Skip to content

fix(chart): pin stabilized platform charts - #5

Merged
rowan-stein merged 2 commits into
mainfrom
noa/issue-4
Jun 4, 2026
Merged

rowan-stein merged 2 commits into
mainfrom
noa/issue-4

Conversation

@casey-brooks

Copy link
Copy Markdown
Collaborator

Summary

Closes #4.

  • Bumps agyn-platform to 0.4.0.
  • Pins the umbrella chart to the patched service chart releases.
  • Removes service subchart default blocks from charts/agyn-platform/values.yaml, including per-subchart fullnameOverride defaults and gateway.env: [].
  • Keeps only umbrella-level configuration and top-level secret contract validation.
  • Does not commit generated Chart.lock, vendored charts/*.tgz, or .dist outputs.

Released service chart versions

Chart Version PR
console-app 0.10.10 agynio/console-app#108
chat-app 0.4.18 agynio/chat-app#105
tracing-app 0.3.1 agynio/tracing-app#53
ziti-management 0.10.11 agynio/ziti-management#62
secrets 0.2.1 agynio/secrets#19
authorization 0.5.5 agynio/authorization#26
llm 0.4.5 agynio/llm#50
threads 0.4.14 agynio/threads#37
notifications 0.2.8 agynio/notifications#33
media-proxy 0.1.6 agynio/media-proxy#16
llm-proxy 0.12.6 agynio/llm-proxy#66
agents-orchestrator 0.13.18 agynio/agents-orchestrator#213
tracing 0.3.7 agynio/tracing#23
files 0.1.6 agynio/files#43
expose 0.1.9 agynio/expose#33
users 0.5.3 agynio/users#25
organizations 0.4.5 agynio/organizations#31
identity 0.2.3 agynio/identity#11
apps 0.4.3 agynio/apps#23
runners 0.5.12 agynio/runners#69
token-counting 0.3.3 agynio/token-counting#18
chat 0.3.5 agynio/chat#32
gateway 0.22.11 agynio/gateway#189

Additional files chart follow-up: agynio/files#42 released files:0.1.5; agynio/files#43 released files:0.1.6 and is the version pinned here.

Validation

Platform chart commands 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 .

Service chart validation previously run across patched service repos:

sh /tmp/validate_charts.sh

media-proxy code 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:

  • Platform Helm lint: 1 chart linted, 0 failed.
  • Platform Helm template: rendered successfully.
  • Platform YAML lint: passed with no errors.
  • Service chart Helm lint/template: all 23 patched service charts passed.
  • media-proxy Go tests: all packages passed.

Notes

chat-app and gateway service PRs still require code-owner review, but their tags were pushed and the released chart packages are available in GHCR and pinned here.

@casey-brooks

Copy link
Copy Markdown
Collaborator Author

Test & Lint Summary

Commands 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:

  • Helm lint: 1 chart linted, 0 failed.
  • Helm template: rendered successfully.
  • YAML lint: passed with no errors.

Generated validation artifacts were removed after running Helm validation; no Chart.lock, vendored charts/*.tgz, or .dist outputs are committed.

@noa-lucent noa-lucent left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Starting review.

Comment thread charts/agyn-platform/Chart.yaml Outdated

@noa-lucent noa-lucent left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

@casey-brooks

Copy link
Copy Markdown
Collaborator Author

Addressed the agents chart review item.

Changes:

  • Patched agynio/agents so the chart now owns fullnameOverride: agents.
  • Patched agynio/agents so DATABASE_URL defaults to valueFrom.secretKeyRef using agyn-platform-database-urls key agents.
  • Merged agents PR: fix(chart): default agents platform wiring agents#60
  • Released and verified agents:0.7.6 in GHCR.
  • Updated this umbrella PR to pin agents from 0.7.4 to 0.7.6.

Validation commands run for agynio/agents:

helm dependency build charts/agents
helm lint charts/agents
helm template test charts/agents >/tmp/agents-chart.yaml

Agents chart results:

  • Helm lint: 1 chart linted, 0 failed.
  • Helm template: rendered successfully.
  • Release workflow: passed for v0.7.6.

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:

  • Helm dependency update pulled agents:0.7.6 successfully.
  • Helm lint: 1 chart linted, 0 failed.
  • Helm template: rendered successfully.
  • YAML lint: passed with no errors.

Generated Helm artifacts were removed after validation and are not committed.

@casey-brooks casey-brooks left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fixed the agents chart pin review item in the latest push. Details and validation summary are in #5 (comment).

@noa-lucent noa-lucent left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

@rowan-stein
rowan-stein merged commit 3a07cc7 into main Jun 4, 2026
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.

Stabilize agyn-platform service subcharts for prod-agyn

3 participants