Update ingress-operator.adoc

[rameshsahoo111]

Added a solution article link to this document for the original client IP details.

Version(s):

Issue:

Link to docs preview: https://84979--ocpdocs-pr.netlify.app/openshift-enterprise/latest/networking/ingress-operator.html#additional-resources

QE review:

• QE has approved this change.

Additional information:

[openshift-ci]

Hi @rameshsahoo111. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

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-sigs/prow repository.

[jldohmann]

thanks for opening! small edit to the existing link. when you've made the change, feel free to ping me

[jldohmann]

the current link doesn't work, but this does:

Suggested change

	* link:https://access.redhat.com/solutions/7096271k[Viewing the original client source IP through the X-Forwarded-For header in OpenShift Ingress access log]
	* link:https://access.redhat.com/solutions/7096271[Viewing the original client source IP through the X-Forwarded-For header in OpenShift Ingress access log]

[jldohmann]

/ok-to-test

[ocpdocs-previewbot]

🤖 Tue Nov 19 17:09:35 - Prow CI generated the docs preview:

https://84979--ocpdocs-pr.netlify.app/openshift-dedicated/latest/networking/ingress-operator.html
https://84979--ocpdocs-pr.netlify.app/openshift-enterprise/latest/networking/ingress-operator.html
https://84979--ocpdocs-pr.netlify.app/openshift-rosa/latest/networking/ingress-operator.html

[jldohmann]

Suggested change

	* link:https://access.redhat.com/solutions/7096271k[Viewing the original client source IP through the X-Forwarded-For header in OpenShift Ingress access log]
	* link:https://access.redhat.com/solutions/7096271k[Viewing the original client source IP through the X-Forwarded-For header in {product-title} Ingress access log]

[openshift-ci]

@rameshsahoo111: all tests passed!

Full PR test history. Your PR dashboard.

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.

[jldohmann]

@melvinjoseph86 can you PTAL at this docs PR for QE ack? thank you

[jldohmann]

@rameshsahoo111 what version(s) of OCP do you mean for this change to apply to?

[melvinjoseph86]

/label qe-approved

[melvinjoseph86]

Working links is 'https://access.redhat.com/solutions/7096271'

[rameshsahoo111]

@rameshsahoo111 what version(s) of OCP do you mean for this change to apply to?

Please apply this to all OCP 4.x version.

[rameshsahoo111]

@rameshsahoo111 what version(s) of OCP do you mean for this change to apply to?

Please apply this to all OCP 4.x version.

I would also request changing the heading as below;

FROM:
Viewing the original client source IP through the X-Forwarded-For header in OpenShift Ingress access log

TO:
Viewing the original client IP through the X-Forwarded-For header in OpenShift Ingress access logs or within applications logs deployed on OpenShift

[jldohmann]

@rameshsahoo111 that sounds fine, just remember to use the attribute {product-title} in place of OpenShift when making the change 😄

[rameshsahoo111]

{product-title}

yes, sure. Thank you.

[openshift-merge-robot]

PR needs rebase.

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.

[rameshsahoo111]

New pull request available at #86578

[bergerhoffer]

The branch/enterprise-4.19 label has been added to this PR.

This is because your PR targets the main branch and is labeled for enterprise-4.18. And any PR going into main must also target the latest version branch (enterprise-4.19).

If the update in your PR does NOT apply to version 4.19 onward, please re-target this PR to go directly into the appropriate version branch or branches (enterprise-4.x) instead of main.

[rameshsahoo111]

Thank you, please merge the pr

[bergerhoffer]

@rameshsahoo111 There is a conflict now, so this PR needs to be rebased (and then squashed to 1 commit).

Or if you can't do that, you can always open it as a new PR. Please ping @jldohmann for help (my comment was just a standard boilerplate for cutting a new version).