Extend artifacts and add debug#1068
Closed
perdasilva wants to merge 77 commits intoopenshift:mainfrom
Closed
Conversation
… they break naming conventions (#3260) Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 3e0230e983397a49306f9f69c6a4561c28fbe67e
* patch flakes Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * Update test/e2e/subscription_e2e_test.go Co-authored-by: Mikalai Radchuk <509198+m1kola@users.noreply.github.com> --------- Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Mikalai Radchuk <509198+m1kola@users.noreply.github.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 161b89ca70027d720347ee9ce7729df4018584ba
* parameterize e2e target Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * pin opm image Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * update default chart values to point to pinned opm image Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> --------- Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 1f5e0dc8e55aba20f3cd637e0b021f081e5761bb
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 71056b532cdea2baa41e78d83635b9980fc0a49b
* update skopeo openshift test Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> * patch subscription e2e flake Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> --------- Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 37dcff4328d76e7307225bffb4620f3f1282d7c3
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: a5230c70dcb90ed25441b72b9b646122ce4bf2c9
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 1404e916dec89d24c5ecfe52fec7f8f093fb917d
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
[release-4.16] OCPBUGS-35219: Unblock CI
… CRD specifies a conversion strategy (#3209) * warn but allow CRD upgrade when validations fail but new CRD has conversion strategy Signed-off-by: everettraven <everettraven@gmail.com> * unexport validationError Signed-off-by: everettraven <everettraven@gmail.com> * updating warning message to be more descriptive, add warning event on installplan Signed-off-by: everettraven <everettraven@gmail.com> * update warning templ per review comments Signed-off-by: everettraven <everettraven@gmail.com> --------- Signed-off-by: everettraven <everettraven@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: c3e17741c5dde8d67203a9e34bba81c4ad2bddcd
…source pods (#3206) * Refactor security context configuration in pod reconciler This change updates the logic for setting security contexts within the OLM pod reconciler. Now, it differentiates between 'Restricted' and 'Legacy' security contexts more explicitly. The 'Restricted' security context applies default security settings unless overridden, while the 'Legacy' context clears all security settings. When no security context is configured, it defaults to restricted. Additionally, the related tests have been updated to reflect these changes and ensure correct behavior. Signed-off-by: btofel <btofel@redhat.com> * Add checking of the namespace PSA restrictions Signed-off-by: btofel <btofel@redhat.com> * Fix linter issues Signed-off-by: btofel <btofel@redhat.com> Signed-off-by: Brett Tofel <btofel@redhat.com> * fixes Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> --------- Signed-off-by: btofel <btofel@redhat.com> Signed-off-by: Brett Tofel <btofel@redhat.com> Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Brett Tofel <btofel@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 9b2802151528e556d78b5fe55bcb8c89070efe58
…ig (openshift#342) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> Upstream-repository: api Upstream-commit: 5d2d3fbe061b7b4a942747877efa58958fa9889e
…-validation-416 OCPBUGS-35373: [release-4.16] Warn and allow CRD upgrade if validation fails but webhook is specified
…iles from git; use rhel8 art builder image Signed-off-by: Jordan Keister <jordan@nimblewidget.com>
…cks directly (#3217) * perform operator apiService certificate validity checks directly Signed-off-by: Ankita Thomas <ankithom@redhat.com> * use sets to track certs to install, revert to checking for installPlan timeouts after API availability checks, add service FQDN to list of hostnames. Signed-off-by: Ankita Thomas <ankithom@redhat.com> --------- Signed-off-by: Ankita Thomas <ankithom@redhat.com>
* fix sorting unpack jobs Signed-off-by: Ankita Thomas <ankithom@redhat.com> * Add test cases to reproduce panic Signed-off-by: Catherine Chan-Tse <cchantse@redhat.com> * handle nil jobs while sorting unpack jobs Signed-off-by: Ankita Thomas <ankithom@redhat.com> --------- Signed-off-by: Ankita Thomas <ankithom@redhat.com> Signed-off-by: Catherine Chan-Tse <cchantse@redhat.com> Co-authored-by: Catherine Chan-Tse <cchantse@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 20a3cbcde1f46ed7f4d56042ccc8b5d962e4d1b8
[release-4.16] OCPBUGS-36138: perform operator apiService certificate validity checks directly
Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 01b44e850bfae0d6775bb7379359b5eaf36f2778
Extends an existing unit test to cover the scenario where an old, failed job missing the label added in later OLM versions is not found with the filtered list call, resulting in an 'AlreadyExists' error when the new job is created. Signed-off-by: Daniel Franz <dfranz@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 8751ca0212e345a0b7eab452b169d926e7f42437
Signed-off-by: Jordan Keister <jordan@nimblewidget.com>
Signed-off-by: Jordan <jordan@nimblewidget.com>
[release-4.16] OCPBUGS-34979: Updates default security context behavior for catalog source pods
[release-4.16] OCPBUGS-36137: fix sorting unpack jobs
[release-4.16] OCPBUGS-25019: fix xplat compile for of-tools image; hide utest files from git; use rhel8 art builder image
[release-4.16] OCPBUGS-36450: Can't install operator on 4.15 after uninstalling it on a prior version
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 8615a10dcce46dadaa627916c76fbbf6dc47ad37
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 25aa03958865dc80693621b37e32405b7b3d4ce0
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
…ry-pick-922-to-release-4.16 [release-4.16] OCPBUGS-47738:catalog-operator: Delete Pods that were evicted (#3459)
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: api Upstream-commit: cbc560493dab9366e79e718ddfce2512ea81c813
Upstream-repository: operator-registry Upstream-commit: 936590e4d29adfc7c1c94eaa2a5dc84d49082614
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 6c8f9c33833389f4e44ffe56e7ac1151bcf7a84d
OCPBUGS-46928, OCPBUGS-46935, OCPBUGS-47315: x/net bump to v0.34.0 [release-4.16]
…n handler (#3483) * switch queue informer to use types.NamespacedName; relocate deletion handler Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * Add error message to object count assertion failure Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> --------- Signed-off-by: Joe Lanford <joe.lanford@gmail.com> Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com> Co-authored-by: Per Goncalves da Silva <pegoncal@redhat.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: edb751b9718ffe29e8462ce17867ec3db3b349ff
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
[release-4.16] OCPBUGS-48661: Fix concurrent namespace resolution
…to GRPC server (#3490) * add log and metric to instrument count of catalog source snapshots Signed-off-by: Joe Lanford <joe.lanford@gmail.com> * use operator cache provider for deprecation updates to limit calls to GRPC server Signed-off-by: Joe Lanford <joe.lanford@gmail.com> --------- Signed-off-by: Joe Lanford <joe.lanford@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 1274d54d885649786733d0b3fe499e9670f3310d
…non-determinism (#3491) Signed-off-by: Joe Lanford <joe.lanford@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: 451e775ea03eb9280399ba0abaf19bd237df167c
[release-4.16] OCPBUGS-48696: Fix excessive catalog source snapshots cause severe performance regression
…ry-pick-985-to-release-4.16 [release-4.16] OCPBUGS-53395: Ensure that PSA label is latest instead of pinning versions
Previously, nonconforming CRDs (missing the label) were logged as INFO, but this caused the to enter a CrashLoopBackOff state due to unhandled inconsistencies. This commit raises the log level to and enhances the message with actionable advice, clarifying that users should either delete CRDs for uninstalled solutions or label them appropriately. The root cause of the scenario is: From an old release (ocp 4.15) the component managed by OLM should have the label olm.managed: true added to comply with the new checks. Upstream-repository: operator-lifecycle-manager Upstream-commit: 1d9eafd4278ec97ea4caa1b6d56d5f901d550374
…rry-pick-1004-to-release-4.16 [release-4.16] OCPBUGS-56358: fix(olm): improve error logging for missing olm.managed label (#3558)
Signed-off-by: grokspawn <jordan@nimblewidget.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: bf9ffe8b04b59e9acd859ac8a9b6698f1f2a3437
…rry-pick-1019-to-release-4.16 [release-4.16] OCPBUGS-57429: reduce cache expiry frequency [release-4.19]
Signed-off-by: Anik Bhattacharjee <anbhatta@gmail.com> Upstream-repository: operator-registry Upstream-commit: 919aefdce3f4d6b77c3872df249815d20749db94
Signed-off-by: Rashmi Gottipati <chowdary.grashmi@gmail.com> Upstream-repository: operator-lifecycle-manager Upstream-commit: d7aaeb11ab80f862fc9c70dfb7865158bb67e28c
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
openshift-merge-robot
added
the
needs-rebase
Contributor
|
PR needs rebase. DetailsInstructions 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-sigs/prow repository. |
openshift-ci
Bot
added
the
do-not-merge/work-in-progress
Contributor
|
Skipping CI for Draft Pull Request. |
Contributor
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: perdasilva The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
Bot
added
the
approved
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
14 participants
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.
No description provided.