| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-02 | |||
| 14:13:04 | efried | Mm, I should clarify that with a TODO in the code. | |
| 14:15:05 | mriedem1 | dansmith: smatzek: ack on https://bugs.launchpad.net/devstack/+bug/1708171 | |
| 14:15:05 | openstack | Launchpad bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] | |
| 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 ? | |
| 14:36:24 | mriedem | this one https://review.openstack.org/#/c/478485/ | |
| 14:36:30 | tikitavi | why extensions stopped working? in particular, extended_volumes.py stopped adding info about attached volumes to server list and server show? | |
| 14:36:50 | mriedem | stephenfin: i also noted in https://review.openstack.org/#/c/477497/ that it's missing a bunch of nova-specific CLI guides that are now gone from the admin guide | |
| 14:36:58 | mriedem | but are still in the stable/ocata branch for openstack-manuals | |
| 14:37:15 | sdague_ | stephenfin: I'm more concerned about breaking up things like the policy doc, so that the only way to find information about nova policy is to already understand it, and know which subdoc to be in | |
| 14:37:19 | stephenfin | mriedem: Yeah, I'm working with asettle on that this week | |
| 14:37:46 | sdague_ | for stuff like that long pages that you can search in browser in are often more useful | |
| 14:37:52 | mriedem | stephenfin: also, do we know if these got in from the user guide? https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/user-guide/source/cli-launch-instances.rst https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/user-guide/source/cli-delete-an-instance.rst ? | |
| 14:38:33 | mriedem | stephenfin: maybe we push https://review.openstack.org/#/c/478485/ to the end and actually just work on getting the migration done for pike | |
| 14:38:35 | stephenfin | sdague_: tbh, I'm not sure what's changed in that doc so I need to look at that. If you haven't left that comment on Gerrit yet, can you do so and I'll tackle it once I get there? :) | |
| 14:38:43 | mriedem | and worry about completely re-arranging everything after that's done | |
| 14:38:45 | sdague_ | stephenfin: I did | |
| 14:38:54 | stephenfin | sdague_: Ta | |
| 14:39:27 | stephenfin | mriedem: Yeah, a mass move and later refactor was my initial plan | |
| 14:39:28 | mriedem | it also seems if we're going to re-arrange the home page for nova to be a certain form, that template should follow suit in the other projects, | |
| 14:39:34 | mriedem | and today it's not for cinder/glance/neutron either | |
| 14:39:36 | mriedem | they are all different | |