Skip to content

Feat s3 share externally #8710

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
Y0Coss merged 8 commits into from
Dec 19, 2025
Merged

Feat s3 share externally #8710

Y0Coss merged 8 commits into from
Dec 19, 2025

Conversation

@OvhValentin
Copy link
Contributor

@OvhValentin OvhValentin commented Nov 18, 2025 •
edited
Loading

What type of Pull Request is this?

  • New guide(s)
  • Index categories update

Description

Refer to SK-2380

Mandatory information

The translations in this Pull Request have been done using:

  • OVHcloud integrated translation LLM
  • Systran
  • Other tool (specify which tool was used)
  • This Pull Request didn't require any translation.
  • This Pull Request can be merged as soon as possible.
  • This Pull Request shouldn't be merged before: YYYY-MM-DD
  • This Pull Request content should be replicated for the US OVHcloud documentation : YES/NO

@OvhValentin OvhValentin self-assigned this Nov 18, 2025
@OvhValentin OvhValentin added the Guide creation The Pull Request contains at least 1 new guide (meta.yaml and index edition needed) label Nov 18, 2025
@OvhValentin OvhValentin marked this pull request as draft November 18, 2025 10:39
@tcpdumpfbacke tcpdumpfbacke self-assigned this Nov 24, 2025
@agoude agoude requested review from agoude and lason-ovh December 2, 2025 10:35
lason-ovh
lason-ovh previously requested changes Dec 3, 2025
View reviewed changes
agoude
agoude previously requested changes Dec 15, 2025
View reviewed changes
Copy link
Contributor

@agoude agoude left a comment

Choose a reason for hiding this comment

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

I would finally remove the part/section on bucket policies to avoid confusion. Thx
Can you remove it? Rest is fine on my side!

@OvhValentin OvhValentin requested a review from agoude December 16, 2025 15:03
@OvhValentin OvhValentin marked this pull request as ready for review December 16, 2025 15:03
@Y0Coss
Copy link
Collaborator

Y0Coss commented Dec 16, 2025

@OvhValentin , please edit the index file to add this new guide.

@Y0Coss Y0Coss assigned Y0Coss and unassigned tcpdumpfbacke and OvhValentin Dec 19, 2025
@Y0Coss Y0Coss merged commit 324eb54 into develop Dec 19, 2025
@Y0Coss Y0Coss deleted the feat-s3-share-externally branch December 19, 2025 07:41
@Y0Coss
Copy link
Collaborator

Y0Coss commented Dec 19, 2025

Thank you for your contribution @OvhValentin

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

Reviewers

@lason-ovh lason-ovh lason-ovh left review comments

@agoude agoude Awaiting requested review from agoude

Assignees

@Y0Coss Y0Coss

Labels

Guide creation The Pull Request contains at least 1 new guide (meta.yaml and index edition needed)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@OvhValentin @Y0Coss @agoude @lason-ovh @tcpdumpfbacke