60 Commits

Author SHA1 Message Date
Trinath Somanchi
af2f0f031f Switch to oslo_log
tackerclient is currently using python logging module to
implement logging functionality.
This patch made a switch to oslo.log(logging for openstack projects)

Reference:-
http://docs.openstack.org/developer/oslo.log

Change-Id: Iecab180e605c788bb820c7a628781abf9a4918ea
Closes-Bug: #1656167
2017-03-14 14:37:31 +05:30
Trinath Somanchi
c80b0c7133 vnfd-delete gives ambiguous success message
When a single vnfd is deleted, the following success message is delivered.

$> tacker vnfd-delete cirros-vnfd-multi-vdu

All vnfd(s) deleted successfully

The above success message has to modified to
" All specified %(resource)s(s) %(msg)s successfully? "

Change-Id: I48443622b718a1d6a1476f53c6378b67ce0d82e4
Closes-Bug: #1672302
2017-03-13 15:31:55 +05:30
janki
f66a4e0ccc Support VNF creation from inline VNF template
Allows creating VNF from the VNFD template directly without the need of
VNFD being stored in Tacker VNFD catalog.

Change-Id: I719237dd04dd7fe13fb7e7964402d7074679b2d6
Implements: blueprint vnf-inline-template
2017-01-20 06:58:02 +05:30
Naoya Harada
59ee90791d Fix invalid unit test code in python-tackerclient
Unit tests are not testing the behavior of command-line client
code properly since mox was replaced with mock in [1].

[1] https://review.openstack.org/#/c/313165/

assert_called_once_with() should be called after the target
call of the patched method.

Change-Id: I99873e9cb1d75d1fa2e41c02b9f87dfa4f9ef1f7
Partial-Bug: #1572706
2017-01-05 09:43:47 +09:00
Jenkins
69009d6dad Merge "Remove passing infra and mgmt driver from client" 2016-12-23 01:23:25 +00:00
Jenkins
3b99b49852 Merge "Replaced e.message with str(e)" 2016-12-23 01:22:51 +00:00
gecong1973
3f7cc06aff Replaced e.message with str(e)
For logging the exception message: e.message has been
deprecated. The preferred way is to call str(e).
more details:
https://www.python.org/dev/peps/pep-0352/

Change-Id: Ic22fbd37376843f714d2c1669d2cdc25a3569225
2016-12-23 00:58:02 +00:00
Jenkins
0373528ff3 Merge "Add __ne__ built-in function" 2016-12-21 01:54:07 +00:00
Janki
1e613792fd Remove passing infra and mgmt driver from client
This patch removes passing infra and mgmt driver as part of VNFD body from
client and API.

Change-Id: I4684e69b8993e6c35503317503dd73e2c13f44dd
Partial-Bug: #1630936
2016-12-20 13:25:24 +05:30
howardlee
2b3b1792d5 Add __ne__ built-in function
In Python 3 __ne__ by default delegates to __eq__ and inverts the
result, but in Python 2 they urge you to define __ne__ when you define
__eq__ for it to work properly [1]. There are no implied relationships
among the comparison operators. The truth of x==y does not imply that
x!=y is false. Accordingly, when defining __eq__(), one should also
define __ne__() so that the operators will behave as expected.

[1]https://docs.python.org/2/reference/datamodel.html#object.__ne__

Change-Id: I6adceadb6e3749e34cf847654f28a3b6eea832fd
2016-11-16 16:20:29 +08:00
Naoya Harada
b45904f7a1 Usability improvements of vim-update options
Tacker CLI does not support updating name and description of VIM.
This patch introduces --name and --description options to vim-update.

Also, existing --is-default option's behavior can be improved.
Currently is_default attribute will be set to False when --is-default
is ommited, and will be set to True when --is-default is specified.
So even a user, who wants to update attributes other than is_default,
needs to be aware of current is_default setting. This patch will make
the behavior of --is-default option clearer like below:

 1.When --is-default True is specified, it'll be set to True.
 2.When --is-default False is specified, it'll be set to False.
 3.When --is-default is ommited, it will stay as-is.

In addition, --config-file should be a required option because both
server and client expect it to be given when updating VIM.

Change-Id: Ic19fb6b0efb48bc3486f94836be4f2ff35f8c84b
Partial-Bug: 1637360
2016-11-11 17:35:52 +09:00
Jenkins
10f286989d Merge "Using assertIsInstance() instead of assertEqual(True, isinstance())" 2016-10-18 04:08:29 +00:00
gong yong sheng
e3f0d522ba make python34 CI job to pass
remove _get_attr_metadata monkey_patch since it is for XML test.
remove iterkeys() call which is not py3.x compatible.

Change-Id: I46df0a3748f470521553793111a21eea72bd7663
Closes-bug: 1632714
2016-10-14 06:39:50 +00:00
Trevor McCasland
65a5198d73 mox to mock refactor
Refactoring unit tests to use mock instead of mox

other mox usages by not run test cases will be dealt with
later.

Co-Authored-By: gong yong sheng <gong.yongsheng@99cloud.net>
Co-Authored-By: Trevor McCasland <TM2086@att.com>

Change-Id: I7e42494b96f1e54f5f04534d3b453422ea1bf4ef
Partial-Bug: #1572706
2016-10-14 14:38:29 +08:00
gong yong sheng
7cbdfabc75 rename vm into vnfm
Change-Id: Ia232eeb94adb190a212e9ec766eeee6dcb63bffd
closes-bug: 1632551
2016-10-12 11:30:04 +08:00
Anh Tran
66c9b13212 Using assertIsInstance() instead of assertEqual(True, isinstance())
Change-Id: I567519f19d415a17a5253d3864031583619586b4
2016-09-30 02:49:34 +00:00
Jenkins
6c78395976 Merge "Fix cString ImportError for py34" 2016-09-30 01:49:58 +00:00
Tony Xu
c3d69c8e22 Remove commented out code in test_shell.py
TrivialFix

This is patch remove pdb in
tackerclient/tests/unit/test_shell.py

Change-Id: Ie13f931b6e914eb511cdc4a073b0577b4bab2767
2016-09-29 09:18:36 +08:00
Lu lei
45ebce43cb Fix cString ImportError for py34
Here cString dose not exist in py3.x. This patch is used to
fix py34 gate errors.

Partial-Bug: #1612071

Change-Id: I0a36777cba8c4602c16be771956d4cd26f7e979f
2016-09-26 12:41:27 +08:00
Jenkins
1612e2227c Merge "Removing test cases for already removed methods" 2016-09-09 17:47:58 +00:00
dharmendra
495ccbd2aa Add support for multi delete
Currently Tacker client support single resource(i.e VNFD, VNF or
VIM) deletion per call. With this fix, multiple VNFs, VNFDs or
VIMs can be deleted in a single client call.

Change-Id: I0819d94e14637d8e9d8a7b2afb5f948587e00ce0
2016-09-09 14:27:09 +09:00
gong yong sheng
3b60ab1aaa Allow auth url without port for vim registration
This patch allows vim-register command to register keystone
URL without port.

Change-Id: Ie04a0253aa3f42ef532ccf8a7bddbbd1f88e3e34
Closes-bug: 1618756
2016-08-31 17:14:27 +08:00
Jenkins
48d458d290 Merge "Modify MyURLComparator to handle deep match on URL" 2016-08-27 14:29:16 +00:00
vish
f1eef18974 Moving test files to vm directory
This commit moves the test files from events and nfvo directory
to the vm directory. Currently tests are executed if they are in
the vm directory.

Change-Id: I7ac08fb5686597926ec3b3c62d8221abf2b0f066
Implements: blueprint: audit-support
2016-08-27 03:40:54 +00:00
Sripriya
f6848f0f31 Modify MyURLComparator to handle deep match on URL
MyURLComparator equals method is enhanced to perfom url comparison
based on url parts such as scheme, netloc, path and query instead
of raw string comparison.

pagination methods are modified to wrap the url requests with pagination
params within MyURLComparator objects.

Change-Id: I69b835dce64d662d1af6a14556fb8281e3b04d63
Closes-Bug: #1617480
2016-08-26 16:03:27 -07:00
Sripriya
aa499bb5f3 Remove list_vnf_resource_pagination unit test
This patch removes the list_vnf_resource_pagination unit test
which tested the pagination support of list_vnf_resources.
Pagination support is not enabled for vnf resources and hence
removed the test case.

Change-Id: I98843a63f6566595ceeb4758926a441fff2ee0ff
2016-08-26 12:54:41 -07:00
Jenkins
c047bf9db2 Merge "Add client support for VNFFG in NFVO" 2016-08-26 17:25:42 +00:00
dharmendra
7951ebb39e Removing test cases for already removed methods
tackerclient/common/utils.py doeas not contains these methods:
  to_primitive
  loads
  dumps.
Removing related test cases

Change-Id: I73035fcd120efc09cd19c94a8fa41671c2b3a7c3
Partial-Bug: 1617157
2016-08-27 01:09:24 +09:00
xu-haiwei
882f6b6c93 Add client support for VNFFG in NFVO
Co-Authored-By: Tim Rozet <trozet@redhat.com>

implements blueprint: tacker-vnffg
Change-Id: Ic5273319a01b8003e9497fbf7f984972602c7698
2016-08-26 09:23:35 -04:00
Tim Rozet
144408331e Adds client commands for listing a VNF resources
Support now for being able list sub resources of a vnf. REST path is
/vnf/<vnf_id>/resources/.  Resources will contain 'name', 'id', and
'type'.

APIImpact
Partial-Bug: 1602112

Change-Id: Ib9f0163c0c86df2a4d17630a5e6f7ca2d2fb22de
Signed-off-by: Tim Rozet <trozet@redhat.com>
2016-08-25 22:34:03 +00:00
Jenkins
7a0f627e2e Merge "Add "Description" parameter while creating VNF with CLI." 2016-08-24 21:03:57 +00:00
Jenkins
6b677a8f38 Merge "Remove '--config' option when create/update a vim" 2016-08-18 02:59:51 +00:00
vish
647baba801 Adds audit support in client
This commit enables support in the client for the resource event
logging

Implements: blueprint: audit-support

Co-Authored-By: Kanagaraj Manickam <mkr1481@gmail.com>

Change-Id: Ia54aed6720043840b00dd2eb00ca03a2ed92da5e
Depends-On: Change-Id: Ib82be521c5aa8b627e3f34a3696b10508371d3a0
2016-08-16 22:35:03 +00:00
xu-haiwei
e5457e5062 Remove '--config' option when create/update a vim
When create/update a vim, only enable '--config-file' option, will
remove '--config' option.

Change-Id: I8a728134ce0229db98de10bff5f4f46e7f2a9f2c
Closes-bug: #1587216
2016-08-15 13:14:39 +00:00
Manikantha Srinivas Tadi
6fecf2adf7 Add "Description" parameter while creating VNF with CLI.
Partial-Bug: #1586300

Change-Id: Ifaea2344b7f00414de743be1c035bd750e0af87b
2016-08-11 04:17:44 -04:00
Jenkins
166368b8b0 Merge "Make VNFD/VNF/VIM Name Mandatory in Tacker CLI" 2016-08-11 02:23:41 +00:00
gong yong sheng
0aa7efd292 Add domain information into auth cred.
If we use v3 keystone, these information should be passed
into tacker server.

Change-Id: Ieb5612c966b2110930faac0401f5dabde064ffc1
Partial-Bug: #1603851
2016-07-28 14:06:48 +08:00
Manikantha Srinivas Tadi
f4daaa09ba Make VNFD/VNF/VIM Name Mandatory in Tacker CLI
Change-Id: I7e94cc43a61fbeb2b406d1e973ca422359e761a0
Partial-Bug: #1474966
2016-07-26 00:09:58 -04:00
Jenkins
e88619e985 Merge "Transition default VIM to API and DB operation" 2016-07-25 18:05:44 +00:00
Kawaguchi Kentaro
c9ac53a722 Remove the mask password logic in vim list and vim show
In vim list and vim show command,
since hashed password masking on server-side,
the "mask_dict_password" modules are removed on client-side.

Change-Id: I463cb04696dc09157fbbc4b0bd64e66850feac84
Depends-On: Ice5c51b6a66cd27f21c144d3a672cf790e4cec41
Closes-bug: #1594495
2016-07-22 14:03:33 +09:00
Saju Madhavan
00d87bde7e Transition default VIM to API and DB operation
Set the default-vim in the tacker-db and provide
API to manage it.

Change-Id: Ie447f13c1d30e3cf5c1756c424fe60882082c211
Partial-Bug: 1592957
2016-07-20 18:27:39 +05:30
xu-haiwei
67c034b8a1 Change bind_port to 9890
Since the bind_port of tacker server is changed to 9890, fix the
document and test case to meet this change.

Change-Id: I5baff302517c8bd8b7cc14b10e771beb9079762b
2016-06-10 17:02:53 +09:00
Hideki Saito
d47747260f Fix incompatible code with python3 for tox pep8 test
"tox -e pep8" fails when python3 is used.
This patch fixes it.

Closes-Bug: #1583984

Change-Id: Idff17f64d2d4789acee08bd98902ae50312ddc92
2016-05-25 10:23:23 +09:00
dharmendra
f7b11685c4 py3.x: Use six.iteritems for Iterable dict items.
py3.x: Use six.iteritems for Iterable dict items instead of dict.iteritems.

Change-Id: Ibc4dbee779c1da52eba5b9ed34b982103c404e7f
Related-Bug: #1547712
2016-04-22 11:07:39 +05:30
Sripriya
07444203ec Do not display password in VIM command outputs
This fix masks the password attr in vim response for vim-list,
vim-show and vim-create commands.

Change-Id: I9b38b45f292a11f4615f3dc8eff672a4206c4a4a
Closes-Bug: #1567585
2016-04-18 17:08:23 -07:00
Sridhar Ramaswamy
20d3658353 Cleanup copyright header
Remove personal references in copyright header.

Change-Id: I05a8939f55acdd108276a37131f8988485801fbe
2016-04-04 03:30:14 +00:00
Martin Oemke
00d3c9771c Cleanup vim settings
This patch cleans up vim settings in .py files.
Settings should be handled in local settings.

Change-Id: Ib7a8c9751d18356070a39798ceedd1e400d979dd
2016-03-28 12:24:24 +02:00
Sripriya
de4aeb2736 Fix attr for vim update
Vim resource attr 'type' does not allow a PUT operation. Updated
the vim args to refelct typeattr for POST operation only. Also
fixed the vim name arg for vnf create command

Change-Id: I3c08945f24343288c2c5614ab4b472d68a1e1d47
2016-03-22 16:26:31 -07:00
Sripriya
5340d09287 Implement client support for multisite VIM
Client side changes for multisite VIM feature including new shell
commands.

Change-Id: Idae463b0cec40bea2edfde5723f7de5077600040
Implements: blueprint multi-site-vim
2016-03-18 20:38:48 -07:00
dharmendra
863084fd99 Removing "device" CLI from master branch.
Removing "device" CLI from master branch.
Related-Bug: #1543393
Change-Id: Ib93df154138b68a7dd0b2c1f79e6e76e05470ad3
2016-03-03 16:28:30 +05:30