You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by koushik-das <gi...@git.apache.org> on 2015/07/01 19:50:38 UTC
[GitHub] cloudstack pull request: CLOUDSTACK-8603: Random list VM failures ...
GitHub user koushik-das opened a pull request:
https://github.com/apache/cloudstack/pull/551
CLOUDSTACK-8603: Random list VM failures at scale (more than 1000 VMs…
…) when VM has resource tags
There is no 'removed' field on the resource_tags table. So 'id' based search may return a record or null in case record is deleted.
Added a check for null or empty in search resource tags based on 'id'.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/koushik-das/cloudstack CLOUDSTACK-8603
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/551.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #551
----
commit 9f8ba5fcc8dc7d96774190bf44f848336dc06d9b
Author: Koushik Das <ko...@apache.org>
Date: 2015-07-01T17:47:46Z
CLOUDSTACK-8603: Random list VM failures at scale (more than 1000 VMs) when VM has resource tags
There is no 'removed' field on the resource_tags table. So 'id' based search may return a record or null in case record is deleted.
Added a check for null or empty in search resource tags based on 'id'.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack pull request: CLOUDSTACK-8603: Random list VM failures ...
Posted by bhaisaab <gi...@git.apache.org>.
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/551#issuecomment-118303081
LGTM, merging.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack pull request: CLOUDSTACK-8603: Random list VM failures ...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/551
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---