| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-28 | |||
| 18:23:51 | mriedem | dansmith: looking for another stable core on this queens backport https://review.opendev.org/#/c/676285/ | |
| 18:33:33 | melwitt | it's another nomadic wifi week. both starbucks nearest me have no internet either. had to go farther out | |
| 18:36:23 | artom | By virtue of being I assume the fires aren't affecting you too much. Must stil suck tho. | |
| 18:39:15 | melwitt | artom: yeah, I'm in the santa cruz mountains. the current fire in norcal is up in sonoma, pretty far from here, few hours drive | |
| 18:40:57 | melwitt | pg&e has been proactively shutting off power to lots of areas, mine is always included in that. our comcast doesn't have generators so no internet in the area even though we have a generator | |
| 18:41:09 | artom | I remember we had forest fires up north, and the winds blew the smoke down here. Weirdly eery... Weak orange sun, everything out of focus | |
| 18:42:03 | artom | That sucks :( That means no hot water and heating too? | |
| 18:42:38 | melwitt | yeah, there was a lot of that last year. didn't get as far south as santa cruz area but the main bay area looked like apocalypse sky | |
| 18:43:21 | melwitt | we have hot water thankfully, that's on propane | |
| 18:43:50 | artom | That's a relief | |
| 18:44:00 | melwitt | yeah, that's kind of my main thing I need is hot water heh | |
| 18:44:13 | artom | Dammit I have a dead pixel right below by blinking IRC cursor | |
| 18:44:47 | melwitt | :( | |
| 18:45:09 | artom | Yeah, being able to take shower is a massive plus | |
| 19:09:57 | efried | gmann: Can you please make the file path match the bp name for policy-defaults-refresh ? | |
| 19:10:14 | efried | If you want to pull https://review.opendev.org/#/c/686058/ out of the gate I'll fast approve it. Or you can put another on top. | |
| 19:10:39 | efried | looks like it was mismatched for train as well :( | |
| 19:10:48 | efried | I thought we had some kind of build check for that. | |
| 19:12:17 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Re-propose policy-defaults-refresh spec for Ussuri https://review.opendev.org/686058 | |
| 19:12:33 | melwitt | I put a check for that into the count_blueprints script, but other than that, I don't know of anything that checks it | |
| 19:12:50 | efried | okay. | |
| 19:13:05 | efried | I made the change, to save having to add a redirect as well --^ | |
| 19:13:13 | melwitt | cool | |
| 19:15:36 | gmann | efried: ah, thanks for update. | |
| 19:15:43 | efried | yahyoubetcha. | |
| 19:34:54 | openstackgerrit | Merged openstack/nova-specs master: Re-propose policy-defaults-refresh spec for Ussuri https://review.opendev.org/686058 | |
| 19:38:40 | mriedem | move_implemented_specs checks for mismatching filenames to blueprint names, | |
| 19:38:44 | mriedem | but there is no pep8 check for it | |
| 19:51:28 | mriedem | dansmith: i'm open to discussion on some of the things in https://review.opendev.org/#/c/688173/ | |
| 19:51:36 | mriedem | but we should at least have *some* test for some of that i think | |
| 19:55:38 | dansmith | mriedem: s'all good. as I said, I care more about the next patch in line and just built that one on top of this because it was structured properly, but never really got around to doing anything here | |
| 19:56:01 | dansmith | then this morning gibi had +2d everything and I fixed his nits in the top patch not really thinking about it | |
| 19:56:19 | dansmith | so I'll add some more tests to the middle one, probably in the morning | |
| 19:56:30 | dansmith | if you could review the next patch so I have all of that feedback together, that'd be cool | |
| 20:00:45 | mriedem | conditions.....ugh.... | |
| 20:00:54 | mriedem | but i wanted to move onto my next todo which was staring at this wall | |
| 20:07:38 | openstackgerrit | Merged openstack/nova master: Add notification sample test for aggregate.cache_images.start|end https://review.opendev.org/691129 | |
| 20:16:10 | mriedem | dansmith: ok reviewed the top as well | |
| 20:34:52 | openstackgerrit | Merged openstack/nova stable/queens: rt: only map compute node if we created it https://review.opendev.org/676285 | |
| 20:45:25 | dansmith | mriedem: thanks | |
| 22:26:29 | openstackgerrit | melanie witt proposed openstack/nova master: Switch to devstack-plugin-ceph-tempest-py3 for ceph https://review.opendev.org/691765 | |
| 22:29:06 | openstackgerrit | melanie witt proposed openstack/nova master: Switch to devstack-plugin-ceph-tempest-py3 for ceph https://review.opendev.org/691765 | |
| 22:33:26 | openstackgerrit | melanie witt proposed openstack/nova master: Switch to devstack-plugin-ceph-tempest-py3 for ceph https://review.opendev.org/691765 | |
| #openstack-nova - 2019-10-29 | |||
| 00:58:36 | openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:volume:Disallow AIO=native when no 'O_DIRECT' is available https://review.opendev.org/682772 | |
| 02:09:05 | openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:volume:Disallow AIO=native when no 'O_DIRECT' is available https://review.opendev.org/682772 | |
| 02:45:39 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 02:49:24 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 02:50:05 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 02:51:03 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 04:56:50 | openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:volume:Disallow AIO=native when no 'O_DIRECT' is available https://review.opendev.org/682772 | |
| 05:59:44 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 06:12:51 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 06:35:18 | brinzhang | stephenfin: alex_xu: https://review.opendev.org/#/c/682772/ this patach find the root issue and update the changes, change the conf.driver_io in the libvirt driver, not change the conf.driver_io in any disk_driver in https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L174-L200 | |
| 06:36:16 | brinzhang | stephenfin: alex_xu: do you have time to review this, and give any suggestion? | |
| 07:02:25 | openstackgerrit | Daniel Pawlik proposed openstack/nova stable/train: Added openssh-client into bindep https://review.opendev.org/691808 | |
| 07:02:37 | openstackgerrit | Daniel Pawlik proposed openstack/nova stable/stein: Added openssh-client into bindep https://review.opendev.org/691809 | |
| 07:14:00 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 07:49:24 | openstackgerrit | sunjiyun-inspur proposed openstack/nova master: On branch vm-query-notsameproject https://review.opendev.org/691629 | |
| 07:51:34 | openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when no 'O_DIRECT' is available https://review.opendev.org/682772 | |
| 08:03:30 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 09:21:45 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: Support fuzzy querying instance by tag https://review.opendev.org/691651 | |
| 10:03:37 | sean-k-mooney | brinzhang: i guess that is one way to adress it. the other way is to say that its invalid to set the cache mode to 'writeback', 'writethrough'or 'unsafe' | |
| 10:04:15 | sean-k-mooney | e.g. use cache mode = node or direct_sysnc | |
| 10:07:33 | brinzhang | sean-k-mooney: yeah, it's the one way to resolve this issue. | |
| 10:07:50 | sean-k-mooney | this is also arguably a libvirt bug given we currently dont set teh driver io mode | |
| 10:08:21 | sean-k-mooney | im pretty sure i have hit that warning in the past and that is how i fixed it for the lvm driver | |
| 10:08:32 | sean-k-mooney | but using thread io is also accpetable | |
| 10:08:57 | sean-k-mooney | thread io is a little more expensive in terms of qemu cpu overhead | |
| 10:09:14 | brinzhang | I am not sure all of the volume driver can be set thread | |
| 10:09:36 | sean-k-mooney | well we dont want to default to threaded | |
| 10:09:47 | sean-k-mooney | that would be incorrect | |
| 10:10:07 | brinzhang | But if donot set driver_io to thread, the bug | |
| 10:10:34 | brinzhang | the bug is also exist, do you have another suggestion to fix it? | |
| 10:10:52 | sean-k-mooney | no what i mean is we dont want to defualt to therad in general outside of this edgecase | |
| 10:10:54 | brinzhang | I know set thread more expensive | |
| 10:11:19 | sean-k-mooney | by the way i dont think | |
| 10:11:20 | sean-k-mooney | if (hasattr(conf, 'driver_io') and | |
| 10:11:22 | sean-k-mooney | conf.driver_io == "native" and conf.driver_cache in | |
| 10:11:24 | sean-k-mooney | ['writeback', 'writethrough', 'unsafe']): | |
| 10:11:26 | sean-k-mooney | will work | |
| 10:11:42 | sean-k-mooney | do you know where conf.driver_io == "native" is set | |
| 10:12:20 | sean-k-mooney | oh its set in the volume plugins | |
| 10:12:55 | brinzhang | it's in the volume driver, such as iscsi https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/iscsi.py#L57 | |
| 10:13:06 | sean-k-mooney | yes im not sure its valid to override that | |
| 10:13:42 | sean-k-mooney | the code you added woudl basically change that to threaded on systems without direct io | |
| 10:14:40 | sean-k-mooney | brinzhang: ok i would want to see at least a release not to call out the behavior change | |
| 10:15:16 | brinzhang | Yeah, we test it in iscsi, change "native" to "thread" on my env, it's ok on the function, | |
| 10:15:28 | sean-k-mooney | and ideally i think we would also want a warning in init_host signally to the operator that we are falling back to threaded io | |
| 10:15:54 | sean-k-mooney | brinzhang: threaded i think will always be supported as it was teh legacy way fo doing io | |
| 10:16:14 | sean-k-mooney | so ya it should work with isci | |
| 10:16:46 | brinzhang | sean-k-mooney: thanks reply my doubt | |
| 10:17:44 | sean-k-mooney | brinzhang: ill leave some comment on the review but i think i would be ok with this as the direction for the fix. it will enable most cases to just work and as long as we warn the opertaotr of the behavior change i think its ok | |
| 10:18:45 | brinzhang | sean-k-mooney: yeah, thanks. | |
| 11:27:19 | openstackgerrit | Maciej Kucia proposed openstack/nova master: SR-IOV passthrough: Check PF only if VF is enabled https://review.opendev.org/476642 | |
| 11:27:46 | openstackgerrit | sunjiyun-inspur proposed openstack/nova master: query diff user in same project https://review.opendev.org/691849 | |
| 13:11:58 | openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772 | |
| 13:24:33 | openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772 | |
| 13:56:45 | openstack | bug 1835822 in OpenStack Compute (nova) "vms loose acess to config drive with CONF.force_config_drive=True after hard reboot" [Medium,Fix released] https://launchpad.net/bugs/1835822 - Assigned to sean mooney (sean-k-mooney) | |
| 13:56:45 | openstackgerrit | sean mooney proposed openstack/nova stable/stein: make config drives sticky bug 1835822 https://review.opendev.org/691864 | |
| 14:02:33 | mriedem | huh, the changePassword server action api requires you specify adminPass but the value can be an empty string, in which case the server (nova-compute service) generates a password, but its not returned to the caller... | |