)]}'
{"tools/ovn_migration/tripleo_environment/playbooks/roles/migration/tasks/cleanup-dataplane.yml":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6104718a7338b7c3cb93fa2fdfb08a2b6db6ccda","unresolved":true,"context_lines":[{"line_number":20,"context_line":"- name: Cleanup neutron trunk subports"},{"line_number":21,"context_line":"  when: do_cleanup.rc \u003d\u003d 0"},{"line_number":22,"context_line":"  shell: |"},{"line_number":23,"context_line":"    ovs-vsctl list interface | awk \u0027/name[ ]*: sp[it]-/ { print $3 }\u0027 | xargs -n1 ovs-vsctl del-port"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Clean neutron datapath security groups from iptables"},{"line_number":26,"context_line":"  shell: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f39c1a8_d6dec2d9","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":100},"updated":"2022-02-01 13:16:37.000000000","message":"Can we just ignore the xargs failure instead? Something like\n\n ovs-vsctl list interface | awk \u0027/name[ ]*: sp[it]-/ { print $3 }\u0027 | xargs -n1 ovs-vsctl del-port \u003e /dev/null || :","commit_id":"9af1e929f526d81229a447c190fd4be7d5ae2ed3"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"512b0eb16d5b38d2c6204ee729bc36aebad78075","unresolved":true,"context_lines":[{"line_number":20,"context_line":"- name: Cleanup neutron trunk subports"},{"line_number":21,"context_line":"  when: do_cleanup.rc \u003d\u003d 0"},{"line_number":22,"context_line":"  shell: |"},{"line_number":23,"context_line":"    ovs-vsctl list interface | awk \u0027/name[ ]*: sp[it]-/ { print $3 }\u0027 | xargs -n1 ovs-vsctl del-port"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Clean neutron datapath security groups from iptables"},{"line_number":26,"context_line":"  shell: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fd8ec9f3_242014f3","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":100},"in_reply_to":"5f39c1a8_d6dec2d9","updated":"2022-02-01 13:19:22.000000000","message":"We could ignore by adding \u0027ignore_errors: True\u0027 as above, but then if the ovs-vsctl del-port fails for some internal error you\u0027ll not able to know it.","commit_id":"9af1e929f526d81229a447c190fd4be7d5ae2ed3"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"51cd670105c8700bf6bcdb764ea636027bddd97b","unresolved":true,"context_lines":[{"line_number":20,"context_line":"- name: Cleanup neutron trunk subports"},{"line_number":21,"context_line":"  when: do_cleanup.rc \u003d\u003d 0"},{"line_number":22,"context_line":"  shell: |"},{"line_number":23,"context_line":"    ovs-vsctl list interface | awk \u0027/name[ ]*: sp[it]-/ { print $3 }\u0027 | xargs -n1 ovs-vsctl del-port"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Clean neutron datapath security groups from iptables"},{"line_number":26,"context_line":"  shell: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4b61b754_19c5a495","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":100},"in_reply_to":"a3da8ebc_38315e3d","updated":"2022-02-01 17:32:48.000000000","message":"Yes it will, I wasn\u0027t considering the fact that the ovs-ofctl list interface could have an error. Since this part of the migration it will only clean the ovs interfaces and the migration shouldn\u0027t be stopped by an error here, I\u0027ll ignore any errors raised here but I\u0027ll print a message informing of the error","commit_id":"9af1e929f526d81229a447c190fd4be7d5ae2ed3"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"f7bd8fe1ad7ceb14b40eed0563a474103200c37a","unresolved":true,"context_lines":[{"line_number":20,"context_line":"- name: Cleanup neutron trunk subports"},{"line_number":21,"context_line":"  when: do_cleanup.rc \u003d\u003d 0"},{"line_number":22,"context_line":"  shell: |"},{"line_number":23,"context_line":"    ovs-vsctl list interface | awk \u0027/name[ ]*: sp[it]-/ { print $3 }\u0027 | xargs -n1 ovs-vsctl del-port"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Clean neutron datapath security groups from iptables"},{"line_number":26,"context_line":"  shell: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a3da8ebc_38315e3d","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":100},"in_reply_to":"fd8ec9f3_242014f3","updated":"2022-02-01 13:22:24.000000000","message":"yes, ignore_errors would be also option. How would be putting ignore_errors to Cleanup neutron trunk subports task different to L18? It will fail with the same error and produce exactly same output, won\u0027t it?","commit_id":"9af1e929f526d81229a447c190fd4be7d5ae2ed3"}]}