search

zipurman / oVIRT_Simple_Backup

A REST API backup using PHP for oVirt 4.2.x
GNU General Public License v3.0
56 stars 32 forks source link

vms list fails to display when too many vms #50

Open natman opened 5 years ago

natman commented 5 years ago

Expected behaviour

I want to do a single backup on datacenters that contains a lot of vm (between 100 and 150). List vms should display so as to select the vm I want to backup

Actual behaviour

blank page is returned

Steps to reproduct the behaviour

works as expected for smaller datacenters (about 40 vms) whereas it is long to display.

zipurman commented 5 years ago

Check the logs of the engine while you are trying to generate the list. It may be getting throttled.

On the engine command line: tail /var/log/ovirt-/.log -f

simpleBackup currently does a single API call for each request, but it also re-authenticates each time. It's on the list to authenticate once which may also solve this issue.

You may also want to try the following on the command line of the simpleBackupVM while trying to generate the list:

tail /var/log/apache2/error.log -f

natman commented 5 years ago

nothing into logs... I tried to set some authentication options thanks to the ovirt-aaa-jdbc-tool, but nothing does the trick. What is strange, I'm able to list the same huge number of VMs without issue when chosing scheduled backups instead of single. The issue may come from more attributes fetched in the single process (comment and cluster). Some other behaviour that could help is that after listed vms with scheduled backup, I'm able to list them with single backup.

andresin commented 5 years ago

Same problem here. I guess it is just a timeout issue, as the Single Backup page loads after long time when asked directly to the VMs IP.

zipurman commented 5 years ago

I will see what I can do to speed this area up as well as extend the timeout issue.

zipurman commented 5 years ago

I just pushed out a change that should help. I was loading in disk sizes for each VM in the list. Using the API, I have to ask for each set of disks for each vm, which on long lists could take a while. I removed the disk size from that page but it will still be shown on the next page once a VM is selected.

Let me know if that improves things.

andresin commented 5 years ago

Those changes fixed the issue for me. Thanks!

andresin commented 5 years ago

In "Scheduled Backups", we have the same issue on "New Schedule" or when editing an "Scheduled Backup Name"

zipurman commented 5 years ago

Just pushed out an update to correct that ;)