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

Revert printer columns #3652

Merged
merged 2 commits into from
Mar 31, 2021

Conversation

carlisia
Copy link
Contributor

Thank you for contributing to Velero!

Please add a summary of your change

I think this needs a changelog.

Ok so, weirdly enough, this change fixes the weird CRD install error that only happened on my machine.

I will dig into it a bit more to see what if anything could/can be done differently to address this, "this" being a more specific cause of this problem, whatever it is.

Does your change fix a particular issue?

Fixes #3600

Please indicate you've done the following:

@carlisia carlisia added the kind/release-blocker Must fix issues for the coming release (milestone) label Mar 31, 2021
@carlisia
Copy link
Contributor Author

Thanks @ashish-amarnath and @zubron for helping me git bisect this, and @dsu-igeek for giving troubleshooting directions and also running tests himself!

zubron
zubron previously approved these changes Mar 31, 2021
@zubron
Copy link
Contributor

zubron commented Mar 31, 2021

It looks like the revert commit also needs to be signed in order to merge.

Carlisia added 2 commits March 31, 2021 14:03
@carlisia carlisia dismissed stale reviews from zubron and ashish-amarnath via 3fec094 March 31, 2021 21:04
@carlisia carlisia force-pushed the c-revert-printer-columns branch from df2a927 to 3fec094 Compare March 31, 2021 21:04
@ashish-amarnath ashish-amarnath merged commit c65c17c into vmware-tanzu:main Mar 31, 2021
@carlisia carlisia deleted the c-revert-printer-columns branch March 31, 2021 23:16
pradeep288 pushed a commit to pradeep288/velero that referenced this pull request May 4, 2021
* Revert "Add additional printer columns for CRDs (vmware-tanzu#2881)"

This reverts commit 4178d9d.

Signed-off-by: Carlisia <[email protected]>

* Add generated files

Signed-off-by: Carlisia <[email protected]>
Signed-off-by: Pradeep Jigalur <[email protected]>
dharmab pushed a commit to dharmab/velero that referenced this pull request May 25, 2021
* Revert "Add additional printer columns for CRDs (vmware-tanzu#2881)"

This reverts commit 4178d9d.

Signed-off-by: Carlisia <[email protected]>

* Add generated files

Signed-off-by: Carlisia <[email protected]>
ywk253100 pushed a commit to ywk253100/velero that referenced this pull request Jun 29, 2021
* Revert "Add additional printer columns for CRDs (vmware-tanzu#2881)"

This reverts commit 4178d9d.

Signed-off-by: Carlisia <[email protected]>

* Add generated files

Signed-off-by: Carlisia <[email protected]>
gyaozhou pushed a commit to gyaozhou/velero-read that referenced this pull request May 14, 2022
* Revert "Add additional printer columns for CRDs (vmware-tanzu#2881)"

This reverts commit 4178d9d.

Signed-off-by: Carlisia <[email protected]>

* Add generated files

Signed-off-by: Carlisia <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-changelog kind/release-blocker Must fix issues for the coming release (milestone)
Projects
None yet
3 participants