| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-21 | |||
| 13:21:19 | efried | I'm *sure* I've seen it done. But I'm not one to stand on principle and go find an example :) | |
| 13:21:28 | sdague | kashyap: yeh, well, the work around was easy for them. Honestly, I'm never going to ding people for deciding to run with what latest upstream things is fine. :) | |
| 13:21:57 | kashyap | Yep, agreed on that bit. | |
| 13:22:14 | kashyap | We still have to figure out that CI job we've been discussing on the list forever | |
| 13:22:42 | kashyap | Where latest libvirt / QEMU (from Git, newest upstream tarballs) are tested w/ Nova Git. | |
| 13:23:40 | jamespage | sdague, mriedem: I'm going to abandon https://review.openstack.org/#/c/505748/ - its way to brittle relying on calling code to know whether to lock or not. | |
| 13:23:40 | mriedem | efried: stephenfin: deprecated options can go after a release | |
| 13:23:50 | mriedem | jamespage: ok | |
| 13:24:06 | jamespage | mriedem: sdague's approach seems a bit more rugged... | |
| 13:24:18 | sdague | well, mriedem actually wrote the patch | |
| 13:24:18 | mriedem | kashyap: you're the one that's always saying we should run against latest libvirt/qemu :) | |
| 13:24:40 | kashyap | mriedem: Yes, I say it to be done in a _separate_ CI job :-) | |
| 13:24:49 | mriedem | oooooooooooooooooooooooooooooooooohhhhhhhhhhhhhhhhhhhhhhhhhh | |
| 13:24:53 | jamespage | must have missed that.. | |
| 13:24:56 | jamespage | apols | |
| 13:24:56 | stephenfin | mriedem: This is more about the deprecated aliases, where we rename an opt or change its group, but still keep it around | |
| 13:25:19 | stephenfin | efried: Proof or it didn't happen ;) | |
| 13:25:20 | sdague | this only showed up in live migration though, honestly, CI isn't going to catch every edge casee | |
| 13:25:32 | mriedem | stephenfin: deprecated is deprecated what's the difference | |
| 13:25:42 | mriedem | this config option deprecation stuff is grating on my brain | |
| 13:26:33 | stephenfin | mriedem: So I _shouldn't_ push a twenty-patch "remove deprecated opts" series? Aww :'( | |
| 13:28:36 | efried | The difference is just code cleanup / tech debt. We could keep every deprecated option around forever and it wouldn't hurt anything. But it sure would look messy. | |
| 13:31:30 | efried | stephenfin Oh, snap. Here's one. Guess who authored it? I41a6be48380999c49d6db3ee5221174a41014002 | |
| 13:32:45 | efried | reviewed, rather. | |
| 13:33:03 | efried | It was in the same blueprint as the one that moved the vnc opts originally (which was your change set IIRC) | |
| 13:34:23 | openstackgerrit | Merged openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | |
| 13:34:41 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | |
| 13:35:04 | efried | stephenfin And here's one you did co-author: https://review.openstack.org/#/c/463220/ | |
| 13:35:06 | efried | I'll stop now. | |
| 13:36:56 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | |
| 13:38:36 | mriedem | efried: stephenfin: i've been dealing with getting my kid to school, so haven't been following along, what is the actual issue? | |
| 13:38:42 | mriedem | i know about the vnc rename thing | |
| 13:39:59 | dansmith | gibi: responded to some of your comments, thanks for looking at this stuff | |
| 13:40:58 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: implement get_inventory() driver method https://review.openstack.org/506175 | |
| 13:41:30 | efried | mriedem That's all. Just needing https://review.openstack.org/#/c/505831/ to merge pdq for the sake of a couple of 3rd party CIs that were testing with those opts. | |
| 13:43:41 | efried | powervm is kinda bound up between that and https://review.openstack.org/#/c/504194/ -- we need ^ to merge first so our rename of < will pass. | |
| 13:46:37 | mriedem | efried: +2 | |
| 13:46:53 | mriedem | alex_xu: sdague: can you take a look at https://review.openstack.org/#/c/505831/ ? unblocks some 3rd party CIs | |
| 13:47:31 | efried | Thanks! | |
| 13:47:37 | mriedem | efried: are you going to follow that up and drop support for the options in the DEFAULT group/ | |
| 13:47:40 | mriedem | which were deprecated | |
| 13:47:49 | efried | Can | |
| 13:49:25 | sdague | what's the no valid host error all about in CI? | |
| 13:49:30 | bauzas | oomichi: you're around ? see my comments in https://review.openstack.org/#/c/506092/1 ? | |
| 13:51:26 | openstackgerrit | Thomas Bechtold proposed openstack/nova stable/pike: Handle deleted instances when refreshing the info_cache https://review.openstack.org/506183 | |
| 13:52:11 | mriedem | fancy pants nova meeting in 8 minutes | |
| 13:52:42 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix definitely move single instance when created concurrently https://review.openstack.org/506093 | |
| 13:52:53 | bauzas | mriedem: you could have interest in ^ | |
| 13:53:31 | takashin | bauzas: Could you review https://review.openstack.org/#/c/408955/ ? | |
| 13:53:56 | bauzas | takashin: yeah, that's in my pipe for a long way :) | |
| 13:54:17 | takashin | bauzas: Thank you. | |
| 13:54:19 | bauzas | takashin: now we merged the spec, let's go back on the implementation side :) | |
| 13:54:31 | mriedem | bauzas: yeah, after the meeting | |
| 13:54:45 | bauzas | mriedem: np, just wanted to make sure you were aware | |
| 13:55:33 | mriedem | dansmith: oh yeah, remind me to talk to you about filtering on IPs | |
| 13:55:35 | mriedem | with your series | |
| 13:55:42 | mriedem | given the discussion we had in denver about how we don't do that in the DB | |
| 13:55:56 | dansmith | mriedem: okay, does my series change anything about that? | |
| 13:56:00 | mriedem | idk :) | |
| 13:56:12 | dansmith | okay | |
| 13:56:18 | mriedem | haven't gone through all of it, i just know it's a special snowflake in the existing code | |
| 13:56:32 | openstackgerrit | Merged openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485 | |
| 13:56:35 | mriedem | i think, however, we have a functional regression test that i added for filtering on ip | |
| 13:57:00 | mriedem | because i remember once last year accusing steve kaufer of regressing that | |
| 13:57:02 | dansmith | oh, filtering not sorting? | |
| 13:57:06 | mriedem | right | |
| 13:57:10 | mriedem | filter by ip regex | |
| 13:57:12 | dansmith | yeah, that shouldn't be any different | |
| 13:57:40 | dansmith | you can only sort on access_ip_vX, which are strings and fine | |
| 13:57:46 | openstackgerrit | sahid proposed openstack/nova master: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/497455 | |
| 13:57:47 | openstackgerrit | sahid proposed openstack/nova master: libvirt: add method to configure migration speed https://review.openstack.org/497456 | |
| 13:57:48 | openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | |
| 13:57:53 | dansmith | filters basically go straight through like they did before | |
| 13:58:00 | openstackgerrit | Merged openstack/nova master: pci: Validate behavior of empty devname https://review.openstack.org/396130 | |
| 13:58:20 | mriedem | straight through to the db api? | |
| 13:58:25 | openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/nova master: Add default configuration files to data_files https://review.openstack.org/506188 | |
| 13:58:39 | mriedem | if yes, that's my concern, since we don't filter ips in the db | |
| 13:59:04 | mriedem | but i'll look for this test before declaring that the sky has fallen | |
| 13:59:07 | dansmith | if they were passed to the db api before then they still are, and if they weren't then they're not | |
| 14:00:10 | dansmith | mriedem: nova/tests/functional/test_list_servers_ip_filter.py | |
| 14:00:39 | mriedem | dansmith: that's the one | |
| 14:00:40 | mriedem | cool | |
| 14:03:05 | sahid | mriedem: can i have at some point your eyes on https://review.openstack.org/#/c/485522/, the code https://review.openstack.org/#/c/458513/ | |
| 14:03:11 | sahid | you were asking for a spec | |
| 14:04:52 | mriedem | sahid: sure, i'll star it for later today | |
| 14:05:32 | sahid | mriedem: ok thanks | |
| 14:07:08 | openstackgerrit | Merged openstack/nova master: Adopt new pypowervm power_off APIs https://review.openstack.org/476945 | |
| 14:22:18 | dansmith | tssurya: this is the place :) | |
| 14:23:16 | tssurya | dansmith: yep I will be here then :) thank you | |
| 14:36:29 | gibi | dansmith: I'm looking at the scatter_gather code and I don't think I spotted the part that prevents us to return a generator as a result. | |
| 14:36:51 | gibi | dansmith: I even replaced the list() call with a generator expression in your first patch and that seems to still pass the tests | |
| 14:39:41 | dansmith | gibi: hmm, I initially was returning a generator from it and was hitting something in there that was trying to count the result | |
| 14:40:17 | dansmith | gibi: but I guess maybe I was generating an error in one of the threads and that threw me off | |
| 14:40:27 | dansmith | let me try to change it again locally now that things are all working and see | |
| 14:41:06 | gibi | the only length calulation is here https://github.com/openstack/nova/blob/193729b93a11ff54da99386e881270209797f020/nova/context.py#L439 but that is only checking the number of results not the lenght of each result | |
| 14:42:26 | dansmith | yeah | |
| 14:42:35 | dansmith | hmm, yeah, that seems to work | |
| 14:43:04 | dansmith | I dunno what I was hitting before, but I initially had this so it was generators all the way down, convinced myself that wouldn't work across that thread boundary and then wrote that docstring about it | |
| 14:43:21 | dansmith | so... I dunno, but that was super early, so maybe something else was going on | |
| 14:43:26 | dansmith | I shall change it | |