Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

Problem: sgx-cargo-1804-hw1 is not easy to run locally #1296

Closed
yihuang opened this issue Mar 25, 2020 · 2 comments
Closed

Problem: sgx-cargo-1804-hw1 is not easy to run locally #1296

yihuang opened this issue Mar 25, 2020 · 2 comments

Comments

@yihuang
Copy link
Collaborator

yihuang commented Mar 25, 2020 •
edited
Loading

Maybe move it into default pipeline, or change pipeline type to default.
Easier to test locally with drone-cli.
@yihuang yihuang changed the title Run sgx-cargo-1804-hw1 in normal pipeline Problem: sgx-cargo-1804-hw1 is not easy to run locally Mar 25, 2020
@tomtau
Copy link
Contributor

tomtau commented Mar 25, 2020 •
edited
Loading

It'll make sense to put it in the default pipeline, as it's all in Docker anyway now.
Local execution is easy though -- just run the same command?

docker run --name <SOME_NAME> --rm -v <PATH_TO_CHAIN_REPO>:/chain --device </dev/sgx or /dev/isgx> cryptocom/chain:latest /bin/bash /chain/ci-scripts/tx-validation-hw-test.sh

It's expected to run in HW mode... for quick ad-hoc testing in SW mode, it's just steps from tx-validation-hw-test.sh without AESM+device testing + export SGX_MODE=SW

yihuang added a commit to yihuang/chain that referenced this issue Mar 25, 2020
@yihuang
Problem (Fix crypto-com#1296): sgx-cargo-1804-hw1 is not easy to run …
5ff7400 
…locally

Solution:
- Move it into default pipeline
yihuang added a commit to yihuang/chain that referenced this issue Mar 25, 2020
@yihuang
Problem (Fix crypto-com#1296): sgx-cargo-1804-hw1 is not easy to run …
85d2bdf 
…locally

Solution:
- Move it into default pipeline
@yihuang
Copy link
Collaborator Author

yihuang commented Mar 25, 2020

It seems quite easy to run in the default pipeline, so I just did it.

drone exec --trusted --include sgx-test

yihuang added a commit to yihuang/chain that referenced this issue Mar 25, 2020
@yihuang
Problem (Fix crypto-com#1296): sgx-cargo-1804-hw1 is not easy to run …
9b292b1 
…locally

Solution:
- Move it into default pipeline
bors bot added a commit that referenced this issue Mar 25, 2020
@bors @yihuang
Merge #1292 #1298
2b30647 
1292: Problem (Fix #1142): revised punishment mechanisms not implemented r=tomtau a=yihuang

Solution:
- revised staking and punishment implementation

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/crypto-com/chain/1292)
<!-- Reviewable:end -->


1298: Problem (Fix #1296): sgx-cargo-1804-hw1 is not easy to run locally r=tomtau a=yihuang

Solution:
    - Move it into default pipeline

```
drone exec --trusted --include sgx-test
```

Co-authored-by: yihuang <[email protected]>
bors bot added a commit that referenced this issue Mar 25, 2020
@bors @yihuang
Merge #1298
0271ac7 
1298: Problem (Fix #1296): sgx-cargo-1804-hw1 is not easy to run locally r=tomtau a=yihuang

Solution:
    - Move it into default pipeline

```
drone exec --trusted --include sgx-test
```

Co-authored-by: yihuang <[email protected]>
@bors bors bot closed this as completed in 448e95b Mar 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yihuang @tomtau