Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

Decomission of SnapshotEnvironmentBinding controller #918

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

mshaposhnik
Copy link
Collaborator

What does this PR do?

Since SEB-s are subject to upcoming removal, there is no need to have a target setting controller anymore.
Targets for push and pull secrets will be created automatically on apllication component processing, (see konflux-ci/image-controller#97)

Screenshot/screencast of this PR

What issues does this PR fix or reference?

https://issues.redhat.com/browse/SVPI-728

How to test this PR?

quay.io/redhat-appstudio/pull-request-builds:spi-controller-#
quay.io/redhat-appstudio/pull-request-builds:spi-oauth-#

Signed-off-by: Max Shaposhnyk <[email protected]>
Signed-off-by: Max Shaposhnyk <[email protected]>
@openshift-ci openshift-ci bot added approved and removed lgtm labels Mar 15, 2024
Copy link

openshift-ci bot commented Mar 15, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mshaposhnik, skabashnyuk

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:
  • OWNERS [mshaposhnik,skabashnyuk]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mshaposhnik mshaposhnik changed the title Decomission of SnapshotENvironmentBinding controller Decomission of SnapshotEnvironmentBinding controller Mar 15, 2024
@mshaposhnik mshaposhnik merged commit 7d1d8aa into konflux-ci:main Mar 18, 2024
12 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants