Skip to content
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.

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

Support s3:// protocol #176

Open
seanorama opened this issue Aug 9, 2022 · 3 comments
Open

Support s3:// protocol #176

seanorama opened this issue Aug 9, 2022 · 3 comments
Labels
carvel-accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request hacktoberfest Hacktoberfest issue priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.

Comments

@seanorama
Copy link

Describe the problem/challenge you have

Need to fetch artifacts from AWS S3 buckets.

We, and our vendors, distribute artifacts with S3. Some buckets are secured so can't be accessed anonymously via the https:// path.

Describe the solution you'd like

Support s3:// like this:

apiVersion: vendir.k14s.io/v1alpha1
kind: Config
directories:
- path: vendor
  contents:
  - path: my-package
    s3:
      url: s3://bucket-name/packages/my-package.tgz

It should use the AWS SDK, such that it automatically uses the authentication from the environment (variables, credential file, credential process, instance metadata, ...).


Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@seanorama seanorama added carvel-triage This issue has not yet been reviewed for validity enhancement This issue is a feature request labels Aug 9, 2022
@cppforlife
Copy link
Contributor

@seanorama that sounds like a good feature. any interest in giving it a shot?

@cppforlife cppforlife added carvel-accepted This issue should be considered for future work and that the triage process has been completed and removed carvel-triage This issue has not yet been reviewed for validity labels Aug 11, 2022
@voor
Copy link

voor commented Nov 10, 2022

This came up today in conversation, it was more of a "oh, I guess it doesn't support this" rather than "it needs to support this" -- just wanted to capture the metric.

@seanorama
Copy link
Author

seanorama commented Nov 15, 2022

@seanorama that sounds like a good feature. any interest in giving it a shot?

That's outside my skillset, sorry.

@neil-hickey neil-hickey added the priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. label Feb 22, 2023
@joaopapereira joaopapereira added the hacktoberfest Hacktoberfest issue label Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
carvel-accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request hacktoberfest Hacktoberfest issue priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Projects
Status: Unprioritized
Development

No branches or pull requests

5 participants