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.

Sign up for GitHub

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

Jump to bottom

operator/pkg/vcluster: add support for vClusters #465

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

operator/pkg/vcluster: add support for vClusters #465

wants to merge 3 commits into from

Conversation

chrisseto
Copy link
Contributor

No description provided.

@chrisseto
pkg/kube: support nested dialers
33e2adf 
Prior to this commit it was not possible to nest PodDialer instances due to
design choices of Kubernetes' spdy package.

This commit re-implements `spdy.RoundTripperFor` with support for overriding
the Dialer used by the SPDY upgrader in favor of HTTP proxy support. This
allows us to easily run the PodDialer over other tunnelling mechanisms.
@chrisseto
b9c3a05
@chrisseto
operator/pkg/vcluster: add support for vClusters
d93e2eb
@chrisseto
Copy link
Contributor Author

This was nasty to get working but we can now run most tests in a single k8s cluster without leaking resources. This will make it possible to hermetically test installs / upgrades of the operator. Spinning up a vCluster is crazy fast. The longest part of the process is waiting for cert-manager to schedule because we install it once per vCluster as syncing is a paid feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RafalKorepta RafalKorepta Awaiting requested review from RafalKorepta RafalKorepta will be requested when the pull request is marked ready for review RafalKorepta is a code owner

@andrewstucki andrewstucki Awaiting requested review from andrewstucki andrewstucki will be requested when the pull request is marked ready for review andrewstucki is a code owner

@gene-redpanda gene-redpanda Awaiting requested review from gene-redpanda gene-redpanda will be requested when the pull request is marked ready for review gene-redpanda is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@chrisseto