[mdsal-dev] Deprecated EntityOwnershipService APIs


Robert Varga <nite@...>
 

On 13/07/17 19:14, Anil Vishnoi wrote:
Hi mdsal-dev,

EntityOwnershipService APIs are deprecated in Nitrogen release, but we
That is not true. None of the APIs are deprecated.

have following two open bug that needs to be resolved for singleton
clustering service API, otherwise project like Openflowplugin/Ovsdb
can't support stable clustering solution. Currently both of these
projects are using the both EOS and singleton clustering service to
resolve these issues.

https://bugs.opendaylight.org/show_bug.cgi?id=7820 (Opened while back)
https://bugs.opendaylight.org/show_bug.cgi?id=8855 (Opened Today)

Is there any plan to fix these bugs in nitrogen cycle ?
These are enhancement requests and no, they did not fit in this
development cycle.

Regards,
Robert


Robert Varga <nite@...>
 

On 25/07/17 12:38, Tom Pantelis wrote:
On Tue, Jul 25, 2017 at 4:12 AM, Robert Varga <nite@...
<mailto:nite@...>> wrote:

On 13/07/17 19:14, Anil Vishnoi wrote:
> Hi mdsal-dev,
>
> EntityOwnershipService APIs are deprecated in Nitrogen release, but we

That is not true. None of the APIs are deprecated.


I think the original controller EOS API is deprecated in favor of the
new one 1n mdsal.
Ah, right, sorry, I mis-grepped in the controller...

Regards,
Robert


Tom Pantelis <tompantelis@...>
 



On Tue, Jul 25, 2017 at 4:12 AM, Robert Varga <nite@...> wrote:
On 13/07/17 19:14, Anil Vishnoi wrote:
> Hi mdsal-dev,
>
> EntityOwnershipService APIs are deprecated in Nitrogen release, but we

That is not true. None of the APIs are deprecated.

I think the original controller EOS API is deprecated in favor of the new one 1n mdsal.
 

> have following two open bug that needs to be resolved for singleton
> clustering service API, otherwise project like Openflowplugin/Ovsdb
> can't support stable clustering solution. Currently both of these
> projects are using the both EOS and singleton clustering service to
> resolve these issues.
>
> https://bugs.opendaylight.org/show_bug.cgi?id=7820 (Opened while back)
> https://bugs.opendaylight.org/show_bug.cgi?id=8855 (Opened Today)
>
> Is there any plan to fix these bugs in nitrogen cycle ?

These are enhancement requests and no, they did not fit in this
development cycle.

Regards,
Robert


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



Sam Hague
 



On Tue, Jul 25, 2017 at 6:31 AM, Robert Varga <nite@...> wrote:
On 25/07/17 12:38, Tom Pantelis wrote:
> On Tue, Jul 25, 2017 at 4:12 AM, Robert Varga <nite@...
> <mailto:nite@...>> wrote:
>
>     On 13/07/17 19:14, Anil Vishnoi wrote:
>     > Hi mdsal-dev,
>     >
>     > EntityOwnershipService APIs are deprecated in Nitrogen release, but we
>
>     That is not true. None of the APIs are deprecated.
>
>
> I think the original controller EOS API is deprecated in favor of the
> new one 1n mdsal.

Ah, right, sorry, I mis-grepped in the controller...
Do Anil's original questions still hold - the EOS APIs are deprecated, but we can't switch to the newer ones because of the two enhancement issues?

Regards,
Robert


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



Robert Varga <nite@...>
 

On 25/07/17 15:49, Sam Hague wrote:
> > EntityOwnershipService APIs are deprecated in Nitrogen release, but we
>
> That is not true. None of the APIs are deprecated.
>
>
> I think the original controller EOS API is deprecated in favor of the
> new one 1n mdsal.

Ah, right, sorry, I mis-grepped in the controller...

Do Anil's original questions still hold - the EOS APIs are deprecated,
but we can't switch to the newer ones because of the two enhancement issues?
Not really, as his basic premise that EOS as such is deprecated does not
hold. As noted in javadoc of
org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipService:

* @deprecated Use
org.opendaylight.mdsal.binding.api.clustering.EntityOwnershipService or
*
org.opendaylight.mdsal.dom.api.clustering.DOMEntityOwnershipService instead

Migration should a relatively straightforward search-and-replace.

Bye,
Robert


Anil Vishnoi
 



On Tue, Jul 25, 2017 at 1:12 AM, Robert Varga <nite@...> wrote:
On 13/07/17 19:14, Anil Vishnoi wrote:
> Hi mdsal-dev,
>
> EntityOwnershipService APIs are deprecated in Nitrogen release, but we

That is not true. None of the APIs are deprecated.

> have following two open bug that needs to be resolved for singleton
> clustering service API, otherwise project like Openflowplugin/Ovsdb
> can't support stable clustering solution. Currently both of these
> projects are using the both EOS and singleton clustering service to
> resolve these issues.
>
> https://bugs.opendaylight.org/show_bug.cgi?id=7820 (Opened while back)
> https://bugs.opendaylight.org/show_bug.cgi?id=8855 (Opened Today)
>
> Is there any plan to fix these bugs in nitrogen cycle ?

These are enhancement requests and no, they did not fit in this
development cycle.
​If we are considering Singleton Clustering as a alternative of EOS and want projects to use these API's, then i think these are bugs, but if we are exposing these as a ​
 
​convinient wrapper API's for users then i think these bugs as an enhancement is fine.

Regards,
Robert




--
Thanks
Anil


Anil Vishnoi
 



On Tue, Jul 25, 2017 at 7:38 AM, Robert Varga <nite@...> wrote:
On 25/07/17 15:49, Sam Hague wrote:
>     >     > EntityOwnershipService APIs are deprecated in Nitrogen release, but we
>     >
>     >     That is not true. None of the APIs are deprecated.
>     >
>     >
>     > I think the original controller EOS API is deprecated in favor of the
>     > new one 1n mdsal.
>
>     Ah, right, sorry, I mis-grepped in the controller...
>
> Do Anil's original questions still hold - the EOS APIs are deprecated,
> but we can't switch to the newer ones because of the two enhancement issues?

Not really, as his basic premise that EOS as such is deprecated does not
hold. As noted in javadoc of
org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipService:

 * @deprecated Use
org.opendaylight.mdsal.binding.api.clustering.EntityOwnershipService or
 *
org.opendaylight.mdsal.dom.api.clustering.DOMEntityOwnershipService instead

Migration should a relatively straightforward search-and-replace.
​Okay, Intellij didn't pull the java-docs from mdsal, so it was just showing ​deprecated. My bad. As far as we have EOS API, i think we are good.

Bye,
Robert


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




--
Thanks
Anil