Re: PR 2670: Removing an old, unused NetApp plug-in

[Tutkowski, Mike]

Just an FYI that I merged this PR into master today.

https://github.com/apache/cloudstack/pull/2670

On 5/24/18, 1:41 AM, "Rohit Yadav"  wrote:

+1 on removal of unused plugin including removal of related files, db 
tables and any global settings.


- Rohit

<https://cloudstack.apache.org>




From: Tutkowski, Mike 
Sent: Thursday, May 24, 2018 1:06:52 PM
To: dev@cloudstack.apache.org; us...@cloudstack.apache.org
    Subject: Re: PR 2670: Removing an old, unused NetApp plug-in

Including users@ on this (below).


rohit.ya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
  
 

On May 23, 2018, at 12:53 PM, Tutkowski, Mike 
mailto:mike.tutkow...@netapp.com>> wrote:

Hi everyone,

I opened the following PR today:

https://github.com/apache/cloudstack/pull/2670

There is an old, what we believe to be unused, NetApp plug-in in the 
CloudStack codebase that I’d like to remove.

I’m sending this e-mail out for the following reasons:


1)   To see if, by chance, anyone is actually using this plug-in.

2)   To direct any potential code reviewers to this PR, where I have 
placed a couple questions about my commit.

To be absolutely clear here, this has nothing to do with any of the 
SolidFire plug-ins (only an old NetApp plug-in that hasn’t been updated in 
years and is probably not being used by anyone).

Thanks,
Mike

Re: PR 2670: Removing an old, unused NetApp plug-in

[Rohit Yadav]

+1 on removal of unused plugin including removal of related files, db tables 
and any global settings.


- Rohit

<https://cloudstack.apache.org>




From: Tutkowski, Mike <mike.tutkow...@netapp.com>
Sent: Thursday, May 24, 2018 1:06:52 PM
To: dev@cloudstack.apache.org; us...@cloudstack.apache.org
Subject: Re: PR 2670: Removing an old, unused NetApp plug-in

Including users@ on this (below).


rohit.ya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
  
 

On May 23, 2018, at 12:53 PM, Tutkowski, Mike 
<mike.tutkow...@netapp.com<mailto:mike.tutkow...@netapp.com>> wrote:

Hi everyone,

I opened the following PR today:

https://github.com/apache/cloudstack/pull/2670

There is an old, what we believe to be unused, NetApp plug-in in the CloudStack 
codebase that I’d like to remove.

I’m sending this e-mail out for the following reasons:


1)   To see if, by chance, anyone is actually using this plug-in.

2)   To direct any potential code reviewers to this PR, where I have placed 
a couple questions about my commit.

To be absolutely clear here, this has nothing to do with any of the SolidFire 
plug-ins (only an old NetApp plug-in that hasn’t been updated in years and is 
probably not being used by anyone).

Thanks,
Mike

Re: PR 2670: Removing an old, unused NetApp plug-in

[Tutkowski, Mike]

Including users@ on this (below).

On May 23, 2018, at 12:53 PM, Tutkowski, Mike 
> wrote:

Hi everyone,

I opened the following PR today:

https://github.com/apache/cloudstack/pull/2670

There is an old, what we believe to be unused, NetApp plug-in in the CloudStack 
codebase that I’d like to remove.

I’m sending this e-mail out for the following reasons:


1)   To see if, by chance, anyone is actually using this plug-in.

2)   To direct any potential code reviewers to this PR, where I have placed 
a couple questions about my commit.

To be absolutely clear here, this has nothing to do with any of the SolidFire 
plug-ins (only an old NetApp plug-in that hasn’t been updated in years and is 
probably not being used by anyone).

Thanks,
Mike

Re: PR 2670: Removing an old, unused NetApp plug-in

[Tutkowski, Mike]

Yeah, Paul, that JAR would no longer be needed.

> On May 23, 2018, at 1:01 PM, Paul Angus <paul.an...@shapeblue.com> wrote:
> 
> +1
> 
> There's a NetApp Jar in the nodist which is related I think - be nice to get 
> rid of that too.
> 
> paul.an...@shapeblue.com 
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue
> 
> 
> 
> 
> -Original Message-
> From: Tutkowski, Mike <mike.tutkow...@netapp.com> 
> Sent: 23 May 2018 19:53
> To: dev@cloudstack.apache.org
> Subject: PR 2670: Removing an old, unused NetApp plug-in
> 
> Hi everyone,
> 
> I opened the following PR today:
> 
> https://github.com/apache/cloudstack/pull/2670
> 
> There is an old, what we believe to be unused, NetApp plug-in in the 
> CloudStack codebase that I’d like to remove.
> 
> I’m sending this e-mail out for the following reasons:
> 
> 
> 1)   To see if, by chance, anyone is actually using this plug-in.
> 
> 2)   To direct any potential code reviewers to this PR, where I have 
> placed a couple questions about my commit.
> 
> To be absolutely clear here, this has nothing to do with any of the SolidFire 
> plug-ins (only an old NetApp plug-in that hasn’t been updated in years and is 
> probably not being used by anyone).
> 
> Thanks,
> Mike

Re: PR 2670: Removing an old, unused NetApp plug-in

[Tutkowski, Mike]

I actually have no idea, Rafael. Nor does there seem to be anyone remaining at 
NetApp who knows about that plug-in.

> On May 23, 2018, at 1:22 PM, Rafael Weingärtner <rafaelweingart...@gmail.com> 
> wrote:
> 
> Those APi methods that you are removing as well (AssociateLunCmd.java,
> CreateLunCmd.java, CreateVolumeOnFilerCmd.java,  CreateVolumePoolCmd.java,
> DeleteVolumePoolCmd.java, DestroyLunCmd.java, DestroyVolumeOnFilerCmd.java,
> DissociateLunCmd.java, ListLunsCmd.java, ListVolumePoolsCmd.java, and so
> on), are they ony used for this NetApp plugin? It seems a considerable
> number of external API methods being removed.
> 
> I also have another question. Are all of these methods broken? I mean, do
> they work?
> 
>> On Wed, May 23, 2018 at 4:13 PM, Will Stevens <wstev...@cloudops.com> wrote:
>> 
>> +1
>> 
>> On Wed, May 23, 2018, 3:03 PM Gabriel Beims Bräscher, <
>> gabr...@pcextreme.nl>
>> wrote:
>> 
>>> I am +1 on removing it.
>>> 
>>>> On May 23, 2018 at 8:59 PM Paul Angus <paul.an...@shapeblue.com>
>> wrote:
>>>> 
>>>> 
>>>> +1
>>>> 
>>>> There's a NetApp Jar in the nodist which is related I think - be nice
>> to
>>> get rid of that too.
>>>> 
>>>> paul.an...@shapeblue.com
>>>> www.shapeblue.com
>>>> 53 Chandos Place, Covent Garden, London WC2N 4HSUK
>>>> @shapeblue
>>>> 
>>>> 
>>>> 
>>>> 
>>>> -Original Message-
>>>> From: Tutkowski, Mike <mike.tutkow...@netapp.com>
>>>> Sent: 23 May 2018 19:53
>>>> To: dev@cloudstack.apache.org
>>>> Subject: PR 2670: Removing an old, unused NetApp plug-in
>>>> 
>>>> Hi everyone,
>>>> 
>>>> I opened the following PR today:
>>>> 
>>>> https://github.com/apache/cloudstack/pull/2670
>>>> 
>>>> There is an old, what we believe to be unused, NetApp plug-in in the
>>> CloudStack codebase that I’d like to remove.
>>>> 
>>>> I’m sending this e-mail out for the following reasons:
>>>> 
>>>> 
>>>> 1) To see if, by chance, anyone is actually using this plug-in.
>>>> 
>>>> 2) To direct any potential code reviewers to this PR, where I have
>>> placed a couple questions about my commit.
>>>> 
>>>> To be absolutely clear here, this has nothing to do with any of the
>>> SolidFire plug-ins (only an old NetApp plug-in that hasn’t been updated
>> in
>>> years and is probably not being used by anyone).
>>>> 
>>>> Thanks,
>>>> Mike
>>> 
>> 
> 
> 
> 
> -- 
> Rafael Weingärtner

Re: PR 2670: Removing an old, unused NetApp plug-in

[Rafael Weingärtner]

Those APi methods that you are removing as well (AssociateLunCmd.java,
CreateLunCmd.java, CreateVolumeOnFilerCmd.java,  CreateVolumePoolCmd.java,
DeleteVolumePoolCmd.java, DestroyLunCmd.java, DestroyVolumeOnFilerCmd.java,
DissociateLunCmd.java, ListLunsCmd.java, ListVolumePoolsCmd.java, and so
on), are they ony used for this NetApp plugin? It seems a considerable
number of external API methods being removed.

I also have another question. Are all of these methods broken? I mean, do
they work?

On Wed, May 23, 2018 at 4:13 PM, Will Stevens <wstev...@cloudops.com> wrote:

> +1
>
> On Wed, May 23, 2018, 3:03 PM Gabriel Beims Bräscher, <
> gabr...@pcextreme.nl>
> wrote:
>
> > I am +1 on removing it.
> >
> > > On May 23, 2018 at 8:59 PM Paul Angus <paul.an...@shapeblue.com>
> wrote:
> > >
> > >
> > > +1
> > >
> > > There's a NetApp Jar in the nodist which is related I think - be nice
> to
> > get rid of that too.
> > >
> > > paul.an...@shapeblue.com
> > > www.shapeblue.com
> > > 53 Chandos Place, Covent Garden, London WC2N 4HSUK
> > > @shapeblue
> > >
> > >
> > >
> > >
> > > -Original Message-
> > > From: Tutkowski, Mike <mike.tutkow...@netapp.com>
> > > Sent: 23 May 2018 19:53
> > > To: dev@cloudstack.apache.org
> > > Subject: PR 2670: Removing an old, unused NetApp plug-in
> > >
> > > Hi everyone,
> > >
> > > I opened the following PR today:
> > >
> > > https://github.com/apache/cloudstack/pull/2670
> > >
> > > There is an old, what we believe to be unused, NetApp plug-in in the
> > CloudStack codebase that I’d like to remove.
> > >
> > > I’m sending this e-mail out for the following reasons:
> > >
> > >
> > > 1) To see if, by chance, anyone is actually using this plug-in.
> > >
> > > 2) To direct any potential code reviewers to this PR, where I have
> > placed a couple questions about my commit.
> > >
> > > To be absolutely clear here, this has nothing to do with any of the
> > SolidFire plug-ins (only an old NetApp plug-in that hasn’t been updated
> in
> > years and is probably not being used by anyone).
> > >
> > > Thanks,
> > > Mike
> >
>



-- 
Rafael Weingärtner

Re: PR 2670: Removing an old, unused NetApp plug-in

[Will Stevens]

+1

On Wed, May 23, 2018, 3:03 PM Gabriel Beims Bräscher, <gabr...@pcextreme.nl>
wrote:

> I am +1 on removing it.
>
> > On May 23, 2018 at 8:59 PM Paul Angus <paul.an...@shapeblue.com> wrote:
> >
> >
> > +1
> >
> > There's a NetApp Jar in the nodist which is related I think - be nice to
> get rid of that too.
> >
> > paul.an...@shapeblue.com
> > www.shapeblue.com
> > 53 Chandos Place, Covent Garden, London WC2N 4HSUK
> > @shapeblue
> >
> >
> >
> >
> > -Original Message-
> > From: Tutkowski, Mike <mike.tutkow...@netapp.com>
> > Sent: 23 May 2018 19:53
> > To: dev@cloudstack.apache.org
> > Subject: PR 2670: Removing an old, unused NetApp plug-in
> >
> > Hi everyone,
> >
> > I opened the following PR today:
> >
> > https://github.com/apache/cloudstack/pull/2670
> >
> > There is an old, what we believe to be unused, NetApp plug-in in the
> CloudStack codebase that I’d like to remove.
> >
> > I’m sending this e-mail out for the following reasons:
> >
> >
> > 1) To see if, by chance, anyone is actually using this plug-in.
> >
> > 2) To direct any potential code reviewers to this PR, where I have
> placed a couple questions about my commit.
> >
> > To be absolutely clear here, this has nothing to do with any of the
> SolidFire plug-ins (only an old NetApp plug-in that hasn’t been updated in
> years and is probably not being used by anyone).
> >
> > Thanks,
> > Mike
>

RE: PR 2670: Removing an old, unused NetApp plug-in

[Gabriel Beims Bräscher]

I am +1 on removing it.

> On May 23, 2018 at 8:59 PM Paul Angus <paul.an...@shapeblue.com> wrote:
>
>
> +1
>
> There's a NetApp Jar in the nodist which is related I think - be nice to get 
> rid of that too.
>
> paul.an...@shapeblue.com 
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London WC2N 4HSUK
> @shapeblue
>
>
>
>
> -Original Message-
> From: Tutkowski, Mike <mike.tutkow...@netapp.com>
> Sent: 23 May 2018 19:53
> To: dev@cloudstack.apache.org
> Subject: PR 2670: Removing an old, unused NetApp plug-in
>
> Hi everyone,
>
> I opened the following PR today:
>
> https://github.com/apache/cloudstack/pull/2670
>
> There is an old, what we believe to be unused, NetApp plug-in in the 
> CloudStack codebase that I’d like to remove.
>
> I’m sending this e-mail out for the following reasons:
>
>
> 1) To see if, by chance, anyone is actually using this plug-in.
>
> 2) To direct any potential code reviewers to this PR, where I have placed a 
> couple questions about my commit.
>
> To be absolutely clear here, this has nothing to do with any of the SolidFire 
> plug-ins (only an old NetApp plug-in that hasn’t been updated in years and is 
> probably not being used by anyone).
>
> Thanks,
> Mike

RE: PR 2670: Removing an old, unused NetApp plug-in

[Paul Angus]

+1

There's a NetApp Jar in the nodist which is related I think - be nice to get 
rid of that too.

paul.an...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
  
 


-Original Message-
From: Tutkowski, Mike <mike.tutkow...@netapp.com> 
Sent: 23 May 2018 19:53
To: dev@cloudstack.apache.org
Subject: PR 2670: Removing an old, unused NetApp plug-in

Hi everyone,

I opened the following PR today:

https://github.com/apache/cloudstack/pull/2670

There is an old, what we believe to be unused, NetApp plug-in in the CloudStack 
codebase that I’d like to remove.

I’m sending this e-mail out for the following reasons:


1)   To see if, by chance, anyone is actually using this plug-in.

2)   To direct any potential code reviewers to this PR, where I have placed 
a couple questions about my commit.

To be absolutely clear here, this has nothing to do with any of the SolidFire 
plug-ins (only an old NetApp plug-in that hasn’t been updated in years and is 
probably not being used by anyone).

Thanks,
Mike

PR 2670: Removing an old, unused NetApp plug-in

[Tutkowski, Mike]

Hi everyone,

I opened the following PR today:

https://github.com/apache/cloudstack/pull/2670

There is an old, what we believe to be unused, NetApp plug-in in the CloudStack 
codebase that I’d like to remove.

I’m sending this e-mail out for the following reasons:


1)   To see if, by chance, anyone is actually using this plug-in.

2)   To direct any potential code reviewers to this PR, where I have placed 
a couple questions about my commit.

To be absolutely clear here, this has nothing to do with any of the SolidFire 
plug-ins (only an old NetApp plug-in that hasn’t been updated in years and is 
probably not being used by anyone).

Thanks,
Mike