Skip to content

Local registry#2326

Merged
FabianKramm merged 41 commits into
devspace-sh:mainfrom
lizardruss:local-repository
Oct 19, 2022
Merged

Local registry#2326
FabianKramm merged 41 commits into
devspace-sh:mainfrom
lizardruss:local-repository

Conversation

@lizardruss

@lizardruss lizardruss commented Sep 20, 2022

Copy link
Copy Markdown
Collaborator

What issue type does this pull request address? (keep at least one, remove the others)
/kind feature

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
Adds a local image registry when user lacks push permissions to the remote registry.

Please provide a short message that should be published in the DevSpace release notes
Fixed an issue where DevSpace ...

What else do we need to know?

TODO

  • Detect push permissions
  • Deploy registry
  • Wait for Registry to be ready
  • Start port forwarding
  • Rewrite image tag for deployed registry
  • Build / Push image
  • Update runtime variables
  • Update config substitution
  • Update ShouldRebuild() to account for local registry usage
  • Acceptance Tests:
    • get_image output
    • ${runtime.images.my-image.image} && ${runtime.images.my-image}
    • image(name)
    • kubectl deployment rewritten
    • helm deployment rewritten
    • dev.*.imageSelector (and all the other different runtime rewritten config sections)
    • dev.*.devImage
  • Move local registry configuration to top level (avoid duplicating for multiple images)
  • Add progress indicator when transferring images
  • devspace init doesn't produce config for local-registry (even in comments).
  • devspace build on minikube with default config(no config for local-registry) spins the registry pod doesn't push the image to it and devspace build command errors out. (Note: updated to skip push for devspace build and minikube, as well as skip local registry for minikube)
  • devspace cleanup local-registry command asks question (sure to cleanup) even when there is no registry pod running.
  • logs need formatting

@lizardruss lizardruss changed the title Local repository Local registry Sep 20, 2022
@lizardruss lizardruss force-pushed the local-repository branch 8 times, most recently from 033cdd6 to 40b0a62 Compare September 23, 2022 14:42
@lizardruss lizardruss marked this pull request as ready for review September 29, 2022 00:14
Comment thread pkg/devspace/build/registry/local_registry.go Outdated
Comment thread pkg/devspace/build/registry/local_registry.go Outdated
Comment thread pkg/devspace/build/registry/local_registry.go Outdated
Comment thread pkg/devspace/config/versions/latest/schema.go Outdated
Comment thread cmd/build.go
@FabianKramm FabianKramm merged commit 04c2339 into devspace-sh:main Oct 19, 2022
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.

2 participants