Skip to content

chore: move back to tech preview#86

Merged
SequeI merged 1 commit intomainfrom
techP
Apr 10, 2026
Merged

chore: move back to tech preview#86
SequeI merged 1 commit intomainfrom
techP

Conversation

@SequeI
Copy link
Copy Markdown
Member

@SequeI SequeI commented Apr 10, 2026

No description provided.

Signed-off-by: SequeI <asiek@redhat.com>
@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Revert to tech preview version 0.1.0 across build and deployment configs

📦 Other

Grey Divider

Walkthroughs

Description
• Revert image version from 1.0.0 to 0.1.0 across all Tekton pipeline files
• Update Dockerfile bundle to use tech-preview channel instead of stable
• Change project maturity status to tech-preview in ClusterServiceVersion
• Update VERSION variable in Makefile to 0.1.0-techpreview
Diagram
flowchart LR
  A["Version 1.0.0"] -- "Downgrade to" --> B["Version 0.1.0"]
  C["Stable Channel"] -- "Switch to" --> D["Tech-Preview Channel"]
  B -- "Applied to" --> E["Tekton Pipelines"]
  B -- "Applied to" --> F["Makefile"]
  B -- "Applied to" --> G["ClusterServiceVersion"]
  D -- "Applied to" --> H["Dockerfile Bundle"]
Loading

Grey Divider

File Changes

1. .tekton/model-validation-operator-agent-pull-request.yaml ⚙️ Configuration changes +1/-1

Downgrade image version to 0.1.0

.tekton/model-validation-operator-agent-pull-request.yaml


2. .tekton/model-validation-operator-agent-push.yaml ⚙️ Configuration changes +1/-1

Downgrade image version to 0.1.0

.tekton/model-validation-operator-agent-push.yaml


3. .tekton/model-validation-operator-bundle-pull-request.yaml ⚙️ Configuration changes +1/-1

Downgrade image version to 0.1.0

.tekton/model-validation-operator-bundle-pull-request.yaml


View more (6)
4. .tekton/model-validation-operator-bundle-push.yaml ⚙️ Configuration changes +1/-1

Downgrade image version to 0.1.0

.tekton/model-validation-operator-bundle-push.yaml


5. .tekton/model-validation-operator-pull-request.yaml ⚙️ Configuration changes +1/-1

Downgrade image version to 0.1.0

.tekton/model-validation-operator-pull-request.yaml


6. .tekton/model-validation-operator-push.yaml ⚙️ Configuration changes +1/-1

Downgrade image version to 0.1.0

.tekton/model-validation-operator-push.yaml


7. Dockerfile.model-validation-bundle.rh ⚙️ Configuration changes +3/-3

Update to tech-preview channel and version 0.1.0

Dockerfile.model-validation-bundle.rh


8. Makefile ⚙️ Configuration changes +1/-1

Update VERSION variable to 0.1.0

Makefile


9. config/manifests/bases/model-validation-operator.clusterserviceversion.yaml ⚙️ Configuration changes +3/-3

Update maturity to tech-preview and version 0.1.0

config/manifests/bases/model-validation-operator.clusterserviceversion.yaml


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review bot commented Apr 10, 2026

Code Review by Qodo

🐞 Bugs (1)   📘 Rule violations (0)   📎 Requirement gaps (0)   🎨 UX Issues (0)
🐞\ ≡ Correctness (1)

Grey Divider


Action required

1. CSV name/version mismatch 🐞
Description
The base ClusterServiceVersion sets metadata.name to model-validation-operator.v0.1.0 while
spec.version is 0.1.0-techpreview, making the CSV internally inconsistent and potentially invalid
for OLM/operator-sdk validation.
Code

config/manifests/bases/model-validation-operator.clusterserviceversion.yaml[R68-72]

+  maturity: tech-preview
  minKubeVersion: 1.24.0
  provider:
    name: Red Hat
-  version: 1.0.0
+  version: 0.1.0-techpreview
Evidence
The CSV declares a different version string in metadata.name than in spec.version. This CSV is
included in the kustomize resources used to generate bundle manifests, and the repo’s bundle build
script runs operator-sdk bundle validation, so an inconsistent CSV can fail validation or produce an
unexpected bundle.

config/manifests/bases/model-validation-operator.clusterserviceversion.yaml[21-22]
config/manifests/bases/model-validation-operator.clusterserviceversion.yaml[68-72]
config/manifests/kustomization.yaml[1-7]
hack/build-bundle.sh[16-18]
hack/build-bundle.sh[29-29]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
The base CSV uses two different version strings: `metadata.name` embeds `v0.1.0` while `spec.version` is `0.1.0-techpreview`. OLM/`operator-sdk bundle validate` expects consistent CSV metadata, and this file is included in the manifests used for bundle generation.

### Issue Context
This CSV is a direct input into the bundle generation kustomize pipeline and is therefore part of what gets validated.

### Fix Focus Areas
- config/manifests/bases/model-validation-operator.clusterserviceversion.yaml[21-22]
- config/manifests/bases/model-validation-operator.clusterserviceversion.yaml[68-72]

### What to change
Choose one consistent version string and apply it to BOTH:
- `metadata.name: model-validation-operator.v<version>`
- `spec.version: <version>`

For example, either:
- keep `spec.version: 0.1.0` (and `metadata.name: ...v0.1.0`), relying on channel/maturity to indicate tech-preview; OR
- use `spec.version: 0.1.0-techpreview` and update `metadata.name` to `model-validation-operator.v0.1.0-techpreview`.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

Comment on lines +68 to +72
maturity: tech-preview
minKubeVersion: 1.24.0
provider:
name: Red Hat
version: 1.0.0
version: 0.1.0-techpreview
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Action required

1. Csv name/version mismatch 🐞 Bug ≡ Correctness

The base ClusterServiceVersion sets metadata.name to model-validation-operator.v0.1.0 while
spec.version is 0.1.0-techpreview, making the CSV internally inconsistent and potentially invalid
for OLM/operator-sdk validation.
Agent Prompt
### Issue description
The base CSV uses two different version strings: `metadata.name` embeds `v0.1.0` while `spec.version` is `0.1.0-techpreview`. OLM/`operator-sdk bundle validate` expects consistent CSV metadata, and this file is included in the manifests used for bundle generation.

### Issue Context
This CSV is a direct input into the bundle generation kustomize pipeline and is therefore part of what gets validated.

### Fix Focus Areas
- config/manifests/bases/model-validation-operator.clusterserviceversion.yaml[21-22]
- config/manifests/bases/model-validation-operator.clusterserviceversion.yaml[68-72]

### What to change
Choose one consistent version string and apply it to BOTH:
- `metadata.name: model-validation-operator.v<version>`
- `spec.version: <version>`

For example, either:
- keep `spec.version: 0.1.0` (and `metadata.name: ...v0.1.0`), relying on channel/maturity to indicate tech-preview; OR
- use `spec.version: 0.1.0-techpreview` and update `metadata.name` to `model-validation-operator.v0.1.0-techpreview`.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

@SequeI SequeI merged commit 91ca2a8 into main Apr 10, 2026
13 checks 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