)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"16b5e20964c3930f58a6848fc0b2c85f4fa83a60","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Note: removed set -u because this can break source activate on old"},{"line_number":18,"context_line":"versions of virtualenv."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Partial-Bug: #1859167"},{"line_number":21,"context_line":"Change-Id: I80bc67c6a17d87c24a2b90ca1577aa1f20902cf6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3fa7e38b_0c95e987","line":20,"updated":"2020-01-10 13:20:45.000000000","message":"that one ? https://bugs.launchpad.net/tripleo/+bug/1858890\n\n[EDIT]: looks like duplicates maybe","commit_id":"6c048ae6499b00c3a07ab825d2914a0136298a33"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"c5a590f196fd6a1daab2cee2eba1c5e4718b75e9","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Note: removed set -u because this can break source activate on old"},{"line_number":18,"context_line":"versions of virtualenv."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Partial-Bug: #1859167"},{"line_number":21,"context_line":"Change-Id: I80bc67c6a17d87c24a2b90ca1577aa1f20902cf6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3fa7e38b_ecb3ad33","line":20,"in_reply_to":"3fa7e38b_0c95e987","updated":"2020-01-10 13:23:46.000000000","message":"Yes, correct but I do not want to delay the patch even more. If something is broken and we have rebuild lets add this to commit message too.","commit_id":"6c048ae6499b00c3a07ab825d2914a0136298a33"}],"roles/build-test-packages/tasks/dlrn-build.yml":[{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"0b7f5746cce1b0c08cd77b9cfecf8167e3e53285","unresolved":false,"context_lines":[{"line_number":35,"context_line":"  shell: \u003e"},{"line_number":36,"context_line":"    set -euo pipefail;"},{"line_number":37,"context_line":"    source {{ build_repo_dir }}/dlrn-venv/bin/activate;"},{"line_number":38,"context_line":"    export PROJECT_NAME\u003d$(basename \"{{ artg_change.project }}\" );"},{"line_number":39,"context_line":"    rdopkg findpkg -s $PROJECT_NAME -l rdoinfo | grep ^name | awk \u0027{print $2}\u0027"},{"line_number":40,"context_line":"  args:"},{"line_number":41,"context_line":"    chdir: \u0027{{ build_repo_dir }}/DLRN\u0027"},{"line_number":42,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3fa7e38b_0174d2ce","line":39,"range":{"start_line":38,"start_character":24,"end_line":39,"end_character":64},"updated":"2020-01-10 12:59:33.000000000","message":"Why overcomplicate instead of using {{ foo | basename }} ?","commit_id":"1c7810f83c739f722950878e1772aec3d7702187"},{"author":{"_account_id":27329,"name":"Federico Ressi","email":"fressi@redhat.com","username":"fressi_redhat"},"change_message_id":"0b967f16f58f325c308ca8e69097b4f3cbf6b13f","unresolved":false,"context_lines":[{"line_number":35,"context_line":"  shell: \u003e"},{"line_number":36,"context_line":"    set -euo pipefail;"},{"line_number":37,"context_line":"    source {{ build_repo_dir }}/dlrn-venv/bin/activate;"},{"line_number":38,"context_line":"    export PROJECT_NAME\u003d$(basename \"{{ artg_change.project }}\" );"},{"line_number":39,"context_line":"    rdopkg findpkg -s $PROJECT_NAME -l rdoinfo | grep ^name | awk \u0027{print $2}\u0027"},{"line_number":40,"context_line":"  args:"},{"line_number":41,"context_line":"    chdir: \u0027{{ build_repo_dir }}/DLRN\u0027"},{"line_number":42,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3fa7e38b_ec0f2dd1","line":39,"range":{"start_line":38,"start_character":24,"end_line":39,"end_character":64},"in_reply_to":"3fa7e38b_0174d2ce","updated":"2020-01-10 13:38:21.000000000","message":"I did it in the next change where I integrated your fixes too:\n https://review.opendev.org/#/c/701921/","commit_id":"1c7810f83c739f722950878e1772aec3d7702187"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"51f0e0c43c328450dc4a173e047d734d2f9cbb07","unresolved":false,"context_lines":[{"line_number":35,"context_line":"  shell: \u003e"},{"line_number":36,"context_line":"    set -eo pipefail;"},{"line_number":37,"context_line":"    source {{ build_repo_dir }}/dlrn-venv/bin/activate;"},{"line_number":38,"context_line":"    rdopkg findpkg -s {{ artg_change.project | basename }} -l rdoinfo | grep ^name | awk \u0027{print $2}\u0027"},{"line_number":39,"context_line":"  args:"},{"line_number":40,"context_line":"    chdir: \u0027{{ build_repo_dir }}/DLRN\u0027"},{"line_number":41,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_ccd35101","line":38,"updated":"2020-01-10 13:53:56.000000000","message":"still failing for me at http://logs.rdoproject.org/39/24339/4/check/periodic-tripleo-ci-centos-7-multinode-1ctlr-featureset010-master/91fd840/job-output.txt\n\nline looks correct \n\n2020-01-10 13:35:07.551724 | primary |     \"cmd\": \"set -eo pipefail; source /home/zuul/dlrn-venv/bin/activate; rdopkg findpkg -s testproject -l rdoinfo | grep ^name | awk \u0027{print $2}\u0027\\n\",\n\nso can we remove the -e ?","commit_id":"6c048ae6499b00c3a07ab825d2914a0136298a33"},{"author":{"_account_id":27329,"name":"Federico Ressi","email":"fressi@redhat.com","username":"fressi_redhat"},"change_message_id":"3954c38f010624f4df1fea1c4b00298f4b905f81","unresolved":false,"context_lines":[{"line_number":35,"context_line":"  shell: \u003e"},{"line_number":36,"context_line":"    set -eo pipefail;"},{"line_number":37,"context_line":"    source {{ build_repo_dir }}/dlrn-venv/bin/activate;"},{"line_number":38,"context_line":"    rdopkg findpkg -s {{ artg_change.project | basename }} -l rdoinfo | grep ^name | awk \u0027{print $2}\u0027"},{"line_number":39,"context_line":"  args:"},{"line_number":40,"context_line":"    chdir: \u0027{{ build_repo_dir }}/DLRN\u0027"},{"line_number":41,"context_line":"  changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_e722e6b6","line":38,"in_reply_to":"3fa7e38b_ccd35101","updated":"2020-01-10 15:09:25.000000000","message":"I think the real problem here is setting set -eo pipefail before sourcing activate script","commit_id":"6c048ae6499b00c3a07ab825d2914a0136298a33"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"42f44c5860b1f63ba45e3ed74d6a7e13d8802cbf","unresolved":false,"context_lines":[{"line_number":47,"context_line":"          WARNING: Unable to build {{ artg_change.project }}"},{"line_number":48,"context_line":"          {{ project_name_mapped }}"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"- when:"},{"line_number":51,"context_line":"    - project_name_mapped is success"},{"line_number":52,"context_line":"    - project_name_mapped.stdout !\u003d \u0027\u0027  # noqa 602"},{"line_number":53,"context_line":"  block:"},{"line_number":54,"context_line":"    - name: Append project name to package list"},{"line_number":55,"context_line":"      set_fact:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3fa7e38b_415ab582","side":"PARENT","line":52,"range":{"start_line":50,"start_character":1,"end_line":52,"end_character":50},"updated":"2020-01-13 13:08:39.000000000","message":"why is this removed?","commit_id":"2e001a3b119d7a0729dd2be803ec7f1f411a0926"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"9b0ef579447a40dfe400545f496a2c59682c5c12","unresolved":false,"context_lines":[{"line_number":47,"context_line":"          WARNING: Unable to build {{ artg_change.project }}"},{"line_number":48,"context_line":"          {{ project_name_mapped }}"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"- when:"},{"line_number":51,"context_line":"    - project_name_mapped is success"},{"line_number":52,"context_line":"    - project_name_mapped.stdout !\u003d \u0027\u0027  # noqa 602"},{"line_number":53,"context_line":"  block:"},{"line_number":54,"context_line":"    - name: Append project name to package list"},{"line_number":55,"context_line":"      set_fact:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3fa7e38b_6113316a","side":"PARENT","line":52,"range":{"start_line":50,"start_character":1,"end_line":52,"end_character":50},"in_reply_to":"3fa7e38b_415ab582","updated":"2020-01-13 13:20:21.000000000","message":"Done","commit_id":"2e001a3b119d7a0729dd2be803ec7f1f411a0926"},{"author":{"_account_id":27329,"name":"Federico Ressi","email":"fressi@redhat.com","username":"fressi_redhat"},"change_message_id":"fa5ce5fdb56dc0ac59aab267a05ea7cdf76789c6","unresolved":false,"context_lines":[{"line_number":47,"context_line":"          WARNING: Unable to build {{ artg_change.project }}"},{"line_number":48,"context_line":"          {{ project_name_mapped }}"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"- when:"},{"line_number":51,"context_line":"    - project_name_mapped is success"},{"line_number":52,"context_line":"    - project_name_mapped.stdout !\u003d \u0027\u0027  # noqa 602"},{"line_number":53,"context_line":"  block:"},{"line_number":54,"context_line":"    - name: Append project name to package list"},{"line_number":55,"context_line":"      set_fact:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3fa7e38b_e48def36","side":"PARENT","line":52,"range":{"start_line":50,"start_character":1,"end_line":52,"end_character":50},"in_reply_to":"3fa7e38b_415ab582","updated":"2020-01-13 13:49:24.000000000","message":"project_name_mapped will fail if project_name_mapped.stdout is \u0027\u0027 (see line 44)","commit_id":"2e001a3b119d7a0729dd2be803ec7f1f411a0926"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"42f44c5860b1f63ba45e3ed74d6a7e13d8802cbf","unresolved":false,"context_lines":[{"line_number":109,"context_line":"      register: repo_built"},{"line_number":110,"context_line":"      when: artg_build_one|bool"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"- name: Ensure artg_rdo_packages is defined"},{"line_number":113,"context_line":"  set_fact:"},{"line_number":114,"context_line":"    artg_rdo_packages: []"},{"line_number":115,"context_line":"  when: artg_rdo_packages is not defined"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3fa7e38b_e14e41c2","side":"PARENT","line":115,"range":{"start_line":112,"start_character":0,"end_line":115,"end_character":40},"updated":"2020-01-13 13:08:39.000000000","message":"why removed?","commit_id":"2e001a3b119d7a0729dd2be803ec7f1f411a0926"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"9b0ef579447a40dfe400545f496a2c59682c5c12","unresolved":false,"context_lines":[{"line_number":109,"context_line":"      register: repo_built"},{"line_number":110,"context_line":"      when: artg_build_one|bool"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"- name: Ensure artg_rdo_packages is defined"},{"line_number":113,"context_line":"  set_fact:"},{"line_number":114,"context_line":"    artg_rdo_packages: []"},{"line_number":115,"context_line":"  when: artg_rdo_packages is not defined"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3fa7e38b_8118ad83","side":"PARENT","line":115,"range":{"start_line":112,"start_character":0,"end_line":115,"end_character":40},"in_reply_to":"3fa7e38b_e14e41c2","updated":"2020-01-13 13:20:21.000000000","message":"Done","commit_id":"2e001a3b119d7a0729dd2be803ec7f1f411a0926"},{"author":{"_account_id":27329,"name":"Federico Ressi","email":"fressi@redhat.com","username":"fressi_redhat"},"change_message_id":"fa5ce5fdb56dc0ac59aab267a05ea7cdf76789c6","unresolved":false,"context_lines":[{"line_number":109,"context_line":"      register: repo_built"},{"line_number":110,"context_line":"      when: artg_build_one|bool"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"- name: Ensure artg_rdo_packages is defined"},{"line_number":113,"context_line":"  set_fact:"},{"line_number":114,"context_line":"    artg_rdo_packages: []"},{"line_number":115,"context_line":"  when: artg_rdo_packages is not defined"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3fa7e38b_4482c301","side":"PARENT","line":115,"range":{"start_line":112,"start_character":0,"end_line":115,"end_character":40},"in_reply_to":"3fa7e38b_e14e41c2","updated":"2020-01-13 13:49:24.000000000","message":"It is already defined this in defaults/main.yaml file","commit_id":"2e001a3b119d7a0729dd2be803ec7f1f411a0926"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"42f44c5860b1f63ba45e3ed74d6a7e13d8802cbf","unresolved":false,"context_lines":[{"line_number":46,"context_line":"  rescue:"},{"line_number":47,"context_line":"    - debug:"},{"line_number":48,"context_line":"        msg: |"},{"line_number":49,"context_line":"          WARNING: Unable to build {{ artg_change.project }}: package not found."},{"line_number":50,"context_line":"          --- STDERR ---"},{"line_number":51,"context_line":"          {{ project_name_mapped.stderr }}"},{"line_number":52,"context_line":"          --- STDERR ---"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"- when: project_name_mapped is success"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3fa7e38b_21ca1919","line":52,"range":{"start_line":49,"start_character":10,"end_line":52,"end_character":24},"updated":"2020-01-13 13:08:39.000000000","message":"You are missing to display the the return code (rc), which is important.\n\nAlso package not found may be mistleading error.","commit_id":"e3beef4bbfc8b34831844f807fe090405fde7213"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"9b0ef579447a40dfe400545f496a2c59682c5c12","unresolved":false,"context_lines":[{"line_number":46,"context_line":"  rescue:"},{"line_number":47,"context_line":"    - debug:"},{"line_number":48,"context_line":"        msg: |"},{"line_number":49,"context_line":"          WARNING: Unable to build {{ artg_change.project }}: package not found."},{"line_number":50,"context_line":"          --- STDERR ---"},{"line_number":51,"context_line":"          {{ project_name_mapped.stderr }}"},{"line_number":52,"context_line":"          --- STDERR ---"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"- when: project_name_mapped is success"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3fa7e38b_210d3940","line":52,"range":{"start_line":49,"start_character":10,"end_line":52,"end_character":24},"in_reply_to":"3fa7e38b_21ca1919","updated":"2020-01-13 13:20:21.000000000","message":"Done","commit_id":"e3beef4bbfc8b34831844f807fe090405fde7213"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"37f5e0c3d94b2c4a54a1775cba9cb01edd05642e","unresolved":false,"context_lines":[{"line_number":50,"context_line":"          --- STDERR ---"},{"line_number":51,"context_line":"          {{ project_name_mapped.stderr }}"},{"line_number":52,"context_line":"          --- STDOUT ---"},{"line_number":53,"context_line":"          {{ project_name_mapped.stdout }}"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"- when: project_name_mapped is success"},{"line_number":56,"context_line":"  block:"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3fa7e38b_602784b7","line":53,"updated":"2020-01-15 08:50:45.000000000","message":"the edits above this line are mainly improvement but not really fixing a bug, if I understood correctly?","commit_id":"13088051ede26d392195558e26581e154484b6c7"},{"author":{"_account_id":27329,"name":"Federico Ressi","email":"fressi@redhat.com","username":"fressi_redhat"},"change_message_id":"2b81da1fb5227931a317b0aa9dfc7373aba25e55","unresolved":false,"context_lines":[{"line_number":50,"context_line":"          --- STDERR ---"},{"line_number":51,"context_line":"          {{ project_name_mapped.stderr }}"},{"line_number":52,"context_line":"          --- STDOUT ---"},{"line_number":53,"context_line":"          {{ project_name_mapped.stdout }}"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"- when: project_name_mapped is success"},{"line_number":56,"context_line":"  block:"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3fa7e38b_e0012209","line":53,"in_reply_to":"3fa7e38b_602784b7","updated":"2020-01-16 11:20:34.000000000","message":"Above changes are intended to help debugging problems with the \"Map project name to DLRN project name\" step. It also print to stderr all the output of \u0027rdopkg findpkg\u0027 so it can be investigated instead of filtering it out with grep. The name of the package is send to stdout as before.\nIt also fixes thing that has been done wrong: sourcing activate script has to be done before setting bash option.  The script hasn\u0027t been designed to support such options and it just add noise to the STDERR when using -x","commit_id":"13088051ede26d392195558e26581e154484b6c7"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"37f5e0c3d94b2c4a54a1775cba9cb01edd05642e","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    - name: Copy from ready directories if configured"},{"line_number":65,"context_line":"      command: \u003e"},{"line_number":66,"context_line":"        cp -raf {{ artg_repos_dir | dirname  }}/{{ artg_change.project }}"},{"line_number":67,"context_line":"        {{ build_repo_dir }}/DLRN/data/{{ project_name_mapped.stdout }}"},{"line_number":68,"context_line":"      ignore_errors: true"},{"line_number":69,"context_line":"      register: copy_dir"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3fa7e38b_8044004a","line":66,"range":{"start_line":66,"start_character":36,"end_line":66,"end_character":43},"updated":"2020-01-15 08:50:45.000000000","message":"this bit in particular is still needed/causes an error when the project is not openstack?\n\nmain hesitation here is that i was working there https://review.rdoproject.org/r/#/c/24339/ which is not an openstack/ project and I have not been hitting errors after zbr fix merged \n\nI\u0027d rather not make changes if they aren\u0027t necessary\n\nTo be clear this bit still causes error in some third party jobs (I think tobiko perhaps?)","commit_id":"13088051ede26d392195558e26581e154484b6c7"},{"author":{"_account_id":27329,"name":"Federico Ressi","email":"fressi@redhat.com","username":"fressi_redhat"},"change_message_id":"2b81da1fb5227931a317b0aa9dfc7373aba25e55","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    - name: Copy from ready directories if configured"},{"line_number":65,"context_line":"      command: \u003e"},{"line_number":66,"context_line":"        cp -raf {{ artg_repos_dir | dirname  }}/{{ artg_change.project }}"},{"line_number":67,"context_line":"        {{ build_repo_dir }}/DLRN/data/{{ project_name_mapped.stdout }}"},{"line_number":68,"context_line":"      ignore_errors: true"},{"line_number":69,"context_line":"      register: copy_dir"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3fa7e38b_4055961e","line":66,"range":{"start_line":66,"start_character":36,"end_line":66,"end_character":43},"in_reply_to":"3fa7e38b_8044004a","updated":"2020-01-16 11:20:34.000000000","message":"This line is what actually fixes problems for tobiko project\n\nIn the case of tobiko artg_repos_dir it would be\n.../src/x/tobiko\nIn the case of openstack projects it would be:\n .../src/openstack/xyz\nIn the case you posted is instead\n .../src/testproject\n\nWhat we need here is .../src/ folder. How to get it? By removing the final bit. I am wrong?","commit_id":"13088051ede26d392195558e26581e154484b6c7"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"df75732e2761bdf7e774c95949eba3ade151c618","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    - name: Copy from ready directories if configured"},{"line_number":65,"context_line":"      command: \u003e"},{"line_number":66,"context_line":"        cp -raf {{ artg_repos_dir | dirname  }}/{{ artg_change.project }}"},{"line_number":67,"context_line":"        {{ build_repo_dir }}/DLRN/data/{{ project_name_mapped.stdout }}"},{"line_number":68,"context_line":"      ignore_errors: true"},{"line_number":69,"context_line":"      register: copy_dir"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3fa7e38b_1b9dcf57","line":66,"range":{"start_line":66,"start_character":36,"end_line":66,"end_character":43},"in_reply_to":"3fa7e38b_8044004a","updated":"2020-01-16 07:49:34.000000000","message":"to answer my own question, yes it does cause error... am hitting it there http://logs.rdoproject.org/39/24339/7/check/tripleo-ci-centos-7-multinode-1ctlr-featureset010/493370c/job-output.txt  but its just not fatal after sorin fix","commit_id":"13088051ede26d392195558e26581e154484b6c7"},{"author":{"_account_id":27329,"name":"Federico Ressi","email":"fressi@redhat.com","username":"fressi_redhat"},"change_message_id":"2b81da1fb5227931a317b0aa9dfc7373aba25e55","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        cmd: |"},{"line_number":39,"context_line":"          source \u0027{{ build_repo_dir }}/dlrn-venv/bin/activate\u0027"},{"line_number":40,"context_line":"          set -xeo pipefail"},{"line_number":41,"context_line":"          rdopkg findpkg -s \u0027{{ artg_change.project | basename }}\u0027 -l rdoinfo | \\"},{"line_number":42,"context_line":"            awk \u0027/^name/{print $2}; {print \"findpkg: \" $0 \u003e \"/dev/stderr\"}\u0027"},{"line_number":43,"context_line":"      changed_when: false"},{"line_number":44,"context_line":"      failed_when: project_name_mapped.rc !\u003d 0 or not project_name_mapped.stdout"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"3fa7e38b_014fa08c","line":42,"range":{"start_line":41,"start_character":10,"end_line":42,"end_character":75},"updated":"2020-01-16 11:20:34.000000000","message":"Marios the error you are pointing out is unrelated with this change that actually removes grep from the script. The log you are pasting do executes grep command it is probably using the master branch version of this role, bur not these changes.\n\nThe command line here is this one:\n\n rdopkg findpkg -s \u0027{{ artg_change.project | basename }}\u0027 -l rdoinfo | \\\n            awk \u0027/^name/{print $2}; {print \"findpkg: \" $0 \u003e \"/dev/stderr\"}\u0027\n\nThe failing script you posted me is executing below commands:\n + rdopkg findpkg -s testproject -l rdoinfo\n + grep \u0027^name\u0027\n + awk \u0027{print $2}\u0027 \n\nTherefore this step should be already failing before this change and the reason is most probably has to be found by printing out some more output from rdopkg findpkg command.","commit_id":"3bff6353f27e13cad8acf7235523fa2a81d0ca7f"}]}