)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"f98faa0e6681f958c96ca82f7d121820bb745819","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deleting only images from kolla build"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Delete images from kolla repository."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Co-Authored-By: Carlos Cesario \u003ccarloscesario@gmail.com\u003e"},{"line_number":12,"context_line":"Change-Id: I3e9652e1cac156f822b388d2b9fd910180b4a892"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1a122d0e_7fc0962b","line":9,"updated":"2016-05-05 16:10:16.000000000","message":"Delete images built by kolla?","commit_id":"d3c9c3eb5ac41b664c8772106282cf886b2662b8"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"14a317cc668ceec9bb69942739fc6817d898aa43","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deleting only images from kolla build"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Delete images from kolla repository."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Co-Authored-By: Carlos Cesario \u003ccarloscesario@gmail.com\u003e"},{"line_number":12,"context_line":"Change-Id: I3e9652e1cac156f822b388d2b9fd910180b4a892"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1a122d0e_5f703a08","line":9,"in_reply_to":"1a122d0e_7fc0962b","updated":"2016-05-05 16:12:57.000000000","message":"??","commit_id":"d3c9c3eb5ac41b664c8772106282cf886b2662b8"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"f98faa0e6681f958c96ca82f7d121820bb745819","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Co-Authored-By: Carlos Cesario \u003ccarloscesario@gmail.com\u003e"},{"line_number":12,"context_line":"Change-Id: I3e9652e1cac156f822b388d2b9fd910180b4a892"},{"line_number":13,"context_line":"DocImpact"},{"line_number":14,"context_line":"Closes-Bug: #1551933"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1a122d0e_5fbddaaf","line":13,"updated":"2016-05-05 16:10:16.000000000","message":"you have fixed the doc in this PS. so no need DocImpact","commit_id":"d3c9c3eb5ac41b664c8772106282cf886b2662b8"}],"doc/operating-kolla.rst":[{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"f8ada8df5159ca493dc5d7d72e29477917f361da","unresolved":false,"context_lines":[{"line_number":32,"context_line":"This can be useful when you want to do a new clean deployment, particularly"},{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove the Docker images related to "},{"line_number":36,"context_line":"kolla from the local Docker registry."}],"source_content_type":"text/x-rst","patch_set":2,"id":"7af24918_8e3e4b26","line":36,"range":{"start_line":35,"start_character":65,"end_line":36,"end_character":5},"updated":"2016-03-02 21:48:58.000000000","message":"maybe \u0027with names related to images built with kolla\u0027","commit_id":"b28b626935d4abe2beb00976f9e88e9dfcf1cbf9"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"f8ada8df5159ca493dc5d7d72e29477917f361da","unresolved":false,"context_lines":[{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove the Docker images related to "},{"line_number":36,"context_line":"kolla from the local Docker registry."}],"source_content_type":"text/x-rst","patch_set":2,"id":"7af24918_2e061786","line":36,"range":{"start_line":36,"start_character":28,"end_line":36,"end_character":36},"updated":"2016-03-02 21:48:58.000000000","message":"should be cache","commit_id":"b28b626935d4abe2beb00976f9e88e9dfcf1cbf9"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"c3c5784bff5849f535eb7f6260a58f7273179a34","unresolved":false,"context_lines":[{"line_number":32,"context_line":"This can be useful when you want to do a new clean deployment, particularly"},{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove the Docker images related to "},{"line_number":36,"context_line":"kolla from the local Docker registry."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7af24918_99c03052","line":36,"range":{"start_line":35,"start_character":65,"end_line":36,"end_character":5},"updated":"2016-03-03 12:21:16.000000000","message":"maybe we should specify here what will be deleted? for example images that have \u0027ubuntu/centos-source/binary\u0027 or \u003cnone\u003e in name or sth like that?","commit_id":"a46cec048dcf691f63c5e7325234eb35bf2b1de6"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"c3c5784bff5849f535eb7f6260a58f7273179a34","unresolved":false,"context_lines":[{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove the Docker images related to "},{"line_number":36,"context_line":"kolla from the local Docker registry."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7af24918_79bc4406","line":36,"range":{"start_line":36,"start_character":28,"end_line":36,"end_character":36},"updated":"2016-03-03 12:21:16.000000000","message":"why not Docker cache? as it was previously","commit_id":"a46cec048dcf691f63c5e7325234eb35bf2b1de6"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"8f1a6a58c968e00a911eda48cac2c45448770b63","unresolved":false,"context_lines":[{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove the Docker images related to "},{"line_number":36,"context_line":"kolla from the local Docker registry."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7af24918_528b1d68","line":36,"range":{"start_line":36,"start_character":28,"end_line":36,"end_character":36},"in_reply_to":"7af24918_79bc4406","updated":"2016-03-03 12:54:57.000000000","message":"Done","commit_id":"a46cec048dcf691f63c5e7325234eb35bf2b1de6"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"8f1a6a58c968e00a911eda48cac2c45448770b63","unresolved":false,"context_lines":[{"line_number":32,"context_line":"This can be useful when you want to do a new clean deployment, particularly"},{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove the Docker images related to "},{"line_number":36,"context_line":"kolla from the local Docker registry."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7af24918_92ef651d","line":36,"range":{"start_line":35,"start_character":65,"end_line":36,"end_character":5},"in_reply_to":"7af24918_99c03052","updated":"2016-03-03 12:54:57.000000000","message":"Done","commit_id":"a46cec048dcf691f63c5e7325234eb35bf2b1de6"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"71f241746515022f271198b0beddea7d91d9aba9","unresolved":false,"context_lines":[{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove all Docker images built by "},{"line_number":36,"context_line":"kolla that have \u0027ubuntu-source/binary\u0027 or \u0027centos-source/binary\u0027 or \u0027\u003cnone\u003e\u0027 "},{"line_number":37,"context_line":"in name from the local Docker cache."},{"line_number":38,"context_line":"Examples: ubuntu-source-base, centos-binary-rsyslog, \u003cnone\u003e."},{"line_number":39,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"7af24918_2f13230f","line":36,"updated":"2016-03-04 23:37:52.000000000","message":"\u003cnone\u003e would not be accurate, nor should we remove none as they may not be from us","commit_id":"242360897ae1f899bc4a68fb900b832951d9df3f"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"588dc19eef8714ae3264d25ea2cde9ae6b7835f8","unresolved":false,"context_lines":[{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove all Docker images from the"},{"line_number":36,"context_line":"local Docker cache."}],"source_content_type":"text/x-rst","patch_set":8,"id":"1a122d0e_6919087a","line":36,"updated":"2016-05-05 15:27:08.000000000","message":"With kolla_version filtering, this documentation should read \"remove all Docker kolla-based images\" or something similar.","commit_id":"31636db9b4fd077ceabee4da67f2499349fc7f31"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"46326f8754314368fa56b2e446369d5f6070c892","unresolved":false,"context_lines":[{"line_number":33,"context_line":"one changing the network topology."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"``tools/cleanup-images`` can be used to remove all Docker images from the"},{"line_number":36,"context_line":"local Docker cache."}],"source_content_type":"text/x-rst","patch_set":8,"id":"1a122d0e_c9b01cdf","line":36,"in_reply_to":"1a122d0e_6919087a","updated":"2016-05-05 15:34:40.000000000","message":"you\u0027re right.","commit_id":"31636db9b4fd077ceabee4da67f2499349fc7f31"}],"tools/cleanup-images":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"ca7599dcb324372220c6715c04ec1a6cc99a343e","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\" | awk \u0027{print $3}\u0027`"},{"line_number":10,"context_line":"UBUNTU_OS\u003d`docker images | grep ^ubuntu | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"CENTOS_OS\u003d`docker images | grep ^centos | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":13,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]] \u0026\u0026 [[ -z \"$UBUNTU_OS\" ]] \u0026\u0026 [[ -z \"$CENTOS_OS\" ]] \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7af24918_1880a1f6","line":10,"updated":"2016-03-03 10:40:55.000000000","message":"Agreed with sam","commit_id":"b28b626935d4abe2beb00976f9e88e9dfcf1cbf9"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"919f116601ca610c9c497715da9829857c4751d9","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\" | awk \u0027{print $3}\u0027`"},{"line_number":10,"context_line":"UBUNTU_OS\u003d`docker images | grep ^ubuntu | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"CENTOS_OS\u003d`docker images | grep ^centos | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":13,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]] \u0026\u0026 [[ -z \"$UBUNTU_OS\" ]] \u0026\u0026 [[ -z \"$CENTOS_OS\" ]] \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7af24918_5cd7cf4a","line":10,"updated":"2016-03-03 03:40:26.000000000","message":"no this is no good. we dont want to delete the base images. we only want to delete images we have built. and this matches _any_ image starting with ubuntu which is also bad","commit_id":"b28b626935d4abe2beb00976f9e88e9dfcf1cbf9"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"c3c5784bff5849f535eb7f6260a58f7273179a34","unresolved":false,"context_lines":[{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]] \u0026\u0026 [[ -z \"$UBUNTU_OS\" ]] \u0026\u0026 \\"},{"line_number":13,"context_line":"    [[ -z \"$CENTOS_OS\" ]] \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":14,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":15,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7af24918_79e3a4a4","line":12,"range":{"start_line":12,"start_character":38,"end_line":12,"end_character":48},"updated":"2016-03-03 12:21:16.000000000","message":"you removed UBUNTU_OS","commit_id":"a46cec048dcf691f63c5e7325234eb35bf2b1de6"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"8f1a6a58c968e00a911eda48cac2c45448770b63","unresolved":false,"context_lines":[{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]] \u0026\u0026 [[ -z \"$UBUNTU_OS\" ]] \u0026\u0026 \\"},{"line_number":13,"context_line":"    [[ -z \"$CENTOS_OS\" ]] \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":14,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":15,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7af24918_d27e0d69","line":12,"range":{"start_line":12,"start_character":38,"end_line":12,"end_character":48},"in_reply_to":"7af24918_79e3a4a4","updated":"2016-03-03 12:54:57.000000000","message":"Done","commit_id":"a46cec048dcf691f63c5e7325234eb35bf2b1de6"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"c3c5784bff5849f535eb7f6260a58f7273179a34","unresolved":false,"context_lines":[{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]] \u0026\u0026 [[ -z \"$UBUNTU_OS\" ]] \u0026\u0026 \\"},{"line_number":13,"context_line":"    [[ -z \"$CENTOS_OS\" ]] \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":14,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":15,"context_line":"    exit 0"},{"line_number":16,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7af24918_594988b6","line":13,"range":{"start_line":13,"start_character":11,"end_line":13,"end_character":21},"updated":"2016-03-03 12:21:16.000000000","message":"you removed CENTOS_OS","commit_id":"a46cec048dcf691f63c5e7325234eb35bf2b1de6"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"8f1a6a58c968e00a911eda48cac2c45448770b63","unresolved":false,"context_lines":[{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]] \u0026\u0026 [[ -z \"$UBUNTU_OS\" ]] \u0026\u0026 \\"},{"line_number":13,"context_line":"    [[ -z \"$CENTOS_OS\" ]] \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":14,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":15,"context_line":"    exit 0"},{"line_number":16,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7af24918_329811c0","line":13,"range":{"start_line":13,"start_character":11,"end_line":13,"end_character":21},"in_reply_to":"7af24918_594988b6","updated":"2016-03-03 12:54:57.000000000","message":"Done","commit_id":"a46cec048dcf691f63c5e7325234eb35bf2b1de6"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"4103ad31a0604ee3b6deec77e56e58fee34ca6b0","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_237b9ef6","line":10,"updated":"2016-03-03 14:29:01.000000000","message":"why `docker images -a` has a -a parameter? do not think it is needed.\n\nanyone could explain this?","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":18652,"name":"Proskurin Kirill","email":"kproskurin@mirantis.com","username":"kproskurin"},"change_message_id":"0c8e99fe6562db7f71f9ae63fe034b6b4a6f1b7b","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_cf913bf3","line":10,"in_reply_to":"7af24918_237b9ef6","updated":"2016-03-03 15:17:03.000000000","message":"Imo it really depends on how \"deep\" this cleanup should be. -a shows intermedeate images. I think its fine here.","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"e81d6b9652d7c662a7bfc89e4a7fb25fe25c4728","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_28b3240c","line":10,"in_reply_to":"7af24918_67af6b2a","updated":"2016-03-04 20:05:20.000000000","message":"Done","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"f6dd8eb684939ccd04ba69f8a4337aacbeec1549","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_1b152248","line":10,"in_reply_to":"7af24918_67af6b2a","updated":"2016-03-03 16:40:32.000000000","message":"I tried to remove without to use -a and fails too.","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"20a2d1ab2901c3537483f37737ae471dcf65cf34","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_67af6b2a","line":10,"in_reply_to":"7af24918_cf913bf3","updated":"2016-03-03 15:47:51.000000000","message":"intermediate images are why the command fails FYI, because it tries to delete an already deleted image","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"4103ad31a0604ee3b6deec77e56e58fee34ca6b0","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_038022bf","line":11,"updated":"2016-03-03 14:29:01.000000000","message":"the grep \u0027-i\u0027 is useless. when it is always marked as \u0027\u003cnone\u003e\u0027 ( lower case)","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"20a2d1ab2901c3537483f37737ae471dcf65cf34","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_c77f5fa9","line":11,"in_reply_to":"7af24918_038022bf","updated":"2016-03-03 15:47:51.000000000","message":"\u003cnone\u003e images should be left alone as those are necessarily from kolla","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":18652,"name":"Proskurin Kirill","email":"kproskurin@mirantis.com","username":"kproskurin"},"change_message_id":"0c8e99fe6562db7f71f9ae63fe034b6b4a6f1b7b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_cf477bba","line":11,"in_reply_to":"7af24918_038022bf","updated":"2016-03-03 15:17:03.000000000","message":"grep is not needed too.\nYou could do something like this:\ndocker images | awk \u0027/\u003cnone\u003e/ {print $3}\u0027","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"e81d6b9652d7c662a7bfc89e4a7fb25fe25c4728","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_68cd2c86","line":11,"in_reply_to":"7af24918_c77f5fa9","updated":"2016-03-04 20:05:20.000000000","message":"Done","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"f6dd8eb684939ccd04ba69f8a4337aacbeec1549","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_60cd9184","line":11,"in_reply_to":"7af24918_c77f5fa9","updated":"2016-03-03 16:40:32.000000000","message":"Even if was it caused by kolla-build?\nIn some times when I built some images was named \u003cnone\u003e.\nI think that should deleted to rebuild, or something like that.\n\nAlso, \u003cnone\u003e images built by kolla are chieldren for other images built by kolla, only after remove this images \u003cnone\u003e, was possible remove another images.","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"f6dd8eb684939ccd04ba69f8a4337aacbeec1549","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images -a | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"FAILED_IMG\u003d`docker images | grep -i \"\u003cnone\u003e\" | awk \u0027{print $3}\u0027`"},{"line_number":12,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_4a6cf6cf","line":11,"in_reply_to":"7af24918_cf477bba","updated":"2016-03-03 16:40:32.000000000","message":"Done","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"83c8d388fa7705d78263a84959ad82e1bee6e84c","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"},{"line_number":15,"context_line":"fi"},{"line_number":16,"context_line":"docker rmi --force $@ $FAILED_IMG $KOLLA_IMAGES"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_746c3e35","line":16,"range":{"start_line":16,"start_character":11,"end_line":16,"end_character":18},"updated":"2016-03-03 15:05:19.000000000","message":"not sure about using force here.\n\nWe discussed in different review (https://review.openstack.org/#/c/286791/), that we should warn a user when he tries to remove images that are used. If he really wants to remove them we should stop and remove containers first and then remove images.","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"e81d6b9652d7c662a7bfc89e4a7fb25fe25c4728","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"},{"line_number":15,"context_line":"fi"},{"line_number":16,"context_line":"docker rmi --force $@ $FAILED_IMG $KOLLA_IMAGES"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_48caf06f","line":16,"range":{"start_line":16,"start_character":11,"end_line":16,"end_character":18},"in_reply_to":"7af24918_746c3e35","updated":"2016-03-04 20:05:20.000000000","message":"Done","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"f6dd8eb684939ccd04ba69f8a4337aacbeec1549","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"},{"line_number":15,"context_line":"fi"},{"line_number":16,"context_line":"docker rmi --force $@ $FAILED_IMG $KOLLA_IMAGES"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_072e6742","line":16,"range":{"start_line":16,"start_character":11,"end_line":16,"end_character":18},"in_reply_to":"7af24918_746c3e35","updated":"2016-03-03 16:40:32.000000000","message":"If the images are been used by a container it would not remove the image, even using --force.\nI test this issue.","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":18652,"name":"Proskurin Kirill","email":"kproskurin@mirantis.com","username":"kproskurin"},"change_message_id":"0c8e99fe6562db7f71f9ae63fe034b6b4a6f1b7b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":14,"context_line":"    exit 0"},{"line_number":15,"context_line":"fi"},{"line_number":16,"context_line":"docker rmi --force $@ $FAILED_IMG $KOLLA_IMAGES"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7af24918_6f258707","line":16,"range":{"start_line":16,"start_character":11,"end_line":16,"end_character":18},"in_reply_to":"7af24918_746c3e35","updated":"2016-03-03 15:17:03.000000000","message":"agree","commit_id":"3c32cdb3f42810b681fa970af16b725780bf0823"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"71f241746515022f271198b0beddea7d91d9aba9","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":12,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":13,"context_line":"    exit 0"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"7af24918_8ff877c6","line":10,"updated":"2016-03-04 23:37:52.000000000","message":"KOLLA_IMAGES\u003d$(docker images --no-trunc | awk \u0027/\u0027\"${FILTER}\"\u0027/ {print $3}\u0027)","commit_id":"242360897ae1f899bc4a68fb900b832951d9df3f"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"71f241746515022f271198b0beddea7d91d9aba9","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":12,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":13,"context_line":"    exit 0"},{"line_number":14,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"7af24918_6ff52bca","line":11,"updated":"2016-03-04 23:37:52.000000000","message":"not sure where FAILED_IMG is coming from","commit_id":"242360897ae1f899bc4a68fb900b832951d9df3f"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"75d259af60e1acd954ecffb8ceeeff0fd1b022d1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"FILTER\u003d\"ubuntu-source-|ubuntu-binary-|centos-source-|centos-binary-\""},{"line_number":10,"context_line":"KOLLA_IMAGES\u003d`docker images | grep -i -E \"$FILTER\" | awk \u0027{print $3}\u0027`"},{"line_number":11,"context_line":"if [[ -z \"$KOLLA_IMAGES\" ]]  \u0026\u0026 [[ -z \"$FAILED_IMG\" ]]; then"},{"line_number":12,"context_line":"    echo \"No images to cleanup, exit now.\""},{"line_number":13,"context_line":"    exit 0"},{"line_number":14,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"7af24918_e6ca001a","line":11,"in_reply_to":"7af24918_6ff52bca","updated":"2016-03-05 03:00:56.000000000","message":"Yes, where is FAILED_IMG in this script ?","commit_id":"242360897ae1f899bc4a68fb900b832951d9df3f"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"0a5564a40e961f6f119d372d79dd69f9d61d4c9e","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    exit 0"},{"line_number":14,"context_line":"fi"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"docker rmi $@ $FAILED_IMG $KOLLA_IMAGES"},{"line_number":17,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"7af24918_162f5bf7","line":16,"range":{"start_line":16,"start_character":14,"end_line":16,"end_character":25},"updated":"2016-03-05 07:17:48.000000000","message":"the same here, FAILED_IMG var does not exist","commit_id":"242360897ae1f899bc4a68fb900b832951d9df3f"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"c03922713302dfd238f953f26c88e0c829283ec6","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"UBUNTU_FILTER\u003d\"ubuntu-source-|ubuntu-binary-\""},{"line_number":10,"context_line":"CENTOS_FILTER\u003d\"centos-source-|centos-binary-|centos-rdo\""},{"line_number":11,"context_line":"RHEL_FILTER\u003d\"rhel-source-\""},{"line_number":12,"context_line":"FEDORA_FILTER\u003d\"fedora-rdo-\""},{"line_number":13,"context_line":"FILTER\u003d\"$UBUNTU_FILTER|$CENTOS_FILTER|$RHEL_FILTER|$FEDORA_FILTER\""}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"5aef4532_5e41f60a","line":10,"updated":"2016-03-07 16:27:55.000000000","message":"as i said back in patchset 2, this is not ok. Please do not do a per distro variable.","commit_id":"1f5819ef68ced10359af6d5ab35e18cddc914fe2"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"43a7efd8d323f39f3f2910440a02e4eda031b184","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"UBUNTU_FILTER\u003d\"ubuntu-source-|ubuntu-binary-\""},{"line_number":10,"context_line":"CENTOS_FILTER\u003d\"centos-source-|centos-binary-|centos-rdo\""},{"line_number":11,"context_line":"RHEL_FILTER\u003d\"rhel-source-\""},{"line_number":12,"context_line":"FEDORA_FILTER\u003d\"fedora-rdo-\""},{"line_number":13,"context_line":"FILTER\u003d\"$UBUNTU_FILTER|$CENTOS_FILTER|$RHEL_FILTER|$FEDORA_FILTER\""}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"5aef4532_4ebf68fb","line":10,"updated":"2016-03-08 03:31:20.000000000","message":"we should respect the configure file (`/etc/kolla/global.yml`) when deleting image.\nuse the\n* kolla_base_distro\n* kolla_install_type\nwe should not touch others images.","commit_id":"1f5819ef68ced10359af6d5ab35e18cddc914fe2"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"498eaf90e4ddc547887c88220127fe5f36a54415","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"UBUNTU_FILTER\u003d\"ubuntu-source-|ubuntu-binary-\""},{"line_number":10,"context_line":"CENTOS_FILTER\u003d\"centos-source-|centos-binary-|centos-rdo\""},{"line_number":11,"context_line":"RHEL_FILTER\u003d\"rhel-source-\""},{"line_number":12,"context_line":"FEDORA_FILTER\u003d\"fedora-rdo-\""},{"line_number":13,"context_line":"FILTER\u003d\"$UBUNTU_FILTER|$CENTOS_FILTER|$RHEL_FILTER|$FEDORA_FILTER\""}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"5aef4532_d574c0d0","line":10,"in_reply_to":"5aef4532_01754ef8","updated":"2016-03-07 18:14:59.000000000","message":"This grep have a problem that the user can build changing the namespace but may be possible to put a option with a default namespace for kollaglue.","commit_id":"1f5819ef68ced10359af6d5ab35e18cddc914fe2"},{"author":{"_account_id":14967,"name":"Thiago Gomes","email":"fthiagogv@outlook.com","username":"fthiagogv"},"change_message_id":"99e1013cb226d79ca8aabfaa2d7b6339babaa416","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"UBUNTU_FILTER\u003d\"ubuntu-source-|ubuntu-binary-\""},{"line_number":10,"context_line":"CENTOS_FILTER\u003d\"centos-source-|centos-binary-|centos-rdo\""},{"line_number":11,"context_line":"RHEL_FILTER\u003d\"rhel-source-\""},{"line_number":12,"context_line":"FEDORA_FILTER\u003d\"fedora-rdo-\""},{"line_number":13,"context_line":"FILTER\u003d\"$UBUNTU_FILTER|$CENTOS_FILTER|$RHEL_FILTER|$FEDORA_FILTER\""}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"5aef4532_61889223","line":10,"in_reply_to":"5aef4532_5e41f60a","updated":"2016-03-07 17:02:03.000000000","message":"I\u0027m sorry, I did not understand clearly. I thought that was because regex used.\nI will search another way to delete precisely all images built.","commit_id":"1f5819ef68ced10359af6d5ab35e18cddc914fe2"},{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"c4404c9b4653a32b4dff53c279f5544fa919b69f","unresolved":false,"context_lines":[{"line_number":7,"context_line":". tools/validate-docker-execute"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"UBUNTU_FILTER\u003d\"ubuntu-source-|ubuntu-binary-\""},{"line_number":10,"context_line":"CENTOS_FILTER\u003d\"centos-source-|centos-binary-|centos-rdo\""},{"line_number":11,"context_line":"RHEL_FILTER\u003d\"rhel-source-\""},{"line_number":12,"context_line":"FEDORA_FILTER\u003d\"fedora-rdo-\""},{"line_number":13,"context_line":"FILTER\u003d\"$UBUNTU_FILTER|$CENTOS_FILTER|$RHEL_FILTER|$FEDORA_FILTER\""}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"5aef4532_01754ef8","line":10,"in_reply_to":"5aef4532_61889223","updated":"2016-03-07 17:06:07.000000000","message":"Delete images built by kolla and _not_ other images.\n\nFor example, the following displays only images built by kolla.\n\n  docker images | grep kollaglue","commit_id":"1f5819ef68ced10359af6d5ab35e18cddc914fe2"}]}