Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use Local Registries with Kind #353

Closed
clshaw01 opened this issue Aug 22, 2024 · 4 comments · Fixed by #383
Closed

Use Local Registries with Kind #353

clshaw01 opened this issue Aug 22, 2024 · 4 comments · Fixed by #383
Assignees
Labels

Comments

@clshaw01
Copy link
Contributor

clshaw01 commented Aug 22, 2024

See https://kind.sigs.k8s.io/docs/user/local-registry/

Running local registry script sets it up and Tilt detects/uses it, but on second run it doesn't seem to pick up the local registry.

Could give us a speed boost when dealing with images.

Image

@clshaw01
Copy link
Contributor Author

example:

  • scaf myproject
  • tilt up (images pulled into cluster's repository, not docker's)

this will create a cache cluster to prevent conflicts

@daveoconnor daveoconnor self-assigned this Aug 27, 2024
@clshaw01
Copy link
Contributor Author

quick fix: add patch in kubernetes kustomization

try: image registry setting in tilt (set to localhost:5000 so tilt will patch all requests through local registry)
see: https://docs.tilt.dev/personal_registry.html

long term: use k3s w/local registry to proxy image requests

@daveoconnor
Copy link
Contributor

daveoconnor commented Sep 18, 2024

default_registry only applies to images tilt builds, not ones it downloads. I wasn't seeing an improvement in pull times, docs have this:

Specifies that any images that Tilt builds should be renamed so that they have the specified Docker registry.

States "builds", but not working for image pulls from what I can see in trying to get it to work.

I'm back to thinking tagging and pushing with patches on the kubernetes kustomization.yaml for the services would be the way for now.

@clshaw01 clshaw01 assigned rochecompaan and unassigned daveoconnor Sep 18, 2024
@clshaw01 clshaw01 linked a pull request Sep 19, 2024 that will close this issue
github-actions bot pushed a commit that referenced this issue Sep 21, 2024
## [1.15.0](v1.14.0...v1.15.0) (2024-09-21)

### Features

* added local registry support and improved setup (close [#353](#353)) ([32de8c7](32de8c7))
* Gfranxman/313 prefer dotlocalbin as install location ([#342](#342)) ([dbb3556](dbb3556))
* implement s3 storage for static and media files on production ([#339](#339)) ([76afd23](76afd23))
* implement scaf challenge for session recording ([#379](#379)) ([0c091af](0c091af))
* **install.sh:** force re-download of Scaf on each install ([#338](#338)) ([f7ef3e3](f7ef3e3))
* op inject secrets (close [#368](#368)) ([58090b5](58090b5))
* reorder environment variables in k8s django config ([#350](#350)) ([f2cb234](f2cb234))
* script to test cookiecutter part of Scaf ([952e276](952e276))

### Bug Fixes

* Fixes frontend tests ([#363](#363)) ([4a2a5b2](4a2a5b2))
* mailhog port config (closes [#249](#249)) ([cea1602](cea1602))
* Specify the full container path, including the host. ([#309](#309)) ([2fa7276](2fa7276)), closes [#308](#308)
* Update logo ([#373](#373)) ([afe2d84](afe2d84))
* use GitHub token for semantic release workflow ([b5eef60](b5eef60))

### Documentation

* add disk space warning to README ([#385](#385)) ([4262b2d](4262b2d))
* update related to optional GraphQL ([#366](#366)) ([92bfc8d](92bfc8d)), closes [#290](#290)
Copy link

🎉 This issue has been resolved in version 1.15.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging a pull request may close this issue.

3 participants