Skip to content

CFE-10/374: binding options#44435

Merged
bobfuru merged 1 commit intoopenshift:mainfrom
Amrita42:CFE-10-374
Apr 14, 2022
Merged

CFE-10/374: binding options#44435
bobfuru merged 1 commit intoopenshift:mainfrom
Amrita42:CFE-10-374

Conversation

@Amrita42
Copy link
Contributor

@Amrita42 Amrita42 commented Apr 11, 2022

NOTE
This PR is duplicated from #43576 but covers only the first file (modules/nw-ingress-controller-configuration-parameters.adoc) . Incorporated SME/peer review feedback.

@sherine-k @quarterpin

UPDATE
The rest_api.adoc file will be updated nearing the time of 4.11 release, as it's going to be built , based on the OpenAPI spec, which OpenShift derives from the openshift/api repository contents.

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Apr 11, 2022
@netlify
Copy link

netlify bot commented Apr 11, 2022

Deploy Preview for osdocs ready!

Name Link
🔨 Latest commit 6abf9eb
🔍 Latest deploy log https://app.netlify.com/sites/osdocs/deploys/625833dbd688890008f7caa6
😎 Deploy Preview https://deploy-preview-44435--osdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@alebedev87
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 12, 2022
@quarterpin
Copy link

/lgtm

@bobfuru bobfuru added this to the Future Release milestone Apr 14, 2022
Copy link
Contributor

@bobfuru bobfuru left a comment

Choose a reason for hiding this comment

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

Overall, LGTM!

Copy link
Contributor

@bobfuru bobfuru Apr 14, 2022

Choose a reason for hiding this comment

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

+1 I thought ISG had guidance for this but couldn't find the citation. 😀

@openshift openshift deleted a comment from jboxman Apr 14, 2022
@bobfuru
Copy link
Contributor

bobfuru commented Apr 14, 2022

Copy link

@jboxman-rh jboxman-rh left a comment

Choose a reason for hiding this comment

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

Few comments, thanks!

@openshift openshift deleted a comment from jboxman Apr 14, 2022
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 14, 2022
@openshift-ci
Copy link

openshift-ci bot commented Apr 14, 2022

New changes are detected. LGTM label has been removed.

@Amrita42
Copy link
Contributor Author

@bobfuru ptal

@bobfuru
Copy link
Contributor

bobfuru commented Apr 14, 2022

LGTM, thanks!

@bobfuru bobfuru merged commit 0df6f40 into openshift:main Apr 14, 2022
@bobfuru
Copy link
Contributor

bobfuru commented Apr 14, 2022

/cherrypick enterprise-4.11

@openshift-cherrypick-robot

@bobfuru: new pull request created: #44611

Details

In response to this:

/cherrypick enterprise-4.11

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.11 size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants