)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"0ec894fad58157a2f098bbae734c24d509b58958","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1793d72e_37026464","updated":"2022-08-11 07:14:17.000000000","message":"needs more testing!","commit_id":"2d950e58d24dc2ebf3cb7d44aba27faeac0d4414"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"b4ff85c2f4395a717765cf62c3926d3f229573de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bda97c5d_1ebe8518","updated":"2022-08-11 14:39:02.000000000","message":"We need to get the Depends-On merged first, and wait for some promotion in order to actually get the nftables package in the OC image... Before that, we\u0027re stuck.","commit_id":"2996c829114c5f46668698c9a1c9c24ddbbbc5b5"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"a40b416971a318681129daa40e2da4900cec8a89","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b0dddb0c_4c0808e3","updated":"2022-08-11 11:09:13.000000000","message":"recheck","commit_id":"2996c829114c5f46668698c9a1c9c24ddbbbc5b5"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"51e62e4a00a6ce6914faea539f5c7d276e1f717d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2588ddef_23093f40","updated":"2022-08-16 11:25:43.000000000","message":"Once we get the OC image with nftables, we should probably be able to merge this patch. at least, a testproject shows it green, with 3 controllers + 1 compute:\nhttps://review.rdoproject.org/r/c/testproject/+/31954/66#message-15129b4bc4172cb4344b6ee9758778ede99b0847","commit_id":"8e915bbb94fec872b43cb898e53ddf75f939fae4"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"95e6125fb82f2c538776cee466c9a5d472cfbb0f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"d07a02dd_c4c54522","updated":"2022-09-08 12:18:46.000000000","message":"recheck","commit_id":"7a969602309b6cf9035502b7aefb66a98a5c332e"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"634eb9523e37de5ab375a7acb0fd948d742f854a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"fe16bc7e_ac4e41b9","updated":"2022-09-09 05:53:50.000000000","message":"recheck","commit_id":"7a969602309b6cf9035502b7aefb66a98a5c332e"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"0c255a9822b014dd578b5a1d2606ec80282135d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"27a56f69_ab3915e4","updated":"2022-09-09 11:12:44.000000000","message":"recheck\n\nfailed before nftables comes to play on the OC node. So it\u0027s unrelated.","commit_id":"d45ec547d5d69c042ce11386b46d0767cb52f6ce"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"e2c7608921935fd5428c2cea99fb3d6e6211e39c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"1d36aa4e_8e6fb1ca","updated":"2022-09-12 13:41:41.000000000","message":"Waiting on 2 other patches for CI:\nhttps://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/857096\nhttps://review.opendev.org/c/openstack/tripleo-quickstart/+/857128","commit_id":"8b1ba7ec3d60225b45c1310e18ff6c36c54ed4b0"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"dc1c6cfa8f6ddd4293e595b4af84ae005f14b265","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"9202896b_655700b4","updated":"2022-09-15 15:42:43.000000000","message":"Although zuul is green (or will be green soon), RDO is broken due to weird content in the \"hostvars\" used in the oooq-extras to generate the firewall rules. We\u0027ll need to filter things.","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"33f2912dbae89494fe1f4ee599bd4cdf19eddc72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"00975fe2_a2ac44ec","updated":"2022-09-19 09:23:38.000000000","message":"check-rdo","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"0c321b50a310215e17db8532e8e11168797a7508","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"08dba12b_deaaef14","updated":"2022-09-19 09:17:55.000000000","message":"check-rdo\n","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"c4fe0ffd2837cba3dfe7acfe70b83926fd3649bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"03711f37_0c9db7d5","updated":"2022-09-15 09:36:33.000000000","message":"recheck","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"60bf21994afcd732abec11bc743cf594aceb5fb6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"05758c93_bdee7f94","updated":"2022-09-16 08:43:06.000000000","message":"recheck","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"c4db4183bf13f40a081928a163f4e8fd0175e9e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"1f001528_2419805a","updated":"2022-09-14 08:07:26.000000000","message":"recheck","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"3df4122240e1d71f9cff0090a02cc3670cf31dbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"5f52ba14_2d82a12a","updated":"2022-09-15 07:41:09.000000000","message":"recheck","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"696c939c0db7a5d661ea054c950e17efdcd073cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"8cfbce15_7e21b26e","updated":"2022-09-14 06:47:01.000000000","message":"recheck","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"6a2008e7555c112a89ad921851fb550e6dbd9c9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"90d8aa72_36c2a149","updated":"2022-09-19 06:31:00.000000000","message":"recheck","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"35c5cdbd5762e2c5f319ebc83198397b65d12590","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"c4c80737_6b2a6055","updated":"2022-09-14 11:09:51.000000000","message":"recheck","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"04f31fb32809745ffac780b2a656ee109054c5ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"cb480d17_93381874","updated":"2022-09-15 06:38:35.000000000","message":"recheck\n\nWe need to (re)open ssh port to the world. This mimic the current behavior set in the \"tripleo-INPUT\" chain.","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"1a9385fe15ab1082ba3ae13585b73ef41d5d5adc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"dcdda42e_fd8a37e5","updated":"2022-09-15 12:17:36.000000000","message":"recheck\n\njinja2 indentation block issue in the oooq-extras patch.","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"3767ac73a807aed4889fbd14a517e686374c591b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"4f97b321_b516374c","updated":"2022-09-14 18:26:55.000000000","message":"recheck\n\nnot sure the failures are really related to the firewall change.","commit_id":"a9a74c7ccf0626095259782ff45d49c68ad1d0be"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"5601a26992d481a33b460d802134e21e1dc62748","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":13,"id":"00c2626b_ae350466","updated":"2022-09-27 07:29:22.000000000","message":"Can we add a release note because this updates the default behavior ?","commit_id":"102e44e643349fb28d06e831a636c225ced30243"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"67a1d23da1fe44f085b71c7d0489fa485a81d668","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":13,"id":"6c3a01c5_b7448e44","updated":"2022-09-27 06:53:28.000000000","message":"Seems all of the dependent changes are already merged. Can we add a release note because this updates the default behavior ?","commit_id":"102e44e643349fb28d06e831a636c225ced30243"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"2b96b2b2a8c46fa10df1c4b34dff43a729c9c506","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"2159a73b_4c149006","updated":"2022-09-21 07:23:54.000000000","message":"check-rdo","commit_id":"102e44e643349fb28d06e831a636c225ced30243"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"3a01e56657bce20383eedf611c48a9d6f14fd38a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"f5a00600_2bce3b13","in_reply_to":"00c2626b_ae350466","updated":"2022-09-27 07:40:33.000000000","message":"Done","commit_id":"102e44e643349fb28d06e831a636c225ced30243"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"2e90610cdc1bab0843c080693ca780d67f6d7b5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"25151f38_9b69e724","in_reply_to":"6c3a01c5_b7448e44","updated":"2022-09-27 07:40:26.000000000","message":"Done","commit_id":"102e44e643349fb28d06e831a636c225ced30243"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"c10d8752b381f4a9528da11603c24faa28541684","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"84eeaa12_34ced39b","updated":"2022-09-27 11:32:59.000000000","message":"check-rdo\n\n2022-09-27 09:03:24 | Error: error creating container storage: the container name \"container-puppet-neutron\" is already in use by 3a05fd95a4b8ea5b980ff7b794340a2180798374c9c53003d123acdcf0fb483f. You have to remove that container to be able to reuse that name: that name is already in use","commit_id":"20d94040dffcbc3c549b170a778e37994c13f868"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b31562705a6bc2d9076992a6e27b199d8ebf2b6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"58d69558_178665bc","updated":"2022-09-28 05:54:57.000000000","message":"ack from ci point of view because you\u0027ve already been working closely with ysandeep for a while on making sure we aren\u0027t breaking any jobs with the switch\n\n@Sandeep or @Cedric did we also test that all is good in the RDO/vexx environment (i.e. periodic integration/component jobs?).\n\n\nI see logs from the jobs here at\n\nhttps://ee942e678189464d44e8-ef070eb0b435ab454e2b8598601fdff9.ssl.cf5.rackcdn.com/852808/15/check/tripleo-ci-centos-9-undercloud-containers/f262693/logs/undercloud/var/log/extra/nftables.txt\n\nhttps://ee942e678189464d44e8-ef070eb0b435ab454e2b8598601fdff9.ssl.cf5.rackcdn.com/852808/15/check/tripleo-ci-centos-9-undercloud-containers/f262693/logs/undercloud/etc/nftables/index.html\n\nany other interesting files/directories we should be aware of? \n\n","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"b80613670edf6bc247559b5f646628e0fa490867","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"8918c1e1_bd37dc04","updated":"2022-09-29 07:06:09.000000000","message":"ah, nope. repo_setup. yay.","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"28cc7a942aadd67567da13b158bf46f3dbfc51dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"6bf095ec_8049111b","updated":"2022-09-28 11:47:19.000000000","message":"based on hackmd results","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"e61f5c4324a6cbad571f962830affca95738e70f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"b087ba0a_3681a936","updated":"2022-09-28 13:23:19.000000000","message":"recheck","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"1875231272539c00f6d2fadfa8b139105428223b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"532f0d45_7f95f400","updated":"2022-09-29 07:05:19.000000000","message":"recheck\n\n-1 is due to the removal of the +W","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e106f649becbfbdded9c0a791b66485a23fa775c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"3438885e_a7f66523","updated":"2022-09-29 07:04:07.000000000","message":"recheck\n\ntimeout during pulling iscsi container \"dial tcp 104.130.172.74:5001: i/o timeout\"","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"3000f2cd67eaf030740127cab970391929dc2daf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"b7db6a15_31a47494","updated":"2022-09-30 05:55:21.000000000","message":"recheck\n\nunrelated.","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e32995bf6129274abcd5c35fe75c2b48070196d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"c0f56396_e0336e63","updated":"2022-09-28 11:46:52.000000000","message":"thanks ","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"1c71337dbbb62770c8a7f162c19a04c8b8167e3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"948a4bc9_d2e3830f","in_reply_to":"58d69558_178665bc","updated":"2022-09-28 08:35:25.000000000","message":"I think there were tests in the testproject, yep. The hackmd should point to all the testing.","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e32995bf6129274abcd5c35fe75c2b48070196d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"a13004f4_654c2e12","in_reply_to":"948a4bc9_d2e3830f","updated":"2022-09-28 11:46:52.000000000","message":"for others\n\nhackmd there https://hackmd.io/F0W2gYw_SiaiWkowjFU9cw?edit\n\ntest patch there https://review.rdoproject.org/r/c/testproject/+/31954","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"2bd245e70a7a6df2197c2191a6bd5fbcfa44561d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"d77e33a6_e66e3fcb","in_reply_to":"b087ba0a_3681a936","updated":"2022-09-28 13:23:42.000000000","message":"(facepalm). It\u0027s missing +W","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"}],"deployment/tripleo-firewall/tripleo-firewall-baremetal-ansible.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b31562705a6bc2d9076992a6e27b199d8ebf2b6e","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    description: Set the actual firewall engine. Can be \"iptables\" or \"nftables\""},{"line_number":39,"context_line":"    type: string"},{"line_number":40,"context_line":"    constraints:"},{"line_number":41,"context_line":"      - allowed_values: [\u0027iptables\u0027, \u0027nftables\u0027]"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"resources:"},{"line_number":44,"context_line":"  # Merging role-specific parameters (RoleParameters) with the default parameters."}],"source_content_type":"text/x-yaml","patch_set":15,"id":"3f13c57e_7bf4067c","line":41,"range":{"start_line":41,"start_character":26,"end_line":41,"end_character":34},"updated":"2022-09-28 05:54:57.000000000","message":"not directly related to *this* patch but do we need to consider keeping one job with iptables instead of the new default?","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"1c71337dbbb62770c8a7f162c19a04c8b8167e3d","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    description: Set the actual firewall engine. Can be \"iptables\" or \"nftables\""},{"line_number":39,"context_line":"    type: string"},{"line_number":40,"context_line":"    constraints:"},{"line_number":41,"context_line":"      - allowed_values: [\u0027iptables\u0027, \u0027nftables\u0027]"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"resources:"},{"line_number":44,"context_line":"  # Merging role-specific parameters (RoleParameters) with the default parameters."}],"source_content_type":"text/x-yaml","patch_set":15,"id":"e28b59c3_62ddb595","line":41,"range":{"start_line":41,"start_character":26,"end_line":41,"end_character":34},"in_reply_to":"3f13c57e_7bf4067c","updated":"2022-09-28 08:35:25.000000000","message":"I don\u0027t think we should keep iptables compat any longer. To be clear, imho that parameter should even be dropped asap - and code in tripleo-ansible should also be cleaned in order to remove tripleo_iptables action and related tasks.\n\nFTR, this FirewallEngine was added in order to allow a smooth testing - not intended to last.\n\nDoes it make sense?","commit_id":"9cdf72876b50499465e3613b4c0ce06e33b1e6f7"}]}