-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix criticals #181
base: staging
Are you sure you want to change the base?
Fix criticals #181
Conversation
@@ -316,7 +316,7 @@ config: | |||
container: | |||
image: | |||
repository: nats | |||
tag: 2.10.20-alpine | |||
tag: 2.10.26-alpine |
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.
@bkochauri-memphis Its not the correct way to upgrade nats or telegraf. You should check updates of the whole helm chart and update it accordingly if needed + version in Chart.yaml of SSM and the components.
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.
@valeraBr Nats chart is updated, regarding telegraf chart we are on the last version.
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.
@bkochauri-memphis Our Chart.yaml should be updated as well with actual helm chart versions
How Has This Been Tested?
Please describe the tests you ran to verify your changes and note any relevant details for your test configuration.
Checklist:
Reviewer Score - 0-100%
Tests have been done and meaningfully described - (25%)
Meeting Task Specifications (25%)
Writing Readable Code (20%)
Attention to the folder structures in automation tools and repositories (20%)
Attention to Edge Cases (10%)