Skip to content

Merge https://github.com/openshift/hypershift-oadp-plugin:main (fd95e11) into main - #298

Open
oadp-rebasebot-app[bot] wants to merge 1 commit into
openshift:mainfrom
oadp-rebasebot:rebase-bot-main
Open

Merge https://github.com/openshift/hypershift-oadp-plugin:main (fd95e11) into main#298
oadp-rebasebot-app[bot] wants to merge 1 commit into
openshift:mainfrom
oadp-rebasebot:rebase-bot-main

Conversation

@oadp-rebasebot-app

@oadp-rebasebot-app oadp-rebasebot-app Bot commented Jul 21, 2026

Copy link
Copy Markdown

This is an automated rebase PR generated by RebaseBot.

Summary

  • Source: https://github.com/openshift/hypershift-oadp-plugin:main
  • Destination: https://github.com/openshift/hypershift-oadp-plugin:main
  • 0 new upstream commits

Summary by CodeRabbit

  • Chores
    • Updated Velero component versions and replacements to align with the latest supported release.
    • Improved dependency consistency across Velero packages.

@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: beb613a4-33ac-4b26-9670-53eca7199cc2

📥 Commits

Reviewing files that changed from the base of the PR and between 6e4984b and dfc66dc.

⛔ Files ignored due to path filters (2)
  • go.sum is excluded by !**/*.sum
  • vendor/modules.txt is excluded by !**/vendor/**, !vendor/**
📒 Files selected for processing (1)
  • go.mod
🚧 Files skipped from review as they are similar to previous changes (1)
  • go.mod

Walkthrough

The Go module configuration changes the Velero API requirement and maps the VMware Velero modules to corresponding OpenShift Velero modules at a newer pseudo-version.

Changes

Velero module resolution

Layer / File(s) Summary
Update Velero module requirements and replacements
go.mod
Changes the Velero API requirement to v0.0.0 and replaces the VMware Velero modules with corresponding OpenShift Velero modules at version v0.10.2-0.20260805134104-2091b09b93bb.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Mergeability Score: ⚪ Minimal · up to dfc66

This automated rebase contains no new upstream commits, and no concrete correctness, security, availability, deployment, or readiness issue is identified; no actionable merge-blocking risk remains beyond normal checks.

Suggested reviewers: enxebre, kaovilai, sjenning

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the automated merge of the upstream repository branch into main.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed The PR changes Go module metadata and vendored Velero code only; the patch adds no Ginkgo test-title calls or dynamic test names.
Test Structure And Quality ✅ Passed The patch changes go.mod, go.sum, and vendored Velero files only; it modifies 0 *_test.go files and adds no Ginkgo assertions or waits.
Microshift Test Compatibility ✅ Passed The diff from base adds no *_test.go files or Ginkgo declarations. It only changes go.mod, go.sum, and vendored Velero code, so the check does not apply.
Single Node Openshift (Sno) Test Compatibility ✅ Passed The pull request changes go.mod, go.sum, and vendored Velero code only; no new Ginkgo e2e test declarations or test files were added.
Topology-Aware Scheduling Compatibility ✅ Passed The commit changes dependency metadata and vendored Velero code only; no deployment manifests, operator/controllers, or scheduling constraints were added or modified.
Ote Binary Stdout Contract ✅ Passed The PR changes only dependencies and Velero API/vendor code; main.go and framework Serve/logger code are unchanged, and framework logging uses logrus's stderr default.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The commit changes go.mod, go.sum, vendor metadata, and Velero source files; it adds no Ginkgo tests or test declarations, so this check is not applicable.
No-Weak-Crypto ✅ Passed The diff updates Velero metadata and data-mover/API code only; added lines contain no MD5, SHA1, DES, RC4, Blowfish, ECB, custom crypto, or secret comparisons.
Container-Privileges ✅ Passed The PR changes Go module metadata and vendored Go APIs only. No manifest or container privilege setting is added, and existing Dockerfiles use non-root USER values.
No-Sensitive-Data-In-Logs ✅ Passed The diff adds no logging calls or sensitive fields; added output is enum-validation errors for dataMover values, not passwords, tokens, keys, PII, hostnames, or customer data.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@openshift-ci
openshift-ci Bot requested review from Joeavaikath and kaovilai July 21, 2026 20:57
@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 Jul 21, 2026
@openshift-ci

openshift-ci Bot commented Jul 21, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: oadp-rebasebot-app[bot]
Once this PR has been reviewed and has the lgtm label, please assign sjenning for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

openshift-ci Bot commented Jul 21, 2026

Copy link
Copy Markdown

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

I'm waiting for a openshift 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.

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.

@oadp-rebasebot-app oadp-rebasebot-app Bot changed the title Merge https://github.com/openshift/hypershift-oadp-plugin:main (f8c5bfb) into main Merge https://github.com/openshift/hypershift-oadp-plugin:main (fd95e11) into main Jul 28, 2026
@redhat-chai-bot

Copy link
Copy Markdown

/ok-to-test


AI-generated. Review for accuracy.

@openshift-ci openshift-ci Bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Aug 5, 2026
@openshift-ci

openshift-ci Bot commented Aug 13, 2026

Copy link
Copy Markdown

@oadp-rebasebot-app[bot]: all tests passed!

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.

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

Labels

ok-to-test Indicates a non-member PR verified by an org member that is safe to test.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants