Skip to content

WIP: 4.10 downstream/upstream sync - first eighth#202

Closed
njhale wants to merge 8 commits intoopenshift:masterfrom
njhale:sync/eighth
Closed

WIP: 4.10 downstream/upstream sync - first eighth#202
njhale wants to merge 8 commits intoopenshift:masterfrom
njhale:sync/eighth

Conversation

@njhale
Copy link
Copy Markdown
Contributor

@njhale njhale commented Oct 13, 2021 •
edited
Loading

timflannagan and others added 8 commits October 13, 2021 14:57
@timflannagan @njhale
.github/workflows: Fix the release quickstart.yaml GH action
b85b9c4 
Update the .github/workflows/quickstart.yml github action and replace
the usage of `kubectl wait ...` with a function that waits until the
various OLM component deployment resources are present and reporting an
available status.

Using `kubectl wait ...` is potentially problematic as it doesn't
support waiting until the creation of that resource, so in the case the
PackageServer deployment doesn't exist yet as the catalog/olm operators
are still being setup, this action will fail as `kubectl wait ...` will
return a non-zero exit code.

Signed-off-by: timflannagan <timflannagan@gmail.com>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: b98ff9d1601b41847fb47445a8366a1145c7cbe6
@benluddy @njhale
Expose errors generated while retrieving catalog content.
986fad7 
Failing to fetch catalog content should not silently return an empty
cache. Instead, it should fail outright with an error that indicates
which catalog(s) could not be fetched.

Signed-off-by: Ben Luddy <bluddy@redhat.com>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: 6ab5f1634093c91d7027b9245a88ede7b807088b
@timflannagan @njhale
v0.18.3 OLM release
053fc41 
Signed-off-by: timflannagan <timflannagan@gmail.com>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: 1e7e1cc81d5c3e0f2de2a06cb4315ea622371cbe
@njhale
Update containerd dependency to fix vulnerabilities (openshift#719)
97d8c25 
Update to containerd 1.4.8

Signed-off-by: Vu Dinh <vudinh@outlook.com>
Upstream-repository: operator-registry
Upstream-commit: aa44cbec0df254fb503a34384a85e197a7e68e6f
@timflannagan @njhale
doc/design: Update the changelog release documentation (#2298)
361a605 
Update the doc/design/release.md documentation around how to generate
the root CHANGELOG.md. Previously, when running into API rate limiting
behaviors locally, the suggestion was to modify a variable in the Ruby
gem package that no longer exists. Now, specify the --max-issues CLI
flag to lower the number of API requests to the OLM repository.

Signed-off-by: timflannagan <timflannagan@gmail.com>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: 5c40752a9605669d34234bf18a93dba948b59439
@timflannagan @njhale
.github/workflows: Avoid running the e2e-tests workflow on doc-only p…
da7bf2d 
…ull requests (#2299)

Update the e2e-tests.yml github workflow and avoid running that action
on PRs that only change the doc directory.

Signed-off-by: timflannagan <timflannagan@gmail.com>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: 939bf94f834181d60a713191f78b13b80c7bec0f
@ecordell @njhale
don't return an error if the singleton rbac doesn't exist (#2309)
7fccc56 
Signed-off-by: Evan <cordell.evan@gmail.com>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: 538b1315eace85e33df7b2893a85a000f8757349
@ecordell @njhale
let net.Listen pick a port for grpc servers generated in tests (#2310)
90f65dc 
Signed-off-by: Evan <cordell.evan@gmail.com>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: 44fc44daf778cd9d36d6b6eb8edf63ff5a202304
@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 Oct 13, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 13, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: njhale

The full list of commands accepted by this bot can be found here.

The pull request process is described 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

@njhale njhale changed the title WIP: downstream/upstream sync - first eighth WIP: 4.10 downstream/upstream sync - first eighth Oct 13, 2021
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 13, 2021
This was referenced Oct 13, 2021
Merged
Closed
Closed
@njhale
Copy link
Copy Markdown
Contributor Author

njhale commented Oct 14, 2021

/retest

4 similar comments
@timflannagan
Copy link
Copy Markdown
Contributor

timflannagan commented Oct 14, 2021

/retest

@njhale
Copy link
Copy Markdown
Contributor Author

njhale commented Oct 14, 2021

/retest

@njhale
Copy link
Copy Markdown
Contributor Author

njhale commented Oct 14, 2021

/retest

@njhale
Copy link
Copy Markdown
Contributor Author

njhale commented Oct 15, 2021

/retest

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 15, 2021

@njhale: The following test 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-upgrade 90f65dc link true /test e2e-upgrade

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.

@njhale
Copy link
Copy Markdown
Contributor Author

njhale commented Oct 16, 2021

/retest

@timflannagan
Copy link
Copy Markdown
Contributor

timflannagan commented Oct 16, 2021

first 1/2 PR merged.

/close

@openshift-ci openshift-ci Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 16, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 16, 2021

@njhale: PR needs rebase.

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.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 16, 2021

@timflannagan: Closed this PR.

Details

In response to this:

first 1/2 PR merged.

/close

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.

@openshift-ci openshift-ci Bot closed this Oct 16, 2021
anik120 pushed a commit to anik120/operator-framework-olm that referenced this pull request Dec 10, 2021
@anik120
feat(cel): Add CEL custom library for semver comparison (openshift#202)
ff19cb1 
* feat(cel): Add CEL custom library for semver comparison

Add CEL custom library for semver comparison to api repo. This
library is used in olm and registry.

Signed-off-by: Vu Dinh <vudinh@outlook.com>

* Simplify the CEL library code structure

Remind the original cel package to constraints pkg to reflect that
there will be more type of constraints.

Remove the evaluator/provider interfaces that are not needed yet.
The generic interface can be designed later when a new type of
constraint is introduced.

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: api
Upstream-commit: 1eb6c0266cce418b5f208d855d678174e02e5e55
anik120 pushed a commit to anik120/operator-framework-olm that referenced this pull request Dec 13, 2021
@anik120
feat(cel): Add CEL custom library for semver comparison (openshift#202)
54b083f 
* feat(cel): Add CEL custom library for semver comparison

Add CEL custom library for semver comparison to api repo. This
library is used in olm and registry.

Signed-off-by: Vu Dinh <vudinh@outlook.com>

* Simplify the CEL library code structure

Remind the original cel package to constraints pkg to reflect that
there will be more type of constraints.

Remove the evaluator/provider interfaces that are not needed yet.
The generic interface can be designed later when a new type of
constraint is introduced.

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: api
Upstream-commit: 1eb6c0266cce418b5f208d855d678174e02e5e55
anik120 pushed a commit to anik120/operator-framework-olm that referenced this pull request Dec 15, 2021
@anik120
chore(deps): upgrade of/api for new constraint type (openshift#880)
12e6af9 
Update operator-framework/api dependency to include a new constraint
type (openshift#202).

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: operator-registry
Upstream-commit: 808de07a2331176414b4cb4a2f2d6fdf54467ff9
anik120 pushed a commit to anik120/operator-framework-olm that referenced this pull request Dec 15, 2021
@anik120
chore(deps): upgrade of/api for new constraint type (openshift#880)
af054d2 
Update operator-framework/api dependency to include a new constraint
type (openshift#202).

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: operator-registry
Upstream-commit: 808de07a2331176414b4cb4a2f2d6fdf54467ff9
anik120 pushed a commit to anik120/operator-framework-olm that referenced this pull request Dec 15, 2021
@anik120
chore(deps): upgrade of/api for new constraint type (openshift#880)
30b3954 
Update operator-framework/api dependency to include a new constraint
type (openshift#202).

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: operator-registry
Upstream-commit: 808de07a2331176414b4cb4a2f2d6fdf54467ff9
timflannagan pushed a commit to timflannagan/operator-framework-olm that referenced this pull request Dec 15, 2021
@timflannagan
chore(deps): upgrade of/api for new constraint type (openshift#880)
6a63e0b 
Update operator-framework/api dependency to include a new constraint
type (openshift#202).

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: operator-registry
Upstream-commit: 808de07a2331176414b4cb4a2f2d6fdf54467ff9
perdasilva pushed a commit to perdasilva/operator-framework-olm that referenced this pull request Mar 4, 2022
@perdasilva
feat(cel): Add CEL custom library for semver comparison (openshift#202)
2515db4 
* feat(cel): Add CEL custom library for semver comparison

Add CEL custom library for semver comparison to api repo. This
library is used in olm and registry.

Signed-off-by: Vu Dinh <vudinh@outlook.com>

* Simplify the CEL library code structure

Remind the original cel package to constraints pkg to reflect that
there will be more type of constraints.

Remove the evaluator/provider interfaces that are not needed yet.
The generic interface can be designed later when a new type of
constraint is introduced.

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: api
Upstream-commit: 1eb6c0266cce418b5f208d855d678174e02e5e55
perdasilva pushed a commit to perdasilva/operator-framework-olm that referenced this pull request Mar 4, 2022
@perdasilva
chore(deps): upgrade of/api for new constraint type (openshift#880)
b938141 
Update operator-framework/api dependency to include a new constraint
type (openshift#202).

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: operator-registry
Upstream-commit: 808de07a2331176414b4cb4a2f2d6fdf54467ff9
perdasilva pushed a commit to perdasilva/operator-framework-olm that referenced this pull request Mar 4, 2022
@perdasilva
feat(cel): Add CEL custom library for semver comparison (openshift#202)
d91d530 
* feat(cel): Add CEL custom library for semver comparison

Add CEL custom library for semver comparison to api repo. This
library is used in olm and registry.

Signed-off-by: Vu Dinh <vudinh@outlook.com>

* Simplify the CEL library code structure

Remind the original cel package to constraints pkg to reflect that
there will be more type of constraints.

Remove the evaluator/provider interfaces that are not needed yet.
The generic interface can be designed later when a new type of
constraint is introduced.

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: api
Upstream-commit: 1eb6c0266cce418b5f208d855d678174e02e5e55
perdasilva pushed a commit to perdasilva/operator-framework-olm that referenced this pull request Mar 4, 2022
@perdasilva
chore(deps): upgrade of/api for new constraint type (openshift#880)
7f00490 
Update operator-framework/api dependency to include a new constraint
type (openshift#202).

Signed-off-by: Vu Dinh <vudinh@outlook.com>

Upstream-repository: operator-registry
Upstream-commit: 808de07a2331176414b4cb4a2f2d6fdf54467ff9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anik120 anik120 Awaiting requested review from anik120

@ankitathomas ankitathomas Awaiting requested review from ankitathomas

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@njhale @timflannagan @benluddy @ecordell