Skip to content
This repository was archived by the owner on Jul 30, 2021. It is now read-only.

Update upstream Knative Eventing operator to 0.13.1#1337

Merged
dmesser merged 1 commit intooperator-framework:masterfrom
aliok:knative-eventing-operator-0.13.0
Apr 9, 2020
Merged

Update upstream Knative Eventing operator to 0.13.1#1337
dmesser merged 1 commit intooperator-framework:masterfrom
aliok:knative-eventing-operator-0.13.0

Conversation

@aliok
Copy link
Copy Markdown
Contributor

@aliok aliok commented Mar 13, 2020

Signed-off-by: Ali Ok aliok@redhat.com

Thanks submitting your Operator. Please check below list before you create your Pull Request.


Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.


New Submissions

Updates to existing Operators

  • Is your new CSV pointing to the previous version with the replaces property?
  • Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
  • Have you tested an update to your Operator when deployed via OLM?
  • Is your submission signed?

Your submission should not

  • Modify more than one operator
  • Modify an Operator you don't own
  • Rename an operator - please remove and add with a different name instead
  • Submit operators to both upstream-community-operators and community-operators at once
  • Modify any files outside the above mentioned folders
  • Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

  1. Description about the managed Application and where to find more information
  2. Features and capabilities of your Operator and how to use it
  3. Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

  • Human readable name and 1-liner description about your Operator
  • Valid category name1
  • One of the pre-defined capability levels2
  • Links to the maintainer, source code and documentation
  • Example templates for all Custom Resource Definitions intended to be used
  • A quadratic logo

Remember that you can preview your CSV here.

--

1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need

2 For more information see here

@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. kubernetes-operator An Operator targeting OperatorHub.io labels Mar 13, 2020
@aliok aliok force-pushed the knative-eventing-operator-0.13.0 branch from 70b6d35 to cb0071e Compare March 13, 2020 19:09
@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 13, 2020

/retest

@openshift-ci-robot
Copy link
Copy Markdown
Collaborator

@aliok: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

Details

In response to this:

/retest

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@aliok aliok force-pushed the knative-eventing-operator-0.13.0 branch 3 times, most recently from 5fe39b6 to eaea61e Compare March 13, 2020 19:48
@J0zi
Copy link
Copy Markdown
Collaborator

J0zi commented Mar 16, 2020

/ok-to-test

@openshift-ci-robot openshift-ci-robot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Mar 16, 2020
@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 16, 2020

/retest

Signed-off-by: Ali Ok <aliok@redhat.com>
@aliok aliok force-pushed the knative-eventing-operator-0.13.0 branch from eaea61e to 99f79b9 Compare March 27, 2020 11:16
@aliok aliok changed the title Update upstream Knative Eventing operator to 0.13.0 Update upstream Knative Eventing operator to 0.13.1 Mar 27, 2020
@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 27, 2020

/retest

1 similar comment
@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 27, 2020

/retest

@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 27, 2020

@J0zi
getting some strange errors:

Install OLM                                       [  Processing  ]
error: error loading config file "/root/.kube/config": read /root/.kube/config: is a directory
time="2020-03-27T11:22:21Z" level=fatal msg="Failed to install OLM version \"0.14.1\": failed to get Kubernetes config: <nil>"
error: error loading config file "/root/.kube/config": read /root/.kube/config: is a directory

Can you have a look please?

@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 27, 2020

Possible cause:

$ scripts/ci/run-deployment-tests.sh
Detected modified Operator in upstream-community-operators/knative-eventing-operator
Detected modified Operator version 0.13.1
/bin/sh: 1: [[: not found
/bin/sh: 1: [[: not found
/bin/sh: 1: [[: not found

@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 27, 2020

/retest

@aliok aliok marked this pull request as ready for review March 30, 2020 07:28
@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 Mar 30, 2020
@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 30, 2020

/retest

@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Mar 30, 2020

still the same error @J0zi

@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Apr 2, 2020

/retest

@aliok
Copy link
Copy Markdown
Contributor Author

aliok commented Apr 2, 2020

@J0zi can we merge this?

@J0zi J0zi closed this Apr 7, 2020
@J0zi J0zi reopened this Apr 7, 2020
@J0zi
Copy link
Copy Markdown
Collaborator

J0zi commented Apr 7, 2020

Hi @aliok, it fails on UnsupportedOperatorGroup

@J0zi J0zi self-assigned this Apr 7, 2020
@dmesser dmesser closed this Apr 8, 2020
@dmesser dmesser reopened this Apr 8, 2020
@dmesser dmesser merged commit 73b5b1a into operator-framework:master Apr 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

kubernetes-operator An Operator targeting OperatorHub.io ok-to-test Indicates a non-member PR verified by an org member that is safe to test.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants