Remove other older and unsupported content from origin

[smarterclayton]

Federation is no longer supported. Cluster capacity is removed in the other prerequisite. Purge the base images down.

Replace origin with origin-control-plane and node with origin node.

Change --image in oc cluster up to mean image format, like ansible. --tag stays the same.

[smarterclayton]

Will probably wait until #19261 is merged, but required for 3.10 in order to get the renames (I can break out those changes if necessary).

Once this is merged we can change ansible to use the new names.

[smarterclayton]

@sdodson @derekwaynecarr fyi

Scott, note that I am removing all of the old master services, and a bunch of unused node cruft. I'd like to fixup the node stuff so that it is correct for all deployments, but not quite there yet (it's going to be broken the moment we split out the node).

[smarterclayton]

/test integration

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: smarterclayton

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [smarterclayton]

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

[sdodson]

/retest

[smarterclayton]

@mfojtik this contains a change to oc cluster up to make --image behave like --imageformat, not like a prefix. We are dropping support for the prefix across all components because of changes in how we're going to do CI, the move to use quay, and making openshift-ansible consistent. This PR removes all non-compliant names in the origin repo, and anything that is required to be part of cluster up and tested as a unit with a cluster install must be in the standard image format. Please review your part (this is blocking OCP because we've already made the renames there).

[smarterclayton]

/retest

[smarterclayton]

/retest

[smarterclayton]

/test extended_conformance_install

[smarterclayton]

/retest

[openshift-ci-robot]

New changes are detected. LGTM label has been removed.

[smarterclayton]

/retest

[sdodson]

/test gcp

[smarterclayton]

/test all

[sdodson]

GCP job looks like a flake, go ahead and get this in?

[smarterclayton]

Yeah wanted one more run then will merge

[nak3]

It looks like test/cmd/newapp.sh started failing because https://raw.githubusercontent.com/openshift/origin/master/examples/wordpress/template/wordpress-mysql.json, which is deleted by this PR, is pointed by the script.

origin/test/cmd/newapp.sh

Line 516 in fedbc3b

os::cmd::expect_success 'oc new-app https://raw.githubusercontent.com/openshift/origin/master/examples/wordpress/template/wordpress-mysql.json --dry-run'

[bparees]

Fun, this pr burned the bridge behind it.

[bparees]

fixing the test here: #19421

[smarterclayton]

Bridges are made to be burned.

…

On Wed, Apr 18, 2018 at 11:12 PM, Ben Parees ***@***.***> wrote: fixing the test here: #19421 <#19421> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#19262 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABG_p2Ut-FkVdLcYyHeBsoFKkJtWK5rvks5tqACogaJpZM4TLS1A> .