Skip to content
This repository was archived by the owner on Jan 28, 2020. It is now read-only.

Conversation

@jlebon
Copy link
Member

@jlebon jlebon commented Jan 30, 2019

This gives RHCOS the capability to pivot right on first boot. We aren't
using this right now (though we were close to) since the MCD just
directly starts the service as needed. But I think it'd be nice for
RHCOS to support this OOTB.

This gives RHCOS the capability to pivot right on first boot. We aren't
using this right now (though we were close to) since the MCD just
directly starts the service as needed. But I think it'd be nice for
RHCOS to support this OOTB.
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 30, 2019
@ashcrow
Copy link
Member

ashcrow commented Jan 31, 2019

@jlebon this should wait until said service is available though, correct?

@ashcrow
Copy link
Member

ashcrow commented Jan 31, 2019

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 31, 2019
@jlebon
Copy link
Member Author

jlebon commented Feb 5, 2019

This can go in before the service exists. It's harmless to have a preset for non-existent services.

@ashcrow
Copy link
Member

ashcrow commented Feb 5, 2019

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 5, 2019
@ashcrow
Copy link
Member

ashcrow commented Feb 5, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 5, 2019
@ashcrow
Copy link
Member

ashcrow commented Feb 5, 2019

/shrug

@openshift-ci-robot openshift-ci-robot added the ¯\_(ツ)_/¯ ¯\\\_(ツ)_/¯ label Feb 5, 2019
@jlebon
Copy link
Member Author

jlebon commented Feb 5, 2019

Weird, not sure why it's not merging. The tide UI does show that the PR meets the label requirements.

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

Labels

lgtm Indicates that a PR is ready to be merged. 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.

4 participants