
On Tue, Oct 9, 2018 at 4:13 PM Gianluca Cecchi <gianluca.cecchi@gmail.com> wrote:
On Thu, Oct 4, 2018 at 12:37 PM Dan Kenigsberg <danken@redhat.com> wrote:
I've identified 3 ancient vdsm-hooks that have been obsoleted by proper oVirt features.
vdsm-hook-isolatedvlan: obsoleted in ovirt-4.2.6 by clean-traffic-gateway filter https://gerrit.ovirt.org/#/q/I396243e1943eca245ab4da64bb286da19f9b47ec
Hello Dan, you mention "clean-traffic-gateway" filter and "clean-traffic" filter. They are to be considered different, correct? Because both in my upstream oVirt engine 4.2.6.4-1.el7 and in my RHV engine 4.2.6.4-0.1.el7ev I only see the "clean-traffic" one. Perhaps it is "filtered out" because the hosts are not 7.6, as I see from https://bugzilla.redhat.com/show_bug.cgi?id=1603115 that it is a libvirt feature to be backported to 7.6, so not available in 7.5 perhaps...?
If so, as RH EL 7.6 is not out yet, and then it will pass some weeks/months at least before CentOS 7.6 will be released and guys updating the hypervisors, I think it's not time yet to remove the old vdsm-hook-isolatedvlan
Any clarification if I was wrong?
No, you are not wrong. I intend to remove this hook only from the master branch, destined for ovirt-4.3. I believe that CentOS-7.6 would be released much before ovirt-4.3, but there is no need to start gambling. These patches can wait. Are you using vdsm-hook-isolatedvlan? Would you agree to check if clean-traffic-gateway answers your needs? All you need is to download https://libvirt.org/git/?p=libvirt.git;a=blob;f=examples/xml/nwfilter/clean-... and then virsh nwfilter-define /tmp/clean-traffic-gateway.xml on each host.