| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 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 | bauzas | andreykurilin: sure, but the novaclient thing is new | |
| 14:44:50 | andreykurilin | bauzas: I attached a bug report in commit message with link to patch that introduced an issue | |
| 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) | |
| 14:59:41 | andreykurilin | bauzas johnthetubaguy: it is another high priority fix - https://review.openstack.org/#/c/424570 . just +10, -1 with release notes | |
| 15:00:55 | openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: set vlan tag for macvtap on SR-IOV VFs https://review.openstack.org/421186 | |
| 15:03:45 | dansmith | johnthetubaguy: thanks.. it's going to be a hard push to get this in by thurs, but I'm still trying | |
| 15:05:33 | mriedem | dansmith: don't forget to breath | |
| 15:05:56 | dansmith | mriedem: hoo hoo heee, hoo hoo heee | |
| 15:06:02 | mriedem | exactly | |
| 15:08:47 | andreykurilin | johnthetubaguy: Can you put +2 again to https://review.openstack.org/424256 ?) | |
| 15:12:38 | bauzas | dansmith: who's core in grenade ? | |
| 15:12:53 | dansmith | bauzas: me, sdague and dtroyer | |
| 15:13:05 | bauzas | dansmith: I'm firing a patch for making sure we run the placement-engine in Ocata when upgrading, which is not the case | |
| 15:13:12 | dansmith | ack | |
| 15:13:19 | bauzas | dansmith: http://logs.openstack.org/61/417961/20/check/gate-grenade-dsvm-neutron-ubuntu-xenial/35a48ec/logs/new/local.conf.txt.gz | |
| 15:13:33 | bauzas | dansmith: there is no enabled service that runs the placement engine | |
| 15:13:45 | bauzas | hence the grenade failure on my change | |
| 15:14:01 | dansmith | ack part deux | |
| 15:14:19 | bauzas | so I'm thinking of updating https://github.com/openstack-dev/grenade/blob/master/projects/60_nova/from-newton/upgrade-nova | |
| 15:14:40 | bauzas | dansmith: but any idea how I would do that ? | |
| 15:14:59 | bauzas | is the above pre calling devstack ? | |
| 15:15:16 | diana_clarke | bauzas: dansmith beat me to it, but for those following along from home, you can see the core here: https://review.openstack.org/#/admin/groups/188,members | |
| 15:15:31 | diana_clarke | s/core/cores | |
| 15:17:05 | dansmith | bauzas: you can call devstack functions if they're included, yeah, like initialize_database there is from devstack | |
| 15:19:09 | bauzas | dansmith: I think I got the idea from https://github.com/openstack-dev/grenade/blob/master/projects/60_nova/from-mitaka/upgrade-nova | |
| 15:19:16 | bauzas | I could source lib/placement | |
| 15:19:16 | johnthetubaguy | dansmith: should I be worried about the order of the list of CELLS we are iterating over being the same on all API workers? | |
| 15:19:38 | dansmith | johnthetubaguy: from a sorting perspective? | |
| 15:19:46 | dansmith | er, instance sorting? | |
| 15:19:56 | johnthetubaguy | dansmith: yeah, I was thinking from a fetching a marker perspective | |
| 15:20:18 | bauzas | johnthetubaguy: you mean adding a new cell meanwhile we're iterating ? | |
| 15:20:29 | dansmith | johnthetubaguy: well, so the deal here is that this code is here to officially support only one cell for ocata, | |
| 15:20:39 | johnthetubaguy | dansmith: good point | |
| 15:20:52 | bauzas | johnthetubaguy: if so, I discussed on that yesterday with dansmith and we agreed on not being a problem for ocata | |
| 15:20:57 | dansmith | johnthetubaguy: but so that we can move to the third patch where we're creating in conductor (and thus potentially multiple cells). None of the naive sorting stuff is in here yet, so I think any concern for that is moot, | |
| 15:21:03 | dansmith | but this enables people to play with it | |
| 15:21:31 | johnthetubaguy | dansmith: in terms of staging things so we get someone, totally makes sense | |
| 15:21:40 | johnthetubaguy | s/someone/somewhere | |
| 15:21:40 | dansmith | johnthetubaguy: right | |
| 15:22:16 | dansmith | johnthetubaguy: but yes, following this, we'll want stable sorting of cells, reloading of the cells list, a naive sort approach, etc | |
| 15:22:55 | johnthetubaguy | dansmith: ack | |
| 15:23:26 | mriedem | oh great device tags is all fubar in the api now too huh | |
| 15:24:46 | dansmith | bauzas: johnthetubaguy: sfinucan: the patch after that, by the way, is just a tweak to the api_samples setup, and an approach sdague suggested, so it should be fairly easy to hit I think | |
| 15:24:58 | dansmith | to prevent api_samples from tripping over itself in the final patch | |
| 15:25:08 | bauzas | dansmith: yeah I just began to look at it | |
| 15:25:17 | dansmith | cool | |