| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-14 | |||
| 21:26:53 | mriedem | jroll: TheJulia: fyi https://bugs.launchpad.net/nova/+bug/1815799 | |
| 21:26:53 | openstack | Launchpad bug 1815799 in OpenStack Compute (nova) "Attempting to delete a baremetal server places server into ERROR state" [Low,Confirmed] | |
| 21:27:03 | mriedem | maybe a simple retry loop in the ironic driver on node destroy would solve that | |
| 21:28:03 | TheJulia | mriedem_away: we should be able to just pass on it if if we've already started the process of deleting the node in ironic | |
| 21:36:45 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | |
| 21:37:29 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.openstack.org/637716 | |
| 21:44:22 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: virt/arch: use 'armv6l' for ARMV6 architecture https://review.openstack.org/643467 | |
| 21:46:33 | hrw | that ^^ change may fix Kolla CI error | |
| 21:57:17 | melwitt | dansmith ^ | |
| 22:02:54 | hrw | and sorry for writing 3rd patch to what looks the same | |
| 22:07:12 | hrw | dansmith: after reading https://review.openstack.org/#/c/643458 and https://review.openstack.org/#/c/643455 I think that 643355 is wrong, your patch is good (left comment) and mine https://review.openstack.org/643467 just fixes to comply with capabilities contents | |
| 22:07:30 | hrw | but that's me - random guy from the internet D: | |
| 22:13:24 | openstackgerrit | Merged openstack/nova master: Customize irrelevant-files for nova-live-migration job https://review.openstack.org/643346 | |
| 22:13:43 | openstackgerrit | Merged openstack/nova master: Re-enable Ceph in live migration testing https://review.openstack.org/643150 | |
| 22:21:35 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get https://review.openstack.org/642899 | |
| 22:26:57 | dansmith | melwitt: what are you pointing me to? | |
| 22:27:25 | melwitt | dansmith: hrw's patch https://review.openstack.org/643467 | |
| 22:28:04 | melwitt | it's related to the unknown arch blowup, sort of | |
| 22:28:28 | dansmith | yeah, I'm not sure which is the official arch name and/or if they're both legit as I'm not an arm specialist | |
| 22:28:52 | dansmith | but my patch is still something we should do as (as hrw said) | |
| 22:29:12 | hrw | dansmith: armv6 armv6l depends on kernel. | |
| 22:29:13 | melwitt | hrw: if you think https://review.openstack.org/643455 is wrong, it would be helpful to put your comment on it about why. I'm curious anyway | |
| 22:29:23 | dansmith | hrw: my concern would be it might also depend on libvirt behavior | |
| 22:29:40 | melwitt | dansmith: yeah, not saying your patch isn't something we should do. just that along that line, if you knew anything about changing that enum, hrw's patch was there | |
| 22:29:52 | dansmith | hrw: because if the l just means long or something, then I would expect both or either could be emulatable | |
| 22:30:51 | dansmith | melwitt: hopefully it's obvious that hrw's patch to change the value of the enum is not okay :) | |
| 22:31:29 | melwitt | dansmith: gah, I thought maybe you could comment on the review | |
| 22:31:40 | melwitt | nevermind me | |
| 22:31:55 | hrw | dansmith: libvirt mentions only armv6l and armv7l. | |
| 22:31:59 | dansmith | melwitt: I just did | |
| 22:32:05 | melwitt | thanks | |
| 22:32:14 | dansmith | hrw: on your system you mean right? | |
| 22:32:17 | hrw | https://gitlab.com/search?utf8=%E2%9C%93&snippets=&scope=&search=VIR_ARCH_ARMV6L&project_id=192693#L30 | |
| 22:32:23 | hrw | in libvirt code | |
| 22:33:29 | dansmith | hrw: okay, like I said, I'm not really able to say whether armv6 or armv6l (or both) are right | |
| 22:33:49 | dansmith | but we can't just make the change you've proposed because it has effects to versioned objects and will break communication between services | |
| 22:33:55 | hrw | ok. | |
| 22:34:03 | hrw | perfectly fine for me. | |
| 22:34:30 | dansmith | hrw: I will say, if you're right that it's always armv6l, then we definitely need a variant of your patch over mgoddard's | |
| 22:34:55 | hrw | dansmith: let me discuss it tomorrow with libvirt guys, ok? | |
| 22:34:58 | hrw | 23:34 here | |
| 22:35:13 | dansmith | sure | |
| 22:36:05 | hrw | as nova/virt/libvirt gets capabilities from libvirt so is it armv6 or armv6l is more or less their decision. if they say 'no, it was always armv6z' then we both are wrong | |
| 22:36:57 | dansmith | yeah | |
| 22:36:59 | hrw | the good point is that armv6 is not able to be a host so it is only guest issue | |
| 22:38:07 | openstackgerrit | Kevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter https://review.openstack.org/633178 | |
| 22:38:17 | dansmith | I'm not sure that's relevant to the thing that nova has to do in this case, but.. okay :) | |
| 22:38:32 | hrw | nothing in this case ;D | |
| 22:42:11 | hrw | ok, went, asked, go to sleep. wake up in ~6h | |
| 22:42:27 | dansmith | go :) | |
| 22:44:21 | hrw | and I promised myself to not play with nova for at least a year... ;d | |
| 22:44:43 | hrw | have a nice day | |
| 23:03:00 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove [ironic]api_endpoint option https://review.openstack.org/643483 | |
| 23:03:27 | openstackgerrit | Julia Kreger proposed openstack/nova master: [ironic] ignore conflict upon cleaning up node https://review.openstack.org/643484 | |
| 23:03:36 | TheJulia | mriedem_away: melwitt: ^^^ | |
| 23:06:11 | melwitt | TheJulia: commented. I had thought jroll or someone had said we shouldn't do that in _cleanup_deploy unilaterally | |
| 23:07:21 | TheJulia | I really don't know what to do then | |
| 23:10:56 | melwitt | TheJulia: well, I just mean that try-except would move to the destroy() method | |
| 23:11:13 | TheJulia | I'm checking to see if we can do that | |
| 23:11:25 | melwitt | if I'm remembering correctly that it should be done for all clean deploys. I might be wrong | |
| 23:11:32 | melwitt | *shouldn't | |
| 23:14:21 | TheJulia | well, making sure we nuke everything and if we can just tolerate it on a destroy | |
| 23:19:13 | TheJulia | melwitt: looks like we can tolerate a conflict anywhere along there because ironic will do cleanup. Some of the things more recently, but yeah... we should be good to just move it | |
| 23:20:04 | melwitt | TheJulia: ok. maybe we'll double check with jroll first before you change anything. bc I'm not 100% sure I remembered that right and don't remember what date to look for #openstack-ironic logs | |
| 23:20:24 | TheJulia | melwitt: I just double checked the paths and coductor behavior | |
| 23:20:56 | melwitt | TheJulia: oh, sorry, I misunderstood. you confirmed it should be only for destroy() and not all cleanup deploy | |
| 23:21:05 | TheJulia | yeah | |
| 23:21:08 | melwitt | ok, cool | |
| 23:27:55 | openstackgerrit | Julia Kreger proposed openstack/nova master: [ironic] ignore conflict upon cleaning up node https://review.openstack.org/643484 | |
| 23:27:59 | TheJulia | melwitt: ^^ | |
| 23:28:39 | melwitt | coolness | |
| 23:29:53 | melwitt | TheJulia: there's comments on PS1 about whether it's ok to ignore any and all conflict (only one way that conflict can occur) | |
| 23:30:33 | melwitt | if you could reply and confirm that | |
| 23:36:08 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add description about sort order in API ref guideline https://review.openstack.org/627282 | |
| #openstack-nova - 2019-03-15 | |||
| 00:36:16 | gmann | takashin: ^^ let's just say "- more information about sorting order of response results". mentioned particular field even as example can create confusion. | |
| 01:01:20 | takashin | gmann: Thanks. I will fix it. | |
| 01:02:39 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add description about sort order in API ref guideline https://review.openstack.org/627282 | |
| 01:31:01 | openstackgerrit | Fan Zhang proposed openstack/nova master: Retry after hitting libvirt error VIR_ERR_OPERATION_INVALID in live migration. https://review.openstack.org/612272 | |
| 01:35:52 | openstackgerrit | Julia Kreger proposed openstack/nova master: [ironic] ignore conflict upon cleaning up node https://review.openstack.org/643484 | |
| 01:40:45 | openstackgerrit | zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403 | |
| 01:51:50 | openstackgerrit | zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403 | |
| 02:45:14 | openstackgerrit | zhufl proposed openstack/nova master: Pass kwargs to exception to get better format of error message https://review.openstack.org/643501 | |
| 02:52:51 | openstackgerrit | Merged openstack/nova stable/queens: Fix WeighedHost logging regression https://review.openstack.org/641359 | |
| 04:13:24 | openstackgerrit | Merged openstack/nova master: Add functional test to delete a server while in VERIFY_RESIZE https://review.openstack.org/615644 | |
| 05:09:56 | openstackgerrit | Merged openstack/nova stable/pike: Migrate nova v2.0 legacy job to zuulv3 https://review.openstack.org/620579 | |
| 07:17:45 | openstackgerrit | zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403 | |
| 08:01:18 | kashyap | efried: dansmith: Caught up with the scrollback on #openstack-meeting. And this is the Intel's equivalent for AMD's SEV: "MKTME": https://www.redhat.com/archives/libvir-list/2019-February/msg01730.html | |
| 08:01:59 | kashyap | dansmith: efried: The enablement of "MKTME" (Multikey Total Memory Encryption) is not merged in kernel itself yet. | |
| 08:02:36 | kashyap | Then comes the design and implementation part in QEMU and libvirt (which is what the above libvirt mailing list URL points to). | |
| 08:17:39 | openstackgerrit | Yongli He proposed openstack/nova master: Clean up orphan instances https://review.openstack.org/627765 | |
| 08:45:18 | bauzas | good morning Nova | |
| 08:47:52 | hrw | morning | |
| 10:07:45 | kashyap | Probably it's OK to not check too. "Distraction is the bane" ;-) | |
| 10:18:38 | hrw | ;D | |
| 10:18:48 | hrw | zuul -> irc gate would be lovely | |
| 11:11:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix ProviderUsageBaseTestCase._run_periodics for multi-cell https://review.openstack.org/641179 | |
| 11:11:29 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Improve CinderFixtureNewAttachFlow https://review.openstack.org/639382 | |
| 11:11:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1818914 https://review.openstack.org/641521 | |
| 11:11:30 | openstack | bug 1818914 in OpenStack Compute (nova) "Hypervisor resource usage on source still shows old flavor usage after resize confirm until update_available_resource periodic runs" [Low,In progress] https://launchpad.net/bugs/1818914 - Assigned to Matt Riedemann (mriedem) | |
| 11:11:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove unused context parameter from RT._get_instance_type https://review.openstack.org/641792 | |