GutoVeronezi commented on PR #8041: URL: https://github.com/apache/cloudstack/pull/8041#issuecomment-1749383613
Just for the record, following there is a quick use case to reproduce the situation with the current patch and to validate the PR: 1. create a cluster (`c1`) with a cluster-wide primary storage (`cw-pri-c1`); create another cluster (`c2`) with a cluster-wide primary storage (`cw-pri-c2`); 2. create a VM (`VM-A`) with template on `c1` (VM's `ROOT` volume must be created in storage `cw-pri-c1`); 3. create a snapshot (`m-snap-1`) of the `ROOT` volume of `VM-A`; 4. create a template (`tmpl-from-snap`) from the snapshot `m-snap-1`; 5. create a VM with template `tmpl-from-snap` on `c2`. With the current patch, step 5 is expected to fail, as hosts in `c2` does not have access to `cw-pri-c1` and template `tmpl-from-snap` will have a backing store reference to the template file in `cw-pri-c1`. With the PR, step 5 is expected to finish successfully, as `tmpl-from-snap` will not have references to any other file. -- 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]
