| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-01 | |||
| 06:07:47 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 06:24:37 | openstackgerrit | Lei Zhang proposed openstack/nova master: placement: extract traits from flavor extra spec https://review.openstack.org/492026 | |
| 06:28:21 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 06:52:58 | openstackgerrit | LiChunlin proposed openstack/nova master: update the description of hypervisor statistics response https://review.openstack.org/462355 | |
| 07:35:06 | openstackgerrit | David Rabel proposed openstack/nova master: Adds support for graceful shutdown for VMware instances https://review.openstack.org/494169 | |
| 07:36:00 | gibi | dansmith: you are right that https://review.openstack.org/#/c/498950 was failed notification tests due to the service version bump | |
| 07:44:51 | openstackgerrit | David Rabel proposed openstack/nova master: VMware: add support for graceful shutdown of instances https://review.openstack.org/494169 | |
| 08:40:04 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451 | |
| 09:08:08 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin https://review.openstack.org/493082 | |
| 09:24:07 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: [WIP][POC] Host plugin information from os-vif https://review.openstack.org/499941 | |
| 09:24:33 | ygl_ | hi all | |
| 09:24:39 | ygl_ | my instance creaiton is failing | |
| 09:24:59 | ygl_ | its is saying "Connection to glance host failed:" | |
| 09:25:05 | ygl_ | but i can list the images | |
| 10:16:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Default the service version in the notification tests https://review.openstack.org/499957 | |
| 10:40:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: doc: fix show-hide sample in notification devref https://review.openstack.org/499963 | |
| 11:33:30 | openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova master: docs: Explain the flow of the "serial console" feature https://review.openstack.org/476188 | |
| 12:15:54 | wlfightup | https://blueprints.launchpad.net/nova/+spec/instance-mem-and-cpu-hot-plugging | |
| 12:36:19 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin https://review.openstack.org/493082 | |
| 12:41:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add '_has_dns_extension' function https://review.openstack.org/480675 | |
| 12:41:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Always refresh cache when checking extensions https://review.openstack.org/480674 | |
| 12:41:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: metadata: Don't use nova-net DNS values with neutron https://review.openstack.org/480616 | |
| 12:41:44 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Store dns_domain value from neutron https://review.openstack.org/480676 | |
| 12:41:44 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add '_has_qos_queue_extension' function https://review.openstack.org/500010 | |
| 12:42:18 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/500011 | |
| 12:47:01 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/500028 | |
| 13:23:50 | openstackgerrit | Eric Fried proposed openstack/nova master: Include /resource_providers/uuid/allocations link https://review.openstack.org/499826 | |
| 13:23:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Make 'get_domain' private https://review.openstack.org/417378 | |
| 13:29:59 | cdent | fried_rice: I’m fairly in favor of making that links change, but I dunno if it will pass the rules | |
| 13:30:22 | fried_rice | cdent Yuh; I'm happy to take my guidance from whoever - but who is that? | |
| 13:30:36 | cdent | I would guess a final arbiter is someone with +2, which is not me, nor figleaf | |
| 13:30:52 | cdent | I’ll go ahead and +1 it to express my opinion | |
| 13:31:30 | fried_rice | Ah, gotcha. Someone like dansmith or leakypipes or mriedem | |
| 13:31:50 | fried_rice | or sdague | |
| 13:32:27 | figleaf | cdent: it's not fixing an error in the sense we intended. It's correcting an oversight | |
| 13:32:43 | mriedem | is it changing the response? | |
| 13:32:52 | figleaf | yes | |
| 13:32:52 | mriedem | if it's changing the response then it's a new microversion | |
| 13:32:56 | cdent | sdague is probably the main arbiter on this kind of thing | |
| 13:33:12 | cdent | mriedem: the squiggle is that we changed the content of that section before without a microversion | |
| 13:33:22 | cdent | thus the lawyering going on now | |
| 13:33:36 | figleaf | two clouds requesting the same microversion would get different data | |
| 13:33:42 | mriedem | past performance is not an indicator of future earnings | |
| 13:34:13 | mriedem | what was the previous non-versioned change? | |
| 13:34:41 | cdent | Yeah, the rules are it must be a new microversion. The reason I’m being squishy about it is just to reinforce that just because I wrote the latest version of the rules doesn’t mean I’m in love with microversions and want to marry them. | |
| 13:34:52 | fried_rice | mriedem https://review.openstack.org/#/c/468923/1/nova/api/openstack/placement/handlers/resource_provider.py | |
| 13:34:53 | cdent | mriedem: it’s linked in fried_rice’s comments | |
| 13:34:54 | cdent | jinx | |
| 13:35:15 | mriedem | let me note that it's too early in my day to have to deal with the age old, "does this require a microversion?" question | |
| 13:35:49 | figleaf | it's the usual problem: a seemingly backwards-compatible change | |
| 13:36:13 | mriedem | so alex_xu and sdague approved https://review.openstack.org/#/c/468923/ | |
| 13:36:20 | mriedem | with no comment on the microversion part of this | |
| 13:36:29 | fried_rice | right | |
| 13:38:10 | cdent | fried_rice: did you find this because you are actually using the links, or just because you were having a poke around. If someone’s using genuine hypertextuality, i’m going to need to sit down. | |
| 13:39:10 | cdent | (somewhat related, I’ve started using genuine hypertextuality in my gabbi tests of placement, just for the sake of it) | |
| 13:39:20 | fried_rice | cdent Poking around. Larnin' up the placement API and resource providers and whatnot. | |
| 13:39:32 | fried_rice | Working towards being able to do generic device management via placement. | |
| 13:39:46 | cdent | Larnin’!!!??? Ain’t no body got time for that. | |
| 13:40:16 | fried_rice | The alternative is putting together a huge pile of incomprehensible sh*t that will waste everybody's time in Denver. | |
| 13:40:36 | fried_rice | This way it gets to be a huge pile of slightly-comprehensible sh*t. | |
| 13:41:19 | kashyap | Huh, the documentation for 'live_migration_scheme' says: "Override the default libvirt live migration scheme", but it doesn't mention _what_ are the possible values? | |
| 13:41:23 | kashyap | From here: https://docs.openstack.org/ocata/config-reference/compute/config-options.html | |
| 13:41:30 | cdent | I hope to introduce you to some vmware folk who are curious about that stuff too, and are also planning to do some funky not-just-for-the-sake-of-nova resource mgt via placement | |
| 13:41:55 | cdent | s/you/fried_rice/ | |
| 13:42:14 | mriedem | kashyap: sec, i have a ML thread link | |
| 13:42:37 | kashyap | mriedem: Ah, I came across it (as I sometimes filter openstack-dev list by your e-mail ID :P) | |
| 13:42:38 | fried_rice | cdent Cool, I've been wanting to talk to VMWare about this. | |
| 13:42:40 | kashyap | Let me go look | |
| 13:42:55 | fried_rice | Cause I think they have some of the same issues as claudiub and I do. | |
| 13:43:09 | fried_rice | that we want to make sure the new solution solves. | |
| 13:43:31 | cdent | fried_rice: yup | |
| 13:43:57 | mriedem | kashyap: http://lists.openstack.org/pipermail/openstack-dev/2017-June/117987.html | |
| 13:44:46 | kashyap | mriedem: BTW, thanks for kickstarting the cleanup of the spaghetti that is LM doc - 9c2518c2 ("Cleanup the plethora of libvirt live migration options") | |
| 13:45:47 | mriedem | yeah it's impossible | |
| 13:46:48 | kashyap | Heh, "landmine of terrible" | |
| 13:47:12 | openstackgerrit | Elod Illes proposed openstack/nova master: Functional test for regression bug #1713783 https://review.openstack.org/500057 | |
| 13:47:13 | openstack | bug 1713783 in OpenStack Compute (nova) pike "After failed evacuation the recovered source compute tries to delete the instance" [High,Triaged] https://launchpad.net/bugs/1713783 | |
| 13:50:00 | kashyap | mriedem: Darn, I missed your e-mail as it got filtered to the 'operators' mail dir. | |
| 13:55:25 | mriedem | elod: thanks for https://review.openstack.org/#/c/500057/ - just a few small things inline | |
| 13:57:55 | elod | mriedem: thanks too, checking | |
| 13:59:03 | kashyap | Although you could figure out for context, but the URL will be useless in future | |
| 13:59:08 | kashyap | s/for/from/ | |
| 14:01:42 | gibi | kashyap: you can have a github link that points to place via a specfic commit hash. That is futureproof | |
| 14:07:07 | mriedem | e.g. https://github.com/openstack/nova/blob/7815108d4892525b0047c787cbd2fe2f26c204c2/nova/virt/libvirt/driver.py#L652 | |
| 14:07:10 | mriedem | like in my email | |
| 14:07:53 | stephenfin | Any chance of a +W on this? Been sitting around for a while https://review.openstack.org/#/c/407055/ | |
| 14:08:19 | kashyap | gibi: Ah, noted | |
| 14:08:35 | mriedem | stephenfin: the change under it has a -1 on it | |
| 14:08:39 | mriedem | so what's the rush | |
| 14:08:42 | mriedem | unless you re-order | |
| 14:08:49 | kashyap | Just reading this one (oh dear, yet more live migration options) - https://review.openstack.org/#/c/456571/ | |
| 14:08:50 | stephenfin | Aha, correct. nvm me | |
| 14:09:08 | stephenfin | My list of open patches has grown rather large/unmanageable :/ | |
| 14:09:19 | kashyap | I know it intends to solve the said valid bug where one can't specify one of the URI variants | |
| 14:09:24 | mriedem | kashyap: the point of that is to remove the URI option and make it specific | |
| 14:09:29 | mriedem | so you can configure the parts of the URI | |
| 14:09:32 | mriedem | right | |
| 14:09:51 | mriedem | live_migration_query_parameters :) | |
| 14:09:58 | kashyap | mriedem: :-) I see the point | |
| 14:10:02 | mriedem | oh i guess that's extra_params | |