<div dir="ltr"><br clear="all"><div><h2 class="gmail-part" id="gmail-Meeting-date-04182018-April-18th-2018-1930-IST-1400-UTC-1000-EDT" style="box-sizing:border-box;font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Helvetica Neue",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol";font-weight:600;line-height:1.25;color:rgb(51,51,51);margin-top:24px;margin-bottom:16px;font-size:1.5em;padding-bottom:0.3em;border-bottom:1px solid rgb(238,238,238);font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;letter-spacing:0.35px;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">Meeting date: 04/18/2018 (April 18th, 2018), 19:30 IST, 14:00 UTC, 10:00 EDT</h2><h3 class="gmail-part" id="gmail-BJ-Link" style="box-sizing:border-box;font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Helvetica Neue",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol";font-weight:600;line-height:1.25;color:rgb(51,51,51);margin-top:24px;margin-bottom:16px;font-size:1.25em;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;letter-spacing:0.35px;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><a class="gmail-anchor gmail-hidden-xs" href="https://hackmd.io/yTC-un5XT6KUB9V37LG6OQ?both#BJ-Link" title="BJ-Link" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none;float:left;padding-right:4px;line-height:1"><span class="gmail-octicon gmail-octicon-link" style="box-sizing:border-box;font-style:normal;font-variant:normal;font-weight:normal;font-stretch:normal;font-size:16px;line-height:1;font-family:octicons;display:inline-block;text-decoration:none;color:rgb(0,0,0);vertical-align:middle"></span></a>BJ Link</h3><ul class="gmail-part" style="box-sizing:border-box;margin-top:0px;margin-bottom:16px;padding-left:2em;color:rgb(51,51,51);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Helvetica Neue",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol";font-size:16px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:0.35px;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><li class="gmail-" style="box-sizing:border-box">Bridge:<span> </span><a href="https://bluejeans.com/205933580" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">https://bluejeans.com/205933580</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Download:<span> </span><a href="https://bluejeans.com/s/qjebZ" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">https://bluejeans.com/s/qjebZ</a></li></ul><h3 class="gmail-part" id="gmail-Attendance" style="box-sizing:border-box;font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Helvetica Neue",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol";font-weight:600;line-height:1.25;color:rgb(51,51,51);margin-top:24px;margin-bottom:16px;font-size:1.25em;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;letter-spacing:0.35px;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><a class="gmail-anchor gmail-hidden-xs" href="https://hackmd.io/yTC-un5XT6KUB9V37LG6OQ?both#Attendance" title="Attendance" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none;float:left;padding-right:4px;line-height:1"><span class="gmail-octicon gmail-octicon-link" style="box-sizing:border-box;font-style:normal;font-variant:normal;font-weight:normal;font-stretch:normal;font-size:16px;line-height:1;font-family:octicons;display:inline-block;text-decoration:none;color:rgb(0,0,0);vertical-align:middle"></span></a>Attendance</h3><ul class="gmail-part" style="box-sizing:border-box;margin-top:0px;margin-bottom:16px;padding-left:2em;color:rgb(51,51,51);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Helvetica Neue",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol";font-size:16px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:0.35px;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><li class="gmail-" style="box-sizing:border-box">Nigel</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Hari</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Nithya</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Kotresh</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Raghavendra M</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Shyam</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Milind</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Sorry Note: ndevos (travelling), kkeithle, amarts</li></ul><h3 class="gmail-part" id="gmail-Agenda" style="box-sizing:border-box;font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Helvetica Neue",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol";font-weight:600;line-height:1.25;color:rgb(51,51,51);margin-top:24px;margin-bottom:16px;font-size:1.25em;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;letter-spacing:0.35px;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><a class="gmail-anchor gmail-hidden-xs" href="https://hackmd.io/yTC-un5XT6KUB9V37LG6OQ?both#Agenda" title="Agenda" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none;float:left;padding-right:4px;line-height:1"><span class="gmail-octicon gmail-octicon-link" style="box-sizing:border-box;font-style:normal;font-variant:normal;font-weight:normal;font-stretch:normal;font-size:16px;line-height:1;font-family:octicons;display:inline-block;text-decoration:none;color:rgb(0,0,0);vertical-align:middle"></span></a>Agenda</h3><ul class="gmail-part" style="box-sizing:border-box;margin-top:0px;margin-bottom:16px;padding-left:2em;color:rgb(51,51,51);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Helvetica Neue",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol";font-size:16px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:0.35px;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><li class="gmail-" style="box-sizing:border-box"><p style="box-sizing:border-box;margin:16px 0px">Format change proposal:</p><ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box"><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1564149" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">Check the bug</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Provide feedbacks<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">We want to do a big bang format change with clang-format.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Which format should we start with as base? Google/LLVM/Mozilla/Webkit/Chromium<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box"><a href="https://github.com/nigelbabu/clang-format-sample" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">Samples present in repo</a>.<span> </span><strong style="box-sizing:border-box;font-weight:700">NOTE</strong>: Samples generated with indent as 4 spaces.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><a href="https://google.github.io/styleguide/cppguide.html" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">Google Style Guide</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><a href="http://llvm.org/docs/CodingStandards.html" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">LLVM Style Guide</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><a href="https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">Mozilla Style Guide</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><a href="https://webkit.org/code-style-guidelines/" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">WebKit style guide</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><a href="https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">Chromium style guide</a></li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">When do we want to make this change? Before the 4.1 branching seems like a good time to make vast changes.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">AGREED to do this prior to 4.1</li></ul></li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><p style="box-sizing:border-box;margin:16px 0px">Gluster’s Adoption of<span> </span><strong style="box-sizing:border-box;font-weight:700">GPL cure period enforcement</strong></p><ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box"><a href="https://www.redhat.com/en/blog/fostering-greater-open-source-development" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">What is it?</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">How to go about implementing it?<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box"><a href="https://github.com/amarts/glusterfs/commit/91918bbe7afaafd8e5bcf4a163ed98ffb39c4d21" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">Intended patch here</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><a href="https://github.com/amarts/glusterfs/blob/commitment/COMMITMENT" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">The Commitment looks like this</a></li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Next steps are to announce this to the lists and implement the same into the repository (possibly this week or early next week)[AI: Amar]</li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><p style="box-sizing:border-box;margin:16px 0px">Automation Update:</p><ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">Run regressions with brick multiplex directly from Gerrit now with a keyword rather than hacky temporary review requests. Keyword is ‘run brick-mux regression’. Example:<span> </span><a href="https://review.gluster.org/#/c/19734/" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">https://review.gluster.org/#/c/19734/</a></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">More automation moving to Python so we have the ability to write unit tests. If you are going to write a complicated shell script as a test runner, please get approval from CI component maintainers.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">We’ve been testing Facebook’s distributed test runner and have managed to get it working. Time for regression drops with every new machine added to the pool. Targetting a few weeks to bring it to production.<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">Test infrastructure has some minor fixes from FB, so need a sync up meeting on the same [AI: Nigel]</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">We can factor in test run times to achieve better overall results, and keep workers equally busy</li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Github Label check is now enforced:<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">Need help from others to identify the needs for going to give the flag.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">As ‘ndevos’ asked, we need to highlight this in Developer Guide and other places in documentation.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Can we fix the ‘gluter spec’ format and ask people to fill the github issues in that format? So that it becomes easier to give the flags.</li></ul></li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><p style="box-sizing:border-box;margin:16px 0px">[Handled Already]<s style="box-sizing:border-box">Regression failures</s></p><ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box"><s style="box-sizing:border-box"><code style="box-sizing:border-box;font-family:Menlo,Monaco,Consolas,"Courier New",monospace;font-size:13.6px;padding:0.2em 0px;background-color:rgba(0,0,0,0.04);border-radius:3px;margin:0px;color:inherit">trash.t</code><span> </span>and<span> </span><code style="box-sizing:border-box;font-family:Menlo,Monaco,Consolas,"Courier New",monospace;font-size:13.6px;padding:0.2em 0px;background-color:rgba(0,0,0,0.04);border-radius:3px;margin:0px;color:inherit">nfs-mount-auth.t</code><span> </span>are failing frequently</s></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><s style="box-sizing:border-box">git bisect shows<span> </span><a href="https://review.gluster.org/19837" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">https://review.gluster.org/19837</a><span> </span>as possible suspect</s></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><s style="box-sizing:border-box">Need to resolve soon as some critical patches are failing regression</s></li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><p style="box-sizing:border-box;margin:16px 0px">Release timelines:</p><ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">Can we extend branching out by a week or two? to compansate for github flag enforcement?<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">Target GA date should remain same.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Things we want for 4.1<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">clang formatting</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Github flag enforcment for all features</li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">We branch out later, but still release at the same time.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">No concerns during meeting. Shyam to announce.</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"></li></ul></li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em"><p style="box-sizing:border-box;margin:16px 0px">Round Table:</p><ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">kkeithle - comment-on-issue test is (still) broken. E.g. see<span> </span><a href="https://review.gluster.org/#/c/19871" target="_blank" style="box-sizing:border-box;background-color:transparent;color:rgb(51,122,183);text-decoration:none">https://review.gluster.org/#/c/19871</a><span> </span>Can we please get this fixed?<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">This is working as intended. Please see automation updates</li></ul></li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">kkeithle - For the record, I’m opposed to any kind of bulk reformat of the source. In my experience this just complicates porting changes between branches. And isn’t it also going to break<span> </span><code style="box-sizing:border-box;font-family:Menlo,Monaco,Consolas,"Courier New",monospace;font-size:13.6px;padding:0.2em 0px;background-color:rgba(0,0,0,0.04);border-radius:3px;margin:0px;color:inherit">git blame</code><span> </span>history?</li><li class="gmail-" style="box-sizing:border-box;margin-top:0.25em">Nithya - Can we split experimental xlators into it’s own package<ul style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;padding-left:2em"><li class="gmail-" style="box-sizing:border-box">AI: Get experimental xlators in a separate package. Shyam will work with Kaleb, et al</li></ul></li></ul></li></ul><br></div><div>-----</div><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Amar Tumballi (amarts)<br></div></div></div></div></div>
</div>