Skip to content

Conversation

@csrwng
Copy link
Contributor

@csrwng csrwng commented Feb 21, 2019

Includes fix for uninstall from installer master repo
Fixes up remotemachineset controller to work with machine-api instead of cluster-api

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 21, 2019
@openshift-ci-robot openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Feb 21, 2019
@dgoodwin
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 21, 2019
@csrwng
Copy link
Contributor Author

csrwng commented Feb 21, 2019

/test e2e

@csrwng csrwng changed the title WIP: Bump to installer master branch Bump to installer master branch Feb 21, 2019
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 21, 2019
@csrwng
Copy link
Contributor Author

csrwng commented Feb 21, 2019

/retest

2 similar comments
@csrwng
Copy link
Contributor Author

csrwng commented Feb 21, 2019

/retest

@dgoodwin
Copy link
Contributor

/retest

@openshift-ci-robot openshift-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed lgtm Indicates that a PR is ready to be merged. labels Feb 22, 2019
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 22, 2019
@dgoodwin
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 22, 2019
@csrwng
Copy link
Contributor Author

csrwng commented Feb 22, 2019

/retest

@dgoodwin
Copy link
Contributor

/test e2e

@dgoodwin
Copy link
Contributor

/test e2e
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 23, 2019
@csrwng
Copy link
Contributor Author

csrwng commented Feb 23, 2019

/retest

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 23, 2019
@dgoodwin
Copy link
Contributor

/lgtm

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

LGTM on new changes from a PR by myself has to be cheating somehow.

@openshift-ci-robot
Copy link

@csrwng: The following test failed for commit df4a5ff, say /retest to rerun them:

Test name Details Rerun command
ci/prow/e2e link /test e2e

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

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. I understand the commands that are listed here.

@dgoodwin
Copy link
Contributor

We have manually completed an e2e test.

/override ci/prow/e2e

@openshift-ci-robot
Copy link

@dgoodwin: Overrode contexts on behalf of dgoodwin: ci/prow/e2e

Details

In response to this:

We have manually completed an e2e test.

/override ci/prow/e2e

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.

@openshift-merge-robot openshift-merge-robot merged commit bc99566 into openshift:master Feb 23, 2019
wking added a commit to wking/openshift-installer that referenced this pull request Feb 26, 2019
When we pivoted to v1beta3 in ccdc32e (installconfig: separate
control plane and compute machine pools, 2019-01-29, openshift#1157), we
dropped support for the old 'machines' JSON property.  Instead of
silently ignoring that property in v1beta2 configs, error out to avoid
surprising users later when they notice us not picking up their
machines configuration.

Or internal consumers pivoted to v1beta3 in openshift/release@e1d729c6
(Modify install-config.yaml to use controlPlane and compute instead of
machines, 2019-02-05, openshift/release#2787) and
openshift/hive@3eda6d12 (Bump to installer master branch, 2019-02-21,
openshift/hive#228).
@csrwng csrwng deleted the bump_installer branch March 8, 2019 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants