Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove oc cluster up in favor of openshift/installer #21399

Merged
merged 2 commits into from
Nov 1, 2018

Conversation

deads2k
Copy link
Contributor

@deads2k deads2k commented Oct 31, 2018

We are focusing our efforts on a fast, easy to use installer that will drive installation and configuration for all flavors of user instead of trying to segregate users into different classes. Doing this will give platform developers one target and should improve reliability, consistency, and velocity. This removes the old oc cluster up flow. Long live openshift-installer.

/assign @derekwaynecarr
@openshift/sig-master @praveenkumar @sspeiche

@openshift-ci-robot openshift-ci-robot added sig/master size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Oct 31, 2018
Copy link
Contributor

@soltysh soltysh left a comment

Choose a reason for hiding this comment

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

lgtm 🎉

@mfojtik
Copy link
Contributor

mfojtik commented Oct 31, 2018

/retest
/approve

@praveenkumar
Copy link
Contributor

/retest

2 similar comments
@praveenkumar
Copy link
Contributor

/retest

@praveenkumar
Copy link
Contributor

/retest

@mfojtik
Copy link
Contributor

mfojtik commented Nov 1, 2018

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 1, 2018
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, mfojtik

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@deads2k
Copy link
Contributor Author

deads2k commented Nov 1, 2018

/retest

2 similar comments
@mfojtik
Copy link
Contributor

mfojtik commented Nov 1, 2018

/retest

@deads2k
Copy link
Contributor Author

deads2k commented Nov 1, 2018

/retest

@openshift-merge-robot openshift-merge-robot merged commit e6bed07 into openshift:master Nov 1, 2018
@cmoulliard
Copy link

As openshift-installer doesn't yet support xhyve/virtualbox, that will then impact the developer's experience ....

@gustavocoding
Copy link

Looking at the openshift installer, it currently supports AWS and openstack and the libvirt setup is detailed here https://github.com/openshift/installer/blob/master/docs/dev/libvirt-howto.md, so far from being trivial as oc cluster up. Is a replacement for oc cluster up planned?

@sspeiche
Copy link
Contributor

sspeiche commented Jan 4, 2019

oc cluster up was a tool for use by the developers of OpenShift itself, there will exist a streamlined localized installer like minishift. Instead of having multiple competing but different options, we are aligning around the installer and using new version of minishift to provide the experience like oc cluster up

@gustavocoding
Copy link

Thanks @sspeiche, looking forward to it.

@gustavocoding
Copy link

gustavocoding commented Jan 5, 2019

Out of curiosity, now that 'oc cluster up' is dead and developers cannot use it anymore, what are they using now? Docker In Docker cluster or they all migrated to virtualization?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. sig/master 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.

10 participants