Skip to content

Conversation

@michele-mangili
Copy link
Contributor

What this PR does / why we need it:

As per the title

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.

@michele-mangili michele-mangili self-assigned this Sep 26, 2023
@michele-mangili michele-mangili force-pushed the mm/cluster-scanner/docker-mirrors branch from 471476c to 9e4c506 Compare September 26, 2023 09:35
@michele-mangili michele-mangili marked this pull request as ready for review September 26, 2023 09:48
@michele-mangili michele-mangili requested a review from a team as a code owner September 26, 2023 09:48
@michele-mangili michele-mangili force-pushed the mm/cluster-scanner/docker-mirrors branch from 9e4c506 to 876435c Compare October 4, 2023 12:44
Copy link
Collaborator

@aroberts87 aroberts87 left a comment

Choose a reason for hiding this comment

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

lgtm, @michele-mangili! just one minor nit that may just be a result of my lack of knowledge in this area. thanks for addressing the earlier concerns by @mavimo as well. 🙂

@michele-mangili michele-mangili merged commit 5b1e964 into master Oct 6, 2023
@michele-mangili michele-mangili deleted the mm/cluster-scanner/docker-mirrors branch October 6, 2023 07:25
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.

5 participants