Tuesday, 2022-04-26

gmannspotz:  do you know if  RDO build for OpenStack yoga is ready ? I did not see any blog or announcement here https://blogs.rdoproject.org/00:25
fungiwe started mirroring the packages from it a week or so ago00:27
fungiat least i think that's what https://static.opendev.org/mirror/centos/8-stream/cloud/x86_64/openstack-yoga/ is00:35
spotzgmann we'll be announcing the release tomorrow or Wednesday so yes00:49
spotzas fungi said it was ready late last week but it didn't make sense to announce on a Friday00:50
gmannok, thanks00:51
spotzNot sure if the docs change has merged I'll double check00:55
spotzYeah docs aren't merged yet but if you need them https://review.rdoproject.org/r/c/rdo-website/+/4213800:56
gmannthanks, that works fine01:14
opendevreviewKendall Nelson proposed openstack/governance master: Defining Tech Preview  https://review.opendev.org/c/openstack/governance/+/83420805:24
*** pojadhav- is now known as pojadhav|pto06:20
opendevreviewHervĂ© Beraud proposed openstack/governance master: Remove oslo's TaCT SIG liaisons  https://review.opendev.org/c/openstack/governance/+/83931708:22
opendevreviewHervĂ© Beraud proposed openstack/governance master: Remove oslo's TaCT SIG liaisons  https://review.opendev.org/c/openstack/governance/+/83931708:44
rosmaitathis is kind of low-priority, and i don't know who to ask, but is the Owner on this patch OK?  https://review.opendev.org/c/openstack/cinder/+/83699612:07
rosmaitaAlso, does CLA checking happen when the review is submitted, or before the patch is merged?)12:07
fungiwhen they try to push a change to a project which has cla enforcement, gerrit checks whether the account pushing the change has agreed to the icla and returns an error refusing the push if not12:13
fungi"DataCore Software Uploaded patch set 1."12:14
fungilooks like they used their third-party ci account to push that change for review12:16
fungiwhoever maintains that account probably agreed to the icla for it12:16
rosmaitafungi: thanks ... is it ok if a rotating cast of characters use that account to submit patches?  It seems kind of non-standard12:19
fungii can't answer that, but i can ask others on the foundation staff. from a gerrit perspective we really have no insight into whether an account gets shared by multiple people12:23
fungii agree it's not usual, and not how we expect contributors to interact, but as to whether it's legally acceptable i'll see if i can find out from people who are more steeped in that12:24
rosmaitafungi: ok, thanks ... we can enforce it from the project side by -2ing the patch (Sean left a comment on there that it should be submitted from a developer account), but i wasn't clear on the whole thing12:25
rosmaitadatacore is on the CLA list: https://openinfra.dev/contributors/corporate12:25
rosmaitai guess it's just a matter of making sure each person posting a patch has signed the icla12:26
fungigerrit itself doesn't have any differentiation between "developer accounts" and "ci accounts" or whatever, accounts are accounts, so we can't really know anyway12:27
fungiexcept when someone names the account and associated e-mail address in ways which give clues12:27
rosmaitaright12:28
fungii am not a lawyer, but from what i gather the ccla is actually what the member companies whose counsel is concerned about contributor agreements really care about, the icla is there to plaster over the gaps where someone might be contributing without being covered by a ccla12:32
fungiin 2015 when the board agreed to let openstack "switch to the dco," it was really that they were okay dropping the icla in favor of the dco but wanted stronger ccla enforcement as a stipulation12:33
rosmaitafungi: i was not aware of the dco, guess i didn't pay attention because i signed the icla back in 2011 or something12:35
fungiwell, we never switched. i'd still like to though12:35
fungiwhat they're mostly concerned about is a lawsuit from some company with a large patent portfolio and well-funded legal team, not some rogue independent dev12:35
rosmaitaoh, so the foundation said we could, but we never did (switch, i mean)12:36
fungiright12:36
rosmaitaIANAL also, but it sounds like from the code review side, we should make sure that whoever is posting the patch has also signed the icla ... though of course, the easiest way to do that is for them to upload the patch themselves using their own gerrit account12:40
rosmaitaactually, after this discussion, i am going to just tell them they MUST do that, and they can follow up with the foundation themselves if there is some reason why datacore wants them to submit from the ci account (which i think there is not, using that account was the path of least resistance)12:41
rosmaitafungi: thanks!12:41
fungiright. gerrit checks that the e-mail address of the committer is an address for the account whose credentials are used, and that the account has agreed to the icla12:41
rosmaitaworks for me12:41
rosmaitaa -2 should get their attention12:42
fungias for the dco sideline, these were the board minutes from the final discussion: https://wiki.openstack.org/wiki/Governance/Foundation/26Oct2015BoardMeetingMinutes12:42
rosmaitacool, ty12:42
gmannfungi: I suspecting if more reply from community member on CFN thread, email rejection and your approval need will be more. that is why I was giving feedback on loosening the number of recipient rule.14:29
fungiyes, it doesn't really increase the work for moderators by leaving it the way it is, but 10 recipients is already a pretty large number. if the suggestion is to increase the limit, then there's probably no point in having a limit and i would just remove that check instead14:33
fungii see benefit in having the listserv remind people that they're sending messages with far too many recipients and that this is not how mailing lists are intended to be used, but if the openstack tc has some consensus that it's irrelevant i'll remove the check14:34
fungirosmaita: i talked to jbryce and he agreed there's no legal concern since the company has a signed ccla on file anyway, but that pushing back on the change proposer to make a separate account for themselves distinct from their third-party ci system is still a good idea14:54
rosmaitafungi: thanks!14:55
fungigmann: i also brought up whether we need to be concerned about imported commits in new deliverables, and the best compromise is probably to make it clear to whoever is proposing the change to add it to governance that by doing so they're effectively vouching for the provenance of any commits from before the icla was enforced on that repo14:56
*** pojadhav|pto is now known as pojadhav15:03
*** lbragstad8 is now known as lbragstad15:05
gmannfungi: on ML restriction, ok but I am sure that is making you to closely monitor that thread and approval the emails in case people reply to thread with no seeing another people reply etc. 15:53
gmannfungi: on imported  commits, thanks for checking. but I am not clear about what exactly we need to ask author to do? asserting some statement in commit message of governance patch or so?15:54
gmannfungi: rosmaita thanks for discussing the different account things. I think it will be good to add these guidelines in p-t-g as many of us might have same question in future? 15:55
fungigmann: i don't know that we necessarily need to *do* anything, but we might want to remind people in documentation somewhere that by proposing to add a deliverable to openstack/governance they're responsible for the the legality of any commits which are being imported even if they're not the author, same as they would be for any change they pushed as an individual16:34
fungias long as they're okay assuming that responsibility then there's no concern16:35
fungibut basically the agreement they made to the icla extends to the commits being imported even if those commits aren't theirs, so they should be sure they're comfortable taking legal responsibility for them16:36
gmannfungi: sounds good. let me add that in documentation somewhere. and adding that checklist  in commit msg or so can be a formal agreement from them. and can be used if any legal issue we face16:36
gmannfungi: great, is that added in icla ? 16:37
fungithere's nothing to add to the icla, i'm just clarifying the meaning of the icla16:38
fungibasically, by proposing a change to import an existing repository into openstack, they're committing those earlier commits to openstack on behalf of their original authors, so the icla applies the same in principle16:39
gmannok, I think this covers it. the  'contribution ' definition  "Contribution" shall mean any original work of authorship, including any modifications or additions to an existing work, that is intentionally submitted by You to the Project Manager for inclusion in,......"  in "1. Definitions - https://review.opendev.org/static/cla.html16:44
fungiyes16:45
gmannso we just add an explicit statement in our documentation also by referring to the icla  right?16:45
fungithat should be sufficient, i think, yes16:46
gmannok, thanks I will add that sometime this week. 16:46
fungibasically by proposing the import, they take responsibility for those imported commits same as if they were pushing the commits themselves16:47
gmann+1, yeah16:48
fungiand so the icla applies, they're implying they have permission from the copyright owners to push those changes into openstack16:49
gmanndansmith: dmendiza[m]:  knikolla: on RBAC meeting on next tuesday. my zoom account have only 45 min meeting limit, if any of you have upgraded account please let me know otherwise I will schedule google meet. 18:31
dmendiza[m]Google Meet works for me.18:37
clarkbwe also host a jitsi meet with no time limits18:42
rosmaitaplease no zoom18:42
gmannclarkb: we used that today and it did not work for many attendees 18:43
clarkbgmann: ok. we did extensive testing just before the ptg acorss as many brwosers as I have installed locally and with my phoen and it all worked. There is a weird firefox thing which got noted in the ptg documents iirc but maybe we need to put it someplace more prominent if that continues to trip people up18:45
clarkbbasically firefox audio autoplay settings affect jitsi meet. WHich means by default on many firefox installs you don't get any audio unless you go an manually enable autoplay for the domain18:45
rosmaitaclarkb: ty, that is good to know18:46
fungifirefox also likes to hide the "enable auto-play audio" option somewhere hard to find18:47
gmannnot sure what browser they used or phone, it did not work for them in joining issue and audio too. and many of them wanted to be part if discussion but they could not18:48
clarkbgmann: right I'm saying the only known issue on that front is solvable, but does force people to use not firefox or be aware of how to configure firefox unfortunately18:49
fungiwhich meeting was that? and do you have any details as to what specific problems they experienced? we can't do much to improve the platform if we don't have information on what issues are encountered18:49
clarkb++ its really hard for us to hear that it never works for anyone else when we do extensive checking before major events like the ptg and debug and do our best to address issues like the firefox thing18:50
gmannRBAC one https://meetpad.opendev.org/secure-rbac18:50
gmannproblem they faced, 1. not allowing to join 2. audio issue both some did not hear our voice and for few their audio did not work18:51
clarkbfor 2 that sounds exactly like the firefox autoplay issue18:51
dansmithgmann: sorry, I don't have a zoom account, but meet is also good for me18:51
fungiif any of the attendees who had problems can provide more specific information and/or help us troubleshoot, that would be appreciated18:51
clarkb1. is curious since anyone should be able to join. I suspect that may be due to local firewalls?18:51
gmanndansmith: dmendiza[m]: ok, let me schedule google meet18:52
fungiclarkb: yes, i'm guessing those were corporate firewalls blocking something, but hard to know without details18:52
gmannyeah, I am not sure what firewall or so but if you have any place to give feedback or exact detail about their issue they faced I will be happy to ask them to add there18:52
clarkbwe're always happy for people to reach out to us on IRC in #opendev, email us at service-discuss@lists.opendev.org or open a storyboard issue18:53
clarkbthat hasn't changed in years...18:54
clarkbI guess the IRC server changed a yaer ago but not the channel name :)18:54
gmannok, I will inform them next time if someone has problem.18:54
gmanndansmith: dmendiza[m] knikolla scheudle on google meet - https://meet.google.com/gie-derw-eer , updated wiki, send on ML. 19:00
dansmithgmann: could we mark the meeting as occurring in -keystone or something so we can merge the patch and get it into the ics file?19:55
gmanndansmith: we had that prevously but issue with that is people are confused with ics file saying meeting on IRC and we do on call. and we forget to ping that call link on IRC.20:21
dansmith...okay20:21
gmannFYI, Board meeting in 8 min from now https://board.openinfra.dev/meetings/2022-04-2620:52

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!