On 27.06.2016 [13:12:17 -0000], Robie Basak wrote:
> +- defaultfor :operatingsystem => :ubuntu, :operatingsystemmajrelease => ["15.04","15.10","16.04"]
> ++ defaultfor :operatingsystem => :ubuntu, :operatingsystemmajrelease => ["15.04","15.10","16.04","16.10"]
>
> This will regress as soon as Yakkety+1 opens. Can we fix it properly?
> The logic should be "systemd for >= 15.04". That may need something
> better than a set of string comparisons. But doing anything different is
> clearly broken.
This logic is "easy" to modify for that, but the other iteration (also
a version list) is not, as it's not using a comparison, but extracting
the value itself.
On 27.06.2016 [13:12:17 -0000], Robie Basak wrote: mmajrelease => ["15.04" ,"15.10" ,"16.04" ] mmajrelease => ["15.04" ,"15.10" ,"16.04" ,"16.10" ]
> +- defaultfor :operatingsystem => :ubuntu, :operatingsyste
> ++ defaultfor :operatingsystem => :ubuntu, :operatingsyste
>
> This will regress as soon as Yakkety+1 opens. Can we fix it properly?
> The logic should be "systemd for >= 15.04". That may need something
> better than a set of string comparisons. But doing anything different is
> clearly broken.
This logic is "easy" to modify for that, but the other iteration (also
a version list) is not, as it's not using a comparison, but extracting
the value itself.
I have asked upstream in https:/ /github. com/puppetlabs/ puppet/ pull/5009
about this and your other suggestion.