#319 fix typo on standard-test-source example
Merged 5 years ago by astepano. Opened 5 years ago by bgoncalv.
bgoncalv/standard-test-roles fix-source-role  into  master

@@ -41,8 +41,8 @@ 

        tags:

        - classic

        vars:

-         # standard-test-basic directory for tests are relative to {{ env_workdir }}

-         env_workdir: /var/test

+         # standard-test-basic directory for tests are relative to {{ tenv_workdir }}

+         tenv_workdir: /var/test

        pre_tasks:

        - import_role:

            name: standard-test-source
@@ -52,7 +52,7 @@ 

        - name: Copy files including source to test environment

          synchronize:

            src: "{{ playbook_dir }}/.."

-           dest: "{{ env_workdir }}"

+           dest: "{{ tenv_workdir }}"

            mode: push

            ssh_args: "-o UserKnownHostsFile=/dev/null"

  
@@ -66,10 +66,10 @@ 

          tests:

          - prepare-source:

              dir: ./

-             run: rpmbuild -bp {{env_workdir}}/*.spec --nodeps --define "_sourcedir {{env_workdir}}" --define "_builddir {{env_workdir}}/source"

+             run: rpmbuild -bp {{tenv_workdir}}/*.spec --nodeps --define "_sourcedir {{tenv_workdir}}" --define "_builddir {{tenv_workdir}}/source"

          - flatten-source:

              dir: ./

-             run: shopt -s dotglob; mv {{env_workdir}}/source/*/* {{env_workdir}}/source

+             run: shopt -s dotglob; mv {{tenv_workdir}}/source/*/* {{tenv_workdir}}/source

          - smoke:

              dir: ./source

              run: make check

the variable should be tenv_workdir and not env_workdir

@bgoncalv thank you for spotting this.

Commit ef2bc10 fixes this pull-request

Pull-Request has been merged by astepano

5 years ago

Pull-Request has been merged by astepano

5 years ago