search

SUSE / velum

Dashboard for CaaS Platform clusters (v1, v2 and v3)
https://www.suse.com/
Apache License 2.0
54 stars 30 forks source link

init: added check for ldap #722

Open vitoravelino opened 5 years ago

vitoravelino commented 5 years ago

Added a new check to verify if the ldap instance is ready before running velum dashboard in order to avoid a connection error when creating a new user.

Signed-off-by: Vítor Avelino vavelino@suse.com

bsc#1121064

vitoravelino commented 5 years ago

I've realized that I added the check to the wrong file. It should be checked in the run file that belongs to the velum-dashboard container and not in the init that belongs to the dashboard initContainer.

MalloZup commented 5 years ago

@mssola, @flavio can we merge this?