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

fix checkPushPermissions by --insecure #683

Closed
wants to merge 19 commits into from

Conversation

cn-sunletian
Copy link

@cn-sunletian cn-sunletian commented May 31, 2019

Our registry is harbor by http. When I push image, error occurred on function CheckPushPermissions.
Even add "--insecure" that doesn't work, but add "--insecure-pull" it can pull well.So i find "push.go" an error here. #737

@abergmeier
Copy link
Contributor

Have a look at #663.

@cn-sunletian cn-sunletian changed the title fix checking push permissions by http fix checking push permissions by --insecure Jun 1, 2019
@cn-sunletian cn-sunletian changed the title fix checking push permissions by --insecure fix checkPushPermissions by --insecure Jun 1, 2019
yehiyam added a commit to kube-HPC/hkube that referenced this pull request Jul 1, 2019
yehiyam added a commit to kube-HPC/kaniko that referenced this pull request Jul 1, 2019
nassiharel pushed a commit to kube-HPC/hkube that referenced this pull request Jul 1, 2019
* support insecure registry
needs PR GoogleContainerTools/kaniko#683  to be merged
hkube-ci pushed a commit to kube-HPC/hkube that referenced this pull request Jul 1, 2019
* support insecure registry
needs PR GoogleContainerTools/kaniko#683  to be merged .... bump version [skip ci]
hkube-ci pushed a commit to kube-HPC/hkube that referenced this pull request Jul 1, 2019
* support insecure registry
needs PR GoogleContainerTools/kaniko#683  to be merged .... bump version [skip ci]
@cn-sunletian
Copy link
Author

@kokoro-team plz check it

@SamyCoenen
Copy link

@priyawadhwa @sharifelgamal With the current release 0.10 I get HTTPS issues, I created my own image from @cn-sunletian fork samyco/kaniko and it works.

Copy link
Collaborator

@priyawadhwa priyawadhwa left a comment

Choose a reason for hiding this comment

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

This LGTM! I think you checked in the .idea directory by accident, could you remove those files? I also don't think the vendor directory needs to be updated in this case :)

@cn-sunletian
Copy link
Author

@priyawadhwa Thank you for reminding me :)

yehiyam added a commit to kube-HPC/hkube that referenced this pull request Jul 29, 2019
* support insecure registry
needs PR GoogleContainerTools/kaniko#683  to be merged

* save

* supprt insecure and skip-verify
hkube-ci pushed a commit to kube-HPC/hkube that referenced this pull request Jul 29, 2019
* support insecure registry
needs PR GoogleContainerTools/kaniko#683  to be merged

* save

* supprt insecure and skip-verify
 .... bump version [skip ci]
hkube-ci pushed a commit to kube-HPC/hkube that referenced this pull request Jul 29, 2019
* support insecure registry
needs PR GoogleContainerTools/kaniko#683  to be merged

* save

* supprt insecure and skip-verify
 .... bump version [skip ci]
@tejal29
Copy link
Member

tejal29 commented Aug 9, 2019

@cn-sunletian can you fix the lint errors? Looking forward to get this in!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants