diff mbox series

[1/1] .gitlab-ci.yml: use gitlab 9.0 varaible names

Message ID 20171023142614.11522-1-thomas.preston@codethink.co.uk
State Accepted
Headers show
Series [1/1] .gitlab-ci.yml: use gitlab 9.0 varaible names | expand

Commit Message

Thomas Preston Oct. 23, 2017, 2:26 p.m. UTC
Gitlab 9.0 prefers "job" instead of "build" for CI variables names.
http://docs.gitlab.com/ce/ci/variables/README.html#9-0-renaming

Signed-off-by: Thomas Preston <thomas.preston@codethink.co.uk>
---
 .gitlab-ci.yml    | 4 ++--
 .gitlab-ci.yml.in | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

Comments

Arnout Vandecappelle Nov. 5, 2017, 3 p.m. UTC | #1
On 23-10-17 16:26, Thomas Preston wrote:
> Gitlab 9.0 prefers "job" instead of "build" for CI variables names.
> http://docs.gitlab.com/ce/ci/variables/README.html#9-0-renaming
> 
> Signed-off-by: Thomas Preston <thomas.preston@codethink.co.uk>

 Applied to master, thanks. Good catch! Did you see a warning somewhere or did
it come naturally after looking at the file?

 Regards,
 Arnout

> ---
>  .gitlab-ci.yml    | 4 ++--
>  .gitlab-ci.yml.in | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index ebaac7a3d9..0dddb22f12 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -8,7 +8,7 @@ image: buildroot/base
>  
>  .defconfig_script: &defconfig_script
>      - echo 'Configure Buildroot'
> -    - make ${CI_BUILD_NAME}
> +    - make ${CI_JOB_NAME}
>      - echo 'Build buildroot'
>      - |
>          make > >(tee build.log |grep '>>>') 2>&1 || {
> @@ -51,7 +51,7 @@ check-DEVELOPERS:
>      # runner will clean up those files for us.
>      # Multiply every emulator timeout by 10 to avoid sporadic failures in
>      # elastic runners.
> -    script: ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${CI_BUILD_NAME}
> +    script: ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${CI_JOB_NAME}
>      artifacts:
>          when: always
>          expire_in: 2 weeks
> diff --git a/.gitlab-ci.yml.in b/.gitlab-ci.yml.in
> index 3abf7d5313..33c7b13725 100644
> --- a/.gitlab-ci.yml.in
> +++ b/.gitlab-ci.yml.in
> @@ -8,7 +8,7 @@ image: buildroot/base
>  
>  .defconfig_script: &defconfig_script
>      - echo 'Configure Buildroot'
> -    - make ${CI_BUILD_NAME}
> +    - make ${CI_JOB_NAME}
>      - echo 'Build buildroot'
>      - |
>          make > >(tee build.log |grep '>>>') 2>&1 || {
> @@ -51,7 +51,7 @@ check-DEVELOPERS:
>      # runner will clean up those files for us.
>      # Multiply every emulator timeout by 10 to avoid sporadic failures in
>      # elastic runners.
> -    script: ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${CI_BUILD_NAME}
> +    script: ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${CI_JOB_NAME}
>      artifacts:
>          when: always
>          expire_in: 2 weeks
>
Thomas Preston Nov. 7, 2017, 5:22 p.m. UTC | #2
On 05/11/17 07:00, Arnout Vandecappelle wrote:
> On 23-10-17 16:26, Thomas Preston wrote:
>> Gitlab 9.0 prefers "job" instead of "build" for CI variables names.
>> http://docs.gitlab.com/ce/ci/variables/README.html#9-0-renaming
>>
>> Signed-off-by: Thomas Preston <thomas.preston@codethink.co.uk>
> 
>   Applied to master, thanks. Good catch! Did you see a warning somewhere or did
> it come naturally after looking at the file?
> 

Just looking at the file, it was my first time using gitlab CI. :)

Thanks
diff mbox series

Patch

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index ebaac7a3d9..0dddb22f12 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -8,7 +8,7 @@  image: buildroot/base
 
 .defconfig_script: &defconfig_script
     - echo 'Configure Buildroot'
-    - make ${CI_BUILD_NAME}
+    - make ${CI_JOB_NAME}
     - echo 'Build buildroot'
     - |
         make > >(tee build.log |grep '>>>') 2>&1 || {
@@ -51,7 +51,7 @@  check-DEVELOPERS:
     # runner will clean up those files for us.
     # Multiply every emulator timeout by 10 to avoid sporadic failures in
     # elastic runners.
-    script: ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${CI_BUILD_NAME}
+    script: ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${CI_JOB_NAME}
     artifacts:
         when: always
         expire_in: 2 weeks
diff --git a/.gitlab-ci.yml.in b/.gitlab-ci.yml.in
index 3abf7d5313..33c7b13725 100644
--- a/.gitlab-ci.yml.in
+++ b/.gitlab-ci.yml.in
@@ -8,7 +8,7 @@  image: buildroot/base
 
 .defconfig_script: &defconfig_script
     - echo 'Configure Buildroot'
-    - make ${CI_BUILD_NAME}
+    - make ${CI_JOB_NAME}
     - echo 'Build buildroot'
     - |
         make > >(tee build.log |grep '>>>') 2>&1 || {
@@ -51,7 +51,7 @@  check-DEVELOPERS:
     # runner will clean up those files for us.
     # Multiply every emulator timeout by 10 to avoid sporadic failures in
     # elastic runners.
-    script: ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${CI_BUILD_NAME}
+    script: ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${CI_JOB_NAME}
     artifacts:
         when: always
         expire_in: 2 weeks