[DNM] Sync Serverless CI config/#79402
Open
maschmid wants to merge 1 commit into
Open
Conversation
Contributor
Author
|
/hold |
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (9)
📒 Files selected for processing (11)
WalkthroughThis pull request adds Slack reporter configuration to CI job definitions across multiple Knative repositories. Each job now reports success, failure, and error states to the appropriate project-specific Slack channel with a templated message that includes the job name and a link to the logs. ChangesSlack Reporter Configuration for CI Jobs
🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 12✅ Passed checks (12 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
openshift-ci
Bot
added
the
do-not-merge/hold
Contributor
Contributor
|
[REHEARSALNOTIFIER]
A total of 37 jobs have been affected by this change. The above listing is non-exhaustive and limited to 25 jobs. A full list of affected jobs can be found here Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
Contributor
|
@maschmid: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
test only
Summary
This pull request synchronizes CI operator configuration for multiple OpenShift Knative projects by adding Slack reporter configurations to test jobs. The changes enable automated job status notifications to be posted to project-specific Slack channels.
Changes
The PR modifies CI job configurations across five Knative-related OpenShift projects:
For each project,
reporter_configblocks are added to CI test job definitions, configuring:#knative-eventing-cifor eventing projects,#serverless-cifor serverless-operator, and#knative-serving-cifor servingsuccess,failure, anderrorThe test jobs receiving these configurations follow the naming pattern
*-c(e.g.,test-e2e-c,test-conformance-c,test-reconciler-c) and vary by project and version.Total impact: ~210 lines added across 10 configuration files with no removals.
Status: Marked as "DNM" (Do Not Merge) and labeled as test-only, indicating this is a draft/experimental change.