Thursday, 2019-05-16

*** diablo_rojo has quit IRC00:45
*** ricolin has joined #openstack-tc00:50
*** lbragstad has quit IRC01:42
*** lbragstad has joined #openstack-tc01:51
*** whoami-rajat has joined #openstack-tc02:06
*** dklyle has joined #openstack-tc02:14
*** dhellmann has quit IRC02:23
*** dhellmann has joined #openstack-tc02:23
*** lbragstad has quit IRC02:40
*** lbragstad has joined #openstack-tc02:43
*** lbragstad has quit IRC04:59
*** Luzi has joined #openstack-tc05:40
*** e0ne has joined #openstack-tc05:53
*** sarob has joined #openstack-tc06:09
*** sarob has quit IRC06:19
*** e0ne has quit IRC06:36
*** openstackgerrit has quit IRC07:03
*** jaosorior has quit IRC07:06
*** tosky has joined #openstack-tc07:48
*** e0ne has joined #openstack-tc08:08
*** dtantsur|afk is now known as dtantsur08:50
*** tosky has quit IRC10:10
*** tosky has joined #openstack-tc10:11
*** jaosorior has joined #openstack-tc10:12
*** zhipeng has joined #openstack-tc10:22
*** ricolin has quit IRC10:39
asettleAFternoooon o/11:00
jroll\o11:28
*** dtantsur is now known as dtantsur|brb11:46
*** lbragstad has joined #openstack-tc13:09
*** mriedem has joined #openstack-tc13:15
smcginnisMooooorning o/ :)13:16
*** dtantsur|brb is now known as dtantsur13:18
*** ianychoi has joined #openstack-tc13:26
*** Luzi has quit IRC13:45
*** openstackgerrit has joined #openstack-tc14:43
openstackgerritSean McGinnis proposed openstack/governance master: Nest tag project output to make sorting look sane  https://review.opendev.org/65931014:43
ttxohai15:01
fungilooks like it's office hour time15:01
zanebo/15:01
ttxI'd like to raise 3 reviews that look stuck15:02
ttxor missing some input from y'all15:03
ttxhttps://review.opendev.org/#/c/655484/15:03
ttx(the winstackers / compute-hyperv one)15:03
ttxi.e. how much do we want two hyperv drivers "in" OpenStack15:04
ttxany opinion on that?15:05
zanebit's worse than that; nobody wants two drivers but the one we do want isn't the one in OpenStack15:05
ttxheh yes15:06
ttxI guess one question would be to remove/deprecate the mainline one15:06
ttxs/question/solution15:06
ttxif nobody can recommend it being used15:07
ttxbasically 3 ways out - A/ have both in OpenStack (approve this review), B/ have only the mainline one in OpenStack (reject that review), C/ have only the winstackers one in openstack (approve review + deprecate the mainline one)15:08
smcginnisGreat detailed response from lpetrut on there.15:09
smcginnisBad yeah, bad situation.15:09
ttxYes, I feel like they are doing the best they can to get people to use the "working" driver15:09
fungithe project reform did allow for competing scope between projects as long as it was not gratuitous duplication of effort15:10
ttxIt's not competition, it's a process artifact15:10
ttxthat creates user confusion15:10
clarkbnova wanting drivers in tree being the process artifact?15:10
ttxclarkb: no, the fact that the driver in tree is not as advanced as the one out, because it takes time to merge things15:11
ttxMaybe we could have both and call one experimental and the other stable15:11
zanebtbh I don't understand why the nova teams takes such a... legalistic approach to things such as this and powervm. people are using this thing and as far as they're concerned it's OpenStack. Nova say they can't be associated with it unless they've reviewed every line of code. well... you're already associated with it so why be difficult?15:12
ttxI guess that is the key question -- which one is actually used15:12
ttxAnd if the people working on it and supporting the users using it are saying they'd rather support people using the experimental one... I'm not sure there is value in the in-tree one15:13
ttxI understand Nova's position that they need to vouch for the in-tree driver design / feature / patches15:14
zanebright, but nobody's suggesting that the experimental driver move in-tree15:14
ttxBut if that process ends up producing somethign that nobody uses because everyone is using the experimental out-tree driver...15:14
ttxzaneb: that would be D/ but I don't think that is a good idea unless Nova is comfortable with it15:15
zanebI think I'm recalling more the discussion around powervm, but this is essentially the same15:16
zanebfrom a TC perspective I think we should probably just approve this review and leave the teams to figure out what they want to do about the duplication15:16
ttxAt this point I'd rather push for C/ than for D/. And if that's not possible, I'm torn between A and B15:17
ttxzaneb: yes, so A with an eye toward potential C or D?15:17
zanebA with an eye toward C15:18
ttxThe problem with A being it perpetuates the user confusion as to what they should be using15:18
zanebor not if the teams so decide15:18
dhellmannit sounds like C is going to put the winstackers team in a position where changes in nova might break hyperv support, though15:19
dhellmannmaybe we can deal with that through gating15:19
zanebaren't they already in that situation15:19
zaneb?15:19
dhellmannapparently with the minimal in-tree driver, they're able to keep that to a minimum? I'm going based on comments on the review15:19
zanebright, but nobody is advised to use that15:21
dhellmannmy position on this has always been that 1 project team does not get to decide if another project team should exist. That's the TC's role. So I think approving this team and encouraging the two groups to sort out the confusion/testing issues is probably the right path forward15:21
dhellmann"We had to keep the upstream driver as well so that Nova would continue to officially support Windows/Hyper-V, otherwise anyone could introduce breaking changes"15:21
dhellmannthat's the comment I was referring to15:21
clarkbre gating we can't do functional testing of hyperv (but if they faked hyperv could probably test large bits of the driver)15:22
dhellmannI assume we would replicate whatever they're doing with that in-tree driver just using the 2nd repo15:23
dhellmannif there's no testing, then it's just false hope that nothing is going to end up broken anyway15:23
smcginnisIsn't that usually addressed by third party CI? I thought we had that for others. (VMware?)15:23
ttxso the hyperv's in tree driver only use is to ensure taht base functions won't get broken15:23
clarkbyes  Ithink third part ci usually fills those gaps (this isn't gating though)15:23
ttxit serves as an undocumented API15:24
smcginnisThey can be though.15:24
dhellmannright, this is unit-test level checks for API breaking changes15:24
dhellmannwe don't need third-party CI to do that15:24
ttxI guess that is what I meant by "process artifact"15:24
ttxIt serves as an intermediary by-product that should probably not be used by users15:25
ttxand yet it is what users see by default15:25
fungithis nova/hyperv discussion reminds me of situations like the qlogic storage drivers in the mainline linux kernel, where production deployments almost always just rebuilt the vendor's out-of-tree version and loaded that instead15:25
clarkbsmcginnis: third party ci cannot gate in our system15:25
clarkb(this is intentional)15:26
smcginnisclarkb: We no longer allow voting?15:26
clarkbsmcginnis: we allow voting, but only the votes from the zuul user control mergeability15:26
smcginnisAh15:26
fungiwe allow voting, we don't allow external ci systems to decide what will merge15:26
fungior block things from merging15:26
*** sarob has joined #openstack-tc15:27
dhellmannanother path forward is to have a declared stable driver interface, but I know some teams are reluctant to do that15:27
dhellmannthat would allow the in-tree driver to be removed safely, and would at least remove the confusion caused by duplicate drivers15:28
smcginnisThere are times when the interface does not change, but something else still is able to break functionality.15:28
ttxa stable API is one thing. Getting warned about changes in an unstable API is another.15:29
ttxMaybe the latter is enough in that case15:29
dhellmannsure15:29
dhellmannit would be good to get the nova team's perspective on this, too. I don't know how recently they were involved in this discussion.15:30
ttxI think the resistance to stable API was mostly around the ability to change it as necessary15:30
dhellmannit sounds like "supporting" out of tree drivers is a bit easier than it used to be15:30
ttxyes -- the question being, do we start the discussion before or after approving that review :)15:30
ttxThe other review I wanted to raise as undecided is https://review.opendev.org/#/c/657142/ -- the Docs SIG one15:31
zanebideally all of the drivers would be out of tree and the API would be stable. but that's not something we can make happen15:31
zanebwhat we can do is eliminate at least one piece of bureaucratic BS that contradicts the facts on the ground15:32
ttxthat one also could use a variety of views, sinc I'm very much on the fence15:32
zanebdhellmann: what is it about a team vs. a SIG that makes it "someone clearly signed up" vs "leaving it to chance"?15:36
zaneb(last comment on the docs review)15:36
smcginnisIsn't the issue that SIG membership is very light weight and easy to come and go? At least with a team, there's a PTL that's ultimately somewhat responsible.15:37
smcginnisThough I could see an argument that a SIG chair would be roughly the same.15:37
smcginnisBut that also makes me wonder why change it if things are basically the same.15:37
zanebwith a project it's more obvious when everybody has gone I guess, because of the PTL election15:37
zanebjust not sure that solves anything15:38
jrollFWIW, nova's virt API is stable in practice, there's just no guarantees15:38
*** e0ne has quit IRC15:38
ttxYes tbh I'm not sure where the idea came from, and teh commit message does not go at length to motivate the change15:38
jrollI've personally had to do extra work to make sure we don't break out of tree drivers15:38
*** ricolin has joined #openstack-tc15:38
ttx(talking about Docs SIG)15:38
jrollhonestly, this out of tree driver just feels like a way to work around review bandwidth, which isn't good but is understandable15:39
zanebnow that most of the docs are in project repos, most contributions to docs are not actually to the docs project. I assume that's what's motivating it?15:39
ttxjroll: also allows questionable shortcuts15:40
jrollttx: that's part of my "isn't good" statement :)15:40
smcginnisWould be good to get some input from asettle when it's her work hours.15:41
zanebjroll: the fact that it's actual stable in practice is what makes it extra-weird15:48
ttxThe third I had was the caching service, but I think a more visible ML thread is the real next step there15:48
ttxI feel like it's very important to not rush base service additions and take the pulse of the ops community on proposed additions15:49
ttxeven if in that specific case it should definitely make sense15:49
jrollzaneb: Lucian's comments on the patch somewhat reflect this as well - I think this out of tree driver came before the relative stability of the driver interface15:50
clarkbttx: (as someone not on the TC) it felt like the etcd base service move was premature too because very little actually uses it (and it took a long time to get there) seems like a better approach is to make something work well then decide to use it more globally rather than deciding globally then hoping it works well or that anyone will even use it15:51
ttxyes... it's always a bit of a chicken-egg problem15:52
cmurphytbh i'm torn on whether to keep pushing caching as a base service, i think it may not really need to be an openstack-wide commitment and the keystone team can probably just make the call that the keystone project alone depends on caching15:52
ttxThe hope is that the news trigger a virtuous cycle but in that case it did not work that well15:52
fungii still intend to propose barbican to base services, castellan was a good first step though15:52
ttxcmurphy: how is it most commonly deployed ? With local memcached on keystone nodes ?15:53
cmurphyttx: usually sharded memcached across keystone nodes15:54
zanebthe good news is that nobody is going to choose not to use keystone in their deployment because it requires memcached15:54
ttxI feel like caching is generally sufficiently sharded that I'm not sure I'd call it a "service" like MySQL or Rabbit15:54
ttxMaybe adding it as a dependency would be enough15:55
zanebttx: other services can make use of the same memcached though15:55
zanebalso there's nothing to stop you from running a separate DB for each service15:55
zanebI don't feel like this is a qualitatively different case15:56
ttxfair15:56
mugsieI don't see the issue really - we added etcd3 before any services actually used it, and from what I can figure out, none of them require it yet15:59
clarkbmugsie: the issue is groups like infra and qa invested a ton off effort for getting it in place and its basically doa16:01
mugsieclarkb: oh, i know - we already have this in place for keystone thought - all of the keystone gates already use it16:02
ttxyeah, if we add a base service and nobody requires it yet, maybe we should remove it16:02
ttxbecause the goal was to be able to actually depend on it being present16:02
clarkbmugsie: ah I see so this caching case is more like what I describe in proving it first (I got the other impression from ttx's comments earlier)16:02
ttxat least in the caching case we have one project signed up to hard-depend on it16:03
smcginnisEtcd is used in some Cinder tests for distributed locking.16:05
*** dims has quit IRC16:05
*** sarob has quit IRC16:16
*** dtantsur is now known as dtantsur|afk16:19
*** sarob has joined #openstack-tc16:26
*** sarob has quit IRC16:32
*** ricolin has quit IRC16:44
dhellmannmy impression of the docs transition is that there's basically no one left contributing to the current team, and operators who might contribute to docs do not want to do it as part of a "team" because they perceive it as burdensome to follow the team processes. so, this is mostly a recognition of the fact that the team largely dissolved over the last year but the tools are still being maintained16:49
dhellmannzaneb : "clearly signed up" in the sense that there is someone who has committed to do it as part of the structure we have for managing producing "openstack", which SIGs aren't automatically doing. so if we're OK with the web site potentially not reflecting the current release status correctly, then we can leave it alone. but if we are not ok with that, we need someone to say they will do it every cycle.16:52
dhellmannI don't care what that group is called, but my impression is that SIGs have a more "when we get around to it" nature than I'm comfortable with for that task.16:53
*** bauzas has quit IRC16:54
zanebteams also have a tendency to evaporate (as you point out, the docs team already mostly did)16:54
clarkbdoes making it a sig unevaporate people?16:54
zaneb"operators who might contribute to docs do not want to do it as part of a "team" because they perceive it as burdensome to follow the team processes"16:56
zaneb(I have no idea if that is actually true)16:56
*** bauzas has joined #openstack-tc16:57
mugsieI would worry that without the team processes we would have a lot of random wiki pages in varing states of decay - which is what happened with the ops guide16:58
mugsieand I am not sure a sig is going to remove the need to use gerrit16:59
fungiin fact the ops guide editors tried relocating the content to a wiki and ultimately decided they'd rather stuff it back into gerrit17:01
*** dims has joined #openstack-tc17:03
fungiso on the expectations of people front, if there is effectively already no docs team except on paper (pun?), then dissolving it officially just seems like a reflection of reality whether it's a reality we like or not17:04
*** sarob has joined #openstack-tc17:05
fungiwhether there are folks interested in writing documentation who don't form a cohesive team is somewhat orthogonal in my mind17:05
*** lbragstad has quit IRC17:05
fungibut if there are and they have sufficient interest in forming a sig around it, then that works for me17:06
zanebyeah, so there are some folks who maintain the docs tooling, and I think everyone agrees that makes sense as a SIG. Doug's point is that there are also actual documentation repos to be maintained, and stuff to do on every release, and a team should handle that17:07
fungiso i'd approach it as two things: 1. official dissolution of the already vacant documentation team, 2. people expressing a desire to form a documentation sig17:07
zanebalthough it's not clear that those latter people exist in either scenario17:07
zanebthat's fair, but step 1 also involves deciding where the repos currently owned by that team end up17:11
*** dims has quit IRC17:12
*** lbragstad has joined #openstack-tc17:13
*** dims has joined #openstack-tc17:13
fungiand my counter to dhellmann's concern is mainly that if the assertion is there's nobody around any longer who wants to do the documentation-related stuff which needs to happen at each release, declaring a team around it won't necessarily make that any less the case17:15
fungiso we need a plan to distribute that work to the folks who have a vested interest in seeing it continue17:15
fungiand if nobody has a vested interest in seeing it continue, then maybe it's not as important as we think it is17:16
zanebdhellmann also suggested moving those repos to the release management team iirc17:16
*** dklyle has quit IRC17:34
*** sarob has quit IRC17:35
-openstackstatus- NOTICE: Gerrit is being restarted to add gitweb links back to Gerrit. Sorry for the noise.17:37
*** ianychoi has quit IRC18:15
dhellmannyeah, I don't want to make a new team, I want someone to pick up the work that used to be done by people that aren't here any more18:55
dhellmannteams do also evaporate, but if we go by ttx's definition of what makes a team vs. a sig, then I think updating docs.openstack.org each cycle to reflect the current state of each release is part of the process of building that release and should be done by a team18:57
fungiand that could be done by infusing the existing team with new people, or by further distributing that work to other existing teams with a vested interest in the work continuing18:57
dhellmannthe "existing team" is basically Stephen, so I don't know about that approach18:57
dhellmannwell, and of course asettle18:58
dhellmannwe have lots of hypothetical solutions to the problem. I'm trying to point out that we need to pick a realistic one. I proposed one such realistic solution in my comments on the patch.18:58
fungibut if the aggregation of documentation-interested folks isn't going to have time to do that per-cycle routine work, then i don't see that telling them they can't form a sig around documentation interests solves the problem either18:59
dhellmannI didn't say anything about creating a sig18:59
dhellmannI don't like the patch as proposed because it moves a repo to a sig that I think should stay owned by a team18:59
dhellmannthe rest of it is fine19:00
fungiyep, that much makes sense19:00
*** sarob has joined #openstack-tc19:27
*** whoami-rajat has quit IRC20:05
*** sarob has quit IRC20:09
*** e0ne has joined #openstack-tc20:10
*** e0ne has quit IRC20:12
*** diablo_rojo has joined #openstack-tc20:48
*** sarob has joined #openstack-tc21:06
*** mriedem has quit IRC21:53
*** tosky has quit IRC21:59
*** sarob has quit IRC22:12
*** dklyle has joined #openstack-tc23:21
*** diablo_rojo has quit IRC23:24
*** dklyle has quit IRC23:35
*** diablo_rojo has joined #openstack-tc23:52

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