-
Notifications
You must be signed in to change notification settings - Fork 20
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
bitacross init #2419
bitacross init #2419
Conversation
5e82070
to
d39923e
Compare
d39923e
to
d037838
Compare
Co-authored-by: Kailai Wang <kailai.wang@trustcomputing.de> Co-authored-by: Zhouhui Tian <zhouhui@liteng.io>
Thanks! We can either accept it now, or wait a bit until P-443 is finished, which would affect the worker registration and mngt, like I'm thinking both identity and bitacross worker will be registered within the same pallet |
Let's merge it, it's already huge commit. |
I think we should merge it, It will be easier to test P-443 with both workers when they are merged in |
Or we can treat this as the staging branch for all bitacross commits |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's merge it first
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Starting point for BitAcross.
Code is taken from
tee-worker
. I've removed unnecessary stuff like identity related trusted calls/getters.Below is the list that contains changed files compared to
tee-worker
. I useddiff -qr tee-worker/ bitacross-worker/
to prepare it.