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

Define and collect input metrics about Kubernetes releases #1527

Open
saschagrunert opened this issue Apr 19, 2021 · 23 comments
Open

Define and collect input metrics about Kubernetes releases #1527

saschagrunert opened this issue Apr 19, 2021 · 23 comments
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/feature Categorizes issue or PR as related to a new feature. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. sig/release Categorizes an issue or PR as relevant to SIG Release.

Comments

@saschagrunert
Copy link
Member

saschagrunert commented Apr 19, 2021

Target is to define a set of metrics around Kubernetes releases to elaborate on the reduced release cadence. This discussion came up during the KEP implementation phase. The planned survey outcome can help us to interpret the raw data later on, too.

The following questions have to be resolved before starting to collect the metrics:

  • Which metrics do we want to collect?
    For example, the number of …
    • exceptions requests
    • PR merged after code freeze
    • tests fixed during test freeze
    • issues open during code/test freeze
    • backport per minor release
    • requests for exceptions after code freeze
  • How do we want to collect the data?
    It could be possible that the release team leads collect the metrics before every retrospective and present them there.
    This would allow us to gather feedback within the retrospective before interpreting the data or correlating it to anything.
  • How to track the metrics over multiple releases?
    Making assertions based on the data needs multiple releases as input. We should take care as a SIG to keep track
    of them and evaluate periodically.

/priority important-longterm
/cc @aojea
/help

@saschagrunert saschagrunert added kind/feature Categorizes issue or PR as related to a new feature. sig/release Categorizes an issue or PR as relevant to SIG Release. labels Apr 19, 2021
@k8s-ci-robot k8s-ci-robot added the priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. label Apr 19, 2021
@saschagrunert saschagrunert added the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Apr 19, 2021
@aojea
Copy link
Member

aojea commented Apr 19, 2021

We have to start defining which areas we want to measure, then define the metrics and the actions associated to each metrics, (I'm making up the examples), i.e.:

Quality:

  • CI reliability:
    Description: This measure the CI health of the project based on some specific jobs considered blocking, this is a daily metric
    Metrics: The metrics is obtained from https://testgrid.k8s.io/sig-release-master-blocking , based on number of failed jobs
    Threshold: 99%
    Actions: Immediate action from all SIGs to fix the failing jobs
  • Stability:
    Description: Measure the stability of the project based on the number of backports to the stable branches:
    Metrics: Cherry-picks approved per minor release
    Threshold: this should be trend based, going higher means we are doing more backports, and it may indicate we are neglecting quality
    Actions: ....

Release:

  • Exception requests:
    Description: It measures the project development speed vs current defined timelines. This is a per release metrics.
    Metrics: The metrics is obtained based on the exceptions request to include features after code free time. The information can be clustered if the process use some kind of form, i.e. what is the reason to request an exception: feature to big, bad planning, ....
    Threshold: 10% , if the release expected to have 50 features, only 5 exceptions max are expected
    Actions: If there are many exception request it is a clear symptom that the developer speed/planning is not aligned to the release timeline, it should require wide discussion within the project (maybe scalated to steering ?? )....

ideally, everything should be automated and you just can build a dashboard with that, per example, the exception requests instead of an email can be an issue or a PR, or a form, so you can automate it.

@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 18, 2021
@saschagrunert
Copy link
Member Author

@aojea are you still planning to work on this? :)

@aojea
Copy link
Member

aojea commented Jul 19, 2021

@aojea are you still planning to work on this? :)

Sorry, I was just sharing some experience and things that worked for me in response to the KEP about the release cadence.
I really don't have time ...

@k8s-triage-robot
Copy link

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

@k8s-ci-robot k8s-ci-robot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Aug 18, 2021
@k8s-triage-robot
Copy link

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Reopen this issue or PR with /reopen
  • Mark this issue or PR as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close

@k8s-ci-robot
Copy link
Contributor

@k8s-triage-robot: Closing this issue.

In response to this:

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Reopen this issue or PR with /reopen
  • Mark this issue or PR as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close

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.

@saschagrunert saschagrunert reopened this Oct 5, 2021
@saschagrunert saschagrunert removed the lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. label Oct 5, 2021
@tpepper
Copy link
Member

tpepper commented Oct 7, 2021

Would it make sense to title this "Define and collect input metrics about Kubernetes releases"? Currently it feels scoped in that way. It would be very useful to additionally have output usage metrics (eg: download counts of different artifacts during specific time spans).

@aojea
Copy link
Member

aojea commented Oct 7, 2021

It would be very useful to additionally have output usage metrics (eg: download counts of different artifacts during specific time spans).

we should be careful with those metrics to avoid falling into the marketing/populism, downloads of artifacts is commonly abused by CI systems ... if you just can differentiate users from bots 🤔 , This problem is similar as counting number of contributors and including people that fixed a typo in one year :/

@tpepper
Copy link
Member

tpepper commented Oct 7, 2021

#1372 (comment) represents a way to get at less game-able data perhaps.

Simple stats gaming bots are possible, but I wouldn't consider CI as inherently a problem. I'm actually interested in seeing the bulk CI usage too. If people are still doing a lot of CI on 1.9 that might be informative.

And I'm really interested in the zeros. Which of our RPM and deb variants have zero downloads?

@aojea
Copy link
Member

aojea commented Oct 7, 2021

agree

@saschagrunert saschagrunert changed the title Define and collect metrics about Kubernetes releases Define and collect input metrics about Kubernetes releases Oct 8, 2021
@saschagrunert
Copy link
Member Author

Sounds good, created #1723 for elaboration.

@k8s-triage-robot
Copy link

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jan 6, 2022
@saschagrunert
Copy link
Member Author

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jan 10, 2022
@k8s-triage-robot
Copy link

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 10, 2022
@saschagrunert
Copy link
Member Author

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 11, 2022
@k8s-triage-robot
Copy link

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 10, 2022
@palnabarun
Copy link
Member

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 14, 2022
@evankanderson
Copy link

I'm not sure what statistics you get from your artifact registry, but here are a few thoughts on the usage side

  • gcr.io VS k8s.io registry during the transition
  • count by client used / count of unique downloads by IP or /24
  • it would be really interesting to know how many are upgrading for each patch / minor vs skipping (this could feed into release cycle discussions)

For within-release management:

  • number of KEPs for release at each cycle checkpoint
    • where do KEPs drop out of the cycle? Should this move left or right?
  • number of days with "green" CI
  • number of bug fixes / regressions detected VS fixed during the cycle
  • (post release) number of bug fixes and which patch they were fixed in

@jberkus
Copy link
Contributor

jberkus commented Jul 25, 2022

  1. Number of Enhancements dropped from the release, with a reason category for why they were dropped. Defer to enhancements team to come up with 2-6 categories for dropping enhancements (e.g. feature cancelled permanently, author incommunicado, etc.)
  2. Number/percent of release team shadows who completed the release work (i.e. didn't drop out).
  3. Number of issues reported based on alpha and beta releases, by day (i.e. is anyone actually testing these?)

@parispittman
Copy link

+1 to calls for metrics on # of regressions, kep status’ (current and future releases; eg how many KEPs not GA yet, how many went to GA, etc), and team shadow metrics

@saschagrunert
Copy link
Member Author

The new project board for enhancements would help us to gather the metrics in a more automated way. I think we should give the board more time to evolve while keeping the necessity for those metrics in mind.

cc @leonardpahlke

Referring discussion: https://kubernetes.slack.com/archives/C2C40FMNF/p1662537798532949?thread_ts=1662477434.202879&cid=C2C40FMNF

@k8s-triage-robot
Copy link

The Kubernetes project currently lacks enough contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/feature Categorizes issue or PR as related to a new feature. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. sig/release Categorizes an issue or PR as relevant to SIG Release.
Projects
None yet
Development

No branches or pull requests

10 participants