| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-11 | |||
| 08:39:49 | openstack | Launchpad bug 1741364 in OpenStack Compute (nova) "ceph ephemeral info not updated during live migrate" [Medium,Confirmed] | |
| 08:39:49 | trident | Any thoughts on https://bugs.launchpad.net/nova/+bug/1741364 ? | |
| 08:41:19 | openstackgerrit | Fan Zhang proposed openstack/nova stable/train: Fix exception translation when creating volume https://review.opendev.org/688072 | |
| 08:43:05 | openstackgerrit | Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073 | |
| 08:43:53 | openstackgerrit | Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073 | |
| 09:03:58 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/train: Fix unit of hw_rng:rate_period https://review.opendev.org/688083 | |
| 09:08:42 | openstackgerrit | ya.wang proposed openstack/nova master: libvirt: Change _compare_cpu to raise InvalidCPUInfo https://review.opendev.org/687808 | |
| 09:21:27 | openstackgerrit | Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073 | |
| 09:26:59 | openstackgerrit | Fan Zhang proposed openstack/nova stable/rocky: Fix exception translation when creating volume https://review.opendev.org/688089 | |
| 09:31:40 | openstackgerrit | Nicolas Belouin proposed openstack/nova master: Add support for Xen PVH VM Mode in libvirt driver https://review.opendev.org/687827 | |
| 09:39:13 | stephenfin | did we not have a meeting last night | |
| 09:41:10 | openstackgerrit | Fan Zhang proposed openstack/nova stable/queens: Fix exception translation when creating volume https://review.opendev.org/688090 | |
| 11:20:27 | kashyap | Any stable cores, mind having a look at this stable/queens thing? It has also merged in other stable branches: https://review.opendev.org/#/c/668111/ (libvirt: Rework 'EBUSY' (SIGKILL) error handling code path | |
| 11:20:31 | kashyap | ) | |
| 11:21:03 | kashyap | (It has merged in Rocky and Stein.) | |
| 11:34:26 | lyarwood | kashyap: yup sorry looking now | |
| 11:34:47 | kashyap | Gracias | |
| 11:56:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Error out interrupted builds https://review.opendev.org/687918 | |
| 11:59:21 | kashyap | Needs a second stable core +2 here: https://review.opendev.org/#/c/668111/ | |
| 12:21:39 | openstack | Launchpad bug 1847759 in tempest "reboot_server_hard can trigger corrupted file system" [Undecided,New] | |
| 12:21:39 | frickler | this bug is targeted on tempest, but I'd like to know from nova ppl how valuable testing the hard reboot is to you, as I'm considering to propose dropping this test if it can't be made more reliable https://bugs.launchpad.net/tempest/+bug/1847759 | |
| 12:25:46 | lyarwood | frickler: IMHO it's useful and that sounds worrying as a bug report | |
| 12:28:02 | frickler | lyarwood: I'm even more worried after seeing how many matches my logstash query has, though I haven't verified yet how many of those match the bug http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22self._test_reboot_server('HARD')%5C%22 | |
| 12:29:41 | lyarwood | frickler: I wonder if we even need to boot an instance using an image to test hard reboot | |
| 12:30:25 | lyarwood | frickler: if corruption of the image by the guest OS dying suddenly is the issue here | |
| 12:36:06 | frickler | lyarwood: o.k., I checked some of the failure logs. 2/3 seem to fail to connect to the instance on the initial boot already, 1/3 looks like fs corruption after the hard reboot, so still a relevant set. if you have any idea how to make that more stable, I think that would be pretty useful | |
| 12:57:42 | bauzas | stephenfin: nope, the nova meeting was cancelled, see ML | |
| 13:23:30 | stephenfin | bauzas: ah, yes, thanks | |
| 13:24:02 | stephenfin | bauzas: Would you do me the honours? https://review.opendev.org/#/c/684402/ https://review.opendev.org/#/c/685929/ https://review.opendev.org/#/c/687264// ? | |
| 13:25:30 | openstackgerrit | Matthew Booth proposed openstack/nova master: Don't take a lock when fetching a cached cell mapping https://review.opendev.org/688122 | |
| 13:29:15 | stephenfin | bauzas: ta | |
| 13:35:40 | openstackgerrit | Merged openstack/nova master: api-guide: Fix available info in handling down cells https://review.opendev.org/686520 | |
| 13:41:54 | openstackgerrit | Merged openstack/nova-specs master: Change the primary assignee to the mainly contributor https://review.opendev.org/688042 | |
| 13:41:55 | openstackgerrit | Merged openstack/nova-specs master: Fix followup comments of policy-defaults-refresh spec https://review.opendev.org/669196 | |
| 13:54:13 | mdbooth | lyarwood: Re https://code.engineering.redhat.com/gerrit/182841 GAH! and thanks. Mind if we don't wait for tempest after I update the commit message? | |
| 13:54:59 | lyarwood | mdbooth: wrong window but yeah of course | |
| 13:55:28 | mdbooth | lyarwood: Hah, so it is | |
| 14:11:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Avoid using image with kernel in BDM large request func test https://review.opendev.org/688132 | |
| 14:40:08 | gibi | stephenfin: quick question in https://review.opendev.org/#/c/686802/6//COMMIT_MSG@18 | |
| 14:47:00 | stephenfin | gibi: replied | |
| 14:47:51 | gibi | stephenfin: +2 | |
| 14:47:58 | stephenfin | ta | |
| 14:55:54 | stephenfin | gibi: If you're on a reviewing streak, I'd appreciate your eyes on the following patch too since it's NeutronFixture'y and you know that stuff, heh https://review.opendev.org/#/c/684344/ | |
| 14:56:04 | stephenfin | Feel free to chuck something my way too | |
| 14:56:10 | gibi | stephenfin: reviewing it right no | |
| 14:56:13 | gibi | w | |
| 14:56:20 | stephenfin | nice :D | |
| 14:56:43 | dansmith | gibi: there are a few patches in this stack for which you reviewed the spec that you could +W if you want :) | |
| 14:56:44 | dansmith | gibi: https://review.opendev.org/#/c/687137/4 | |
| 14:57:05 | gibi | dansmith: ack, I will look at it after stephenfin's patch | |
| 14:57:28 | dansmith | gibi: thanks | |
| 14:57:48 | gibi | stephenfin: -1 due to https://review.opendev.org/#/c/684344/15/nova/tests/functional/api_sample_tests/test_floating_ips.py@158 | |
| 14:58:23 | openstackgerrit | Dan Smith proposed openstack/nova master: Add image caching API for aggregates https://review.opendev.org/687140 | |
| 14:58:23 | openstackgerrit | Dan Smith proposed openstack/nova master: Add cache_images() to conductor https://review.opendev.org/687139 | |
| 14:58:24 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Add image precaching docs for aggregates https://review.opendev.org/687348 | |
| 14:58:58 | stephenfin | gibi: Would a follow-up be okay? Can post that now | |
| 14:59:05 | gibi | stephenfin: sure | |
| 14:59:10 | gibi | let me change my vote | |
| 15:03:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Use deepcopy on value returned by NeutronFixture https://review.opendev.org/688139 | |
| 15:29:29 | stephenfin | gibi: Back at you https://review.opendev.org/#/c/688132/ | |
| 15:30:26 | gibi | stephenfin: thanks. Make sense. I need to get back to that | |
| 15:53:44 | gibi | dansmith: reviewd the image chache series left some comments here and there | |
| 15:55:19 | dansmith | gibi: so the return dict thing is leaving us some room for improvement in the future without having to bump the compute rpc version again | |
| 15:55:41 | dansmith | gibi: are you -1 on that or the logging asserts? | |
| 15:55:54 | dansmith | oh you said, because the driver bit is unused | |
| 15:56:06 | gibi | dansmith: you mean that now the rpc returns a dict so later we can add whatever we want to that dict without rpc bump? | |
| 15:56:36 | dansmith | gibi: no, it's returning that so that I could follow on with some stats logging in conductor.. not expecting to add stuff to the dict later, just expecting to use it | |
| 15:56:54 | dansmith | gibi: there was some discussion in those patches (IIRC) but definitely some here about what we might do in the future to log some stats, | |
| 15:57:15 | dansmith | gibi: like "image X seemed to fail on every host" or "34 new downloads, 127 existing", that kind of thing | |
| 15:57:18 | gibi | dansmith: if it will be used in the future then I'm OK with it | |
| 15:57:51 | dansmith | gibi: yep, plan is to use it, just trying to keep the base functionality to these patches and then we can hem and haw over how to calculate some stats in a future patch | |
| 16:01:16 | gibi | dansmith: cool. changed my votes as the rest of my comments can be done in a fup | |
| 16:01:43 | dansmith | gibi: roger, just saw as I was replying, I'll post a fup for the nits and get a WIP enqueued for the stats so I can point at that if the question comes up again :D | |
| 16:02:21 | gibi | dansmith: thanks | |
| 16:04:44 | dansmith | gibi: oh, heh on that libvirt test name... it's totally opposite, I dunno how I did that :P | |
| 16:05:53 | gibi | :) | |
| 16:06:24 | dansmith | I'll call it a "testing gibi's attention to detail" easter egg | |
| 16:06:31 | gibi | it worked :) | |
| 16:06:56 | dansmith | yeah you passed the test this time | |
| 16:07:41 | gibi | I'm wondering was there other tests I did not even notice?! | |
| 16:09:05 | dansmith | gibi: muahah :D | |
| 16:09:12 | gibi | stephenfin: do you want me to refactor the fake image service ? https://review.opendev.org/#/c/688132/1/nova/tests/functional/test_boot_from_volume.py@203 | |
| 16:11:34 | gibi | stephenfin: https://github.com/openstack/nova/blob/ef6e49d5bc721840b331c87c6391a69309253ade/nova/tests/unit/image/fake.py#L45 | |
| 16:12:18 | stephenfin | gibi: If you have time, but I won't block on that now | |
| 16:12:48 | gibi | stephenfin: I can do that later. making a todo... | |
| 16:13:02 | stephenfin | tbh, I'd like to stop using the 'stub_out_image_service' function entirely since it obscures things | |
| 16:13:15 | stephenfin | I've a big functional test cleanup in-progress. Can include that | |
| 16:15:28 | gibi | stephenfin: OK, I will ping you when I reach my todo to see if you have already started on it | |
| 16:15:40 | stephenfin | (y) | |
| 16:16:10 | stephenfin | +2 | |
| 16:18:25 | gibi | Im leaving for today. Have a nice weeked you all! | |
| 16:22:17 | stephenfin | O/ | |
| 16:53:47 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Log some stats for image pre-cache https://review.opendev.org/688173 | |
| 16:53:47 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix up some feedback on image precache support https://review.opendev.org/688172 | |
| 17:37:35 | openstackgerrit | Dan Smith proposed openstack/nova master: Add image caching API for aggregates https://review.opendev.org/687140 | |
| 17:37:36 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Add image precaching docs for aggregates https://review.opendev.org/687348 | |
| 17:37:37 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Log some stats for image pre-cache https://review.opendev.org/688173 | |
| 17:37:37 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix up some feedback on image precache support https://review.opendev.org/688172 | |
| 18:06:28 | openstackgerrit | Stephen Finucane proposed openstack/nova master: setup.cfg: Cleanup https://review.opendev.org/677969 | |
| 18:06:29 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop testing Python 2 https://review.opendev.org/687954 | |