)]}'
{"playbooks/defaults/source_install.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"5d7353a8affb59cdbd7f141cbed4e956e9db9ad3","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"cinder_bin: \"/openstack/venvs/cinder-{{ venv_tag }}/bin\""},{"line_number":17,"context_line":"glance_bin: \"/openstack/venvs/glance-{{ venv_tag }}/bin\""},{"line_number":18,"context_line":"keystone_bin: \"/openstack/venvs/keystone-{{ venv_tag }}/bin\""},{"line_number":19,"context_line":"nova_bin: \"/openstack/venvs/nova-{{ venv_tag }}/bin\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3f79a3b5_9907661e","line":16,"range":{"start_line":16,"start_character":40,"end_line":16,"end_character":48},"updated":"2018-09-03 11:50:40.000000000","message":"nit: as this becomes common, wouldn\u0027t it be wise to use directly openstack_release, and also change the few plays still using venv_tag?","commit_id":"1409c3ad9ddff714a8d1f42639106ee96367b288"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"d152bdc7fdedd7de887f69a2daeba0a02393e88d","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"cinder_bin: \"/openstack/venvs/cinder-{{ venv_tag }}/bin\""},{"line_number":17,"context_line":"glance_bin: \"/openstack/venvs/glance-{{ venv_tag }}/bin\""},{"line_number":18,"context_line":"keystone_bin: \"/openstack/venvs/keystone-{{ venv_tag }}/bin\""},{"line_number":19,"context_line":"nova_bin: \"/openstack/venvs/nova-{{ venv_tag }}/bin\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3f79a3b5_1cd5841b","line":16,"range":{"start_line":16,"start_character":40,"end_line":16,"end_character":48},"in_reply_to":"3f79a3b5_9907661e","updated":"2018-09-03 12:07:56.000000000","message":"Yep, I kinda started doing it, then realised I was turning everything into megapatches with too many changes mixed in.\n\nI actually think that instead of using openstack_release, we can just remove the use of the tag for the venv path. It would be far simpler to just have the venv be /openstack/cinder, for example. We have no need to rebuild the venv for every release.","commit_id":"1409c3ad9ddff714a8d1f42639106ee96367b288"}]}