Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-13
19:41:11 lyarwood ah cool
19:41:13 mriedem been a long week
19:41:21 lyarwood yeah, tell me about it
19:41:46 mriedem the other series in newton for placement that i have here https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:bug/1651678
19:42:00 mriedem one of the changes towards the end is failing functional tests for some reason which i need to investigate
19:42:03 mriedem so you can hold off on that series
19:42:18 lyarwood don't worry I was going to :)
19:42:23 mriedem heh
19:42:50 mriedem i mean at quarter to 8 on a friday, reviewing that series is about as much fun as you should have
19:44:40 lyarwood haha aye, unfortunately my presence is required downstream to fight another fire, otherwise I'd be all over that at this time on Friday ;)
19:51:06 dims mriedem : so got one +2 from sdague and a review from cdent in the morning. any chance for getting it merged today? https://review.openstack.org/#/c/419476/
19:51:37 dims mriedem : thanks in advance!
19:51:38 mriedem not from me right now, i have other fires
19:52:25 dims :) no worries. just did not want to keep asking
19:52:49 dims melwitt : how about you? :)
19:53:31 melwitt maybe later, working on something related to cells v2 fire right now
19:54:23 dims ack thanks melwitt
20:02:12 sdague timing on that sucks, mostly with dansmith heading out to the wood with only a ham radio
20:02:36 mriedem well i think he's around a bit yet
20:02:44 mriedem but yes
20:03:21 melwitt he said he'll be around this afternoon, we'll have him on monday too. but starting tuesday he's out next week
20:05:27 mriedem ok i've replied to the ML thread
20:05:31 mriedem with my brain dump
20:06:18 dims hmm, looks like cdent is done for the day as well. seeing some gabbit related issues
20:06:54 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968
20:08:05 mriedem sdague: we also need https://review.openstack.org/#/c/392569/ updaed
20:08:07 mriedem *updated
20:09:11 mriedem although i haven't gone through that whole patch yet
20:09:28 mriedem and the things i pointed out already probably aren't the end of the world
20:10:15 melwitt mriedem: I'm fixing up the create_cell patch right now, just fyi
20:11:01 mriedem melwitt: yup, i know (already) :)
20:12:59 EmilienM mriedem: thanks for your help, very appreciated
20:13:09 mriedem yw
20:41:00 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968
20:46:41 openstackgerrit Slawek Kaplonski proposed openstack/nova: Set sysinfo_serial="none" in LibvirtDriverTestCase https://review.openstack.org/420176
20:48:19 openstackgerrit Matt Riedemann proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775
20:48:19 openstackgerrit Matt Riedemann proposed openstack/nova: Make metadata server know about cell mappings https://review.openstack.org/417962
20:48:20 openstackgerrit Matt Riedemann proposed openstack/nova: Make api_samples tests use simple cell environment https://review.openstack.org/410473
20:48:23 openstackgerrit Matt Riedemann proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379
21:04:00 mriedem sdague: i'm cool with this pretty simple patch https://review.openstack.org/#/c/417962/
21:08:39 openstackgerrit melanie witt proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713
21:14:57 openstackgerrit sean mooney proposed openstack/os-vif: [DO NOT MERGE] this a test patch for os-vif gate. https://review.openstack.org/420179
21:17:07 openstackgerrit Diana Clarke proposed openstack/nova: copy pasta error https://review.openstack.org/420180
21:17:22 diana_clarke ^ easy review for Friday brains ;)
21:34:02 openstackgerrit Diana Clarke proposed openstack/nova: DNM: check experimental queue https://review.openstack.org/420186
21:38:45 mriedem sean-k-mooney: i think https://review.openstack.org/#/c/410737/ is depending on the wrong patch
21:39:05 mriedem that's why gate-nova-requirements failed
21:39:43 mriedem i'll update it quick
21:40:19 openstackgerrit Matt Riedemann proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061
21:40:20 openstackgerrit Matt Riedemann proposed openstack/nova: os-vif-util: set vif_name for vhostuser ovs os-vif port https://review.openstack.org/410737
21:51:22 dtp anyone need anything re cells v2?
21:51:40 mriedem dtp: for right now i don't think so
21:51:53 dtp ok
22:22:04 melwitt mriedem: trying to make sure I have this right in my head ... is the expectation during an upgrade to ocata that the appropriate cells v2 setup commands be run using the ocata code, before running the db sync? or is the expectation to run the setup using the newton code before starting the upgrade
22:24:25 mriedem melwitt: in grenade we run simple_cell_setup in newton before going to ocata: https://review.openstack.org/#/c/393441/2/projects/60_nova/from-newton/upgrade-nova
22:24:28 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968
22:24:42 mriedem but simple_cell_setup is meant to be idempotent so you can run it as many times as needed
22:25:04 mriedem but you're asking because we don't have create_cell in newton
22:25:08 melwitt okay. I was wondering if it's also fine to run them in ocata, just before the db sync
22:25:11 melwitt yes, pretty much
22:25:34 mriedem well, for the nova_api db sync to work, you have to have created the cell mappings and cell0
22:25:49 mriedem so (1) simple_cell_setup, (2) api-db sync
22:25:50 melwitt and also the simple_cell_setup has an enhancement in ocata to take the transport url from the conf, which is pretty important. I think it can be backported (I checked oslo.messaging stuff)
22:25:55 mriedem because (2) has the blocking schema migration
22:25:57 melwitt but I was thinking, do we need to backport all this stuff or no
22:26:10 melwitt right
22:26:20 mriedem melwitt: i remember you asking about backporting that one, i think if it was ok w/ the oslo.messaging version in newton i think we can backport it
22:26:23 mriedem if you haven't already done so
22:26:34 openstackgerrit Dan Smith proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198
22:26:42 mriedem well look who it is
22:26:44 dansmith mriedem: melwitt ^
22:26:46 melwitt yeah, I was just wondering if it's necessary. maybe not necessary but it would be nice to have the option to run them in newton?
22:27:33 dansmith after writing that, I almost think maybe we should do away with the simple command, or at least not stress its usage
22:27:40 dansmith once we have the create command
22:27:55 dansmith I think it's more straightforward to think about "create cell, discovery hosts"
22:28:18 dansmith once we merge the create command I will change devstack to do the same thing for cellsv1 and normal, removing the simple usage
22:29:04 melwitt yeah, not surprisingly the highly niched batch command ends up being more confusing than it's worth
22:29:53 dansmith well, it was supposed to make it easy for people, but...
22:29:58 mriedem dansmith: yeah i did mention something to that effect in the ML, i.e. doing different things for upgrade scripts (grenade) and fresh install scripts (devstack)
22:30:04 melwitt dansmith: I dunno if you saw my earlier question but, I was asking is it equally okay to run cells v2 setup stuff after you've gotten ocata code, just make sure you do it before any db sync?
22:30:04 mriedem but it depends on where we end up probably
22:30:13 openstackgerrit Scott DAngelo proposed openstack/nova: WIP Do Not Merge https://review.openstack.org/420201
22:30:14 melwitt or do we need to backport all these command changes to newton
22:30:33 mriedem if create_cell is going to be required for upgrade, then we're going to have to backport it to newton and change grenade
22:30:39 dansmith yeah
22:30:50 dansmith although the simple command will still work for people upgrading,
22:30:52 mriedem right
22:30:52 melwitt okay
22:31:04 mriedem i like simple_cell_setup in grenade, but not devstack
22:31:08 dansmith so maybe we just backport changes to it to make it work, and bill it as the "get you to ocata command" and then do away with it later
22:31:12 dansmith yeah
22:31:19 mriedem sure
22:31:37 scottda mriedem: That patch fixes the problem with not having cinderv3 in Nova's catalog ^^^^
22:31:47 dansmith if one of you will skim that patch to see if there's anything majorly wrong with it, then I can send it to the ML on that thread for some wider view
22:32:28 mriedem omfg my eyes https://review.openstack.org/#/c/420201/1/nova/context.py
22:32:36 mriedem scottda: you could have thrown that in the other WIP couldn't you?
22:32:43 mriedem not sure how many DNM patches we are going to have here :)
22:33:14 melwitt dansmith: the main thing I noticed is I wouldn't want to imply the db and mq urls are required by putting them in the examples. operators really don't want to put secret creds on the command line
22:33:53 dansmith melwitt: it just seems clearer to me to do that
22:34:13 scottda mriedem: Yeah, I'm going to abandon the first...was too messy to keep and change all the logic.

Earlier   Later