Skip to content

NO-JIRA: Remove ChunkSizeMib featuregate#1240

Open
deepsm007 wants to merge 2 commits intoopenshift:mainfrom
deepsm007:remove-chunksizemib-var
Open

NO-JIRA: Remove ChunkSizeMib featuregate#1240
deepsm007 wants to merge 2 commits intoopenshift:mainfrom
deepsm007:remove-chunksizemib-var

Conversation

@deepsm007
Copy link
Contributor

@openshift-ci openshift-ci bot requested a review from flavianmissi August 5, 2025 16:01
@deepsm007 deepsm007 force-pushed the remove-chunksizemib-var branch 2 times, most recently from b349985 to 641b9d7 Compare August 6, 2025 00:54
@deepsm007 deepsm007 force-pushed the remove-chunksizemib-var branch 3 times, most recently from 74bd59a to 9ae0bc1 Compare August 18, 2025 17:55
@deepsm007
Copy link
Contributor Author

/retest

1 similar comment
@flavianmissi
Copy link
Member

/retest

@deepsm007
Copy link
Contributor Author

/retest-required

@deepsm007
Copy link
Contributor Author

/retest

@deepsm007 deepsm007 force-pushed the remove-chunksizemib-var branch from 340c2b6 to edba1e4 Compare October 20, 2025 16:26
@deepsm007
Copy link
Contributor Author

I have rebased the PR, the previous failures were not related to the code changes. Hypershift test failed due to infrastructure issue, retesting again.

@deepsm007
Copy link
Contributor Author

/retest

@deepsm007 deepsm007 force-pushed the remove-chunksizemib-var branch from edba1e4 to 41c150c Compare October 21, 2025 14:55
@deepsm007
Copy link
Contributor Author

/retest

@flavianmissi
Copy link
Member

/approve
/retest

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 22, 2025
@deepsm007
Copy link
Contributor Author

/retest-required

@deepsm007
Copy link
Contributor Author

/test e2e-vsphere-operator e2e-aws-operator

@flavianmissi
Copy link
Member

the e2e-*-operator tests are flaking so much that at this point they're nearly perma-failing, and this isn't the case for this PR only, it's happening across the board.
I'm going to override the required tests so we can unblock this PR.

/override ci/prow/e2e-aws-operator
/override ci/prow/e2e-vsphere-operator

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 24, 2025

@flavianmissi: Overrode contexts on behalf of flavianmissi: ci/prow/e2e-aws-operator, ci/prow/e2e-vsphere-operator

Details

In response to this:

the e2e-*-operator tests are flaking so much that at this point they're nearly perma-failing, and this isn't the case for this PR only, it's happening across the board.
I'm going to override the required tests so we can unblock this PR.

/override ci/prow/e2e-aws-operator
/override ci/prow/e2e-vsphere-operator

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-sigs/prow repository.

@deepsm007 deepsm007 changed the title Remove ChunkSizeMib featuregate [No-Jira]: Remove ChunkSizeMib featuregate Oct 24, 2025
@deepsm007 deepsm007 force-pushed the remove-chunksizemib-var branch from 41c150c to 503ccc8 Compare October 27, 2025 15:29
@flavianmissi
Copy link
Member

/override ci/prow/e2e-aws-operator
/override ci/prow/e2e-vsphere-operator

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 28, 2025

@flavianmissi: Overrode contexts on behalf of flavianmissi: ci/prow/e2e-aws-operator, ci/prow/e2e-vsphere-operator

Details

In response to this:

/override ci/prow/e2e-aws-operator
/override ci/prow/e2e-vsphere-operator

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-sigs/prow repository.

@flavianmissi
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 28, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 28, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deepsm007, flavianmissi

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

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Oct 29, 2025
@openshift-ci-robot
Copy link
Contributor

@xiuwang: This PR has been marked as verified by @xiuwang.

Details

In response to this:

/verified by @xiuwang

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 2212ae9 and 2 for PR HEAD 503ccc8 in total

@deepsm007
Copy link
Contributor Author

@xiuwang can you override the one failing test? #1240 (comment) @flavianmissi is not around for several weeks now

@xiuwang
Copy link

xiuwang commented Nov 4, 2025

/retest-required

@xiuwang
Copy link

xiuwang commented Nov 4, 2025

/override ci/prow/e2e-gcp-operator
/override ci/prow/e2e-azure-operator
/override ci/prow/e2e-vsphere-operator

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 4, 2025

@xiuwang: xiuwang unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight openshift-staff-engineers openshift-sustaining-engineers.

Details

In response to this:

/override ci/prow/e2e-gcp-operator
/override ci/prow/e2e-azure-operator
/override ci/prow/e2e-vsphere-operator

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-sigs/prow repository.

@xiuwang
Copy link

xiuwang commented Nov 4, 2025

@deepsm007 I have no permission to do override
@ricardomaraschini Could you help do override in comment ?

@ricardomaraschini
Copy link
Contributor

@deepsm007 I am concerned with the failing tests. Can you please see what is going on with those ? We seem to have unit tests failing as well.

@deepsm007
Copy link
Contributor Author

@deepsm007 I am concerned with the failing tests. Can you please see what is going on with those ? We seem to have unit tests failing as well.

I see a lot of connection refused for all 3 failures

Also #1240 (comment), i will spend some time investigating further. And also #1240 (comment)

@flavianmissi
Copy link
Member

TestScaleUp is basically perma-failing now :/ and TestS3Minio is really flaky too, we might have to override.

/retest-required

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 86f6c08 and 1 for PR HEAD 503ccc8 in total

@flavianmissi
Copy link
Member

unfortunately the e2e-.*-operator tests seem to be perma-failing.

/override ci/prow/e2e-vsphere-operator

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 1, 2025

@flavianmissi: Overrode contexts on behalf of flavianmissi: ci/prow/e2e-vsphere-operator

Details

In response to this:

unfortunately the e2e-.*-operator tests seem to be perma-failing.

/override ci/prow/e2e-vsphere-operator

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-sigs/prow repository.

@flavianmissi
Copy link
Member

/retest-required

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 2, 2025
@deepsm007 deepsm007 force-pushed the remove-chunksizemib-var branch from 503ccc8 to 282878f Compare December 2, 2025 13:34
@openshift-ci-robot openshift-ci-robot removed the verified Signifies that the PR passed pre-merge verification criteria label Dec 2, 2025
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Dec 2, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 2, 2025

New changes are detected. LGTM label has been removed.

@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 2, 2025
@deepsm007 deepsm007 force-pushed the remove-chunksizemib-var branch from 282878f to 49bccf7 Compare December 2, 2025 14:08
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 2, 2025

@deepsm007: 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-azure-operator 49bccf7 link false /test e2e-azure-operator
ci/prow/e2e-vsphere-operator 49bccf7 link true /test e2e-vsphere-operator
ci/prow/e2e-gcp-operator 49bccf7 link false /test e2e-gcp-operator
ci/prow/e2e-openstack 49bccf7 link false /test e2e-openstack

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-sigs/prow repository. I understand the commands that are listed here.

@openshift-bot
Copy link
Contributor

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci openshift-ci bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 3, 2026
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 3, 2026
@openshift-merge-robot
Copy link
Contributor

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-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. docs-approved Signifies that Docs has signed off on this PR jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. px-approved Signifies that Product Support has signed off on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants