Skip to content

[SRVLOGIC-443] Document Kubernetes service discovery#109172

Open
ochromy wants to merge 1 commit intoopenshift:serverless-docs-mainfrom
ochromy:SRVLOGIC-443f
Open

[SRVLOGIC-443] Document Kubernetes service discovery#109172
ochromy wants to merge 1 commit intoopenshift:serverless-docs-mainfrom
ochromy:SRVLOGIC-443f

Conversation

@ochromy
Copy link
Copy Markdown

@ochromy ochromy commented Mar 30, 2026

Version(s):

  • serverless-docs-1.36
  • serverless-docs-1.37
  • serverless-docs-1.38

Issue:

Link to docs preview:

QE review:

@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Mar 30, 2026
@ocpdocs-previewbot
Copy link
Copy Markdown

ocpdocs-previewbot commented Mar 30, 2026

Copy link
Copy Markdown

@kaldesai kaldesai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work :)
Just a tiny nitpicks you can consider, otherwise LGTM

Peer review is complete

improvements no1

improvements no2

improvements no3

improvements no4

improvements no5

improvements no6

improvements no7

improvements no8

[SRVLOGIC-442] Document Custom podSpec definition.

improvements no1

improvements no2

improvements no3

improvements no4

improvements no5

improvements no6

improvements no7

improvements no8

peer review no1

resolve conflict with 442

add include module

peer review feedback no1
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 30, 2026

@ochromy: all tests passed!

Full PR test history. Your PR dashboard.

Details

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-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants