Neutron L3/Router tests use admin credentials unnecessarily

Bug #1579136 reported by Mark T. Voelker
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
defcore
Medium
Mark T. Voelker

Bug Description

DefCore is working on adding basic tenant-facing networking capabilities to future Guidelines. Neutron has several L3 and router capabilities that scored sufficiently high for inclusion...however upon further review, it appears that the tests for these currently use admin credentials even though the capabilities themselves are generally exposed to tenants.[1] The use of admin credentials here is probably an artifact of code reuse. We should refactor these to not require admin credentials so they can be included in the 2016.08 Guideline.

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

Revision history for this message
Chris Hoge (hoge) wrote :
Download full text (4.2 KiB)

A json formatted list of tests to evaluate

{
    "networks-l3-router": {
        "tempest.api.network.test_routers_negative.RoutersNegativeTest.test_router_add_gateway_invalid_network_returns_404": {
           "idempotent_id": "id-37a94fc0-a834-45b9-bd23-9a81d2fd1e22"
         },
         "tempest.api.network.test_routers_negative.RoutersNegativeTest.test_router_add_gateway_net_not_external_returns_400": {
           "idempotent_id": "id-11836a18-0b15-4327-a50b-f0d9dc66bddd"
         },
         "tempest.api.network.test_routers_negative.RoutersNegativeTest.test_router_remove_interface_in_use_returns_409": {
           "idempotent_id": "id-04df80f9-224d-47f5-837a-bf23e33d1c20"
         }
    },
    "networks-l3-crud": {
        "tempest.api.network.test_routers.RoutersTest.test_add_multiple_router_interfaces": {
          "idempotent_id": "id-802c73c9-c937-4cef-824b-2191e24a6aab"
        },
        "tempest.api.network.test_routers.RoutersTest.test_add_remove_router_interface_with_port_id": {
          "idempotent_id": "id-2b7d2f37-6748-4d78-92e5-1d590234f0d5"
        },
        "tempest.api.network.test_routers.RoutersTest.test_add_remove_router_interface_with_subnet_id": {
          "idempotent_id": "id-b42e6e39-2e37-49cc-a6f4-8467e940900a"
        },
        "tempest.api.network.test_routers_negative.RoutersNegativeTest.test_add_router_interfaces_on_overlapping_subnets_returns_400": {
          "idempotent_id": "id-957751a3-3c68-4fa2-93b6-eb52ea10db6e"
        },
        "tempest.api.network.test_routers.RoutersTest.test_create_router_setting_tenant_id": {
          "idempotent_id": "id-e54dd3a3-4352-4921-b09d-44369ae17397",
          "aliases": ["tempest.api.network.test_routers.RoutersTest.test_create_router_setting_project_id"]
        },
        "tempest.api.network.test_routers.RoutersTest.test_create_router_with_default_snat_value": {
          "idempotent_id": "id-847257cc-6afd-4154-b8fb-af49f5670ce8"
        },
        "tempest.api.network.test_routers.RoutersTest.test_create_router_with_snat_explicit": {
          "idempotent_id": "id-ea74068d-09e9-4fd7-8995-9b6a1ace920f"
        },
        "tempest.api.network.test_routers.RoutersTest.test_create_show_list_update_delete_router": {
          "idempotent_id": "id-f64403e2-8483-4b34-8ccd-b09a87bcc68c"
        },
        "tempest.api.network.test_routers_negative.RoutersNegativeTest.test_delete_non_existent_router_returns_404": {
          "idempotent_id": "id-c7edc5ad-d09d-41e6-a344-5c0c31e2e3e4"
        },
        "tempest.api.network.test_routers_negative.RoutersNegativeTest.test_show_non_existent_router_returns_404": {
          "idempotent_id": "id-c2a70d72-8826-43a7-8208-0209e6360c47"
        },
        "tempest.api.network.test_routers_negative.RoutersNegativeTest.test_show_non_existent_router_returns_404": {
          "idempotent_id": "id-c2a70d72-8826-43a7-8208-0209e6360c47"
        },
        "tempest.api.network.test_routers.RoutersTest.test_update_extra_route": {
          "idempotent_id": "id-c86ac3a8-50bd-4b00-a6b8-62af84a0765c"
        },
        "tempest.api.network.test_routers.RoutersTest.test_update_router_admin_state": {
          "idempotent_id": "id-a8902683-c...

Read more...

Changed in defcore:
assignee: nobody → Chris Hoge (hoge)
Revision history for this message
Chris Hoge (hoge) wrote :
Revision history for this message
Mark T. Voelker (mvoelker) wrote :

Patch to address router tests: https://review.openstack.org/#/c/335558/

Changed in defcore:
assignee: Chris Hoge (hoge) → Mark T. Voelker (mvoelker)
status: New → In Progress
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers