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

Unique Chain 1.14 #1096

Open
wants to merge 139 commits into
base: develop
Choose a base branch
from
Open

Unique Chain 1.14 #1096

wants to merge 139 commits into from

Conversation

vklachkov
Copy link

Much-anticipated update of Unique to Polkadot 1.14.

This update has been prepared through the joint efforts of @fairax, @BuddyGlas, myself, and @mrshiposha.

In this update

  • Updated all dependencies
  • Setuped async backing
  • Fixed migrations issues that have persisted since 0.9 versions
  • Fixed a lot of tests
  • Revived our CI

Despite the release of version 1.16.1, we decided to keep the SDK version at 1.14 as it has already been tested and works great.

Regarding the previous PR #1091, I have restructured the commit history and corrected all messages, as well as added the missing changes.

Update note №1

The node in this release with Async Backing performs very poorly. To address this, it was decided to update the node separately to the latest Polkadot SDK to enable Fork Aware Tx Pool in a separate branch. After the new release from Parity, the branches may be merged.

Update note №2

The release has been delayed so much that many outdated branches with intermediate changes have formed during the process. After a successful release, we need to remember to clean up.

@BuddyGlas BuddyGlas added the CI-xcm for CI workflows label Dec 1, 2024
@BuddyGlas BuddyGlas closed this Dec 1, 2024
@BuddyGlas BuddyGlas reopened this Dec 1, 2024
@BuddyGlas BuddyGlas added CI-just-tests just tests without update and removed CI-xcm for CI workflows labels Dec 9, 2024
@BuddyGlas BuddyGlas removed the CI-just-tests just tests without update label Dec 9, 2024
modify:: bdk.mixer([
m.genericRelay($),
{
genesis+: {
Copy link
Member

Choose a reason for hiding this comment

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

Обновите на simplify+_genesis, выяснили что runtimeGenesis формат устарел

Copy link
Collaborator

Choose a reason for hiding this comment

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

это тестовый файл, на него не обращай внимания.

@@ -26,7 +26,7 @@ local relay = {
wantedKeys: 'relay',
expectedDataPath: '/parity',
},
for name in ['alice', 'bob', 'charlie', 'dave', 'eve', 'ferdie']
for name in ['alice', 'bob', 'charlie', 'dave', 'eve', 'ferdie', 'gregory', 'holly', 'iggy', 'john', 'kurt']
Copy link
Member

Choose a reason for hiding this comment

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

  • пропущена конфигурация async-backing, для последней версии реле она требуется

.baedeker/xcm-unique.jsonnet Outdated Show resolved Hide resolved
# of it to this day (Ignoring incremental compilation, which is explicitly disabled by CARGO_INCREMENTAL=0). Only the
# fully built crates will be cached depending on the crate source, version and feature set, which are locked by
# Cargo.lock, which can't be updated because of the --locked flag.
RUN --mount=type=cache,target=/cargo-home/registry \
Copy link
Member

Choose a reason for hiding this comment

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

Кеш был для локальных сборок, он не вредит, и игнорируется с --no-cache который стоит у нас на CI, давайте его оставим, я его уже N раз возвращал, но его N+1 раз удаляли)

.docker/Dockerfile-unique-base-image Show resolved Hide resolved
CARGO_INCREMENTAL=0 cargo build --release --features="${FEATURES}" --locked && \
mv ./target/release/unique-collator /unique_parachain/unique-chain/
mv ./target/release/unique-collator /workdir/unique-chain/ && \
cd target/release/wbuild && find . -name "*.wasm" -exec sh -c 'mkdir -p "../../../wasm/$(dirname {})"; cp {} "../../../wasm/{}"' \;
Copy link
Member

Choose a reason for hiding this comment

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

Для /wasm давайте абсолютный путь использовать, а то читать тяжело)

@@ -138,7 +169,9 @@ jobs:
inputs: |
.baedeker/forkless-data.jsonnet
snippet:(import 'baedeker-library/ops/rewrites.libsonnet').rewriteNodePaths({'bin/polkadot':{dockerImage:'${{ steps.polkadot.outputs.image }}:${{ steps.polkadot.outputs.tag }}'}})
snippet:(import 'baedeker-library/ops/rewrites.libsonnet').rewriteNodePaths({'bin/unique':{dockerImage:'${{ steps.latest.outputs.name }}'}}, extra_node_mixin={extraArgs: []})
# with data build uses old runtime, but new node, thus we use mainnet image for spec generation, and then target image for nodes.
snippet:(import 'baedeker-library/ops/rewrites.libsonnet').rewriteNodePaths({'bin/unique':{dockerImage:'${{ steps.node.outputs.name }}'}}, extra_node_mixin={extraArgs: []}, for_chain = false)
Copy link
Member

Choose a reason for hiding this comment

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

Внимание: тут ОЧИЩАЕТСЯ extraArgs, не уверен что нам это нужно

Copy link
Collaborator

Choose a reason for hiding this comment

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

Этот воркфлоу будет удалён

inputs: |
.baedeker/just-tests.jsonnet
snippet:(import 'baedeker-library/ops/rewrites.libsonnet').rewriteNodePaths({'bin/polkadot':{dockerImage:'${{ steps.polkadot.outputs.image }}:${{ steps.polkadot.outputs.tag }}'}})
snippet:(import 'baedeker-library/ops/rewrites.libsonnet').rewriteNodePaths({'bin/unique':{dockerImage:'${{ steps.node.outputs.name }}'}}, extra_node_mixin={extraArgs: []})
Copy link
Member

Choose a reason for hiding this comment

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

Очищается extraArgs, перепроверьте что это нам нужно

Copy link
Collaborator

Choose a reason for hiding this comment

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

Этот воркфлоу будет удалён

.github/workflows/main-process-update.yml Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-main-process for CI workflows CI-minimal Only execute jobs specified as labels, no defaults.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants