enhancements icon indicating copy to clipboard operation
enhancements copied to clipboard

changes KEP-2644 beta target to 1.30

Open carlory opened this issue 1 year ago • 5 comments

  • One-line PR description: changes KEP-2644 beta target to 1.30
  • Issue link: #2644
  • Other comments:

carlory avatar Jan 09 '24 08:01 carlory

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: carlory Once this PR has been reviewed and has the lgtm label, please assign xing-yang for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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 Jan 09 '24 08:01 k8s-ci-robot

/cc @deepakkinni

carlory avatar Jan 09 '24 08:01 carlory

@carlory We can't move this to beta yet, we need additional tests to be written for that.

deepakkinni avatar Jan 23 '24 21:01 deepakkinni

@deepakkinni thanks for your review. https://github.com/kubernetes/kubernetes/pull/123151 will add e2e test.

carlory avatar Feb 06 '24 11:02 carlory

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 06 '24 11:05 k8s-triage-robot

/remove-lifecycle stale

carlory avatar May 13 '24 06:05 carlory

/hold

carlory avatar May 13 '24 06:05 carlory

@carlory Why do you add the "hold" to this PR? Are you waiting for anything else?

xing-yang avatar May 13 '24 14:05 xing-yang

This KEP can be merged now since the e2e tests PR has already been merged.

xing-yang avatar May 13 '24 14:05 xing-yang

/lgtm /approve

xing-yang avatar May 13 '24 14:05 xing-yang

/lgtm /approve

deepakkinni avatar May 13 '24 17:05 deepakkinni

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: carlory, deepakkinni, xing-yang

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 May 13 '24 17:05 k8s-ci-robot

@carlory Why do you add the "hold" to this PR? Are you waiting for anything else?

@xing-yang It needs to be rewritten with the latest template.

- [Design Details](#design-details)
  - [Test Plan](#test-plan)
    - [Prerequisite testing updates](#prerequisite-testing-updates)
    - [Unit tests](#unit-tests)
    - [Integration tests](#integration-tests)

- [Production Readiness Review Questionnaire](#production-readiness-review-questionnaire)
  - [Rollout, Upgrade and Rollback Planning](#rollout-upgrade-and-rollback-planning)
  - [Dependencies](#dependencies)
  - [Troubleshooting](#troubleshooting)

the above chapters do not exist. Do we need to add these to this PR?

carlory avatar May 14 '24 08:05 carlory

/hold cancel

carlory avatar May 14 '24 11:05 carlory

/hold

carlory avatar May 14 '24 11:05 carlory

/hold cancel

carlory avatar May 14 '24 11:05 carlory