You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2021/02/25 10:01:08 UTC
[GitHub] [cloudstack] andrijapanicsb opened a new issue #4729: XenServer, management server sends pool UUID instead of the PATH, as the SR name
andrijapanicsb opened a new issue #4729:
URL: https://github.com/apache/cloudstack/issues/4729
<!--
Verify first that your issue/request is not already reported on GitHub.
Also test if the latest release and master branch are affected too.
Always add information AFTER of these HTML comments, but no need to delete the comments.
-->
##### ISSUE TYPE
<!-- Pick one below and delete the rest -->
* Bug Report
##### COMPONENT NAME
<!--
Categorize the issue, e.g. API, VR, VPN, UI, etc.
-->
~~~
~~~
##### CLOUDSTACK VERSION
<!--
New line separated list of affected versions, commit ID for issues on master branch.
-->
~~~
4.14.0.0 tested
~~~
##### CONFIGURATION
<!--
Information about the configuration if relevant, e.g. basic network, advanced networking, etc. N/A otherwise
-->
Original Cloudplatform 4.7.1 with XenServer with preSetup storage (iscis) - all this is less relevant, see down
##### OS / ENVIRONMENT
<!--
Information about the environment if relevant, N/A otherwise
-->
NA
##### SUMMARY
<!-- Explain the problem/feature briefly -->
After upgrading ACP 4.7.1 to ACS 4.14.0.0, preSetup storage type can not be mounted on XS servers, as ACS DB is sending UUID (instead of the pool name) to the XS to mount it - see details below
DB upgraded to 4.14, but the same data is in 4.7.1 before the upgrade - storage_pool table:
id: 2
name: XenServer-Primary
**uuid: 1e189b97-974d-35b6-bc76-50bca7704ea0**
pool_type: PreSetup
port: 0
data_center_id: 2
pod_id: 2
cluster_id: 3
used_bytes: 28898754560
capacity_bytes: 214714810368
host_address: localhost
user_info: NULL
path: /XEN_PRIMARY
created: 2021-02-24 10:38:37
removed: NULL
update_time: NULL
status: Maintenance
storage_provider_name: DefaultPrimary
scope: CLUSTER
hypervisor: NULL
managed: 0
capacity_iops: NULL
In CLoudPlatform 4.7.1. this used to work fine, "PATH" filed was sent as the SR name to the XS to mount it.
In CloudStack 4.14.0.0, mgmt server sends UUID (1e189b97-974d-35b6-bc76-50bca7704ea0) as the SR name parameter for XS when mounting it, instead of "path" (XEN_PRIMARY), so the SR mount fails:
2021-02-25 03:49:45,306 INFO [c.c.v.VirtualMachineManagerImpl] (Work-Job-Executor-3:ctx-7a7ee26b job-118/job-119 ctx-6e3dd2e5) (logid:edf9d62a) Unable to contact resource.
com.cloud.exception.StorageUnavailableException: Resource [StoragePool:2] is unreachable: Unable to create Vol[26|vm=26|ROOT]:**There are 0 SRs with same name: 1e189b97-974d-35b6-bc76-50bca7704ea0**
The workaround was (this is customer env) to replace UUID value with the value of the "PATH" field so that XS can mount it (what consequences this might have, is unknown)
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [cloudstack] andrijapanicsb commented on issue #4729: XenServer, management server sends pool UUID instead of the PATH, as the SR name
Posted by GitBox <gi...@apache.org>.
andrijapanicsb commented on issue #4729:
URL: https://github.com/apache/cloudstack/issues/4729#issuecomment-791369928
Current code puts the XS SR name (when doing preSetup storage) into the "UUID" field in DB...
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd closed issue #4729: XenServer, management server sends pool UUID instead of the PATH, as the SR name
Posted by GitBox <gi...@apache.org>.
rhtyd closed issue #4729:
URL: https://github.com/apache/cloudstack/issues/4729
--
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:
users@infra.apache.org
[GitHub] [cloudstack] DaanHoogland commented on issue #4729: XenServer, management server sends pool UUID instead of the PATH, as the SR name
Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on issue #4729:
URL: https://github.com/apache/cloudstack/issues/4729#issuecomment-785781822
@rhtyd @shwstppr , not sure if this should go in a point release. Looks like an extra version of the XS Resource(Base) it might be needed. cc @andrijapanicsb
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [cloudstack] rhtyd commented on issue #4729: XenServer, management server sends pool UUID instead of the PATH, as the SR name
Posted by GitBox <gi...@apache.org>.
rhtyd commented on issue #4729:
URL: https://github.com/apache/cloudstack/issues/4729#issuecomment-813285886
Fixed in https://github.com/apache/cloudstack/pull/4816
--
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:
users@infra.apache.org