enhancements icon indicating copy to clipboard operation
enhancements copied to clipboard

Transition from SPDY to WebSockets

Open seans3 opened this issue 2 years ago • 58 comments

Enhancement Description

  • One-line enhancement description (can be used as a release note): Transition the bi-directional streaming protocol of Kubernetes clients from SPDY/3.1 to WebSockets.

  • Kubernetes Enhancement Proposal: https://github.com/kubernetes/enhancements/tree/master/keps/sig-api-machinery/4006-transition-spdy-to-websockets

  • Discussion Link:

  • Primary contact (assignee): @seans3

  • Responsible SIGs: SIG API Machinery, SIG CLI

  • Enhancement target (which target equals to which milestone):

    • Alpha release target (x.y): 1.29
    • Beta release target (x.y): 1.31
    • Stable release target (x.y): 1.32
  • [x] Alpha

    • [x] KEP (k/enhancements) update PR(s):
      • https://github.com/kubernetes/enhancements/pull/4016
      • https://github.com/kubernetes/enhancements/pull/4169
    • [x] Code (k/k) update PR(s):
      • https://github.com/kubernetes/kubernetes/pull/119157
      • https://github.com/kubernetes/kubernetes/pull/119186
      • https://github.com/kubernetes/kubernetes/pull/123413
    • [x] Docs (k/website) update PR(s):
      • https://github.com/kubernetes/website/pull/43620
  • [x] Beta

    • [x] KEP (k/enhancements) update PR(s):
      • https://github.com/kubernetes/enhancements/pull/4416
      • https://github.com/kubernetes/enhancements/pull/4666
    • [x] Code (k/k) update PR(s):
      • https://github.com/kubernetes/kubernetes/pull/123281
      • https://github.com/kubernetes/kubernetes/pull/125528
    • [x] Docs (k/website) update(s):
      • https://github.com/kubernetes/website/pull/45270
      • https://github.com/kubernetes/website/pull/46955

Please keep this description up to date. This will help the Enhancement Team to track the evolution of the enhancement efficiently.

seans3 avatar May 15 '23 19:05 seans3

/sig api-machinery

seans3 avatar May 15 '23 19:05 seans3

/sig cli /stage alpha

seans3 avatar May 15 '23 19:05 seans3

/milestone v1.28 /label lead-opted-in

deads2k avatar Jun 02 '23 14:06 deads2k

Hello @seans3 👋, Enhancements team here.

Just checking in as we approach enhancements freeze on 01:00 UTC Friday, 16th June 2023.

This enhancement is targeting for stage alpha for 1.28 (correct me, if otherwise)

Here's where this enhancement currently stands:

  • [X] KEP readme using the latest template has been merged into the k/enhancements repo.
  • [X] KEP status is marked as implementable for latest-milestone: 1.28
  • [X] KEP readme has a updated detailed test plan section filled out
  • [X] KEP readme has up to date graduation criteria
  • [ ] KEP has a production readiness review that has been completed and merged into k/enhancements.

For this KEP, we would just need to update the following:

  • Ensure that the PRs are merged.

The status of this enhancement is marked as at risk. Please keep the issue description up-to-date with appropriate stages as well. Thank you!

salehsedghpour avatar Jun 03 '23 22:06 salehsedghpour

@salehsedghpour

Hello. Since the PRR reviewer @deads2k approved the KEP and since the KEP has been merged, I believe this KEP is now ready for v1.28. Please let me know if there are any other tasks needed for this release.

Thanks.

seans3 avatar Jun 09 '23 00:06 seans3

Hello @seans3! 1.28 Docs Shadow here.

Does this enhancement work planned for 1.28 require any new docs or modification to existing docs?

If so, please follows the steps here to open a PR against dev-1.28 branch in the k/website repo. This PR can be just a placeholder at this time and must be created before Thursday 20th July 2023.

Also, take a look at Documenting for a release to get yourself familiarize with the docs requirement for the release.

Thank you!

katcosgrove avatar Jun 21 '23 13:06 katcosgrove

Hey @seans3 , could you please create a docs PR even if it is a draft PR with no content yet against dev-1.28 branch in the k/website repo. The deadline to create this draft PR is Thursday 20th July 2023.

Rishit-dagli avatar Jul 12 '23 15:07 Rishit-dagli

Hey again @seans3 :wave: Just checking in as we approach Code freeze at 01:00 UTC Friday, 19th July 2023 .

I don't see any code (k/k) update PR(s) in the issue description so if there are any k/k related PR(s) that we should be tracking for this KEP please link them in the issue description above.

As always, we are here to help if any questions come up. Thanks!

salehsedghpour avatar Jul 17 '23 20:07 salehsedghpour

Hello @seans3 👋, 1.28 Enhancements Lead here.

Unfortunately, the implementation (code related) PRs associated with this enhancement are not in the merge-ready state by code-freeze and hence this enhancement is now removed from the v1.28 milestone.

If you still wish to progress this enhancement in v1.28, please file an exception request. Thanks!

/milestone clear

Atharva-Shinde avatar Jul 19 '23 04:07 Atharva-Shinde

Hello @seans3 👋, 1.29 Enhancements team here!

Just checking in as we approach enhancements freeze on 01:00 UTC, Friday, 6th October, 2023.

This enhancement is targeting for stage alpha for 1.29 (correct me, if otherwise)

Here's where this enhancement currently stands:

  • [x] KEP readme using the latest template has been merged into the k/enhancements repo.
  • [ ] KEP status is marked as implementable for latest-milestone: 1.29. KEPs targeting stable will need to be marked as implemented after code PRs are merged and the feature gates are removed.
  • [x] KEP readme has up-to-date graduation criteria
  • [x] KEP has a production readiness review that has been completed and merged into k/enhancements. (For more information on the PRR process, check here).

For this KEP, please update the latest-milestone to v1.29 and the alpha milestone to 1.29 in the kep.yaml.

The status of this enhancement is marked as at risk for enhancement freeze. Please keep the issue description up-to-date with appropriate stages as well.

Thank you!

sanchita-07 avatar Sep 29 '23 04:09 sanchita-07

  • KEP status is marked as implementable for latest-milestone: 1.29. KEPs targeting stable will need to be marked as implemented after code PRs are merged and the feature gates are removed.

that doesn't really make sense... the KEP is still in alpha and is remaining implementable for 1.29 in alpha, not completing

liggitt avatar Oct 02 '23 20:10 liggitt

/label lead-opted-in (for 1.29)

jpbetz avatar Oct 02 '23 20:10 jpbetz

that doesn't really make sense... the KEP is still in alpha and is remaining implementable for 1.29 in alpha, not completing

Apologies for any confusion, @liggitt. The box was left unchecked because the latest-milestone was set for v1.28. We need to mark the status implementable for v1.29 which I think has been addressed by the PR #4169.

Just to clarify, when the KEP is targeting for stable and the code PRs are merged as well as the feature gates are removed, then we mark the status as implemented.

I hope this clarifies any uncertainties.

sanchita-07 avatar Oct 04 '23 03:10 sanchita-07

Hi @seans3 , checking in once more as we approach the 1.29 enhancement freeze deadline on 01:00 UTC, Friday, 6th October, 2023. The status of this enhancement is marked as at risk for enhancement freeze. It looks like #4169 will address the requirements. Let me know if I missed anything. Thanks!

sanchita-07 avatar Oct 04 '23 07:10 sanchita-07

@sanchita-07 #4169 is merged, can you confirm this is tracked for 1.29?

liggitt avatar Oct 05 '23 19:10 liggitt

With KEP PR https://github.com/kubernetes/enhancements/pull/4169 approved, the enhancement is ready for the enhancements freeze. The status is now marked as tracked for enhancement freeze for 1.29. 🚀 Thank you!

npolshakova avatar Oct 06 '23 00:10 npolshakova

Hey there @seans3 👋, v1.29 Docs team shadow here. Does this enhancement work planned for v1.29 require any new docs or modifications to existing docs? If so, please follow the steps here to open a PR against dev-1.29 branch in the k/website repo. This PR can be just a placeholder at this time and must be created before Thursday, 19 October 2023. Also, take a look at Documenting for a release to get yourself familiarized with the docs requirement for the release. Thank you!

taniaduggal avatar Oct 11 '23 04:10 taniaduggal

Hey there @seans3 👋, The deadline to open a placeholder PR against k/website for required documentation is Thursday, 19 October. Could you please update me on the status of docs for this enhancement? Thank you!

taniaduggal avatar Oct 18 '23 15:10 taniaduggal

Hey there Sean Sullivan 👋, The deadline to open a placeholder PR against k/website for required documentation is Thursday, 19 October. Could you please update me on the status of docs for this enhancement? Thank you!

Hello. Sorry for the delay. The placeholder PR for the docs for this enhancement is at: https://github.com/kubernetes/website/pull/43620

Please let me know if there is anything else required. Thanks.

seans3 avatar Oct 20 '23 23:10 seans3

Hey again @seans3 👋, 1.29 Enhancements team here.

Just checking in as we approach code freeze at 01:00 UTC Wednesday 1st November 2023:

Here's where this enhancement currently stands:

  • [ ] All PRs to the Kubernetes repo that are related to your enhancement are linked in the above issue description (for tracking purposes).
  • [ ] All PR/s are ready to be merged (they have approved and lgtm labels applied) by the code freeze deadline. This includes tests.

For this enhancement, it looks like the following PRs are open and need to be merged before code freeze:

Are there additional code related PRs that need to be merged for 1.29? It looks like https://github.com/kubernetes/kubernetes/pull/120889 is mentioned and needs to be included in the issue description.

With all this, the status of this KEP is at risk for code freeze.

Also, please let me know if there are other PRs in k/k we should be tracking for this KEP. As always, we are here to help if any questions come up. Thanks!

sanchita-07 avatar Oct 24 '23 03:10 sanchita-07

@sanchita-07, #120889 is not targeting 1.29, only #119186 is (and that is now in the merge queue)

liggitt avatar Oct 24 '23 13:10 liggitt

Thanks @liggitt for mentioning :)

With all the PRs related to the enhancement are linked in the above issue description and :

  • https://github.com/kubernetes/kubernetes/pull/119157 merged
  • https://github.com/kubernetes/kubernetes/pull/119186 having approved and lgtm label

This enhancement is now Tracked for Code Freeze. Thanks!

sanchita-07 avatar Oct 24 '23 14:10 sanchita-07

Hi @seans3, 👋 from the v1.29 Release Team-Communications! We would like to check if you have any plans to publish a blog for this KEP regarding new features, removals, and deprecations for this release.

If so, you need to open a PR placeholder in the website repository. The deadline will be on Tuesday 14th November 2023 (after the Docs deadline PR ready for review)

Here is the 1.29 calendar

a-mccarthy avatar Oct 30 '23 18:10 a-mccarthy

/remove-label lead-opted-in

salehsedghpour avatar Jan 06 '24 16:01 salehsedghpour

Hello 👋 1.30 Enhancements Lead here,

I'm closing milestone 1.29 now, If you wish to progress this enhancement in v1.30, please follow the instructions here to opt in the enhancement and make sure the lead-opted-in label is set so it can get added to the tracking board and finally add /milestone v1.30. Thanks!

/milestone clear

salehsedghpour avatar Jan 16 '24 23:01 salehsedghpour

/stage beta

seans3 avatar Jan 19 '24 06:01 seans3

/label lead-opted-in /milestone v1.30

jpbetz avatar Jan 19 '24 14:01 jpbetz

@jpbetz: You must be a member of the kubernetes/milestone-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your Milestone Maintainers Team and have them propose you as an additional delegate for this responsibility.

In response to this:

/label lead-opted-in /milestone v1.30

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 Jan 19 '24 14:01 k8s-ci-robot

/milestone v1.30

seans3 avatar Jan 19 '24 22:01 seans3

@seans3: You must be a member of the kubernetes/milestone-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your Milestone Maintainers Team and have them propose you as an additional delegate for this responsibility.

In response to this:

/milestone v1.30

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 Jan 19 '24 22:01 k8s-ci-robot