initscripts/sysconfig, branch 9.49.48

ifup-post: fix incorrect condition for RESOLV_MODS

2018-08-24T10:23:27+00:00

  This was causing the /etc/resolv.conf file to be always updated when
  RESOLV_MODS was not set...

  Before the commit 5d6156454bf8f6dab4a5fdd7e1bf6 we were not updating
  the /etc/resolv.conf file if the RESOLV_MODS was empty.

  See https://bugzilla.redhat.com/show_bug.cgi?id=1610411 for more info.

ifdown-eth: no longer needed 'pidof -x dhclient' condition removed

2018-07-24T13:52:11+00:00

  Previously, there were 2 additional conditions related to this part of
  code, but they were removed in time, so this last condition has
  remained an artifact and is de-facto obsolete.

  However, the 'pidof -x dhclient' condition could cause the network
  service to hang on stop/restart when processes were executed from a
  network share... Removing that condition should hypothetically fix it.

network-scripts: setting of firewall ZONE fixed

2018-06-07T12:26:23+00:00

  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: use tr to upper case strings rather than awk

2018-05-17T15:47:07+00:00

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) }' /dev/null

    real    0m0,227s
    user    0m0,224s
    sys     0m0,004s
    $ time tr '[a-z]' '[A-Z]' /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

sysconfig/readonly-root: Clarify the usage of readonly-root.

2018-05-17T13:18:03+00:00

  ... Without appending the 'ro' to mount options of '/' mount point the
  system will be still mounted as read-write, even when READONLY=yes.

network-functions: add error messages for bonding installation

2018-05-17T12:49:45+00:00

  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

ifdown-post: fix logical error in commit 5d61564

2018-01-02T16:29:40+00:00

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

  Resolves: #155

network-functions: use POSIX forwarding instead of bash-ism

2017-11-09T13:58:51+00:00

ifup-post: always update nameserver & search entries in /etc/resolv.conf

2017-11-02T10:54:44+00:00

  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

ifup-eth: wait for STP to complete setup on bridge if $DELAY is not set

2017-10-31T11:36:19+00:00

  Otherwise obtaining IP from DHCP might fail. Based on patch from:
   * Rich Alloway  | CentOS BZ #0011138