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

[graphite] bump apiversion to v2 in order to support helm chart dependencies #464

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

JonoNiemann
Copy link
Contributor

@JonoNiemann JonoNiemann commented Jul 12, 2023

What this PR does / why we need it:

  • Support for apiversion 2, allowing use of dependencies in the chart.

Special notes for your reviewer:

Checklist

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

  • DCO signed
  • Chart Version bumped (if the pr is an update to an existing chart)
  • Title of the PR starts with chart name (e.g. [fluentd-elasticsearch])

@JonoNiemann JonoNiemann changed the title bump apiversion to v2 in order to support helm chart dependencies graphite: bump apiversion to v2 in order to support helm chart dependencies Jul 12, 2023
@JonoNiemann JonoNiemann force-pushed the master branch 7 times, most recently from 9dbf22a to 3af9d5d Compare July 12, 2023 08:23
Signed-off-by: Jonathan Niemann <zoeyruthniemann@gmail.com>
@JonoNiemann JonoNiemann changed the title graphite: bump apiversion to v2 in order to support helm chart dependencies [graphite] bump apiversion to v2 in order to support helm chart dependencies Jul 12, 2023
@JonoNiemann
Copy link
Contributor Author

cc: @monotek @rpahli

@rpahli rpahli merged commit cbd6f7c into kiwigrid:master Jul 17, 2023
11 checks passed
@JonoNiemann
Copy link
Contributor Author

hey @rpahli

Would you mind taking a look or trying to rerun action...

the deploy step failed in actions

https://github.com/kiwigrid/helm-charts/actions/runs/5575998523/job/15098440124

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