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

Updating private registry support to handle private dockerhub images #2516

Merged
merged 8 commits into from
Feb 25, 2021

Conversation

cdchris12
Copy link
Contributor

Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated
  • PR title is ready for changelog and subsystem label(s) applied

This PR adds support for private docker hub images to kubectl-build-deploy-dind, as this support is non functional at the moment due to a hardcoded dependency on the imagecache

Closing issues

Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

@cdchris12 cdchris12 added the 2-build-deploy Build & Deploy subsystem label Feb 18, 2021
@cdchris12 cdchris12 self-assigned this Feb 18, 2021
@shreddedbacon
Copy link
Member

Have done some testing with the environment reporting the issue using the latest image built by jenkins

testlagoon/kubectl-build-deploy-dind:pr-2516@sha256:020e5623c7e4eff8bde590b2aa2cc0953bee100864c149f2e5978fdb69f46aeb

The environment was able to successfully pull the image from private registry and push to harbor using skopeo, where before it would fail due to imagecache.

We also ran it through a drupal example site in another cluster and it completed successfully, with no private registries defined, it used the imagecache as it was supposed to.

cdchris12 added a commit that referenced this pull request Feb 18, 2021
@cdchris12 cdchris12 added this to the v1.14 milestone Feb 19, 2021
@tobybellwood
Copy link
Member

@cdchris12 - if this needs to make it to Lagoon in the next few weeks, this PR will need to be repointed to the master branch instead of main before we release 1.14 (as 1.14 is a Lagoon 1-only release)

@cdchris12
Copy link
Contributor Author

@tobybellwood I've actually already backported this, I just tagged the wrong PR for release in 1.14 :(

You'll want to merge #2517 instead. It's the same code, just applied to the master branch instead of the main branch

@cdchris12 cdchris12 modified the milestones: v1.14, v2.0.0 Feb 22, 2021
tobybellwood pushed a commit that referenced this pull request Feb 23, 2021
Backporting private docker registry fixes from #2516 to master branch
@tobybellwood
Copy link
Member

Sorry; this is my mistake. :( I'll push up a fix for 1.x so we can at least get this sorted

adds to the list of things we should probably have tests for, but don't 😬

@tobybellwood tobybellwood merged commit 60a5948 into main Feb 25, 2021
@tobybellwood tobybellwood deleted the private_registry_fix branch February 25, 2021 04:45
tobybellwood pushed a commit that referenced this pull request Feb 25, 2021
Matching kubectlbuilddeploy to tested version in #2516
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2-build-deploy Build & Deploy subsystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants