Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-10-28
18:16:56 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: add troubleshooting guide for cleaning up orphaned allocations https://review.opendev.org/691427
18:20:59 openstackgerrit Merged openstack/nova stable/queens: Fix exception translation when creating volume https://review.opendev.org/688090
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 openstackgerrit sean mooney proposed openstack/nova stable/stein: make config drives sticky bug 1835822 https://review.opendev.org/691864

Earlier   Later