Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

[CI] Test SGX remote attestation examples on Ubuntu 18.04 and 20.04 #2534

Merged
merged 1 commit into from
Jul 19, 2021

Conversation

dimakuv
Copy link

@dimakuv dimakuv commented Jul 13, 2021

Description of the changes

Previously, it was tested only on Ubuntu 18.04 (bionic) due to concerns over sending too many EPID requests to the Intel Attestation Service (IAS). This shouldn't be an issue anymore.

How to test this PR?

Trigger Jenkins several times to be sure.


This change is Reviewable

@dimakuv
Copy link
Author

dimakuv commented Jul 13, 2021

Jenkins, test this please

1 similar comment
@dimakuv
Copy link
Author

dimakuv commented Jul 13, 2021

Jenkins, test this please

mkow
mkow previously approved these changes Jul 14, 2021
Copy link
Member

@mkow mkow left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, all discussions resolved, not enough approvals from maintainers (1 more required)

Previously, it was tested only on Ubuntu 18.04 (bionic) due to concerns
over sending too many EPID requests to the Intel Attestation Service
(IAS). This shouldn't be an issue anymore.

Signed-off-by: Dmitrii Kuvaiskii <dmitrii.kuvaiskii@intel.com>
@dimakuv dimakuv force-pushed the dimakuv/enable-epid-on-both-cis branch from 47f1e7b to ea8c0d3 Compare July 19, 2021 08:25
Copy link
Member

@mkow mkow left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@dimakuv dimakuv merged commit ea8c0d3 into master Jul 19, 2021
@dimakuv dimakuv deleted the dimakuv/enable-epid-on-both-cis branch July 19, 2021 12:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants