-
Notifications
You must be signed in to change notification settings - Fork 245
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
Enable pipeline SA to read and list plrs #4413
Enable pipeline SA to read and list plrs #4413
Conversation
- This PR enables the appstudio-pipeline service account to get and list PipelineRuns - The motivation for this is to allow Integration Test Scenarios PipelineRuns to query itself to inspect label and annotations added by the Integration Service such as: ``` appstudio.openshift.io/component: acb-service test.appstudio.openshift.io/type: component ``` which tells you that the underlying Snapshot for this Test was created by a component that was recently built called `abc-service` Signed-off-by: Scott Hebert <[email protected]>
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.
LGTM
- Introducing Single Component Mode - See KONFLUX-4059 - This step, if activated by the SINGLE_COMPONENT parameter and a valid pipelinerun name, will attempt to reduce the incoming snapshot to contain a single component. That component being the one that caused the snapshot to create for. - This is determined by examining the labels of the currently running pipelinerun: ``` appstudio.openshift.io/component: acb-service test.appstudio.openshift.io/type: component ``` - This tells you that the underlying snapshot for this Test was created by a `component` that was recently built called `abc-service` NOTE: This requires redhat-appstudio/infra-deployments#4413 Signed-off-by: Scott Hebert <[email protected]>
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hugares, ralphbean, scoheb The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
a454002
into
redhat-appstudio:main
- Introducing Single Component Mode - See KONFLUX-4059 - This step, if activated by the SINGLE_COMPONENT parameter and a valid pipelinerun name, will attempt to reduce the incoming snapshot to contain a single component. That component being the one that caused the snapshot to create for. - This is determined by examining the labels of the currently running pipelinerun: ``` appstudio.openshift.io/component: acb-service test.appstudio.openshift.io/type: component ``` - This tells you that the underlying snapshot for this Test was created by a `component` that was recently built called `abc-service` NOTE: This requires redhat-appstudio/infra-deployments#4413 Signed-off-by: Scott Hebert <[email protected]>
- Introducing Single Component Mode - See KONFLUX-4059 - This step, if activated by the SINGLE_COMPONENT parameter and a valid pipelinerun name, will attempt to reduce the incoming snapshot to contain a single component. That component being the one that caused the snapshot to create for. - This is determined by examining the labels of the currently running pipelinerun: ``` appstudio.openshift.io/component: acb-service test.appstudio.openshift.io/type: component ``` - This tells you that the underlying snapshot for this Test was created by a `component` that was recently built called `abc-service` NOTE: This requires redhat-appstudio/infra-deployments#4413 Signed-off-by: Scott Hebert <[email protected]>
which tells you that the underlying Snapshot for this Test was created by a component that was recently built called
abc-service
Addresses KONFLUX-4171