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

Add api-name in cannot query the runtime API version warning #3653

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

alexggh
Copy link
Contributor

@alexggh alexggh commented Mar 12, 2024

Sometimes we see nodes printing this warning:

cannot query the runtime API version: Api called for an unknown Block: State already discarded for

The log is harmless, but let's print the api we got this for, so that we can track its call site and truly confirm it is harmless or fix it.

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
@tdimitrov tdimitrov added the R0-silent Changes should not be mentioned in any release notes label Mar 12, 2024
@tdimitrov tdimitrov enabled auto-merge March 12, 2024 08:19
@tdimitrov tdimitrov added this pull request to the merge queue Mar 12, 2024
Merged via the queue into master with commit 1ead597 Mar 12, 2024
133 of 136 checks passed
@tdimitrov tdimitrov deleted the alexaggh/more_logs branch March 12, 2024 09:07
ordian added a commit that referenced this pull request Mar 16, 2024
* master: (65 commits)
  collator protocol changes for elastic scaling (validator side) (#3302)
  Contracts use polkavm workspace deps (#3715)
  Add elastic scaling support in ParaInherent BenchBuilder  (#3690)
  Removes `as [disambiguation_path]` from `derive_impl` usage (#3652)
  fix(paseo-spec): New Paseo Bootnodes (#3674)
  Improve Penpal runtime + emulated tests (#3543)
  Staking ledger bonding fixes (#3639)
  DescribeAllTerminal for HashedDescription (#3349)
  Increase timeout for assertions (#3680)
  Add subsystems regression tests to CI (#3527)
  Always print connectivity report (#3677)
  Revert "FRAME: Create `TransactionExtension` as a replacement for `SignedExtension` (#2280)" (#3665)
  authority-discovery: Add log for debugging DHT authority records (#3668)
  Construct Runtime v2  (#1378)
  Support for `keyring` in runtimes (#2044)
  Add api-name in `cannot query the runtime API version` warning (#3653)
  Add a PolkaVM-based executor (#3458)
  Adds default config for assets pallet (#3637)
  Bump handlebars from 4.3.7 to 5.1.0 (#3248)
  [Collator Selection] Fix weight refund for `set_candidacy_bond` (#3643)
  ...
dharjeezy pushed a commit to dharjeezy/polkadot-sdk that referenced this pull request Mar 24, 2024
…ytech#3653)

Sometimes we see nodes printing this warning:
```
cannot query the runtime API version: Api called for an unknown Block: State already discarded for
```

The log is harmless, but let's print the api we got this for, so that we
can track its call site and truly confirm it is harmless or fix it.

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants