Project

General

Profile

Actions

Task #7283

closed

Task #6869: Challenge OpenStack and OpenNebula with ucloud

Cleanups for ucloud 0.0.2

Added by Nico Schottelius about 5 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
-
Start date:
11/01/2019
Due date:
% Done:

0%

Estimated time:
PM Check date:

Description

  • host/main.py: do not hard code /var/vm -> read this from a key in etcd
    • Not sure if we want to use threading or multiprocess - I'd say the latter is probably easier
  • don't just call main(), protect it by if name == '__main__'!
Actions #1

Updated by Nico Schottelius about 5 years ago

  • Description updated (diff)
Actions #2

Updated by Ahmed Bilal about 5 years ago

Nico Schottelius wrote:

  • host/main.py: do not hard code /var/vm -> read this from a key in etcd
    • Not sure if we want to use threading or multiprocess - I'd say the latter is probably easier
  • don't just call main(), protect it by if name '__main__'!
  • I already removed hard coded thingies but didn't pushed them yet.
  • Threading should be used in our case because we want to share variables e.g etcd_client handle
  • name '__main__' DONE.
Actions #3

Updated by Nico Schottelius about 5 years ago

Not sure if I agree with you in terms of sharing - why would we want to
share the etcd_client handle and not reallocate/take a new one?

writes:

Actions #4

Updated by Ahmed Bilal about 5 years ago

We can create new ones, but suppose we want to change some parameters i.e passing certificates path when opening connection to etcd key/value store than we have to change it in two places :). Still, if you say I would do it with processes. It may be better. Don't know yet.

Actions #5

Updated by Ahmed Bilal about 5 years ago

Now, using process instead of threads.

Actions #6

Updated by Ahmed Bilal about 5 years ago

  • Status changed from New to Resolved
Actions #7

Updated by Ahmed Bilal almost 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF