| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-05 | |||
| 18:54:10 | mriedem | probably the placement job | |
| 18:54:17 | mriedem | which is our test bed for new things | |
| 18:54:28 | scottda | mriedem: OK, just bringing it up since we're talking about it. | |
| 18:54:43 | mriedem | is there a devstack configure to set the cinder catalog info in nova.conf? | |
| 18:54:48 | mriedem | *config | |
| 18:55:09 | scottda | I don't think so, but not sure. | |
| 18:55:14 | mriedem | CINDER_NOVA_CATALOG_INFO | |
| 18:55:28 | mriedem | oh that's to configure cinder | |
| 18:56:07 | mriedem | if there were a devstack variable we could piggyback on from the cinder side that would be good, | |
| 18:56:20 | mriedem | because we can't yet pass localrc through from the job defintiion through to devstack | |
| 18:56:33 | mriedem | and the devstack cores aren't happy about adding more one-off config variables like that | |
| 18:57:09 | mriedem | i don't see anything | |
| 18:57:34 | mriedem | you could talk to sdague about the plumbing bits...but my guess is he'll say he wants it done with localrc | |
| 18:58:26 | dansmith | mriedem: well, progress. now all that fails is "all the listing tests" which is what that patch touches | |
| 18:58:55 | mriedem | from tempest? | |
| 18:59:17 | mriedem | well, baby steps :) | |
| 18:59:29 | mriedem | that's probably an easier fix than what we started thinking was broken this morning | |
| 18:59:54 | dansmith | yeah | |
| 18:59:56 | dansmith | that's my point | |
| 19:00:03 | mriedem | unless the cells v1 api isn't getting any instances from the cell mapping | |
| 19:00:07 | mriedem | that might suck | |
| 19:00:19 | dansmith | well, it's likely easier to resolve still | |
| 19:00:22 | mriedem | does map_cell_and_hosts also call map_instances or whatever? | |
| 19:00:30 | mriedem | my guess is the instances aren't mapped? | |
| 19:01:09 | dansmith | there are no instances at that point, and all created ones should be mapped | |
| 19:01:10 | mriedem | https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1182 | |
| 19:01:14 | mriedem | oh right | |
| 19:04:10 | openstackgerrit | Matt Riedemann proposed openstack/nova: WIP Get Cinder v3 microversion https://review.openstack.org/385682 | |
| 19:04:11 | mriedem | scottda: ildikov: ^ | |
| 19:05:06 | mriedem | that's not going to work right no | |
| 19:05:07 | mriedem | *now | |
| 19:05:15 | ildikov | mriedem: sweet, thank you! | |
| 19:06:06 | scottda | ildikov: Just tested using cinder v3 in nova.conf and it works for me. | |
| 19:07:01 | openstackgerrit | Matt Riedemann proposed openstack/nova: WIP Get Cinder v3 microversion https://review.openstack.org/385682 | |
| 19:07:54 | mriedem | ok well ^ should see if things work in a tempest run | |
| 19:08:51 | openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | |
| 19:08:52 | openstackgerrit | Dan Smith proposed openstack/nova: Make the SingleCellSimple fixture a little more comprehensive https://review.openstack.org/410472 | |
| 19:08:52 | openstackgerrit | Dan Smith proposed openstack/nova: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/410471 | |
| 19:12:49 | mriedem | speaking of, | |
| 19:12:56 | mriedem | i always thought they said 'fucked up music' in https://www.youtube.com/watch?v=HV3zWSawJiw | |
| 19:12:58 | mriedem | is that just me? | |
| 19:13:06 | mriedem | i'm pretty sure it's 'funked up' | |
| 19:14:02 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | |
| 19:14:16 | ildikov | scottda: thanks, I will reinstall my Devstack then and see | |
| 19:17:03 | ildikov | mriedem: does not really sound 'funked up' to me, but maybe I'm not native enough to hear it especially after having a hint of what I should hear instead :) | |
| 19:17:44 | mriedem | heh | |
| 19:17:48 | mriedem | well, | |
| 19:17:50 | mriedem | i wouldn't go out tonight, | |
| 19:17:55 | mriedem | there is a bathroom on the right | |
| 19:18:17 | mriedem | https://www.youtube.com/watch?v=ExViB7ArQNA | |
| 19:19:49 | ildikov | LOL :) | |
| 19:22:08 | openstackgerrit | Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | |
| 19:22:09 | openstackgerrit | Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | |
| 19:25:03 | ildikov | mriedem: BTW, as you don't seem to go out tonight, if you would happen to have time to check the Remove check_attach patch ^^ that would be great :) | |
| 19:25:52 | openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | |
| 19:26:00 | mriedem | my dance card is full all of a sudden | |
| 19:31:53 | ildikov | dansmith: enjoy lunch :) | |
| 19:33:30 | openstackgerrit | Matt Riedemann proposed openstack/nova: Return uuid attribute for aggregates https://review.openstack.org/415031 | |
| 19:33:31 | mriedem | jaypipes: ok i think the change here ^ tightens up the api sample test validation a bit | |
| 19:40:53 | openstack | Launchpad bug 1632852 in OpenStack Compute (nova) "placement api responses should not be cacehable" [Medium,In progress] - Assigned to Prateek Arora (parora) | |
| 19:40:53 | cdent | macsz: is there code up for review for this bug or are you just indicating it is in progress because there is code in progress on someone's computer? https://bugs.launchpad.net/nova/+bug/1632852 | |
| 19:43:36 | macsz | cdent: there is an assignee, the policy is that if someone is assigned the status should be 'In progress'. Normally I would question if someone is still working on it, but Prateek was assigned just a month ago, and there were Christmas on the way, so effectively it leaves like two weeks of work | |
| 19:44:40 | cdent | macsz: Secure, was just curious; wondering if maybe it was up for review but the link to gerrit didn't get in there for some reason. If so, I wanted to look :) | |
| 19:44:50 | macsz | cdent: I would probably make it to this bug again next week or two and ask for a status | |
| 19:44:54 | macsz | cdent: cool :) | |
| 19:45:17 | cdent | s/Secure/Sure/ (I have no idea what I was thinking there ... ) | |
| 19:46:25 | macsz | well, secureness cheered me up for a moment :D | |
| 19:48:43 | cdent | :) | |
| 19:56:10 | cdent | edleafe: did you achieve satisfaction? | |
| 19:56:37 | mriedem | jaypipes: cdent: ok comments in https://review.openstack.org/#/c/407742/ | |
| 19:57:28 | cdent | thanks mriedem will chew on that | |
| 19:58:39 | cdent | yeah that all makes sense, will dig | |
| 19:59:16 | mriedem | hmm http://docs.sqlalchemy.org/en/latest/core/sqlelement.html#sqlalchemy.sql.operators.ColumnOperators.in_ | |
| 19:59:19 | mriedem | str is a tuple right? | |
| 19:59:30 | mriedem | that would be a pretty gd sneaky bug | |
| 20:06:25 | cdent | mriedem: I reckon the thing to do is to just special case member_of for 'in:'. The filter code is only doing the in_ sql call on member_of | |
| 20:06:34 | cdent | so we just need to make sure that's a list (even if it is just one) | |
| 20:07:05 | openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | |
| 20:10:20 | mordred | sdague: we're still heading towards not having project_id in the catalog endpoint for nova, yeah? | |
| 20:11:00 | sdague | mordred: yeh, you can do that now | |
| 20:11:16 | sdague | we've been testing that way for about a year | |
| 20:12:09 | mordred | sdague: cool - so lack of project_id in the url just means "use the project from the currently scoped user" ? | |
| 20:12:14 | mordred | s/user/token/ | |
| 20:12:30 | mriedem | cdent: that's what i was thinking | |
| 20:12:33 | sdague | mordred: yes | |
| 20:12:42 | sdague | mordred: which is actually the way the code always worked | |
| 20:12:45 | cdent | ✔ | |
| 20:12:48 | mriedem | cdent: we could also ask zzzeek but to be safe it's probably safest to just always make it a list | |
| 20:13:21 | sdague | the project id in the url was nothing more than belt and suspenders since I've been on the project | |
| 20:13:21 | mriedem | did i mention that safety is very important :) | |
| 20:13:21 | openstackgerrit | Sarafraj Singh proposed openstack/nova: WIP: Add service_token for nova-cinder interaction https://review.openstack.org/397399 | |
| 20:14:00 | sdague | mriedem / dims - something just seems to have started melting down on python3 unit tests for nova - http://status.openstack.org/zuul/ | |
| 20:14:05 | sdague | on gate queue | |
| 20:14:44 | sdague | http://logs.openstack.org/23/403423/9/gate/gate-nova-python35-db/60675e6/console.html#_2017-01-05_19_02_54_521661 | |
| 20:16:06 | mordred | sdague: ah! neat. good to know | |
| 20:16:44 | mordred | sdague: if I am an admin with access rights and I add a project id to the url, that should show me things in that project? | |
| 20:17:57 | sdague | no | |
| 20:18:24 | sdague | it's... weird, and we all had to read that code a bunch to convince ourselves that wasn't working that way | |
| 20:19:11 | sdague | but basically if project_id is in the url, it is checked against token project_id, if they aren't the same, 403 | |
| 20:19:28 | sdague | the token project_id is always used | |