Rename/Cleanup CLI Flags #872
Labels
area/open-source
Open Source (community, enablement, contributions, conferences, CNCF, etc.) related
area/quality
Output qualification (tests, checks, scans, automation in general, etc.) related
area/usability
Usability related
kind/enhancement
Enhancement, improvement, extension
How to categorize this issue?
/area open-source
/area quality
/area usability
/kind enhancement
What would you like to be added:
leader-election-resource-lock
is marked as deprecated but its value is honoured. Ideally we should ignore the value that is set via this flag. in v0.23.0 this was marked as deprecated and it should be removed in v0.27.0disable-lease-cache
we currently do not know why this flag is required. This is not configured/set in g/g use case.etcd-member-notready-threshold
andetcd-member-unknown-threshold
. Currently it is not very clear on the need for it and even if these flags are set what are the magical values that one can set is also not very clear.etcd-events-threshold
is meant for compaction controller but its name is too generic. This flag should be renamed as from its name its purpose is not clear at all. Similarly there are other CLI flags which have the same problem:active-deadline-duration
,metrics-scrape-wait-duration
reconciler-service-account
is very generic but only applies to the etcd controller.disable-etcd-serviceaccount-automount
should be marked deprecated and removed later. Since k8s 1.24, projected service account token is the default. What we therefore need is only for the user to specify a differentexpirationSeconds
.Why is this needed:
Many of the etcd-druid CLI args are badly named and they do not clearly indicate what their purpose is. Some of these flags are questionable and should be considered to be marked as deprecated and in future removed. Not having well-defined CLI flags make it not-so-easy to consume druid and only adds cognitive overhead to first understand each of the flags and keep that mapping fresh in your mind as well as their name do not suggest what they really do.
The text was updated successfully, but these errors were encountered: