[09:57:11] lunch [13:31:20] \o [13:31:32] o/ [13:36:02] o/ [13:54:22] patch up to update opensearch plugins package for regex: https://gitlab.wikimedia.org/repos/search-platform/opensearch-plugins-deb/-/merge_requests/7 [14:32:00] ebernhardson: approved [14:38:08] and, same as last time, i forgot to update the changelog and bump the build number :P Almost tempted to make that also a ci thing. Maybe make a changelog a summary of commits, and auto-bump the version number [14:43:27] hmm, curious... `level=fatal msg="run app" cli=release-cli error="failed to create release: API Error Response status_code: 409 message: Release already exists" version=0.24.0` [14:44:11] fun things :P [14:51:18] * ebernhardson failed to update version numbers appropriately :P [14:53:13] * inflatador has made that mistake before [15:01:05] ryankemper if you have time today, could you look into T403738 ? It's just package updates for envoy, should be apt-update + restart the envoy service. We can talk it over at pairing if you like [15:01:06] T403738: Apply Envoy updates to wcqs and wdqs hosts - https://phabricator.wikimedia.org/T403738 [15:03:56] workout, back in ~40 [15:03:57] \o [15:04:06] inflatador: yeah! can take a look in an hour [15:05:57] worked. Suspect we might want to change debian/rules to source it versioning from the changelog [15:06:00] https://gitlab.wikimedia.org/repos/search-platform/opensearch-plugins-deb/-/packages/1616 [15:58:21] ebernhardson created T403749 for the plugin apply process, feel free to link to the original regex ticket if you like [15:58:22] T403749: Install new wmf-opensearch-search-plugins package/roll-restart CirrusSearch clusters - https://phabricator.wikimedia.org/T403749 [16:12:33] thinking this might improve the release process in the future: https://gitlab.wikimedia.org/repos/search-platform/opensearch-plugins-deb/-/merge_requests/10 [16:12:46] (make it harder to forget to update versions, and limit the places to upate) [16:15:31] +1 from me, for whatever that's worth ;) [16:18:27] amusingly i kinda wanted to add a CI step that verifies the changelog always changes. But then this exact commit has no reason to change the changelog, since it reworks packaging without changing contents [16:35:02] hmm, amusing idea: Have a CI stage that posts a comment on new mr's reminding you to update the changelog [16:36:36] or i guess it could post a warning if the mr doesn't change the changelog [16:47:03] sure [16:51:25] the merge_requests/$CI_MERGE_REQUEST_IID/notes are comments? [16:53:49] dinner [16:53:58] Sigh power outage right when I’m in the middle of apt upgrading envoyproxy on wcqs [16:54:18] so far no figuring out the right set of conditions to get it to trigger [17:02:38] Ugh, power will be out for the next 7 hours RIP [17:02:47] Will head to a cafe but taking dog out first [17:41:52] sigh..finally figured out why i wasn't authing. Usually we mark variables with tokens as protected, thats fine for something like a build on a tag. But protected variables aren't available in an MR from an unprotected branch [17:41:59] basically have to unprotect the variable [18:06:29] inflatador: T403738 is all done [18:06:30] T403738: Apply Envoy updates to wcqs and wdqs hosts - https://phabricator.wikimedia.org/T403738 [18:07:06] actually i still see a few hosts in https://debmonitor.wikimedia.org/packages/envoyproxy ... checking [18:09:55] these must be hosts not yet assigned to a role, doing manually [18:12:47] ok, actually done now [19:20:26] back [19:20:42] ryankemper thanks for taking care of that