oms4suse / python-ceph-cfg

Simple API to help deploying ceph.
Other
7 stars 12 forks source link

Requires superuser privileges #15

Open bsanders opened 8 years ago

bsanders commented 8 years ago

In hindsight, it's a bit obvious, but perhaps worth mentioning that certain functionality requires admin access. After installing the the library, the first thing I did was ceph_cfg.partition_list(), which calls partition_table_refresh() which eventually calls parted, which hangs waiting for "you're not root, so I can't do that, do you want to retry?" input.

Unfortunately no error is raised or printed, so it took me a little digging through the source code to figure out what happened.

oms4suse commented 8 years ago

Thanks for the feed back. Expect me to work on this next week, as I have a partial fix for this already.