/srv/irclogs.ubuntu.com/2020/03/11/#ubuntu-devel.txt

* mwhudson pokes at glibc's build system, fails01:57
=== ben_r_ is now known as ben_r
=== pieq_ is now known as pieq
cpaelzerrbasak: any idea why "git ubuntu tag --upload" would complain about "ERROR:Working tree must be clean to continue." while "git status --ignored" is empty?09:02
cpaelzerah, could it be that it has an empty dir issue?09:03
cpaelzerI just checked out my current branch and got the "WARNING: empty directories exist but are not tracked by git:" ...09:03
cpaelzerrbasak: by that the upload won't be matched anyway, would that also trigger the "must be clean" issue09:04
rbasakI'm not sure, sorry. That behaviour always seemed a little buggy to me.09:21
cpaelzerwhat confuses me is that it depends on the type of tag09:23
cpaelzer--split almost always works whiel --logical almost never works (for me)09:24
rbasakdoko: would you mind reviewing bug 1864766 please? It looks like the right thing to do to me, but I'd appreciate your eyes.10:40
ubottubug 1864766 in python-pip (Ubuntu Xenial) "[SRU] pip in xenial is installing packages incompatible with Python 2.7 (and those are becoming common)" [Medium,In progress] https://launchpad.net/bugs/186476610:40
heliocastroMorning/afternoon11:07
heliocastroAfter update from eoan to focal, ina XPS 13, X11 simply stop to proper work:11:09
heliocastroHere's what is happening https://photos.app.goo.gl/MPv8Ef7BnUYA7VXb911:09
heliocastroRight now i'm using wayland11:09
heliocastroIs there any report open against intel graphic drivers ?11:09
=== bpsecret- is now known as bpsecret
AsciiWolfhmm, the snap version of gnome-software on latest Ubuntu Focal does not uninstall unused dependencies when uninstalling a deb application?11:18
AsciiWolfor is there something wrong on my system?11:18
AsciiWolfalso, I was able to uninstall the Snap Store snap itself from the Snap Store app... this should not be possible at all11:20
AsciiWolfI am not sure if I should report these as issues on Launchpad since they happen only in the snap version of gnome-software11:22
AsciiWolfoh and another small thing: the "Open in Software" button from Applications tab in GNOME Settings does nothing when there is only a Snap version of gnome-software installed11:24
=== Wryhder is now known as Lucas_Gray
rbasakAsciiWolf: https://snapcraft.io/snap-store says to use https://bugs.launchpad.net/snap-store/12:03
AsciiWolfrbasak, thanks!12:08
kanashirohi, I am investigating the gem2deb/1.0.5 autopkgtest failure on armhf and it is complaining (reprotest actually) about the kernel version: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-focal/focal/armhf/g/gem2deb/20200311_020802_c6e4e@/log.gz12:08
kanashiroif you check the log you will find: autopkgtest [01:38:52]: testbed running kernel: Linux 4.15.0-72-generic #81-Ubuntu SMP Tue Nov 26 12:21:09 UTC 201912:09
kanashirois linux 4.15.0 expected in an ubuntu focal testbed?12:09
ahasenackvorlon: hi, for when you are online, when you have a moment, this is a seed change for a geoip library. I added a review slot for you at the request of the MIR team, because you were the one who added the geoip1 version to this seed way back: https://code.launchpad.net/~ahasenack/ubuntu-seeds/+git/ubuntu/+merge/38054713:22
ahasenackteward: hi, I'm in the process of trying to bring libmaxminddb (for GeoIP2 access) from universe to main, and demote geoip1. There are only two rdeps in main, and nginx is one of them. I believe I just need to reshuffle the meta packages to require the geoip2 subpackage instead of the geoip1, and I'll put an MP up shortly and add a review slot for you13:26
ahasenackteward: when you are around, for the moment the diff is in https://bugs.launchpad.net/ubuntu/+source/libmaxminddb/+bug/1861101/comments/8 as I can't push to a new git repo in lp (it's being worked on)13:52
ubottuLaunchpad bug 1861101 in nginx (Ubuntu) "[MIR]: dependency of bind9" [Undecided,In progress]13:52
rbasakrafaeldtinoco: will your fix for bug 1866458 definitely also work on the release pocket Bionic kernel? Because we'll build with NLA_F_NESTED defined to non-zero, right?13:57
ubottubug 1866458 in drbd-utils (Ubuntu Bionic) "drbd not working after kernel upgrade 5.0.x -> 5.3.x" [Medium,In progress] https://launchpad.net/bugs/186645813:57
rafaeldtinocoyep13:58
rafaeldtinocowithout that flag set command rets error13:58
rbasakrafaeldtinoco: I don't follow14:02
rbasakI thought this worked without the fix on the original kernel?14:02
rafaeldtinocorbasak: new kernels containing a specific commit need that fix14:03
rafaeldtinocoso if user is using HWE and upgrades to new kernel.. might have to have it14:03
rafaeldtinocoah I see what you mean14:04
rafaeldtinocosorry rbasak I was doing 3 simult things14:04
rafaeldtinocoyou are afraid of the flag14:04
rafaeldtinocobeing added to a kernel that does not need it14:04
rafaeldtinoco+#ifndef NLA_F_NESTED14:04
rafaeldtinoco+#define NLA_F_NESTED 014:04
rafaeldtinoco+#endif14:04
rafaeldtinocothat part will make it ok14:05
rafaeldtinocoor maybe I didnt understand what u meant14:06
rbasakrafaeldtinoco: sort of14:06
rbasakAnd yes, that ifndef would nearly make me happy14:06
rbasakHowever that's a build time thing14:06
rbasakIn our case, presumably NLA_F_NESTED is always available at build time and is non-zero14:07
rafaeldtinocoI see what u mean14:08
rafaeldtinocoi was checking that14:08
rbasakSo how does something compiled with that work with an old kernel being given a flag value it doesn't recognize?14:08
rafaeldtinocolinux/netlink.h:#define NLA_F_NESTED            (1 << 15)14:08
* rafaeldtinoco checks14:08
rafaeldtinocorbasak: https://bugs.launchpad.net/ubuntu/+source/drbd-utils/+bug/1866458/comments/1114:12
ubottuLaunchpad bug 1866458 in drbd-utils (Ubuntu Bionic) "drbd not working after kernel upgrade 5.0.x -> 5.3.x" [Medium,In progress]14:12
rbasakrafaeldtinoco: thank you for checking!14:13
rafaeldtinocoof course. im sorry for not having checked it before14:13
rafaeldtinocoill pay more attention to compile-time only variables being added14:14
rbasakIt's fine. That's the point of reviews :)14:14
rafaeldtinocoyep14:14
rafaeldtinoco=)14:14
=== Wryhder is now known as Lucas_Gray
seb128vorlon, hey, could you get wpebackend-fdo to build on i386? it's a build-depends of webkit2gtk now15:36
Laneywe need mozjs68 for gjs now as well15:39
Laneyhttps://code.launchpad.net/~laney/ubuntu-archive-tools/mozjs68/+merge/38056215:42
vorlonseb128: wpebackend-fdo added to the whitelist. so webkit2gtk got uploaded while there was a version entangled in the icu transition, and there were autopkgtest failures on webkit2gtk's revdeps in particular that were blocking.  are we going to get this through?15:49
seb128vorlon, I noticed those but I didn't want to investage on an rc version when a new stable is out which might fix it, I will look at those failure if there are still there with 2.28.015:50
vorlonok, as long as there's follow-through, thanks15:51
seb128np!15:51
tewardahasenack: NACK unless your rejiggered the NGINX code.  the inbuilt GeoIP doesnt work with geoip215:59
tewardwe had to pull in a third party plugin for it so unless libmaxmind is going to Main I have to disable some components on NGINX builds16:00
tewardI have no issues doing that but that removes the geoip functions from nginx-core16:01
tewardhad this discussion with upstream when GeoIP Legacy went the way of the dinosaur and we pulled in geoip2 as another third party module16:01
tewardah wait I misread.  bringing libmaxmind to main.16:02
tewarderror:deadbrain16:02
tewardahasenack: sec team needs to review geoip2 then16:02
tewardbecause that's a third party module16:03
teward(the geoip2 in the package for nginx) - if Security OKs that third party module going to main then I can apply it16:03
ahasenackteward: you mean review nginx's geoip2 core, not libmaxminddb16:03
tewardahasenack: ehhhhhhhhhhhh16:03
tewardno16:03
ahasenackis that a separate source tarball? Or a contrib/?16:03
tewardhang on a moment IRC from phone is inefficient16:04
tewardlet me get to my PC16:04
ahasenackteward: I'm on a call too :)16:04
ahasenackteward: we can talk in 1h or later16:04
tewardlol i"m not heh16:04
cjwatsonFWIW I switched Launchpad to geoip2 in production a while back16:04
tewardjust not out of bed :)16:04
cjwatson(because geoip legacy is (a) legacy and (b) doesn't support ipv6)16:05
tewardcjwatson: right, that's why i pushed to get geoip2 added to the package as a third party dep16:07
tewardbut here's the problem16:07
tewardhttps://bugs.launchpad.net/ubuntu/+source/nginx/+bug/1825895 is the inclusion of geoip2 module16:07
ubottuLaunchpad bug 1825895 in nginx (Ubuntu) "Please include GeoIP2 module." [Wishlist,Fix released]16:07
tewardwhich is not from nginx.org upstream16:07
tewardit's in debian/modules/16:08
tewardwhich is where we load 3rd party modules (Debian's to blame here)16:08
tewardnow, we had a strict thing with nginx-core to not have any third party modules back in the nginx MIR16:08
ahasenackteward: I have to leave for about 1h, will be back later to read history16:08
tewardgeoip is from upstream, geoip2 is *not* from upstream nginx and is third party16:08
tewardso if we're going to add that as a dep in nginx-core we need the SEcurity team to vet that module's code more thoroughly16:09
teward(cc sarnold because I'm about to give you more work_)16:09
tewardnow, if the Security team ACK's the geoip2 module that is third party for Main, then I can support this and push the diffs in16:09
cjwatsonI presume this is only an issue due to the desire to only have one or the other of the core geoip{1,2} libraries in main16:09
cjwatsonsince I don't think nginx uses the python geoip bindings either way16:10
cjwatsonanyway, not my problem, I just wanted to mention the Launchpad change16:10
tewardcjwatson: the only reason it's an issue is because ahasenack wants to change nginx-core to pull in libnginx-mod-http-geoip2 instead of libnginx-mod-http-geoip16:13
cjwatsonPresumably ultimately for the reason I gave16:13
tewardahasenack: this said, the two can work at the same time, they're designed to be coinstallable and work with slightly different commands16:13
tewardyep16:13
tewardcjwatson: yep.16:13
tewardregardless, because of the nature of -geoip2 being a third party module that was NOT ACK'd in the original NGINX MIR (and was not given an ACK for main because libmaxminddb is not in Main) it needs a Security Team review of that code before it can be ACK'd and included in Main (the -geoip2 module).16:15
tewards/that code/the third party module's code/16:15
teward(it also required libmaxminddb0 to be installed at runtime so... hence why it couldn't be in Main directly)16:17
tewardahasenack: also, I would suggest that we keep -geoip in as well as -geoip2 because -geoip2 isn't AIUI reverse compatible with GeoLite or GeoIP1 which some older deployments may still be using.16:18
teward(because they target different geoip datasets and there's a LOT of legacy apps I can think of in the Universe that use the legacy geoip data)16:18
tewardah i just read the bug data.  ACK for dropping -geoip as a dep in core IF security team ACKs -geoip2 for Main16:21
tewardcpaelzer: ^ pulling you in since you pointed ahasenack my way16:21
tewardcpaelzer: needs security team reivew of -geoip2 for Main inclusion before it can move anywhere16:22
teward(because -geoip2 is NOT from nginx upstream, it's a third party contrib.)16:22
cjwatsonI feel like you've said this a lot and maybe it would be better to just write it down once in a bug or MP or whatever's appropriate?16:22
tewardalready made a note in the bug16:22
teward*yawn* now where the heck is my coffee...16:23
vorlonLaney: well, mozjs68 ftbfs on i386 :P16:33
vorlonLaney: (and on amd64 apparently)16:33
LaneyWHAT16:33
vorlonall I did was a no-change rebuild to pick up i38616:33
vorlonso I dunno16:34
Laneydunno about amd64, no log?16:34
Laneyi386... yeah... ok that looks plausibly real16:35
ahasenackteward: I commented in the bug also. What if we demoted the geoip(1) nginx module?17:19
tewardahasenack: doable.  would require a few tweaks to the build rules too I believe because it references the dynamic modules there.  give me a bit, doing some net maintenance at home atm so stuck on my phone's crap signal.  Release Notes will need a blurb then though17:21
ahasenacksure17:21
tewardso upgraders know the functionality changed17:22
ahasenackI added some reasons to my comment17:22
ahasenackwhich we could use to justify this17:22
teward*disappears to finish the net changes to bring wifi back up*17:22
ahasenackgood luck!17:22
tewardahasenack: given the MIR backlog currently at the Sec Team's level (per sarnold) I'd say just drop geoip from core.  I'll have to tweak the build rules give me a bit to poke some things and I'll have a debdiff17:39
ahasenackthanks teward. This also doesn't prevent the review of geoip2 from happening17:39
tewardcorrect.17:39
tewardbut it removes NGINX from the equation17:40
tewardfor now17:40
ahasenackagreed17:40
tewardwe can MIR geoip2 in nginx later17:40
ahasenackif you want I can file that bug and start the analysis, like I did for bind9 (see the long description)17:40
tewardyep, feel free, but let me get this diff first and let you once-over it to see if I missed stuff17:41
ahasenackright, let's see how it works17:42
ahasenackand if it works17:42
sarnoldwoo, thanks teward, ahasenack :)17:43
ahasenackteamwork++17:43
tewardahasenack: https://paste.ubuntu.com/p/4q3Bv9Sy53/ <-- the debdiff I've got that would make these changes.17:49
tewardthere's an extra change in there for consistency sake on indentation from the 'dynamic module' section defining what is/isn't a dynamic module but the core_configure_flags in debian/rules is the core change that needed to be made17:49
tewardotherwise it'd explode at compile :P17:49
* ahasenack doesn't like explosions outside of cinema17:50
ahasenackteward: I'll give that a whirl in a ppa together with the rest, and MP afterwards17:51
ahasenackunless you are doing that right now17:51
tewardnah i need foods.  feel free to do the PPA testing, if it works i can upload this direct :P17:51
ahasenackok17:51
ahasenackthanks a lot17:51
tewardwow LAG.  my debdiff post JUST went throug bleh18:47
teward*shoots computer*18:47
tewardahasenack: i just uploaded the debdiff as well18:55
tewardto the bug in case paste.u.c drops18:55
ahasenackteward: my only change is a whitespace at the and of the changelog message, and adding the bug number18:59
tewardcool cook18:59
ahasenack+  * Drop GeoIP from nginx-core due to demotion of libgeoip (LP: #1861101):18:59
ubottuLaunchpad bug 1861101 in nginx (Ubuntu) "[MIR]: dependency of bind9" [Undecided,In progress] https://launchpad.net/bugs/186110118:59
tewardcool cool*18:59
ahasenackit's building in a ppa18:59
tewardawesome let me know how it goes19:00
ahasenackhttps://launchpad.net/~ahasenack/+archive/ubuntu/bind9-geoip19:00
tewardahasenack: I'll run tests after it builds and publishes19:13
ahasenackit's published, I'm about to install it19:13
ahasenacktw^19:13
ahasenackteward: ^19:13
tewardok.  i need to check some config bits tho on it too19:13
tewardmake sure its disabled too - because default it's off :P19:13
tewardin a few once my Focal container starts19:13
ahasenackteward: note I enabled proposed in the ppa, to mimic what will happen once it's uploaded to focal19:17
tewardawesome19:17
ahasenackthere is a new libcrypt1 dep19:17
ahasenackyou might have to enable proposed in your container as well19:17
teward(I already do heh)19:17
ahasenackyay19:17
ahasenack nginx-core : Depends: libcrypt1 (>= 1:4.1.0) but it is not installable19:18
ahasenackI didn't :P19:18
ahasenackI see it there, it has the so, linked to the expected libraries19:23
ahasenackteward: modules-available/ is empty after installs?19:23
ahasenackin /etc, I mean19:23
tewardyes19:23
tewardahasenack: mine's enabled too but it won't install until i specify `-t focal-proposed` to apt :P19:25
tewardinstalling now19:25
ahasenacksafer19:25
teward(I pinned it at 400 per https://wiki.ubuntu.com/Testing/EnableProposed)19:26
ahasenackI always forget about -t19:26
tewardahasenack: modules-enabled is where the system installed ones are19:27
tewardgeoip isn't in there which is a good sign19:28
tewardbut let me do some tests19:28
tewardnginx: [emerg] unknown directive "geoip_country" in /etc/nginx/conf.d/geoip.conf:1  <-- this is *good*19:29
tewardmeans geoip isn't available which is what our proposal does to nginx-core - removes 'geoip' from the available command set19:29
tewardso from my perspective LGTM with those changes in, as geoip isn't available19:29
tewardthat said, we'll start getting [emerg] level log warnings for people still using legacy GeoIP in nginx-core and upgrading19:30
tewardbut that's something we'll have to justify19:30
ahasenackis this something that prevents nginx from starting after an upgrade?19:33
ahasenackI guess no, because we are not removing the geoip package19:33
ahasenackit will just be upgraded as usual, but from universe19:33
ahasenackthis was a fresh install case?19:33
tewardyes19:35
tewardbut the upgrade case will cause errors as well19:35
ahasenackhow so?19:35
ahasenackif someone doesn't have universe enabled, you mean?19:35
tewardwell if the upgrade process does an autoremove it'll remove the geoip module19:35
tewardand its modules-enabled reference19:36
tewardand restart nginx and get an 'unknown directive' error19:36
tewardthis'll require a release notes notice about it19:36
ahasenackwill it fail to restart?19:36
tewardyes19:36
ahasenackI think it prompts for autoremove19:36
ahasenackbut still, if there are many packages in that list, it might be missed19:36
tewardyep19:36
tewardif it autoremoves libnginx-mod-http-geoip it'll error with the error I indicated19:37
ahasenackreplacing geoip with geoip2 would have caused the same thing, no?19:37
tewardcorrect19:37
tewardbecause it's a separate set of config arguments19:37
tewardthis is ExpectedIssues with any major change like this19:38
ahasenackI mean, even the original plan would have caused this19:38
tewardcorrect19:38
ahasenackok, document and proceed. I'll add a release notes task to the bug19:38
ahasenackif I remember what the release notes project is19:38
tewardheh19:38
tewardahasenack: i'll tweak my debdiff, do you want me to upload direct to proposed then?19:40
ahasenackteward: that /etc/nginx/conf.d/geoip.conf you had, was that you who created it, or the package?19:40
tewardi created it19:41
ahasenackok19:41
tewardno package makes anything in conf.d19:41
ahasenackso even purging would leave it there, and the error19:41
tewardbut it beats the heck out of editing nginx.conf directly (it has an `include /etc/nginx/conf.d/*.conf`)19:41
tewardcorrect, which is another example of how this upgrade can break things19:41
ahasenackyeah, isn't that one of the reasons to have .d? To have packages place stuff there?19:41
tewardahasenack: indeed, but nginx doesn't enable geoip by default19:41
tewardupstream indicates as much19:41
ahasenackbut if you install it, it loads it by default19:42
tewardhttp://nginx.org/en/docs/http/ngx_http_geoip_module.html19:42
ahasenackjust not uses it?19:42
tewardcorrect19:42
tewardbut loads vs. uses is the point here19:42
ahasenackok, several layers19:42
tewardthe module is default off unless you configure things19:42
tewardonce you add the configuration (a-la that link's example at the top of its docs) it tries to use the module19:42
tewardby 'default off' i mean enabled anda vailable but not used.19:42
tewards/enabled/loaded/19:42
tewardjust like a2enmod {INSERTMODULE} that doesn't necessarily *activate* any configurations for some modules just makes it available for conf to use.19:43
teward(in some edge cases)19:43
ahasenackok19:43
ahasenackI added a comment about release notes19:44
ahasenackwrt uploading now, I don't know, cpaelzer isn't aware of our new plan19:44
ahasenackmaybe wait until tomorrow?19:44
tewardack19:44
tewardi'll prep the changes on my side though19:44
tewardjust in case ;)19:44
ahasenackok, thanks19:44
ahasenackI can't create an MP, launchpad is timing out when I git push19:44
ahasenackknown issue, I raised a support ticket19:45
tewardright.19:45
tewardi don't push to the Git with nginx, though, I usually upload direct.19:45
teward(old school package - I haven't gotten a git workflow for it worked yet)19:45
tewards/package/packaging/19:45
ahasenackit's fine19:45
tewardunlike my GBP workflow in Debian for vmfs6-tools and xca :P19:45
ahasenackteward: what tz are you in?19:46
ahasenackit's 16h46 for me now19:46
ahasenack(19h46 UTC)19:46
tewardEastern US (15:46 local, 19:46 UTC)19:47
teward+- 1m30s for typing resulting in timeoffset19:47
ahasenackoh, look, same utc19:47
ahasenack:D19:47
ahasenackclose enough19:47
tewardyep xD19:47
kanashiroLocutusOfBorg, I mistakenly sync'ed a new version of src:puma on top of your delta and now I am trying to introduce the changes again to fix a regression. However, some questions arose during the review process, could you check this bug and try to give us some insight? https://bugs.launchpad.net/ubuntu/+source/puma/+bug/186688119:48
ubottuLaunchpad bug 1866881 in puma (Ubuntu) "Autopkgtest regression triggered by Ruby 2.7" [Undecided,New]19:48
ahasenackteward: thanks for your assistance on this today19:49
tewardahasenack: you're welcome.  do me a solid though and prep the MIR for -geoip2 module as well?19:49
tewardwe'll throw that stuff onto Security's backlog ;)19:49
tewardif not in time for release here, for G-series19:50
ahasenackalready created the card: https://trello.com/c/ySFJNH0x19:50
tewardnice19:50
ahasenackyou can subscribe to it, it will have the mir bug shortly19:50
tewardyep watching the card now :)19:51
tewardand i'm subbed to all nginx bugs so when that MIR is filed I'll get a ping :)19:52
tewardsorry I made things complex but :)19:53
tewardbest we're all in the loop :)19:53
ahasenackyou did not make it comple20:00
ahasenackcomplex20:00
ahasenackI missed the libmaxminddb requirement in my bind9 9.16 update, I thought I could still use the old geoip120:00
bdmurrayvorlon: I ran across this MP the other day - do you have an opinion on the new sizes? https://code.launchpad.net/~xnox/ubuntu-release-upgrader/eoan-kernel-sizes/+merge/37412520:08
LocutusOfBorgkanashiro, will have a look20:17
vorlonbdmurray: seems reasonable to me, particularly as these are fallbacks20:21
LocutusOfBorgkanashiro, I did sponsor the patch thanks20:39
kanashiroLocutusOfBorg, thank you20:43
LocutusOfBorgkanashiro, I added also the previous changelog entries21:01
LocutusOfBorgso we don't loose the history21:01
mwhudsonvorlon: bah ubuntu-dev-tools in focal is not compatible with python2 any more22:06
vorlonheh22:06
vorlonsil2100: pleeease22:07
mwhudsoni'll poke a bit to see how non-trivially22:07
mwhudsonbluuurgh python2 sucks22:55

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!