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

acbuild: add symlinks to /usr/local/bin/etcd* #6653

Merged
merged 1 commit into from
Oct 21, 2016
Merged

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented Oct 14, 2016

And uses latest acbuild (v0.4.0, --to-dir flag is deprecated).

Fix #6057.

/cc @steveej @jonboulle @xiang90

And uses latest acbuild (v0.4.0, --to-dir flag is deprecated).

For etcd-io#6057.
@gyuho gyuho added the WIP label Oct 14, 2016
@xiang90
Copy link
Contributor

xiang90 commented Oct 14, 2016

the docker image is fine?

@gyuho
Copy link
Contributor Author

gyuho commented Oct 14, 2016

This tries to make acbuild image similar to https://github.com/coreos/etcd/blob/master/Dockerfile-release.
So it doesn't touch any Docker builds?

Previous commands with rkt with acbuild image would still work.

Default data directory changes though.

I will test more when I do the next release.

@xiang90
Copy link
Contributor

xiang90 commented Oct 14, 2016

OK. Right.

@gyuho gyuho removed the WIP label Oct 17, 2016
@gyuho
Copy link
Contributor Author

gyuho commented Oct 17, 2016

@jonboulle @steveej Any feedback?

We would like to get this done before v3.1 release.

@jonboulle
Copy link
Contributor

@dgonyeo can you review?

@cgonyeo
Copy link

cgonyeo commented Oct 17, 2016

Nothing seems incorrect to me in the script, so LGTM if it works.

@gyuho
Copy link
Contributor Author

gyuho commented Oct 17, 2016

@dgonyeo @jonboulle Thanks!

I will double-check this script in the next release before merging.

@gyuho gyuho merged commit 791aeb3 into etcd-io:master Oct 21, 2016
@gyuho gyuho deleted the acbuild branch October 21, 2016 17:48
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants