tempest

Testcase: Write testcase for security -tenant id by pass

Bug #930482 reported by Ravikumar Venkatesan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Fix Released
Medium
Rajalakshmi Ganesan

Bug Description

Add a test - to start server passing another tenant id using REST Calls
http://127.0.0.1:8774/v1.1/different_tenant/servers

Ravikumar Venkatesan (ravikumar-venkatesan)
Changed in tempest:
assignee: nobody → Sapan Kona (sapan-kona)
Revision history for this message
Sapan Kona (sapan-kona) wrote :

Hi Ravi,

I am done with writing the test script. However, some of the existing tests in test_list_servers.py script into which i was supposed to add this test are failing when i run it in devstack.

Should i still go ahead and submit?

Thanks,
Sapan

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

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

Changed in tempest:
status: Triaged → In Progress
Revision history for this message
Sapan Kona (sapan-kona) wrote :

Nosetests Log:

muralik@openstack27:~/tempest_sapan/tempest/tempest/tests$ nosetests -sv test_list_servers.py

Negative test: List servers of one tenant from alternate tenant ... ok

----------------------------------------------------------------------
Ran 1 test in 40.374s

OK

Pep8 Log:

muralik@openstack27:~/tempest_sapan/tempest/tempest/tests$pep8 test_list_servers.py
muralik@openstack27:~/tempest_sapan/tempest/tempest/tests$

Code Submitted : https://review.openstack.org/#change,4497

Revision history for this message
Sapan Kona (sapan-kona) wrote :

Hi Ravi,

Presently the test script for this bug is in the list servers script. However, Rajalakshmi feels that it would be more logical if we could add this in the test_authorization.py file.

Please suggest me on how I should move forward with this.

Thanks,
Sapan

Revision history for this message
Ravikumar Venkatesan (ravikumar-venkatesan) wrote :

Sapan ,

Absolutely . This needs to be added to test_authorization.py . Can you make new submission incorporating jay's feedback.

Regards,
Ravi

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
Sapan Kona (sapan-kona) wrote :

Nosetests Log:

muralik@openstack27:~/tempest_sapan/tempest/tempest/tests$ nosetests -sv test_authorization.py

A list on servers from one tenant should not ... ok

----------------------------------------------------------------------
Ran 1 test in 42.236s

OK

Pep8 Log:

muralik@openstack27:~/tempest_sapan/tempest/tempest/tests$pep8 test_authorization.py
muralik@openstack27:~/tempest_sapan/tempest/tempest/tests$

New Patch Submitted : https://review.openstack.org/#/c/7743/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Sapan Kona (sapan-kona)
Changed in tempest:
assignee: Sapan Kona (sapan-kona) → meenakshi m (meenakshi-m)
Rajalakshmi Ganesan (rajalakshmi-ganesan)
Changed in tempest:
assignee: meenakshi m (meenakshi-m) → Rajalakshmi Ganesan (rajalakshmi-ganesan)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/9099
Committed: http://github.com/openstack/tempest/commit/3793976f20d7f17da4a16e46290bc249db7d299e
Submitter: Jenkins
Branch: master

commit 3793976f20d7f17da4a16e46290bc249db7d299e
Author: sapan-kona <email address hidden>
Date: Thu Jun 28 20:22:43 2012 +0530

    Fixes LP Bug# 930482 - Test for security -tenanid by pass

    Change-Id: Idd6b6d0b7086ddf7682cfa6ac94daf5020a9eb82

Changed in tempest:
status: In Progress → Fix Committed
David Kranz (david-kranz)
Changed in tempest:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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