-
Notifications
You must be signed in to change notification settings - Fork 54
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
docs: add simple example for network field #550
docs: add simple example for network field #550
Conversation
Hi @googs1025. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
✅ Deploy Preview for kubernetes-sigs-jobset ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I'm not sure if we need to add documentation demonstrating how to ping between two pods. root@VM-0-4-ubuntu:/home/ubuntu# kubectl get pods
NAME READY STATUS RESTARTS AGE
failure-policy-workers-0-0-c59sq 0/1 Completed 0 20h
failure-policy-workers-0-1-98htc 0/1 Completed 0 20h
network-jobset-leader-0-0-5dmv7 1/1 Running 0 20m
network-jobset-workers-0-0-g5n2t 1/1 Running 0 20m
network-jobset-workers-0-1-rghv6 1/1 Running 0 20m
paralleljobs-driver-0-0-dw5mh 0/1 Completed 0 20h
paralleljobs-workers-0-0-mfwtk 0/1 Completed 0 20h
paralleljobs-workers-0-1-5g6gp 0/1 Completed 0 20h
paralleljobs-workers-0-2-7n9j5 0/1 Completed 0 20h
paralleljobs-workers-0-3-vp8jc 0/1 Completed 0 20h
success-policy-leader-0-0-8rwff 0/1 Completed 0 45h
success-policy-workers-0-0-c58gl 0/1 Completed 0 45h
success-policy-workers-0-1-dhfq9 0/1 Completed 0 45h
root@VM-0-4-ubuntu:/home/ubuntu# kubectl exec -it network-jobset-leader-0-0-5dmv7 -- sh
/ # cat /etc/hosts
# Kubernetes-managed hosts file.
127.0.0.1 localhost
::1 localhost ip6-localhost ip6-loopback
fe00::0 ip6-localnet
fe00::0 ip6-mcastprefix
fe00::1 ip6-allnodes
fe00::2 ip6-allrouters
10.6.2.23 network-jobset-leader-0-0.network-jobset.default.svc.cluster.local network-jobset-leader-0-0
/ # ping network-jobset-workers-0-1.network-jobset.default.svc.cluster.local
PING network-jobset-workers-0-1.network-jobset.default.svc.cluster.local (10.6.2.24): 56 data bytes
64 bytes from 10.6.2.24: seq=0 ttl=63 time=0.069 ms
64 bytes from 10.6.2.24: seq=1 ttl=63 time=0.062 ms
64 bytes from 10.6.2.24: seq=2 ttl=63 time=0.066 ms
^C
--- network-jobset-workers-0-1.network-jobset.default.svc.cluster.local ping statistics ---
3 packets transmitted, 3 packets received, 0% packet loss
round-trip min/avg/max = 0.062/0.065/0.069 ms
|
/ok-to-test |
007edaa
to
930bcea
Compare
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.
left one comment
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.
Hmm I'm unsure if we want this network policy example in it's own folder, rather than in examples/simple. I guess you did this since you wanted to add a README explaining some extra steps the user can take to verify the pod-to-pod communication via pod hostnames is working correctly.
@kannon92 what do you think about this? The readme is a nice touch but I don't want the examples folder to get disorganized. Maybe we can think about each example having a short readme to accompany it like this?
Yes, I separated them because I wanted to have a README.md document to describe . |
@danielvegamyhre @kannon92 After thinking for a while, I've decided to put it back into the examples/simple directory. How about this? I will move the README.md file to the Troubleshooting section here because network connectivity issues are something that users should be aware of. If users are unsure, they can refer to the Troubleshooting section for guidance. |
854317c
to
32ded05
Compare
913a140
to
f266536
Compare
Signed-off-by: googs1025 <googs1025@gmail.com>
Signed-off-by: googs1025 <googs1025@gmail.com>
@danielvegamyhre /PTAL Any other suggestions or modifications? thanks! |
b629c63
to
ed48512
Compare
/lgtm Thanks @googs1025! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: danielvegamyhre, googs1025 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 |
I would like to add a usage example of the network field in simple/example and demonstrate how to access each other between two pods. This will allow users to better understand the functions provided by the jobset itself.