Skip to content

Commit

Permalink
Added Change from Jukka
Browse files Browse the repository at this point in the history
it seems that the problem is here:
https://github.com/learning-layers/openldapaccount-Dockerfiles/blob/master/docker-compose.yml

’links:’ is defined twice, once as
-openldap
-oidc

and second time:
-openldap

It seems that docker-compose versions deal with this in different ways. …And when layersbox -script reads this, it ends up writing this kind of definition:

openldapaccount:
... links: [openldap]...
 
if this is replaced with:
  links: [openldap, oidc]

Then the registration works.
So if you can remove the second ’links’ definition in openldapaccount’s docker-compose.yml , layersbox script should be able to write it the correct way.
  • Loading branch information
nicolaescu committed Nov 25, 2015
1 parent 2df92f7 commit 6ea2dde
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,15 +6,11 @@ openldapaccount:
image: learninglayers/openldapaccount:0.0.5
volumes_from:
- openldapaccountdata
links:
- openldap
- oidc
links: [openldap, oidc]
environment:
- PWM_LDAP_ADMINS=<value>koren</value><value>nicolaescu</value>
env_file:
- common.env
- services/openldap/LDAP.env
links:
- openldap
volumes:
- /dev/urandom:/dev/random

0 comments on commit 6ea2dde

Please sign in to comment.