Skip to content

Conversation

@IgorEulalio
Copy link
Contributor

Fix for the below discussion:
https://sysdigcloud.slack.com/archives/C8R29J8KE/p1697724328414589?thread_ts=1697651280.472299&cid=C8R29J8KE

What this PR does / why we need it:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • 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

Check Contribution guidelines in README.md for more insight.

@IgorEulalio IgorEulalio requested a review from a team as a code owner October 23, 2023 23:11
@github-actions
Copy link
Contributor

Hi @IgorEulalio. Thanks for your PR.

After inspecting your changes someone with write access to this repo needs
to approve and run the workflow with the following slash_command.

NOTE: the workflow triggered this way will only report the final status to this PR,
in order to check the logs please go to the Actions tab and look for approve-test-run-command.

  • /approve-test-run sha=<short commit sha to test>: Run helm chart linting and testing workflow

@github-actions github-actions bot added the no-tests Chart templates modified without test changes label Oct 23, 2023
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR title does not comply with regex: ^(\w*)(?:\(([\w\$\.\,\-\*\s]*)\))?\:\s?(.*)$!
Check PR guidelines at https://github.com/sysdiglabs/charts/blob/master/README.md#pull-requests

@IgorEulalio IgorEulalio changed the title Change ClusterScanner instalation condition based on nodeAnalyzer.runtimeScanner feat(agent): change ClusterScanner instalation condition based on nodeAnalyzer.runtimeScanner Oct 24, 2023
@github-actions github-actions bot dismissed their stale review October 24, 2023 12:28

All good!

@AlbertoBarba
Copy link
Contributor

/approve-test-run sha=b42b4b1

@IgorEulalio
Copy link
Contributor Author

/approve-test-run sha=c1e772e

@AlbertoBarba
Copy link
Contributor

/approve-test-run sha=0aed3a7

@AlbertoBarba AlbertoBarba merged commit 8d36884 into sysdiglabs:master Oct 25, 2023
@IgorEulalio IgorEulalio deleted the patch-1 branch October 25, 2023 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-tests Chart templates modified without test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants