Skip to content
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

make keystone_example_app working #1

Open
wants to merge 15 commits into
base: dayeol/packaging
Choose a base branch
from

Conversation

dayeol
Copy link

@dayeol dayeol commented Jun 25, 2023

stacked PR on top of ccc-certifier-framework#102
cc @evgenyp67

@evgenyp67 evgenyp67 force-pushed the dayeol/packaging branch 2 times, most recently from 14a6aaf to b851314 Compare June 25, 2023 23:42
gapisback and others added 10 commits June 25, 2023 17:43
… APIs and in code

This commit renames stuff like 'cca', 'CCA' in various Islet
APIs and interfaces, and code-dir / artifact-names to become
'islet' or ISLET, as appropriate. This naming is inline with
what was done for Keystone-based simple-app.

There are just a couple of remaining instances of cca, only
used for shim-produced 'bins', in the driver shell script.

Deleted file src/certifier.proto had crept back in. Delete it again.
…r repo. Add CI-checks

This commit does a one-time apply of default 'gofmt' formatting to all
.go files in the Certifier repo. To prevent future divergence, a new
CI/scripts/check-gofmt.sh script is added, which gets invoked in early
steps of build.yml . CI-builds will fail if any gofmt violations are
seen.
…iles

This commit cleans up docs related to simple_app_under_keystone
and simple_app_under_islet:
 - Adds islet/instructions.md;
 - Fixes sample-app README.md to list 2 new sample-apps
 - renames temporary_instructions.txt to runnable 'script'
 - Other misc edits, corrections
…rtifier-framework-for-confidential-computing into keystone-example-app
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants