Skip to content

[fix] update max interval and enable audi timestamp#2470

Merged
kamilsa merged 8 commits into
masterfrom
feature/audi-timestamp
Jun 18, 2025
Merged

[fix] update max interval and enable audi timestamp#2470
kamilsa merged 8 commits into
masterfrom
feature/audi-timestamp

Conversation

@kamilsa

@kamilsa kamilsa commented Jun 18, 2025

Copy link
Copy Markdown
Contributor

Referenced issues

#2351

Description of the Change

  • Returns audi timestamp
  • Decreases address publishing interval to 5 minutes
  • Starts publishing addresses before we became an authority (same behaviour as in polkadot-sdk)

Possible Drawbacks

Too frequent publishing interval comparing to polkadot sdk

Checklist Before Opening a PR

Before you open a Pull Request (PR), please make sure you've completed the following steps and confirm by answering 'Yes' to each item:

  1. Code is formatted: Have you run your code through clang-format to ensure it adheres to the project's coding standards? Yes
  2. Code is documented: Have you added comments and documentation to your code according to the guidelines in the project's contributing guidelines? Yes
  3. Self-review: Have you reviewed your own code to ensure it is free of typos, syntax errors, logical errors, and unresolved TODOs or FIXME without linking to an issue? Yes
  4. Zombienet Tests: Have you ensured that the zombienet tests are passing? Zombienet is a network simulation and testing tool used in this project. It's important to ensure that these tests pass to maintain the stability and reliability of the project. Yes

@kamilsa kamilsa requested review from turuslan and xDimon June 18, 2025 04:42
Comment thread core/authority_discovery/publisher/address_publisher.cpp Outdated
@kamilsa kamilsa force-pushed the feature/audi-timestamp branch from f54a0f7 to ffe71b8 Compare June 18, 2025 07:14
@kamilsa kamilsa force-pushed the feature/audi-timestamp branch from ffe71b8 to e7b1424 Compare June 18, 2025 07:16
@kamilsa kamilsa enabled auto-merge (squash) June 18, 2025 08:30
@kamilsa kamilsa merged commit 44cde1e into master Jun 18, 2025
19 of 26 checks passed
@kamilsa kamilsa deleted the feature/audi-timestamp branch June 18, 2025 09:24
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.

3 participants