| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-05 | |||
| 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 | |
| 09:48:00 | kashyap | Or: | |