Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-10
20:30:13 openstackgerrit Merged openstack/nova master: Add regression test for persisted RequestSpec.retry from failed resize https://review.openstack.org/505771
20:31:56 cfriesen_ dansmith: I see relevent entries in "instance_actions_events" and "instance_faults", but I don't know of any API to get either of those. Would the fault entry show up when doing a GET
20:31:57 cfriesen_ GET
20:31:59 cfriesen_ /servers/{server_id}
20:32:06 cfriesen_ whoops, paste error
20:32:07 dansmith cfriesen_: the instance actions api
20:32:15 mriedem GET /os-instance-actions
20:32:23 mriedem the fault is stored on the server
20:32:35 mriedem https://developer.openstack.org/api-ref/compute/#servers-actions-servers-os-instance-actions
20:32:40 dansmith cfriesen_: https://developer.openstack.org/api-ref/compute/#list-actions-for-server
20:32:43 dansmith yeah
20:32:44 mriedem https://developer.openstack.org/api-guide/compute/faults.html
20:32:56 mriedem ^ might be stale
20:33:02 mriedem most things in the compute api guide are old
20:33:45 mriedem e.g. https://developer.openstack.org/api-guide/compute/faults.html#instance-faults
20:33:50 mriedem "However, there is currently no API to retrieve this information."
20:33:52 mriedem that's totally bogus
20:34:19 mriedem GET /server/{id} has a 'fault' key in the response
20:34:20 mriedem "A fault object. Only displayed in the failed response. Default keys are code, created, and message (response code, created time, and message respectively). In addition, the key details (stack trace) is available if you have the administrator privilege."
20:38:21 cfriesen_ thanks, looks like heat should be able to get the necessary info
20:38:59 openstackgerrit Merged openstack/nova master: Don't persist RequestSpec.retry https://review.openstack.org/559447
20:39:30 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Add regression test for persisted RequestSpec.retry from failed resize https://review.openstack.org/560142
20:39:31 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Don't persist RequestSpec.retry https://review.openstack.org/560143
20:44:32 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Add regression test for persisted RequestSpec.retry from failed resize https://review.openstack.org/560145
20:44:33 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Don't persist RequestSpec.retry https://review.openstack.org/560146
21:10:10 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
21:10:11 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
21:15:15 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Add regression test for persisted RequestSpec.retry from failed resize https://review.openstack.org/560145
21:15:16 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Don't persist RequestSpec.retry https://review.openstack.org/560146
21:16:16 jackie-truong I have a patch that adds documentation for the certificate validation feature. Should that be linked with the current nova-validate-certificates patches? I'll still include the bp link in the commit message
21:17:02 cfriesen_ mriedem: dansmith: have you ever seen anything like this? http://paste.openstack.org/show/718865/ It seems like we ask nova to detach, but libvirt doesn't recognize the device.
21:17:39 mriedem yes
21:17:46 mriedem cfriesen_: you should be talking to lyarwood and melwitt
21:17:51 cfriesen_ cool, will do
21:17:54 mriedem they've been in the bowels of that code a few times
21:17:57 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
21:18:30 mriedem cfriesen_: which branch are you on? do you have all the latest fixes?
21:19:05 cfriesen_ this would be Pike, I'd have to check if it was totally up to date
21:19:59 cfriesen_ gotta run, parent-teacher interview
21:20:52 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
21:20:53 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
21:20:54 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
21:22:57 mriedem cfriesen_away: i was thinking of https://review.openstack.org/#/c/488545/ but that's in pike GA
21:41:25 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Add regression test for persisted RequestSpec.retry from failed resize https://review.openstack.org/560162
21:53:33 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300
21:59:06 arvindn05 dansmith: can you please review the changes made per your suggestions? https://review.openstack.org/#/c/557795/ - Update ImageMetaProp object to expose traits
22:04:05 openstackgerrit Ed Leafe proposed openstack/nova master: Add a microversion for consumer generation https://review.openstack.org/558958
22:15:04 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242
22:15:05 openstackgerrit Michael Still proposed openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078
22:15:06 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605
22:15:07 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
22:15:08 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
22:15:09 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
22:23:04 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Don't persist RequestSpec.retry https://review.openstack.org/560167
22:55:34 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242
22:55:35 openstackgerrit Michael Still proposed openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078
22:55:36 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605
22:55:37 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
22:55:38 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
22:55:39 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
23:17:56 openstackgerrit Eric Fried proposed openstack/nova-specs master: Add history behind nullable project_id and user_id https://review.openstack.org/560174
23:18:17 efried dansmith, edleafe, jaypipes: ^^
23:19:23 efried mikal: +2!
23:19:44 efried mikal: Thanks for putting up with me.
23:20:37 openstackgerrit Ed Leafe proposed openstack/nova master: Add handlers for consumer_generation https://review.openstack.org/557959
23:20:40 openstackgerrit Ed Leafe proposed openstack/nova master: Add a microversion for consumer generation https://review.openstack.org/558958
23:21:02 edleafe efried: ^^ answered *some* of your concerns. The remainder can be discussed tomorrow.
23:21:15 efried edleafe: Roger that. I gotta bail.
23:34:14 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: cleanup docs about instance faults https://review.openstack.org/560178
23:34:25 mriedem cfriesen_away: ^ that one is for you
23:48:00 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: cleanup docs about instance faults https://review.openstack.org/560178
23:49:31 mriedem alex_xu: gmann: Kevin_Zheng: ^ I just realized that we don't have 'fault' as a server blacklisted query parameter or sort key
23:49:50 mriedem but it is a joined table
23:50:37 mriedem looking at https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L298
23:51:04 Kevin_Zheng mriedem: got it, will check it the first thing when got to office
23:54:50 openstack Launchpad bug 1762870 in OpenStack Compute (nova) "server fault is not blacklisted for filtering/sorting" [Medium,Triaged]
23:54:50 mriedem https://bugs.launchpad.net/nova/+bug/1762870
#openstack-nova - 2018-04-11
00:00:16 cfriesen mriedem: aww, shucks.
00:02:22 mriedem Kevin_Zheng: i tested it out and invalidated the bug, details are in the bug - looks like we handle it appropriately by default; it's ignored as a query parameter, and you get a 400 if you try to sort on 'fault'
00:12:31 tetsuro good morning
00:27:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_virt_drivers.py https://review.openstack.org/559878
00:28:33 gmann mriedem: yea, it is not in valid sort key as well as in ignore list also so this gives 400 - https://github.com/openstack/nova/blob/94b150bd79e5dfb4bdbb1aa9437c103ad5b80543/nova/api/openstack/compute/schemas/servers.py#L379
00:30:07 gmann tetsuro: morning
00:33:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (2/3) https://review.openstack.org/550648
00:33:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (3/3) https://review.openstack.org/550659
00:34:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
00:34:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/test_utils.py https://review.openstack.org/557883
00:49:49 mriedem if anyone wants to chase down a functional test race bug over breakfast https://bugs.launchpad.net/nova/+bug/1762876
00:49:49 openstack Launchpad bug 1762876 in OpenStack Compute (nova) "KeyError during move operation functional tests" [Medium,Confirmed]
00:55:26 naichuans mriedem: Hi, Matt. Currently we have no test environments about live-migration in citrix CI, but we do have test jobs about it in Citrix XenRT, it is a internal test environment. And all the test related with the two patches are passed.https://review.openstack.org/#/c/489451/ and https://review.openstack.org/#/c/554154/
00:56:56 naichuans efried: Thank you for the feedback. Will follow up blueprint nested-resource-providers-allocation-candidates
01:31:00 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
01:31:01 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
01:31:02 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
01:31:41 Kevin_Zheng mriedem strange, I thought that one went well..
02:08:29 eandersson How does database pools work with workers? Does the conductor have one db pool per proccess?
02:08:48 eandersson and is the nova_api different?
02:33:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/api/openstack/compute/test_hosts.py https://review.openstack.org/559266
02:57:24 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
02:57:25 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204

Earlier   Later