)]}'
{".zuul.yaml":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"5c3f5d66297d60f11615f1399676a3d297a83343","unresolved":false,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":"- job:"},{"line_number":133,"context_line":"    name: swift-probetests-centos-7"},{"line_number":134,"context_line":"    parent: unittests"},{"line_number":135,"context_line":"    nodeset: centos-7"},{"line_number":136,"context_line":"    voting: false"},{"line_number":137,"context_line":"    description: |"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5f7c97a3_94c41961","line":134,"range":{"start_line":134,"start_character":4,"end_line":134,"end_character":21},"updated":"2018-05-23 16:44:33.000000000","message":"I think this maybe sets up some extra yum repos that we need above?","commit_id":"50fb39cf4486f20908fb6cdf28c5c51bf6a27cd5"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"e794a80016d6f7d18a1ee28cfd9fa4877770017d","unresolved":false,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":"- job:"},{"line_number":133,"context_line":"    name: swift-probetests-centos-7"},{"line_number":134,"context_line":"    parent: unittests"},{"line_number":135,"context_line":"    nodeset: centos-7"},{"line_number":136,"context_line":"    voting: false"},{"line_number":137,"context_line":"    description: |"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5f7c97a3_b8c0237e","line":134,"range":{"start_line":134,"start_character":4,"end_line":134,"end_character":21},"in_reply_to":"5f7c97a3_94c41961","updated":"2018-05-24 00:03:41.000000000","message":"Hum... let me try it. I didn\u0027t where the unittests defined at tho.","commit_id":"50fb39cf4486f20908fb6cdf28c5c51bf6a27cd5"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"6fc2cd537a4a69ad0fa1c52d15a418abb24099e3","unresolved":false,"context_lines":[{"line_number":251,"context_line":"              - ^(.gitreview|.mailmap|AUTHORS|CHANGELOG)$"},{"line_number":252,"context_line":"        - swift-tox-func-s3api-ceph-s3tests-tempauth:"},{"line_number":253,"context_line":"            irrelevant-files:"},{"line_number":254,"context_line":"              - ^(api-ref|doc|releasenotes)/.*$"},{"line_number":255,"context_line":"              - ^test/probe/.*$"},{"line_number":256,"context_line":"              - ^(.gitreview|.mailmap|AUTHORS|CHANGELOG)$"},{"line_number":257,"context_line":"        - swift-probetests-centos-7:"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"3f79a3b5_f8d234fd","line":254,"range":{"start_line":254,"start_character":26,"end_line":254,"end_character":29},"updated":"2018-09-13 19:55:19.000000000","message":"We still want to run this when just things change under doc/saio or doc/s3api/conf.","commit_id":"a927fe290ff91fd6869e2dfdc4717efa83ba3c4e"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"331412b9bd9fdf02a27e0abccde74559033af7c4","unresolved":false,"context_lines":[{"line_number":251,"context_line":"              - ^(.gitreview|.mailmap|AUTHORS|CHANGELOG)$"},{"line_number":252,"context_line":"        - swift-tox-func-s3api-ceph-s3tests-tempauth:"},{"line_number":253,"context_line":"            irrelevant-files:"},{"line_number":254,"context_line":"              - ^(api-ref|doc|releasenotes)/.*$"},{"line_number":255,"context_line":"              - ^test/probe/.*$"},{"line_number":256,"context_line":"              - ^(.gitreview|.mailmap|AUTHORS|CHANGELOG)$"},{"line_number":257,"context_line":"        - swift-probetests-centos-7:"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"3f79a3b5_c9320cf3","line":254,"range":{"start_line":254,"start_character":26,"end_line":254,"end_character":29},"in_reply_to":"3f79a3b5_0e256a0a","updated":"2018-09-13 21:13:49.000000000","message":"We could still include a line for\n\n - ^doc/(requirements.txt|(manpages|source)/.*)$\n\nso we don\u0027t run this on pure doc cleanup, but that could happen later. Better to run too often than not often enough.","commit_id":"a927fe290ff91fd6869e2dfdc4717efa83ba3c4e"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"46725c433570fc9229bbebff6f99a9948d0e2160","unresolved":false,"context_lines":[{"line_number":251,"context_line":"              - ^(.gitreview|.mailmap|AUTHORS|CHANGELOG)$"},{"line_number":252,"context_line":"        - swift-tox-func-s3api-ceph-s3tests-tempauth:"},{"line_number":253,"context_line":"            irrelevant-files:"},{"line_number":254,"context_line":"              - ^(api-ref|doc|releasenotes)/.*$"},{"line_number":255,"context_line":"              - ^test/probe/.*$"},{"line_number":256,"context_line":"              - ^(.gitreview|.mailmap|AUTHORS|CHANGELOG)$"},{"line_number":257,"context_line":"        - swift-probetests-centos-7:"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"3f79a3b5_0e256a0a","line":254,"range":{"start_line":254,"start_character":26,"end_line":254,"end_character":29},"in_reply_to":"3f79a3b5_f8d234fd","updated":"2018-09-13 20:29:49.000000000","message":"Correct. I just got the means with your comment.\n\nSorry it was just copy and paste from other jobs.","commit_id":"a927fe290ff91fd6869e2dfdc4717efa83ba3c4e"}],"playbooks/ceph-s3tests/post.yaml":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"716512f14e3bd5d67ec9a0d8e9746f5573fbb54a","unresolved":false,"context_lines":[{"line_number":17,"context_line":"        verify_host: true"},{"line_number":18,"context_line":"      with_items:"},{"line_number":19,"context_line":"        - \u0027/var/log/swift\u0027"},{"line_number":20,"context_line":"        - \u0027{{ ansible_env.HOME }}/s3compat/output\u0027"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_e76e3c95","line":20,"range":{"start_line":20,"start_character":11,"end_line":20,"end_character":49},"updated":"2018-06-05 18:25:42.000000000","message":"I wonder if it would be better to have this playbook *just* target this log, and have the zuul job definition reference both post.yaml books...","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"4cd01c3c89f149189cf6aaf7ea27274416995dc4","unresolved":false,"context_lines":[{"line_number":17,"context_line":"        verify_host: true"},{"line_number":18,"context_line":"      with_items:"},{"line_number":19,"context_line":"        - \u0027/var/log/swift\u0027"},{"line_number":20,"context_line":"        - \u0027{{ ansible_env.HOME }}/s3compat/output\u0027"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_9ff7848d","line":20,"range":{"start_line":20,"start_character":11,"end_line":20,"end_character":49},"in_reply_to":"5f7c97a3_e76e3c95","updated":"2018-06-06 21:07:05.000000000","message":"Done","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"}],"playbooks/ceph-s3tests/run.yaml":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"5c3f5d66297d60f11615f1399676a3d297a83343","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    - name: Add allowed_headers config"},{"line_number":26,"context_line":"      lineinfile:"},{"line_number":27,"context_line":"          regexp: \"^# allowed_headers \u003d\""},{"line_number":28,"context_line":"          line: \"allowed_headers \u003d Content-Disposition, Content-Encoding, X-Delete-At, X-Object-Manifest, X-Static-Large-Object Cache-Control, Content-Language, Expires, and X-Robots-Tag\""},{"line_number":29,"context_line":"      become: true"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    - name: Reload all configs on main modules"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5f7c97a3_94a739a8","line":28,"updated":"2018-05-23 16:44:33.000000000","message":"These changes are also something we should consider for swift in general.","commit_id":"50fb39cf4486f20908fb6cdf28c5c51bf6a27cd5"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"e794a80016d6f7d18a1ee28cfd9fa4877770017d","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    - name: Add allowed_headers config"},{"line_number":26,"context_line":"      lineinfile:"},{"line_number":27,"context_line":"          regexp: \"^# allowed_headers \u003d\""},{"line_number":28,"context_line":"          line: \"allowed_headers \u003d Content-Disposition, Content-Encoding, X-Delete-At, X-Object-Manifest, X-Static-Large-Object Cache-Control, Content-Language, Expires, and X-Robots-Tag\""},{"line_number":29,"context_line":"      become: true"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    - name: Reload all configs on main modules"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5f7c97a3_d8407f0a","line":28,"in_reply_to":"5f7c97a3_94a739a8","updated":"2018-05-24 00:03:41.000000000","message":"yup","commit_id":"50fb39cf4486f20908fb6cdf28c5c51bf6a27cd5"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"5c3f5d66297d60f11615f1399676a3d297a83343","unresolved":false,"context_lines":[{"line_number":63,"context_line":"      shell: |"},{"line_number":64,"context_line":"        ./bin/get_ceph_test_attributes.py"},{"line_number":65,"context_line":"        ./bin/report.py --detailed \"{{ zuul.executor.log_root }}/ceph-s3.out.yaml\" \\"},{"line_number":66,"context_line":"            --known-failures \"{{ zuul.project.src_dir }}/doc/s3api/conf/ceph-known-failures-tempauth.yaml\" \\"},{"line_number":67,"context_line":"            --detailedformat console \"{{ zuul.executor.log_root }}/ceph-s3-details.xml\" | \\"},{"line_number":68,"context_line":"            tee \"{{ zuul.executor.log_root }}}/ceph-s3-summary.log\""},{"line_number":69,"context_line":"      chdir:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5f7c97a3_348f4524","line":66,"updated":"2018-05-23 16:44:33.000000000","message":"I like the tree you\u0027re making -- I wonder if this would be better as playbooks/ceph-s3tests/known-failures-tempauth.yaml","commit_id":"50fb39cf4486f20908fb6cdf28c5c51bf6a27cd5"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"e794a80016d6f7d18a1ee28cfd9fa4877770017d","unresolved":false,"context_lines":[{"line_number":63,"context_line":"      shell: |"},{"line_number":64,"context_line":"        ./bin/get_ceph_test_attributes.py"},{"line_number":65,"context_line":"        ./bin/report.py --detailed \"{{ zuul.executor.log_root }}/ceph-s3.out.yaml\" \\"},{"line_number":66,"context_line":"            --known-failures \"{{ zuul.project.src_dir }}/doc/s3api/conf/ceph-known-failures-tempauth.yaml\" \\"},{"line_number":67,"context_line":"            --detailedformat console \"{{ zuul.executor.log_root }}/ceph-s3-details.xml\" | \\"},{"line_number":68,"context_line":"            tee \"{{ zuul.executor.log_root }}}/ceph-s3-summary.log\""},{"line_number":69,"context_line":"      chdir:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5f7c97a3_d8595ffe","line":66,"in_reply_to":"5f7c97a3_348f4524","updated":"2018-05-24 00:03:41.000000000","message":"moving it to the playbooks dir might be better. let me do it after i got succeeded with the gate.","commit_id":"50fb39cf4486f20908fb6cdf28c5c51bf6a27cd5"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"36df705a25cabe24bccab765baea6b00bf072e1c","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    - name: Clone s3compat repository"},{"line_number":42,"context_line":"      git:"},{"line_number":43,"context_line":"          repo: \"https://github.com/swiftstack/s3compat.git\""},{"line_number":44,"context_line":"          dest: \u0027{{ zuul.project.src_dir }}/s3compat\u0027"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    - name: Update pip"},{"line_number":47,"context_line":"      pip:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"5f7c97a3_df3920d7","line":44,"updated":"2018-05-28 07:54:34.000000000","message":"The result said this works as *chaned* so the clone should be successful.","commit_id":"72a0e767e928f7cf4954db92303236f0908ed05b"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"36df705a25cabe24bccab765baea6b00bf072e1c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"          state: latest"},{"line_number":50,"context_line":"      become: true"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    - name: Install s3compat requirements"},{"line_number":53,"context_line":"      pip:"},{"line_number":54,"context_line":"          requirements: \u0027{{ zuul.project.src_dir }}/s3compat/requirements.txt\u0027"},{"line_number":55,"context_line":"      become: true"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    - name: Show packages"},{"line_number":58,"context_line":"      shell: \"pip freeze\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"5f7c97a3_1f2478ae","line":55,"range":{"start_line":52,"start_character":1,"end_line":55,"end_character":18},"updated":"2018-05-28 07:54:34.000000000","message":"???\n\nI don\u0027t get why the requirements file is not found in the gate. This works well in my environment. Am I missing something?","commit_id":"72a0e767e928f7cf4954db92303236f0908ed05b"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"6a67e3ff3481b01284aeb062df325b10924f9559","unresolved":false,"context_lines":[{"line_number":49,"context_line":"          state: latest"},{"line_number":50,"context_line":"      become: true"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    - name: Install s3compat requirements"},{"line_number":53,"context_line":"      pip:"},{"line_number":54,"context_line":"          requirements: \u0027{{ zuul.project.src_dir }}/s3compat/requirements.txt\u0027"},{"line_number":55,"context_line":"      become: true"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    - name: Show packages"},{"line_number":58,"context_line":"      shell: \"pip freeze\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"5f7c97a3_bf746469","line":55,"range":{"start_line":52,"start_character":1,"end_line":55,"end_character":18},"in_reply_to":"5f7c97a3_1f2478ae","updated":"2018-05-28 08:08:40.000000000","message":"It seems like the requirements path requires absolute path to the file and src_dir is relative.","commit_id":"72a0e767e928f7cf4954db92303236f0908ed05b"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"716512f14e3bd5d67ec9a0d8e9746f5573fbb54a","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":"- hosts: all"},{"line_number":16,"context_line":"  tasks:"},{"line_number":17,"context_line":"    # TODO: remove this task when s3api is in the pipeline by default"},{"line_number":18,"context_line":"    - name: Add s3api in proxy-server.conf"},{"line_number":19,"context_line":"      replace:"},{"line_number":20,"context_line":"          path: \"/etc/swift/proxy-server.conf\""}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_8700a8ad","line":17,"updated":"2018-06-05 18:25:42.000000000","message":"Another option would be to establish a precedent of having override confs stored alongside the relevant playbooks...\n\nThen we could copy initial confs from the saio docs, and layer on any job-specific configs...\n\nProbably out of scope, but something to think about as we add more jobs.","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"4cd01c3c89f149189cf6aaf7ea27274416995dc4","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":"- hosts: all"},{"line_number":16,"context_line":"  tasks:"},{"line_number":17,"context_line":"    # TODO: remove this task when s3api is in the pipeline by default"},{"line_number":18,"context_line":"    - name: Add s3api in proxy-server.conf"},{"line_number":19,"context_line":"      replace:"},{"line_number":20,"context_line":"          path: \"/etc/swift/proxy-server.conf\""}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_dfe41ce3","line":17,"in_reply_to":"5f7c97a3_8700a8ad","updated":"2018-06-06 21:07:05.000000000","message":"It sounds good idea. Plus, it might be useful to consider that we could break the conf file into small chunks (parts of conf) and then using directory style configuration like we\u0027ve done in vagrant-swift-all-in-one. WIth that, we will be able to overwrite only the part of pipeline config.\n\nAnyway, it seems to be more bigger refactoring so that I\u0027m inclined to do that in the future other patches.","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"716512f14e3bd5d67ec9a0d8e9746f5573fbb54a","unresolved":false,"context_lines":[{"line_number":22,"context_line":"          replace: \"container_sync s3api tempauth\""},{"line_number":23,"context_line":"      become: true"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    - name: find object-server config files to modify allowed_headers option"},{"line_number":26,"context_line":"      find: paths\u003d\"/etc/swift/object/\" patterns\u003d\"*.conf\" recurse\u003dyes"},{"line_number":27,"context_line":"      register: find_object_configs"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    - name: Add allowed_headers config"},{"line_number":30,"context_line":"      lineinfile:"},{"line_number":31,"context_line":"          path: \"{{ item.path }}\""},{"line_number":32,"context_line":"          regexp: \"^# allowed_headers \u003d\""},{"line_number":33,"context_line":"          line: \"allowed_headers \u003d Content-Disposition, Content-Encoding, X-Delete-At, X-Object-Manifest, X-Static-Large-Object Cache-Control, Content-Language, Expires, and X-Robots-Tag\""},{"line_number":34,"context_line":"      with_items: \"{{ find_object_configs.files }}\""},{"line_number":35,"context_line":"      become: true"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    - name: Starts main swift servers"},{"line_number":38,"context_line":"      shell: \"swift-init main start\""}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_07b498f1","line":35,"range":{"start_line":25,"start_character":0,"end_line":35,"end_character":18},"updated":"2018-06-05 18:25:42.000000000","message":"No longer necessary following https://github.com/openstack/swift/commit/36dbd38","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"4cd01c3c89f149189cf6aaf7ea27274416995dc4","unresolved":false,"context_lines":[{"line_number":22,"context_line":"          replace: \"container_sync s3api tempauth\""},{"line_number":23,"context_line":"      become: true"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    - name: find object-server config files to modify allowed_headers option"},{"line_number":26,"context_line":"      find: paths\u003d\"/etc/swift/object/\" patterns\u003d\"*.conf\" recurse\u003dyes"},{"line_number":27,"context_line":"      register: find_object_configs"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    - name: Add allowed_headers config"},{"line_number":30,"context_line":"      lineinfile:"},{"line_number":31,"context_line":"          path: \"{{ item.path }}\""},{"line_number":32,"context_line":"          regexp: \"^# allowed_headers \u003d\""},{"line_number":33,"context_line":"          line: \"allowed_headers \u003d Content-Disposition, Content-Encoding, X-Delete-At, X-Object-Manifest, X-Static-Large-Object Cache-Control, Content-Language, Expires, and X-Robots-Tag\""},{"line_number":34,"context_line":"      with_items: \"{{ find_object_configs.files }}\""},{"line_number":35,"context_line":"      become: true"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    - name: Starts main swift servers"},{"line_number":38,"context_line":"      shell: \"swift-init main start\""}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_bf6e0094","line":35,"range":{"start_line":25,"start_character":0,"end_line":35,"end_character":18},"in_reply_to":"5f7c97a3_07b498f1","updated":"2018-06-06 21:07:05.000000000","message":"Done","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"716512f14e3bd5d67ec9a0d8e9746f5573fbb54a","unresolved":false,"context_lines":[{"line_number":55,"context_line":"      shell: \u0027{{ ansible_env.HOME }}/s3compat/bin/run_ceph_tests.py \"$@\" || true\u0027"},{"line_number":56,"context_line":"      environment:"},{"line_number":57,"context_line":"          S3TEST_CONF: \"{{ ansible_env.HOME }}/{{ zuul.project.src_dir }}/playbooks/ceph-s3tests/ceph-s3.conf\""},{"line_number":58,"context_line":"          S3ACL: \"true\""},{"line_number":59,"context_line":"          DNS_BUCKET_NAMES: \"false\""},{"line_number":60,"context_line":"          CHECK_BUCKET_OWNER: \"true\""},{"line_number":61,"context_line":"      args:"},{"line_number":62,"context_line":"          chdir: \u0027{{ ansible_env.HOME }}/s3compat\u0027"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_07825801","line":60,"range":{"start_line":58,"start_character":0,"end_line":60,"end_character":36},"updated":"2018-06-05 18:25:42.000000000","message":"These were necessary given how swift3 set up functional tests, but they shouldn\u0027t be needed now.","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"4cd01c3c89f149189cf6aaf7ea27274416995dc4","unresolved":false,"context_lines":[{"line_number":55,"context_line":"      shell: \u0027{{ ansible_env.HOME }}/s3compat/bin/run_ceph_tests.py \"$@\" || true\u0027"},{"line_number":56,"context_line":"      environment:"},{"line_number":57,"context_line":"          S3TEST_CONF: \"{{ ansible_env.HOME }}/{{ zuul.project.src_dir }}/playbooks/ceph-s3tests/ceph-s3.conf\""},{"line_number":58,"context_line":"          S3ACL: \"true\""},{"line_number":59,"context_line":"          DNS_BUCKET_NAMES: \"false\""},{"line_number":60,"context_line":"          CHECK_BUCKET_OWNER: \"true\""},{"line_number":61,"context_line":"      args:"},{"line_number":62,"context_line":"          chdir: \u0027{{ ansible_env.HOME }}/s3compat\u0027"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_9f7b0428","line":60,"range":{"start_line":58,"start_character":0,"end_line":60,"end_character":36},"in_reply_to":"5f7c97a3_07825801","updated":"2018-06-06 21:07:05.000000000","message":"OIC. They are in saio conf example. Make sense that we can remove those settings from here.","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"eb1029119ee58d062b291d49abcc253b2884ce4a","unresolved":false,"context_lines":[{"line_number":37,"context_line":"      become: true"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    - name: Show packages"},{"line_number":40,"context_line":"      shell: \"pip freeze\""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    - name: Run s3compat tests"},{"line_number":43,"context_line":"      shell: \u0027{{ ansible_env.HOME }}/s3compat/bin/run_ceph_tests.py \"$@\" || true\u0027"}],"source_content_type":"text/x-yaml","patch_set":27,"id":"5f7c97a3_ff58e157","line":40,"updated":"2018-06-06 23:42:04.000000000","message":"Maybe not necessary? I think I\u0027d done it before because of the second virtualenv... but IDK that it\u0027s still necessary? IIRC, there was a version conflict on requests... which I think it only really needed by authtoken and s3token?","commit_id":"339768cd09df811c4576f22870a5020a19e33731"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"8b72237ee509ad7f9d5164d25e404362e393bf5b","unresolved":false,"context_lines":[{"line_number":37,"context_line":"      become: true"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    - name: Show packages"},{"line_number":40,"context_line":"      shell: \"pip freeze\""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    - name: Run s3compat tests"},{"line_number":43,"context_line":"      shell: \u0027{{ ansible_env.HOME }}/s3compat/bin/run_ceph_tests.py \"$@\" || true\u0027"}],"source_content_type":"text/x-yaml","patch_set":27,"id":"5f7c97a3_d77e29ae","line":40,"in_reply_to":"5f7c97a3_ff58e157","updated":"2018-06-07 06:36:13.000000000","message":"Done","commit_id":"339768cd09df811c4576f22870a5020a19e33731"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"eb1029119ee58d062b291d49abcc253b2884ce4a","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        ./bin/get_ceph_test_attributes.py"},{"line_number":55,"context_line":"        ./bin/report.py --detailed \"{{ ansible_env.HOME }}/s3compat/output/ceph-s3.out.yaml\" \\"},{"line_number":56,"context_line":"            --known-failures \"{{ ansible_env.HOME}}/{{ zuul.project.src_dir }}/doc/s3api/conf/ceph-known-failures-tempauth.yaml\" \\"},{"line_number":57,"context_line":"            --detailedformat console \"{{ ansible_env.HOME }}/s3compat/output/ceph-s3.out.xml\" | \\"},{"line_number":58,"context_line":"            tee \"{{ ansible_env.HOME }}/s3compat/output/ceph-s3-summary.log\""},{"line_number":59,"context_line":"      args:"},{"line_number":60,"context_line":"          chdir:"},{"line_number":61,"context_line":"              \"{{ ansible_env.HOME }}/s3compat\""}],"source_content_type":"text/x-yaml","patch_set":27,"id":"5f7c97a3_9f7e8dad","line":58,"range":{"start_line":57,"start_character":94,"end_line":58,"end_character":15},"updated":"2018-06-06 23:42:04.000000000","message":"FWIW, I *think* this is why https://review.openstack.org/#/c/572508/ didn\u0027t fail. We probably ought to\n\n set -o pipefail\n\nor even\n\n set -e\n\nat the start of the shell block; see https://stackoverflow.com/questions/33304866/ansible-enforce-pipefail and https://github.com/openstack/tripleo-quickstart-extras/commit/69ad943","commit_id":"339768cd09df811c4576f22870a5020a19e33731"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"8b72237ee509ad7f9d5164d25e404362e393bf5b","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        ./bin/get_ceph_test_attributes.py"},{"line_number":55,"context_line":"        ./bin/report.py --detailed \"{{ ansible_env.HOME }}/s3compat/output/ceph-s3.out.yaml\" \\"},{"line_number":56,"context_line":"            --known-failures \"{{ ansible_env.HOME}}/{{ zuul.project.src_dir }}/doc/s3api/conf/ceph-known-failures-tempauth.yaml\" \\"},{"line_number":57,"context_line":"            --detailedformat console \"{{ ansible_env.HOME }}/s3compat/output/ceph-s3.out.xml\" | \\"},{"line_number":58,"context_line":"            tee \"{{ ansible_env.HOME }}/s3compat/output/ceph-s3-summary.log\""},{"line_number":59,"context_line":"      args:"},{"line_number":60,"context_line":"          chdir:"},{"line_number":61,"context_line":"              \"{{ ansible_env.HOME }}/s3compat\""}],"source_content_type":"text/x-yaml","patch_set":27,"id":"5f7c97a3_974af180","line":58,"range":{"start_line":57,"start_character":94,"end_line":58,"end_character":15},"in_reply_to":"5f7c97a3_9f7e8dad","updated":"2018-06-07 06:36:13.000000000","message":"Let\u0027s try it.","commit_id":"339768cd09df811c4576f22870a5020a19e33731"}],"playbooks/saio_single_node_setup/install_dependencies.yaml":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"1c9002a2c826ae6cd553936b912bd866b4583b43","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        # Memo: eventlet seems like to be installed via swift setup"},{"line_number":48,"context_line":"        # - python-eventlet"},{"line_number":49,"context_line":"        # Memo: pyeclib as well"},{"line_number":50,"context_line":"        # but system package is maybe faster?"},{"line_number":51,"context_line":"        - python-pyeclib"},{"line_number":52,"context_line":"        - python-pip"},{"line_number":53,"context_line":"        - python-nose"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"5f7c97a3_749c6a0d","line":50,"updated":"2018-05-31 23:47:53.000000000","message":"I\u0027m sad that this is apparently true :-(","commit_id":"ff564219ffd6818803779b30b83ef9441134fcd2"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"716512f14e3bd5d67ec9a0d8e9746f5573fbb54a","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        - python-pip"},{"line_number":49,"context_line":"        - python-nose"},{"line_number":50,"context_line":"        - python-swiftclient"},{"line_number":51,"context_line":"      become: true"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    - name: update pip version to be the latest"},{"line_number":54,"context_line":"      pip:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_c77b0027","line":51,"updated":"2018-06-05 18:25:42.000000000","message":"I\u0027m a little surprised it worked without this before...","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"4cd01c3c89f149189cf6aaf7ea27274416995dc4","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        - python-pip"},{"line_number":49,"context_line":"        - python-nose"},{"line_number":50,"context_line":"        - python-swiftclient"},{"line_number":51,"context_line":"      become: true"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    - name: update pip version to be the latest"},{"line_number":54,"context_line":"      pip:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_bfe120ff","line":51,"in_reply_to":"5f7c97a3_c77b0027","updated":"2018-06-06 21:07:05.000000000","message":"indeed","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"716512f14e3bd5d67ec9a0d8e9746f5573fbb54a","unresolved":false,"context_lines":[{"line_number":62,"context_line":"          # The upper constraints of eventlet is pinned to 0.20.0"},{"line_number":63,"context_line":"          # at https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt"},{"line_number":64,"context_line":"          # FIXME: hopefully, -c should work to pin dinamically but it affects other modules"},{"line_number":65,"context_line":"          #        try to resolve the constraints and, fall into conflicts with disturibution installation"},{"line_number":66,"context_line":"          #        so this attempts to install only eventlet \u003c\u003d 0.20.0"},{"line_number":67,"context_line":"          version: 0.20.0"},{"line_number":68,"context_line":"      become: true"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_87750835","line":65,"range":{"start_line":65,"start_character":80,"end_line":65,"end_character":93},"updated":"2018-06-05 18:25:42.000000000","message":"nit: distribution","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"4cd01c3c89f149189cf6aaf7ea27274416995dc4","unresolved":false,"context_lines":[{"line_number":62,"context_line":"          # The upper constraints of eventlet is pinned to 0.20.0"},{"line_number":63,"context_line":"          # at https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt"},{"line_number":64,"context_line":"          # FIXME: hopefully, -c should work to pin dinamically but it affects other modules"},{"line_number":65,"context_line":"          #        try to resolve the constraints and, fall into conflicts with disturibution installation"},{"line_number":66,"context_line":"          #        so this attempts to install only eventlet \u003c\u003d 0.20.0"},{"line_number":67,"context_line":"          version: 0.20.0"},{"line_number":68,"context_line":"      become: true"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_dfb3bc09","line":65,"range":{"start_line":65,"start_character":80,"end_line":65,"end_character":93},"in_reply_to":"5f7c97a3_87750835","updated":"2018-06-06 21:07:05.000000000","message":"Done","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"716512f14e3bd5d67ec9a0d8e9746f5573fbb54a","unresolved":false,"context_lines":[{"line_number":63,"context_line":"          # at https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt"},{"line_number":64,"context_line":"          # FIXME: hopefully, -c should work to pin dinamically but it affects other modules"},{"line_number":65,"context_line":"          #        try to resolve the constraints and, fall into conflicts with disturibution installation"},{"line_number":66,"context_line":"          #        so this attempts to install only eventlet \u003c\u003d 0.20.0"},{"line_number":67,"context_line":"          version: 0.20.0"},{"line_number":68,"context_line":"      become: true"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_0797b83c","line":66,"range":{"start_line":66,"start_character":52,"end_line":66,"end_character":70},"updated":"2018-06-05 18:25:42.000000000","message":"Or is it attempting to install\n\n eventlet \u003d\u003d 0.20.0\n\n? *shrug* Not that I\u0027m expecting 0.20.0 to get unpublished or anything...","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"4cd01c3c89f149189cf6aaf7ea27274416995dc4","unresolved":false,"context_lines":[{"line_number":63,"context_line":"          # at https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt"},{"line_number":64,"context_line":"          # FIXME: hopefully, -c should work to pin dinamically but it affects other modules"},{"line_number":65,"context_line":"          #        try to resolve the constraints and, fall into conflicts with disturibution installation"},{"line_number":66,"context_line":"          #        so this attempts to install only eventlet \u003c\u003d 0.20.0"},{"line_number":67,"context_line":"          version: 0.20.0"},{"line_number":68,"context_line":"      become: true"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f7c97a3_9f922458","line":66,"range":{"start_line":66,"start_character":52,"end_line":66,"end_character":70},"in_reply_to":"5f7c97a3_0797b83c","updated":"2018-06-06 21:07:05.000000000","message":"Done","commit_id":"58d23d748e68c8f636e4982d68d1770615493ec7"}],"requirements.txt":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"1c9002a2c826ae6cd553936b912bd866b4583b43","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":"dnspython\u003e\u003d1.14.0;python_version\u003d\u003d\u00272.7\u0027  # http://www.dnspython.org/LICENSE"},{"line_number":6,"context_line":"# Try to cap eventlet version to avoid 0.23.0"},{"line_number":7,"context_line":"eventlet\u003e\u003d0.17.4,!\u003d0.23.0  # MIT"},{"line_number":8,"context_line":"greenlet\u003e\u003d0.3.1"},{"line_number":9,"context_line":"netifaces\u003e\u003d0.5,!\u003d0.10.0,!\u003d0.10.1"},{"line_number":10,"context_line":"PasteDeploy\u003e\u003d1.3.3"}],"source_content_type":"text/plain","patch_set":22,"id":"5f7c97a3_7438eaec","line":7,"updated":"2018-05-31 23:47:53.000000000","message":"I wonder if we should instead be using something like\n\n pip install -chttps://github.com/openstack/requirements/blob/master/upper-constraints.txt -e .\n\nover in make_rings.yaml instead...\n\nThat\u0027s what saves the rest of our gate, yeah?","commit_id":"ff564219ffd6818803779b30b83ef9441134fcd2"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"e177cbad8b82605a6795131a8017bf59d4aecb45","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":"dnspython\u003e\u003d1.14.0;python_version\u003d\u003d\u00272.7\u0027  # http://www.dnspython.org/LICENSE"},{"line_number":6,"context_line":"# Try to cap eventlet version to avoid 0.23.0"},{"line_number":7,"context_line":"eventlet\u003e\u003d0.17.4,!\u003d0.23.0  # MIT"},{"line_number":8,"context_line":"greenlet\u003e\u003d0.3.1"},{"line_number":9,"context_line":"netifaces\u003e\u003d0.5,!\u003d0.10.0,!\u003d0.10.1"},{"line_number":10,"context_line":"PasteDeploy\u003e\u003d1.3.3"}],"source_content_type":"text/plain","patch_set":22,"id":"5f7c97a3_8059bc8e","line":7,"in_reply_to":"5f7c97a3_7438eaec","updated":"2018-06-01 05:06:36.000000000","message":"Sounds good idea!!\n\nI noticed the 0.22.1 has the problem too so I was looking for the version that works well.\n\nI checked some gate jobs work with 0.20.0 that is capped by the upper-requirements so let\u0027s have the requirements within the install_dependency playbook.\n\nPerhaps, we may think of the upper constraint for swift as well but that should be another patch.","commit_id":"ff564219ffd6818803779b30b83ef9441134fcd2"}],"tools/playbooks/saio_single_node_setup/install_dependencies.yaml":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"6fc2cd537a4a69ad0fa1c52d15a418abb24099e3","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    - name: installing python dependencies"},{"line_number":34,"context_line":"      pip:"},{"line_number":35,"context_line":"          name: eventlet"},{"line_number":36,"context_line":"          # The upper constraints of eventlet is pinned to 0.20.0"},{"line_number":37,"context_line":"          # at https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt"},{"line_number":38,"context_line":"          # FIXME: hopefully, -c should work to pin dinamically but it affects other modules"},{"line_number":39,"context_line":"          #        try to resolve the constraints and, fall into conflicts with distribution installation"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"3f79a3b5_4e6da2ca","line":36,"range":{"start_line":36,"start_character":12,"end_line":36,"end_character":65},"updated":"2018-09-13 19:55:19.000000000","message":"Not anymore! Is this still necessary?","commit_id":"a927fe290ff91fd6869e2dfdc4717efa83ba3c4e"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"46725c433570fc9229bbebff6f99a9948d0e2160","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    - name: installing python dependencies"},{"line_number":34,"context_line":"      pip:"},{"line_number":35,"context_line":"          name: eventlet"},{"line_number":36,"context_line":"          # The upper constraints of eventlet is pinned to 0.20.0"},{"line_number":37,"context_line":"          # at https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt"},{"line_number":38,"context_line":"          # FIXME: hopefully, -c should work to pin dinamically but it affects other modules"},{"line_number":39,"context_line":"          #        try to resolve the constraints and, fall into conflicts with distribution installation"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"3f79a3b5_8e5f3a74","line":36,"range":{"start_line":36,"start_character":12,"end_line":36,"end_character":65},"in_reply_to":"3f79a3b5_4e6da2ca","updated":"2018-09-13 20:29:49.000000000","message":"Indeed. Let\u0027s remove the comment and version pin.\nI\u0027m bit worried we cannot black list the problem version but probably safe pypi is already over the version.","commit_id":"a927fe290ff91fd6869e2dfdc4717efa83ba3c4e"}]}