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

Error when --image-is-bundle-check flag is set to false while using -b flag #563

Merged
merged 2 commits into from Aug 25, 2023

Conversation

ashpect
Copy link
Contributor

@ashpect ashpect commented Aug 19, 2023

This solves #544

Issue

The command

imgpkg pull -b index.docker.io/<user>/<repo>:<tag> --image-is-bundle-check=false -o <location>

successfully runs and will pull it as an image, which is a bit weird because we are trying to pull a bundle using the bundle flag.

Fix

Now running the command throws this error

imgpkg: Error: Setting --image-is-bundle-check flag to false is not allowed when using -b flag.

@ashpect ashpect had a problem deploying to TanzuNet Registry Dev e2e August 19, 2023 08:17 — with GitHub Actions Failure
@ashpect ashpect changed the title Fix#544 Error when --image-is-bundle-check flag is set to false while using -b flag Aug 19, 2023
@ashpect ashpect temporarily deployed to TanzuNet Registry Dev e2e August 19, 2023 08:18 — with GitHub Actions Inactive
Signed-off-by: ashpect <ashishndiitr@gmail.com>
@ashpect ashpect temporarily deployed to TanzuNet Registry Dev e2e August 20, 2023 15:25 — with GitHub Actions Inactive
@ashpect ashpect temporarily deployed to GCR e2e August 20, 2023 15:25 — with GitHub Actions Inactive
Copy link
Member

@praveenrewar praveenrewar left a comment

Choose a reason for hiding this comment

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

Thank you for working on this @ashpect. The change looks good to me. Would you like to add a test for this as well?

@ashpect
Copy link
Contributor Author

ashpect commented Aug 21, 2023

Yes, definitely.

@ashpect ashpect temporarily deployed to TanzuNet Registry Dev e2e August 24, 2023 17:34 — with GitHub Actions Inactive
@ashpect ashpect temporarily deployed to GCR e2e August 24, 2023 17:34 — with GitHub Actions Inactive
@ashpect
Copy link
Contributor Author

ashpect commented Aug 24, 2023

Hello @praveenrewar. I added a test for the fix.
Please review it whenever you get the time. 🙌

Screenshot 2023-08-24 at 11 07 14 PM

@ashpect ashpect temporarily deployed to TanzuNet Registry Dev e2e August 24, 2023 19:25 — with GitHub Actions Inactive
@ashpect ashpect temporarily deployed to GCR e2e August 24, 2023 19:25 — with GitHub Actions Inactive
Signed-off-by: ashpect <ashishndiitr@gmail.com>
@ashpect ashpect temporarily deployed to TanzuNet Registry Dev e2e August 24, 2023 19:28 — with GitHub Actions Inactive
@ashpect ashpect temporarily deployed to TanzuNet Registry Dev e2e August 25, 2023 03:22 — with GitHub Actions Inactive
@ashpect ashpect temporarily deployed to GCR e2e August 25, 2023 14:37 — with GitHub Actions Inactive
@ashpect ashpect temporarily deployed to TanzuNet Registry Dev e2e August 25, 2023 14:37 — with GitHub Actions Inactive
Copy link
Member

@joaopapereira joaopapereira left a comment

Choose a reason for hiding this comment

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

LGTM

@joaopapereira joaopapereira merged commit c3b6970 into carvel-dev:develop Aug 25, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants