root / svc / TODO @ ed2ee198
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 |