| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-16 | |||
| 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 | ++ | |
| 19:59:09 | fried_rice | I'm done mucking with it; have at r. | |
| 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 | |
| 21:15:55 | mtreinish | oomichi, mriedem: it looks like it's just a table in the rst: https://git.openstack.org/cgit/openstack/nova/tree/doc/source/cli/nova-api.rst#n9 | |
| 21:15:57 | smcginnis | Any man page specific stuff probably hasn't been touched since then. | |
| 21:16:53 | mtreinish | the man pages are just sphinx rendering the rst: https://github.com/openstack/nova/blob/master/doc/source/conf.py#L137 with the manpage builder | |
| 21:17:17 | mtreinish | I just spent a bunch of time reformatting: https://stestr.readthedocs.io/en/latest/MANUAL.html to be a proper manpage when using that sphinx builder | |
| 21:17:32 | mtreinish | I can give you a hand with that if it's something you'd want to do | |
| 21:19:20 | oomichi | mtreinish: I might want to remove them (Date, Version) if we can | |
| 21:19:51 | mtreinish | yeah, I don't think that table adds anything. I think just push a patch to delete it, or change it to a comment so it doesn't render in the output | |
| 21:21:44 | oomichi | mtreinish: thanks, let me try | |
| 21:24:32 | openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | |
| 21:24:33 | openstackgerrit | Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 21:24:33 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 21:24:34 | openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | |
| 21:24:34 | openstackgerrit | Eric Fried proposed openstack/nova master: Make generation optional in ProviderTree https://review.openstack.org/539324 | |
| 21:24:35 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728 | |
| 21:26:02 | oomichi | mriedem: mtreinish: cinder team has removes Date and Version on the same reason which we feel: https://review.openstack.org/#/c/325465/ | |
| 21:32:03 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Clean up man page and remove version/date https://review.openstack.org/553903 | |
| 21:33:03 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove version/date from CLI documantation https://review.openstack.org/553903 | |
| 21:33:32 | oomichi | mriedem: ^^^: done | |
| 21:40:02 | mriedem | +2 | |
| 21:40:17 | mriedem | POST_FAILURE on https://review.openstack.org/#/c/553794/ so it's going to have to get rechecked | |
| 21:57:47 | mriedem | cfriesen: found the code that won't allow it | |
| 21:58:09 | mriedem | https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L210 | |
| 21:58:11 | mriedem | in the conductor task | |
| 21:58:18 | mriedem | also checks the version | |
| 22:00:38 | fried_rice | jgwentworth, superdan: Are we rolling out runways yet? I put one in the queue :) | |
| 22:01:46 | oomichi | thanks | |
| 22:01:50 | jgwentworth | fried_rice: heh, you're ready2rock. gonna ping gibi and bauzas on monday and see if they have anything to add/change before unleashing it to the dev ML | |
| 22:02:20 | fried_rice | cool cool. Meanwhile, same patches are in the rocky priorities etherpad, so I figger I'm covered :) | |
| 22:02:27 | jgwentworth | speaking of, while you're here oomichi if you have any feedback about our rough draft about runways for rocky https://etherpad.openstack.org/p/nova-runways-rocky | |
| 22:02:59 | jgwentworth | I'm sorry this hasn't been mentioned yet as I haven't written the PTG summary for the friday miscellaenous bag of topics yet | |
| 22:03:14 | jgwentworth | *miscellaneous | |
| 22:03:54 | jgwentworth | alex_xu_ ^ | |
| 22:04:01 | mriedem | jgwentworth: if you're going to attempt to summarize the friday misc stuff from the ptg, i'd recommend not trying to cover every single item, and only focus on highlights for any bigger or more contentious items | |
| 22:04:10 | mriedem | otherwise the summary will essentually be the etherpad itself | |
| 22:04:24 | jgwentworth | mriedem: yeah, I was wondering about that. thanks for the advice | |
| 22:04:32 | oomichi | jgwentworth: OK, let me check. (that is already huge btw :-) | |
| 22:04:34 | mriedem | i never tried to summarize all the misc stuff before | |
| 22:05:13 | jgwentworth | good to know | |
| 22:08:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548 | |
| 22:08:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add code to bind a port against a dest host during live migration https://review.openstack.org/523604 | |
| 22:08:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | |
| 22:08:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | |
| 22:08:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | |
| 22:08:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | |
| 22:08:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: compute: use port binding extended API during live migration https://review.openstack.org/551371 | |
| 22:08:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | |
| 22:08:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Turn on new port binding extended live migrate flow https://review.openstack.org/552173 | |
| 22:08:45 | jgwentworth | superdan: ty for reviewing the hypervisor_type patch | |
| 22:10:20 | mriedem | superdan: i'll bug you about this on monday https://review.openstack.org/#/c/515423/9/nova/objects/migrate_data.py@186 but i assume it's because the field is defined in the parent class rather than each subclass | |
| 22:10:32 | mriedem | so my option is moving that field into all subclasses | |
| 22:10:39 | mriedem | which is easy and mechanical | |
| 22:18:42 | jgwentworth | TheJulia, jroll, mriedem: here's the nova/ironic PTG summary if you have anything to add/correct before I send it out to the ML https://etherpad.openstack.org/p/nova-ptg-rocky-ironic-summary | |
| 22:33:18 | superdan | jgwentworth: yep | |
| 22:33:27 | superdan | mriedem: left a comment, will help you look on monday | |
| 22:50:42 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove version/date from CLI documantation https://review.openstack.org/553903 | |
| 22:55:31 | openstackgerrit | Dan Smith proposed openstack/nova master: Add --by-service to discover_hosts https://review.openstack.org/552691 | |
| 23:09:00 | openstackgerrit | Merged openstack/nova master: Follow the new PTI for document build https://review.openstack.org/553751 | |
| #openstack-nova - 2018-03-17 | |||
| 00:07:13 | openstackgerrit | Merged openstack/nova master: Change TestNewtonCellsCheck to not rely on objects https://review.openstack.org/553082 | |
| 00:18:16 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/553918 | |
| 00:21:07 | Spaz-Home | Have a good weekend, Novaers | |
| 00:21:13 | Spaz-Home | I shall be here, guarding your seats! | |