Something wrong with configdb command: fuel2 config set

Bug #1644814 reported by Dmitry Belyaninov
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Dmitry Belyaninov
Newton
Fix Released
High
Dmitry Belyaninov

Bug Description

There are 4 failed tests:
https://product-ci.infra.mirantis.net/job/9.x.acceptance.ubuntu.tests_configdb_api/8/

Errors like:
    assert_equal(res_obj['key'], {'a': 1, 'b': None})
KeyError: 'key'

Previous tests were failed with
https://bugs.launchpad.net/fuel/+bug/1637504

Revision history for this message
Dmitry Belyaninov (dbelyaninov) wrote :
Revision history for this message
Alexey. Kalashnikov (akalashnikov) wrote :

Seems like this issue related to fix:

https://bugs.launchpad.net/fuel/+bug/1642330

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Dmitry Belyaninov (dbelyaninov)
Revision history for this message
Dmitry Belyaninov (dbelyaninov) wrote :

console output for commands from resource_value_without_level test
http://paste.openstack.org/show/591217/

So the problem is in step #5. That is why next check of resource failed.

Changed in fuel:
assignee: Dmitry Belyaninov (dbelyaninov) → Alexander Kislitsky (akislitsky)
summary: - [fuel-qa] something wrong with configdb commands output
+ Something wrong with configdb command: fuel2 config set
tags: added: area-configdb
Revision history for this message
Alexander Kislitsky (akislitsky) wrote :

The issue occurs because of resource is not created and we are trying to set key/value to it.
For fix the qa test we should add the following step:

fuel2 config set --env 2 --resource 3 --format json

# Here we should pass to the stdin value: {}

And after this we can execute:

fuel2 config set --env 2 --resource 3 --value '{"a": 1, "b": null}' --key key --type json

Changed in fuel:
status: New → Triaged
assignee: Alexander Kislitsky (akislitsky) → Fuel QA Team (fuel-qa)
Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Dmitry Belyaninov (dbelyaninov)
Changed in fuel:
assignee: Dmitry Belyaninov (dbelyaninov) → Alexander Kislitsky (akislitsky)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tuning-box (master)

Reviewed: https://review.openstack.org/409017
Committed: https://git.openstack.org/cgit/openstack/tuning-box/commit/?id=9f3302d26d1bfd9167d8dc676dda007c13891851
Submitter: Jenkins
Branch: master

commit 9f3302d26d1bfd9167d8dc676dda007c13891851
Author: Alexander Kislitsky <email address hidden>
Date: Fri Dec 9 11:28:16 2016 +0300

    Resource value modification inhanced in the fuel2

    Now it is possible to set whole resource value through fuel2
    using --value and --type without --key.

    Partial-Bug: #1644814
    Change-Id: Iea3c949c86e1376cfdfafbc86f7e3e3a4f5e1516

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tuning-box (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/409097

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tuning-box (stable/mitaka)

Reviewed: https://review.openstack.org/409097
Committed: https://git.openstack.org/cgit/openstack/tuning-box/commit/?id=c548853225446e264a71ab7c9a5a7343e66cad78
Submitter: Jenkins
Branch: stable/mitaka

commit c548853225446e264a71ab7c9a5a7343e66cad78
Author: Alexander Kislitsky <email address hidden>
Date: Fri Dec 9 11:28:16 2016 +0300

    Resource value modification enhanced in the fuel2

    Now it is possible to set whole resource value through fuel2
    using --value and --type without --key.

    Partial-Bug: #1644814
    Change-Id: Iea3c949c86e1376cfdfafbc86f7e3e3a4f5e1516
    (cherry picked from commit 9f3302d26d1bfd9167d8dc676dda007c13891851)

tags: added: in-stable-mitaka
Changed in fuel:
assignee: Alexander Kislitsky (akislitsky) → Fuel QA Team (fuel-qa)
Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Dmitry Belyaninov (dbelyaninov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/411226

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/mitaka)

Reviewed: https://review.openstack.org/411226
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=fd1f0bfd97c8a93162b3c20095b4f5b6c3247bcc
Submitter: Jenkins
Branch: stable/mitaka

commit fd1f0bfd97c8a93162b3c20095b4f5b6c3247bcc
Author: Dmitry Belyaninov <email address hidden>
Date: Thu Dec 15 10:42:39 2016 +0000

    Configdb tests should be changed according to the new CLI

    Fuel CLI was changed, so appropriate tests modification
    was prepared.

    Change-Id: Ie1f2d2eba713b22dffebb7e0ac911de0f9222687
    Closes-Bug: 1644814

Revision history for this message
Dmitry Belyaninov (dbelyaninov) wrote :
Changed in fuel:
status: In Progress → Fix Committed
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/436350

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/newton)

Reviewed: https://review.openstack.org/436350
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=c4e0b7f8648795774bb612c2ac917e6a42f31800
Submitter: Jenkins
Branch: stable/newton

commit c4e0b7f8648795774bb612c2ac917e6a42f31800
Author: Dmitry Belyaninov <email address hidden>
Date: Thu Dec 15 10:42:39 2016 +0000

    Configdb tests should be changed according to the new CLI

    Fuel CLI was changed, so appropriate tests modification
    was prepared.

    Change-Id: Ie1f2d2eba713b22dffebb7e0ac911de0f9222687
    Closes-Bug: 1644814
    (cherry picked from commit fd1f0bfd97c8a93162b3c20095b4f5b6c3247bcc)

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

Reviewed: https://review.openstack.org/436348
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=0724bb79522353073195c1db630a0b07ee0f2961
Submitter: Jenkins
Branch: master

commit 0724bb79522353073195c1db630a0b07ee0f2961
Author: Dmitry Belyaninov <email address hidden>
Date: Thu Dec 15 10:42:39 2016 +0000

    Configdb tests should be changed according to the new CLI

    Fuel CLI was changed, so appropriate tests modification
    was prepared.

    Change-Id: Ie1f2d2eba713b22dffebb7e0ac911de0f9222687
    Closes-Bug: 1644814
    (cherry picked from commit fd1f0bfd97c8a93162b3c20095b4f5b6c3247bcc)

Revision history for this message
Dmitry Belyaninov (dbelyaninov) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-qa 11.0.0.0rc1

This issue was fixed in the openstack/fuel-qa 11.0.0.0rc1 release candidate.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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