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

Small improvements #1503

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

Conversation

tudorpintea999
Copy link

Hello
I improved a bit rpc docker file and the start bash script , description available in the commits.
Hope this helps you , if you need any additional help I m here !
Thanks
Tudor.

 handle_error function ensures any error stops execution and logs where it occurred.
		-retries for Katana: A retries count limits the wait loop for Katana to avoid infinite loops.
		-function Modularity: Each step is encapsulated in its own function, making the script modular and improving readability.
	 -values are exported from manifest.json using environment variables, making the script adaptable to different environments.
for cleaner logs, use --quiet while updating the system
installing all required system dependencies (libclang-dev, libssl-dev, ca-certificates, tini, curl) in the chef or builder stage to avoid multiple apt-get calls.

Using cargo chef cook with caching can significantly improve build speed.


Remove unnecessary files after apt-get installs to reduce image size.
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