)]}'
{"requirements.txt":[{"author":{"_account_id":7219,"name":"git-harry","email":"git-harry@live.co.uk","username":"git-harry"},"change_message_id":"a71394fdb1f3f53aba626e945a0fdc294bb468a5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"cloudlib\u003e\u003d0.3.0         # scripts/os-ansible-role-requirements.py"},{"line_number":2,"context_line":"ecdsa\u003e\u003d0.11             # playbooks/roles/{os_cinder,os_swift}"},{"line_number":3,"context_line":"Jinja2\u003e\u003d2.6             # ansible"},{"line_number":4,"context_line":"netaddr\u003e\u003d0.7.12         # playbooks/inventory/dynamic_inventory.py"},{"line_number":5,"context_line":"paramiko\u003e\u003d1.13.0        # ansible"}],"source_content_type":"text/plain","patch_set":6,"id":"da9b358b_52cf5eb0","line":2,"updated":"2015-03-30 13:38:00.000000000","message":"Would this not get installed directly by those roles from the wheel repo built as part of the install?","commit_id":"c268d8f59fa5092c133165baa5e2d8102d50d8f9"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"8aad19a78181c07f8a10e5fd0c965330f62d6d94","unresolved":false,"context_lines":[{"line_number":1,"context_line":"cloudlib\u003e\u003d0.3.0         # scripts/os-ansible-role-requirements.py"},{"line_number":2,"context_line":"ecdsa\u003e\u003d0.11             # playbooks/roles/{os_cinder,os_swift}"},{"line_number":3,"context_line":"Jinja2\u003e\u003d2.6             # ansible"},{"line_number":4,"context_line":"netaddr\u003e\u003d0.7.12         # playbooks/inventory/dynamic_inventory.py"},{"line_number":5,"context_line":"paramiko\u003e\u003d1.13.0        # ansible"}],"source_content_type":"text/plain","patch_set":6,"id":"baa041b7_80549654","line":2,"in_reply_to":"da9b358b_4f3c02e2","updated":"2015-04-01 13:39:40.000000000","message":"I removed ecdsa from requirements.txt and tried a test build and everything worked as expected.  The ecdsa package actually gets installed into a number of containers (nova, utility, cinder, swift) and it gets built properly as a wheel as it\u0027s required by paramiko.","commit_id":"c268d8f59fa5092c133165baa5e2d8102d50d8f9"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"55129ce45bfc78b8b086b77bd5527b555d421500","unresolved":false,"context_lines":[{"line_number":1,"context_line":"cloudlib\u003e\u003d0.3.0         # scripts/os-ansible-role-requirements.py"},{"line_number":2,"context_line":"ecdsa\u003e\u003d0.11             # playbooks/roles/{os_cinder,os_swift}"},{"line_number":3,"context_line":"Jinja2\u003e\u003d2.6             # ansible"},{"line_number":4,"context_line":"netaddr\u003e\u003d0.7.12         # playbooks/inventory/dynamic_inventory.py"},{"line_number":5,"context_line":"paramiko\u003e\u003d1.13.0        # ansible"}],"source_content_type":"text/plain","patch_set":6,"id":"da9b358b_cfbff2f9","line":2,"in_reply_to":"da9b358b_52cf5eb0","updated":"2015-04-01 07:19:11.000000000","message":"It\u0027s not clear why this package is needed by those roles, however my guess is that there is no explicit requirement for this package anywhere and therefore to have the package built we have to list it in the top level requirements.txt file.","commit_id":"c268d8f59fa5092c133165baa5e2d8102d50d8f9"},{"author":{"_account_id":7219,"name":"git-harry","email":"git-harry@live.co.uk","username":"git-harry"},"change_message_id":"0e525727342b7e7b5d69a1f12567af36c4081c27","unresolved":false,"context_lines":[{"line_number":1,"context_line":"cloudlib\u003e\u003d0.3.0         # scripts/os-ansible-role-requirements.py"},{"line_number":2,"context_line":"ecdsa\u003e\u003d0.11             # playbooks/roles/{os_cinder,os_swift}"},{"line_number":3,"context_line":"Jinja2\u003e\u003d2.6             # ansible"},{"line_number":4,"context_line":"netaddr\u003e\u003d0.7.12         # playbooks/inventory/dynamic_inventory.py"},{"line_number":5,"context_line":"paramiko\u003e\u003d1.13.0        # ansible"}],"source_content_type":"text/plain","patch_set":6,"id":"da9b358b_4f3c02e2","line":2,"in_reply_to":"da9b358b_cfbff2f9","updated":"2015-04-01 07:43:30.000000000","message":"If those roles require that package and it\u0027s not covered by the requirements file for the project I think it needs to be listed in defaults/main.yml for the role.\n\nDo those roles fail if the package isn\u0027t installed by the bootstrap script?","commit_id":"c268d8f59fa5092c133165baa5e2d8102d50d8f9"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"56580e407310e8b90a3f61f9ff4d75a1d8842911","unresolved":false,"context_lines":[{"line_number":8,"context_line":"pycrypto\u003e\u003d2.6           # ansible"},{"line_number":9,"context_line":"PyYAML\u003e\u003d3.1.0           # ansible"},{"line_number":10,"context_line":"requests\u003e\u003d2.2.0,!\u003d2.4.0 # scripts/os-ansible-role-requirements.py"},{"line_number":11,"context_line":"wheel"}],"source_content_type":"text/plain","patch_set":6,"id":"da9b358b_31c5fc8d","line":11,"updated":"2015-03-30 12:19:53.000000000","message":"It\u0027d probably be good to know whether this is required only for the building or the consumption of the python wheels. I suspect it\u0027s only for consumption.","commit_id":"c268d8f59fa5092c133165baa5e2d8102d50d8f9"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"5a72bc08e12574f560f5ec4786c041cf5505392f","unresolved":false,"context_lines":[{"line_number":8,"context_line":"pycrypto\u003e\u003d2.6           # ansible"},{"line_number":9,"context_line":"PyYAML\u003e\u003d3.1.0           # ansible"},{"line_number":10,"context_line":"requests\u003e\u003d2.2.0,!\u003d2.4.0 # scripts/os-ansible-role-requirements.py"},{"line_number":11,"context_line":"wheel"}],"source_content_type":"text/plain","patch_set":6,"id":"da9b358b_8fa20aad","line":11,"in_reply_to":"da9b358b_31c5fc8d","updated":"2015-04-01 07:27:51.000000000","message":"The only container that I see w/ wheel installed is repo_container.  As wheel is listed in the top level requirements.txt, it gets installed on the physical hosts also.  The repo_container containers seem to install python packages from pypi, so I\u0027m not entirely sure it\u0027s necessary we list this here.  I\u0027ll have to check w/ @cloudnull to confirm.","commit_id":"c268d8f59fa5092c133165baa5e2d8102d50d8f9"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"5e75631458c1b8e07750a434e40ac64c72c01e4d","unresolved":false,"context_lines":[{"line_number":8,"context_line":"pycrypto\u003e\u003d2.6           # ansible"},{"line_number":9,"context_line":"PyYAML\u003e\u003d3.1.0           # ansible"},{"line_number":10,"context_line":"requests\u003e\u003d2.2.0,!\u003d2.4.0 # scripts/os-ansible-role-requirements.py"},{"line_number":11,"context_line":"wheel"}],"source_content_type":"text/plain","patch_set":6,"id":"baa041b7_00bc066b","line":11,"in_reply_to":"da9b358b_8fa20aad","updated":"2015-04-01 13:40:54.000000000","message":"Checked w/ @cloudnull, this was a requirement of pip \u003c 6.0 if you were using pip to install wheels.  As we have bumped pip to \u003e\u003d 6.0 this is no longer required.","commit_id":"c268d8f59fa5092c133165baa5e2d8102d50d8f9"}]}