Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-10-03
15:44:37 mriedem but only on thursdays
15:45:20 efried Also noting that reviews from experienced devs are valuable even if they aren't able to approve
15:45:48 mriedem examples would be good for thinking through this,
15:46:14 mriedem so on dan's spec, if i am listed as an official sponsor, it's not because i need to hold dan's hand, it's because i'm signing up to be an official reviewer feet to the fire person
15:46:25 mriedem and he has all rights to harass me in irc if i do'nt do so
15:46:37 efried yes
15:46:53 mriedem on the other hand, i could approve some non-core newb contributor but not sponsor them and absolve myself of all review responsibility
15:47:13 mriedem and thus being quite a dick
15:47:40 efried sure, I'm not trying to create a process to prevent dickishness.
15:47:43 openstackgerrit Dan Smith proposed openstack/nova master: Add reserved schema migrations for Ussuri https://review.opendev.org/686411
15:47:48 mriedem in reality i just wouldn't approve the spec
15:47:52 mriedem if i don't intend to review it
15:48:16 dansmith that's what I'm hearing,
15:48:20 efried We need more than one reviewer (hopefully more than two, but I'm a realist) to approve code anyway.
15:48:27 dansmith and I think it's important to make sure all the cores get that message
15:49:01 efried If we had some way of making that message clear and making it stick, I would be all over it.
15:49:46 mriedem it probably needs to at least be documented in the specs review for code review process
15:49:52 mriedem *specs repo
15:50:33 mriedem https://specs.openstack.org/openstack/nova-specs/readme.html#specification-review-policies
15:52:14 mriedem i have voted on the liaison patch forewith
15:52:18 mriedem forthwith?
15:52:26 mriedem yes
15:55:31 efried thanks
15:57:06 mriedem btw i know this isn't fun,
15:57:08 mriedem welcome to process land
15:57:14 mriedem and my world for 2 years
15:57:23 efried thanks
15:57:39 mriedem yo'ure also very handsome
15:58:25 gmann efried: i have re-proposed the policy spec for ussuri - https://review.opendev.org/#/c/686058/
16:02:26 efried mriedem: is that a thing? Three thank-yous and the bot automatically picks it up?
16:02:34 efried I'll have to be more careful.
16:02:36 efried gmann: ack
16:02:46 mriedem efried: ha, no
16:02:51 mriedem not what i meant
16:03:24 efried gmann: stay tuned for an update on how we're going to be deciding on approvals (once we figure it out ourselves...)
16:06:00 gmann efried: you mean the on ML thread ? but this is re-proposal of already approved spec
16:06:54 efried gmann: yup, that's understood. We're leaning towards making even those conform to the same criteria.
16:07:12 gmann humm
16:08:05 efried gmann: mriedem makes a good point about needing to update the nova-specs readme accordingly, because previously-reviewed specs will need more than just a random single-core +2 now.
16:08:15 gmann I thought re-approved things are fast approved one and new one goes into that criteria
16:08:46 efried gmann: not an unreasonable assumption. We're still working out the details.
16:09:52 dansmith gmann: that's being changed
16:09:56 dansmith well, potentially
16:10:25 dansmith efried: fwiw, I think the re-approval of specs has always been flawed because we just kick things down the road instead of requiring them to muster enough support to be approved again
16:10:28 cdent if a pre-approved spec didn't get done in the previous cycle, one would think that it deserves extra inspection to see if it is maybe a lost cause
16:10:39 cdent yeah, what dansmith said
16:10:42 efried ++
16:10:42 dansmith yup
16:11:08 efried requiring the sponsor to +2 might be enough
16:11:59 efried or we could just completely do away with official special treatment, understanding that previously-approved specs will probably require less technical scrutiny just naturally.
16:17:11 mriedem dansmith: i hope i answered your question in https://review.opendev.org/#/c/686047/ and the one after it
16:23:44 dansmith mriedem: not really, but doesn't matter
16:24:55 SonPham hi
16:25:23 SonPham any way to create live-snapshot + keep vms state ( process) ?
16:29:59 gmann ok, i will wait for the new process or let me know if spec sponsor thing is final so that i can add the same in policy spec.
16:37:14 mriedem would be nice if someone can check my assertions here http://lists.openstack.org/pipermail/openstack-discuss/2019-October/009937.html
16:37:19 mriedem i'm writing a bug
16:42:05 openstack Launchpad bug 1846527 in OpenStack Compute (nova) "online_data_migrations docs don't mention using --config-file to run the migrations per cell db" [Undecided,New]
16:42:05 mriedem mgoddard: https://bugs.launchpad.net/nova/+bug/1846527 if that makes sense to you
16:43:57 mgoddard mriedem: the --config-file option is useful, but the upgrade doc doesn't say ODM needs to be run multiple times at all
16:44:43 mriedem multiple times meaning once per cell db right?
16:44:47 mgoddard mriedem: would be nice if it just clearly said "run at least once with an API DB connection, and for each cell (including cell0?)"
16:44:49 mgoddard mriedem: yeah
16:44:53 mriedem right
16:45:19 mgoddard --config-file option makes that easier for some
16:46:55 SonPham hi If I want to suggest a new feature? What do I do after writing the blueprint?
16:47:06 mriedem an --all-cells would make it uber easy for deployment tooling
16:47:10 mriedem *option
16:47:32 mgoddard mriedem: +1
16:47:34 mriedem SonPham: https://docs.openstack.org/nova/latest/contributor/blueprints.html
16:48:28 mgoddard mriedem: is it possible to run ODM for a cell DB before other cells have been upgraded?
16:49:07 mgoddard just wondering about the upgrade one cell at a time scenario, should I include the data migrations as part of that cell's upgrade, or all together at the end?
16:49:07 mriedem possible? sure
16:49:17 mgoddard advisable?
16:49:47 mriedem i'm not sure
16:49:48 mriedem dansmith: ^?
16:50:04 mriedem i would think it's ok to do it per cell upgrade
16:50:12 dansmith mriedem: mgoddard at the end
16:50:24 mgoddard dansmith: ack, thanks
16:50:44 dansmith mriedem: do you mean target the cell and do data migrations just within that cell?
16:50:53 mriedem i think that's the question yeah
16:51:10 dansmith I mean, I guess that's okay, but since they don't need to be run until later, I would just do them later I guess,
16:51:19 dansmith and if the api is all upgraded first then those migrations would be fine
16:51:29 dansmith so I guess either way, but I'd just do it later personally
16:51:33 dansmith mgoddard: ^
16:51:46 mriedem mayhap we should be capturing some of this in the faqs https://docs.openstack.org/nova/latest/user/cells.html#faqs
16:52:06 mriedem not it
16:58:43 efried Does takashin usually propose the move-implemented-specs thing?
16:58:58 efried we're probably ready for that now
17:01:20 mriedem idk, just do it
17:01:22 openstackgerrit Matt Riedemann proposed openstack/nova master: Use scatter/gather in HostAPI.compute_node_get https://review.opendev.org/686442
17:01:34 mriedem takashi usually does the 'prep for the next release' thing
17:01:39 efried fiber mriedem. fiber.
17:01:48 mriedem delegate to someone else to do it
17:01:49 mriedem that's not me
17:02:17 mriedem dude, you want to talk about fiber. i've been eating raisin bran for breakfast and leftover chili for lunch most of this week.
17:02:25 efried ye gods
17:02:34 mriedem i only put together the reason for the side effects last night
17:05:04 sean-k-mooney lol isnit awsome that everything we type is archive for ever :)
17:06:21 dansmith artom: by "double download niggle" you mean "what happens if an image is re-cached" and/or "what happens if two requests come in for the same image quickly" right?

Earlier   Later