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

Push tarball of linux/osx binaries to s3://binaries.cockroachdb.com #5094

Merged
merged 1 commit into from
Mar 10, 2016

Conversation

mberhault
Copy link
Contributor

These will be the links used on the download page.
We still push linux-only binaries to the current bucket.
binaries.cockroachdb.com is special because it is used by cloudfront
which provides https.

Download links will end up being:
https://binaries.cockroachdb.com/cockroach.{linux,darwin-10.9}-amd64.tgz

See discussion in cockroachdb/docs#34


This change is Review on Reviewable

These will be the links used on the download page.
We still push linux-only binaries to the current bucket.
binaries.cockroachdb.com is special because it is used by cloudfront
which provides https.

Download links will end up being:
https://binaries.cockroachdb.com/cockroach.{linux,darwin-10.9}-amd64.tgz
./push-one-binary.sh ${SHA} sql/sql${OSARCH}.test
./push-one-binary.sh ${SHA} acceptance/acceptance${OSARCH}.test
./push-one-binary.sh ${SHA} static-tests${OSARCH}.tar.gz
# Linux binaries: cockroach + tests.
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 am intentionally dropping the (currently unused) OSARCH bit. We're only pushing those binaries for linux.

@tbg tbg self-assigned this Mar 10, 2016
@tbg
Copy link
Member

tbg commented Mar 10, 2016

LGTM. Some duplication of variables, but OSX doesn't bump their versions all that often.

mberhault added a commit that referenced this pull request Mar 10, 2016
Push tarball of linux/osx binaries to s3://binaries.cockroachdb.com
@mberhault mberhault merged commit 23717c7 into master Mar 10, 2016
@mberhault mberhault deleted the marc/push_binaries branch March 10, 2016 18:56
@jseldess
Copy link
Contributor

Tarballs linked from Install CockroachDB.

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

Successfully merging this pull request may close these issues.

None yet

3 participants