)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"49591f3127e8469528884131da696a1694e260bc","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Change lookup to ansible_env in C8 release files"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Performing local actions on executor is forbidden,"},{"line_number":10,"context_line":"changing lookup to ansible_env."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Signed-off-by: Amol Kahat \u003camolkahat@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c5573b98_cd82951f","line":9,"updated":"2022-01-24 08:42:27.000000000","message":"this is correct but i\u0027d be against making this change unless we need it?\n\nplease see inline comment... as far as I know we only faced this problem with centos9 ... where/are we seeing this for centos8 too?","commit_id":"1567cf0a35fa89bc37dc197bfec054e36a154fcf"},{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"da584fc745d877c70727f83fbcf2c10ead44c924","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Change lookup to ansible_env in C8 release files"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Performing local actions on executor is forbidden,"},{"line_number":10,"context_line":"changing lookup to ansible_env."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Signed-off-by: Amol Kahat \u003camolkahat@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d1456ba3_f5c17fd0","line":9,"in_reply_to":"c5573b98_cd82951f","updated":"2022-01-24 12:06:04.000000000","message":"We are using release files for CentOS-8. We also need it downstream.","commit_id":"1567cf0a35fa89bc37dc197bfec054e36a154fcf"}],"config/release/tripleo-ci/CentOS-8/master.yml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"49591f3127e8469528884131da696a1694e260bc","unresolved":true,"context_lines":[{"line_number":72,"context_line":"    source /etc/ci/mirror_info.sh"},{"line_number":73,"context_line":"  else"},{"line_number":74,"context_line":"    # Otherwise, fallback to official mirrors provided by CentOS."},{"line_number":75,"context_line":"    export NODEPOOL_CENTOS_MIRROR\u003d{{ lookup(\u0027env\u0027,\u0027NODEPOOL_CENTOS_MIRROR\u0027)|default(\u0027http://mirror.centos.org/centos\u0027, true) }}"},{"line_number":76,"context_line":"    export NODEPOOL_RDO_PROXY\u003dhttps://trunk.rdoproject.org"},{"line_number":77,"context_line":"  fi"},{"line_number":78,"context_line":"  {% if dlrn_hash_path_newest is defined and dlrn_hash_path_newest %}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f57fba7a_74a0c656","side":"PARENT","line":75,"range":{"start_line":75,"start_character":37,"end_line":75,"end_character":43},"updated":"2022-01-24 08:42:27.000000000","message":"where is this a problem... for centos9 it became a problem (and we included ansible_env instead of lookup) because we run the lookup as \u0027ansible native\u0027 trying to execute the repo-setup role for the image build jobs (for centos8 it is done differently)\n\ndo we have a bug about centos8 for this issue?","commit_id":"945c08a0c173a2526dc581ecc3b32c1c8afff8a7"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"fa65cf72bd8203e434c41c7e75cd21ee51a81db2","unresolved":true,"context_lines":[{"line_number":72,"context_line":"    source /etc/ci/mirror_info.sh"},{"line_number":73,"context_line":"  else"},{"line_number":74,"context_line":"    # Otherwise, fallback to official mirrors provided by CentOS."},{"line_number":75,"context_line":"    export NODEPOOL_CENTOS_MIRROR\u003d{{ lookup(\u0027env\u0027,\u0027NODEPOOL_CENTOS_MIRROR\u0027)|default(\u0027http://mirror.centos.org/centos\u0027, true) }}"},{"line_number":76,"context_line":"    export NODEPOOL_RDO_PROXY\u003dhttps://trunk.rdoproject.org"},{"line_number":77,"context_line":"  fi"},{"line_number":78,"context_line":"  {% if dlrn_hash_path_newest is defined and dlrn_hash_path_newest %}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"128ecfc8_a16f90a0","side":"PARENT","line":75,"range":{"start_line":75,"start_character":37,"end_line":75,"end_character":43},"in_reply_to":"b3f833b7_27714990","updated":"2022-01-28 07:25:46.000000000","message":"i see thanks, i didn\u0027t know we were switching the centos8 to also use release files fro the image build like we did for c9...","commit_id":"945c08a0c173a2526dc581ecc3b32c1c8afff8a7"},{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"da584fc745d877c70727f83fbcf2c10ead44c924","unresolved":true,"context_lines":[{"line_number":72,"context_line":"    source /etc/ci/mirror_info.sh"},{"line_number":73,"context_line":"  else"},{"line_number":74,"context_line":"    # Otherwise, fallback to official mirrors provided by CentOS."},{"line_number":75,"context_line":"    export NODEPOOL_CENTOS_MIRROR\u003d{{ lookup(\u0027env\u0027,\u0027NODEPOOL_CENTOS_MIRROR\u0027)|default(\u0027http://mirror.centos.org/centos\u0027, true) }}"},{"line_number":76,"context_line":"    export NODEPOOL_RDO_PROXY\u003dhttps://trunk.rdoproject.org"},{"line_number":77,"context_line":"  fi"},{"line_number":78,"context_line":"  {% if dlrn_hash_path_newest is defined and dlrn_hash_path_newest %}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b3f833b7_27714990","side":"PARENT","line":75,"range":{"start_line":75,"start_character":37,"end_line":75,"end_character":43},"in_reply_to":"f57fba7a_74a0c656","updated":"2022-01-24 12:06:04.000000000","message":"This is hitting here: https://review.opendev.org/c/openstack/tripleo-ci/+/823688/14#message-6f7a95943587b8f2078ee373724cc1fc7179ed12, this is one reason we changed to ansible_env.\n\n\nWe have JIRA (TRIPLEOCI-765) for using release files for c8.","commit_id":"945c08a0c173a2526dc581ecc3b32c1c8afff8a7"}]}