Guest agent initial work for a postgresql instance hitting prepare() got an unexpected keyword argument 'cluster_config'

Bug #1380745 reported by Sreedhar Chidambaram
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Fix Released
Critical
Nikhil Manchanda

Bug Description

I spun off a trove instance from devstack using below command. I wanted 9.2 version of postgresql to be installed instead of 9.1. I had made required changes that I thought complete for this purpose. But guest instance ended up with version 9.1.

trove create testvm1 2 --size 2 --availability_zone nova --datastore 169774df-89f7-446d-a5ff-39a930c56db1

The trove-guestagent.log shows that the initial setup hitting an exception. Below is the error and complete trace.

2014-10-13 18:09:14.584 ERROR trove.openstack.common.rpc.amqp [^[[01;36mreq-e15a51a6-5ec5-4875-9d58-961dbbf0916b ^[[00;36m6f1997785c39498dbfd5e4c9e3b2efc6 31f90f566449478484e72f8e1073fb36] ^[[01;35mException during message handling^[[00m
2014-10-13 18:09:14.584 TRACE trove.openstack.common.rpc.amqp ^[[01;35m^[[00mTraceback (most recent call last):
2014-10-13 18:09:14.584 TRACE trove.openstack.common.rpc.amqp ^[[01;35m^[[00m File "/home/ubuntu/trove/trove/openstack/common/rpc/amqp.py", line 440, in _process_data
2014-10-13 18:09:14.584 TRACE trove.openstack.common.rpc.amqp ^[[01;35m^[[00m **args)
2014-10-13 18:09:14.584 TRACE trove.openstack.common.rpc.amqp ^[[01;35m^[[00m File "/home/ubuntu/trove/trove/openstack/common/rpc/dispatcher.py", line 172, in dispatch
2014-10-13 18:09:14.584 TRACE trove.openstack.common.rpc.amqp ^[[01;35m^[[00m result = getattr(proxyobj, method)(ctxt, **kwargs)
2014-10-13 18:09:14.584 TRACE trove.openstack.common.rpc.amqp ^[[01;35m^[[00mTypeError: prepare() got an unexpected keyword argument 'cluster_config'
2014-10-13 18:09:14.584 TRACE trove.openstack.common.rpc.amqp ^[[01;35m^[[00m

I am not sure if this exception is anything to do with the issue I am having with installing 9.2 version. By default the postgresql image that devstack downloads, installs version 9.1. I wanted the version to be 9.2 and for that I made necessary changes.

Here is what trove shows for datastore and datastore versions for postgresql.

ubuntu@trove-test:~/trove-integration/report/logs$ trove datastore-list
+--------------------------------------+--------------------+
| ID | Name |
+--------------------------------------+--------------------+
| 10000000-0000-0000-0000-000000000001 | Legacy MySQL |
| 169774df-89f7-446d-a5ff-39a930c56db1 | postgresql |
| 6a02d4da-99dd-4765-a3bc-f798155f49b7 | Test_Datastore_1 |
| 6d465049-4cd0-42e3-9398-8e36e579abcd | Inactive_Datastore |
| 93dcb542-2ee5-4bcb-bb93-508858b09f2d | mysql |
+--------------------------------------+--------------------+

ubuntu@trove-test:~/kcs$ trove datastore-version-list 169774df-89f7-446d-a5ff-39a930c56db1
+--------------------------------------+------------------+
| ID | Name |
+--------------------------------------+------------------+
| 7bf2108e-0e51-4e1c-9983-46ae41b56ae6 | inactive_version |
| a51bea07-3828-41ca-98ce-9bee2acce7f7 | 9.1 |
| b51bea07-3828-41ca-98ce-9bee2acce7f8 | 9.2 |
+--------------------------------------+------------------+

ubuntu@trove-test:~/kcs$ trove datastore-version-show b51bea07-3828-41ca-98ce-9bee2acce7f8
+-----------+--------------------------------------+
| Property | Value |
+-----------+--------------------------------------+
| active | True |
| datastore | 169774df-89f7-446d-a5ff-39a930c56db1 |
| id | b51bea07-3828-41ca-98ce-9bee2acce7f8 |
| image | 6560822f-4e1f-483d-ada1-1a1f8844d85e |
| name | 9.2 |
| packages | postgresql-9.2,pgadmin3 |
+-----------+--------------------------------------+

ubuntu@trove-test:~/kcs$ trove datastore-version-show a51bea07-3828-41ca-98ce-9bee2acce7f7
+-----------+--------------------------------------+
| Property | Value |
+-----------+--------------------------------------+
| active | False |
| datastore | 169774df-89f7-446d-a5ff-39a930c56db1 |
| id | a51bea07-3828-41ca-98ce-9bee2acce7f7 |
| image | 6560822f-4e1f-483d-ada1-1a1f8844d85e |
| name | 9.1 |
| packages | postgresql-9.1 |
+-----------+--------------------------------------+

Denis M. (dmakogon)
Changed in trove:
status: New → Confirmed
importance: Undecided → High
Changed in trove:
assignee: nobody → Sreedhar Chidambaram (sreedhara-chidambaram)
Changed in trove:
importance: High → Critical
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to trove (master)

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

Changed in trove:
status: Confirmed → In Progress
Changed in trove:
assignee: Sreedhar Chidambaram (sreedhara-chidambaram) → Nikhil Manchanda (slicknik)
tags: added: juno-rc-potential
Thierry Carrez (ttx)
Changed in trove:
milestone: none → juno-rc3
tags: removed: juno-rc-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to trove (proposed/juno)

Fix proposed to branch: proposed/juno
Review: https://review.openstack.org/128360

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

Reviewed: https://review.openstack.org/128150
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=1bb74ed0602d1dd7319f6939efc5c948b2b8402c
Submitter: Jenkins
Branch: master

commit 1bb74ed0602d1dd7319f6939efc5c948b2b8402c
Author: Sreedhar Chidambaram <email address hidden>
Date: Tue Oct 14 01:08:17 2014 +0000

    cluster_config argument missing in prepare()

    Due to a bad merge, the prepare() call in postgresql does
    not have cluster_config argument. Added that to make sure
    postgres is in line with other data stores.

    Change-Id: I1c3e5032447124dae47ae23ebf820fdcf8080588
    Closes-Bug: #1380745

Changed in trove:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to trove (proposed/juno)

Reviewed: https://review.openstack.org/128360
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=4a21bd9d24a7e2cce219e2cbd6880309c975d2de
Submitter: Jenkins
Branch: proposed/juno

commit 4a21bd9d24a7e2cce219e2cbd6880309c975d2de
Author: Sreedhar Chidambaram <email address hidden>
Date: Tue Oct 14 01:08:17 2014 +0000

    cluster_config argument missing in prepare()

    Due to a bad merge, the prepare() call in postgresql does
    not have cluster_config argument. Added that to make sure
    postgres is in line with other data stores.

    Change-Id: I1c3e5032447124dae47ae23ebf820fdcf8080588
    Closes-Bug: #1380745

Changed in trove:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in trove:
milestone: juno-rc3 → 2014.2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to trove (master)

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

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

Reviewed: https://review.openstack.org/128900
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=54785d1b12d13db7545cd24cbc39381c377e3059
Submitter: Jenkins
Branch: master

commit 4a21bd9d24a7e2cce219e2cbd6880309c975d2de
Author: Sreedhar Chidambaram <email address hidden>
Date: Tue Oct 14 01:08:17 2014 +0000

    cluster_config argument missing in prepare()

    Due to a bad merge, the prepare() call in postgresql does
    not have cluster_config argument. Added that to make sure
    postgres is in line with other data stores.

    Change-Id: I1c3e5032447124dae47ae23ebf820fdcf8080588
    Closes-Bug: #1380745

commit 08af8e029d6e3d785f6387eef294eafad28ad3e1
Author: Craig Vyvial <email address hidden>
Date: Mon Oct 13 10:56:44 2014 -0500

    restart_required cfg-param is bool and not string

    There was a break to the previous version of the configuration
    paramenters API when the configurations parameters were moved to
    the database. The actual type of the restart_required parameter
    should be a JSON boolean value, and not a string to be compatible
    with the previous configuration parameters API.

    Change-Id: If45944e25a55813ddf8f80010811da04d667ca19
    Closes-Bug: #1379665

commit e94c68c7861ea82324f8078ccfbc3037ef286d99
Author: Craig Vyvial <email address hidden>
Date: Fri Oct 10 10:07:29 2014 -0500

    convert restart_required to 'true' or 'false' string

    The output of the configuration-parameters-list/show was
    inadvertently changed when the parameters were moved into
    the database in Juno.

    Change-Id: Ic7d215482f5a9ab8ba2059cae7b91a99b35f2795
    Closes-Bug: #1379665

commit 8af6a9adaad83bfbc3c26107dc82c444fd9aa1b4
Author: Nikhil Manchanda <email address hidden>
Date: Fri Oct 10 07:53:24 2014 +0000

    Refreshed translations

    Updated translations by running the following:
    $ python setup.py extract_messages
    $ python setup.py update_catalog --no-fuzzy-matching \
      --ignore-obsolete=true
    $ source \
      ../infra/project-config/jenkins/scripts/common_translation_update.sh
    $ setup_loglevel_vars
    $ cleanup_po_files trove

    Change-Id: I2fcb4e3a0714a73a7026ce2d2c3ffde8aaee1236

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.