Re: adding parameters to create_volume function - encryption

From: Rayson Ho <no email>
Date: Wed, 2 Jul 2014 23:58:03 -0400

Hmm, I was not very clear in my previous email...

create_volume() called in volume.py is defined in the StarCluster's EasyAWS
class (awsutils.py). So if you want to pass encypted=True to boto's
create_volume(), you will need to add the extra parameter in awsutils.py:

    def create_volume(self, size, zone, snapshot_id=None):
        ...
        return self.conn.create_volume(size, zone, snapshot_id,
encypted=True)

So you just need to work your way up the call chain so that you can
determine the value passed in by the user from the command line.

(When your code is general purpose enough to be included in the StarCluster
repo, create a pull request in github.)

Rayson

==================================================
Open Grid Scheduler - The Official Open Source Grid Engine
http://gridscheduler.sourceforge.net/
http://gridscheduler.sourceforge.net/GridEngine/GridEngineCloud.html


On Wed, Jul 2, 2014 at 11:07 PM, ronen artzi <ronen.artzi_at_gmail.com> wrote:

> Thanks Rayson for your prompt response.
>
> I am running on the "Cluster Launcher/controller" node with Boto 2.30.0
> installed.
> in my code I am using (encrypt is the option I am getting from the user :
> True/False):
>
> log.info(">>>>>>>>>>>> Creating new Volume with encrypt
> option : %s" % encrypt)
> vol = self.ec2.create_volume(size, zone,
> None,None,None,encrypt,None)
> log.info("New volume id: %s" % vol.id)
>
> The error I am getting :
>
> File "/home/ubuntu/.starcluster/plugins/glusterfsencrypt.py", line 110,
> in run
> volume = self._create_ec2_volume(self.name, self.ebs_size, zone ,
> self.encrypt)
>
> File "/home/ubuntu/.starcluster/plugins/glusterfsencrypt.py", line 363, in
> _create_ec2_volume
> vol = self.ec2.create_volume(size, zone, None,None,None,encrypt)
>
> TypeError: create_volume() takes at most 4 arguments (8 given)
>
>
>
> On another note : when I am calling the create_function with
> encypted=encrypt
>
> i am getting :
>
> vol = self.ec2.create_volume(size, zone, None,encrypted=encrypt)
> TypeError: create_volume() got an unexpected keyword argument 'encrypted'
>
>
>
>
> On Wed, Jul 2, 2014 at 6:35 PM, Rayson Ho <raysonlogin_at_gmail.com> wrote:
>
>> StarCluster only needs the first 3 function parameters, thus it relies on
>> the default parameters for boto's create_volume() function. If all you want
>> is to pass in the encrypted boolean, then you can add encrypted=True to the
>> list of parameters for create_volume().
>>
>> For you code to be general purpose enough to be included, you will need
>> to add logic so that is does not always pass the encrypted=True bool into
>> the function all the time, and instead read the user's choice from the
>> command line (see createvolume.py).
>>
>> Rayson
>>
>> ==================================================
>> Open Grid Scheduler - The Official Open Source Grid Engine
>> http://gridscheduler.sourceforge.net/
>> http://gridscheduler.sourceforge.net/GridEngine/GridEngineCloud.html
>>
>>
>> On Wed, Jul 2, 2014 at 5:42 PM, ronen artzi <ronen.artzi_at_gmail.com>
>> wrote:
>>
>>> Hello,
>>>
>>> I am trying to extend a glusterfs plugin we have , and would like to
>>> introduce the options of encrypting the ESB bricks.
>>>
>>> I'd like to be able to use the full parameter list coming with ec2 fro
>>> volume creation :
>>>
>>> create_volume(*size*, *zone*, *snapshot=None*, *volume_type=None*,
>>> *iops=None*, *encrypted=False*, *dry_run=False*)
>>>
>>> Being very new to the starcluster code base I am not sure what areas are
>>> dealign with such volume creation.
>>>
>>> it seems that under star cluster python packages area , volume.py
>>> support only the first 4 parameters.
>>>
>>> Any insights here are appreciated.
>>> --
>>> Ronen
>>>
>>>
>>> _______________________________________________
>>> StarCluster mailing list
>>> StarCluster_at_mit.edu
>>> http://mailman.mit.edu/mailman/listinfo/starcluster
>>>
>>>
>>
>
>
> --
> - Ronen
> Cell: +1-508-308-2417
> My profiles: [image: LinkedIn]
> <http://www.linkedin.com/pub/ronen-artzi/0/30/4b9> [image: Tungle.me]
> <http://tungle.me/ronenartzi>
>
>
Received on Wed Jul 02 2014 - 23:58:05 EDT