weizhouapache commented on PR #6699: URL: https://github.com/apache/cloudstack/pull/6699#issuecomment-1241599433
> > @DK101010 > > multiple local storage is supported since #6147 can you test it ? > > @weizhouapache Damn, the last time I checked the PRs, I didn't see that one.... never mind > > Hmm ... you implemented this via agent properties and my implementation works via UI like the other storages. For user acceptance reasons I prefer my solution. User can use local storages like other storage types without touch agent properties. Also no restarts required. Excuse me for insisting, but It will be hard to explain my users that they have to manipulate the agent properties instead to use the ui to add local storages but for other storages they have to use the ui. :/ > > Perhaps we find a compromise. But I'm afraid that our implementations are disjunct. Do you have an idea? @DK101010 The reason I choose `agent properties` is, this can only be operated by administrator. they can do it by automation tools like chef, ansible, etc. agree with you it would be nice to support it via api and UI, it will be more user-friendly. for example (1) add host with multiple local storage pools on UI (2) list all local storage pools of a host on UI (of course API as well) (3) add local storage pool to a host (create libvirt storage pool, update db and agent.properties) (4) remove local storage from a host (remove libvirt storage pool, update db and agent.properties) 1 is optional if 2,3,4 are supported. we can start by adding a host with only 1 local storage pool. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
