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

AA: Support CoCo-AS Attestation Token #439

Closed
wants to merge 3 commits into from

Conversation

jialez0
Copy link
Member

@jialez0 jialez0 commented Jan 18, 2024

No description provided.

attestation-agent/token/Cargo.toml Show resolved Hide resolved
attestation-agent/lib/src/lib.rs Show resolved Hide resolved
attestation-agent/lib/src/lib.rs Show resolved Hide resolved
attestation-agent/lib/Cargo.toml Show resolved Hide resolved
attestation-agent/token/src/lib.rs Show resolved Hide resolved
attestation-agent/token/src/lib.rs Show resolved Hide resolved
attestation-agent/lib/src/lib.rs Show resolved Hide resolved
attestation-agent/lib/src/lib.rs Show resolved Hide resolved
use std::path::Path;
use std::sync::OnceLock;
use tokio::fs;

const PEER_POD_CONFIG_PATH: &str = "/run/peerpod/daemon.json";
pub const DEFAULT_AA_CONFIG_PATH: &str = "/etc/attestation.toml";
Copy link
Member

@Xynnn007 Xynnn007 Jan 19, 2024

Choose a reason for hiding this comment

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

Maybe attestation-agent.toml?

Also there are some interesting tradeoffs. There was once a discussion with @danmihai1 , he suggested to use JSONs for readibility in initdata. I prefer TOML as we can add comments but JSONs cannot.

Copy link
Member

Choose a reason for hiding this comment

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

Since the primary PR is about supporting attestation token, I think the changes to add config file mechanism should be in a separate PR.

This will help with the discussions and review especially w.r.to format, name and extensibility

cc @Xynnn007 @fitzthum

@jialez0 jialez0 closed this Jan 19, 2024
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