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

Replace removed tolerate-unready-endpoints annotation with publishNotReadyAddresses #1066

Merged

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jul 10, 2023

Motivation

StreamNative Pulsar helm chart isn't compatible with k8s 1.24+ since there isn't a way to configure Zookeeper service with publishNotReadyAddresses: true.

Modifications

  • set publishNotReadyAddresses: true for zookeeper services
  • remove traces of removed service.alpha.kubernetes.io/tolerate-unready-endpoints annotation

Documentation

  • no-need-doc

…notation that got removed in k8s 1.24

- service.alpha.kubernetes.io/tolerate-unready-endpoints was removed in k8s 1.24
  with commit kubernetes/kubernetes@ae9f117
@lhotari lhotari requested review from a team, sijie and addisonj as code owners July 10, 2023 13:33
@github-actions github-actions bot added the no-need-doc This pr does not need any document label Jul 10, 2023
@ericsyh ericsyh requested a review from a team July 21, 2023 17:41
@ericsyh ericsyh added this to the sn-platform-1.11 milestone Jul 25, 2023
@ericsyh ericsyh requested a review from a team August 4, 2023 06:19
@ericsyh ericsyh merged commit 9e94e40 into streamnative:master Aug 9, 2023
ericsyh pushed a commit that referenced this pull request Aug 24, 2023
…ReadyAddresses (#1066)

Use publishNotReadyAddresses instead of tolerate-unready-endpoints annotation that got removed in k8s 1.24

- service.alpha.kubernetes.io/tolerate-unready-endpoints was removed in k8s 1.24
  with commit kubernetes/kubernetes@ae9f117

Co-authored-by: Eric Shen <[email protected]>
(cherry picked from commit 9e94e40)
ericsyh pushed a commit that referenced this pull request Aug 24, 2023
…ReadyAddresses (#1066)

Use publishNotReadyAddresses instead of tolerate-unready-endpoints annotation that got removed in k8s 1.24

- service.alpha.kubernetes.io/tolerate-unready-endpoints was removed in k8s 1.24
  with commit kubernetes/kubernetes@ae9f117

Co-authored-by: Eric Shen <[email protected]>
(cherry picked from commit 9e94e40)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-need-doc This pr does not need any document release/1.10.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants