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

Added S3 support for pelican quickstart. #775

Merged
merged 1 commit into from Mar 24, 2013

Conversation

Projects
None yet
4 participants
@lexual
Contributor

lexual commented Mar 15, 2013

Now users can do:

$ make s3_upload

And it will be uploaded to s3 via s3cmd.

@almet

This comment has been minimized.

Show comment
Hide comment
@almet

almet Mar 22, 2013

Member

That's actually something we need to discuss with the whole team: do we want to support every single option in the pelican codebase, or do we prefer to not go this path?

What do you guys think? @bbinet @tbunnyman @justinmayer

Member

almet commented Mar 22, 2013

That's actually something we need to discuss with the whole team: do we want to support every single option in the pelican codebase, or do we prefer to not go this path?

What do you guys think? @bbinet @tbunnyman @justinmayer

@bbinet

This comment has been minimized.

Show comment
Hide comment
@bbinet

bbinet Mar 22, 2013

Contributor

These changes are related to the makefile only, so I don't mind supporting every single option in the pelican makefile.

Contributor

bbinet commented Mar 22, 2013

These changes are related to the makefile only, so I don't mind supporting every single option in the pelican makefile.

@onlyhavecans

This comment has been minimized.

Show comment
Hide comment
@onlyhavecans

onlyhavecans Mar 22, 2013

Member

I think S3 is becoming such a big contender in the static hosting market since it has been gaining mainstream media press[^1] and Amazon directly supports it. Considering we already have so many options I don't think we hurt support taking our out of box upload support from %80 to %90. Most of the support issues from the makefile tend to be with edge case cross compatibility.

However, if we are to pull it in I think it it needs to be made crystal clear in the quickstart and documented that using this option requires external software and configuration in advance of use. The point here that we can't support s3 tools we have no experience or use with.

[^1]: I can't find it NOW of course but I happened upon two separate articles about fairly mainstream media using static hosting on S3 to keep their site strong and fast during election coverage.

Member

onlyhavecans commented Mar 22, 2013

I think S3 is becoming such a big contender in the static hosting market since it has been gaining mainstream media press[^1] and Amazon directly supports it. Considering we already have so many options I don't think we hurt support taking our out of box upload support from %80 to %90. Most of the support issues from the makefile tend to be with edge case cross compatibility.

However, if we are to pull it in I think it it needs to be made crystal clear in the quickstart and documented that using this option requires external software and configuration in advance of use. The point here that we can't support s3 tools we have no experience or use with.

[^1]: I can't find it NOW of course but I happened upon two separate articles about fairly mainstream media using static hosting on S3 to keep their site strong and fast during election coverage.

@almet

This comment has been minimized.

Show comment
Hide comment
@almet

almet Mar 24, 2013

Member

Okay, let's go this way then.

Member

almet commented Mar 24, 2013

Okay, let's go this way then.

almet added a commit that referenced this pull request Mar 24, 2013

Merge pull request #775 from lexual/s3_support_for_quickstart
Added S3 support for pelican quickstart.

@almet almet merged commit 5dbb8c3 into getpelican:master Mar 24, 2013

1 check failed

default The Travis build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment