<div dir="ltr">I have also accidentally done this. Sometimes i copy/paste text from gerrit/phab/etc to use as the sync line and i manage to copy a \n at the beginning, which syncs with no message and puts my message on the next line. Semi-related, what if scap required either the message to be present, or a flag that says i really don't want to send a message?</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 7, 2017 at 8:21 AM, Željko Filipin <span dir="ltr"><<a href="mailto:zfilipin@wikimedia.org" target="_blank">zfilipin@wikimedia.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I am guilty of a few[0] deploys with no justification:<br><br>13:09 zfilipin@tin: Synchronized portals: (no justification provided) (duration: 00m 52s)<br>13:08 zfilipin@tin: Synchronized portals/prod/<a href="http://wikipedia.org/assets" target="_blank">wikipedia.org/<wbr>assets</a>: (no justification provided) (duration: 00m 52s)<br><br>The documentation on how to deploy portals[1] says to run sync-portals[2] script. I have updated the documentation to make it explicit that a message should be provided, similar to SWAT deploys/Deployers[3] documentation.<br><br>Željko<br>--<br>0: <a href="https://wikitech.wikimedia.org/wiki/Server_Admin_Log#2017-08-16" target="_blank">https://wikitech.wikimedia.<wbr>org/wiki/Server_Admin_Log#<wbr>2017-08-16</a><br>1: <a href="https://www.mediawiki.org/wiki/Wikipedia.org_Portal#Portal_Deployment" target="_blank">https://www.mediawiki.org/<wbr>wiki/Wikipedia.org_Portal#<wbr>Portal_Deployment</a><br>2: <a href="https://phabricator.wikimedia.org/diffusion/WPOR/browse/master/sync-portals" target="_blank">https://phabricator.wikimedia.<wbr>org/diffusion/WPOR/browse/<wbr>master/sync-portals</a><br>3: <a href="https://wikitech.wikimedia.org/wiki/SWAT_deploys/Deployers#Full_deployment" target="_blank">https://wikitech.wikimedia.<wbr>org/wiki/SWAT_deploys/<wbr>Deployers#Full_deployment</a><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Fri, Sep 1, 2017 at 3:23 AM, Chad Horohoe <span dir="ltr"><<a href="mailto:chorohoe@wikimedia.org" target="_blank">chorohoe@wikimedia.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">Hi,<div><br></div><div>I hope you were confused by my incredibly vague subject line. That's the point ;-)</div><div><br></div><div>As you're probably aware, whenever you do something with scap, it sends log entries off to the SAL. This is great because it lets us help build a history of what was deployed when--absolutely crucial when we're trying to figure out why something started breaking. What I think people might not know is that these deployment log entries can contain custom text provided by you the deployer. Here's a few samples chosen from the last couple of days:</div><div><br></div><div>```</div><div><div>20:03 demon@tin: rebuilt wikiversions.php and synchronized wikiversions files: (no justification provided)</div><div>19:27 mattflaschen@tin: Finished scap: Watchlist filters: Convert edit watchlist button to new UX and fix server-side tag filtering. T172030 (duration: 21m 23s)</div><div>19:05 mattflaschen@tin: Started scap: Watchlist filters: Convert edit watchlist button to new UX and fix server-side tag filtering. T172030</div><div>20:16 ayounsi@tin: Finished deploy [librenms/librenms@5fea59c]: (no justification provided) (duration: 00m 05s)</div><div>20:16 ayounsi@tin: Started deploy [librenms/librenms@5fea59c]: (no justification provided)</div><div>18:43 aaron@tin: Synchronized php-1.30.0-wmf.15/includes/job<wbr>queue/jobs/HTMLCacheUpdateJob.<wbr>php: 2d83569397 - Fix old regression in HTMLCacheUpdate de-duplication (duration: 00m 44s)</div></div><div>```</div><div><br></div><div>See how useful those are? But wait, what's with the (no justification provided)...that's basically useless! You're right. And I forget sometimes too. But I think a lot of people might not realize that this is the case. Basically the way it works is whatever text you type after your scap command is used as the message text. For example:</div><div><br></div><div>```</div><div># scap sync-file foo.php This is an awesome file everyone needs</div><div># scap deploy -v Nobody wants this code anyway, undeploying</div><div>```</div><div><br></div><div>This e-mail is not meant to shame anyone (or shame equally? I did include myself...), but merely as a pointer to a feature that I think is getting underused. In the last couple of months I count at least 250 deployments that lack a reason. I think we can do better :)</div><div><br></div><div>Have a great Friday!</div><span class="m_-6401259213860668024HOEnZb"><font color="#888888"><div><br></div><div>-Chad</div></font></span></div>
<br></div></div>______________________________<wbr>_________________<br>
Engineering mailing list<br>
<a href="mailto:Engineering@lists.wikimedia.org" target="_blank">Engineering@lists.wikimedia.or<wbr>g</a><br>
<a href="https://lists.wikimedia.org/mailman/listinfo/engineering" rel="noreferrer" target="_blank">https://lists.wikimedia.org/ma<wbr>ilman/listinfo/engineering</a><br>
<br></blockquote></div><br></div>
<br>______________________________<wbr>_________________<br>
Engineering mailing list<br>
<a href="mailto:Engineering@lists.wikimedia.org">Engineering@lists.wikimedia.<wbr>org</a><br>
<a href="https://lists.wikimedia.org/mailman/listinfo/engineering" rel="noreferrer" target="_blank">https://lists.wikimedia.org/<wbr>mailman/listinfo/engineering</a><br>
<br></blockquote></div><br></div>