| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-23 | |||
| 14:31:05 | rgerganov | cdent, correct | |
| 14:31:35 | cdent | It behaves the same way with the debugger not involved? | |
| 14:31:43 | rgerganov | yes | |
| 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 | 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:45:02 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 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: Usage docs and initial release note for osc-placement https://review.openstack.org/536858 | |
| 15:50:30 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Add missing runtime requirements https://review.openstack.org/536870 | |
| 15:50:31 | alex_xu | dansmith: ok, thanks | |
| 15:50:31 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Address review comments from allocations patch https://review.openstack.org/536871 | |
| 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 aggregates (v1.1) https://review.openstack.org/505643 | |
| 16:04:49 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI for resource classes (v1.2) https://review.openstack.org/511182 | |
| 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:50 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | |
| 16:04:51 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | |
| 16:04:51 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | |
| 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:52 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | |
| 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: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346 | |
| 16:10:40 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | |
| 16:10:41 | openstackgerrit | Matthew Booth proposed openstack/nova master: Expose driver_block_device fields consistently https://review.openstack.org/528362 | |
| 16:10:41 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029 | |
| 16:10:42 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363 | |
| 16:10:42 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328 | |