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

runtime: Bump Oasis SDK to 0.10.0 and prepare 0.9.0-testnet #445

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

matevz
Copy link
Member

@matevz matevz commented Oct 24, 2024

No description provided.

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for oasisprotocol-sapphire-paratime canceled.

Name Link
🔨 Latest commit 627e429
🔍 Latest deploy log https://app.netlify.com/sites/oasisprotocol-sapphire-paratime/deploys/671b4a655f149e00088c1cbc

@matevz matevz force-pushed the matevz/feature/prepare-0.9.0-testnet branch from bfd2e10 to 9f09542 Compare October 24, 2024 12:23
@matevz matevz requested a review from kostko October 24, 2024 12:28
kostko
kostko previously approved these changes Oct 24, 2024
@kostko kostko dismissed their stale review October 24, 2024 15:52

Stale.

@kostko
Copy link
Member

kostko commented Oct 24, 2024

We need to add explicit keys for mock SGX to support localnet builds. For trusted_signers you need to specify TrustedSigners::unsafe_mock under mock SGX.

@CedarMist
Copy link
Member

This works :D Thanks

    #[cfg(feature = "debug-mock-sgx")]
    fn trusted_signers() -> Option<TrustedSigners> {
        return Some(TrustedSigners::unsafe_mock());
    }

    #[cfg(not(feature = "debug-mock-sgx"))]
    fn trusted_signers() -> Option<TrustedSigners> {
        let tps = keymanager::trusted_policy_signers();
        // The `keymanager` crate may use a different version of `oasis_core`
        // so we need to convert the `TrustedSigners` between the versions.
        Some(TrustedSigners {
            signers: tps.signers.into_iter().map(|s| PublicKey(s.0)).collect(),
            threshold: tps.threshold,
        })
    }

@matevz matevz force-pushed the matevz/feature/prepare-0.9.0-testnet branch from 9f09542 to 3f77e0f Compare October 25, 2024 06:47
runtime/src/lib.rs Outdated Show resolved Hide resolved
@matevz matevz force-pushed the matevz/feature/prepare-0.9.0-testnet branch from 3f77e0f to 0ee88ac Compare October 25, 2024 07:07
runtime/src/lib.rs Outdated Show resolved Hide resolved
runtime/Cargo.toml Outdated Show resolved Hide resolved
@matevz matevz force-pushed the matevz/feature/prepare-0.9.0-testnet branch 3 times, most recently from 0a420a2 to 7964d17 Compare October 25, 2024 07:25
@matevz matevz force-pushed the matevz/feature/prepare-0.9.0-testnet branch 3 times, most recently from 9bd3230 to a03a86c Compare October 25, 2024 07:32
@matevz matevz force-pushed the matevz/feature/prepare-0.9.0-testnet branch from a03a86c to 627e429 Compare October 25, 2024 07:36
@matevz matevz merged commit 624ac5d into main Oct 25, 2024
12 checks passed
@matevz matevz deleted the matevz/feature/prepare-0.9.0-testnet branch October 25, 2024 07:53
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