Skip to content

feat(cudn-bgp-routing-operator): in-cluster BuildConfig + Namespace (0.1.2) - #126

Merged
foster-rh merged 2 commits into
mainfrom
feature/cudn-bgp-build-operator
Aug 5, 2026
Merged

feat(cudn-bgp-routing-operator): in-cluster BuildConfig + Namespace (0.1.2)#126
foster-rh merged 2 commits into
mainfrom
feature/cudn-bgp-build-operator

Conversation

@paulczar

@paulczar paulczar commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add createNamespace (default true) so the chart works with app-of-apps CreateNamespace=false (fixes missing openshift-cudn-bgp-routing ns).
  • Add optional buildOperator that renders ImageStream + BuildConfig to build rosa-bgp-operator into the local registry path the Deployment already expects.
  • Deployment gets OpenShift image triggers + later sync-wave when building; chart version 0.1.2.
  • Default build resources: requests 2 CPU / 4Gi, limits 4 CPU / 8Gi (validated on a ROSA HCP BGP PoC cluster).

Test plan

  • helm lint with buildOperator.enabled=true
  • helm template renders Namespace / ImageStream / BuildConfig when enabled
  • Live install on bgp cluster: BuildConfig completed (operator-4), ImageStream tag dev, manager Deployment Running
  • CI / chart release workflow publishes 0.1.2
  • Follow-up: enable in rosa-cluster-config dev/bgp with targetRevision: 0.1.2 and buildOperator.enabled: true

Related

Made with Cursor

paulczar and others added 2 commits August 5, 2026 10:46
…Namespace

Add createNamespace (app-of-apps uses CreateNamespace=false) and optional
buildOperator ImageStream/BuildConfig so PoC clusters can build the operator
into the local registry. Bump chart to 0.1.2; default build resources 4Gi/8Gi.

Co-authored-by: Cursor <cursoragent@cursor.com>
chart-testing requires maintainers; matches other charts in this repo.

Co-authored-by: Cursor <cursoragent@cursor.com>
@foster-rh
foster-rh merged commit 4f675bc into main Aug 5, 2026
1 check passed
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