router create with external_gateway_info fails with 500 always.

Bug #1064235 reported by Itsuro Oda
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
yong sheng gong
Folsom
Fix Released
High
Gary Kotton
quantum (Ubuntu)
Fix Released
Undecided
Unassigned
Precise
Won't Fix
Undecided
Unassigned
Quantal
Fix Released
Undecided
Unassigned

Bug Description

This should be OK since allow_post of external_gateway_info is True.

--- log ---
$ quantum net-show net-ext
+---------------------------+--------------------------------------+
| Field | Value |
+---------------------------+--------------------------------------+
| admin_state_up | True |
| id | 364db7e1-6343-497b-bc9b-0c6ec72fce3f |
| name | net-ext |
| provider:network_type | local |
| provider:physical_network | |
| provider:segmentation_id | |
| router:external | True |
| shared | False |
| status | ACTIVE |
| subnets | 9454a527-5e59-4e36-8bb2-ee188b24fb85 |
| tenant_id | be3fc4694da2413c833574d6e018d338 |
+---------------------------+--------------------------------------+
$ quantum -v router-create router-test -- --external_gateway_info type=dict network_id=364db7e1-6343-497b-bc9b-0c6ec72fce3f
DEBUG: quantumclient.quantum.v2_0.router.CreateRouter get_data(Namespace(admin_state_down=True, columns=[], formatter='table', name='router-test', prefix='', request_format='json', tenant_id=None, value_specs=['--external_gateway_info', 'type=dict', 'network_id=364db7e1-6343-497b-bc9b-0c6ec72fce3f'], variables=[]))
DEBUG: quantumclient.client REQ: curl -i http://172.17.190.3:5000/v2.0/tokens -X POST -H "Content-Type: application/json" -H "Accept: application/json" -H "User-Agent: python-quantumclient"

DEBUG: quantumclient.client REQ BODY: {"auth": {"tenantName": "admin", "passwordCredentials": {"username": "admin", "password": "oda"}}}

DEBUG: quantumclient.client RESP:{'date': 'Tue, 09 Oct 2012 07:39:16 GMT', 'vary': 'X-Auth-Token', 'content-length': '2727', 'status': '200', 'content-type': 'application/json'}

DEBUG: quantumclient.client RESP BODY:{"access": {"token": {"expires": "2012-10-10T07:39:16Z", "id": "3dd65a586af1412c802643d04c4cf0f2", "tenant": {"enabled": true, "description": null, "name": "admin", "id": "be3fc4694da2413c833574d6e018d338"}}, "serviceCatalog": [{"endpoints": [{"adminURL": "http://172.17.190.3:8774/v2/be3fc4694da2413c833574d6e018d338", "region": "RegionOne", "publicURL": "http://172.17.190.3:8774/v2/be3fc4694da2413c833574d6e018d338", "internalURL": "http://172.17.190.3:8774/v2/be3fc4694da2413c833574d6e018d338"}], "endpoints_links": [], "type": "compute", "name": "Compute Service"}, {"endpoints": [{"adminURL": "http://172.17.190.3:9696/", "region": "RegionOne", "publicURL": "http://172.17.190.3:9696/", "internalURL": "http://172.17.190.3:9696/"}], "endpoints_links": [], "type": "network", "name": "Quantum Service"}, {"endpoints": [{"adminURL": "http://172.17.190.3:3333", "region": "RegionOne", "publicURL": "http://172.17.190.3:3333", "internalURL": "http://172.17.190.3:3333"}], "endpoints_links": [], "type": "s3", "name": "S3 Service"}, {"endpoints": [{"adminURL": "http://172.17.190.3:9292", "region": "RegionOne", "publicURL": "http://172.17.190.3:9292", "internalURL": "http://172.17.190.3:9292"}], "endpoints_links": [], "type": "image", "name": "Image Service"}, {"endpoints": [{"adminURL": "http://172.17.190.3:8776/v1/be3fc4694da2413c833574d6e018d338", "region": "RegionOne", "publicURL": "http://172.17.190.3:8776/v1/be3fc4694da2413c833574d6e018d338", "internalURL": "http://172.17.190.3:8776/v1/be3fc4694da2413c833574d6e018d338"}], "endpoints_links": [], "type": "volume", "name": "Volume Service"}, {"endpoints": [{"adminURL": "http://172.17.190.3:8773/services/Admin", "region": "RegionOne", "publicURL": "http://172.17.190.3:8773/services/Cloud", "internalURL": "http://172.17.190.3:8773/services/Cloud"}], "endpoints_links": [], "type": "ec2", "name": "EC2 Service"}, {"endpoints": [{"adminURL": "http://172.17.190.3:8000/v1", "region": "RegionOne", "publicURL": "http://172.17.190.3:8000/v1", "internalURL": "http://172.17.190.3:8000/v1"}], "endpoints_links": [], "type": "orchestration", "name": "Heat Service"}, {"endpoints": [{"adminURL": "http://172.17.190.3:35357/v2.0", "region": "RegionOne", "publicURL": "http://172.17.190.3:5000/v2.0", "internalURL": "http://172.17.190.3:5000/v2.0"}], "endpoints_links": [], "type": "identity", "name": "Identity Service"}], "user": {"username": "admin", "roles_links": [], "id": "af4f007f3197436dad5e6e6df0a75003", "roles": [{"name": "KeystoneServiceAdmin"}, {"name": "admin"}, {"name": "KeystoneAdmin"}], "name": "admin"}, "metadata": {"is_admin": 0, "roles": ["0351c121ed544e79a32c599f84ee898c", "61a36e1107e84ddc941317a6cedaae66", "7988289d6b0740259640c39a47c0d7c5"]}}}

DEBUG: quantumclient.client REQ: curl -i http://172.17.190.3:9696/v2.0/routers.json -X POST -H "User-Agent: python-quantumclient" -H "Content-Type: application/json" -H "Accept: application/json" -H "X-Auth-Token: 3dd65a586af1412c802643d04c4cf0f2"

DEBUG: quantumclient.client REQ BODY: {"router": {"external_gateway_info": {"network_id": "364db7e1-6343-497b-bc9b-0c6ec72fce3f"}, "name": "router-test", "admin_state_up": true}}

DEBUG: quantumclient.client RESP:{'date': 'Tue, 09 Oct 2012 07:39:16 GMT', 'status': '500', 'content-length': '88', 'content-type': 'application/json'}

DEBUG: quantumclient.client RESP BODY:{"QuantumError": "Request Failed: internal server error while processing your request."}

DEBUG: quantumclient.v2_0.client Error message: {"QuantumError": "Request Failed: internal server error while processing your request."}
ERROR: cliff.app Request Failed: internal server error while processing your request.
DEBUG: quantumclient.shell clean_up CreateRouter
DEBUG: quantumclient.shell got an error: Request Failed: internal server error while processing your request.
---

Revision history for this message
dan wendlandt (danwent) wrote :

Yes, I see this following traceback:

2012-10-09 16:07:35 ERROR [quantum.api.v2.resource] create failed
Traceback (most recent call last):
  File "/opt/stack/quantum/quantum/api/v2/resource.py", line 95, in resource
    result = method(request=request, **args)
  File "/opt/stack/quantum/quantum/api/v2/base.py", line 331, in create
    obj = obj_creator(request.context, **kwargs)
  File "/opt/stack/quantum/quantum/db/l3_db.py", line 154, in create_router
    return self._make_router_dict(router_db)
  File "/opt/stack/quantum/quantum/db/l3_db.py", line 131, in _make_router_dict
    nw_id = router.gw_port['network_id']
TypeError: 'NoneType' object has no attribute '__getitem__'

This has to do with some unexpected behavior from SQLAlchemy in which the "gw_port" relationship on the Router object is not loaded in create_port(), even though the router['gw_port_id'] is populated. Very similar logic for update_port() seems to behave correctly, so this requires more investigation (ideally by someone who understands sqlalchemy relationships well)

Changed in quantum:
status: New → Confirmed
importance: Undecided → High
tags: added: folsom-backport-potential
Changed in quantum:
assignee: nobody → yong sheng gong (gongysh)
milestone: none → grizzly-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (master)

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

Changed in quantum:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (master)

Reviewed: https://review.openstack.org/14674
Committed: http://github.com/openstack/quantum/commit/66e99dea67c16614da79bb2aef2235dac7949276
Submitter: Jenkins
Branch: master

commit 66e99dea67c16614da79bb2aef2235dac7949276
Author: gongysh <email address hidden>
Date: Tue Oct 23 22:40:26 2012 +0800

    use object directly instead of the foreigh key to update master db object.

    Bug #1064235

    Change-Id: I74fe3e758681c40590de87c82fd093224d329f51

Changed in quantum:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (stable/folsom)

Fix proposed to branch: stable/folsom
Review: https://review.openstack.org/14945

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (stable/folsom)

Reviewed: https://review.openstack.org/14945
Committed: http://github.com/openstack/quantum/commit/5cbaff430fc85db80c5c36b6976f43a6367d5ddc
Submitter: Jenkins
Branch: stable/folsom

commit 5cbaff430fc85db80c5c36b6976f43a6367d5ddc
Author: gongysh <email address hidden>
Date: Tue Oct 23 22:40:26 2012 +0800

    use object directly instead of the foreigh key to update master db object.

    Bug #1064235

    Change-Id: I74fe3e758681c40590de87c82fd093224d329f51

tags: added: in-stable-folsom
Gary Kotton (garyk)
tags: removed: folsom-backport-potential
Chuck Short (zulcss)
Changed in quantum (Ubuntu):
status: New → Fix Released
Changed in quantum (Ubuntu Precise):
status: New → Confirmed
Thierry Carrez (ttx)
Changed in quantum:
status: Fix Committed → Fix Released
Changed in quantum (Ubuntu Quantal):
status: New → Confirmed
Revision history for this message
Clint Byrum (clint-fewbar) wrote : Please test proposed package

Hello Itsuro, or anyone else affected,

Accepted quantum into quantal-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/quantum/2012.2.1-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in quantum (Ubuntu Quantal):
status: Confirmed → Fix Committed
tags: added: verification-needed
Mark McLoughlin (markmc)
tags: removed: in-stable-folsom
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.8 KiB)

This bug was fixed in the package quantum - 2012.2.1-0ubuntu1

---------------
quantum (2012.2.1-0ubuntu1) quantal-proposed; urgency=low

  * Resynchronize with stable/folsom (1e774867) (LP: #1085255):
    - [aeabb42] There are routing problems when the dnsmasq port does not come
      first in the routing table (LP: #1083238)
    - [04aab72] Quantum linux bridge not optimized with libvirt (LP: #1078210)
    - [ca7fc10] getting quotas from database has severe performance implications
      (LP: #1075369)
    - [66605e8] failed to update an external network into non external network
      (LP: #1083387)
    - [c60051a] Quantum test suite leaks memory like a sieve (LP: #1065276)
    - [3179dfc] clear_db() does incomplete db teardown (LP: #1080988)
    - [c1e19d7] Unauthorized command: cat /proc/None/cmdline (LP: #1077651)
    - [af9e076] At times a instance will not receive an IP address from the DHCP
      agent (LP: #1081664)
    - [e0d1a7d] allow multiple floating-ip on single port if they use different
      fixed ips and/or external nets (LP: #1057844)
    - [8471d79] Delete port fails to gateway ip (LP: #1079980)
    - [aca8b4a] fixed_ip allocation which is not included within
      allocation_pools makes error when delete port or re-create port
      (LP: #1077292)
    - [eacc9d3] Mapping same bridge to different phyiscal networks succeed
      (LP: #1067669)
    - [51b4c82] python-quantum: not region aware (LP: #1080793)
    - [6f0a486] delete floatingip should be in one transaction to delete port
      (LP: #1080516)
    - [db6cda7] Remove qpid configuration variables no longer supported
    - [a112840] Allow NVP plugin to use per-tenant quota extension
    - [82b1a55] Quantum service does not restart after reboot (LP: #1073999)
    - [c01a839] There are some cases that L3 API with an invalid parameter
      returns 500. (LP: #1064765)
    - [26b383f] external network can be plugged also as internal network for one
      router (LP: #1053633)
    - [49f649c] There is a lot of cases that API with an invalid parameter
      returns 500. (LP: #1062046)
    - [4546a18] When create subnet, you con set up the value as cidr (the value
      isn't cidr form). (LP: #1067959)
    - [9ba453a] killfilter should handle updated/deleted executables
      (LP: #1073768)
    - [7c8a55c] a port which is not able to delete is made when floatingip
      create fails. (LP: #1064748)
    - [c9b84cf] Linux bridge port update causes exception (LP: #1072713)
    - [cb57932] I can't add interface to router, if there is another port in
      non-shared network of other tenant (LP: #1057558)
    - [574e278] Ryu plugin does not support Security Groups (LP: #1059393)
    - [607f486] tap device added to integration bridge without tag
      (LP: #1064070)
    - [21a0fdf] L3 agent external network flag (LP: #1056720)
    - [5cbaff4] router create with external_gateway_info fails with 500 always.
      (LP: #1064235)
    - [63b81f6] l3 db operations failed in multiple transactions (LP: #1070335)
    - [bff17fb] Ensure that the SqlSoup import is still supported.
    - [e091a29] l3_nat_agent was renamed to l3_agent
    - [9030969] remove default value of 'local_ip' of 10...

Read more...

Changed in quantum (Ubuntu Quantal):
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in quantum:
milestone: grizzly-1 → 2013.1
Revision history for this message
Steve Langasek (vorlon) wrote :

The Precise Pangolin has reached end of life, so this bug will not be fixed for that release

Changed in quantum (Ubuntu Precise):
status: Confirmed → Won't Fix
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.