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

--registry-insecure broken for push and pull since v0.26.0 #378

Closed
ChristianCiach opened this issue Apr 21, 2022 · 2 comments · Fixed by #384
Closed

--registry-insecure broken for push and pull since v0.26.0 #378

ChristianCiach opened this issue Apr 21, 2022 · 2 comments · Fixed by #384
Labels
bug This issue describes a defect or unexpected behavior priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.

Comments

@ChristianCiach
Copy link

What steps did you take:
Starting with imgpkg v0.26.0 the option --registry-insecure has no effect for push and pull commands.

What happened:

$ imgpkg push --registry-insecure -i energy80.workstations.ems:5000/test:bundle -f test.bin
file: test.bin

imgpkg: Error: Writing 'energy80.workstations.ems:5000/test:bundle':
  Error while preparing a transport to talk with the registry:
    Unable to create round tripper:
      Get "https://energy80.workstations.ems:5000/v2/": http: server gave HTTP response to HTTPS client

What did you expect:

$ imgpkg push --registry-insecure -i energy80.workstations.ems:5000/test:bundle -f test.bin
file: test.bin
Pushed 'energy80.workstations.ems:5000/test@sha256:aeab02abc8ce72464a829decf4e4f98d3b0fe24901ed804a6bfd3022f74a486a'
Succeeded

Environment:

  • imgpkg version (use imgpkg --version): imgpkg version 0.28.0
  • Docker registry used (e.g. Docker HUB): Locally hosted registry:2
  • OS (e.g. from /etc/os-release): Fedora Linux 35 (Workstation Edition)

Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@ChristianCiach ChristianCiach added bug This issue describes a defect or unexpected behavior carvel triage This issue has not yet been reviewed for validity labels Apr 21, 2022
@ChristianCiach
Copy link
Author

Probably related to #334

@joaopapereira
Copy link
Member

After some investigation we made a fix for this issue

@joaopapereira joaopapereira added priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. and removed carvel triage This issue has not yet been reviewed for validity labels May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue describes a defect or unexpected behavior priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants