| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-05-07 | |||
| 17:28:37 | sean-k-mooney | and that does it via the image properties | |
| 17:29:31 | sean-k-mooney | dansmith: with that said your partly adressing that now | |
| 17:32:21 | opendevreview | Dan Smith proposed openstack/nova master: Fix device_type=lun with boot_index https://review.opendev.org/c/openstack/nova/+/918470 | |
| 17:32:28 | dansmith | sean-k-mooney: this fixes the other problem ^ with BFV | |
| 17:32:44 | dansmith | I have a tempest change to test BFV with multi-attach type=lun, which seems to work fine | |
| 17:32:52 | dansmith | I'll get them all re-swizzled after this run finishes | |
| 17:33:30 | sean-k-mooney | sure no rush ah you had to add lun as a possible boot dev | |
| 17:33:43 | sean-k-mooney | i wonder if this was previously only "supported" for data voluems | |
| 17:33:48 | sean-k-mooney | based on that list | |
| 17:33:58 | dansmith | just never tried, I imagine | |
| 17:34:26 | dansmith | it would be easy to say "don't do that for root" but I'm sure someone will want it and the kinds of pets that want this sort of thing are likely to be in that boat | |
| 17:34:54 | sean-k-mooney | im not sure if there is any disadvantage ot having root on a lun | |
| 17:34:56 | dansmith | I think there are some potential performance improvements with this for root if you have a very static pet.. I was reading that it's common to do this on vmware | |
| 17:35:04 | dansmith | maybe something related to the OS knowing its root is basically an iscsi device | |
| 17:35:08 | sean-k-mooney | i mean you proably dont need the extra locking apis | |
| 17:35:18 | dansmith | right, not for locking, but for perf | |
| 17:35:24 | sean-k-mooney | and we dont supprot multiatcch BFV right | |
| 17:35:35 | dansmith | we do actually. it's already tested even | |
| 17:35:37 | dansmith | in tempest | |
| 17:35:48 | dansmith | so I'm doing this as multi-attach BFV just to make sure it's covered that way | |
| 17:35:49 | sean-k-mooney | oh well in that case you may want the locking apis | |
| 17:36:11 | dansmith | presumably for hot spare type things I guess | |
| 17:36:34 | sean-k-mooney | ya so we cant do the vmware "keep the memroy in sync" type of failover | |
| 17:36:48 | sean-k-mooney | but if your application can do that or the bits it needs | |
| 17:37:00 | sean-k-mooney | then i can see that use case kind of | |
| 17:37:26 | dansmith | or you just want another instance already configured potentially hanging at the bootloader screen ready to boot, recover journal and be the master | |
| 17:37:31 | sean-k-mooney | it would be interesting to here form anyone who has used BFV multi attach | |
| 17:38:30 | sean-k-mooney | ... using grub as a active/passive lock sound like a nightmare to debug | |
| 17:38:57 | sean-k-mooney | but i could see that | |
| 17:39:40 | opendevreview | Vasyl Saienko proposed openstack/nova master: [ironic] Fix rebooting instance https://review.opendev.org/c/openstack/nova/+/918195 | |
| 17:39:43 | sean-k-mooney | that feels like "hay this is how we did it in my enteriprise virt solution, make it work the same in cloud" | |
| 17:41:49 | dansmith | yep, that's what I mean by "these sorts of pets" :) | |
| 17:43:10 | dansmith | Because of the boot order thing, I opened https://bugs.launchpad.net/nova/+bug/2065084 for the combined "device_type=lun is broken" issue and will tag both fixes to it | |
| 18:19:11 | opendevreview | Dan Smith proposed openstack/nova master: Avoid setting serial on raw LUN devices https://review.opendev.org/c/openstack/nova/+/918089 | |
| 18:19:12 | opendevreview | Dan Smith proposed openstack/nova master: Fix device_type=lun with boot_index https://review.opendev.org/c/openstack/nova/+/918470 | |
| 18:19:12 | opendevreview | Dan Smith proposed openstack/nova master: Enable virtio-scsi in nova-next https://review.opendev.org/c/openstack/nova/+/918458 | |
| 18:49:29 | opendevreview | Dan Smith proposed openstack/nova master: Enable virtio-scsi in nova-next https://review.opendev.org/c/openstack/nova/+/918458 | |
| #openstack-nova - 2024-05-08 | |||
| 01:10:35 | opendevreview | Merged openstack/nova master: [ironic] Fix rebooting instance https://review.opendev.org/c/openstack/nova/+/918195 | |
| 05:14:25 | opendevreview | Vasyl Saienko proposed openstack/nova stable/2024.1: [ironic] Fix rebooting instance https://review.opendev.org/c/openstack/nova/+/918180 | |
| 09:33:22 | stephenfin | gibi: don't know if you've time but if you were able to get back to https://review.opendev.org/c/openstack/nova/+/821423 before this week is out I'd love to close it out | |
| 12:57:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject AZ changes during aggregate add / remove host https://review.opendev.org/c/openstack/nova/+/821423 | |
| 12:58:05 | gibi | stephenfin: replied and fixed ^^ | |
| 13:55:27 | opendevreview | Merged openstack/nova master: Make overcommit check for pinned instance pagesize aware https://review.opendev.org/c/openstack/nova/+/631053 | |
| 15:04:06 | dansmith | gibi: could you look at this mostly trivial series to fix the direct lun attach? https://review.opendev.org/c/openstack/nova/+/918089 | |
| 15:04:36 | dansmith | turns out we can actually test this with LVM backend, which is cool, because that means we can also do this in upstream tempest | |
| 15:04:38 | gibi | lookoing... | |
| 15:05:18 | dansmith | sean-k-mooney: and I replied to your comment on the middle patch about the boot index | |
| 15:14:24 | opendevreview | Biser Milanov proposed openstack/nova master: Hardcode the use of iothreads for KVM. https://review.opendev.org/c/openstack/nova/+/918669 | |
| 16:29:02 | dansmith | gibi: thanks | |
| 16:32:34 | gibi | dansmith: it was simple indeed | |
| 17:57:50 | opendevreview | Dan Smith proposed openstack/nova master: Upload glance image with --file in ceph job https://review.opendev.org/c/openstack/nova/+/918689 | |
| 18:36:12 | opendevreview | Merged openstack/nova master: Avoid setting serial on raw LUN devices https://review.opendev.org/c/openstack/nova/+/918089 | |
| #openstack-nova - 2024-05-09 | |||
| 14:01:24 | WJeffs7508790194 | Hey, Random question for PCI passhthrough I have 2 models of the same GPUs with different product IDs, can I repeat the same alias again so they will both have the same name? or do they have to unique? thanks | |
| 14:51:37 | sean-k-mooney | WJeffs7508790194: you cant have a single alais refer to two diffent product ids | |
| 14:52:14 | WJeffs7508790194 | sean-k-mooney: Thanks for the clarification, I thought that might have been the case. | |
| 17:32:29 | opendevreview | Merged openstack/nova master: Fix device_type=lun with boot_index https://review.opendev.org/c/openstack/nova/+/918470 | |
| 18:50:10 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/osc-placement/+/904612 | |
| 19:30:38 | opendevreview | Dan Smith proposed openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets https://review.opendev.org/c/openstack/nova/+/913744 | |
| 19:31:00 | opendevreview | sean mooney proposed openstack/nova master: enable numa live migration in the ceph job https://review.opendev.org/c/openstack/nova/+/913842 | |
| 19:47:29 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Update testing of python versions https://review.opendev.org/c/openstack/osc-placement/+/904612 | |
| 19:58:01 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Update testing of python versions https://review.opendev.org/c/openstack/osc-placement/+/904612 | |
| 20:17:33 | opendevreview | Merged openstack/nova master: Reject AZ changes during aggregate add / remove host https://review.opendev.org/c/openstack/nova/+/821423 | |
| #openstack-nova - 2024-05-10 | |||
| 01:19:03 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: DNM: test dependency on devstack-plugin-ceph changes https://review.opendev.org/c/openstack/nova/+/918957 | |
| 06:13:59 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix KeyError on assisted snapshot call https://review.opendev.org/c/openstack/nova/+/900783 | |
| 09:36:39 | opendevreview | sean mooney proposed openstack/nova stable/2024.1: Reject AZ changes during aggregate add / remove host https://review.opendev.org/c/openstack/nova/+/918650 | |
| 09:37:21 | sean-k-mooney | stephenfin: gibi: ^ now that thats is merged ill start the cherry pick process | |
| 09:38:58 | sean-k-mooney | the functional repoducer was merged in yoga so we only need the actual fix patch | |
| 10:41:46 | gibi | sean-k-mooney: thanks. Ping me and I can review the backports | |
| 10:43:31 | sean-k-mooney | the first one is up but zuul is still runnig | |
| 12:06:16 | opendevreview | sean mooney proposed openstack/nova master: tweak emulation job to avoid OOM errors https://review.opendev.org/c/openstack/nova/+/918464 | |
| 12:06:17 | opendevreview | sean mooney proposed openstack/nova master: [DNM] test nova-emulation https://review.opendev.org/c/openstack/nova/+/918465 | |
| 12:07:17 | opendevreview | sean mooney proposed openstack/nova master: tweak emulation job to avoid OOM errors https://review.opendev.org/c/openstack/nova/+/918464 | |
| 12:07:18 | opendevreview | sean mooney proposed openstack/nova master: [DNM] test nova-emulation https://review.opendev.org/c/openstack/nova/+/918465 | |
| 12:12:20 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 12:12:20 | opendevreview | ribaudr proposed openstack/nova master: Amend DB model add a unique constraint. https://review.opendev.org/c/openstack/nova/+/912518 | |
| 12:12:21 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 12:12:26 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 12:12:27 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 12:12:28 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 12:12:31 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 12:12:33 | 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 | |
| 12:12:35 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 12:12:40 | 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 | |
| 12:12:40 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 12:12:41 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 12:12:42 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 12:12:46 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 12:12:50 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 12:12:53 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 12:12:54 | opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | |
| 12:13:01 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 12:13:02 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 12:13:03 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 12:13:03 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 12:13:18 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 12:13:26 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 12:13:26 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 12:13:27 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 12:13:27 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 12:13:28 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |