You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by "andrijapanicsb (via GitHub)" <gi...@apache.org> on 2023/12/20 22:34:11 UTC

[I] Improvement on collecting/updating Secondary Storage total size/capacity [cloudstack]

andrijapanicsb opened a new issue, #8391:
URL: https://github.com/apache/cloudstack/issues/8391

   Improvement request:
   
   - the capacity of the Secondary Storage seem updated/obtained from time to time, by SSVMs and the value (capacity) is held in the memory of the mgmt server.
   
   In certain cases, this becomes a very... useless... design, where the values can not be altered in case of edge cases where the Secondary Storage capacity needs to be expanded, but SSVMs are not running/existing, for whatever reason.
   
   Proposal:
   - store the capacity of the SS in the DB (perhaps op_host_capacity table, like for the Primary data, and other resource values)
   - Enable a global setting that will control the interval of the capacity collection from the NFS Secondary Storage
   - Consider offloading the work to the mgmt server, as per architecture design, all mgmt servers MUST be able to mount the secondary storage NFS shares (for various other tasks)
   - Consider implementing an explicit API call that will trigger updating the SS NFS capacities (in cases where the backend capacity has been expanded)


-- 
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: commits-unsubscribe@cloudstack.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Improvement on collecting/updating Secondary Storage total size/capacity [cloudstack]

Posted by "andrijapanicsb (via GitHub)" <gi...@apache.org>.
andrijapanicsb commented on issue #8391:
URL: https://github.com/apache/cloudstack/issues/8391#issuecomment-2097749602

   As discussed, consider the value from the DB during the boot of the mgmt server (instead of having null or zero value, whatever is the case currently), but when the stats collectors runs for the next time (when SSVM returns the size of the SS), then update in-memory capacity as well as the capacity inside the DB (that was set manually/via API/UI).


-- 
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: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Improvement on collecting/updating Secondary Storage total size/capacity [cloudstack]

Posted by "rohityadavcloud (via GitHub)" <gi...@apache.org>.
rohityadavcloud closed issue #8391: Improvement on collecting/updating Secondary Storage total size/capacity
URL: https://github.com/apache/cloudstack/issues/8391


-- 
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: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Improvement on collecting/updating Secondary Storage total size/capacity [cloudstack]

Posted by "DaanHoogland (via GitHub)" <gi...@apache.org>.
DaanHoogland commented on issue #8391:
URL: https://github.com/apache/cloudstack/issues/8391#issuecomment-2084622960

   ​@anddrijapanicsb, so far I implemented a API/UI change to update the value in the DB, but the value returned is still coming from the StatsCollector. When a manual change has been done that disagrees with the actual/discovered size, which do we consider when/on what grounds? 
   


-- 
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: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Improvement on collecting/updating Secondary Storage total size/capacity [cloudstack]

Posted by "rohityadavcloud (via GitHub)" <gi...@apache.org>.
rohityadavcloud commented on issue #8391:
URL: https://github.com/apache/cloudstack/issues/8391#issuecomment-2175304329

   Fixed by https://github.com/apache/cloudstack/pull/8941 tested by @kiranchavala
   @andrijapanicsb pl review and discuss if this is enough, re-open the issue if more work in needed.


-- 
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: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org