On Tue, Jun 11, 2019 at 12:41:23PM +0200, Jan Tluka wrote:
Tue, Jun 11, 2019 at 11:22:12AM CEST, olichtne(a)redhat.com wrote:
>From: Ondrej Lichtner <olichtne(a)redhat.com>
>
>Hi,
>
>the following patchset is a collection of various minor fixes and a semi
>large refactoring of the netlink related methods of the Device class
>hierarchy.
>
>These were implemented as a preparation for the following patchset that
>refactors and redesigns the ENRT recipes hierarchy to both enable a more
>flexible recipe design and to enable an easy generation of test_wide
>configuration and sub configuration descriptions.
>
>-Ondrej
>
>Ondrej Lichtner (9):
> lnst.Tests.Ping: don't fail due to stderr
> lnst.Devices.Device: refactor nl_sync methods
> lnst.Controller.Job: add host info to __str__
> lnst.Controller.RecipeResults: reduce the level of DeviceConfigResults
> lnst.Devices.Device: minor adaptive coalescing improvement
> lnst.Common.IpAddress: add flags attribute
> lnst.Common.IpAddress: add is_tentative attribute
> lnst.Devices.Device: add flags to parsed ip addresses
> lnst.Controller.RecipeControl: add timeout parameter to
> wait_for_condition
>
> lnst/Common/IpAddress.py | 28 +++++---
> lnst/Controller/Job.py | 2 +
> lnst/Controller/RecipeControl.py | 4 +-
> lnst/Controller/RecipeResults.py | 2 +-
> lnst/Devices/BondDevice.py | 52 +++++++--------
> lnst/Devices/BridgeDevice.py | 6 +-
> lnst/Devices/Device.py | 109 +++++++++++++------------------
> lnst/Devices/MacvlanDevice.py | 4 +-
> lnst/Devices/SoftDevice.py | 2 +-
> lnst/Devices/VethDevice.py | 2 +-
> lnst/Devices/VlanDevice.py | 4 +-
> lnst/Devices/VtiDevice.py | 10 +--
> lnst/Devices/VxlanDevice.py | 10 +--
> lnst/Tests/Ping.py | 1 -
> 14 files changed, 114 insertions(+), 122 deletions(-)
>
>--
>2.22.0
>_______________________________________________
>LNST-developers mailing list -- lnst-developers(a)lists.fedorahosted.org
>To unsubscribe send an email to lnst-developers-leave(a)lists.fedorahosted.org
>Fedora Code of Conduct:
https://getfedora.org/code-of-conduct.html
>List Guidelines:
https://fedoraproject.org/wiki/Mailing_list_guidelines
>List Archives:
https://lists.fedorahosted.org/archives/list/lnst-developers@lists.fedora...
This one looks good.
Acked-by: Jan Tluka <jtluka(a)redhat.com>
thanks for the review, pushed.
-Ondrej