Exception propagation from aaa-encryption service


Atul Gosain <agosain@...>
 

Hi All 


Here is the patch to propagate the exceptions from aaa-encryption service in AAA. This is as per the conversations regarding the change on kernel call today. 


https://git.opendaylight.org/gerrit/60742 ​


Please review and merge if appropriate. 


Thanks

Atul


Stephen Kitt <skitt@...>
 

Hi Atul,

On Tue, 25 Jul 2017 19:52:08 +0000
Atul Gosain <agosain@...> wrote:
Here is the patch to propagate the exceptions from aaa-encryption
service in AAA. This is as per the conversations regarding the change
on kernel call today.


https://git.opendaylight.org/gerrit/60742 ?


Please review and merge if appropriate.
Was this really intended for Carbon rather than Nitrogen? It’s a big
API change for a stable branch, and is causing build failures in other
projects (e.g. https://git.opendaylight.org/gerrit/60936).

Regards,

Stephen


Ryan Goulding <ryandgoulding@...>
 

I don't think this belongs in Carbon;  just Nitrogen as discussed on last weeks Kernel projects call.  I am going to revert it in Carbon.

Regards,

Ryan Goulding

On Mon, Jul 31, 2017 at 11:12 AM, Stephen Kitt <skitt@...> wrote:
Hi Atul,

On Tue, 25 Jul 2017 19:52:08 +0000
Atul Gosain <agosain@...> wrote:
> Here is the patch to propagate the exceptions from aaa-encryption
> service in AAA. This is as per the conversations regarding the change
> on kernel call today.
>
>
> https://git.opendaylight.org/gerrit/60742 ?
>
>
> Please review and merge if appropriate.

Was this really intended for Carbon rather than Nitrogen? It’s a big
API change for a stable branch, and is causing build failures in other
projects (e.g. https://git.opendaylight.org/gerrit/60936).

Regards,

Stephen

_______________________________________________
aaa-dev mailing list
aaa-dev@...
https://lists.opendaylight.org/mailman/listinfo/aaa-dev



Atul Gosain <agosain@...>
 

Hi Stephen

I needed this change in one of the netconf password encryption patches (for carbon and nitrogen) and changed it after discussing on the kernel call. I will take care of not making API changes to stable branches now. As Ryan has already reverted this patch from carbon, I will go ahead with the plan for putting it on Nitrogen and hopefully there will be enough time to resolve downstream dependencies.

Thanks
Atul

-----Original Message-----
From: Stephen Kitt [mailto:skitt@...]
Sent: Monday, July 31, 2017 8:12 AM
To: Atul Gosain <agosain@...>
Cc: aaa-dev@...; release@...
Subject: Re: [Aaa-dev] Exception propagation from aaa-encryption service

Hi Atul,

On Tue, 25 Jul 2017 19:52:08 +0000
Atul Gosain <agosain@...> wrote:
Here is the patch to propagate the exceptions from aaa-encryption
service in AAA. This is as per the conversations regarding the change
on kernel call today.


https://git.opendaylight.org/gerrit/60742 ?


Please review and merge if appropriate.
Was this really intended for Carbon rather than Nitrogen? It’s a big API change for a stable branch, and is causing build failures in other projects (e.g. https://git.opendaylight.org/gerrit/60936).

Regards,

Stephen