| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 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 ? | |
| 14:41:17 | bauzas | andreykurilin: I mean, I'm theorically the release liaison so release management cores would be okay with me providing this change without matt +1ing it | |
| 14:41:57 | andreykurilin | bauzas: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-01-23.log.html#t2017-01-23T17:31:51 | |
| 14:42:49 | andreykurilin | bauzas: yesturday I informed Matt about an issue and notified him about required release, he gave ack :) | |
| 14:43:22 | openstackgerrit | Radomir Dopieralski proposed openstack/python-novaclient: ListExtResource given in place of ListExtManager https://review.openstack.org/424570 | |
| 14:44:04 | bauzas | andreykurilin: johnthetubaguy: I have a nit about the reno note | |
| 14:44:22 | andreykurilin | bauzas: again. CLI brackage was introduced a year ago | |
| 14:44:24 | andreykurilin | in 2015 | |
| 14:44:25 | andreykurilin | :) | |
| 14:44:32 | bauzas | andreykurilin: I'd prefer if you could respin very quick so I could apply a +2/+W based on john's already giving you +2 | |
| 14:44:50 | andreykurilin | bauzas: I attached a bug report in commit message with link to patch that introduced an issue | |
| 14:44:50 | bauzas | andreykurilin: sure, but the novaclient thing is new | |
| 14:45:23 | bauzas | andreykurilin: please consider application developers that fetch novaclient and look at the release notes to see what changed | |
| 14:45:32 | bauzas | andreykurilin: those people wouldn't read the commit msg | |
| 14:45:52 | bauzas | andreykurilin: I got that the CLI breakage is old, but the novaclient thing is new, right? | |
| 14:46:13 | bauzas | andreykurilin: so, maybe 2 bullets is fine by me, but you should at least explain since when we had that regression | |
| 14:46:25 | andreykurilin | bauzas: usage via python api yes - it is new:) via cli is old. | |
| 14:46:33 | bauzas | or people would say "WAT? I wasn't able to use token auth? seriously?" | |
| 14:47:02 | andreykurilin | We broke CLI in 3.1.0 release of novaclient. lol | |
| 14:47:12 | bauzas | 3.1.0 is not that old | |
| 14:47:20 | bauzas | we have people running it | |
| 14:47:23 | bauzas | that I know | |
| 14:47:25 | andreykurilin | heh | |
| 14:47:27 | dansmith | johnthetubaguy: if you have any time to review the bottom two patches on the cells series I'd appreciate it: https://review.openstack.org/#/c/396775/42 | |
| 14:47:41 | dansmith | johnthetubaguy: those three patches basically get us to our goal for cellsv2 for the cycle | |
| 14:47:43 | bauzas | so, yeah, I think that's important for us to comment when the breakage was introduced | |
| 14:47:47 | andreykurilin | ok. I'll update my patch in seconds to not scare people:) | |
| 14:47:53 | bauzas | dansmith: I haven't forgotten you | |
| 14:48:11 | johnthetubaguy | dansmith: I will take a peak at that now | |
| 14:50:48 | mdbooth | pkoniszewski: Just been reading through the review history on that patch. moshele mentioned the save() issue here: https://review.openstack.org/#/c/244489/46/nova/compute/manager.py | |
| 14:51:31 | mdbooth | My confidence is growing that the rollback issue is real. | |
| 14:51:59 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [CLI] Fix token auth type https://review.openstack.org/424256 | |
| 14:52:06 | andreykurilin | bauzas: ^ | |
| 14:53:24 | pkoniszewski | mdbooth: that's right | |
| 14:53:51 | pkoniszewski | mdbooth: but the pre_live_migration bit is correct | |
| 14:54:40 | mdbooth | Yeah. I didn't see a problem there, btw. | |
| 14:55:55 | mdbooth | I think it's probably futile to try to prevent calling save on the instance object on such a wide ranging function. Perhaps we should just concede it will happen and deal with it? | |
| 14:56:33 | mdbooth | Well, for live migration we *know* it happens, because we do it in compute manager. | |
| 14:57:21 | openstackgerrit | Radomir Dopieralski proposed openstack/python-novaclient: ListExtResource given in place of ListExtManager https://review.openstack.org/424570 | |
| 14:58:26 | andreykurilin | bauzas: and yes, it will be cool, if you propose a new novaclient's release. Not sure about right version- 7.0.1 (since it fixes several issues from 7.0.0) or 7.1.0 (since it contains a bit feauture - support of osprofiler) | |