-
Notifications
You must be signed in to change notification settings - Fork 46
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
Publish mrenclave #1473
Publish mrenclave #1473
Conversation
600af1d
to
78bfcee
Compare
c2afb65
to
0cfb560
Compare
SGX_MODE_LOWERCASE=$(echo "${${{ matrix.sgx_mode }},,}") | ||
echo "IMAGE_SUFFIX=$SGX_MODE_LOWERCASE-${{ matrix.flavor_id }}-${{ github.sha }}" >> $GITHUB_ENV |
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.
A docker image tag must have lower case letters only.
- name: Create Enclave Digest File | ||
run: | | ||
mrenclave_hex=$(docker run integritee-worker-${{ env.IMAGE_SUFFIX }} mrenclave | grep -o -E '[0-9a-fA-F]{64}') | ||
echo "$mrenclave_hex" > mrenclave-${{ env.IMAGE_SUFFIX }}.hex |
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.
I would like to confirm consensus here: We create a mrenclave-${{ env.IMAGE_SUFFIX }}.hex
for each flavour we build, which contains the mrenclave, example: mrenclave-hw-sidechain-<github-sha>.hex
.
Objections: @brenzi?
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.
Nice job! LGTM
I merge this for now, so that I can make an upstream merge in OLI's merge. Any amendments can be done later. |
mrenclave-${{ env.IMAGE_SUFFIX }}.hex
containing the mrenclave for each flavor we build, e.g.,mrenclave-hw-sidechain-<github-sha>.hex
. Closes Get mrenclave at build time #1472mrenclave
to the docker image we build, which prints the mrenclave:docker run integritee-service mrenclave