Mentions légales du service

Skip to content
Snippets Groups Projects

[oar-properties] fix get oar properties error header

Merged Pierre Neyron requested to merge fix_get_oar_properties_error_header into master
1 unresolved thread

pour le diff, j'ai pas trouvé ou corriger, mais c'est pas tres lisible ce "Output format" Output format: ['~', 'key', 'old value', 'new value'] chifflot-1: ["-", "gpu", 18] ["-", "gpudevice", "1"] chifflot-2: ["-", "gpu", 20] ["-", "gpudevice", "1"] [...] ["chifflot-1", "sdb.chifflot-1"]: OK

-> "+" c'est les propriétés ajoutées, "-" les propriétés enlevées, "~" celles modifiées

Merge request reports

Pipeline #86885 passed

Pipeline passed for e9bf86e5 on fix_get_oar_properties_error_header

Merged by Lucas NussbaumLucas Nussbaum 6 years ago (Jul 12, 2019 1:33pm UTC)

Loading

Pipeline #86886 passed

Pipeline passed for 3d5e6b50 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
673 673 when 2
674 674 # Give more details
675 675 if header == false
676 puts "Output format: ['~', 'key', 'old value', 'new value']"
676 puts "Output format: [ '-', 'key', 'value'] for missing, [ '+', 'key', 'value'] for added, ['~', 'key', 'old value', 'new value'] for changed properties"
  • bah "missing" = "removed" or "never added" ?

  • pour le cas present, on a - devant gpu et gpudevice car ils ne sont pas encore dans le ref repo, si je comprends tout.

  • je crois pas qu'il y ait une notion de "propriété dans le ref-repo". Ou alors je ne comprends pas ce que ça veut dire.

    Je crois que ce code compare les propriétés générées, et celles qui sont présentes dans OAR. Mais c'est peut-être à clarifier aussi dans la sortie.

  • Pierre Neyron added 1 commit

    added 1 commit

    • 007b9722 - [oar-properties] fix get oar properties error header

    Compare with previous version

  • Pierre Neyron added 1 commit

    added 1 commit

    • e9bf86e5 - [oar-properties] fix get oar properties error header

    Compare with previous version

  • Lucas Nussbaum enabled an automatic merge when the pipeline for e9bf86e5 succeeds

    enabled an automatic merge when the pipeline for e9bf86e5 succeeds

  • Lucas Nussbaum mentioned in commit 3d5e6b50

    mentioned in commit 3d5e6b50

  • Please register or sign in to reply
    Loading