Skip to content

Conversation

akarol
Copy link
Contributor

@akarol akarol commented Oct 7, 2025

  • Added comprehensive S3CERTIFY.md guide for S3-compatible storage certification

@openshift-ci openshift-ci bot requested review from mrnold and sseago October 7, 2025 12:43
Copy link
Contributor

@weshayutin weshayutin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great, let's just add our emails and I'll get it merged up :)
THANK YOU!

S3CERTIFY.md Outdated

For questions or to begin the S3 certification process:

- **OADP Team**: [[email protected]](mailto:[email protected])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have added my and Wes's email addresses.

This document outlines the general requirements, access needs, and steps involved in certifying an **S3-compatible object storage product** for use with **OADP (OpenShift API for Data Protection)**.

> ⚠️ **Important**: This certification process ensures your S3-compatible storage solution meets the reliability and compatibility standards required for production OpenShift backup and restore operations.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also link our current docs where we list the supported/certified s3 compatible backup storage providers list ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can add that reference here.

@akarol
Copy link
Contributor Author

akarol commented Oct 8, 2025

looks great, let's just add our emails and I'll get it merged up :) THANK YOU!

looks great, let's just add our emails and I'll get it merged up :) THANK YOU!

--- Email address added!

@weshayutin
Copy link
Contributor

/cherry-pick oadp-1.5

@openshift-cherrypick-robot
Copy link
Contributor

@weshayutin: once the present PR merges, I will cherry-pick it on top of oadp-1.5 in a new PR and assign it to you.

In response to this:

/cherry-pick oadp-1.5

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.

- Add comprehensive S3CERTIFY.md guide for S3-compatible storage certification
- Include access requirements, credentials, endpoint configuration
- Add support and communication guidelines
- Provide certification timeline and process overview
- Include quick checklist and contact information
- Document applies to OADP 1.3+ and OpenShift 4.12+
@weshayutin
Copy link
Contributor

/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 15, 2025
Copy link
Member

@kaovilai kaovilai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

may the managers write lgtm when they are ready

@shubham-pampattiwar
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 15, 2025
@weshayutin weshayutin merged commit 382fa92 into openshift:oadp-dev Oct 15, 2025
7 checks passed
Copy link

openshift-ci bot commented Oct 15, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: akarol, kaovilai, shubham-pampattiwar, weshayutin

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:
  • OWNERS [kaovilai,shubham-pampattiwar]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-cherrypick-robot
Copy link
Contributor

@weshayutin: new pull request created: #1996

In response to this:

/cherry-pick oadp-1.5

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.

Copy link

openshift-ci bot commented Oct 15, 2025

@akarol: all tests passed!

Full PR test history. Your PR dashboard.

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

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants