| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-31 | |||
| 21:21:58 | cdent | omg, "scheduler calling the placement api" just merged. | |
| 21:22:07 | bauzas | indeed | |
| 21:22:15 | mriedem | i thought it was getting bumped again | |
| 21:22:17 | bauzas | back from the cave | |
| 21:22:54 | bauzas | mriedem: you okay if I'm just providing a new revision for the reno branch ? | |
| 21:23:13 | mriedem | bauzas: yes | |
| 21:23:19 | bauzas | k | |
| 21:23:20 | dansmith | mriedem: we need to add the aliases once know it's never going to change for a release, which usually means "after the release" | |
| 21:23:58 | browne | ah ok | |
| 21:25:14 | cdent | mriedem: I'm done for the day, but can I assume that https://etherpad.openstack.org/p/nova-ocata-rc1-todos is still the place to go tomorrow morning for things, like, to do? | |
| 21:25:46 | mriedem | cdent: yup | |
| 21:25:49 | mriedem | good progress today | |
| 21:25:51 | cdent | cool | |
| 21:25:58 | cdent | goodnight everybody | |
| 21:28:22 | mriedem | dansmith: https://wiki.openstack.org/wiki/Nova/ReleaseChecklist#Pre-release_Checklist is a bit confusing since it says "Ideally add the RPC version alias just before opening the next release" | |
| 21:28:29 | mriedem | i guess ideally is, well, ideal | |
| 21:28:30 | mriedem | but optional | |
| 21:28:31 | kencjohnston | Where would I find the name of Nova's default read-only role, if there is one? | |
| 21:28:35 | dansmith | mriedem: right | |
| 21:33:08 | openstackgerrit | Matt Riedemann proposed openstack/nova: doc: refer to the cell_v2 man pages from the cells v2 doc https://review.openstack.org/427459 | |
| 21:33:09 | openstackgerrit | Matt Riedemann proposed openstack/nova: doc: update the man page entry for nova-manage db sync https://review.openstack.org/427460 | |
| 21:51:01 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737 | |
| 21:51:01 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Add release note for nova-status upgrade check CLI https://review.openstack.org/426911 | |
| 21:51:02 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Add prelude section for Ocata https://review.openstack.org/426864 | |
| 22:15:32 | sfinucan | mriedem: Bit late, but ':doc:/<path>' is relative to root | |
| 22:15:59 | sfinucan | re: mriedem> sfinucan: do you know the ReST magic to find the root of a doc tree? for a link | |
| 22:16:11 | mriedem | sfinucan: not too late yet | |
| 22:16:12 | mriedem | thanks | |
| 22:19:21 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: localdisk https://review.openstack.org/409401 | |
| 22:26:37 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: console https://review.openstack.org/409402 | |
| 22:26:52 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (cfg drv) https://review.openstack.org/409404 | |
| 22:29:35 | mriedem | so another thought occurs to me, | |
| 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? | |