Earlier  
Posted Nick Remark
#openstack-nova - 2021-08-05
12:48:39 songwenping ack
12:49:54 sean-k-mooney oh https://github.com/openstack/nova/blob/35ddf1ad40207dee681a3c92cc9e86b061234edd/nova/db/sqlalchemy/api.py#L545-L550
12:50:18 sean-k-mooney so we do have that patteren
12:50:52 sean-k-mooney that would have changed form ServiceBinaryExists to ServiceTopicExists silently
12:52:03 songwenping so this also have problem
12:52:49 songwenping the tempest doesnot check the ServiceBinaryExists exception?
12:53:00 sean-k-mooney kind of becaue we use .get it wont fail
12:53:12 sean-k-mooney ya we likely dont have tempest coverage for this
12:53:23 sean-k-mooney although we should have and api funct tests for thsi
12:53:42 songwenping +1
12:53:47 gibi bauzas: OK, so you have plans to amend the audit support for MDEV. then that is really a good time to add some support for heal if possible.
12:54:19 bauzas gibi: well, I have around 12 hours for doing this until 3 weeks :p
12:54:27 songwenping i will commit one patch to coverage it.
12:54:34 bauzas 8 hours for tomorrow and 4 jours this afternoon :)
12:54:38 bauzas hours*
12:54:39 gibi bauzas: I'm not saying that you have to do it now. :D
12:54:56 gibi bauzas: it is totally ok to do it in Yoga
12:55:04 gibi bauzas: I just noted that we have a gap in heal
12:55:16 bauzas gibi: create a bug report and I'll own it
12:55:19 gibi I will make sure that it is now documented
12:55:27 gibi bauzas: OK I can file a bug too
12:55:30 bauzas or a blueprint maybe
12:55:31 gibi that is a good idea
12:55:45 gibi I will create a bug and later on we can transform that to a bp
12:55:50 bauzas this is a tech debt fix
12:55:51 gibi if we ever get to it
13:24:22 opendevreview Pierre Libeau proposed openstack/nova master: Reset state during service nova-compute init https://review.opendev.org/c/openstack/nova/+/803600
13:37:54 opendevreview Merged openstack/nova master: trivial: Cleanup a comment about a now removed libvirt version check https://review.opendev.org/c/openstack/nova/+/802617
13:51:14 gibi bauzas: filed the bug about heal_allocation https://bugs.launchpad.net/nova/+bug/1939020
13:51:58 bauzas gibi: and triaged
13:53:34 opendevreview Balazs Gibizer proposed openstack/nova master: Block servers wit vGPU and device profile in heal_allocations https://review.opendev.org/c/openstack/nova/+/803603
13:53:42 gibi bauzas: thanks, and here is the doc and block ^^
13:57:01 bauzas gibi: looking
15:08:27 melwitt bauzas: I'm gonna respond on the review too but the "unknown" is not a default consumer type, it's a cosmetic-only label for friendliness to the user. (same with the "all" label as a query parameter). a consumer with no type has NULL in the column. the reproposal of the spec has removed mention of the original proposed "UNKNOWN" official default type. during review reviewers preferred to leave NULL to represent the absence of a consumer
15:08:29 melwitt type
15:08:46 bauzas melwitt: oh ok
15:08:58 bauzas I only looked at the train spec
15:09:08 bauzas when this was something as a gerrit spec :p
15:09:09 melwitt ah gotcha
15:09:25 bauzas melwitt: will then remove my -1
15:09:41 melwitt cool thanks :)
15:10:08 bauzas (and yeah sorry I cheated :p )
15:11:13 melwitt haha no worry
15:14:08 lyarwood https://review.opendev.org/c/openstack/nova/+/803585 btw gibi / stephenfin / melwitt ; moving the LM jobs to NV for the time being while someone looks at the underlying QEMU failure(s)
15:14:48 melwitt /sad trombone sound
15:18:41 melwitt kashyap: I see you have commented on https://launchpad.net/bugs/1912310 before ^ but no leads yet right?
15:18:54 kashyap melwitt: Hi, /me looks
15:19:13 kashyap (The bot doesn't seem to higlight the summary on IRC anymore)
15:20:29 melwitt hm yeah not sure what happened to the bot
15:20:51 kashyap melwitt: Yeah, no leads yet
15:22:07 kashyap melwitt: Is it hitting more often? Looks like so from gibi's #7 on 31Jul
15:22:46 melwitt kashyap: yeah, lyarwood is proposing above making the live migration job non-voting because of it
15:23:03 kashyap melwitt: Oh, yeah - just saw that
15:55:48 bauzas lyarwood: unfortunately +Wd the live-mig job be non-votign
15:56:02 lyarwood ack thanks
16:06:35 gibi about the bug resolver bot I asked infra they said that https://meetings.opendev.org/irclogs/%23opendev/%23opendev.2021-08-03.log.html#t2021-08-03T16:31:32
16:11:16 melwitt thanks gibi
16:14:20 gibi so probably we lost the both the topic handling and the bug resolving capability of the bot during that transformatin
16:43:02 gibi If I had time I would be happy working on those bots
18:11:55 opendevreview Merged openstack/nova master: zuul: Mark live migration jobs as non-voting due to bug #1912310 https://review.opendev.org/c/openstack/nova/+/803585
21:12:41 nurdie Hey OS nova! I had to evacuate a failed compute node (it's super dead) on shared storage (ceph). 3 instances are hanging in "nova migration-list" as "accepted" but are already on another compute node. Does anyone know what galera db.table I can edit real quick to clear that up?
23:23:58 opendevreview melanie witt proposed openstack/placement master: Add consumer_types migration, database and object changes https://review.opendev.org/c/openstack/placement/+/669170
23:24:06 opendevreview melanie witt proposed openstack/placement master: Microversion 1.38: API support for consumer types https://review.opendev.org/c/openstack/placement/+/679441
23:24:11 opendevreview melanie witt proposed openstack/placement master: Switch ConsumerType to use an AttributeCache https://review.opendev.org/c/openstack/placement/+/679486
23:24:35 melwitt bauzas: updated first patch per your comments ^
#openstack-nova - 2021-08-06
00:18:06 opendevreview Brin Zhang proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs https://review.opendev.org/c/openstack/nova/+/765311
01:45:00 opendevreview Qiu Fossen proposed openstack/nova master: Allow migrating PMEM's data https://review.opendev.org/c/openstack/nova/+/802225
03:06:33 opendevreview melanie witt proposed openstack/placement master: Switch ConsumerType to use an AttributeCache https://review.opendev.org/c/openstack/placement/+/679486
07:55:55 lyarwood gibi: Morning, https://bugs.launchpad.net/nova/+bug/1939108 but on the positive side, logsearch++
08:06:18 aarents bauzas: thanks for https://review.opendev.org/c/openstack/nova/+/764435 review, I applied the rephrasing.
08:13:22 gibi lyarwood: morning! could this be related to the guest blocking or delaying the detach attempt? I vaguly remember we discussed such option before
08:14:52 lyarwood gibi: yeah it's likely that, I need to push the tempest change ahead to rework detach so we actually dump the console to know either way
08:15:08 lyarwood iirc it was failing on something random the last time I rechecked
08:15:18 lyarwood https://review.opendev.org/c/openstack/tempest/+/794757
08:15:45 lyarwood yeah unrelated test_create_list_show_delete_interfaces_by_network_port ssh timeouts
08:15:48 lyarwood /o\
08:15:50 lyarwood rechecks again
08:16:17 lyarwood actually rebase as it has been that long
08:20:11 lyarwood gibi: https://github.com/gibizer/zuul-log-search-config/pull/1 btw, have a few ideas about the tool itself but I might not have time to hack on it until next week
08:29:56 gibi ohh, ok, lets hope the console log reveale something, other than that we can only increase timeout for the single detach attempt or increase the amount of retry we do
08:30:39 lyarwood indeed, 1 hit in the last 100 failures however suggests we are somewhere close to getting this right in CI finally
08:31:11 gibi lyarwood: if you have ideas about the feel free to note them in an github issue or directly in the TODO.md in the repo
08:31:53 lyarwood ack will do, it's mostly basic stuff like an argument for number of days back to search for etc and maybe a handler for logstash but that's it at the moment
08:31:56 gibi lyarwood: thanks for looking at that func faiure I had the intention to look at both func faiures but I had no time since and probablny not even today
08:40:49 bauzas aarents: thanks for the ping, just rechecked and +2d
08:43:53 aarents bauzas: thanks
09:08:16 opendevreview Pierre Libeau proposed openstack/nova master: Move file system freeze after end of mirroring https://review.opendev.org/c/openstack/nova/+/803713
09:08:59 opendevreview Lee Yarwood proposed openstack/nova master: func: Increase rpc_response_timeout in TestMultiCellMigrate tests https://review.opendev.org/c/openstack/nova/+/803714
09:09:52 lyarwood gibi / bauzas / stephenfin ; ^ okay I think that's a pretty trivial fix to the functional failures, I don't get why it's suddenly an issue but as this RPC call to the conductor then calls down to the source I'm assuming we've added some additional overhead somewhere and were not creeping over the 1 second timeout in the test.
09:10:10 lyarwood are now*
09:12:58 gibi lyarwood: will check soon
09:42:57 kashyap lyarwood: What's the "best" manual way to change the video model for an existing instance? (Asking for a friend.) Is the following painful approach the only way?
09:43:30 kashyap lyarwood: (1) Stop the instance. (2) snapshot it (what kind?). (3) Update the metadata on the snapshot image to use 'virtio' display dev. (4) Then rebuild the instance using the snapshot
09:43:34 kashyap Any better way than that?
09:43:54 kashyap (Besides the future goodness of 'nova-manage' command akin to machine_type update)
09:44:27 lyarwood if you want to persist the instance data then yeah I think that's the only way
09:44:42 lyarwood I don't think resize would change it
09:45:16 kashyap I see.
09:45:41 kashyap And for new instances, obviously start with: openstack image set --property hw_video_model=virtio [...]
09:46:54 lyarwood yeah

Earlier   Later