Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

Conversation

kbroughton
Copy link
Contributor

This resolves #1277

  • Installs cleanly on a fresh build of most recent master branch
  • Upgrades cleanly from the most recent release
  • Updates documentation relevant to the changes

@kbroughton kbroughton force-pushed the kbroughton/feature/1277-make-naming-consistent-hosts-route53 branch 3 times, most recently from d897174 to aeacb0d Compare March 29, 2016 19:23
@darKoram
Copy link

test this please

@kbroughton kbroughton force-pushed the kbroughton/feature/1277-make-naming-consistent-hosts-route53 branch 4 times, most recently from baa9beb to d9c902f Compare March 29, 2016 19:52
@kbroughton kbroughton force-pushed the kbroughton/feature/1277-make-naming-consistent-hosts-route53 branch from d9c902f to b66c719 Compare March 29, 2016 20:03
@langston-barrett
Copy link
Contributor

@kbroughton Can you outline the acceptance criteria for testing this?

@kbroughton
Copy link
Contributor Author

terraform apply
open route53 and view the server names. eg.
https://drive.google.com/file/d/0B5bhZOuD4S4FLTRxLTduRU91blE/view?usp=sharing

ssh into any host.
cat /etc/hosts

You should see that worker-001 is consistent in both locations. Previously, /etc/hosts was worker-01

@kbroughton
Copy link
Contributor Author

btw, i noticed that some of the other providers, openstack, cisco-os already have this mechanism.
I'm guessing gce will have the same issue as aws. They even named the variable the same as me!

./terraform/openstack-modules.sample.tf: count_format = "%03d"
./terraform/openstack-modules.sample.tf: count_format = "%02d"
./testing/cisco-os.tf: count_format = "%03d"
./testing/cisco-os.tf: count_format = "%02d"

@langston-barrett
Copy link
Contributor

Testing now.

@langston-barrett
Copy link
Contributor

Works for me (AWS, obviously):

[centos@ip-10-1-1-199 ~]$ cat /etc/hosts
127.0.0.1   localhost localhost.localdomain localhost4 localhost4.localdomain4
::1         localhost localhost.localdomain localhost6 localhost6.localdomain6
10.1.3.64  lb1-control-03
10.1.3.34  lb1-worker-003
10.1.2.145  lb1-edge-02
10.1.1.65  lb1-edge-01
10.1.2.37  lb1-control-02
10.1.2.187  lb1-worker-002
10.1.1.199  lb1-control-01
10.1.1.169  lb1-worker-004
10.1.1.57  lb1-worker-001

screenshot

@langston-barrett langston-barrett merged commit 55addee into mantl:master Apr 5, 2016
@ryane ryane modified the milestone: 1.1 May 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

inconsistent names between /etc/hosts, Route53
5 participants