Skip to content
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(node-analyzer,sysdig-deploy): Ability to disable sslVerifyCertificate per container on node-analyzer #1445

Merged
merged 5 commits into from
Nov 6, 2023

Conversation

dark-vex
Copy link
Collaborator

@dark-vex dark-vex commented Nov 2, 2023

What this PR does / why we need it:

This is a continuation of PR #1432 these changes will introduce the possibility to disable the certificate verification per container or disable it globally

Checklist

  • Title of the PR starts with type and scope, (e.g. feat(agent,node-analyzer,sysdig-deploy):)
  • Chart Version bumped for the respective charts
  • Variables are documented in the README.md (or README.tpl in some charts)
  • Check GithubAction checks (like lint) to avoid merge-check stoppers
  • All test files are added in the tests folder of their respective chart and have a "_test" suffix

@dark-vex dark-vex self-assigned this Nov 2, 2023
@dark-vex dark-vex requested a review from a team as a code owner November 2, 2023 14:26
@dark-vex dark-vex force-pushed the fix-sslverifycertificate-per-container branch from 5a625c2 to 391b7a2 Compare November 3, 2023 13:16
@mavimo mavimo force-pushed the fix-sslverifycertificate-per-container branch from b69df2f to 36f747a Compare November 6, 2023 08:51
@mavimo mavimo merged commit 5ea8986 into master Nov 6, 2023
5 checks passed
@mavimo mavimo deleted the fix-sslverifycertificate-per-container branch November 6, 2023 09:52
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