pokt-network / poktroll

The official Shannon upgrade implementation of the Pocket Network Protocol implemented using the Cosmos SDK
MIT License
15 stars 8 forks source link

[Proof Module] Add proof_missing_penalty param #596

Closed bryanchriswhite closed 4 months ago

bryanchriswhite commented 5 months ago

Summary

Add proof_missing_penalty param to the proof module.

Issue

Type of change

Select one or more:

Testing

Documentation changes (only if making doc changes)

Local Testing (only if making code changes)

PR Testing (only if making code changes)

Sanity Checklist

github-actions[bot] commented 5 months ago

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks.

You may need to run make trigger_ci to submit an empty commit that'll trigger the tests.

GCP workloads (requires changing the namespace to 596)) Grafana network dashboard for devnet-issue-{issue-id}

github-actions[bot] commented 5 months ago

The image is going to be pushed after the next commit.

You can use make trigger_ci to push an empty commit.

If you also want to run E2E tests, please add devnet-test-e2e label.

bryanchriswhite commented 4 months ago

I've confirmed that the two failing E2E scenarios is a result of a bug that was merged to main and is fixed in #607. I am able to reproduce this specific failure locally and the logs present the same symptoms as this bug, where the proof window start height is much larger than expected:

image