Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

Fix Ansible deployment scripts #2099

Merged
merged 1 commit into from
Feb 20, 2019
Merged

Fix Ansible deployment scripts #2099

merged 1 commit into from
Feb 20, 2019

Conversation

bakhtin
Copy link
Contributor

@bakhtin bakhtin commented Feb 14, 2019

Signed-off-by: Artyom Bakhtin [email protected]

Description of the Change

  • change how Docker network is created. Split down a step into two: depending on whether overlay_network variable is set to true or false. Also add network removal step as it might have interfered with different overlay_network modes.
  • fix docker-compose.yml template not using network name variable but hardcoded value instead

Benefits

less buggy deployment scripts

Possible Drawbacks

None

- change how Docker network is created. Split down a step into two: depending on whether `overlay_network` variable is set to `true` or `false`. Also add network removal step as it might have interfered with different `overlay_network` modes.
- fix docker-compose.yml template not using network name variable but hardcoded value instead

Signed-off-by: Artyom Bakhtin <[email protected]>
@bakhtin bakhtin requested review from BulatSaif and kogeler February 14, 2019 14:03
@neewy neewy requested review from neewy and removed request for kogeler February 20, 2019 09:53
@neewy neewy merged commit e759b1d into develop Feb 20, 2019
@neewy neewy deleted the fix/ansible-deployment branch February 20, 2019 09:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants