Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-01
14:49:22 mriedem or https://s-media-cache-ak0.pinimg.com/originals/49/d7/a0/49d7a0676e7945b3fb7bcf4c4533fd99.jpg
14:49:25 sdague gibi: honestly, on the test front, duplication is not a bad thing. You are trying to understand individual test functions in their own context, and if you have to chase a bunch of common code, it's a lot harder to put the state in your head
14:52:25 gibi sdague: I have no firm oppinion I think for me the usefullness of deduplication depends on how well I can trust the utility function to do what it promise
14:53:55 dansmith sdague: exactly
14:53:57 gibi sdague: to build that trust during review I guess I inline the utils in my head at the first usage but then later I just trust them
14:57:32 openstackgerrit Jay Pipes proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958
14:57:33 openstackgerrit Jay Pipes proposed openstack/nova master: placement: remove existing allocs when set allocs https://review.openstack.org/489273
14:57:33 openstackgerrit Jay Pipes proposed openstack/nova master: remove source provider allocs in confirm_resize() https://review.openstack.org/488510
14:57:35 jaypipes dansmith: made those changes you pointed out in the review ^^
14:58:44 openstackgerrit khushbu parakh proposed openstack/nova master: Closes-Bug: #1454515 https://review.openstack.org/488941
14:58:44 openstack bug 1454515 in OpenStack Compute (nova) "Property "instance.disable_terminate" is always "False" and never actually used." [Low,In progress] https://launchpad.net/bugs/1454515 - Assigned to Khushbuparakh (khushbuparakh)
14:59:24 sdague gibi: right, but that's because you are familiar with them. People reading tests will not be, so it takes longer to understand what is going on if they also need to digest utils
15:05:59 dfisher ok, i'm still struggling trying to get a compute node talking to a devstack rig. What services need to be running on the compute node? Obviously nova-compute but what else?
15:06:06 dfisher this is all with beta3
15:06:32 dfisher nova service-list sees my compute node, but cell_v2 discover_hosts can't find it
15:07:02 gibi sdague: sure, there is an overhead due to the extra indirection until the reader gets familiar with the utils but after that it is a lot easier to read the deduplicated code (not to mention chaging it)
15:07:19 jaypipes gibi, dansmith, mriedem, cdent: for the record, I was wrong, dansmith and gibi were right. sorry that I jumped the gun.
15:07:30 mriedem dfisher: what does nova-manage cell_v2 list_cells show?
15:07:34 mriedem do you have a cell0 and cell1?
15:07:55 jaypipes gibi, dansmith, mriedem, cdent: and I did not mean to shut anyone down. I just thought I'd figured out the issue
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 :(

Earlier   Later