| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-31 | |||
| 20:19:46 | sdague | mordred: yeh, test-config | |
| 20:20:00 | sdague | mordred: so change https://github.com/openstack-infra/nodepool/blob/12d3caf560aa827ed47f320259cdb66a20f1e24b/devstack/plugin.sh#L465-L469 | |
| 20:20:05 | sdague | s/extra/test-config/ | |
| 20:20:14 | mordred | cool. thank you! that'll be super helpful | |
| 20:20:39 | sdague | that executes here in stack.sh - https://github.com/openstack-dev/devstack/blob/e6f8dbb55fc8fccb18f4ccb5ed5723a2efc3b025/stack.sh#L1401 | |
| 20:22:32 | mriedem | mordred: sdague: here is the devstack change https://review.openstack.org/427392 | |
| 20:23:17 | mordred | mriedem: I tell you what - we can get some code written aroud here | |
| 20:23:28 | mriedem | just not reviewed or merged :) | |
| 20:23:30 | mriedem | hi-o! | |
| 20:24:30 | mordred | :) | |
| 20:24:44 | mordred | I've been trying to land a shade patch since last tuesday | |
| 20:24:59 | mordred | I'm going to get it landed today if I have to write and land a patch to every repo in openstack | |
| 20:27:14 | openstackgerrit | melanie witt proposed openstack/nova: Read instances from API cell for cells v1 https://review.openstack.org/427394 | |
| 20:27:27 | mtreinish | mordred: don't you have ninja merge powers... | |
| 20:28:17 | mordred | mtreinish: don't tempt me | |
| 20:29:51 | melwitt | dansmith: ^ see if that's what you expect for the cells v1 get_all | |
| 20:30:38 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | |
| 20:46:03 | mriedem | melwitt: lgtm for the most part - would be nice to have a comment in there about the conditional | |
| 20:46:06 | mriedem | like we have in _get_instance | |
| 20:46:19 | cdent | we collided a bit on that | |
| 20:46:21 | melwitt | mriedem: yeah, adding now | |
| 20:46:30 | melwitt | I saw cdent's comment | |
| 20:46:58 | mriedem | i had to pull it down to make sure the else condition was being hit in tests | |
| 20:47:12 | mriedem | since the compute api unit tests are confusing with the cells v1 and non-cells v1 stuff | |
| 20:49:43 | dansmith | melwitt: effectively, yeah | |
| 20:51:16 | openstackgerrit | melanie witt proposed openstack/nova: Read instances from API cell for cells v1 https://review.openstack.org/427394 | |
| 20:53:02 | mriedem | +2 | |
| 20:53:31 | openstackgerrit | Anusha Unnam proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | |
| 20:54:19 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | |
| 20:55:55 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 20:56:31 | mriedem | oomichi: you were +2 before https://review.openstack.org/#/c/422171/ want to get that one now? | |
| 20:56:53 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | |
| 21:02:54 | openstackgerrit | Matt Riedemann proposed openstack/nova: doc: add some detail to the map_cell0 man page https://review.openstack.org/427450 | |
| 21:02:58 | mriedem | diana_clarke: ^ addresses something i noted in your map_cell0 man page change after it was approved | |
| 21:04:06 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: spawn/destroy https://review.openstack.org/391288 | |
| 21:06:27 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 21:07:18 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | |
| 21:16:20 | openstackgerrit | Scott DAngelo proposed openstack/nova: WIP Support cinder v3 API https://review.openstack.org/420201 | |
| 21:17:45 | mriedem | browne: so looking at when the newton version aliases were made, that happened in ocata https://review.openstack.org/#/c/386625/ | |
| 21:18:03 | mriedem | browne: which leads me to believe we should wait to do https://review.openstack.org/#/c/425958/ in pike | |
| 21:18:10 | mriedem | dansmith: is that right? or does it not matter? ^ | |
| 21:19:46 | browne | mriedem: ok, so maybe it can wait | |
| 21:20:22 | mriedem | ooo i found the checklist https://wiki.openstack.org/wiki/Nova/ReleaseChecklist | |
| 21:21:35 | openstackgerrit | Chris Dent proposed openstack/nova: Optional separate database for placement API https://review.openstack.org/362766 | |
| 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: Add release note for nova-status upgrade check CLI https://review.openstack.org/426911 | |
| 21:51:01 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737 | |
| 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 | openstack | Launchpad bug 1654183 in tripleo "Token based authentication in Client class does not work" [High,Triaged] | |
| 23:07:04 | panda | dansmith: hi, did someone already ping you about https://bugs.launchpad.net/python-novaclient/+bug/1654183 today ? | |
| 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 | |