GTest - Runtime upgrade 1100

Si en fait en sélectionnant la troncature on peut copier et ça copie bien l’intégralité du code. Contre intuitif je sais.

C’est ce que j’expliquais ici:

Raison de plus pour re-proposer via system.authorizeUpgrade : vous n’aurez pas besoin de copier le code pour vérifier le proposal

Je n’y arrive pas, essayes : Duniter Portal

Ou alors tu parles d’autre chose.

1 Like

Oui, je parle de la page qui s’ouvre quand on clic ici depuis la page des preimages:

1 Like

Je l’avais fait comme ça à l’époque : Proposition runtime 900. Avec authorizeUpgrade.

1 Like

Proposition 9

@technical-committee j’ai donc refait la proposition avec la nouvelle manière décrite par @elois : Duniter Portal

@HugoTrentesaux si tu peux revoter ^^

En effet le hash du runtime est directement visible là (codeHash: H256), beaucoup plus simple, plus besoin de toute cette procédure de vérification du hash, jsute compilez en local pour vérifier que vous avez bien le bon hash !

0x87592e7a3f47558f2bb2194e236629bab8fbe894675a5d3e91662499c07c9cf8

4 Likes

Ah oui c’est bon cette fois ! Désolé pas l’habitude.

bash-5.2$ WASM_BYTES=$(cat /Users/cgeek/set_code_1100)
bash-5.2$ echo $WASM_BYTES | xxd -r -p | python3 -c   "import sys, hashlib; print('0x' + hashlib.blake2b(sys.stdin.buffer.read(), digest_size=32).hexdigest())"
0x87592e7a3f47558f2bb2194e236629bab8fbe894675a5d3e91662499c07c9cf8

J’ai voté les deux propositions :

g1cli --no-indexer -u wss://gtest.cgeek.fr -S cesium tech vote 0xe663a5af20c61416c8aaa7332db35986bcacefe6f158345011417977d4028d35 8 1
g1cli --no-indexer -u wss://gtest.cgeek.fr -S cesium tech vote 0x805fabfbbc254c9588f3bd16bccc49d02c047518a0b1af975ae8a44400da9b7a 9 1
2 Likes

A voté !
J’ai rencontré des problèmes de permission sur les répertoires runtime et release et proposé une MR.
La MR pointe vers la branche de runtime, je rebaserai si besoin…

1 Like

Précision : j’avais migré mon identité mais j’ai dû utiliser l’ancienne clé pour voter.

@aya j’ai push sur ta branche pour améliorer tes changements afin de ne pas utiliser root pour garder des permissions propres sur les fichiers générés:

Détails:
  • retire -u 0:0, sed /etc/passwd et su builder
  • utilise --user <uid>:<gid> (hôte)
  • garde l’accès toolchain avec --group-add (configurable via SRTOOL_BUILDER_GID, défaut 1001)

EDIT: j’ai rebase sur runtime/gtest-1100 et merge

2 Likes

Bien vu c’est plus propre en effet.

1 Like

Merci @aya pour le fix et merci @elois d’avoir mergé :slight_smile:

Ah ben je peux pas voter :

"Inability to pay some fees (e.g. account balance too low)"

tu dois reùettre des sous sur cette clé.

@Pini tu ne peux pas voter de toute façon car tu n’es pas dans le comité technique

1 Like

Ah ben j’étais plus sûr :slight_smile:
Comme ça rien à faire !

J’ai eu un doute vu que j’étais notifié des posts sur ce fil.

Les membres du comité technique gtest sont:

g1Kah4oS1PLp8N6QwbiVXpqfgqomoF1offxbtvx2aeMZbEvHe
g1KcETSPpBk7A84ZQp5tTt5Z1cjMCp8Qetrk2Z5MBdPtj57ha
g1L1ebhZUTNzbnHgKDGKwTCkFBgqc55WP6EC53saBTA6sGgaU
g1M5PtxdtmkVnaL6GFT94h6FW7MfF2rTfBitVkDtKX2kBTkQL
g1PH3gQ4n53EqpcLVLtG77jRRo71tLUjvDYcm87MENkAa1NPz
g1PiFLg2KQAHJej733C8fbpoZLHuiusCXdKcSktDYs4vgYdVb
g1PqkJ26truXzzYLKmgrHRr4o4V7Gje5UZwPSmvXUm8tJ4eJJ
g1QEs1J7Z6wpXZUe3t3KxYmhbXUeUJx283YJpu9uZrWEcZzJP

1 Like

@Moul @vit @1000i100 @tuxmain plus que 2 votes nécessaired pour executer ce runtime upgrade et tester les correctif d’elois avant le lancement de Samedi :slight_smile:

2 Likes

J’ai voté pour !
Cette fois, la vérification du hash ressemble à ce que j’ai connu par le passé. Bien plus simple. Merci.
Je valide les changements qui ont eu lieu. Je ne suis pas assez compétent pour review/auditer le code rust et je n’aurais pas le temps. J’ai regardé ce qui a été fait à l’échelle du changelog, des MR et des échanges sur le forum. Ça me semble cohérent et puis à un moment il faut faire confiance.

1 Like

Compilation en cours…
Je voterai dans la foulée avec Tikka, c’est tellement simple avec Tikka ! (petite autopromotion… :wink: ).

2 Likes