Diff comments:

> diff --git a/jenkins/jobs/ansible_playbook.groovy 
> b/jenkins/jobs/ansible_playbook.groovy
> index 2c953c0..ce7d330 100644
> --- a/jenkins/jobs/ansible_playbook.groovy
> +++ b/jenkins/jobs/ansible_playbook.groovy
> @@ -38,6 +30,15 @@ pipeline {
>                  """
>              }
>          }
> +        stage('Fetch commit sha') {
> +            steps {
> +                script {
> +                    env.COMMIT_SHA = sh(script: "git ls-remote 
> ${params.ANSIBLE_REPO} ${params.ANSIBLE_BRANCH} | cut -c1-40", returnStdout: 
> true).trim()

instead of cut with columns, just use `awk '{print $1}'` to get the first 
space-separated word and then we don't need to care if the sha1 changes length

> +                    println "commit: ${env.COMMIT_SHA}"
> +                }
> +                updateCommitStatus("${params.ANSIBLE_REPO}", 
> "${env.COMMIT_SHA}", "${env.BUILD_URL}", "PENDING")
> +            }
> +        }
>          stage('System Tests') {
>              steps {
>                  writeFile file: 'system-tests/config.yaml', text: """


-- 
https://code.launchpad.net/~maas-committers/maas-ci/+git/maas-ci-config/+merge/438519
Your team MAAS Committers is requested to review the proposed merge of 
~maas-committers/maas-ci/+git/maas-ci-config:fix-commit-sha-collection into 
~maas-committers/maas-ci/+git/maas-ci-config:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp

Reply via email to