Correct typing for IOReadIOPSMax, IOWriteIOPSMax,... in systemd::manage_dropin

[traylenator]

Pull Request (PR) description

Using any of the following directives in systemd::manage_dropin or systemd::manage_unit resulted in a compilation error.

• IODeviceWeight
• IOReadBandwidthMax
• IOWriteBandwidthMax
• IOReadIOPSMax
• IOWriteIOPSMax

The types for these directives in Systemd::Unit::Slice and Systemd::Unit::Service have now been updated.

Example usage:

systemd::manage_dropin { 'devicelimits.conf':
  unit          =>  'special.service',
  service_entry => {
   'IOReadIOPSMax' => [
     ['/dev/afs',100],
     ['/dev/gluster','1000K'],
   ],
  },
}

would result in a drop in file of:

[Service]
IOReadIOPSMax=/dev/afs 100
IOReadIOPSMax=/dev/gluster 1000K

This Pull Request (PR) fixes the following issues

• Fixes IODeviceWeight, IOReadIOPSMax, .. do not work in systemd::manage_unit or systemd::dropin_file #424

Worth Noting

• The Types here are different to what the existing systemd::service_limit uses for these parameters.
• My aim following this is to make systemd::service_limits a wrapper to systemd::manage_dropin.
  it contains a translation for these parameters to maintain backwards compatibility.
• systemd::service_limits could then be deprecated.

[traylenator]

Very open to the simpler:

systemd::manage_dropin { 'devicelimits.conf':
  unit          =>  'special.service',
  service_entry => {
   'IOReadIOPSMax' => [
      '/dev/afs 100',
      '/dev/gluster  1000K',
   ],
  },
}

👍 or 👎 for this simpler one.

[TheMeier]

Maybe make a type for

Variant[Tuple[Stdlib::Absolutepath, Systemd::Unit::Amount],Array[Tuple[Stdlib::Absolutepath, Systemd::Unit::Amount]]]

and

Array[Tuple[Stdlib::Absolutepath, Systemd::Unit::Amount]]

Systemd::Unit::Bandwidth?

[traylenator]

Maybe make a type for

Variant[Tuple[Stdlib::Absolutepath, Systemd::Unit::Amount],Array[Tuple[Stdlib::Absolutepath, Systemd::Unit::Amount]]]

and

Array[Tuple[Stdlib::Absolutepath, Systemd::Unit::Amount]]

Systemd::Unit::Bandwidth?

I don't think this is used anywhere else.
I certainly don't want to move away from the obvious to read.

variant[array[thing],thing]

So would only leave the tuple to simplify and that's simple already. :-)