| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-04 | |||
| 17:50:50 | spatel | opening bug now | |
| 17:51:34 | sean-k-mooney | i know this used to be a bug when we first intoduced hugepages back in like 2014 but its totally posibel that that bug has been reinotduce or that we just never got aroud to fixing it | |
| 17:55:14 | sean-k-mooney | this should be correct https://github.com/openstack/nova/blob/master/nova/objects/numa.py#L140-L157 but that said the error came from qemu so perhaps this is a qemu issue | |
| 18:02:46 | openstackgerrit | Merged openstack/nova master: db: Remove cell APIs https://review.opendev.org/651309 | |
| 18:04:46 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Workaround missing RequestSpec.instance_group.uuid https://review.opendev.org/663110 | |
| 18:06:12 | spatel | sean-k-mooney: https://bugs.launchpad.net/nova/+bug/1831652 | |
| 18:06:13 | openstack | Launchpad bug 1831652 in OpenStack Compute (nova) "fixing the case where we use every single page in 1 vm" [Undecided,New] | |
| 18:06:24 | spatel | sean-k-mooney: Thank you for help :) | |
| 18:08:17 | sean-k-mooney | spatel: no worries as i said the error seams to be coming form qemu so its likely that we fixed the nova issue already and qemu also has a bug but at least we have something to track figuring that out | |
| 18:30:18 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Use SATA bus for cdrom devices when using q35 machine type https://review.opendev.org/663011 | |
| 18:30:19 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887 | |
| 18:40:39 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Use SATA bus for cdrom devices when using q35 machine type https://review.opendev.org/663011 | |
| 18:40:40 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887 | |
| 18:50:56 | mriedem | dansmith: thanks replied on that pre-filter for disabled computes spec | |
| 18:51:48 | mriedem | tl;dr it sounds like don't worry about down services | |
| 19:05:41 | dansmith | mriedem: ack, replied, but yeah, that's MHO | |
| 19:06:57 | mriedem | regarding old computes that won't be reporting the trait but they get disabled in the api, what are your thoughts on that? the api manages the trait until the compute is upgraded, or just ignore it until upgraded, or add a sync CLI? | |
| 19:07:30 | mriedem | if we do'nt add a sync CLI for old computes that are already disabled, then i'd just ignore those requests in the API | |
| 19:10:11 | dansmith | neither | |
| 19:10:17 | dansmith | ignore until upgrade | |
| 19:10:42 | dansmith | we set disabled=true on the compute record, let the computefilter continue to exclude those at great expense | |
| 19:10:58 | dansmith | when the compute is upgraded and restarted, the u-p-t sync will update placement | |
| 19:11:11 | mriedem | upt won't set this trait | |
| 19:11:17 | dansmith | why? | |
| 19:11:32 | mriedem | well, i hadn't planned on adding that - it was an alternative but not one i'd baked into the proposed change | |
| 19:11:43 | dansmith | but why not? it solves a lot right? | |
| 19:12:02 | mriedem | solves as in upgrades, dropped calls, out of sync, etc? | |
| 19:12:05 | dansmith | it's kindof our "heal placement" loop as it is, so I don't see why we wouldn't | |
| 19:12:10 | dansmith | yeah | |
| 19:13:01 | mriedem | ok. in my reply i said we could build that in, so i might as well i guess, i'll move that out of alternatives. | |
| 19:31:57 | efried | melwitt: Is there a reason you're holding back from approving https://review.opendev.org/#/c/579897/ ? | |
| 19:34:06 | melwitt | efried: yeah, I hadn't reviewed deep into the change previously and don't know much about the context (the original feature). but since I had expressed a reno would be helpful, I reviewed the change and acked on that basis | |
| 19:34:44 | efried | okay, thanks. | |
| 19:34:47 | mriedem | efried: dustinc: i'm marking https://review.opendev.org/#/c/662881/ as a WIP since it's clearly a WIP | |
| 19:34:49 | efried | Not sure who we're looking for to send it | |
| 19:34:59 | efried | ack mriedem | |
| 19:35:31 | mriedem | efried: just spin the roulette wheel of nova cores | |
| 19:36:04 | efried | mriedem: Tempted to send it myself, adding my almost+2 to the testament of people who tested it live | |
| 19:36:25 | mriedem | probably shouldn't, | |
| 19:36:33 | mriedem | just wait for stephenfin or johnthetubaguy to take a look i'd say | |
| 19:36:39 | efried | I didn't touch much of it. But yeah. | |
| 19:37:09 | mriedem | https://www.youtube.com/watch?v=JGftIcp2SC0 ? | |
| 19:37:26 | dansmith | rock on | |
| 19:37:56 | melwitt | lol, a song for every occasion | |
| 19:38:36 | mriedem | bon was ahead of his time when it came to collaborative software development | |
| 19:39:42 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add regression recreate test for bug 1830747 https://review.opendev.org/663124 | |
| 19:39:43 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Workaround missing RequestSpec.instance_group.uuid https://review.opendev.org/663125 | |
| 19:39:44 | openstack | bug 1830747 in OpenStack Compute (nova) rocky "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem) | |
| 19:40:08 | sean-k-mooney | efried: for what its worth i think the hypervior hiding chance makes sense the only reservation i have is nvida or other may work around this in the future so i tond want us to have to keep addign these workaround but for this specific case i think it makes sense | |
| 19:44:13 | dustinc | mriedem: thanks, I meant to put WIP in the title | |
| 20:13:45 | efried | dustinc: In case it's not clear, if it looks like I'm answering questions in the sdk spec, me (or anyone) answering in comments isn't the end of the road - the salient points should be included in the document itself. | |
| 20:27:03 | dustinc | efried: very clear, thanks | |
| 20:40:57 | mriedem | i'm assuming i'm not alone in saying if we add support to pass an az to the unshelve api https://review.opendev.org/#/c/624689/ we should/would not support the zone:host:node format like on server create which bypasses the scheduler and forces the server onto the specified host and/or node | |
| 20:41:16 | mriedem | from earlier review on the spec i think at least alex_xu agrees with me on ^ | |
| 20:44:06 | efried | mriedem: You appear to be alone in saying that, at least right now. But I wouldn't take that as dissent. | |
| 20:44:27 | mriedem | https://review.opendev.org/#/c/624689/7/specs/train/approved/support-specifying-az-when-restore-shelved-server.rst@49 | |
| 20:46:40 | melwitt | if it's "obvious" that the only utility for zone:host:node was getting a server into a specific az, then I would agree, but I don't feel sure that's the only reason people were using it | |
| 20:46:54 | melwitt | *for unshelve | |
| 20:59:18 | mriedem | i'm not sure what that means | |
| 20:59:41 | mriedem | zone:host:node is used during server create to force the server on a specific host and/or node if you're an admin | |
| 21:00:24 | mriedem | if you're non-admin you can only specify zone | |
| 21:05:34 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Add regression recreate test for bug 1830747 https://review.opendev.org/663143 | |
| 21:05:36 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Workaround missing RequestSpec.instance_group.uuid https://review.opendev.org/663144 | |
| 21:05:36 | openstack | bug 1830747 in OpenStack Compute (nova) rocky "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem) | |
| 21:05:40 | melwitt | oh, sorry, I misunderstood what was happening in the spec. I thought that today zone:host:node is supported and that they wish to add specifying az (for non-admins) and that they were proposing removing zone:host:node as part of the change | |
| 21:06:26 | mriedem | nope, just wanting to add az to unshelve | |
| 21:06:45 | melwitt | I see now that today there is no zone:host:node support and they want to add specification of az and they're proposing doing it via zone:host:node. I agree it should not be done that way because that's more than only adding the az, that's adding scheduler bypass, as you said | |
| 21:06:58 | mriedem | yar | |
| 21:15:47 | mriedem | thanks | |
| 21:25:52 | openstackgerrit | Dustin Cowles proposed openstack/nova-specs master: WIP: Spec: Use OpenStack SDK in Nova https://review.opendev.org/662881 | |
| 22:06:34 | cfriesen | when a nova-compute service is deleted in nova, is that supposed to also remove the relevent row in the resource_providers table? | |
| 22:07:37 | efried | cfriesen: If it's deleted through proper channels, I think so? | |
| 22:07:54 | efried | that would seem to be a pretty obvious miss if not | |
| 22:08:51 | efried | You're supposed to stop the service before you delete it | |
| 22:09:05 | efried | otherwise the compute will reassert its RP the next time periodics run. | |
| 22:09:07 | cfriesen | I'm debugging a situation with Pike where deleting nova-compute and then adding it back with the same name ends up not updating the resource_providers row so it points at the deleted entry in compute_nodes. | |
| 22:09:24 | efried | oh, yeah, pretty sure there's been a bug or three about that. | |
| 22:09:37 | efried | mriedem_away would be able to recite the number off the top of his head. | |
| 22:10:23 | efried | Fraid I gotta run. Good luck :) | |
| 22:10:39 | cfriesen | thanks. :) | |
| 22:12:36 | openstackgerrit | sean mooney proposed openstack/nova-specs master: add libvirt pqos spec https://review.opendev.org/662264 | |
| 22:41:32 | cfriesen | efried: FYI, pretty sure I found the missing commits. seems we weren't up-to-date with upstream stable/pike | |
| 23:16:14 | openstackgerrit | Dustin Cowles proposed openstack/nova-specs master: WIP: Spec: Use OpenStack SDK in Nova https://review.opendev.org/662881 | |
| #openstack-nova - 2019-06-05 | |||
| 00:08:35 | openstackgerrit | Merged openstack/python-novaclient master: Blacklist python-cinderclient 4.0.0 https://review.opendev.org/662912 | |
| 00:26:55 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Replace 'is comprised of' with 'comprises' https://review.opendev.org/663175 | |
| 00:37:47 | openstackgerrit | Brin Zhang proposed openstack/nova stable/rocky: Replace the invalid index of nova-rocky releasenote https://review.opendev.org/663178 | |
| 00:59:43 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove unnecessary setUp methods https://review.opendev.org/663179 | |
| 01:01:23 | alex_xu | The vPMEM spec is ready for review https://review.opendev.org/#/c/601596/, in case of anyone still at the day of spec review day~ | |
| 01:52:57 | openstackgerrit | ya.wang proposed openstack/nova-specs master: Add spec for expose-auto-converge-post-copy https://review.opendev.org/651681 | |
| 07:20:44 | openstackgerrit | Merged openstack/nova master: Make all functional tests reusable by other projects https://review.opendev.org/657659 | |
| 08:08:13 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Validate requested host/node during servers create https://review.opendev.org/661237 | |
| 08:08:14 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520 | |
| 08:14:33 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Specifying az when restore shelved server https://review.opendev.org/624689 | |
| 08:33:27 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Use SATA bus for cdrom devices when using q35 machine type https://review.opendev.org/663011 | |
| 08:33:28 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887 | |
| 08:54:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Ensure controllers all call super https://review.opendev.org/660950 | |
| 08:55:07 | stephenfin | bauzas: Is this clear(er) now? https://review.opendev.org/#/c/660774/3/nova/compute/manager.py | |
| 09:01:15 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520 | |
| 09:20:09 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Use SATA bus for cdrom devices when using q35 machine type https://review.opendev.org/663011 | |
| 09:20:10 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887 | |