Skip to content

CLOUDSTACK-8603: Random list VM failures at scale (more than 1000 VMs…#551

Closed
koushik-das wants to merge 1 commit into
apache:masterfrom
koushik-das:CLOUDSTACK-8603
Closed

CLOUDSTACK-8603: Random list VM failures at scale (more than 1000 VMs…#551
koushik-das wants to merge 1 commit into
apache:masterfrom
koushik-das:CLOUDSTACK-8603

Conversation

@koushik-das
Copy link
Copy Markdown
Contributor

…) 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'.

…) 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'.
@asfbot
Copy link
Copy Markdown

asfbot commented Jul 1, 2015

cloudstack-pull-requests #684 SUCCESS
This pull request looks good

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Jul 3, 2015

LGTM, merging.

@asfgit asfgit closed this in 5d9f851 Jul 3, 2015
asfgit pushed a commit that referenced this pull request Jul 3, 2015
…) 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'.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

This closes #551

(cherry picked from commit 5d9f851)
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
lucas-a-martins pushed a commit to scclouds/cloudstack that referenced this pull request Jan 29, 2024
Limitar visibilidade na API `listRoles`

Closes apache#551

See merge request scclouds/scclouds!585
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants