| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-27 | |||
| 10:03:34 | takashin | stephenfin: The stable/train branch has been created in nova. So would you remove -2 in https://review.opendev.org/#/c/576020/ and vote workflow +1 for the patch and subsequent patches? | |
| 10:04:05 | stephenfin | takashin: look two lines up :) | |
| 10:04:39 | stephenfin | I'm trying to figure out if there's any reason not to do it now. It shouldn't cause too many conflicts but I'd like someone else's ack first | |
| 10:04:58 | takashin | Okay. | |
| 10:11:45 | gibi | stephenfin: does these patches makes backporting potential rc bugs harder? | |
| 10:14:33 | stephenfin | gibi: I would say no because it only touches the one file (nova/tests/unit/network/test_neutronv2.py) and there have been very few non-mox removal changes against that file this cycle | |
| 10:14:45 | stephenfin | So the chances of there being bugs introduced that we'd have to fix in the file are low | |
| 10:21:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Stop overriding the 'install_command' https://review.opendev.org/684775 | |
| 10:21:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Use common 'command' definition for unit tests https://review.opendev.org/684774 | |
| 10:21:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Start using global stdout, stderr buffers https://review.opendev.org/685310 | |
| 10:21:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Nuke OS_STDOUT_CAPTURE, OS_STDERR_CAPTURE https://review.opendev.org/684773 | |
| 10:44:38 | gibi | stephenfin: cool. if you need a second core to push them through then ping me | |
| 10:48:31 | stephenfin | gibi: I think oomichi has reviewed most of them too so we should be good. Think you could look at https://review.opendev.org/#/c/684762/ though? | |
| 11:53:12 | efried | stephenfin: re mox, go for it. | |
| 11:53:48 | efried | and yes, having looked briefly, the only delta since +A on those was updating the bp name in the commit messages. If that's true, feel free to fast approve. | |
| 11:54:17 | efried | Is the bp already approved for ussuri? I think I did that. /me looks... | |
| 11:54:53 | efried | nope, lemme hit that... | |
| 11:55:19 | efried | oh, I did but forgot to no-op update the series goal | |
| 11:55:21 | efried | done now | |
| 11:58:52 | efried | kashyap: Could http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009743.html be https://review.opendev.org/#/c/684375/ ? | |
| 12:00:18 | kashyap | efried: Indeed, that's it -- the "kvm_init_vcpu failed" error gives it away (it's _not_ main memory allocation, but a prob w/ NUMA interaction) | |
| 12:00:37 | efried | kashyap: sweet. Can you respond? | |
| 12:00:46 | kashyap | efried: I'll reply there; first to try with kernel 4.19 | |
| 12:00:52 | efried | beaut | |
| 12:00:54 | efried | thank you | |
| 12:01:00 | kashyap | efried: Will do; seems like it _just_ arrived? | |
| 12:01:14 | efried | yeah, <5h ago | |
| 12:01:16 | efried | serendipity | |
| 12:01:37 | kashyap | Ah, missed it; as I was reading only the [nova] tag | |
| 12:02:10 | efried | Couldn't tell you why I actually read this one. Maybe because it said "numa affinity"? | |
| 12:02:10 | kashyap | (And this didn't have the tag) | |
| 12:02:20 | kashyap | Likely | |
| 12:02:44 | kashyap | I don't know if we should make it into yet-another tunable; I'd think we should do the _right_ thing there. Still mulling | |
| 12:03:07 | efried | but someone running into this can resolve by upgrading kernel? | |
| 12:03:08 | kashyap | (Where just use the conservative, 'preferred'. I'll let Sean come back to triple-check on me) | |
| 12:03:23 | kashyap | efried: That's my theory. Let's see if this reporter confirms. | |
| 12:03:31 | efried | is there not a case where you would truly *want* to use strict? | |
| 12:03:48 | kashyap | Also the kernel fix is extremely small (yes, yes, that doesn't say _anything_ about risk) and isoloated; so I'd suggest the Linux distros backport it. | |
| 12:03:52 | efried | seems like something something edge HPC realtime something | |
| 12:04:27 | kashyap | efried: There are cases, but I'll double-check w/ the libvirt dev who meditates on this | |
| 12:05:01 | efried | cool. /me watches from a safe distance | |
| 12:05:04 | kashyap | Heh | |
| 12:06:26 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Stop silently ignoring invalid 'nova boot --hint' options https://review.opendev.org/684762 | |
| 12:08:36 | kashyap | efried: Completely unrelated - I wonder if all the 'no-reply' emails can be directed to another 'openstack-releases' list | |
| 12:08:51 | kashyap | Otherwise the noise feels just too much. | |
| 12:08:56 | kashyap | Maybe I should ask it on the list | |
| 12:09:22 | kashyap | I know that the point of it is to give a heads-up to devs that a new release came out. | |
| 12:09:29 | efried | kashyap: respond to http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009744.html | |
| 12:09:59 | kashyap | Ah, will do | |
| 12:10:03 | kashyap | Thanks for the pointer | |
| 12:10:50 | efried | yahyoubetcha. I didn't care enough to respond; I'm happy to hit [Delete] a couple dozen times twice a year. | |
| 12:11:54 | kashyap | My gripe is, the mailing list archives turn into a toilet | |
| 12:12:10 | kashyap | It just becomes so polluted that the signal is drowned in that noise. | |
| 12:12:34 | kashyap | And it shoos away those who're reading the list to ignore it. I'll respond on that thread. :D | |
| 12:22:13 | zigo | Is Nova really broken with cryptography 2.6 ? | |
| 12:22:27 | zigo | That's kind of annoying me to require 2.7, because the Debian package isn't realy. | |
| 12:22:33 | zigo | (and I'm not the maintainer of that one...) | |
| 12:45:00 | zigo | bauzas: ^ Do you know ? | |
| 12:46:08 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.opendev.org/576020 | |
| 12:46:16 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.opendev.org/576027 | |
| 12:46:30 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.opendev.org/576031 | |
| 12:46:40 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.opendev.org/576299 | |
| 12:46:51 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.opendev.org/576344 | |
| 12:47:02 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.opendev.org/576673 | |
| 12:47:14 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.opendev.org/576676 | |
| 12:47:23 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.opendev.org/576689 | |
| 12:47:31 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.opendev.org/576709 | |
| 12:47:42 | openstackgerrit | Merged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.opendev.org/576712 | |
| 13:04:06 | openstack | Launchpad bug 1845322 in python-novaclient "'--hint' argument to 'nova boot ' should be a key-value pair" [Low,In progress] - Assigned to Stephen Finucane (stephenfinucane) | |
| 13:04:06 | gibi | stephenfin: regarding https://bugs.launchpad.net/python-novaclient/+bug/1845322 I think the same bug exists in openstackclient as well | |
| 13:09:47 | gibi | stephenfin: opened a story for it https://storyboard.openstack.org/#!/story/2006628 | |
| 13:18:10 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.opendev.org/576758 | |
| 13:22:49 | openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova stable/pike: Avoid redundant initialize_connection on source post live migration https://review.opendev.org/683008 | |
| 13:27:34 | efried | stephenfin: Is your nova-net removal series | |
| 13:27:43 | efried | ... "done"? Like, at least proposed all the way up? | |
| 13:27:47 | efried | (I haven't looked) | |
| 13:28:13 | efried | curious whether we could just do away with https://review.opendev.org/#/c/576758/ and/or do the actual removal on top of your series. | |
| 13:30:14 | efried | ...looks like just the test bits are proposed at this point. | |
| 13:45:44 | stephenfin | gibi: Yeah, I pushed patches for that | |
| 13:46:08 | stephenfin | Was trying to figure out why I couldn't file against openstackclient on launchpad. That would be why,heh | |
| 13:46:45 | stephenfin | efried: Yeah, just the functional test piece pushed so far. I'm working on the rest of it slowly | |
| 13:47:06 | stephenfin | Trying to avoid a mega "remove all the things" patch | |
| 13:47:20 | efried | stephenfin: as for cellsv1, good approach. | |
| 13:48:11 | stephenfin | efried: And the ec2 cleanup...which isn't actually merged yet (I thought it was). Would you mind looking at them if/when you have a chance? https://review.opendev.org/#/q/topic:ec2-cleanup+(status:open+OR+status:merged) | |
| 13:48:12 | gibi | stephenfin: cool reading the openstackclient patch now | |
| 13:51:34 | dansmith | mriedem: I think I'm +2 on all the +2able things in stein | |
| 13:51:42 | dansmith | mriedem: (re: your mention from the meeting yesterday) | |
| 13:51:55 | openstackgerrit | melanie witt proposed openstack/nova master: Reduce scope of 'path' query parameter to noVNC consoles https://review.opendev.org/685194 | |
| 13:52:22 | mriedem | dansmith: ack | |
| 13:53:11 | mriedem | lyarwood: if you're about can you hit these? https://review.opendev.org/#/q/project:openstack/nova+branch:stable/stein+status:open+label:Code-Review=2 | |
| 13:55:13 | mriedem | dansmith: can you hit https://review.opendev.org/#/c/680871/ and https://review.opendev.org/#/c/676498/ for rocky? | |
| 13:55:27 | artom | mriedem, you saw ya.wang's review on that NUMA fix? | |
| 13:55:34 | dansmith | I've got the rocky set up right now, are those two more important? | |
| 13:56:07 | mriedem | more important? | |
| 13:56:28 | mriedem | the former is a long-standing issue for baremetal instances, the latter is an lxc thing which is low priority | |
| 13:56:33 | mriedem | lxc+py3 | |
| 13:56:38 | mriedem | artom: not yet | |
| 13:56:41 | dansmith | mriedem: meaning you pointed me at two rocky reviews, just wondering if I can hit those when I get to them in the rocky queue or if I should do those first for some reason | |
| 13:56:42 | mriedem | artom: does it work? | |
| 13:56:48 | artom | mriedem, yeah, tl;dr, he's +1 | |