)]}'
{"diskimage-create/diskimage-create.sh":[{"author":{"_account_id":12320,"name":"Pino Toscano","email":"ptoscano@redhat.com","username":"ptoscano"},"change_message_id":"ed769a5d3d74b71e0dffec58c12c02ba8e768012","unresolved":false,"context_lines":[{"line_number":615,"context_line":"    fi"},{"line_number":616,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos\" ]; then"},{"line_number":617,"context_line":"        ambari_centos_image_name\u003d${ambari_centos_image_name:-centos_sahara_ambari}"},{"line_number":618,"context_line":"        ambari_element_sequence\u003d\"ambari mysql redhat-lsb $JAVA_ELEMENT\""},{"line_number":619,"context_line":"        image_create centos $ambari_centos_image_name $ambari_element_sequence"},{"line_number":620,"context_line":"    fi"},{"line_number":621,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos7\" ]; then"}],"source_content_type":"text/x-sh","patch_set":4,"id":"7a2fa921_abce1db2","line":618,"updated":"2015-10-07 13:02:04.000000000","message":"Why is redhat-lsb needed? It drags in a lot of dependencies... are you really sure it is needed by the ambari guests?","commit_id":"7ec5891ede4bb37fafaeb21ee86adbea0e58dfaa"},{"author":{"_account_id":12320,"name":"Pino Toscano","email":"ptoscano@redhat.com","username":"ptoscano"},"change_message_id":"f04b3443b409e53ca7261b9bf4f5c19ab852dcc6","unresolved":false,"context_lines":[{"line_number":615,"context_line":"    fi"},{"line_number":616,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos\" ]; then"},{"line_number":617,"context_line":"        ambari_centos_image_name\u003d${ambari_centos_image_name:-centos_sahara_ambari}"},{"line_number":618,"context_line":"        ambari_element_sequence\u003d\"ambari mysql redhat-lsb $JAVA_ELEMENT\""},{"line_number":619,"context_line":"        image_create centos $ambari_centos_image_name $ambari_element_sequence"},{"line_number":620,"context_line":"    fi"},{"line_number":621,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos7\" ]; then"}],"source_content_type":"text/x-sh","patch_set":4,"id":"7a2fa921_1c92f99e","line":618,"in_reply_to":"7a2fa921_31559e1c","updated":"2015-10-07 13:40:54.000000000","message":"Again, I\u0027m asking *why* ambari needs them. And if it really needs them, why the ambari RPMs are not requiring it? Otherwise this patch seems just a workaround.","commit_id":"7ec5891ede4bb37fafaeb21ee86adbea0e58dfaa"},{"author":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"change_message_id":"ef553979d53a04e3b2010712638309a62be274e9","unresolved":false,"context_lines":[{"line_number":615,"context_line":"    fi"},{"line_number":616,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos\" ]; then"},{"line_number":617,"context_line":"        ambari_centos_image_name\u003d${ambari_centos_image_name:-centos_sahara_ambari}"},{"line_number":618,"context_line":"        ambari_element_sequence\u003d\"ambari mysql redhat-lsb $JAVA_ELEMENT\""},{"line_number":619,"context_line":"        image_create centos $ambari_centos_image_name $ambari_element_sequence"},{"line_number":620,"context_line":"    fi"},{"line_number":621,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos7\" ]; then"}],"source_content_type":"text/x-sh","patch_set":4,"id":"7a2fa921_31559e1c","line":618,"in_reply_to":"7a2fa921_91a03245","updated":"2015-10-07 13:36:30.000000000","message":"ambari installs redhat-lsb stuff during installing services on cluster, I think that we should include that on our images","commit_id":"7ec5891ede4bb37fafaeb21ee86adbea0e58dfaa"},{"author":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"change_message_id":"74676ce9c4818a6c34f5235e3149c47018f10899","unresolved":false,"context_lines":[{"line_number":615,"context_line":"    fi"},{"line_number":616,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos\" ]; then"},{"line_number":617,"context_line":"        ambari_centos_image_name\u003d${ambari_centos_image_name:-centos_sahara_ambari}"},{"line_number":618,"context_line":"        ambari_element_sequence\u003d\"ambari mysql redhat-lsb $JAVA_ELEMENT\""},{"line_number":619,"context_line":"        image_create centos $ambari_centos_image_name $ambari_element_sequence"},{"line_number":620,"context_line":"    fi"},{"line_number":621,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos7\" ]; then"}],"source_content_type":"text/x-sh","patch_set":4,"id":"7a2fa921_d12c6a04","line":618,"in_reply_to":"7a2fa921_abce1db2","updated":"2015-10-07 13:31:36.000000000","message":"yep, ambari uses that during hdp cluster installing","commit_id":"7ec5891ede4bb37fafaeb21ee86adbea0e58dfaa"},{"author":{"_account_id":12320,"name":"Pino Toscano","email":"ptoscano@redhat.com","username":"ptoscano"},"change_message_id":"b615c406c902a23d74a7bd79d74a72dbda4690de","unresolved":false,"context_lines":[{"line_number":615,"context_line":"    fi"},{"line_number":616,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos\" ]; then"},{"line_number":617,"context_line":"        ambari_centos_image_name\u003d${ambari_centos_image_name:-centos_sahara_ambari}"},{"line_number":618,"context_line":"        ambari_element_sequence\u003d\"ambari mysql redhat-lsb $JAVA_ELEMENT\""},{"line_number":619,"context_line":"        image_create centos $ambari_centos_image_name $ambari_element_sequence"},{"line_number":620,"context_line":"    fi"},{"line_number":621,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos7\" ]; then"}],"source_content_type":"text/x-sh","patch_set":4,"id":"7a2fa921_91a03245","line":618,"in_reply_to":"7a2fa921_d12c6a04","updated":"2015-10-07 13:33:47.000000000","message":"What is redhat-lsb needed for, by ambari?","commit_id":"7ec5891ede4bb37fafaeb21ee86adbea0e58dfaa"},{"author":{"_account_id":12320,"name":"Pino Toscano","email":"ptoscano@redhat.com","username":"ptoscano"},"change_message_id":"0d11de14a4ea24f602b23c60c55959a7fb52f807","unresolved":false,"context_lines":[{"line_number":615,"context_line":"    fi"},{"line_number":616,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos\" ]; then"},{"line_number":617,"context_line":"        ambari_centos_image_name\u003d${ambari_centos_image_name:-centos_sahara_ambari}"},{"line_number":618,"context_line":"        ambari_element_sequence\u003d\"ambari mysql redhat-lsb $JAVA_ELEMENT\""},{"line_number":619,"context_line":"        image_create centos $ambari_centos_image_name $ambari_element_sequence"},{"line_number":620,"context_line":"    fi"},{"line_number":621,"context_line":"    if [ -z \"$BASE_IMAGE_OS\" -o \"$BASE_IMAGE_OS\" \u003d \"centos7\" ]; then"}],"source_content_type":"text/x-sh","patch_set":5,"id":"7a2fa921_33dd9be0","line":618,"updated":"2015-10-07 16:05:36.000000000","message":"Still no explanation why redhat-lsb is needed.","commit_id":"ab1e29b9f2566605017ff308e346e492b369bc72"}],"elements/ambari/element-deps":[{"author":{"_account_id":12320,"name":"Pino Toscano","email":"ptoscano@redhat.com","username":"ptoscano"},"change_message_id":"45f11103f395bc9b43d97a869459232679a0ef4c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"disable-firewall"},{"line_number":2,"context_line":"java"},{"line_number":3,"context_line":"package-installs"},{"line_number":4,"context_line":"pkg-map"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"7a2fa921_30090e32","line":4,"updated":"2015-10-07 16:37:21.000000000","message":"package-installs is enough.","commit_id":"bfd5bea16f824a791f3bb68e9a591acf30fb1766"}],"elements/ambari/pkg-map":[{"author":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"change_message_id":"37578669c2567642ed9652241fd0ea4ef538e623","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"  \"family\": {"},{"line_number":3,"context_line":"    \"debian\": {"},{"line_number":4,"context_line":"      \"nc\": \"netcat\""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"7a2fa921_853c70e3","line":1,"updated":"2015-10-07 12:28:44.000000000","message":"pkg-map should be added in element-deps","commit_id":"7ec5891ede4bb37fafaeb21ee86adbea0e58dfaa"},{"author":{"_account_id":12320,"name":"Pino Toscano","email":"ptoscano@redhat.com","username":"ptoscano"},"change_message_id":"ed769a5d3d74b71e0dffec58c12c02ba8e768012","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"  \"family\": {"},{"line_number":3,"context_line":"    \"debian\": {"},{"line_number":4,"context_line":"      \"nc\": \"netcat\""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"7a2fa921_e8be5bd3","line":1,"in_reply_to":"7a2fa921_853c70e3","updated":"2015-10-07 13:02:04.000000000","message":"There\u0027s package-installs already, which uses pkg-map.","commit_id":"7ec5891ede4bb37fafaeb21ee86adbea0e58dfaa"},{"author":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"change_message_id":"37578669c2567642ed9652241fd0ea4ef538e623","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    },"},{"line_number":6,"context_line":"    \"redhat\": {"},{"line_number":7,"context_line":"      \"nc\": \"nc\""},{"line_number":8,"context_line":"    },"},{"line_number":9,"context_line":"  }"},{"line_number":10,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"7a2fa921_055aa04f","line":8,"range":{"start_line":8,"start_character":5,"end_line":8,"end_character":6},"updated":"2015-10-07 12:28:44.000000000","message":"need to remove it","commit_id":"7ec5891ede4bb37fafaeb21ee86adbea0e58dfaa"},{"author":{"_account_id":12320,"name":"Pino Toscano","email":"ptoscano@redhat.com","username":"ptoscano"},"change_message_id":"0d11de14a4ea24f602b23c60c55959a7fb52f807","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  \"family\": {"},{"line_number":3,"context_line":"    \"debian\": {"},{"line_number":4,"context_line":"      \"nc\": \"netcat\","},{"line_number":5,"context_line":"      \"snappy\": \"snappy\""},{"line_number":6,"context_line":"    },"},{"line_number":7,"context_line":"    \"redhat\": {"},{"line_number":8,"context_line":"      \"nc\": \"nc\","}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a2fa921_535b7754","line":5,"updated":"2015-10-07 16:05:36.000000000","message":"\"snappy\" in Debian is a multimedia player. If you want the development package for the snappy library, it is libsnappy-dev.\n\nWhy does it need the development stuff of a library?","commit_id":"ab1e29b9f2566605017ff308e346e492b369bc72"}]}