Skip to content

introduce image catalog#12

Merged
bennyz merged 7 commits into
centos-automotive-suite:mainfrom
bennyz:001-image-catalog
Jan 6, 2026
Merged

introduce image catalog#12
bennyz merged 7 commits into
centos-automotive-suite:mainfrom
bennyz:001-image-catalog

Conversation

@bennyz

@bennyz bennyz commented Jan 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@bennyz bennyz force-pushed the 001-image-catalog branch from 375af16 to a5cbe88 Compare January 4, 2026 17:38
bennyz added 5 commits January 6, 2026 11:58
…les)

  The constitution is now ready to guide development practices and can be referenced in code reviews, feature planning, and architectural decisions. The principles specifically address the unique requirements of a Kubernetes operator that manages automotive OS image builds through container-based workflows.

Signed-off-by: Benny Zlotnik <bzlotnik@redhat.com>
Assisted-by: Claude Code
Signed-off-by: Benny Zlotnik <bzlotnik@redhat.com>
Signed-off-by: Benny Zlotnik <bzlotnik@redhat.com>
Signed-off-by: Benny Zlotnik <bzlotnik@redhat.com>
Assisted-by: claude-opus-4.5
Signed-off-by: Benny Zlotnik <bzlotnik@redhat.com>
Assited-by: claude-opus-4.5
@bennyz bennyz force-pushed the 001-image-catalog branch from 8f465ea to da2d41c Compare January 6, 2026 09:58
bennyz and others added 2 commits January 6, 2026 13:24
The CatalogImage CRD was generated but not included in the kustomization
resources list, causing e2e tests to fail when the controller tried to
create a field index on a non-existent CRD.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Benny Zlotnik <bzlotnik@redhat.com>
@bennyz bennyz force-pushed the 001-image-catalog branch from da2d41c to ca17588 Compare January 6, 2026 12:32
@bennyz bennyz merged commit addde39 into centos-automotive-suite:main Jan 6, 2026
1 check passed
@bennyz bennyz deleted the 001-image-catalog branch January 6, 2026 12:40
bennyz added a commit that referenced this pull request Jan 21, 2026
vtz added a commit to vtz/automotive-dev-operator that referenced this pull request Apr 11, 2026
CRD changes (softwarebuild_types.go):
- Remove hostPath source type (not implemented) (centos-automotive-suite#7)
- Add CEL validation for source type/field consistency (centos-automotive-suite#11)
- Remove unused destination types (registry, artifactory, quay) (centos-automotive-suite#16)
- Tighten Git URL and revision validation patterns (centos-automotive-suite#1)

Pipeline generation (software_build.go):
- Fix command injection: quote URL and revision in git clone (centos-automotive-suite#1)
- Sanitize revision with regex, fall back to "main" on invalid input
- Deterministic PipelineRun naming using sb.Generation (centos-automotive-suite#3)
- Propagate TimeoutSeconds and BuildTimeoutMinutes to PipelineRun (centos-automotive-suite#6)
- Add SecurityContext with AllowPrivilegeEscalation=false (centos-automotive-suite#12)
- Wire per-stage image override (centos-automotive-suite#17)
- Use ParseQuantity with fallback instead of MustParse (centos-automotive-suite#26)
- Wire ServiceAccountName to PipelineRun (centos-automotive-suite#28)
- Wire OperatorConfig DefaultImage through BuildConfig (centos-automotive-suite#14)

Controller (softwarebuild/controller.go):
- Handle status update errors instead of discarding them (centos-automotive-suite#8)
- Distinguish Pending from Running using StartTime (centos-automotive-suite#18)
- Add error context wrapping on all return paths (centos-automotive-suite#19)
- Replace magic strings with typed constants (centos-automotive-suite#20)
- Extract mapPipelineRunPhase and buildStageStatuses helpers (centos-automotive-suite#24)

OperatorConfig controller:
- Propagate deploySoftwareBuilds error and set Failed status (centos-automotive-suite#4)
- Add cleanupSoftwareBuilds for when feature is disabled (centos-automotive-suite#5)
- Wire SoftwareBuildsConfig fields to pipeline generation (centos-automotive-suite#14)

Made-with: Cursor
vtz added a commit to vtz/automotive-dev-operator that referenced this pull request Apr 11, 2026
CRD changes (softwarebuild_types.go):
- Remove hostPath source type (not implemented) (centos-automotive-suite#7)
- Add CEL validation for source type/field consistency (centos-automotive-suite#11)
- Remove unused destination types (registry, artifactory, quay) (centos-automotive-suite#16)
- Tighten Git URL and revision validation patterns (centos-automotive-suite#1)

Pipeline generation (software_build.go):
- Fix command injection: quote URL and revision in git clone (centos-automotive-suite#1)
- Sanitize revision with regex, fall back to "main" on invalid input
- Deterministic PipelineRun naming using sb.Generation (centos-automotive-suite#3)
- Propagate TimeoutSeconds and BuildTimeoutMinutes to PipelineRun (centos-automotive-suite#6)
- Add SecurityContext with AllowPrivilegeEscalation=false (centos-automotive-suite#12)
- Wire per-stage image override (centos-automotive-suite#17)
- Use ParseQuantity with fallback instead of MustParse (centos-automotive-suite#26)
- Wire ServiceAccountName to PipelineRun (centos-automotive-suite#28)
- Wire OperatorConfig DefaultImage through BuildConfig (centos-automotive-suite#14)

Controller (softwarebuild/controller.go):
- Handle status update errors instead of discarding them (centos-automotive-suite#8)
- Distinguish Pending from Running using StartTime (centos-automotive-suite#18)
- Add error context wrapping on all return paths (centos-automotive-suite#19)
- Replace magic strings with typed constants (centos-automotive-suite#20)
- Extract mapPipelineRunPhase and buildStageStatuses helpers (centos-automotive-suite#24)

OperatorConfig controller:
- Propagate deploySoftwareBuilds error and set Failed status (centos-automotive-suite#4)
- Add cleanupSoftwareBuilds for when feature is disabled (centos-automotive-suite#5)
- Wire SoftwareBuildsConfig fields to pipeline generation (centos-automotive-suite#14)

Made-with: Cursor
@ambient-code ambient-code Bot mentioned this pull request Jun 1, 2026
10 tasks
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.

1 participant