| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-25 | |||
| 16:19:00 | gibi | o/ | |
| 16:19:09 | elodilles | thanks, too o/ | |
| 16:21:14 | gmann | thanks o/ | |
| 16:58:39 | opendevreview | sean mooney proposed openstack/nova master: [WIP] use disk caching to hide slow cinder performance https://review.opendev.org/c/openstack/nova/+/889383 | |
| 16:59:11 | sean-k-mooney | if we decied this is the correct approch in general i can do ^ in devstack instead | |
| 16:59:45 | sean-k-mooney | or in the base job but im adding it to all the in tree job to get more input on if it affect the ci jobs positivly or negitivly | |
| 17:02:11 | dansmith | sean-k-mooney: I think doing it in devstack itself is the wrong move, but maybe in some of the devstack job defs.. this is really an optimization for our own CI and not something we'd want anyone running without knowing it, even in devstack, IMHO | |
| 17:02:41 | dansmith | I'll be interested to see if that helps significantly.. I've seen that bug manifest only occasionally in my surveying recently | |
| 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 | |