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.

Sign up for GitHub

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

Jump to bottom

Remove quick installer #5563

Merged
merged 3 commits into from
Apr 7, 2018
Merged

Remove quick installer #5563

merged 3 commits into from
Apr 7, 2018

Conversation

sdodson
Copy link
Member

@sdodson sdodson commented Sep 27, 2017 •
edited
Loading

We don't intend to support the quick installer any longer. If you have a cluster that's been previously installed or managed via the quick installer you should have a .config/openshift/hosts file which can be used by ansible-playbook directly following the Advanced installer documentation.

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Sep 27, 2017
@mtnbikenc
Copy link
Member

/retest

1 similar comment
@sdodson
Copy link
Member Author

sdodson commented Oct 9, 2017

/retest

@sdodson
Copy link
Member Author

sdodson commented Oct 9, 2017

Need to update CI tooling to use 'openshift-ansible' rather than 'atomic-openshift-utils'. :-(

@sdodson sdodson force-pushed the remove-quick-installer branch 2 times, most recently from 52a7b0e to e0e779f Compare October 13, 2017 17:32
@sdodson sdodson mentioned this pull request Oct 13, 2017
Merged
@sdodson
Copy link
Member Author

sdodson commented Oct 13, 2017

/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 Oct 13, 2017
@openshift-merge-robot
Copy link
Contributor

@sdodson PR needs rebase

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 1, 2017
@mtnbikenc
Copy link
Member

@sdodson Thoughts on coming back to this and getting it done?

@mtnbikenc
Copy link
Member

@sdodson Where do we stand on this?

@sdodson
Copy link
Member Author

sdodson commented Dec 8, 2017

@sdodson Where do we stand on this?

We have to meet with PM and CE&E to determine a solution that fills the need. I think a simplified all in one inventory like #6129 will likely suffice but we need to make sure those parties are bought in. I'll arrange a meeting.

@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 6, 2018
@sdodson sdodson changed the title [WIP] Remove quick installer Remove quick installer Apr 6, 2018
@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 Apr 6, 2018
@sdodson
Copy link
Member Author

sdodson commented Apr 6, 2018

/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 Apr 6, 2018
@sdodson
Copy link
Member Author

sdodson commented Apr 6, 2018

We'll flatten everything down to just one package in the near future.

mtnbikenc
mtnbikenc approved these changes Apr 6, 2018
View reviewed changes
Copy link
Member

@mtnbikenc mtnbikenc left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 6, 2018
michaelgugino
michaelgugino approved these changes Apr 6, 2018
View reviewed changes
Copy link
Contributor

@michaelgugino michaelgugino left a comment

Choose a reason for hiding this comment

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

/lgtm

@sdodson
Copy link
Member Author

sdodson commented Apr 6, 2018

/test gcp

1 similar comment
@sdodson
Copy link
Member Author

sdodson commented Apr 6, 2018

/test gcp

@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 7, 2018 •
edited
Loading

@sdodson: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
ci/openshift-jenkins/upgrade e0e779f link /test upgrade
ci/openshift-jenkins/logging e0e779f link /test logging
ci/openshift-jenkins/containerized e0e779f link /test containerized

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.

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.

@sdodson
Copy link
Member Author

sdodson commented Apr 7, 2018 via email

@openshift-merge-robot openshift-merge-robot merged commit ac710a3 into openshift:master Apr 7, 2018
@DanyC97 DanyC97 mentioned this pull request Apr 7, 2018
Closed
@runcom runcom mentioned this pull request Apr 18, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michaelgugino michaelgugino michaelgugino approved these changes

@mtnbikenc mtnbikenc mtnbikenc approved these changes

Assignees

@michaelgugino michaelgugino

@mtnbikenc mtnbikenc

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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@sdodson @mtnbikenc @openshift-merge-robot @openshift-ci-robot @michaelgugino @openshift-bot