sample/quote: Add Makefile and README #71
Conversation
binxing
force-pushed
the
pr/bxing/sample_quote
branch
from
50e132d to
6b330e4
Compare
samples/quote/Makefile
Outdated
|
|
||
| server.json: server/Dockerfile signer.pem signer.cer | .acon/ | ||
| $(DOCKER) build -t sampleserver -f $< $(PWD)/../.. | ||
| $(ACONCLI) generate -i sampleserver $@ |
There was a problem hiding this comment.
I remember that "writableFS" in the manifest needs to be changed to "true".
There was a problem hiding this comment.
Does quote_server need to write disk files? If no, then we don't need writable FS.
There was a problem hiding this comment.
No. Seems there is a bug of acond.
There was a problem hiding this comment.
@binxing, is /run/user/0 a must directory for readonly file system?
There was a problem hiding this comment.
/run is a separate tmpfs mount that shouldn't be affected by writableFS.
There was a problem hiding this comment.
I mean /run/user/*, not /run.
samples/quote/README.md
Outdated
| - The server must be started first, by | ||
|
|
||
| ```sh | ||
| TCP_PORT=5555 ATD=1 ATD_KERNEL=/path/to/vmlinuz ATD_RD=/path/to/initrd.img aconcli run -ni -c:$TCP_PORT server.json |
There was a problem hiding this comment.
Does it need a TCP port mapping between server and client?
There was a problem hiding this comment.
You are right! Which port?
There was a problem hiding this comment.
@xxu36 and @xiangquanliu, I added ATD_TCPFWD=8080:8085. Let me know if I'm wrong.
There was a problem hiding this comment.
yes, currently we are using this mapping
binxing
force-pushed
the
pr/bxing/sample_quote
branch
5 times, most recently
from
5399f2f to
6ae42f0
Compare
samples/quote/Makefile
Outdated
| $(OPENSSL) req -x509 -sha384 -key $< -out $@ -outform der -subj /CN=self-signed-$< | ||
|
|
||
| client/sampleclient: | ||
| $(GO) -C $(@D) build -v |
There was a problem hiding this comment.
Not sure if it is better to build a static linked binary. For example, to build it in development environment and test in EMR environment - may cause link errors.
Signed-off-by: Cedric Xing <[email protected]>
binxing
force-pushed
the
pr/bxing/sample_quote
branch
from
6ae42f0 to
d7f6407
Compare
No description provided.