Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

restructured introduction to own webserver deployments #2352

Closed
wants to merge 1 commit into from

Conversation

thigg
Copy link

@thigg thigg commented Dec 20, 2022

As a follow up to #2351.

This is a bit more opinionated (and maybe even wrong), but I just wrote it in the way I would have needed it.
For me, together with the changes in the first method this would have made the perfect match.

spantaleev added a commit that referenced this pull request Dec 20, 2022
This warning was added because of:
#2090

The problem has since been fixed by:
e9e8434

This current patch was provoked by:
#2352
@spantaleev
Copy link
Owner

Thanks! This PR makes things better.. However, it seems like the documentation was outdated.

There's no need to do separate things for worker and non-worker setups - since e9e8434, the playbook works equally well with both of these.

As such, I've simplified the documentation in 362954a by completely removing the warning.

@thigg thigg closed this Dec 20, 2022
mgoerens added a commit to mgoerens/matrix-docker-ansible-deploy that referenced this pull request Apr 12, 2023
Similar to:
spantaleev@362954a

This warning was added because of:
spantaleev#2090

The problem has since been fixed by:
spantaleev@e9e8434

This current patch was provoked by:
spantaleev#2352
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants