Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

malus: add the nemesis use cases #3639

Closed
wants to merge 157 commits into from
Closed

malus: add the nemesis use cases #3639

wants to merge 157 commits into from

Conversation

drahnr
Copy link
Contributor

@drahnr drahnr commented Aug 13, 2021

Initial draft of a concrete nemesis impl

  • finalize malus node impl
  • add a docker file for usage /w gurke
  • extend by gurke configs
    • add tera to get good coverage around thresholds
  • unify dir layout: simnet_tests with node/malus/integrationtests have some shared ideas, docs, and content. defered, need to clarify the intended scope of simnet_tests

@drahnr drahnr added A3-in_progress Pull request is in progress. No review needed at this stage. I5-tests Tests need fixing, improving or augmenting. B0-silent Changes should not be mentioned in any release notes labels Aug 13, 2021
@drahnr drahnr self-assigned this Aug 13, 2021
@drahnr
Copy link
Contributor Author

drahnr commented Aug 16, 2021

Not sure what's happening, but I hit

  |
  = help: This is a known issue with the compiler. Run `cargo clean -p westend_runtime` or `cargo clean` to allow your project to compile
  = note: Please follow the instructions below to create a bug report with the provided information
  = note: See <https://github.com/rust-lang/rust/issues/84970> for more information

thread 'rustc' panicked at 'Found unstable fingerprints for evaluate_obligation(87dcc842a9ce649f-5ba6d09e73d8d6c1): Ok(EvaluatedToOk)', /rustc/a178d0322ce20e33eac124758e837cbd80a6f633/compiler/rustc_query_system/src/query/plumbing.rs:620:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

error: internal compiler error: unexpected panic

note: the compiler unexpectedly panicked. this is a bug.

note: we would appreciate a bug report: https://github.com/rust-lang/rust/issues/new?labels=C-bug%2C+I-ICE%2C+T-compiler&template=ice.md

note: rustc 1.54.0 (a178d0322 2021-07-26) running on x86_64-unknown-linux-gnu

note: compiler flags: -C embed-bitcode=no -C debuginfo=2 -C incremental --crate-type lib

note: some of the compiler flags provided by cargo are hidden

query stack during panic:
#0 [evaluate_obligation] evaluating trait selection obligation `pallet_transaction_payment::ChargeTransactionPayment<Runtime>: sp_runtime::traits::SignedExtension`
#1 [normalize_projection_ty] normalizing `Canonical { max_universe: U0, variables: [], value: ParamEnvAnd { param_env: ParamEnv { caller_bounds: [], reveal: All }, value: ProjectionTy { substs: [sp_runtime::generic::Block<sp_runtime::generic::Header<u32, sp_runtime::traits::BlakeTwo256>, sp_runtime::generic::UncheckedExtrinsic<sp_runtime::MultiAddress<sp_runtime::AccountId32, ()>, Call, sp_runtime::MultiSignature, (frame_system::CheckSpecVersion<Runtime>, frame_system::CheckTxVersion<Runtime>, frame_system::CheckGenesis<Runtime>, frame_system::CheckMortality<Runtime>, frame_system::CheckNonce<Runtime>, frame_system::CheckWeight<Runtime>, pallet_transaction_payment::ChargeTransactionPayment<Runtime>)>>], item_def_id: DefId(213:1595 ~ sp_runtime[8ec7]::traits::Block::Extrinsic) } } }`
end of query stack
error: aborting due to previous error

awefully lot. Every second compile I have to clean all runtime artifacts.

@drahnr drahnr force-pushed the bernhard-malus-fx branch from b4f74ee to f06f7a4 Compare August 16, 2021 13:25
node/malus/Cargo.toml Outdated Show resolved Hide resolved
@drahnr drahnr marked this pull request as ready for review August 16, 2021 14:17
@drahnr drahnr added the C1-low PR touches the given topic and has a low impact on builders. label Aug 16, 2021
@drahnr drahnr force-pushed the bernhard-malus-fx branch 2 times, most recently from 90e47c9 to d95aec5 Compare August 18, 2021 08:15
ordian and others added 26 commits October 4, 2021 18:48
* master:
  Add extrinsic ordering filtering (#3631)
  chore: ci list files that spellcheck finds (#3992)
  Use background tasks properly in candidate-validation (#4002)
  Fix unoccupied bitfields (#4004)
  Bump syn from 1.0.77 to 1.0.78 (#4006)
* master:
  chore: update `builder` image (#3884)
  Free disputed cores before processing bitfields (#4008)
  Make candidate validation timeouts configurable (#4001)
* master: (40 commits)
  use pipeline-scripts for dependent projects script (#4017)
  companion for substrate/10013 (#4078)
  fix restart issue of staking miner (#4073)
  use `WEIGHT_PER_SECOND` (#4077)
  Switch to compressed runtimes (#4061)
  Create a README for XCMv1 detailing notable changes (#4058)
  Bump proc-macro2 from 1.0.29 to 1.0.30 (#4071)
  Move runtime to with_transaction in order to Rollback state changes (#4067)
  Update wasmtime config to match new version in substrate (#4063)
  Fix XCM Weights on Westend (#4066)
  Dispatchable XCMs should translate to v0 (#4062)
  Add xcm fuzzer (#3869)
  Bump trybuild from 1.0.49 to 1.0.50 (#4060)
  Companion PR for ss58 crate (#3953)
  overseer: simplify debugging some more (#4053)
  Bump thiserror from 1.0.29 to 1.0.30 (#4048)
  Bump strum from 0.21.0 to 0.22.0 (#4047)
  Add para pallets to polkadot benchmarks (#4045)
  Fixes broken docker link in docker descriptiob (#4043)
  Bump trybuild from 1.0.48 to 1.0.49 (#4039)
  ...
@drahnr
Copy link
Contributor Author

drahnr commented Nov 25, 2021

If we want part of the changes to live within the polkadot repo, please create a new PR with the relevant changes. Thank you!

@acatangiu acatangiu deleted the bernhard-malus-fx branch December 12, 2022 10:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-in_progress Pull request is in progress. No review needed at this stage. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. I5-tests Tests need fixing, improving or augmenting.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants