Federico Simoncelli has posted comments on this change.
Change subject: provider: glance images listing size and limits
......................................................................
Patch Set 1:
(1 comment)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/OpenStackImageProviderProxy.java
Line 238:
Line 239: for (Image glanceImage : images) {
Line 240: RepoImage repoImage = imageToRepoImage(glanceImage);
Line 241: repoImage.setLastRefreshed(currentTime);
Line 242: repoImages.add(repoImage);
Good question, it might be better to report in some way that we're not going to
proceed and list all the remaining images.
Line 243: }
Line 244: } while((images.getList().size() >= listSize) &&
repoImages.size() < totalListSize);
Line 245:
Line 246: return repoImages;
--
To view, visit http://gerrit.ovirt.org/19223
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7b534caccbdcf121b81571384e3420f1a6efde00
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Sergey Gotliv <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
