| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-15 | |||
| 15:51:58 | aspiers | so maybe you'll even be able to ignore not just my spec, but my patches too! | |
| 15:52:29 | mriedem | that was supposed to be angry eyebrows | |
| 15:52:49 | aspiers | and if you need help with that, I remember corvus pointing out that gertty makes ignoring reviews easier than ever ;-) | |
| 15:53:29 | melwitt | speaking of ignoring specs, I've been thinking whether we should organize a spec review day for next week ahead of s-1 (even though s-1 is not the spec freeze) | |
| 15:53:31 | aspiers | maybe he can even add a -3 hotkey specially for jaypipes ... | |
| 15:59:44 | mriedem | melwitt: yes we likely need a spec review day; i know i haven't been doing a good job of reviewing specs for stein. | |
| 16:05:01 | melwitt | we usually do them on tuesday. are y'all game for a spec review day next tuesday oct 23? alex_xu dansmith efried johnthetubaguy stephenfin bauzas | |
| 16:05:35 | efried | wfm | |
| 16:18:45 | openstackgerrit | Corey Bryant proposed openstack/osc-placement master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610074 | |
| 16:19:53 | openstackgerrit | Corey Bryant proposed openstack/os-vif master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610068 | |
| 16:21:44 | openstackgerrit | Jan Gutter proposed openstack/os-vif master: Extend host_info to cover port profiles https://review.openstack.org/610636 | |
| 16:24:31 | openstackgerrit | Corey Bryant proposed openstack/os-traits master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610065 | |
| 16:40:02 | mriedem | tuesday the 23rd works for me | |
| 16:54:34 | melwitt | I'll send a mail to the ML proposing next tuesday to get some async responses | |
| 17:36:30 | openstackgerrit | Merged openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068 | |
| 17:38:40 | openstackgerrit | melanie witt proposed openstack/nova stable/rocky: Use nova-consoleauth only if workaround enabled https://review.openstack.org/610673 | |
| 17:42:32 | openstackgerrit | Merged openstack/nova master: Add microversion 2.67 to rest api version history https://review.openstack.org/610349 | |
| 17:57:39 | mriedem | efried: jackding: a few things in https://review.openstack.org/#/c/607989/ | |
| 17:57:46 | mriedem | mostly to just tighten up that spec | |
| 18:00:19 | jackding | mriedem, will have a look thanks | |
| 18:07:07 | efried | mriedem, jackding: Responded to a few of them. | |
| 18:10:03 | openstackgerrit | Corey Bryant proposed openstack/osc-placement master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610074 | |
| 18:10:40 | openstackgerrit | Corey Bryant proposed openstack/os-traits master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610065 | |
| 18:11:02 | openstackgerrit | Corey Bryant proposed openstack/os-vif master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610068 | |
| 18:17:05 | coreycb | melwitt: hi, you may have noticed we started proposing patches to enable py37 unit tests and disable py35 unit tests. just wanted to give you a heads up to ensure it's not a surprise. just a few more to go. | |
| 18:18:07 | melwitt | coreycb: I have noticed and thank you for the heads up :) | |
| 18:19:51 | openstackgerrit | Corey Bryant proposed openstack/nova master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610694 | |
| 18:21:00 | mriedem | efried: it wasn't clear to me the compute code was going to be setting the trait | |
| 18:21:35 | efried | mriedem: Guess that's a thing to be called out in the spec, then :p | |
| 18:21:43 | mriedem | yeah | |
| 18:21:49 | mriedem | otherwise this is all straight-forward | |
| 18:22:39 | efried | I took it for granted based on the code, but that shouldn't be a prereq for understanding the spec of course | |
| 18:40:28 | mriedem | artom: a few small things in https://review.openstack.org/#/c/609517/ | |
| 18:41:08 | artom | mriedem, much thanks, will address now! | |
| 18:46:16 | mriedem | artom: also, something is a false positive in the tests | |
| 18:46:28 | artom | mriedem, how so? | |
| 18:46:48 | mriedem | i moved the try/except around only the terminate_connection call and both tests passed | |
| 18:46:54 | mriedem | but the attachment_delete one should have failed | |
| 18:47:21 | artom | Dammi :( OK, I'll look into that | |
| 18:47:41 | artom | (I *did* do that with a previous version, I swear!) | |
| 18:49:35 | openstackgerrit | Merged openstack/nova master: Add some more docs for upgrade checkers https://review.openstack.org/604486 | |
| 18:57:10 | mriedem | Kevin_Zheng: i replied to your -1 in https://review.openstack.org/#/c/608777/ - are you still -1 on that or can it be dropped? | |
| 19:04:21 | mriedem | this seems...excessive: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22detaching%20network%20adapter%20failed.%3A%20libvirtError%3A%20internal%20error%3A%20End%20of%20file%20from%20qemu%20monitor%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d | |
| 19:07:32 | mriedem | oh i see, | |
| 19:07:54 | mriedem | we're tearing down, we get a network-vif-deleted event right before the guest is deleted in the hypervisor | |
| 19:08:30 | mriedem | doesn't "End of file from qemu monitor" usually indicate the guest is gone? | |
| 19:13:19 | openstackgerrit | Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | |
| 19:29:12 | artom | mriedem, ah, it's because my poor man's mock checks for volume_id, but is called with an attachment_id. | |
| 19:29:20 | artom | So it doesn't actually raise | |
| 19:30:56 | mriedem | yeah... | |
| 19:31:08 | mriedem | so track call count and raise count? | |
| 19:38:31 | mriedem | jackding: just a couple of small updates in that spec | |
| 19:38:33 | mriedem | for hpet | |
| 19:38:59 | mriedem | btw, how does the libvirt driver know it can report the hpet trait? | |
| 19:39:03 | mriedem | something on the host capabilities? | |
| 19:40:10 | dansmith | I think so yeah | |
| 19:47:06 | imacdonn | dansmith mriedem efried: please re-review this - would like to get it wrapped up, so I can look at applying the fix to cinder too.... https://review.openstack.org/608091 | |
| 19:47:33 | efried | imacdonn: My hands are off that one. It's down to dansmith and mriedem. | |
| 19:48:01 | imacdonn | efried: OK. The logic did change a bit, so thought you might want to look again | |
| 19:48:07 | dansmith | I've had it open in the queue | |
| 19:48:19 | dansmith | I keep coming up with excuses :) | |
| 19:48:22 | imacdonn | :) | |
| 19:48:43 | efried | imacdonn: I would, if I weren't completely underwater, or if there weren't already two cores on it who are way more familiar with the domain. | |
| 19:49:04 | imacdonn | efried: understood - thanks! | |
| 19:51:20 | jackding | mriedem: I'm not familiar with traits, but I thought that's something virt driver could query the host from. | |
| 19:53:55 | mriedem | jackding: efried: well efried said the virt driver would report the hpet trait, | |
| 19:54:05 | mriedem | but looking at the nova code change, i don't see it doing anything like that | |
| 19:54:16 | efried | mriedem: The code change hasn't caught up yet. | |
| 19:54:42 | efried | What I said earlier was based on the fact that the virt driver code is doing the error checking. | |
| 19:56:26 | mriedem | so...we need to configure the guest with a timer element https://libvirt.org/formatdomain.html#elementsTime | |
| 19:56:46 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in _post_live_migration https://review.openstack.org/609517 | |
| 19:56:53 | mriedem | i guess it already does, but always sets it to false | |
| 19:58:41 | dansmith | imacdonn: you're going to love me | |
| 19:58:46 | imacdonn | uh oh | |
| 19:58:49 | dansmith | imacdonn: but I -1d for a single character change | |
| 19:59:01 | dansmith | imacdonn: however, fix it fast and I shall stand by for +2ing | |
| 19:59:25 | imacdonn | dang .. ok | |
| 19:59:55 | mriedem | jackding: i guess it's not clear to me how the libvirt driver will determine that the underlying hypervisor supports hpet | |
| 20:00:10 | mriedem | is it just assumed that all x86 hosts support hpet? | |
| 20:00:46 | openstackgerrit | iain MacDonnell proposed openstack/nova master: Handle online_data_migrations exceptions https://review.openstack.org/608091 | |
| 20:00:47 | artom | Back when I was compiling my own kernels this was an option you could disable, so I don't think we should assume that | |
| 20:01:23 | imacdonn | dansmith: how fast is fast? :) | |
| 20:01:26 | dansmith | imacdonn: that fast | |
| 20:01:29 | imacdonn | cool | |
| 20:01:54 | dansmith | artom: correct | |
| 20:01:57 | jackding | in our implementation we assume x86 support hpet | |
| 20:02:06 | mriedem | blech | |
| 20:02:15 | mriedem | this is when i need kashyap | |
| 20:02:55 | mriedem | i realize windriver can be super opinionated on the stuff they deploy, but not so much upstream | |
| 20:04:08 | dansmith | mriedem: if all x86 could be assumed to have it, you wouldn't need this trait, just require arch=x86 | |
| 20:05:53 | mriedem | good point | |
| 20:08:36 | artom | tsc hpet acpi_pm | |
| 20:08:36 | artom | So, hardware-wise, it's apparently as easy as [artom@jayne ~]$ cat /sys/devices/system/clocksource/clocksource0/available_clocksource | |
| 20:08:46 | artom | But. Do we need anything from libvirt? | |
| 20:09:34 | mriedem | i was hoping something would be in the host capabilities API | |
| 20:09:35 | mriedem | http://logs.openstack.org/39/587439/4/check/tempest-full/fdb1cf9/controller/logs/screen-n-cpu.txt.gz#_Sep_11_14_44_27_658449 | |
| 20:09:37 | mriedem | but i don't see that | |
| 20:10:28 | mriedem | or maybe hpet just doesn't show up there | |
| 20:10:33 | mriedem | ? | |
| 20:10:36 | mriedem | because it's nested virt? | |