| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-31 | |||
| 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 | mriedem | so it's your gut | |
| 23:16:06 | andreykurilin | I checked the code | |
| 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 | openstackgerrit | John Griffith proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | |
| 23:18:18 | mriedem | because 6.0 is the minimum in g-r right now | |
| 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 | 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:51:12 | andreykurilin | so it is just +2;-2 change (with tests) | |
| 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 | openstack | Launchpad bug 1660761 in kolla-ansible "VM stuck in scheduling" [Critical,Confirmed] - Assigned to Michał Jastrzębski (inc007) | |
| 01:38:51 | inc0 | hello guys, we (kolla) started to experience issues with nova - https://bugs.launchpad.net/kolla-ansible/+bug/1660761 | |
| 01:39:15 | inc0 | cells v2 landed? I assume there is new step durng deployment process right? | |
| 01:39:47 | inc0 | and we *have to* configure cells now? | |
| 01:41:43 | diana_clarke | inc0: yes, cells v2 setup is required as of ocata | |
| 01:42:31 | inc0 | http://docs.openstack.org/developer/nova/cells.html#cells-v2 this is correct doc right? | |
| 01:42:57 | diana_clarke | inc0: I'm just pulling up some links for you (sorry if that sounded short/terse a moment ago). | |
| 01:43:21 | diana_clarke | inc0: yup, that a good one to read | |
| 01:43:24 | diana_clarke | inc0: https://review.openstack.org/#/c/426864/5/releasenotes/notes/ocata_prelude-cfa8793d07f963e7.yaml | |
| 01:43:26 | inc0 | diana_clarke, it did not:) it sounded nothing but helpful:) | |
| 01:43:53 | inc0 | thank you, I'll get to writing deployment code for that | |
| 01:45:00 | diana_clarke | inc0: http://docs.openstack.org/developer/nova/man/nova-manage.html#nova-cells-v2 | |
| 01:45:57 | inc0 | thanks, I'll check if other deployment tools got that in their deploy as well | |
| 01:46:13 | inc0 | this will have to be part of upgrade task too I assume? | |