Skip to content

test(RELEASE-2325): add idempotent retrigger e2e test for rh-advisories#2170

Merged
elenagerman merged 1 commit into
konflux-ci:developmentfrom
elenagerman:release-2325
Jun 9, 2026
Merged

test(RELEASE-2325): add idempotent retrigger e2e test for rh-advisories#2170
elenagerman merged 1 commit into
konflux-ci:developmentfrom
elenagerman:release-2325

Conversation

@elenagerman

@elenagerman elenagerman commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

Adds a new integration test suite rh-advisories-idempotent that validates safe re-release behavior when the same snapshot is released twice through the rh-advisories pipeline.

Test flow:

  • First release: all tasks run, advisory created (skip_release=false)
  • Second release (same snapshot): filter-already-released-advisory-images detects the existing advisory in Pyxis → skip_release=true
  • Test asserts the second release completes successfully (Released=True), that all major downstream tasks appear in skippedTasks, and that advisory.url is present in the second release status (populated from the filter result, not create-advisory)

Relevant Jira

RELEASE-2325

Checklist before requesting a review

  • I have marked as draft or added do not merge label if there's a dependency PR
    • If you want reviews on your draft PR, you can add reviewers or add the release-service-maintainers handle if you are unsure who to tag
  • My commit message includes Signed-off-by: My name <email>
  • I read CONTRIBUTING.MD and commit formatting
  • I have run the README.md generator script in .github/scripts/readme_generator.sh and verified the results using .github/scripts/check_readme.sh
  • If an AI agent was used, I marked that via a commit footer like Assisted-By: Cursor

@elenagerman elenagerman marked this pull request as draft April 21, 2026 14:56
@elenagerman elenagerman force-pushed the release-2325 branch 4 times, most recently from fc0274a to 71bd31c Compare April 21, 2026 17:30
@elenagerman

Copy link
Copy Markdown
Contributor Author

/retest

@elenagerman elenagerman force-pushed the release-2325 branch 6 times, most recently from 1a0d2a1 to 1a4c840 Compare April 22, 2026 15:47
@elenagerman

Copy link
Copy Markdown
Contributor Author

/retest

@elenagerman

Copy link
Copy Markdown
Contributor Author

/retest

@elenagerman

Copy link
Copy Markdown
Contributor Author

/retest

@elenagerman

Copy link
Copy Markdown
Contributor Author

/retest

@elenagerman elenagerman marked this pull request as ready for review May 20, 2026 03:07
@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@mmalina

mmalina commented May 20, 2026

Copy link
Copy Markdown
Contributor

There is a conflict (I think Johnny changed this yesterday). Please fix it first.

@qodo-code-review

This comment was marked as duplicate.

@qodo-code-review

This comment was marked as duplicate.

Comment thread integration-tests/rh-advisories-idempotent/test.sh
Comment thread integration-tests/rh-advisories-idempotent/test.sh Outdated
@qodo-code-review

This comment was marked as duplicate.

Comment thread integration-tests/rh-advisories-idempotent/test.sh
Comment thread integration-tests/rh-advisories-idempotent/test.sh Outdated
Comment thread integration-tests/rh-advisories-idempotent/test.sh Outdated
@qodo-code-review

This comment was marked as duplicate.

Comment thread integration-tests/rh-advisories-idempotent/test.sh Outdated
@qodo-code-review

This comment was marked as duplicate.

2 similar comments
@qodo-code-review

This comment was marked as duplicate.

@qodo-code-review

This comment was marked as duplicate.

@elenagerman

Copy link
Copy Markdown
Contributor Author

/retest

@qodo-code-review

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

Warning

/improve is deprecated. Use /agentic_review instead (removal date not yet scheduled).

Inline suggestions were posted as code suggestions.

Comment thread integration-tests/lib/test-functions.sh
Comment thread integration-tests/lib/test-functions.sh
Comment thread integration-tests/rh-advisories-idempotent/test.sh
@elenagerman

Copy link
Copy Markdown
Contributor Author

/retest

1 similar comment
@elenagerman

Copy link
Copy Markdown
Contributor Author

/retest

@qodo-code-review

This comment was marked as duplicate.

Comment thread integration-tests/lib/test-functions.sh
@konflux-ci-qe-bot

Copy link
Copy Markdown

Scenario: push-to-external-registry-self-hosted-quay-e2e-test-go
@elenagerman: The following test has Failed, say /retest to rerun failed tests.

PipelineRun Name Status Rerun command Build Log Test Log
push-to-external-registry-self-hosted-quay-e2e-test-go-pplg9 Failed /retest View Pipeline Log View Test Logs

Inspecting Test Artifacts

To inspect your test artifacts, follow these steps:

  1. Install ORAS (see the ORAS installation guide).
  2. Download artifacts with the following commands:
mkdir -p oras-artifacts
cd oras-artifacts
oras pull quay.io/konflux-test-storage/konflux-team/release-service-catalog:push-to-external-registry-self-hosted-quay-e2e-test-go-pplg9

Test results analysis

🚨 Couldn't detect a specific failure, see the related PipelineRun for more details or consult with Konflux DevProd team.

OCI Artifact Browser URL

<not enabled>

Adds a new integration test suite rh-advisories-idempotent that
validates safe re-release behavior when the same snapshot is
released twice through the rh-advisories pipeline.

Signed-off-by: Elena German <elgerman@redhat.com>
Assisted-by: Claude
Co-authored-by: Cursor <cursoragent@cursor.com>
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.

4 participants