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

Setup side chain on local #5337

Merged
merged 50 commits into from
Oct 23, 2023
Merged

Conversation

eniolam1000752
Copy link
Contributor

@eniolam1000752 eniolam1000752 commented Sep 25, 2023

What was the problem?

This PR resolves #5173

How was it solved?

  • Setup enevti core on jenkins
  • Setup enevti service on jenkins
  • Refactor lisk chain service, enevti chain service and e2e test job procedure.
  • Ensure existing e2e test works

How was it tested?

  • Spin up core for both chains (i.e lisk and enevti) by running the following;

    • Open up a bash terminal and ensure you are in the lisk-desktop root directory, then run CORE=lisk source e2e/scripts/run-core.sh on that terminal. This should spin up lisk core on that terminal.
    • Open up a new bash terminal, while also still in the lisk-desktop root directory and run CORE=enevti source e2e/scripts/run-core.sh. This should spin up enevti core on that terminal.
  • Spin up service for both enevti and lisk

    • Ensure you have docker available on your local machine.
    • Ensure you have cloned the lisk-service repo and you have separate copies of the service folder.
    • Name one of the above folders lisk-service and the other enevti-service
    • Open up a fresh new bash terminal and ensure you are in the lisk-desktop root directory then run LISK_SERVICE_FILE_PATH="<relative path to the lisk-service folder>" GITHUB_APP_REGISTRY_REPO_BRANCH=jenkins-deployment sh e2e/scripts/run-service.sh on that terminal. This should spin up a lisk-service that connects to the lisk-core initially created on that terminal.
    • Again, open up a new bash terminal and also ensure you are in the lisk-desktop root directory then run ENEVTI_SERVICE_FILE_PATH="<relative path to the enveti service folder>" GITHUB_APP_REGISTRY_REPO_BRANCH=jenkins-deployment source e2e/scripts/run-service.sh on that terminal. This should spin up an enevti-service instance that connects to the enevti-core instance created initially on that terminal.
  • With the above, you should have core and service for enevti and lisk up and running. Then you should be able to run the e2e test or add a local network connection to a running lisk-deskop application. With this, all endpoints should point to 127.0.0.1 and enevti should be part of the explorable applications if viewed from the lisk local main chain

@ManuGowda ManuGowda removed their request for review September 29, 2023 10:21
@eniolam1000752 eniolam1000752 marked this pull request as ready for review October 16, 2023 15:05
Jenkinsfile Outdated Show resolved Hide resolved
Jenkinsfile Outdated Show resolved Hide resolved
e2e/scripts/run-core.sh Outdated Show resolved Hide resolved
e2e/scripts/run-service.sh Outdated Show resolved Hide resolved
@ikem-legend ikem-legend self-requested a review October 18, 2023 07:42
Comment on lines +194 to +199
## kill enevti-core process
# kill $( cat enevti-core.pid ) || true
# sleep 10
# kill -9 $( cat enevti-core.pid ) || true
# cat enevti-core.out
# cat enevti-core.err
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since enevti-service and lisk-core were shut down properly, why wasn't enevti-core shut down?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reason was because for now the enevti-core isn't running at the moment(there is a sight issue with enevti running on jeinkins which I would be resolving in this issue) because of those reasons, there would be no pid, out or err files created. Reason for me commenting those lines out @ikem-legend

@ikem-legend ikem-legend self-requested a review October 23, 2023 09:55
ikem-legend
ikem-legend previously approved these changes Oct 23, 2023
oskarleonard
oskarleonard previously approved these changes Oct 23, 2023
@ikem-legend ikem-legend self-requested a review October 23, 2023 12:36
@ManuGowda ManuGowda merged commit f43a4c7 into release/3.0.0 Oct 23, 2023
6 checks passed
@ManuGowda ManuGowda deleted the 5173-setup-side-chain-on-jenkins branch October 23, 2023 16:27
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.

None yet

4 participants