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

Rockon with multiple containers requires same env variables for all containers #2202

Closed
freaktechnik opened this issue Aug 15, 2020 · 7 comments

Comments

@freaktechnik
Copy link

Define a rockon with multiple containers. The two containers require a different set of environment variables to be set by the user. Installing the rockon will fail, due to an env variable that was set in the wizard not existing for one of the containers.

@FroggyFlox
Copy link
Member

FroggyFlox commented Aug 15, 2020

Hi @freaktechnik ,

Thanks a lot for your interest in multi-containers rock-ons; it's nice to see there's a lot of interest in this.

Unfortunately, I believe you are highlighting an issue that was previously raised (#1588 (comment)) and for which a pull-request was submitted (#1688). I personally am very motivated in getting to that one as soon as things settle down on the relaunch, so it's very encouraging to see that this interest is shared by other users. We have now seen multiple users that would have benefited from this being fixed/enabled, so I will definitely get on that once I can.

Sorry for not being able to provide you with a solution right away, unfortunately.

Hope this helps anyway...

@freaktechnik
Copy link
Author

freaktechnik commented Aug 15, 2020

I have found a workaround, which is to have the same set of env vars for all containers. Curiously this repeats the fields in the env var input, but will only show them once in the end table. Of course this only works if there are no env var clashes.

@freaktechnik
Copy link
Author

This is what the resulting definition looks like btw: https://github.com/freaktechnik/rockons/blob/master/matrix-synapse.json

In theory only matrix-synapse needs GID and UID and only the postgres container needs the pw (though the pw bootstrapping doesn't seem to have worked in the first place)

@FroggyFlox
Copy link
Member

That's a great candidate for a rock-on, especially given its reputation as being a little complex to setup!
Let's hope we'll be able to fix that underlying limitation relatively soon.

Thanks!

@freaktechnik
Copy link
Author

freaktechnik commented Aug 16, 2020

Yeah, I've noted some of the caveats with the rockon I ended up with in the README. Essentially the TURN server can't STUN, and I didn't manage to get Element as part of the setup. I've now simply used the Element app for Nextcloud to self-host Element (that plus having set up the database and synapse config manually, which I actually wanted to do since I migrated my synapse). I guess an even more advanced package would also include jitsi, though I think that could be a separate rockon that you can just integrate at rockon setup time. Either way, seeing a more streamlined version would definitively be great!

@freaktechnik
Copy link
Author

This also appears to apply to devices.

@phillxnet
Copy link
Member

Closing as:
Duplicate of #1588

"Rockons: Invalid environment variabled" #1588

As per @FroggyFlox comment #2202 (comment)

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

No branches or pull requests

3 participants