| opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | 06:44 |
|---|---|---|
| opendevreview | ribaudr proposed openstack/nova master: Amend DB model add a unique constraint. https://review.opendev.org/c/openstack/nova/+/912518 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Use client token when talking to manila https://review.opendev.org/c/openstack/nova/+/925277 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares https://review.opendev.org/c/openstack/nova/+/860287 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Reports instance events to the DB regarding attaching and detaching a share https://review.opendev.org/c/openstack/nova/+/927088 | 06:44 |
| opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | 06:45 |
| opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | 06:45 |
| opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | 06:45 |
| *** bauzas_ is now known as bauzas | 07:28 | |
| *** __ministry is now known as Guest3703 | 07:36 | |
| zigo | Hi there! When should I expect the RC1 for Nova? | 10:29 |
| zigo | What's blocking? | 10:29 |
| opendevreview | Merged openstack/placement stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/placement/+/929290 | 10:41 |
| opendevreview | Merged openstack/placement stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/placement/+/929291 | 10:41 |
| stephenfin | zigo: https://review.opendev.org/c/openstack/requirements/+/929504 | 10:51 |
| tosky | uhm, is there a bug which tracks the failure of nova-live-migration? | 11:06 |
| zigo | stephenfin: FYI, I have all packages ready for Dalmatian but Nova, and I'm not affected by the issue with openstackclient (since 7.1.1 is there it's going to be used). | 11:15 |
| zigo | Maybe I should package tip of master? Or is this already in a stable branch? | 11:15 |
| stephenfin | it's already in a stable branch | 11:15 |
| zigo | Ok, thanks. | 11:15 |
| zigo | I'll do an ~rc1~git.<something> then. | 11:16 |
| zigo | Not correct, I can't see a stable/2024.2 branch at https://opendev.org/openstack/nova | 11:17 |
| stephenfin | I'm talking about OSC | 11:18 |
| zigo | Oh... :P | 11:18 |
| zigo | So nova is still on master, then I guess. | 11:18 |
| gibi | just to set expectations after https://review.opendev.org/c/openstack/requirements/+/929504 lands we have at least 3 regression bug fixes lined up to land before we can cut RC1. | 11:36 |
| elodilles | gibi or bauzas or other cores o:) when you have time, could you please do a quick review on these reno patches? https://review.opendev.org/c/openstack/python-novaclient/+/928366 and https://review.opendev.org/c/openstack/placement/+/929292 | 11:46 |
| gibi | on it | 11:52 |
| gibi | I lost, stephenfin was faster :) | 11:53 |
| stephenfin | 🥷 | 11:53 |
| gibi | :) | 11:53 |
| opendevreview | Merged openstack/python-novaclient master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/python-novaclient/+/928366 | 12:00 |
| opendevreview | Merged openstack/placement master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/placement/+/929292 | 12:01 |
| elodilles | thx :] | 12:03 |
| *** __ministry is now known as Guest3730 | 13:21 | |
| zigo | https://paste.opendev.org/show/bSMhXjOBcgfTTiNF0s1I/ <--- Is this a new behavior expected in Dalmatian ? | 14:40 |
| zigo | If I provide --domain it works ... | 14:40 |
| sean-k-mooney | i dont know but the keyston folks might. i doubt it woudl hav echanged but its possible | 14:53 |
| frickler | zigo: sean-k-mooney: that's a bug in OSC, see https://review.opendev.org/c/openstack/python-openstackclient/+/929456 | 14:59 |
| zigo | Yeah, found the patch. | 14:59 |
| zigo | Thanks. | 14:59 |
| zigo | I'm already building OSC with it. | 14:59 |
| sean-k-mooney | frickler: so we would need another osc release for that right | 15:00 |
| sean-k-mooney | 7.1.2 | 15:00 |
| zigo | Agreed. | 15:00 |
| frickler | I've been wondering whether the bug is critical enough to warrant another OSC release right away or whether it could wait until 2024.2 is out, but likely you're right | 15:05 |
| dansmith | surely that's done in a devstack setup right? | 15:07 |
| dansmith | how are the jobs supposedly testing osc not catching that? | 15:07 |
| sean-k-mooney | it might always pass the domain | 15:07 |
| dansmith | ah yeah I guess it must | 15:08 |
| dansmith | probably because it's possible to set the domain in devstack I guess | 15:08 |
| zigo | frickler: IMO, it is. It broke my puppet provider for creating users... | 15:09 |
| dansmith | zigo: yeah agree, it would certainly have broken some of my own scripts that test glance-remote | 15:10 |
| sean-k-mooney | dansmith: https://github.com/openstack/devstack/blob/0ff627286297a3957143577412884dc50ff8a57a/functions-common#L902-L921 | 15:16 |
| sean-k-mooney | so ya we alwasy pass a domain | 15:16 |
| dansmith | yeah | 15:16 |
| frickler | yes, devstack always is explicit in specifying the domain | 15:19 |
| sean-k-mooney | to be fair this should be tested in the devstack-functionatl job but there is obviously a testing gap | 15:20 |
| opendevreview | Masahito Muroi proposed openstack/nova master: Use boot_roles_count and boot_role_<count> key for system_metadata https://review.opendev.org/c/openstack/nova/+/925163 | 15:26 |
| elodilles | bauzas: fyi, this has just merged: 'update constraint for python-openstackclient to new release 7.1.1' https://review.opendev.org/c/openstack/requirements/+/929504 | 15:57 |
| bauzas | I saw | 16:00 |
| bauzas | I'm late for the meeting, let me run it | 16:00 |
| bauzas | #startmeeting | 16:01 |
| opendevmeet | bauzas: Error: A meeting name is required, e.g., '#startmeeting Marketing Committee' | 16:01 |
| bauzas | #startmeeting nova | 16:01 |
| opendevmeet | Meeting started Tue Sep 17 16:01:19 2024 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
| opendevmeet | The meeting name has been set to 'nova' | 16:01 |
| bauzas | hey everyone | 16:01 |
| fwiesel | o/ | 16:01 |
| bauzas | the agenda isn't written yet, lemme do it | 16:01 |
| elodilles | o/ | 16:01 |
| dansmith | o/ | 16:01 |
| bauzas | okay, I'm done with the agenda | 16:02 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:02 |
| bauzas | #topic Bugs (stuck/critical) | 16:02 |
| bauzas | #info One critical bug | 16:03 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.importance=Critical&field.status=New&field.status=Incomplete&field.status=Confirmed&field.status=Triaged&field.status=In+Progress | 16:03 |
| bauzas | #link https://bugs.launchpad.net/nova/+bug/2079850 | 16:03 |
| gibi | o/ | 16:03 |
| bauzas | so we know about it, we just need to merge the bugfix | 16:03 |
| bauzas | but, we'll discuss why we weren't able to do it previously in a second | 16:04 |
| bauzas | anything else about bugs ? | 16:04 |
| bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:04 |
| bauzas | looks not | 16:05 |
| bauzas | #topic Gate status | 16:05 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:05 |
| bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:05 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:05 |
| bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:05 |
| bauzas | #info Please try to provide meaningful comment when you recheck | 16:05 |
| bauzas | so, you probably saw but the gate is blocked | 16:05 |
| Uggla | o/ | 16:06 |
| bauzas | #info Gate is currently blocked due to a bug in OpenStackClient 7.1.0 | 16:06 |
| gibi | bauzas: https://review.opendev.org/c/openstack/requirements/+/929504 is merged 7.1.1 is in uc now | 16:07 |
| bauzas | elodilles just told me that u-c now supports 7.1.1 where the bug is fixed (due to a parameter modification) | 16:07 |
| bauzas | gibi: yup, was writing it :) | 16:07 |
| bauzas | so, now, we can recheck, which I willk | 16:07 |
| gibi | I already did | 16:07 |
| bauzas | cool | 16:07 |
| gibi | rechecked the bottom of the 3 RC bug we track in the etherpad | 16:08 |
| bauzas | cool | 16:08 |
| bauzas | we need to recheck the prelude and another Dalmatian change | 16:08 |
| bauzas | but we'll discuss that in the next topic | 16:08 |
| gibi | the prelude is already merged :) | 16:09 |
| bauzas | #info Gate should now be unblocked as upper-constraints now supports OSC 7.1.1 which fixes the cause | 16:09 |
| bauzas | oh cool | 16:09 |
| * bauzas was in meetings for the last 2 hours | 16:09 | |
| bauzas | that's how you can be productive, right? | 16:09 |
| bauzas | gibi: thanks then | 16:10 |
| gibi | I think the dalmatian service version patch is still open, I rechecked it now | 16:10 |
| bauzas | ack | 16:10 |
| bauzas | I can basically now take a beer then :) | 16:10 |
| bauzas | 'hold my beer' literally | 16:10 |
| gibi | sure, but keep your fingres crossed :) | 16:10 |
| bauzas | my toes are even crossed | 16:11 |
| bauzas | we're super late and I feel bad about it | 16:11 |
| bauzas | but we shouldn't need another RC2 | 16:11 |
| bauzas | #topic Release Planning | 16:11 |
| bauzas | #link https://releases.openstack.org/dalmatian/schedule.html | 16:11 |
| gibi | so we just name this RC2 and say we have an early :) | 16:11 |
| bauzas | #info Dalmatian RC1 not yet tagged, hopefully tomorrow | 16:12 |
| bauzas | #link https://etherpad.opendev.org/p/nova-dalmatian-rc-potential Tracking etherpad for RC candidates | 16:12 |
| bauzas | okay, I think we discussed that enough, unless someone has any question ? | 16:12 |
| bauzas | #topic Review priorities | 16:13 |
| bauzas | #link https://etherpad.opendev.org/p/nova-2025.1-status | 16:14 |
| bauzas | please add the series you'd want us to look at | 16:14 |
| bauzas | I already see two proposals | 16:14 |
| bauzas | moving on | 16:15 |
| bauzas | #topic PTG planning | 16:15 |
| bauzas | #info as a reminder, we'll meet (virtually) at the PTG on Oct 21-25 2024 | 16:16 |
| bauzas | I'm starting to draft an etherpad where you could add your topics of interest | 16:16 |
| bauzas | #link https://etherpad.opendev.org/p/nova-2025.1-ptg | 16:16 |
| bauzas | I'll send an email to openstack-discuss soon to tell about it | 16:17 |
| bauzas | I think I'm done | 16:18 |
| bauzas | next topic | 16:18 |
| bauzas | #topic Stable Branches | 16:18 |
| bauzas | elodilles: floor is yours | 16:18 |
| elodilles | o7 | 16:18 |
| elodilles | #info stable/202*.* gates seem to be OK | 16:18 |
| elodilles | #info stable/2024.2 branch is not yet cut for nova, but all the other repos the new branch exist | 16:18 |
| elodilles | and that's all from me about stable branches for now | 16:18 |
| bauzas | cool | 16:19 |
| bauzas | last topic | 16:20 |
| bauzas | well | 16:20 |
| bauzas | #topic vmwareapi 3rd-party CI efforts Highlights | 16:20 |
| bauzas | fwiesel: anything to report ? | 16:20 |
| fwiesel | #info No updates | 16:20 |
| fwiesel | Not really... patch still stuck in ci... I'll recheck it soon | 16:21 |
| bauzas | cool | 16:21 |
| bauzas | last topic then | 16:21 |
| bauzas | #topic Open discussion | 16:21 |
| bauzas | (noonedeadpunk) Enable serving NoVNC under subpath (not only via port/subdomain) | 16:21 |
| bauzas | noonedeadpunk: around ? | 16:21 |
| bauzas | #link https://review.opendev.org/c/openstack/nova/+/928839 | 16:21 |
| sean-k-mooney | so its tecnnically possible ot do ^ externally with the correct revers proxy config but its a bit of a pain | 16:22 |
| sean-k-mooney | im generally supportive of this as a minor feature (specless blueprint) | 16:23 |
| sean-k-mooney | tl;dr | 16:23 |
| sean-k-mooney | if you want to deploy all services on port 443 including novnc | 16:23 |
| gibi | I only skimmed it, is this only support a single new query param "path"? | 16:23 |
| sean-k-mooney | (either beacuse of yoru isp or firewall issues) you can do it with subdomains but its tricky to do with subfolders | 16:24 |
| sean-k-mooney | yep path is the subpath to append ot the websocket | 16:24 |
| sean-k-mooney | so in there case they wanted to deply openstac.my.domain/vnc | 16:24 |
| bauzas | I'm having a few concerns by the implementation but I don't have any concenr about the design | 16:24 |
| sean-k-mooney | without setting path it will connect ot / | 16:25 |
| noonedeadpunk | yeah, around jsut a bit busy | 16:25 |
| bauzas | like, I don't want nova to just sub the path and see whether there is path | 16:25 |
| sean-k-mooney | so this is just exposing a way to pass path to novnc | 16:25 |
| bauzas | maybe a config option, maybe something like a relnote | 16:25 |
| noonedeadpunk | nova does not sub the path - but it jsut respects whatever user defiens in config | 16:26 |
| bauzas | but I don't want us to not expose that | 16:26 |
| noonedeadpunk | by not respecting query, and not just fqdn | 16:26 |
| bauzas | well your change will parse the qs | 16:26 |
| noonedeadpunk | and append to it | 16:26 |
| sean-k-mooney | so we brifly dicussed having a config option like the exisitng port option for the query sting | 16:27 |
| noonedeadpunk | vs currnet behaviour where query is hardcoded basically | 16:27 |
| sean-k-mooney | for libvirt we decieded that just passing a uri for the connection was better | 16:28 |
| bauzas | well, here you're hard parsing the query with only 'path' | 16:28 |
| noonedeadpunk | well, I'm not sure about config option ,as there could be more usecases whre you wnat to pass smth as admin to novnc | 16:28 |
| sean-k-mooney | so that why i was mostly ok with noonedeadpunk approch as its consitent with that | 16:28 |
| sean-k-mooney | bauzas: there may be other query arges that novnc uses tha tcould be reasoanable to pass | 16:28 |
| noonedeadpunk | bauzas: yeah, but current code will jsut add `?path=?token=<uuid>` | 16:29 |
| sean-k-mooney | so i woudl prefer not to limit it to path | 16:29 |
| bauzas | anyway, as I said, this is a technical detail | 16:29 |
| noonedeadpunk | so if you try to add like anything to the `novncproxy_base_url` - you will end up with invalid token | 16:29 |
| bauzas | I don't oppose to the usecase and I think keeping an URI is important | 16:29 |
| noonedeadpunk | probably I jsut don't get what you're proposing isntead... | 16:30 |
| bauzas | as I said, I only have implementation concerns | 16:30 |
| bauzas | your current patch only accepts 'path' as a qs param | 16:30 |
| noonedeadpunk | but like unit tests shows it should be quite consistent... I can add more test cases if I'll get what these concerns are | 16:30 |
| bauzas | if we restrict to 'path' then we need to document it better than "oh and now you can pass *any* qs" | 16:31 |
| sean-k-mooney | noonedeadpunk: bauzas is askign to make sure it will work with any arg | 16:31 |
| noonedeadpunk | it's not restricted with path? | 16:31 |
| noonedeadpunk | l133 of https://review.opendev.org/c/openstack/nova/+/928839/3/nova/tests/unit/objects/test_console_auth_token.py | 16:32 |
| bauzas | "@item(hotlist):button2 | 16:32 |
| bauzas | doh | 16:32 |
| bauzas | the documentation says "You can also supply extra request arguments" | 16:32 |
| bauzas | anyway, can we just (for the sake of my beer) accept that blueprint as specless and nitpick over the change as much as we want over the Epoxy timeframe ? :) | 16:33 |
| noonedeadpunk | ++ | 16:33 |
| sean-k-mooney | +1 form me also | 16:33 |
| bauzas | any objection ? | 16:34 |
| bauzas | looks not | 16:34 |
| bauzas | #agreed https://blueprints.launchpad.net/nova/+spec/novnc-base-url-respect-extra-params accepted as specless for the Epoxy release | 16:35 |
| bauzas | I'll file the paperwork | 16:35 |
| gibi | OK for me | 16:35 |
| bauzas | that's it for the agenda | 16:35 |
| bauzas | any other bit you'd like to raise ? | 16:35 |
| bauzas | looks not | 16:35 |
| bauzas | my beer won't heat, cool | 16:36 |
| bauzas | thanks all | 16:36 |
| bauzas | #endmeeting | 16:36 |
| opendevmeet | Meeting ended Tue Sep 17 16:36:13 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:36 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-09-17-16.01.html | 16:36 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-09-17-16.01.txt | 16:36 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-09-17-16.01.log.html | 16:36 |
| sean-k-mooney | o/ | 16:36 |
| elodilles | thanks o/ | 16:36 |
| gibi | o/ | 16:38 |
| *** bauzas_ is now known as bauzas | 19:28 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!