)]}'
{".zuul.yaml":[{"author":{"_account_id":21961,"name":"Lance Albertson","email":"lance@osuosl.org","username":"ramereth"},"change_message_id":"d93df67601b33ed225275ed3571cbd1f3ce5a89a","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        # - openstack-chef-integration-ubuntu"},{"line_number":72,"context_line":"        - openstack-cinc-integration-ubuntu"},{"line_number":73,"context_line":"        - openstack-cinc-integration-centos"},{"line_number":74,"context_line":"        # - openstack-chef-integration-centos"},{"line_number":75,"context_line":"    gate:"},{"line_number":76,"context_line":"      queue: openstack-chef"},{"line_number":77,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"ff570b3c_5d72084b","line":74,"updated":"2020-06-09 17:50:44.000000000","message":"I\u0027m assuming we\u0027ll enable these other jobs too? Any reason we can\u0027t do that now?","commit_id":"d41ba6c75df2ed74712b50b963d93044c80043e0"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"dceacbb0e64ddef8dffc41e880ab5a839e82f295","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        # - openstack-chef-integration-ubuntu"},{"line_number":72,"context_line":"        - openstack-cinc-integration-ubuntu"},{"line_number":73,"context_line":"        - openstack-cinc-integration-centos"},{"line_number":74,"context_line":"        # - openstack-chef-integration-centos"},{"line_number":75,"context_line":"    gate:"},{"line_number":76,"context_line":"      queue: openstack-chef"},{"line_number":77,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"ff570b3c_0765d245","line":74,"in_reply_to":"ff570b3c_5d72084b","updated":"2020-06-10 07:54:10.000000000","message":"I was just waiting for the testing to be successful.","commit_id":"d41ba6c75df2ed74712b50b963d93044c80043e0"}],"playbooks/pre-integration.yaml":[{"author":{"_account_id":21961,"name":"Lance Albertson","email":"lance@osuosl.org","username":"ramereth"},"change_message_id":"2360db4b433b263a18f9dcabbb2b148b0b8f8643","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  tasks:"},{"line_number":9,"context_line":"  - name: Install chef package"},{"line_number":10,"context_line":"    shell:"},{"line_number":11,"context_line":"      cmd: wget http://downloads.cinc.sh/files/stable/cinc/ubuntu/18.04/cinc_15.3.14-1_amd64.deb \u0026\u0026 dpkg -i cinc_15.3.14-1_amd64.deb"},{"line_number":12,"context_line":"      chdir: /tmp"},{"line_number":13,"context_line":"    become: yes"},{"line_number":14,"context_line":"  - name: Install berkshelf"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_bf0060ad","line":11,"updated":"2019-10-01 20:59:38.000000000","message":"We now have an omnitruck instance up, so you can create a cinc-install.sh by pulling down https://omnitruck.cinc.sh/install.sh","commit_id":"29231d5cfd1892aefaa54545ed949406eaeffffc"},{"author":{"_account_id":21961,"name":"Lance Albertson","email":"lance@osuosl.org","username":"ramereth"},"change_message_id":"97cbbc15be32ed08afed13c869145397c8c2c5b5","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  tasks:"},{"line_number":9,"context_line":"  - name: Install chef package"},{"line_number":10,"context_line":"    shell:"},{"line_number":11,"context_line":"      cmd: wget http://downloads.cinc.sh/files/stable/cinc/15.4.45/ubuntu/18.04/cinc_15.4.45-1_amd64.deb \u0026\u0026 dpkg -i cinc_15.4.45-1_amd64.deb"},{"line_number":12,"context_line":"      chdir: /tmp"},{"line_number":13,"context_line":"    become: yes"},{"line_number":14,"context_line":"  - name: Install berkshelf"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_9dcec2b2","line":11,"updated":"2019-10-18 16:32:59.000000000","message":"I\u0027d prefer to keep this with using the install.sh if possible. Can you just download https://omnitruck.cinc.sh/install.sh and replace it with what we have in openstack-chef and change it to \u0027-v 15\u0027?\n\nAny reason why you\u0027re doing it this way instead of the other method?","commit_id":"96388b505e9517f02ce1fdf2bc06af6a8e443e51"},{"author":{"_account_id":21961,"name":"Lance Albertson","email":"lance@osuosl.org","username":"ramereth"},"change_message_id":"f93bbe7e77dc031ba899a19c9f55b1f0c5327260","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  tasks:"},{"line_number":9,"context_line":"  - name: Install cinc package"},{"line_number":10,"context_line":"    shell:"},{"line_number":11,"context_line":"      cmd: wget http://downloads.cinc.sh/files/stable/cinc/15.10.12/ubuntu/18.04/cinc_15.10.12-1_amd64.deb \u0026\u0026 dpkg -i cinc_15.10.12-1_amd64.deb \u0026\u0026 ln -s /opt/cinc /opt/chef"},{"line_number":12,"context_line":"      chdir: /tmp"},{"line_number":13,"context_line":"    become: yes"},{"line_number":14,"context_line":"  - name: Install berkshelf"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"ff570b3c_049ae5c3","line":11,"updated":"2020-05-26 16:12:44.000000000","message":"If we\u0027re going to be using Cinc, I recommend we install it in a similar fashion we\u0027ve been doing with Chef using the install.sh script. In order to get it to work with the Cinc omnitruck instance, you\u0027ll want to download the script from https://omnitruck.cinc.sh/install.sh and then use that one instead.\n\nAre you just testing or intending to replace entirely? We may want to just create a new install-cinc.sh script instead of replacing this one depending on which.","commit_id":"d8cd6c755d5b53176e61979883b5612b65c8e96c"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"118ba851e116925818f629f46f12bcdc18e266f7","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  tasks:"},{"line_number":9,"context_line":"  - name: Install cinc package"},{"line_number":10,"context_line":"    shell:"},{"line_number":11,"context_line":"      cmd: wget http://downloads.cinc.sh/files/stable/cinc/15.10.12/ubuntu/18.04/cinc_15.10.12-1_amd64.deb \u0026\u0026 dpkg -i cinc_15.10.12-1_amd64.deb \u0026\u0026 ln -s /opt/cinc /opt/chef"},{"line_number":12,"context_line":"      chdir: /tmp"},{"line_number":13,"context_line":"    become: yes"},{"line_number":14,"context_line":"  - name: Install berkshelf"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"ff570b3c_b576c53d","line":11,"in_reply_to":"ff570b3c_049ae5c3","updated":"2020-05-26 18:49:45.000000000","message":"My idea was to create an ansible role that would install either chef or cinc depending on some variable, similar to e.g. ensure-python in the upstream jobs.\n\nI\u0027ll take a look at the install script, though at least for stable branches I\u0027d actually prefer to install a pinned client version, too, instead of \"latest 15.x\". That avoids regressions that may go unnoticed for some time unless one creates and watches a periodic testing job. It would also be more inline with our real-world deployments, where we don\u0027t update the client for the lifetime of a stable branch. (Unless required by some bug, but I don\u0027t remember that happen.)","commit_id":"d8cd6c755d5b53176e61979883b5612b65c8e96c"},{"author":{"_account_id":21961,"name":"Lance Albertson","email":"lance@osuosl.org","username":"ramereth"},"change_message_id":"afe6d80e048e214d93dd5c3b9f7d20c022d45f86","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  tasks:"},{"line_number":9,"context_line":"  - name: Install cinc package"},{"line_number":10,"context_line":"    shell:"},{"line_number":11,"context_line":"      cmd: wget http://downloads.cinc.sh/files/stable/cinc/15.10.12/ubuntu/18.04/cinc_15.10.12-1_amd64.deb \u0026\u0026 dpkg -i cinc_15.10.12-1_amd64.deb \u0026\u0026 ln -s /opt/cinc /opt/chef"},{"line_number":12,"context_line":"      chdir: /tmp"},{"line_number":13,"context_line":"    become: yes"},{"line_number":14,"context_line":"  - name: Install berkshelf"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"ff570b3c_e20cfe3a","line":11,"in_reply_to":"ff570b3c_b576c53d","updated":"2020-05-26 20:02:58.000000000","message":"I like the idea of testing both via some variable!\n\nRegarding latest 15.x. From my experience there are very few regressions between the minor releases and if there is one, they usually release a fix quickly to resolve it. I think it\u0027s important to test the latest major release no matter what it is so we can catch any issues that may happen.\n\nAt any rate, you can still use the script to pin specific versions, i.e. \"-v 15.10\" or even \"-v 15.10.12\". For stable branches, it might make sense to pin it at least the minor version since patch version bumps in the stable channel only include critical regression fixes and no feature additions.\n\nHowever for Workstation, I do think it\u0027s important to pin that since they update cookstyle regularly which can result in extra work depending on how things are.\n\nSo in short:\n\n1. I recommend we put to just the major version in the master branch\n2. Pin to the minor version on the stable branches\n\nThoughts?","commit_id":"d8cd6c755d5b53176e61979883b5612b65c8e96c"}],"playbooks/roles/install-chef/files/cinc-install.sh":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9e1e1bb2783c9f107e7dbc49b9a5c70647aef961","unresolved":false,"context_lines":[{"line_number":643,"context_line":"  echo \"Getting information for $project $channel $version for $platform...\""},{"line_number":644,"context_line":""},{"line_number":645,"context_line":"  metadata_filename\u003d\"$tmp_dir/metadata.txt\""},{"line_number":646,"context_line":"  metadata_url\u003d\"https://omnitruck.cinc.sh/$channel/$project/metadata?v\u003d$version\u0026p\u003d$platform\u0026pv\u003d$platform_version\u0026m\u003d$machine\""},{"line_number":647,"context_line":""},{"line_number":648,"context_line":"  do_download \"$metadata_url\"  \"$metadata_filename\""},{"line_number":649,"context_line":""}],"source_content_type":"text/x-sh","patch_set":13,"id":"ff570b3c_2793b602","line":646,"updated":"2020-06-10 08:00:47.000000000","message":"Actually this is the only significant difference between this script and chef-install.sh , maybe we want to make the base URL a parameter and avoid the need to have this script twice?","commit_id":"d41ba6c75df2ed74712b50b963d93044c80043e0"},{"author":{"_account_id":21961,"name":"Lance Albertson","email":"lance@osuosl.org","username":"ramereth"},"change_message_id":"14d9ef8890d225e45420ef457daf5b8f44408a44","unresolved":false,"context_lines":[{"line_number":643,"context_line":"  echo \"Getting information for $project $channel $version for $platform...\""},{"line_number":644,"context_line":""},{"line_number":645,"context_line":"  metadata_filename\u003d\"$tmp_dir/metadata.txt\""},{"line_number":646,"context_line":"  metadata_url\u003d\"https://omnitruck.cinc.sh/$channel/$project/metadata?v\u003d$version\u0026p\u003d$platform\u0026pv\u003d$platform_version\u0026m\u003d$machine\""},{"line_number":647,"context_line":""},{"line_number":648,"context_line":"  do_download \"$metadata_url\"  \"$metadata_filename\""},{"line_number":649,"context_line":""}],"source_content_type":"text/x-sh","patch_set":13,"id":"ff570b3c_9fbac207","line":646,"in_reply_to":"ff570b3c_2793b602","updated":"2020-06-10 16:46:23.000000000","message":"I\u0027d rather not just in case we change additional things later.","commit_id":"d41ba6c75df2ed74712b50b963d93044c80043e0"}]}