ravening commented on pull request #4078:
URL: https://github.com/apache/cloudstack/pull/4078#issuecomment-630071057


   > @ravening, I have seen problems with re-downloads of templates and the 
main issue was not the URL but the checksum stored. Did you encounter this? I 
don't see how the URL of a template would be the problem, unless I don't 
understand your description.
   > Also CLSTACK-7189 is an internal issue for you guys isn't it? Can you 
create a github issue describing the issue and rename the PR? (using the 
hashtag-issue contruct to point to the public github issue)
   > finally, you are speaking of both volumes and templates in the PR but only 
of templates in the title. Just a remark, i'm sure the source will make it 
clear, but not everybody will read the source :(
   
   @dahn, When we download volume/template, it generates a download_url link in 
the table. When we destroy the secondary storage vm, the download_url still 
exists in the table but it is not valid anymore as we cant download the 
template/volume from the same link.
   
   So when we destroy the SSVM, we clean up the download_url link so that it 
will generate a new download_url link when template/volume is downloaded again.
   
   This applies for both volumes and templates.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to