Magnum

tempest init is throwing warning UserWarning: converting 'True' to a string with Magnum tempest plugin

Bug #1685797 reported by chandan kumar on 2017-04-24

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Magnum	Fix Released	Undecided	yatin

Bug Description

Tempest init is throwing UserWarning: converting 'True' to a string with magnum tempest plugin
(.thor) [chkumar246@assassin tempest_tracebin]$ tempest init foobar
2017-04-24 18:26:58.017 30056 INFO tempest [-] Using tempest config file /etc/tempest/tempest.conf
/home/chkumar246/killbox/rdo/tempest_tracebin/.thor/lib/python2.7/site-packages/oslo_config/types.py:54: UserWarning: converting 'True' to a string
warnings.warn('converting \'%s\' to a string' % str_val)
A workspace already exists with name: foobar.
(.thor) [chkumar246@assassin tempest_tracebin]$ tempest list-plugins
+--------------+------------------------------------------------------------------+
| Name | EntryPoint |
+--------------+------------------------------------------------------------------+
| magnum_tests | magnum.tests.functional.tempest_tests.plugin:MagnumTempestPlugin |
+--------------+------------------------------------------------------------------+
(.thor) [chkumar246@assassin tempest_tracebin]$

It appears that https://github.com/openstack/magnum/blob/master/magnum/tests/functional/tempest_tests/config.py#L64
copy_logs is set default to True which is boolean but config type is defined to string that's why it is throwing warning while doing tempest init. We need to fix it,

Tags:

chandan kumar (chkumar246) on 2017-04-24

Changed in magnum:
assignee:	nobody → chandan kumar (chkumar246)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-04-24: Fix proposed to magnum (master)

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

Changed in magnum:
status:	New → In Progress

OpenStack Infra (hudson-openstack) on 2017-04-27

Changed in magnum:
assignee:	chandan kumar (chkumar246) → yatin (yatinkarel)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-04-27: Fix merged to magnum (master)

Reviewed: https://review.openstack.org/459318
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=32b468f7e11b674c36256e30f23e5fd317f5bfe8
Submitter: Jenkins
Branch: master

commit 32b468f7e11b674c36256e30f23e5fd317f5bfe8
Author: Chandan Kumar <email address hidden>
Date: Mon Apr 24 19:32:13 2017 +0530

Fix config type of copy_logs from string to Boolean

    * As per the default value of copy_logs is Boolean
      but config type is defined as string, It needs to
      be fixed other wise tempest init will throw warning.

Change-Id: I9abf0c989e707b879c160d4df14546dd87c16f95
Closes-Bug: #1685797

Changed in magnum:
status:	In Progress → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-05-01: Fix proposed to magnum (stable/ocata)

Fix proposed to branch: stable/ocata
Review: https://review.openstack.org/461430

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-05-04: Fix merged to magnum (stable/ocata)

Reviewed: https://review.openstack.org/461430
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=7d30f94e2f096020aec0704f78bb6527b2e78f97
Submitter: Jenkins
Branch: stable/ocata

commit 7d30f94e2f096020aec0704f78bb6527b2e78f97
Author: Chandan Kumar <email address hidden>
Date: Mon Apr 24 19:32:13 2017 +0530

Fix config type of copy_logs from string to Boolean

    * As per the default value of copy_logs is Boolean
      but config type is defined as string, It needs to
      be fixed other wise tempest init will throw warning.

    Change-Id: I9abf0c989e707b879c160d4df14546dd87c16f95
    Closes-Bug: #1685797
    (cherry picked from commit 32b468f7e11b674c36256e30f23e5fd317f5bfe8)