-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
Expand debugging services tips #362
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
3df3d19
to
faa8225
Compare
* If you meant it to be a named port, do your `Pod`s expose a port | ||
with the same name? | ||
* Is the port's `protocol` the same as the `Pod`'s? | ||
* Does the service has a `selector` and is correctly written? |
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.
- Does the service have a selector and is it spelled correctly?
cc @kubernetes/sig-network |
I signed it! as Advanced Programming Solutions SL (APSL) |
faa8225
to
eea8f1d
Compare
eea8f1d
to
07ed66e
Compare
@johndmulhausen I added and example of both questions. Is there anything else to improve? |
@paurullan Can you address the issues related to DNS and alpine linux? I'll be happy to review this after that's done. Overall this look pretty good. |
@paurullan Friendly bump. |
@devin-donnelly Would be good to either close this, or merge it and fix the issues as a follow up. Would you be able to drive this? |
@pwittrock @kelseyhightower sorry for the delay, I had a disconnection for the whole August. |
Update: Adding TLS link to mTLS term
During my first tests I got a lot of trouble for having non-kubernetes compatible Alpine linux images. Also, one time I incorrectly wrote the service selectors.
This PR is just an expansion for those papercuts.