-
Notifications
You must be signed in to change notification settings - Fork 492
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
Support for proxying Juju client/controller connections through the Kubernetes API #12512
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tlm
force-pushed
the
k8s-tunnel-2
branch
3 times, most recently
from
January 14, 2021 07:53
2c92bea
to
f17ae20
Compare
tlm
changed the title
Init commit for kubernetes proxying
Support for proxying Juju client/controller connections through the Kubernetes API
Jan 14, 2021
tlm
force-pushed
the
k8s-tunnel-2
branch
2 times, most recently
from
January 14, 2021 10:10
e0b38e9
to
102803c
Compare
hpidcock
reviewed
Jan 14, 2021
tlm
force-pushed
the
k8s-tunnel-2
branch
2 times, most recently
from
January 15, 2021 01:22
a598277
to
0695c4d
Compare
hpidcock
approved these changes
Jan 15, 2021
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, just a few little details and one bug.
tlm
force-pushed
the
k8s-tunnel-2
branch
8 times, most recently
from
January 15, 2021 06:04
c16c557
to
41bf5b3
Compare
When Juju clients want to talk with Juju controllers inside of k8s they require the controller to be publicly accessible. This commit introduces a change allowing clients to port-forward to the controller when a route able connection is not possible. Main features of the PR: - k8s tunnel and proxier for the k8s api - Start of a wider proxying framework for developing other types of proxy's for Juju controller connections - Changes to the users controller.yaml file describing the proxy to use when making connections.
|
jujubot
added a commit
that referenced
this pull request
Jan 25, 2021
#12550 There are conflicts in this, so we need to be careful about what's landed. Most of the conflicts are in the go.mod/go.sum files. 2940934 (upstream/2.9, origin/2.9, 2.9) Merge pull request #12532 from SimonRichardson/charmhub-download-refresh-api fe8e779 Merge pull request #12527 from SimonRichardson/charmstore-arch 20d95c6 Merge pull request #12546 from hmlanigan/case-sensative 8de1f74 Merge pull request #12542 from hmlanigan/no-charmhub-charm-by-revision e4f02f8 Merge pull request #12544 from jujubot/increment-to-2.9-rc5 6162d5f Merge pull request #12543 from ycliuhw/2.8-into-2.9 81c7a77 Merge pull request #12541 from SimonRichardson/rename-charmhub-model-config f2d798a Merge pull request #12540 from wallyworld/merge-2.8-20210121 0f28abb Merge pull request #12539 from manadart/2.9-lxd-default-root-disk-source a8378b9 Merge pull request #12536 from achilleasa/2.9-fix-panic-when-serializing-proxier 1868862 Merge pull request #12517 from hmlanigan/snapstore-to-charmhub f5ebff1 Merge pull request #12508 from manadart/2.9-remove-unused-retry 283e7bc Merge pull request #12534 from tlm/lp-1912348-socat-kube-proxy 3729449 Merge pull request #12526 from tlm/kube-config-multi 4a49a1a Merge pull request #12521 from SimonRichardson/charmhub-arch-charm-url 5035c50 Merge pull request #12523 from hmlanigan/new-refresh-resources-charmhub 7252d6f Merge pull request #12493 from manadart/2.9-login-spaces-discovery b119871 Merge pull request #12522 from hmlanigan/no-info-find-download-with-older-controller 639114c Merge pull request #12530 from manadart/2.8-into-2.9 963fba8 Merge pull request #12529 from wallyworld/merge-2.8-20210119 9ffbd77 Merge pull request #12525 from tlm/k8s-log-id 65cfe5a Merge pull request #12512 from tlm/k8s-tunnel-2 daa4214 Merge pull request #12488 from SimonRichardson/fallback-architecture 8aa5f70 Merge pull request #12505 from hmlanigan/charmhub-hash-change ec71db6 Merge pull request #12514 from wallyworld/fix-azure-bootstrap-custom-vnet 04f94d3 Merge pull request #12511 from benhoyt/aws-sdk-29-backport 22015bd Merge pull request #12509 from manadart/2.8-into-2.9 182c0b6 Merge pull request #12502 from ycliuhw/fix/ingress-class 6c4a40e Merge pull request #12496 from hpidcock/bootstrap-incluster f1710b7 Merge pull request #12499 from hmlanigan/bug-1910591-cs-channel-resource 5fa2ca9 Merge pull request #12490 from benhoyt/merge-28-29-2 e80aed5 Merge pull request #12487 from hmlanigan/no-cs-required 2dc1434 Merge pull request #12485 from hmlanigan/relation-test-fix
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Support for proxying Juju client through k8s
When Juju clients want to talk with Juju controllers inside of k8s they require the controller to be publicly accessible. This commit introduce a change allowing clients to port-forward to the controller when a route able connection is not possible.
Main features of the PR:
Checklist
Yes, leaving for the moment for another PR.
QA steps
QA Steps is starting a Juju bootstrap. Install a Kubernetes cluster that Juju has previously failed on. Examples such as minikube or kind.
Then bootstrap juju to one of these clusters. The end result should be a perfectly working Juju.
The local users controller.yaml file can be checked for the new proxying information.
Documentation changes
** Coming Soon