Disroot-Ansible
/
nginx
11
1
Fork 0
Code Pull Requests 4 Projects Releases Activity

cryptpad - commented out by default datastore location #10

Merged
muppeth merged 5 commits from cryptpad into master 2021-04-27 21:17:18 +02:00
Conversation 5 Commits 5 Files Changed 1 +2
muppeth commented 2021-04-25 08:55:14 +02:00
Owner
Copy Link
No description provided.
meaz was assigned by muppeth 2021-04-25 08:55:14 +02:00
antilopa was assigned by muppeth 2021-04-25 08:55:14 +02:00
muppeth added 1 commit 2021-04-25 08:55:16 +02:00
meaz approved these changes 2021-04-25 09:03:48 +02:00
meaz added 1 commit 2021-04-25 09:05:45 +02:00
muppeth commented 2021-04-25 11:07:41 +02:00
Author
Owner
Copy Link

@meaz we will also have to rebase your cryptapd new branch right? Or (maybe easiser) instead of pull this you add the changes to your cryptapd branch and we pull yours all together in one go.

@meaz we will also have to rebase your cryptapd new branch right? Or (maybe easiser) instead of pull this you add the changes to your cryptapd branch and we pull yours all together in one go.
antilopa approved these changes 2021-04-25 22:22:37 +02:00
antilopa commented 2021-04-25 23:24:29 +02:00
Owner
Copy Link

building cryptpad via ansible with this branch set fails to create vhost, as if the template is broken. It builds fine when nginx role is set to master or cryptpad4.4.0 branch. Can you check if you also have this issue?

building cryptpad via ansible with this branch set fails to create vhost, as if the template is broken. It builds fine when nginx role is set to master or cryptpad4.4.0 branch. Can you check if you also have this issue?
antilopa requested changes 2021-04-25 23:26:09 +02:00
antilopa left a comment
Owner
Copy Link

This seems to break the build process as I commented above.

This seems to break the build process as I commented above.
meaz added 1 commit 2021-04-26 11:31:17 +02:00
meaz commented 2021-04-26 11:48:07 +02:00
Owner
Copy Link

@muppeth my branch was already merged. So we need to merge this one too.

@antilopa I fixed the issue. It was just the new debug var that needed to be added in the vars files in the cryptpad role.

@muppeth my branch was already merged. So we need to merge this one too. @antilopa I fixed the issue. It was just the new debug var that needed to be added in the vars files in the cryptpad role.
meaz approved these changes 2021-04-26 11:48:49 +02:00
antilopa commented 2021-04-26 16:27:45 +02:00
Owner
Copy Link

@meaz it is still not solved for me:
"msg": "AnsibleError: template error while templating string: expected token 'end of statement block', got 'defined'.

@meaz it is still not solved for me: `"msg": "AnsibleError: template error while templating string: expected token 'end of statement block', got 'defined'.`
muppeth added 2 commits 2021-04-27 05:44:26 +02:00
muppeth commented 2021-04-27 05:49:24 +02:00
Author
Owner
Copy Link

Actually the issue was missing is in the if statement. It's fixed now.

Actually the issue was missing `is` in the if statement. It's fixed now.
antilopa approved these changes 2021-04-27 20:53:34 +02:00
meaz approved these changes 2021-04-27 20:54:20 +02:00
muppeth merged commit 1dd23b29d0 into master 2021-04-27 21:17:18 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
antilopa
meaz
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
meaz
antilopa
3 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: Disroot-Ansible/nginx#10
No description provided.
Delete Branch "cryptpad"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?