-
Notifications
You must be signed in to change notification settings - Fork 17
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
WIP: allow multiple videobridges #55
WIP: allow multiple videobridges #55
Conversation
- jitsi-meet-prosody | ||
- jitsi-meet-web | ||
- jitsi-meet-web-config | ||
jitsi_meet_apt_repositories_pin_packages: "{{ ['jitsi-videobridge2'] + (jitsi_meet_single_instance | ternary(_jitsi_meet_apt_repositories_pin_packages, [])) }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is ugly, but works.
Thank you for permitting this PR. I think we shouldn't refactor that much. Instead of moving everything in subfolders, make videobridge an independent role that can work on it's own and can be tested independently. Afterwards, make videobridge a conditional dependency of jitsit-meet in |
Hello, as you might have guessed by my absence, i don't have the time to put more work into this, right now. I will close this PR for now and might come back to it later. Feel free to use my code, if you plan to tackle the multi videobridge setup yourself in the future. |
This is a WIP draft for configuring multiple videobridges with this role.
Most definitely missing are:
In my tests, these changes were backwards compatible, as long as one did not change the name of the role to something else then
jitsi-meet
(since that would break the galaxy dependencies inmeta/main.yml
).Be aware that i'm using a custom version of this branch with some customizations specific to my own setup, so i never tested this code as is. That should probably happen, before merging.. :)
Fixes #53.