Skip to content
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

Merged

Conversation

scoheb
Copy link
Member

@scoheb scoheb commented Aug 28, 2024

  • 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

Addresses KONFLUX-4171

- 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]>
@scoheb scoheb requested review from ralphbean, hugares and gabemontero and removed request for MatousJobanek and mfrancisc August 28, 2024 16:17
Copy link
Contributor

@hugares hugares left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

scoheb added a commit to scoheb/ec-cli that referenced this pull request Aug 28, 2024
- 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]>
Copy link
Member

@ralphbean ralphbean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Copy link

openshift-ci bot commented Aug 29, 2024

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit a454002 into redhat-appstudio:main Aug 29, 2024
4 checks passed
scoheb added a commit to scoheb/ec-cli that referenced this pull request Aug 29, 2024
- 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]>
scoheb added a commit to scoheb/ec-cli that referenced this pull request Sep 4, 2024
- 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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants