Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

Docker installation method #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alangecker
Copy link

as requested by @matiasilva this PR adds the docker installation method to the documentation.

closes alangecker/bigbluebutton-docker#15

@basisbit
Copy link
Contributor

There are still these "Open tasks" mentioned in the readme of that repository:

  • add support for recording
  • add prometheus exporter
  • further separate bbb-core into individual container
  • enable IPv6 support
  • switch html5 to node v12
  • drop root privileges in webrtc-sfu

I'd suggest to not merge this until a certain quality and completeness has been reached.

@alangecker
Copy link
Author

There are still these "Open tasks" mentioned in the readme of that repository:
[...]

I'd suggest to not merge this until a certain quality and completeness has been reached.

these points are just nice to have and neither included in the other installation methods so I don't think that they are somehow relevant as a blocking issue.

  • add prometheus exporter
  • switch html5 to node v12 (other methods rely on node v8)
  • further separate bbb-core into individual container (other methods don't work with isolation of components)

but yeps, these two can be considered as blocking issues:

  • missing recording
  • missing IPv6 support

@alangecker
Copy link
Author

alangecker commented Sep 9, 2020

so, both blocking issues are resolved with the latest Release v2.2.23-1
@basisbit I would be happy about continuing the discussion, especially what further requirement / concerns you see for adding this method :)

@ffdixon
Copy link
Member

ffdixon commented Jan 15, 2022

@alangecker We can merge this in for 2.4 if you can resolve the merge conflicts.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add this to the documentation
3 participants