| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-02-27 | |||
| 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: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 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: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: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 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: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: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 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: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: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 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:12 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | |
| 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: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 helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 08:23:14 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 08:23:16 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 08:23:16 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 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:18 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 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:20 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 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. | |
| 09:15:08 | bauzas | Uggla: ack, and thanks all for the fish :) | |
| 09:20:07 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 09:20:07 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 09:20:08 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 09:20:08 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 09:20:09 | opendevreview | melanie witt proposed openstack/nova master: libvirt: make |
|
| 09:20:09 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 09:20:10 | opendevreview | melanie witt proposed openstack/nova master: Support (resize|cold migration) with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870933 | |
| 09:20:11 | opendevreview | melanie witt proposed openstack/nova master: Support live migration with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/905512 | |
| 09:20:11 | opendevreview | melanie witt proposed openstack/nova master: Support rebuild with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870939 | |
| 09:20:13 | opendevreview | melanie witt proposed openstack/nova master: Support rescue with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/873675 | |
| 09:20:13 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 09:20:15 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870937 | |
| 09:20:15 | opendevreview | melanie witt proposed openstack/nova master: Add backing_encryption_secret_uuid to BlockDeviceMapping https://review.opendev.org/c/openstack/nova/+/907960 | |
| 09:20:17 | opendevreview | melanie witt proposed openstack/nova master: Support encrypted backing files for qcow2 https://review.opendev.org/c/openstack/nova/+/907961 | |
| 09:20:17 | opendevreview | melanie witt proposed openstack/nova master: Support cross cell resize with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/909595 | |
| 09:20:19 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 09:20:19 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 09:20:21 | opendevreview | melanie witt proposed openstack/nova master: Report ephemeral disk encryption in the metadata API https://review.opendev.org/c/openstack/nova/+/909945 | |
| 09:20:21 | opendevreview | melanie witt proposed openstack/nova master: Deprecate legacy ephemeral storage encryption using dm-crypt https://review.opendev.org/c/openstack/nova/+/909947 | |
| 09:20:23 | opendevreview | melanie witt proposed openstack/nova master: Documentation for ephemeral encryption https://review.opendev.org/c/openstack/nova/+/910034 | |
| 09:26:45 | Uggla | bauzas, for the fish ? You lost me ? | |
| 09:28:52 | bauzas | Uggla: no, just a famous H2G2 sentence :) | |
| 09:29:13 | bauzas | so long and thanks all for the fish :) | |
| 09:30:05 | bauzas | thanks for *all* the fish (btw.) | |
| 09:30:23 | Uggla | Uggla, I was not on that page. :) | |
| 09:51:32 | opendevreview | Steven Blatzheim proposed openstack/nova master: Fix nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/910466 | |
| 09:52:28 | gibi | Uggla: ack, start looking... | |
| 09:52:37 | Uggla | gibi, thx | |
| 09:53:18 | opendevreview | Steven Blatzheim proposed openstack/nova master: Fix nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/910466 | |
| 09:59:36 | opendevreview | Steven Blatzheim proposed openstack/nova master: Fix nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/910466 | |
| 10:02:40 | Uggla | gibi, fyi https://review.opendev.org/c/openstack/openstacksdk/+/889519 is merged | |
| 10:02:58 | Uggla | so no need to cherry pick it | |
| 10:24:49 | opendevreview | Steven Blatzheim proposed openstack/nova master: fix indention https://review.opendev.org/c/openstack/nova/+/910479 | |
| 10:29:11 | opendevreview | Steven Blatzheim proposed openstack/nova master: Fix nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/910466 | |
| 11:21:44 | opendevreview | Sylvain Bauza proposed openstack/nova master: Modify the mdevs in the migrate XML https://review.opendev.org/c/openstack/nova/+/904258 | |
| 11:21:44 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: add a periodic check for migrations https://review.opendev.org/c/openstack/nova/+/909598 | |
| 11:36:38 | sean-k-mooney[m] | tkajinam: i had a look at the castellan pathch. i have +2’d it but form my perspective you missed the non-clinet lib freeze so im treating this as an expcetion because its small but i dont think its in line wiht our normal release process | |
| 11:38:26 | sean-k-mooney[m] | bauzas: i was holding off on comeitng on the docs change because i didnt want you to repsin it but since you have im going to ask you to correct some other issues | |
| 11:56:15 | frickler | sean-k-mooney[m]: the lib was released in time last week, it is just the reqs update that got stuck because of test failures. technically there's time for that until friday without an exception | |
| 11:57:10 | sean-k-mooney[m] | frickler that not how i managed this when i was the release leasion for nova and i was asked ot get an excption in the past | |
| 11:57:23 | sean-k-mooney[m] | for libs in the same situration | |
| 11:58:36 | sean-k-mooney[m] | as i said im not blockign this but i dont really like late breakign chnages like this espcially when the lib in question could have added the new api witout breaking the existing users | |
| 11:59:23 | stblatzheim | sean-k-mooney[m]: Regarding https://review.opendev.org/c/openstack/nova/+/910466 i don't get why this should be solved in neutron. It's just a changed behaviour which wasn't adapted in the decision making of dhcp in nova-api. Can you clarify this? | |