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

Release 1.3.4 #486

Merged
merged 14 commits into from
Nov 1, 2024
Merged

Release 1.3.4 #486

merged 14 commits into from
Nov 1, 2024

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Oct 17, 2024

Release for the updated inflation formula from #471

This release only affects the Polkadot runtime.

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Copy link
Contributor

@acatangiu acatangiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't it need a spec bump to the Polkadot Relay Chain runtime?

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Copy link
Contributor

@acatangiu acatangiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to the CHANGELOG the only diff between 1.3.3 and this proposed 1.3.4 is:

Change Polkadot inflation to 120M DOT per year (#471)

So why bump and release all (unaffected) runtimes? We could bump and release only Polkadot Relay.

@ggwpez
Copy link
Member Author

ggwpez commented Oct 21, 2024

So why bump and release all (unaffected) runtimes? We could bump and release only Polkadot Relay.

Yes true. We did partial bumps in the past, and it then becomes difficult to know what runtimes need to be bumped to what version on the next release.
Bumping all to 1.3.4 in the 1.3.4 release keeps the rule simple - just bump all to the release version.

@acatangiu
Copy link
Contributor

Releasing same runtime again but with a different spec version is not cool IMO. It adds unnecessary operational overhead to do useless runtime upgrades.

I suggest we achieve what you describe by bumping all relevant runtimes to latest spec version.

In this case, only Polkadot Relay gets 1.3.4. E.g. later say we want to release AH runtime, it would get 1.3.5 (and so will any other runtime that gets changes between now and then).

difficult to know what runtimes need to be bumped to what version on the next release

"what runtimes" -> check the changelog or git history
"what version" -> always use next "globally unique" version (1.3.4 now, next 1.3.5 or 1.4.0)

@ggwpez
Copy link
Member Author

ggwpez commented Oct 23, 2024

It adds unnecessary operational overhead to do useless runtime upgrades.

We dont upgrade them. They are just deleted from the release page. But it makes it simpler to ctrl+f update them all.

"what version" -> always use next "globally unique" version (1.3.4 now, next 1.3.5 or 1.4.0)

Yea we can also do that. Will change now to get this merged.

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Copy link
Contributor

@seadanda seadanda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to re-run benchmarks for MQ bump? AHK and Encointer bumps LGTM

source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "20e65a37881d1998546254a5e50a1f768b3f82deabe774e750f4ea95aba8030c"
checksum = "8bb779dbfd58b3a4e6a84245392fe9d8c31a746ff7c6db414e91c012663859c3"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To pick up this fix for other runtimes we need to bump them too. Not sure if the fix is urgent, if the other system chains have any overweight stuck messages in the queue.
But if they don't, I don't personally think it's worth the runtime upgrade, and we can go without bumping them.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good Q. I think it should be fine to only bump PAH. Last time we checked it was only Hydra affected (Astar was false positive).

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Dónal Murray <[email protected]>
@ggwpez
Copy link
Member Author

ggwpez commented Nov 1, 2024

/merge

@fellowship-merge-bot fellowship-merge-bot bot enabled auto-merge (squash) November 1, 2024 15:45
@fellowship-merge-bot
Copy link
Contributor

Enabled auto-merge in Pull Request

Available commands
  • /merge: Enables auto-merge for Pull Request
  • /merge cancel: Cancels auto-merge for Pull Request
  • /merge help: Shows this menu

For more information see the documentation

@fellowship-merge-bot fellowship-merge-bot bot merged commit 3b18d94 into main Nov 1, 2024
45 of 49 checks passed
@ggwpez ggwpez deleted the oty-1.3.4 branch November 1, 2024 18:29
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.

7 participants