Wednesday, 2020-02-19

*** slaweq has joined #openstack-tc00:11
*** slaweq has quit IRC00:15
*** slaweq has joined #openstack-tc01:10
*** slaweq has quit IRC01:15
*** jamesmcarthur has joined #openstack-tc01:17
*** jamesmcarthur has quit IRC01:39
*** jamesmcarthur has joined #openstack-tc01:51
*** jamesmcarthur has quit IRC02:01
*** slaweq has joined #openstack-tc02:10
*** slaweq has quit IRC02:16
*** purplerbot has quit IRC02:34
*** purplerbot has joined #openstack-tc02:36
*** sapd1_ has quit IRC02:43
*** sapd1_ has joined #openstack-tc02:44
*** jamesmcarthur has joined #openstack-tc02:48
*** slaweq has joined #openstack-tc03:10
*** slaweq has quit IRC03:16
*** jamesmcarthur has quit IRC03:26
*** jamesmcarthur has joined #openstack-tc03:48
*** slaweq has joined #openstack-tc04:10
*** slaweq has quit IRC04:16
*** lbragstad has quit IRC04:22
*** jamesmcarthur has quit IRC04:58
*** jamesmcarthur has joined #openstack-tc04:59
*** jamesmcarthur has quit IRC05:07
*** jamesmcarthur has joined #openstack-tc05:10
*** slaweq has joined #openstack-tc05:11
*** slaweq has quit IRC05:16
*** jamesmcarthur has quit IRC05:22
*** jamesmcarthur has joined #openstack-tc05:28
*** jamesmcarthur has quit IRC05:32
*** evrardjp has quit IRC05:34
*** evrardjp has joined #openstack-tc05:34
*** jamesmcarthur has joined #openstack-tc06:08
*** slaweq has joined #openstack-tc06:11
*** jamesmcarthur has quit IRC06:14
*** slaweq has quit IRC06:15
*** jamesmcarthur has joined #openstack-tc06:30
*** jamesmcarthur has quit IRC06:35
*** openstack has joined #openstack-tc07:19
*** ChanServ sets mode: +o openstack07:19
*** openstack has joined #openstack-tc07:43
*** ChanServ sets mode: +o openstack07:43
*** slaweq has joined #openstack-tc07:50
*** tetsuro has joined #openstack-tc08:12
*** tetsuro__ has joined #openstack-tc08:13
*** tetsuro_ has quit IRC08:14
openstackgerritMerged openstack/governance master: Introduce 2020 upstream investment opportunities.  https://review.opendev.org/70712008:16
*** tetsuro has quit IRC08:16
*** tetsuro has joined #openstack-tc08:17
*** tetsuro__ has quit IRC08:21
*** jamesmcarthur has joined #openstack-tc08:31
*** mwhahaha has quit IRC08:34
*** jbryce has quit IRC08:35
*** jbryce has joined #openstack-tc08:36
*** jamesmcarthur has quit IRC08:36
*** tosky has joined #openstack-tc08:37
*** mwhahaha has joined #openstack-tc08:37
*** rpittau|afk is now known as rpittau08:37
*** tosky has quit IRC08:51
*** tosky has joined #openstack-tc08:51
*** tetsuro_ has joined #openstack-tc09:09
*** tetsuro has quit IRC09:09
*** Jeffrey4l has quit IRC09:12
*** Jeffrey4l has joined #openstack-tc09:13
*** tetsuro has joined #openstack-tc09:21
*** tetsuro_ has quit IRC09:25
*** tetsuro has quit IRC09:27
*** e0ne has joined #openstack-tc09:41
*** slaweq has quit IRC11:06
*** slaweq has joined #openstack-tc11:11
*** slaweq has quit IRC11:32
*** slaweq has joined #openstack-tc11:34
*** rpittau is now known as rpittau|bbl11:40
*** rpittau|bbl is now known as rpittau13:02
*** jamesmcarthur has joined #openstack-tc13:17
*** ijolliffe has joined #openstack-tc13:33
*** jamesmcarthur has quit IRC13:36
*** lbragstad has joined #openstack-tc13:47
*** jamesmcarthur has joined #openstack-tc13:48
*** lbragsta_ has joined #openstack-tc13:52
*** lbragstad has quit IRC13:54
*** jamesmcarthur has quit IRC14:14
*** jaosorior has joined #openstack-tc14:16
*** lbragsta_ has quit IRC14:20
*** slaweq has quit IRC14:27
*** slaweq has joined #openstack-tc14:30
*** jamesmcarthur has joined #openstack-tc14:32
*** jamesmcarthur_ has joined #openstack-tc14:33
*** jamesmcarthur has quit IRC14:36
*** jaosorior has quit IRC14:38
*** irclogbot_3 has quit IRC14:56
*** irclogbot_3 has joined #openstack-tc15:02
*** lpetrut has quit IRC15:11
zanebfungi: I really think we are making a mistake with https://review.opendev.org/70851115:37
smcginniszaneb: Sorry, I completely missed your commments on there.15:41
zanebjust saw that :D15:41
smcginnisFeel free to revert. I do actually agree with what you're saying there.15:42
fungiif the important information you refer to appears in the first paragraph then i think folks are just as likely to read it as if it's the only paragraph15:42
zanebfungi: I don't think that's how folks work15:43
fungiyour assertion is that if they see there's a lot of information there, then they'll read none of it? that's remarkably cynical15:43
zanebyes, and yes. but is it inaccurate?15:46
zanebwhen was the last time you read an EULA?15:46
smcginnisfungi might be the one person I would expect to answer that with a fairly recent response. :D15:47
zanebfungi might be the one person who has never even encountered one15:48
fungii do read software licenses, but i understand i'm generally brain-damaged in that regard15:49
fungiit seems like we should be able to highlight the relevant important paragraph at the top in a note:: type admonition to draw the eye to it (provides attention-getting indentation in plain-text view and an alternate background color in rendered rst/html)15:51
zanebpeople *especially* don't read those little note boxes. they look like ads. eyes float right over them without even seeing15:53
fungibut also this is the cookiecutter template. if new projects are going to mostly refer back to the contributor guide for common tasks then the initial CONTRIBUTING.rst is likely to be small anyway. as their community grows in complexity they can of course make the decision to stick some content in the doc/source/contributor/contributing.rst file which isn't covered by the include (or split them entirely)15:53
fungianyway, by your logic, it sounds like most potential contributors aren't going to read no matter how much or how little information we put in front of them. they probably also aren't going to read our responses on their changes if they do figure out how to send in a patch either15:54
zanebthe reason we're talking about the cookiecutter is that we intend to align every project with this by a community goal due by the end of this cycle15:55
zanebdespite the fact that halfway through the cycle we still can't agree on what we want people to do :(15:55
fungiexcept we're not aligning them because we're still allowing them to document their own team-specific workflows15:55
fungiif we really were "aligning" them then we could get them all to use one consistent CONTRIBUTING.rst file which refers back to the central contributor guide, and then stop with the extra content in doc/source/contributor/...15:58
zanebwe have that now! it's great! we just wanted to add a place to document stuff like PTL duties. but now that place is going to be in CONTRIBUTING.rst where it gets shoved in people's faces when they're just trying to open a pull request16:00
fungiyeah, but moving all the instructions on how/where to propose patches out of the CONTRIBUTING.rst file makes that harder16:01
zanebI don't understand, we're not moving anything out. in e.g. https://review.opendev.org/708672 CONTRIBUTING.rst stays exactly the same as it has always been and just adds a link to the project-specific docs16:02
zanebin addition to the general openstack contribution docs that were already linked16:03
openstackgerritMerged openstack/governance-sigs master: Merge Auto-scaling SIG and Self-healing SIG  https://review.opendev.org/70838116:04
fungihaving some extra information in doc/source/contributor/contributing.rst which isn't in CONTRIBUTING.rst isn't my main concern (i'm not as cynical as you on the risk of putting it all in CONTRIBUTING.rst but that's a separate issue), my problem was with https://review.opendev.org/70773516:09
fungie.g., removing things like "pull requests should not be submitted through github" from the CONTRIBUTING.rst file16:09
fungiand expecting people to dig up some other file by reference to discover that. because i *am* of the opinion that folks are even less likely to go find additional documents you refer to than they are to read the information you put in front of them16:11
zanebright, yeah, I hadn't reviewed that patchset https://review.opendev.org/#/c/707735/2/CONTRIBUTING.rst but I wouldn't have agreed with it either16:15
fungimy position on it (and i'm not even a core reviewer on that repo so take my -1 with whatever degree of weight you like) is that CONTRIBUTING.rst should at least directly contain text explaining the things we think people are likely to get wrong when trying to figure out how to submit a patch, file a bug or contact the maintainers16:19
fungisecondarily, i feel like duplicating content between CONTRIBUTING.rst and another document in the repo risks divergence over time, so CONTRIBUTING.rst should get transcluded within it instead of duplicated16:21
zanebthat's a fair point, but the only thing that's really duplicated is the bug tracker link16:23
fungitertiarily and leastly, i think having contributor information split between different files is messy unless the topics they cover are significantly divergent and the prose required for them is massive enough to warrant different pages of text entirely16:23
jrollFWIW, I think including content in a cookiecutter repo will also cause divergence over time, unless we have bots updating the older repos16:25
jroll(and people merging said updates)16:26
fungiyep, this is basically the issue we've had with any file in the cookiecutter repos16:27
fungievery project contains (some variation on) cookiecutter files contemporary with when the project was created16:27
jrollergo, a link to a central doc makes sense to me16:27
fungias for me, but there's still important information people won't follow that link to find out16:28
jrollyeah, fair point16:29
zanebto be fair, one advantage of having a project-wide goal is that we get to update everyone's cookiecutter-supplied doc16:29
jrollonce :)16:29
zanebyes :)16:29
fungiin particular, things you are likely to want to do with a copy of the source code (the repositories on github and elsewhere are read-only mirrors, if you want to propose a patch don't open a pull request instead read this, if you want to report a bug don't open a github issue instead go here, if you want to contact the maintainers don't use a github group message go here...)16:30
*** rpittau is now known as rpittau|afk16:48
*** tosky has quit IRC17:16
gmannzaneb: what if we highlight those 2 line in bold that can have clear attention from reader, better than from current CONTRIBUTING.rst - https://review.opendev.org/#/c/708511/217:25
zanebgmann: sorry, uploaded a new patch before I saw your comments on there17:27
gmanneither approach works for me. 1. BODLing the 2 line in modified CONTRIBUTING.rst havign all details or 2. zaneb approach of keeping detail separate and link in CONTRIBUTING.rst + my opinion of clearing CONTRIBUTING.rst more17:27
zanebgmann: so I changed it to have a link to opendev at the very top, followed by the line about ignoring PRs. what do you think of that? We could bold it as well17:29
* diablo_rojo tries to catch up with scrollback but is also double meeting-ing17:29
zanebdiablo_rojo: you're gonna wish you hadn't17:29
zaneb;)17:29
gmannzaneb: i think we can remove L1-L25 with those 2 liner and then link - https://review.opendev.org/#/c/708672/2/%257B%257Bcookiecutter.repo_name%257D%257D/CONTRIBUTING.rst17:31
gmannmy point was those infra manual, bug tracker and workflow link are duplicating of details in doc/source/contributing.rst17:31
zanebgmann: I think we need the bug tracker thing because people try to open issues on GitHub as well17:32
gmannzaneb: ok, or we can say 'no Github issue' also. because anyway users have to go to detail doc to learn about gerrit things.17:33
*** evrardjp has quit IRC17:34
*** evrardjp has joined #openstack-tc17:34
zanebI'm trying to address fungi's (valid) objection that we should also consider the audience of people just doing "cat CONTRIBUTING.rst"17:36
fungii consider that to be the primary purpose of having a shouty-named file in the top level of a source tree17:38
fungilike README or LICENSE17:38
diablo_rojozaneb, you are correct. I'm like 1/3 of the way through and feel a bit dizzy.17:38
fungidiablo_rojo: take your time, it'll keep17:39
diablo_rojozaneb, so. You dont like the patch that got merged because it doesn't loudly proclaim that we don't yell about not using github at the top of CONTRIBUTING.rst?17:39
diablo_rojoOr have I oversimiplified?17:39
diablo_rojofungi, will it keep till happy hour? ;)17:40
zanebfungi: that's fair, but when other tools (like GitHub) start treating those as special then you also have to consider how audiences who arrive through that tool will see it17:40
zanebwhich is hard, because writing for different audiences is hard17:40
fungiyep, i agree17:40
zanebdiablo_rojo: tl;dr I think we should do this: https://review.opendev.org/#/c/708672/2 (or something like it, pending any feedback from fungi)17:41
fungishe and i are in the same two-day-long meeting, but i'll definitely stick it on my priority list17:42
diablo_rojoMy general thoughts: I dont care where all the template lives, but I think everything should be in one place because we have a real problem with duplication.17:42
diablo_rojozaneb, I am largely fine with that patch, though I prefer the state of things right now with it all being in CONTRIBUTING.rst and just being connected to the other file. The one actual problem I have with that patch is that we need to link to the contributor guide and NOT the infra manual since that is being migrated out of TC oversight and going to be opendev specific..Or if we do want to link to it we also need the contributor guide and17:44
diablo_rojothen this page starts to get lengthy which I think was your concern in the first place.17:44
* diablo_rojo will go put all this on the review now17:45
diablo_rojowell maybe not now17:45
diablo_rojobut before the end of the day.17:45
gmannin that case, making two line Bold in starting of CONTRIBUTING.rst(one having all info) can solve zaneb concern of having everything in one file make it unreadable. but zaneb to validate :)17:47
fungiwell, the infra manual is going to be less openstack-specific, so it will be "opendev-general" you might say17:48
gmanni agree to link to contributor guide instead of infra manual. diablo_rojo i think we have those diagrams(infra manual and workflow) in contributing guide ?17:48
diablo_rojogmann, I am totally fine to put at the very top of CONTRIBUTING.rst (as it is right now) DONE USE GITHUB FOR PRs WE USE GERRIT.17:48
diablo_rojofungi, correct.17:48
gmannfungi: yeah or i thought i can be deleted in favor of ownership to contributor guide etc17:49
diablo_rojoI see the general flow of 'getting started' as being the opendev docs (nee infra-manual), then the contrib guide (which will be general openstack stuff) and then the project specific docs17:49
gmanndiablo_rojo: i cannot find the gerrit workflow in contributor guide. did we miss ?17:49
diablo_rojogmann, havent migrated that over, but it is a more opendev specific thing that will continue to live in the infra manual17:50
diablo_rojothe contributor guide should mention the opendev docs (infra manual) and then go on to talk about the openstack general stuff17:50
fungigmann: there's https://docs.openstack.org/contributors/common/setup-gerrit.html#preparing-to-send-a-review17:51
gmannit has all info we have in contributor guide now like account setup to code review etc- https://docs.openstack.org/infra/manual/developers.html17:52
diablo_rojoOnce the governance change to move the infra manual out lands I think both it and the contributor guide will need some changes to clean up the deduplication and cross link and what not.17:53
gmannfungi: i mean the diagram in starting which is very helpful for newby - https://docs.openstack.org/infra/manual/developers.html17:53
fungisure, openstack's patch submission workflow is the opendev workflow, so having the openstack contributor guide refer to the opendev manual's section on gerrit code review would make sense17:54
diablo_rojo+2 that would be my vision17:54
fungior find a way to transclude content into it without duplicating it in the contributor guide repo17:54
diablo_rojoYeah that works too. If it gets updated at the source it will update there too17:55
gmannopendev workflow etc make sense.17:55
zanebI was halfway through editing the patch to link to the openstack contrib guide when I actually looked at it and... there's no obvious entry point like there is with the opendev one17:58
zanebif we're catching someone halfway through opening a pull request, we can't be like "don't do that, read this book instead"17:58
fungiyeah, i think the in-flux situation with the contributor guide complicates this a little at the moment18:00
*** e0ne has quit IRC18:00
diablo_rojozaneb, I think if you are concerned about where in the process we are catching them I would just link to the specific section of the contrib guide about setting up gerrit18:00
gmannzaneb: we can have entry point once migration of infra manual to opendev and updates to contributor guide are done18:00
diablo_rojoWhat gmann says works too.18:01
diablo_rojoEither way I think we are going to need to update that link everywhere once the docs are all cleaned up and sorted correctly.18:01
gmanndiablo_rojo: fungi i think very first page of contributor guide having opendev flow (diagram) is needed otherwise zaneb concern of ''read this book instead" is very valid18:02
diablo_rojogmann, yeah I get that.18:02
diablo_rojoNo matter what, people are going to need to read to effectively contribute.18:02
zanebgmann: that sounds promising. when do we expect that to be done?18:03
gmanni do not know, may be fungi or diablo_rojo can answer18:03
diablo_rojoAnd no matter what approach we take there will be people that don't follow the logical path that we landed on because their logic is different.18:03
diablo_rojozaneb, the moving opendev out of the TC governance patch needs to land. Then we need to remove all the openstack specific stuff from the infra manual (which will eventually be JUST opendev).18:04
diablo_rojoThe openstack specific stuff will live in the contributor guide18:04
diablo_rojoand the contributor guide will link to various parts of the opendev docs18:04
fungibut *could* instead build in content pulled from opendev's manual sources if linking away to another site is seen as a problem18:08
diablo_rojoYes. That.18:08
diablo_rojoDefinitely cool with that.18:08
fungi(one of the benefits of how we're publishing both with a cross-project-aware ci system)18:08
* diablo_rojo admits to not know exactly how that would work18:08
fungithe ugly sledge hammer would be to have zuul checkout both repos and then copy (parts of?) files from one into the other before invoking sphinx18:09
diablo_rojoOof.18:09
fungithat would be fairly simple, there are almost certainly far more elegant ways though18:10
diablo_rojoIf thats the only way of doing it.. I think I'd rather link? But am open to opinions.18:10
fungii'm sure that's not the only way, just the first thing which came to mind (i'm not super deep in sphinx know-how)18:11
diablo_rojofungi, me neither, but it definitely seems like a thing18:11
gmannon goal side, if we keep infra manual in 708672 approach then we have to update it again once migration to opendev happen OR if we remove the infra manula then we do not have entry point in contributor guide which is same issue in modified CONTRIBUTING.rst.18:16
diablo_rojogmann, I think either way we will need to update the link everywhere..I guess we can keep it as the infra manual now if zaneb feels strongly about not linking to the contrib guide.18:17
zanebwill there be a redirect once the infra manual moves?18:18
gmannin later approach it will be just contributor guide update not all CONTRIBUTING.rst on each repo. but that give complex doc to read to understand the workflow.18:19
fungithere will be redirects from the old infra manual url to the new one, but that's not even something we've talked heavily about moving yet. the current "move" is about putting the git repository under opendev's governance instead of openstack's18:20
gmannor if we link the current infra manual in contributor guide introduction.html then we can keep entry point (current infra manual and later updatae to opendev one) -  https://docs.openstack.org/contributors/code-and-documentation/introduction.html18:20
fungihow it gets published is a conversation we can have after the governance change is settled18:21
gmannthat way we can avoid infra manual links from CONTRIBUTING.rst and update only contributor guide once migration happen.18:21
diablo_rojoI'm all for less updates across all the repos.18:22
*** witek has quit IRC18:25
openstackgerritNate Johnston proposed openstack/governance master: Add QA upstream contribution opportunity  https://review.opendev.org/70663718:32
*** jamesmcarthur has joined #openstack-tc18:39
*** jamesmcarthur_ has quit IRC18:41
jungleboyjzaneb: Finally getting some bandwidth to respond to this:  https://review.opendev.org/#/c/698582/2/doc/source/user_survey/analysis-12-2019.rst19:23
jungleboyjzaneb:  Nevermind.  Found where we have the number of people who report using the different projects.19:36
jungleboyjThough this graph is impossible to read for a color blind person.19:38
jungleboyjdiablo_rojo:  Do you know who might have the raw numbers for how many respondents reporting each service?19:38
fungiaprice would, but she's ni an all-day meeting currently19:40
fungiit may also be in the analytics interface... checking19:40
jungleboyjfungi:  Cool.  Thanks for looking.19:41
fungiat https://www.openstack.org/analytics if you click on deployments at the left, the charts there have percentages but so crowded it's basically unreadable19:43
jungleboyjfungi: Yep, that was what I was looking at and if you zoom in it is no better.19:43
jungleboyjThe colors used are also not friendly to a Red/Green color blind person.  They all look grey to me.19:43
apricejungleboyj: yeah, we can get that data. are you looking for particular projects or all? and when do you need?19:44
apricethat is my least favorite chart lol19:44
jungleboyjaprice: Awesome.  All of them please.  If it is possible by End of the Week that would be great, but whenever you can is awesome.19:45
fungii'm not color-blind (i don't think) and the colors there are sandwiched so close together they still look like mud19:45
apricejungleboyj: eow is possible. will likely be friday since i have all day meetings today and tomorrow, but feel free to ping me if you dont hear from me19:45
jungleboyjaprice: Sounds good.  Thanks for the help!19:45
jungleboyjfungi: Thanks for the pointer!19:45
apriceof course!19:45
jungleboyjfungi: Ok, good to know it isn't just me.19:46
jungleboyjfungi: It is amazing how many we interfaces put a small think red line around missed or bad input.  I can't see that.  I just sit there wondering why the heck the form isn't processing.19:47
fungijungleboyj: just think, if you were chinese (and not color blind) you'd think the red outline was telling you everything's just peachy19:48
jungleboyj:-)19:49
*** jamesmcarthur has quit IRC20:06
*** iurygregory has quit IRC20:27
*** jamesmcarthur has joined #openstack-tc21:07
*** jamesmcarthur has quit IRC21:07
*** jamesmcarthur has joined #openstack-tc21:07
*** jamesmcarthur has quit IRC21:19
*** jamesmcarthur has joined #openstack-tc21:21
*** jamesmcarthur has quit IRC21:26
smcginnisSome good news (cc mugsie): https://www.theregister.co.uk/2020/02/19/dot_org_sale_isoc_advice/21:45
* fungi cheers while his cats stare at him strangely21:46
*** jamesmcarthur has joined #openstack-tc21:56
*** jamesmcarthur has quit IRC21:58
*** jamesmcarthur has joined #openstack-tc21:58
*** tosky has joined #openstack-tc22:00
*** jamesmcarthur_ has joined #openstack-tc22:09
*** ijolliffe has quit IRC22:11
*** jamesmcarthur has quit IRC22:13
*** slaweq has quit IRC22:29
*** slaweq has joined #openstack-tc22:40
*** slaweq has quit IRC22:45
*** jamesmcarthur_ has quit IRC23:32
*** jamesmcarthur has joined #openstack-tc23:33
*** tosky has quit IRC23:34
*** jamesmcarthur has quit IRC23:39

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