-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
Revise concept & reference docs for Services, Ingress & networking #30817
Conversation
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
5ca4285
to
71a71f3
Compare
47a4706
to
bf9ad16
Compare
bf9ad16
to
c0d2a62
Compare
c0d2a62
to
3cdcf4d
Compare
May help with #32013 |
3cdcf4d
to
151c504
Compare
The service glossary entry ({{< glossary_definition term_id="service" length="short" >}}) does not look right. Perhaps replace with: Kubernetes defines a Service as a scalable and resilient method for exposing an application service running on one or more pods in your cluster. Kubernetes employs several techniques to enable a service including per-Pod IPv4/IPv6 addressing, dynamic Pod creation and deletion, Pod replicas and service discovery. |
In service.md file under #services-in-kubernetes, wondering if micro-service term should be included. What about the abstraction? Isn't it just one piece of a service? A text suggestion: In Kubernetes, a Service employs an abstraction which defines a logical set of Pods and a policy by which to access these pods. |
6deaa65
to
93d7531
Compare
93d7531
to
48cb5c2
Compare
Also see #41173 |
/remove-label refactor |
48cb5c2
to
8e92a98
Compare
8e92a98
to
9892771
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: shannonxtreme The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
9892771
to
613d6b8
Compare
All split out PRs have merged or been superseded. /close |
@sftim: Closed this PR. 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. |
Overall PR (changes have been split out).
Individual PRs:
The effect of this PR, overall:
/language en
/label refactor
/sig network
Helps with #14770