Merge upstream tag v1.20.0 #243
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
ryantking
commented
May 4, 2022
ryantking
commented
This change adds the labels of the deployment to the ClusterServiceVersions deployment spec. So the labels defined for the deployment (and defined in its manifest) will be applied too, if the operator gets installed via OLM. Signed-off-by: Christoph Stäbler <cstabler@redhat.com>
`memcached-molecule-operator` used in e2e molecule test watches k8s core resources (Secret). Running the operator in cluster-scope causes watching all Secrets in the cluster, which causes high load of the operator and leads to resource creation timeout in the test case. Configuring the operator to watch own namespace workarounds the issue. Fixes #5547 Signed-off-by: Ryo Imai <iryo@jp.ibm.com>
Signed-off-by: Venkat Ramaraju <vramaraj@redhat.com>
Bug Fixes - Fixes issue 74 - chore: update to JOSDK extension 3.0.4 and Quarkus 2.7.3 (PR 76) Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
Signed-off-by: Venkat Ramaraju <vramaraj@redhat.com>
Signed-off-by: Venkat Ramaraju <vramaraj@redhat.com>
Signed-off-by: Venkat Ramaraju <vramaraj@redhat.com>
Signed-off-by: James Lakin <james@jameslakin.co.uk>
* fix: K8S_VERSION in Makefile Signed-off-by: jesus m. rodriguez <jesusr@redhat.com> * make envtest match Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
…receives an error (#5486) Made for Issue #5409 Signed-off-by: aemperador <alexis.emperador@ibm.com>
Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
* log error for url verification failure Signed-off-by: Clyde Germond <clyde.germond@gmail.com>
…as --skip-tls flag is deprecated. (#5617) * change is to add opm new flags --skip-tls-verify and --use-http Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> updated versions Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> add changelog for the change Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> updated variable names Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> removed content files Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> modified code according to version changes Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> updated tests Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> modified tests for --use-http Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> modified due to test sanity Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> modified due to test sanity Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> modified due to test sanity Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> modified due to test sanity Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> updated varibale description Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * updated according to PR review comments Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com>
log could not be set as it was unexported. Signed-off-by: Frederic Giloux <fgiloux@redhat.com>
Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
Signed-off-by: Jonathan Berkhahn <jaberkha@us.ibm.com>
This significantly lowers the load on the test-operator, which was getting restarted due to resource limits. Re #5547 Signed-off-by: Austin Macdonald <austin@redhat.com>
Signed-off-by: Venkat Ramaraju <vramaraj@redhat.com>
Signed-off-by: Venkat Ramaraju <vramaraj@redhat.com>
There is a known controller-tools issue where rbac markers are not parsed correctly if they are not followed by a newline. Fixes: Bugzilla 2061611 Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
Signed-off-by: Prashant Arya <prary2@gmail.com>
Signed-off-by: Venkat Ramaraju <vramaraj@redhat.com>
* Update python dependencies - Bump openshift 0.12 to 0.13 - Skip pipenv check for pyup-45114 https://nvd.nist.gov/vuln/detail/CVE-2021-29923 This CVE is filed against k8s on Golang < 1.17 and is attached to this lib because python-kubernetes uses the Kubernetes API. Signed-off-by: Austin Macdonald <austin@redhat.com> * remove pipfile lock, which was used to update the pipfiles Signed-off-by: Austin Macdonald <austin@redhat.com> * take 2 Signed-off-by: Austin Macdonald <austin@redhat.com>
* revert back --skip-tls with deprecation notice Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * added changelog Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * modified according to test sanity changes Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * modified according to suggestion Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * removed use-http flag Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * modified tag to use http Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * added typos Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * removed comment Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * removed field as it is not used in registry pod Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com>
Part of the image pinning logic is currently missing from master. This PR adds that back along with the released version of Operator Manifest Tools. Signed-off-by: Ryan King <ryking@redhat.com>
Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
Co-authored-by: Prashant Arya <prary2@gmail.com> Signed-off-by: jesus m. rodriguez <jesusr@redhat.com> Co-authored-by: Prashant Arya <prary2@gmail.com>
Signed-off-by: Austin Macdonald <austin@redhat.com>
* Add instructions for archiving unsupported docs Fixes: #5631 Signed-off-by: Austin Macdonald <austin@redhat.com> * Change release perms to be like the other files Signed-off-by: Austin Macdonald <austin@redhat.com>
* update olm bindata operator-sdk was behind on a few OLM versions. updated the supported versions and bindata. resolves #5680 Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * update olm version in testutils Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * update olm based on review Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * update to be 3 latest minor releases Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * update changelog Signed-off-by: Bryce Palmer <bpalmer@redhat.com> * fix netlify badge link Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
* Fix ansible-operator finalizer concurrency issue For ansible-based operators, this change fixes an issue that caused finalizers to fail to run if the watched resource (CR) is deleted during reconciliation. Fixes #4909 Signed-off-by: Austin Macdonald <austin@redhat.com> * dont change constants Signed-off-by: Austin Macdonald <austin@redhat.com> * change name so we dont have a half-made test Signed-off-by: Austin Macdonald <austin@redhat.com> * PR cleanup Signed-off-by: Austin Macdonald <austin@redhat.com> * rm extra --- Signed-off-by: Austin Macdonald <austin@redhat.com>
* Release v1.20.0 Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * added changelog Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * removed website/config.toml-E file Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com> * modified changelog content Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com>
Operator SDK v1.20.0 Merge executed via ./UPSTREAM-MERGE.sh v1.20.0 upstream master Overwritten conflicts: Makefile testdata/ansible/memcached-operator/Dockerfile testdata/ansible/memcached-operator/Makefile testdata/ansible/memcached-operator/bundle/tests/scorecard/config.yaml testdata/ansible/memcached-operator/config/scorecard/patches/basic.config.yaml testdata/ansible/memcached-operator/config/scorecard/patches/olm.config.yaml testdata/go/v2/memcached-operator/bundle/tests/scorecard/config.yaml testdata/go/v2/memcached-operator/config/scorecard/patches/basic.config.yaml testdata/go/v2/memcached-operator/config/scorecard/patches/olm.config.yaml testdata/go/v3/memcached-operator/bundle/tests/scorecard/config.yaml testdata/go/v3/memcached-operator/config/scorecard/patches/basic.config.yaml testdata/go/v3/memcached-operator/config/scorecard/patches/olm.config.yaml testdata/helm/memcached-operator/Dockerfile testdata/helm/memcached-operator/Makefile testdata/helm/memcached-operator/bundle/tests/scorecard/config.yaml testdata/helm/memcached-operator/config/scorecard/patches/basic.config.yaml testdata/helm/memcached-operator/config/scorecard/patches/olm.config.yaml website/content/en/docs/installation/_index.md
openshift-ci
bot
added
the
needs-ok-to-test
|
Hi @ryantking. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
|
/ok-to-test |
openshift-ci
bot
added
ok-to-test
|
@ryantking: all tests passed! Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
|
/label docs-approved |
openshift-ci
bot
added
the
docs-approved
|
/label qe-approved |
openshift-ci
bot
added
the
qe-approved
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jmrodri, ryantking 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 |
openshift-ci
bot
added
the
approved
|
/approved |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.