Identity Change Hook Error: TypeError: <charm.openstack.designate.DesignateConfigurationAdapter object> is not JSON serializable

Bug #1756412 reported by Eric Vasquez
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Designate Charm
Fix Released
Critical
David Ames

Bug Description

Keystone is requiring a relation change and waits on designate.
Designate seems to blow up during configuration.

Log snippet output from designate here:
https://pastebin.canonical.com/p/3DY9cvxR6w

Revision history for this message
Eric Vasquez (envas) wrote :
tags: added: cdo-qa foundations-engine
removed: cdoqa
Revision history for this message
Alexander Litvinov (alitvinov) wrote :

Facing this issue infield

Ryan Beisner (1chb1n)
Changed in charm-designate:
milestone: none → 18.05
importance: Undecided → Critical
assignee: nobody → Alex Kavanagh (ajkavanagh)
status: New → Confirmed
Revision history for this message
Alex Kavanagh (ajkavanagh) wrote :

The bug was in charm-helpers (a recent introduces feature to fix key order problems in hashing). PR#136 (https://github.com/juju/charm-helpers/pull/136) just merged should fix it, so that designate charm just needs to be rebuilt and tested with keystone.

Revision history for this message
Alexander Litvinov (alitvinov) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-designate (master)

Fix proposed to branch: master
Review: https://review.openstack.org/554247

Changed in charm-designate:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to charm-designate (stable/18.02)

Related fix proposed to branch: stable/18.02
Review: https://review.openstack.org/554249

Ryan Beisner (1chb1n)
Changed in charm-designate:
assignee: Alex Kavanagh (ajkavanagh) → David Ames (thedac)
Revision history for this message
David Ames (thedac) wrote :

Still waiting on OSCI runs for the designate forced rebuilds.

https://review.openstack.org/#/c/554247/
https://review.openstack.org/#/c/554249/

Revision history for this message
David Ames (thedac) wrote :

The following are merging and should be in the charm store in the next 20-30 minutes.

https://review.openstack.org/#/c/554247/
https://review.openstack.org/#/c/554249/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-designate (master)

Reviewed: https://review.openstack.org/554247
Committed: https://git.openstack.org/cgit/openstack/charm-designate/commit/?id=293fc11bc0d4f8f18467a4bebec251db57e76300
Submitter: Zuul
Branch: master

commit 293fc11bc0d4f8f18467a4bebec251db57e76300
Author: Alex Kavanagh <email address hidden>
Date: Mon Mar 19 14:48:13 2018 +0000

    Rebuild charm to get charm-helpers 0.18.7

    This is to fix a regression introduced in charm-helpers 0.18.6
    which caused designate to crash when it tried access cached hookenv
    config in charms.openstack ConfigurationAdapter class

    Change-Id: I5813229481f660b6da8918fa4d72f78610ea3b55
    Closes-Bug: #1756412

Changed in charm-designate:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to charm-designate (stable/18.02)

Reviewed: https://review.openstack.org/554249
Committed: https://git.openstack.org/cgit/openstack/charm-designate/commit/?id=82d2b9236b993935ed1ca3113fc1ebd6566f8695
Submitter: Zuul
Branch: stable/18.02

commit 82d2b9236b993935ed1ca3113fc1ebd6566f8695
Author: Alex Kavanagh <email address hidden>
Date: Mon Mar 19 14:56:36 2018 +0000

    Rebuild charm to get charm-helpers 0.18.7 for stable build

    This is to fix a regression introduced in charm-helpers 0.18.6
    which caused designate to crash when it tried access cached hookenv
    config in charms.openstack ConfigurationAdapter class

    Note: related bug below, as fix is in master.

    Change-Id: I6041c0eacf09d7fda1529f0f657bf6fe12b8b3e7
    Related-Bug: #1756412

Liam Young (gnuoy)
Changed in charm-designate:
status: Fix Committed → Fix Released
Revision history for this message
Alex Kavanagh (ajkavanagh) wrote :

The charmstore has the updated charm built with charmhelpers 0.18.7. Can we confirm onsite please so we can close the bug and/or reduce the severity please?

Changed in charm-designate:
status: Fix Released → Fix Committed
Changed in charm-designate:
status: Fix Committed → In Progress
Revision history for this message
Alexander Litvinov (alitvinov) wrote :
Revision history for this message
Alexander Litvinov (alitvinov) wrote :

/var/log dump from unit designate/0

Revision history for this message
David Ames (thedac) wrote :

Alex Litvinov,

Can you please re-check. I just pulled designate down from the charm store and ran a test I used that showed this bug, and it is no longer showing it.

Please make sure the charm is not cached.

Revision history for this message
Alexander Litvinov (alitvinov) wrote :

David,
It is working now with 18th charm.
Thank you for a quick fix

David Ames (thedac)
Changed in charm-designate:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.