Friday, 2018-09-07

alex_xugood morning placement00:32
openstackgerritElancheran S proposed openstack/nova stable/pike: Add apply_cells to nova-manage  https://review.openstack.org/59905001:07
*** rubasov has quit IRC01:15
openstackgerritfupingxie proposed openstack/nova master: Add an example to add more pci devices in nova.conf  https://review.openstack.org/59224301:18
*** rubasov has joined #openstack-placement01:23
*** rubasov has quit IRC01:27
*** rubasov has joined #openstack-placement01:28
*** lei-zh has joined #openstack-placement01:34
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec  https://review.openstack.org/60060802:08
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Spec to add per instance timeout for LM - Stein  https://review.openstack.org/60061302:18
*** rubasov has quit IRC02:34
openstackgerritMerged openstack/nova stable/rocky: Fix a failure to format config sample  https://review.openstack.org/60033603:01
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Detach and attach boot volumes - Stein  https://review.openstack.org/60062803:09
openstackgerritBrin Zhang proposed openstack/nova-specs master: Resource retrieving: add changes-before filter  https://review.openstack.org/59197603:30
*** nicolasbock has quit IRC03:36
*** e0ne has joined #openstack-placement05:23
*** e0ne has quit IRC05:24
*** e0ne has joined #openstack-placement05:28
*** e0ne has quit IRC05:29
openstackgerritMerged openstack/nova master: Fix TypeError in nova-manage cell_v2 list_cells  https://review.openstack.org/59986105:38
*** e0ne has joined #openstack-placement05:43
openstackgerritfupingxie proposed openstack/nova master: Add an example to add more pci devices in nova.conf  https://review.openstack.org/59224305:46
*** e0ne has quit IRC05:47
*** lei-zh has quit IRC05:48
*** e0ne has joined #openstack-placement05:48
*** e0ne has quit IRC05:50
*** e0ne has joined #openstack-placement05:51
*** e0ne has quit IRC05:57
*** e0ne has joined #openstack-placement06:16
*** e0ne has quit IRC06:18
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Detach and attach boot volumes - Stein  https://review.openstack.org/60062806:26
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node  https://review.openstack.org/58289906:28
*** lei-zh has joined #openstack-placement06:36
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec  https://review.openstack.org/60060806:43
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node  https://review.openstack.org/58289906:44
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node  https://review.openstack.org/58289906:53
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Spec to add per instance timeout for LM - Stein  https://review.openstack.org/60061306:55
openstackgerritRadoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler  https://review.openstack.org/59861107:03
openstackgerritRadoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler  https://review.openstack.org/59861107:04
openstackgerritChen proposed openstack/nova master: doc: update info for hypervisors  https://review.openstack.org/59955407:08
openstackgerritChen proposed openstack/nova master: doc: update info for hypervisors  https://review.openstack.org/59955407:36
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Update launch-from-volume doc to latest version.  https://review.openstack.org/57224108:03
openstackgerritYikun Jiang (Kero) proposed openstack/placement master: Refresh maximum version info in rest history doc  https://review.openstack.org/60067208:15
*** lei-zh has quit IRC08:23
*** ttsiouts has joined #openstack-placement08:31
*** e0ne has joined #openstack-placement08:32
openstackgerritSilvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts  https://review.openstack.org/52224509:07
openstackgerritChen proposed openstack/nova master: Use six.string_types to improve python2/3 compatibility  https://review.openstack.org/60067709:07
*** mriedem has joined #openstack-placement09:23
*** nicolasbock has joined #openstack-placement09:39
*** nicolasbock has quit IRC09:40
*** e0ne has quit IRC09:47
*** ttsiouts has quit IRC09:59
*** tetsuro has joined #openstack-placement09:59
*** tetsuro has quit IRC10:10
*** cdent has joined #openstack-placement10:13
* cdent waves10:14
openstackgerritArtom Lifshitz proposed openstack/nova master: DNM: Use claim context during live migration  https://review.openstack.org/57622210:17
*** cdent has quit IRC10:23
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder  https://review.openstack.org/58547510:35
openstackgerritMerged openstack/nova master: Update volume-attachment API url in policies  https://review.openstack.org/59672611:15
openstackgerritMerged openstack/nova master: Docs: update link for remote debugging  https://review.openstack.org/59131611:16
openstackgerritMerged openstack/nova master: Add encrypted volume support to feature matrix docs  https://review.openstack.org/57025511:16
openstackgerritMerged openstack/nova master: fup: Fix import order and test nit  https://review.openstack.org/60047411:16
*** nicolasbock has joined #openstack-placement11:29
*** rubasov has joined #openstack-placement11:30
*** ttsiouts has joined #openstack-placement11:43
*** cdent has joined #openstack-placement11:49
efriedō/12:14
*** efried is now known as fried_rice12:14
fried_ricecdent: https://review.openstack.org/#/admin/groups/1936,members12:18
cdenthuzzah, thanks12:19
openstackgerritKonstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Modelling passthrough devices for report to placement  https://review.openstack.org/59103712:36
edleafehuzzah to cdent!12:42
*** e0ne has joined #openstack-placement12:49
cdentI guess I better merge something12:49
openstackgerritRadoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler  https://review.openstack.org/59861112:54
openstackgerritMerged openstack/placement master: Fix aesthetic issues from I4974a28de541aace043504f  https://review.openstack.org/59984613:00
openstackgerritMerged openstack/placement master: Rename PlacementPolicyFixture to PolicyFixture  https://review.openstack.org/60032613:01
openstackgerritMerged openstack/placement master: Unify utils.py and util.py  https://review.openstack.org/60035313:03
edleafecdent: fried_rice: I have to take my car in for service, and will have unknown connectivity for the next few hours.13:12
openstackgerritMerged openstack/nova master: Configure placement DB context manager for nova-manage/status  https://review.openstack.org/59974413:15
cdentedleafe: noted. If Im gone by the time you're back see you in Denver.13:23
cdentIt's gonna be a damn shame if/when we decide to add integration/tempest tests to the placement's gate. Right now no single job takes more than 5 minutes.13:24
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge extended_status extension response into server view builder  https://review.openstack.org/59209213:34
dansmithfried_rice: on the placement nova.conf patch thing,13:41
* fried_rice listens13:42
dansmithfried_rice: it seems like you've just completely papered over my opinion on the thing and said that jaypipes' concern is the only one we need to address or something13:42
dansmithis that the intent? I haven't seen anyone address my concerns of pointing placement, with its own schema (which happens to be identical to nova-api today) at a database with a different schema13:43
fried_ricedansmith: I may have misread the wall-o-comments, but I *thought* you had voiced in favor of the try-placement.conf-then-nova.conf-with-warning13:43
fried_ricedansmith: I wasn't intending to dismiss that at all.13:43
fried_riceapologies if it came across that way13:43
dansmithno, I said I think we shouldn't,13:43
dansmithbut if we do we should log holy hell about it13:44
fried_riceI was saying, we should do that --^ now, because it would be easy to miss later, and then discuss the full strategy in a different and more durable forum.13:44
dansmithhowever, I'm reading the comments in the review instead of in my gerrit notifications13:44
dansmithand noticing that those comments came in while I was writing mine13:44
dansmithbut seemed like after based on when I read the gerrit mails13:44
fried_riceoh, okay. Yes, I noticed you had come in after with some other words, but haven't gotten back to it yet since then.13:45
fried_riceI have been really buried the last couple of days, and it's only getting worse13:45
dansmithcombining your comments with mnaser's seeming agreement that we should just read nova.conf I guess it sounded like it was just decided13:45
fried_riceWell, my review (like any review) was expressing an opinion.13:45
fried_riceI don't know that anything has been decided.13:46
dansmithis your point that we should only look at placement.conf in this change, and then propose the "or nova" behavior in a second one we can all argue about?13:46
fried_riceMy point was: My preference is to do the try-placement.conf-then-nova.conf-with-warning thing via this patch, or this plus a followon to be merged together; and move the discussion about the other gory aspects of upgrade to a spec.13:47
*** jaypipes is now known as leakypipes13:47
dansmithbummer13:48
fried_riceIf I misunderstood your commentary and you're truly opposed to the conf cascade idea, I apologize, but only for lumping you in with "everyone" when I said everyone had acked that idea.13:48
fried_riceDid I at least get it right that everyone *else* had acked that idea? Maybe I was out of my head more than I thought yesterday.13:49
dansmithI don't think cdent did13:51
* cdent is still kind of in love with the idea of two different wsgi apps: one that uses the nova conf and one that uses the placement conf. Choose a different app to get a different conf.13:51
dansmithcdent: I think that's a backwards way into the thing I'm concerned about,13:51
cdentyeah, understood13:51
dansmithwhich is operators treating the schemas as if they're the same13:51
leakypipesI share dansmith's general concern about the conf stuff w.r.t the DB. My primary concern is the situation where due to not finding a conf file the placement service loads default values and the deployer has no idea that happened.13:51
cdentand given that, I think the right thing is for the wsgi app to do one thing13:52
dansmithleakypipes: it's not going to load the database connection value properly, so what would that hurt?13:52
dansmithit'd fail to start and then they'd finish reading the docs?13:52
cdentyeah, I think what we want is something that fails early rather than guessing13:52
dansmithright, so that's why I say do a hard cutover13:53
* cdent marks the calendar13:53
dansmithif you guys need to make a schema change before the end of stein, this is all moot anyway13:53
cdentaw, c'mon dansmith "us guys" includes you13:53
dansmithcdent: sorry, I meant "placement" :)13:53
leakypipesdansmith: the default database connection is a sqlite file DB: https://github.com/openstack/nova/blob/master/nova/conf/database.py#L2113:54
dansmithleakypipes: ah, so that's your concern yeah?13:54
leakypipesdansmith: which would get auto-created13:54
dansmiththat's  the same for all the services, right?13:54
leakypipesdansmith: and the deployer would sit there and say "oh, why does my placement service no longer have any data in it"?13:54
dansmithI know because my glance uses sqlite :P13:54
dansmithleakypipes: okay, fair enough, but can we please address that somehow other than munging the schema problem into it?13:55
leakypipesdansmith: I don't know... I'm just trying to prevent situations where default conf values cause confusion during an upgrade.13:55
leakypipesdansmith: I'm definitely not trying to munge the two issues together. :(13:55
fried_ricerecognizing that there's more to this, doesn't the "conf cascade" address jay's main concern (unexpectedly empty/default values)?13:57
dansmithleakypipes: so, what if we refuse to start if there's no placement config, and if they create one and don't set connection= then we assume that's what they want, maybe with a warning?13:57
dansmithfried_rice: it does, but it triggers mine13:57
dansmithdespite me thinking it's very wrong13:57
dansmithin general13:57
fried_riceokay.13:57
fried_ricejust making sure I understand some part of this discussion13:57
dansmithI have to jump on my least favorite call of the week in a few13:58
dansmithcan we hangout this in an hour?13:58
cdentwfm, gives me a good window to pack13:59
dansmithI don't want to block us merging this patch, but I'd prefer either we leave out the cascade for the moment or just hash it out in an hour and find some solution13:59
dansmithprefer the latter13:59
dansmithAlso, mriedem's email to the list makes it sound like the cascade would be a way to avoid having to do some grenade work, which is not the right approach, IMHO14:00
dansmithand I'm willing to do the right grenade work to do the proper upgrade14:00
mriedemtbc,14:01
mriedemi was trying to summarize options from discussion yesterday for others - mostly ops and deployment project people14:01
mriedemnot saying, "this is what we decided"14:01
mriedemb/c i don't think we should really do this in a vacuum since it's a big change14:02
mriedemwe can say, "this is what we think we should do and why"14:02
mriedembut we should have some ack from ops14:02
mriedemIMO of course14:02
dansmithwell, I'm worried that operators will opt for the most automatic thing, despite it being known problematic14:02
mriedemthat's where the "this is why" comes in14:02
leakypipesdansmith: you prefer a hard failure to happen during "build" time, ok. that's probably the safest option. I just brought up my concerns because we've always been so obsessed with not breaking folks during an upgrade process. sorry for adding noise to the conversation. :(14:04
*** rubasov has quit IRC14:05
dansmithleakypipes: yeah, but we've already thrown that out the window because we're not going to have an overlap period14:05
dansmithso they have to install a new package, set up a new service, and all the things that come with it14:05
dansmithbelieve me, I'm sad we're not providing the overlap behavior, but alas14:05
leakypipesdansmith: oh, sorry, I didn't realize there was no overlap period. if I had realized that, I would not have pushed for this :(14:05
dansmithleakypipes: AFAIK, the plan is to have placement not runnable from the nova tree in the stein release14:06
mriedemdansmith: you should also reply in the ML thread, but maybe you're waiting until after we hash this out in an hour?14:06
*** rubasov has joined #openstack-placement14:07
dansmithmriedem: I've got a reply started14:07
mriedemmnaser expects this to be like standard config option deprecation handling, which you're asserting we aren't/can't14:07
mriedemwell, s/can't/shouldn't/14:07
dansmithright14:07
mriedemso we should be clear to him/other ops about why14:07
dansmithI thought I was clear in the review, but he didn't seem to even acknowledge it14:08
mriedemmight have just glossed over it, idk14:08
mriedemi'm sure he's busy14:08
mriedemi've added this as an item to the ops meetup and upgrade sig etherpads for the ptg14:29
mriedemupgrades sig is monday afternoon, ops meetup is all day tuesday14:29
dansmithI guess I'm being an asshole,14:30
dansmithbut I don't really understand why the ops opinion on whether or not we should do a broken thing matters14:31
dansmithif the point is to determine how freaked out they are about this in order to inform us doing _different_ things entirely, then that's fine14:32
*** rubasov_ has joined #openstack-placement14:32
*** rubasov has quit IRC14:32
mriedemi want any input from ops14:33
mriedemsince they have to deal with this upgrade, not me14:33
mriedemi fully realize they could say, "write a magical easy button that makes this all go away" which isn't an option14:33
mriedemif the majority of ops said you shouldn't extract this way and instead cutover in T, then that's something else we'd want to know - but would obviously throw a big wrench into the works14:35
dansmithack14:36
mriedembtw, did you get the many pings about grenade not even working properly in stein yet?14:36
mriedemi mean, it doesn't fail, but it doesn't work14:37
mriedemuntil https://review.openstack.org/#/c/591594/ merges anyway14:37
*** rubasov_ has quit IRC14:38
dansmithmriedem: no14:40
dansmithI'm off my call, are we going to have a hangout now?14:40
*** rubasov has joined #openstack-placement14:40
dansmithleakypipes: fried_rice cdent mriedem ^14:41
fried_riceI'm available14:42
cdentI can if we want. someone make a shout if/when14:42
* mriedem gets on his makeup14:42
edleafeI'm back too14:42
dansmithhttps://hangouts.google.com/call/LCkw2qSKsdXz_tzCaZg3AAEE14:42
openstackgerritArtom Lifshitz proposed openstack/nova-specs master: WIP: Re-propose numa-aware-live-migration spec  https://review.openstack.org/59958714:44
dansmithcdent: leakypipes: waiting on you14:44
cdentone sec14:45
dansmithleakypipes: cdent it's awkward staring at these guys14:45
dansmithhurry up14:46
edleafeYou'd rather stare at them?14:46
mriedemdoes my mic work?14:46
mriedemgo ahead and start14:47
cdentso much for packing15:16
cdentnaked PTG15:16
mriedemi'm sure the airport sells clothes15:17
mriedemrugged rocky mountain gear15:17
mriedemand i have a working mic now15:20
mriedemanyone want to hangout again?15:20
*** e0ne has quit IRC15:21
fried_riceheh15:22
mriedemdansmith: congratulations, you've sold mnaser15:28
* fried_rice reboots...15:29
*** fried_rice has quit IRC15:29
*** efried has joined #openstack-placement15:32
dansmithmriedem: mriedem \o/15:33
dansmithmriedem: that grenade target thing needs to happen every release, and I think sean just used to keep track of it15:41
mriedemi know; gmann had the patch up, but was failing due to some breakages in osc based on changes in glance/cinder at the end of rocky15:45
mriedemand then gmann was out so i think it was forgotten15:45
mriedemgmann actually did a lot of heavy lifting on unf'ing that for the community15:45
dansmithokay15:46
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Resource retrieving: add changes-before filter  https://review.openstack.org/59197616:10
*** mriedem is now known as mriedem_bbq16:22
*** ttsiouts has quit IRC16:27
*** cdent has quit IRC16:27
*** e0ne has joined #openstack-placement16:28
*** e0ne has quit IRC16:42
*** e0ne has joined #openstack-placement17:02
*** e0ne has quit IRC17:11
*** mriedem_bbq is now known as mriedem17:52
openstackgerritMerged openstack/nova-specs master: Resource retrieving: add changes-before filter  https://review.openstack.org/59197618:11
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec  https://review.openstack.org/60060818:18
openstackgerritMatt Riedemann proposed openstack/nova master: Fix formatting in changes-since guide  https://review.openstack.org/60015018:30
*** efried is now known as fried_rice18:56
openstackgerritEd Leafe proposed openstack/placement master: Rename files to remove 'placement'  https://review.openstack.org/60050619:26
openstackgerritDavid Bingham proposed openstack/nova stable/pike: Add apply_cells to nova-manage  https://review.openstack.org/59905019:35
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec  https://review.openstack.org/60060819:40
*** e0ne has joined #openstack-placement19:45
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Spec to add per instance timeout for LM - Stein  https://review.openstack.org/60061319:56
openstackgerritEric Fried proposed openstack/nova master: Add an example to add more pci devices in nova.conf  https://review.openstack.org/59224320:02
openstackgerritEd Leafe proposed openstack/placement master: Rename files to remove 'placement'  https://review.openstack.org/60050620:11
openstackgerritEd Leafe proposed openstack/placement master: Remove old pathing references from the docs  https://review.openstack.org/60085720:11
openstackgerritEric Fried proposed openstack/placement master: De-nova-ify and reformat contributor guide  https://review.openstack.org/59986420:12
openstackgerritEric Fried proposed openstack/placement master: Name arguments to _get_provider_ids_matching  https://review.openstack.org/60085820:15
mriedemfried_rice: there are some broken links in https://review.openstack.org/#/c/600268/ - do we care about those now or follow ups?20:24
fried_ricemriedem: Definitely definitely don't care about them now. Per commit message, the intent was *just* to get it passing the build; much to fix up later.20:24
fried_ricecontent-wise20:24
mriedemok20:25
openstackgerritMerged openstack/placement master: Add api-ref job  https://review.openstack.org/59952420:27
openstackgerritMerged openstack/placement master: Make docs build  https://review.openstack.org/60026820:34
openstackgerritMerged openstack/placement master: Use templates in .zuul.yaml  https://review.openstack.org/60045920:46
openstackgerritMerged openstack/placement master: Add lower-constraints job  https://review.openstack.org/60046020:46
openstackgerritMatt Riedemann proposed openstack/nova master: Update docs for live_migration_progress_timeout option  https://review.openstack.org/60086420:57
*** e0ne has quit IRC20:57
*** tetsuro has joined #openstack-placement21:14
*** mriedem is now known as mriedem_mowtown21:28
openstackgerritMerged openstack/os-traits stable/rocky: Update .gitreview for stable/rocky  https://review.openstack.org/58610121:33
openstackgerritMerged openstack/os-traits stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/58610221:33
*** tetsuro has quit IRC21:46
*** tetsuro has joined #openstack-placement21:46
openstackgerritEric Fried proposed openstack/placement master: Remove redundant reference to nova ConfFixture  https://review.openstack.org/60087321:52
*** tetsuro has quit IRC22:10
openstackgerritMerged openstack/nova master: Fix formatting in changes-since guide  https://review.openstack.org/60015022:48
*** tetsuro has joined #openstack-placement22:54
*** tetsuro has quit IRC22:58
*** rubasov has quit IRC23:06
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Network bandwidth resource provider  https://review.openstack.org/59524323:18
*** mriedem_mowtown is now known as mriedem23:19
openstackgerritMatt Riedemann proposed openstack/nova master: Update docs for live_migration_progress_timeout option  https://review.openstack.org/60086423:22
*** mriedem has quit IRC23:28
openstackgerritMerged openstack/nova-specs master: Network bandwidth resource provider  https://review.openstack.org/59524323:31
openstackgerritJonte Watford proposed openstack/nova master: Modified version of 0027-Numa-object-string-representations.patch with some updates from the current numa files for nova: numa.py instance_numa_topology.py  https://review.openstack.org/60026923:40

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!