)]}'
{"nova/values.yaml":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"73033f30edae2ab1348bbde831fc2edbadd2d5e2","unresolved":false,"context_lines":[{"line_number":561,"context_line":"      StrictHostKeyChecking no"},{"line_number":562,"context_line":"      UserKnownHostsFile /dev/null"},{"line_number":563,"context_line":"      Port {{ .Values.network.ssh.port }}"},{"line_number":564,"context_line":"  ssh_private: \u0027nulhttps://review.opendev.org/#/c/683759/l\u0027"},{"line_number":565,"context_line":"  ssh_public: \u0027null\u0027"},{"line_number":566,"context_line":"  rally_tests:"},{"line_number":567,"context_line":"    run_tempest: false"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_65541922","line":564,"range":{"start_line":564,"start_character":16,"end_line":564,"end_character":58},"updated":"2019-09-24 14:02:02.000000000","message":"Looks like this string got messed up by accident.","commit_id":"a1bac011e163e093b76a64ffedd78847b13936e3"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"8bc07b0da3e351e6e394f0c908a8d652c693fb13","unresolved":false,"context_lines":[{"line_number":561,"context_line":"      StrictHostKeyChecking no"},{"line_number":562,"context_line":"      UserKnownHostsFile /dev/null"},{"line_number":563,"context_line":"      Port {{ .Values.network.ssh.port }}"},{"line_number":564,"context_line":"  ssh_private: \u0027nulhttps://review.opendev.org/#/c/683759/l\u0027"},{"line_number":565,"context_line":"  ssh_public: \u0027null\u0027"},{"line_number":566,"context_line":"  rally_tests:"},{"line_number":567,"context_line":"    run_tempest: false"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_29befbd4","line":564,"range":{"start_line":564,"start_character":16,"end_line":564,"end_character":58},"in_reply_to":"3fa7e38b_65541922","updated":"2019-09-24 15:32:05.000000000","message":"weird - will fix.","commit_id":"a1bac011e163e093b76a64ffedd78847b13936e3"},{"author":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"change_message_id":"eecf21f91bfe12bff16e0b5ef581c79be95f0e29","unresolved":false,"context_lines":[{"line_number":566,"context_line":"  rally_tests:"},{"line_number":567,"context_line":"    run_tempest: false"},{"line_number":568,"context_line":"    clean_up: |"},{"line_number":569,"context_line":"      FLAVORS\u003d$(openstack flavor list | grep -e \"^s_rally_\" | awk \"{ print $2 }\")"},{"line_number":570,"context_line":"      if [ -n \"$FLAVORS\" ]; then"},{"line_number":571,"context_line":"        echo $FLAVORS | xargs openstack flavor delete"},{"line_number":572,"context_line":"      fi"},{"line_number":573,"context_line":"    tests:"},{"line_number":574,"context_line":"      NovaAgents.list_agents:"},{"line_number":575,"context_line":"        - runner:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_4046abc2","line":572,"range":{"start_line":569,"start_character":0,"end_line":572,"end_character":8},"updated":"2019-09-24 14:29:56.000000000","message":"Should we check for any left over VMs created by rally and clean them up?","commit_id":"a1bac011e163e093b76a64ffedd78847b13936e3"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"65971d115b0854f5e984cdfd69b111ba56f55330","unresolved":false,"context_lines":[{"line_number":566,"context_line":"  rally_tests:"},{"line_number":567,"context_line":"    run_tempest: false"},{"line_number":568,"context_line":"    clean_up: |"},{"line_number":569,"context_line":"      FLAVORS\u003d$(openstack flavor list | grep -e \"^s_rally_\" | awk \"{ print $2 }\")"},{"line_number":570,"context_line":"      if [ -n \"$FLAVORS\" ]; then"},{"line_number":571,"context_line":"        echo $FLAVORS | xargs openstack flavor delete"},{"line_number":572,"context_line":"      fi"},{"line_number":573,"context_line":"    tests:"},{"line_number":574,"context_line":"      NovaAgents.list_agents:"},{"line_number":575,"context_line":"        - runner:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_297b5bf5","line":572,"range":{"start_line":569,"start_character":0,"end_line":572,"end_character":8},"in_reply_to":"3fa7e38b_4046abc2","updated":"2019-09-24 15:42:43.000000000","message":"we can - but would these VMs start with the string s_rally_? I presume yes?","commit_id":"a1bac011e163e093b76a64ffedd78847b13936e3"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"fe44d02d48cb88942ddfb38c6e412317b867feae","unresolved":false,"context_lines":[{"line_number":566,"context_line":"  rally_tests:"},{"line_number":567,"context_line":"    run_tempest: false"},{"line_number":568,"context_line":"    clean_up: |"},{"line_number":569,"context_line":"      FLAVORS\u003d$(openstack flavor list | grep -e \"^s_rally_\" | awk \"{ print $2 }\")"},{"line_number":570,"context_line":"      if [ -n \"$FLAVORS\" ]; then"},{"line_number":571,"context_line":"        echo $FLAVORS | xargs openstack flavor delete"},{"line_number":572,"context_line":"      fi"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_35c3a85f","line":569,"range":{"start_line":569,"start_character":40,"end_line":569,"end_character":80},"updated":"2019-09-24 18:39:07.000000000","message":"When I run it in my own environment, this does not work. \n1) awk needs single quotes; when I used double quotes, awk didn\u0027t do anything with it.  \n2) When you list the flavors, you get tabular format, so you have to navigate around the \"|\" characters on each line, so the name of the flavor would actually be the 4th string on the line, not the second.  But if you used \"-f value\" on the flavor list command you don\u0027t have to worry about the \"|\" char  (and you can use awk \u0027{ print $2 }\u0027 in that case).\n\nSame issues on line 573.","commit_id":"e5faad1b237a7e578d7166107041c6c6d00e70ab"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"cd6a622891190b5f8f46eefe873a192049934280","unresolved":false,"context_lines":[{"line_number":566,"context_line":"  rally_tests:"},{"line_number":567,"context_line":"    run_tempest: false"},{"line_number":568,"context_line":"    clean_up: |"},{"line_number":569,"context_line":"      FLAVORS\u003d$(openstack flavor list | grep -e \"^s_rally_\" | awk \"{ print $2 }\")"},{"line_number":570,"context_line":"      if [ -n \"$FLAVORS\" ]; then"},{"line_number":571,"context_line":"        echo $FLAVORS | xargs openstack flavor delete"},{"line_number":572,"context_line":"      fi"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_cdfabd41","line":569,"range":{"start_line":569,"start_character":40,"end_line":569,"end_character":80},"in_reply_to":"3fa7e38b_326bac1e","updated":"2019-09-25 13:53:27.000000000","message":"Yep, that\u0027s right. Thanks.","commit_id":"e5faad1b237a7e578d7166107041c6c6d00e70ab"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"dfe1112fe330ae89e4930a7c00d9a44e84b3f1e4","unresolved":false,"context_lines":[{"line_number":566,"context_line":"  rally_tests:"},{"line_number":567,"context_line":"    run_tempest: false"},{"line_number":568,"context_line":"    clean_up: |"},{"line_number":569,"context_line":"      FLAVORS\u003d$(openstack flavor list | grep -e \"^s_rally_\" | awk \"{ print $2 }\")"},{"line_number":570,"context_line":"      if [ -n \"$FLAVORS\" ]; then"},{"line_number":571,"context_line":"        echo $FLAVORS | xargs openstack flavor delete"},{"line_number":572,"context_line":"      fi"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_326bac1e","line":569,"range":{"start_line":569,"start_character":40,"end_line":569,"end_character":80},"in_reply_to":"3fa7e38b_35c3a85f","updated":"2019-09-25 13:15:03.000000000","message":"Will fix the quote error here. I actually want a list of ID\u0027s for the purpose of delete, so $2 should be correct with | or $1 with \"-f value\". The name is for the purpose of filtering the resource.","commit_id":"e5faad1b237a7e578d7166107041c6c6d00e70ab"}]}