Skip to content

OSDOCS-3918: Adds release note for making status.HostIP for pods visible in console#51185

Merged
opayne1 merged 1 commit intoopenshift:enterprise-4.12from
opayne1:OSDOCS-3918
Oct 20, 2022
Merged

OSDOCS-3918: Adds release note for making status.HostIP for pods visible in console#51185
opayne1 merged 1 commit intoopenshift:enterprise-4.12from
opayne1:OSDOCS-3918

Conversation

@opayne1
Copy link
Contributor

@opayne1 opayne1 commented Oct 4, 2022

This PR adds a release note for making status.HostIP for pods visible in console.

Version(s):
4.12 only

Issue:
OSDOCS-3918

Link to docs preview:
https://51185--docspreview.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-12-release-notes.html#ocp-4-12-developer-perspective

QE review:

  • QE has approved this change.

@opayne1 opayne1 added this to the Planned for 4.12 GA milestone Oct 4, 2022
@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 4, 2022
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Oct 4, 2022

🤖 Updated build preview is available at:
https://51185--docspreview.netlify.app

Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/2081

@opayne1
Copy link
Contributor Author

opayne1 commented Oct 4, 2022

Hi @XiyunZhao could you help review this small release note for making status.HostIP visible on the pod details page?

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 8, 2022
@XiyunZhao
Copy link

@opayne1 The update looks good
/label qe-approved

@openshift-ci openshift-ci bot added the qe-approved Signifies that QE has signed off on this PR label Oct 19, 2022
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 19, 2022
@opayne1 opayne1 added the peer-review-needed Signifies that the peer review team needs to review this PR label Oct 19, 2022
@jboxman-rh jboxman-rh added peer-review-in-progress Signifies that the peer review team is reviewing this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Oct 19, 2022

Choose a reason for hiding this comment

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

@opayne1, out of scope, but I wonder if it makes sense to parallelize these and drop the redundant With this update, now that there are four of these in a row?

We've been using IP address instead of just IP when possible.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point! I have a plan to reorg the web console sections and break up with admin/dev rel notes to make clearer. I will address that in that PR and i can share with dev perspective writers.

I will update with IP address. I was looking for a word to make that sentence flow better. Thanks!

@jboxman-rh
Copy link

@opayne1, left a comment, thanks!

@jboxman-rh jboxman-rh added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Oct 19, 2022
@jboxman-rh
Copy link

@opayne1, LGTM, thanks!

@opayne1 opayne1 merged commit b24fd9a into openshift:enterprise-4.12 Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.12 peer-review-done Signifies that the peer review team has reviewed this PR qe-approved Signifies that QE has signed off on this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants