release icon indicating copy to clipboard operation
release copied to clipboard
verified publisher icon kubernetes

Add new tool: patch release cherry pick notification

Open cpanato opened this issue 4 years ago • 25 comments

What type of PR is this?

/kind feature

What this PR does / why we need it:

This is a tool that will run in a prow job and uses the https://raw.githubusercontent.com/kubernetes/website/main/data/releases/schedule.yaml published with the patch releases and will notify when the cherry pick deadline is close by sending an email to the k-dev mailing list.

Opened as WIP to collect feedback and see what we can improve

After approval and merge, will add the prow job to run the tool

/assign @saschagrunert @puerco @xmudrii @Verolop @justaugustus cc @kubernetes/release-engineering

Which issue(s) this PR fixes:

Fixes partially https://github.com/kubernetes/release/issues/2174

Special notes for your reviewer:

Does this PR introduce a user-facing change?

Add new tool: patch release cherry pick notification

cpanato avatar Feb 16 '22 15:02 cpanato

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cpanato

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Feb 16 '22 15:02 k8s-ci-robot

/assign

puerco avatar May 10 '22 14:05 puerco

Feel free to reassign me when this is ready! /unassign

justaugustus avatar Jul 23 '22 14:07 justaugustus

sample email sent

Screenshot 2022-09-02 at 13 43 20

cpanato avatar Sep 02 '22 11:09 cpanato

will setup the job in a mock stage for now

cc @saschagrunert

cpanato avatar Sep 02 '22 11:09 cpanato

@cpanato: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-release-image-distroless-iptables 06d31379c063aa201c9537a123379e81cfa89e4c link true /test pull-release-image-distroless-iptables

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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. I understand the commands that are listed here.

k8s-ci-robot avatar Sep 02 '22 12:09 k8s-ci-robot

job: https://github.com/kubernetes/test-infra/pull/27489

cpanato avatar Sep 13 '22 12:09 cpanato

for now it will send the email to the test announcement mailing list to validate everything

cpanato avatar Sep 13 '22 12:09 cpanato

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-triage-robot avatar Dec 12 '22 13:12 k8s-triage-robot

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-triage-robot avatar Jan 11 '23 14:01 k8s-triage-robot

when you all have time can please take a look at this? pretty please 😇

cc @saschagrunert @xmudrii @Verolop @palnabarun

cpanato avatar Jan 17 '23 13:01 cpanato

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and 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:

  • Reopen this PR with /reopen
  • Mark this 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-triage-robot avatar Feb 17 '23 09:02 k8s-triage-robot

@k8s-triage-robot: Closed this PR.

In response to this:

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and 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:

  • Reopen this PR with /reopen
  • Mark this 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.

k8s-ci-robot avatar Feb 17 '23 09:02 k8s-ci-robot

/remove-lifecycle rotten

cpanato avatar Feb 17 '23 10:02 cpanato

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-triage-robot avatar May 18 '23 10:05 k8s-triage-robot

PR needs rebase.

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.

k8s-ci-robot avatar May 19 '23 17:05 k8s-ci-robot

The Kubernetes project currently lacks enough active 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 rotten
  • Close this PR with /close
  • Offer to help out with Issue Triage

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

/lifecycle rotten

k8s-triage-robot avatar Jun 18 '23 17:06 k8s-triage-robot

/remove-lifecycle rotten

cpanato avatar Jun 19 '23 06:06 cpanato

@cpanato – should we resurrect this PR?

justaugustus avatar Nov 18 '23 18:11 justaugustus

@justaugustus yes!! i know we are discussing which email service to use, so I would like to align that. do you have any updates @jimangel @ameukam? what is the email service?

then I can update this PR and we can move forward

cpanato avatar Nov 19 '23 11:11 cpanato

what is the email service?

The solution I proposed is for manual sending (releng folks cutting releases). This seems like it's attempting to be a more automated solution? If so, we'd need to evaluate a different bulk-sending service like @ameukam mentioned.

TL;DR: I don't think there's a solution yet for automated emails.

We also might be able to modify my solution to use an offline / refresh token, but then we'd have to figure out which account we'd want to authorize to allow "sending email on behalf."

jimangel avatar Nov 19 '23 14:11 jimangel

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-triage-robot avatar Feb 17 '24 14:02 k8s-triage-robot

/remove-lifecycle remove

cpanato avatar Feb 17 '24 15:02 cpanato

The Kubernetes project currently lacks enough active 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 rotten
  • Close this PR with /close
  • Offer to help out with Issue Triage

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

/lifecycle rotten

k8s-triage-robot avatar Mar 18 '24 15:03 k8s-triage-robot

/remove-lifecycle rotten

working on that

cpanato avatar Mar 19 '24 22:03 cpanato

closing in favor of https://github.com/kubernetes/release/pull/3627

cpanato avatar May 30 '24 08:05 cpanato