| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-10 | |||
| 06:54:03 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix keypair API parameters https://review.openstack.org/420626 | |
| 06:55:32 | openstackgerrit | Wei Cao proposed openstack/nova master: vmware: Change the VM Create spec https://review.openstack.org/444119 | |
| 06:59:44 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix keypair API parameters https://review.openstack.org/420626 | |
| 07:21:06 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436 | |
| 07:21:13 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738 | |
| 08:07:45 | openstackgerrit | Michael Henkel proposed openstack/nova master: Implements: blueprint opencontrail-dpdk-support https://review.openstack.org/441183 | |
| 08:23:34 | openstackgerrit | Tomi Juvonen proposed openstack/nova-specs master: Planned host maintenance https://review.openstack.org/310510 | |
| 08:26:19 | bauwser | good morning Nova | |
| 08:27:40 | arne_r | good morning bauwser :) | |
| 08:59:12 | openstackgerrit | Huan Xie proposed openstack/nova master: WIP: add linux bridge qbr for live migration on XenServer https://review.openstack.org/444163 | |
| 09:37:58 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/435954 | |
| 09:49:30 | openstackgerrit | Michael Henkel proposed openstack/nova master: Implements: blueprint opencontrail-dpdk-support https://review.openstack.org/441183 | |
| 10:26:38 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove duplicate attributes in sample files https://review.openstack.org/444199 | |
| 10:27:29 | gmann | alex_xu: oomichi ^^ easy one | |
| 10:38:34 | kashyap | mdbooth: Hello there, wonder if you have time / energy today to make any comment here -- https://review.openstack.org/#/c/246910/ | |
| 10:38:50 | kashyap | I know this involves Neutron voodoo, so if you shy away from it with a "meh", can understand :-) | |
| 10:39:14 | kashyap | Just this is causing a nasty interaction bug between Nova, Neutron, libvirt, QEMU and Virtio code | |
| 10:43:09 | mdbooth | kashyap: Eww | |
| 10:43:23 | kashyap | :-) | |
| 10:43:24 | kashyap | Sorry | |
| 10:43:34 | mdbooth | To me that just says the flow is wrong and we should be casting | |
| 10:43:38 | mdbooth | With callbacks | |
| 10:43:42 | mdbooth | if necessary | |
| 10:44:12 | mdbooth | kashyap: I don't feel I have the bandwidth to jump into that meaningfully right now, I'm afraid | |
| 10:44:33 | kashyap | mdbooth: No worries at all | |
| 10:44:38 | mdbooth | At first glance I'm not a fan, but I don't have an alternative | |
| 10:45:23 | kashyap | Yeah, I hear you. It's just feels (to me) impenetrable sometimes, with so many moving parts | |
| 10:46:15 | markus_z | What's our current policy for images in our docs? I see different types of image sources (*.dia, *.graphml, *odp) and also image generators like "graphviz". | |
| 10:46:27 | mdbooth | There are non-neutron reasons for not using a call in that flow | |
| 11:03:06 | gmann | sfinucan: if you can fix missing 's' on this - https://review.openstack.org/#/c/430914/4 | |
| 11:34:45 | openstackgerrit | Huan Xie proposed openstack/nova master: WIP: Create ovs port with other params together https://review.openstack.org/444230 | |
| 12:04:05 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Deprecate the Cinder API v2 support https://review.openstack.org/444253 | |
| 12:04:41 | andreaf | hello nova folks, https://bugs.launchpad.net/nova/+bug/1532809 pop up again in the gate, it's now the top recheck issue: http://status.openstack.org/elastic-recheck/ | |
| 12:04:41 | openstack | Launchpad bug 1532809 in OpenStack Compute (nova) liberty "Gate failures when DHCP lease cannot be acquired" [High,In progress] - Assigned to Sean Dague (sdague) | |
| 12:07:54 | andreaf | uhm - it's only on openstack/networking-odl | |
| 12:17:41 | alex_xu | gmann: thanks, just done | |
| 12:27:27 | steveq | hi all, I want to test reproducethis bug, https://bugs.launchpad.net/nova/+bug/1481262, is there any way where I can create 100K instances without invoking boot api? | |
| 12:27:27 | openstack | Launchpad bug 1481262 in OpenStack Compute (nova) ""nova usage" taking too much time with many VMs in database" [Medium,In progress] - Assigned to Guillaume Espanel (guillaume-espanel) | |
| 12:27:53 | steveq | s/test reproducethis/reproduce this | |
| 12:55:59 | avolkov | https://docs.openstack.org/ocata/config-reference/compute/config-options.html looks outdated for live_migration_uri option | |
| 12:56:19 | avolkov | could somebody point me from where it's generated? | |
| 13:00:51 | markus_z | avolkov: It gets pulled from these string here. https://github.com/openstack/nova/blob/master/nova/conf/libvirt.py | |
| 13:04:51 | avolkov | markus_z: ok, maybe it takes only first paragraph for description, but live_migration_scheme is missed in docs but exists in libvirt.py, do you know why? | |
| 13:57:57 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Deprecate the Cinder API v2 support https://review.openstack.org/444253 | |
| 14:15:35 | mriedem | o/ | |
| 14:18:02 | Dinesh_Bhor | mriedem: Hi, if you get time could you please check this https://review.openstack.org/#/c/441750/? | |
| 14:35:14 | mriedem | dansmith: i think we're happy with this now https://review.openstack.org/#/c/437222/ - the notification delete patch | |
| 14:40:58 | ioni | hello guys | |
| 14:41:22 | ioni | sorry for bothering you, i have noticed something and I don't know if is a bug in nova or a bug in documentation | |
| 14:41:30 | ioni | https://docs.openstack.org/ocata/config-reference/compute/config-options.html | |
| 14:41:44 | ioni | is mentioned that notify_on_state_change is in [DEFAULT] | |
| 14:42:11 | ioni | while looking at the logs, i have noticed that notifications.notify_on_state_change = None | |
| 14:42:44 | ioni | the only way to set this up is to create a new [notifications] section, having notify_on_state_change = vm_and_task_state | |
| 14:42:57 | ioni | bug in nova or bug in documentation? | |
| 14:45:27 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/435954 | |
| 14:47:32 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Websockify security proxy framework https://review.openstack.org/443234 | |
| 14:54:59 | figleaf | ioni: that's a doc bug | |
| 14:55:21 | figleaf | are you running ocata? | |
| 14:55:45 | ioni | figleaf, yes | |
| 14:57:04 | figleaf | ioni: https://github.com/openstack/nova/blob/master/nova/conf/notifications.py#L108-L110 | |
| 14:57:39 | figleaf | ioni: that shows the opts defined above it, including notify_state_on_change, as being part of the notifications_group | |
| 14:57:41 | mriedem | ioni: are you sure that it wasn't deprecated in the DEFAULT group in ocata and still pass through to [notifications]? | |
| 14:57:59 | mriedem | we generally deprecate the group for a release when moving them | |
| 14:58:04 | mriedem | so the old config will work | |
| 14:58:11 | mriedem | so it depends on when using it in DEFAULT was deprecated | |
| 14:58:29 | ioni | mriedem, i'm looking at: https://docs.openstack.org/ocata/config-reference/compute/config-options.html | |
| 14:58:38 | mriedem | https://docs.openstack.org/developer/nova/ocata/sample_config.html | |
| 14:58:46 | mriedem | yeah, the docs.o.o might be wrong | |
| 14:58:50 | mriedem | ^ is generated from the nova repo | |
| 14:58:55 | ioni | i don't know what's in the code, i'm just looking at them :D | |
| 14:59:08 | mriedem | Deprecated group/name - [default]/notify_on_state_change | |
| 14:59:37 | ioni | ok | |
| 14:59:42 | mriedem | https://github.com/openstack/nova/blob/stable/ocata/nova/conf/notifications.py#L33 | |
| 14:59:52 | mriedem | so if you have it in [DEFAULT] in ocata it should still work | |
| 14:59:55 | mriedem | you might get a warning | |
| 15:00:01 | ioni | it doesn't work | |
| 15:00:08 | ioni | no warning, it defaults to None | |
| 15:00:08 | mriedem | sfinucan: does deprecated_group='default' work for [DEFAULT]? | |
| 15:00:19 | mriedem | dims_: ^? | |
| 15:00:20 | openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | |
| 15:00:20 | openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | |
| 15:00:21 | openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | |
| 15:00:22 | openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | |
| 15:00:23 | openstackgerrit | Dan Smith proposed openstack/nova master: Add identity helper property to CellMapping https://review.openstack.org/444038 | |
| 15:00:24 | openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | |
| 15:00:24 | openstackgerrit | Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | |
| 15:01:09 | mriedem | ioni: let me try something quick in ocata | |
| 15:01:43 | sfinucan | mriedem: I think macsz tried that and it does | |
| 15:02:45 | mriedem | ok i'm changing a unit test in ocata that sets it into the 'notifications' group, putting it back in DEFAULT and seeing if that passes | |
| 15:03:14 | mriedem | oslo_config.cfg.NoSuchOptError: no such option notify_on_state_change in group [DEFAULT] | |
| 15:04:53 | mriedem | changing deprecated_group="DEFAULT" | |
| 15:05:33 | ioni | mriedem, https://paste.xinu.at/VmpI/ | |
| 15:05:58 | ioni | mriedem, so in the first two lines, i had notify_on_state_change in [DEFAULT] | |
| 15:06:08 | ioni | no warning, nothing related to that | |
| 15:06:42 | mriedem | this doesn't work for me either https://gist.github.com/mriedem/d20978819dbbc28c77f89e118fd4f52f | |
| 15:06:52 | mriedem | i get oslo_config.cfg.NoSuchOptError: no such option notify_on_state_change in group [DEFAULT] | |
| 15:06:55 | sfinucan | mriedem: AFAIK the deprecated_group and deprecated_name properties only affects reading from config files. When accessing it programatically (cfg.CONF) you need to use the current group and name | |
| 15:06:57 | mriedem | might be a problem in the unit tests | |
| 15:07:04 | mriedem | sfinucan: ah | |
| 15:07:29 | sfinucan | which, imo, is the correct behavior | |