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

Make additional validation for CM of cozystack #284

Open
gecube opened this issue Aug 13, 2024 · 1 comment
Open

Make additional validation for CM of cozystack #284

gecube opened this issue Aug 13, 2024 · 1 comment

Comments

@gecube
Copy link
Collaborator

gecube commented Aug 13, 2024

Hi!

Right now it is easy to make a typo when preparing the cozy-stack configmap:

apiVersion: v1
kind: ConfigMap
metadata:
  name: cozystack
  namespace: cozy-system
data:
  bundle-name: "paas-full"
  ipv4-pod-cidr: "100.64.0.0/16"
  ipv4-pod-gateway: "10.64.0.1"
  ipv4-svc-cidr: "10.65.0.0/16"
  ipv4-join-cidr: "100.66.0.0/16"

I think it could be beneficial to install before applying some kind of webhook which will check the parameters and write meaningful errors (like "bundle name is not supported") or even switch from CM to our own CRD with a proper schema.

@mrkhachaturov
Copy link
Contributor

@gecube

I am working on operator for cozystack.

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

No branches or pull requests

2 participants