| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-02-27 | |||
| 16:33:37 | bauzas | kgube: do you have a follow-up patch for Tempest about it ? | |
| 16:33:47 | bauzas | like another attribute | |
| 16:34:50 | kgube | so the only job that will be testing the new volume action is the devstack-plugin-nfs one | |
| 16:35:43 | kgube | I have a patch for this, but it also needs changing once the feature is merged | |
| 16:35:47 | kgube | https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/896196 | |
| 16:35:59 | kgube | so it's currently marked as WIP | |
| 16:36:50 | bauzas | okay, then we need to have a consensus for accepting your feature without Tempest scenarios | |
| 16:37:07 | bauzas | what people think about ^ ? | |
| 16:37:34 | bauzas | as kgube said, we only know it doesn't trample the other usages | |
| 16:39:24 | kgube | well, the second change I posted has successful tempest runs for nfs online extend, with all of the changes merged in | |
| 16:39:57 | kgube | it just isn't part of the nova tempest jobs | |
| 16:40:44 | bauzas | kgube: I'll ping other cores async | |
| 16:40:52 | bauzas | to see what they think | |
| 16:41:04 | kgube | thank you! | |
| 16:41:32 | bauzas | any other things before we close ? | |
| 16:42:34 | bauzas | looks not | |
| 16:42:36 | bauzas | thanks all | |
| 16:42:38 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-02-27-16.00.log.html | |
| 16:42:38 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-02-27-16.00.txt | |
| 16:42:38 | bauzas | #endmeeting | |
| 16:42:38 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-02-27-16.00.html | |
| 16:42:38 | opendevmeet | Meeting ended Tue Feb 27 16:42:38 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:47:55 | sean-k-mooney | bauzas: the tempest test is not the issue we cant merge the patch without the client bump | |
| 16:48:06 | sean-k-mooney | so we should ask for an early cinder clinent release | |
| 16:48:13 | sean-k-mooney | and if they need to do another one they can | |
| 16:48:17 | sean-k-mooney | at ff | |
| 16:48:55 | sean-k-mooney | the release process is fully automated out sied of the patch review so that should be quick and simple to do | |
| 16:48:57 | bauzas | well, the depends-on is a cinderclient patch | |
| 16:49:11 | sean-k-mooney | right but the nova patch need a min version bump | |
| 16:49:13 | bauzas | once the patch is merged, then Zuul can accept | |
| 16:49:30 | sean-k-mooney | no because our unit/fucntional test will fail | |
| 16:49:32 | bauzas | sean-k-mooney: correct, we should target the required need | |
| 16:50:01 | sean-k-mooney | well it depend on that the client change is doing | |
| 16:50:14 | sean-k-mooney | depends on is not used for the tox changes | |
| 16:50:26 | sean-k-mooney | so those will fial if we are usign a new api form it but we are probaly mocking it | |
| 16:50:32 | sean-k-mooney | so maybe they wont fial | |
| 16:51:24 | bauzas | sean-k-mooney: my point is that nothing blocks the nova change to be merged | |
| 16:51:53 | bauzas | because I guess it's only additive opt-in changes | |
| 16:53:39 | bauzas | paperwork-ish, we should ask for an reqs.txt update | |
| 16:53:59 | bauzas | anyway, I need to leave now b/c of a physical local meetup in Grenoble | |
| 16:56:47 | sean-k-mooney | bauzas: my point is it really shoudl be blocked | |
| 16:56:53 | sean-k-mooney | bauzas: that not how we handel depencies | |
| 16:57:05 | sean-k-mooney | the path that need the new feature does the requirements bump | |
| 16:57:53 | bauzas | sean-k-mooney: please then add a comment in the nova patch | |
| 17:31:53 | sean-k-mooney | i did thats why i -1 it | |
| 17:32:43 | sean-k-mooney | bauzas: on the good news front https://zuul.opendev.org/t/openstack/build/2a3f86e4974b4a4a9e875477c77f81e0/logs | |
| 17:32:52 | sean-k-mooney | bauzas: that fialed one test that shoudl not work in that job config | |
| 17:33:34 | sean-k-mooney | bauzas: i have confirmed that we have 2 compute nodes and each is only reporting 3 ironic nodes | |
| 17:33:47 | sean-k-mooney | each compute is exposing diffent shards | |
| 17:34:16 | sean-k-mooney | adn we can boot nova instnace across either shard | |
| 17:35:23 | sean-k-mooney | bauzas: i was just chatting with JayF and im goign to rework this to be mergable and start wroking on a simple post hook to validate that each compute service is only reportin the ironic nodes we expect | |
| 17:36:16 | sean-k-mooney | long term we may be able to simplfy the setup too but i think this give nova the coverage we ant for the shard work. ironic will seperaly be testing the api in there jobs too | |
| 17:54:20 | JayF | that is already merged and has been running for weeks now (ironic api test job) | |
| 17:54:38 | JayF | https://opendev.org/openstack/ironic-tempest-plugin/commit/f2b9c74c9cb8459a979d9002aae3c1a41737c77a | |
| 21:36:45 | opendevreview | sean mooney proposed openstack/nova master: add multinode ironic shard job https://review.opendev.org/c/openstack/nova/+/910333 | |
| 21:49:53 | sean-k-mooney | bauzas: https://review.opendev.org/c/openstack/nova/+/910333 is enabling the ironic job in experimetal and perodic weekly | |
| 21:50:43 | sean-k-mooney | assuming that passes i think that woudl be enough with the exiting ironic testing on there side to progress and we can spend some time adding better coverage post FF and early next cycle | |
| 21:52:17 | sean-k-mooney | the coverage in the job is minimal as it stands today but i have partsed the logs myself and convicned my self that its actully sharding properly | |
| 21:52:39 | sean-k-mooney | see https://review.opendev.org/c/openstack/nova/+/910333/4/.zuul.yaml#974 | |
| 21:52:59 | sean-k-mooney | i am not conviced this job woudl cache if that broken on teh ironic side | |
| 21:53:25 | sean-k-mooney | but we can evolve it to do that | |
| 22:04:30 | JayF | We also have explicit API testing that will make loud noises if we ever break shard querying in our API... | |
| #openstack-nova - 2024-02-28 | |||
| 00:41:46 | opendevreview | Merged openstack/nova master: block_device: Add encryption attributes to swap disks https://review.opendev.org/c/openstack/nova/+/884312 | |
| 02:18:02 | opendevreview | Takashi Kajinami proposed openstack/nova master: Override new KeyManager interfaces https://review.opendev.org/c/openstack/nova/+/910459 | |
| 06:18:10 | opendevreview | Merged openstack/nova master: libvirt: Support maxphysaddr. https://review.opendev.org/c/openstack/nova/+/907516 | |
| 06:26:51 | opendevreview | Steven Blatzheim proposed openstack/nova master: fixed nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/910466 | |
| 07:00:09 | opendevreview | Takashi Kajinami proposed openstack/nova master: Override new KeyManager interfaces https://review.opendev.org/c/openstack/nova/+/910459 | |
| 07:10:49 | tkajinam | o/ I'd like to ask for some attention to https://review.opendev.org/c/openstack/nova/+/896100 to unblock u-c update to pull in the new castellan release for Caracal. | |
| 07:13:53 | tkajinam | (thanks stephenfin for that patch, by the way | |
| 08:13:35 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing requested az in output https://review.opendev.org/c/openstack/nova/+/904568 | |
| 08:22:55 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 08:22:55 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 08:22:59 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 08:23:00 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 08:23:00 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 08:23:02 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 08:23:04 | 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 | |
| 08:23:05 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 08:23:06 | 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 | |
| 08:23:07 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 08:23:07 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 08:23:08 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 08:23:09 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 08:23:10 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 08:23:11 | opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | |
| 08:23:11 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 08:23:12 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 08:23:12 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | |
| 08:23:13 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (API) https://review.opendev.org/c/openstack/nova/+/850499 | |
| 08:23:14 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 08:23:14 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 08:23:16 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 08:23:16 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 08:23:18 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 08:23:18 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 08:23:20 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 08:23:20 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 08:23:22 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 08:25:04 | Uggla | gibi, bauzas , new update of virtiofs ^ please note that I have not fully tested it yet with devstack. I'm gonna do it now. The doc may need a review as well. However this is a new round to show you the new code. | |
| 08:36:03 | stblatzheim | Can someone have a look at https://bugs.launchpad.net/nova/+bug/2055245 ? I already posted a suitable solution for this behaviour. | |