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.

Sign up for GitHub

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

Jump to bottom

partial and strict are deprecated for helm installation #27060

Closed
2 tasks done
xtineskim opened this issue Jul 25, 2023 · 0 comments · Fixed by #27066 or #27433
Closed
2 tasks done

partial and strict are deprecated for helm installation #27060

xtineskim opened this issue Jul 25, 2023 · 0 comments · Fixed by #27066 or #27433
Assignees
@michi-covalent
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/servicemesh GH issues or PRs regarding servicemesh kind/bug This is a bug in the Cilium logic. kind/community-report This was reported by a user in the Cilium community, eg via Slack.

Comments

@xtineskim
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

What happened?

When trying to install v1.14 rc with helm, the cilium-operator and agents never initialize correctly
partial and strict values for kubeProxyReplacement are deprecated, and the GatewayAPI show that as a prereq

Cilium Version

v1.14 rc and greater

Kernel Version

not relevant

Kubernetes Version

v1.27.3

Sysdump

No response

Relevant log output

No response

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@xtineskim xtineskim added kind/bug This is a bug in the Cilium logic. kind/community-report This was reported by a user in the Cilium community, eg via Slack. needs/triage This issue requires triaging to establish severity and next steps. labels Jul 25, 2023
@michi-covalent michi-covalent self-assigned this Jul 25, 2023
@michi-covalent michi-covalent added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/servicemesh GH issues or PRs regarding servicemesh and removed needs/triage This issue requires triaging to establish severity and next steps. labels Jul 25, 2023
@michi-covalent michi-covalent mentioned this issue Jul 25, 2023
Merged
michi-covalent added a commit that referenced this issue Jul 25, 2023
@michi-covalent
docs: Use kubeProxyReplacement=true for Gateway API docs
e720a34 
- Use `kubeProxyReplacement=true` as `partial` and `strict` settings
  have been deprecated in v1.14.
- Specify the same Helm values for Helm and Cilium-CLI instructions.
- Move the Cilium CLI installation instructions out of the tabs. It's
  required for both Helm and Cilium-CLI instructions.

Fixes: #27060

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
qmonnet pushed a commit that referenced this issue Jul 26, 2023
@michi-covalent @qmonnet
docs: Use kubeProxyReplacement=true for Gateway API docs
4772468 
- Use `kubeProxyReplacement=true` as `partial` and `strict` settings
  have been deprecated in v1.14.
- Specify the same Helm values for Helm and Cilium-CLI instructions.
- Move the Cilium CLI installation instructions out of the tabs. It's
  required for both Helm and Cilium-CLI instructions.

Fixes: #27060

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
nbusseneau pushed a commit that referenced this issue Jul 26, 2023
@michi-covalent @nbusseneau
docs: Use kubeProxyReplacement=true for Gateway API docs
a3d8506 
[ upstream commit 4772468 ]

- Use `kubeProxyReplacement=true` as `partial` and `strict` settings
  have been deprecated in v1.14.
- Specify the same Helm values for Helm and Cilium-CLI instructions.
- Move the Cilium CLI installation instructions out of the tabs. It's
  required for both Helm and Cilium-CLI instructions.

Fixes: #27060

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
joestringer pushed a commit that referenced this issue Jul 26, 2023
@michi-covalent @joestringer
docs: Use kubeProxyReplacement=true for Gateway API docs
176b3db 
[ upstream commit 4772468 ]

- Use `kubeProxyReplacement=true` as `partial` and `strict` settings
  have been deprecated in v1.14.
- Specify the same Helm values for Helm and Cilium-CLI instructions.
- Move the Cilium CLI installation instructions out of the tabs. It's
  required for both Helm and Cilium-CLI instructions.

Fixes: #27060

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
@xtineskim xtineskim mentioned this issue Aug 10, 2023
Merged
ldelossa pushed a commit to ldelossa/cilium that referenced this issue Sep 27, 2023
@michi-covalent
docs: Use kubeProxyReplacement=true for Gateway API docs
6920ef1 
[ upstream commit 4772468 ]

- Use `kubeProxyReplacement=true` as `partial` and `strict` settings
  have been deprecated in v1.14.
- Specify the same Helm values for Helm and Cilium-CLI instructions.
- Move the Cilium CLI installation instructions out of the tabs. It's
  required for both Helm and Cilium-CLI instructions.

Fixes: cilium#27060

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michi-covalent michi-covalent

Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/servicemesh GH issues or PRs regarding servicemesh kind/bug This is a bug in the Cilium logic. kind/community-report This was reported by a user in the Cilium community, eg via Slack.
Projects
None yet
Milestone
No milestone
2 participants
@michi-covalent @xtineskim