Skip to content

Addressing metaparam override in datadog_agent::tag#338

Merged
truthbk merged 2 commits intoDataDog:masterfrom
craigwatson:bugfix/tag_metaparam
Jul 20, 2017
Merged

Addressing metaparam override in datadog_agent::tag#338
truthbk merged 2 commits intoDataDog:masterfrom
craigwatson:bugfix/tag_metaparam

Conversation

@craigwatson
Copy link
Copy Markdown
Contributor

No description provided.

@truthbk truthbk self-requested a review July 19, 2017 15:56
@truthbk truthbk added this to the 1.11.0 milestone Jul 19, 2017
@truthbk
Copy link
Copy Markdown
Member

truthbk commented Jul 19, 2017

Thanks @craigwatson 🙇 - taking a look at what happened in the CI.

@truthbk
Copy link
Copy Markdown
Member

truthbk commented Jul 19, 2017

@craigwatson fixed the TravisCI issue on master. Could you please rebase to the latest master - that should get the entire test-suite to pass.

@craigwatson
Copy link
Copy Markdown
Contributor Author

craigwatson commented Jul 19, 2017

@truthbk thanks, have now rebased :)

Copy link
Copy Markdown
Member

@truthbk truthbk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Despite the parameter name change, this is not a breaking change as the previous behavior was actually overriding the meta parameter - this now actually does what it was meant. Thanks @craigwatson

@truthbk truthbk merged commit 23a0720 into DataDog:master Jul 20, 2017
@craigwatson craigwatson deleted the bugfix/tag_metaparam branch August 21, 2017 07:06
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Jan 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants