| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-17 | |||
| 16:49:01 | mriedem | gibi: great, thanks for doing that | |
| 16:49:03 | mriedem | <3 | |
| 16:50:00 | gibi | mriedem: it was a nice challenge :) | |
| 16:50:34 | mriedem | nice to do something other than bw provider i imagine :) | |
| 16:53:30 | gibi | mriedem: yeah, I'm pretty much waiting for the decision which direction to go with the bwm | |
| 17:10:29 | mriedem | watch out | |
| 17:10:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Isolate cell-targeting code in MigrationTask https://review.openstack.org/621310 | |
| 17:10:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012 | |
| 17:10:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Extract compute API _create_image to compute.utils https://review.openstack.org/621311 | |
| 17:10:45 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036 | |
| 17:10:45 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field https://review.openstack.org/614035 | |
| 17:10:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.openstack.org/614353 | |
| 17:10:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.openstack.org/614037 | |
| 17:10:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.openstack.org/631123 | |
| 17:10:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method https://review.openstack.org/621313 | |
| 17:10:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask https://review.openstack.org/627890 | |
| 17:10:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add TargetDBSetupTask https://review.openstack.org/627892 | |
| 17:10:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add CrossCellMigrationTask https://review.openstack.org/631581 | |
| 17:10:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891 | |
| 17:10:56 | cdent | jeebus | |
| 17:11:16 | mriedem | i believe gibi's is more impressive | |
| 17:20:06 | mriedem | dansmith: melwitt: are you aware of any test utilities to compare to objects and ignore some fields? I found nova.tests.objects.test_objects.compare_obj but that takes an object and a primitive and doesn't take a blacklist method for fields to ignore | |
| 17:20:34 | melwitt | I thought we do. /me looks | |
| 17:20:36 | dansmith | hmm, I thought there was one | |
| 17:20:53 | dansmith | you could just del the fields you don't want of course | |
| 17:22:50 | adrianc | sean-k-mooney: ^ | |
| 17:23:11 | adrianc | whoops :) disregard | |
| 17:23:36 | melwitt | mriedem: this is the thing I have used before https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_db_api.py#L1395 | |
| 17:23:53 | mriedem | yeah just found this https://github.com/openstack/nova/blob/master/nova/tests/functional/db/test_console_auth_token.py#L47 | |
| 17:24:13 | mriedem | that db api util is more like what i'm looking for | |
| 17:57:14 | cdent | efried: lgtm | |
| 18:15:13 | openstackgerrit | Maxim Nestratov proposed openstack/nova master: libvirt: Virtuozzo containers config drive support https://review.openstack.org/449818 | |
| 18:20:05 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025 | |
| 19:06:14 | cfriesen | I'm trying to work out the logic in the image-fetching and caching code on the compute node but it's not easy to follow. If the glance image is stored in ceph and nova-compute is using local storage, will the image data be pulled down via the glance API or can nova short-circuit it and pull it in directly from ceph? | |
| 19:10:22 | mriedem | cfriesen: https://review.openstack.org/#/c/572805/ might be related | |
| 19:11:11 | cfriesen | mriedem: how apropos | |
| 19:43:18 | efried | mriedem: So close to using TaskFlow, remind me why we aren't doing that? | |
| 19:44:17 | mriedem | efried: b/c i've never taskflowed? | |
| 19:45:04 | mriedem | learning a new toolset is kind of real low priority when i have a mountain to climb first, but i know | |
| 19:45:12 | efried | I just mean, looking at TaskBase, it's essentially the same as a TaskFlow Task - execute and rollback methods - but we had to write the plumbing for things like doing the actual rollbacks or whatever. | |
| 19:45:39 | mriedem | task flow also has a lot of threading stuff in it though doesn't it? | |
| 19:45:41 | efried | more just wondering why it wasn't used in the first place. I get following momentum | |
| 19:45:50 | efried | you can thread, or not. | |
| 19:45:54 | mriedem | conductor tasks were written long long ago | |
| 19:46:26 | efried | I see TaskBase is from 2015 yeah | |
| 19:47:31 | mriedem | i believe there was also anecdotes (from years ago) when cinder refactored their volume create flow to use taskflow and it was kind of a mess | |
| 19:47:40 | mriedem | *were | |
| 19:48:49 | smcginnis | And then many debates were had on whether to do the work of removing taskflow by those that could no longer figure out what was going on. | |
| 19:48:59 | mriedem | glancing at the taskflow docs, conductor tasks in nova are essentially this https://docs.openstack.org/taskflow/latest/user/patterns.html#module-taskflow.patterns.linear_flow | |
| 19:49:29 | mriedem | yeah, it just seems more complicated than what i need | |
| 19:50:08 | mriedem | i'm sure it's very fancy and all, but fancy is kind of the last thing i need to deal with in the cross-cell resize series | |
| 19:50:40 | mriedem | jroll: ironic has a task engine thing right? does that use taskflow? | |
| 19:50:46 | efried | The fanciest part of it is making the task a class with an execute (and optional rollback) method. You're already doing that, so... | |
| 19:51:16 | mriedem | but but but i might need to raise a pinky when drinking that tea | |
| 19:51:37 | mriedem | anyway, | |
| 19:51:56 | mriedem | what i'd like to do is get an end to end working flow of this (again, since my initial poc with shelve had one) and then iterate on how to fancy it up | |
| 19:51:57 | efried | Yeah, I'm not pushing it, just something I noticed with your *Task patches. | |
| 19:52:00 | mriedem | sure | |
| 19:52:05 | mriedem | i knew you'd bring it up eventually | |
| 19:52:08 | efried | :) | |
| 19:52:09 | mriedem | but it's not what i'm losing sleep over :) | |
| 19:52:15 | efried | no doubt | |
| 19:52:32 | openstackgerrit | Jack Ding proposed openstack/nova master: Flavor extra spec and image properties validation https://review.openstack.org/620706 | |
| 19:52:52 | mriedem | if you're rechecking tempest ssh failures, | |
| 19:53:22 | mriedem | there is a known neutron bug | |
| 19:53:56 | mriedem | https://bugs.launchpad.net/neutron/+bug/1811515 | |
| 19:54:03 | openstack | Launchpad bug 1811515 in neutron "pyroute2.NetNS don't work properly with concurrency in oslo.privsep" [Critical,In progress] - Assigned to Slawek Kaplonski (slaweq) | |
| 19:54:31 | mriedem | luckily https://review.openstack.org/#/c/631275/ | |
| 20:50:16 | melwitt | nova meeting in 10 minutes | |
| 20:55:25 | efried | mriedem: thanks. Some of them are ssh failures, yes. | |
| 21:04:03 | mriedem | melwitt: os-vif was released with that change https://review.openstack.org/#/c/602384/ | |
| 21:04:34 | melwitt | mriedem: ack | |
| 21:05:16 | sean-k-mooney | yes there is a conflict with libvirt | |
| 21:05:29 | sean-k-mooney | os-vif creates teh port then libvirt deletes it and creates it again | |
| 21:05:41 | sean-k-mooney | tests that only check for port active race | |
| 21:05:53 | sean-k-mooney | tetsts taht check the interface is pingable would be fien | |
| 21:06:19 | sean-k-mooney | e.g. if they wait for the vm to be accessabel before sshing then it wont break | |
| 21:07:09 | sean-k-mooney | im looking in to this week | |
| 21:08:59 | mriedem | dansmith: https://bugs.launchpad.net/nova/+bug/1812196 | |
| 21:09:04 | openstack | Launchpad bug 1812196 in OpenStack Compute (nova) "when using templated cells urls, validation fails with create_cell when using {scheme} and {port}" [Undecided,New] | |
| 21:10:12 | dansmith | um | |
| 21:10:28 | dansmith | those shouldn't be parsed by nova-manage | |
| 21:12:18 | dansmith | hrm | |
| 21:12:35 | dansmith | well, that's kindof the problem I guess, but.. hmm | |
| 21:14:27 | dansmith | cripes can this always have been broken with nova-manage? | |
| 21:23:34 | mriedem | i haven't dug into the bug, just noticed it in triage | |
| 21:27:00 | dansmith | yeah, this is a pretty dumb ass mistake.. mah bad | |
| 21:29:47 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix using template cell urls with nova-manage https://review.openstack.org/631649 | |
| 21:29:51 | dansmith | mriedem: ^ we'll want that backported to .. at least rocky, IIRC | |
| 21:41:40 | melwitt | thanks tonyb | |
| 21:43:01 | tonyb | first nova review in $ages | |
| 21:43:26 | melwitt | :) | |
| 21:58:46 | openstackgerrit | Dakshina Ilangovan proposed openstack/nova-specs master: Nova changes to support Local Resource Management that uses Resource Management Daemon https://review.openstack.org/630817 | |
| 22:01:19 | mriedem | dansmith: might have just been a result of https://github.com/openstack/nova/commit/bc6ca87a6a01ecbda926a78d07cadbd4e3ab97e0 not taking into account templated urls | |
| 22:01:27 | mriedem | i'm not sure which came first | |
| 22:01:55 | dansmith | mriedem: nah, mine was after | |
| 22:18:24 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix using template cell urls with nova-manage https://review.openstack.org/631649 | |
| 22:20:06 | mriedem | e-r uncategorized should be fixed, will take time to regen the page | |
| 22:20:09 | mriedem | https://review.openstack.org/#/c/631653/1 | |
| 22:41:23 | mriedem | dansmith: +2 | |