Skip to content

ci: check notification - #1174

Closed
saw-jan wants to merge 4 commits into
mainfrom
ci/pr-event-notification
Closed

saw-jan wants to merge 4 commits into
mainfrom
ci/pr-event-notification

Conversation

@saw-jan

@saw-jan saw-jan commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@saw-jan saw-jan self-assigned this Oct 14, 2026
@saw-jan saw-jan added Type:Test Type:DevOps Related to CI/CD labels Oct 14, 2026
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
@saw-jan
saw-jan force-pushed the ci/pr-event-notification branch from 68b8307 to 608c0bd Compare October 15, 2026 11:57
@saw-jan

saw-jan commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

This is by design that if the pipeline fails, the notification workflow also fails with exit code 1.
I was just confused why the workflow failed even after the successful notify action.
https://github.com/opencloud-eu/qa/blob/b6afd9ffc6ef60fed01815c45a5c65ef16f6bced/scripts/matrix-notification/matrix-notification.go#L127-L130

@saw-jan saw-jan closed this Oct 15, 2026
@saw-jan
saw-jan deleted the ci/pr-event-notification branch October 15, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type:DevOps Related to CI/CD Type:Test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant