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

centos6 Initial commit #1002

Merged
merged 1 commit into from
Jun 6, 2015
Merged

centos6 Initial commit #1002

merged 1 commit into from
Jun 6, 2015

Conversation

sathiyas
Copy link
Contributor

@sathiyas sathiyas commented Jun 4, 2015

This change enables users to create centos6 based Consul cluster. CentOS.org does not allow launch AMIs directly since it is available only through Market place. The execution will throw an error asking to accept terms and conditions of market place one and provides a link specific to the account that is used to execute. Once you accept the terms, next execution will succeed. Another option is to replace the default AMI with another trusted centos6 AMI, the caveat is it may not be endorsed by centos.org

@sathiyas
Copy link
Contributor Author

sathiyas commented Jun 4, 2015

Please see note from centos @ http://wiki.centos.org/Cloud/AWS

“Using the AMI's directly is currently deprecated. We are working to resolve issues and establish automation as well as monitoring around resources and process that will allow us to re-enable direct AMI instantiation. In the mean time, the AMI's are listed at the bottom of this document for legacy / reference purposes."

@sathiyas
Copy link
Contributor Author

sathiyas commented Jun 5, 2015

@phinze , the build seems to have failed, I hope it's nothing to do with this pull, please check

@ryanuber
Copy link
Member

ryanuber commented Jun 5, 2015

@sathiyas it's definitely not related to your changes. Unfortunately Consul has a hard time with its unit tests on Travis-CI sometimes.

armon added a commit that referenced this pull request Jun 6, 2015
@armon armon merged commit 9f081e3 into hashicorp:master Jun 6, 2015
@armon
Copy link
Member

armon commented Jun 6, 2015

Thanks!

duckhan pushed a commit to duckhan/consul that referenced this pull request Oct 24, 2021
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