Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-24
18:51:44 cburgess dansmith Yes... very yes.
18:52:01 cburgess Wait there is a polish for that? Maybe thats why I've been stuck in this same job for 6 years... :P
18:52:21 edmondsw seems odd that we have an API to allow you to restrict access by tenant but not to open it up to all tenants
18:52:26 dansmith cburgess: https://cdn.dribbble.com/users/327319/screenshots/1695561/resume_polish-01_1x.png
18:52:27 edmondsw oh well
18:53:16 cburgess dansmith OMG I'm going to have to use that on social media some how. Thats brilliant.
18:58:16 mriedem hmm looks like remove_provider_from_instance_allocation handles the resize to same host for me
19:30:28 openstackgerrit Dan Smith proposed openstack/nova master: Add uuid to migration object and migrate-on-load https://review.openstack.org/496934
19:35:27 mriedem ok fix incoming
19:35:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Cleanup allocations in failed prep_resize https://review.openstack.org/497592
19:35:55 mriedem dansmith: ^ should handle both resize to same host and different hosts, but don't have a functional test for the resize to same host case
19:35:56 mriedem yet
19:39:45 mriedem oomichi: can you help review ^ too please?
19:39:52 mriedem we're pretty short staffed right now
19:44:05 dansmith mriedem: why not do the flavor->resources conversion in the caller and then just re-use the existing migration method in the RT?
19:44:41 dansmith other than that the method is identical to the above in terms of functionality
19:44:49 mriedem because the existing method uses instance.flavor
19:44:52 mriedem which is not the new flavor yet
19:45:25 mriedem the instance.flavor = new_flavor in finish_resize
19:45:29 dansmith ah, I misread the first line
19:45:44 mriedem it's sneaky
19:46:00 dansmith could still refactor it to take a flavor
19:46:09 dansmith just seems like it's too similar to duplicate
19:46:24 mriedem well, the name is misleading, and the error message if it fails
19:48:19 mriedem cdent was doing similar dedup here https://review.openstack.org/#/c/496936/
19:48:20 guimaluf hi guys, I'm getting "Instance failed network setup after 1 attempt(s)" followed by "Timed out waiting for a reply to message ID Timed out waiting for a reply to message ID". My compute node run neutron-{ovs,dhcp,metadata}-agent, and neutron-server and ml2 plugin on my neutron node. Rabbitmq is working. Any clue or direction?
19:48:51 guimaluf I couldn't find nothing meaningful on neutron-server logs, nor neutron-agents
20:04:39 mriedem cool got the resize to same host test done too
20:07:23 cfriesen_ guimaluf: maybe enable debug logs in neutron/nova? try to isolate whether neutron got the message, whether it processed the message, whether it sent a reply, and whether nova received the reply.
20:08:34 guimaluf cfriesen_, is so difficult to check neutron agents logs, cause there is an process in loop filling the screen with trash info
20:08:46 guimaluf cfriesen_, I've enabled the debug and trying to check it out... :/
20:08:50 mriedem dansmith: i've got the func test for the resize to same host part of this, so i'll push that up quick also
20:09:06 mriedem and deal with merging those methods in a follow u
20:09:08 mriedem *up
20:10:21 dansmith okay
20:11:00 openstackgerrit Matt Riedemann proposed openstack/nova master: Cleanup allocations in failed prep_resize https://review.openstack.org/497592
20:11:02 mriedem there it is
20:12:20 mriedem if the func test refactor part bothers you i could pull the new resize test into the patch before this one
20:12:24 mriedem might make review cleaner
20:13:05 dansmith already +2d it
20:13:18 mriedem sweet exhaustion
20:13:34 mriedem now, who do we make temporary core
20:14:18 cfriesen_ mriedem: https://review.openstack.org/#/c/497592 looks okay (with lots of room for cleanup though). were you planning on adding that testcase in test_servers.py or leaving that for a future patch?
20:14:31 mriedem cfriesen_: see PS2
20:14:55 dansmith mriedem: we can wait a bit and see if someone pops up and if not I can just slam it in with an emergency war declaration
20:15:18 mriedem ok, but keep in mind we have to backport and merge in stable/pike too before the rc2 tag
20:15:30 dansmith oh today is thursday huh
20:15:33 mriedem and dims and doug are east coast choppers
20:16:00 dansmith so better do it now then eh?
20:16:36 dims dansmith : mriedem : i will be up until say 10:30-11:00 pm my time
20:16:37 mriedem let cfriesen_ +1 it :)
20:17:20 cfriesen_ heh
20:19:55 dansmith ooh, oomichi for the win
20:20:23 mriedem dansmith: oomichi: don't forget this one in the middle https://review.openstack.org/#/c/497541/1
20:20:30 mriedem the middle child is always left out
20:22:00 dansmith mriedem: I just slammed that one since it's just a test
20:28:58 mriedem ok
20:32:10 cfriesen_ mriedem: it seems wrong to have a test called "test_rescheduling_when_migrating_instance" that doesn't actually test anything around rescheduling.
20:32:48 mriedem1 starting the backport parade
20:35:58 mriedem dansmith: ok we've got 4 backports for stable/pike, the 3 for this series starting here https://review.openstack.org/#/c/497542/
20:36:08 mriedem and this guy https://review.openstack.org/#/c/497461/
20:36:10 mriedem then that's rc2
20:36:32 dansmith waiting for master to merge though right?
20:36:56 mriedem yeah i suppose, you could line up +2s w/o approvals
20:37:00 mriedem or jus twait
20:37:09 mriedem gate seems backed up today
20:37:44 dansmith okay all set
20:37:46 mriedem queues aren't big though so not sure if it's just me
20:38:05 mriedem cool
20:38:11 mriedem i'll start the rc2 proceedings
20:38:23 dansmith yeah I dunno why it's so slow with a relatively low workload
20:39:07 dansmith that conf thing is about to blow through
20:41:11 dansmith mriedem: your prep resize one at the end has a bunch of fails
20:41:21 dansmith like pep8 even I think
20:41:43 mriedem https://review.openstack.org/#/c/497610/
20:42:03 mriedem no way
20:42:15 dansmith (497606,1) Cleanup allocations in failed prep_resize (6m/~--~~~--~-~~_~/rem:12m)
20:42:36 mriedem pretty sure i ran that before pushing
20:42:40 mriedem doing it again
20:43:04 dansmith but that looks like maybe functional too?
20:43:16 mriedem passed locally
20:43:17 mriedem pep8 that is
20:43:27 dansmith everything
20:43:38 dansmith py27, pep8, cells, functional, devstack
20:43:53 dansmith http://logs.openstack.org/06/497606/1/check/gate-nova-tox-functional-ubuntu-xenial/0481e72/console.html
20:43:54 dansmith nice
20:44:07 mriedem oh infra
20:44:09 mriedem ok
20:44:11 mriedem whew
20:44:16 openstackgerrit Merged openstack/nova master: Fix sample configuration generation for compute-related options https://review.openstack.org/497460
20:44:24 dansmith all of them failed the same way
20:44:37 dansmith mriedem: well, whew, if it's not about to block us from landing anything else
20:46:50 openstackgerrit Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: config drive https://review.openstack.org/409404
20:55:27 dansmith well, I guess I'll wander off for a bit and then come back to check on stuff
21:05:43 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/497058
21:07:30 openstackgerrit Ed Leafe proposed openstack/nova master: Return alternates along with their allocations https://review.openstack.org/486253
21:07:30 openstackgerrit Ed Leafe proposed openstack/nova master: Add allocations to the values returned from the scheduler https://review.openstack.org/495854
21:12:22 mriedem dansmith: i just realized that vikings game one of the season is monday night 9/11 at the PTG at ~5:30pm, therefore we will need to be out early and at a sports bar
21:12:25 mriedem post haste
21:12:47 mriedem my beloved vikings will be trouncing mordred's loser saints

Earlier   Later