| opendevreview | sean mooney proposed openstack/os-vif master: [WIP] set default qos policy https://review.opendev.org/c/openstack/os-vif/+/881751 | 01:12 |
|---|---|---|
| han-guangyu | hello, I created instance by ISO image. Installing system to a cinder volume. | 05:30 |
| han-guangyu | But after instance reboot， instance still enter the system installation interface | 05:30 |
| han-guangyu | I know I can detach the volume, create a new instance | 05:31 |
| han-guangyu | I would like to ask, is there a way to make this instance, after restarting, use the volume installed with the system as the startup item | 05:32 |
| han-guangyu | Similar to libvirt, after installing the operating system with iso, it will automatically restart and enter the system | 05:32 |
| han-guangyu | Or if I want to try to achieve this requirement through small changes in a private environment, any suggestions? | 05:33 |
| han-guangyu | thanks for any help | 05:33 |
| opendevreview | Merged openstack/nova master: Handle zero pinned CPU in a cell with mixed policy https://review.opendev.org/c/openstack/nova/+/862687 | 12:46 |
| sean-k-mooney | yey :) | 12:57 |
| opendevreview | Danylo Vodopianov proposed openstack/nova master: Napatech SmartNIC support https://review.opendev.org/c/openstack/nova/+/859577 | 13:35 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: wip: convert usage of neutronclient to openstacksdk https://review.opendev.org/c/openstack/nova/+/882714 | 13:44 |
| opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Added api for healthcheck https://review.opendev.org/c/openstack/nova/+/882718 | 13:51 |
| opendevreview | Merged openstack/nova stable/2023.1: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882313 | 14:00 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Check shares support https://review.opendev.org/c/openstack/nova/+/850499 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/852085 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | 14:36 |
| 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 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/854824 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | 14:36 |
| 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 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/860285 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/860288 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | 14:36 |
| opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | 14:36 |
| bauzas | reminder : nova meeting in 50 mins-ish | 15:09 |
| bauzas | here | 15:09 |
| *** dmellado1 is now known as dmellado | 15:25 | |
| opendevreview | Merged openstack/nova stable/2023.1: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882314 | 15:27 |
| * gibi finds a seat | 16:01 | |
| elodilles | ~o~ | 16:01 |
| bauzas | #startmeeting nova | 16:01 |
| opendevmeet | Meeting started Tue May 9 16:01:59 2023 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 | sorry I'm late but some urgency came today that people will hear tomorrow | 16:02 |
| * bauzas tries to not spoil it too earlty | 16:02 | |
| Uggla | o/ | 16:02 |
| Uggla | bauzas, you are teasing us. | 16:02 |
| elodilles | o/ | 16:02 |
| auniyal_ | o/ | 16:03 |
| gmann | o/ | 16:04 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:04 |
| bauzas | gibi: can you please run the meeting now I'll jump back as soon as I can | 16:04 |
| bauzas | ? | 16:04 |
| gibi | sure | 16:04 |
| gibi | gathering thoughts... | 16:05 |
| gibi | #topic Bugs (stuck/critical) | 16:05 |
| gibi | there is one critical bug https://bugs.launchpad.net/nova/+bug/2012993 | 16:06 |
| gibi | the fix is up https://review.opendev.org/c/openstack/nova/+/882052 | 16:06 |
| gibi | it has +A but we need to push it through the gate | 16:07 |
| bauzas | yeah | 16:07 |
| bauzas | it got a lot of ci failures, mostly on gate pipeline | 16:07 |
| gibi | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 15 new untriaged bugs (-3 since the last meeting) | 16:08 |
| gibi | thanks whoever had the baton this week | 16:08 |
| gibi | ahh I see, thanks Uggla | 16:09 |
| gibi | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:09 |
| gibi | Uggla: any bug worth to mention? | 16:09 |
| Uggla | yep | 16:09 |
| Uggla | just to say I looked at https://bugs.launchpad.net/nova/+bug/2016967 | 16:09 |
| Uggla | I discussed about it with Artom, I suggested to add tracing, so I will propose a little patch to do that, but I have not change the status of the bug. | 16:10 |
| Uggla | s/I/Artom | 16:10 |
| gibi | Uggla: ack good idea | 16:11 |
| Uggla | also I guess to _unshelve_server closure is useless, I will remove it and explain in another patch. | 16:12 |
| Uggla | that's all for me. | 16:12 |
| gibi | thanks Uggla | 16:13 |
| gibi | #info bug baton is being passed to sean-k-mooney | 16:13 |
| gibi | sean-k-mooney: could you take it? | 16:13 |
| sean-k-mooney | i should proably say no but sure ill try | 16:13 |
| gibi | thanks :) | 16:13 |
| gibi | is there any bug we need to discuss today? | 16:14 |
| sean-k-mooney | the os-vif one i am working on is almost ready for wider review but ill ping people after the meeting once im happy with it. noting else i wanted ot highlight | 16:15 |
| gibi | thanks | 16:16 |
| Uggla | just if we can merge the patches: https://review.opendev.org/c/openstack/nova/+/879705 (Don't provide MTU value in metadata service if DHCP is enabled) | 16:16 |
| sean-k-mooney | thatns the backport | 16:16 |
| Uggla | I think I have backport patches down to train. | 16:16 |
| gibi | ack I see them lined up in https://review.opendev.org/q/topic:bug%252F1899487 | 16:17 |
| sean-k-mooney | that failed on ceph multi store it seams | 16:17 |
| sean-k-mooney | but ya i agree it would be nice ot land that | 16:17 |
| Uggla | yep at least if we can merged some of them, not to forget. | 16:18 |
| sean-k-mooney | it still need ot merge on all the newer branches | 16:18 |
| gibi | yepp | 16:18 |
| sean-k-mooney | oh so the zed patch still | 16:18 |
| sean-k-mooney | has an open qustion form elodilles | 16:18 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/877500/1/releasenotes/notes/Do-not-send-mtu-value-in-metadata-for-networks-with-enabled-dhcp-641506f2a13b540f.yaml | 16:19 |
| Uggla | and if we can land this one also: https://review.opendev.org/c/openstack/nova/+/868088 | 16:19 |
| sean-k-mooney | we should proably do a bug review day | 16:20 |
| gibi | sean-k-mooney, elodilles: I think it is fine to merge it with a minor bump | 16:20 |
| elodilles | gibi: ack | 16:20 |
| sean-k-mooney | we have a stable one schduled soon right? | 16:20 |
| gibi | sean-k-mooney: I'm not sure we have bug review day scheduled | 16:21 |
| elodilles | actually if i'm not mistaken that day is today :) | 16:21 |
| gibi | I think we only scheduled spec review day | 16:21 |
| elodilles | https://releases.openstack.org/bobcat/schedule.html#b-nova-stable-review-day | 16:21 |
| elodilles | today :) | 16:21 |
| sean-k-mooney | https://releases.openstack.org/bobcat/schedule.html#b-nova-stable-review-day | 16:21 |
| gibi | hupps, I totally missed that | 16:22 |
| sean-k-mooney | i think that didnt happen | 16:22 |
| sean-k-mooney | should we try again next week | 16:22 |
| gibi | we should | 16:22 |
| elodilles | i reviewed some stable backport o:) | 16:22 |
| gibi | elodilles++ :) | 16:22 |
| sean-k-mooney | elodilles++ | 16:22 |
| elodilles | though i also had review duties with Bobcat-1 release patches :X | 16:23 |
| gibi | bauzas: could you make sure that we try the stable review day next week same day? | 16:23 |
| gibi | or elodilles ^^ | 16:23 |
| sean-k-mooney | other things were on my mind so i forgot about this until now | 16:23 |
| gibi | it was a total blackout from my side | 16:23 |
| gibi | anyhow | 16:23 |
| gibi | moving on | 16:24 |
| elodilles | i can ping the team here next Tuesday if that is what you mean gibi | 16:24 |
| elodilles | on IRC | 16:24 |
| gibi | elodilles: that is exactly what I mean :) | 16:24 |
| gibi | thanks | 16:24 |
| elodilles | \o/ | 16:24 |
| gibi | #topic Gate status | 16:24 |
| gibi | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:24 |
| gibi | I'm not up to date with the gate status | 16:25 |
| gibi | but I saw https://bugs.launchpad.net/nova/+bug/1946339 happening again on couple of my patches | 16:25 |
| gibi | so I added fresh links there | 16:25 |
| gibi | *fresh job links | 16:25 |
| gibi | how others feel about the gate? | 16:25 |
| gibi | then I guess everything is acceptable | 16:27 |
| gibi | :) | 16:27 |
| gibi | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement | 16:27 |
| gibi | periodics are green | 16:27 |
| gibi | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:27 |
| gibi | #info STOP DOING BLIND RECHECKS aka. 'recheck' https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures | 16:28 |
| gibi | anything else about our gate? | 16:28 |
| gibi | then moving on | 16:29 |
| gibi | #topic Release Planning | 16:29 |
| gibi | #link https://releases.openstack.org/bobcat/schedule.html | 16:29 |
| gibi | #info Nova deadlines are set in the above schedule | 16:29 |
| gibi | #info Bobcat-1 is today | 16:29 |
| elodilles | (* this week, rather) | 16:29 |
| gibi | elodilles: do you need any help from us to make B-1 happen? | 16:29 |
| gibi | elodilles: thanks, correct | 16:29 |
| elodilles | gibi: i haven't seen any nova related patch | 16:30 |
| gibi | is it a good or a bad sign? :) | 16:30 |
| elodilles | that means we don't have any merged (and unreleased) patches | 16:30 |
| elodilles | in our libraries / clients | 16:30 |
| gibi | sounds good to me | 16:31 |
| gibi | #info Next Tuesday 16th is stable branches review day | 16:31 |
| elodilles | if anyone aware of sg that needs to be released, then this is the time, though :) | 16:31 |
| elodilles | gibi: thx! \o/ | 16:31 |
| gibi | elodilles: thanks | 16:31 |
| elodilles | ++ | 16:31 |
| gibi | hm either somebody updated the agenda while we were talking about the stable review day or it was already delayed | 16:32 |
| sean-k-mooney | elodilles: once the os-vif bug is fixed we will want to do a release pretty quickily i think but it does not need to be aligned to any milestone | 16:32 |
| gibi | sean-k-mooney: good point | 16:32 |
| elodilles | sean-k-mooney: ACK, thanks for the heads up! | 16:32 |
| elodilles | ping me and i can review it | 16:33 |
| sean-k-mooney | its release wiht intermedirary so ill just submit the patch after the issue is fixed | 16:33 |
| sean-k-mooney | ack | 16:33 |
| gibi | cool, any other topic about our releases? | 16:34 |
| gibi | then | 16:34 |
| gibi | #topic Review priorities | 16:34 |
| gibi | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | 16:34 |
| gibi | #info As a reminder, cores eager to review changes can +1 to indicate their interest, +2 for committing to the review | 16:35 |
| gibi | just the usual :) | 16:35 |
| gibi | any comments? | 16:35 |
| gibi | then I'm passing the keyboard to elodilles | 16:36 |
| gibi | #topic Stable Branches | 16:36 |
| elodilles | nothing to report, really, | 16:36 |
| elodilles | so the usual: | 16:36 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:36 |
| elodilles | and i'll ping the team next week for the stable review day :) | 16:37 |
| gibi | awesome | 16:37 |
| elodilles | that's all from me | 16:37 |
| gibi | thank you | 16:37 |
| elodilles | ++ | 16:37 |
| gibi | #topic Open discussion | 16:37 |
| gibi | nothing on the agenda | 16:37 |
| gibi | does anybody has anything to talk about? | 16:38 |
| auniyal_ | this patch https://review.opendev.org/c/openstack/nova/+/857339 | 16:38 |
| auniyal_ | I was trying to replace hasattr with obj_attr_is_set as per comments but few existing test cases are failing | 16:39 |
| auniyal_ | because instance.new_flavor is getting set as None | 16:39 |
| auniyal_ | I have added a detailed comments in patch | 16:40 |
| auniyal_ | sean-k-mooney, suggested earlier need to change in test cases, but I am not sure what | 16:41 |
| gibi | you need to do if instance.new_flavor and instance.new_flavor.obj_attr_is_set | 16:41 |
| gibi | so basically check if instance.new_falvor is not None | 16:41 |
| gibi | before callin obj_attr_is_set on it | 16:42 |
| gibi | >>> hasattr(None, "foo") | 16:42 |
| gibi | False | 16:42 |
| gibi | hashattr handles None internally | 16:42 |
| auniyal_ | but then we won't be running these TC's, is it okay | 16:42 |
| gibi | I only checked this from hashattr vs obj_attr_is_set perspective | 16:42 |
| gibi | with hasattr you also skipped these test if new_flavor was None | 16:44 |
| auniyal_ | yes | 16:44 |
| gibi | based on the review many people has more context on this change than me so I will defer to them | 16:45 |
| gibi | but they seems not to be present right now. So I suggest to ping them | 16:45 |
| auniyal_ | actually later I understood that, when I removed the hasattr I was skipping these test | 16:46 |
| auniyal_ | sean-k-mooney, :) | 16:46 |
| gibi | OK. Anything else before we close the meeting? | 16:48 |
| auniyal_ | no, nothing from side thanks gibi | 16:48 |
| gibi | #endmeeting | 16:49 |
| gibi | bauzas: we need you to end the meeting | 16:49 |
| gibi | the bot refuse to listen to me | 16:49 |
| bauzas | gibi: shit, forgot you to add as a chair | 16:49 |
| bauzas | also, about the stable branch review day, yeah let's punt that to next week | 16:50 |
| gibi | ack | 16:50 |
| bauzas | anyway, thanks a lot gibi | 16:50 |
| bauzas | thanks all | 16:50 |
| gibi | bauzas: no worries | 16:51 |
| bauzas | #endmeeting | 16:51 |
| opendevmeet | Meeting ended Tue May 9 16:51:03 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:51 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-09-16.01.html | 16:51 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-09-16.01.txt | 16:51 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-09-16.01.log.html | 16:51 |
| gibi | thanks folks | 16:51 |
| * gibi leaves the room in a hurry | 16:51 | |
| elodilles | thanks o/ | 16:52 |
| opendevreview | sean mooney proposed openstack/os-vif master: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/881751 | 18:32 |
| opendevreview | sean mooney proposed openstack/os-vif master: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/881751 | 18:55 |
| opendevreview | sean mooney proposed openstack/os-vif master: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/881751 | 19:02 |
| bauzas | dansmith: now that we merged a lot of volume-related ssh stuff, I guess I only need to recheck https://review.opendev.org/c/openstack/nova/+/882052 ? or shall I rebase it from master ? | 19:49 |
| bauzas | after a week, we're still unable to land this critical bugfix | 19:49 |
| dansmith | did that fail on volume things? | 19:49 |
| bauzas | yes, https://878773ae5f0379c9d548-8605f807f0b1cdc197533279c15edd0d.ssl.cf1.rackcdn.com/882052/1/check/tempest-integrated-compute-enforce-scope-new-defaults/2d80ba4/testr_results.html | 19:50 |
| bauzas | (at least one test, not the second one which is a panicking guest) | 19:50 |
| dansmith | bauzas: that failed with a kernel panic | 19:50 |
| dansmith | unrelated to the things I've been working on fixing, but related to the thing I asked kashyap to try to chase down | 19:51 |
| dansmith | but cirros 0.5.2's kernel being unsupported now, we don't really have any recourse | 19:51 |
| bauzas | dansmith: but the first test failed on teardown after a timeout | 19:51 |
| dansmith | because the guest was totally hung | 19:52 |
| bauzas | oh, shit | 19:53 |
| dansmith | bauzas: only one test failed, the first row is the teardown for the test that failed | 19:53 |
| bauzas | missed that the first test failing is actually a teardownclass | 19:53 |
| bauzas | yeah that | 19:53 |
| dansmith | yeah | 19:54 |
| bauzas | roger and do we have the cirros bug report ? | 19:54 |
| bauzas | nvm, found it https://bugs.launchpad.net/nova/+bug/1939108 | 19:56 |
| dansmith | actually I have a newer one | 19:58 |
| dansmith | although my kernel crash looks different than the one you have here | 19:58 |
| dansmith | yours looks like that one in the above bug indeed | 19:59 |
| bauzas | yup | 19:59 |
| dansmith | mine is quite different, but we see it a fair bit too | 19:59 |
| dansmith | https://bugs.launchpad.net/nova/+bug/2018612\ | 19:59 |
| dansmith | https://bugs.launchpad.net/nova/+bug/2018612 | 19:59 |
| bauzas | I see | 20:00 |
| bauzas | different problem indeed | 20:00 |
| bauzas | kernel taintaing | 20:00 |
| bauzas | tainting* | 20:00 |
| dansmith | yours looks like crash on startup, mine is on attach, I think | 20:00 |
| bauzas | yes | 20:00 |
| bauzas | the acpi boot fails | 20:00 |
| bauzas | we also have a problem with udhcpd leases, related to cirros too | 20:01 |
| dansmith | yeaj | 20:01 |
| opendevreview | sean mooney proposed openstack/os-vif master: remove focal based jobs https://review.opendev.org/c/openstack/os-vif/+/882755 | 20:41 |
| opendevreview | Merged openstack/nova master: Revert "Debug Nova APIs call failures" https://review.opendev.org/c/openstack/nova/+/882052 | 23:54 |
| dansmith | last time's a charm ^ | 23:54 |
| melwitt | lol finally | 23:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!