)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"60b25c50f74fbf523800015ef5ab055b6b4d4087","unresolved":true,"context_lines":[{"line_number":10,"context_line":"openstack-ansible-collections starting in Zed, this updates"},{"line_number":11,"context_line":"quickstart.sh to install the version of the collections that\u0027s"},{"line_number":12,"context_line":"compatible with the release being deployed."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I014ef50269fc195f582d6d7e5cfe49294025ee30"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1b8e0c5d_6b355820","line":13,"updated":"2022-06-01 11:29:15.000000000","message":"https://review.opendev.org/c/openstack/tripleo-ci/+/844141 is the depends on added otherway around.","commit_id":"a40701b54260af2eb2121ff3a9e2d2f82e779497"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"ff815ea5d13fd1a46f621c7344ad94598949b011","unresolved":false,"context_lines":[{"line_number":10,"context_line":"openstack-ansible-collections starting in Zed, this updates"},{"line_number":11,"context_line":"quickstart.sh to install the version of the collections that\u0027s"},{"line_number":12,"context_line":"compatible with the release being deployed."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I014ef50269fc195f582d6d7e5cfe49294025ee30"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"372e9b93_0162e863","line":13,"in_reply_to":"1b8e0c5d_6b355820","updated":"2022-06-01 12:25:54.000000000","message":"Done","commit_id":"a40701b54260af2eb2121ff3a9e2d2f82e779497"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"7e9214d3670987ef4b171e8863afffcbe0ce9300","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ec1cfd8e_c5db61ed","updated":"2022-05-27 07:22:18.000000000","message":"Oh, you already submitted a patch for branching. Now we have two proposals.. 😂 \n\nhttps://review.opendev.org/c/openstack/tripleo-quickstart/+/843570","commit_id":"0401f9d0b3efa65dd398671ef895e6c4cc2ed706"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"d7eb5605a1881444e88e319ff10623a9b1e5d537","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9f19238b_ecc88390","updated":"2022-05-30 08:04:41.000000000","message":"lgtm i think we should discuss about this vs https://review.opendev.org/c/openstack/tripleo-quickstart/+/843570 \n \nimo i think this one is currently more complete","commit_id":"0401f9d0b3efa65dd398671ef895e6c4cc2ed706"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"728bdbbe9aad69e172a70219147548c2ad63cd98","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"39704f75_664f7d99","updated":"2022-05-31 14:14:28.000000000","message":"lgtm, just a few minor changes requested.","commit_id":"0401f9d0b3efa65dd398671ef895e6c4cc2ed706"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"6196edff5b60f627dad0eb9e48b0485774a72d70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a6818461_90ad1321","updated":"2022-05-26 19:21:02.000000000","message":"looks reasonable - needs more eyes","commit_id":"0401f9d0b3efa65dd398671ef895e6c4cc2ed706"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e6485812fcd52a1001e6e5d5ebb8c9f92ac6496b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"aefc1dea_34d7fdd1","updated":"2022-05-31 19:50:04.000000000","message":"As we mentioned in review meeting, stable/wallaby is installing \u0027openstack.cloud:2.0.0-dev\u0027[1]\n\n[1] https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d9a/843497/3/check/tripleo-ci-centos-9-containers-multinode-wallaby/d9a32b0/job-output.txt","commit_id":"a40701b54260af2eb2121ff3a9e2d2f82e779497"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b4444ee213ebec2aac550e64afa56c122594b707","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"92a90204_4eaa62bf","updated":"2022-06-02 06:21:58.000000000","message":"Thanks Depends-On changed!","commit_id":"279f1e512def2e5afa2e32562288d379f3b42f09"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"658e1addd6e10f69150c147f12658c0537e1e98a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"247d333d_e3668592","updated":"2022-07-18 08:43:03.000000000","message":"The Depends-On patch has been reverted because it broke TripleO CI\u0027s periodic jobs, so this patch does not rebase cleanly. For now, we settled with decoupling TripleO and Ansible OpenStack collection (AOC) by pinning all TripleO jobs to stable/1.0.0 branch of AOC, since no TripleO branch is currently compatible to master branch of AOC anyway:\n\n  https://review.opendev.org/c/openstack/tripleo-quickstart/+/849620/\n  \nWe might want to reconsider this patch in future though, when we need a way to install different versions of AOC in TripleO Quickstart.\n\n-1\u0027ing here to prevent merging and to mark it has not required atm.","commit_id":"279f1e512def2e5afa2e32562288d379f3b42f09"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"bfbf62eb5adde06f6ea122f6ce1b1c1031623dad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bf0b3910_073d7ea6","updated":"2022-06-01 12:25:07.000000000","message":"recheck","commit_id":"279f1e512def2e5afa2e32562288d379f3b42f09"}],"quickstart.sh":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"3e3685c19d0ac259ce13feafa684c5cb59962e56","unresolved":true,"context_lines":[{"line_number":131,"context_line":"        -p $VIRTUAL_ENV/share/ansible/collections"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|ussuri|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"ab5f789a_4d800fc3","line":134,"updated":"2022-05-30 10:57:51.000000000","message":"Can we tweak this existing function https://opendev.org/openstack/tripleo-quickstart/src/branch/master/install-deps.sh#L286 to handle the installation of collection?","commit_id":"0401f9d0b3efa65dd398671ef895e6c4cc2ed706"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"7556e2494984a16ee3a54ca11b15da09fb05a9e2","unresolved":true,"context_lines":[{"line_number":131,"context_line":"        -p $VIRTUAL_ENV/share/ansible/collections"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|ussuri|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"2913c0b3_3fae8baa","line":134,"in_reply_to":"ab5f789a_4d800fc3","updated":"2022-05-31 15:06:56.000000000","message":"This is right under where we were installing the collection before. I think it makes sense to have this here.","commit_id":"0401f9d0b3efa65dd398671ef895e6c4cc2ed706"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"d7eb5605a1881444e88e319ff10623a9b1e5d537","unresolved":true,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|ussuri|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"3cfb35e0_b8fd0d7f","line":135,"range":{"start_line":135,"start_character":35,"end_line":135,"end_character":41},"updated":"2022-05-30 08:04:41.000000000","message":"remove U as it doesn\u0027t exist for tripleo any more","commit_id":"0401f9d0b3efa65dd398671ef895e6c4cc2ed706"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"7556e2494984a16ee3a54ca11b15da09fb05a9e2","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|ussuri|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"50cb4c94_f1ee2687","line":135,"range":{"start_line":135,"start_character":35,"end_line":135,"end_character":41},"in_reply_to":"3cfb35e0_b8fd0d7f","updated":"2022-05-31 15:06:56.000000000","message":"Done","commit_id":"0401f9d0b3efa65dd398671ef895e6c4cc2ed706"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a3343bba21951e97ddad8ad7d19f9355e5734f29","unresolved":true,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"962a8d20_0a491688","line":135,"range":{"start_line":135,"start_character":11,"end_line":135,"end_character":22},"updated":"2022-06-01 06:04:34.000000000","message":"Check what is OPT_RELEASE here, seems like it\u0027s \"master\" in branchful jobs like ...multinode-wallaby","commit_id":"a40701b54260af2eb2121ff3a9e2d2f82e779497"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"ff815ea5d13fd1a46f621c7344ad94598949b011","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"9875e605_1cc60fdc","line":135,"range":{"start_line":135,"start_character":11,"end_line":135,"end_character":22},"in_reply_to":"962a8d20_0a491688","updated":"2022-06-01 12:25:54.000000000","message":"Yeah sorry about that. This change working depends on https://review.opendev.org/c/openstack/tripleo-ci/+/844141\n\nAdded","commit_id":"a40701b54260af2eb2121ff3a9e2d2f82e779497"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"d33dbdd53ddde94b9ad2684a1678eb637a12ebb2","unresolved":true,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"37c46642_a7535fb3","line":135,"range":{"start_line":135,"start_character":3,"end_line":135,"end_character":23},"updated":"2022-06-02 07:38:03.000000000","message":"looks good\n\nwhere are the tests please? \n\nI mean did you post a patch for train/victoria/wallaby and make sure the right versions of ACO are installed with this patch? If not I think we should before we merge. If you have can you please add links in the commit message (maybe have a gerrit topic to point to?)","commit_id":"279f1e512def2e5afa2e32562288d379f3b42f09"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"83c4fc324765bb54449dc1690c94c171bf48d3a1","unresolved":true,"context_lines":[{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    if [[ -d ~/src/opendev.org/openstack/ansible-collections-openstack ]]; then"}],"source_content_type":"text/x-sh","patch_set":4,"id":"bda4b1e0_28a1ec5c","line":136,"range":{"start_line":136,"start_character":8,"end_line":136,"end_character":31},"updated":"2022-06-02 11:32:02.000000000","message":"Still see in wallaby job[1]:\n\n Installing \u0027openstack.cloud:1.8.0\u0027 to \u0027/home/zuul/workspace/.quickstart/share/ansible/collections/ansible_collections/openstack/cloud\u0027\n\nIs that correct?\n\n[1] https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_979/843497/4/check/tripleo-ci-centos-9-containers-multinode-wallaby/97949c8/job-output.txt","commit_id":"279f1e512def2e5afa2e32562288d379f3b42f09"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"72c58f548cf1b42308849634e1e17c0debfd16ea","unresolved":true,"context_lines":[{"line_number":133,"context_line":"    # Install ansible-collections-openstack separately to account for different releases"},{"line_number":134,"context_line":"    ACO_BRANCH\u003dmaster"},{"line_number":135,"context_line":"    if [[ $OPT_RELEASE \u003d~ .*(train|victoria|wallaby).* ]]; then"},{"line_number":136,"context_line":"        ACO_BRANCH\u003dstable/1.0.0"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    if [[ -d ~/src/opendev.org/openstack/ansible-collections-openstack ]]; then"}],"source_content_type":"text/x-sh","patch_set":4,"id":"09bb4111_a0acdce1","line":136,"range":{"start_line":136,"start_character":8,"end_line":136,"end_character":31},"in_reply_to":"bda4b1e0_28a1ec5c","updated":"2022-06-07 06:36:06.000000000","message":"Yes, stable/1.0.0 branch contains 1.8.0 version of collection. Master contains 2.0.0-dev version.","commit_id":"279f1e512def2e5afa2e32562288d379f3b42f09"}]}