Skip to content

Generate operator/v1alpha1 CRDs, update for https://github.com/openshift/api/pull/384#73

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
mtrmac:icsp-explicit-source
Jul 23, 2019
Merged

Generate operator/v1alpha1 CRDs, update for https://github.com/openshift/api/pull/384#73
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
mtrmac:icsp-explicit-source

Conversation

@mtrmac
Copy link
Copy Markdown
Contributor

@mtrmac mtrmac commented Jul 18, 2019

DO NOT MERGE: This includes UNMERGED #70, and will need to be rebased on top of it.

This PR updates vendor openshift/api for openshift/api#384 , adds generation of the CRDs for the operator/v1alpha1 namespace (or was it omitted intentionally?) and runs the CRD generator.

@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 Jul 18, 2019
@damemi
Copy link
Copy Markdown
Contributor

damemi commented Jul 19, 2019

@mtrmac thanks for catching that missing operator/v1alpha1, it definitely wasn't intentional. #70 has been tagged and will hopefully merge soon

@mtrmac mtrmac force-pushed the icsp-explicit-source branch from cf5aebd to 4e309a5 Compare July 19, 2019 19:31
@mtrmac mtrmac changed the title WIP: Generate operator/v1alpha1 CRDs, update for https://github.com/openshift/api/pull/384 Generate operator/v1alpha1 CRDs, update for https://github.com/openshift/api/pull/384 Jul 19, 2019
@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 Jul 19, 2019
mtrmac added 2 commits July 19, 2019 21:32
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Jul 19, 2019

Rebased on top of merged #70, PTAL.

@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Jul 19, 2019

/retest

3 similar comments
@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Jul 19, 2019

/retest

@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Jul 20, 2019

/retest

@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Jul 20, 2019

/retest

@umohnani8
Copy link
Copy Markdown
Contributor

@sttts this is ready to be merged, PTAL.

@umohnani8
Copy link
Copy Markdown
Contributor

@deads2k @abhinavdahiya PTAL

@abhinavdahiya
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 22, 2019
@mrunalp
Copy link
Copy Markdown
Member

mrunalp commented Jul 22, 2019

@deads2k @sttts ptal.

@sttts
Copy link
Copy Markdown
Contributor

sttts commented Jul 23, 2019

/approve

@openshift-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abhinavdahiya, mtrmac, sttts

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 Jul 23, 2019
@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@mrunalp
Copy link
Copy Markdown
Member

mrunalp commented Jul 23, 2019

/test e2e-aws

@mrunalp
Copy link
Copy Markdown
Member

mrunalp commented Jul 23, 2019

/test e2e-aws-upgrade

@umohnani8
Copy link
Copy Markdown
Contributor

/retest

@openshift-merge-robot openshift-merge-robot merged commit 149670d into openshift:master Jul 23, 2019
@mtrmac mtrmac deleted the icsp-explicit-source branch July 23, 2019 22:35
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. 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.

9 participants