Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-01
15:07:57 dfisher mriedem: yes. I have both cell0 and cell1
15:08:10 cdent jaypipes: I’ll have a aggregate map WIP up shortly
15:08:20 jaypipes cdent: cool, thanks.
15:08:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Deprecate cells v1 https://review.openstack.org/489632
15:08:23 gibi jaypipes: you made me think for a while, but I like these mental challenges so no worries
15:08:24 sdague dfisher: what versions of devstack code do you have?
15:08:29 dansmith dfisher: and cell1 points to the same transport and db url as the config for the conductor and the compute?
15:08:32 mriedem dfisher: and does the transport_url for cell1 match the transport_url in nova.conf on the compute node?
15:08:48 dfisher sdague: git pull'd as of yesterday
15:08:59 mriedem dfisher: you can see the transport_url for cell1 by using --verbose with the list_cells command
15:09:11 dfisher dansmith/mriedem: hang on, checking...
15:09:22 mriedem dansmith: sdague: here is the cellsv1 deprecation https://review.openstack.org/489632
15:09:34 dansmith dfisher: this is what I asked you yesterday fwiw :)
15:09:48 dfisher I know. I'm thick :(
15:09:50 jaypipes gibi: no, I feel bad because I didn't see dansmith's comment above "oh you're right, so I should just stop looking then eh?". I didn't mean to shut him down.
15:10:20 sdague mriedem: ++
15:11:58 dfisher dansmith: i apologize but I didn't understand you meant the transport_url
15:12:04 dfisher dansmith: you said, "dfisher: make sure your conductor's config is pointed at the cell1 db, matching the cell1 cell_mapping record"
15:12:11 dansmith dfisher: yesterday or now?
15:12:12 dfisher dansmith: and I thought you meant the database connection string :(
15:12:14 dfisher yesterday
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]

Earlier   Later