Skip to content

feat: add conditional annotations for the e2e PipelineRun#186

Closed
Roming22 wants to merge 1 commit into
redhat-appstudio:mainfrom
Roming22:feat/e2e
Closed

feat: add conditional annotations for the e2e PipelineRun#186
Roming22 wants to merge 1 commit into
redhat-appstudio:mainfrom
Roming22:feat/e2e

Conversation

@Roming22
Copy link
Copy Markdown
Member

@Roming22 Roming22 commented May 1, 2026

Introduced annotations to the e2e-main-pipeline.yaml to trigger on specific files paths.

Assisted-by: Cursor

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

Introduced annotations to the e2e-main-pipeline.yaml to trigger on specific files paths.

Assisted-by: Cursor

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@Roming22 Roming22 requested review from rhopp, smatula and xinredhat May 1, 2026 19:18
@konflux-ci-qe-bot
Copy link
Copy Markdown

@Roming22: The following matrix E2E test has Failed, say /retest to rerun failed tests.

Total child pipelines: 2
Failed child pipelines: 2

Failed child pipelines

PipelineRun Name Status Rerun command Build Log Test Log
tsf-e2e-4.20-xsz2l Failed /retest View Pipeline Log View Test Logs
tsf-e2e-4.21-4mk98 Failed /retest View Pipeline Log View Test Logs

@rhopp
Copy link
Copy Markdown
Collaborator

rhopp commented May 5, 2026

This pipelinerun is triggered/created by Integration tests controller and not by PaC. I'm not sure this will work. I don't see this being mentioned in konflux documentation - and even quick search in konflux-users I don't see any mentions of on-cel-expression at IntegrationTestScenario. Even consultation with konflux user support bot and the konux user notebookLM say this won't work.

@xinredhat
Copy link
Copy Markdown
Member

Because the e2e PipelineRun is triggered after pull-request/push pipelinerun, my understanding is that the above cel expression can be moved to pull-request/push pipelinerun.

@Roming22 Roming22 closed this May 12, 2026
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.

4 participants