-
Notifications
You must be signed in to change notification settings - Fork 48
OCPBUGS-62517: Synchronize from Upstream Repositories #710
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
Merged
openshift-merge-bot
merged 104 commits into
openshift:main
from
openshift-bot:synchronize-upstream
May 1, 2026
Merged
Changes from all commits
Commits
Show all changes
104 commits
Select commit
Hold shift + click to select a range
0e70fe3
tests: address CodeRabbit review comments from downstream PR (#2672)
tmshort e12d90b
:bug: Fix catalogd ha readiness (#2674)
tmshort cadb7a7
:seedling: Bump github.com/google/go-containerregistry (#2677)
dependabot[bot] aa70192
:seedling: Bump marocchino/sticky-pull-request-comment (#2676)
dependabot[bot] ee337de
:seedling: Bump github.com/cert-manager/cert-manager (#2678)
dependabot[bot] 4ce4bab
adds support for parsing explicit `pkg.Release` field (#2543)
rashmigottipati 74e39f5
🐛 fix: allow reconciliation of deadline-exceeded ClusterObjectSets (#…
joelanford b177be3
fix(catalogd): follow-up fixes after HA PR merge (#2679)
tmshort 1312457
:seedling: Bump sigs.k8s.io/structured-merge-diff/v6 (#2681)
dependabot[bot] 0fae07a
:seedling: Bump github.com/containerd/containerd from 1.7.30 to 1.7.3…
dependabot[bot] 09ccf9e
:seedling: Bump packaging from 26.0 to 26.1 (#2683)
dependabot[bot] e0f05f9
fix(e2e): use per-component namespaces in HA and upgrade steps (#2685)
tmshort c8a978d
Merge branch 'main' into synchronize
644f51d
UPSTREAM: <carry>: Add OpenShift specific files
dtfranz 5623e00
UPSTREAM: <carry>: Add new tests for single/own namespaces install modes
camilamacedo86 c647696
UPSTREAM: <carry>: Upgrade OCP image from 4.20 to 4.21
camilamacedo86 ad6715c
UPSTREAM: <carry>: [Default Catalog Tests] - Change logic to get ocp …
camilamacedo86 281a037
UPSTREAM: <carry>: Update OCP catalogs to v4.21
tmshort f6a9328
UPSTREAM: <carry>: support singleown cases in disconnected
kuiwang02 7fb4682
UPSTREAM: <carry>: fix cases 81696 and 74618 for product code changes
kuiwang02 c4a1ec6
UPSTREAM: <carry>: Define Default timeouts and apply their usage accr…
camilamacedo86 cbe76a7
UPSTREAM: <carry>: Update to new feature-gate options in helm
tmshort 107ad99
UPSTREAM: <carry>: Fix flake for single/own ns tests by ensuring uniq…
camilamacedo86 beaf9e3
UPSTREAM: <carry>: [OTE]: Enhance single/own ns based on review comme…
camilamacedo86 a6f8a92
UPSTREAM: <carry>: Update OwnSingle template to use spec.config.inlin…
kuiwang02 1d0ddb3
UPSTREAM: <carry>: [OTE]: Add webhook cleanup validation on extension…
camilamacedo86 81fb8ef
UPSTREAM: <carry>: Add [OTP] to migrated cases
kuiwang02 f2cf2d0
UPSTREAM: <carry>: [OTE]: Upgrade dependencies used
camilamacedo86 3a89012
UPSTREAM: <carry>: fix(OTE): fix OpenShift Kubernetes replace version…
camilamacedo86 f2244e3
UPSTREAM: <carry>: [Default Catalog Tests] Upgrade go 1.24.6 and depe…
camilamacedo86 347614c
UPSTREAM: <carry>: add disconnected environment support with custom p…
kuiwang02 4164edd
UPSTREAM: <carry>: migrate jiazha test cases to OTE
jianzhangbjz 88389ad
UPSTREAM: <carry>: migrate clustercatalog case to ote
Xia-Zhao-rh ce45b6e
UPSTREAM: <carry>: migrate olmv1 QE stress cases
kuiwang02 0b848a6
UPSTREAM: <carry>: Use busybox/httpd to simulate probes
tmshort d4e07ba
UPSTREAM: <carry>: migrate olmv1 QE cases
Xia-Zhao-rh cdbe0d9
UPSTREAM: <carry>: add agent for olmv1 qe cases
kuiwang02 ac053c9
UPSTREAM: <carry>: Disable upstream PodDisruptionBudget
tmshort ce0340d
UPSTREAM: <carry>: Add AGENTS.md for AI code contributions
rashmigottipati c29705d
UPSTREAM: <carry>: address review comments through addl prompts
rashmigottipati c958dbe
UPSTREAM: <carry>: addressing some more review comments
rashmigottipati 0a592a0
UPSTREAM: <carry>: remove DCO line
rashmigottipati a853715
UPSTREAM: <carry>: migrate bandrade test cases to OTE
bandrade 1add9dc
UPSTREAM: <carry>: update metadata
bandrade a9b19de
UPSTREAM: <carry>: remove originalName
bandrade 29b9586
UPSTREAM: <carry>: update 80458's timeout to 180s
jianzhangbjz dd6e942
UPSTREAM: <carry>: update 83026 to specify the clustercatalog
jianzhangbjz 0b37871
UPSTREAM: <carry>: Update to golang 1.25 and ocp 4.22
oceanc80 6111a81
UPSTREAM: <carry>: Use oc client for running e2e tests
pedjak fb62521
UPSTREAM: <carry>: Run upstream e2e tests tagged with `@catalogd-update`
pedjak 78bacbe
UPSTREAM: <carry>: enhance case to make it more stable
kuiwang02 f111b87
UPSTREAM: <carry>: add service account to curl job
ehearne-redhat 92f4cdc
UPSTREAM: <carry>: move sa creation out of buildCurlJob()
ehearne-redhat e95eab7
UPSTREAM: <carry>: comment out delete service account
ehearne-redhat 8297b77
UPSTREAM: <carry>: move defercleanup for sa for LIFO
ehearne-redhat 04ea2c0
UPSTREAM: <carry>: add polling so job fully deleted before proceed
ehearne-redhat 145557a
UPSTREAM: <carry>: Revert "Merge pull request #594 from ehearne-redha…
sosiouxme b9ec9a7
UPSTREAM: <carry>: Remove openshift-redhat-marketplace catalog tests
camilamacedo86 464f015
UPSTREAM: <carry>: config watchnamespace cases
kuiwang02 43b4f62
UPSTREAM: <carry>: enhance ocp-79770
Xia-Zhao-rh d062f97
UPSTREAM: <carry>: upgrade version support case
kuiwang02 496a954
UPSTREAM: <carry>: Remove installed condition check from auth preflig…
4abce8f
UPSTREAM: <carry>: Add openshift/api dependency
d25c162
UPSTREAM: <carry>: Add boxcutter specific preflight auth test
e47eff5
UPSTREAM: <carry>: adjust watchnamespace case based on change
kuiwang02 1d81b46
UPSTREAM: <carry>: fix(ote): Use as operator-controller dep from root…
camilamacedo86 971bc07
UPSTREAM: <carry>: add 83979 automation
bandrade 0a05aef
UPSTREAM: <carry>: add 85889 automation
bandrade 318539f
UPSTREAM: <carry>: Update test-operator startup script to fix pod pro…
ff62660
UPSTREAM: <carry>: Fix up own-namespace invalid configuration test
a2a3836
UPSTREAM: <carry>: Preflight tests use in-cluster catalog and bundles…
camilamacedo86 b70b3b1
UPSTREAM: <carry>: adjust sa and permission test cases per new change…
kuiwang02 cf92e0e
UPSTREAM: <carry>: Update OCP catalogs to v4.22
camilamacedo86 b5a170e
UPSTREAM: <carry>: chore(OTE and Default Catalog Tests) Update go and…
camilamacedo86 3800687
UPSTREAM: <carry>: fix 83026 for TP cluster
jianzhangbjz 4cbfb17
UPSTREAM: <carry>: serviceAccount validation unified across all runtimes
kuiwang02 99e4f0e
UPSTREAM: <carry>: Fix OLMv1 test operator to listen on IPv6
stbenjam 3cbd0a0
UPSTREAM: <carry>: Increase install timeout and add diagnostic loggin…
camilamacedo86 2618267
UPSTREAM: <carry>: add service account to curl job
ehearne-redhat 18d6afa
UPSTREAM: <carry>: update OCP-75441 to support multi-arch
jianzhangbjz b4cfe92
UPSTREAM: <carry>: deployment config cases
kuiwang02 d7739de
UPSTREAM: <carry>: Add OTE tests for OLMv1 DeploymentConfig support
tmshort 9fbb7a3
UPSTREAM: <carry>: Update openshift/api and client-go
tmshort 77d2e9c
UPSTREAM: <carry>: Add boxcutter tests
camilamacedo86 1d8fe9b
UPSTREAM: <carry>: enhance QE cases
Xia-Zhao-rh a7d3f94
UPSTREAM: <carry>: Update quay-operator version to one containing arm…
dtfranz 5f52f80
UPSTREAM: <carry>: verify volume/volumeMount override
kuiwang02 211ab6e
UPSTREAM: <carry>: Add long-duration test script and documents
jianzhangbjz acb28b8
UPSTREAM: <carry>: Update grpc in default-catalog-consistency tests
tmshort 4b37eb3
UPSTREAM: <carry>: Rename ClusterExtensionRevision to ClusterObjectSe…
camilamacedo86 ce512a5
UPSTREAM: <carry>: Skip incompatible operator test when Boxcutter use…
camilamacedo86 b5a535c
UPSTREAM: <carry>: add ocp-87557
bandrade cab6b58
UPSTREAM: <carry>: Add fgiudici as reviewer
fgiudici cf0a5c0
UPSTREAM: <carry>: Remove skip for incompatible operator check after …
camilamacedo86 5a2c66b
UPSTREAM: <carry>: Test empty affinity erasure and cleanup
kuiwang02 a13d640
UPSTREAM: <carry>: Fix boxcutter finalizer ResourceNames in prefligh…
camilamacedo86 43ac099
UPSTREAM: <carry>: Expand OTE docs with more comprehensive details
camilamacedo86 eed0be2
UPSTREAM: <carry>: Disable upstream TLSProfile tests
tmshort 9023ebe
UPSTREAM: <carry>: OTE: Simplify by remove option to configure tests …
camilamacedo86 eb89728
UPSTREAM: <carry>: OTE - Make OTE local output easier to read
camilamacedo86 0ef8b5d
UPSTREAM: <carry>: remove dead e2e registry push job and related vari…
joelanford fcea10f
UPSTREAM: <drop>: go mod vendor
9f2702c
UPSTREAM: <drop>: remove upstream GitHub configuration
e048679
UPSTREAM: <drop>: configure the commit-checker
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.
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.
MaxLength=20introduces a schema/runtime mismatch that can block reconciliation.Line 489 caps
status.install.bundle.releaseto 20 chars, but release parsing/propagation paths accept valid longer values. A valid release string over 20 chars would fail status validation at write time instead of being handled deterministically earlier.💡 Proposed fix (align API with current parser behavior)
- // +kubebuilder:validation:MaxLength=20 // +kubebuilder:validation:XValidation:rule="self.matches(\"^$|^(0|[1-9][0-9]*|[0-9]*[A-Za-z-][0-9A-Za-z-]*)(\\\\.(0|[1-9][0-9]*|[0-9]*[A-Za-z-][0-9A-Za-z-]*))*$\")",message="release must be empty or consist of dot-separated identifiers (numeric without leading zeros, or alphanumeric)" Release *string `json:"release,omitempty"`If the 20-char cap is required, enforce that same limit at all ingestion points before status assignment, and return a clear validation error there.
🤖 Prompt for AI Agents
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.
@grokspawn @rashmigottipati do we care about this?
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.
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.
This is already consistently handled in each location where a release value can be generated EXCEPT for the legacy build-metadata approach in the legacy handling, but SEPARATELY we have already verified that all available catalog metadata which uses this approach is acceptable to these limits (operator-framework/operator-registry#1852).
We no longer 'grandfather' build metadata approaches to infer release versions, nor do we allow new submissions using that approach to SQLite/legacy catalog formats, so I'm reasonably confident we've bolted the doors and locked the windows here.