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

KeepalivedGroup should be a cluster-level resource #8

Open
raffaelespazzoli opened this issue Mar 2, 2020 · 8 comments
Open

KeepalivedGroup should be a cluster-level resource #8

raffaelespazzoli opened this issue Mar 2, 2020 · 8 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@raffaelespazzoli
Copy link
Collaborator

Currently the KeepalivedGroup CRD is a namespaced resource, but it makes more sense for it to be a cluster-level resource.

@raffaelespazzoli raffaelespazzoli added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Mar 2, 2020
@osirisstar-hub
Copy link

Perhaps I am confused but this seems to only work in a single namespace. So each namespace would have to have their own operator rather than their being a cluster operator. However, when installing two operators I get a port conflict. Do I just simply change the port number used to avoid the conflict?

@osirisstar-hub
Copy link

In order to get more than one operator running in a cluster so that multiple namespaces can get external IPs, I had to change the Prometheus port number in the daemonset.

Warning FailedScheduling pod/keepalivedgroup-workers-zpx8f 0/11 nodes are available: 10 node(s) didn't match node selector, 8 node(s) didn't have free ports for the requested pod ports.

@raffaelespazzoli
Copy link
Collaborator Author

I am not sure I am following.
Anyway the operator is designed to be a cluster-wide operator, but due to an issue with the last release, OLM installs it as a namespaced operator. If you deploy the operator with watch_namespace="", it will work as expected.
This will be fixed in the next release.

@osirisstar-hub
Copy link

watch_namespace goes into the operator yaml or into the subscription yaml?

@raffaelespazzoli
Copy link
Collaborator Author

raffaelespazzoli commented Jul 22, 2020 via email

@raffaelespazzoli
Copy link
Collaborator Author

the latest release should have solved the issue with OLM.

@slauger
Copy link

slauger commented Jul 28, 2020

When could we expect a new release in the OperatorHub @raffaelespazzoli?

Furthermore it would be great if you could create a "real" release on GH (not just a tag). That would allow us to get notified when a new release is available.

@raffaelespazzoli
Copy link
Collaborator Author

raffaelespazzoli commented Jul 28, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants