Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-10-05 Thread Mingshen Sun
Merged #559 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/apache/incubator-teaclave/pull/559#event-5412730954

Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-10-05 Thread Gordon King
> I have fixed in PR #561. Please help to review and approve this PR (we won't > merge any PR without review). Thanks. Will review and verify it, thanks. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-10-04 Thread Mingshen Sun
I have fixed in PR #561. Please help to review and approve this PR (we won't merge any PR without review). Thanks. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-10-04 Thread Mingshen Sun
> The CI identified the following issue. > ![image](https://user-images.githubusercontent.com/10856796/135905616-60640390-262a-47a3-9201-495bb7f62377.png) > Is it related to this PR? or anything I should fix in this PR? thanks. This is caused by the CI environment. We didn't pin openssl to a

Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-10-04 Thread Gordon King
The CI identified the following issue. ![image](https://user-images.githubusercontent.com/10856796/135905616-60640390-262a-47a3-9201-495bb7f62377.png) Is it related to this PR? anything I should fix in this PR? thanks. -- You are receiving this because you are subscribed to this thread. Reply to

Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-09-30 Thread Gordon King
> I see several docs also contain instructions to use `docker-compose` to start > services. Can you also change these? Thanks. > > * `docs/my-first-function.md` > * `docs/azure-confidential-computing.md` sure, will do. thanks. -- You are receiving this because you are subscribed to this

Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-09-30 Thread Mingshen Sun
I see several docs also contain instructions to use `docker-compose` to start services. Can you also change these? Thanks. - `docs/my-first-function.md` - `docs/azure-confidential-computing.md` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or

Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-09-28 Thread Gordon King
My colleague helped me to identified an issue on his icelake server, I will add a fixup on top of this PR, thanks. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-09-28 Thread Mingshen Sun
Thanks. This is really cool! Let me test it on servers with different kinds of SGX drivers. (This also bother me since we cannot test this in our CI) -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[apache/incubator-teaclave] [docker] start Teaclave docker services with auto-detection mechanism (#559)

2021-09-28 Thread Gordon King
## Description Added a bash script to detect sgx device and aesm service heuristically, and start teaclave services without needing to specify corresponding docker compose file manually. this script can detect and handle containerized aesm service as well. Fixes #553 ## Type of change (select