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

Support external LINSTOR Controllers #451

Merged
merged 1 commit into from
Apr 6, 2023
Merged

Conversation

WanzenBug
Copy link
Member

Allow users to specify the URL of an external LINSTOR Controller. When specified, no LINSTOR Controller is deployed. Instead the operator will reconcile the Kubernetes resources so that they use the external controller instead.

Fixes #447 for v2

@WanzenBug WanzenBug force-pushed the external-controller branch 2 times, most recently from 035e5d9 to ab21e7f Compare April 6, 2023 09:35
@WanzenBug WanzenBug requested a review from JoelColledge April 6, 2023 09:40
Copy link
Collaborator

@JoelColledge JoelColledge left a comment

Choose a reason for hiding this comment

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

Looks great to me.

pkg/linstorhelper/client.go Outdated Show resolved Hide resolved
docs/how-to/external-controller.md Outdated Show resolved Hide resolved
Allow users to specify the URL of an external LINSTOR Controller. When specified,
no LINSTOR Controller is deployed. Instead the operator will reconcile the
Kubernetes resources so that they use the external controller instead.

Also adds docs on how to use the external controller.

Signed-off-by: Moritz "WanzenBug" Wanzenböck <moritz.wanzenboeck@linbit.com>
@WanzenBug WanzenBug force-pushed the external-controller branch from ab21e7f to 78e2a49 Compare April 6, 2023 11:31
@JoelColledge JoelColledge merged commit 50f9d10 into v2 Apr 6, 2023
@JoelColledge JoelColledge deleted the external-controller branch April 6, 2023 11:56
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.

Piraeus operators connected to external linstor storage clusters
2 participants