| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-26 | |||
| 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 | |
| 07:20:31 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | |
| 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 | |
| 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 | |
| 10:39:00 | bauzas | last successful run was 2023-07-24 09:18:03 | |
| 10:39:01 | sean-k-mooney | ya it stated on the 24th | |
| 10:39:22 | sean-k-mooney | im not really sure what could have merged then | |
| 10:39:59 | bauzas | looking at gerrit | |
| 10:40:08 | bauzas | it doesn't sound a race, rather a regression | |
| 10:40:10 | sean-k-mooney | https://github.com/openstack/nova/commits/master | |
| 10:40:34 | sean-k-mooney | we had one commit on the 21st and then the 25th | |
| 10:40:48 | sean-k-mooney | so i dont see anything in nova that aligns | |
| 10:42:39 | bauzas | yeah | |
| 10:43:51 | bauzas | sean-k-mooney: but tempest has suspects https://github.com/openstack/tempest/commits/04cb0adc822ffea6c7bfccce8fa08b03739894b7 | |
| 10:44:16 | sean-k-mooney | ya i looked at the optimize one | |
| 10:44:26 | sean-k-mooney | i dont think that is touching the relevent test | |
| 10:45:11 | bauzas | https://github.com/openstack/tempest/commit/eea2c1cfac1e5d240cad4f8be68cff7d72f220a8 | |
| 10:45:31 | sean-k-mooney | oh the merge commit | |
| 10:45:37 | bauzas | we have a problem with server actions but this patch touches this | |
| 10:47:37 | sean-k-mooney | yep it moved the test | |
| 10:48:36 | sean-k-mooney | this is the orginal commit https://github.com/openstack/tempest/commit/1643536c9f7407dceccfaa18a2dd95156de1419c looks like dan wrote it | |
| 10:49:02 | bauzas | probably an unnotified merge conflict when rebasing | |
| 10:50:20 | sean-k-mooney | i dont think so | |
| 10:50:38 | sean-k-mooney | well maybe so what dan did was split up the large class into 3 | |
| 10:50:46 | sean-k-mooney | so that we got bettter parallisum | |
| 10:51:12 | sean-k-mooney | reducing the over all time of the tests form 15 min to 5 on there system | |
| 10:51:21 | sean-k-mooney | but that might mean we have more load | |
| 10:51:34 | sean-k-mooney | and the vm boots might now be slower | |
| 10:53:42 | bauzas | yeah sounds plausible | |
| 10:54:06 | sean-k-mooney | im just going to change server = self.create_test_server(wait_until='ACTIVE') | |
| 10:54:08 | sean-k-mooney | to server = self.create_test_server(wait_until='SSHABLE') | |
| 10:56:09 | bauzas | shoot | |
| 10:56:20 | bauzas | and depends-on it with a nova dnm please | |
| 10:56:31 | sean-k-mooney | yep | |
| 10:57:37 | sean-k-mooney | now i want one :) | |
| 10:57:45 | sean-k-mooney | enjoy | |
| 11:03:46 | opendevreview | sean mooney proposed openstack/nova master: use disk caching to hide slow cinder performance https://review.opendev.org/c/openstack/nova/+/889383 | |
| 11:04:40 | sean-k-mooney | ill use my diskcache mode change to test it instead ^ that way we can test both and save ci time | |
| 12:56:42 | opendevreview | alecorps proposed openstack/nova-specs master: new spec: support of vnc console for mks https://review.opendev.org/c/openstack/nova-specs/+/889857 | |