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

Clean Makefile #474

Open
zhuyuanmao opened this issue Jul 26, 2022 · 2 comments
Open

Clean Makefile #474

zhuyuanmao opened this issue Jul 26, 2022 · 2 comments

Comments

@zhuyuanmao
Copy link
Member

As the Cello project grows, the Makefile commands are outdated. We would like to know which commands aren't working. Please list the commands here.

@alienishi
Copy link

1: make start : This command starts the service, but it doesn't specify which service to start. It relies on the start-docker-compose rule, which starts the services defined in the bootup/docker-compose-files/${COMPOSE_FILE} file. However, the value of ${COMPOSE_FILE} is determined by the MODE variable, which is set to prod by default. If you want to use this command, ensure that the appropriate value is set for ${COMPOSE_FILE} and that the necessary configuration files are present.

2: make stop: This command stops the service, but it doesn't specify which service to stop. It relies on the stop-docker-compose rule, which stops the services defined in the bootup/docker-compose-files/${COMPOSE_FILE} file. Similar to the start command, ensure that the value of ${COMPOSE_FILE} is correctly set.

3: make restart: This command restarts the services, but it relies on the stop and start commands. As mentioned above, ensure that the necessary configurations are in place for those commands to work correctly.

4: make clean-images: This command is intended to clean all Cello-related Docker images. However, it executes the clean rule, which stops services and cleans Docker containers. It doesn't specifically target cleaning Docker images. If you want to clean Docker images, you may need to modify the command to remove the images instead of stopping services and cleaning containers.

5: make check-dashboard: This command checks the dashboard but relies on the docker-compose command with a specific configuration file. Ensure that the necessary configuration files are present and correctly set up for the dashboard check to work as expected.

@alienishi
Copy link

Hi @zhuyuanmao
Sure i can make a patch to fix these makefile rules
Please do assign this issue to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants