-
Notifications
You must be signed in to change notification settings - Fork 32
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
Bug 2045878: v1.16.0 rebase master #203
Bug 2045878: v1.16.0 rebase master #203
Conversation
* bump kb with the latest changes/fixes - v3.2.0
* bump operator-frawork-api Signed-off-by: Camila Macedo <cmacedo@redhat.com> * add changelog Signed-off-by: Camila Macedo <cmacedo@redhat.com> * Update changelog/fragments/bump-operator-frameworkapi.yaml Co-authored-by: Austin Macdonald <austin@redhat.com> * Update changelog/fragments/bump-operator-frameworkapi.yaml Co-authored-by: Austin Macdonald <austin@redhat.com> * Update changelog/fragments/bump-operator-frameworkapi.yaml Co-authored-by: Austin Macdonald <austin@redhat.com> Co-authored-by: Austin Macdonald <austin@redhat.com>
Signed-off-by: austin <austin@redhat.com>
This order seems easier to understand at least to a non-native-speaker. Signed-off-by: Marcin Owsiany <porridge@redhat.com>
…r (#5414) Signed-off-by: Camila Macedo <cmacedo@redhat.com>
…(#5407) * feat: add the deprecated APIs optional checker, , to the suite operatorframework Signed-off-by: Camila Macedo <cmacedo@redhat.com> * Update changelog/fragments/suite.yaml Co-authored-by: Varsha <varshaprasad96@gmail.com> Co-authored-by: Varsha <varshaprasad96@gmail.com>
* doc: add best practices for multi-tenancy solutions Signed-off-by: Camila Macedo <cmacedo@redhat.com> Co-authored-by: Jesus Rodriguez <jmrodri@gmail.com> Co-authored-by: Alexander Greene <agreene@redhat.com> Co-authored-by: Frederic Giloux <fgiloux@redhat.com> Co-authored-by: Daniel Messer <dmesser@redhat.com> Co-authored-by: Ben Parees <bparees@users.noreply.github.com>
* add new good practices validator * fin nit typo issue in the fragment
Signed-off-by: austin <austin@redhat.com>
Signed-off-by: austin <austin@redhat.com>
Signed-off-by: Ish Shah <ishah@redhat.com>
* Upgrade OPM version used Signed-off-by: Thierry Wasylczenko <thierry.wasylczenko@gmail.com> * Upgrade OPM version to v1.18 and update 00-template.yaml Signed-off-by: Thierry Wasylczenko <thierry.wasylczenko@gmail.com> * Use correct OPM version with 3 parts Signed-off-by: Thierry Wasylczenko <thierry.wasylczenko@gmail.com> * Dedicated fragment for release notes Signed-off-by: Thierry Wasylczenko <thierry.wasylczenko@gmail.com> * [upgrade-opm-version] Using OPM 1.19.1 Signed-off-by: Thierry Wasylczenko <thierry.wasylczenko@gmail.com>
* Ignore CVEs 2021-3533 and 2021-3532 in base image The following CVEs are causing `pipenv check` step of image build to fail: https://nvd.nist.gov/vuln/detail/CVE-2021-3533 https://nvd.nist.gov/vuln/detail/CVE-2021-3532 Both of these CVEs are against Ansible 2.9. The Ansible team is aware of them, but they require deliberately insecure configuration from the user and the Ansible team considers them invalid CVEs. There are no plans for these to be fixed in the 2.9.z stream and will not cause a security problem with Ansible-based operators. Signed-off-by: austin <austin@redhat.com> * Rerun sanity tests in 2022 Signed-off-by: austin <austin@redhat.com>
* feat: add resource limits for ansible Signed-off-by: Camila Macedo <cmacedo@redhat.com> * nit suggestion abut changelog info
…work/ansible-operator-base:master-69ba8b0e9b375529ab47788bd0dbe4e3524ddc12 (#5467) * [master] image(ansible-operator): bump base to quay.io/operator-framework/ansible-operator-base:master-69ba8b0e9b375529ab47788bd0dbe4e3524ddc12 Signed-off-by: asmacdo <asmacdo@users.noreply.github.com> * [master] image(ansible-operator-2.11-preview): bump base to quay.io/operator-framework/ansible-operator-2.11-preview-base:master-69ba8b0e9b375529ab47788bd0dbe4e3524ddc12 (#5468) Signed-off-by: asmacdo <asmacdo@users.noreply.github.com> Co-authored-by: asmacdo <asmacdo@users.noreply.github.com> Co-authored-by: asmacdo <asmacdo@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
Signed-off-by: Ryan King <ryking@redhat.com>
Signed-off-by: Ryan King <ryking@redhat.com>
Operator SDK v1.16.0 Merge executed via ./UPSTREAM-MERGE.sh v1.16.0 upstream master Overwritten conflicts: <NONE>
Signed-off-by: Fabian von Feilitzsch <fabian@fabianism.us>
b85e415
to
9bf710d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack
@@ -4,7 +4,7 @@ SHELL = /bin/bash | |||
# This value must be updated to the release tag of the most recent release, a change that must | |||
# occur in the release commit. IMAGE_VERSION will be removed once each subproject that uses this | |||
# version is moved to a separate repo and release process. | |||
export IMAGE_VERSION = v1.15.0 | |||
export IMAGE_VERSION = v1.16.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
@@ -7,7 +7,7 @@ diff -up ./Makefile.setversion ./Makefile | |||
# Build-time variables to inject into binaries | |||
-export SIMPLE_VERSION = $(shell (test "$(shell git describe)" = "$(shell git describe --abbrev=0)" && echo $(shell git describe)) || echo $(shell git describe --abbrev=0)+git) | |||
-export GIT_VERSION = $(shell git describe --dirty --tags --always) | |||
+export SIMPLE_VERSION = v1.15.0-ocp | |||
+export SIMPLE_VERSION = v1.16.0-ocp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
export GIT_VERSION = $(SIMPLE_VERSION) | ||
export GIT_COMMIT = $(shell git rev-parse HEAD) | ||
-export K8S_VERSION = 1.21 | ||
+export K8S_VERSION = 1.22 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
@fabianvf: This pull request references Bugzilla bug 2045878, which is valid. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (jfan@redhat.com), skipping review request. In response to this:
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. |
/retest |
1 similar comment
/retest |
/retest |
@fabianvf: 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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabianvf, 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:
Approvers can indicate their approval by writing |
@fabianvf: All pull requests linked via external trackers have merged: Bugzilla bug 2045878 has been moved to the MODIFIED state. In response to this:
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. |
Description of the change:
Motivation for the change:
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments
(seechangelog/fragments/00-template.yaml
)website/content/en/docs