[DOC] integration tests require both OS_TENANT_ID and OS_TENANT_NAME parameters

Bug #1289607 reported by Andrew Lazarev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Sahara
Fix Released
Low
Sergey Lukjanov

Bug Description

Integration tests config sample should be updated with note that both OS_TENANT_ID and OS_TENANT_NAME should be specified.

=================================

Integration tests require OS_TENANT_ID and OS_TENANT_NAME parameters at the same time.

Exception if OS_TENANT_ID is not specified:

  File "/home/ubuntu/savanna/.tox/integration/local/lib/python2.7/site-packages/savannaclient/api/node_group_templates.py", line 61, in create
    'node_group_template')
...
  File "/home/ubuntu/savanna/.tox/integration/local/lib/python2.7/site-packages/simplejson/decoder.py", line 389, in raw_decode
    return self.scan_once(s, idx=_w(s, idx).end())
JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Exception if OS_TENANT_NAME is not specified:
...
  File "/Users/andrew/Savanna/savanna/savanna/tests/integration/tests/base.py", line 90, in setUp
    self.common_config.OS_TENANT_ID
  File "/Users/andrew/Savanna/savanna/.tox/py27/lib/python2.7/site-packages/savannaclient/client.py", line 45, in Client
    return client_class(*args, **kwargs)
...
  File "/Users/andrew/Savanna/savanna/.tox/py27/lib/python2.7/site-packages/keystoneclient/v2_0/client.py", line 199, in _base_authN
    resp, body = self.request(url, 'POST', body=params, headers=headers)
...
Unauthorized: The request you have made requires authentication. (HTTP 401)

I see two problems here:
1. User should be able to specify any of these two params, not both
2. Exceptions doesn't help at all

Changed in savanna:
status: New → Won't Fix
status: Won't Fix → Confirmed
Revision history for this message
Andrew Lazarev (alazarev) wrote :

Also itest.conf.sample doesn't contain OS_TENANT_ID

Revision history for this message
Sergey Lukjanov (slukjanov) wrote :

It's as designed, sample should be updated with note about it.

summary: - integration tests require OS_TENANT_ID and OS_TENANT_NAME parameters
+ [DOC] integration tests require OS_TENANT_ID and OS_TENANT_NAME
+ parameters
Changed in savanna:
importance: Undecided → Low
milestone: none → icehouse-rc1
description: updated
summary: - [DOC] integration tests require OS_TENANT_ID and OS_TENANT_NAME
+ [DOC] integration tests require both OS_TENANT_ID and OS_TENANT_NAME
parameters
Revision history for this message
Sergey Lukjanov (slukjanov) wrote :
Changed in sahara:
assignee: nobody → Sergey Lukjanov (slukjanov)
status: Confirmed → In Progress
Revision history for this message
Sergey Lukjanov (slukjanov) wrote :
Changed in sahara:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in sahara:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in sahara:
milestone: icehouse-rc1 → 2014.1
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.