Skip to content

WIP: Downstream#484

Closed
tmshort wants to merge 18 commits into
openshift:masterfrom
tmshort:downstream2
Closed

WIP: Downstream#484
tmshort wants to merge 18 commits into
openshift:masterfrom
tmshort:downstream2

Conversation

@tmshort

@tmshort tmshort commented Apr 26, 2023

Copy link
Copy Markdown
Contributor

Includes (most) of API and some registry changes.

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 26, 2023
@openshift-ci

openshift-ci Bot commented Apr 26, 2023

Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@tmshort

tmshort commented Apr 26, 2023

Copy link
Copy Markdown
Contributor Author

/test all

@openshift-ci

openshift-ci Bot commented Apr 26, 2023

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: tmshort
Once this PR has been reviewed and has the lgtm label, please assign perdasilva for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 10, 2023
bcrochet and others added 16 commits May 23, 2023 14:47
@bcrochet @tmshort
Fix a panic when reading annotations if the bundle is not there (open…
210f450 
…shift#255)

If the bundle is not present, the current bundleLoader will panic
when it gets to addChannelsFromAnnotationsFile. If the bundle is
nil, addChannelsFromAnnotationsFile should not attempt to do anything
so instead it just returns now.

Signed-off-by: Brad P. Crochet <brad@redhat.com>

Signed-off-by: Brad P. Crochet <brad@redhat.com>
Upstream-repository: api
Upstream-commit: 5f99430d4ec47d59daafa3b818229f0466531dea
@grokspawn @tmshort
introduce spec.runAsRoot field (openshift#258)
4becd85 
Signed-off-by: Jordan <jordan@nimblewidget.com>

Signed-off-by: Jordan <jordan@nimblewidget.com>
Upstream-repository: api
Upstream-commit: 7339a22050af53df7b6f97a652b8e2d73698765a
@laxmikantbpandhare @tmshort
bump k8s to 1.25 and go to 1.19 (openshift#260)
8f8c460 
updated

format

defs

Upstream-repository: api
Upstream-commit: ff2dbc53d3817df9b62a83dbe83cb221e7c043f4
@timflannagan @tmshort
Makefile: Remove the -v go mod tidy flag to fix the verify check (ope…
3e53d80 
…nshift#262)

Signed-off-by: timflannagan <timflannagan@gmail.com>
Upstream-repository: api
Upstream-commit: 72295edd2bb11d414a1db6d6643fe3f3918c5ae4
@timflannagan @tmshort
go.*,pkg: Remove the duplicate github.com/blang/semver dependency (op…
4f71e4c 
…enshift#264)

Signed-off-by: timflannagan <timflannagan@gmail.com>

Signed-off-by: timflannagan <timflannagan@gmail.com>
Upstream-repository: api
Upstream-commit: e4d13db375b88f764a953ce34925e4ad4e79e5fa
@timflannagan @tmshort
crds,Makefile: Bump controller-tools version to v0.9.0 (openshift#263)
8aadf69 
Signed-off-by: timflannagan <timflannagan@gmail.com>
Upstream-repository: api
Upstream-commit: 4d4ed5a299506cab8898036a873031e6faa4b0fd
@anik120 @tmshort
(makefile) Upgrade mikefarah/yq to v4 (openshift#267)
6a38002 
This PR upgrades the version of yq used in the makefile from
v3@latest to v4@v4.28.1

Upstream-repository: api
Upstream-commit: b527a19c8e14e2249d5a5cdb88063cfb136fdafb
@everettraven @tmshort
update k8s 1.25 validation logic (openshift#270)
aa33dfd 
* update k8s 1.25 validation logic

to look for deprecated k8s APIs in various CSV fields

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* update validation logic

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* remove debug statements

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Upstream-repository: api
Upstream-commit: b611f6cef49cb8c6d621145c4e31d8ddfd4c59f4
@grokspawn @tmshort
adding grokspawn to tide owners file (openshift#268)
e2c4060 
Signed-off-by: Jordan Keister <jordan@nimblewidget.com>

Signed-off-by: Jordan Keister <jordan@nimblewidget.com>
Upstream-repository: api
Upstream-commit: 028731a4f915f3a843634554ccc21d78b1d89859
@everettraven @tmshort
(bugfix): make k8s 1.25 validation logic check api group before issui…
18f6068 
…ng a warning (openshift#274)

* fix a bug in k8s 1.25 validation logic

to now check the apiGroup/resource to determine if an api is deprecated.

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* update warning and error checks to use a map

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Upstream-repository: api
Upstream-commit: f1b729684854a053f229464eb327527222188fd1
@dependabot @tmshort
build(deps): bump golang.org/x/text from 0.3.7 to 0.3.8 (openshift#277)
1e54d64 
Bumps [golang.org/x/text](https://github.com/golang/text) from 0.3.7 to 0.3.8.
- [Release notes](https://github.com/golang/text/releases)
- [Commits](golang/text@v0.3.7...v0.3.8)

Upstream-repository: api
Upstream-commit: e8bb2e01756424cd3de5ec8521ef370623459cae

---
updated-dependencies:
- dependency-name: golang.org/x/text
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@grokspawn @tmshort
Updating workflows to pull go version from go.mod file (openshift#1025)
6335ddf 
Signed-off-by: Jordan Keister <jordan@nimblewidget.com>

Signed-off-by: Jordan Keister <jordan@nimblewidget.com>
Upstream-repository: operator-registry
Upstream-commit: bd3c80489dbf6942005c345a33733fb782e952fe
@asmacdo @tmshort
bump k8s to 1.25 (openshift#1024)
b3e09cd 
Signed-off-by: Austin Macdonald <austin@redhat.com>

Signed-off-by: Austin Macdonald <austin@redhat.com>
Upstream-repository: operator-registry
Upstream-commit: 6d762ada58dfd90f41cc002d8f34c01811312f6f
@tmshort
OPRUN-2913: Update vendor directory
79fe794 
Signed-off-by: Todd Short <todd.short@me.com>
@jcho02 @tmshort
go upgrade to 1.19 (openshift#1029)
55f4eb8 
Signed-off-by: jcho02 <jason.cho2@ibm.com>

Signed-off-by: jcho02 <jason.cho2@ibm.com>
Upstream-repository: operator-registry
Upstream-commit: 0271e7847d0b71cd969a2426f5532a4cb1cddd40
@grokspawn @tmshort
left-right swap edge vertices (openshift#1041)
73ac45c 
Signed-off-by: Jordan Keister <jordan@nimblewidget.com>

Signed-off-by: Jordan Keister <jordan@nimblewidget.com>
Upstream-repository: operator-registry
Upstream-commit: 89594183c5c9ea38c183d2f38c2644dba0f37024
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 23, 2023
@tmshort

tmshort commented May 25, 2023

Copy link
Copy Markdown
Contributor Author

/retest

joelanford and others added 2 commits May 25, 2023 13:55
@joelanford @tmshort
opm migrate: add help text note about concatenated json format (opens…
6e3de3e 
…hift#1055)

Signed-off-by: Joe Lanford <joe.lanford@gmail.com>

Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
Upstream-repository: operator-registry
Upstream-commit: b7307b57c5f67b296098e66bfe38458e762c99aa
@rashmigottipati @tmshort
OPECO-2646: exclude bundles with olm.deprecated property when rende…
e4e9bf8 
…ring (openshift#1063)

* exclude bundles with `olm.deprecated` property when rendering

Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com>

* Exclude bundles based on the property type and not value

Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com>

---------

Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com>
Upstream-repository: operator-registry
Upstream-commit: 0aeffa3f44f5e36bd2c0bcc63a94eda000a5f257
@tmshort

tmshort commented May 30, 2023

Copy link
Copy Markdown
Contributor Author

/retest

@openshift-ci

openshift-ci Bot commented May 30, 2023

Copy link
Copy Markdown
Contributor

@tmshort: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-gcp-olm e4e9bf8 link true /test e2e-gcp-olm
ci/prow/unit-olm e4e9bf8 link true /test unit-olm
ci/prow/verify e4e9bf8 link true /test verify

Full PR test history. Your PR dashboard.

Details

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.

@tmshort tmshort closed this Jun 29, 2023
@tmshort tmshort deleted the downstream2 branch October 23, 2023 13:35
openshift-bot pushed a commit to openshift-bot/operator-framework-olm that referenced this pull request Jun 6, 2026
@dependabot
Bump codecov/codecov-action from 5 to 6 (openshift#484)
0e8c4f5 
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 5 to 6.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v5...v6)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Upstream-repository: api
Upstream-commit: 8ad46923e7cf7a8b66f7b5b647288bb652497c12
@openshift-bot openshift-bot mentioned this pull request Jun 6, 2026
Open
openshift-bot pushed a commit to openshift-bot/operator-framework-olm that referenced this pull request Jun 9, 2026
@dependabot
Bump codecov/codecov-action from 5 to 6 (openshift#484)
7720fab 
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 5 to 6.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v5...v6)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Upstream-repository: api
Upstream-commit: 8ad46923e7cf7a8b66f7b5b647288bb652497c12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.