[07:46:46] !log tool.stewardbots Regain StewardBot IRC nick via NickServ's REGAIN command [07:46:47] Urbanecm: Unknown project "tool.stewardbots" [07:46:52] !log tools.stewardbots Regain StewardBot IRC nick via NickServ's REGAIN command [07:46:53] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Tools.stewardbots/SAL [09:08:36] So sccache isn't as useful as I thought it would be because it caches absolute path names, so it can't be shared across tools [09:11:07] For CI it works fine because we clone everything into /src [10:08:55] !log tools.wikiloves Deployed a50d262, 94fe39e and ce95b92 and upgraded all dependencies in the virtual environment. [10:08:58] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Tools.wikiloves/SAL [21:04:25] question about the domainproxy. I am moving my project from wmflabs.org to wmcloud.org as you suggested. It works well so far to create new proxy and then delete old proxy but keep redirects working. Just wondering about HTTP status 301 vs 308 for that redirect. A ticket tells me to expect '308 - Permanent Redirect' but what i actually get is '301 - Moved Permanently'. So i got curious what the [21:04:31] difference is since both are permanent and it that 308 forbids POST to GET conversion. [21:05:19] and the question is...? [21:05:57] initially, the semantic on whether when following a 30x with POST the result should be POST or GET was not clear [21:06:08] that's why it later new codes were added [21:06:14] is it a bug that I don't get a 308? and.. was there maybe a discussion about 308 vs 301 before and it changed [21:08:04] * Platonides doesn't really knows... [21:10:40] thanks. it was just because Southparkfan made this really detailed ticket how to test it and specifically put the 308 and then i wanted to know the actual difference [21:11:45] my question that is left is whether we do want to disallow the conversion [21:17:23] bd808: bump [21:17:29] * ffs3___ is trying to help here [21:20:06] mutante: hmmm... I remember that we were going to use a 308. I can't remember if we switched to a 301 because of client support (there are at least some widely used java libraries that don't understand 308), or if we just typed the wrong code. andrewbogott may or may not remember more. [21:21:17] I'll look at the code, it might be that nginx decided what to call it [21:21:22] bd808: *nod* thank you. [21:21:46] cool [21:22:02] the main difference as you found is that the HTTP/1.0 spec lefts an ambiguity about how a user-agent should handle receiving a 301/302 in response to a POST request. And the later RFS that introduced 307/308 clarified the expected behavior for those new codes. [21:22:16] *RFCs [21:25:39] it looks like the lua backend we're using doesn't define 308. [21:25:39] https://openresty-reference.readthedocs.io/en/latest/Lua_Nginx_API/#http-status-constants [21:25:49] I mean, we could just jam a 308 in there anyway... [21:26:33] ah [21:27:37] huh. they have ngx.HTTP_TEMPORARY_REDIRECT for 307 but not a ngx.HTTP_PERMANENT_REDIRECT for 308. good times [21:28:19] the inversion of permanent/temporary between 301/302 and 308/307 is annoying too [21:29:59] did not expect this rabbit hole or even wanted to claim it's an actual problem. i was more curious why the change [21:30:33] but maybe it's worth fixing upstream, because 307 but no 308 does seem inconsistnt, yea [21:30:37] :) we are all about rabbit holes here mutante [21:30:52] yea:) [21:32:37] replacing some more wikistats.wmflabs.org with wmcloud.org links in the repo. conversion worked fine. [21:34:24] !log admin restarting nova-compute on cloudvirt1033; it seems stuck [21:34:26] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Admin/SAL [21:35:52] bd808, mutante, lmk if you think this matters otherwise I'm going to consider the status quo good enough [21:36:09] (Unless I get an itch to collect an nginx/lua hat at 2AM tonight) [21:36:21] I think the 301 is fine, but if we documented it as a 308 somewhere we should fix that [21:36:52] getting a new version of resty compiled as a deb... yuck [21:49:36] andrewbogott: I don't see an actual problem with the status quo, afaict. [21:50:03] i will take care of that part by asking SPF where he got that from ..next time i see him online [21:50:10] and on that ticket he made [21:54:31] it was interesting to know where it comes from, from that Lua nginx API [21:55:16] thanks mutante [21:57:36] thanks andrewbogott