Fuel cli ignores optional arguments "--json"

Bug #1307516 reported by Egor Kotko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Alexandr Notchenko

Bug Description

{"build_id": "2014-04-14_01-00-26", "mirantis": "yes", "build_number": "101", "nailgun_sha": "61410bcf3201cd737e68ece8ab15313acc746476", "production": "dev", "ostf_sha": "118c955085ea7829f3a34decd38d63554b74451c", "fuelmain_sha": "ddc94c52c267f0276cbd8485d6e704aea05b23a5", "astute_sha": "401bc474b1d8cebb8ba70b3b6154107e08fd725d", "release": "5.0", "fuellib_sha": "101f3645ead182bc47024ff7568b04554de06bba"}

Steps to reproduce:
1) Create environment
2) Execute on master node:
#fuel settings --download --env 1 --json

Expected result:
The command should generates file settings_1.json,

Actual result:
The command generates file settings_1.yaml

The same is about command:
fuel settings --upload --dir /root/settings --json --env 1 - It ignores argument --json, appears alert that there is no *.yaml file in the directory.

Tags: cli
Revision history for this message
Egor Kotko (ykotko) wrote :
Changed in fuel:
status: New → Confirmed
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/87331
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=cf54bd27a0beb20edee611ea55f1dcc5070b67db
Submitter: Jenkins
Branch: master

commit cf54bd27a0beb20edee611ea55f1dcc5070b67db
Author: Alexandr Notchenko <email address hidden>
Date: Mon Apr 14 21:18:28 2014 +0400

    File Serialization problem

     * Fixed serialization with flags
     * Fixed several command stdout messages
     * Fixed empty facts exception

     Closes-bug: #1307577
     Closes-bug: #1307516

    Change-Id: Iecbd12959c4f8e707249c6a58df0cc5e432fd1df

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Egor Kotko (ykotko) wrote :

Checked on:
{"build_id": "2014-05-16_01-10-31", "mirantis": "yes", "build_number": "208", "ostf_sha": "353f918197ec53a00127fd28b9151f248a2a2d30", "nailgun_sha": "89e7224f0b87284f75bf02b956b640768fabe352", "production": "docker", "api": "1.0", "fuelmain_sha": "861f4410bd07ecf40c1f87785dd0c86de387f2cf", "astute_sha": "c0418e1739cceb864fd2c548a7c6b9ad6a46ed86", "release": "5.0", "fuellib_sha": "68ef90fdc9fe5a3978f756a96a3fa29d5f1a2929"}

Changed in fuel:
status: Fix Committed → 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.