| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-25 | |||
| 17:02:42 | sean-k-mooney | i was thinking of doing it like the mysql memory thing | |
| 17:03:08 | sean-k-mooney | e.g. off by default with a macro/var to turn it on in ci | |
| 17:03:11 | dansmith | yeah, it could be a flag, sure | |
| 17:03:41 | sean-k-mooney | basically i just dont want to have to copy paste that to even more jobs over time :) | |
| 17:04:23 | sean-k-mooney | im not sure if it will help or not but we will see | |
| 17:04:38 | sean-k-mooney | in theory it should not sure | |
| 19:24:20 | opendevreview | Maxim Monin proposed openstack/nova master: Server Rescue leads to Server ERROR state if original image is deleted https://review.opendev.org/c/openstack/nova/+/872385 | |
| #openstack-nova - 2023-07-26 | |||
| 02:04:51 | opendevreview | Merged openstack/nova master: Add dest_compute_id to Migration object https://review.opendev.org/c/openstack/nova/+/879682 | |
| 10:41:28 | carloss | o/ Uggla I believe we finally have some testable code in the Manila side for the VirtioFS implementation | |
| 10:42:07 | carloss | https://review.opendev.org/c/openstack/manila/+/887725 - Allow restricting access rules fields and deletion | |
| 10:42:07 | carloss | https://review.opendev.org/c/openstack/manila/+/887115 - Resource Locks: Support for share deletion lock | |
| 10:42:07 | carloss | the Manila changes: | |
| 10:43:12 | carloss | https://review.opendev.org/c/openstack/python-manilaclient/+/888355 - Allow restricting access rules | |
| 10:43:12 | carloss | https://review.opendev.org/c/openstack/python-manilaclient/+/888006 - Support for resource locks | |
| 10:43:12 | carloss | OSC changes in case you want to interact with manila through the CLI: | |
| 10:43:50 | carloss | https://review.opendev.org/c/openstack/openstacksdk/+/889519 - Resource locks and access rules restrictions | |
| 10:43:50 | carloss | and finally, the SDK part of the implementation: | |
| 10:44:00 | carloss | please let me know if I can help with something :) | |
| 10:44:40 | carloss | you'll notice some of the changes are still WIP, but that is mostly for some missing coverage in unit/functional tests | |
| 10:57:42 | opendevreview | sean mooney proposed openstack/nova master: use disk caching to hide slow cinder performance https://review.opendev.org/c/openstack/nova/+/889383 | |
| 11:00:31 | sean-k-mooney | carloss: cool the nova code shoudl use the sdk not the manilaclient | |
| 11:00:52 | sean-k-mooney | so we actully only depend on the maniala change and the sdk changes | |
| 11:01:34 | sean-k-mooney | its cutting it kind of tight to get this all landed this cycle | |
| 11:03:17 | carloss | yep... the sdk change should be in a good shape as well... I did some testing on it yesterday and it behave fine when I did stuff with it manually | |
| 11:03:17 | carloss | > carloss: cool the nova code shoudl use the sdk not the manilaclient | |
| 11:03:50 | carloss | sean-k-mooney: ack - we'll do our best to have it landing asap in manila | |
| 11:03:50 | carloss | > its cutting it kind of tight to get this all landed this cycle | |
| 11:04:17 | sean-k-mooney | at least with depend on we can test some of this before its landed | |
| 11:04:26 | carloss | ++ | |
| 11:04:38 | carloss | I think both manila side changes are ready for some feedback as well, so if you want to share your thoughts, please do so | |
| 11:04:56 | sean-k-mooney | have you considered addding tempest coverage? | |
| 11:05:24 | sean-k-mooney | or testing it in the manila ci beyond unit/funcitonal tests | |
| 11:06:34 | carloss | https://review.opendev.org/c/openstack/manila-tempest-plugin/+/889331 - Add access rules restriction tests | |
| 11:06:34 | carloss | https://review.opendev.org/c/openstack/manila-tempest-plugin/+/888618 - Test resource locks | |
| 11:06:34 | carloss | yeah, we have some changes up for functional testing on manila-tempest-plugin as well: | |
| 11:06:46 | carloss | resource locks is failing because of a known issue in our CI | |
| 11:07:25 | carloss | and access rules restrictions is on me... I need to fix some issues with testing, but I also tested it locally and it worked | |
| 11:07:43 | carloss | I should figure out such issues with access rules restriction testing over the course of this week | |
| 11:08:55 | carloss | s/locally/in a devstack env | |
| 11:09:37 | carloss | sean-k-mooney: forgot to share those changes in advance with the other ones I sent, sorry 😅 | |
| 11:09:37 | carloss | sean-k-mooney: forgot to share those changes in advance with the other ones I sent, sorry 😅 | |
| 11:37:37 | sean-k-mooney | no worries im sure they will be useful for Uggla as a refernce for how to use the apis | |
| 12:36:37 | Uggla | Hey carloss, thanks for the information. As sean-k-mooney said I'm currently moving from the manilaclient to sdk. As soon as ready (soon). I will try to implement and test the new "lock and restricted access rules" features. | |
| 12:38:07 | Uggla | carloss, also thanks for the hard work, I can't stand to integrate that. | |
| 12:41:52 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unused mocks https://review.opendev.org/c/openstack/nova/+/867789 | |
| 12:41:53 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/c/openstack/nova/+/670213 | |
| 12:41:53 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/c/openstack/nova/+/659691 | |
| 12:41:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_provision_state https://review.opendev.org/c/openstack/nova/+/867791 | |
| 12:41:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for volume_target.* https://review.opendev.org/c/openstack/nova/+/867790 | |
| 12:41:55 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.vif* https://review.opendev.org/c/openstack/nova/+/867793 | |
| 12:41:55 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_power_state https://review.opendev.org/c/openstack/nova/+/867792 | |
| 12:41:56 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining ironic driver calls https://review.opendev.org/c/openstack/nova/+/867795 | |
| 12:41:56 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining network operations https://review.opendev.org/c/openstack/nova/+/867794 | |
| 12:41:58 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove Ironic client wrapper https://review.opendev.org/c/openstack/nova/+/867797 | |
| 12:41:58 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for deprecated baremetal proxy API https://review.opendev.org/c/openstack/nova/+/867796 | |
| 12:42:00 | opendevreview | Stephen Finucane proposed openstack/nova master: Use real SDK objects in tests https://review.opendev.org/c/openstack/nova/+/867799 | |
| 12:42:00 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unnecessary aliasing https://review.opendev.org/c/openstack/nova/+/867798 | |
| 12:42:32 | sean-k-mooney | stephenfin: just one or two ptaches ya | |
| 12:42:45 | sean-k-mooney | is that all of the convertion to the sdk | |
| 12:43:08 | stephenfin | only two changed, yes 😇 (the rest are just rebases) | |
| 12:43:08 | stephenfin | only two changed, yes 😇 (the rest are just rebases) | |
| 12:43:15 | stephenfin | yup, full conversion of ironic calls to SDK | |
| 12:43:41 | sean-k-mooney | oh just ironic i tough ti say some neutron ones | |
| 12:44:08 | stephenfin | nah, the network operations referred to are still ironic calls | |
| 12:44:17 | stephenfin | neutron is a much bigger fish | |
| 12:44:42 | sean-k-mooney | ok and same for volume it just the ironic volume bits | |
| 12:45:13 | sean-k-mooney | still woul be nice to get that done this cycle | |
| 12:45:29 | carloss | Uggla: yw - happy to help... gouthamr and I did our best to get this to a testable state, so have fun testing! :D | |
| 12:45:35 | stephenfin | yup, ironic volumes | |
| 12:46:11 | Uggla | carloss, \o/ | |
| 14:40:05 | opendevreview | Merged openstack/nova master: db: Store unmasked password https://review.opendev.org/c/openstack/nova/+/880669 | |
| 15:11:55 | opendevreview | Vladislav Belogrudov proposed openstack/nova master: Nova instance snapshot should wait for volumes https://review.opendev.org/c/openstack/nova/+/848638 | |
| 15:49:20 | opendevreview | Stephen Finucane proposed openstack/nova master: Use real SDK objects in tests https://review.opendev.org/c/openstack/nova/+/867799 | |
| 16:17:16 | sean-k-mooney | test_get_console_output_server_id_in_shutoff_status is failling alot i nthe nova_lvm job | |
| 16:32:15 | dansmith | bauzas: are you around? | |
| 17:24:49 | opendevreview | Dan Smith proposed openstack/nova master: Add upgrade check for compute-object-ids linkage https://review.opendev.org/c/openstack/nova/+/889740 | |
| 17:24:53 | dansmith | sean-k-mooney: ^ | |
| 17:25:24 | dansmith | I wonder if you could proxy bauzas' +2 on this patch, since he's already +2d the followup which was his reason for not +2ing the first place: https://review.opendev.org/c/openstack/nova/+/884752/2 | |
| 17:25:40 | dansmith | that way I can continue my slow grind to recheck these mofos through the gate :/ | |
| 17:27:45 | sean-k-mooney[m] | im kind of done for today so ill review the final patch tomorrow but maybe melwitt can take a look | |
| 17:28:26 | dansmith | ack, I won't need that +2 for probably several more weeks, but I just want to be able to land it without asking in case I get super lucky ;) | |
| 17:30:30 | dansmith | to be clear, the proxy vote is on a patch you already +2d, not the new one | |
| #openstack-nova - 2023-07-27 | |||
| 01:38:07 | opendevreview | Merged openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | |
| 01:38:58 | opendevreview | Takashi Kajinami proposed openstack/nova master: Enforce correct choice for [DEFAULT] enabled_ssl_apis https://review.opendev.org/c/openstack/nova/+/889787 | |
| 07:10:00 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 07:10:00 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add encryption attributes to swap disks https://review.opendev.org/c/openstack/nova/+/884312 | |
| 07:10:01 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 07:10:01 | 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 | |
| 07:10:02 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |
| 07:10:02 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 07:10:03 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 07:10:03 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 07:10:04 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | |
| 07:20:27 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 07:20:27 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add encryption attributes to swap disks https://review.opendev.org/c/openstack/nova/+/884312 | |
| 07:20:28 | 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 | |
| 07:20:29 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 07:20:29 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 07:20:30 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 07:20:30 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |