)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b332eefe472cae33eacabe993f6aeba4d7b28c4e","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Retrive non-tripleo containers from buildah images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Due to fallback registry, non_tripleo_containers list does not"},{"line_number":10,"context_line":"contains the list of actual pulled containers, it needs to be"},{"line_number":11,"context_line":"retrived from buildah images output otherwise while pushing"},{"line_number":12,"context_line":"the containers it will fail complaining containers does not"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"42e02f37_a8f7901f","line":9,"range":{"start_line":9,"start_character":26,"end_line":9,"end_character":1},"updated":"2021-03-24 08:25:51.000000000","message":"a bit confused by this though.\n\nthe code is _still_ using non_tripleo_containers list https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/782362/2/roles/container-build/tasks/non_tripleo_containers.yml#112\n\nso what we need is the build id. You are using grep/awk to retrieve the image id ?","commit_id":"77ce99452a4cdb7fd3d52603a51fc07ac651c21d"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e4f7c14d8bf0e9a3e1c4f2ef583e6b9f1d5a51b0","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Retrive non-tripleo containers from buildah images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Due to fallback registry, non_tripleo_containers list does not"},{"line_number":10,"context_line":"contains the list of actual pulled containers, it needs to be"},{"line_number":11,"context_line":"retrived from buildah images output otherwise while pushing"},{"line_number":12,"context_line":"the containers it will fail complaining containers does not"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"2972762c_c12dbfee","line":9,"range":{"start_line":9,"start_character":26,"end_line":9,"end_character":1},"in_reply_to":"42e02f37_a8f7901f","updated":"2021-05-10 12:48:24.000000000","message":"We need the image id so that we can push the containers to content provider registry.","commit_id":"77ce99452a4cdb7fd3d52603a51fc07ac651c21d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"6e0bed430fd7d638eef6223ab2c241eed9ef984e","unresolved":true,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Related-Bug: #1920873"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tripleo-common/+/791299"},{"line_number":22,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"},{"line_number":23,"context_line":"Change-Id: I972d03650653a63c0f5df0b3c2ef7bb710b4afba"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"9353f1e2_327647c9","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":7},"updated":"2021-05-14 06:14:02.000000000","message":"so this is just for testing looks like","commit_id":"8f50659c961bb1e1d95ff71e39d987576ecdb394"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"05b6468ad1f306d26c55d45111af19e01319adb3","unresolved":true,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Related-Bug: #1920873"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tripleo-common/+/791299"},{"line_number":22,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"},{"line_number":23,"context_line":"Change-Id: I972d03650653a63c0f5df0b3c2ef7bb710b4afba"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"90058f8f_aca100a5","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":7},"in_reply_to":"9353f1e2_327647c9","updated":"2021-05-14 06:53:34.000000000","message":"Yes, it is just for testing, updated a new patchset on tripleo-common side","commit_id":"8f50659c961bb1e1d95ff71e39d987576ecdb394"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"ca1e069403c513e5832f94bfda10db85b69db5d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"27f61645_ef4e550b","updated":"2021-10-14 14:34:27.000000000","message":"Result from https://review.opendev.org/c/openstack/tripleo-common/+/813957\nhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4b3/813957/1/check/tripleo-ci-centos-8-content-provider/4b38de5/job-output.txt\n```\n2021-10-14 12:47:40.012639 | primary | ok: [undercloud] \u003d\u003e (item\u003d{\u0027msg\u0027: \u0027non-zero return code\u0027, \u0027cmd\u0027: [\u0027podman\u0027, \u0027pull\u0027, \u0027quay.ceph1.io/app-sre/grafana:6.7.4\u0027], \u0027stdout\u0027: \u0027\u0027, \u0027stderr\u0027: \u0027Trying to pull quay.ceph1.io/app-sre/grafana:6.7.4...\\n  Get \"https://quay.ceph1.io/v2/\": dial tcp: lookup quay.ceph1.io on 127.0.0.1:53: no such host\\nError: Error initializing source docker://quay.ceph1.io/app-sre/grafana:6.7.4: error pinging docker registry quay.ceph1.io: Get \"https://quay.ceph1.io/v2/\": dial tcp: lookup quay.ceph1.io on 127.0.0.1:53: no such host\u0027, \u0027rc\u0027: 125, \u0027start\u0027: \u00272021-10-14 12:47:39.460274\u0027, \u0027end\u0027: \u00272021-10-14 12:47:39.773375\u0027, \u0027delta\u0027: \u00270:00:00.313101\u0027, \u0027changed\u0027: True, \u0027failed\u0027: True, \u0027invocation\u0027: {\u0027module_args\u0027: {\u0027_raw_params\u0027: \u0027podman pull quay.ceph1.io/app-sre/grafana:6.7.4\u0027, \u0027warn\u0027: False, \u0027_uses_shell\u0027: False, \u0027stdin_add_newline\u0027: True, \u0027strip_empty_ends\u0027: True, \u0027argv\u0027: None, \u0027chdir\u0027: None, \u0027executable\u0027: None, \u0027creates\u0027: None, \u0027removes\u0027: None, \u0027stdin\u0027: None}}, \u0027stdout_lines\u0027: [], \u0027stderr_lines\u0027: [\u0027Trying to pull quay.ceph1.io/app-sre/grafana:6.7.4...\u0027, \u0027  Get \"https://quay.ceph1.io/v2/\": dial tcp: lookup quay.ceph1.io on 127.0.0.1:53: no such host\u0027, \u0027Error: Error initializing source docker://quay.ceph1.io/app-sre/grafana:6.7.4: error pinging docker registry quay.ceph1.io: Get \"https://quay.ceph1.io/v2/\": dial tcp: lookup quay.ceph1.io on 127.0.0.1:53: no such host\u0027], \u0027item\u0027: \u0027quay.ceph1.io/app-sre/grafana:6.7.4\u0027, \u0027ansible_loop_var\u0027: \u0027item\u0027})\n2021-10-14 12:47:40.032334 | primary |\n2021-10-14 12:47:40.032404 | primary | TASK [container-build : Pull failed containers from RDO registry] **************\n2021-10-14 12:47:40.032455 | primary | Thursday 14 October 2021  12:47:40 +0000 (0:00:00.195)       1:01:40.274 ******\n2021-10-14 12:47:53.780317 | primary | changed: [undercloud] \u003d\u003e (item\u003dquay.ceph1.io/app-sre/grafana:6.7.4)\n2021-10-14 12:47:53.798464 | primary |\n2021-10-14 12:47:53.798534 | primary | TASK [container-build : List down missing containers] **************************\n2021-10-14 12:47:53.798747 | primary | Thursday 14 October 2021  12:47:53 +0000 (0:00:13.766)       1:01:54.041 ******\n2021-10-14 12:47:53.901787 | primary | skipping: [undercloud] \u003d\u003e (item\u003d{\u0027cmd\u0027: [\u0027podman\u0027, \u0027pull\u0027, \u0027docker://trunk.registry.rdoproject.org/ceph/grafana:6.7.4\u0027], \u0027stdout\u0027: \u0027be4c69a1aae8ddac3b68d8f43567f3ff15d87fc418b1639981d2205fd7d7bb88\u0027, \u0027stderr\u0027: \u0027Trying to pull docker://trunk.registry.rdoproject.org/ceph/grafana:6.7.4...\\nGetting image source signatures\\nCopying blob sha256:d48fedd385a56fdcb3e52d8aeaf76eaff856287992a2dda435cbbec2e2b63857\\nCopying blob sha256:82fc1c29ef77957f54987480afc080cb0dad823ea340b29371d83f065c30c6b5\\nCopying blob sha256:aa0174112d037bf446eb5a6e707a4bb48debac5b280b61124bd02fc16976dabd\\nCopying blob sha256:8094032001f383ef91cd6a394f4262fbe421dc1eea5e28fd0343418d3affc7b4\\nCopying blob sha256:9d16cba9fb961d1aafec9542f2bf7cb64acfc55245f9e4eb5abecd4cdc38d749\\nCopying blob sha256:1834ce9eb7cede6c1ae0fc995bb595d22ad1a3189a63bbb9aa01d48965132a02\\nCopying blob sha256:a40562c358bdb8a1c864c91a80bf48c51d030fa1195f5c25170009d74ed87972\\nCopying blob sha256:1da048eb9017d76b76f72713b0594357a95ddf2bda4c88eb4d059530ec2accbc\\nCopying config sha256:be4c69a1aae8ddac3b68d8f43567f3ff15d87fc418b1639981d2205fd7d7bb88\\nWriting manifest to image destination\\nStoring signatures\u0027, \u0027rc\u0027: 0, \u0027start\u0027: \u00272021-10-14 12:47:40.566751\u0027, \u0027end\u0027: \u00272021-10-14 12:47:53.727363\u0027, \u0027delta\u0027: \u00270:00:13.160612\u0027, \u0027changed\u0027: True, \u0027invocation\u0027: {\u0027module_args\u0027: {\u0027_raw_params\u0027: \u0027podman pull docker://trunk.registry.rdoproject.org/ceph/grafana:6.7.4\u0027, \u0027warn\u0027: False, \u0027_uses_shell\u0027: False, \u0027stdin_add_newline\u0027: True, \u0027strip_empty_ends\u0027: True, \u0027argv\u0027: None, \u0027chdir\u0027: None, \u0027executable\u0027: None, \u0027creates\u0027: None, \u0027removes\u0027: None, \u0027stdin\u0027: None}}, \u0027stdout_lines\u0027: [\u0027be4c69a1aae8ddac3b68d8f43567f3ff15d87fc418b1639981d2205fd7d7bb88\u0027], \u0027stderr_lines\u0027: [\u0027Trying to pull docker://trunk.registry.rdoproject.org/ceph/grafana:6.7.4...\u0027, \u0027Getting image source signatures\u0027, \u0027Copying blob sha256:d48fedd385a56fdcb3e52d8aeaf76eaff856287992a2dda435cbbec2e2b63857\u0027, \u0027Copying blob sha256:82fc1c29ef77957f54987480afc080cb0dad823ea340b29371d83f065c30c6b5\u0027, \u0027Copying blob sha256:aa0174112d037bf446eb5a6e707a4bb48debac5b280b61124bd02fc16976dabd\u0027, \u0027Copying blob sha256:8094032001f383ef91cd6a394f4262fbe421dc1eea5e28fd0343418d3affc7b4\u0027, \u0027Copying blob sha256:9d16cba9fb961d1aafec9542f2bf7cb64acfc55245f9e4eb5abecd4cdc38d749\u0027, \u0027Copying blob sha256:1834ce9eb7cede6c1ae0fc995bb595d22ad1a3189a63bbb9aa01d48965132a02\u0027, \u0027Copying blob sha256:a40562c358bdb8a1c864c91a80bf48c51d030fa1195f5c25170009d74ed87972\u0027, \u0027Copying blob sha256:1da048eb9017d76b76f72713b0594357a95ddf2bda4c88eb4d059530ec2accbc\u0027, \u0027Copying config sha256:be4c69a1aae8ddac3b68d8f43567f3ff15d87fc418b1639981d2205fd7d7bb88\u0027, \u0027Writing manifest to image destination\u0027, \u0027Storing signatures\u0027], \u0027failed\u0027: False, \u0027item\u0027: \u0027quay.ceph1.io/app-sre/grafana:6.7.4\u0027, \u0027ansible_loop_var\u0027: \u0027item\u0027})\n```","commit_id":"5a7f81dd138a91df25fa923c63ffa3edfe03218b"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9dfcc7fa1fa0668227113515d214ada0c9a7767b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"1681c118_272d2458","updated":"2021-10-14 10:44:59.000000000","message":"ack - let\u0027s see test results here","commit_id":"5a7f81dd138a91df25fa923c63ffa3edfe03218b"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9f19c1c61adf154bd94a1475c0d81339c14b0ace","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"7d28e801_f07d7077","updated":"2021-10-14 14:38:33.000000000","message":"looking at test results","commit_id":"5a7f81dd138a91df25fa923c63ffa3edfe03218b"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a59e99eb823747691f06fc1e055862ef50676744","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"7f10bf7b_a4cc3ea7","updated":"2021-10-14 14:40:08.000000000","message":"looks good to proceed, tks","commit_id":"5a7f81dd138a91df25fa923c63ffa3edfe03218b"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"3efb73e933b5ff557e35a425ade3a297d1137d21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"d68c9146_fee5da4d","updated":"2021-10-14 07:10:25.000000000","message":"testing here: https://review.opendev.org/c/openstack/tripleo-common/+/813957","commit_id":"5a7f81dd138a91df25fa923c63ffa3edfe03218b"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"2c062a9a970f62defb3f599f8e87d33dd2d70f14","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"385d8647_9e2f763f","updated":"2021-10-14 09:53:25.000000000","message":"will merge once we have results from https://review.opendev.org/c/openstack/tripleo-common/+/813957","commit_id":"5a7f81dd138a91df25fa923c63ffa3edfe03218b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f66663147e37b01a4ad6b5b7e1014971f3c79282","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"d17b8d4e_760669a0","in_reply_to":"27f61645_ef4e550b","updated":"2021-10-14 14:43:52.000000000","message":"something wrong as output not saved in https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4b3/813957/1/check/tripleo-ci-centos-8-content-provider/4b38de5/logs/undercloud/home/zuul/non_tripleo_containers.log","commit_id":"5a7f81dd138a91df25fa923c63ffa3edfe03218b"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"aa4e11f4b0b2bf14a4ec71567ea2b4d9673e741a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"b30f23a7_65175f2c","updated":"2021-10-14 20:25:29.000000000","message":"https://f12c37c8a25912360378-40001c60cc2cc6a7bddcc7547f0d5486.ssl.cf5.rackcdn.com/782362/17/check/tripleo-ci-centos-8-content-provider/510b501/logs/undercloud/home/zuul/non_tripleo_containers.log\n\nPS17 wasn\u0027t copying anything to non_tripleo_containers.log","commit_id":"bdb6d4de28c02c494d3b0bba252965079f39e75c"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e8295d67b459000f7d6a01a073091315ab86a11c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"78687631_898ec794","updated":"2021-10-15 06:49:47.000000000","message":"minor inline comment for consideration","commit_id":"bdb6d4de28c02c494d3b0bba252965079f39e75c"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"67a77bfcd2265ba91b9280364a31dd83bc3153c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"1fabe99e_169d6bdd","in_reply_to":"b30f23a7_65175f2c","updated":"2021-10-15 07:36:39.000000000","message":"It is fixed in a new patchset.","commit_id":"bdb6d4de28c02c494d3b0bba252965079f39e75c"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"1ad608172229081cedb2389fae274c6a5dd86d59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"7a34546c_857ff66c","updated":"2021-10-15 12:09:45.000000000","message":"https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_be5/782362/19/check/tripleo-ci-centos-8-content-provider/be5aad4/logs/undercloud/home/zuul/non_tripleo_containers.log\n\nOutputs are available at \u0027stderr\u0027, not sure why, can we spot any issue in outputs? ","commit_id":"c6db9237f6ed45fe050e91eb1d44f752353a329b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"2e64057481691e899b44b8053d61312011686196","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"63df0c17_4d380937","in_reply_to":"7a34546c_857ff66c","updated":"2021-10-19 11:36:34.000000000","message":"likely due to debug mode it redirects output to stderr, collecting stdout/stderr only should have been better though. but ok will not block.","commit_id":"c6db9237f6ed45fe050e91eb1d44f752353a329b"}],"roles/container-build/tasks/non_tripleo_containers.yml":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"e564d46f18221aa8739bd632060cfe7d3e1062a0","unresolved":true,"context_lines":[{"line_number":107,"context_line":"    - name: Push non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        export image_name\u003d\"$( buildah images | grep {{ item.split(\u0027/\u0027)[-1] }} | awk \u0027{ print $1 }\u0027 )\";"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug $image_name \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""},{"line_number":113,"context_line":"      retries: 3"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"439846e0_596f40f3","line":110,"updated":"2021-03-23 06:21:53.000000000","message":"don\u0027t you need to pass tag along with name? or use image id instead?","commit_id":"b2291f7f052766f8cd820b8fd84581d5f1b3aabc"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b332eefe472cae33eacabe993f6aeba4d7b28c4e","unresolved":true,"context_lines":[{"line_number":109,"context_line":"        export image_id\u003d\"$( buildah images | grep {{ item.split(\u0027/\u0027)[-1] }} | awk \u0027{ print $3 }\u0027 )\";"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug $image_id \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""},{"line_number":113,"context_line":"      retries: 3"},{"line_number":114,"context_line":"      delay: 5"},{"line_number":115,"context_line":"      register: non_tripleo"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f6c0a943_c35eb75d","line":112,"range":{"start_line":112,"start_character":6,"end_line":112,"end_character":44},"updated":"2021-03-24 08:25:51.000000000","message":"but it still uses the non_tripleo_containers list ?","commit_id":"77ce99452a4cdb7fd3d52603a51fc07ac651c21d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"165b446f7260c36d336ce44719f95fd1903d5ec5","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Push non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e-"},{"line_number":109,"context_line":"        export image_id\u003d\"$(buildah images | grep {{ item.split(\u0027/\u0027)[-1] }} | awk \u0027{ print $3 }\u0027)\";"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug $image_id \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"62f58e6f_ece760c6","line":109,"range":{"start_line":109,"start_character":27,"end_line":109,"end_character":98},"updated":"2021-05-13 06:37:24.000000000","message":"seems to do the right thing here - eg for local image \n\nREPOSITORY                   TAG      IMAGE ID       CREATED       SIZE\ndocker.io/library/registry   latest   1fd8e1b0bb7e   4 weeks ago   26.8 MB\n\n\nbut i think we can avoid the grep awk... have you tried buildah images -q ?\n\ne.g. buildah images -q registry \n1fd8e1b0bb7e","commit_id":"0c9a902d3c501ea7d1ec08505285fc75706da075"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1c6946dde8d6fbcae03d8f4b8222aa77268dce95","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Push non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e-"},{"line_number":109,"context_line":"        export image_id\u003d\"$(buildah images | grep {{ item.split(\u0027/\u0027)[-1] }} | awk \u0027{ print $3 }\u0027)\";"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug $image_id \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"fcd57ff8_0f2ef0e2","line":109,"range":{"start_line":109,"start_character":27,"end_line":109,"end_character":98},"in_reply_to":"62f58e6f_ece760c6","updated":"2021-05-13 06:40:45.000000000","message":"i forgot to include the \u0027seems to do the right thing\u0027 output ;D\n\nbuildah images | grep registry | awk \u0027{ print $3 }\u0027;\n1fd8e1b0bb7e\n\n\nanyway if you want to change it (I think we should unless there is reason not to should be like\n\n\n\"$(buildah images -q {{ item }})\"","commit_id":"0c9a902d3c501ea7d1ec08505285fc75706da075"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fa9c95c96e446ba4e1a675a6cc518c1275a09186","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Push non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e-"},{"line_number":109,"context_line":"        export image_id\u003d\"$(buildah images | grep {{ item.split(\u0027/\u0027)[-1] }} | awk \u0027{ print $3 }\u0027)\";"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug $image_id \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"e7f3651f_56bba8e3","line":109,"range":{"start_line":109,"start_character":27,"end_line":109,"end_character":98},"in_reply_to":"fcd57ff8_0f2ef0e2","updated":"2021-05-13 06:55:56.000000000","message":"Thanks marios, I was not knowing about that option, it seems pretty easy, we can pass the image name with tag, it returns the id.","commit_id":"0c9a902d3c501ea7d1ec08505285fc75706da075"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"fa70d045020eff84bf0d2a735834239c606776cb","unresolved":true,"context_lines":[{"line_number":105,"context_line":"    # NOTE(mwhahaha): use buildah because --format can only be specified with buildah push."},{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Push non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e-"},{"line_number":109,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug \\"},{"line_number":110,"context_line":"          $(buildah images -q {{ item.split(\u0027/\u0027)[-1] }}) \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"fbb625c6_6f3a34ec","line":108,"range":{"start_line":108,"start_character":14,"end_line":108,"end_character":15},"updated":"2021-05-13 11:09:40.000000000","message":"just noticed that... did you mean to add it? why do we want to strip the newline at the end of the shell string?","commit_id":"83b563c285ab09b8755370c33b46ffb863fc78fb"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"5661e60a4c01ec2d61a3a5b1710c2b31661bc28a","unresolved":true,"context_lines":[{"line_number":105,"context_line":"    # NOTE(mwhahaha): use buildah because --format can only be specified with buildah push."},{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Push non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e-"},{"line_number":109,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug \\"},{"line_number":110,"context_line":"          $(buildah images -q {{ item.split(\u0027/\u0027)[-1] }}) \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"77381927_c87311b2","line":108,"range":{"start_line":108,"start_character":14,"end_line":108,"end_character":15},"in_reply_to":"fbb625c6_6f3a34ec","updated":"2021-05-13 12:25:14.000000000","message":"Got added by mistake removing that.","commit_id":"83b563c285ab09b8755370c33b46ffb863fc78fb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e24ecb701c45bdba16460e103450a5fecf77c25b","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"6fb22280_a1b88d5a","side":"PARENT","line":109,"range":{"start_line":109,"start_character":8,"end_line":109,"end_character":22},"updated":"2021-05-17 06:55:05.000000000","message":"i understand we are now pushing using the id but it looks like we were also listing the images first do we want to keep this one?","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"131aeebaef7e34f20d6df0a81c29b148f5a6c6aa","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"84e42a8f_498b234f","side":"PARENT","line":109,"range":{"start_line":109,"start_character":8,"end_line":109,"end_character":22},"in_reply_to":"3765f0a4_96c9890a","updated":"2021-05-19 07:35:54.000000000","message":"ok then will downgrade vote until you update thanks for checking","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"201a1199a658a9052be920133837840ec7732c2b","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"3765f0a4_96c9890a","side":"PARENT","line":109,"range":{"start_line":109,"start_character":8,"end_line":109,"end_character":22},"in_reply_to":"6fb22280_a1b88d5a","updated":"2021-05-17 13:22:30.000000000","message":"I think we can keep this one to see what all images got pulled locally before pushing it. it would be useful for debugging.","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b5a2b906e4c94bd7a9197c33b6320b6dc901d23a","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"d086fc37_1055a943","side":"PARENT","line":109,"range":{"start_line":109,"start_character":8,"end_line":109,"end_character":22},"in_reply_to":"84e42a8f_498b234f","updated":"2021-06-17 05:35:45.000000000","message":"Done in the new patchset.","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"f04d69ae8391b7e0edf583bfa0522cc4f17540e2","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"f0aff823_94a6f7e0","side":"PARENT","line":109,"range":{"start_line":109,"start_character":8,"end_line":109,"end_character":22},"in_reply_to":"d086fc37_1055a943","updated":"2021-06-17 06:19:42.000000000","message":"ack thanks\nand welcome back :D","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"f9946d0cc6ac4ce52843bde421245a4d6174d65d","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"a86816e9_49e594a2","side":"PARENT","line":109,"range":{"start_line":109,"start_character":8,"end_line":109,"end_character":22},"in_reply_to":"f0aff823_94a6f7e0","updated":"2021-06-17 07:40:14.000000000","message":"Thanks :-)","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f322eb84fd884dba3f887d5dcac7f6c725b3a2e5","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"fe3a4c89_551fa167","side":"PARENT","line":109,"range":{"start_line":109,"start_character":8,"end_line":109,"end_character":23},"updated":"2021-10-07 06:47:03.000000000","message":"don\u0027t think need to run this for all the items so can move it to seperate task so it runs just once. or if looking for only related images then can do buildah images|grep {{ item }}","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"8b4110bbeee9c17bda36afc7edb1ae5dc61b08ee","unresolved":true,"context_lines":[{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug \\"},{"line_number":111,"context_line":"          $(buildah images -q {{ item.split(\u0027/\u0027)[-1] }}) \\"},{"line_number":112,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":113,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""},{"line_number":114,"context_line":"      retries: 3"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"e4e1acdb_c2c59461","line":111,"updated":"2021-06-28 09:04:42.000000000","message":"this is LGTM, but an unrelated question: https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/roles/container-build/tasks/main.yaml#L115 imports these tasks without specifying a list of items. I hope that\u0027s fine...","commit_id":"0a4768f03096d0343cff41c56fc0fb8da28bdb97"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"53ed4a42aa56728587c807815fe2a8ea995b9360","unresolved":true,"context_lines":[{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug \\"},{"line_number":111,"context_line":"          $(buildah images -q {{ item.split(\u0027/\u0027)[-1] }}) \\"},{"line_number":112,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":113,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""},{"line_number":114,"context_line":"      retries: 3"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"7ca77f5c_6bf952b5","line":111,"in_reply_to":"e4e1acdb_c2c59461","updated":"2021-06-28 09:09:08.000000000","message":"Good question, Yes, It  imports these tasks without specifying a list of items.As the above list is getting generated here https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/roles/container-build/tasks/non_tripleo_containers.yml#L2 with in the imported playbook itself by reading the tripleo-common tripleo_container.yaml file.","commit_id":"0a4768f03096d0343cff41c56fc0fb8da28bdb97"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f322eb84fd884dba3f887d5dcac7f6c725b3a2e5","unresolved":true,"context_lines":[{"line_number":119,"context_line":"      tags:"},{"line_number":120,"context_line":"        - skip_ansible_lint"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"- name: Dump non-tripleo containers output to file"},{"line_number":123,"context_line":"  copy:"},{"line_number":124,"context_line":"    content: \"{{ non_tripleo.stdout }}\""},{"line_number":125,"context_line":"    dest: \"{{ ansible_user_dir }}/non_tripleo_containers.log\""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"3d9b096d_eb4932de","line":122,"range":{"start_line":122,"start_character":8,"end_line":122,"end_character":12},"updated":"2021-10-07 06:47:03.000000000","message":"this failing with The task includes an option with an undefined variable. The error was: \u0027dict object\u0027 has no attribute \u0027stdout\u0027, may be you looking for filtering from results like \"{{ non_tripleo.results | map(attribute\u003d\u0027stdout\u0027) | join(\u0027\\n\u0027) }}\"","commit_id":"0a4768f03096d0343cff41c56fc0fb8da28bdb97"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e8295d67b459000f7d6a01a073091315ab86a11c","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Push non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images | grep {{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug \\"},{"line_number":111,"context_line":"          $(buildah images -q {{ item.split(\u0027/\u0027)[-1] }}) \\"},{"line_number":112,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"72caee23_3fb07ebc","line":109,"range":{"start_line":109,"start_character":23,"end_line":109,"end_character":56},"updated":"2021-10-15 06:49:47.000000000","message":"why though lets have all the info i am guessing this was added for debug purposes?\n\njust commenting on this specifically as it is the only diff from my last vote on v15 in june ;)","commit_id":"bdb6d4de28c02c494d3b0bba252965079f39e75c"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"67a77bfcd2265ba91b9280364a31dd83bc3153c7","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Push non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images | grep {{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug \\"},{"line_number":111,"context_line":"          $(buildah images -q {{ item.split(\u0027/\u0027)[-1] }}) \\"},{"line_number":112,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"f9790b9a_5122be2e","line":109,"range":{"start_line":109,"start_character":23,"end_line":109,"end_character":56},"in_reply_to":"72caee23_3fb07ebc","updated":"2021-10-15 07:36:39.000000000","message":"It was debugging purpose. We have removed it in new patchset.","commit_id":"bdb6d4de28c02c494d3b0bba252965079f39e75c"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"bbb2e3378331cf7ee894afbc286cb9325723853e","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":19,"id":"38b2e962_5155af99","side":"PARENT","line":109,"range":{"start_line":109,"start_character":0,"end_line":109,"end_character":1},"updated":"2021-10-15 14:55:24.000000000","message":"this has come up before in this change but why are you removing this.\n\nthis is pre-existing i.e. we have it in there for debug during jobs. sure we want to remove?","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"760d5b669dc53cd75b50b76526893a5931c2ba91","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    # Additionally we could switch to oci, but v2v2 is what we convert on the undercloud."},{"line_number":107,"context_line":"    - name: Pull non-tripleo containers (ceph, alertmanager, prometheus) to the content provider registry"},{"line_number":108,"context_line":"      shell: \u003e"},{"line_number":109,"context_line":"        buildah images;"},{"line_number":110,"context_line":"        buildah push --format\u003dv2s2 --tls-verify\u003dFalse --log-level debug {{ item }} \\"},{"line_number":111,"context_line":"          docker://{{ content_provider_registry_ip }}:{{ content_provider_registry_port }}/{{ docker_registry_namespace }}/{{ item.split(\u0027/\u0027)[-1] }};"},{"line_number":112,"context_line":"      with_items: \"{{ non_tripleo_containers }}\""}],"source_content_type":"text/x-yaml","patch_set":19,"id":"ff9d012e_3a6fa910","side":"PARENT","line":109,"range":{"start_line":109,"start_character":0,"end_line":109,"end_character":1},"in_reply_to":"38b2e962_5155af99","updated":"2021-10-15 15:30:41.000000000","message":"Since we are logging the output in a file https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_be5/782362/19/check/tripleo-ci-centos-8-content-provider/be5aad4/logs/undercloud/home/zuul/non_tripleo_containers.log\n\nSo that would be enough for debugging for that. So we are removing it.","commit_id":"530c677f3ea47db7f8569b5a292b361faec67a1e"}]}