| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 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: :) | |
| 14:38:08 | kuzko | hello,I would like to know if there are special command lines or something special to do to allow for UEFI boot of a volume? | |
| 14:39:26 | andreykurilin | bauzas johnthetubaguy: there is one more high priority bug, which affects cinder and horizon(and openstackclient ?!). Fix is almost ready, just require proper release note. I'll try to reach the author of fix to make it asap | |
| 14:39:48 | johnthetubaguy | andreykurilin: cool, thanks for chasing, as always! | |
| 14:39:57 | bauzas | andreykurilin: have you discussed with matt about a release soon ? | |
| 14:40:10 | bauzas | andreykurilin: do you want me to tag the release ? | |