This group is locked. No changes can be made to the group while it is locked.
Date
1 - 4 of 4
Exception propagation from aaa-encryption service
|
Atul Gosain <agosain@...>
Hi Stephen
toggle quoted message
Show quoted text
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-encryptionWas 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, |
|
|
|
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-encryptionWas 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 |
|
|
|
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 |
|
|