Skip to content

Merge https://github.com/openshift/oadp-operator:oadp-dev (513180a) into oadp-dev #1990

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.

Sign up for GitHub

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

Jump to bottom
Merged
kaovilai merged 3 commits into from
Nov 5, 2025
Merged

Merge https://github.com/openshift/oadp-operator:oadp-dev (513180a) into oadp-dev #1990

kaovilai merged 3 commits into from
Nov 5, 2025

Conversation

@oadp-rebasebot-app
Copy link
Contributor

@oadp-rebasebot-app oadp-rebasebot-app bot commented Oct 9, 2025

No description provided.

@openshift-ci openshift-ci bot requested review from kaovilai and mrnold October 9, 2025 17:39
@openshift-ci openshift-ci bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Oct 9, 2025
@openshift-ci
Copy link

openshift-ci bot commented Oct 9, 2025

Hi @oadp-rebasebot-app[bot]. Thanks for your PR.

I'm waiting for a github.com member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

kaovilai
kaovilai previously approved these changes Oct 9, 2025
View reviewed changes
@kaovilai
Copy link
Member

kaovilai commented Oct 9, 2025

/ok-to-test

@openshift-ci openshift-ci bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. approved Indicates a PR has been approved by an approver from all required OWNERS files. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Oct 9, 2025
weshayutin
weshayutin previously approved these changes Oct 10, 2025
View reviewed changes
@mpryc
Copy link
Contributor

mpryc commented Oct 10, 2025

/hold - this is in must-gather, probably we need to rip out this folder, still the tests relay on it?

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 10, 2025
@weshayutin
Copy link
Contributor

weshayutin commented Oct 10, 2025

@mpryc perhaps for the time being.. we SKIP_MUST_GATHER ?= true

@kaovilai
Copy link
Member

kaovilai commented Oct 10, 2025

the test does run mustgather.. its one of the requirements it may need updates

@oadp-rebasebot-app oadp-rebasebot-app bot changed the title Merge https://github.com/openshift/oadp-operator:oadp-dev (790106e) into oadp-dev Merge https://github.com/openshift/oadp-operator:oadp-dev (b296ca1) into oadp-dev Oct 15, 2025
@oadp-rebasebot-cloner oadp-rebasebot-cloner bot dismissed stale reviews from weshayutin and kaovilai via 50e261b October 15, 2025 08:42
@openshift-ci openshift-ci bot removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 15, 2025
@oadp-rebasebot-app oadp-rebasebot-app bot changed the title Merge https://github.com/openshift/oadp-operator:oadp-dev (b296ca1) into oadp-dev Merge https://github.com/openshift/oadp-operator:oadp-dev (5c1e63f) into oadp-dev Oct 17, 2025
@oadp-rebasebot-app oadp-rebasebot-app bot changed the title Merge https://github.com/openshift/oadp-operator:oadp-dev (5c1e63f) into oadp-dev Merge https://github.com/openshift/oadp-operator:oadp-dev (44d4afc) into oadp-dev Oct 22, 2025
@oadp-rebasebot-cloner oadp-rebasebot-cloner bot force-pushed the rebase-bot-oadp-dev branch 2 times, most recently from 4cfe228 to bc21ecb Compare October 26, 2025 03:17
@oadp-rebasebot-app oadp-rebasebot-app bot changed the title Merge https://github.com/openshift/oadp-operator:oadp-dev (44d4afc) into oadp-dev Merge https://github.com/openshift/oadp-operator:oadp-dev (513180a) into oadp-dev Oct 29, 2025
@oadp-rebasebot-cloner oadp-rebasebot-cloner bot force-pushed the rebase-bot-oadp-dev branch 2 times, most recently from f0e872a to 8e7e6cb Compare October 29, 2025 16:18
@mpryc
Copy link
Contributor

mpryc commented Oct 29, 2025

/ok-to-test

@weshayutin
Copy link
Contributor

weshayutin commented Oct 29, 2025

/retest

@weshayutin
Copy link
Contributor

weshayutin commented Oct 29, 2025

temp until I can ensure this is more stable #2002

weshayutin
weshayutin previously approved these changes Nov 5, 2025
View reviewed changes
@kaovilai
Copy link
Member

kaovilai commented Nov 5, 2025

/retest

ai-retester: The Mongo application CSI test timed out after 540 seconds because the todolist pod was stuck in a PodInitializing state. The underlying cause isn't immediately obvious, but the pod never became ready.

The e2e test e2e-test-aws failed because the "Mongo application CSI" test timed out after 540 seconds, and the todolist pod remained in PodInitializing state, indicating issues with pod initialization.

The failure was in the e2e-test-cli-aws test, specifically the Mongo application CSI via CLI test. The tests/e2e/backup_restore_cli_suite_test.go timed out waiting for a pod to succeed. The container "test" exited with code 2, reason Error

@kaovilai
Copy link
Member

kaovilai commented Nov 5, 2025

/retest

ai-retester: The e2e-test-cli-aws test failed because the Mongo application CSI via CLI test timed out after 540 seconds. The todolist pod was stuck in the "PodInitializing" state and never became ready.

The e2e-test-aws test failed because the Mongo application CSI test timed out while waiting for a pod to succeed, indicating a problem with the backup and restore process of a MongoDB application using CSI snapshots. The container todolist in pod todolist-6d7bb9554c-mf9bn was stuck in PodInitializing state.

The e2e-test-aws step failed because the Mongo application CSI test timed out after 540 seconds waiting for a Pod to succeed. The test container exited with code 2.

@oadp-rebasebot-cloner oadp-rebasebot-cloner bot dismissed stale reviews from weshayutin and kaovilai via b039f8b November 5, 2025 03:17
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 5, 2025
@kaovilai
Copy link
Member

kaovilai commented Nov 5, 2025

/retest

ai-retester: The end-to-end test e2e-test-aws failed because the "Mongo application CSI" test timed out and a container "todolist" in pod "todolist-6d7bb9554c-mf9bn" was waiting to start due to PodInitializing, implying issues deploying or accessing the MongoDB application.

@kaovilai
Copy link
Member

kaovilai commented Nov 5, 2025

only mongo csi failed overriding due to consensus on this case removal #2002
/override ci/prow/4.20-e2e-test-aws ci/prow/4.20-e2e-test-hcp-aws

@openshift-ci
Copy link

openshift-ci bot commented Nov 5, 2025

@kaovilai: Overrode contexts on behalf of kaovilai: ci/prow/4.20-e2e-test-aws, ci/prow/4.20-e2e-test-hcp-aws

In response to this:

only mongo csi failed overriding due to consensus on this case removal #2002
/override ci/prow/4.20-e2e-test-aws ci/prow/4.20-e2e-test-hcp-aws

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.

@mpryc
Copy link
Contributor

mpryc commented Nov 5, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 5, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 5, 2025

/retest-required

Remaining retests: 0 against base HEAD 513180a and 2 for PR HEAD 5747e96 in total

@kaovilai
Copy link
Member

kaovilai commented Nov 5, 2025

/retest

ai-retester: The e2e test Mongo application CSI timed out and failed because a pod never reached a succeeded state, likely related to a 'PodInitializing' status issue.

The e2e tests for the OADP CLI failed because the Mongo application CSI via CLI test timed out when waiting for a pod to become ready, and the test suite itself returned a non-zero exit code.

The Mongo application CSI e2e test timed out and failed due to a PodInitializing error. The todolist container in the specified pod never became ready.

@kaovilai
Copy link
Member

kaovilai commented Nov 5, 2025

/override ci/prow/4.20-e2e-test-aws ci/prow/4.20-e2e-test-hcp-aws ci/prow/4.19-e2e-test-kubevirt-aws ci/prow/4.19-e2e-test-aws

@openshift-ci
Copy link

openshift-ci bot commented Nov 5, 2025

@kaovilai: Overrode contexts on behalf of kaovilai: ci/prow/4.19-e2e-test-aws, ci/prow/4.19-e2e-test-kubevirt-aws, ci/prow/4.20-e2e-test-aws, ci/prow/4.20-e2e-test-hcp-aws

In response to this:

/override ci/prow/4.20-e2e-test-aws ci/prow/4.20-e2e-test-hcp-aws ci/prow/4.19-e2e-test-kubevirt-aws ci/prow/4.19-e2e-test-aws

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.

@kaovilai
Copy link
Member

kaovilai commented Nov 5, 2025

/override ci/prow/4.19-e2e-test-cli-aws

@openshift-ci
Copy link

openshift-ci bot commented Nov 5, 2025

@kaovilai: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • [ci/prow/4.19-e2e-test-cli-aws](https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_oadp-operator/1990/pull-ci-openshift-oadp-operator-oadp-dev-4.19-e2e-test-cli-aws/1986062536658653184)

Only the following failed contexts/checkruns were expected:

  • CodeRabbit
  • ci/prow/4.19-ci-index
  • ci/prow/4.19-e2e-test-aws
  • ci/prow/4.19-e2e-test-cli-aws
  • ci/prow/4.19-e2e-test-hcp-aws
  • ci/prow/4.19-e2e-test-kubevirt-aws
  • ci/prow/4.19-images
  • ci/prow/4.20-ci-index
  • ci/prow/4.20-e2e-test-aws
  • ci/prow/4.20-e2e-test-hcp-aws
  • ci/prow/4.20-e2e-test-kubevirt-aws
  • ci/prow/4.20-images
  • ci/prow/images
  • ci/prow/unit-test
  • pull-ci-openshift-oadp-operator-oadp-1.0-images
  • pull-ci-openshift-oadp-operator-oadp-1.0-unit-test
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.19-ci-index
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.19-e2e-test-aws
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.19-e2e-test-hcp-aws
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.19-e2e-test-kubevirt-aws
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.19-images
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.20-ci-index
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.20-e2e-test-aws
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.20-e2e-test-hcp-aws
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.20-e2e-test-kubevirt-aws
  • pull-ci-openshift-oadp-operator-oadp-1.5-4.20-images
  • pull-ci-openshift-oadp-operator-oadp-dev-4.19-e2e-test-cli-aws
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override ci/prow/4.19-e2e-test-cli-aws

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.

@kaovilai
Copy link
Member

kaovilai commented Nov 5, 2025

/override ci/prow/4.19-e2e-test-cli-aws

@openshift-ci
Copy link

openshift-ci bot commented Nov 5, 2025

@kaovilai: Overrode contexts on behalf of kaovilai: ci/prow/4.19-e2e-test-cli-aws

In response to this:

/override ci/prow/4.19-e2e-test-cli-aws

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.

@openshift-ci
Copy link

openshift-ci bot commented Nov 5, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kaovilai, mpryc, oadp-rebasebot-app[bot]

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

The pull request process is described here

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

@kaovilai kaovilai merged commit 5048a8f into openshift:oadp-dev Nov 5, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kaovilai kaovilai kaovilai approved these changes

@mpryc mpryc mpryc approved these changes

@weshayutin weshayutin weshayutin left review comments

@mrnold mrnold Awaiting requested review from mrnold

Assignees

@kaovilai kaovilai

@mpryc mpryc

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kaovilai @mpryc @weshayutin @openshift-ci-robot @oadp-maintainers