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.

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

fix the labels.provider of packagemanifest issue #766

Merged

Conversation

jianzhangbjz
Copy link
Contributor

As title, fix bug 1690241, any comments are welcome!

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 19, 2019
@ecordell
Copy link
Member

/retest

@ecordell ecordell added the kind/bug Categorizes issue or PR as related to a bug. label Mar 19, 2019
@jianzhangbjz
Copy link
Contributor Author

@ecordell I guess it's waiting for the /approve label.

@ecordell
Copy link
Member

/retest

@jianzhangbjz
Copy link
Contributor Author

/test e2e-aws-olm

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 2, 2019
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 15, 2019
@jianzhangbjz
Copy link
Contributor Author

/retest

@ecordell
Copy link
Member

/retest

1 similar comment
@ecordell
Copy link
Member

/retest

@jianzhangbjz
Copy link
Contributor Author

/test e2e-aws-olm

1 similar comment
@jianzhangbjz
Copy link
Contributor Author

/test e2e-aws-olm

@cuppett
Copy link

cuppett commented Apr 22, 2019

/retest e2e-aws-olm

@jianzhangbjz
Copy link
Contributor Author

/retest

3 similar comments
@ecordell
Copy link
Member

/retest

@ecordell
Copy link
Member

/retest

@jianzhangbjz
Copy link
Contributor Author

/retest

@cuppett
Copy link

cuppett commented Apr 29, 2019

/retest e2e-aws-olm

1 similar comment
@jianzhangbjz
Copy link
Contributor Author

/retest e2e-aws-olm

@jianzhangbjz
Copy link
Contributor Author

/retest

1 similar comment
@ecordell
Copy link
Member

ecordell commented May 1, 2019

/retest

@ecordell
Copy link
Member

ecordell commented May 1, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels May 1, 2019
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label May 1, 2019
@ecordell
Copy link
Member

ecordell commented May 1, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 1, 2019
@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ecordell, jianzhangbjz

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:

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 openshift-merge-robot merged commit 8ee9de6 into operator-framework:master May 1, 2019
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. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants