| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-02 | |||
| 14:09:48 | mordred | is so that nova can say "I can handle v1-v2 of glance and prefer internal interface then public interface" | |
| 14:10:14 | mordred | efried: we should supply defaults for adapter values so that it works ifyou don't set endpoint_override too | |
| 14:10:22 | sdague_ | mordred: I'm all for this effort, just wanted to make sure I can land - https://review.openstack.org/#/c/490031/ | |
| 14:10:33 | sdague_ | I legit want to land that as default devstack | |
| 14:10:43 | mordred | sdague_: yes - what I'm saying is that we added a bunch of things to ksa so that you can | |
| 14:10:47 | sdague_ | cool | |
| 14:10:56 | mordred | sdague_: because otherwise it's not possible to actually express a good enough default value | |
| 14:11:00 | sdague_ | ok, great | |
| 14:11:13 | sdague_ | I might have misunderstood efried | |
| 14:11:19 | efried | sdague_ That (getting glance auth from... somewhere else) was going to be a subsequent step in the process. | |
| 14:11:43 | mordred | efried: we should be able to use what's there - it's already getting auth from somewhere else ... OH - I think I get whatyou're saying | |
| 14:11:47 | efried | But it's the reason I coded the util to take an auth param. | |
| 14:11:55 | mordred | efried: the util function doesn't currently have 'pass auth in' plumbed in | |
| 14:11:59 | efried | mordred Yeah, there's no auth in that method | |
| 14:12:23 | mordred | ok. I grok the whole end to end | |
| 14:12:24 | efried | Exactly. And adding it in would have required some pretty far-reaching changes, which I wanted to put off for a followup. | |
| 14:12:42 | efried | And focus this change on setting up and proving the viability of the util itself. | |
| 14:13:04 | efried | Mm, I should clarify that with a TODO in the code. | |
| 14:15:05 | openstack | Launchpad bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] | |
| 14:15:05 | mriedem1 | dansmith: smatzek: ack on https://bugs.launchpad.net/devstack/+bug/1708171 | |
| 14:15:44 | mordred | efried: yes - I think a TODO will help - I have a couple of more comments - and also I see why it's complex to pass in the auth | |
| 14:15:47 | mriedem1 | fwiw the affinity filter tests in tempest seem fine with the superconductor change | |
| 14:15:55 | mriedem1 | maybe trove tests things differently | |
| 14:16:52 | sdague_ | mriedem1: so you think that the wait for nova patch is needed back on ocata as well | |
| 14:16:54 | mordred | sdague_: (for context, get_api_servers in nova/image/glance.py doesn't have the auth context atm since it's currently just dealing with config data - so efried is going to have to plumb that through in nova/image/glance.py | |
| 14:17:04 | efried | mordred Roger, will respin today. Also ( sdague_ ) updated https://review.openstack.org/#/c/489671/ to see if *just* setting endpoint_override will work. I actually suspect it won't - I think we'll fail to build the Adapter. | |
| 14:17:29 | smatzek | does nova have a negative test to ensure anti-affinity fails? (deploying 2 instances in 1 server group when we have 1 host) | |
| 14:17:38 | mordred | efried: I agree with you - we will fail to do that - but I think that patch will be a good testcase of when the things have been plumbed all the way through | |
| 14:17:41 | mriedem1 | smatzek: maybe in functional | |
| 14:17:52 | mriedem1 | smatzek: the affinity filter test in tempest is here http://logs.openstack.org/32/489632/1/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/ae3cb6f/console.html#_2017-08-01_15_57_46_709238 | |
| 14:18:01 | dansmith | smatzek: is that what you're doing and failing? | |
| 14:18:02 | mriedem1 | i'm working on an anti-affinity multi-node test in tempest | |
| 14:19:20 | smatzek | anti-affinity tests with 2 instances, 1 servergroup, 1 host, and verifying that the 2nd instance goes to error. The gate is failing because the second server goes to active. | |
| 14:19:23 | mriedem1 | sdague: the wait for nova patch isn't working on all CIs | |
| 14:20:07 | dansmith | smatzek: ack | |
| 14:20:45 | dansmith | mriedem: I really thought that a test like that requires the send-instance-info stuff in order to actually work | |
| 14:20:49 | mriedem | smatzek: has anyone posted a patch for trove? | |
| 14:20:58 | dansmith | I forget why all, something about when it refreshes the host list | |
| 14:21:13 | sdague_ | mriedem: xenserver looks like the last one right? | |
| 14:21:22 | sdague_ | I just pushed a patch to skip there | |
| 14:21:23 | mriedem | dansmith: and i thought the scheduler / host manager would just pull the instance info if it wasn't getting updates | |
| 14:21:37 | mriedem | sdague_: who knows how many others don't run on devstack changes which would be broken | |
| 14:21:42 | smatzek | mriedem, no, I've been the only one digging into this gate failure and finally narrowed it down to that cells change yesterday late afternoon | |
| 14:21:43 | dansmith | mriedem: but not fast enough or something | |
| 14:22:17 | dansmith | smatzek: you have what you need now to set that conductor mode as a workaround though right? | |
| 14:22:19 | mriedem | smatzek: is the trove job that's failing controlled through the trove repo? | |
| 14:22:21 | mriedem | or something else? | |
| 14:23:49 | mriedem | nvm i see it | |
| 14:23:52 | mriedem | i can push a patch in a minute | |
| 14:23:53 | smatzek | I'm relatively new to Trove in the Trove channel have asked amrith the proper place to set the CELLSV2_SETUP=singleconductor env var. The trove job is controlled through the trove repo and its own devstack plugin.sh | |
| 14:24:02 | mriedem | yeah i'm on it | |
| 14:24:49 | smatzek | Unfortunately I'm going to be afk for the rest of the day starting in a bit. I'm going to take a stab and putting up a review that sets CELLSV2_SETUP where I think it needs to go and we'll see if the gate passes. | |
| 14:24:50 | sdague_ | mriedem: it is possible, that's why I sent an email. | |
| 14:25:08 | sdague_ | the xenserver folks popped up with their concerns | |
| 14:25:31 | dansmith | smatzek: sounds like mriedem is doing that | |
| 14:25:57 | gibi | mriedem: just for your info I pushed functional test for resize to same host https://review.openstack.org/#/c/489973/ | |
| 14:26:19 | smatzek | dansmith, mriedem thanks | |
| 14:27:04 | cdent | gibi++ | |
| 14:27:09 | dansmith | gibi: does that pass with the normal assertions on top of jaypipesjuryduty's current set? | |
| 14:27:29 | dansmith | gibi: I found it super useful to make sure the other one passed on top, then rebase on master and comment out the assertions that failed | |
| 14:27:42 | mriedem | gibi: yup i saw it, thanks | |
| 14:27:53 | mriedem | jay's on jury duty?! | |
| 14:28:03 | dansmith | again | |
| 14:28:09 | dansmith | third time in four years or some such | |
| 14:28:19 | mriedem | he is the youngest citizen in the county | |
| 14:28:30 | dansmith | might be why they want him | |
| 14:29:16 | cdent | dansmith: we don’t have a strategy for accounting for the allocations on same host yet, do we? | |
| 14:29:21 | gibi | dansmith: I haven't tried yet and I think it won't pass there | |
| 14:29:27 | gibi | cdent: exactly | |
| 14:29:51 | gibi | so my commented out asserts there are only visionary things | |
| 14:29:58 | gibi | feel free to comment it | |
| 14:29:59 | dansmith | cdent: that's why I'm asking.. make sure it it passes on top of jay's current set, iterate on jay's set until it does, then flip the ordering | |
| 14:30:00 | cdent | the current strategy is based on different resource provider ids | |
| 14:30:29 | dansmith | I have a call now, biab | |
| 14:30:40 | cdent | dansmith: yeah, but as far I know we haven’t got any ideas | |
| 14:30:48 | gibi | I also have to go offline for 30 mins or so but I be back | |
| 14:31:00 | cdent | me too, I haven’t eaten yet | |
| 14:31:34 | mdbooth | mriedem: Incidentally, mind if I keep the functional test separate for backporting goodness: https://review.openstack.org/#/c/489654/ ? | |
| 14:32:50 | stephenfin | mriedem: Ahh, you're about. Anything super important that I missed in the last week? | |
| 14:33:02 | mriedem | mdbooth: incidentally to what? | |
| 14:33:16 | mdbooth | mriedem: whatever you're doing right now? | |
| 14:33:33 | mdbooth | Your original comment. | |
| 14:33:34 | mriedem | mdbooth: i just got online like 10 minutes ago | |
| 14:33:36 | mriedem | so i'm not doing much | |
| 14:33:52 | mriedem | stephenfin: the docs stuff | |
| 14:34:07 | mdbooth | mriedem: I probably over use that word. | |
| 14:34:16 | mriedem | stephenfin: main issue is redirects were not done so when the pages moved things went to hell | |
| 14:34:44 | mriedem | stephenfin: i'm tracking rc1 todos here https://etherpad.openstack.org/p/nova-pike-release-candidate-todo | |
| 14:34:57 | mriedem | stephenfin: https://review.openstack.org/#/c/489650/ fixes a bunch of links, | |
| 14:35:05 | mriedem | but some are still broken https://review.openstack.org/#/c/489650/ | |
| 14:35:14 | stephenfin | mriedem: Yeah, looks like we got bogged down with those patches. So just that one change left? | |
| 14:35:26 | mriedem | oops https://review.openstack.org/#/c/489641/ | |
| 14:35:32 | stephenfin | Oh, two so | |
| 14:35:37 | stephenfin | I assume the docs patches still open aren't subject to the feature freeze? | |
| 14:35:38 | mriedem | well, those are the 2 we know of | |
| 14:35:44 | mriedem | no they aren't | |
| 14:35:53 | stephenfin | ...but should now include additions to this htaccess file | |
| 14:35:56 | mriedem | sdague and i are concerned about the massive one that changes the indexes | |
| 14:36:03 | tikitavi | hi, I’m from ec2 team, we need some help | |
| 14:36:17 | stephenfin | mriedem: index.rst ? | |