Simplify / harmonize packaging step in Jenkinsfile (#74572) #47

Merged
aberriot merged 1 commits from wip/74572-simplify-jenkinsfile into main 2023-02-20 14:05:38 +01:00
Owner
No description provided.
aberriot force-pushed wip/74572-simplify-jenkinsfile from 4c25601c27 to 8052ec17cb 2023-02-16 10:19:59 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from 8052ec17cb to f62fe81162 2023-02-16 10:22:26 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from f62fe81162 to d4771480cc 2023-02-16 10:23:45 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from d4771480cc to 70410bff0f 2023-02-16 10:42:51 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from 70410bff0f to 2f63ebc859 2023-02-16 10:43:16 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from 2f63ebc859 to 7f40f1ebd2 2023-02-16 10:50:49 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from 7f40f1ebd2 to 509e3d0c7e 2023-02-16 10:51:26 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from 509e3d0c7e to f51360c638 2023-02-16 10:54:38 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from f51360c638 to 9244d9ffec 2023-02-16 10:56:27 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from 9244d9ffec to 11602b50b3 2023-02-16 10:57:35 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from 11602b50b3 to c419ede8e6 2023-02-16 10:59:04 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from c419ede8e6 to 8882fa2064 2023-02-16 11:00:08 +01:00 Compare
aberriot force-pushed wip/74572-simplify-jenkinsfile from 8882fa2064 to 0e4f021d72 2023-02-16 11:00:27 +01:00 Compare
aberriot reviewed 2023-02-16 11:01:13 +01:00
Jenkinsfile Outdated
@ -33,0 +33,4 @@
script: '''
echo "${JOB_NAME}" | sed "s/gitea\\///" | awk -F/ '{print $1}'
'''
).trim()
Author
Owner

Basé sur https://stackoverflow.com/a/59359806/2844093, pour sortir la bonne partie du job name et la passer à EObuilder.

Basé sur https://stackoverflow.com/a/59359806/2844093, pour sortir la bonne partie du job name et la passer à EObuilder.
aberriot reviewed 2023-02-16 11:03:48 +01:00
Jenkinsfile Outdated
@ -33,0 +37,4 @@
if (env.BRANCH_NAME == 'main') {
sh "sudo -H -u eobuilder /usr/local/bin/eobuilder -d bullseye ${SHORT_JOB_NAME}"
} else if (env.BRANCH_NAME.startsWith('hotfix/')) {
sh "sudo -H -u eobuilder /usr/local/bin/eobuilder -d bullseye --branch ${env.BRANCH_NAME} --hotfix ${SHORT_JOB_NAME}"
Author
Owner

Cette partie là devient indépendante du projet, on peut copier/coller d'un projet à l'autre sans soucis. J'utilise env.BRANCH_NAME au lieu de env.GIT_BRANCH, car normalement cette variable ne contient pas de prefixe type origin/.

Sur une PR, BRANCH_NAME ressemble à gitea/combo/PR-47, mais autrement, ça doit matcher la branche git en train d'être buildée.

Cette partie là devient indépendante du projet, on peut copier/coller d'un projet à l'autre sans soucis. J'utilise `env.BRANCH_NAME` au lieu de `env.GIT_BRANCH`, car normalement cette variable ne contient pas de prefixe type `origin/`. Sur une PR, BRANCH_NAME ressemble à `gitea/combo/PR-47`, mais autrement, ça doit matcher la branche git en train d'être buildée.
Owner

On pourrait avoir if (env.BRANCH_NAME == 'main' || .BRANCH_NAME == 'origin/main') {, pour garantie sur l'existant ?

Pour bien tout vérifier, si tu veux faire des tests sur la branche main, tu peux migrer le projet https://git.entrouvert.org/bidon.git/ vers gitea, là-dedans tu pourras commiter directement sur main pour tester.

On pourrait avoir `if (env.BRANCH_NAME == 'main' || .BRANCH_NAME == 'origin/main') {`, pour garantie sur l'existant ? Pour bien tout vérifier, si tu veux faire des tests sur la branche main, tu peux migrer le projet https://git.entrouvert.org/bidon.git/ vers gitea, là-dedans tu pourras commiter directement sur main pour tester.
Author
Owner

Ah génial, je vais faire mes tests sur le projet bidon pour m'assurer de la compat sur main (déjà ça plante avec le code de la PR).

Ah génial, je vais faire mes tests sur le projet bidon pour m'assurer de la compat sur main (déjà ça plante avec le code de la PR).
Author
Owner

Du coup, BRANCH_NAME n'est simplement pas disponible sur les jobs existants. Donc j'ai rebasculé sur GIT_BRANCH, avec un or pour gérer le préfixe avec origin/, cf 80d7fb2fbb/Jenkinsfile (L22-L25)

(qui passe comme attendu et builde un package, cf https://jenkins.entrouvert.org/job/bidon/1067/console)

Du coup, BRANCH_NAME n'est simplement pas disponible sur les jobs existants. Donc j'ai rebasculé sur GIT_BRANCH, avec un or pour gérer le préfixe avec origin/, cf https://gitea.entrouvert.org/entrouvert/bidon/src/commit/80d7fb2fbbff8540efc94ef031b503102bcb3037/Jenkinsfile#L22-L25 (qui passe comme attendu et builde un package, cf https://jenkins.entrouvert.org/job/bidon/1067/console)
aberriot changed title from WIP: Simplify / harmonize packaging step in Jenkinsfile (#74572) to Simplify / harmonize packaging step in Jenkinsfile (#74572) 2023-02-16 11:04:01 +01:00
aberriot force-pushed wip/74572-simplify-jenkinsfile from 0e4f021d72 to 4e23c0cfff 2023-02-16 11:44:52 +01:00 Compare
csechet approved these changes 2023-02-20 10:33:04 +01:00
aberriot merged commit d84c97351f into main 2023-02-20 14:05:38 +01:00
aberriot deleted branch wip/74572-simplify-jenkinsfile 2023-02-20 14:05:38 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/combo#47
No description provided.