Exotic

Décision prise en CT :

• https://www.grid5000.fr/w/TechTeam:CT-310#bug_.2311160:Support_de_ARM.28David_L..2FPierre.2C_arm.29
• https://www.grid5000.fr/w/TechTeam:CT-311#bug_.2311160:Support_de_ARM.28David_L..2FPierre.2C_arm.29

Dans cette MR :

• Ajout du champs "exotic" en tant que "optional_boolean" :
  • Input : au niveau de input/.../cluster.yaml ou input/.../nodes/nodes-x.yaml
  • Output : au niveau du node.json ( = false si champs absent du input )
  • Actuellement, exotic = true sur pyxis, exotic = false sur tous les autres clusters
• Modification du générateur de propriétés OAR pour ajouter la propriété "exotic"
• Modification du générateur wiki Hardware
  • Change la colonne "queue" en "restrictions" pour y faire apparaitre les clusters réservable avec le type de job "exotic"
  • Nouvelle page Hardware global : https://www.grid5000.fr/w/User:Dloup/exotic_global_hardware
  • Nouvelle page Hardware de site : https://www.grid5000.fr/w/User:Dloup/exotic_site_hardware

changed title from WIP : Exotic to WIP: Exotic

changed the description

added 13 commits

• 59792079...b56c1190 - 8 commits from branch master
• 27ec2684 - [lib/reference-api,valid/schema] Add 'exotic' field as an optional Boolean
• a7b728be - Generate ref api with 'exotic' field (true pyxis, false for others)
• e114071e - Add exotic properties to oar generator and validator
• 297351b0 - [wiki:gen hardware] Replace 'queue' columns with 'restrictions' to inform about exotic job type
• 33bacde6 - [rspec] Update input json and add exotic to stub_oar_properties

Compare with previous version

unmarked as a Work In Progress

changed the description

Quelques remarques/questions:

• exotic=yes apparait dans data/.../clusters/pyxis.yaml et dans data/.../cluster/nodes/pyxis-<N>.yaml, mais exotic=no n'apparait que dans data/.../clusters/nodes/<node>-<N>.yaml mais pas dans le yaml data des clusters. Est-ce ce qu'on veut effectivement ?
• le titre Restrictions dans les page wiki me semble pas très compréhensible: je propose Acces condition, qui tiendra sur 2 lignes dans le header des tableaux, mais ce n'est pas un soucis.
• dans l'affichage des queues/restrictions/...
  • Avant, on affichait default là ou il n'y avait pas besoin d'utiliser la queue testing ou production (ce qui était d'ailleurs trompeur, car les queues utilisables sont en fait default + besteffort + challenge (+ admin), → default ne se rapportait donc pas un nom de la queue, bien que cela soit un nom de queue.
  • Dans la proposition actuelle, on n'affiche rien quand il n'y pas pas de "restriction" (ou "access condition") → je propose d'afficher none.
• j'ai l'impression que le generateur se base sur les données input et non data ? (test queue == '', alors que dans data, queue est tjs defini ?)

@lnussbau un avis ?

• Pour le 1er point, ce n'est pas le comportement voulu en effet, c'est corrigé dans le commit 812fb787 : On a maintenant "exotic": false par défaut au niveau du data/.../cluster.json si rien n'est spécifié en input.

• Access condition me parait aussi plus claire que Restrictions, par contre j'ai testé le none au lieu d'un champs vide et je trouve que ça rends la tableau moins lisible. Je propose de simplement changer Restrictions en Access condition et laisser la case vide si il n'y en a pas, comme c'est fait pour la colonne Accelerators par exemple.

• Le générateur prends bien les data ( récupérés avec G5K::get_global_hash -> load_data_hierarchy ). C'est à partir de la ligne 202 que queue peut être vide : queue = cluster_hash['queues'] - ['admin', 'default']

je n'ai pas regardé le code, mais l'état actuel de la discussion avec les propositions me semble OK

assigned to @lnussbau

added 15 commits

• 33bacde6...14607c52 - 9 commits from branch master
• 142b9133 - [lib/reference-api,valid/schema] Add 'exotic' field as an optional Boolean
• 34b35a65 - Generate ref api with 'exotic' field (true pyxis, false for others)
• fcac8e27 - Add exotic properties to oar generator and validator
• fecfe2dc - [wiki:gen hardware] Replace 'queue' columns with 'restrictions' to inform about exotic job type
• 28a1102e - [rspec] Update input json and add exotic to stub_oar_properties
• 812fb787 - [gen/reference-api] Set 'exotic=no' at cluster level if not specified in input yaml

Compare with previous version

added 1 commit

• 0c090b13 - [gen/wiki/site_hardware] Change column 'Restrictions' to 'Access Condition'

Compare with previous version

c'est ok pour moi

pouce haut aussi

added 14 commits

• 0c090b13...8592bbc7 - 7 commits from branch master
• 9ddafbb1 - [lib/reference-api,valid/schema] Add 'exotic' field as an optional Boolean
• 5e8eaeda - Generate ref api with 'exotic' field (true pyxis, false for others)
• 109a64ec - Add exotic properties to oar generator and validator
• e16a3619 - [wiki:gen hardware] Replace 'queue' columns with 'restrictions' to inform about exotic job type
• 5847a36f - [rspec] Update input json and add exotic to stub_oar_properties
• d4a25f46 - [gen/reference-api] Set 'exotic=no' at cluster level if not specified in input yaml
• 4c44f7ab - [gen/wiki/site_hardware] Change column 'Restrictions' to 'Access Condition'

Compare with previous version

merged

mentioned in commit 5dca20e5