Fuel CLI returns error 'is not JSON serializable' when trying to remove node from environment

[Artem Panchenko]

api: '1.0'
astute_sha: 17b1afa5f0dc8f4fca5ed4eb03ec566fbfb5ed19
build_id: 2014-06-11_00-30-36
build_number: '250'
fuellib_sha: d9b8daa61e131e3c346e93d3870620897a71c593
fuelmain_sha: e59cb325d99fba49085ed5b0b6cfb0d9a83d2fc8
mirantis: 'yes'
nailgun_sha: f851199db57d9561b51c429956c056b463ad3f0b
ostf_sha: 67b61ed3788297fa5d985afec32498d8c0f812db
production: docker
release: '5.1'

Removing of nodes from environment using Fuel CLI fails when 'env-id' parameter isn't specified:

http://paste.openstack.org/show/84138/

But it should work according to 'fuel node --help' otput:

---
       Remove nodes no matter to which environment they were assigned:
            fuel node remove --node 2,3,6,7

I attached the patch for 'fuelclient/cli/actions/node.py' script which possibly fixes this issue.

[Dima Shulyak (dshulyak)]

Thanks, maybe you will submit this patch to fuel-web repo ?

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

commit 930c54c6b7016eab421169ecb12d027ac9e484d1
Author: Artem Panchenko <email address hidden>
Date: Tue Jun 17 22:44:29 2014 +0300

    Fix removing of nodes via CLI without env-id

    Removing of nodes from environment using Fuel CLI
    fails when 'env-id' parameter isn't specified.
    Also it shouldn't try to remove node which isn't
    assigned to some environment.

    Change-Id: I9e108be401984c2cc42cab63a895c82af9e83610
    Closes-bug: #1330412

[Egor Kotko (ykotko)]

{u'build_id': u'2014-07-17_22-02-59', u'ostf_sha': u'9863db951a6e159f4fa6e6861c8331e1af069cf8', u'build_number': u'329', u'auth_required': False, u'nailgun_sha': u'0af26b4d29a7f540911466f2374e256e8c899e60', u'production': u'docker', u'api': u'1.0', u'fuelmain_sha': u'079bf8c7d80d1c957343ac217747fdce00f2f85a', u'astute_sha': u'd90cad0130da014eded5c21fa5f31054ce999dac', u'feature_groups': [u'mirantis'], u'release': u'5.1', u'fuellib_sha': u'baf62a223eaa610a47a964ee11110ee60830e5ac'}