| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 13:51:05 | bauzas | but that's tied to the grenade issue | |
| 13:51:31 | bauzas | maybe an hangout could help with folks around | |
| 13:51:52 | bauzas | but we would be missing jaypipes and matt | |
| 13:51:57 | bauzas | :( | |
| 13:52:04 | bauzas | in the meantime, I can do the naming thing | |
| 13:52:33 | cdent | bauzas, johnthetubaguy : I'm not yet caught up on that review, but will be within about two hours. Will try to add my perspective, if it differs from anything already there. | |
| 13:52:51 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441 | |
| 13:53:05 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | |
| 13:53:10 | bauzas | cdent: refrain your pencil, I have a patch to do for modifying the interface name | |
| 13:53:26 | cdent | ✔ | |
| 13:53:34 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [CLI] Fix token auth type https://review.openstack.org/424256 | |
| 13:54:22 | andreykurilin | bauzas: hi! please look at https://review.openstack.org/#/c/419441 it is critical fix for several projects | |
| 13:59:43 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | |
| 13:59:43 | takashin | bauzas: Could you review https://review.openstack.org/#/c/408955/ again and check my comments? | |
| 13:59:53 | tdurakov | paul-carlton2 eliqiao shaohe_feng pkoniszewski yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov andreas_s abhishekk mrhillsman raj_singh wznoinsk siva_krishnan hi folks, live migration subteam meeting in a minute | |
| 14:01:17 | takashin | bauzas: Are you around? | |
| 14:01:34 | johnthetubaguy | bauzas: I think I don't really understand fully what the current proposal for update is / means, so its hard to comment on it. | |
| 14:02:14 | bauzas | johnthetubaguy: that's why I think an hangout could be nice | |
| 14:02:28 | takashin | bauzas: Could you review https://review.openstack.org/#/c/408955/ again and check my comments? | |
| 14:02:49 | bauzas | johnthetubaguy: so you could give your insight once knowing all | |
| 14:03:05 | johnthetubaguy | bauzas: yeah, that could work | |
| 14:03:06 | bauzas | johnthetubaguy: and honestly, I tend to listen to your voice for operating insights :) | |
| 14:03:24 | bauzas | takashin: yup, in my queue, I haven't forgotten | |
| 14:03:34 | bauzas | just feeling the FF breeze chilling me | |
| 14:03:50 | takashin | bauzas: Thank you. | |
| 14:04:24 | openstackgerrit | Radomir Dopieralski proposed openstack/python-novaclient: ListExtResource given in place of ListExtManager https://review.openstack.org/424570 | |
| 14:04:58 | bauzas | unrelated, having the UTs running with osprofiler anytime seems sooo wrong to me | |
| 14:05:05 | bauzas | DinaBelova: around ? | |
| 14:05:26 | bauzas | DinaBelova: why are we needing to verify osprofiler anytime we run U | |
| 14:05:28 | bauzas | UTs? | |
| 14:05:40 | bauzas | can't it be part of the regular unittest job? | |
| 14:06:06 | bauzas | as a developer, I don't really need to verify that anytime I'm modifying an internal change | |
| 14:06:15 | bauzas | that's... silly | |
| 14:06:31 | bauzas | but I truly understand it could be part of the UT coverage | |
| 14:06:41 | bauzas | or a specific job | |
| 14:11:28 | jsheeren | hi all, the default flavors in mitaka and earlier had a naming like m1.small etc, what is the origin of this? to be similar to amazon? | |
| 14:12:54 | andreykurilin | ACHTUNG! Nova's intergration with Horizon, Trove, Murano-dashboard, Cinder(?!) is broken - https://review.openstack.org/#/c/419441 | |
| 14:13:18 | openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | |
| 14:13:28 | andreykurilin | jaypipes ^ | |
| 14:14:17 | bauzas | andreykurilin: on it | |
| 14:14:27 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_attach notification https://review.openstack.org/401992 | |
| 14:14:33 | andreykurilin | bauzas: thanks :) | |
| 14:14:40 | bauzas | andreykurilin: AFAIK, it's impacting TripleO as well, so I'd recommend to tag the bug as critical | |
| 14:15:06 | andreykurilin | bauzas: done | |
| 14:17:39 | bauzas | andreykurilin: diana_clarke: there is a follow-up for fixing CLI, correct? | |
| 14:17:47 | diana_clarke | bauzas: yup | |
| 14:17:52 | andreykurilin | bauzas: https://review.openstack.org/#/c/424256 | |
| 14:17:58 | bauzas | okay, let's merge the first one and see the CLI then | |
| 14:18:06 | andreykurilin | bauzas: it is broken for more than a year :( | |
| 14:18:17 | bauzas | diana_clarke: could you please mark your change as Related-bug ? | |
| 14:18:47 | diana_clarke | ^ andreykurilin | |
| 14:19:02 | diana_clarke | (I don't have a change to mark as related) | |
| 14:19:21 | andreykurilin | bauzas: I think we do not have a right bug-report for CLI, but I can create it for proper history | |
| 14:19:39 | bauzas | andreykurilin: oh wait, the bug is 6 month old ? | |
| 14:20:06 | bauzas | if so, that's definitely not a critical :p | |
| 14:20:10 | andreykurilin | lol | |
| 14:20:12 | andreykurilin | bauzas: token auth via python API is broken since 7.0.0 | |
| 14:20:20 | andreykurilin | it is just several weeks | |
| 14:20:48 | openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect job completion https://review.openstack.org/399735 | |
| 14:21:02 | andreykurilin | bauzas: keystoneauth1 provides helpers for CLI layer and usage of it was broken from the beggining (~1 year) | |
| 14:21:52 | johnthetubaguy | andreykurilin: had some concerns with the CLI patch, not sure if I am correct though | |
| 14:22:51 | johnthetubaguy | bauzas: andreykurilin: when is release freeze for the client though? | |
| 14:23:46 | johnthetubaguy | I think its this week | |
| 14:24:03 | bauzas | johnthetubaguy: it was last week | |
| 14:24:13 | bauzas | hence my concern about the prio of the bug | |
| 14:24:15 | johnthetubaguy | wasn't that non-clients | |
| 14:24:19 | bauzas | oh wait | |
| 14:24:23 | bauzas | I guess you're right | |
| 14:24:26 | bauzas | sec, verifying | |
| 14:24:29 | johnthetubaguy | seems like we just released something that is really quite broken | |
| 14:24:40 | johnthetubaguy | and it used to work in the previous "stable" release | |
| 14:24:47 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_attach notification https://review.openstack.org/401992 | |
| 14:24:47 | bauzas | Jan 23 - Jan 27 | |
| 14:24:53 | bauzas | https://releases.openstack.org/ocata/schedule.html | |
| 14:24:59 | johnthetubaguy | yeah, thats how I read it too | |
| 14:25:00 | bauzas | we're all good | |
| 14:25:07 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 14:25:46 | johnthetubaguy | bauzas: it does feel quite urgent to me | |
| 14:26:32 | bauzas | johnthetubaguy: agreed, just wanted to understand the problem statement | |
| 14:26:45 | bauzas | and why it's broken since weeks | |
| 14:26:58 | bauzas | the bug is open since early January | |
| 14:27:15 | diana_clarke | bauzas: because it wasn't used by other projects until u-c was bumped, IIUC. | |
| 14:27:24 | bauzas | when was the bump ? | |
| 14:27:30 | diana_clarke | bauzas: 19th | |
| 14:27:44 | bauzas | diana_clarke: ah ok | |
| 14:27:45 | diana_clarke | bauzas: https://review.openstack.org/#/c/414170/ | |
| 14:27:47 | bauzas | that explains more | |
| 14:28:19 | johnthetubaguy | yeah, I was just checking that | |
| 14:28:27 | johnthetubaguy | that makes sense | |
| 14:30:50 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 14:31:39 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [CLI] Fix token auth type https://review.openstack.org/424256 | |
| 14:31:49 | andreykurilin | johnthetubaguy: fixed your comment ^ | |
| 14:32:56 | johnthetubaguy | andreykurilin: sorry, I added another comment after I spotted something else | |
| 14:33:05 | johnthetubaguy | around the domain being optional | |
| 14:33:25 | andreykurilin | johnthetubaguy: I left an answer to it :) | |
| 14:33:44 | andreykurilin | johnthetubaguy: it is optional but has default values | |
| 14:34:38 | andreykurilin | johnthetubaguy: since it exist in both token and password auth methods, it is ok. user_id exists only in password method and in case of token one, it is just filtered and no metter do we have default values or not | |
| 14:34:47 | johnthetubaguy | andreykurilin: doh, I can't read problem, I see your comment now, makes sense | |
| 14:34:59 | johnthetubaguy | s/read problem/read properly/ | |
| 14:35:32 | andreykurilin | johnthetubaguy: :) | |