| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-02-06 | |||
| 11:41:11 | opendevreview | Merged openstack/nova stable/xena: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871415 | |
| 13:05:40 | auniyal_ | O/, | |
| 13:05:41 | auniyal_ | hyper_v.limit_cpu_features configuration parameter | |
| 13:05:41 | auniyal_ | https://opendev.org/openstack/nova/src/commit/72370a188c0755bc9c864b5a5e4a972077cb8dd6/nova/conf/hyperv.py#L72 | |
| 13:05:41 | auniyal_ | "live migration to hosts with different CPU features, ..... limit the CPU features used by the instance (while creation)" | |
| 13:05:41 | auniyal_ | what is it mean to limit? and to be true of false ? | |
| 13:16:25 | opendevreview | Merged openstack/os-vif master: Increase the swap size to 8GB in tempest jobs https://review.opendev.org/c/openstack/os-vif/+/872655 | |
| 13:25:42 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: fup: support evacuate target state https://review.opendev.org/c/openstack/nova/+/872413 | |
| 14:22:02 | opendevreview | Merged openstack/nova stable/wallaby: Cleanup old resize instances dir before resize https://review.opendev.org/c/openstack/nova/+/864691 | |
| 14:28:17 | dvo-plv | Hello, Sean. I would like to continue our coversation, which we had at friday | |
| 14:29:28 | dvo-plv | We talked about packed_ring option | |
| 14:33:07 | dvo-plv | I would like to discuss the situation when use did not ask about COMPUTE_NET_VIRTIO_PACKED trait, but we need to handle migration in some way. I found that scheduler has ALL_REQUEST_FILTERS array with different filters. My eye falls on the accelerators_filter. I suggest implement packed_ring filtering in the same way as in this method | |
| 14:40:45 | bauzas | dvo-plv: sean-k-mooney is on PTO today | |
| 14:57:22 | dvo-plv | Thank you | |
| 15:07:57 | bauzas | by default, now my disk is getting a lot of shit from dstat and other dbtracing | |
| 15:08:04 | bauzas | so, huge logs size | |
| 15:10:34 | bauzas | man, I just miss the old g'days of screen-based services :) | |
| 15:41:46 | prometheanfire | working on an updated constraints patchset, so far it's just nova and tempest that are still failing, anyone around to help figure out which lib update is causing the fail? https://zuul.opendev.org/t/openstack/build/4ba64ca08116467daca31551ce8a9753 | |
| 15:59:35 | bauzas | gmann: I lost a bit of scope for tempest wallaby, what's up ? | |
| 15:59:45 | bauzas | can we recheck wallaby changes ? | |
| 16:58:54 | gibi | bauzas: I guess it might make sense now to start an etherpad about CI failures on master and try to get the grouped and maybe tracking how is looking at which failure | |
| 16:59:01 | gibi | we can talk about this tomorrow ^^ | |
| 16:59:16 | bauzas | gibi: looks a need, indeed | |
| 16:59:44 | bauzas | gibi: wants to start it, or do you want me to pull things I found first ? | |
| 17:09:15 | gibi | bauzas: please start it. I won't have time to look at it today, but maybe tomorrow I can take a look them | |
| 17:09:41 | bauzas | gibi: ok, I'll try to gather my findings tomorrow morning | |
| 17:09:55 | bauzas | and we can improve it as a team | |
| 17:10:40 | gibi | cool, thanks | |
| 18:32:46 | opendevreview | Merged openstack/os-vif master: Implement "BaseCommand" result property https://review.opendev.org/c/openstack/os-vif/+/872391 | |
| 18:58:37 | gmann | bauzas: gibi, yes, after tempest, sdks broke it and with this it fixed and green https://review.opendev.org/c/openstack/nova/+/871798 | |
| 18:58:55 | gmann | it is good to recheck now | |
| 21:24:42 | tobias-urdin | found a little nasty spelling mistake in compute rpc api aliasing introduced here https://review.opendev.org/c/openstack/nova/+/858383/30/nova/compute/rpcapi.py#428 | |
| 21:24:49 | tobias-urdin | unless there is a patch pending for it | |
| 21:25:03 | tobias-urdin | alias for 6.2 rpc version says antilope and not antelope | |
| 21:26:40 | opendevreview | Tobias Urdin proposed openstack/nova master: Fix 6.2 compute RPC version alias https://review.opendev.org/c/openstack/nova/+/872804 | |
| 21:26:45 | tobias-urdin | ^ patch | |
| #openstack-nova - 2023-02-07 | |||
| 02:17:43 | opendevreview | Merged openstack/nova master: Protect against a deleted node id file https://review.opendev.org/c/openstack/nova/+/872204 | |
| 07:56:47 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 07:56:47 | 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:56:48 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Cache the key manager when disk is encrypted https://review.opendev.org/c/openstack/nova/+/826756 | |
| 07:56:48 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 07:56:49 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |
| 07:56:49 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 07:56:50 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 07:56:51 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 07:56:51 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | |
| 07:56:53 | opendevreview | melanie witt proposed openstack/nova master: Add reset_encryption_fields() and save_all() to BlockDeviceMappingList https://review.opendev.org/c/openstack/nova/+/870938 | |
| 07:56:53 | opendevreview | melanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties https://review.opendev.org/c/openstack/nova/+/870939 | |
| 07:56:55 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 08:13:18 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 08:13:18 | 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 | |
| 08:13:19 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Cache the key manager when disk is encrypted https://review.opendev.org/c/openstack/nova/+/826756 | |
| 08:13:19 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 08:13:20 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |
| 08:13:20 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 08:13:21 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 08:13:21 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 08:13:22 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | |
| 08:13:23 | opendevreview | melanie witt proposed openstack/nova master: Add reset_encryption_fields() and save_all() to BlockDeviceMappingList https://review.opendev.org/c/openstack/nova/+/870938 | |
| 08:13:23 | opendevreview | melanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties https://review.opendev.org/c/openstack/nova/+/870939 | |
| 08:13:25 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 09:04:17 | bauzas | tobias-urdin: excellent catch on the RPC pin alias, many thanks | |
| 09:04:31 | bauzas | it would have broken seriously our users if we were having it released | |
| 09:27:36 | bauzas | yet another strange DB issue https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d00/868237/9/check/nova-tox-functional-py38/d00d1ff/testr_results.html | |
| 09:27:50 | bauzas | gibi: I'm writing the etherpad | |
| 09:28:03 | bauzas | and I'm reporting a new gate failure | |
| 09:29:15 | gibi | bauzas: the failure you linked is probably https://bugs.launchpad.net/nova/+bug/1946339 | |
| 09:29:51 | gibi | so no need for a new bug report | |
| 09:30:45 | bauzas | gibi: seems unrelated | |
| 09:30:56 | bauzas | I get a DB issue due to a missing table | |
| 09:31:09 | bauzas | oh | |
| 09:31:29 | gibi | both fails with | |
| 09:31:29 | gibi | sqlite3.OperationalError: no such table: instance_faults | |
| 09:31:41 | gibi | after a message timeout | |
| 09:31:51 | bauzas | yeah, not the same table but quite the same behaviour | |
| 09:32:03 | bauzas | well, the bug report title is too specific | |
| 09:33:06 | bauzas | gibi: agreed, seems the same | |
| 09:47:20 | bauzas | gibi: https://etherpad.opendev.org/p/nova-ci-failures | |
| 09:49:49 | bauzas | damn shit, opensearch urls aren't shareable | |
| 09:52:40 | gibi | bauzas: feel free to update the title of https://bugs.launchpad.net/nova/+bug/1946339 | |
| 09:52:58 | bauzas | gibi: I'm not good at naming | |
| 09:53:10 | bauzas | and I'm not sure of the rootcause | |
| 09:53:26 | bauzas | I guess this is due to long-running threads | |
| 09:53:40 | bauzas | so, due to the length it takes, then we delete the DB | |
| 09:54:06 | gibi | bauzas: changed the title then | |
| 09:54:08 | bauzas | and so, when the thread stops, then we try to lookup the table which no longer exists | |
| 09:54:22 | bauzas | amirite on the root cause ? | |
| 09:54:52 | gibi | bauzas: one of the root cause is described in the commit message here https://review.opendev.org/c/openstack/nova/+/814036 | |
| 09:56:24 | gibi | what I we need to figure out what is the new sequence of events that leads to a similar leak | |
| 09:56:25 | bauzas | yeah ok, so we agree | |
| 09:56:35 | bauzas | we have spawned greenlets | |
| 09:56:42 | bauzas | that finish after the db is cleaned up | |
| 09:56:59 | bauzas | we should hold the test until those greenlets finish | |
| 09:58:16 | gibi | 1) we need to figure out which eventlet leaks 2) then we need to figure out how that leaked eventlet affects the later test case, i.e. where it the global variable that supports the leak 3) then we can figure out how to fix it | |
| 09:59:12 | tobias-urdin | bauzas: no worries, funny coincidence that i was digging around in the RPC layer and saw it :) | |
| 09:59:53 | gibi | in the fixed case it was RPC handling eventlet was leaked that after 60 sec waked up do to timeout and used nova.rpc.get_versioned_notifier() to get a fresh notifier to the actually running test case | |
| 10:00:21 | opendevreview | Jorge San Emeterio proposed openstack/nova master: Dividing global privsep profile https://review.opendev.org/c/openstack/nova/+/871729 | |
| 10:00:51 | gibi | the fix there was to ignore notifications if it is comming from a test case that is different from the currently running one | |
| 10:03:53 | bauzas | gibi: grabbing a coffee, this is a hard issue to dig into | |
| 10:04:11 | gibi | bauzas: ack I will add a logsearch query for it as soon as I have one | |