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

Add tdx guide for osdk #849

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Hsy-Intel
Copy link
Contributor

No description provided.

@tatetian
Copy link
Contributor

@StanPlatinum Could you give some feedback for this PR?

@StanPlatinum
Copy link
Contributor

@StanPlatinum Could you give some feedback for this PR?

Sure!

docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
docs/src/SUMMARY.md Outdated Show resolved Hide resolved
docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
-monitor pty \
-no-hpet \
-nodefaults \
-drive file=<Your OS image path in qcow2 format>,if=virtio,format=qcow2 \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This OSDK guide is intended to provide step-by-step instructions so that a reader can follow them to reproduce the result. So we must give concrete values that work out of box. What is the concrete value for <Your OS image path in qcow2 format>?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pending on issue #922.

docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
docs/src/osdk/guide/intel-tdx-guide.md Outdated Show resolved Hide resolved
osdk/tools/Dockerfile.tdx Outdated Show resolved Hide resolved
osdk/tools/Dockerfile.tdx Outdated Show resolved Hide resolved
@Hsy-Intel Hsy-Intel force-pushed the osdk_tdx_env branch 2 times, most recently from 3bf8cc0 to 86909cb Compare June 10, 2024 06:24
Copy link
Contributor

@tatetian tatetian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Now the revised version looks much better, although it still has some minor issues.

osdk/tools/gen_dockerfile.sh Outdated Show resolved Hide resolved
osdk/tools/gen_dockerfile.sh Outdated Show resolved Hide resolved
osdk/tools/build_image.sh Outdated Show resolved Hide resolved
osdk/tools/build_image.sh Show resolved Hide resolved
@Hsy-Intel Hsy-Intel force-pushed the osdk_tdx_env branch 3 times, most recently from 71e14f5 to e714569 Compare June 11, 2024 05:51
@Hsy-Intel
Copy link
Contributor Author

Commit updated. I will verify this PR once issue #922 is solved.

osdk/tools/docker/gen_dockerfile.sh Outdated Show resolved Hide resolved
osdk/tools/docker/gen_dockerfile.sh Outdated Show resolved Hide resolved
osdk/tools/docker/gen_dockerfile.sh Outdated Show resolved Hide resolved
osdk/tools/docker/gen_dockerfile.sh Outdated Show resolved Hide resolved
.github/workflows/docker_build.yml Outdated Show resolved Hide resolved
@Hsy-Intel Hsy-Intel force-pushed the osdk_tdx_env branch 2 times, most recently from e4790ba to 94dfc38 Compare June 11, 2024 08:14
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.

None yet

4 participants