Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-02-09
16:10:36 kashyap I'll report back my result, if it works
16:10:55 sean-k-mooney stephenfin: lets try keeping it for this cycle
16:11:16 sean-k-mooney stephenfin: if it break agian before the end of the cycle we can drop it
16:12:39 sean-k-mooney stephenfin: our lower constratis job is a littel different since we actully run our unit tests and docs
16:14:13 stephenfin lyarwood: Is there any way to map a migration request (via '/servers/{server_id}/action (migrate)') to a migration record ('/servers/{server_id}/migrations')
16:14:31 stephenfin I can't see any. It's weird that the former doesn't return an ID in the success case
16:15:34 sean-k-mooney stephenfin: tthe request id might be usable if you are looking at logs
16:15:58 stephenfin I'm thinking of OSC
16:15:59 stephenfin https://review.opendev.org/c/openstack/osc-lib/+/773877
16:16:21 stephenfin Rather than polling the server status, I'd like to map the migration request to the record as above and check the status of that
16:16:37 stephenfin but there doesn't appear to be any way to do that :-\
16:18:50 sean-k-mooney /servers/{server_id}/migrations
16:18:56 sean-k-mooney so you can list the migrates for a server
16:19:02 sean-k-mooney and get the last one
16:19:25 stephenfin Yeah, I was hoping to avoid that
16:19:42 sean-k-mooney maybe via the server events?
16:19:52 stephenfin Hmm, that could work
16:20:06 stephenfin I think the issue is that the Migration object isn't created until we land on a compute noe
16:20:09 stephenfin *node
16:20:24 stephenfin and we use a cast so can't wait for that
16:20:44 stephenfin sounds like a potential usability RFE for Xena
16:22:33 lyarwood yeah I did wonder why we didn't list the request-id in https://docs.openstack.org/api-ref/compute/?expanded=show-migration-details-detail#show-migration-details tbh
16:22:44 lyarwood so you could then lookup by request-id or something
16:22:58 dansmith stephenfin: we didn't originally expose any details of migrations to users other than a resize I think, which is why we don't have a lot of stuff baked around making that trackable by a user
16:22:59 lyarwood definitely something to improve in X
16:23:02 sean-k-mooney well the issue is that we return nothing for the action
16:23:15 lyarwood we should always have the request-id
16:23:41 dansmith stephenfin: there's also the migration context on the instance, so we can stash things like that there before we create the migration if that helps
16:23:41 sean-k-mooney lyarwood: sure but we dont save request ids normally
16:23:42 dansmith like that being the req-id
16:24:18 sean-k-mooney dansmith: ya we do but there is not realy a good way to acess that form the api
16:24:36 lyarwood well that's my point sean-k-mooney, stashing it in the migration record
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 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:06:59 lyarwood depends-on libs that is
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: extra specs/image props: add `socket` PCI NUMA affinity https://review.opendev.org/c/openstack/nova/+/772748
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: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: WIP: pci: implement the `socket` NUMA affinity policy https://review.opendev.org/c/openstack/nova/+/772779
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: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/

Earlier   Later