-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(RELEASE-119): add support to run a tenant PipelineRun #451
Conversation
Skipping CI for Draft Pull Request. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #451 +/- ##
==========================================
- Coverage 84.95% 84.69% -0.27%
==========================================
Files 26 26
Lines 1396 1411 +15
==========================================
+ Hits 1186 1195 +9
- Misses 152 156 +4
- Partials 58 60 +2 ☔ View full report in Codecov by Sentry. |
7f54b9f
to
8fc1c54
Compare
Quality Gate passedIssues Measures |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #451 +/- ##
==========================================
- Coverage 84.95% 83.93% -1.02%
==========================================
Files 26 26
Lines 1396 1488 +92
==========================================
+ Hits 1186 1249 +63
- Misses 152 170 +18
- Partials 58 69 +11 ☔ View full report in Codecov by Sentry. |
/ok-to-test |
c5750a6
to
c3067d4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@davidmogar is this how we want it? I was under the understanding that the tenant pipeline would execute before, not in place of, the managed pipeline
/test release-service-e2e |
a1f89f6
to
d671826
Compare
Moving it back to draft as changes are required. |
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am good with the changes. Now just to get David's approval 🙂
/retest |
/test all |
This commit adds support for running tenant Pipelines. Signed-off-by: Leandro Mendes <lmendes@redhat.com>
New changes are detected. LGTM label has been removed. |
This commit includes: - konflux-ci/release-service#473 - konflux-ci/release-service#472 - konflux-ci/release-service#451 Signed-off-by: Leandro Mendes <lmendes@redhat.com>
This commit includes: - konflux-ci/release-service#473 - konflux-ci/release-service#472 - konflux-ci/release-service#451 Signed-off-by: Leandro Mendes <lmendes@redhat.com>
This commit includes: - konflux-ci/release-service#473 - konflux-ci/release-service#472 - konflux-ci/release-service#451 Signed-off-by: Leandro Mendes <lmendes@redhat.com>
This commit adds support for running tenant PipelineRuns
Signed-off-by: Leandro Mendes lmendes@redhat.com