Earlier  
Posted Nick Remark
#openstack-nova - 2017-01-31
22:29:57 mriedem nova-status upgrade check is checking for resource providers and compute nodes and fails if there are compute nodes but not RPs,
22:30:06 mriedem i'm fixing that due to the fallback change in the filter scheduler,
22:30:30 mriedem but i was just thinking, none of this matters unless you're using the filter scheduler - since the caching scheduler doesn't use placement at all in ocata, RPs or not
22:34:27 openstackgerrit Sylvain Bauza proposed openstack/nova: Add a status check for enabled filters https://review.openstack.org/427200
23:05:48 openstackgerrit Matt Riedemann proposed openstack/nova: nova-status: relax the resource providers check https://review.openstack.org/427499
23:07:04 panda dansmith: hi, did someone already ping you about https://bugs.launchpad.net/python-novaclient/+bug/1654183 today ?
23:07:04 openstack Launchpad bug 1654183 in tripleo "Token based authentication in Client class does not work" [High,Triaged]
23:08:45 mriedem hopefully ^ gets nova-status working in grenade now https://review.openstack.org/#/c/426926/
23:09:00 mriedem panda: that's fixed in novaclient 7.1.0
23:09:58 mriedem andreykurilin: ^
23:10:14 panda mriedem: even using 7.1.0 we're still having problems when retrieving nova keypair using token authentication
23:10:43 andreykurilin panda: what project we are talking about?
23:10:45 andreykurilin hi
23:10:51 mriedem andreykurilin: tripleo
23:10:57 mriedem andreykurilin: http://logs.openstack.org/31/426231/3/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/6015c34/logs/undercloud/var/log/undercloud_install.txt.gz#_2017-01-31_18_25_17_000
23:11:38 andreykurilin let me check
23:11:56 mriedem http://logs.openstack.org/31/426231/3/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/6015c34/logs/undercloud/var/log/keystone/keystone.txt.gz#_2017-01-31_18_25_17_339
23:12:02 mriedem 2017-01-31 18:25:17.339 32100 DEBUG keystone.middleware.auth [req-74362ac9-6544-4b45-83f8-e8dc56b1d7e2 - - - - -] There is either no auth token in the request or the certificate issuer is not trusted. No auth context will be set. fill_context /usr/lib/python2.7/site-packages/keystone/middleware/auth.py:188
23:12:28 andreykurilin ok, it looks like I know the reason
23:12:33 andreykurilin of failure
23:13:55 andreykurilin it is about that commit - https://github.com/openstack/python-novaclient/commit/9bbe5a87b7df62a7962debba5db7c96555da6761
23:13:56 panda andreykurilin: anything we should check on our side ?
23:14:15 andreykurilin panda: fix should be simple
23:14:24 andreykurilin just +1;-1
23:14:58 andreykurilin the problem is that projectid argument of novaclient had different meaning in different cases
23:15:14 andreykurilin we fix that
23:15:27 andreykurilin and now projectid means Project ID in case of keystone termas
23:15:30 mriedem andreykurilin: where do you see that from the logs?
23:15:53 andreykurilin there is nothing in logs:)
23:15:59 mriedem heh
23:16:06 andreykurilin I checked the code
23:16:06 mriedem so it's your gut
23:16:26 andreykurilin https://github.com/openstack/instack-undercloud/blob/master/instack_undercloud/undercloud.py#L1239-L1240
23:16:40 panda +1;-1 ok, and they said that math wasn't useful at school.
23:16:41 andreykurilin https://github.com/openstack/instack-undercloud/blob/master/instack_undercloud/undercloud.py#L1099
23:16:54 andreykurilin tenant is a tenant_name
23:17:00 andreykurilin but we expects tenant name
23:17:08 andreykurilin *but we expect tenand id
23:17:11 andreykurilin *tenant
23:17:39 panda so it must be fixed in instack-undercloud ?
23:17:43 andreykurilin yes
23:18:10 mriedem andreykurilin: will that also work with novaclient 6.0?
23:18:18 mriedem because 6.0 is the minimum in g-r right now
23:18:18 openstackgerrit John Griffith proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285
23:18:21 openstackgerrit John Griffith proposed openstack/nova: Remove check_attach https://review.openstack.org/335358
23:20:20 panda andreykurilin: ah, interesting
23:21:10 andreykurilin mriedem: I think no, if we just transmit it `via project_name=tenant`. But I can assume that if we obtain OS_TENANT_ID at https://github.com/openstack/instack-undercloud/blob/master/instack_undercloud/undercloud.py#L1099 and use tenant_id keyword argument while novaclient initialization it should work with 6.0 and 7.1
23:21:39 mriedem ok
23:21:50 mriedem well, i've got to run, but thanks for the quick investigation
23:21:56 mriedem i posted those notes into the bug report
23:22:11 andreykurilin panda: are you ok with that change^
23:22:24 andreykurilin I can quickly propose a patch
23:23:15 andreykurilin mriedem: I'm glad I could help
23:24:02 panda andreykurilin: fine with me
23:27:29 panda bnemec: ^
23:28:18 openstackgerrit Eric Fried proposed openstack/nova: PowerVM Driver: spawn/destroy https://review.openstack.org/391288
23:28:33 openstackgerrit Eric Fried proposed openstack/nova: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380
23:28:50 openstackgerrit Eric Fried proposed openstack/nova: PowerVM Driver: localdisk https://review.openstack.org/409401
23:28:51 bnemec Umm, we don't generate OS_TENANT_ID, so it's not quite that simple.
23:30:33 openstackgerrit Eric Fried proposed openstack/nova: PowerVM Driver: console https://review.openstack.org/409402
23:35:14 jaypipes #success \0/ Finally got a good functional test created that stresses the Ironic and Nova integration and migration from Newton to Ocata.
23:35:16 openstackstatus jaypipes: Added success to Success page
23:41:35 openstackgerrit Eric Fried proposed openstack/nova: PowerVM Driver: config drive https://review.openstack.org/409404
23:45:29 andreykurilin bnemec: hm
23:46:17 andreykurilin bnemec: is it difficult to add that?
23:47:11 bnemec andreykurilin: I'm not sure how we would. We'd need to look up the id in nova, which we can't do until we can create a client.
23:48:27 bnemec andreykurilin: Can we just switch to using a project name or something?
23:49:18 andreykurilin bnemec: yes, but it will not be compatible with min version in g-r. We can use simple check of novaclient version to support all versions. Are you ok about that solution?
23:50:32 openstackgerrit Scott DAngelo proposed openstack/nova: WIP Support cinder v3 API https://review.openstack.org/420201
23:50:35 bnemec andreykurilin: I'm not sure we're as concerned about the minimum. Our repos are generated in such a way that I think if we find any way that works with the new novaclient we can start using that.
23:50:52 andreykurilin oh
23:50:54 andreykurilin cool
23:51:12 andreykurilin so it is just +2;-2 change (with tests)
23:51:12 bnemec Anyone on the old novaclient would be using the old instack-undercloud, and when they update to the new novaclient they should also get the new instack-undercloud.
23:55:11 andreykurilin bnemec panda: https://review.openstack.org/427515
23:56:03 andreykurilin it should fix an issue
23:56:55 bnemec andreykurilin: Cool, thanks.
23:57:04 panda not until we put OS_TENANT_ID in stackrc ?
23:57:55 andreykurilin panda: that solution should work with OS_TENANT_NAME and with latest novaclient without changing stackrc
23:59:11 panda andreykurilin: nice, thanks
#openstack-nova - 2017-02-01
00:07:41 openstackgerrit Jay Pipes proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472
00:18:29 openstackgerrit David Medberry proposed openstack/nova: Explain how allow_resize_to_same_host is useful https://review.openstack.org/426531
00:19:41 openstackgerrit David Medberry proposed openstack/nova: Explain how allow_resize_to_same_host is useful https://review.openstack.org/426531
00:43:26 andreykurilin bnemec: for some reasons CI failed.
00:43:46 andreykurilin any other places of novaclient initialization?
00:46:51 bnemec andreykurilin: Crud, maybe I was wrong about not having to worry about older novaclients.
00:47:22 bnemec It's working on our ocata jobs, but they're using the latest novaclient too.
00:47:59 andreykurilin bnemec: u-c is used in all gates, even for those projects who ignores g-r :) so latest novaclient should be there
00:48:23 andreykurilin oh...there is 6.0
00:48:29 andreykurilin http://logs.openstack.org/15/427515/2/check/gate-tripleo-ci-centos-7-undercloud/962e5b9/logs/pip2-freeze.txt.gz
00:48:39 bnemec andreykurilin: We don't install from source though. We use package repos, and non-tripleo projects are pinned to whatever version last passed our promotion jobs.
00:48:46 bnemec And this bug is blocking the promotion jobs. :-)
00:49:25 bnemec andreykurilin: I have to get going. I'll pick this back up tomorrow though.
00:52:11 andreykurilin bnemec: give me a sec, I'll resubmit a patch
00:53:12 andreykurilin bnemec: I submitted a new patch-set with hack for old release
00:54:02 openstackgerrit Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285
00:54:06 openstackgerrit Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358
01:38:51 inc0 hello guys, we (kolla) started to experience issues with nova - https://bugs.launchpad.net/kolla-ansible/+bug/1660761
01:38:51 openstack Launchpad bug 1660761 in kolla-ansible "VM stuck in scheduling" [Critical,Confirmed] - Assigned to Michał Jastrzębski (inc007)
01:39:15 inc0 cells v2 landed? I assume there is new step durng deployment process right?

Earlier   Later