| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-15 | |||
| 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 | |
| 19:12:18 | mriedem | bah | |
| 19:12:25 | mriedem | i know he can't quite this early | |
| 19:14:00 | mriedem | https://www.youtube.com/watch?v=nsmY0tFJEIE | |
| 19:17:17 | melwitt | yessssss func test "OK" | |
| 19:17:43 | artom | mriedem, that's a very unhealthy relationship to have with a piece of software ;) | |
| 19:18:08 | artom | I suppose that could change in the future once AI becomes better | |
| 19:18:27 | mriedem | artom: i'll let you make the firmware joke here | |
| 19:19:37 | artom | You have to flash it. | |
| 19:19:47 | mriedem | HI-O | |
| 19:20:07 | artom | *finger guns* | |