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

[Tracking] Consolidate and automate creation of sig-release owned testgrid dashboards #11977

Closed
1 of 4 tasks
mariantalla opened this issue Mar 28, 2019 · 14 comments
Closed
1 of 4 tasks
Assignees
Labels
area/config Issues or PRs related to code in /config area/testgrid kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. sig/release Categorizes an issue or PR as relevant to SIG Release.
Milestone

Comments

@mariantalla
Copy link
Contributor

mariantalla commented Mar 28, 2019

(Following up from the last sig-release catchup (notes))

Context: At the end of the 1.13 release, we rearranged sig-release owned dashboards into blocking and informing, based on criteria discussed in kubernetes/sig-release#347.

What should be cleaned up or changed:

Related/Complementary, but out of scope for this work:

  • Upgrade tests should move to use a community supported way of upgrading clusters

cc @alejandrox1 @spiffxp @claurence @neolit123 @tpepper @calebamiles @jberkus
/sig release

@mariantalla mariantalla added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label Mar 28, 2019
@k8s-ci-robot k8s-ci-robot added the sig/release Categorizes an issue or PR as relevant to SIG Release. label Mar 28, 2019
@mariantalla
Copy link
Contributor Author

mariantalla commented Mar 28, 2019

Re: consolidating master- dashboards, we spoke about moving them to -informing rather than -blocking, because they use /cluster (=non-community-supporting) scripts to upgrade clusters. For 1.14, CI signal treated them as equally blocking as master-blocking. They're also currently the only way we have to get signal on how stable upgrades are, so I feel there is a good argument for them to move into blocking too.

@alejandrox1 @claurence thoughts?

@spiffxp
Copy link
Member

spiffxp commented May 6, 2019

/assign @spiffxp @mariantalla
I would like to help push this with maria (whom I've assigned because she's started pushing this in #12441)

/assign @alejandrox1
CI Signal for v1.15

FYI @claurence as 1.15 release lead

FYI @Katharine given you are trying to help simply forking of the jobs that end up on these dashboards (ref: #12443)

@spiffxp
Copy link
Member

spiffxp commented May 6, 2019

/milestone v1.15

@k8s-ci-robot k8s-ci-robot added this to the v1.15 milestone May 6, 2019
@spiffxp
Copy link
Member

spiffxp commented May 7, 2019

Create tooling to automate creation of - dashboards and their contents (e.g. prow jobs), using master- dashboards as input (add issue/PR here)

/assign @Katharine
Now that I'm reading a little more closely, this is what #12443 is for, so assigning to make sure we agree this is done

@spiffxp
Copy link
Member

spiffxp commented May 10, 2019

I am, FWIW, trying to link anything related to "clean up release dashboards" or "clean up release jobs" or "make these things more consistent" etc, because this seems like the most relevant umbrella

@spiffxp
Copy link
Member

spiffxp commented May 13, 2019

/area config
/area testgrid

@k8s-ci-robot k8s-ci-robot added area/config Issues or PRs related to code in /config area/testgrid labels May 13, 2019
@spiffxp
Copy link
Member

spiffxp commented May 13, 2019

#11977 is adding -informing dashboards for all branches

@spiffxp
Copy link
Member

spiffxp commented May 14, 2019

We are now in an intermediary state I would like to get out of:

  • release-x.y-blocking
  • release-x.y-informing
  • release-x.y-all

I want us to get rid of -all

@spiffxp
Copy link
Member

spiffxp commented Jul 9, 2019

/milestone v1.16

@k8s-ci-robot k8s-ci-robot modified the milestones: v1.15, v1.16 Jul 9, 2019
@spiffxp
Copy link
Member

spiffxp commented Aug 21, 2019

ref: #13883

@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-testing, kubernetes/test-infra and/or fejta.
/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 Nov 19, 2019
@justaugustus
Copy link
Member

Hopping on this since I've done recent-ish work to spruce these boards up:

What remains for us to do to close this out?

/assign
/remove-lifecycle stale
/lifecycle frozen

@k8s-ci-robot k8s-ci-robot added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Dec 9, 2019
@spiffxp
Copy link
Member

spiffxp commented Apr 10, 2020

/close
I can't think of anything off the top of my head so I think we should close. If there's something specific that comes up, use followup issues.

@k8s-ci-robot
Copy link
Contributor

@spiffxp: Closing this issue.

In response to this:

/close
I can't think of anything off the top of my head so I think we should close. If there's something specific that comes up, use followup issues.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/config Issues or PRs related to code in /config area/testgrid kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. sig/release Categorizes an issue or PR as relevant to SIG Release.
Projects
None yet
Development

No branches or pull requests

7 participants