)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"ea48d3e53d31de805ad5fc76746d2e6068df9489","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP: Add spec for single sourcing repo setup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Story: https://projects.engineering.redhat.com/browse/TRIPLEOCI-262"},{"line_number":10,"context_line":"Change-Id: Idd2663d4db54e9bedf6ea067ca14e287e55d3159"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"b968ecdb_399dfe88","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":5},"updated":"2021-02-02 08:27:37.000000000","message":"please remove that, unfortunately our jira is not yet public facing","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"4a693acf32b672e92c716e6067b8fac2db2a0767","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP: Add spec for single sourcing repo setup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Story: https://projects.engineering.redhat.com/browse/TRIPLEOCI-262"},{"line_number":10,"context_line":"Change-Id: Idd2663d4db54e9bedf6ea067ca14e287e55d3159"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"250f01ad_b84d6118","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":5},"in_reply_to":"b968ecdb_399dfe88","updated":"2021-02-02 14:14:56.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"}],"specs/wallaby/tripleo-repos-single-source.rst":[{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"01d6db69d6c5c71ef3c5522f9b07ac36ffe0e4ab","unresolved":true,"context_lines":[{"line_number":21,"context_line":"and versions are set. Some of these include:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* tripleo repos"},{"line_number":24,"context_line":"    * https://github.com/openstack/tripleo-repos"},{"line_number":25,"context_line":"* repo setup role"},{"line_number":26,"context_line":"    * https://github.com/openstack/tripleo-quickstart/tree/master/roles/repo-setup"},{"line_number":27,"context_line":"* release config files"}],"source_content_type":"text/x-rst","patch_set":4,"id":"35b6e2e3_8dad6ec3","line":24,"updated":"2021-02-01 21:40:16.000000000","message":"Please add:\nhttps://opendev.org/openstack/tripleo-heat-templates/src/branch/master/environments/lifecycle/undercloud-upgrade-prepare.yaml\n\nhttps://opendev.org/openstack/tripleo-common/src/branch/master/container-images/tcib/base/base.yaml#L62\n\nhttps://opendev.org/openstack/tripleo-operator-ansible/src/branch/master/roles/tripleo_container_image_build/molecule/default/converge.yml#L172","commit_id":"d9d52c4371e0799dc17cf6ebc8378efe12524148"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85819f5c5e118dd9d5852852dc82eaac19d8a15c","unresolved":false,"context_lines":[{"line_number":21,"context_line":"and versions are set. Some of these include:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* tripleo repos"},{"line_number":24,"context_line":"    * https://github.com/openstack/tripleo-repos"},{"line_number":25,"context_line":"* repo setup role"},{"line_number":26,"context_line":"    * https://github.com/openstack/tripleo-quickstart/tree/master/roles/repo-setup"},{"line_number":27,"context_line":"* release config files"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9ad15b56_8511cf0e","line":24,"in_reply_to":"35b6e2e3_8dad6ec3","updated":"2021-02-01 23:48:08.000000000","message":"Done","commit_id":"d9d52c4371e0799dc17cf6ebc8378efe12524148"},{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"01d6db69d6c5c71ef3c5522f9b07ac36ffe0e4ab","unresolved":true,"context_lines":[{"line_number":75,"context_line":"Functionality to set repos, modules and versions is already available today."},{"line_number":76,"context_line":"It would be possible to leave the status quo or:"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"* Create an entirely new repo - outside of anything used in TripleO"},{"line_number":79,"context_line":"right now to do the settings work"},{"line_number":80,"context_line":"* Use rdo config to set one version per release - however, this would not"},{"line_number":81,"context_line":"address the issue of changing DLRN hashes"},{"line_number":82,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"772afece_ea292591","line":79,"range":{"start_line":78,"start_character":1,"end_line":79,"end_character":33},"updated":"2021-02-01 21:40:16.000000000","message":"meh.. I don\u0027t think this one is even worth proposing as an alternative","commit_id":"d9d52c4371e0799dc17cf6ebc8378efe12524148"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85819f5c5e118dd9d5852852dc82eaac19d8a15c","unresolved":false,"context_lines":[{"line_number":75,"context_line":"Functionality to set repos, modules and versions is already available today."},{"line_number":76,"context_line":"It would be possible to leave the status quo or:"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"* Create an entirely new repo - outside of anything used in TripleO"},{"line_number":79,"context_line":"right now to do the settings work"},{"line_number":80,"context_line":"* Use rdo config to set one version per release - however, this would not"},{"line_number":81,"context_line":"address the issue of changing DLRN hashes"},{"line_number":82,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"d0ba5209_e5bf8786","line":79,"range":{"start_line":78,"start_character":1,"end_line":79,"end_character":33},"in_reply_to":"772afece_ea292591","updated":"2021-02-01 23:48:08.000000000","message":"Done","commit_id":"d9d52c4371e0799dc17cf6ebc8378efe12524148"},{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"01d6db69d6c5c71ef3c5522f9b07ac36ffe0e4ab","unresolved":true,"context_lines":[{"line_number":78,"context_line":"* Create an entirely new repo - outside of anything used in TripleO"},{"line_number":79,"context_line":"right now to do the settings work"},{"line_number":80,"context_line":"* Use rdo config to set one version per release - however, this would not"},{"line_number":81,"context_line":"address the issue of changing DLRN hashes"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Security Impact"},{"line_number":84,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1063c274_23631134","line":81,"updated":"2021-02-01 21:40:16.000000000","message":"Maybe an alternative could be a rpm that lays down /etc/tripleo-release /etc/osp-release info and container-tools could be meta data in that.  Similar  /etc/os-release or /etc/redhat-release","commit_id":"d9d52c4371e0799dc17cf6ebc8378efe12524148"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85819f5c5e118dd9d5852852dc82eaac19d8a15c","unresolved":false,"context_lines":[{"line_number":78,"context_line":"* Create an entirely new repo - outside of anything used in TripleO"},{"line_number":79,"context_line":"right now to do the settings work"},{"line_number":80,"context_line":"* Use rdo config to set one version per release - however, this would not"},{"line_number":81,"context_line":"address the issue of changing DLRN hashes"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Security Impact"},{"line_number":84,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"b3e9fc1a_5aa79de5","line":81,"in_reply_to":"1063c274_23631134","updated":"2021-02-01 23:48:08.000000000","message":"Done","commit_id":"d9d52c4371e0799dc17cf6ebc8378efe12524148"},{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"b5dad161f65a89eebe2a9bd3c38a19791408a192","unresolved":true,"context_lines":[{"line_number":126,"context_line":"The functionality added to tripleo-repos will be writen as a Python module"},{"line_number":127,"context_line":"with a CLI and will be able to perform the following primary functions:"},{"line_number":128,"context_line":"  * Single source the installation of non-base OS repos for TripleO"},{"line_number":129,"context_line":"  * Include the functionality current available in the repo-setup role:"},{"line_number":130,"context_line":"        * Create repos from templates and files"},{"line_number":131,"context_line":"            * example:"},{"line_number":132,"context_line":"              ```yaml"},{"line_number":133,"context_line":"               - type: file"},{"line_number":134,"context_line":"                 filename: delorean.repo"},{"line_number":135,"context_line":"                 down_url: \"https://trunk.rdoproject.org/{{ distro_ver }}-{{ release }}/${DLRN_PATH_TAG}/delorean.repo\""},{"line_number":136,"context_line":"              ```"},{"line_number":137,"context_line":"              ```yaml"},{"line_number":138,"context_line":"              - type: generic"},{"line_number":139,"context_line":"                reponame: quickstart-centos-base"},{"line_number":140,"context_line":"                filename: quickstart-centos-base.repo"},{"line_number":141,"context_line":"                baseurl: ${NODEPOOL_CENTOS_MIRROR}/8/BaseOS/x86_64/os/"},{"line_number":142,"context_line":"              ```"},{"line_number":143,"context_line":"        * The ```before_cmd``` and ```after_cmd``` functions such as"},{"line_number":144,"context_line":"          ```dnf clean```, ```dnf update```, ```enable```, ```disable```"},{"line_number":145,"context_line":"          will be left in release files where they are currently"},{"line_number":146,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":147,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":148,"context_line":"  * Get and transform dlrn hashes"},{"line_number":149,"context_line":"      * Transform human readable dlrn links ( e.g current-tripleo )"},{"line_number":150,"context_line":"          * to a url with a pinned hash"},{"line_number":151,"context_line":"          * (dlrn new) args: centos8, master, current-tripleo"},{"line_number":152,"context_line":"              * RETURN: https://trunk.rdoproject.org/centos8-master/current-tripleo/19/ac/19ac8a26034e818128be95835e02f49f/"},{"line_number":153,"context_line":"              * RETURN: 19ac8a26034e818128be95835e02f49f"},{"line_number":154,"context_line":"              * RETURN: 19/ac/19ac8a26034e818128be95835e02f49f"},{"line_number":155,"context_line":"           * (dlrn component) args: centos8, master, compute, current-tripleo"},{"line_number":156,"context_line":"               * RETURN: https://trunk.rdoproject.org/centos8-master/component/compute/96/e9/96e9124ad705df82fb4866c9241fac5e5e98b761_73a51637/"},{"line_number":157,"context_line":"               * RETURN: 96e9124ad705df82fb4866c9241fac5e5e98b761_73a51637"},{"line_number":158,"context_line":"               * RETURN: 96/e9/96e9124ad705df82fb4866c9241fac5e5e98b761_73a51637"},{"line_number":159,"context_line":"        * (classic dlrn)args: centos7, stein, current-tripleo"},{"line_number":160,"context_line":"            * RETURN: https://trunk.rdoproject.org/centos7-stein/03/78/037897775d2d628562fb259a7f4c7d5f7f78ea54_8d9f1861/"},{"line_number":161,"context_line":"            * RETURN: /03/78/037897775d2d628562fb259a7f4c7d5f7f78ea54_8d9f1861"},{"line_number":162,"context_line":"            * RETURN 037897775d2d628562fb259a7f4c7d5f7f78ea54_8d9f1861"},{"line_number":163,"context_line":"  * Set dlrn_hash, dlrn_hash_newest, dlrn_hash_tag_newest etc."},{"line_number":164,"context_line":"    https://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-8/master.yml#L7"},{"line_number":165,"context_line":"  * Support setting  yum modules such as container-tools"},{"line_number":166,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"87b857bb_8b5ac88a","line":163,"range":{"start_line":129,"start_character":1,"end_line":163,"end_character":62},"updated":"2021-02-01 21:43:24.000000000","message":"this is not helpful to the reader imho...\n\nspeak can you pull this out and make it more generic?","commit_id":"d9d52c4371e0799dc17cf6ebc8378efe12524148"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85819f5c5e118dd9d5852852dc82eaac19d8a15c","unresolved":false,"context_lines":[{"line_number":126,"context_line":"The functionality added to tripleo-repos will be writen as a Python module"},{"line_number":127,"context_line":"with a CLI and will be able to perform the following primary functions:"},{"line_number":128,"context_line":"  * Single source the installation of non-base OS repos for TripleO"},{"line_number":129,"context_line":"  * Include the functionality current available in the repo-setup role:"},{"line_number":130,"context_line":"        * Create repos from templates and files"},{"line_number":131,"context_line":"            * example:"},{"line_number":132,"context_line":"              ```yaml"},{"line_number":133,"context_line":"               - type: file"},{"line_number":134,"context_line":"                 filename: delorean.repo"},{"line_number":135,"context_line":"                 down_url: \"https://trunk.rdoproject.org/{{ distro_ver }}-{{ release }}/${DLRN_PATH_TAG}/delorean.repo\""},{"line_number":136,"context_line":"              ```"},{"line_number":137,"context_line":"              ```yaml"},{"line_number":138,"context_line":"              - type: generic"},{"line_number":139,"context_line":"                reponame: quickstart-centos-base"},{"line_number":140,"context_line":"                filename: quickstart-centos-base.repo"},{"line_number":141,"context_line":"                baseurl: ${NODEPOOL_CENTOS_MIRROR}/8/BaseOS/x86_64/os/"},{"line_number":142,"context_line":"              ```"},{"line_number":143,"context_line":"        * The ```before_cmd``` and ```after_cmd``` functions such as"},{"line_number":144,"context_line":"          ```dnf clean```, ```dnf update```, ```enable```, ```disable```"},{"line_number":145,"context_line":"          will be left in release files where they are currently"},{"line_number":146,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":147,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":148,"context_line":"  * Get and transform dlrn hashes"},{"line_number":149,"context_line":"      * Transform human readable dlrn links ( e.g current-tripleo )"},{"line_number":150,"context_line":"          * to a url with a pinned hash"},{"line_number":151,"context_line":"          * (dlrn new) args: centos8, master, current-tripleo"},{"line_number":152,"context_line":"              * RETURN: https://trunk.rdoproject.org/centos8-master/current-tripleo/19/ac/19ac8a26034e818128be95835e02f49f/"},{"line_number":153,"context_line":"              * RETURN: 19ac8a26034e818128be95835e02f49f"},{"line_number":154,"context_line":"              * RETURN: 19/ac/19ac8a26034e818128be95835e02f49f"},{"line_number":155,"context_line":"           * (dlrn component) args: centos8, master, compute, current-tripleo"},{"line_number":156,"context_line":"               * RETURN: https://trunk.rdoproject.org/centos8-master/component/compute/96/e9/96e9124ad705df82fb4866c9241fac5e5e98b761_73a51637/"},{"line_number":157,"context_line":"               * RETURN: 96e9124ad705df82fb4866c9241fac5e5e98b761_73a51637"},{"line_number":158,"context_line":"               * RETURN: 96/e9/96e9124ad705df82fb4866c9241fac5e5e98b761_73a51637"},{"line_number":159,"context_line":"        * (classic dlrn)args: centos7, stein, current-tripleo"},{"line_number":160,"context_line":"            * RETURN: https://trunk.rdoproject.org/centos7-stein/03/78/037897775d2d628562fb259a7f4c7d5f7f78ea54_8d9f1861/"},{"line_number":161,"context_line":"            * RETURN: /03/78/037897775d2d628562fb259a7f4c7d5f7f78ea54_8d9f1861"},{"line_number":162,"context_line":"            * RETURN 037897775d2d628562fb259a7f4c7d5f7f78ea54_8d9f1861"},{"line_number":163,"context_line":"  * Set dlrn_hash, dlrn_hash_newest, dlrn_hash_tag_newest etc."},{"line_number":164,"context_line":"    https://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-8/master.yml#L7"},{"line_number":165,"context_line":"  * Support setting  yum modules such as container-tools"},{"line_number":166,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"c777fb6f_52f91207","line":163,"range":{"start_line":129,"start_character":1,"end_line":163,"end_character":62},"in_reply_to":"87b857bb_8b5ac88a","updated":"2021-02-01 23:48:08.000000000","message":"Done","commit_id":"d9d52c4371e0799dc17cf6ebc8378efe12524148"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d04a828ae9e4a27290a3625fddf46105ae2376f6","unresolved":true,"context_lines":[{"line_number":1,"context_line":".."},{"line_number":2,"context_line":" This work is licensed under a Creative Commons Attribution 3.0 Unported"},{"line_number":3,"context_line":" License."},{"line_number":4,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"e0f58551_1a3d1e1f","line":1,"updated":"2021-02-02 08:44:28.000000000","message":"Looks promising","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d04a828ae9e4a27290a3625fddf46105ae2376f6","unresolved":true,"context_lines":[{"line_number":21,"context_line":"and versions are set. Some of these include:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* tripleo repos"},{"line_number":24,"context_line":"  https://github.com/openstack/tripleo-repos"},{"line_number":25,"context_line":"* repo setup role"},{"line_number":26,"context_line":"  https://github.com/openstack/tripleo-quickstart/tree/master/roles/repo-setup"},{"line_number":27,"context_line":"* release config files"}],"source_content_type":"text/x-rst","patch_set":6,"id":"f8ef8f7c_d30a0069","line":24,"range":{"start_line":24,"start_character":10,"end_line":24,"end_character":20},"updated":"2021-02-02 08:44:28.000000000","message":"please replace github.com with opendev.org","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":21,"context_line":"and versions are set. Some of these include:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* tripleo repos"},{"line_number":24,"context_line":"  https://github.com/openstack/tripleo-repos"},{"line_number":25,"context_line":"* repo setup role"},{"line_number":26,"context_line":"  https://github.com/openstack/tripleo-quickstart/tree/master/roles/repo-setup"},{"line_number":27,"context_line":"* release config files"}],"source_content_type":"text/x-rst","patch_set":6,"id":"28e25da3_e09f7039","line":24,"range":{"start_line":24,"start_character":10,"end_line":24,"end_character":20},"in_reply_to":"f8ef8f7c_d30a0069","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d04a828ae9e4a27290a3625fddf46105ae2376f6","unresolved":true,"context_lines":[{"line_number":26,"context_line":"  https://github.com/openstack/tripleo-quickstart/tree/master/roles/repo-setup"},{"line_number":27,"context_line":"* release config files"},{"line_number":28,"context_line":"  https://github.com/openstack/tripleo-quickstart/tree/master/config/release/tripleo-ci/CentOS-8"},{"line_number":29,"context_line":"* container tooling (base tcib file)"},{"line_number":30,"context_line":"  https://github.com/openstack/tripleo-common/blob/master/container-images/tcib/base/base.yaml#L59"},{"line_number":31,"context_line":"* tripleo-ansible"},{"line_number":32,"context_line":"  https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_podman/vars/redhat-8.2.yml"}],"source_content_type":"text/x-rst","patch_set":6,"id":"af04c360_3eee76ab","line":29,"updated":"2021-02-02 08:44:28.000000000","message":"Can we also mention about \n-\u003e getting and resolving mirrors\n-\u003e Getting the proper dlrn hashes https://opendev.org/openstack/tripleo-quickstart/src/branch/master/config/release/tripleo-ci/CentOS-8/master.yml#L86 (whole shell script needs to be replaced with python code)","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d04a828ae9e4a27290a3625fddf46105ae2376f6","unresolved":true,"context_lines":[{"line_number":71,"context_line":"* release files"},{"line_number":72,"context_line":"* container and overcloud image builds"},{"line_number":73,"context_line":"* rdo config"},{"line_number":74,"context_line":"* yum/dnf repos and modules"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Alternatives"},{"line_number":77,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"0bf72619_da6a9929","line":74,"updated":"2021-02-02 08:44:28.000000000","message":"Pulling proper hashes for each promotion pipelines","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":71,"context_line":"* release files"},{"line_number":72,"context_line":"* container and overcloud image builds"},{"line_number":73,"context_line":"* rdo config"},{"line_number":74,"context_line":"* yum/dnf repos and modules"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Alternatives"},{"line_number":77,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"b4f9e7e4_91d2ca98","line":74,"in_reply_to":"0bf72619_da6a9929","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"983a23e3845d96b542180ef56aa81ec88fae1ca2","unresolved":true,"context_lines":[{"line_number":71,"context_line":"* release files"},{"line_number":72,"context_line":"* container and overcloud image builds"},{"line_number":73,"context_line":"* rdo config"},{"line_number":74,"context_line":"* yum/dnf repos and modules"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Alternatives"},{"line_number":77,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"6cd9c58a_a2c85184","line":74,"in_reply_to":"0bf72619_da6a9929","updated":"2021-02-02 14:24:54.000000000","message":"please also note that since that makes the tool much more complex and fragile, unit tests should become a must","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"c1a63315774ed6f6dafe9f8997ee2dcf04d4ac02","unresolved":false,"context_lines":[{"line_number":71,"context_line":"* release files"},{"line_number":72,"context_line":"* container and overcloud image builds"},{"line_number":73,"context_line":"* rdo config"},{"line_number":74,"context_line":"* yum/dnf repos and modules"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Alternatives"},{"line_number":77,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"962f984c_794e8f76","line":74,"in_reply_to":"6cd9c58a_a2c85184","updated":"2021-02-02 20:48:55.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a5b9559f5d1c669e117e0209160a9be6efa16954","unresolved":true,"context_lines":[{"line_number":91,"context_line":"open-source repos and will remain there - just in a consoldated"},{"line_number":92,"context_line":"place and format."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Upgrade Impact"},{"line_number":95,"context_line":"--------------"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Currently there wil be no upgrade impact. The new CLI will support"}],"source_content_type":"text/x-rst","patch_set":6,"id":"d519a175_877c31e1","line":94,"range":{"start_line":94,"start_character":0,"end_line":94,"end_character":14},"updated":"2021-02-02 13:16:30.000000000","message":"I wonder if we need to include there also \"emit-releases\" file functionality","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":91,"context_line":"open-source repos and will remain there - just in a consoldated"},{"line_number":92,"context_line":"place and format."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Upgrade Impact"},{"line_number":95,"context_line":"--------------"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Currently there wil be no upgrade impact. The new CLI will support"}],"source_content_type":"text/x-rst","patch_set":6,"id":"adf7b7df_2f910bf1","line":94,"range":{"start_line":94,"start_character":0,"end_line":94,"end_character":14},"in_reply_to":"d519a175_877c31e1","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"change_message_id":"9382555a0e4e05a1523897506734eb69c776810a","unresolved":true,"context_lines":[{"line_number":94,"context_line":"Upgrade Impact"},{"line_number":95,"context_line":"--------------"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Currently there wil be no upgrade impact. The new CLI will support"},{"line_number":98,"context_line":"all release versions under support and in use. At a later date,"},{"line_number":99,"context_line":"when the old CLI is deprecated there may be some update"},{"line_number":100,"context_line":"implications."}],"source_content_type":"text/x-rst","patch_set":6,"id":"a8db26dd_f8439c2a","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":19},"updated":"2021-02-02 11:58:00.000000000","message":"s/wil/will","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":94,"context_line":"Upgrade Impact"},{"line_number":95,"context_line":"--------------"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Currently there wil be no upgrade impact. The new CLI will support"},{"line_number":98,"context_line":"all release versions under support and in use. At a later date,"},{"line_number":99,"context_line":"when the old CLI is deprecated there may be some update"},{"line_number":100,"context_line":"implications."}],"source_content_type":"text/x-rst","patch_set":6,"id":"d97f1677_b70a527f","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":19},"in_reply_to":"a8db26dd_f8439c2a","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a5b9559f5d1c669e117e0209160a9be6efa16954","unresolved":true,"context_lines":[{"line_number":102,"context_line":"Other End User Impact"},{"line_number":103,"context_line":"---------------------"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"The design will aim to keep the interfaces the same."},{"line_number":106,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":107,"context_line":"End users would be able to select which version of the CLI to use - until"},{"line_number":108,"context_line":"the old CLI is deprecated."}],"source_content_type":"text/x-rst","patch_set":6,"id":"c90f0b7c_e05b84b6","line":105,"range":{"start_line":105,"start_character":0,"end_line":105,"end_character":52},"updated":"2021-02-02 13:16:30.000000000","message":"Actually it aims to change the design.The explanation below is enough, but this line is confusing.","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":102,"context_line":"Other End User Impact"},{"line_number":103,"context_line":"---------------------"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"The design will aim to keep the interfaces the same."},{"line_number":106,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":107,"context_line":"End users would be able to select which version of the CLI to use - until"},{"line_number":108,"context_line":"the old CLI is deprecated."}],"source_content_type":"text/x-rst","patch_set":6,"id":"db9127b3_2c4acfba","line":105,"range":{"start_line":105,"start_character":0,"end_line":105,"end_character":52},"in_reply_to":"c90f0b7c_e05b84b6","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fea3347edf1415090d8d8355842610c43ea639e0","unresolved":true,"context_lines":[{"line_number":106,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":107,"context_line":"End users would be able to select which version of the CLI to use - until"},{"line_number":108,"context_line":"the old CLI is deprecated."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"Performance Impact"},{"line_number":112,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"4004381b_5a00225a","line":109,"updated":"2021-02-02 12:46:59.000000000","message":"May be we can add cli interface if we do this , it will produce this?(Not sure it would be good to give what is going to do)","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d04a828ae9e4a27290a3625fddf46105ae2376f6","unresolved":true,"context_lines":[{"line_number":132,"context_line":"  * Single source the installation of non-base OS repos for TripleO"},{"line_number":133,"context_line":"  * Include the functionality current available in the repo-setup role"},{"line_number":134,"context_line":"  including creating repos from templates and files"},{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"}],"source_content_type":"text/x-rst","patch_set":6,"id":"dae1b799_ee8be70c","line":135,"range":{"start_line":135,"start_character":4,"end_line":135,"end_character":1},"updated":"2021-02-02 08:44:28.000000000","message":"May be we add about this also in problem statement, how messy the current situation is","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"c1a63315774ed6f6dafe9f8997ee2dcf04d4ac02","unresolved":false,"context_lines":[{"line_number":132,"context_line":"  * Single source the installation of non-base OS repos for TripleO"},{"line_number":133,"context_line":"  * Include the functionality current available in the repo-setup role"},{"line_number":134,"context_line":"  including creating repos from templates and files"},{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"}],"source_content_type":"text/x-rst","patch_set":6,"id":"c2a47cc1_74328da2","line":135,"range":{"start_line":135,"start_character":4,"end_line":135,"end_character":1},"in_reply_to":"21963383_187b6b25","updated":"2021-02-02 20:48:55.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":true,"context_lines":[{"line_number":132,"context_line":"  * Single source the installation of non-base OS repos for TripleO"},{"line_number":133,"context_line":"  * Include the functionality current available in the repo-setup role"},{"line_number":134,"context_line":"  including creating repos from templates and files"},{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"}],"source_content_type":"text/x-rst","patch_set":6,"id":"21963383_187b6b25","line":135,"range":{"start_line":135,"start_character":4,"end_line":135,"end_character":1},"in_reply_to":"dae1b799_ee8be70c","updated":"2021-02-02 14:09:20.000000000","message":"yes - will add","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a5b9559f5d1c669e117e0209160a9be6efa16954","unresolved":true,"context_lines":[{"line_number":133,"context_line":"  * Include the functionality current available in the repo-setup role"},{"line_number":134,"context_line":"  including creating repos from templates and files"},{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"},{"line_number":139,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"39e32419_89ea69df","line":136,"range":{"start_line":136,"start_character":35,"end_line":136,"end_character":39},"updated":"2021-02-02 13:16:30.000000000","message":"DLRN","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":133,"context_line":"  * Include the functionality current available in the repo-setup role"},{"line_number":134,"context_line":"  including creating repos from templates and files"},{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"},{"line_number":139,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"8d41e643_0fd76439","line":136,"range":{"start_line":136,"start_character":35,"end_line":136,"end_character":39},"in_reply_to":"39e32419_89ea69df","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a5b9559f5d1c669e117e0209160a9be6efa16954","unresolved":true,"context_lines":[{"line_number":134,"context_line":"  including creating repos from templates and files"},{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"The repo-setup role will remain but it will invoke tripleo-repos."}],"source_content_type":"text/x-rst","patch_set":6,"id":"898c3383_92d333d4","line":137,"range":{"start_line":137,"start_character":28,"end_line":137,"end_character":35},"updated":"2021-02-02 13:16:30.000000000","message":"readable","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":134,"context_line":"  including creating repos from templates and files"},{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"The repo-setup role will remain but it will invoke tripleo-repos."}],"source_content_type":"text/x-rst","patch_set":6,"id":"604180b6_dbd3a527","line":137,"range":{"start_line":137,"start_character":28,"end_line":137,"end_character":35},"in_reply_to":"898c3383_92d333d4","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a5b9559f5d1c669e117e0209160a9be6efa16954","unresolved":true,"context_lines":[{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"The repo-setup role will remain but it will invoke tripleo-repos."},{"line_number":141,"context_line":"All options required to be passed to tripleo-repos should be in the release file."}],"source_content_type":"text/x-rst","patch_set":6,"id":"6185eab9_6fc58d15","line":138,"range":{"start_line":138,"start_character":2,"end_line":138,"end_character":4},"updated":"2021-02-02 13:16:30.000000000","message":"Also enabling and disabling repositories","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":135,"context_line":"  * Perform proxy handling such as is done in the release files"},{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"The repo-setup role will remain but it will invoke tripleo-repos."},{"line_number":141,"context_line":"All options required to be passed to tripleo-repos should be in the release file."}],"source_content_type":"text/x-rst","patch_set":6,"id":"af78ab55_f47242ae","line":138,"range":{"start_line":138,"start_character":2,"end_line":138,"end_character":4},"in_reply_to":"6185eab9_6fc58d15","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d04a828ae9e4a27290a3625fddf46105ae2376f6","unresolved":true,"context_lines":[{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"The repo-setup role will remain but it will invoke tripleo-repos."},{"line_number":141,"context_line":"All options required to be passed to tripleo-repos should be in the release file."},{"line_number":142,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7325f7d0_ce001be5","line":139,"updated":"2021-02-02 08:44:28.000000000","message":"I think there was a requirements there old unit tests will work fine with the new refactored code?","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    (mirrors, using rdoproject for DLNR repos)"},{"line_number":137,"context_line":"  * Get and transform human-readble dlrn hashes"},{"line_number":138,"context_line":"  * Support setting yum modules such as container-tools"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"The repo-setup role will remain but it will invoke tripleo-repos."},{"line_number":141,"context_line":"All options required to be passed to tripleo-repos should be in the release file."},{"line_number":142,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"5b06b279_53450ea9","line":139,"in_reply_to":"7325f7d0_ce001be5","updated":"2021-02-02 14:09:20.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a5b9559f5d1c669e117e0209160a9be6efa16954","unresolved":true,"context_lines":[{"line_number":140,"context_line":"The repo-setup role will remain but it will invoke tripleo-repos."},{"line_number":141,"context_line":"All options required to be passed to tripleo-repos should be in the release file."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"The design will aim to keep the interfaces the same."},{"line_number":144,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":145,"context_line":"Unit tests will be added on this branch to test the new CLI directly."},{"line_number":146,"context_line":"CI would be flipped to run in the new branch when approved by DF and CI teams."}],"source_content_type":"text/x-rst","patch_set":6,"id":"942bbacf_1b766c17","line":143,"range":{"start_line":143,"start_character":0,"end_line":143,"end_character":1},"updated":"2021-02-02 13:16:30.000000000","message":"Seems like this section is a duplicate from line 105?","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":false,"context_lines":[{"line_number":140,"context_line":"The repo-setup role will remain but it will invoke tripleo-repos."},{"line_number":141,"context_line":"All options required to be passed to tripleo-repos should be in the release file."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"The design will aim to keep the interfaces the same."},{"line_number":144,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":145,"context_line":"Unit tests will be added on this branch to test the new CLI directly."},{"line_number":146,"context_line":"CI would be flipped to run in the new branch when approved by DF and CI teams."}],"source_content_type":"text/x-rst","patch_set":6,"id":"62f6687f_5dbc8196","line":143,"range":{"start_line":143,"start_character":0,"end_line":143,"end_character":1},"in_reply_to":"942bbacf_1b766c17","updated":"2021-02-02 14:09:20.000000000","message":"Ack","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"983a23e3845d96b542180ef56aa81ec88fae1ca2","unresolved":true,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"The design will aim to keep the interfaces the same."},{"line_number":144,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":145,"context_line":"Unit tests will be added on this branch to test the new CLI directly."},{"line_number":146,"context_line":"CI would be flipped to run in the new branch when approved by DF and CI teams."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"Open question:"}],"source_content_type":"text/x-rst","patch_set":6,"id":"ea91b54b_b4339e19","line":145,"updated":"2021-02-02 14:24:54.000000000","message":"s/will/shall","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"c1a63315774ed6f6dafe9f8997ee2dcf04d4ac02","unresolved":false,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"The design will aim to keep the interfaces the same."},{"line_number":144,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":145,"context_line":"Unit tests will be added on this branch to test the new CLI directly."},{"line_number":146,"context_line":"CI would be flipped to run in the new branch when approved by DF and CI teams."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"Open question:"}],"source_content_type":"text/x-rst","patch_set":6,"id":"f8d926d1_23e1a85f","line":145,"in_reply_to":"ea91b54b_b4339e19","updated":"2021-02-02 20:48:55.000000000","message":"Done","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"45607ef4a4a55a957943c998e38b68d172cc44bd","unresolved":true,"context_lines":[{"line_number":157,"context_line":"* weshay"},{"line_number":158,"context_line":"* chandankumar"},{"line_number":159,"context_line":"* ysandeep"},{"line_number":160,"context_line":"* others in CI?"},{"line_number":161,"context_line":"* rlandy"},{"line_number":162,"context_line":"* other DF members (cloudnull?)"},{"line_number":163,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"51400747_aa43690c","line":160,"updated":"2021-02-02 12:08:04.000000000","message":"Me me, pick me!","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a5b9559f5d1c669e117e0209160a9be6efa16954","unresolved":true,"context_lines":[{"line_number":168,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"Investigative work will be begin in the W-release cycle on a project branch"},{"line_number":171,"context_line":"in tripleo-repos. The spec will be put forward for approval in the U-release"},{"line_number":172,"context_line":"cycle and impactful and integration work will be visible once the spec"},{"line_number":173,"context_line":"is approved."},{"line_number":174,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"83eddc20_a5154296","line":171,"range":{"start_line":171,"start_character":67,"end_line":171,"end_character":69},"updated":"2021-02-02 13:16:30.000000000","message":"W?","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9d9e4e7c348a2c441f5413695435f791985afa6f","unresolved":true,"context_lines":[{"line_number":168,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"Investigative work will be begin in the W-release cycle on a project branch"},{"line_number":171,"context_line":"in tripleo-repos. The spec will be put forward for approval in the U-release"},{"line_number":172,"context_line":"cycle and impactful and integration work will be visible once the spec"},{"line_number":173,"context_line":"is approved."},{"line_number":174,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"75874c58_46f6f3cc","line":171,"range":{"start_line":171,"start_character":67,"end_line":171,"end_character":69},"in_reply_to":"83eddc20_a5154296","updated":"2021-02-02 14:09:20.000000000","message":"I think W is closed","commit_id":"b56f83d65d614065a02109b243731e2547489389"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b5f950be945d2482a5bdb48952570cbf660d0206","unresolved":true,"context_lines":[{"line_number":23,"context_line":"* tripleo repos"},{"line_number":24,"context_line":"  https://opendev.org/openstack/tripleo-repos"},{"line_number":25,"context_line":"* repo setup role"},{"line_number":26,"context_line":"  https://opendev.org/openstack/tripleo-quickstart/tree/master/roles/repo-setup"},{"line_number":27,"context_line":"* release config files"},{"line_number":28,"context_line":"  https://opendev.org/openstack/tripleo-quickstart/tree/master/config/release/tripleo-ci/CentOS-8"},{"line_number":29,"context_line":"* container tooling (base tcib file)"}],"source_content_type":"text/x-rst","patch_set":11,"id":"47d0b10c_d3e031cc","line":26,"range":{"start_line":26,"start_character":2,"end_line":26,"end_character":79},"updated":"2021-02-03 07:51:38.000000000","message":"also 404 https://opendev.org/openstack/tripleo-quickstart/src/branch/master/roles/repo-setup i think maybe these were the github links and you changed to opendev.org?","commit_id":"7138628cc1756097d1eaa9421f16ba9ebe79f5a9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b5f950be945d2482a5bdb48952570cbf660d0206","unresolved":true,"context_lines":[{"line_number":28,"context_line":"  https://opendev.org/openstack/tripleo-quickstart/tree/master/config/release/tripleo-ci/CentOS-8"},{"line_number":29,"context_line":"* container tooling (base tcib file)"},{"line_number":30,"context_line":"  https://opendev.org/openstack/tripleo-common/blob/master/container-images/tcib/base/base.yaml#L59"},{"line_number":31,"context_line":"* tripleo-ansible"},{"line_number":32,"context_line":"  https://opendev.org/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_podman/vars/redhat-8.2.yml"},{"line_number":33,"context_line":"* rdo config"},{"line_number":34,"context_line":"  https://review.rdoproject.org/r/31439 (example)"}],"source_content_type":"text/x-rst","patch_set":11,"id":"a1e3899e_dfd385cd","line":31,"range":{"start_line":31,"start_character":2,"end_line":31,"end_character":9},"updated":"2021-02-03 07:51:38.000000000","message":"i think it would be easier to read this if instead of providing the link here just use the place name e.g. \"tripleo-ansible\" and either make it a hyperlink or provider references at the botton like \"tripleo-ansible [1]\" type thing","commit_id":"7138628cc1756097d1eaa9421f16ba9ebe79f5a9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"114c18432dbc77717c7ba913a30e3c58f8701f36","unresolved":true,"context_lines":[{"line_number":28,"context_line":"  https://opendev.org/openstack/tripleo-quickstart/tree/master/config/release/tripleo-ci/CentOS-8"},{"line_number":29,"context_line":"* container tooling (base tcib file)"},{"line_number":30,"context_line":"  https://opendev.org/openstack/tripleo-common/blob/master/container-images/tcib/base/base.yaml#L59"},{"line_number":31,"context_line":"* tripleo-ansible"},{"line_number":32,"context_line":"  https://opendev.org/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_podman/vars/redhat-8.2.yml"},{"line_number":33,"context_line":"* rdo config"},{"line_number":34,"context_line":"  https://review.rdoproject.org/r/31439 (example)"}],"source_content_type":"text/x-rst","patch_set":11,"id":"13707ead_dc103b42","line":31,"range":{"start_line":31,"start_character":2,"end_line":31,"end_character":9},"in_reply_to":"a1e3899e_dfd385cd","updated":"2021-02-03 16:09:59.000000000","message":"update incoming to fix up the links","commit_id":"7138628cc1756097d1eaa9421f16ba9ebe79f5a9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b5f950be945d2482a5bdb48952570cbf660d0206","unresolved":true,"context_lines":[{"line_number":29,"context_line":"* container tooling (base tcib file)"},{"line_number":30,"context_line":"  https://opendev.org/openstack/tripleo-common/blob/master/container-images/tcib/base/base.yaml#L59"},{"line_number":31,"context_line":"* tripleo-ansible"},{"line_number":32,"context_line":"  https://opendev.org/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_podman/vars/redhat-8.2.yml"},{"line_number":33,"context_line":"* rdo config"},{"line_number":34,"context_line":"  https://review.rdoproject.org/r/31439 (example)"},{"line_number":35,"context_line":"* other examples:"}],"source_content_type":"text/x-rst","patch_set":11,"id":"9a1c1592_73140053","line":32,"updated":"2021-02-03 07:51:38.000000000","message":"that is 404 it needs to be: \nhttps://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_podman/vars/redhat-8.2.yml","commit_id":"7138628cc1756097d1eaa9421f16ba9ebe79f5a9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b5f950be945d2482a5bdb48952570cbf660d0206","unresolved":true,"context_lines":[{"line_number":47,"context_line":"for setting repo configurations, modules and supported versions"},{"line_number":48,"context_line":"is confusing and error prone."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"There should be one source of truth for which repos are installed"},{"line_number":51,"context_line":"within a tripleo deployment and how they are installed."},{"line_number":52,"context_line":"Single-sourcing all these functions will avoid the current"},{"line_number":53,"context_line":"problems of duplication, over-writing settings and version confusion."}],"source_content_type":"text/x-rst","patch_set":11,"id":"2bcc7cf5_f75265ec","line":50,"range":{"start_line":50,"start_character":20,"end_line":50,"end_character":26},"updated":"2021-02-03 07:51:38.000000000","message":"one tool for laying down the tripleo related repos prior to a deployment (just suggestion)","commit_id":"7138628cc1756097d1eaa9421f16ba9ebe79f5a9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b5f950be945d2482a5bdb48952570cbf660d0206","unresolved":true,"context_lines":[{"line_number":66,"context_line":"Having a single source of truth for repo config, modules, etc will make"},{"line_number":67,"context_line":"development and testing more consistent, reliable and easier to debug."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"One example review: https://review.opendev.org/c/openstack/tripleo-repos/+/767214"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Integration Points"}],"source_content_type":"text/x-rst","patch_set":11,"id":"794bdc2e_7d56bcfd","line":69,"range":{"start_line":69,"start_character":0,"end_line":69,"end_character":18},"updated":"2021-02-03 07:51:38.000000000","message":"can you expand on that a bit. what is the review showing etc","commit_id":"7138628cc1756097d1eaa9421f16ba9ebe79f5a9"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"149f070c618211f367cf9e1267fa213b8afef4b8","unresolved":true,"context_lines":[{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This proposal lays out the plan to use TripleO Repos as a single source"},{"line_number":12,"context_line":"to install and config non-base OS repos for TripleO - including"},{"line_number":13,"context_line":"setting the required DLRN hashes."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"https://blueprints.launchpad.net/tripleo/+spec/tripleo-repos-single-source"}],"source_content_type":"text/x-rst","patch_set":12,"id":"58a15345_4b48f77a","line":12,"range":{"start_line":12,"start_character":15,"end_line":12,"end_character":21},"updated":"2021-02-08 09:14:19.000000000","message":"configure ?","commit_id":"ffdbd45f75452bbef72966101e0e6df4cc005d0d"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"45c57519d017d95079ed071d4529d9a48711a9ae","unresolved":true,"context_lines":[{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This proposal lays out the plan to use TripleO Repos as a single source"},{"line_number":12,"context_line":"to install and config non-base OS repos for TripleO - including"},{"line_number":13,"context_line":"setting the required DLRN hashes."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"https://blueprints.launchpad.net/tripleo/+spec/tripleo-repos-single-source"}],"source_content_type":"text/x-rst","patch_set":12,"id":"2450cd5a_9fe54eed","line":12,"range":{"start_line":12,"start_character":15,"end_line":12,"end_character":21},"in_reply_to":"58a15345_4b48f77a","updated":"2021-02-08 13:19:22.000000000","message":"updating this one","commit_id":"ffdbd45f75452bbef72966101e0e6df4cc005d0d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"252352bc7c231ddb908f65224b051783ee23d33b","unresolved":true,"context_lines":[{"line_number":17,"context_line":"Problem Description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Reviewing the code base, there are multiple places where repos are set up"},{"line_number":21,"context_line":"and versions are set. Some of these include:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* `tripleo repos`_"}],"source_content_type":"text/x-rst","patch_set":14,"id":"f9fbfaad_7d41eac0","line":20,"range":{"start_line":20,"start_character":68,"end_line":20,"end_character":73},"updated":"2021-02-16 08:42:05.000000000","message":"let\u0027s say \u0027specified\u0027 and/or set-up e.g. in the release files we are just setting the configuration that is actually applied by repo setup role","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Problem Description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Reviewing the code base, there are multiple places where repos are set up"},{"line_number":21,"context_line":"and versions are set. Some of these include:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* `tripleo repos`_"}],"source_content_type":"text/x-rst","patch_set":14,"id":"02b5ed7e_7de60cc5","line":20,"range":{"start_line":20,"start_character":68,"end_line":20,"end_character":73},"in_reply_to":"f9fbfaad_7d41eac0","updated":"2021-02-23 11:29:17.000000000","message":"Done","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"b589d31400f2307e02a3a5cd9943837ccaf14124","unresolved":true,"context_lines":[{"line_number":21,"context_line":"and versions are set. Some of these include:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* `tripleo repos`_"},{"line_number":24,"context_line":"* `repo setup role`_"},{"line_number":25,"context_line":"* `release config files`_"},{"line_number":26,"context_line":"* `container tooling (base tcib file)`_"},{"line_number":27,"context_line":"* `tripleo-ansible`_"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5eabd55c_44eb1c84","line":24,"updated":"2021-02-11 06:10:40.000000000","message":"For repo setup i think currently these(tripleo-repos CLI and repo-setup ansible role) are the only two ways upstream, Other below are using one of them or are setting or providing(like tripleo-ansible) a way to set up dnf modules or defining set of packages to install.","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"252352bc7c231ddb908f65224b051783ee23d33b","unresolved":true,"context_lines":[{"line_number":21,"context_line":"and versions are set. Some of these include:"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* `tripleo repos`_"},{"line_number":24,"context_line":"* `repo setup role`_"},{"line_number":25,"context_line":"* `release config files`_"},{"line_number":26,"context_line":"* `container tooling (base tcib file)`_"},{"line_number":27,"context_line":"* `tripleo-ansible`_"}],"source_content_type":"text/x-rst","patch_set":14,"id":"e82780ba_e070ad54","line":24,"in_reply_to":"5eabd55c_44eb1c84","updated":"2021-02-16 08:42:05.000000000","message":"ack, let\u0027s call this \u0027we have multiple places where repos are specified\u0027 then release files are ok in this list","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"252352bc7c231ddb908f65224b051783ee23d33b","unresolved":true,"context_lines":[{"line_number":39,"context_line":".. _`tripleo-operator-ansible`: https://opendev.org/openstack/tripleo-operator-ansible/src/commit/14a601a47be217386df83512fae3a2e5aa5444a3/roles/tripleo_container_image_build/molecule/default/converge.yml#L172"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"The process of setting repo versions often requires getting and"},{"line_number":43,"context_line":"transforming DLRN hashes and specifying the correct proxies."},{"line_number":44,"context_line":"Currently a large portion of this work is done in the release files"},{"line_number":45,"context_line":"resulting in sections of complicated and fragile Bash scripts -"}],"source_content_type":"text/x-rst","patch_set":14,"id":"c75395ad_b446d319","line":42,"range":{"start_line":42,"start_character":37,"end_line":42,"end_character":42},"updated":"2021-02-16 08:42:05.000000000","message":"remove","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":false,"context_lines":[{"line_number":39,"context_line":".. _`tripleo-operator-ansible`: https://opendev.org/openstack/tripleo-operator-ansible/src/commit/14a601a47be217386df83512fae3a2e5aa5444a3/roles/tripleo_container_image_build/molecule/default/converge.yml#L172"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"The process of setting repo versions often requires getting and"},{"line_number":43,"context_line":"transforming DLRN hashes and specifying the correct proxies."},{"line_number":44,"context_line":"Currently a large portion of this work is done in the release files"},{"line_number":45,"context_line":"resulting in sections of complicated and fragile Bash scripts -"}],"source_content_type":"text/x-rst","patch_set":14,"id":"616e5d41_725cc035","line":42,"range":{"start_line":42,"start_character":37,"end_line":42,"end_character":42},"in_reply_to":"c75395ad_b446d319","updated":"2021-02-23 11:29:17.000000000","message":"Done","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"252352bc7c231ddb908f65224b051783ee23d33b","unresolved":true,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"The process of setting repo versions often requires getting and"},{"line_number":43,"context_line":"transforming DLRN hashes and specifying the correct proxies."},{"line_number":44,"context_line":"Currently a large portion of this work is done in the release files"},{"line_number":45,"context_line":"resulting in sections of complicated and fragile Bash scripts -"},{"line_number":46,"context_line":"duplicated over numerous release files."}],"source_content_type":"text/x-rst","patch_set":14,"id":"b94b52d5_ec13354b","line":43,"updated":"2021-02-16 08:42:05.000000000","message":"DLRN hashes, for example resolving \u0027current-tripleo\u0027 to a particular DLRN build ID","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"The process of setting repo versions often requires getting and"},{"line_number":43,"context_line":"transforming DLRN hashes and specifying the correct proxies."},{"line_number":44,"context_line":"Currently a large portion of this work is done in the release files"},{"line_number":45,"context_line":"resulting in sections of complicated and fragile Bash scripts -"},{"line_number":46,"context_line":"duplicated over numerous release files."}],"source_content_type":"text/x-rst","patch_set":14,"id":"60c2b6f4_f6ef030c","line":43,"in_reply_to":"b94b52d5_ec13354b","updated":"2021-02-23 11:29:17.000000000","message":"Done","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"252352bc7c231ddb908f65224b051783ee23d33b","unresolved":true,"context_lines":[{"line_number":43,"context_line":"transforming DLRN hashes and specifying the correct proxies."},{"line_number":44,"context_line":"Currently a large portion of this work is done in the release files"},{"line_number":45,"context_line":"resulting in sections of complicated and fragile Bash scripts -"},{"line_number":46,"context_line":"duplicated over numerous release files."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"This duplication, coupled with the various locations in use"},{"line_number":49,"context_line":"for setting repo configurations, modules and supported versions"}],"source_content_type":"text/x-rst","patch_set":14,"id":"bb8f10bd_78149450","line":46,"range":{"start_line":46,"start_character":11,"end_line":46,"end_character":15},"updated":"2021-02-16 08:42:05.000000000","message":"across","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":false,"context_lines":[{"line_number":43,"context_line":"transforming DLRN hashes and specifying the correct proxies."},{"line_number":44,"context_line":"Currently a large portion of this work is done in the release files"},{"line_number":45,"context_line":"resulting in sections of complicated and fragile Bash scripts -"},{"line_number":46,"context_line":"duplicated over numerous release files."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"This duplication, coupled with the various locations in use"},{"line_number":49,"context_line":"for setting repo configurations, modules and supported versions"}],"source_content_type":"text/x-rst","patch_set":14,"id":"e9a139c7_509a0cec","line":46,"range":{"start_line":46,"start_character":11,"end_line":46,"end_character":15},"in_reply_to":"bb8f10bd_78149450","updated":"2021-02-23 11:29:17.000000000","message":"Done","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"252352bc7c231ddb908f65224b051783ee23d33b","unresolved":true,"context_lines":[{"line_number":74,"context_line":"Integration Points"},{"line_number":75,"context_line":"------------------"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"The new work in tripleo-repos will have to support and integrate with all"},{"line_number":78,"context_line":"the cases currently in use and will have to integrate with:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* DLRN API"}],"source_content_type":"text/x-rst","patch_set":14,"id":"cafaadb3_4f026f11","line":77,"range":{"start_line":77,"start_character":50,"end_line":77,"end_character":69},"updated":"2021-02-16 08:42:05.000000000","message":"remove (duplicated at end of line)","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":false,"context_lines":[{"line_number":74,"context_line":"Integration Points"},{"line_number":75,"context_line":"------------------"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"The new work in tripleo-repos will have to support and integrate with all"},{"line_number":78,"context_line":"the cases currently in use and will have to integrate with:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* DLRN API"}],"source_content_type":"text/x-rst","patch_set":14,"id":"268d57ab_5c9af69c","line":77,"range":{"start_line":77,"start_character":50,"end_line":77,"end_character":69},"in_reply_to":"cafaadb3_4f026f11","updated":"2021-02-23 11:29:17.000000000","message":"Done","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"b589d31400f2307e02a3a5cd9943837ccaf14124","unresolved":true,"context_lines":[{"line_number":77,"context_line":"The new work in tripleo-repos will have to support and integrate with all"},{"line_number":78,"context_line":"the cases currently in use and will have to integrate with:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* DLRN API"},{"line_number":81,"context_line":"* release files"},{"line_number":82,"context_line":"* container and overcloud image builds"},{"line_number":83,"context_line":"* rdo config"}],"source_content_type":"text/x-rst","patch_set":14,"id":"a5148579_cb1a6091","line":80,"range":{"start_line":80,"start_character":7,"end_line":80,"end_character":10},"updated":"2021-02-11 06:10:40.000000000","message":"didn\u0027t got the API part, or u just meant DLRN repos here, is the plan to call dlrnapi to get some information and use that information to setup repos?","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":true,"context_lines":[{"line_number":77,"context_line":"The new work in tripleo-repos will have to support and integrate with all"},{"line_number":78,"context_line":"the cases currently in use and will have to integrate with:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"* DLRN API"},{"line_number":81,"context_line":"* release files"},{"line_number":82,"context_line":"* container and overcloud image builds"},{"line_number":83,"context_line":"* rdo config"}],"source_content_type":"text/x-rst","patch_set":14,"id":"0f17743f_273f38f2","line":80,"range":{"start_line":80,"start_character":7,"end_line":80,"end_character":10},"in_reply_to":"a5148579_cb1a6091","updated":"2021-02-23 11:29:17.000000000","message":"DLRN repos","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"b5150dfb9899b3c96a976f66395e89c8eecc7268","unresolved":true,"context_lines":[{"line_number":104,"context_line":"---------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"No security impact is anticipated. The work is currently in the tripleo"},{"line_number":107,"context_line":"open-source repos and will remain there - just in a consoldated"},{"line_number":108,"context_line":"place and format."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"Upgrade Impact"}],"source_content_type":"text/x-rst","patch_set":14,"id":"422eb3ab_e44e7c16","line":107,"range":{"start_line":107,"start_character":52,"end_line":107,"end_character":63},"updated":"2021-02-10 19:16:12.000000000","message":"nit, typo","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":false,"context_lines":[{"line_number":104,"context_line":"---------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"No security impact is anticipated. The work is currently in the tripleo"},{"line_number":107,"context_line":"open-source repos and will remain there - just in a consoldated"},{"line_number":108,"context_line":"place and format."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"Upgrade Impact"}],"source_content_type":"text/x-rst","patch_set":14,"id":"6038286c_51756536","line":107,"range":{"start_line":107,"start_character":52,"end_line":107,"end_character":63},"in_reply_to":"422eb3ab_e44e7c16","updated":"2021-02-23 11:29:17.000000000","message":"Done","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"252352bc7c231ddb908f65224b051783ee23d33b","unresolved":true,"context_lines":[{"line_number":149,"context_line":"The functionality added to tripleo-repos will be writen as a Python module"},{"line_number":150,"context_line":"with a CLI and will be able to perform the following primary functions:"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"* Single source the installation of non-base OS repos for TripleO"},{"line_number":153,"context_line":"* Include the functionality current available in the repo-setup role"},{"line_number":154,"context_line":"  including creating repos from templates and files"},{"line_number":155,"context_line":"* Perform proxy handling such as is done in the release files"}],"source_content_type":"text/x-rst","patch_set":14,"id":"5788dc93_9f466cb4","line":152,"range":{"start_line":152,"start_character":37,"end_line":152,"end_character":65},"updated":"2021-02-16 08:42:05.000000000","message":"all TripleO related repos","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":false,"context_lines":[{"line_number":149,"context_line":"The functionality added to tripleo-repos will be writen as a Python module"},{"line_number":150,"context_line":"with a CLI and will be able to perform the following primary functions:"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"* Single source the installation of non-base OS repos for TripleO"},{"line_number":153,"context_line":"* Include the functionality current available in the repo-setup role"},{"line_number":154,"context_line":"  including creating repos from templates and files"},{"line_number":155,"context_line":"* Perform proxy handling such as is done in the release files"}],"source_content_type":"text/x-rst","patch_set":14,"id":"8c78b22a_9491a1bc","line":152,"range":{"start_line":152,"start_character":37,"end_line":152,"end_character":65},"in_reply_to":"5788dc93_9f466cb4","updated":"2021-02-23 11:29:17.000000000","message":"Done","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"b5150dfb9899b3c96a976f66395e89c8eecc7268","unresolved":true,"context_lines":[{"line_number":171,"context_line":"user to invoke the Python CLI from within Ansible."},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"It is an open question whether to support ansible collections"},{"line_number":174,"context_line":"from Asible Galaxy."},{"line_number":175,"context_line":""},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"Assignee(s)"}],"source_content_type":"text/x-rst","patch_set":14,"id":"ca41d5a4_33a1cc07","line":174,"range":{"start_line":174,"start_character":5,"end_line":174,"end_character":11},"updated":"2021-02-10 19:16:12.000000000","message":"nit: typo","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aff37d07d3d1a7722a9f52b40b928591c83c95e3","unresolved":false,"context_lines":[{"line_number":171,"context_line":"user to invoke the Python CLI from within Ansible."},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"It is an open question whether to support ansible collections"},{"line_number":174,"context_line":"from Asible Galaxy."},{"line_number":175,"context_line":""},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"Assignee(s)"}],"source_content_type":"text/x-rst","patch_set":14,"id":"b5503321_693536d7","line":174,"range":{"start_line":174,"start_character":5,"end_line":174,"end_character":11},"in_reply_to":"ca41d5a4_33a1cc07","updated":"2021-02-23 11:29:17.000000000","message":"Done","commit_id":"4ae8967fb13c667aecab58f274f6882f023a69ee"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"26de38dc7c67da427efe7d3c87ce2a113610acb4","unresolved":true,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Reviewing the code base, there are multiple places where repos are"},{"line_number":21,"context_line":"specified e.g. in the release files we are just setting the"},{"line_number":22,"context_line":"configuration that is actually applied by repo setup role and"},{"line_number":23,"context_line":"versions are set. Some of these include:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"* `tripleo repos`_"}],"source_content_type":"text/x-rst","patch_set":19,"id":"4aec8cf7_c2c98d71","line":22,"updated":"2021-03-01 08:28:28.000000000","message":"the `repo setup role`_ \n\nwe have a link to it","commit_id":"a3a3181d5f93aec2656d81d3f47c7876280ee643"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c15dfee62552ac410299d6fb5447c94ae45841ab","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Reviewing the code base, there are multiple places where repos are"},{"line_number":21,"context_line":"specified e.g. in the release files we are just setting the"},{"line_number":22,"context_line":"configuration that is actually applied by repo setup role and"},{"line_number":23,"context_line":"versions are set. Some of these include:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"* `tripleo repos`_"}],"source_content_type":"text/x-rst","patch_set":19,"id":"22b44e8d_431e7d50","line":22,"in_reply_to":"4aec8cf7_c2c98d71","updated":"2021-03-01 08:51:04.000000000","message":"Done","commit_id":"a3a3181d5f93aec2656d81d3f47c7876280ee643"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"26de38dc7c67da427efe7d3c87ce2a113610acb4","unresolved":true,"context_lines":[{"line_number":72,"context_line":"development and testing more consistent, reliable and easier to debug."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"One example review: https://review.opendev.org/c/openstack/tripleo-repos/+/767214"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Integration Points"},{"line_number":78,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":19,"id":"7563d8ca_069f0056","line":75,"updated":"2021-03-01 08:28:28.000000000","message":"maybe we need a sentence here for clarification; the intent is to use the existing tripleo-repos repo for this work and not to create a new repo. It is as yet to be determined if we will add a v2/versioned api or how we will handle the integration with the existing functionality there.","commit_id":"a3a3181d5f93aec2656d81d3f47c7876280ee643"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c15dfee62552ac410299d6fb5447c94ae45841ab","unresolved":false,"context_lines":[{"line_number":72,"context_line":"development and testing more consistent, reliable and easier to debug."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"One example review: https://review.opendev.org/c/openstack/tripleo-repos/+/767214"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Integration Points"},{"line_number":78,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":19,"id":"3e87168f_94c73cd5","line":75,"in_reply_to":"7563d8ca_069f0056","updated":"2021-03-01 08:51:04.000000000","message":"Done","commit_id":"a3a3181d5f93aec2656d81d3f47c7876280ee643"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"26de38dc7c67da427efe7d3c87ce2a113610acb4","unresolved":true,"context_lines":[{"line_number":227,"context_line":"CLI Design"},{"line_number":228,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"Here is the cli design for tripleo-repos to solve the above needs."},{"line_number":231,"context_line":"It covers most of the needs discussed at multiple places."},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"Scenario 1"}],"source_content_type":"text/x-rst","patch_set":19,"id":"90a4a278_79aca679","line":230,"range":{"start_line":230,"start_character":8,"end_line":230,"end_character":11},"updated":"2021-03-01 08:28:28.000000000","message":"\"an abstract sketch of the intended cli design for the new tripleo-repos\"","commit_id":"a3a3181d5f93aec2656d81d3f47c7876280ee643"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c15dfee62552ac410299d6fb5447c94ae45841ab","unresolved":false,"context_lines":[{"line_number":227,"context_line":"CLI Design"},{"line_number":228,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"Here is the cli design for tripleo-repos to solve the above needs."},{"line_number":231,"context_line":"It covers most of the needs discussed at multiple places."},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"Scenario 1"}],"source_content_type":"text/x-rst","patch_set":19,"id":"79eb92c1_83335dce","line":230,"range":{"start_line":230,"start_character":8,"end_line":230,"end_character":11},"in_reply_to":"90a4a278_79aca679","updated":"2021-03-01 08:51:04.000000000","message":"Done","commit_id":"a3a3181d5f93aec2656d81d3f47c7876280ee643"},{"author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"change_message_id":"731d197cce5e68c7bc1b205788aa098b47084521","unresolved":true,"context_lines":[{"line_number":214,"context_line":"Dependencies"},{"line_number":215,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"This work has a dependency on the DLRN API and on yum/dnf."},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"Testing"},{"line_number":220,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":20,"id":"b5b737ad_860297bc","line":217,"range":{"start_line":217,"start_character":34,"end_line":217,"end_character":42},"updated":"2021-03-02 13:53:20.000000000","message":"Nit (please ignore if no more iterations are happening): it might be good to add a link to https://dlrn.readthedocs.io/en/latest/api.html to describe the DLRN API.","commit_id":"af97836793077eaaa547ba43207951e8294dbc2e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d6297b9ea367a280cd856016429db15f1dea5a6d","unresolved":false,"context_lines":[{"line_number":214,"context_line":"Dependencies"},{"line_number":215,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"This work has a dependency on the DLRN API and on yum/dnf."},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"Testing"},{"line_number":220,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":20,"id":"572242fd_1273936c","line":217,"range":{"start_line":217,"start_character":34,"end_line":217,"end_character":42},"in_reply_to":"b5b737ad_860297bc","updated":"2021-03-04 12:56:31.000000000","message":"Done","commit_id":"af97836793077eaaa547ba43207951e8294dbc2e"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"0267b1b48c68c311e24ee23a855bc2d27378b8de","unresolved":false,"context_lines":[{"line_number":66,"context_line":"This proposal puts forward using tripleo-repos as the \u0027source of truth\u0027"},{"line_number":67,"context_line":"for setting repo configurations, modules and supported versions -"},{"line_number":68,"context_line":"including setting the DLRN hashes required to specify exact repo"},{"line_number":69,"context_line":"versions to install."},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Having a single source of truth for repo config, modules, etc will make"},{"line_number":72,"context_line":"development and testing more consistent, reliable and easier to debug."}],"source_content_type":"text/x-rst","patch_set":21,"id":"959c8d5a_62a0cbe1","line":69,"updated":"2021-03-09 17:43:39.000000000","message":"I think this should be clarified for just the upstream development/CI workflows. As previously mentioned we should have stripped down tool that doesn\u0027t take all the various development paths for the regular end user workflows. https://github.com/mwhahaha/rhos-bootstrap (to be moved later) would be the source of truth for the \u0027production\u0027 repos that an end user would be expected to use under normal circumstances.","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"baad52cfb1e08697736e2c6168e71b539d8a62ce","unresolved":false,"context_lines":[{"line_number":66,"context_line":"This proposal puts forward using tripleo-repos as the \u0027source of truth\u0027"},{"line_number":67,"context_line":"for setting repo configurations, modules and supported versions -"},{"line_number":68,"context_line":"including setting the DLRN hashes required to specify exact repo"},{"line_number":69,"context_line":"versions to install."},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Having a single source of truth for repo config, modules, etc will make"},{"line_number":72,"context_line":"development and testing more consistent, reliable and easier to debug."}],"source_content_type":"text/x-rst","patch_set":21,"id":"f238a957_4c24d1ce","line":69,"in_reply_to":"959c8d5a_62a0cbe1","updated":"2021-03-09 21:12:17.000000000","message":"+1 for emphasizing this difference","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d77d028498734983cc6990736447f666e86de688","unresolved":false,"context_lines":[{"line_number":66,"context_line":"This proposal puts forward using tripleo-repos as the \u0027source of truth\u0027"},{"line_number":67,"context_line":"for setting repo configurations, modules and supported versions -"},{"line_number":68,"context_line":"including setting the DLRN hashes required to specify exact repo"},{"line_number":69,"context_line":"versions to install."},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Having a single source of truth for repo config, modules, etc will make"},{"line_number":72,"context_line":"development and testing more consistent, reliable and easier to debug."}],"source_content_type":"text/x-rst","patch_set":21,"id":"f237bc38_e8958cac","line":69,"in_reply_to":"f238a957_4c24d1ce","updated":"2021-03-11 12:00:40.000000000","message":"Done","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"0267b1b48c68c311e24ee23a855bc2d27378b8de","unresolved":false,"context_lines":[{"line_number":76,"context_line":"The intent is to use the existing tripleo-repos repo for this work and"},{"line_number":77,"context_line":"not to create a new repo. It is as yet to be determined if we will add"},{"line_number":78,"context_line":"a v2/versioned api or how we will handle the integration with the"},{"line_number":79,"context_line":"existing functionality there."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Integration Points"},{"line_number":82,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":21,"id":"69f0e26d_d3789f91","line":79,"updated":"2021-03-09 17:43:39.000000000","message":"As previously mentioned, this should probably be clarified that the reuse is to cover the development/CI use cases.","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d77d028498734983cc6990736447f666e86de688","unresolved":false,"context_lines":[{"line_number":76,"context_line":"The intent is to use the existing tripleo-repos repo for this work and"},{"line_number":77,"context_line":"not to create a new repo. It is as yet to be determined if we will add"},{"line_number":78,"context_line":"a v2/versioned api or how we will handle the integration with the"},{"line_number":79,"context_line":"existing functionality there."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Integration Points"},{"line_number":82,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":21,"id":"c319204f_3e039e73","line":79,"in_reply_to":"69f0e26d_d3789f91","updated":"2021-03-11 12:00:40.000000000","message":"Covered above","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"59f82ac49172ebafa1c2e5f2dbc69d8c6b2d8a48","unresolved":true,"context_lines":[{"line_number":248,"context_line":""},{"line_number":249,"context_line":".. code-block:: bash"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"    tripleo-repos configure-repo delorean.repo --repo-type dlrn \\"},{"line_number":252,"context_line":"                    --repo-hash  6ab585bbd38fa96125006c5ddcdbe6ad \\"},{"line_number":253,"context_line":"                    --promotion-pipeline current-tripleo \\"},{"line_number":254,"context_line":"                    --install-packages python-tripleoclient \\"}],"source_content_type":"text/x-rst","patch_set":21,"id":"062d709c_8d35d03d","line":251,"range":{"start_line":251,"start_character":47,"end_line":251,"end_character":63},"updated":"2021-03-09 13:14:18.000000000","message":"what is the intention behind that one please \n\nso in this example you\u0027re saying configure delorean.repo from current-tripleo and install tht/python-tripleoclient.\n\nso what does the --repo-type dlrn do","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"baad52cfb1e08697736e2c6168e71b539d8a62ce","unresolved":true,"context_lines":[{"line_number":249,"context_line":".. code-block:: bash"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"    tripleo-repos configure-repo delorean.repo --repo-type dlrn \\"},{"line_number":252,"context_line":"                    --repo-hash  6ab585bbd38fa96125006c5ddcdbe6ad \\"},{"line_number":253,"context_line":"                    --promotion-pipeline current-tripleo \\"},{"line_number":254,"context_line":"                    --install-packages python-tripleoclient \\"},{"line_number":255,"context_line":"                    --include_packages tripleo-heat-templates \\"}],"source_content_type":"text/x-rst","patch_set":21,"id":"4ce248f2_4a9cb31f","line":252,"range":{"start_line":252,"start_character":20,"end_line":252,"end_character":65},"updated":"2021-03-09 21:12:17.000000000","message":"I\u0027d probably set: --dlrn-hash 6ab585bbd38fa96125006c5... because only DLRN repos have hashes, afaiu.","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"ebf6ec83de2dbf4f3c267ac668a444216a4625c6","unresolved":true,"context_lines":[{"line_number":250,"context_line":""},{"line_number":251,"context_line":"    tripleo-repos configure-repo delorean.repo --repo-type dlrn \\"},{"line_number":252,"context_line":"                    --repo-hash  6ab585bbd38fa96125006c5ddcdbe6ad \\"},{"line_number":253,"context_line":"                    --promotion-pipeline current-tripleo \\"},{"line_number":254,"context_line":"                    --install-packages python-tripleoclient \\"},{"line_number":255,"context_line":"                    --include_packages tripleo-heat-templates \\"},{"line_number":256,"context_line":"                    --proxy-url https://x.y.z \\"}],"source_content_type":"text/x-rst","patch_set":21,"id":"b5e8988c_6c1438d6","line":253,"updated":"2021-03-08 13:34:01.000000000","message":"What does mean specifying both \"current-tripleo\" and repo-hash?","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3d44080773fc724a09de2cad31b4907aad44f780","unresolved":true,"context_lines":[{"line_number":252,"context_line":"                    --repo-hash  6ab585bbd38fa96125006c5ddcdbe6ad \\"},{"line_number":253,"context_line":"                    --promotion-pipeline current-tripleo \\"},{"line_number":254,"context_line":"                    --install-packages python-tripleoclient \\"},{"line_number":255,"context_line":"                    --include_packages tripleo-heat-templates \\"},{"line_number":256,"context_line":"                    --proxy-url https://x.y.z \\"},{"line_number":257,"context_line":"                release master"},{"line_number":258,"context_line":""}],"source_content_type":"text/x-rst","patch_set":21,"id":"594cff20_d500e615","line":255,"range":{"start_line":255,"start_character":22,"end_line":255,"end_character":38},"updated":"2021-03-09 13:11:22.000000000","message":"is there some intended difference between install-packages and include_packages or is that just typo","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"ebf6ec83de2dbf4f3c267ac668a444216a4625c6","unresolved":true,"context_lines":[{"line_number":257,"context_line":"                release master"},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"    Detecting Distro and Version: CentOS and 8.0"},{"line_number":260,"context_line":"    Relase: Master"},{"line_number":261,"context_line":"    Found promotion pipeline: current-tripleo"},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"    Checking the connectivity for proxy url https://x.y.z REACHABLE"}],"source_content_type":"text/x-rst","patch_set":21,"id":"9fa2987a_711b8a86","line":260,"range":{"start_line":260,"start_character":4,"end_line":260,"end_character":10},"updated":"2021-03-08 13:34:01.000000000","message":"Release","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"ebf6ec83de2dbf4f3c267ac668a444216a4625c6","unresolved":true,"context_lines":[{"line_number":261,"context_line":"    Found promotion pipeline: current-tripleo"},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"    Checking the connectivity for proxy url https://x.y.z REACHABLE"},{"line_number":264,"context_line":"    Getting dlrn md5 hash from https://x.y.z/centos8-master/tripleo-ci-testing/delorean.repo.md5"},{"line_number":265,"context_line":"    received: 6ab585bbd38fa96125006c5ddcdbe6ad"},{"line_number":266,"context_line":"    Saving dlrn md5 hash in /tmp/dlrn_md5_hash_tripleo_ci_testing.txt"},{"line_number":267,"context_line":"    setting env var DLRN_MD5_HASH 6ab585bbd38fa96125006c5ddcdbe6ad"}],"source_content_type":"text/x-rst","patch_set":21,"id":"0061ec89_102979c6","line":264,"range":{"start_line":264,"start_character":60,"end_line":264,"end_character":78},"updated":"2021-03-08 13:34:01.000000000","message":"why tripleo-ci-testing if in cli \"current-tripleo\"?","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d77d028498734983cc6990736447f666e86de688","unresolved":true,"context_lines":[{"line_number":261,"context_line":"    Found promotion pipeline: current-tripleo"},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"    Checking the connectivity for proxy url https://x.y.z REACHABLE"},{"line_number":264,"context_line":"    Getting dlrn md5 hash from https://x.y.z/centos8-master/tripleo-ci-testing/delorean.repo.md5"},{"line_number":265,"context_line":"    received: 6ab585bbd38fa96125006c5ddcdbe6ad"},{"line_number":266,"context_line":"    Saving dlrn md5 hash in /tmp/dlrn_md5_hash_tripleo_ci_testing.txt"},{"line_number":267,"context_line":"    setting env var DLRN_MD5_HASH 6ab585bbd38fa96125006c5ddcdbe6ad"}],"source_content_type":"text/x-rst","patch_set":21,"id":"357ccdd8_98f2cb73","line":264,"range":{"start_line":264,"start_character":60,"end_line":264,"end_character":78},"in_reply_to":"0061ec89_102979c6","updated":"2021-03-11 12:00:40.000000000","message":"Updating it in a new patch","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"ebf6ec83de2dbf4f3c267ac668a444216a4625c6","unresolved":true,"context_lines":[{"line_number":281,"context_line":""},{"line_number":282,"context_line":".. code-block:: bash"},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"    tripleo-repos module container-tools --enable rhel8 \\"},{"line_number":285,"context_line":"                                    --disable 2.0 \\"},{"line_number":286,"context_line":"                                    --install-packages podman"},{"line_number":287,"context_line":""}],"source_content_type":"text/x-rst","patch_set":21,"id":"51a9e92e_74c88146","line":284,"updated":"2021-03-08 13:34:01.000000000","message":"probably worth to wrap name of module in separate variable like:\n  tripleo-repos module --name container-tools --enable ... --disable ...","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d77d028498734983cc6990736447f666e86de688","unresolved":false,"context_lines":[{"line_number":281,"context_line":""},{"line_number":282,"context_line":".. code-block:: bash"},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"    tripleo-repos module container-tools --enable rhel8 \\"},{"line_number":285,"context_line":"                                    --disable 2.0 \\"},{"line_number":286,"context_line":"                                    --install-packages podman"},{"line_number":287,"context_line":""}],"source_content_type":"text/x-rst","patch_set":21,"id":"615a2988_a21fd578","line":284,"in_reply_to":"51a9e92e_74c88146","updated":"2021-03-11 12:00:40.000000000","message":"Done","commit_id":"c7f909d6414af43b688187df4a16de560568610d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c1442936d83d319fb720a9e982da212f4895ed51","unresolved":true,"context_lines":[{"line_number":29,"context_line":"* `tripleo-ansible`_"},{"line_number":30,"context_line":"* `rdo config`_ (example)"},{"line_number":31,"context_line":"* `tripleo-heat-templates`_"},{"line_number":32,"context_line":"* `tripleo-operator-ansible`_ (not sure this example is valid any more)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":".. _`tripleo repos`: https://opendev.org/openstack/tripleo-repos"},{"line_number":35,"context_line":".. _`repo setup role`: https://opendev.org/openstack/tripleo-quickstart/src/branch/master/roles/repo-setup"}],"source_content_type":"text/x-rst","patch_set":25,"id":"abe82290_d9ae1fb1","line":32,"range":{"start_line":32,"start_character":31,"end_line":32,"end_character":34},"updated":"2021-05-20 15:42:44.000000000","message":"removing this comment in brackets","commit_id":"da150203de081eee166cb9401910cd646c575992"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c1442936d83d319fb720a9e982da212f4895ed51","unresolved":true,"context_lines":[{"line_number":35,"context_line":".. _`repo setup role`: https://opendev.org/openstack/tripleo-quickstart/src/branch/master/roles/repo-setup"},{"line_number":36,"context_line":".. _`release config files`: https://opendev.org/openstack/tripleo-quickstart/src/branch/master/config/release/tripleo-ci/CentOS-8"},{"line_number":37,"context_line":".. _`container tooling (base tcib file)`: https://opendev.org/openstack/tripleo-common/src/branch/master/container-images/tcib/base/base.yaml#L59"},{"line_number":38,"context_line":".. _`tripleo-ansible`: https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_podman/vars/redhat-8.2.yml"},{"line_number":39,"context_line":".. _`rdo config`: https://review.rdoproject.org/r/31439"},{"line_number":40,"context_line":".. _`tripleo-heat-templates`: https://opendev.org/openstack/tripleo-heat-templates/src/commit/125f45820255efe370af1024080bafc695892faa/environments/lifecycle/undercloud-upgrade-prepare.yaml"},{"line_number":41,"context_line":".. _`tripleo-operator-ansible`: https://opendev.org/openstack/tripleo-operator-ansible/src/commit/14a601a47be217386df83512fae3a2e5aa5444a3/roles/tripleo_container_image_build/molecule/default/converge.yml#L172"}],"source_content_type":"text/x-rst","patch_set":25,"id":"a72b9fb7_7231c6a8","line":38,"range":{"start_line":38,"start_character":137,"end_line":38,"end_character":143},"updated":"2021-05-20 15:42:44.000000000","message":"that one is now 404 so updating to use permalink (pointing to a specific commit before it was deleted)","commit_id":"da150203de081eee166cb9401910cd646c575992"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c1442936d83d319fb720a9e982da212f4895ed51","unresolved":true,"context_lines":[{"line_number":71,"context_line":"Having a single source of truth for repo config, modules, etc will make"},{"line_number":72,"context_line":"development and testing more consistent, reliable and easier to debug."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"One example review: https://review.opendev.org/c/openstack/tripleo-repos/+/767214"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"The intent is to use the existing tripleo-repos repo for this work and"},{"line_number":77,"context_line":"not to create a new repo. It is as yet to be determined if we will add"}],"source_content_type":"text/x-rst","patch_set":25,"id":"f8cbf0b2_5d355c6a","line":74,"range":{"start_line":74,"start_character":6,"end_line":74,"end_character":16},"updated":"2021-05-20 15:42:44.000000000","message":"removing this -  without more info/context i don\u0027t think it adds anything here","commit_id":"da150203de081eee166cb9401910cd646c575992"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c1442936d83d319fb720a9e982da212f4895ed51","unresolved":true,"context_lines":[{"line_number":172,"context_line":"All options required to be passed to tripleo-repos should be in the"},{"line_number":173,"context_line":"release file."},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"The new tripleo-repos will inherit the re-usable python modules(repo"},{"line_number":176,"context_line":"configuration and layout using DNF, yaml version schema) from"},{"line_number":177,"context_line":"`rhos-bootstrap`_ python project. The tripleo developer and upstream CI"},{"line_number":178,"context_line":"specific code will leave in tripleo-repos."},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Later on,  rhos-bootstrap tool would be the source of truth for the"},{"line_number":181,"context_line":"\u0027production\u0027 repos that an end user would be expected to use"},{"line_number":182,"context_line":"under normal circumstances."},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":185,"context_line":"Unit tests will be added on this branch to test the new CLI directly."}],"source_content_type":"text/x-rst","patch_set":25,"id":"8718dec1_d6cbdf49","line":182,"range":{"start_line":175,"start_character":4,"end_line":182,"end_character":15},"updated":"2021-05-20 15:42:44.000000000","message":"removing this bit too... will try and some words about the personas that we want to support.","commit_id":"da150203de081eee166cb9401910cd646c575992"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c1442936d83d319fb720a9e982da212f4895ed51","unresolved":true,"context_lines":[{"line_number":245,"context_line":""},{"line_number":246,"context_line":"It covers most of the needs discussed at multiple places."},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"Scenario 1"},{"line_number":249,"context_line":"----------"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"Goal is to construct following repo with correct hash for a integration"}],"source_content_type":"text/x-rst","patch_set":25,"id":"981014d2_943cf9af","line":248,"range":{"start_line":248,"start_character":0,"end_line":248,"end_character":8},"updated":"2021-05-20 15:42:44.000000000","message":"I am tempted to remove all of these... they are a bit outdated now and I think we just shouldn\u0027t give this much detail in the spec.\n\nmaybe keep one of these but not all of them","commit_id":"da150203de081eee166cb9401910cd646c575992"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"ba9b1509ce3e3e2d19f792d145e30f9d923f94c1","unresolved":true,"context_lines":[{"line_number":245,"context_line":""},{"line_number":246,"context_line":"It covers most of the needs discussed at multiple places."},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"Scenario 1"},{"line_number":249,"context_line":"----------"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"Goal is to construct following repo with correct hash for a integration"}],"source_content_type":"text/x-rst","patch_set":25,"id":"06d72d2f_dce4aade","line":248,"range":{"start_line":248,"start_character":0,"end_line":248,"end_character":8},"in_reply_to":"981014d2_943cf9af","updated":"2021-05-25 13:13:08.000000000","message":"discussing withi rlandy now on a call\n\nlets keep the 2/3 that show  the basic functionality like add repos and add modules\nalso include example of tripleo-get-hash cli\n\ndon\u0027t need all the parameters in this much detail","commit_id":"da150203de081eee166cb9401910cd646c575992"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c1442936d83d319fb720a9e982da212f4895ed51","unresolved":true,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":".. code-block:: bash"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"    tripleo-repos get-hash --help"},{"line_number":257,"context_line":"                  --name - Name of the repo (mandatory/default: delorean.repo)"},{"line_number":258,"context_line":"                  --release - Name of the release (mandatory/default: master)"},{"line_number":259,"context_line":"                  --distro - Name of the distribution (optional/default: centos)"}],"source_content_type":"text/x-rst","patch_set":25,"id":"28d45d4b_e9f58746","line":256,"updated":"2021-05-20 15:42:44.000000000","message":"cli examples need updating","commit_id":"da150203de081eee166cb9401910cd646c575992"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":12,"context_line":"to install and configure non-base OS repos for TripleO - including"},{"line_number":13,"context_line":"setting the required DLRN hashes."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"https://blueprints.launchpad.net/tripleo/+spec/tripleo-repos-single-source"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Problem Description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":26,"id":"b2117de2_125cac7d","line":15,"updated":"2021-05-25 12:58:43.000000000","message":"link does not exist","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Reviewing the code base, there are multiple places where repos are"},{"line_number":21,"context_line":"specified e.g. in the release files we are just setting the"},{"line_number":22,"context_line":"configuration that is actually applied by `repo setup role`_ and"},{"line_number":23,"context_line":"versions are set. Some of these include:"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-rst","patch_set":26,"id":"388910d2_a5663962","line":21,"updated":"2021-05-25 12:58:43.000000000","message":"for example:","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"70961ecb1f261b44f67e3d00fc106302cb96ea95","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Reviewing the code base, there are multiple places where repos are"},{"line_number":21,"context_line":"specified e.g. in the release files we are just setting the"},{"line_number":22,"context_line":"configuration that is actually applied by `repo setup role`_ and"},{"line_number":23,"context_line":"versions are set. Some of these include:"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-rst","patch_set":26,"id":"34186f54_4a8908fd","line":21,"in_reply_to":"388910d2_a5663962","updated":"2021-05-25 16:32:15.000000000","message":"Done","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":20,"context_line":"Reviewing the code base, there are multiple places where repos are"},{"line_number":21,"context_line":"specified e.g. in the release files we are just setting the"},{"line_number":22,"context_line":"configuration that is actually applied by `repo setup role`_ and"},{"line_number":23,"context_line":"versions are set. Some of these include:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"* `tripleo repos`_"},{"line_number":26,"context_line":"* `repo setup role`_"}],"source_content_type":"text/x-rst","patch_set":26,"id":"c4762ff4_5e2def20","line":23,"updated":"2021-05-25 12:58:43.000000000","message":"This is confusing. Are we trying to say that that repo_setup role sets the versions of all these entities OR that there are other places with definitions?","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"70961ecb1f261b44f67e3d00fc106302cb96ea95","unresolved":false,"context_lines":[{"line_number":20,"context_line":"Reviewing the code base, there are multiple places where repos are"},{"line_number":21,"context_line":"specified e.g. in the release files we are just setting the"},{"line_number":22,"context_line":"configuration that is actually applied by `repo setup role`_ and"},{"line_number":23,"context_line":"versions are set. Some of these include:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"* `tripleo repos`_"},{"line_number":26,"context_line":"* `repo setup role`_"}],"source_content_type":"text/x-rst","patch_set":26,"id":"e161fb28_e8e3b73f","line":23,"in_reply_to":"c4762ff4_5e2def20","updated":"2021-05-25 16:32:15.000000000","message":"Done","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":68,"context_line":"including setting the DLRN hashes required to specify exact repo"},{"line_number":69,"context_line":"versions to install for upstream development/CI workflows."},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Having a single source of truth for repo config, modules, etc will make"},{"line_number":72,"context_line":"development and testing more consistent, reliable and easier to debug."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"The intent is to use the existing tripleo-repos repo for this work and"}],"source_content_type":"text/x-rst","patch_set":26,"id":"854111fd_145c7e70","line":71,"updated":"2021-05-25 12:58:43.000000000","message":"etc.","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"70961ecb1f261b44f67e3d00fc106302cb96ea95","unresolved":false,"context_lines":[{"line_number":68,"context_line":"including setting the DLRN hashes required to specify exact repo"},{"line_number":69,"context_line":"versions to install for upstream development/CI workflows."},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Having a single source of truth for repo config, modules, etc will make"},{"line_number":72,"context_line":"development and testing more consistent, reliable and easier to debug."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"The intent is to use the existing tripleo-repos repo for this work and"}],"source_content_type":"text/x-rst","patch_set":26,"id":"322dcc60_7f39fd1c","line":71,"in_reply_to":"854111fd_145c7e70","updated":"2021-05-25 16:32:15.000000000","message":"Done","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"We aim to modularize the design and implementation of the proposed tripleo-repos"},{"line_number":80,"context_line":"work. Two sub systems in particular have been identified that can be"},{"line_number":81,"context_line":"implemented independently of and ultimately to be consumed by tripleo-repos;"},{"line_number":82,"context_line":"the resolution of delorean build hashes from known tags (i.e. resolving"},{"line_number":83,"context_line":"\u0027current-tripleo\u0027 to a particular DLRN build ID) and the configuration of dnf"},{"line_number":84,"context_line":"repos and modules will be implemented as independent python modules, with"}],"source_content_type":"text/x-rst","patch_set":26,"id":"b944ec4b_db8a8467","line":81,"updated":"2021-05-25 12:58:43.000000000","message":"independently of, and... consumed by,","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"70961ecb1f261b44f67e3d00fc106302cb96ea95","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"We aim to modularize the design and implementation of the proposed tripleo-repos"},{"line_number":80,"context_line":"work. Two sub systems in particular have been identified that can be"},{"line_number":81,"context_line":"implemented independently of and ultimately to be consumed by tripleo-repos;"},{"line_number":82,"context_line":"the resolution of delorean build hashes from known tags (i.e. resolving"},{"line_number":83,"context_line":"\u0027current-tripleo\u0027 to a particular DLRN build ID) and the configuration of dnf"},{"line_number":84,"context_line":"repos and modules will be implemented as independent python modules, with"}],"source_content_type":"text/x-rst","patch_set":26,"id":"a9e649f6_3dd78db7","line":81,"in_reply_to":"b944ec4b_db8a8467","updated":"2021-05-25 16:32:15.000000000","message":"Done","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":82,"context_line":"the resolution of delorean build hashes from known tags (i.e. resolving"},{"line_number":83,"context_line":"\u0027current-tripleo\u0027 to a particular DLRN build ID) and the configuration of dnf"},{"line_number":84,"context_line":"repos and modules will be implemented as independent python modules, with"},{"line_number":85,"context_line":"their own unit tests, cli, ansible modules etc."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"Integration Points"},{"line_number":88,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":26,"id":"3eed9bb9_9aec1df1","line":85,"updated":"2021-05-25 12:58:43.000000000","message":"clis?","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"70961ecb1f261b44f67e3d00fc106302cb96ea95","unresolved":false,"context_lines":[{"line_number":82,"context_line":"the resolution of delorean build hashes from known tags (i.e. resolving"},{"line_number":83,"context_line":"\u0027current-tripleo\u0027 to a particular DLRN build ID) and the configuration of dnf"},{"line_number":84,"context_line":"repos and modules will be implemented as independent python modules, with"},{"line_number":85,"context_line":"their own unit tests, cli, ansible modules etc."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"Integration Points"},{"line_number":88,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":26,"id":"528f6fb6_78557f16","line":85,"in_reply_to":"3eed9bb9_9aec1df1","updated":"2021-05-25 16:32:15.000000000","message":"Done","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":153,"context_line":"is done correctly and consistently."},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"Other Deployer Impact"},{"line_number":156,"context_line":"---------------------"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Developer Impact"}],"source_content_type":"text/x-rst","patch_set":26,"id":"e14b5812_00ac7777","line":156,"updated":"2021-05-25 12:58:43.000000000","message":"Remove this?","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":172,"context_line":"  including creating repos from templates and files"},{"line_number":173,"context_line":"* Perform proxy handling such as is done in the release files"},{"line_number":174,"context_line":"  (mirrors, using rdoproject for DLRN repos)"},{"line_number":175,"context_line":"* Get and transform human-readable dlrn hashes - to be implemented as an"},{"line_number":176,"context_line":"  independent module."},{"line_number":177,"context_line":"* Support setting yum modules such as container-tools - to be implemented"},{"line_number":178,"context_line":"  as an independent module."}],"source_content_type":"text/x-rst","patch_set":26,"id":"9eb7616f_4eda747d","line":175,"updated":"2021-05-25 12:58:43.000000000","message":"DLRN","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"70961ecb1f261b44f67e3d00fc106302cb96ea95","unresolved":false,"context_lines":[{"line_number":172,"context_line":"  including creating repos from templates and files"},{"line_number":173,"context_line":"* Perform proxy handling such as is done in the release files"},{"line_number":174,"context_line":"  (mirrors, using rdoproject for DLRN repos)"},{"line_number":175,"context_line":"* Get and transform human-readable dlrn hashes - to be implemented as an"},{"line_number":176,"context_line":"  independent module."},{"line_number":177,"context_line":"* Support setting yum modules such as container-tools - to be implemented"},{"line_number":178,"context_line":"  as an independent module."}],"source_content_type":"text/x-rst","patch_set":26,"id":"4879d540_1aa0105f","line":175,"in_reply_to":"9eb7616f_4eda747d","updated":"2021-05-25 16:32:15.000000000","message":"Done","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"3f22849724302bf9add4598f3402cdde18ff7273","unresolved":true,"context_lines":[{"line_number":176,"context_line":"  independent module."},{"line_number":177,"context_line":"* Support setting yum modules such as container-tools - to be implemented"},{"line_number":178,"context_line":"  as an independent module."},{"line_number":179,"context_line":"* Support enabling and disabling repos"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"The repo-setup role shall remain but it will invoke tripleo-repos."},{"line_number":182,"context_line":"All options required to be passed to tripleo-repos should be in the"}],"source_content_type":"text/x-rst","patch_set":26,"id":"b69158b0_41401cb7","line":179,"updated":"2021-05-25 12:58:43.000000000","message":"Setting priorities?","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"70961ecb1f261b44f67e3d00fc106302cb96ea95","unresolved":false,"context_lines":[{"line_number":176,"context_line":"  independent module."},{"line_number":177,"context_line":"* Support setting yum modules such as container-tools - to be implemented"},{"line_number":178,"context_line":"  as an independent module."},{"line_number":179,"context_line":"* Support enabling and disabling repos"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"The repo-setup role shall remain but it will invoke tripleo-repos."},{"line_number":182,"context_line":"All options required to be passed to tripleo-repos should be in the"}],"source_content_type":"text/x-rst","patch_set":26,"id":"327a5c56_a5cb1e5e","line":179,"in_reply_to":"b69158b0_41401cb7","updated":"2021-05-25 16:32:15.000000000","message":"Done","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"70961ecb1f261b44f67e3d00fc106302cb96ea95","unresolved":true,"context_lines":[{"line_number":294,"context_line":"Scenario 2"},{"line_number":295,"context_line":"----------"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"Goal is to construct any type of repo."},{"line_number":298,"context_line":""},{"line_number":299,"context_line":".. code-block::"},{"line_number":300,"context_line":""}],"source_content_type":"text/x-rst","patch_set":26,"id":"d902d7c6_ac4d5eff","line":297,"updated":"2021-05-25 16:32:15.000000000","message":"any type of yum repo","commit_id":"1e45243bd7ef906019cee157b8af6ddce67d365e"},{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"8493df8ed0def12d463294702fc2e18f2da93e51","unresolved":true,"context_lines":[{"line_number":12,"context_line":"to install and configure non-base OS repos for TripleO - including"},{"line_number":13,"context_line":"setting the required DLRN hashes."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"https://blueprints.launchpad.net/tripleo/+spec/tripleo-repos-single-source"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Problem Description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":28,"id":"3eb1d14a_b94eeef2","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":74},"updated":"2021-05-25 19:15:56.000000000","message":"? https://blueprints.launchpad.net/tripleo/?searchtext\u003dtripleo-repos-single-source  \nThat blueprint does not exist","commit_id":"6f44696232f26653b4dc8e3c1349579391fe6b7a"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"617a46df5f66aa021f9f77408a448d6c408fd95c","unresolved":true,"context_lines":[{"line_number":12,"context_line":"to install and configure non-base OS repos for TripleO - including"},{"line_number":13,"context_line":"setting the required DLRN hashes."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"https://blueprints.launchpad.net/tripleo/+spec/tripleo-repos-single-source"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Problem Description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":28,"id":"bc9b3b62_e8e0781c","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":74},"in_reply_to":"3eb1d14a_b94eeef2","updated":"2021-05-25 19:32:34.000000000","message":"ack - we were waiting until this review ready","commit_id":"6f44696232f26653b4dc8e3c1349579391fe6b7a"},{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"8493df8ed0def12d463294702fc2e18f2da93e51","unresolved":true,"context_lines":[{"line_number":184,"context_line":""},{"line_number":185,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":186,"context_line":"Unit tests will be added on this branch to test the new CLI directly."},{"line_number":187,"context_line":"CI would be flipped to run in the new branch when approved by DF and CI teams."},{"line_number":188,"context_line":"All current unit tests should pass with the new code."},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"An Ansible module will be added to call the tripleo-repos"}],"source_content_type":"text/x-rst","patch_set":28,"id":"a4b4700e_545bc6bb","line":187,"range":{"start_line":187,"start_character":58,"end_line":187,"end_character":75},"updated":"2021-05-25 19:15:56.000000000","message":"approved by TripleO is good enough.. Don\u0027t use DF.\nhttps://specs.openstack.org/openstack/tripleo-specs/specs/policy/squads.html","commit_id":"6f44696232f26653b4dc8e3c1349579391fe6b7a"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"617a46df5f66aa021f9f77408a448d6c408fd95c","unresolved":false,"context_lines":[{"line_number":184,"context_line":""},{"line_number":185,"context_line":"Work done on the new project branch will offer a different version of CLI, v2."},{"line_number":186,"context_line":"Unit tests will be added on this branch to test the new CLI directly."},{"line_number":187,"context_line":"CI would be flipped to run in the new branch when approved by DF and CI teams."},{"line_number":188,"context_line":"All current unit tests should pass with the new code."},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"An Ansible module will be added to call the tripleo-repos"}],"source_content_type":"text/x-rst","patch_set":28,"id":"991efe2e_9e914cb0","line":187,"range":{"start_line":187,"start_character":58,"end_line":187,"end_character":75},"in_reply_to":"a4b4700e_545bc6bb","updated":"2021-05-25 19:32:34.000000000","message":"Done","commit_id":"6f44696232f26653b4dc8e3c1349579391fe6b7a"},{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"8493df8ed0def12d463294702fc2e18f2da93e51","unresolved":true,"context_lines":[{"line_number":190,"context_line":"An Ansible module will be added to call the tripleo-repos"},{"line_number":191,"context_line":"options from Ansible directly without requiring the end"},{"line_number":192,"context_line":"user to invoke the Python CLI from within Ansible."},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"The aim is for tripleo-repos to be the single source for all repo related"},{"line_number":195,"context_line":"configuration. In particular the goal is to serve the following 3 personas:"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"* Upstream/OpenStack CI jobs"},{"line_number":198,"context_line":"* Downstream/OSP/RHEL jobs"},{"line_number":199,"context_line":"* Customer installations"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"The configuration required to serve each of these use cases is slightly"},{"line_number":202,"context_line":"different. In upstream CI jobs we need to configure the latest current-tripleo"},{"line_number":203,"context_line":"promoted content repos. In downstream/OSP jobs we need to use rhos-release"}],"source_content_type":"text/x-rst","patch_set":28,"id":"89111dbb_949a3b6a","line":200,"range":{"start_line":193,"start_character":0,"end_line":200,"end_character":0},"updated":"2021-05-25 19:15:56.000000000","message":"nice","commit_id":"6f44696232f26653b4dc8e3c1349579391fe6b7a"},{"author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"change_message_id":"8493df8ed0def12d463294702fc2e18f2da93e51","unresolved":true,"context_lines":[{"line_number":336,"context_line":"References to old sources of settings such as tripleo-ansible,"},{"line_number":337,"context_line":"release files in tripleo-quickstart and the repo-setup role"},{"line_number":338,"context_line":"will have to be removed and replaced to point to the new"},{"line_number":339,"context_line":"workflow."}],"source_content_type":"text/x-rst","patch_set":28,"id":"6c25ec3a_86f70316","line":339,"updated":"2021-05-25 19:15:56.000000000","message":"Very nice.. thanks for pushing this forward.","commit_id":"6f44696232f26653b4dc8e3c1349579391fe6b7a"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"a3050da5ffdac94535c3f74fa6712a4a26a16675","unresolved":true,"context_lines":[{"line_number":1,"context_line":".."},{"line_number":2,"context_line":" This work is licensed under a Creative Commons Attribution 3.0 Unported"},{"line_number":3,"context_line":" License."},{"line_number":4,"context_line":""}],"source_content_type":"text/x-rst","patch_set":29,"id":"e8c93253_806973bc","line":1,"updated":"2021-05-25 19:45:11.000000000","message":"should be moved to xena as well","commit_id":"13283e5b2798b3da99a0f31d62373fad48154c57"}],"specs/xena/tripleo-repos-single-source.rst":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"621206a0df4b95c9fe8a1154078227c9df37f17c","unresolved":true,"context_lines":[{"line_number":196,"context_line":""},{"line_number":197,"context_line":"* Upstream/OpenStack CI jobs"},{"line_number":198,"context_line":"* Downstream/OSP/RHEL jobs"},{"line_number":199,"context_line":"* Customer installations"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"The configuration required to serve each of these use cases is slightly"},{"line_number":202,"context_line":"different. In upstream CI jobs we need to configure the latest current-tripleo"}],"source_content_type":"text/x-rst","patch_set":31,"id":"763fae5f_7613229e","line":199,"range":{"start_line":199,"start_character":2,"end_line":199,"end_character":10},"updated":"2021-05-27 07:31:09.000000000","message":"lets call this one \"Production deployment\" i think it sounds better than \u0027customer\u0027 (even though I wrote that ;) )","commit_id":"d7b47eb28eca211ffd5ab9951ea6766619a8b4ed"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"621206a0df4b95c9fe8a1154078227c9df37f17c","unresolved":true,"context_lines":[{"line_number":304,"context_line":""},{"line_number":305,"context_line":"For this scenario:"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"* Specify"},{"line_number":308,"context_line":"* module name"},{"line_number":309,"context_line":"* which version to disable"},{"line_number":310,"context_line":"* which version to enable"}],"source_content_type":"text/x-rst","patch_set":31,"id":"ef9bcfb9_58d203cf","line":307,"range":{"start_line":307,"start_character":2,"end_line":307,"end_character":9},"updated":"2021-05-27 07:31:09.000000000","message":"?","commit_id":"d7b47eb28eca211ffd5ab9951ea6766619a8b4ed"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"621206a0df4b95c9fe8a1154078227c9df37f17c","unresolved":true,"context_lines":[{"line_number":316,"context_line":""},{"line_number":317,"context_line":"The goal is to enable or disable some repos,"},{"line_number":318,"context_line":"remove any associated repo files no longer needed,"},{"line_number":319,"context_line":"and then perform a system update."},{"line_number":320,"context_line":""},{"line_number":321,"context_line":"For this scenario:"},{"line_number":322,"context_line":""}],"source_content_type":"text/x-rst","patch_set":31,"id":"cbfc542e_c61a7664","line":319,"range":{"start_line":319,"start_character":19,"end_line":319,"end_character":33},"updated":"2021-05-27 07:31:09.000000000","message":"not sure we want that one? I mean is it going to be part of the tripleo-repos functionality? I think it should not do this, i mean, lets use it to lay down the config repos modules and all, but then the operator can do the yum update/upgrade as a subsequent step outside tripleo-repos.","commit_id":"d7b47eb28eca211ffd5ab9951ea6766619a8b4ed"}]}