Re: [openflowplugin-dev] [OpenDaylight TSC] [release] Weather: TSC-45: MDSAL codegen breakage landing this weekend

2018-04-02 Thread Sam Hague
Where do we stand on this now? I see the ofp patch is failing distrocheck -
is that safe to ignore?

We should move this forward as quick as possible since we are trying to get
the Fluorine managed release in place. looks like other project jobs are
also failing so we don't want to propagate other issues.

On Sat, Mar 31, 2018 at 12:26 PM, Robert Varga  wrote:

> On 31/03/18 16:35, Robert Varga wrote:
> > On 30/03/18 23:24, Robert Varga wrote:
> >> Hello everyone,
> >>
> >> this is a reminder that the weather item tracked at
> >> https://jira.opendaylight.org/browse/TSC-45 will be landing this
> >> Saturday, 3/31/2018 around 14:00 UTC.
> >>
> >> Once the two patches involved(*) are merged, all downstream projects
> >> will require re-merging current sources in project dependency order. As
> >> usual in these situations, downstreams will remain blocked by their
> >> upstreams and distribution-check is not guaranteed to run successfully
> >> until all projects in distribution are re-merged.
> >
> > This item is now underway with controller being merged right now.
>
> We have hit a snag in OFP tests, which are defining test identities in
> code. https://git.opendaylight.org/gerrit/70227 fixes them up, but
> really there should be test models so that the test identities are
> generated rather than hard-coded.
>
> distcheck for that patch is going to fail, we need it force-merged.
>
> Regards,
> Robert
>
>
> ___
> TSC mailing list
> t...@lists.opendaylight.org
> https://lists.opendaylight.org/mailman/listinfo/tsc
>
>
___
openflowplugin-dev mailing list
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev


[openflowplugin-dev] Zoom meeting for new time

2018-04-02 Thread Anil Vishnoi
Hi Casey,

Sorry for the short notice. All the openflow plugin projects committers decided 
to move the meeting to Monday 8:30 AM PST time. Can you please change the 
existing meeting to the new time and send the new invite. 

Thanks

iPhone.iTypos.iApologize
___
openflowplugin-dev mailing list
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev


Re: [openflowplugin-dev] PortReason Flag not set properly during port deletes

2018-04-02 Thread Faseela K
Arun,
  Can we get the patch merged, and then get cherry-picked to Oxygen?
  We are waiting for this, to get the dependent patches in genius to be merged.
Thanks,
Faseela

From: D Arunprakash
Sent: Thursday, March 15, 2018 5:05 PM
To: Faseela K ; Suja T ; 
openflowplugin-dev 
Cc: genius-...@lists.opendaylight.org; odl netvirt dev 

Subject: RE: PortReason Flag not set properly during port deletes

Hi Faseela,
Have raised a tentative patch for the issue, could you please check and let us 
know if it's working as expected.
https://git.opendaylight.org/gerrit/#/c/69422/

Regards,
Arun

From: Faseela K
Sent: Thursday, March 15, 2018 4:51 PM
To: Suja T >; D Arunprakash 
>; 
openflowplugin-dev 
>
Cc: 
genius-...@lists.opendaylight.org; 
odl netvirt dev 
>
Subject: RE: PortReason Flag not set properly during port deletes

Hi,
   I have raised the below JIRA to track the issue :
  https://jira.opendaylight.org/browse/OPNFLWPLUG-987

Thanks,
Faseela

From: Faseela K
Sent: Tuesday, March 13, 2018 7:56 AM
To: Suja T >; D Arunprakash 
>; 
'openflowplugin-dev' 
>
Cc: 'genius-...@lists.opendaylight.org' 
>; 
'odl netvirt dev' 
>
Subject: RE: PortReason Flag not set properly during port deletes

Suja,
   We have reverted the changes to use PortReason flag in master as well, as 
the current implementation is buggy and we have tests failing because of that.
Thanks,
Faseela

From: Faseela K
Sent: Saturday, March 10, 2018 1:31 AM
To: Suja T >; D Arunprakash 
>; 
'openflowplugin-dev' 
>
Cc: 'genius-...@lists.opendaylight.org' 
>; 
'odl netvirt dev' 
>
Subject: RE: PortReason Flag not set properly during port deletes

Just an FYI.
Suja was later on able to reproduce the issue, with some openflowplugin logs 
enabled.
@Suja : Please update if you have some conclusion on the root cause.

Thanks,
Faseela

From: Faseela K
Sent: Friday, March 09, 2018 12:31 PM
To: Suja T >; D Arunprakash 
>; 
openflowplugin-dev 
>
Cc: 
genius-...@lists.opendaylight.org; 
odl netvirt dev 
>
Subject: RE: PortReason Flag not set properly during port deletes

Suja,

   The logs which you have referred somehow gets these test cases passing, and 
that is when Reason flag is coming properly. If you check the run below where 
the test-case fails, you can see the PortReason flag coming as "Update".


https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-oxygen/50/odl_1/odl1_karaf.log.gz
  ==> tapcad3a2b8-50

I have triggered some more runs, let us see whether we will hit the error or 
not.


Thanks,
Faseela

From: Suja T
Sent: Friday, March 09, 2018 8:06 AM
To: Faseela K >; D 
Arunprakash >; 
openflowplugin-dev 
>
Cc: 
genius-...@lists.opendaylight.org; 
odl netvirt dev 
>
Subject: RE: PortReason Flag not set properly during port deletes

Hi Faseela,
We have added logs in openflowplugin and we started CSIT to identify whether we 
are receiving the proper reason and updating the reason flag with exact reason.
When there is actual port delete we are submitting the transaction with the 
reason as delete for both update and remove event.
I have verified this scenario for the problematic port which you have mentioned 
for the 

Re: [openflowplugin-dev] PortReason Flag not set properly during port deletes

2018-04-02 Thread D Arunprakash
Hi Faseela,
Waiting for openflowplugin committers to review and merge.

https://git.opendaylight.org/gerrit/#/c/69422/

Will discuss in today's community meeting and close.

Regards,
Arun

From: Faseela K
Sent: Monday, April 02, 2018 12:52 PM
To: D Arunprakash ; Suja T ; 
openflowplugin-dev 
Cc: genius-...@lists.opendaylight.org; odl netvirt dev 

Subject: RE: PortReason Flag not set properly during port deletes

Arun,
  Can we get the patch merged, and then get cherry-picked to Oxygen?
  We are waiting for this, to get the dependent patches in genius to be merged.
Thanks,
Faseela

From: D Arunprakash
Sent: Thursday, March 15, 2018 5:05 PM
To: Faseela K >; Suja T 
>; openflowplugin-dev 
>
Cc: 
genius-...@lists.opendaylight.org; 
odl netvirt dev 
>
Subject: RE: PortReason Flag not set properly during port deletes

Hi Faseela,
Have raised a tentative patch for the issue, could you please check and let us 
know if it's working as expected.
https://git.opendaylight.org/gerrit/#/c/69422/

Regards,
Arun

From: Faseela K
Sent: Thursday, March 15, 2018 4:51 PM
To: Suja T >; D Arunprakash 
>; 
openflowplugin-dev 
>
Cc: 
genius-...@lists.opendaylight.org; 
odl netvirt dev 
>
Subject: RE: PortReason Flag not set properly during port deletes

Hi,
   I have raised the below JIRA to track the issue :
  https://jira.opendaylight.org/browse/OPNFLWPLUG-987

Thanks,
Faseela

From: Faseela K
Sent: Tuesday, March 13, 2018 7:56 AM
To: Suja T >; D Arunprakash 
>; 
'openflowplugin-dev' 
>
Cc: 'genius-...@lists.opendaylight.org' 
>; 
'odl netvirt dev' 
>
Subject: RE: PortReason Flag not set properly during port deletes

Suja,
   We have reverted the changes to use PortReason flag in master as well, as 
the current implementation is buggy and we have tests failing because of that.
Thanks,
Faseela

From: Faseela K
Sent: Saturday, March 10, 2018 1:31 AM
To: Suja T >; D Arunprakash 
>; 
'openflowplugin-dev' 
>
Cc: 'genius-...@lists.opendaylight.org' 
>; 
'odl netvirt dev' 
>
Subject: RE: PortReason Flag not set properly during port deletes

Just an FYI.
Suja was later on able to reproduce the issue, with some openflowplugin logs 
enabled.
@Suja : Please update if you have some conclusion on the root cause.

Thanks,
Faseela

From: Faseela K
Sent: Friday, March 09, 2018 12:31 PM
To: Suja T >; D Arunprakash 
>; 
openflowplugin-dev 
>
Cc: 
genius-...@lists.opendaylight.org; 
odl netvirt dev 
>
Subject: RE: PortReason Flag not set properly during port deletes

Suja,

   The logs which you have referred somehow gets these test cases passing, and 
that is when Reason flag is coming properly. If you check the run below where 
the test-case fails, you can see the PortReason flag coming as "Update".


https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-oxygen/50/odl_1/odl1_karaf.log.gz
  ==> tapcad3a2b8-50

I have triggered some more runs, let us see whether we will hit the error or 
not.


Thanks,
Faseela

From: Suja T
Sent: Friday, March 09, 2018 8:06 AM
To: Faseela K >; D 
Arunprakash >; 

Re: [openflowplugin-dev] [OpenDaylight TSC] [release] Weather: TSC-45: MDSAL codegen breakage landing this weekend

2018-04-02 Thread Sam Hague
Anil, looks like the auth is back now. I was finally able to login.

On Mon, Apr 2, 2018 at 9:23 AM, Anil Vishnoi  wrote:

> It says
>
> "Authentication unavailable at this time."
>
> I didn't go through all the mails, so is anybody else facing this issue?
>
> On Mon, Apr 2, 2018 at 9:22 AM, Anil Vishnoi 
> wrote:
>
>>
>>
>> On Mon, Apr 2, 2018 at 8:58 AM, Sam Hague  wrote:
>>
>>> Where do we stand on this now? I see the ofp patch is failing
>>> distrocheck - is that safe to ignore?
>>>
>> ​I think failing distrocheck is fine.​
>>
>>
>>>
>>> We should move this forward as quick as possible since we are trying to
>>> get the Fluorine managed release in place. looks like other project jobs
>>> are also failing so we don't want to propagate other issues.
>>>
>> ​I am trying to merge this patch, but looks like there is some
>> authorization issue with gerrit? I am not able to log-in.​
>>
>>
>>>
>>> On Sat, Mar 31, 2018 at 12:26 PM, Robert Varga  wrote:
>>>
 On 31/03/18 16:35, Robert Varga wrote:
 > On 30/03/18 23:24, Robert Varga wrote:
 >> Hello everyone,
 >>
 >> this is a reminder that the weather item tracked at
 >> https://jira.opendaylight.org/browse/TSC-45 will be landing this
 >> Saturday, 3/31/2018 around 14:00 UTC.
 >>
 >> Once the two patches involved(*) are merged, all downstream projects
 >> will require re-merging current sources in project dependency order.
 As
 >> usual in these situations, downstreams will remain blocked by their
 >> upstreams and distribution-check is not guaranteed to run
 successfully
 >> until all projects in distribution are re-merged.
 >
 > This item is now underway with controller being merged right now.

 We have hit a snag in OFP tests, which are defining test identities in
 code. https://git.opendaylight.org/gerrit/70227 fixes them up, but
 really there should be test models so that the test identities are
 generated rather than hard-coded.

 distcheck for that patch is going to fail, we need it force-merged.

 Regards,
 Robert


 ___
 TSC mailing list
 t...@lists.opendaylight.org
 https://lists.opendaylight.org/mailman/listinfo/tsc


>>>
>>> ___
>>> TSC mailing list
>>> t...@lists.opendaylight.org
>>> https://lists.opendaylight.org/mailman/listinfo/tsc
>>>
>>>
>>
>>
>> --
>> Thanks
>> Anil
>>
>
>
>
> --
> Thanks
> Anil
>
___
openflowplugin-dev mailing list
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev


Re: [openflowplugin-dev] [OpenDaylight TSC] [release] Weather: TSC-45: MDSAL codegen breakage landing this weekend

2018-04-02 Thread Anil Vishnoi
On Mon, Apr 2, 2018 at 8:58 AM, Sam Hague  wrote:

> Where do we stand on this now? I see the ofp patch is failing distrocheck
> - is that safe to ignore?
>
​I think failing distrocheck is fine.​


>
> We should move this forward as quick as possible since we are trying to
> get the Fluorine managed release in place. looks like other project jobs
> are also failing so we don't want to propagate other issues.
>
​I am trying to merge this patch, but looks like there is some
authorization issue with gerrit? I am not able to log-in.​


>
> On Sat, Mar 31, 2018 at 12:26 PM, Robert Varga  wrote:
>
>> On 31/03/18 16:35, Robert Varga wrote:
>> > On 30/03/18 23:24, Robert Varga wrote:
>> >> Hello everyone,
>> >>
>> >> this is a reminder that the weather item tracked at
>> >> https://jira.opendaylight.org/browse/TSC-45 will be landing this
>> >> Saturday, 3/31/2018 around 14:00 UTC.
>> >>
>> >> Once the two patches involved(*) are merged, all downstream projects
>> >> will require re-merging current sources in project dependency order. As
>> >> usual in these situations, downstreams will remain blocked by their
>> >> upstreams and distribution-check is not guaranteed to run successfully
>> >> until all projects in distribution are re-merged.
>> >
>> > This item is now underway with controller being merged right now.
>>
>> We have hit a snag in OFP tests, which are defining test identities in
>> code. https://git.opendaylight.org/gerrit/70227 fixes them up, but
>> really there should be test models so that the test identities are
>> generated rather than hard-coded.
>>
>> distcheck for that patch is going to fail, we need it force-merged.
>>
>> Regards,
>> Robert
>>
>>
>> ___
>> TSC mailing list
>> t...@lists.opendaylight.org
>> https://lists.opendaylight.org/mailman/listinfo/tsc
>>
>>
>
> ___
> TSC mailing list
> t...@lists.opendaylight.org
> https://lists.opendaylight.org/mailman/listinfo/tsc
>
>


-- 
Thanks
Anil
___
openflowplugin-dev mailing list
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev


Re: [openflowplugin-dev] [OpenDaylight TSC] [release] Weather: TSC-45: MDSAL codegen breakage landing this weekend

2018-04-02 Thread Anil Vishnoi
I forced merge the openflowplugin patch, so i believe we are unblocked from
openflowplugin side.

Anil

On Mon, Apr 2, 2018 at 9:49 AM, Sam Hague  wrote:

> Anil, looks like the auth is back now. I was finally able to login.
>
> On Mon, Apr 2, 2018 at 9:23 AM, Anil Vishnoi 
> wrote:
>
>> It says
>>
>> "Authentication unavailable at this time."
>>
>> I didn't go through all the mails, so is anybody else facing this issue?
>>
>> On Mon, Apr 2, 2018 at 9:22 AM, Anil Vishnoi 
>> wrote:
>>
>>>
>>>
>>> On Mon, Apr 2, 2018 at 8:58 AM, Sam Hague  wrote:
>>>
 Where do we stand on this now? I see the ofp patch is failing
 distrocheck - is that safe to ignore?

>>> ​I think failing distrocheck is fine.​
>>>
>>>

 We should move this forward as quick as possible since we are trying to
 get the Fluorine managed release in place. looks like other project jobs
 are also failing so we don't want to propagate other issues.

>>> ​I am trying to merge this patch, but looks like there is some
>>> authorization issue with gerrit? I am not able to log-in.​
>>>
>>>

 On Sat, Mar 31, 2018 at 12:26 PM, Robert Varga  wrote:

> On 31/03/18 16:35, Robert Varga wrote:
> > On 30/03/18 23:24, Robert Varga wrote:
> >> Hello everyone,
> >>
> >> this is a reminder that the weather item tracked at
> >> https://jira.opendaylight.org/browse/TSC-45 will be landing this
> >> Saturday, 3/31/2018 around 14:00 UTC.
> >>
> >> Once the two patches involved(*) are merged, all downstream projects
> >> will require re-merging current sources in project dependency
> order. As
> >> usual in these situations, downstreams will remain blocked by their
> >> upstreams and distribution-check is not guaranteed to run
> successfully
> >> until all projects in distribution are re-merged.
> >
> > This item is now underway with controller being merged right now.
>
> We have hit a snag in OFP tests, which are defining test identities in
> code. https://git.opendaylight.org/gerrit/70227 fixes them up, but
> really there should be test models so that the test identities are
> generated rather than hard-coded.
>
> distcheck for that patch is going to fail, we need it force-merged.
>
> Regards,
> Robert
>
>
> ___
> TSC mailing list
> t...@lists.opendaylight.org
> https://lists.opendaylight.org/mailman/listinfo/tsc
>
>

 ___
 TSC mailing list
 t...@lists.opendaylight.org
 https://lists.opendaylight.org/mailman/listinfo/tsc


>>>
>>>
>>> --
>>> Thanks
>>> Anil
>>>
>>
>>
>>
>> --
>> Thanks
>> Anil
>>
>
>


-- 
Thanks
Anil
___
openflowplugin-dev mailing list
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev


Re: [openflowplugin-dev] [OpenDaylight TSC] [release] Weather: TSC-45: MDSAL codegen breakage landing this weekend

2018-04-02 Thread Anil Vishnoi
It says

"Authentication unavailable at this time."

I didn't go through all the mails, so is anybody else facing this issue?

On Mon, Apr 2, 2018 at 9:22 AM, Anil Vishnoi  wrote:

>
>
> On Mon, Apr 2, 2018 at 8:58 AM, Sam Hague  wrote:
>
>> Where do we stand on this now? I see the ofp patch is failing distrocheck
>> - is that safe to ignore?
>>
> ​I think failing distrocheck is fine.​
>
>
>>
>> We should move this forward as quick as possible since we are trying to
>> get the Fluorine managed release in place. looks like other project jobs
>> are also failing so we don't want to propagate other issues.
>>
> ​I am trying to merge this patch, but looks like there is some
> authorization issue with gerrit? I am not able to log-in.​
>
>
>>
>> On Sat, Mar 31, 2018 at 12:26 PM, Robert Varga  wrote:
>>
>>> On 31/03/18 16:35, Robert Varga wrote:
>>> > On 30/03/18 23:24, Robert Varga wrote:
>>> >> Hello everyone,
>>> >>
>>> >> this is a reminder that the weather item tracked at
>>> >> https://jira.opendaylight.org/browse/TSC-45 will be landing this
>>> >> Saturday, 3/31/2018 around 14:00 UTC.
>>> >>
>>> >> Once the two patches involved(*) are merged, all downstream projects
>>> >> will require re-merging current sources in project dependency order.
>>> As
>>> >> usual in these situations, downstreams will remain blocked by their
>>> >> upstreams and distribution-check is not guaranteed to run successfully
>>> >> until all projects in distribution are re-merged.
>>> >
>>> > This item is now underway with controller being merged right now.
>>>
>>> We have hit a snag in OFP tests, which are defining test identities in
>>> code. https://git.opendaylight.org/gerrit/70227 fixes them up, but
>>> really there should be test models so that the test identities are
>>> generated rather than hard-coded.
>>>
>>> distcheck for that patch is going to fail, we need it force-merged.
>>>
>>> Regards,
>>> Robert
>>>
>>>
>>> ___
>>> TSC mailing list
>>> t...@lists.opendaylight.org
>>> https://lists.opendaylight.org/mailman/listinfo/tsc
>>>
>>>
>>
>> ___
>> TSC mailing list
>> t...@lists.opendaylight.org
>> https://lists.opendaylight.org/mailman/listinfo/tsc
>>
>>
>
>
> --
> Thanks
> Anil
>



-- 
Thanks
Anil
___
openflowplugin-dev mailing list
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev


Re: [openflowplugin-dev] Zoom meeting for new time

2018-04-02 Thread Anil Vishnoi
Hi Casey,

Just a reminder.

iPhone.iTypos.iApologize

> On Apr 2, 2018, at 12:08 AM, Anil Vishnoi  wrote:
> 
> Hi Casey,
> 
> Sorry for the short notice. All the openflow plugin projects committers 
> decided to move the meeting to Monday 8:30 AM PST time. Can you please change 
> the existing meeting to the new time and send the new invite. 
> 
> Thanks
> 
> iPhone.iTypos.iApologize
___
openflowplugin-dev mailing list
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev