Statistics
| Branch: | Revision:

root / svc_extstorage / TODO @ 2eba2338

History | View | Annotate | Download (1.3 kB)

1
- Fix _is_vdisk_defined (see FIXME comment there)
2
- Code duplication (subprocess wrappers, anything else?)
3
- Config file opts 'parsing' (too complex? )
4
- Consistent use of datatypes (int for VOL_SIZE, port, host_id, string
5
  for everything else). Related to the above.
6
- Make consistent use of exceptions / consistent error handling
7
- What are the semantics of the verify op?
8
- Refactoring:
9
    *) _get_params / _read_config are not 'consistent' (set attrs and
10
       return some opts for SVC connection)
11
    *) make the interface more clear (how do we init the provider,
12
       SVC connection, how to we issue the ops, do we need oop /
13
       classes
14
- Check code taken from the original cinder driver (driver_assert,
15
  paramiko code, svc cmds)
16
- Verify multipath / dmsetup / scsi rescan cmds work as intended
17
  (and that they work ok with udev, when certain rules are disabled)
18
- Lots of rtts to the SVC for no reason (is_vdisk_defined, get_attrs
19
  etc). Maybe have a separate volume class (overkill) or private attrs
20
  for volume specific stuff?
21
- Unittests probably need rewriting from scratch
22
- We currently rely on rescan-scsi-bus etc for scanning. Check which
23
  package provides it (probably scsi-tools). Fallback to manual
24
  scanning (via sysfs files) if not available?
25
- Fix vdisk attribute getting (wrapper fun to return SVCError on key
26
  error / malformed vdiks attrs