For currently unknown reason the dbus-send calls will fail to set the
  firewall zone for the given interface if we omit the --print-reply
  option...

  This looks like some kind of race-condition in dbus-send, since the
  --print-reply makes the call synchronous and slower.

  Hopefully this is only a temporary workaround until DBus is fixed.

  Resolves: #1586284

network-functions uses awk to tranform the mac address to upper case.
Replace awk with tr which is two order of magnitude faster:

    $ time awk '{ print toupper($0) }' </usr/share/dict/words >/dev/null

    real    0m0,227s
    user    0m0,224s
    sys     0m0,004s
    $ time tr '[a-z]' '[A-Z]' </usr/share/dict/words >/dev/null

    real    0m0,005s
    user    0m0,000s
    sys     0m0,005s

Also use here-strings instead of spawning a subshell and a pipe.

Signed-off-by: Matteo Croce <mcroce@redhat.com>

  Instead of displaying messages without context, like this:

  ./network-functions: line 571: echo: write error: Invalid argument
  ./network-functions: line 598: echo: write error: Permission denied

  We will now display pretty error messages (via net_log) about what has
  actually failed...

  This has been requested by our customer:
  https://bugzilla.redhat.com/show_bug.cgi?id=1542514

  Empty ${RESOLV_MODS} was causing a deletion of config files in
  /etc/ppp/peers/*

  Resolves: #155

  This is complete rework of how we generate the /etc/resolv.conf.

  Fixes:
   * always update 'nameserver' & 'search' entries when DNS* & DOMAIN
     options (respectively) were updated in ifcfg-* files
   * always update the order of 'nameserver' entries when the order of
     DNS* options was updated

  Enhancements:
   * added support for DNS3 option (equals to MAXNS value in GLIBC)
   * added logic to process duplicate DNS* options
   * added logic to process randomly omitted DNS* options (e.g. omitting
     DNS1 while specifying DNS2 and/or DNS3 will still work now)

  This work was based on these two RHEL-7.5 BZs:
  https://bugzilla.redhat.com/show_bug.cgi?id=1364895
  https://bugzilla.redhat.com/show_bug.cgi?id=1357658

  Otherwise obtaining IP from DHCP might fail. Based on patch from:
   * Rich Alloway <richard.alloway@roguewave.com> | CentOS BZ #0011138

  And network-scripts/network-functions was patched to use convert2sec().

  This function can be used to convert the value of its first parameter
  to a number of seconds - based on the time unit (specified as the
  second parameter).

  This is mostly useful for converting values for use with the sleep(1).

  The time units had to be manually converted from microseconds to
  seconds. Hopefully this was done right.