search

aenix-io / etcd-operator

New generation community-driven etcd-operator!
https://etcd.aenix.io
Apache License 2.0
90 stars 13 forks source link

Validate extraArgs field in kind:EtcdCluster #77

Closed sergeyshevch closed 7 months ago

sergeyshevch commented 7 months ago

We should validate extraArgs field and prevent usage of args generated by operator itself. Follow up for #72 #69

sircthulhu commented 7 months ago

Should we check that etcd supports such flags or just reject collisions with flags used by the operator?

sergeyshevch commented 7 months ago

The initial request was just to reject collisions. I guess it will be enough for now

andreykont commented 7 months ago

Which of 3 behavior we want:

  1. Validation web hook
  2. status error
  3. skip the unwanted argument
sergeyshevch commented 7 months ago

I guess validation webhook error

andreykont commented 7 months ago

99