Use new operator - Githubissues

openshift / ansible-service-broker

Ansible Service Broker

Apache License 2.0

226 stars 84 forks source link

Use new operator #1175

Closed djzager closed 5 years ago

djzager commented 5 years ago

This removes the old operator artifacts in favor of the new /operator directory.

djzager commented 5 years ago

This should fail until Prow is updated to reflect the new location.

djzager commented 5 years ago

/hold until

[x] Prow Updated (and CI Passes) ~~- [ ] ART approves of the move~~ Since they haven't created the necessary configuration yet. This point is moot.

eriknelson commented 5 years ago

/hold

coveralls commented 5 years ago

Coverage remained the same at 40.97% when pulling df79d67e6e10302c46ffe9c5d62a6ca20e98f166 on djzager:use-new-operator into 37bec41e082571d032b1c687cd762d233c4652ea on openshift:master.

djzager commented 5 years ago

/hold cancel

djzager commented 5 years ago

/test images

djzager commented 5 years ago

/retest

jmrodri commented 5 years ago

/lgtm

openshift-ci-robot commented 5 years ago

New changes are detected. LGTM label has been removed.

openshift-ci-robot commented 5 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: djzager, eriknelson, jmontleon, jmrodri

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: - ~~[OWNERS](https://github.com/openshift/ansible-service-broker/blob/master/OWNERS)~~ [djzager,eriknelson,jmontleon,jmrodri] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

openshift-merge-robot commented 5 years ago

/retest

openshift-merge-robot commented 5 years ago

/retest