-
Notifications
You must be signed in to change notification settings - Fork 415
Conversation
I think that by v1.0, we should have terraform.py as a submodule, so upkeep is simpler. |
@sehqlr Agreed. |
This passed a basic install + healthy test. Are there any specific tests for this one? |
@sehqlr Nope! This should be ready to merge after we release 0.6. |
attrs.update({ | ||
'consul_dc': _clean_dc(attrs['metadata'].get('dc', attrs['region'])), | ||
'role': attrs['metadata'].get('role', 'none') | ||
'role': attrs['metadata'].get('role', 'none'), | ||
'ansible_python_interpreter': attrs['metadata'].get('python_bin','python') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why we need to touch stuff related to python_interpreter here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure, maybe @BrianHicks would know.
When we move this to a submodule, where would the clone go? In the root directory? |
@sehqlr Let's make an issue for that and discuss there. |
I made an issue: #1120 |
Fixes #986
cd5c908
to
b429ed0
Compare
Failure on AWS was quota-related. On GCE, Traefik health checks are failing, but that doesn't seem related. DO build passed. |
Fixes #986