On Sat, Apr 21, 2018 at 6:22 PM, Sam Hague <[email protected]> wrote:

> Well, maybe we can keep the change.. I ran ovsdb [1] and netvirt csit [2]
> using the ovsdb patch Vishal had for the issue and they both look to have
> passed. NetVirt relies on ofp, ovsdb and genius so that seems to show
> things work. I ran this in the last couple hours using the ovsdb distro
> from yesterday which should have been before Tom reverted the patch. If
> someone could verify my comment that the artifacts used in the CSIT run
> truly do have the changes, then maybe we can keep this change.
>

That's good that it passed. But folks were asking to revert so that's what
I did. If someone wants to "unrevert" it and drive that and ensure there's
no other breakages, that's fine.



>
> Thanks, Sam
>
> [1] https://jenkins.opendaylight.org/releng/job/ovsdb-patch-
> test-core-fluorine/16/
>
> [2] https://jenkins.opendaylight.org/releng/job/netvirt-
> csit-1node-openstack-queens-gate-stateful-fluorine/236
>
> On Sat, Apr 21, 2018 at 7:13 AM, Abhijit Kumbhare <[email protected]>
> wrote:
>
>> Thanks Robert.
>>
>> On Sat, Apr 21, 2018 at 5:27 AM, Robert Varga <[email protected]> wrote:
>>
>>> On 21/04/18 01:49, Robert Varga wrote:
>>> >> The weather process
>>> >> needed to have been followed - with the possibility for downstream
>>> >> projects to not accept the change for legitimate reasons. Please do so
>>> >> in the future.
>>> >>
>>> >> The most ideal solution as suggested by Luis below & initially agreed
>>> by
>>> >> Tom would have been to back out the change, discuss it completely (in
>>> a
>>> >> the TWS call) and go ahead with the decision after the TWS call. As it
>>> >> stands, and pointed by Tom, backing out the change looks to be a
>>> >> daunting task. Since that is the case, let us do the following:
>>> >> 0) Unblock the projects by whatever means - whether it is projects
>>> >> merging the changes to accommodate the original patch or reverting
>>> >> 1) Robert, please create the weather report with the existing change
>>> > Sorry for the delay, today has been exceptionally busy, I will file a
>>> > weather item tomorrow.
>>> >
>>>
>>> https://jira.opendaylight.org/browse/TSC-96
>>>
>>> Regards,
>>> Robert
>>>
>>>
>>
>> _______________________________________________
>> openflowplugin-dev mailing list
>> [email protected]
>> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
>>
>>
>
> _______________________________________________
> release mailing list
> [email protected]
> https://lists.opendaylight.org/mailman/listinfo/release
>
>
_______________________________________________
openflowplugin-dev mailing list
[email protected]
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev

Reply via email to