| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-05-02 | |||
| 13:12:48 | bauzas | stephenfin: my only concern if we merge your patch is that enduser wouldn' longer see that they need to create their own keypairs | |
| 13:13:26 | bauzas | stephenfin: so if they use openstacksdk directly, they would see then | |
| 13:13:33 | sean-k-mooney | i can check | |
| 13:13:46 | stephenfin | wdym? They don't. We're still doing it for them, only on the client rather than the server | |
| 13:13:48 | stephenfin | Ah | |
| 13:14:08 | stephenfin | I mean, I think that's okay. We do a whole load of helpful things in OSC that don't happen in SDK | |
| 13:15:36 | stephenfin | Like the 'server ssh' command. There's no equivalent for that in SDK, mainly because it's not needed. We could add a docstring in SDK noting that the private key field must be explicitly provided in newer microversions | |
| 13:18:33 | sean-k-mooney | stephenfin: for osc i do not have +2 currently | |
| 13:18:54 | sean-k-mooney | likely the same for sdk i have not actuly looked | |
| 13:26:08 | bauzas | ditto, I'm not osc-core (and not sure I want it :) ) | |
| 13:27:19 | dansmith | kashyap: thanks for looking..that looks like a kernel bug to me and so it'd be good to know if it's been fixed as it's not uncommon to see it | |
| 13:27:31 | stephenfin | Ah, okay, it seems I'd misunderstood what 'python-openstackclient-service-core' was being used for. I'd thought it would contain all the service core groups by default. Evidently not. | |
| 13:28:27 | stephenfin | bauzas: https://review.opendev.org/c/openstack/openstacksdk/+/881965 | |
| 13:29:15 | kashyap | dansmith: Yeah, it looks like "somehow" the loading of the kernel modules itself is failing: "failed loading these modules: nls_ascii nls_iso8859-1 nls_utf8 ip_tables ahci" | |
| 13:30:21 | kashyap | Also, I hate to say this, but I wonder if it's CirrOS-specific | |
| 13:30:57 | kashyap | dansmith: I take it that this segfault is consistently reproducible | |
| 13:31:54 | dansmith | kashyap: I can't push a button and make it happen, but I see it fairly often | |
| 13:32:21 | kashyap | dansmith: Noted; I'm trying to wade through the logs to also find the QEMU command-line of this. And the host kernel version | |
| 13:32:28 | dansmith | kashyap: entirely possible that it's cirros related, but since this there is not much going on in userland and this is a kernel crash, I imagine it's not the actual fault of cirros | |
| 13:32:36 | kashyap | Have you got the instance name / UUID, per chance? | |
| 13:35:42 | dansmith | kashyap: if you look right above the top of the console dump you'll see requests tempest was making to watch the status and you should find the uuid there | |
| 13:35:59 | kashyap | Okido, this seems the UUID: e0dc4c98-95e2-4421-a6ba-ae07213914a7 | |
| 13:36:00 | kashyap | Thanks! | |
| 13:40:46 | dansmith | :) | |
| 13:53:25 | dansmith | bauzas: FYI, I've got a few good passing runs on the nova patch with new ceph on jammy, after *much* work on the ceph devstack plugin, cinder tempest plugin, and tempest itself | |
| 13:53:41 | dansmith | so I'm hoping we can start to land those things soon and flip that job over, with your blessing | |
| 13:53:44 | opendevreview | yatin proposed openstack/nova master: Add config option to configure TB cache size https://review.opendev.org/c/openstack/nova/+/868419 | |
| 13:53:52 | dansmith | but let me know if you want to examine results closely | |
| 14:03:03 | bauzas | dansmith: oh that's excellent news, I got distracted from following the series | |
| 14:03:22 | dansmith | gouthamr: I think I need to make the devtack-ceph-plugin dependent on the cinder test plugin, which is dependent on the tempest stack.. are you okay with that? and do you want me to do anything else to the devstack plugin before we merge? or just merge and clean it up after? | |
| 14:03:37 | dansmith | bauzas: not expecting you to follow it, just giving the boss the executive summary :) | |
| 14:03:59 | bauzas | dansmith: don't call me like this, man, I'm just a cat herder :) | |
| 14:04:05 | dansmith | heh | |
| 14:04:51 | bauzas | dansmith: so, what's the plan ? merging the tempest bits first, and then the ceph devstack plugin one ? | |
| 14:05:32 | dansmith | bauzas: well, I was just saying above that I think it needs to be devstack-plugin -> cinder tempest -> tempest (so they merge in reverse order of this) | |
| 14:06:02 | bauzas | sorry missed your line | |
| 14:06:12 | dansmith | and probably change my DNM nova patch to stop commenting out all the jobs to make sure they're all still happy, other than just the ceph one | |
| 14:06:19 | bauzas | yeah | |
| 14:08:43 | bauzas | dansmith: at least that's a good thing to tell in the nova meeting today :) | |
| 14:08:52 | dansmith | yeah | |
| 14:18:57 | sean-k-mooney | dansmith: by the way i pushed this yesterday https://review.opendev.org/c/openstack/nova/+/881912 the image it pulled does not have resize2fs installed | |
| 14:19:09 | sean-k-mooney | so that is why some of the tests failed | |
| 14:19:27 | dansmith | sean-k-mooney: because it can't resize itself to fill the root on boot? | |
| 14:20:08 | sean-k-mooney | yep | |
| 14:20:35 | sean-k-mooney | i can fix that pretty simply i need to look and see if any other packages are missing | |
| 14:21:04 | dansmith | ack, cool | |
| 14:27:40 | sean-k-mooney | in at least some of the case the inablity to resize the root fs prevent the ssh hostkeys from being genreated which prevented ssh access to the vm | |
| 14:28:26 | dansmith | it'll prevent other things too, like writing the timestamp files | |
| 14:28:34 | dansmith | (assuming it's really that constrained) | |
| 14:29:58 | sean-k-mooney | virtual size: 292 MiB (305659904 bytes) | |
| 14:30:00 | sean-k-mooney | disk size: 83 MiB | |
| 14:50:49 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 15:03:07 | bauzas | reminder : nova meeting in one hour-ish | |
| 15:04:38 | enriquetaso | thanks bauzas | |
| 15:16:52 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/868419 is almost ready to merge just one last issues to adress if people want to review | |
| 15:41:27 | bauzas | sean-k-mooney: I plan to review this feature by tomorrow | |
| 15:41:36 | bauzas | (the tb cache size one) | |
| 15:51:31 | opendevreview | Sylvain Bauza proposed openstack/nova master: Revert "Debug Nova APIs call failures" https://review.opendev.org/c/openstack/nova/+/882052 | |
| 15:56:02 | bauzas | dansmith: melwitt: you may appreciate a quick review of the logging leak ^ | |
| 15:56:49 | bauzas | (did it on purpose before the meeting) | |
| 16:00:02 | bauzas | #startmeeting nova | |
| 16:00:02 | opendevmeet | Meeting started Tue May 2 16:00:02 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:02 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:02 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:11 | bauzas | welcome everyone | |
| 16:00:19 | elodilles | o/ | |
| 16:00:25 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:00:42 | gmann | o/ | |
| 16:00:50 | gibi | o/ | |
| 16:00:54 | gibi | (on a spotty connection) | |
| 16:01:19 | dansmith | bauzas: got it | |
| 16:02:02 | bauzas | coolio, let's start | |
| 16:02:11 | bauzas | #topic Bugs (stuck/critical) | |
| 16:02:21 | bauzas | #info One Critical bug | |
| 16:02:30 | bauzas | #link https://bugs.launchpad.net/nova/+bug/2012993 | |
| 16:02:35 | bauzas | shall be quickly reverted | |
| 16:02:45 | bauzas | and I'll propose backports as soon as it lands | |
| 16:03:25 | bauzas | not sure we need to discuss on it by now | |
| 16:03:38 | bauzas | so we can move on, unless people wanna know more about it | |
| 16:04:07 | sean-k-mooney | i have +w'ed it so its on its way | |
| 16:04:21 | bauzas | yup, and as you wrote, two backports are planned | |
| 16:04:26 | bauzas | down to 2023.1 and Zed | |
| 16:04:45 | bauzas | anyway, moving on | |
| 16:04:56 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 20 new untriaged bugs (+3 since the last meeting) | |
| 16:05:16 | bauzas | artom: I assume you didn't had a lot of time for spinning around those bugs ? | |
| 16:05:29 | bauzas | again, no worries | |
| 16:05:32 | Uggla | o/ | |
| 16:05:53 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:06:08 | bauzas | Uggla: you're the next on the list, happy to axe some of the bugs ? | |
| 16:06:19 | artom | bauzas, trying to do a few now | |
| 16:06:28 | bauzas | cool, no rush | |
| 16:06:36 | Uggla | bauzas, yep ok for me. | |
| 16:06:43 | bauzas | thanbks | |
| 16:06:45 | artom | For https://bugs.launchpad.net/nova/+bug/2018172 I think we need kashyap to weigh in, 'virtio' may not be a thing in RHEL 9.1? | |
| 16:06:51 | bauzas | #info bug baton is being passed to Uggla | |
| 16:06:55 | bauzas | artom: looking | |
| 16:07:01 | dansmith | uh, what | |
| 16:07:03 | artom | Or maybe just close immediately and direct them to a RHEL... thing | |
| 16:07:07 | kashyap | artom: Wait; it's impossible "virtio" can't be a thing in RHEL9.1. /me looks | |
| 16:07:21 | dansmith | hah, yeah | |