Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-01
15:12:23 dfisher ok, so with the transport_url set right, i'm in business
15:12:26 dansmith dfisher: well, it's a little of both
15:12:29 dfisher thank you all.
15:12:30 dansmith dfisher: cool
15:12:41 mriedem little bit country? little bit rock&roll?
15:13:42 mriedem dfisher: we could put something in https://docs.openstack.org/nova/latest/user/cells.html#faqs for your ase
15:13:43 mriedem *case
15:14:59 mriedem by we i think i mean you
15:18:16 openstackgerrit Chris Dent proposed openstack/nova master: WIP Update RT aggregate map less frequently https://review.openstack.org/489633
15:19:09 sdague dfisher: ok, it would be good to figure out where that went wrong
15:19:25 sdague because devstack should be setting it up correctly on both controller and compute
15:19:37 sdague and if something is falling through, would be good to fix
15:19:40 mriedem sdague: i assume dfisher is doign multinode
15:19:45 mriedem w/o devstack-gate
15:19:47 sdague mriedem: sure
15:20:13 sdague but the only thing in d-g is the discover_hosts
15:20:16 openstackgerrit Jay Pipes proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958
15:20:17 openstackgerrit Jay Pipes proposed openstack/nova master: placement: remove existing allocs when set allocs https://review.openstack.org/489273
15:20:17 openstackgerrit Jay Pipes proposed openstack/nova master: remove source provider allocs in confirm_resize() https://review.openstack.org/488510
15:20:19 cdent jaypipes: that patch above does what I think we’re after ^ but needs some reactions and tweaks. gonna get lunch and then will check back
15:20:28 jaypipes cdent: cheers
15:20:28 mriedem sdague: d-g also sets up the local.conf for the subnodes doesn't it?
15:20:36 mriedem otherwise you'd have n-api in enabled_services
15:20:37 cdent bbs
15:20:48 sdague mriedem: yeh, but I didn't think we had cells specific logic there
15:21:01 sdague I'd like devstack multinode without d-g to work for folks
15:21:29 mriedem sdague: there is conditional logic in devstack to know if it's a subnode or not based on n-api running
15:21:30 mriedem is my point
15:22:29 sdague mriedem: yeh
15:23:04 sdague dfisher: anyway, if you have suggestions on how to make this more clear and straightforward to setup on the devstack side, let me know
15:23:26 dfisher sdague: just mention that the transport_url on the compute node needs to match the cell1 transport_url
15:23:28 dfisher that's all you need.
15:24:26 dfisher and, again, thank you sdague, mriedem, and dansmith for putting up with dumb questions from the peanut gallery
15:24:43 gibi jaypipes, dansmith: I have one more question at https://review.openstack.org/#/c/487958/21/nova/tests/functional/test_servers.py@1370
15:25:20 sdague dfisher: any idea where that should be mentioned that it would be more obvious?
15:25:54 dfisher sdague:
15:25:54 dfisher 1.Configure and start your compute hosts. Before step 7, make sure you have compute hosts in the database by running nova service-list --binary nova-compute.
15:25:59 dfisher gah. that should be #6
15:27:00 dfisher maybe something like: 'Configure your compute node's transport_url to point to the cell1 transport_url (visible with nova-manage cell_v2 list_cells —verbose) and start the nova-compute service'
15:27:39 jaypipes gibi: that's concerning to me. I can't explain why that would be. :(
15:29:54 gibi jaypipes: sorry I pointed to the wrong assert it fails the assert 'self.assertEqual(0, len(allocations))'
15:29:55 mriedem dfisher: step 5 says, "The transport url is taken from the [DEFAULT]/transport_url setting in the Nova configuration file if not specified on the command line." - which would assume that if you created cell1 with a transport_url from the config file, you'd be using that same config file on the compute service if you have a single MQ
15:30:19 mriedem dfisher: devstack is a bit different now in that by default there are 2 conductors
15:30:50 mriedem what you suggest isn't a bad reminder though
15:32:03 mriedem dfisher: did you want to push a change to adjust that part of the docs?
15:32:10 bauzas oh man, my computer crashed
15:32:27 mriedem bauzas: it didn't crash, it's on french government mandated PTO
15:32:32 bauzas (says the one who uses AndChat)
15:32:35 mriedem hence the smoke
15:32:39 jaypipes gibi: I believe there should be a *separate* assertion above line 1370 that shows we have the correct allocation after CONFIRM_RESIZE but after the periodics run, that allocation is messed up. I think that's what dansmith was going for there.
15:32:45 dfisher I can. Not sure how, however. That and me no word good.
15:33:02 bauzas mriedem: it's a trap
15:33:12 mriedem dfisher: have you ever pushed a change to review.openstack.org?
15:33:16 dfisher yep.
15:33:26 dansmith isn't dfisher from oracle?
15:33:30 dfisher that's me
15:33:33 dansmith he might burst into flame if he pushes a patch
15:33:36 dansmith so we should probably do it for him
15:33:37 dfisher :(
15:33:57 mriedem dfisher: http://git.openstack.org/cgit/openstack/nova/tree/doc/source/user/cells.rst#n391 is the line you want
15:34:06 dfisher on it.
15:34:16 gibi jaypipes: make sense
15:34:36 jaypipes gibi: I'll add that assertion.
15:34:42 gibi jaypipes: thanks
15:35:46 dfisher dansmith: i'll let you know if you if any parts of me (fleshy or ethereal) start to smoke as I work on this
15:36:17 dansmith dfisher: I'll stand by with a fire extinguisher
15:36:30 dfisher even though I'm an Oracle employee? :)
15:36:39 mriedem sdague: redirect works https://docs.openstack.org/nova/latest/cells.html
15:36:40 dansmith dfisher: you're still a human...
15:36:45 dansmith dfisher: um, right?
15:36:58 gibi jaypipes: would it make sense if I start building some more funcitonal tests top of the your bug fix to see further ahead?
15:37:05 dansmith heh
15:37:14 mriedem gibi: yes we want a single node functional set of tests for resize confirm/revert
15:37:18 mriedem resize to same host
15:37:23 openstackgerrit Matthew Booth proposed openstack/nova master: fake_notifier: allow waiting on either notification type https://review.openstack.org/489637
15:37:38 dfisher as a Denver resident I can suggest places to eat (and not to eat) for the PTG
15:37:54 mriedem gibi: that's going to be needed for this bug https://bugs.launchpad.net/nova/+bug/1707252
15:37:55 jaypipes gibi: not at this time, I don't think so.
15:37:55 openstack Launchpad bug 1707252 in OpenStack Compute (nova) "Claims in the scheduler does not account for doubling allocations on resize to same host" [Medium,Confirmed]
15:37:55 dfisher i can trade that for fire-putting-outting, maybe
15:38:29 mdbooth mriedem: Incidentally, I now have a patch on top of gibi's patch, but because it also has another dependency I can't push it
15:38:58 mdbooth Any chance we could keep the scope of gibi's patch small so it can merge soon?
15:39:04 mdbooth And add more later.
15:39:40 gibi mdbooth: If you are refering to https://review.openstack.org/#/c/487958/ then I think that patch is pretty close to land
15:39:47 mriedem mdbooth: i was assuming any new tests would be built on top
15:40:01 mdbooth gibi: Yep, that's the one.
15:40:08 mdbooth mriedem: Excellent, just checking.
15:40:08 mriedem mdbooth: believe me, i'd like to see that test patch land asap
15:40:17 gibi mdbooth: I think that patch will not grow further
15:40:37 mdbooth gibi: Now could you land it in the next 10 minutes? ;)
15:40:51 gibi mdbooth: I have no power over it ;)
15:41:30 mdbooth I have your patch stashed in the middle of my own series, so I need to be really careful not to submit it!
15:43:01 gibi mdbooth: to be clear it is not my patch, it is shared at least between jaypipes, dansmith, cdent and myself. Just to make it easy ;)
15:43:58 gibi mriedem: so I'm not committing myself to start the resize to same host today, but if nobody starts that until I wake up tomorrow then I will stat
15:44:17 gibi mriedem: s/stat/start/
15:44:32 mriedem gibi: ok i might start that today
15:44:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Add redirect for api-microversion-history doc https://review.openstack.org/489641
15:45:05 gibi mriedem: ping me when you start to avoid double work
15:45:31 mriedem aye aye
15:46:45 mriedem fyi, also sent to the ML https://etherpad.openstack.org/p/nova-pike-release-candidate-todo

Earlier   Later