| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-12-12 | |||
| 14:40:44 | lyarwood | efried: not really tbh | |
| 14:40:49 | lyarwood | efried: yeah it was | |
| 14:41:04 | efried | Then meh, use a new change-id but ref the old one in the commit message | |
| 14:41:28 | lyarwood | ack will do | |
| 14:43:44 | dansmith | mriedem: for sure | |
| 14:44:09 | dansmith | mriedem: theoretically you could handle it in rpcapi but definitely should have a bump imho | |
| 14:44:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move _get_request_group_mapping() to RequestSpec https://review.opendev.org/696541 | |
| 14:44:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move _update_pci_request_spec_with_allocated_interface_name https://review.opendev.org/696574 | |
| 14:46:15 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support live migration with qos ports https://review.opendev.org/695905 | |
| 14:47:41 | huaqiang | stephenfin: morning | |
| 14:48:12 | huaqiang | stephenfin: for https://review.opendev.org/#/c/668656/, can you make a review when you have time | |
| 14:48:27 | gibi | efried, mriedem: fixed up your comments in the qos live migration series | |
| 14:48:40 | stephenfin | mriedem: Yeah. I can reproduce that too. Will try root out a fix, assuming sean-k-mooney hasn't fixed it yet | |
| 14:49:18 | sean-k-mooney | luyao: left a few comments but more or less i think the direction of the spec is good. just grabing something to eat so i wont be around for a the next 30 mins or so | |
| 14:49:39 | mriedem | gibi: thanks, much better | |
| 14:49:55 | sean-k-mooney | stephenfin: nope havent started was going to take a look later today but have a few other thing to get done first | |
| 14:54:05 | luyao | sean-k-mooney: thanks for your comments:) | |
| 14:56:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Keep pre-commit inline with hacking and fix whitespace https://review.opendev.org/674057 | |
| 14:57:17 | stephenfin | sean-k-mooney: Cool | |
| 14:57:58 | stephenfin | also, "foo is 'string'" raises a syntax deprecation warning in Python 3, so we should probably merge https://review.opendev.org/#/c/695732/ sooner rather than later | |
| 14:59:01 | huaqiang | :xa | |
| 15:01:15 | efried | stephenfin: +2 | |
| 15:02:28 | stephenfin | ta | |
| 15:02:48 | stephenfin | huaqiang: I've a few other things ahead of it, but I'll try | |
| 15:05:28 | huaqiang | stephenfin: I see. I especially need comments for if we need to create 'mixed' instance through 'resources:PCPU' and 'resources:VCPU' and specifying the count of dedicated vCPU number in flavor/image or using a CPU mask for them | |
| 15:05:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Don't hardcore Python versions in test https://review.opendev.org/698732 | |
| 15:05:58 | stephenfin | efried, mriedem, sean-k-mooney: ^ | |
| 15:06:05 | stephenfin | hardcore. FML | |
| 15:06:18 | efried | love it | |
| 15:06:18 | mriedem | stephenfin: efried: what's wrong with the dict usage for urlencode here? https://review.opendev.org/#/c/695732/2/nova/tests/unit/api/openstack/compute/test_volumes.py | |
| 15:06:19 | huaqiang | stephenfin: I know we have had some discussion on these two questions, but there are some other opionions on them | |
| 15:06:35 | efried | mriedem: URLs allow repetition of keys; dicts don't. | |
| 15:06:45 | efried | mriedem: querystrings are also ordered. | |
| 15:06:50 | mriedem | oh guh | |
| 15:06:56 | mriedem | didn't even think about the duplicate keys | |
| 15:06:58 | efried | So old encoders just kinda went with it | |
| 15:07:09 | efried | but good code should be explicit | |
| 15:07:44 | huaqiang | stephenfin: so I want to reach consitency on these two questions. | |
| 15:08:52 | KeithMnemonic | Hello mridem. Do you think there will be any movement on the queens/pike cherry picks of your "marker" fix? https://review.opendev.org/#/c/690725/2. I know those are EOL but since the patches were created was not sure if you thought they could still get in? | |
| 15:09:08 | KeithMnemonic | argh mriedem: | |
| 15:09:34 | awalende | Anyone knows what nova-compute wants to tell me when trying to attach a volume to a new instance on a freshly deployed ubuntu 18.04 compute node?: unable to execute QEMU command 'device_add': Property 'virtio-blk-device.drive' can't find value 'drive-virtio-disk1' | |
| 15:11:13 | mriedem | KeithMnemonic: you have to ask stable cores | |
| 15:11:31 | KeithMnemonic | ok thanks | |
| 15:11:38 | mriedem | those branches aren't EOL either, | |
| 15:11:42 | mriedem | they are EM = extended maintenance | |
| 15:11:53 | mriedem | https://review.opendev.org/#/admin/groups/540,members | |
| 15:15:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Don't hardcore Python versions in test https://review.opendev.org/698732 | |
| 15:15:58 | KeithMnemonic | mriedem is there a preferred way to ping the "stable cores" | |
| 15:16:08 | efried | stephenfin: I thought you were going to fix that commit title? | |
| 15:16:10 | KeithMnemonic | should i attend the next IRC meeting? | |
| 15:16:21 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use reasonable name for provider mapping https://review.opendev.org/698735 | |
| 15:16:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Don't hardcode Python versions in test https://review.opendev.org/698732 | |
| 15:16:39 | efried | KeithMnemonic: you might be waiting a while. bauzas dansmith melwitt lyarwood | |
| 15:16:44 | stephenfin | efried: yeah, done | |
| 15:16:58 | efried | stephenfin: though I rather liked it the way it was before. | |
| 15:17:40 | efried | ah | |
| 15:17:54 | dansmith | KeithMnemonic: I've reviewed the other backports of this | |
| 15:18:05 | dansmith | mriedem: any concern about the queens one to keep me from slamming this? | |
| 15:18:11 | mriedem | stephenfin: "hardcore" | |
| 15:18:12 | mriedem | filthy | |
| 15:18:12 | KeithMnemonic | thanks efried: | |
| 15:18:19 | efried | yahyoubetcha | |
| 15:20:51 | mriedem | Intel OpenStack CI is failing on merge conflicts - it must be patching something into the CI | |
| 15:21:04 | mriedem | isn't that the one asking for vpmem live migration support? | |
| 15:21:12 | stephenfin | mriedem: Can you restore this for me? https://review.opendev.org/#/c/685756/ | |
| 15:21:25 | efried | luyao: ---^ | |
| 15:21:30 | mriedem | stephenfin: done | |
| 15:21:31 | stephenfin | (Gerrit won't let me backport the fixes otherwise) | |
| 15:21:32 | stephenfin | ta | |
| 15:21:47 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: Also enable iommu for virtio controllers and video in libvirt https://review.opendev.org/685756 | |
| 15:21:47 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: Switch to uses_virtio to enable iommu driver for AMD SEV https://review.opendev.org/698738 | |
| 15:21:48 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: Create a controller for qga when SEV is used https://review.opendev.org/698739 | |
| 15:23:03 | mriedem | dansmith: when is the last time we actually dropped old tables? https://review.opendev.org/#/q/topic:db-cleanup+(status:open+OR+status:merged) | |
| 15:23:18 | bbobrov | oh nice | |
| 15:23:19 | dansmith | tables? | |
| 15:23:25 | bbobrov | so we backport the fixes to train | |
| 15:23:37 | dansmith | never? | |
| 15:23:44 | mriedem | that's what i thought | |
| 15:23:47 | stephenfin | bbobrov: You mean you (SUSE) have done it already | |
| 15:23:55 | stephenfin | dansmith: You sit on a throne of LIES | |
| 15:25:01 | stephenfin | Search for 'assertTableNotExists' | |
| 15:25:10 | mriedem | i guess there are some old migrations that dropped really old tables | |
| 15:25:10 | stephenfin | in 'nova/tests/unit/db/test_migrations.py' | |
| 15:25:19 | mriedem | 232, 233 | |
| 15:25:25 | dansmith | mriedem: what were the tables? | |
| 15:25:30 | stephenfin | 346 | |
| 15:25:40 | stephenfin | https://review.opendev.org/#/c/698705/1/nova/tests/unit/db/test_migrations.py@234 | |
| 15:25:44 | mriedem | dump_*, compute_node_stats, instance_group_metadata | |
| 15:26:14 | mriedem | iscsi_targets, volumes | |
| 15:26:30 | mriedem | 292 was the most recent table drop migration | |
| 15:26:31 | dansmith | oh, gosh of course, from glance and cinder splits | |
| 15:26:49 | mriedem | https://review.opendev.org/#/c/166214/ being the most recent | |
| 15:26:50 | dansmith | stephenfin: 346 was a column | |
| 15:27:25 | mriedem | liberty, 2015 | |
| 15:27:28 | stephenfin | ah, that explains why it told me I wasn't allowed drop a column when I was dropping a table. Same heuristic, I guess | |
| 15:27:30 | mriedem | the wild west of yore | |
| 15:28:02 | stephenfin | I assume there's a reason not to drop tables? | |
| 15:28:13 | dansmith | mriedem: liberty was the 292? | |
| 15:28:18 | mriedem | dansmith: yeah | |
| 15:28:26 | dansmith | mriedem: if so, those tables must've been unused for longer than that | |