[Pulp-list] Failure migrating database on 2.8.3 upgrade

[Sparks, Alan]

Appreciate the fast reply.   Was able to work around it similarly to your suggestion (actually wrapped the call in a try/except, your idea is cleaner).
Was able to complete the migration and get back running.   Thanks!
-Alan

-----Original Message-----
From: Randy Barlow [mailto:rbarlow at redhat.com] 
Sent: Wednesday, May 18, 2016 3:22 PM
To: Sparks, Alan <alan.l.sparks at hpe.com>
Cc: pulp-list at redhat.com
Subject: Re: [Pulp-list] Failure migrating database on 2.8.3 upgrade

On Wed, May 18, 2016 at 08:24:46PM +0000, Sparks, Alan wrote:
> Not sure what I've done wrong, but migrating after my 2.8.3 package updates from 2.8.2, I am getting the following error:
> 
> Applying pulp.server.db.migrations version 23 Applying migration 
> pulp.server.db.migrations.0023_importer_tls_storage failed.
> 
> Halting migrations due to a migration failure.
> 'config'
> Traceback (most recent call last):
>   File "/usr/lib/python2.6/site-packages/pulp/server/db/manage.py", line 193, in main
>     return _auto_manage_db(options)
>   File "/usr/lib/python2.6/site-packages/pulp/server/db/manage.py", line 256, in _auto_manage_db
>     migrate_database(options)
>   File "/usr/lib/python2.6/site-packages/pulp/server/db/manage.py", line 124, in migrate_database
>     update_current_version=not options.test)
>   File "/usr/lib/python2.6/site-packages/pulp/server/db/migrate/models.py", line 186, in apply_migration
>     migration.migrate()
>   File "/usr/lib/python2.6/site-packages/pulp/server/db/migrations/0023_importer_tls_storage.py", line 25, in migrate
>     _write_pem_file(pki_path, importer['config'], key, filename)
> KeyError: 'config'
> 
> 
> Any advice how to debug why I seem to be missing some config item?
> -Alan

> _______________________________________________
> Pulp-list mailing list
> Pulp-list at redhat.com
> https://www.redhat.com/mailman/listinfo/pulp-list
Hello Alan!

I've filed an issue about this for you:

https://pulp.plan.io/issues/1929

I don't think it's a problem that you are missing the config item, but an issue in the code. The fix will probably be something like:

if 'config' in importer:
    _write_pem_file(pki_path, importer['config'], key, filename)

If you are comfortable working in Python, you could try something like that at line 25 in /usr/lib/python2.6/site-packages/pulp/server/db/migrations/0023_importer_tls_storage.py

I apologize for the assumption and the inconvenience!

--
Randy Barlow
irc:   bowlofeggs