Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-23
14:32:07 rgerganov on the first call the provider tree has both parent and child and the call to placement succeds
14:32:53 rgerganov on the second call it has only the parent and when the driver adds the child and placement is called, we get 409
14:34:44 cdent rgerganov: and the message with the 409 is what?
14:35:24 rgerganov cdent, Failed to create resource provider record in placement API for UUID 4be176f1-b09f-48dd-82d7-d9c1c86fb68b. Got 409: {"errors": [{"status": 409, "request_id": "req-69d31b34-bc50-4178-a569-ebaef4734cbd", "detail": "There was a conflict when trying to complete your request.\n\n Conflicting resource provider name: vdc already exists. ", "title": "Conflict"}]}.
14:37:00 cdent rgerganov: thanks, looking at the calling code now
14:44:47 edmondsw gibi tx for reviewing the powervm SEA patch! Are you planning to look at the powervm vscsi patch as well?
14:45:02 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408
14:45:02 cdent rgerganov: on the second request, when the provider tree doesnt have the child, have you traced what's happening in _ensure_resource_provider. It seems like in there is where it could be going awry and not creating a full tree
14:46:05 gibi edmondsw: I cannot promise that it will fit before the feature freeze
14:47:19 edmondsw gibi ok. If you get to it, thanks. If not, maybe someone else will
14:48:55 gibi edmondsw: please ping me daily, that will help me not to forget
14:49:28 edmondsw gibi you asked for it... ;)
14:49:34 edmondsw thanks
14:50:55 edmondsw stephenfin do you also want daily reminders? ;)
14:50:58 gibi :)
14:51:22 stephenfin edmondsw: Nope, it'll be done :)
14:51:33 openstack bugzilla.redhat.com bug 1534669 in openstack-nova "emulator_threads_policy needs improvement when hyper threading is enabled" [Medium,New] - Assigned to sfinucan
14:51:43 edmondsw stephenfin thanks!
14:51:56 rgerganov cdent, correct, this method doesn't fetch the full tree on the second call
14:52:13 rgerganov cdent, self._refresh_associations doesn't do what we expect
14:58:34 edmondsw bauzas johnthetubaguy any chance you could look at https://review.openstack.org/#/c/526094/ adding powervm cinder volume support?
15:02:31 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
15:03:43 mriedem melwitt: want to hit this 3 LOC patch for 2.60 support in novaclient? https://review.openstack.org/#/c/536621/
15:09:45 melwitt mriedem: got it
15:09:47 bauzas mriedem: I can look
15:09:54 bauzas heh, snap
15:10:12 mriedem thanks, that'll likely be it for queens
15:14:39 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usage docs and initial release note for osc-placement https://review.openstack.org/536858
15:16:39 openstackgerrit int32bit proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268
15:22:14 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates https://review.openstack.org/500396
15:25:50 alex_xu mriedem: in case you have time https://review.openstack.org/#/q/topic:bp/request-traits-in-nova+(status:open+OR+status:merged), I have last two patches waiting for one more +2, appreciate if you can help it
15:27:56 mriedem alex_xu: ok, also looks like https://review.openstack.org/#/c/536083/4/nova/scheduler/client/report.py overlaps with something dansmith was working on
15:28:05 mriedem to use the 1.12 allocation candidates format for the move operations in the report client
15:29:40 dansmith mriedem: I think that patch just has two remaining failures to look at but I've been kinda absorbed with other things lately
15:31:43 melwitt mriedem: on the set status to ERROR upon rebuild failure patch, do you think it would be worth adding a fault message to the instance along with the ERROR state? I was thinking since the instance is sitting with the wrong imageref, it might be helpful to give some guidance in the fault message in case someone doesn't know how to recover https://review.openstack.org/#/c/536268/
15:32:50 sean-k-mooney stephenfin: for the record that refactor is annoying. i have all but 1 test passing now howerver the pci request spec object has like 8 required field not just the 4 in the alias schema definition
15:32:54 mriedem yeah a fault is probably necessary, i thought we'd have one on there but apparently we only use wrap_instance_fault in the compute manager
15:33:14 stephenfin sean-k-mooney: Really? What are the other ones?
15:33:48 dansmith melwitt: so can you push up a change to add a fault so we can get moving on that?
15:34:07 sean-k-mooney stephenfin: http://paste.openstack.org/show/650978/
15:34:26 melwitt dansmith: yep, let me do that
15:34:44 sean-k-mooney stephenfin: the commented out ones should be deleted
15:35:20 dansmith melwitt: cool
15:35:48 sean-k-mooney stephenfin: also i found that some test use device_id instead of product id so i have to support both and while the alias uses device_type the db uses dev_type so i have to convert when i create the spec
15:37:00 stephenfin sean-k-mooney: You can probably drop capability_type
15:37:06 sean-k-mooney stephenfin: anyway when i get the last test passing ill push up the new revision.
15:37:12 stephenfin Per my comment in nova/pci/request.py
15:37:40 stephenfin But otherwise, yeah, I figured it wouldn't be super easy but it is a much better design long-term, IMO
15:37:44 sean-k-mooney stephenfin: nope you will get an invalid key exception unless i specificlally add code to drop it
15:38:12 stephenfin sean-k-mooney: I'm working through the CPU pinning code again. Every time I go through this, I have to relearn it
15:38:34 stephenfin sean-k-mooney: But nothing except tests use it
15:38:36 stephenfin *set it
15:39:17 stephenfin and we could just filter it out of any '[pci]spec' options with a warning that it must be manually removed
15:39:20 damini 111Pls can anyone tell me how to write tempest test for shelve instance and unshelve .I am new to openstack.
15:39:50 sean-k-mooney haha well now you mention it the code that is failing is in consume_requests a pop from an empty list.
15:40:17 sean-k-mooney stephenfin: ya its easy to ignore it and log a message the way i have it set up instead of a exception
15:41:49 stephenfin damini: Tempest has a helper function for that. You might want to start there https://github.com/openstack/tempest/blob/master/tempest/common/compute.py#L263
15:42:26 stephenfin sean-k-mooney: (y) that would be my recommendation
15:42:34 alex_xu dansmith: do you have the link? I didn't found in the gerrit
15:42:39 stephenfin No point adding something we're going to immediately deprecate
15:42:49 dansmith alex_xu: link for what?
15:43:17 alex_xu dansmith: the 1.12 which mriedem mentioned, I guess you hit 1.12 by this patch https://review.openstack.org/#/c/531517/2
15:43:53 mriedem yeah that's the one
15:46:11 alex_xu mriedem: ah, I got it, dansmith already said he is working on other things. sorry, i misunderstand few words previously
15:46:38 mriedem np, i'm just pointing out you were both working on part of the same problem i think
15:47:24 dansmith alex_xu: why are you only bumping to 1.12 instead of to current?
15:47:31 dansmith alex_xu: because we're just going to have to bump right after this one
15:47:55 alex_xu dansmith: I have followup patch to support traits https://review.openstack.org/536085, which is 1.17
15:48:57 dansmith alex_xu: okay seems like you could just start calling at 1.17 in the first patch, but okay
15:49:02 dansmith I guess I'll rebase mine on your last change
15:50:29 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Address comments from original inventory patch https://review.openstack.org/521578
15:50:30 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add missing runtime requirements https://review.openstack.org/536870
15:50:30 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usage docs and initial release note for osc-placement https://review.openstack.org/536858
15:50:31 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Address review comments from allocations patch https://review.openstack.org/536871
15:50:31 alex_xu dansmith: ok, thanks
16:00:18 openstackgerrit Merged openstack/nova master: Add reference to policy sample https://review.openstack.org/531713
16:00:31 openstackgerrit Merged openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540
16:01:02 openstackgerrit Merged openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168
16:02:36 openstackgerrit melanie witt proposed openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268
16:04:49 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI for resource classes (v1.2) https://review.openstack.org/511182
16:04:49 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI for aggregates (v1.1) https://review.openstack.org/505643
16:04:50 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642
16:04:50 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183
16:04:51 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644
16:04:51 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643
16:04:52 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
16:04:52 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
16:04:53 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: [WIP] Get resource provider by uuid or name https://review.openstack.org/527791
16:07:58 bauzas sorry folks, I just migrated my IRC bouncer
16:08:06 bauzas in case some pinged me, lemme know
16:09:16 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates https://review.openstack.org/500396
16:10:40 openstackgerrit Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347
16:10:40 openstackgerrit Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346
16:10:41 openstackgerrit Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029
16:10:41 openstackgerrit Matthew Booth proposed openstack/nova master: Expose driver_block_device fields consistently https://review.openstack.org/528362
16:10:42 openstackgerrit Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328
16:10:42 openstackgerrit Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363
16:10:43 openstackgerrit Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786
16:10:43 openstackgerrit Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329
16:10:44 openstackgerrit Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787

Earlier   Later