Skip to content

Conversation

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 2, 2021
@netlify
Copy link

netlify bot commented Sep 2, 2021

✔️ Deploy Preview for osdocs ready!

🔨 Explore the source changes: bb48461

🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/61572cb19b82e80007157fec

😎 Browse the preview: https://deploy-preview-36065--osdocs.netlify.app/openshift-enterprise/latest/networking/ingress-operator

@skrthomas skrthomas force-pushed the OSDOCS-2456 branch 2 times, most recently from 754ae62 to 93283d4 Compare September 2, 2021 20:57
Copy link
Contributor

@jboxman jboxman left a comment

Choose a reason for hiding this comment

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

@skrthomas, drive-by comments. We're using config map and code font for object names and name space names.

@skrthomas skrthomas force-pushed the OSDOCS-2456 branch 3 times, most recently from 20e7b2d to de78741 Compare September 8, 2021 18:42
@skrthomas skrthomas force-pushed the OSDOCS-2456 branch 5 times, most recently from 4054cd2 to acdd81d Compare September 13, 2021 21:07
@miheer
Copy link

miheer commented Sep 14, 2021

@skrthomas PTAL at my comments. Thanks for working on this!

@skrthomas skrthomas force-pushed the OSDOCS-2456 branch 3 times, most recently from bc2b01d to 99ece85 Compare September 14, 2021 17:36
@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 14, 2021
@skrthomas skrthomas force-pushed the OSDOCS-2456 branch 7 times, most recently from 53f8000 to c8d839c Compare September 17, 2021 14:59
@miheer
Copy link

miheer commented Sep 23, 2021

@skrthomas lgtm

Copy link

@mikemckiernan mikemckiernan left a comment

Choose a reason for hiding this comment

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

As always, please let me know what I can clarify.

@mikemckiernan mikemckiernan added the peer-review-done Signifies that the peer review team has reviewed this PR label Sep 28, 2021
@skrthomas skrthomas force-pushed the OSDOCS-2456 branch 5 times, most recently from 8bb2ba7 to 66844e5 Compare September 30, 2021 16:39
Copy link

@mikemckiernan mikemckiernan left a comment

Choose a reason for hiding this comment

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

Nits, comments, and up-to-yous. Please let me know what I can clarify.

Copy link
Contributor

@sferich888 sferich888 left a comment

Choose a reason for hiding this comment

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

LGTM

@skrthomas skrthomas force-pushed the OSDOCS-2456 branch 4 times, most recently from 7f0e1db to 00bf129 Compare October 1, 2021 14:01
Copy link

@mikemckiernan mikemckiernan left a comment

Choose a reason for hiding this comment

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

A few more items. Apologies if these are items that I overlooked previously and that I'm burning your time.

@mikemckiernan mikemckiernan merged commit 31565ec into openshift:main Oct 1, 2021
@mikemckiernan
Copy link

/cherrypick enterprise-4.9

@openshift-cherrypick-robot

@mikemckiernan: new pull request created: #36983

In response to this:

/cherrypick enterprise-4.9

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.

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

Labels

branch/enterprise-4.9 peer-review-done Signifies that the peer review team has reviewed this PR 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.

8 participants