[virt-tools-list] [PATCH virt-viewer v2] ovirt-foreign-menu: Fix endpoint for storage domains query

Eduardo Lima (Etrunko) etrunko at redhat.com
Mon Aug 12 14:00:14 UTC 2019


On 8/12/19 9:48 AM, Victor Toso wrote:
> On Mon, Aug 12, 2019 at 09:36:39AM -0300, Eduardo Lima (Etrunko) wrote:
>> Instead of fetching toplevel REST API query, we use the one relative
>> from the data center, which returns more detailed information,
>> especially the status of the storage domain.
>>
>> This fixes https://bugzilla.redhat.com/show_bug.cgi?id=1427467
>>
>> Signed-off-by: Eduardo Lima (Etrunko) <etrunko at redhat.com>
>> ---
>>
>> in V2:
>>  - Improve usage of boolean domain_valid
> 
> Sure,
> Acked-by: Victor Toso <victortoso at redhat.com>

Thanks, pushed.

> 
>>
>> ---
>>  src/ovirt-foreign-menu.c | 26 ++++++++++++++++++++------
>>  1 file changed, 20 insertions(+), 6 deletions(-)
>>
>> diff --git a/src/ovirt-foreign-menu.c b/src/ovirt-foreign-menu.c
>> index 59c3d48..75f9ac4 100644
>> --- a/src/ovirt-foreign-menu.c
>> +++ b/src/ovirt-foreign-menu.c
>> @@ -683,6 +683,7 @@ static void storage_domains_fetched_cb(GObject *source_object,
>>      OvirtCollection *collection = OVIRT_COLLECTION(source_object);
>>      GHashTableIter iter;
>>      OvirtStorageDomain *domain;
>> +    gboolean domain_valid = FALSE;
>>  
>>      ovirt_collection_fetch_finish(collection, result, &error);
>>      if (error != NULL) {
>> @@ -696,7 +697,8 @@ static void storage_domains_fetched_cb(GObject *source_object,
>>      while (g_hash_table_iter_next(&iter, NULL, (gpointer *)&domain)) {
>>          OvirtCollection *file_collection;
>>  
>> -        if (!storage_domain_validate(menu, domain))
>> +        domain_valid = storage_domain_validate(menu, domain);
>> +        if (!domain_valid)
>>              continue;
>>  
>>          file_collection = ovirt_storage_domain_get_files(domain);
>> @@ -713,9 +715,11 @@ static void storage_domains_fetched_cb(GObject *source_object,
>>      if (menu->priv->files != NULL) {
>>          ovirt_foreign_menu_next_async_step(menu, task, STATE_STORAGE_DOMAIN);
>>      } else {
>> -        g_debug("Could not find iso file collection");
>> -        g_task_return_new_error(task, OVIRT_ERROR, OVIRT_ERROR_FAILED,
>> -                                "Could not find ISO file collection");
>> +        const char *msg = domain_valid ? "Could not find ISO file collection"
>> +                                       : "Could not find valid ISO storage domain";
>> +
>> +        g_debug(msg);
>> +        g_task_return_new_error(task, OVIRT_ERROR, OVIRT_ERROR_FAILED, msg);
>>          g_object_unref(task);
>>      }
>>  }
>> @@ -724,9 +728,19 @@ static void storage_domains_fetched_cb(GObject *source_object,
>>  static void ovirt_foreign_menu_fetch_storage_domain_async(OvirtForeignMenu *menu,
>>                                                            GTask *task)
>>  {
>> -    OvirtCollection *collection = ovirt_api_get_storage_domains(menu->priv->api);
>> +    OvirtCollection *collection = NULL;
>> +
>> +#ifdef HAVE_OVIRT_DATA_CENTER
>> +    g_return_if_fail(OVIRT_IS_FOREIGN_MENU(menu));
>> +    g_return_if_fail(OVIRT_IS_PROXY(menu->priv->proxy));
>> +    g_return_if_fail(OVIRT_IS_DATA_CENTER(menu->priv->data_center));
>> +
>> +    collection = ovirt_data_center_get_storage_domains(menu->priv->data_center);
>> +#else
>> +    collection = ovirt_api_get_storage_domains(menu->priv->api);
>> +#endif
>>  
>> -    g_debug("Start fetching oVirt REST collection");
>> +    g_debug("Start fetching iso file collection");
>>      ovirt_collection_fetch_async(collection, menu->priv->proxy,
>>                                   g_task_get_cancellable(task),
>>                                   storage_domains_fetched_cb, task);
>> -- 
>> 2.21.0
>>
>> _______________________________________________
>> virt-tools-list mailing list
>> virt-tools-list at redhat.com
>> https://www.redhat.com/mailman/listinfo/virt-tools-list


-- 
Eduardo de Barros Lima (Etrunko)
Software Engineer - Red Hat
etrunko at redhat.com

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20190812/68885ee7/attachment.sig>


More information about the virt-tools-list mailing list