allow the engine more time to update the OVFSTORE

arachmani commented 4 years ago

I'm not sure why we need that, we need to check with @eslutsky.

eslutsky commented 4 years ago

i was investigating some race condition with PR, it didn't prove to solve anything. closing PR

kobihk commented 4 years ago

I just realized that I have the exact failure because of a short timeout. in the failure I can see: \"description\": \"{\\"Updated\\":false,\\"Last Updated\\":\\"Wed Jul 01 11:18:33 IDT 2020\\",\\"Storage Domains\\":[{\\"uuid\\":\\"b7c64b97-0c98-47c4-a81e-2955e781e384\\"}],\\"Disk Description\\":\\"OVF_STORE\\"}\",

but when I run it manually after a while I got: "description": "{\"Updated\":true,\"Size\":22528,\"Last Updated\":\"Wed Jul 01 11:43:07 IDT 2020\",\"Storage Domains\":[{\"uuid\":\"b7c64b97-0c98-47c4-a81e-2955e781e384\"}],\"Disk Description\":\"OVF_STORE\"}",

maybe it will be better to increase the retries

oVirt / ovirt-ansible-hosted-engine-setup

allow the engine more time to update the OVFSTORE #335