)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"fb686934ce32fd62c232cf207cc9dea6dfa4d866","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5e3939c3_ec9cd40d","updated":"2026-02-02 07:35:01.000000000","message":"Just a couple of small things, lgtm otherwise","commit_id":"f15d99f28578ac94ea9a6e05aeb809e64abb94cb"}],"ceilometer/network/services/vpnaas.py":[{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"b39418ddc6ade8fcad87398220fa8cb18a9559f5","unresolved":true,"context_lines":[{"line_number":72,"context_line":"              \u0027psk\u0027,"},{"line_number":73,"context_line":"              \u0027initiator\u0027,"},{"line_number":74,"context_line":"              \u0027ikepolicy_id\u0027,"},{"line_number":75,"context_line":"              \u0027dpd\u0027,"},{"line_number":76,"context_line":"              \u0027ipsecpolicy_id\u0027,"},{"line_number":77,"context_line":"              \u0027vpnservice_id\u0027,"},{"line_number":78,"context_line":"              \u0027mtu\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1925692b_9b41efe3","side":"PARENT","line":75,"updated":"2026-02-03 15:28:01.000000000","message":"Is the dpd field not present in the data returned by the new client? Or why is it getting removed?","commit_id":"7039a99b9863c3d7260182175651acacbe5fe693"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"912382782bb8e6eacda98422a4cc32006370c718","unresolved":true,"context_lines":[{"line_number":72,"context_line":"              \u0027psk\u0027,"},{"line_number":73,"context_line":"              \u0027initiator\u0027,"},{"line_number":74,"context_line":"              \u0027ikepolicy_id\u0027,"},{"line_number":75,"context_line":"              \u0027dpd\u0027,"},{"line_number":76,"context_line":"              \u0027ipsecpolicy_id\u0027,"},{"line_number":77,"context_line":"              \u0027vpnservice_id\u0027,"},{"line_number":78,"context_line":"              \u0027mtu\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"afa184e7_0c082ea5","side":"PARENT","line":75,"in_reply_to":"1925692b_9b41efe3","updated":"2026-02-03 15:52:26.000000000","message":"It is not present in the data returned by the new client.\n\nThese are the data that are available in the VPNService resource in the opesntacksdk:\nhttps://github.com/openstack/openstacksdk/blob/master/openstack/network/v2/vpn_service.py#L30","commit_id":"7039a99b9863c3d7260182175651acacbe5fe693"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"0ccd5a93398eac51fefbedd62a248c5645990d32","unresolved":false,"context_lines":[{"line_number":72,"context_line":"              \u0027psk\u0027,"},{"line_number":73,"context_line":"              \u0027initiator\u0027,"},{"line_number":74,"context_line":"              \u0027ikepolicy_id\u0027,"},{"line_number":75,"context_line":"              \u0027dpd\u0027,"},{"line_number":76,"context_line":"              \u0027ipsecpolicy_id\u0027,"},{"line_number":77,"context_line":"              \u0027vpnservice_id\u0027,"},{"line_number":78,"context_line":"              \u0027mtu\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"b44c8125_b9c408ae","side":"PARENT","line":75,"in_reply_to":"afa184e7_0c082ea5","updated":"2026-02-03 16:10:48.000000000","message":"Acknowledged","commit_id":"7039a99b9863c3d7260182175651acacbe5fe693"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"fb686934ce32fd62c232cf207cc9dea6dfa4d866","unresolved":true,"context_lines":[{"line_number":58,"context_line":"                unit\u003d\u0027vpnservice\u0027,"},{"line_number":59,"context_line":"                volume\u003dstatus,"},{"line_number":60,"context_line":"                user_id\u003dNone,"},{"line_number":61,"context_line":"                project_id\u003dvpn[\u0027project_id\u0027] or vpn[\u0027tenant_id\u0027],"},{"line_number":62,"context_line":"                resource_id\u003dvpn[\u0027id\u0027],"},{"line_number":63,"context_line":"                resource_metadata\u003dself.extract_metadata(vpn)"},{"line_number":64,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":1,"id":"31f7a4fb_9c7970ac","line":61,"updated":"2026-02-02 07:35:01.000000000","message":"Replace this with access-safe, the same way it is in fwaas.py\n\n```suggestion\n                project_id\u003dvpn.get(\u0027project_id\u0027) or vpn.get(\u0027tenant_id\u0027),\n```","commit_id":"f15d99f28578ac94ea9a6e05aeb809e64abb94cb"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"912382782bb8e6eacda98422a4cc32006370c718","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                unit\u003d\u0027vpnservice\u0027,"},{"line_number":59,"context_line":"                volume\u003dstatus,"},{"line_number":60,"context_line":"                user_id\u003dNone,"},{"line_number":61,"context_line":"                project_id\u003dvpn[\u0027project_id\u0027] or vpn[\u0027tenant_id\u0027],"},{"line_number":62,"context_line":"                resource_id\u003dvpn[\u0027id\u0027],"},{"line_number":63,"context_line":"                resource_metadata\u003dself.extract_metadata(vpn)"},{"line_number":64,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":1,"id":"6b3c404b_00076f83","line":61,"in_reply_to":"31f7a4fb_9c7970ac","updated":"2026-02-03 15:52:26.000000000","message":"Acknowledged","commit_id":"f15d99f28578ac94ea9a6e05aeb809e64abb94cb"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"fb686934ce32fd62c232cf207cc9dea6dfa4d866","unresolved":true,"context_lines":[{"line_number":100,"context_line":"                unit\u003d\u0027ipsec_site_connection\u0027,"},{"line_number":101,"context_line":"                volume\u003d1,"},{"line_number":102,"context_line":"                user_id\u003dNone,"},{"line_number":103,"context_line":"                project_id\u003dconn[\u0027project_id\u0027] or conn[\u0027tenant_id\u0027],"},{"line_number":104,"context_line":"                resource_id\u003dconn[\u0027id\u0027],"},{"line_number":105,"context_line":"                resource_metadata\u003dself.extract_metadata(conn)"},{"line_number":106,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":1,"id":"6d49b437_387d044e","line":103,"updated":"2026-02-02 07:35:01.000000000","message":"```suggestion\n                project_id\u003dconn.get(\u0027project_id\u0027) or conn.get(\u0027tenant_id\u0027),\n```","commit_id":"f15d99f28578ac94ea9a6e05aeb809e64abb94cb"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"912382782bb8e6eacda98422a4cc32006370c718","unresolved":false,"context_lines":[{"line_number":100,"context_line":"                unit\u003d\u0027ipsec_site_connection\u0027,"},{"line_number":101,"context_line":"                volume\u003d1,"},{"line_number":102,"context_line":"                user_id\u003dNone,"},{"line_number":103,"context_line":"                project_id\u003dconn[\u0027project_id\u0027] or conn[\u0027tenant_id\u0027],"},{"line_number":104,"context_line":"                resource_id\u003dconn[\u0027id\u0027],"},{"line_number":105,"context_line":"                resource_metadata\u003dself.extract_metadata(conn)"},{"line_number":106,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":1,"id":"e9edad44_a26fd686","line":103,"in_reply_to":"6d49b437_387d044e","updated":"2026-02-03 15:52:26.000000000","message":"Acknowledged","commit_id":"f15d99f28578ac94ea9a6e05aeb809e64abb94cb"}]}