-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/lasting #248
Feature/lasting #248
Conversation
ba0dd99
to
6e3ce3e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some suggestions in comments
472b2ce
to
9588fb1
Compare
9588fb1
to
e8d753d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems good for me, please you have to add this new parameter in https://github.com/claranet/terraform-signalfx-detectors/blob/master/scripts/templates/values.yaml file for documentation purpose
e980337
to
94287b4
Compare
94287b4
to
1573b42
Compare
811fd22
to
fee6735
Compare
fee6735
to
4efe269
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Closes #247
Needed for #246