#9 Add docker support
Merged by firstyear. Opened by firstyear.
firstyear/389-ds-portal 3-docker  into  master

This adds a dockerfile and support for docker to the 389-ds-portal. This may not work for others yet due to:

  • app.secret_key is per-thread and should be coming from a config ( https://pagure.io/389-ds-portal/issue/2 )
  • lib389 fails when unable to find defaults.inf ( https://pagure.io/389-ds-base/issue/50649 )
- docker run -i -t -p 8080 -v testdata:/data 389-ds-portal:test
+ docker run -i -t --name=389-ds-portal -p 8080:8080 -v testdata:/data 389-ds-portal:devel
- docker run -i -t -v testdata:/data 389ds/dirsrv:latest
+ docker run -i -t --name=389-ds -v testdata:/data 389ds/dirsrv:latest

Here and below then replace relaxed_yonath with 389-ds. All that for a mere convenience.

Other than the comments above this seems to work just fine, me only being unable to actually log in most likely due to issues described in the description.

These are all great comments @mhonek, thanks so much! I'll apply them now :)

1 new commit added

  • Update based on mhonek feedback

Thanks @mhonek, how is this?

I'm not sure I get this change. Especially the use of the Tumbleweed container instead of the actual 389-ds-portal we just built. I still think this should actually read the following:

    docker run -i -t -p 8080:8080 -v 389data:/data -i -t 389-ds-portal:devel

BTW, I use here the 8080:8080 so that it does not just bind to a random port, but always 8080 instead.

I probably just blindly copy pasted like an idiot ... I'll cehck to see what I messed up :)

1 new commit added

  • One line fix to docs

That should fix it, sorry I'm a bit silly.

rebased onto 6f27c3575fd707d37b44f8f28986aeb916c6b60d

rebased onto c1354b331c10a0151fa0fe51b573fa61860dce7f

Pull-Request has been merged by firstyear