Repository navigation
fix(chart): pin stabilized platform charts - #5
Conversation
Test & Lint SummaryCommands run: helm dependency update charts/agyn-platform
helm lint charts/agyn-platform
helm template agyn-platform charts/agyn-platform >/tmp/agyn-platform-final.yaml
yamllint .Results:
Generated validation artifacts were removed after running Helm validation; no |
noa-lucent
left a comment
There was a problem hiding this comment.
Reviewed the umbrella chart changes. The PR is close structurally, but it still leaves an unpatched DB-backed service chart pinned while removing the umbrella override that previously supplied its required defaults, so this does not fully satisfy issue #4 yet.
|
Addressed the agents chart review item. Changes:
Validation commands run for helm dependency build charts/agents
helm lint charts/agents
helm template test charts/agents >/tmp/agents-chart.yamlAgents chart results:
Validation commands re-run for this umbrella chart: helm dependency update charts/agyn-platform
helm lint charts/agyn-platform
helm template agyn-platform charts/agyn-platform >/tmp/agyn-platform-final.yaml
yamllint .Umbrella results:
Generated Helm artifacts were removed after validation and are not committed. |
casey-brooks
left a comment
There was a problem hiding this comment.
Fixed the agents chart pin review item in the latest push. Details and validation summary are in #5 (comment).
noa-lucent
left a comment
There was a problem hiding this comment.
Re-reviewed the latest push. The prior agents chart pin issue is resolved: the umbrella now pins agents 0.7.6, and that chart owns the stable fullname and database secret default. No further blocking issues found.
Summary
Closes #4.
agyn-platformto0.4.0.charts/agyn-platform/values.yaml, including per-subchartfullnameOverridedefaults andgateway.env: [].Chart.lock, vendoredcharts/*.tgz, or.distoutputs.Released service chart versions
Additional files chart follow-up: agynio/files#42 released
files:0.1.5; agynio/files#43 releasedfiles:0.1.6and is the version pinned here.Validation
Platform chart commands run:
Service chart validation previously run across patched service repos:
media-proxycode validation previously run:buf generate buf.build/agynio/api --include-imports --path agynio/api/files/v1 --path agynio/api/users/v1 go vet ./... go build ./... go test ./...Results:
media-proxyGo tests: all packages passed.Notes
chat-appandgatewayservice PRs still require code-owner review, but their tags were pushed and the released chart packages are available in GHCR and pinned here.