| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-15 | |||
| 17:13:24 | mriedem | artom: well it's not really only RH, it's any distro that EOLs an old series and customers are forced to move up | |
| 17:13:41 | aspiers | any other mock experts wanna review that? | |
| 17:13:48 | mriedem | which is why we get all the RH queens bug reports bubblin gup now | |
| 17:14:23 | artom | mriedem, are there other significant distros besides OSP at this point? Honest question | |
| 17:14:36 | cdent | efried: I assume you probably know this already and we're merely following protocol: but I didn't write that "single pass instance info fetch in host manager". My ressurection of it was solely based on the profiling benefits. So a) I'll have to dig deeper to respond to the comment, b) the comments make that digging easier. | |
| 17:14:44 | artom | Commercial, I mean | |
| 17:14:45 | mriedem | canonical? VIO? suse? idk ask their devs | |
| 17:14:47 | sean-k-mooney | i think ubuntu still has more deplopyment then osp | |
| 17:14:52 | efried | aspiers: lgtm, thank you. | |
| 17:15:00 | artom | Ah, SUSE, right | |
| 17:15:08 | artom | (With apologies to aspiers) | |
| 17:15:08 | mriedem | artom: you can ask aspiers how suse is ramping up for rolling out rocky with SEV forks :) | |
| 17:15:31 | sean-k-mooney | and canonical openstack | |
| 17:15:32 | aspiers | artom: what are you apologising for? :) | |
| 17:15:45 | artom | aspiers, forgetting you exist, essentially ;) | |
| 17:15:49 | aspiers | haha | |
| 17:15:55 | aspiers | easily done | |
| 17:16:06 | aspiers | mriedem: yeah sadly that's probably gonna happen | |
| 17:16:33 | efried | cdent: ack. If we want to land this pretty much as-is, we just need unit/func test coverage in this patch. The rest can be FUPed. | |
| 17:16:34 | aspiers | Not 100% sure though. We have a policy of trying to stay as close to upstream as humanly possible, even on stable branches. | |
| 17:16:58 | aspiers | artom: let me know if I can help with anything from the distro angle | |
| 17:17:07 | sean-k-mooney | so do we but PMs have a different opipion on that when there are $$$$ | |
| 17:17:22 | artom | aspiers, thank you :) Though we were just discussing dev/QE lag | |
| 17:17:36 | artom | aspiers, and how the RH QE is lagging behind upstream | |
| 17:17:55 | artom | So we'll do an RFE, and like a year later, we find out if never worked | |
| 17:18:07 | artom | (Only slightly exaggerating) | |
| 17:18:12 | aspiers | artom: well at least you do QE on upstream :) | |
| 17:18:21 | artom | aspiers, we don't, hence the lag | |
| 17:18:34 | aspiers | I mean eventually | |
| 17:18:40 | artom | By the time stuff's in our builds, it's been months, if not years | |
| 17:18:45 | aspiers | lag implies it happens eventually :) | |
| 17:18:53 | aspiers | If it never happens, that's infinite lag | |
| 17:19:09 | aspiers | although maybe I kinda just contradicted myself | |
| 17:20:04 | openstackgerrit | Matt Riedemann proposed openstack/nova master: API microversion 2.76: Add 'power-update' external event https://review.opendev.org/645611 | |
| 17:20:09 | mriedem | dansmith: ^ | |
| 17:35:11 | dansmith | mriedem: looks good | |
| 18:03:57 | melwitt | argh why does it do "An auth plugin is required to determine endpoint URL" when I'm using the PlacementFixture | |
| 18:04:17 | efried | melwitt: You need to use the sdk fixture :) | |
| 18:04:19 | melwitt | this is what happens when I don't use ServersTestBase | |
| 18:04:39 | melwitt | oh. ok | |
| 18:04:53 | efried | (as well as PlacementFixture, if what you're trying to do involves real placement calls) | |
| 18:05:32 | melwitt | the sdk fixture is in test.TestCase | |
| 18:06:15 | efried | melwitt: are you getting the error locally or in zuul? | |
| 18:06:49 | efried | locally => upgrade your ksa and sdk in your virtualenv | |
| 18:06:49 | efried | zuul => rebase and recheck, since the u-c bump landed overnight. | |
| 18:06:58 | melwitt | locally. I'm trying to change from deriving from ServersTestCase to just TestCase | |
| 18:07:07 | melwitt | ok | |
| 18:07:39 | efried | yeah, I bet it's because ksa and/or sdk in your venv is downlevel OR those are latest but you don't have the new fixture you merged last night. | |
| 18:10:23 | melwitt | I had just recreated my venv. but yeah let me just rebase again in case that helps. I can never remember how this works with pulling u-c | |
| 18:12:08 | efried | melwitt: If you recreated your venv without https://review.opendev.org/676707 then you might still have the old libs, I'm not actually sure. | |
| 18:12:27 | efried | You can check with | |
| 18:12:27 | efried | .tox/$env/bin/pip show openstacksdk keystoneauth1 | |
| 18:13:14 | efried | ...and of course you'll want your change rebased against latest master to pick up last night's fixture change to accommodate that. | |
| 18:13:19 | melwitt | 0.34.0 and 3.16.0 | |
| 18:13:49 | efried | those are latest, good; so you'll have a problem if your change is based earlier than the fixture commit from yesterday. | |
| 18:14:30 | efried | ...and if all that is up to date, then it's a legit code problem :) | |
| 18:14:30 | melwitt | it is. thanks, rebasing now | |
| 18:14:53 | efried | okay, cool, lmk if you still get errors of this ilk, I'm tracking closely. | |
| 18:15:00 | melwitt | thanks | |
| 18:16:43 | openstackgerrit | sean mooney proposed openstack/nova master: Deprecate Aggregate[Core|Ram|Disk]Filters https://review.opendev.org/673496 | |
| 18:16:47 | sean-k-mooney | melwitt: care to take a look at ^ | |
| 18:17:23 | melwitt | no | |
| 18:17:25 | melwitt | jk | |
| 18:18:03 | sean-k-mooney | i would not blame you :P | |
| 18:18:35 | sean-k-mooney | but you more or less deprecated them with your mail post like 18 months ago but we never actully did | |
| 18:18:49 | melwitt | I feel attacked | |
| 18:19:02 | melwitt | yeah, I know. I'll look at it | |
| 18:19:44 | sean-k-mooney | cool im going to finish one last test and then call it a day | |
| 18:19:48 | sean-k-mooney | thanks | |
| 18:20:29 | mriedem | "no" was the correct answer | |
| 18:23:27 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add |
|
| 18:23:28 | openstackgerrit | Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 18:23:28 | openstackgerrit | Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 18:32:49 | openstackgerrit | Adam Spiers proposed openstack/nova master: Allow assertXmlEqual() to pass options to matchers.XMLMatches https://review.opendev.org/674628 | |
| 18:32:49 | openstackgerrit | Adam Spiers proposed openstack/nova master: Fix libvirt driver tests to use LibvirtConfigCapsGuest instances https://review.opendev.org/674629 | |
| 18:32:50 | openstackgerrit | Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151 | |
| 18:32:50 | openstackgerrit | Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | |
| 18:32:51 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |
| 18:32:52 | openstackgerrit | Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 18:32:52 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add |
|
| 18:32:53 | openstackgerrit | Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 18:32:53 | openstackgerrit | Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 18:43:18 | mriedem | sean-k-mooney: melwitt: i reviewed that deprecation patch (again) | |
| 18:43:24 | mriedem | more docs fun | |
| 18:44:49 | mriedem | melwitt: btw i missed that you split this all up https://review.opendev.org/#/c/640898/ - bug me next week to review that series; i wanted to play with it in devstack while reviewing, which i won't have time for today | |
| 18:45:42 | sean-k-mooney | mriedem: jsut looked ya i can fix up those issues and repin ill likely do it tomrrow however | |
| 18:46:02 | melwitt | mriedem: cool, will do. I'm gonna respin it to address tetsuro's comments, then I'll bug you next week | |
| 18:46:08 | melwitt | thanks | |
| 18:46:10 | mriedem | it's only ~8pm, you've got 5 more hours to put in sean | |
| 18:46:31 | sean-k-mooney | for once i started at 9 this moring | |
| 18:46:48 | sean-k-mooney | well 9 ish | |
| 18:58:20 | mriedem | what time do you usually start? | |
| 18:58:41 | sean-k-mooney | around 10:30 or 11 | |
| 18:58:47 | mriedem | cripes | |
| 18:58:57 | mriedem | i could watch the price is right every day | |
| 18:59:52 | sean-k-mooney | and i usually take an hour or two break between around 6:30 ish and come back on line around 7 or 8 with the intent of finsih up before 8 | |
| 18:59:55 | sean-k-mooney | *10 | |
| 19:00:15 | sean-k-mooney | i often dont but that in theory is the plan | |
| 19:11:27 | mriedem | sean-k-mooney: vpmu comments as well https://review.opendev.org/#/c/671338/ | |
| 19:12:02 | artom | He waved us good night on the internal channel | |