Skip to content

github: Fix attest invalid image name #114

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

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

TimePrinciple
Copy link

Summary of the PR

Strip tag from IMAGE_NAME used for attest, which should not contain column and tag. IMAGE_NAME is now specified explicitly to rustvmm/dev.

Requirements

Before submitting your PR, please make sure you addressed the following
requirements:

  • All commits in this PR have Signed-Off-By trailers (with
    git commit -s), and the commit message has max 60 characters for the
    summary and max 75 characters for each description line.
  • All added/changed functionality has a corresponding unit/integration
    test.
  • All added/changed public-facing functionality has entries in the "Upcoming
    Release" section of CHANGELOG.md (if no such section exists, please create one).
  • Any newly added unsafe code is properly documented.

@TimePrinciple TimePrinciple force-pushed the fix-invalid-image-name branch from f6207bd to f6998d7 Compare August 30, 2024 14:54
Add `print-registry` and `print-image-name` to `docker.sh`, enabling
`docker.sh` to work with attestation.

Co-authored-by: Jiayi Yin <jiayi@iscas.ac.cn>
Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
@TimePrinciple TimePrinciple force-pushed the fix-invalid-image-name branch from f6998d7 to 457b4ed Compare August 30, 2024 15:00
@roypat roypat merged commit 44e5d18 into rust-vmm:main Aug 30, 2024
2 of 3 checks passed
@TimePrinciple TimePrinciple deleted the fix-invalid-image-name branch August 30, 2024 15:31
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.

4 participants