Skip to content

Conversation

@maxwelldb
Copy link
Contributor

@maxwelldb maxwelldb added this to the Future Release milestone Dec 23, 2020
@maxwelldb maxwelldb self-assigned this Dec 23, 2020
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 23, 2020
@openshift-docs-preview-bot

The preview will be available shortly at:

@maxwelldb maxwelldb force-pushed the setting-osp-machine-subnet-osdocs1663 branch from 3d6848c to 3a63e72 Compare December 23, 2020 19:11
@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Dec 23, 2020
@maxwelldb
Copy link
Contributor Author

FYI @iamemilio

@maxwelldb maxwelldb changed the title [OSP] Machine set primary subnet support [OSP] MachineSet primary subnet support Dec 23, 2020
@maxwelldb maxwelldb requested a review from iamemilio January 5, 2021 17:29
@iamemilio
Copy link

lgtm

@maxwelldb
Copy link
Contributor Author

TY!

@maxwelldb maxwelldb force-pushed the setting-osp-machine-subnet-osdocs1663 branch from 17ad430 to 4f28bf0 Compare January 7, 2021 22:30
@ghost
Copy link

ghost commented Jan 11, 2021

LGTM

@maxwelldb
Copy link
Contributor Author

Thanks!

@maxwelldb maxwelldb added the peer-review-needed Signifies that the peer review team needs to review this PR label Jan 11, 2021
@maxwelldb maxwelldb removed the request for review from iamemilio January 11, 2021 19:20
@maxwelldb
Copy link
Contributor Author

@openshift/team-documentation PTAL when you have a moment. Thanks!

Copy link
Contributor

@kalexand-rh kalexand-rh 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 picky things, but it largely LGTM

@kalexand-rh kalexand-rh added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Jan 18, 2021
@maxwelldb
Copy link
Contributor Author

@kalexand-rh TY!

@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 18, 2021
@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 18, 2021
@maxwelldb maxwelldb merged commit fefda3b into openshift:master Jan 18, 2021
@maxwelldb maxwelldb deleted the setting-osp-machine-subnet-osdocs1663 branch January 18, 2021 21:23
@maxwelldb
Copy link
Contributor Author

maxwelldb commented Jan 18, 2021

/cherry-pick enterprise-4.7

@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Jan 18, 2021

@maxwelldb: new pull request created: #28663

Details

In response to this:

/cherry-pick enterprise-4.7

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