Skip to content

test_verify_tee_expired_attestation_triggers_resharing is flaky #1900

Closed as not planned

Description

@gilcu3

Background

test test_verify_tee_expired_attestation_triggers_resharing was introduced recently in #1725 and seems to be flaky:

thread 'sandbox::tee::test_verify_tee_expired_attestation_triggers_resharing' panicked at crates/contract/tests/sandbox/tee.rs:660:14:
    expected Resharing state, got Running(RunningContractState { domains: DomainRegistry { domains: [DomainConfig { id: DomainId(0), scheme: Secp256k1 }], next_domain_id: 2 }, keyset: Keyset { epoch_id: EpochId(5), domains: [KeyForDomain { domain_id: DomainId(0), key: Secp256k1 { near_public_key: PublicKey { data: [1, 228, 165, 129, 33, 42, 127, 18, 178, 247, 71, 203, 41, 192, 7, 227, 100, 119, 95, 206, 53, 191, 29, 130, 8, 79, 246, 116, 250, 191, 9, 17, 174, 93, 201, 170, 28, 121, 198, 103, 113, 72, 177, 156, 100, 123, 35, 30, 68, 168, 50, 115, 47, 62, 39, 235, 4, 165, 2, 138, 210, 39, 17, 71, 44] } }, attempt: AttemptId(0) }] }, parameters: ThresholdParameters { participants: Participants { next_id: ParticipantId(3), participants: [(AccountId("dev-20260129092359-57050154508676.test.near"), ParticipantId(0), ParticipantInfo { url: "127.0.0.1", sign_pk: PublicKey { data: [0, 70, 107, 211, 11, 235, 181, 203, 137, 128, 194, 23, 93, 112, 245, 186, 25, 206, 117, 166, 115, 237, 204, 115, 121, 120, 219, 193, 23, 161, 190, 21, 97] } }), (AccountId("dev-20260129092359-48724867042456.test.near"), ParticipantId(1), ParticipantInfo { url: "127.0.0.1", sign_pk: PublicKey { data: [0, 16, 240, 175, 180, 167, 110, 81, 29, 45, 141, 41, 0, 181, 34, 148, 117, 63, 236, 48, 181, 243, 125, 4, 126, 216, 8, 21, 236, 90, 205, 132, 144] } }), (AccountId("dev-20260129092359-75784008687763.test.near"), ParticipantId(2), ParticipantInfo { url: "127.0.0.1", sign_pk: PublicKey { data: [0, 5, 168, 158, 89, 238, 104, 243, 163, 96, 4, 193, 81, 106, 141, 0, 154, 135, 193, 24, 166, 142, 199, 250, 120, 251, 89, 78, 55, 248, 210, 173, 241] } })] }, threshold: Threshold(2) }, parameters_votes: ThresholdParametersVotes { proposal_by_account: {} }, add_domains_votes: AddDomainsVotes { proposal_by_account: {} }, previously_cancelled_resharing_epoch_id: None })

example runs:

User Story

No response

Acceptance Criteria

The test is fixed so that it is no longer flaky

Resources & Additional Notes

No response

Activity

  1. added theissue type on Jan 29, 2026
  2. kevindeforth commented on Jan 30, 2026

    @kevindeforth
    Contributor
  3. self-assigned this
    on Feb 2, 2026
  4. pbeza commented on Feb 23, 2026

    @pbeza
    Contributor

    Reopening this — it’s still flaky as of 23.02.2026:
    https://github.com/near/mpc/actions/runs/22303646112/job/64517376434?pr=2169

  5. reopened this on Feb 23, 2026
  6. gilcu3 commented on Feb 23, 2026

    @gilcu3
    ContributorAuthor

    Reopening this — it’s still flaky as of 23.02.2026: https://github.com/near/mpc/actions/runs/22303646112/job/64517376434?pr=2169

    I believe the error for this one is slightly different, sharing here for future reference:

    thread 'sandbox::tee::test_verify_tee_expired_attestation_triggers_resharing' panicked at crates/contract/tests/sandbox/tee.rs:664:5:
        verify_tee call failed: ExecutionFinalResult { total_gas_burnt: NearGas { inner: 3225543660406 }, transaction: ExecutionOutcome { transaction_hash: 4Jdi8NZvwA6WDx1F7WrEKvZt9urQp2mDBxyfQsfVPmPK, block_hash: 8X4YsLbSpPxZ6U6j6hD5AdHcDhNp56YUTjL3eGuT9jAm, logs: [], receipt_ids: [E1rabLDBBtg3CfBU9XR3Z3doayL4PgLmZhU4WtyYzbeL], gas_burnt: NearGas { inner: 308631704580 }, tokens_burnt: NearToken { inner: 30863170458000000000 }, executor_id: AccountId("dev-20260223111522-52416385007290.test.near"), status: SuccessReceiptId(E1rabLDBBtg3CfBU9XR3Z3doayL4PgLmZhU4WtyYzbeL) }, receipts: [ExecutionOutcome { transaction_hash: E1rabLDBBtg3CfBU9XR3Z3doayL4PgLmZhU4WtyYzbeL, block_hash: ESXktMG2zxwM2N7urHxy1MHTJ3zvq1v1PPemrqGBfBJS, logs: ["verify_tee: signer=dev-20260223111522-52416385007290.test.near"], receipt_ids: [FmeN8m7y5kkxaDDNp2LCaAqouZuPPhnbJfASu49Hq8rd], gas_burnt: NearGas { inner: 2693729393326 }, tokens_burnt: NearToken { inner: 269372939332600000000 }, executor_id: AccountId("dev-20260223111522-43636279420496.test.near"), status: Failure(ActionError(ActionError { index: Some(0), kind: FunctionCallError(ExecutionError("Smart contract panicked: The protocol is not Running.")) })) }, ExecutionOutcome { transaction_hash: FmeN8m7y5kkxaDDNp2LCaAqouZuPPhnbJfASu49Hq8rd, block_hash: J8PJ5DiSbzpTJuxYCpKxDXDYPDJZGKGoE7CC9j8WkznF, logs: [], receipt_ids: [], gas_burnt: NearGas { inner: 223182562500 }, tokens_burnt: NearToken { inner: 0 }, executor_id: AccountId("dev-20260223111522-52416385007290.test.near"), status: SuccessValue('') }], status: Failure(ActionError(ActionError { index: Some(0), kind: FunctionCallError(ExecutionError("Smart contract panicked: The protocol is not Running.")) })) }
    

    If it begins failing with noticeable frequency I will work on it again, for the moment I will assume it was a one-off. We still could have those at any time, given that our tests have timeouts that might fail in extraordinary cases

  7. removed their assignment
    on Feb 23, 2026
  8. netrome commented on May 4, 2026

    @netrome
    Collaborator

    Has anyone still seen this? If there are no more notes of this failure in 60 days let's close it.

  9. netrome commented on May 4, 2026

    @netrome
    Collaborator

    This might have made the test less flaky #2742

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions