Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Correct s3 access snippets#939

Open
oliviermeslin wants to merge 1 commit intoInseeFrLab:mainfrom
oliviermeslin:fix_s3_access_snippets
Open

Correct s3 access snippets#939
oliviermeslin wants to merge 1 commit intoInseeFrLab:mainfrom
oliviermeslin:fix_s3_access_snippets

Conversation

Copy link

oliviermeslin commented Mar 3, 2025

Hi Onyxia team,

The S3 access snippets in R with R (aws.s3 and paws) were not working because listing buckets on S3 storages with many buckets does not seem to work for some reason. I propose a correction, along with a few comments.

Copy link

sonarqubecloud bot commented Mar 3, 2025

Copy link
Author

oliviermeslin commented Mar 3, 2025

@garronej , @ddecrulle : I'm quite aware that username <- gsub("user-", "", Sys.getenv("KUBERNETES_NAMESPACE")) is not the proper way to retrieve the username (and it won't work anyway for project namespaces). But I really think it is useful to show how to list the content of a bucket (get_bucket(username, region = "")). Do you know how to do it in a less hacky way?

Copy link
Contributor

fcomte commented Mar 5, 2025

Hello ,

We don't like those script in the opensource project.
We can only merge the removal of the bucket list function.
Sorry

garronej force-pushed the main branch from a16d955 to 6dd222d Compare March 28, 2025 15:43
ddecrulle force-pushed the main branch from 0a69285 to d42d38e Compare June 6, 2025 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants