[ovirt-devel] upgrade script failing in latest master

[Greg Padgett]

On 07/27/2015 03:24 PM, Alexander Wels wrote:
> On Monday, July 27, 2015 03:07:19 PM Greg Padgett wrote:
>
>  > On 07/27/2015 02:48 PM, Alexander Wels wrote:
>
>  > > Hi,
>
>  > >
>
>  > > When I run engine-setup in the latest master I get the following error in
>
>  > > my setup log:
>
>  > Hi Alexander,
>
>  >
>
>  > It looks like a bug. The upgrade script casts the
>
>  > vm_static.dedicated_vm_for_vds value to a UUID, but in VmBaseDaoDbFacade
>
>  > we see that this field may consist of a comma-delimited list of UUIDs.
>
>  >
>
>  > To work around it for now you can update the offending
>
>  > vm_static.dedicated_vm_for_vds values to have only a single UUID.
>
>  >
>
>  > Unfortunately I don't know the "proper" fix (guessing vm_host_pinning_map
>
>  > needs one value per UUID in dedicated_vm_for_vds?), but this should at
>
>  > least get you up and running.
>
>  >
>
> Aha, that tells me what the problem is, I have been working on a UI change
> to allow one to select multiple hosts to run a VM on. And I believe the
> result is a comma separated list of host ids. I had one of my VMs still
> marked for 2 hosts, and as soon as I changed that to one host, this
> problem went away.

Glad it worked.

> Of course immediately a new one showed up...
>
> 2015-07-27 15:11:47 DEBUG otopi.context context._executeMethod:141 Stage
> misc METHOD
> otopi.plugins.ovirt_**FILTERED**_setup.ovirt_**FILTERED**.config.public_glance_repository.Plugin._misc
>
> 2015-07-27 15:11:47 DEBUG
> otopi.ovirt_**FILTERED**_setup.**FILTERED**_common.database
> database.execute:171 Database: 'None', Statement: '
>                 select inst_add_glance_provider(
>                     %(provider_id)s,
>                     %(provider_name)s,
>                     %(provider_description)s,
>                     %(provider_url)s,
>                     %(storage_domain_id)s
>                 )
>             ', args: {'provider_url': 'http://glance.ovirt.org:9292',
> 'provider_name': 'ovirt-image-repository', 'storage_domain_id':
> '072fbaa1-08f3-4a40-9f34-a5ca22dd1d74', 'provider_description': 'Publ
> ic Glance repository for oVirt', 'provider_id':
> 'ceab03af-7220-4d42-8f5c-9b557f5d29af'}
> 2015-07-27 15:11:47 DEBUG otopi.context context._executeMethod:155 method
> exception
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/site-packages/otopi/context.py", line 145, in
> _executeMethod
>     method['method']()
>   File
> "/home/awels/ovirt-**FILTERED**/share/ovirt-**FILTERED**/setup/bin/../plugins/ovirt-**FILTERED**-setup/ovirt-**FILTERED**/config/public_glance_repository.py",
> line 77, in _misc
>     storage_domain_id="072fbaa1-08f3-4a40-9f34-a5ca22dd1d74"
>   File
> "/home/awels/ovirt-**FILTERED**/share/ovirt-**FILTERED**/setup/ovirt_**FILTERED**_setup/**FILTERED**_common/database.py",
> line 195, in execute
>     args,
> ProgrammingError: function inst_add_glance_provider(unknown, unknown,
> unknown, unknown, unknown) does not exist
> LINE 2:                 select inst_add_glance_provider(
>                                ^
> HINT:  No function matches the given name and argument types. You might
> need to add explicit type casts.
>
> 2015-07-27 15:11:47 ERRORotopi.context context._executeMethod:164 Failed
> to execute stage 'Misc configuration': function
> inst_add_glance_provider(unknown, unknown, unknown, unknown, unknown) does
> not ex
> ist
> LINE 2:                 select inst_add_glance_provider(
>                                ^
> HINT:  No function matches the given name and argument types. You might
> need to add explicit type casts.
>
> Again, this is strange as I do have that function in my database, and it
> looks like the appropriate fields are there.

I saw this before too, and I suspect it's a symptom of the db update 
failing--but I never nailed down the root cause, thinking it was something 
unique I did to myself.  It went away on its own just as unexpectedly as 
it had arrived.  While it plagued me, I worked around it by modifying 
public_glance_repository.py to return early.