-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
updating cilium network policy docs #29079
updating cilium network policy docs #29079
Conversation
@Ritikaa96: The label(s) In response to this:
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. |
/assign @celestehorgan The latest reference link provide installation information of Cilium with other tools too. Maybe we can also generalise it too here and reference to the link only like we have done here. |
✔️ Deploy Preview for kubernetes-io-main-staging ready! 🔨 Explore the source changes: cee22da 🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/60f97bbdf43f8e0007ca74a7 😎 Browse the preview: https://deploy-preview-29079--kubernetes-io-main-staging.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
LGTM label has been added. Git tree hash: 92ce262370d017639d23f0a7cf536ae73b3d385f
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the suggestion @Ritikaa96
I recommend rewording more of the text, to suggest that this guide assumes you already have a cluster. We could signpost folks towards https://kubernetes.io/docs/tasks/tools/ if they don't have one already.
@@ -24,7 +24,7 @@ For background on Cilium, read the [Introduction to Cilium](https://docs.cilium. | |||
## Deploying Cilium on Minikube for Basic Testing | |||
|
|||
To get familiar with Cilium easily you can follow the | |||
[Cilium Kubernetes Getting Started Guide](https://docs.cilium.io/en/stable/gettingstarted/minikube/) | |||
[Cilium Kubernetes Getting Started Guide](https://docs.cilium.io/en/stable/gettingstarted/k8s-install-default/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit:
[Cilium Kubernetes Getting Started Guide](https://docs.cilium.io/en/stable/gettingstarted/k8s-install-default/) | |
Cilium [Quick Installation](https://docs.cilium.io/en/stable/gettingstarted/k8s-install-default/) guide |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@celestehorgan PTAL |
I'm going to deem @sftim's comment as non-blocking :) /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aanm, celestehorgan 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 |
The Document : https://kubernetes.io/docs/tasks/administer-cluster/network-policy-provider/cilium-network-policy/
has error link for installing cilium on minikube.
fixed it to stable version: https://docs.cilium.io/en/stable/gettingstarted/k8s-install-default/
However, the details are for cilium v1.8 and can be updated.
/kind bug