Skip to content

Fix reverse content match#524

Merged
truthbk merged 7 commits intoDataDog:masterfrom
dorg-kanderson:fix-reverse-content-match
May 20, 2019
Merged

Fix reverse content match#524
truthbk merged 7 commits intoDataDog:masterfrom
dorg-kanderson:fix-reverse-content-match

Conversation

@dorg-kanderson
Copy link
Copy Markdown
Contributor

Add reverse_content_match boolean to http_check

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.

This is almost ready. A spec test would be perfect if you could please add that! 🙇

@dorg-kanderson
Copy link
Copy Markdown
Contributor Author

Please review and let me know if I constructed the spec correctly.

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.

Almost there! :)

Comment thread spec/classes/datadog_agent_integrations_http_check_spec.rb Outdated
Comment thread spec/classes/datadog_agent_integrations_http_check_spec.rb Outdated
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.

Found the last missing detail. Once you fix this tests will be ✅

Comment thread templates/agent-conf.d/http_check.yaml.erb Outdated
@dorg-kanderson
Copy link
Copy Markdown
Contributor Author

Please let me know if you need anything additional from me to get this merged. Thank you!

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.

Let's merge this guy! 🙇

Thank you @dorg-kanderson!

@truthbk truthbk merged commit 9f9efc2 into DataDog:master May 20, 2019
@dorg-kanderson dorg-kanderson deleted the fix-reverse-content-match branch May 20, 2019 21:31
@truthbk truthbk added this to the 2.6.0 milestone May 31, 2019
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Apr 6, 2020
* Update http_check.yaml.erb

* Update http_check.pp

* Update datadog_agent_integrations_http_check_spec.rb

* Update datadog_agent_integrations_http_check_spec.rb

* Update datadog_agent_integrations_http_check_spec.rb

* Update datadog_agent_integrations_http_check_spec.rb

* Update http_check.yaml.erb
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