[10:06:02] hello, Grafana Grizzly is deployed using Puppet `git::clone` and I'd like to remove the `mode` parameter from that call to simply another change of mine (which removes `umask` from git::clone) [10:06:04] https://gerrit.wikimedia.org/r/c/operations/puppet/+/1054892 [10:06:54] that removes the mode 0440 , the default is 0755 which is fine given the cloned repo is public anyway ;) [10:14:19] hashar: checking [10:14:32] thanks! [10:16:13] may you puppet-merge it for me please or do you want someone else from your team to validate it? [10:16:56] oh [10:16:58] pending Keith :) [10:16:59] sorry [10:17:02] thank you godog ! [10:17:18] yw hashar [13:54:23] +1'd thx for the patch [16:52:00] looks like the unassigned shard email alerts are bouncing, because the status line is too long for email [16:52:02] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=logstash2024&service=OpenSearch+unassigned+shard+check+-+9200 [16:55:54] yeah, that status output is less than ideal :/ [17:05:31] I'm creating a task to track this issue, thanks for the report jhathaway ! [17:05:47] sounds good, thanks denisse [17:13:05] Here's the task: https://phabricator.wikimedia.org/T370680 [17:14:07] The Gmail Subject limit is 998 characters, the status information of that alert is 1023 characters long. [17:14:30] Source: https://support.google.com/a/answer/14016360 [19:08:22] herron: denisse: hi! Thanks for the review about my git clone patch earlier, is there anything left missing to get it deployed? ( https://gerrit.wikimedia.org/r/c/operations/puppet/+/1054892 ) [19:08:28] I am not sure what your team process is :) [19:11:13] hashar: My pleasure, thank you for the patch. [19:11:13] I'm running PCC with your patch on the grafana hosts to get a better overview of the changes. [19:11:13] It looks good to me but I'd like to wait for herron's feedback before merging it. :) [19:11:34] sure :) [19:13:30] there is no rush anyway [19:13:54] denisse: please feel free to merge or I'm happy to do it as well shortly [19:14:10] Sure, I'll merge it. [19:17:10] I merged the patch and manually ran Puppet on the Grafana hosts for the changes to take effect, thanks for the patch hashar! [19:17:22] awesome thank you very muc ! [19:17:24] much