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

renamed adore_cli services to accommodate the stricter docker compose… #70

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

akoerner1
Copy link
Contributor

@akoerner1 akoerner1 commented Apr 12, 2024

This PR addresses the stricter static checks intruduced in docker compose version >= 2.24.6 with the issue is detailed in: #69

  • This PR introduces container name changes for the adore_cli, libadore and plotlabserver services and removes the depends_on and version tag and from the docker compose file due to deprecation and stricter static checks

  • The adore_cli, plotlabserver, and apt_cacher_ng_docker submodules were updated with the PR.

  • The CI action successfully passed with the corresponding commit for this PR:
    https://github.com/akoerner1/adore/actions/runs/8612759571

Before accepting this PR please do a clean clone, build and run of the baseline_test scenario.

@akoerner1 akoerner1 requested a review from tlobig April 12, 2024 10:01
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

Successfully merging this pull request may close these issues.

1 participant