Skip to content
This repository was archived by the owner on Jun 14, 2019. It is now read-only.

Conversation

@smarterclayton
Copy link
Contributor

As we move templates into the ci-operator, we need a way to stage the
changes for sets of jobs and move gradually. When a user provides
--template on the command line, ignore any job with the same name so that
we can begin moving those to internal definitions.

How would we move jobs over to built in steps:

  1. Land this PR to allow templates to override build in steps
  2. Identify a set of ProwJobs that can omit the --template parameter that are not generated
  3. Update the cluster install e2e step to match the latest template
  4. Change the ProwJobs to omit the template parameter
  5. Soak for several days
  6. Identify any gaps in how we implement the template steps, fix ci-operator (secrets, profiles)
  7. Soak more
  8. Change ci-operator-prowgen to omit the --template parameter for all generated jobs
  9. React and validate

…defined

As we move templates into the ci-operator, we need a way to stage the changes
for sets of jobs and move gradually. When a user provides --template on the
command line, ignore any job with the same name so that we can begin moving
those to internal definitions.
@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 1, 2019
@openshift-ci-robot
Copy link

[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:

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

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 1, 2019
@smarterclayton
Copy link
Contributor Author

@petr-muller @bbguimaraes

In order for us to start trying to move templates inline, we need a way to keep backcompat with existing jobs (which use --template) and then allow us to test a few using the version coming out of this binary, then migrate the rest by regenerating ci-operator prowgen. This PR allows us to let --template override any implicit step.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants