| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-26 | |||
| 11:01:34 | sean-k-mooney | its cutting it kind of tight to get this all landed this cycle | |
| 11:03:17 | carloss | > carloss: cool the nova code shoudl use the sdk not the manilaclient | |
| 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:50 | carloss | > its cutting it kind of tight to get this all landed this cycle | |
| 11:03:50 | carloss | sean-k-mooney: ack - we'll do our best to have it landing asap in manila | |
| 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 | yeah, we have some changes up for functional testing on manila-tempest-plugin as well: | |
| 11:06:34 | carloss | https://review.opendev.org/c/openstack/manila-tempest-plugin/+/888618 - Test resource locks | |
| 11:06:34 | carloss | https://review.opendev.org/c/openstack/manila-tempest-plugin/+/889331 - Add access rules restriction tests | |
| 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 add/remove instance info from node https://review.opendev.org/c/openstack/nova/+/659691 | |
| 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:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for volume_target.* https://review.opendev.org/c/openstack/nova/+/867790 | |
| 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: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:55 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.vif* https://review.opendev.org/c/openstack/nova/+/867793 | |
| 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: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:58 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for deprecated baremetal proxy API https://review.opendev.org/c/openstack/nova/+/867796 | |
| 12:41:58 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove Ironic client wrapper https://review.opendev.org/c/openstack/nova/+/867797 | |
| 12:42:00 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unnecessary aliasing https://review.opendev.org/c/openstack/nova/+/867798 | |
| 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: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: block_device: Add encryption attributes to swap disks https://review.opendev.org/c/openstack/nova/+/884312 | |
| 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: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:01 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 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:02 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |
| 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: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: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: block_device: Add encryption attributes to swap disks https://review.opendev.org/c/openstack/nova/+/884312 | |
| 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: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: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 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:30 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |
| 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:31 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 07:20:31 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | |
| 08:54:46 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM: Verify whether we persist some wrong ReqSpec values https://review.opendev.org/c/openstack/nova/+/889843 | |
| 09:02:49 | opendevreview | alecorps proposed openstack/nova-specs master: new spec: support of vnc console for mks https://review.opendev.org/c/openstack/nova-specs/+/889844 | |
| 10:20:44 | sean-k-mooney | bauzas: so we have a gate blocker in the nova-lvm job | |
| 10:20:57 | bauzas | shit shit shit | |
| 10:20:58 | sean-k-mooney | test_get_console_output_server_id_in_shutoff_status is failing with AssertionError: '' is not true : Console output was empty. | |
| 10:21:27 | sean-k-mooney | im seeing that quite a lot | |
| 10:22:13 | sean-k-mooney | always in the lvm job for some reason and not others | |
| 10:23:32 | sean-k-mooney | looking at the test im wondering if this could be a test bug or not | |
| 10:23:34 | sean-k-mooney | https://github.com/openstack/tempest/blob/04cb0adc822ffea6c7bfccce8fa08b03739894b7/tempest/api/compute/servers/test_server_actions.py#L713 | |
| 10:23:54 | sean-k-mooney | im wondering if we are perhaps stoping the server before the guest writes to the console? | |
| 10:27:21 | sean-k-mooney | https://opensearch.logs.openstack.org/_dashboards/app/discover?security_tenant=global#/?_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-30d,to:now))&_a=(columns:!(hosts_region,project,build_name),filters:!(('$state':(store:appState),meta:(alias:!n,disabled:!f,index:'94869730-aea8-11ec-9e6a-83741af3fdcd',key:project,negate:!f,params:(query:openstack%2Fnova),ty | |
| 10:27:23 | sean-k-mooney | pe:phrase),query:(match_phrase:(project:openstack%2Fnova)))),index:'94869730-aea8-11ec-9e6a-83741af3fdcd',interval:d,query:(language:kuery,query:'message:%22Console%20output%20was%20empty.%22'),sort:!()) | |
| 10:27:31 | sean-k-mooney | hum let me get a short link | |
| 10:28:31 | sean-k-mooney | https://tinyurl.com/kvcc9289 | |
| 10:29:06 | sean-k-mooney | 132 failures in the last 3 days | |
| 10:36:32 | bauzas | sean-k-mooney: ack, ok | |
| 10:37:19 | bauzas | sorry, was on meeting | |
| 10:37:32 | sean-k-mooney | no worries just filed https://bugs.launchpad.net/nova/+bug/2028851 | |
| 10:38:06 | sean-k-mooney | im going ot add a wiat for sshabel before the server stop in tempest and see if that helps | |
| 10:38:39 | bauzas | sean-k-mooney: it looks to me it regressed two days before https://zuul.openstack.org/builds?job_name=nova-lvm&skip=0 | |