)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"cc8c39c9b98ace37e3cd71b2498490e974c3f090","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch does the following:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"- updates the Master SHAs for new development work based on the"},{"line_number":12,"context_line":"  OpenStack Mitaka-2 releases"},{"line_number":13,"context_line":"- includes updates to the policy files as required, and moves the Aodh"},{"line_number":14,"context_line":"  repository to openstack_services as it now has implemented a stable"},{"line_number":15,"context_line":"  branch"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3a7e1126_49b61ffa","line":12,"updated":"2015-12-21 14:42:14.000000000","message":"mitaka-2 is only in January, so I\u0027d suggest replacing this with something else... ref: http://docs.openstack.org/releases/schedules/mitaka.html","commit_id":"a4c66bd37ed70f9e6bfaa203b8505d3a8086b159"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"cc8c39c9b98ace37e3cd71b2498490e974c3f090","unresolved":false,"context_lines":[{"line_number":13,"context_line":"- includes updates to the policy files as required, and moves the Aodh"},{"line_number":14,"context_line":"  repository to openstack_services as it now has implemented a stable"},{"line_number":15,"context_line":"  branch"},{"line_number":16,"context_line":"- pulls in a fresh copy of os_cinder/files/volume.filters since the"},{"line_number":17,"context_line":"  existing filters do not permit volumes from being created"},{"line_number":18,"context_line":"- updates requirements.txt to use the same version of requests as"},{"line_number":19,"context_line":"  global-requirements.txt (without this we end up with 2.9.0 being"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3a7e1126_4984df09","line":16,"updated":"2015-12-21 14:42:14.000000000","message":"Good catch - it\u0027s probably best to verify that all the service rootwraps are up to date.","commit_id":"a4c66bd37ed70f9e6bfaa203b8505d3a8086b159"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"cc8c39c9b98ace37e3cd71b2498490e974c3f090","unresolved":false,"context_lines":[{"line_number":16,"context_line":"- pulls in a fresh copy of os_cinder/files/volume.filters since the"},{"line_number":17,"context_line":"  existing filters do not permit volumes from being created"},{"line_number":18,"context_line":"- updates requirements.txt to use the same version of requests as"},{"line_number":19,"context_line":"  global-requirements.txt (without this we end up with 2.9.0 being"},{"line_number":20,"context_line":"  built, which doesn\u0027t work)"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I933c24c03518865d9d40519dafb2ba46769a5453"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3a7e1126_c999af7a","line":19,"updated":"2015-12-21 14:42:14.000000000","message":"This may be indicative of a different issue, because the wheel building is done using upper-constraints as an upper cap... and right now that has 2.8.1 as the upper constraint.","commit_id":"a4c66bd37ed70f9e6bfaa203b8505d3a8086b159"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"320e48ebc6dce09af7262c6cf0679d36d3b3ebbc","unresolved":false,"context_lines":[{"line_number":16,"context_line":"- pulls in a fresh copy of os_cinder/files/volume.filters since the"},{"line_number":17,"context_line":"  existing filters do not permit volumes from being created"},{"line_number":18,"context_line":"- updates requirements.txt to use the same version of requests as"},{"line_number":19,"context_line":"  global-requirements.txt (without this we end up with 2.9.0 being"},{"line_number":20,"context_line":"  built, which doesn\u0027t work)"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I933c24c03518865d9d40519dafb2ba46769a5453"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3a7e1126_cf452f43","line":19,"in_reply_to":"3a7e1126_c999af7a","updated":"2015-12-21 15:18:58.000000000","message":"local requirements are sourced first in this order https://github.com/openstack/openstack-ansible/blob/master/playbooks/plugins/lookups/py_pkgs.py#L33-L39 then local roles https://github.com/openstack/openstack-ansible/blob/master/playbooks/repo-build.yml#L24-L27 and then items in global requirements and constraints.\n\nThe requirements that we have locally have the most precedence over once sourced remotely. In the various \"requirements_.*.txt\" files the repo_build role creates we can see that the local requirements is overriding the global ones sourced in from the OpenStack requirements repository. The main issue here is that we\u0027re defining some of these requirements and will need to remain in lock step with OpenStack to ensure compatibility.","commit_id":"a4c66bd37ed70f9e6bfaa203b8505d3a8086b159"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"653143d3b6420a1e8fbc7883a4f7c003e79a93e5","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch does the following:"},{"line_number":10,"context_line":"- updates the Master SHAs for new development work based on the"},{"line_number":11,"context_line":"  OpenStack Mitaka-2 releases"},{"line_number":12,"context_line":"- includes updates to the policy files as required, and moves the Aodh"},{"line_number":13,"context_line":"  repository to openstack_services as it now has implemented a stable"},{"line_number":14,"context_line":"  branch"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"fa69d971_2928e2e1","line":11,"updated":"2016-01-04 14:40:54.000000000","message":"These SHA\u0027s are not Mitaka-2. I would prefer that we get the SHA\u0027s to Mitaka-1 (ie the latest master branch tag in each project) and that we don\u0027t SHA bump again until Mitaka-2.","commit_id":"1e8a902cb56201f878cb7ac7ef9024ffbe25da11"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"653143d3b6420a1e8fbc7883a4f7c003e79a93e5","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  OpenStack Mitaka-2 releases"},{"line_number":12,"context_line":"- includes updates to the policy files as required, and moves the Aodh"},{"line_number":13,"context_line":"  repository to openstack_services as it now has implemented a stable"},{"line_number":14,"context_line":"  branch"},{"line_number":15,"context_line":"- pulls in a fresh copy of os_cinder/files/volume.filters since the"},{"line_number":16,"context_line":"  existing filters do not permit volumes from being created"},{"line_number":17,"context_line":"- updates requirements.txt to use \"requests\u003c2.9.0\" (without this we"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"fa69d971_a9c85223","line":14,"updated":"2016-01-04 14:40:54.000000000","message":"I\u0027d like us to figure out a way to rather use the files from the local copy of the git repo as the base templates instead of carrying them like we currently do.","commit_id":"1e8a902cb56201f878cb7ac7ef9024ffbe25da11"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"5daa98880c6bfb0287b38de0da02430affb7e99b","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  OpenStack Mitaka-2 releases"},{"line_number":12,"context_line":"- includes updates to the policy files as required, and moves the Aodh"},{"line_number":13,"context_line":"  repository to openstack_services as it now has implemented a stable"},{"line_number":14,"context_line":"  branch"},{"line_number":15,"context_line":"- pulls in a fresh copy of os_cinder/files/volume.filters since the"},{"line_number":16,"context_line":"  existing filters do not permit volumes from being created"},{"line_number":17,"context_line":"- updates requirements.txt to use \"requests\u003c2.9.0\" (without this we"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"fa69d971_8448d388","line":14,"in_reply_to":"fa69d971_a9c85223","updated":"2016-01-04 15:24:55.000000000","message":"I dont think this is possible especially with the impending role break out work that we\u0027ll be doing. In particular the policy/filter files are part of what allows a given OS role to function and without them the service will more or less not function. I would been keen on the main openstack-ansible repo having a helper script to update the policy/filter files however when the roles get broken out of the main repo that script may be usless. Additionally I think we should avoid downloading the raw policy/filter files from git because that would create yet another hard dependency on an upstream which I\u0027d like to avoid. Personally I think this is a nice to have however I dont believe updating the policy/filter files once or twice a given releases is that large of a chore.","commit_id":"1e8a902cb56201f878cb7ac7ef9024ffbe25da11"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"653143d3b6420a1e8fbc7883a4f7c003e79a93e5","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  repository to openstack_services as it now has implemented a stable"},{"line_number":14,"context_line":"  branch"},{"line_number":15,"context_line":"- pulls in a fresh copy of os_cinder/files/volume.filters since the"},{"line_number":16,"context_line":"  existing filters do not permit volumes from being created"},{"line_number":17,"context_line":"- updates requirements.txt to use \"requests\u003c2.9.0\" (without this we"},{"line_number":18,"context_line":"  end up with 2.9.0 being built, which doesn\u0027t work)"},{"line_number":19,"context_line":"- Updated the keystone-wsgi file as it was still running the code from"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"fa69d971_2956a25c","line":16,"updated":"2016-01-04 14:40:54.000000000","message":"I\u0027d like us to figure out a way to rather use the file from the local copy of the git repo as the base template for this file instead of carrying it like we currently do.","commit_id":"1e8a902cb56201f878cb7ac7ef9024ffbe25da11"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"5daa98880c6bfb0287b38de0da02430affb7e99b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  repository to openstack_services as it now has implemented a stable"},{"line_number":14,"context_line":"  branch"},{"line_number":15,"context_line":"- pulls in a fresh copy of os_cinder/files/volume.filters since the"},{"line_number":16,"context_line":"  existing filters do not permit volumes from being created"},{"line_number":17,"context_line":"- updates requirements.txt to use \"requests\u003c2.9.0\" (without this we"},{"line_number":18,"context_line":"  end up with 2.9.0 being built, which doesn\u0027t work)"},{"line_number":19,"context_line":"- Updated the keystone-wsgi file as it was still running the code from"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"fa69d971_047e23ae","line":16,"in_reply_to":"fa69d971_2956a25c","updated":"2016-01-04 15:24:55.000000000","message":"I dont think this is possible especially with the impending role break out work that we\u0027ll be doing. In particular the policy/filter files are part of what allows a given OS role to function and without them the service will more or less not function. I would been keen on the main openstack-ansible repo having a helper script to update the policy/filter files however when the roles get broken out of the main repo that script may be usless. Additionally I think we should avoid downloading the raw policy/filter files from git because that would create yet another hard dependency on an upstream which I\u0027d like to avoid. Personally I think this is a nice to have however I dont believe updating the policy/filter files once or twice a given releases is that large of a chore.","commit_id":"1e8a902cb56201f878cb7ac7ef9024ffbe25da11"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"653143d3b6420a1e8fbc7883a4f7c003e79a93e5","unresolved":false,"context_lines":[{"line_number":14,"context_line":"  branch"},{"line_number":15,"context_line":"- pulls in a fresh copy of os_cinder/files/volume.filters since the"},{"line_number":16,"context_line":"  existing filters do not permit volumes from being created"},{"line_number":17,"context_line":"- updates requirements.txt to use \"requests\u003c2.9.0\" (without this we"},{"line_number":18,"context_line":"  end up with 2.9.0 being built, which doesn\u0027t work)"},{"line_number":19,"context_line":"- Updated the keystone-wsgi file as it was still running the code from"},{"line_number":20,"context_line":"  liberty"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"fa69d971_c97d86e8","line":17,"updated":"2016-01-04 14:40:54.000000000","message":"I\u0027ve submitted https://review.openstack.org/263214 which removes the pin and the file that requires it. I do think that we need to re-think our use of requirements.txt and figure out how to handle the repo build in a way that the contents of this file don\u0027t affect the entire repo.","commit_id":"1e8a902cb56201f878cb7ac7ef9024ffbe25da11"}]}
