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

Metrics Stability Enhancement #1209

Closed
logicalhan opened this issue Aug 8, 2019 · 32 comments
Closed

Metrics Stability Enhancement #1209

logicalhan opened this issue Aug 8, 2019 · 32 comments
Labels
sig/instrumentation Categorizes an issue or PR as relevant to SIG Instrumentation. stage/stable Denotes an issue tracking an enhancement targeted for Stable/GA status tracked/no Denotes an enhancement issue is NOT actively being tracked by the Release Team
Milestone

Comments

@logicalhan
Copy link
Member

Enhancement Description

  • One-line enhancement description (can be used as a release note): Introduce a framework for providing stability guarantees for Kubernetes control plane metrics.
  • Kubernetes Enhancement Proposals: Metrics Stability Framework, Metrics Validation and Verification, Metrics Migration
  • Primary contact (assignee): @logicalhan, @serathius
  • Responsible SIGs: sig-instrumentation
  • Enhancement target (which target equals to which milestone): This isn't a feature per se, but we intend to have the metrics migrated as of 1.16. The migration itself will be a backwards compatible change.
@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Aug 8, 2019
@brancz brancz added the sig/instrumentation Categorizes an issue or PR as relevant to SIG Instrumentation. label Aug 8, 2019
@k8s-ci-robot k8s-ci-robot removed the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Aug 8, 2019
@kacole2
Copy link

kacole2 commented Aug 12, 2019

@logicalhan How is this different than #1206? Is this an on going genesis of #1206? Or is this a completely new Enhancement?

We need to find an umbrella issue for this if this will be in 1.16

@kacole2
Copy link

kacole2 commented Aug 13, 2019

/close
in favor of #1206

@k8s-ci-robot
Copy link
Contributor

@kacole2: Closing this issue.

In response to this:

/close
in favor of #1206

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@serathius
Copy link
Contributor

@kacole2 This is someone different effort driven by different people. "Kubernetes Metrics Overhaul" is about unifying how metrics are written. This effort is about introducing stability to
K8s metrics by introducing new library.

@kacole2
Copy link

kacole2 commented Aug 13, 2019

@serathius I think there is a disconnect. There needs to be an issue created for each type of Enhancement. I see 3 KEPs here. We've never had 3 KEPs assigned to a single issue.

Can we update the title to be more descriptive? Can we also update the description to be more descriptive of what this enhancement should be tracking? Do all 3 KEPs align to this "single" enhancement?

The way I see it.

  1. Metric Migration is one issue (is this Kubernetes Metrics Overhaul #1206 ?)
  2. New Metric Library is another issue. (is stability part of this? and it needs to be called out in graduation criteria in the KEP)

@lachie83 @mrbobbytables @justaugustus

@kacole2 kacole2 reopened this Aug 13, 2019
@logicalhan
Copy link
Member Author

We are actually going to merge these under the metrics overhaul KEP, since the stability work is (at this point) a precondition of the overhaul metric deprecation.

@ehashman
Copy link
Member

I'm going to reopen this as the original tracker for metrics stability, as it's still in beta and hasn't GA'd

@ehashman ehashman reopened this Jan 21, 2021
@ehashman ehashman added the stage/beta Denotes an issue tracking an enhancement targeted for Beta status label Jan 21, 2021
@ehashman
Copy link
Member

ehashman commented Jan 21, 2021

It's not letting me edit the first comment but here's my updated description.

I'm going to try to merge the three different docs into a single lifecycled KEP.

Enhancement Description

@ehashman
Copy link
Member

As a SIG (per 2021-01-21 meeting) we want to graduate this to stable in 1.21. We need to update the KEP (and get that merged) in order to track what's remaining for this...

@ehashman
Copy link
Member

/milestone v1.21

@k8s-ci-robot k8s-ci-robot added this to the v1.21 milestone Jan 27, 2021
@annajung annajung added the tracked/yes Denotes an enhancement issue is actively being tracked by the Release Team label Feb 4, 2021
@JamesLaverack
Copy link
Member

👋 Hey @logicalhan and @serathius, 1.21 enhancements shadow here.

This enhancement is being tracked for the 1.21 release, but I have a few queries about your KEP.

  • I understand that this is going GA in 1.21, but the stage field in your kep.yaml states it's going to beta. Is this out of date?
  • There's a TODO, mentioning @logicalhan in the Beta -> GA graduation section, does this need to be resolved?
  • Five different SIGs (Api Machinery, Node, Scheduling, Cluster Lifecycle, and Cloud Provider) are marked as parcitipating. Do they need to do anything for the GA graduation, and if so are they signed on?
  • You don't have a production readiness review. As of 1.21 this is a requirement for enhancements. The stub section in the KEP mentions that @logicalhan can write one?

@JamesLaverack JamesLaverack added stage/stable Denotes an issue tracking an enhancement targeted for Stable/GA status and removed stage/beta Denotes an issue tracking an enhancement targeted for Beta status labels Feb 5, 2021
@ehashman
Copy link
Member

ehashman commented Feb 8, 2021

I believe all concerns in the comment above were addressed in #2431.

@JamesLaverack
Copy link
Member

Hi @logicalhan and @serathius,

Since your Enhancement is scheduled to be in 1.21, please keep in mind the important upcoming dates:

  • Tuesday, March 9th: Week 9 — Code Freeze
  • Tuesday, March 16th: Week 10 — Docs Placeholder PR deadline
    • If this enhancement requires new docs or modification to existing docs, please follow the steps in the Open a placeholder PR doc to open a PR against k/website repo.

As a reminder, please link all of your k/k PR(s) and k/website PR(s) to this issue so we can track them.

Thanks!

@JamesLaverack
Copy link
Member

Hi @logicalhan and @serathius

Enhancements team is marking this enhancement as "At Risk" for the upcoming code freeze due to not seeing any linked k/k PR(s) for this enhancement.

Please make sure to provide all k/k PR(s) and k/website PR(s) to this issue so it can be tracked by the release team.

@ehashman
Copy link
Member

ehashman commented Mar 4, 2021

@logicalhan can you link the PRs adding stable metrics here?

@JamesLaverack
Copy link
Member

Hi @logicalhan and @serathius,

A friendly reminder that Code freeze is 3 days away, March 9th EOD PST

Any enhancements that are NOT code complete by the freeze will be removed from the milestone and will require an exception to be added back.

Please also keep in mind that if this enhancement requires new docs or modification to existing docs, you'll need to follow the steps in the Open a placeholder PR doc to open a PR against k/website repo by March 16th EOD PST

Thanks!

@logicalhan
Copy link
Member Author

@logicalhan can you link the PRs adding stable metrics here?

kubernetes/kubernetes#99788
kubernetes/kubernetes#99785

@logicalhan
Copy link
Member Author

Also, these two PRs were relevant: kubernetes/kubernetes#99217 & kubernetes/kubernetes#98739

@logicalhan
Copy link
Member Author

Enhancements team is marking this enhancement as "At Risk" for the upcoming code freeze due to not seeing any linked k/k PR(s) for this enhancement.

Ah, sorry about that. I linked the PRs. There is only one pending (which is #99785). It's pretty much good to go, once I hear back from @brancz, I can lgtm/approve it. Personally, I'd characterize this as low risk.

@logicalhan
Copy link
Member Author

Oh there is one more PR: kubernetes/kubernetes#99925

(it's been approved and is passing CI so it should merge today)

@JamesLaverack
Copy link
Member

Thanks for the clarification @logicalhan. Is the only one left to merge kubernetes/kubernetes#99785?

@logicalhan
Copy link
Member Author

Thanks for the clarification @logicalhan. Is the only one left to merge kubernetes/kubernetes#99785?

Yes and I have the requisite approval OWNER ACLs for those files, so there isn't any reason it shouldn't merge today. I'm just waiting on a reply from @brancz.

@logicalhan
Copy link
Member Author

Hi @logicalhan, with the following PRs merged, we're marking this as code complete for the 1.21 release.

Should be code complete now 😄

Thanks for your help!!

@PI-Victor
Copy link
Member

Hello @logicalhan , 1.21 Docs shadow here.
Does this enhancement work planned for 1.21 require any new docs or modification to existing docs?
If so, please follows the steps here to open a PR against dev-1.21 branch in the k/website repo. This PR can be just a placeholder at this time and must be created by March 16 EOD PST
Also take a look at Documenting for a release to get yourself familiarize with the docs requirement for the release.
Thank you!

@ehashman
Copy link
Member

Exception request filed for fixing one stable metric's name: https://groups.google.com/g/kubernetes-sig-instrumentation/c/Uqx4uXxnseg

@PI-Victor we have a list of stable metrics in k/k that is automatically pulled into documentation. I don't believe we have any additional documentation required for graduation.

@ehashman
Copy link
Member

If RT is looking for blog posts, I think this would be an appropriate KEP to feature for the 1.21 release. @logicalhan and I had drafted a blog post for 1.16 that never got run which we can update.

@palnabarun
Copy link
Member

@ehashman -- thank you for the ping! 👍🏽

cc: @divya-mohan0209 you may want to have a look at this for the feature blogs.

@divya-mohan0209
Copy link
Contributor

divya-mohan0209 commented Mar 17, 2021

@ehashman -- thank you for the ping! 👍🏽

cc: @divya-mohan0209 you may want to have a look at this for the feature blogs.

@Pensu : PTAL & liaise with @ehashman for further tracking of the feature blog. Thank you @palnabarun & @ehashman for notifying us!

@annajung
Copy link
Contributor

annajung commented Apr 8, 2021

Hi @logicalhan @serathius 1.21 Enhancement Lead here.

Can you update the kep.yaml to reflect a status of implemented:

Once that merges, we can close out this issue.

@logicalhan
Copy link
Member Author

Hi @logicalhan @serathius 1.21 Enhancement Lead here.

Can you update the kep.yaml to reflect a status of implemented:

Once that merges, we can close out this issue.

#2612

@annajung
Copy link
Contributor

annajung commented Apr 9, 2021

with enhancement graduated to stable and #2612 merged, I am closing out this issue

@annajung annajung closed this as completed Apr 9, 2021
@JamesLaverack JamesLaverack added tracked/no Denotes an enhancement issue is NOT actively being tracked by the Release Team and removed tracked/yes Denotes an enhancement issue is actively being tracked by the Release Team labels Apr 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/instrumentation Categorizes an issue or PR as relevant to SIG Instrumentation. stage/stable Denotes an issue tracking an enhancement targeted for Stable/GA status tracked/no Denotes an enhancement issue is NOT actively being tracked by the Release Team
Projects
None yet
Development

No branches or pull requests