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

Fix TB setup #1335

Merged
merged 1 commit into from
Apr 11, 2023
Merged

Fix TB setup #1335

merged 1 commit into from
Apr 11, 2023

Conversation

georgimld
Copy link
Collaborator

@georgimld georgimld commented Apr 6, 2023

Checklist

  • I have followed (at least) the PR section of the contributing guide.
  • I fixed all necessary PR warnings
  • The commit history is clean
  • The E2E tests are passing
  • If possible, the issue has been divided into more subtasks
  • I did a self review before requesting a review from another team member

Description

The operation script and the docker-compose setup were not working when a volume was attached to the blockchain container, since TB v2.1.0, with an error like:

terminate called after throwing an instance of 'boost::filesystem::filesystem_error'\n what(): boost::filesystem::create_directory: Permission denied: \"/root/.multichain/TrubudgetChain\"\n","message":"Failed to start the alpha node: "

Todo

  • The fix included granting permissions to the node user in the blockchain dockerfile
  • setting the correct environment variable as MULTICHAIN_DIR=/root
  • correctly using volumes in the operations docker-compose

How to test

  • check out branch 1330-fix-operation-script
  • using the operations script: make sure to build the blockchain image locally, for this you must edit one line in the docker-compose file in the /operation folder. This must be edited to build: ../../blockchain. Then you can run bash ../scripts/operation/start-trubudget.sh --slim
  • using the docker-compose setup: cd docker-compose and then docker-compose --project-directory . -f blockchain/docker-compose.alphanode.yml -f blockchain/persistence.docker-compose.yml -f blockchain/local-build.docker-compose.yml up

Closes #1330

@georgimld georgimld requested a review from egdmrsy April 6, 2023 14:45
@openkfwCI
Copy link

openkfwCI commented Apr 6, 2023

NotesTime
Note for Reviewer: E2E tests on remote server succeededTue, 11 Apr 2023 07:36:46 +0000

Generated by E2E-Test

@georgimld georgimld merged commit a0c42c2 into main Apr 11, 2023
@georgimld georgimld deleted the 1330-fix-operation-script branch April 11, 2023 09:39
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.

Fix TB operation script
3 participants