Skip to content

Remove component adoption labels before initial CSV copy. #2164

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

Conversation

benluddy
Copy link
Contributor

@benluddy benluddy commented May 18, 2021

Operator component labels were not being stripped before initially
creating a CSV copy (they were only being stripped on subsequent
updates to copies).

This explains why the end-to-end test for copied CSV adoption has been failing sporadically.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 18, 2021
@openshift-ci
Copy link

openshift-ci bot commented May 18, 2021

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot requested review from awgreene and timflannagan May 18, 2021 15:00
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 18, 2021
@benluddy benluddy force-pushed the copied-csv-adoption-flake branch from d5dd67d to a5890d4 Compare May 18, 2021 17:11
@benluddy
Copy link
Contributor Author

#2091

@benluddy benluddy marked this pull request as ready for review May 18, 2021 17:24
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 18, 2021
@benluddy benluddy force-pushed the copied-csv-adoption-flake branch from a5890d4 to 6959fe2 Compare May 18, 2021 17:27
@benluddy
Copy link
Contributor Author

/hold

Looking into a test failure that looks related:

[Fail] ClusterServiceVersion when a copied csv exists [It] is synchronized with the original csv

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 18, 2021
Copy link
Member

@dinhxuanvu dinhxuanvu left a comment

Choose a reason for hiding this comment

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

One nit. Otherwise, lgtm.
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 18, 2021
@benluddy benluddy force-pushed the copied-csv-adoption-flake branch from 6959fe2 to bfb97a4 Compare May 21, 2021 13:17
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label May 21, 2021
Operator component labels were not being stripped before initially
creating a CSV copy (they were only being stripped on subsequent
updates to copies).

Signed-off-by: Ben Luddy <[email protected]>
@benluddy benluddy force-pushed the copied-csv-adoption-flake branch from bfb97a4 to 7a63093 Compare May 21, 2021 13:18
@benluddy
Copy link
Contributor Author

/hold cancel

Should be resolved now -- the cache-get is load-bearing and has to stay.

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 21, 2021
@benluddy benluddy requested a review from dinhxuanvu May 21, 2021 13:26
Copy link
Member

@dinhxuanvu dinhxuanvu 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 May 21, 2021
@openshift-ci
Copy link

openshift-ci bot commented May 21, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: benluddy, dinhxuanvu

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-robot openshift-merge-robot merged commit 4871fe3 into operator-framework:master May 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants