[09:56:05] pfischer: o/ any objections to merge https://gerrit.wikimedia.org/r/c/wikidata/query/rdf/+/886117 (relates to flink 1.16)? It's the same patch chain you reviewed on the master branch but now on a flink_1_16 branch [09:57:03] testing on yarn did work and now I'm hoping to use this version to test the flink-k8s-operator on the dse-k8s cluster [09:58:46] reason for the separate branch is that we deploy these artifacts to deployment.eqiad.wmnet and I would not want to have a flink-1.16 artifacts being deployed there inadvertently [10:15:54] gehel: yes could do that too but I worry that the way we deploy we'll push a flink-1.16 artifact (i.e. while deploying a new version of wdqs) to prod in which case it'll be complicated to do some of the maint operation we have to do on the job [10:17:26] dcausse: all good! [14:09:43] o/ [14:56:47] FYI, this just popped for cookbooks: https://phabricator.wikimedia.org/T328775 [14:57:32] looks trivial at first glance but IDK yet [15:34:34] Weekly update out: https://app.asana.com/0/0/1203891560804762 Ping me if I forgot anything important [15:49:08] PR for the reload cookbook if anyone has time to review https://gerrit.wikimedia.org/r/c/operations/cookbooks/+/886173 [15:58:25] \o [15:58:41] o/ [16:01:41] inflatador: I'm looking at that CR. Why do we drop the --reuse-munge option? [16:05:00] I'll let a comment as well. [16:05:26] gehel it's no longer used. dcausse and I talked about it and neither of us felt it was worth much, so we dropped it [16:05:34] ok [16:05:55] it should probably be explained in the commit message. I'll let a comment. [16:06:05] definitely, sorry for missing that [16:10:27] a few minor comments, otherwise, LGTM [16:23:15] gehel Thanks for the review! [16:26:52] np [16:29:30] heading out, have a nice week-end [16:29:39] dcausse: enjoy the weekend! [17:54:25] lunch, back in ~1h [19:08:22] back