Skip to content

Conversation

fabianvf
Copy link
Member

No description provided.

id: prep
run: |
IMG=quay.io/joelanford/helm-operator
IMG=quay.io/operator-framework/helm-operator-plugins
Copy link
Member Author

Choose a reason for hiding this comment

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

TODO: I think we probably should just skip the deploy step for now since the actual image is being released from the main SDK repo, I didn't want to rename this to quay.io/operator-framework/helm-operator out of fear we may accidentally overwrite the real image before we were ready

@fabianvf fabianvf requested a review from joelanford June 22, 2021 16:43
Copy link
Member

@varshaprasad96 varshaprasad96 left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 22, 2021
@openshift-ci
Copy link

openshift-ci bot commented Jun 24, 2021

New changes are detected. LGTM label has been removed.

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jun 24, 2021
fabianvf and others added 2 commits June 24, 2021 22:52
Signed-off-by: varshaprasad96 <varshaprasad96@gmail.com>
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.

2 participants