Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 1 | From 49cc847aeedacaabdb285b7cdea0e6ccede6042e Mon Sep 17 00:00:00 2001 |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 2 | From: Tomas Pecka <peckato1@users.noreply.github.com> |
| 3 | Date: Wed, 6 Oct 2021 10:11:31 +0200 |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 4 | Subject: [PATCH 7/7] networkctl: JSON output in networkctl lldp |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 5 | |
| 6 | `networkctl lldp` now outputs also in JSON format when `--json=*` |
| 7 | argument passed. The LLDP neighbors are listed in per interface lists. |
| 8 | --- |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 9 | src/network/networkctl.c | 84 +++++++++++++++++++++++++++++----------- |
| 10 | 1 file changed, 62 insertions(+), 22 deletions(-) |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 11 | |
| 12 | diff --git a/src/network/networkctl.c b/src/network/networkctl.c |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 13 | index 04a89e015f..2eb21ad4ff 100644 |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 14 | --- a/src/network/networkctl.c |
| 15 | +++ b/src/network/networkctl.c |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 16 | @@ -2467,12 +2467,14 @@ typedef struct LLDPUserdata { |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 17 | uint16_t capabilities_all; |
| 18 | |
| 19 | Table *table; |
| 20 | + JsonVariant *json; |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 21 | |
| 22 | char *link_name; |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 23 | } LLDPUserdata; |
| 24 | |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 25 | static void lldp_userdata_freep(LLDPUserdata* p) { |
| 26 | table_unref(p->table); |
| 27 | + json_variant_unref(p->json); |
| 28 | } |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 29 | |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 30 | static int lldp_neighbors_varlink_reply(Varlink *link, JsonVariant *parameters, const char *error_id, VarlinkReplyFlags flags, void *userdata) { |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 31 | @@ -2516,6 +2518,34 @@ static int lldp_neighbors_varlink_reply(Varlink *link, JsonVariant *parameters, |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 32 | udata->neighbors_count += 1; |
| 33 | udata->capabilities_all |= entry.capabilities; |
| 34 | |
| 35 | + if (udata->json) { |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 36 | + _cleanup_(json_variant_unrefp) JsonVariant *v = NULL; |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 37 | + _cleanup_(json_variant_unrefp) JsonVariant *neighbors = NULL; |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 38 | + |
| 39 | + capabilities = lldp_capabilities_to_string(entry.capabilities, false); |
| 40 | + |
| 41 | + r = json_build(&v, JSON_BUILD_OBJECT( |
| 42 | + JSON_BUILD_PAIR("neighbor", JSON_BUILD_OBJECT( |
| 43 | + JSON_BUILD_PAIR_CONDITION(entry.chassis_id, "chassisId", JSON_BUILD_STRING(entry.chassis_id)), |
| 44 | + JSON_BUILD_PAIR_CONDITION(entry.port_id, "portId", JSON_BUILD_STRING(entry.port_id)), |
| 45 | + JSON_BUILD_PAIR_CONDITION(entry.system_name, "systemName", JSON_BUILD_STRING(entry.system_name)), |
| 46 | + JSON_BUILD_PAIR_CONDITION(entry.port_description, "portDescription", JSON_BUILD_STRING(entry.port_description)), |
| 47 | + JSON_BUILD_PAIR_CONDITION(entry.capabilities, "enabledCapabilities", JSON_BUILD_STRING(capabilities)))))); |
| 48 | + if (r < 0) |
| 49 | + return r; |
| 50 | + |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 51 | + |
| 52 | + neighbors = json_variant_ref(json_variant_by_key(udata->json, udata->link_name)); |
| 53 | + |
| 54 | + r = json_variant_append_array(&neighbors, v); |
| 55 | + if (r < 0) |
| 56 | + return r; |
| 57 | + |
| 58 | + r = json_variant_set_field(&udata->json, udata->link_name, neighbors); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 59 | + if (r < 0) |
| 60 | + return r; |
| 61 | + } |
| 62 | + |
| 63 | return 0; |
| 64 | } |
| 65 | |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 66 | @@ -2528,7 +2558,6 @@ static int link_lldp_status(int argc, char *argv[], void *userdata) { |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 67 | _cleanup_(sd_netlink_unrefp) sd_netlink *rtnl = NULL; |
| 68 | _cleanup_(link_info_array_freep) LinkInfo *links = NULL; |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 69 | _cleanup_(lldp_userdata_freep) LLDPUserdata udata = {}; |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 70 | - TableCell *cell; |
| 71 | |
| 72 | r = varlink_connect_address(&link, address); |
| 73 | if (r < 0) |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 74 | @@ -2550,24 +2579,31 @@ static int link_lldp_status(int argc, char *argv[], void *userdata) { |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 75 | return c; |
| 76 | |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 77 | pager_open(arg_pager_flags); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 78 | + if (arg_json_format_flags == JSON_FORMAT_OFF) { |
| 79 | + TableCell *cell; |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 80 | + |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 81 | + udata.table = table_new("link", |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 82 | + "chassis-id", |
| 83 | + "system-name", |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 84 | + "caps", |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 85 | + "port-id", |
| 86 | + "port-description"); |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 87 | + if (!udata.table) |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 88 | + return log_oom(); |
Tomáš Pecka | 5e07025 | 2022-05-20 20:41:21 +0200 | [diff] [blame] | 89 | |
| 90 | - udata.table = table_new("link", |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 91 | - "chassis-id", |
| 92 | - "system-name", |
Tomáš Pecka | 5e07025 | 2022-05-20 20:41:21 +0200 | [diff] [blame] | 93 | - "caps", |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 94 | - "port-id", |
| 95 | - "port-description"); |
Tomáš Pecka | 5e07025 | 2022-05-20 20:41:21 +0200 | [diff] [blame] | 96 | - if (!udata.table) |
| 97 | - return log_oom(); |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 98 | + if (arg_full) |
| 99 | + table_set_width(udata.table, 0); |
Tomáš Pecka | 5e07025 | 2022-05-20 20:41:21 +0200 | [diff] [blame] | 100 | |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 101 | - if (arg_full) |
| 102 | - table_set_width(udata.table, 0); |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 103 | + table_set_header(udata.table, arg_legend); |
Tomáš Pecka | 5e07025 | 2022-05-20 20:41:21 +0200 | [diff] [blame] | 104 | |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 105 | - table_set_header(udata.table, arg_legend); |
| 106 | - |
Tomáš Pecka | 5e07025 | 2022-05-20 20:41:21 +0200 | [diff] [blame] | 107 | - assert_se(cell = table_get_cell(udata.table, 0, 3)); |
| 108 | - table_set_minimum_width(udata.table, cell, 11); |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 109 | - table_set_ersatz_string(udata.table, TABLE_ERSATZ_DASH); |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 110 | + assert_se(cell = table_get_cell(udata.table, 0, 3)); |
| 111 | + table_set_minimum_width(udata.table, cell, 11); |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 112 | + table_set_ersatz_string(udata.table, TABLE_ERSATZ_DASH); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 113 | + } else { |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 114 | + r = json_build(&udata.json, JSON_BUILD_EMPTY_OBJECT); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 115 | + if (r < 0) |
| 116 | + return r; |
| 117 | + } |
| 118 | |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 119 | varlink_set_userdata(link, &udata); |
| 120 | |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 121 | @@ -2590,13 +2626,17 @@ static int link_lldp_status(int argc, char *argv[], void *userdata) { |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 122 | return r; |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 123 | } |
| 124 | |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 125 | - r = table_print(udata.table, NULL); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 126 | - if (r < 0) |
| 127 | - return table_log_print_error(r); |
| 128 | + if (arg_json_format_flags == JSON_FORMAT_OFF) { |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 129 | + r = table_print(udata.table, NULL); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 130 | + if (r < 0) |
| 131 | + return table_log_print_error(r); |
| 132 | |
| 133 | - if (arg_legend) { |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 134 | - lldp_capabilities_legend(udata.capabilities_all); |
| 135 | - printf("\n%i neighbors listed.\n", udata.neighbors_count); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 136 | + if (arg_legend) { |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 137 | + lldp_capabilities_legend(udata.capabilities_all); |
| 138 | + printf("\n%i neighbors listed.\n", udata.neighbors_count); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 139 | + } |
| 140 | + } else { |
Václav Kubernát | 8cd6156 | 2021-12-08 13:27:31 +0100 | [diff] [blame] | 141 | + json_variant_dump(udata.json, arg_json_format_flags, NULL, NULL); |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 142 | } |
| 143 | |
| 144 | return 0; |
| 145 | -- |
Tomáš Pecka | 6936b91 | 2023-06-12 11:30:36 +0200 | [diff] [blame] | 146 | 2.41.0 |
Tomáš Pecka | ae30176 | 2021-10-13 10:50:37 +0200 | [diff] [blame] | 147 | |