Skip to content

Isv 6537#880

Merged
Allda merged 2 commits into
mainfrom
ISV-6537
Jan 14, 2026
Merged

Isv 6537#880
Allda merged 2 commits into
mainfrom
ISV-6537

Conversation

@Allda
Copy link
Copy Markdown
Contributor

@Allda Allda commented Jan 12, 2026

Merge Request Checklists

  • Development is done in feature branches
  • Code changes are submitted as pull request into a primary branch [Provide reason for non-primary branch submissions]
  • Code changes are covered with unit and integration tests.
  • Code passes all automated code tests:
    • Linting
    • Code formatter - Black
    • Security scanners
    • Unit tests
    • Integration tests
  • Code is reviewed by at least 1 team member
  • Pull request is tagged with "risk/good-to-go" label for minor changes

Allda added 2 commits January 12, 2026 12:09
The test verifies that operator version directory always matches the
version from the CSV. In the past we noticed several bundles with wrong
version which can lead to undefined events.

JIRA: ISV-6537

Signed-off-by: Ales Raszka <araszka@redhat.com>
Signed-off-by: Ales Raszka <araszka@redhat.com>
@Allda
Copy link
Copy Markdown
Contributor Author

Allda commented Jan 12, 2026

We have several bundles that violate these tests in certified and community repositories. Since they are already merged and released, we can't do anything with them. The new test checks only the new bundles and makes sure it won't happen for new content.

The unmatched versions might cause unexpected behaviour for the operator.

Copy link
Copy Markdown
Contributor

@bclindner bclindner left a comment

Choose a reason for hiding this comment

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

definitely thought csv meant "comma-separated values" for a bit lol - other than that looks good

@Allda Allda merged commit 4d476f0 into main Jan 14, 2026
22 of 34 checks passed
@Allda Allda deleted the ISV-6537 branch January 14, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants