Conversation
webknjaz
left a comment
webknjaz
left a comment
There was a problem hiding this comment.
I think we can keep Ansible content in the root of the repository as it'll be the main thing anyway and we'll remove shell scripts.
webknjaz
left a comment
webknjaz
left a comment
There was a problem hiding this comment.
also, rm unnecessary stat task
|
plz put ansible content in the root of the repo, also put the apps list into a variable and remove parts of |
|
bump |
inventory
Outdated
| @@ -0,0 +1,3 @@ | |||
| [target] | |||
There was a problem hiding this comment.
I'd be more specific here. Also it's a group so maybe plural is appropriate.
[dev-machines]
inventory
Outdated
| @@ -0,0 +1,3 @@ | |||
| [dev-machines] | |||
There was a problem hiding this comment.
Plz add another inventory file with the same group and localhost and connection local set.
|
So how about adding other parts of the original script here? |
RootenberG
force-pushed
the
automating_applications
branch
from
85514fd to
f798047
Compare
| @@ -0,0 +1,6 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
This should be in the playbook instead
inventory
Outdated
| @@ -0,0 +1,3 @@ | |||
| [dev-machines] | |||
|
|
|||
| localhost ansible_connection=local No newline at end of file | |||
There was a problem hiding this comment.
So one inventory file should have the host with connection local and the other inventory should point to the VM over SSH.
install-apps.yml
Outdated
| apt_repository: | ||
| repo: "deb [arch=amd64] https://packages.microsoft.com/repos/vscode stable main" | ||
| state: present | ||
| - name: Install vscode depts |
There was a problem hiding this comment.
| - name: Install vscode depts | |
| - name: Install vscode deps |
|
Bump |
|
bump |
3 participants
Closes #5