Docker compose improvements

[Silex]

Describe your changes

I can wrap my head around why management service would depend on dashboard, so here is a PR removing the dependency.

If the dependency is actually needed, I might transform this PR into one where I add HEALTHCHECK to the dashboard.

EDIT: ok the PR changed its goal toward general docker compose improvements.

Issue ticket number and link

Stack

Checklist

• [ ] Is it a bug fix
• [ ] Is a typo/documentation fix
• [ ] Is a feature enhancement
• [x] It is a refactor
• [ ] Created tests that fail without the change (if possible)
• [ ] Extended the README / documentation, if necessary

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

[mlsmaycon]

is mainly because we mount the certificates generated by the dashboard into the management

[Silex]

@mlsmaycon ok thanks, I removed the commit. The rest should still apply, ask if you have questions.

[Silex]

@mlsmaycon wait actually, why is it needed to mount the certificates generated by the dashboard into the management? is it if they both use the same domain on different ports?

I don't quite get it because by default it looks like only the dashboard serves HTTPS, and management is used internally on port 33080.

Or maybe it's simply so the letsencrypt folder contains all certificates, and the dependencies forces some kind of order for the file creation.

[sonarqubecloud[bot]]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud