fix(runtime): upgrade ingress-nginx to 4.15.0 - #169
Open
plural-copilot[bot] wants to merge 1 commit into
Open
Conversation
Contributor
Author
There was a problem hiding this comment.
This PR was generated by the codex Plural Agent Runtime. Here's some useful information you might want to know to evaluate the ai's perfomance:
| Name | Details |
|---|---|
| 💬 Prompt | In pluralsh/bootstrap current main, inspect the runtime Helm chart and create exactly one bootstrap-only PR needed to remediate the management EKS 1.35 ingress-nginx blocker. The management consumer is plrl-up-demos bootstrap/runtime.yaml and uses this runtime chart; its encrypted runtime values must never be edited. Determine exact runtime chart source/dependency/template ownership and current version. Upgrade management ingress-nginx to controller/app 1.15.0/chart 4.15.0 only if it is actua... |
| 🔗 Run history | View run history |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updates only the bootstrap management runtime chart packaging to remediate the EKS 1.35 ingress-nginx blocker.
Version changes
0.1.36→0.1.37ingress-nginxchart dependency:4.14.1→4.15.01.14.1→1.15.0Both the public
ingress-nginxdependency and theingress-nginx-privatealias are upgraded together.Runtime ownership evidence
charts/runtime/Chart.yamlis theruntimechart source and directly declares both ingress-nginx dependencies, each conditioned by its runtime chart value.charts/runtime/values.yamlenables and configures both releases, including the public and private IngressClass/controller settings.setup/runtime.yamlinstalls the runtime chart into the management cluster (clusterRef: mgmt) fromhttps://pluralsh.github.io/bootstrap..github/workflows/chart.yamlpublishes chart changes when this repository'smainbranch receives the merge.Files changed
charts/runtime/Chart.yaml— bumps the runtime package version and both ingress-nginx dependency pins.charts/runtime/Chart.lock— regenerates the lock metadata for 4.15.0.charts/runtime/charts/ingress-nginx-4.14.1.tgz— removes the prior vendored dependency archive.charts/runtime/charts/ingress-nginx-4.15.0.tgz— adds the regenerated vendored dependency archive.No plrl-up-demos files, encrypted runtime values, or workload resources are changed.
Required consumer follow-up and merge order
runtimechart0.1.37tohttps://pluralsh.github.io/bootstrap.plrl-up-demos/bootstrap/runtime.yamlto set the runtime chart reference to:The consumer reference must not be merged ahead of this chart PR/publication, or it can resolve a chart version that does not yet exist.
Validation
3.16.4.helm lint charts/runtimepassed (with only Helm's informational icon recommendation).1.35.0; both ingress controllers renderregistry.k8s.io/ingress-nginx/controller:v1.15.0, including the private alias.runtime-0.1.37.tgzsuccessfully.The encrypted plrl-up-demos runtime values were intentionally not accessed or modified, so validation is limited to the runtime chart dependency/default render rather than the encrypted consumer overrides.