[03:34] sarnold: I think he has some idea that because now he knows how to install thing in linux his career will be to be a sysadmin [03:34] genii: aye.. it's perhaps a longer road than he suspects, but still, I want to help folks as I can :) [03:35] Hehe, helluva long road [03:45] yeah :) [06:12] Good morning [10:16] coreycb: I've uploaded SRU's for bionic, cosmic and disco for the neutron internal dns fixes === cpaelzer__ is now known as cpaelzer [12:01] jamespage: thanks === BlackDex_ is now known as BlackDex [13:39] sahid: I've merged/uploaded everything except neutron so far for bug 1830695 [13:39] bug 1830695 in nova (Ubuntu Cosmic) "[SRU] rocky stable releases" [High,Triaged] https://launchpad.net/bugs/1830695 [13:39] jamespage: fyi ^ [14:13] about the xenailxerux release notes, in it they say that PHP was upgraded to PHP7 and packages were either rebuilt or upgraded to php7, does the rebuild then take place against the new version of php5? [14:26] supaman: Xenial only ships with PHP 7.0 packages so those are all build against this version [14:31] sdeziel: even cakephp? [14:32] https://packages.ubuntu.com/xenial/web/cakephp === cpaelzer__ is now known as cpaelzer [14:34] supaman: yes [14:37] sdeziel: teward I'll leave Bug #1581864 up to you to respond [14:37] bug 1581864 in nginx (Ubuntu Eoan) "nginx.service: Failed to read PID from file /run/nginx.pid: Invalid argument" [Low,Confirmed] https://launchpad.net/bugs/1581864 [14:43] teward: do you plan to provide another PPA with the TJ's new patch? If yes I would give it a test run [14:47] sdeziel: TJ-: yes, but right now i'm fixing a major network bug at work [14:47] i'll upload with p1 attached to the version string I had uploaded previously so it bumps up [14:47] give me a minute i have to fix this crap [14:47] thank you both [14:49] ahasenack: RE: your comment about intrusive patches [14:49] i think Upstream would want evidence it works before including [14:49] sure [14:49] we can test-drive it in a PPA first [14:49] if it works there then we can SRU it here and next NGINX release with it in it I can upload it to Eoan [14:49] I checked fedora, the same thing happens there, fwiw [14:49] and drop the delta [14:50] ahasenack: yeah it's an upstream issue, TJ ID'd it in the code [14:50] grrr this darn network bug... [14:50] was an upstream bug even filed? [14:50] wonder if a switch is broken... BRB [14:50] ahasenack: I didn't see one, but we could always search Trac [14:50] ugh, trac [14:50] most things don't end up in the Trac tracker and just on nginx-devel@ [14:54] Trac is a good deterrent for bug reporters ;) [14:56] didn't want to say it, but there it is :) [15:20] ahasenack: I didn't want to report a bug until we're sure we're aiming at the correct target! [15:24] TJ-: fair :) [15:39] TJ-: PM me yoru email address to use on the DEP3 headers? [15:39] 'cause i'm adding headers xD [15:39] teward: same as last time; ubuntu@iam.tj or hacker@iam.tj [15:40] yeah but i lost my logs :P [15:40] thanks [15:40] :p [15:40] didn't keep the patch? tut tut :D [15:40] lost the headers in my logs [15:49] TJ-: sdeziel: PPA uploads complete, made a version snafu with Eoan in the PPA that caused the latest patch to be rejected but fixed in the PPA. Should build 'soon'. [15:49] p1 appended to version strings [15:50] building: Xenial, Bionic, Cosmic, Disco, Eoan. [16:07] TJ-: I'm fairly sure we've narrowed it down to an actual NGINX race condition problem, for how it handles PIDfiles, so... :P In either case, the PPAs are buildilng now time for tests :P [16:08] teward: it's certainly due to not creating the PIDfile in the parent process, since systemd checks for the PID file as soon that exits [16:09] *nods* [16:09] just lucky on SMP the forked child gets going and creates that before parent exits on the other thread [16:09] looks like s390x is the only arch in the PPAs that's backlogged and the rest were built. [16:09] right [16:10] teward: odd, asking to clean them. [16:10] xnox: heh, didn't know you were tracking xD [16:11] xnox: ahhh i think that's because many are stuck in a cleaning loop and a few are disabled [16:11] of the 20 s390x builders, 5 are disabled 4 are doing builds and the rest stuck on cleaning [16:11] teward: tracking what? =) [16:12] xnox: when people complain about PPA buidlers being weird :P [16:22] teward: TJ-: test successful on bionic [16:23] let me know if you want some other distro tested prior to proposing upstream [16:23] sdeziel: might need to rework the patch to make it look prettier :) [16:26] sdeziel: although there's only 2 things can be changed; values assigned to the pid_child tell-tale values, and the if (pid_child < NGX_OK || pid_child > NGX_OK) which could be just if (pid_child !=0) or even (pid_child) - although those make the intent of the code obscure [16:39] sdeziel: I'd say test Debian and CentOS if you can, I just can't build CentOS packages [16:39] Upstream has to support Cent too so I mean :| [16:41] this reminds me I *really* need to get my private Debian cloud builders working again [16:43] xnox: FYI, 4 of the s390x builders that were building are not stuck in Disabled, the rest are "Idle" so I think that fixed the broken workers problem. [16:43] now stuck* [16:46] TJ-: well sdeziel seems to suggest the code fixes the headache, do you want to test Eoan and Xenial as well? [16:46] Eoan because this lands there first, Xenial because it's LTS too. [16:46] and needs this fix heh [16:46] teward: I don't actually have nginx installed, I just thought I'd hack the patch. It works in a 19.10 container though :) [16:46] sarnold: rbasak: if either of you want to review the patch for glaring WTH problems let me know, before I drop this in Eoan, it SEEMS like TJ-'s hacked patch works :P [16:47] TJ-: isn't that the point of all patches/hacks? xD [16:47] teward: well I like to think I bring some finesse to the job, usually :p [16:47] heh [16:48] sdeziel: if you can spin an eoan container and make sure it works there that'd be great [16:48] this'd land in Eoan first, even if it's just a distropatch [16:53] teward,TJ-, any changes since last time I looked? [16:53] sarnold: when did you last look? patch was attached last friday [16:54] TJ-: sounds about right [16:54] (well, really, it feels like I just reviewed that yesterday morning.) [16:54] lol [16:55] sarnold: even if this is a nasty distropatch only I want this race fixed, We'll let TJ suggest the fix upstream xD [16:55] or sdeziel [16:55] this race condition is a nasty :p [16:57] teward: eoan test successful [17:01] sarnold: If I recall the events correctly, you reviewed an old version of TJ's patch. Here's the latest one that tested OK on Bionic/Eoan: https://launchpadlibrarian.net/426320897/nginx-fix-pidfile.8.patch [17:01] indeed this line is new to me + child_pid--; [17:03] alright, it's even more subtle than before, BUT I prefer this patch a thousand times to the systemd-based workarounds that were suggested [17:03] I wouldn't be surprised if upstream wants to do it differently [17:03] sarnold: nor me, as they know their code best [17:03] lets hope :) [17:03] they have to do something similar though to pull writing of the PID file into the parent process [17:05] regardless, if upstream fixes the issue, we can drop the distropatch but I would still want a distropatch that 'works' for older versions of NGINX. in the short term this is SRUable, if Upstream comes out with a fix we can backport, we can drop this one and replace it with an upstream patch. :P [17:07] yeah, and I think I heard someone say nginx-devel is the place to report this rather than trac? [17:07] Usually patches get submitted to nginx-devel there's a "Submitting Patches" guide let me grab it [17:10] I've subscribed so I'll write something later on [17:11] fooey I can't find the documentation, so I made a request for them to provide it again :p [17:11] and I know Maxim on that list is very good at such things. [17:11] sarnold: so, not a horridly nasty patch then? :P [17:12] * TJ- does dinner, bbl [17:12] teward: exactly [18:14] TJ-: looks like there had been patches suggested before for this, and were rejected. See Maxim's response. http://mailman.nginx.org/pipermail/nginx-devel/2019-June/012292.html [18:14] cc sarnold [18:14] looks like Upstream knows this may be an issue and previous patches in the past were rejected [18:17] one of the rejected patch was a sleep 0.1 baked in nginx itself :) [18:17] hah, and here's them pointing out the race in the sleep 0.1 approach :) http://mailman.nginx.org/pipermail/nginx-ru/2017-November/060631.html [18:17] "sleep 0.1 - это race condition на ровном месте, плохой workaround. [18:17] along the lines of "this is race condition here, it's a bad workaround [18:17] https://mailman.nginx.org/pipermail/nginx-ru/2017-November/060628.html looks better [18:18] that pipe-based approach is very similar to .. cgmanager? lxd? lxc? iirc something that hallyn worked on; I *really* liked it, but figured that would be a drastic enuogh change that I didn't suggest it [18:20] * hallyn perks up [18:20] heh [18:20] hey Hackerpcs :) [18:20] sigh. [18:20] had a feeling that'd happen :P [18:20] hey hallyn :) [18:20] yo [18:20] hey sarnold :) [18:20] sarnold: alt-tab with a "Last active" setting on it instead of alphabetical :P [18:21] Hackerpcs: sarnold mispinged :P [18:21] is this from ggn? xD [18:23] s/alt-tab/tab-complete/ [18:57] teward: well, of the 2 patches that were rejected, the 100ms sleep is the least invasive, but if upstream won't fix it... no point us submitting my patch! [18:58] TJ-: i mean you can try [18:59] teward: I'm not sure I've the energy to want to push it if they have had two proposed patches, rejected both, and NOT come up with a solution themselves [18:59] but yeah i'm probably going to keep this as an added patch/delta [18:59] TJ-: i'm not even sure if they consider it an 'issue' TBH [19:02] teward: I agree, which strikes me as a poor attitude because it plainly is not what is expected behaviour, but they don't seem to publish the systemd unit file - is that originating in Debian or ubuntu? [19:02] Debian actually [19:02] right, so they're adopting the "someone elses problem" attitude [19:03] I know with my solution there is no race condition, but it'll get objected to for likely being too invasive [19:08] indeed. [19:09] the tricky part here is, it sounds like this invasiveness is *needed* to make things actually work right... [19:09] TJ-: I'm going to let it sit in the PPA for a bit, let sdeziel and others do some more testing [19:09] see if this works fine for most production use cases [19:09] if it does I'll go the route of uploading for Eoan and getting SRU reviews [19:12] TJ-: it would be too bad to admit defeat before sending it upstream but I'd understand you don't want to fight that battle. [19:13] TJ-: looking at https://launchpadlibrarian.net/426320897/nginx-fix-pidfile.8.patch, I noticed the 'if (ngx_create_pidfile(&ccf->pid, cycle->log) != NGX_OK) {' line has 1 too many space indent, same for the one after [19:14] [20:29] my netplan setup fails to set the MTU properly, anyone knows about this? [20:31] here is the yaml config: https://paste.ubuntu.com/p/zkSV8jrkRk/ [20:32] once 'netplan apply' is done, the MTU is left at 1500 requiring me to do 'ip link set enp1s0f0 mtu 9000' [20:37] sdeziel: do you need to set that on the bridge instead? [20:38] sarnold: shouldn't be needed because the bridge always adapt to lowest MTU of the enslaved ports. That said, I tried it anyway and it doesn't work ;) [20:39] sdeziel: dang and double-dang. but thanks for reporting back :) [20:39] I also tried setting the MTU on both but no dice, thanks for suggesting [20:40] I think the issue is the NIC seems to go down/up on MTU changes [20:41] sdeziel: AWS? [20:41] * sdeziel misses ifupdown post-up scripts for quick and dirty fixes [20:42] cyphermox: no, iron @Home [20:42] ok, let me rephrase then, it's DHCP? [20:42] cyphermox: no, all static [20:42] oh you did paste config [20:43] to get the MTU to apply you must match: by MAC or something else that is specific enough to apply to the interface [20:43] oh, that would make a fantastic addition to https://netplan.io/examples ;) [20:44] cyphermox: my bad, it's already there in https://netplan.io/reference [20:44] testing this now, thanks! [20:44] well, I guess it's worth having in examples [20:45] It would be a good addition IMHO as tuning the MTU is probably something many folks are looking for [20:50] cyphermox: it worked, thank you very much!