Skip to content
  • P
    Projects
  • G
    Groups
  • S
    Snippets
  • Help

CIRCLE / cloud

  • This project
    • Loading...
  • Sign in
Go to a project
  • Project
  • Repository
  • Issues 94
  • Merge Requests 10
  • Pipelines
  • Wiki
  • Snippets
  • Members
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Merged
Merge request !23 opened Mar 06, 2014 by Guba Sándor@gubasandor 
  • Report abuse
Report abuse

Refactoring save as function

⚠ Requires support in circle/storagedriver (at least storagedriver/94379fb).

  • ✅ add InstanceTemplate.save_as/_async
  • ✅ #24 (closed) create disk from iso url
  • ✅ add Disk.create (replace all Disk.objects.create and Disk.init calls)
  • ✅ add Disk.create_empty
  • Discussion 6
  • Commits 25
  • Changes
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Őry Máté
    @orymate started a discussion on the diff Mar 06, 2014
    Last updated by Őry Máté Mar 13, 2014
    circle/storage/models.py
    65 65 TYPES = [('qcow2-norm', 'qcow2 normal'), ('qcow2-snap', 'qcow2 snapshot'),
    66 66 ('iso', 'iso'), ('raw-ro', 'raw read-only'), ('raw-rw', 'raw')]
    67 67 name = CharField(blank=True, max_length=100, verbose_name=_("name"))
    68 filename = CharField(max_length=256, verbose_name=_("filename"))
    68 filename = CharField(max_length=256, unique=True,
    • Őry Máté @orymate commented Mar 06, 2014
      Owner

      Migration is missing?

      Migration is missing?
    • Őry Máté @orymate commented Mar 13, 2014
      Owner

      fixed @ b644bc7c

      Edited Mar 13, 2014
      fixed @ b644bc7c8
    Please register or sign in to reply
  • Őry Máté
    @orymate started a discussion on the diff Mar 06, 2014
    circle/storage/models.py
    403 393 local_tasks.restore.apply_async(args=[self, user],
    404 394 queue='localhost.man')
    405 395  
    406 def save_as(self, user=None, task_uuid=None, timeout=120):
    396 def save_as_async(self, disk, task_uuid=None, timeout=300, user=None):
    397 return local_tasks.save_as.apply_async(args=[disk, timeout, user],
    398 queue="localhost.man")
    399
    400 def save_as(self, user=None, task_uuid=None, timeout=300):
    • Őry Máté @orymate commented Mar 06, 2014
      Owner

      Is this timeout of 5 minutes normally enough?

      Is this timeout of 5 minutes normally enough?
    Please register or sign in to reply
  • Őry Máté
    @orymate started a discussion on the diff Mar 06, 2014
    circle/vm/models/instance.py
    1001 # save template
    1002 tmpl.save()
    1003 try:
    1004 # create interface templates
    1005 for i in self.interface_set.all():
    1006 i.save_as_template(tmpl)
    1007 except:
    1008 tmpl.delete()
    1009 raise
    1010 else:
    1011 return tmpl
    975 def save_as_template(self, name, task_uuid=None, user=None,
    976 timeout=300, **kwargs):
    977 with instance_activity(code_suffix="save_as_template", instance=self,
    978 task_uuid=task_uuid, user=user):
    979 # prepare parameters
    • Őry Máté @orymate commented Mar 06, 2014
      Owner

      defaults = {...}, defaults.update(kwargs) ?

      `defaults = {...}, defaults.update(kwargs)` ?
    Please register or sign in to reply
  • Őry Máté @orymate commented Mar 09, 2014
    Owner

    ping

    ping
  • Kálmán Viktor @kviktor commented Mar 09, 2014
    Owner

    Oh, I'm assigned :o

    Oh, I'm assigned :o
  • Őry Máté
    @orymate started a discussion on commit 6014f0de Mar 13, 2014
    • Őry Máté @orymate

      mentioned in merge request !23 (merged)

      Mar 13, 2014

      mentioned in merge request !23 (merged)

      mentioned in merge request !23
      Toggle commit list
    Please register or sign in to reply
  • Őry Máté
    @orymate started a discussion on commit b644bc7c Mar 13, 2014
    • Őry Máté @orymate

      mentioned in merge request !23 (merged)

      Mar 13, 2014

      mentioned in merge request !23 (merged)

      mentioned in merge request !23
      Toggle commit list
    Please register or sign in to reply
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Guba Sándor
Assignee
Guba Sándor @gubasandor
Assign to
Pilot deploy
Milestone
Pilot deploy
Assign milestone
Time tracking
0
Labels
None
Assign labels
  • View labels
3
3 participants
Reference: circle/cloud!23