Fix Ansible installation, add uninstall playbook #19

Manually merged
dessalines merged 0 commits from nutomic/lemmy:ansible into master 2020-02-01 04:08:44 +00:00
Owner
No description provided.
dessalines reviewed 2020-01-29 05:28:25 +00:00
Owner

Hrm, maybe we should also externalize the version into inventory... not sure tho.

Hrm, maybe we should also externalize the version into inventory... not sure tho.
nutomic reviewed 2020-01-29 13:27:06 +00:00
Author
Owner

Yeah I'm thinking about that for the dev deploy right now. Inventory is probably not the best place, because its not under version control so every user has to update it manually. How about adding another VERSION file that only contains the version as plaintext?

Yeah I'm thinking about that for the dev deploy right now. Inventory is probably not the best place, because its not under version control so every user has to update it manually. How about adding another `VERSION` file that only contains the version as plaintext?
dessalines reviewed 2020-01-30 01:48:12 +00:00
Owner

Hrm... I feel like for the ansible it should only update when explicitly told to, who knows what kind of changes might break things.

But if we wanted to do that, the VERSION would go in the templates folder? And then the docker compose would do {{ version }} ?

Hrm... I feel like for the ansible it should only update when explicitly told to, who knows what kind of changes might break things. But if we wanted to do that, the VERSION would go in the templates folder? And then the docker compose would do `{{ version }}` ?
nutomic reviewed 2020-01-31 11:06:28 +00:00
Author
Owner

Templates sounds like the wrong place, that is for files which will be copied to the server. But something like ansible/VERSION I guess. And yes it can be used as a variable then.

Templates sounds like the wrong place, that is for files which will be copied to the server. But something like `ansible/VERSION` I guess. And yes it can be used as a variable then.
nutomic reviewed 2020-01-31 11:23:36 +00:00
Author
Owner

I will probably fix that in #20 because I need to add the variable there anyway to support dev builds.

I will probably fix that in #20 because I need to add the variable there anyway to support dev builds.
dessalines reviewed 2020-01-31 20:17:42 +00:00
Owner

Okay cool, I'll take a look at this within a few hours.

Okay cool, I'll take a look at this within a few hours.
dessalines closed this pull request 2020-02-01 04:08:44 +00:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: LemmyNet/lemmy#19
No description provided.