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

add support for local registry discovery #1437

Merged
merged 1 commit into from
Aug 14, 2020
Merged

add support for local registry discovery #1437

merged 1 commit into from
Aug 14, 2020

Conversation

nicks
Copy link
Contributor

@nicks nicks commented Jul 27, 2020

Fixes #1173

@nicks nicks requested a review from ktsakalozos as a code owner July 27, 2020 15:18

# If no local-registry-hosting documentation has been installed,
# install a help guide that points to the microk8s registry docs.
if [ $installed_registry_help -eq 0 ] &&
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I put this code in the apiservice-kicker, because it needs to happen after the apiservice comes up.

This seemed like the best fit for that kind of a life-cycle hook in the current code (even if the fit isn't that great)

But I'm completely open to suggestions on better places to put it!

Copy link
Member

Choose a reason for hiding this comment

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

Indeed we do not have a place to put any maifests we want to apply at boot time.

As you said, it is good enough for now. We will explore alternatives if this causes problems.

Copy link
Member

@ktsakalozos ktsakalozos left a comment

Choose a reason for hiding this comment

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

LGTM +1 thank you @nicks

@ktsakalozos
Copy link
Member

@nicks is there any doc/blog post we could supplement the 1.19 release announcement regarding this feature and push to our social channels? Thanks.

@ktsakalozos ktsakalozos merged commit 4b85f5f into canonical:master Aug 14, 2020
@nicks
Copy link
Contributor Author

nicks commented Aug 19, 2020

@ktsakalozos Not a lot right now aside from https://docs.tilt.dev/choosing_clusters.html#discovering-the-registry and the KEP itself (https://github.com/kubernetes/enhancements/tree/master/keps/sig-cluster-lifecycle/generic/1755-communicating-a-local-registry)

I'm hoping to get a few more implementations in the field and then write about it more on our blog

@nicks nicks deleted the nicks/issue1173 branch August 19, 2020 11:46
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.

feature request: Standard way for microk8s to communicate about a local registry
2 participants