Resolve "Industrialize Releases production"

Closes #125 (closed)

assigned to @c-geek

added 1 commit

• 9872d5cf - fix(#125 (closed)): dissociate genesis config from genesis migration

Compare with previous version

mentioned in commit 1d48d12e

mentioned in merge request !176 (merged)

added 6 commits

• 9872d5cf...b145f36f - 3 commits from branch master
• 40823408 - fix(#125 (closed)): dissociate genesis config from genesis migration
• 10eee6cc - fix(#125 (closed)): remove identities from genesis config file (gdev.json)
• 60b5a7d6 - fix(#125 (closed)): GenesisConfig must be derived from GenesisData (GDev)

Compare with previous version

added 4 commits

• 28ba7b98 - fix(#125 (closed)): start GDev chains refactoring
• 6fc41d0e - fix(#125 (closed)): allow indexer export file through `DUNITER_GENESIS_EXPORT` env var
• c3da2a25 - fix(#125 (closed)): config(all()) for multiple required features
• f419837f - fix(#125 (closed)): first working gdev_live

Compare with previous version

added 1 commit

• f1d99e21 - fix(#125 (closed)): ignore "session_keys" error if authority is forced

Compare with previous version

added RN-binary label

changed milestone to %runtime-600

added 1 commit

• 5ed52cb0 - fix(#125 (closed)): clippy

Compare with previous version

added 2 commits

• df8e673b - fix(#125 (closed)): industrialize the start of a new currency
• c8fea737 - fix(#125 (closed)): naive indexer export

Compare with previous version

Cette branche est basée sur b145f36f avant la fusion de !172 (merged) qui introduit plusieurs modifications liés à la mise à jour de substrate. Je propose de la reviewer une première fois sans tenir compte de ces changements puis de m'occuper moi-même du rebase sur master parce qu'il y aura probablement pas mal de conflits.

Ce code ne compile pas avec la feature gtest.

cargo check --features gtest --no-default-features 

donne "borrowed data escapes outside of function".

Il y a quelques trucs à corriger, donc.

How about splitting this MR into two parts:

• a first one related to the code
• a second one related to the CI / release stuff

And I think we could save some time with a meeting about the second part, so that we understand each other about which problems it is trying to solve.

Une première passe de réponses.

Ne vous formalisez pas trop sur le Dockerfile, je dois le remettre en ordre. J'avais besoin d'un espace de travail pour tester mes modifications, mais je vais les mettre à part dans le fichier une fois que tout sera bien rôdé de mon côté.

J'ai créé une branche hugo-125-merge-master où je merge cette branche avec master pour anticiper la résolution de conflits. Je corrige aussi certains warnings, la compilation des tests, et adapte certains tests à la nouvelle valeur du existential deposit. Par contre, il faut modifier les fichiers json des tests end2end pour que le format de clé soit accepté.

Une première passe de réponses.

Ne vous formalisez pas trop sur le Dockerfile, je dois le remettre en ordre. J'avais besoin d'un espace de travail pour tester mes modifications, mais je vais les mettre à part dans le fichier une fois que tout sera bien rôdé de mon côté.

added 2 commits

• eb06efdc - fix(#125 (closed)): review: documentation
• e13931ae - fix(#125 (closed)): feat: transaction_history

Compare with previous version

added 1 commit

• edd76a6e - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit

• b9e4cf31 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit

• d0a5deb5 - fix(#125 (closed)): review: invalid pubkeys (len 33)

Compare with previous version

Une première passe de réponses.

Ne vous formalisez pas trop sur le Dockerfile, je dois le remettre en ordre. J'avais besoin d'un espace de travail pour tester mes modifications, mais je vais les mettre à part dans le fichier une fois que tout sera bien rôdé de mon côté.

added 1 commit

• 07714a17 - fix(#125 (closed)): review: pallet-im-online position in Cargo.toml

Compare with previous version

added 1 commit

• fda2ec7b - fix(#125 (closed)): review: dead code

Compare with previous version

added 5 commits

• c914b396 - fix(#125 (closed)): review: typo
• ff48760d - fix(#125 (closed)): review: remove test log
• b8dc4d61 - fix(#125 (closed)): review: invalid pubkeys (len 33)
• 1557202e - fix(#125 (closed)): review: pallet-im-online position in Cargo.toml
• 0730417a - fix(#125 (closed)): review: dead code

Compare with previous version

added 1 commit

• 51fed219 - fix(#125 (closed)): review: gtest does not compile

Compare with previous version

added 1 commit

• 5b6db601 - fix(#125 (closed)): tmp: disable release creation

Compare with previous version

added 1 commit

• 1861539d - fix(#125 (closed)): tmp: disable release creation (fix)

Compare with previous version

added 1 commit

• 4fc4f735 - fix(#125 (closed)): tmp: disable release creation (fix 2)

Compare with previous version

added 1 commit

• 6955e2c3 - fix(#125 (closed)): fix: fmt --check

Compare with previous version

added 1 commit

• b57e068c - fix(#125 (closed)): fix: clippy

Compare with previous version

added 1 commit

• 4bc600ce - fix(#125 (closed)): fix: clippy

Compare with previous version

added 1 commit

• f4bd8bde - fix(#125 (closed)): review: revert Dockerfile changes + build on podman

Compare with previous version

Seconde passe : j'ai revert complètement le Dockerfile, je n'avais pas besoin de le modifier.

Aussi : je résout les threads, mais si vous n'êtes pas d'accord avec ma réponse vous pouvez rouvrir. Je résout simplement pour y voir plus clair.

added 1 commit

• d5e691f1 - fix(#125 (closed)): fix: broken tests (existential deposit 200 -> 100)

Compare with previous version

added 1 commit

• bb51146d - fix(#125 (closed)): fix: end2end GENESIS_CONFIG files

Compare with previous version

added 2 commits

• 240fb981 - fix(#125 (closed)): fix: e2e tests were keeping the node running of failing tests
• 1d254dba - fix(#125 (closed)): fix: e2e: use --chain=gdev_dev

Compare with previous version

added 2 commits

• ebdc3fcf - fix(#125 (closed)): debug(e2): explicitely say that issuer does not exist
• d0547ba2 - fix(#125 (closed)): fix: clippy

Compare with previous version

added 1 commit

• f01e0313 - fix(#125 (closed)): feat(e2e): add --no-spawn option

Compare with previous version