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(agent): Reorder volume list #1899

Closed
wants to merge 2 commits into from

Conversation

gso-dc
Copy link
Contributor

@gso-dc gso-dc commented Aug 23, 2024

What this PR does / why we need it:

OpenShift stores the volume list in alphabetical order. To avoid unnecessary reconcile runs when using GitOps - align the order in helm chart with OpenShift expected order.

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
  • Check GithubAction checks (like lint) to avoid merge-check stoppers

@gso-dc gso-dc requested a review from a team as a code owner August 23, 2024 11:18
Copy link
Contributor

Hi @gso-dc. Thanks for your PR.

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

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/main/README.md#pull-requests

@gso-dc gso-dc changed the title fix(agent) Reorder volume list fix(agent): Reorder volume list Aug 23, 2024
mavimo added a commit that referenced this pull request Sep 5, 2024
@mavimo
Copy link
Contributor

mavimo commented Sep 5, 2024

@gso-dc unfortunately your PR need a rebase and to bump the chart version. I can't operate on your repo so I made a new PR (#1917 ) giving you the commit ownership.

@mavimo mavimo closed this Sep 5, 2024
@gso-dc gso-dc deleted the fix_volume_order branch September 6, 2024 11:05
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