1
0
mirror of https://github.com/systemd/systemd.git synced 2025-09-14 09:44:38 +03:00

network: downgrade log level in dhcp6_configure()

And make the callers warn about failures.
This commit is contained in:
Yu Watanabe
2021-06-15 22:16:51 +09:00
parent 5aebe9c898
commit bc8bd68db8
3 changed files with 21 additions and 21 deletions

View File

@@ -113,7 +113,7 @@ static int link_configure_and_start_dhcp_delayed(Link *link) {
if (!link->dhcp6_client) { if (!link->dhcp6_client) {
r = dhcp6_configure(link); r = dhcp6_configure(link);
if (r < 0) if (r < 0)
return r; return log_link_warning_errno(link, r, "Failed to configure DHCP6 client: %m");
} }
if (link->set_flags_messages > 0) if (link->set_flags_messages > 0)

View File

@@ -1505,7 +1505,7 @@ static int dhcp6_set_hostname(sd_dhcp6_client *client, Link *link) {
else { else {
r = gethostname_strict(&hostname); r = gethostname_strict(&hostname);
if (r < 0 && r != -ENXIO) /* ENXIO: no hostname set or hostname is "localhost" */ if (r < 0 && r != -ENXIO) /* ENXIO: no hostname set or hostname is "localhost" */
return r; return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to get hostname: %m");
hn = hostname; hn = hostname;
} }
@@ -1513,9 +1513,9 @@ static int dhcp6_set_hostname(sd_dhcp6_client *client, Link *link) {
r = sd_dhcp6_client_set_fqdn(client, hn); r = sd_dhcp6_client_set_fqdn(client, hn);
if (r == -EINVAL && hostname) if (r == -EINVAL && hostname)
/* Ignore error when the machine's hostname is not suitable to send in DHCP packet. */ /* Ignore error when the machine's hostname is not suitable to send in DHCP packet. */
log_link_warning_errno(link, r, "DHCP6 CLIENT: Failed to set hostname from kernel hostname, ignoring: %m"); log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set hostname from kernel hostname, ignoring: %m");
else if (r < 0) else if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set hostname: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set hostname: %m");
return 0; return 0;
} }
@@ -1585,7 +1585,7 @@ int dhcp6_configure(Link *link) {
return 0; return 0;
if (link->dhcp6_client) if (link->dhcp6_client)
return -EBUSY; return log_link_debug_errno(link, SYNTHETIC_ERRNO(EBUSY), "DHCP6 client is already configured.");
r = dhcp_configure_duid(link, link_get_dhcp6_duid(link)); r = dhcp_configure_duid(link, link_get_dhcp6_duid(link));
if (r <= 0) if (r <= 0)
@@ -1593,24 +1593,24 @@ int dhcp6_configure(Link *link) {
r = sd_dhcp6_client_new(&client); r = sd_dhcp6_client_new(&client);
if (r == -ENOMEM) if (r == -ENOMEM)
return log_oom(); return log_oom_debug();
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to create DHCP6 client: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to create DHCP6 client: %m");
r = sd_dhcp6_client_attach_event(client, link->manager->event, 0); r = sd_dhcp6_client_attach_event(client, link->manager->event, 0);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to attach event: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to attach event: %m");
r = dhcp6_set_identifier(link, client); r = dhcp6_set_identifier(link, client);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set identifier: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set identifier: %m");
ORDERED_HASHMAP_FOREACH(send_option, link->network->dhcp6_client_send_options) { ORDERED_HASHMAP_FOREACH(send_option, link->network->dhcp6_client_send_options) {
r = sd_dhcp6_client_add_option(client, send_option); r = sd_dhcp6_client_add_option(client, send_option);
if (r == -EEXIST) if (r == -EEXIST)
continue; continue;
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set option: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set option: %m");
} }
r = dhcp6_set_hostname(client, link); r = dhcp6_set_hostname(client, link);
@@ -1619,18 +1619,18 @@ int dhcp6_configure(Link *link) {
r = sd_dhcp6_client_set_ifindex(client, link->ifindex); r = sd_dhcp6_client_set_ifindex(client, link->ifindex);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set ifindex: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set ifindex: %m");
if (link->network->dhcp6_rapid_commit) { if (link->network->dhcp6_rapid_commit) {
r = sd_dhcp6_client_set_request_option(client, SD_DHCP6_OPTION_RAPID_COMMIT); r = sd_dhcp6_client_set_request_option(client, SD_DHCP6_OPTION_RAPID_COMMIT);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set request flag for rapid commit: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set request flag for rapid commit: %m");
} }
if (link->network->dhcp6_mudurl) { if (link->network->dhcp6_mudurl) {
r = sd_dhcp6_client_set_request_mud_url(client, link->network->dhcp6_mudurl); r = sd_dhcp6_client_set_request_mud_url(client, link->network->dhcp6_mudurl);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set MUD URL: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set MUD URL: %m");
} }
SET_FOREACH(request_options, link->network->dhcp6_request_options) { SET_FOREACH(request_options, link->network->dhcp6_request_options) {
@@ -1642,19 +1642,19 @@ int dhcp6_configure(Link *link) {
continue; continue;
} }
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set request flag for '%u': %m", option); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set request flag for '%u': %m", option);
} }
if (link->network->dhcp6_user_class) { if (link->network->dhcp6_user_class) {
r = sd_dhcp6_client_set_request_user_class(client, link->network->dhcp6_user_class); r = sd_dhcp6_client_set_request_user_class(client, link->network->dhcp6_user_class);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set user class: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set user class: %m");
} }
if (link->network->dhcp6_vendor_class) { if (link->network->dhcp6_vendor_class) {
r = sd_dhcp6_client_set_request_vendor_class(client, link->network->dhcp6_vendor_class); r = sd_dhcp6_client_set_request_vendor_class(client, link->network->dhcp6_vendor_class);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set vendor class: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set vendor class: %m");
} }
ORDERED_HASHMAP_FOREACH(vendor_option, link->network->dhcp6_client_send_vendor_options) { ORDERED_HASHMAP_FOREACH(vendor_option, link->network->dhcp6_client_send_vendor_options) {
@@ -1662,23 +1662,23 @@ int dhcp6_configure(Link *link) {
if (r == -EEXIST) if (r == -EEXIST)
continue; continue;
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set vendor option: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set vendor option: %m");
} }
r = sd_dhcp6_client_set_callback(client, dhcp6_handler, link); r = sd_dhcp6_client_set_callback(client, dhcp6_handler, link);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set callback: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set callback: %m");
if (dhcp6_enable_prefix_delegation(link)) { if (dhcp6_enable_prefix_delegation(link)) {
r = sd_dhcp6_client_set_prefix_delegation(client, true); r = sd_dhcp6_client_set_prefix_delegation(client, true);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set prefix delegation: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set prefix delegation: %m");
} }
if (link->network->dhcp6_pd_length > 0) { if (link->network->dhcp6_pd_length > 0) {
r = sd_dhcp6_client_set_prefix_delegation_hint(client, link->network->dhcp6_pd_length, &link->network->dhcp6_pd_address); r = sd_dhcp6_client_set_prefix_delegation_hint(client, link->network->dhcp6_pd_length, &link->network->dhcp6_pd_address);
if (r < 0) if (r < 0)
return log_link_error_errno(link, r, "DHCP6 CLIENT: Failed to set prefix hint: %m"); return log_link_debug_errno(link, r, "DHCP6 CLIENT: Failed to set prefix hint: %m");
} }
link->dhcp6_client = TAKE_PTR(client); link->dhcp6_client = TAKE_PTR(client);

View File

@@ -1133,7 +1133,7 @@ static int link_configure(Link *link) {
r = dhcp6_configure(link); r = dhcp6_configure(link);
if (r < 0) if (r < 0)
return r; return log_link_warning_errno(link, r, "Failed to configure DHCP6 client: %m");
r = ndisc_configure(link); r = ndisc_configure(link);
if (r < 0) if (r < 0)