)]}'
{"elements/proliant-tools/install.d/65-proliant-tools-install":[{"author":{"_account_id":15074,"name":"Aparna","email":"aparnavtce@gmail.com","username":"Aparna"},"change_message_id":"bddfde989253b7b0375b491de33ff878ef238486","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Install proliantutils python module"},{"line_number":38,"context_line":"IPA_VENV\u003d/usr/share/ironic-python-agent/venv"},{"line_number":39,"context_line":"if [[ -d \"$IPA_VENV\" ]]; then"},{"line_number":40,"context_line":"    $IPA_VENV/bin/pip install proliantutils"},{"line_number":41,"context_line":"else"},{"line_number":42,"context_line":"    pip install proliantutils"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5a18252c_204a1184","line":39,"range":{"start_line":39,"start_character":0,"end_line":39,"end_character":2},"updated":"2016-04-12 12:29:52.000000000","message":"error in the syntax, \u0027if\u0027 is not closed.","commit_id":"b10367667269ad3a55b2054259add4ac75f08c54"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"802da31c4b4650bdca5565f819c59a1dd494d33e","unresolved":false,"context_lines":[{"line_number":35,"context_line":"rm -f /tmp/hpssacli.rpm"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Install proliantutils python module"},{"line_number":38,"context_line":"IPA_VENV\u003d/usr/share/ironic-python-agent/venv"},{"line_number":39,"context_line":"if [[ -d \"$IPA_VENV\" ]]; then"},{"line_number":40,"context_line":"    $IPA_VENV/bin/pip install proliantutils"},{"line_number":41,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"1a122d0e_e217afa0","line":38,"updated":"2016-04-21 05:08:20.000000000","message":"I dunno about this ... hidden dependencies and different behaviour based on what\u0027s included.  That said it\u0027s probably not worth re-engineering too much for this.  \n\nThus I\u0027d just like to see a comment here that this is pre-installed by 60-ironic-agent-install and that we want it there mostly for that, so at least in the future we know what\u0027s going on.\n\nI can be considered +2 after that","commit_id":"ca3752be50571fcf8723720052f892ecdd4f68f0"},{"author":{"_account_id":11297,"name":"Nisha Agarwal","email":"agarwalnisha1980@gmail.com","username":"Nisha"},"change_message_id":"442e5bb6c95a4805c7fdaab024829db0942c4fd4","unresolved":false,"context_lines":[{"line_number":35,"context_line":"rm -f /tmp/hpssacli.rpm"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Install proliantutils python module"},{"line_number":38,"context_line":"IPA_VENV\u003d/usr/share/ironic-python-agent/venv"},{"line_number":39,"context_line":"if [[ -d \"$IPA_VENV\" ]]; then"},{"line_number":40,"context_line":"    $IPA_VENV/bin/pip install proliantutils"},{"line_number":41,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"1a122d0e_5ae29f02","line":38,"in_reply_to":"1a122d0e_e217afa0","updated":"2016-05-02 08:29:38.000000000","message":"ok","commit_id":"ca3752be50571fcf8723720052f892ecdd4f68f0"}]}