Earlier  
Posted Nick Remark
#openstack-nova - 2021-02-09
16:24:43 dansmith sean-k-mooney: no, I mean as support for making the connection possible between the migration request and the actual migration
16:26:36 sean-k-mooney we coudl certenly improt the api respocne to make that eaiser ya
16:26:53 gibi bauzas: thanks for the review, I will reply hopefully tomorrow
16:27:10 bauzas gibi: not sure about my question if it's a problem, honestly
16:27:23 bauzas gibi: wdyt about this ?
16:27:28 sean-k-mooney dansmith: just looking at what currently avaiable however its non trivial for osc to figure out in a way that will work with existing clouds
16:27:48 dansmith yes, clearly
16:28:02 gibi bauzas: the detach case has already implemented couple of cycles ago
16:28:09 gibi let me dig out the lunk
16:28:11 gibi link
16:28:35 dansmith I was just saying I think the lack of obvious linkage is because we haven't always exposed much, and was suggesting to stephenfin that the migration context could likely be used to stash things before we've created the migration record if indeed that happens late as he says (I don't really remember
16:29:15 gibi bauzas: the detach has functional test coverage already on master https://github.com/openstack/nova/blob/941f4624638090b27e9e18601cde0529fcdbd22f/nova/tests/functional/test_servers.py#L6134
16:30:11 gibi bauzas: it was implemented in https://review.opendev.org/c/openstack/nova/+/622421 in stein
16:31:27 bauzas gibi: ah ok
16:31:47 bauzas gibi: all good then, just answer my nits
16:31:52 gibi bauzas: will do
16:49:57 kashyap melwitt: Hi
16:50:07 kashyap melwitt: I'm talking to libvirt block layer dev on that bug you filed
16:50:22 kashyap melwitt: Do you have a link to the libvirtd.log from where you posted this snippet? -- https://launchpadlibrarian.net/521114497/bug1914282_log_excerpts.txt
16:50:52 kashyap melwitt: In the bug itself you posted the "ready": true bit ... but right after that we should see for an event called BLOCK_JOB_COMPLETED
16:52:33 kashyap melwitt: Ah, ignore me; it's the 'controller' dir
17:29:58 openstackgerrit Stephen Finucane proposed openstack/nova master: Drop support for custom schedulers https://review.opendev.org/c/openstack/nova/+/773637
17:29:59 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Drop '[scheduler] periodic_task_interval' https://review.opendev.org/c/openstack/nova/+/773638
17:30:01 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Clean up docs for scheduler options https://review.opendev.org/c/openstack/nova/+/773639
17:30:03 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Remove 'USES_ALLOCATION_CANDIDATES' https://review.opendev.org/c/openstack/nova/+/773640
17:30:05 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Remove 'hosts_up' https://review.opendev.org/c/openstack/nova/+/773641
17:30:06 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Remove FakeScheduler (for realz) https://review.opendev.org/c/openstack/nova/+/773642
17:30:07 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Merge 'FilterScheduler' into base class https://review.opendev.org/c/openstack/nova/+/773643
17:30:11 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Merge driver into manager https://review.opendev.org/c/openstack/nova/+/773644
17:30:15 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Drop references to non-filter scheduler drivers https://review.opendev.org/c/openstack/nova/+/773645
17:30:18 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Merge 'test_utils', 'test_scheduler_utils' https://review.opendev.org/c/openstack/nova/+/773646
17:30:22 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Remove deprecated aliases https://review.opendev.org/c/openstack/nova/+/773647
18:34:26 openstackgerrit Merged openstack/nova master: db: Compact Pike database migrations https://review.opendev.org/c/openstack/nova/+/759087
18:35:21 openstackgerrit Stephen Finucane proposed openstack/nova master: Use neutronclient's port binding APIs https://review.opendev.org/c/openstack/nova/+/706295
20:52:05 artom CI jobs for https://review.opendev.org/c/openstack/nova/+/772779/6 seem to not have picked up the os-traits dependency at https://review.opendev.org/c/openstack/os-traits/+/771705
20:52:16 artom And I don't understand what's going on...
20:54:47 artom If I `pip install -e .` my os-traits patch into the tox venv locally, functional tests pass...
21:02:57 lyarwood ouch, did you not get the trait in before the lib freeze?
21:03:28 lyarwood ah wait that isn't for a while
21:03:44 lyarwood why did I think that was at M2
21:03:56 artom Lib freeze is normally later, no?
21:04:51 artom Ah, it's the week before M3 actually
21:06:03 artom Should un-WIP it then
21:06:10 artom Though the CI thing is still baffling
21:06:17 artom Where do I even start lookingÉ
21:06:18 artom ?
21:06:45 lyarwood yeah iirc we don't pull libs into the tox envs for func and unit so you need an actual release to start using them there
21:06:59 lyarwood depends-on libs that is
21:06:59 openstackgerrit Artom Lifshitz proposed openstack/os-traits master: Add COMPUTE_SOCKET_PCI_NUMA_AFFINITY trait https://review.opendev.org/c/openstack/os-traits/+/771705
21:07:45 lyarwood just need to finish the machine type stuff first that I've had to rewrite most of today after finding the hyperv driver uses hw_machine_type as well /o\
21:07:47 artom lyarwood, release or just landed in master?
21:07:59 lyarwood artom: release and bump requirements I think
21:10:26 openstackgerrit Artom Lifshitz proposed openstack/nova master: libvirt: start tracking NUMACell.socket for hosts https://review.opendev.org/c/openstack/nova/+/766816
21:10:26 openstackgerrit Artom Lifshitz proposed openstack/nova master: extra specs/image props: add `socket` PCI NUMA affinity https://review.opendev.org/c/openstack/nova/+/772748
21:10:27 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add `socket` PCI NUMA affinity policy request prefilter https://review.opendev.org/c/openstack/nova/+/772749
21:10:28 openstackgerrit Artom Lifshitz proposed openstack/nova master: Track host NUMA topology in PCI manager https://review.opendev.org/c/openstack/nova/+/774149
21:10:28 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: pci: implement the `socket` NUMA affinity policy https://review.opendev.org/c/openstack/nova/+/772779
21:34:17 gmann gibi: melwitt stephenfin multiple network fix is merged, you can recheck failure if there is any https://review.opendev.org/c/openstack/tempest/+/770169
21:34:41 gmann let's see if we see any other test failing for that error.
21:36:43 lyarwood \o/
21:46:19 openstackgerrit Ghanshyam proposed openstack/nova master: DNM:try l-c with direct deps https://review.opendev.org/c/openstack/nova/+/772780
22:09:23 melwitt thanks gmann
23:12:24 openstackgerrit Merged openstack/nova master: Add requested_networks field to RequestSpec object https://review.opendev.org/c/openstack/nova/+/749977
#openstack-nova - 2021-02-10
00:36:28 openstackgerrit Ghanshyam proposed openstack/nova master: DNM:try l-c with direct deps https://review.opendev.org/c/openstack/nova/+/772780
00:55:45 openstackgerrit Merged openstack/placement master: policy: Deprecate 'admin_api' rule https://review.opendev.org/c/openstack/placement/+/772334
00:56:08 openstackgerrit Merged openstack/placement master: policy: Add note about keystone's expansion of roles https://review.opendev.org/c/openstack/placement/+/772752
07:14:01 gibi gmann: awesome, thanks
09:15:16 stephenfin Can someone hit this RBAC release note before I shove that context out of my brain? https://review.opendev.org/c/openstack/placement/+/774654/
09:22:46 kashyap lyarwood: Hmm, I'm hitting this one on F32 DevStack - https://bugs.launchpad.net/devstack/+bug/1885198
09:22:47 openstack Launchpad bug 1885198 in devstack "table `broute' is incompatible, use 'nft' tool" [Undecided,New]
09:28:59 openstackgerrit Vlad Gusev proposed openstack/nova stable/stein: Avoid lazy-loading instance.flavor in cold migration https://review.opendev.org/c/openstack/nova/+/774796
09:37:52 lyarwood kashyap: the failure was before that,
09:37:56 lyarwood 2021-02-09 18:08:18.069 | Cloning into '/home/stack/src/cloud/nova'...
09:37:58 lyarwood 2021-02-09 18:11:10.431 | error: RPC failed; curl 18 transfer closed with outstanding read data remaining
09:38:07 kashyap lyarwood: Yeah, just noticed the 'curl' thing; ugh
09:38:17 lyarwood kashyap: why is it cloning into /home/stack/src/cloud/nova?
09:38:37 kashyap lyarwood: Because I used a custom data dir out of habit
09:40:45 lyarwood stephenfin: did you want to add a link as suggested by lbragstad ?
09:41:23 kashyap lyarwood: But it's not the dir that is reason; still looking why the 'curl' failed - perhaps my poor n/w succumbed to a Nova repo clone
09:48:19 stephenfin lyarwood: ah, I hadn't seen that. Looking
09:55:19 gibi stephenfin: regarding the RBAC; the patch looks good to me. What do you want to do with Lance's comment?
09:55:38 stephenfin I'm reworking it now, but perhaps I can use a follow-up?
09:55:50 stephenfin Or not
09:55:52 stephenfin It'll just take me a bit to write this
09:56:00 stephenfin there's a lot of policies to account for :)
09:56:03 stephenfin *there are
09:56:06 gibi stephenfin: do the write up and then ping me, I will push the patch through
09:56:10 stephenfin (y)
09:56:24 gibi thanks
10:07:32 kashyap lyarwood: Now it fails by trying not find a tenant n/w -- http://paste.openstack.org/show/802501/
10:08:39 lyarwood kashyap: how many nics does your vm have?
10:08:58 kashyap lyarwood: Just one; shall I add one more? /me looks at his working older DevStack setup
10:09:30 lyarwood yeah I've always had two, I assume that's why it's unhappy
10:10:27 kashyap lyarwood: Gah. But ... in the name Zeus, I got it working w/ one NIC on an F31 deployment
10:10:33 kashyap Lemme try; sorry for the noise
10:25:27 openstackgerrit Merged openstack/placement stable/ussuri: Fix Placement Doc https://review.opendev.org/c/openstack/placement/+/762607
10:25:34 openstackgerrit Merged openstack/placement stable/train: Fix Placement Doc https://review.opendev.org/c/openstack/placement/+/762608
11:10:49 openstackgerrit Stephen Finucane proposed openstack/placement master: policy: Add releasenote for RBAC work https://review.opendev.org/c/openstack/placement/+/774654
11:10:49 openstackgerrit Stephen Finucane proposed openstack/placement master: Implement secure RBAC for reshaper https://review.opendev.org/c/openstack/placement/+/774827
11:11:01 stephenfin gibi, lyarwood: ^

Earlier   Later