Skip to content

Use enterprise config-syncer image and fix catalog image discovery#496

Merged
tamalsaha merged 1 commit into
masterfrom
fix
Jul 8, 2026
Merged

Use enterprise config-syncer image and fix catalog image discovery#496
tamalsaha merged 1 commit into
masterfrom
fix

Conversation

@tamalsaha

@tamalsaha tamalsaha commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Switch the catalog to the enterprise config-syncer image and make catalog image discovery work without requiring a real license.

Changes

  • catalog/imagelist.yaml: Use ghcr.io/appscode/config-syncer:v0.15.5-ent.
  • hack/scripts/update-catalog.sh: Temporarily set a fake license in charts/config-syncer/ci/ci-values.yaml so config-syncer templates render during image-packer list, then reset the license back afterwards (via an EXIT trap).
  • kmodules.xyz/image-packer: Bump the dependency, and the vendored pkg/lib/image.go now passes a chart's ci/ci-values.yaml as --values when present, so per-chart CI values feed image discovery.
  • tests/check-charts_test.go: Drop the now-unneeded ignoreMissingList entry for config-syncer since its image is discovered correctly.
  • .github/workflows/ci.yml: Bump CI kubectl to v1.31.0 via dl.k8s.io.

@tamalsaha tamalsaha force-pushed the fix branch 6 times, most recently from d6dd934 to ffb6f57 Compare July 8, 2026 06:02
@tamalsaha tamalsaha changed the title Fix CI Use enterprise config-syncer image and fix catalog image discovery Jul 8, 2026
- Switch catalog to ghcr.io/appscode/config-syncer:v0.15.5-ent
- Render config-syncer with a fake license during update-catalog.sh so
  its images are discovered, resetting the license afterwards
- Bump image-packer to pick up per-chart ci/ci-values.yaml support
- Drop the now-unneeded config-syncer entry from ignoreMissingList
- Bump CI kubectl to v1.31.0 via dl.k8s.io

Signed-off-by: Tamal Saha <tamal@appscode.com>
@tamalsaha tamalsaha merged commit 9ba7746 into master Jul 8, 2026
3 of 4 checks passed
@tamalsaha tamalsaha deleted the fix branch July 8, 2026 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant