Regarding sfc removal for sodium onwards.


JayaPr
 

Hi Jamo/Thanh,

 

As Faseela says

https://git.opendaylight.org/gerrit/#/c/releng/autorelease/+/84544/

Faseela K

10:47 AM

Patch Set 1:

The decision we had in TSC was to first stop running netvirt sfc csit jobs. Not to stop sfc builds right? I am still confused whether this change is supposed to be merged :

Do we get this reverted and only do changes in netvirt/.yaml files to stop the csit jobs rather than removing this module completely.

Regards

Jaya


Thanh ha <zxiiro@...>
 

I thought we decided to remove SFC as a managed project from the magnesium release and _maybe_ from the Sodium SR1 release?

In any case yes removing it from autorelease removes SFC as a managed project. If the intention was only to remove the CSIT jobs then a patch be created for the CSIT job instead.

Regards,
Thanh


On Fri, 20 Sep 2019 at 01:46, Jaya Priyadarshini <jaya.priyadarshini@...> wrote:

Hi Jamo/Thanh,

 

As Faseela says

https://git.opendaylight.org/gerrit/#/c/releng/autorelease/+/84544/

Faseela K

10:47 AM

Patch Set 1:

The decision we had in TSC was to first stop running netvirt sfc csit jobs. Not to stop sfc builds right? I am still confused whether this change is supposed to be merged :

Do we get this reverted and only do changes in netvirt/.yaml files to stop the csit jobs rather than removing this module completely.

Regards

Jaya


Luis Gomez
 

It is being long time we do not remove projects from AR/distribution, just remind the order is:

- To remove: first distribution, second AR
- To add: first AR, second distribution

It is actually weird the patch verified because I remember we had something in place to prevent the wrong order.

BR/Luis



On Sep 20, 2019, at 6:16 AM, Thanh ha <zxiiro@...> wrote:

I thought we decided to remove SFC as a managed project from the magnesium release and _maybe_ from the Sodium SR1 release?

In any case yes removing it from autorelease removes SFC as a managed project. If the intention was only to remove the CSIT jobs then a patch be created for the CSIT job instead.

Regards,
Thanh

On Fri, 20 Sep 2019 at 01:46, Jaya Priyadarshini <jaya.priyadarshini@...> wrote:

Hi Jamo/Thanh,

 

As Faseela says

https://git.opendaylight.org/gerrit/#/c/releng/autorelease/+/84544/

Faseela K

10:47 AM

Patch Set 1:

The decision we had in TSC was to first stop running netvirt sfc csit jobs. Not to stop sfc builds right? I am still confused whether this change is supposed to be merged :

Do we get this reverted and only do changes in netvirt/.yaml files to stop the csit jobs rather than removing this module completely.

Regards

Jaya

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#13724): https://lists.opendaylight.org/g/integration-dev/message/13724
Mute This Topic: https://lists.opendaylight.org/mt/34227521/1217165
Group Owner: integration-dev+owner@...
Unsubscribe: https://lists.opendaylight.org/g/integration-dev/unsub  [ecelgp@...]
-=-=-=-=-=-=-=-=-=-=-=-


Thanh ha <zxiiro@...>
 

Right totally forgot about the order but you're right I do remember we had a job in place to catch that issue. Will have to revisit and see why we didn't catch it.

Regards,
Thanh


On Fri, 20 Sep 2019 at 11:48, Luis Gomez <ecelgp@...> wrote:
It is being long time we do not remove projects from AR/distribution, just remind the order is:

- To remove: first distribution, second AR
- To add: first AR, second distribution

It is actually weird the patch verified because I remember we had something in place to prevent the wrong order.

BR/Luis



On Sep 20, 2019, at 6:16 AM, Thanh ha <zxiiro@...> wrote:

I thought we decided to remove SFC as a managed project from the magnesium release and _maybe_ from the Sodium SR1 release?

In any case yes removing it from autorelease removes SFC as a managed project. If the intention was only to remove the CSIT jobs then a patch be created for the CSIT job instead.

Regards,
Thanh

On Fri, 20 Sep 2019 at 01:46, Jaya Priyadarshini <jaya.priyadarshini@...> wrote:

Hi Jamo/Thanh,

 

As Faseela says

https://git.opendaylight.org/gerrit/#/c/releng/autorelease/+/84544/

Faseela K

10:47 AM

Patch Set 1:

The decision we had in TSC was to first stop running netvirt sfc csit jobs. Not to stop sfc builds right? I am still confused whether this change is supposed to be merged :

Do we get this reverted and only do changes in netvirt/.yaml files to stop the csit jobs rather than removing this module completely.

Regards

Jaya

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#13724): https://lists.opendaylight.org/g/integration-dev/message/13724
Mute This Topic: https://lists.opendaylight.org/mt/34227521/1217165
Group Owner: integration-dev+owner@...
Unsubscribe: https://lists.opendaylight.org/g/integration-dev/unsub  [ecelgp@...]
-=-=-=-=-=-=-=-=-=-=-=-


Faseela K
 

I just created a revert patch.

 

@Luis : Did we conclude on the discussion about SFC? Robert had raised some concerns, and was saying we should be waiting till the respective checkpoint before doing the removal for Magnesium.

 

And Jaya’s patch was on master, so I am really not sure whether it is taking care of removal in Sodium. Feel free to abandon my revert patch, if Jaya’s patch was good to go.

 

 

Thanks,

Faseela

 

From: Thanh Ha <zxiiro@...>
Sent: Friday, September 20, 2019 11:17 PM
To: Luis Gomez <ecelgp@...>
Cc: Jaya Priyadarshini <jaya.priyadarshini@...>; Jamo Luhrsen <jluhrsen@...>; netvirt-dev@...; integration-dev@...; Faseela K <faseela.k@...>
Subject: Re: [integration-dev] Regarding sfc removal for sodium onwards.

 

Right totally forgot about the order but you're right I do remember we had a job in place to catch that issue. Will have to revisit and see why we didn't catch it.

 

Regards,

Thanh

 

On Fri, 20 Sep 2019 at 11:48, Luis Gomez <ecelgp@...> wrote:

It is being long time we do not remove projects from AR/distribution, just remind the order is:

 

- To remove: first distribution, second AR

- To add: first AR, second distribution

 

It is actually weird the patch verified because I remember we had something in place to prevent the wrong order.

 

BR/Luis

 

 



On Sep 20, 2019, at 6:16 AM, Thanh ha <zxiiro@...> wrote:

 

I thought we decided to remove SFC as a managed project from the magnesium release and _maybe_ from the Sodium SR1 release?

 

In any case yes removing it from autorelease removes SFC as a managed project. If the intention was only to remove the CSIT jobs then a patch be created for the CSIT job instead.

 

Regards,

Thanh

 

On Fri, 20 Sep 2019 at 01:46, Jaya Priyadarshini <jaya.priyadarshini@...> wrote:

Hi Jamo/Thanh,

 

As Faseela says

https://git.opendaylight.org/gerrit/#/c/releng/autorelease/+/84544/

Faseela K

10:47 AM

Patch Set 1:

The decision we had in TSC was to first stop running netvirt sfc csit jobs. Not to stop sfc builds right? I am still confused whether this change is supposed to be merged :

Do we get this reverted and only do changes in netvirt/.yaml files to stop the csit jobs rather than removing this module completely.

Regards

Jaya

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#13724): https://lists.opendaylight.org/g/integration-dev/message/13724
Mute This Topic: https://lists.opendaylight.org/mt/34227521/1217165
Group Owner: integration-dev+owner@...
Unsubscribe: https://lists.opendaylight.org/g/integration-dev/unsub  [ecelgp@...]
-=-=-=-=-=-=-=-=-=-=-=-