DaanHoogland commented on PR #10132:
URL: https://github.com/apache/cloudstack/pull/10132#issuecomment-2631326931

   > Just to have it noted here, I'll probably create an issue out of it. It 
isn't the `cloud-install-sys-tmplt` script (maybe its usage), but the actual 
problem is in:
   > 
   > 
https://github.com/apache/cloudstack/blob/cd81398cb7176698196f72002df5d6088befbaed/engine/storage/image/src/main/java/org/apache/cloudstack/storage/image/TemplateServiceImpl.java#L1242
   > 
   > For pre-seeded templates, the `template_store_ref` entries get pre filled 
with 0 and never updated, which is bad I guess for most primary storages. Using 
preseeded templates and this commit will break using those templates, until the 
DB entries are fixed.
   
   ok, that to me sounds like we shouldn't merge this untill we fix that as 
well.


-- 
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]

Reply via email to