https://redmine.koumbit.net/https://redmine.koumbit.net/themes/purplemine2/favicon/favicon.ico2020-04-10T22:30:14ZKoumbit's RedmineGit hooks - Bug #33101: Ligne manquante dans site/profile/manifests/icinga2/master.pp provoque une erreur de linthttps://redmine.koumbit.net/issues/33101?journal_id=1768592020-04-10T22:30:14ZGabriel Filiongabriel@koumbit.org
<ul></ul><p>j'ai réussi à reproduire. j'ai créé une branche sur le point de départ avec:</p>
<pre>
git checkout 1b802d5e
git branch test_ligne_manquante_redmine_33101
git checkout test_ligne_manquante_redmine_33101
git push origin test_ligne_manquante_redmine_33101
</pre>
<p>donc là le point de départ est sur le puppetmaster.</p>
<p>pour reproduire le bug, on peut déplacer notre branche locale vers le point du merge qui a causé le problème:</p>
<pre>
git merge bb85532b
</pre>
<p>maintenant quand on essaye de faire <code>git push origin test_ligne_manquante_redmine_33101</code> ça devrait donner l'erreur de lint causée par la ligne manquante. on peut pousser ça autant de fois qu'on veut pour tester des choses avec le hook tant qu'on s'arrange pas pour que le hook bypass le check de lint.</p> Git hooks - Bug #33101: Ligne manquante dans site/profile/manifests/icinga2/master.pp provoque une erreur de linthttps://redmine.koumbit.net/issues/33101?journal_id=1768602020-04-10T22:31:22ZGabriel Filiongabriel@koumbit.org
<ul></ul><p>l'output de l'erreur quand on pousse:</p>
<pre>
$ git push origin test_ligne_manquante_redmine_33101
Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
remote: INFO: Skipping review check on commit "bb85532baf9e2fb9dc99f54f728e7f8b3db5b27b" - no changes to Puppetfile{,.lock}
remote: Linting YAML file: hieradata/common.yaml OK
remote: Verifying syntax for file: site/profile/manifests/icinga2/master.pp OK
remote: Verifying lint errors for file: site/profile/manifests/icinga2/master.ppWARNING: indentation of => is not properly aligned (expected in column 15, but found it in column 19) on line 97
remote: WARNING: indentation of => is not properly aligned (expected in column 15, but found it in column 19) on line 98
remote: WARNING: indentation of => is not properly aligned (expected in column 15, but found it in column 19) on line 99
remote: WARNING: indentation of => is not properly aligned (expected in column 15, but found it in column 19) on line 104
remote: WARNING: indentation of => is not properly aligned (expected in column 15, but found it in column 19) on line 105
remote: Puppet lint failed for file: site/profile/manifests/icinga2/master.pp
remote: OK
remote: error: hook declined to update refs/heads/test_ligne_manquante_redmine_33101
To ssh://puppet1.koumbit.net/srv/git/control-repo.git
! [remote rejected] test_ligne_manquante_redmine_33101 -> test_ligne_manquante_redmine_33101 (hook declined)
error: failed to push some refs to 'ssh://puppet1.koumbit.net/srv/git/control-repo.git
</pre> Git hooks - Bug #33101: Ligne manquante dans site/profile/manifests/icinga2/master.pp provoque une erreur de linthttps://redmine.koumbit.net/issues/33101?journal_id=1768612020-04-10T22:36:48ZGabriel Filiongabriel@koumbit.org
<ul><li><strong>Statut</strong> changé de <i>New</i> à <i>In progress</i></li></ul><p>ok j'ai mis le doigt dessus je crois:</p>
<pre>
59 # Avoid the --ignoreimport bug by filtering out import lines
60 git cat-file blob "$new_sha1" | grep -v "^[ ]*import" > "$tmp"
</pre>
<p>la ligne qui manque c'est (le # au début de la ligne c'est vim.. juste pour indication de où c'est. le début de la ligne c'est juste des espaces):</p>
<pre>
100 import_schema => true,
</pre>
<p>donc ça veut dire qu'il faudrait ré-expoter le contenu du fichier sans retirer les lignes import pour faire un check de lint qui a plus d'allure.</p> Git hooks - Bug #33101: Ligne manquante dans site/profile/manifests/icinga2/master.pp provoque une erreur de linthttps://redmine.koumbit.net/issues/33101?journal_id=1768632020-04-10T22:58:53ZGabriel Filiongabriel@koumbit.org
<ul><li><strong>Statut</strong> changé de <i>In progress</i> à <i>Closed</i></li></ul><p>la solution c'était de retirer le grep -v qui enlevait les lignes qui commençaient par "import". c'était un fix à un bug pour une option qui a disparue dans puppet 4.0. donc complètement inutile (et problématique en plus). ciao!</p>
<p>j'ai envoyé le commit 7600627 sur les hooks git, pull'ed ça sur le puppet master, et maintenant je peux pousser le merge problématique sans problème!</p>