| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-16 | |||
| 17:45:54 | superdan | maybe I was just psychic | |
| 17:46:22 | mriedem | i.e. it's easy to copy the ovs multinode job, tweak for lb, and run it through ci until it works | |
| 17:46:27 | mriedem | not like i'm spending a lot of time on it | |
| 17:47:03 | mriedem | add PHYSICAL_NETWORK=default back in | |
| 17:47:05 | mriedem | *adding | |
| 17:55:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add a note in DELETE /os-services about deleting computes https://review.openstack.org/553598 | |
| 18:00:05 | jgwentworth | superdan, mriedem, leakypipes, fried_rolls, anyone else interested: here's a really rough draft on runways, please add comments and questions and if we can polish it a bit, I can send it out to the dev ML to start using the process https://etherpad.openstack.org/p/nova-runways-rocky | |
| 18:00:39 | jgwentworth | noting that this is going to be really not perfect and we just want to start with something. I should probably write that on the etherpad too | |
| 18:58:00 | fried_rolls | superdan: Gah, I thought I did that before I went to roll, sorry about that. Done. | |
| 18:58:21 | superdan | fried_rolls: thanks | |
| 19:02:18 | fried_rice | superdan, cdent: and by the way, here's my contribution: https://youtu.be/RYwgtJ7kJTI?t=130 | |
| 19:03:22 | superdan | fried_rice: now that's just silly | |
| 19:03:55 | fried_rice | Well, compared to the high cinema you presented, of course. | |
| 19:05:11 | fried_rice | superdan: I think the volume fails will continue to fail until https://review.openstack.org/#/c/553794/ merges? | |
| 19:05:43 | superdan | fried_rice: intermittently I think | |
| 19:06:03 | fried_rice | mm. Well, I was gonna wait to recheck mine, which is underneath his, so nyah. | |
| 19:06:25 | superdan | well, whatever, I was trying to help | |
| 19:06:32 | fried_rice | :) | |
| 19:06:39 | superdan | gonna go cry in my corner now | |
| 19:07:21 | fried_rice | What does that make the score? efried 1, dansmith 10000? | |
| 19:07:34 | fried_rice | I'm making my comeback | |
| 19:07:40 | mriedem | gah we don't have a Dict<?> field? | |
| 19:07:59 | superdan | mriedem: DictOfStrings | |
| 19:08:10 | mriedem | yeah but i've got a big ol bag of goodies | |
| 19:08:16 | mriedem | i know what to do, | |
| 19:08:17 | superdan | mriedem: a generic dict just makes it easy to throw complex types in and call it versioned | |
| 19:08:18 | mriedem | i'm just complaining | |
| 19:08:21 | mriedem | i know | |
| 19:08:31 | mriedem | bag-o-dicts | |
| 19:08:55 | mriedem | *space | |
| 19:09:01 | mriedem | my typing has been really terrible lately | |
| 19:09:06 | mriedem | talking to sean-k-mooney too much | |
| 19:21:14 | mriedem | same g-api failed to start error in that job http://logs.openstack.org/08/553608/4/check/neutron-tempest-linuxbridge/0a5acff/logs/subnode-2/devstacklog.txt.gz#_2018-03-16_19_15_53_511 | |
| 19:24:41 | mriedem | looking at devstack-gate, when it sets up multinode connectivity between the two hosts, it looks like it runs some ansible playbooks and those all assume ovs | |
| 19:25:27 | superdan | mriedem: assume how? | |
| 19:26:21 | mriedem | https://github.com/openstack-infra/devstack-gate/tree/master/playbooks/roles | |
| 19:29:20 | mriedem | although it looks like those are only used for nova-net multinode | |
| 19:29:24 | mriedem | or if using dvr | |
| 19:29:27 | mriedem | which this job isn't | |
| 19:38:08 | mriedem | the subnode code is trying to hit g-api on the primary node, not the subnode where g-api is also running | |
| 19:48:44 | superdan | aaaand, etherpad discussions fail. | |
| 19:49:40 | superdan | fried_rice: we might need a way to draw a cut line on a long series and say "this is what we expect to get done during this window".. it'd be nice if that was always a whole blueprint, but maybe that's not always realistic | |
| 19:50:05 | fried_rice | superdan: Yes, exactly so. | |
| 19:50:28 | fried_rice | For two reasons: 1) reviewer bandwidth, 2) reasonable expectation to have the whole series completely ready at the same time. | |
| 19:50:56 | fried_rice | There was a time when the upt series was over a dozen patches tall. | |
| 19:50:58 | superdan | entirely possible that early patches will be reviewed to oblivion making later patches irrelevant until substantially changed | |
| 19:51:05 | fried_rice | Yes, that also. | |
| 19:51:20 | superdan | well, the point of this is to maintain focus so we can chew through long series easier, | |
| 19:51:43 | superdan | so maybe it just needs to be "have approximately two weeks of stuff to review at the head of your series" | |
| 19:51:53 | superdan | nobody will ever be able to measure that of course | |
| 19:52:04 | fried_rice | No, but we'll get better at it with practice. | |
| 19:52:06 | superdan | and hopefully most blueprints will wholly fit into two weeks, but.. | |
| 19:52:10 | fried_rice | "Bottom N patches of bp X" | |
| 19:52:18 | fried_rice | kind of thing. | |
| 19:52:23 | mriedem | it's likely a discussion that happens when talking about the next thing to go into a slot, | |
| 19:52:27 | superdan | well, aim for whole blueprints, but for special problem children like you... | |
| 19:52:31 | mriedem | for single-patch small bp's, it's not an issue | |
| 19:52:56 | superdan | yeah | |
| 19:53:32 | mriedem | "is bp x ready for a slot? if so, are you going to be around for the next 2 weeks to respond to reviews? if so, i see that it's 20 patches already, what is the reasonable target?" | |
| 19:54:38 | mriedem | this shouldn't be all that process heavy | |
| 19:54:51 | mriedem | says the guy that loves process | |
| 19:54:55 | superdan | yeah for serious | |
| 19:55:11 | mriedem | hey, | |
| 19:55:13 | superdan | I expect we wing this for a cycle, and decide either to scrap it, or it becomes clear what some of the rules should be | |
| 19:55:16 | mriedem | when it's not *my* process i don't care as much | |
| 19:55:20 | superdan | heh | |
| 19:57:00 | jgwentworth | I'll ping gibi_ and bauzas next week to add their feedback to this too | |
| 19:58:17 | jgwentworth | since they're on PTO this week | |
| 19:58:38 | superdan | jgwentworth: might want to clean up the pad a bit and move discussion to the bottom or something | |
| 19:58:39 | jgwentworth | and it's friday late for them | |
| 19:58:42 | superdan | it's a fairly confusing read right now | |
| 19:59:09 | fried_rice | I'm done mucking with it; have at r. | |
| 19:59:09 | fried_rice | ++ | |
| 19:59:12 | superdan | a summary at the top or somethimg | |
| 19:59:27 | jgwentworth | yeah, I was debating whether to copy-paste the whole thing into a -discussion etherpad (should have done that first) and leave this as the trimmed down thing | |
| 19:59:48 | fried_rice | jgwentworth: I meant to mention, wouldn't mind if the definition was the first thing. | |
| 19:59:55 | fried_rice | ("What is a runway?") | |
| 20:00:01 | jgwentworth | oh, yeah, okay | |
| 20:15:20 | openstackgerrit | Merged openstack/nova master: Update contributor guide for Rocky https://review.openstack.org/550288 | |
| 20:15:30 | openstackgerrit | Merged openstack/nova master: Removed unnecessary parantheses in yield statements https://review.openstack.org/550379 | |
| 20:38:51 | mriedem | cfriesen: any idea if you can live migrate between compute hosts with different libvirt virt_types today? like qemu->kvm and back? | |
| 20:41:32 | mriedem | that pair might work, but i'm pretty sure something like kvm->xen wouldn't | |
| 20:52:40 | jgwentworth | superdan: this is a patch having to do with the hypervisor_type in the libvirt driver. it looked reasonable to me but I wasn't 100% sure whether filtering hypervisor_type by virt_type changes the semantics of hypervisor_type/capabilities. I'd appreciate your input about it https://review.openstack.org/#/c/531347/ | |
| 21:10:52 | oomichi | hi, I have a question about Date(2012-09-27), Version(2012.1) on OpenStack Doc like https://docs.openstack.org/nova/latest/cli/nova-api-metadata.html | |
| 21:11:34 | oomichi | They make me feel the doc seems super old, I am not sure we need to update or remove them | |
| 21:12:14 | mriedem | oomichi: those have always annoyed me too | |
| 21:12:38 | oomichi | mriedem: hehe, let me check the other projects too | |
| 21:12:46 | mriedem | i'm not sure if they are needed for man page builds | |
| 21:12:51 | mriedem | or were just originally informational | |
| 21:13:37 | mriedem | https://docs.openstack.org/cinder/latest/man/cinder-manage.html | |
| 21:13:45 | mriedem | cinder doesn't have the version or date in theirs | |
| 21:14:07 | oomichi | mriedem: cool, thanks for picking it up | |
| 21:14:32 | oomichi | it would be nice to send a mail to get answer from doc team | |
| 21:14:34 | cfriesen | mriedem: I've never tried live migration between different libvirt types...I'd expect it to fail. | |
| 21:14:37 | oomichi | will do that | |
| 21:15:02 | mriedem | apparently there is a whole spec http://man7.org/linux/man-pages/man7/man-pages.7.html | |
| 21:15:15 | mriedem | but i'm sure we don't follow the actual man page spec closely | |
| 21:15:24 | oomichi | btw cinder's "Manual group:cloud computing" seems a little odd | |
| 21:15:35 | mriedem | likely copied from nova | |
| 21:15:38 | mriedem | when cinder was split out | |