)]}'
{"devstack/lib/freezer-api":[{"author":{"_account_id":14101,"name":"yapeng Yang","email":"a394954369@gmail.com","username":"yapeng"},"change_message_id":"3bda4090befd501846159cc989b1c99e735dfc26","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    sudo chown $USER $FREEZER_API_LOG_DIR"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    sudo cp $FREEZER_API_DIR/etc/freezer-api.conf.sample $FREEZER_API_CONF_DIR/freezer-api.conf"},{"line_number":91,"context_line":"    sudo cp $FREEZER_API_DIR/etc/freezer-paste.ini $FREEZER_API_CONF_DIR/freezer-paste.ini"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    # set paste configuration"},{"line_number":94,"context_line":"    iniset $FREEZER_API_CONF \u0027paste_deploy\u0027 config_file $FREEZER_API_CONF_DIR/freezer-paste.ini"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"bab6814e_409b7318","line":91,"range":{"start_line":91,"start_character":9,"end_line":91,"end_character":11},"updated":"2016-05-26 01:39:49.000000000","message":"if we shoud  `sudo chown $USER $FREEZER_API_CONF_DIR` follow it .","commit_id":"2aa86103fc6b3999efd9925480bfff102b5f296b"},{"author":{"_account_id":13940,"name":"Saad Zaher","email":"eng.szaher@gmail.com","username":"szaher"},"change_message_id":"231453c1c39b6e764055729192048f8e6cf30e29","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    sudo chown $USER $FREEZER_API_LOG_DIR"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    sudo cp $FREEZER_API_DIR/etc/freezer-api.conf.sample $FREEZER_API_CONF_DIR/freezer-api.conf"},{"line_number":91,"context_line":"    sudo cp $FREEZER_API_DIR/etc/freezer-paste.ini $FREEZER_API_CONF_DIR/freezer-paste.ini"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    # set paste configuration"},{"line_number":94,"context_line":"    iniset $FREEZER_API_CONF \u0027paste_deploy\u0027 config_file $FREEZER_API_CONF_DIR/freezer-paste.ini"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"bab6814e_1c3ee18a","line":91,"range":{"start_line":91,"start_character":9,"end_line":91,"end_character":11},"in_reply_to":"bab6814e_409b7318","updated":"2016-05-26 11:17:53.000000000","message":"I\u0027m not sure about that !","commit_id":"2aa86103fc6b3999efd9925480bfff102b5f296b"},{"author":{"_account_id":14101,"name":"yapeng Yang","email":"a394954369@gmail.com","username":"yapeng"},"change_message_id":"f9e55e4ee8341c8d4c0d2610c18adf0899de6b8e","unresolved":false,"context_lines":[{"line_number":82,"context_line":"function configure_freezer_api {"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    [ ! -d $FREEZER_API_CONF_DIR ] \u0026\u0026 sudo mkdir -m 755 -p $FREEZER_API_CONF_DIR"},{"line_number":85,"context_line":"    sudo chown $USER $FREEZER_API_CONF_DIR"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    [ ! -d $FREEZER_API_LOG_DIR ] \u0026\u0026  sudo mkdir -m 755 -p $FREEZER_API_LOG_DIR"},{"line_number":88,"context_line":"    sudo chown $USER $FREEZER_API_LOG_DIR"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"bab6814e_31ce662d","line":85,"range":{"start_line":85,"start_character":20,"end_line":85,"end_character":21},"updated":"2016-05-26 13:28:53.000000000","message":"this shoud be delete","commit_id":"59123aa10a646bc6f6d4bfe660023d1d4e1966ab"},{"author":{"_account_id":14101,"name":"yapeng Yang","email":"a394954369@gmail.com","username":"yapeng"},"change_message_id":"f9e55e4ee8341c8d4c0d2610c18adf0899de6b8e","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    iniset $FREEZER_API_CONF \u0027paste_deploy\u0027 config_file $FREEZER_API_CONF_DIR/freezer-paste.ini"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # make sure the stack user has the right permissions on the config folder"},{"line_number":97,"context_line":"    sudo chown $USER $FREEZER_API_CONF_DIR"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    #set elasticsearch configuration"},{"line_number":100,"context_line":"    iniset $FREEZER_API_CONF \u0027storage\u0027 db elasticsearch"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"bab6814e_71558e4e","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":20},"updated":"2016-05-26 13:28:53.000000000","message":"We shoud the above the same,USER change STACKUSER can be better.,","commit_id":"59123aa10a646bc6f6d4bfe660023d1d4e1966ab"},{"author":{"_account_id":14084,"name":"Einst Crazy","email":"yu.changcai@99cloud.net","username":"Einst"},"change_message_id":"3c5da89d6d3b8b6968f3eacd493c61d3650ce291","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    iniset $FREEZER_API_CONF \u0027paste_deploy\u0027 config_file $FREEZER_API_CONF_DIR/freezer-paste.ini"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # make sure the stack user has the right permissions on the config folder"},{"line_number":97,"context_line":"    sudo chown $USER $FREEZER_API_CONF_DIR"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    #set elasticsearch configuration"},{"line_number":100,"context_line":"    iniset $FREEZER_API_CONF \u0027storage\u0027 db elasticsearch"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9abb7d3a_6fc1b280","line":97,"updated":"2016-05-27 16:37:54.000000000","message":"this line is duplicated with line 85","commit_id":"7269203e07ac6eabf753efd36f781d8fb2fd3a57"},{"author":{"_account_id":13940,"name":"Saad Zaher","email":"eng.szaher@gmail.com","username":"szaher"},"change_message_id":"2eef03361c3cf3a45ad3a63b4d2573822dd46fe3","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    iniset $FREEZER_API_CONF \u0027paste_deploy\u0027 config_file $FREEZER_API_CONF_DIR/freezer-paste.ini"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # make sure the stack user has the right permissions on the config folder"},{"line_number":97,"context_line":"    sudo chown $USER $FREEZER_API_CONF_DIR"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    #set elasticsearch configuration"},{"line_number":100,"context_line":"    iniset $FREEZER_API_CONF \u0027storage\u0027 db elasticsearch"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9abb7d3a_b5a61b51","line":97,"in_reply_to":"9abb7d3a_6fc1b280","updated":"2016-05-30 09:51:19.000000000","message":"We need this as we moved some files with the root user using sudo","commit_id":"7269203e07ac6eabf753efd36f781d8fb2fd3a57"},{"author":{"_account_id":14084,"name":"Einst Crazy","email":"yu.changcai@99cloud.net","username":"Einst"},"change_message_id":"70b9f5e19c2f10d5e59ed86f0cf93a04a67837ca","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    iniset $FREEZER_API_CONF \u0027paste_deploy\u0027 config_file $FREEZER_API_CONF_DIR/freezer-paste.ini"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # make sure the stack user has the right permissions on the config folder"},{"line_number":97,"context_line":"    sudo chown $USER $FREEZER_API_CONF_DIR"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    #set elasticsearch configuration"},{"line_number":100,"context_line":"    iniset $FREEZER_API_CONF \u0027storage\u0027 db elasticsearch"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9abb7d3a_f5f8824b","line":97,"in_reply_to":"9abb7d3a_b5a61b51","updated":"2016-05-31 01:43:33.000000000","message":"So we can delete the line 85? and need the args of -R?","commit_id":"7269203e07ac6eabf753efd36f781d8fb2fd3a57"},{"author":{"_account_id":13940,"name":"Saad Zaher","email":"eng.szaher@gmail.com","username":"szaher"},"change_message_id":"0a6fecf261eeb35a3c92df7e5ab77c7c24ebbc43","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    iniset $FREEZER_API_CONF \u0027paste_deploy\u0027 config_file $FREEZER_API_CONF_DIR/freezer-paste.ini"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # make sure the stack user has the right permissions on the config folder"},{"line_number":97,"context_line":"    sudo chown $USER $FREEZER_API_CONF_DIR"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    #set elasticsearch configuration"},{"line_number":100,"context_line":"    iniset $FREEZER_API_CONF \u0027storage\u0027 db elasticsearch"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9abb7d3a_9fa2e397","line":97,"in_reply_to":"9abb7d3a_f5f8824b","updated":"2016-05-31 09:26:24.000000000","message":"it will cause problems !!\nPlease, take a look at the previous replies and previous patches if you want !","commit_id":"7269203e07ac6eabf753efd36f781d8fb2fd3a57"}],"etc/freezer-api.conf.sample":[{"author":{"_account_id":14028,"name":"Jonas Pfannschmidt","email":"jonas.pfannschmidt@hp.com","username":"jonaspf"},"change_message_id":"69ce89cd8c4bb0c90c6883d3d0fcd91ed6ce2aea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bab6814e_167703a0","updated":"2016-05-25 17:10:50.000000000","message":"Since we have more than one config file now, shouldn\u0027t we move both files into one sub-directory? Similar to how it is done in nova and neutron?","commit_id":"2aa86103fc6b3999efd9925480bfff102b5f296b"},{"author":{"_account_id":13940,"name":"Saad Zaher","email":"eng.szaher@gmail.com","username":"szaher"},"change_message_id":"231453c1c39b6e764055729192048f8e6cf30e29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bab6814e_fecf9453","in_reply_to":"bab6814e_167703a0","updated":"2016-05-26 11:17:53.000000000","message":"Yes, It\u0027s a user issue freezer-api is ready for that. You can have /etc/freezer/freezer-api.conf and this will work fine and freezer will be able to load the configuration.","commit_id":"2aa86103fc6b3999efd9925480bfff102b5f296b"},{"author":{"_account_id":14101,"name":"yapeng Yang","email":"a394954369@gmail.com","username":"yapeng"},"change_message_id":"3bda4090befd501846159cc989b1c99e735dfc26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bab6814e_e021c74e","in_reply_to":"bab6814e_167703a0","updated":"2016-05-26 01:39:49.000000000","message":"yes,i agree, we shoud move  it to /etc/freezer/freezer-api.conf ,freezer-scheduler.conf .eg.","commit_id":"2aa86103fc6b3999efd9925480bfff102b5f296b"},{"author":{"_account_id":13940,"name":"Saad Zaher","email":"eng.szaher@gmail.com","username":"szaher"},"change_message_id":"231453c1c39b6e764055729192048f8e6cf30e29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bab6814e_fe8794b2","in_reply_to":"bab6814e_e021c74e","updated":"2016-05-26 11:17:53.000000000","message":"It\u0027s supported !","commit_id":"2aa86103fc6b3999efd9925480bfff102b5f296b"}],"freezer_api/cmd/wsgi.py":[{"author":{"_account_id":14028,"name":"Jonas Pfannschmidt","email":"jonas.pfannschmidt@hp.com","username":"jonaspf"},"change_message_id":"69ce89cd8c4bb0c90c6883d3d0fcd91ed6ce2aea","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    db \u003d driver.get_db()"},{"line_number":29,"context_line":"except Exception as err:"},{"line_number":30,"context_line":"    message \u003d \u0027Unable to start server: {0} \u0027.format(err)"},{"line_number":31,"context_line":"    _LOG.error(message)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"application \u003d get_application(db)"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_f3c41141","side":"PARENT","line":31,"range":{"start_line":31,"start_character":22,"end_line":31,"end_character":23},"updated":"2016-05-25 17:10:50.000000000","message":"Shouldn\u0027t we stop here, rather than trying to continue without db?","commit_id":"fc37320b92f76f9525433e96cab989b5f32aee3d"},{"author":{"_account_id":13940,"name":"Saad Zaher","email":"eng.szaher@gmail.com","username":"szaher"},"change_message_id":"231453c1c39b6e764055729192048f8e6cf30e29","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    db \u003d driver.get_db()"},{"line_number":29,"context_line":"except Exception as err:"},{"line_number":30,"context_line":"    message \u003d \u0027Unable to start server: {0} \u0027.format(err)"},{"line_number":31,"context_line":"    _LOG.error(message)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"application \u003d get_application(db)"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_3c948589","side":"PARENT","line":31,"range":{"start_line":31,"start_character":22,"end_line":31,"end_character":23},"in_reply_to":"bab6814e_f3c41141","updated":"2016-05-26 11:17:53.000000000","message":"The application checks if the db doesn\u0027t exist and will try to load it \ncheck freezer_api/cmd/api.py:get_application","commit_id":"fc37320b92f76f9525433e96cab989b5f32aee3d"}]}