)]}'
{"scripts/localrc.rc":[{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"4b9787936b507c8ed63048b177985ad40385bf85","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    disable_service n-net"},{"line_number":20,"context_line":"fi"},{"line_number":21,"context_line":"# OSprofiler depends on Ceilometer"},{"line_number":22,"context_line":"if [[ $ENABLE_CEILOMETER \u003d true ]] || [[ $ENABLE_PROFILER \u003d true ]]; then"},{"line_number":23,"context_line":"    CEILOMETER_BACKEND\u003dmysql"},{"line_number":24,"context_line":"    CEILOMETER_NOTIFICATION_TOPICS\u003dnotifications,profiler"},{"line_number":25,"context_line":"    ENABLED_SERVICES+\u003d,ceilometer-acompute,ceilometer-acentral"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"da9b358b_49ce47b5","line":22,"updated":"2015-04-01 00:37:41.000000000","message":"Using || would make this statement true if one of the conditions is true. If OSProfiler depends on Ceilometer and OSProfiler is enabled but Ceilometer is not, this will fail. We should have an \u0026\u0026 instead.","commit_id":"9be09ef8c71de31325a8a1f1bb4948797c6380e0"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"b2ba84d76aa51eeb62173e75b6b4ceb0ce877f62","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    disable_service n-net"},{"line_number":20,"context_line":"fi"},{"line_number":21,"context_line":"# OSprofiler depends on Ceilometer"},{"line_number":22,"context_line":"if [[ $ENABLE_CEILOMETER \u003d true ]] || [[ $ENABLE_PROFILER \u003d true ]]; then"},{"line_number":23,"context_line":"    CEILOMETER_BACKEND\u003dmysql"},{"line_number":24,"context_line":"    CEILOMETER_NOTIFICATION_TOPICS\u003dnotifications,profiler"},{"line_number":25,"context_line":"    ENABLED_SERVICES+\u003d,ceilometer-acompute,ceilometer-acentral"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"da9b358b_4c1095a8","line":22,"in_reply_to":"da9b358b_49ce47b5","updated":"2015-04-01 01:54:07.000000000","message":"If either one is set, we need to activate Ceilometer, hence the \u0027or\u0027:\n\n  ubuntu:~/work\u003e CEIL\u003dtrue  PROF\u003dtrue;  if [[ $PROF \u003d true ]] || [[ $CEIL \u003d true ]]; then echo \"Ceil active\"; else echo \"Skip Ceil\"; fi\n  Ceil active\n  ubuntu:~/work\u003e CEIL\u003dtrue  PROF\u003dfalse; if [[ $PROF \u003d true ]] || [[ $CEIL \u003d true ]]; then echo \"Ceil active\"; else echo \"Skip Ceil\"; fi\n  Ceil active\n  ubuntu:~/work\u003e CEIL\u003dfalse PROF\u003dtrue;  if [[ $PROF \u003d true ]] || [[ $CEIL \u003d true ]]; then echo \"Ceil active\"; else echo \"Skip Ceil\"; fi\n  Ceil active\n  ubuntu:~/work\u003e CEIL\u003dfalse PROF\u003dfalse; if [[ $PROF \u003d true ]] || [[ $CEIL \u003d true ]]; then echo \"Ceil active\"; else echo \"Skip Ceil\"; fi\n  Skip Ceil\n\nThe code block will only be skipped if both variables are set to \u0027false\u0027 (i.e. not set to \u0027true\u0027)","commit_id":"9be09ef8c71de31325a8a1f1bb4948797c6380e0"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"43a50a0d3c1faac6f79d6e5386c4055f77161d15","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    disable_service n-net"},{"line_number":20,"context_line":"fi"},{"line_number":21,"context_line":"# OSprofiler depends on Ceilometer"},{"line_number":22,"context_line":"if [[ $ENABLE_CEILOMETER \u003d true ]] || [[ $ENABLE_PROFILER \u003d true ]]; then"},{"line_number":23,"context_line":"    CEILOMETER_BACKEND\u003dmysql"},{"line_number":24,"context_line":"    CEILOMETER_NOTIFICATION_TOPICS\u003dnotifications,profiler"},{"line_number":25,"context_line":"    ENABLED_SERVICES+\u003d,ceilometer-acompute,ceilometer-acentral"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"da9b358b_cc7dc5b2","line":22,"in_reply_to":"da9b358b_4c1095a8","updated":"2015-04-01 02:02:28.000000000","message":"I see what you mean. Thanks!","commit_id":"9be09ef8c71de31325a8a1f1bb4948797c6380e0"}],"scripts/redstack.rc":[{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"d7b767726fee6c95c5386711b07f86d5a3bd9f9e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"# Enable ceilometer"},{"line_number":12,"context_line":"ENABLE_CEILOMETER\u003d$(get_bool ENABLE_CEILOMETER false)"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"# Enable osprofiler - note: Enables Ceilometer as well"},{"line_number":15,"context_line":"ENABLE_PROFILER\u003d$(get_bool ENABLE_PROFILER false)"},{"line_number":16,"context_line":"PROFILER_TRACE_SQL\u003d$(get_bool PROFILER_TRACE_SQL false)"},{"line_number":17,"context_line":"[ -z $PROFILER_HMAC_KEYS ] \u0026\u0026 PROFILER_HMAC_KEYS\u003dSECRET_KEY"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"9a80dd14_4cca8f6f","line":14,"updated":"2015-03-11 04:45:12.000000000","message":"Please use the case osprofiler as earlier\n\"OSprofiler\"","commit_id":"9be09ef8c71de31325a8a1f1bb4948797c6380e0"}]}
