Skip to content

Add internal registry#44

Merged
Kidswiss merged 3 commits intomasterfrom
add/internal_registry
Nov 18, 2025
Merged

Add internal registry#44
Kidswiss merged 3 commits intomasterfrom
add/internal_registry

Conversation

@Kidswiss
Copy link
Contributor

This adds an internal registry to kindev.

It's exposed at localhost:5000. It needs to be added as an insecure registry to docker though.

To pull pods from the internal registry localhost:5000 can also be used as the registry.

For helm charts via provider-helm the registry has to be set as registry.kube-system.svc.cluster.local:5000.

@Kidswiss Kidswiss force-pushed the add/internal_registry branch from 851245a to fda305b Compare November 12, 2025 12:10
@Kidswiss Kidswiss requested review from a team, TheBigLee, mdnix, mikeshootzz and zugao and removed request for a team November 14, 2025 08:27
- name: server-cert
secret:
secretName: tls-server-certificate
# Localprovisioner doesn't want to provision the PVC on the control plane
Copy link

Choose a reason for hiding this comment

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

Is this still necessary to be kept?

- name: server-cert
secret:
secretName: tls-server-certificate
# Localprovisioner doesn't want to provision the PVC on the control plane
Copy link
Member

Choose a reason for hiding this comment

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

Drop this if we don't need it.

@Kidswiss
Copy link
Contributor Author

@zugao & @mdnix my reasoning for this comment was so that we still know in 6 months why it's this way. The problem with using emptydir is that all the images have to be re-pushed when things restart...

@mdnix
Copy link
Member

mdnix commented Nov 18, 2025

@zugao & @mdnix my reasoning for this comment was so that we still know in 6 months why it's this way. The problem with using emptydir is that all the images have to be re-pushed when things restart...

Ok got it, leave it there for future us

@Kidswiss Kidswiss merged commit 24b872f into master Nov 18, 2025
1 check passed
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.

3 participants