Skip to content

AS: fix build with rvps features#553

Merged
Xynnn007 merged 1 commit into
confidential-containers:mainfrom
Xynnn007:fix-rvps-feature
Nov 22, 2024
Merged

AS: fix build with rvps features#553
Xynnn007 merged 1 commit into
confidential-containers:mainfrom
Xynnn007:fix-rvps-feature

Conversation

@Xynnn007
Copy link
Copy Markdown
Member

@Xynnn007 Xynnn007 commented Nov 1, 2024

Before this commit when we only enable restful-bin,rvps-grpc features, the attestation service cannot be built.

This patch fixes this issue. Also, this commit deletes the rvps-builtin feature as the code of rvps does not bring in any new dependencies.

Related configurations are updated to make it more robust, together with the documents.

@Xynnn007
Copy link
Copy Markdown
Member Author

Xynnn007 commented Nov 1, 2024

Let's mark this as draft because there is some overlap to #514

@Xynnn007 Xynnn007 force-pushed the fix-rvps-feature branch 2 times, most recently from b1d224c to 2e8cbe9 Compare November 7, 2024 05:47
Before this commit when we only enable `restful-bin,rvps-grpc` features,
the attestation service cannot be built.

This patch fixes this issue. Also, this commit deletes the `rvps-builtin`
feature as the code of rvps does not bring in any new dependencies.

Related configurations are updated to make it more robust, together with
the documents.

Signed-off-by: Xynnn007 <xynnn@linux.alibaba.com>
@Xynnn007
Copy link
Copy Markdown
Member Author

cc @lmilleri @spotlesstofu this might influence operator side i am not sure

@Xynnn007 Xynnn007 marked this pull request as ready for review November 16, 2024 10:03
@Xynnn007 Xynnn007 requested a review from a team as a code owner November 16, 2024 10:03
@lmilleri
Copy link
Copy Markdown
Member

cc @lmilleri @spotlesstofu this might influence operator side i am not sure

@Xynnn007 thank you for the heads-up!

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