)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"fae39d8b9477e9ae9d031aee41f60d9dac7c96be","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Always install Ansible with pip"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"instaling and using Ansible from source for bifrost has several"},{"line_number":10,"context_line":"drawbacks, mainly due to how Ansible\u0027s \u0027ansible/hacking/env-setup\u0027"},{"line_number":11,"context_line":"script mangles with PATH and PYTHONPATH, which complicates running it as"},{"line_number":12,"context_line":"part of other scripts. Besides, cloning the whole repo and its"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"fa31d9ce_43756af7","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":9},"updated":"2017-02-10 20:38:32.000000000","message":"Installing","commit_id":"b24ce2199f9eb69d28f8ba013e1046f552e782e9"},{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"fae39d8b9477e9ae9d031aee41f60d9dac7c96be","unresolved":false,"context_lines":[{"line_number":9,"context_line":"instaling and using Ansible from source for bifrost has several"},{"line_number":10,"context_line":"drawbacks, mainly due to how Ansible\u0027s \u0027ansible/hacking/env-setup\u0027"},{"line_number":11,"context_line":"script mangles with PATH and PYTHONPATH, which complicates running it as"},{"line_number":12,"context_line":"part of other scripts. Besides, cloning the whole repo and its"},{"line_number":13,"context_line":"submodules is somewhat longer."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The main reason why we were doing that at all was a necessity to install"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"fa31d9ce_a35aa681","line":12,"range":{"start_line":12,"start_character":59,"end_line":12,"end_character":62},"updated":"2017-02-10 20:38:32.000000000","message":"it\u0027s","commit_id":"b24ce2199f9eb69d28f8ba013e1046f552e782e9"}],"README.rst":[{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"26dd252237744fcf903a71e6281d8c987370118b","unresolved":false,"context_lines":[{"line_number":136,"context_line":"install the library requirements. Alternatively the ``env-setup.sh`` script"},{"line_number":137,"context_line":"will install ansible and all of bifrost\u0027s dependencies."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"If you use ``env-setup.sh``, you can select the target folder using the"},{"line_number":140,"context_line":"``ANSIBLE_INSTALL_ROOT`` environment variable, otherwise the default of"},{"line_number":141,"context_line":"``/opt/stack`` will be utilized."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"Note:"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"  Only ansible installation location will be moved as part of the"},{"line_number":146,"context_line":"  environment variable.  The other components will continue to be cloned under"},{"line_number":147,"context_line":"  ``/opt/stack``"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"If using the environment setup script::"},{"line_number":150,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"fa31d9ce_d7bbdc70","line":147,"range":{"start_line":139,"start_character":0,"end_line":147,"end_character":16},"updated":"2017-02-10 18:49:14.000000000","message":"that\u0027s not how it works in this patch - ANSIBLE_INSTALL_ROOT is only used to create a fake ansible/hacking/env-setup for those third-party scripts that still try to source it, as Ansible will be attempted to be installed into the system anyway.","commit_id":"1ed566a606cb3ddd20e4baeb6101c3ace9fa05b8"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"be067e1d7ea6e0fc7cad82776cc58f7a5c1f8cf2","unresolved":false,"context_lines":[{"line_number":136,"context_line":"install the library requirements. Alternatively the ``env-setup.sh`` script"},{"line_number":137,"context_line":"will install ansible and all of bifrost\u0027s dependencies."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"If you use ``env-setup.sh``, you can select the target folder using the"},{"line_number":140,"context_line":"``ANSIBLE_INSTALL_ROOT`` environment variable, otherwise the default of"},{"line_number":141,"context_line":"``/opt/stack`` will be utilized."},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"Note:"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"  Only ansible installation location will be moved as part of the"},{"line_number":146,"context_line":"  environment variable.  The other components will continue to be cloned under"},{"line_number":147,"context_line":"  ``/opt/stack``"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"If using the environment setup script::"},{"line_number":150,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"fa31d9ce_3a789ff2","line":147,"range":{"start_line":139,"start_character":0,"end_line":147,"end_character":16},"in_reply_to":"fa31d9ce_d7bbdc70","updated":"2017-02-10 19:17:33.000000000","message":"Doh! I got so focused on the text that I completely forgot :(","commit_id":"1ed566a606cb3ddd20e4baeb6101c3ace9fa05b8"},{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"fae39d8b9477e9ae9d031aee41f60d9dac7c96be","unresolved":false,"context_lines":[{"line_number":154,"context_line":""},{"line_number":155,"context_line":"Otherwise::"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"  pip install -r requirements.txt"},{"line_number":158,"context_line":"  cd playbooks"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"The second part is an Ansible playbook that installs and configures ironic"}],"source_content_type":"text/x-rst","patch_set":13,"id":"fa31d9ce_63798efc","line":157,"updated":"2017-02-10 20:38:32.000000000","message":"Are we assuming the user already has sudoer privileges or do we need to change the permissions for requirements.txt? We should include a note if that\u0027s the case.","commit_id":"b24ce2199f9eb69d28f8ba013e1046f552e782e9"}],"releasenotes/notes/ansible-pip-install-2b66bd82ce9ed4f6.yaml":[{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"fae39d8b9477e9ae9d031aee41f60d9dac7c96be","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Bifrost has been changed to prefer use of a system installed Ansible"},{"line_number":5,"context_line":"    that has been installed. When this is the case, execution of the"},{"line_number":6,"context_line":"    ``env-setup.sh`` script is not required as it is geared for development"},{"line_number":7,"context_line":"    and testing use of bifrost."},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"fa31d9ce_c35ddaea","line":5,"range":{"start_line":4,"start_character":32,"end_line":5,"end_character":27},"updated":"2017-02-10 20:38:32.000000000","message":"give preference to the use of a system that comes with an installation of Ansible.","commit_id":"b24ce2199f9eb69d28f8ba013e1046f552e782e9"},{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"fae39d8b9477e9ae9d031aee41f60d9dac7c96be","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    In order to use the playbooks with a system installed Ansible,"},{"line_number":10,"context_line":"    the library requirements must be installed prior to playbook"},{"line_number":11,"context_line":"    execution.:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"        ``pip install -r requirements.txt``"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"fa31d9ce_2304f6be","line":11,"range":{"start_line":11,"start_character":13,"end_line":11,"end_character":15},"updated":"2017-02-10 20:38:32.000000000","message":"You can lose the unnecessary period.","commit_id":"b24ce2199f9eb69d28f8ba013e1046f552e782e9"},{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"fae39d8b9477e9ae9d031aee41f60d9dac7c96be","unresolved":false,"context_lines":[{"line_number":10,"context_line":"    the library requirements must be installed prior to playbook"},{"line_number":11,"context_line":"    execution.:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"        ``pip install -r requirements.txt``"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"  - |"},{"line_number":16,"context_line":"    The environment setup script will now attempt to install bifrost from PyPI"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"fa31d9ce_63d34e39","line":13,"range":{"start_line":13,"start_character":10,"end_line":13,"end_character":13},"updated":"2017-02-10 20:38:32.000000000","message":"Are we assuming the user already has sudoer privileges or do we need to change the permissions for requirements.txt?","commit_id":"b24ce2199f9eb69d28f8ba013e1046f552e782e9"},{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"fae39d8b9477e9ae9d031aee41f60d9dac7c96be","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    To install the Ansible 2.1 series as part of the env-setup script,"},{"line_number":30,"context_line":"    execute ``env ANSIBLE_GIT_BRANCH\u003d\"stable-2.1\" scripts/env-setup``."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    Simillarlly, ``ANSIBLE_PIP_VERSION`` can be utilized to specifiy"},{"line_number":33,"context_line":"    the exact version, or range of version desired. Example:"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"        ``ANSIBLE_PIP_VERSION\u003d2.1.0.1`` or ``ANSIBLE_PIP_VERSION\u003d\u003c2.2``"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"fa31d9ce_638fce23","line":32,"range":{"start_line":32,"start_character":4,"end_line":32,"end_character":15},"updated":"2017-02-10 20:38:32.000000000","message":"Similarly,","commit_id":"b24ce2199f9eb69d28f8ba013e1046f552e782e9"},{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"fae39d8b9477e9ae9d031aee41f60d9dac7c96be","unresolved":false,"context_lines":[{"line_number":38,"context_line":"  - |"},{"line_number":39,"context_line":"    If installing bifrost in a virtualenv (venv) and running playbooks"},{"line_number":40,"context_line":"    against localhost, you must install the basic python requirements"},{"line_number":41,"context_line":"    on a system-wide level due to the operationg behavior of Ansible."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"fixes:"},{"line_number":44,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"fa31d9ce_43c2ca34","line":41,"range":{"start_line":41,"start_character":38,"end_line":41,"end_character":48},"updated":"2017-02-10 20:38:32.000000000","message":"operating","commit_id":"b24ce2199f9eb69d28f8ba013e1046f552e782e9"},{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"ee721c5cce4018e86d10df5d155b9f8e97bdd0c7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Bifrost has been changed to preference to the use a system that"},{"line_number":5,"context_line":"    already has Ansible installed. When this is the case, execution of the"},{"line_number":6,"context_line":"    ``env-setup.sh`` script is not required as it is geared for development"},{"line_number":7,"context_line":"    and testing use of bifrost."}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_5455f81a","line":4,"range":{"start_line":4,"start_character":21,"end_line":4,"end_character":28},"updated":"2017-02-13 10:34:45.000000000","message":"i\u0027m not an native english speaker, but this sentence looks very strange to me. Maybe \"Bifrost now prefers to use a system with Ansible already installed\", or something around these lines?","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"3b74d939e59ab41a10f7517485d3504559650a69","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Bifrost has been changed to preference to the use a system that"},{"line_number":5,"context_line":"    already has Ansible installed. When this is the case, execution of the"},{"line_number":6,"context_line":"    ``env-setup.sh`` script is not required as it is geared for development"},{"line_number":7,"context_line":"    and testing use of bifrost."}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_424b76c9","line":4,"range":{"start_line":4,"start_character":21,"end_line":4,"end_character":28},"in_reply_to":"fa31d9ce_5455f81a","updated":"2017-02-13 11:10:48.000000000","message":"Done","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"ee721c5cce4018e86d10df5d155b9f8e97bdd0c7","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    ``env-setup.sh`` script is not required as it is geared for development"},{"line_number":7,"context_line":"    and testing use of bifrost."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    In order to use the playbooks with a system installed Ansible,"},{"line_number":10,"context_line":"    the library requirements must be installed prior to playbook"},{"line_number":11,"context_line":"    execution:"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_d43f28d4","line":9,"range":{"start_line":9,"start_character":34,"end_line":9,"end_character":38},"updated":"2017-02-13 10:34:45.000000000","message":"on a system with Ansible already installed?","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"3b74d939e59ab41a10f7517485d3504559650a69","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    ``env-setup.sh`` script is not required as it is geared for development"},{"line_number":7,"context_line":"    and testing use of bifrost."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    In order to use the playbooks with a system installed Ansible,"},{"line_number":10,"context_line":"    the library requirements must be installed prior to playbook"},{"line_number":11,"context_line":"    execution:"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_8213aeb7","line":9,"range":{"start_line":9,"start_character":34,"end_line":9,"end_character":38},"in_reply_to":"fa31d9ce_d43f28d4","updated":"2017-02-13 11:10:48.000000000","message":"Done","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"ee721c5cce4018e86d10df5d155b9f8e97bdd0c7","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"        ``pip install -r requirements.txt``"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    Administrative privilges may be required if the packages"},{"line_number":16,"context_line":"    must be installed system wide."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_1422a0b9","line":15,"range":{"start_line":15,"start_character":19,"end_line":15,"end_character":28},"updated":"2017-02-13 10:34:45.000000000","message":"privileges","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"3b74d939e59ab41a10f7517485d3504559650a69","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"        ``pip install -r requirements.txt``"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    Administrative privilges may be required if the packages"},{"line_number":16,"context_line":"    must be installed system wide."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_026fbe2b","line":15,"range":{"start_line":15,"start_character":19,"end_line":15,"end_character":28},"in_reply_to":"fa31d9ce_1422a0b9","updated":"2017-02-13 11:10:48.000000000","message":"Done","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"3b74d939e59ab41a10f7517485d3504559650a69","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    To use such installed Ansible, modifications of ``$PATH``"},{"line_number":27,"context_line":"    environment variable might be required to include ``~/.local/bin`` path."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    Some backwards compatability is provided via the use of the"},{"line_number":30,"context_line":"    ``ANSIBLE_GIT_BRANCH`` variable, where a user can define ``stable-X.Y``"},{"line_number":31,"context_line":"    and the latest available version in that series will be installed."},{"line_number":32,"context_line":"    To install the Ansible 2.1 series as part of the env-setup script,"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_c284c6d5","line":29,"range":{"start_line":29,"start_character":19,"end_line":29,"end_character":32},"updated":"2017-02-13 11:10:48.000000000","message":"yet another typo - compatibility","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"ee721c5cce4018e86d10df5d155b9f8e97bdd0c7","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    To install the Ansible 2.1 series as part of the env-setup script,"},{"line_number":33,"context_line":"    execute ``env ANSIBLE_GIT_BRANCH\u003d\"stable-2.1\" scripts/env-setup``."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    Similarlly, ``ANSIBLE_PIP_VERSION`` can be utilized to specifiy"},{"line_number":36,"context_line":"    the exact version, or range of version desired. Example:"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        ``ANSIBLE_PIP_VERSION\u003d2.1.0.1`` or ``ANSIBLE_PIP_VERSION\u003d\u003c2.2``"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_f7142e45","line":35,"range":{"start_line":35,"start_character":4,"end_line":35,"end_character":14},"updated":"2017-02-13 10:34:45.000000000","message":"Similarly","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"3b74d939e59ab41a10f7517485d3504559650a69","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    To install the Ansible 2.1 series as part of the env-setup script,"},{"line_number":33,"context_line":"    execute ``env ANSIBLE_GIT_BRANCH\u003d\"stable-2.1\" scripts/env-setup``."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    Similarlly, ``ANSIBLE_PIP_VERSION`` can be utilized to specifiy"},{"line_number":36,"context_line":"    the exact version, or range of version desired. Example:"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        ``ANSIBLE_PIP_VERSION\u003d2.1.0.1`` or ``ANSIBLE_PIP_VERSION\u003d\u003c2.2``"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_02ddfedf","line":35,"range":{"start_line":35,"start_character":59,"end_line":35,"end_character":67},"updated":"2017-02-13 11:10:48.000000000","message":"another typo - specify","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"3b74d939e59ab41a10f7517485d3504559650a69","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    To install the Ansible 2.1 series as part of the env-setup script,"},{"line_number":33,"context_line":"    execute ``env ANSIBLE_GIT_BRANCH\u003d\"stable-2.1\" scripts/env-setup``."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    Similarlly, ``ANSIBLE_PIP_VERSION`` can be utilized to specifiy"},{"line_number":36,"context_line":"    the exact version, or range of version desired. Example:"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        ``ANSIBLE_PIP_VERSION\u003d2.1.0.1`` or ``ANSIBLE_PIP_VERSION\u003d\u003c2.2``"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"fa31d9ce_22ff6235","line":35,"range":{"start_line":35,"start_character":4,"end_line":35,"end_character":14},"in_reply_to":"fa31d9ce_f7142e45","updated":"2017-02-13 11:10:48.000000000","message":"Done","commit_id":"1df4e0592a9ed902d24a8bdd79ef978bd86b3107"}],"scripts/ansible-pip-str.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"0dfb3d909dbc2e49907d5d2e2bf39dcf09394d32","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/usr/bin/env python"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"\"\"\"Helper script to choose which ansible version to install for bifrost\"\"\""},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"from __future__ import print_function"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa31d9ce_7a41450b","line":2,"updated":"2017-02-10 13:54:05.000000000","message":"nit: license header","commit_id":"933e0989f81a56f7ab8557d552460ca006cefb4f"}],"scripts/env-setup.sh":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"0dfb3d909dbc2e49907d5d2e2bf39dcf09394d32","unresolved":false,"context_lines":[{"line_number":57,"context_line":"fi"},{"line_number":58,"context_line":"sudo -H chown -R $u:$g ${ANSIBLE_INSTALL_ROOT}"},{"line_number":59,"context_line":"mkdir -p ${ANSIBLE_INSTALL_ROOT}/ansible/hacking"},{"line_number":60,"context_line":"echo \"echo Sourcing this file is no longer needed! Ansible is always installed from PyPI\" \u003e ${ANSIBLE_INSTALL_ROOT}/ansible/hacking/env-setup"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"echo"},{"line_number":63,"context_line":"echo \"To use bifrost, do\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"fa31d9ce_faf1b5ce","line":60,"range":{"start_line":60,"start_character":5,"end_line":60,"end_character":10},"updated":"2017-02-10 13:54:05.000000000","message":"Heh, I guess this makes sense. :)\nI think we ought to add instructions because system wide can complicate things with different versions and different comparabilities.  Part of the reason we did our own hacking based install initially.  That or stress virtualenv use.","commit_id":"933e0989f81a56f7ab8557d552460ca006cefb4f"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"066cdbeddc7d69dfbf519a397606d5dfb970dfd5","unresolved":false,"context_lines":[{"line_number":57,"context_line":"fi"},{"line_number":58,"context_line":"sudo -H chown -R $u:$g ${ANSIBLE_INSTALL_ROOT}"},{"line_number":59,"context_line":"mkdir -p ${ANSIBLE_INSTALL_ROOT}/ansible/hacking"},{"line_number":60,"context_line":"echo \"echo Sourcing this file is no longer needed! Ansible is always installed from PyPI\" \u003e ${ANSIBLE_INSTALL_ROOT}/ansible/hacking/env-setup"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"echo"},{"line_number":63,"context_line":"echo \"To use bifrost, do\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"fa31d9ce_3a4a6d84","line":60,"range":{"start_line":60,"start_character":5,"end_line":60,"end_character":10},"in_reply_to":"fa31d9ce_faf1b5ce","updated":"2017-02-10 14:04:01.000000000","message":"This particular place was added specifically to pass OPNFV CI as they use their own version of test-bifrost.sh that is trying to source this file.\n\nAlso it appears our testing was actually relying on /opt/stack being pre-created before (prep-install was trying to ensure access to it w/o become)\n\nI am actually planning to add a patch / rework this one to *always* pip-install ansible itself to a venv (regardless of use_venv) - for example not to clobber an existing Ansible installation :) especially when running bifrost against remote target from existing mgmt machine (actually happened to me while testing this change :) )","commit_id":"933e0989f81a56f7ab8557d552460ca006cefb4f"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"d2ae4c359206f5fcffeb97f4a9bebd052a5a7c4d","unresolved":false,"context_lines":[{"line_number":58,"context_line":"sudo -H chown -R $u:$g ${ANSIBLE_INSTALL_ROOT}"},{"line_number":59,"context_line":"mkdir -p ${ANSIBLE_INSTALL_ROOT}/ansible/hacking"},{"line_number":60,"context_line":"echo \"echo Sourcing this file is no longer needed! Ansible is always installed from PyPI\" \u003e ${ANSIBLE_INSTALL_ROOT}/ansible/hacking/env-setup"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"echo"},{"line_number":63,"context_line":"echo \"To use bifrost, do\""},{"line_number":64,"context_line":""}],"source_content_type":"text/x-sh","patch_set":11,"id":"fa31d9ce_3aab3f1f","line":61,"updated":"2017-02-10 19:18:04.000000000","message":"perhaps we should exit 1?","commit_id":"1ed566a606cb3ddd20e4baeb6101c3ace9fa05b8"}]}