8000 build: fix beta tag build by jordanlewis · Pull Request #8886 · cockroachdb/cockroach · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

build: fix beta tag build #8886

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

Merged
merged 1 commit into from
Aug 29, 2016

Conversation

jordanlewis
Copy link
Member
@jordanlewis jordanlewis commented Aug 29, 2016

Previously, the beta-* tag builder built static binaries after the
push-docker-deploy step, which prevents push-docker-deploy from running
the acceptance tests. It made up for this by running the acceptance
tests manually. Now push-docker-deploy will run the acceptance tests
instead.


This change is Reviewable

Previously, the beta-* tag builder built static binaries after the
push-docker-deploy step, which prevents push-docker-deploy from running
the acceptance tests. It made up for this by running the acceptance
tests manually. Now push-docker-deploy will run the acceptance tests
instead.
@tbg
Copy link
Member
tbg commented Aug 29, 2016

LGTM. push-docker-deploy already runs the acceptance tests, right? Just double checking because not in diff.

@jordanlewis
Copy link
Member Author

Correct, it does run the acceptance tests.

@petermattis
Copy link
Collaborator

:lgtm:


Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks pending.


Comments from Reviewable

@jordanlewis jordanlewis merged commit ce2bc50 into cockroachdb:master Aug 29, 2016
@jordanlewis jordanlewis deleted the fix-beta-builds branch August 29, 2016 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0