Change:
) var ( <del> encryptionConfigAutomaticReloadFailureTotal = metrics.NewCounterVec( </del> <ins> encryptionConfigAutomaticReloadsTotal = metrics.NewCounterVec( </ins> &metrics.CounterOpts{ Namespace:      namespace, Subsystem:      subsystem, <del> Name:           "automatic_reload_failures_total", Help:           "Total number of failed automatic reloads of encryption configuration split by apiserver identity.", </del> <ins> Name:           "automatic_reloads_total", Help:           "Total number of reload successes and failures of encryption configuration split by apiserver identity.", </ins> StabilityLevel: metrics.ALPHA, }, <ins> []string{"status", "apiserver_id_hash"}, ) // deprecatedEncryptionConfigAutomaticReloadFailureTotal has been deprecated in 1.30.0 // use encryptionConfigAutomaticReloadsTotal instead deprecatedEncryptionConfigAutomaticReloadFailureTotal = metrics.NewCounterVec( &metrics.CounterOpts{ Namespace:         namespace, Subsystem:         subsystem, Name:              "automatic_reload_failures_total", Help:              "Total number of failed automatic reloads of encryption configuration split by apiserver identity.", StabilityLevel:    metrics.ALPHA, DeprecatedVersion: "1.30.0", }, </ins> []string{"apiserver_id_hash"}, ) <del> encryptionConfigAutomaticReloadSuccessTotal = metrics.NewCounterVec( </del> <ins> // deprecatedEncryptionConfigAutomaticReloadSuccessTotal has been deprecated in 1.30.0 // use encryptionConfigAutomaticReloadsTotal instead deprecatedEncryptionConfigAutomaticReloadSuccessTotal = metrics.NewCounterVec( </ins> &metrics.CounterOpts{ <del> Namespace:      namespace, Subsystem:      subsystem, Name:           "automatic_reload_success_total", Help:           "Total number of successful automatic reloads of encryption configuration split by apiserver identity.", StabilityLevel: metrics.ALPHA, </del> <ins> Namespace:         namespace, Subsystem:         subsystem, Name:              "automatic_reload_success_total", Help:              "Total number of successful automatic reloads of encryption configuration split by apiserver identity.", StabilityLevel:    metrics.ALPHA, DeprecatedVersion: "1.30.0", </ins> }, []string{"apiserver_id_hash"}, )

Discussion:
Change encryption config controller reload metrics to be consistent with other places (Authn, Authz) xref: Combine and into a single metric with with an enum value of and . /sig auth /triage accepted
cc
We probably should keep the existing metrics for some time to allow folks to migrate to the new one.
Ah I was thinking about this as well! Looking at "may be deleted at any time", I was leaning towards just removing it. If the common approach is to keep it for a release before removing, we could do that. I believe we can set a for the 2 metrics we want to remove.
I don't have a good sense of metrics stability because they tend to stay alpha forever and folks don't go back to update them. Since the associated feature set is GA, I think we should be conservative with the changes we make. Perhaps SIG instrumentation folks can provide some guidance here?
As part of the PR, I was going to ask when is the right time to graduate these metrics do you have thoughts on to take while deleting an alpha metric metrics should be graduated from alpha to beta and stable?