yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #62451
[Bug 1664117] Re: Error message should not include SQL command
Reviewed: https://review.openstack.org/436773
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=d88a2f00e184475258eebe4f6a2bff8a6b9e341f
Submitter: Jenkins
Branch: master
commit d88a2f00e184475258eebe4f6a2bff8a6b9e341f
Author: liyingjun <yingjun.li@xxxxxxxxxxxxxxx>
Date: Wed Feb 22 11:20:12 2017 +0800
Error message should not include SQL command
There is SQL command included in the http response for conflicting
resource provider. We should remove SQL related errors from the
error message.
Change-Id: I47726919f9a2d2a767564909f454c334e42ffbe0
Closes-bug: 1664117
** Changed in: nova
Status: In Progress => Fix Released
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1664117
Title:
Error message should not include SQL command
Status in OpenStack Compute (nova):
Fix Released
Bug description:
Description
===========
When i create one resource provider with existed one's name,
returned error message includes SQL command.
Steps to reproduce
==================
* Create one resource provider with name 'RP_test'
* Create another resource provider with name 'RP_test'
Expected result
===============
"detail": "There was a conflict when trying to complete your request.\n\n Conflicting resource provider RP_test already exists.",
I think message above is detailed enough.
Actual result
=============
"detail": "There was a conflict when trying to complete your request.\n\n Conflicting resource provider already exists: (pymysql.err.IntegrityError) (1062, u\"Duplicate entry 'RP_test' for key 'uniq_resource_providers0name'\") [SQL: u'INSERT INTO resource_providers (created_at, updated_at, uuid, name, generation, can_host) VALUES (%(created_at)s, %(updated_at)s, %(uuid)s, %(name)s, %(generation)s, %(can_host)s)'] [parameters: {'uuid': 'cfafc096-4b15-4dc1-bb44-2bad0cd6d9e5', 'generation': 0, 'created_at': datetime.datetime(2017, 2, 13, 5, 27, 41, 686138), 'updated_at': None, 'can_host': 0, 'name': u'RP_test'}] ",
Environment
===========
1. nova version
[root@controller nova]# git log
commit 50d402821be7476eb58ccd791c50d8ed801e85eb
Author: Matt Riedemann <mriedem@xxxxxxxxxx>
Date: Wed Feb 8 10:23:14 2017 -0500
Consider startup scenario in _get_compute_nodes_in_db
2. Which hypervisor did you use?
devstack + libvirt + kvm
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1664117/+subscriptions
References