/srv/irclogs.ubuntu.com/2017/04/26/#ubuntu-devel.txt

naccrbasak: well that's nice to see -- latest importer picked up the unapplied where it was, and is imported all the applied, so far successfully, so i can do the php7.0 merge00:06
rbasakNice!00:12
=== Spydar007 is now known as Guest92796
=== cpaelzer_ is now known as cpaelzer
=== JanC is now known as Guest5365
=== JanC_ is now known as JanC
=== klebers_ is now known as klebers
=== _salem is now known as salem_
naccrbasak: ah i see i had to revert HEAD -- we need an indevelopment feature of git to allow us to skip known empty changes15:31
naccrbasak: but that would end up skipping (potentially) changes that are now empty becuase they can be dropped15:32
naccrbasak: which we don't want15:32
rbasakI see, OK.15:33
naccrbasak: i put a comment in the comment with a link to a patch for git that might let us specify it more clearly15:34
naccrbasak: already empty commits vs. commits that have become empty15:34
naccrbasak: but not yet existent, so i reverted from `usd merge` and fixed the wiki15:35
=== salem_ is now known as _salem
juliankarges: I saw that you accepted apt 1.2.21 and 1.3.6 for xenial/yakkety, but not 1.4.1~17.04.1 for zesty. Any specific reason for that?16:05
juliankIt's a bit odd to have the SRUs for older releases in -proposed without the one for the newer release16:06
LocutusOfBorgNoskcaj, FYI I'll sync dhelp as soon as it is picked up16:31
LocutusOfBorgafter only 9 years seems that LP: #205308 is fixed16:31
ubottuLaunchpad bug 205308 in dhelp (Ubuntu) "dhelp postinst error on gutsy->hardy upgrade" [High,Fix released] https://launchpad.net/bugs/20530816:31
LocutusOfBorgcjwatson, ^^ :)16:31
xnoxjuliank, we have more stuff to do for apt =)))))) we shall talk ;-) in a second.16:31
juliankxnox: More stuff to do sounds scary16:46
juliankMy veggies are in the oven now, so I do have some time :)16:48
xnoxjuliank, hehehehe =) i had steamed veggies, quicker ;-)16:48
xnoxjuliank, so. we spoke to sysadmins and we have a huge spike at 6am..6:30am UTC from all the pre-xenial machines.16:49
juliankRight. That's expected16:49
xnoxjuliank, switching xenial+ machines to that window will decrease the baseload, and will spike the window even more. Which will take us over our bandwidth limit in the key datacentres.16:50
xnoxjuliank, but we had a brainstorm session and this is what we established.16:50
xnoxjuliank, currently we have 1 timer which triggers/does 3 things as a monolithic block: apt update; apt upgrade --download-only; apt upgrade.16:51
xnoxthe first two things, ideally should happen universally spread across the 24h window.16:51
xnoxthe end users (server/cloud sysadmins) care a lot about the precise time of upgrades application. And they do not care when the bandwidth is used up.16:51
juliankWell, yes, that's not really new :)16:51
juliankWhat about memory usage during update, though?16:52
xnoxthus my current proposal is to decouple unattended upgrades application; from the update  & download-only16:52
juliankslangasek said his machine went really slow.16:52
juliankat the 1800 update due to memory spikes16:52
slangasekjuliank: xnox speaks for me16:52
xnoxjuliank, i did not consider memory usage at all. And my expectations is that one should not run ubuntu if one cannot do apt update.16:53
slangasekjuliank: fwiw I think it was ultimately the unattended-upgrades part (auto-installation of security updates) that broke me rather than the list updates16:53
xnoxslangasek, plus i suspect slangasek has like a gazzilion of repositories enabled, unlike our core user base, which is actually hitting security pocket only.16:53
slangasekwhich I hadn't considered when I talked to you16:53
juliankOK16:53
slangasekjuliank: also, fundamentally, 1) I've upgraded the RAM in my machine since then :P, and 2) I've been larted by our IS team ;-)16:54
juliankWhat is "larted"?16:54
xnoxjuliank, currently i'm writting up a back report with the decoupling plan, and will work on updating the units and/or configs; in apt and/or unattended upgrades. Such that the following behaviour happens for the long running instances:16:54
xnoxrun apt update smeared across 24h; if unattended-upgrades are enabled, execute --download-only straight after; and make unattanded-upgrades scheduled at the 6am..7am window.16:55
xnoxon boot/resume: if we missed the cycle, execute on boot the whole lot, as enabled.16:55
juliankThat's what I'd have seen as the optimal solution last week ...16:55
xnoxjuliank, this kind of means we are putting the sru on hold =/16:56
xnoxjuliank, if you want the other fixes in, i can redo the sru's without the timer change.16:56
xnoxjuliank, unless you really want to DDoS canonical datacentres and see an angry elmo =)16:56
juliankxnox: Nah16:56
juliankxnox: But then we basically just have to revert the change in apt, and set APT::Periodic::Unattended-Upgrade to 0 in unattended-upgrades, and add a timer for that16:58
xnoxjuliank, yes.16:58
juliankAlthough maybe we want 0:00 as time + random delay of 24 hours instead of two start + 12 hour delay16:58
xnoxjuliank, and the unattended-upgrade timer will be in the unattended-upgrade package.16:58
juliankYep, sure16:59
xnoxjuliank, i think two times a day is good.16:59
xnoxjuliank, because on ubuntu it has other effects apart from unattaned upgrades.16:59
xnoxthere is MOTD updates; and other messenging that is triggered by apt update.16:59
juliankYou gotta realize that there is a timestamp file in the apt script itself, so you could run it every minute, it only does something if 24 hours have passed since the last time17:00
juliank:D17:00
xnoxslangasek, i am also affected!!!!!!! i've now realised why i have random alerts texted to me on telegram whenever my database/webserver are down. And it was all weird to me, as to why they are restarting, when i didn't touch it.17:00
xnoxjuliank, can we get rid of that, at least in ubuntu, and rely on the systemd timer alone?17:01
xnox(persistent systemd timer alone)17:01
xnoxjuliank, doesn't like anacron keep persistent timestamps like that as well?17:01
juliankWell, the script has multiple distinct timers17:01
xnox/o\17:01
juliankAPT::Periodic::AutocleanInterval17:01
juliankAPT::Periodic::CleanInterval17:01
juliankAPT::Periodic::Update-Package-Lists17:02
juliankAPT::Periodic::Download-Upgradeable-Packages17:02
juliankAPT::Periodic::BackupArchiveInterval17:02
juliankall in the unit of days17:02
xnoxjuliank, so apt can be a cron replacement?! =)17:02
juliankMaybe it makes sense to split this into 5 systemd timers, but that's kind of weird17:03
juliankThe two times a day should be enough to make it run often enough17:04
xnoxjuliank, not 5. Update-Package-Lists & Download-Upgradeable-Packages should be one timer triggered thing. And the rest could be the other.17:04
xnoxbecause you do want to download things, straight after update, if enabled to do so.17:05
juliankWell, you say that.17:05
xnox(e.g. if unattended upgrades are enabled, or download-upgradable-packages is enabled)17:05
xnoxi totally want to support download Update-Package-Lists only.17:05
juliankxnox: Now, I'd really like to have the same logic in Debian, I don't really want to have distinct Ubuntu branches again :)17:08
juliankSo, I'd like keep the current setting of 6 to 7, and install a drop-in unit from the Ubuntu vendor directory.17:09
xnoxjuliank, are unattended upgrades enabled by default in debian?17:09
juliankDoes that make sense?17:09
juliankxnox: I hope not.17:09
xnoxjuliank, drop-in: yes, that is fine, the ubuntu drop-in can go into /lib/systemd/system/apt-update-whatever-the-name-is.timer.d/ubuntu.conf17:09
juliankBut we actually should have at least fixed the race conditions now, so that's something :)17:09
xnox[Timer]17:09
xnoxOnCalendar=17:09
xnoxOnCalendar=<ubuntu setting>17:10
juliankYep17:10
juliank+ Adjustments for RandomizedDelay17:11
juliankxnox: I also see u-u already has a service17:14
juliankSo adding a timer should be easy17:14
juliankAh, but that's just for shutdown17:15
xnoxjuliank, i think it would be more elaborate yes.17:17
xnoxjuliank, i was thinking to override apt settings when calling apt timers, and in ubuntu at least have something with -oAPT::DO-THIS-ONLY17:17
xnoxand -oAPT::DO_THAT_ONLY17:17
xnoxor some such, or more verbose scripts that determine what happens when.17:18
xnoxi'm currently writting the plan up, and will be working on a proposal of patches to apt and/or unattended-upgrades packages17:18
juliankxnox: The script does not take -o options, that's very ... involved.17:20
juliankYou could use config files, but I think it's pointless.17:21
juliankWhat you want as a default according to your description is APT::Periodic::Update-Package-Lists=1, APT::Periodic::Download-Upgradeable-Packages=1 and APT::Periodic::Unattended-Upgrade=017:22
slangasekjuliank: larted> from LART, luser attitude readjustment tool ;)17:22
juliankand then just run unattended-upgrades from the unattended-upgrades-daily service or whatever17:22
juliankAlthough, clean and autoclean would be weird in the apt job17:23
juliankCan we just split up the apt job in two?17:23
xnoxjuliank, clean and autoclean should be in the unattended-ugprade now?17:23
xnoxjuliank, we can do that too.17:23
juliankclean and autoclean run after unattended-upgrades17:23
juliankif configured17:24
juliankSo if you'd move only unattended-upgrades out, you'd end up with update, download, clean if configured, and your files would be gone again17:24
juliankThis way we can have apt-daily and apt-daily-dangerous17:25
juliank:D17:25
juliank(OK, maybe a better name for the second one)17:25
juliankI guess we can specify After betweens them, so *if* both are started at the same time, the upgrade&clean job runs later17:28
xnoxjuliank, yeah, we will need to do the after/before orderings right, for the onboot case.17:29
xnoxjuliank, so could you please checkout and comment on:17:29
xnoxhttps://bugs.launchpad.net/ubuntu/artful/+source/unattended-upgrades/+bug/168647017:29
ubottuLaunchpad bug 1686470 in unattended-upgrades (Ubuntu Artful) "Apt updates that are uniformally spread across all timezones, with predictable application windows" [High,Triaged]17:29
juliankThat sounds right17:30
juliankIf we just split up the apt job in two, we can just keep the changes localized to apt though, and not change unattended-upgrades at all :D17:31
xnoxyeah \o/ no need to do versioned breaks and depends17:32
xnoxjuliank, u-u service is for shutdown, i believe. not for on-boot stuff.17:33
juliankxnox: yeah17:35
juliankxnox: Another advantage of that scheme is that I can probably sell that the Debian release team :D17:36
juliank"Oh, we discovered that this might cause unwanted spikes in memory bandwidth if a lot of machines update at the same time, let's just do this all day, and special case upgrades"17:37
xnoxjuliank, yes.17:39
xnoxjuliank, "memory bandwidth" you mean "network bandwidth" right?17:40
juliankyes17:40
juliankodd mistake17:40
xnoxjuliank, and not can, but does take out a DC center we can see spikes to 10Gbps, vs the baseload of 4Gbps and some of our pipes are limited to 10Gbps17:41
xnoxat which point we have protections to limit the bandwidth, such that #is can ssh into those boxes at least; and end users get timeouts doing apt update.17:42
juliankxnox: The diff is stupid, but https://github.com/Debian/apt/compare/master...julian-klode:lp1686470?expand=1 should be it on the script side17:44
juliank+ install rules and a new timer and service17:44
xnoxdestructive-daily17:45
* xnox giggles17:45
juliankxnox: Well, find a better name!17:46
xnoxapt-download-updates, apt-install-upgrades17:48
xnox... by definition installing things, breaks toys.17:48
xnoxjuliank, however we should probably not change the original timer name though =/ because $reasons17:48
juliankxnox: Yep. Maybe I should also keep the script together and just put the parts in an if/else, that's a bit easier to look at and build.17:52
juliankThen we can just pass upgrade as $1 to run upgrade, update to update, and if nothing passed it does all, or something like that17:53
juliankHas the benefit of being completely easy to review17:53
juliankjust do a diff -w17:53
juliank:D17:53
juliankhttps://paste.ubuntu.com/24461931/18:00
juliankxnox: With the additional timer and service: https://paste.ubuntu.com/24461953/18:06
juliankOh, also needs an adjustment to rules18:06
juliankthat's a -w git, the unreadable version is at https://github.com/Debian/apt/compare/master...julian-klode:lp1686470?diff=split&expand=1&name=lp168647018:07
juliankExciting times!18:09
juliankI guess I'll just put this in a PPA quickly for zesty and artful to play with18:10
juliankBut first, my own system18:11
juliankxnox: Stuff seems ready to go (built successfully, and I installed it :D), I could upload the branch to artful to give it a test drive.18:44
xnoxjuliank, artful would be nice; many of us run artful things.18:45
juliankxnox: One question, though: Does it make sense to have the u-u/clean stuff not depend on network-online, as I did, or should both depend on that?18:49
juliankI wonder if u-u works without network18:49
xnoxjuliank, u-u/clean should not need network, correct.18:50
xnoxand i do not care about msft-fonts-package that does wget in postinst =)18:50
juliankIt's theoretical anyway.18:50
juliankThe network-online target is a boot thing18:51
juliankand apt-daily-upgrade has after=apt-daily which has after=network-online18:51
juliank(and wants=network-online)18:51
xnoxjuliank, possibly both, however the chain should be enough18:54
juliankyeah18:54
juliankxnox: Currently writing an email to deity@l.d.o and debian-release, do you want to be CCed on that?19:02
juliank(and anyone else)19:03
juliankMostly collecting additional feedback19:03
xnoxjuliank, sure19:08
juliankxnox: which email address?19:09
xnoxjuliank, xnox@ anything - ubuntu.com, debian.org, spi-inc.org, surgut.co.uk......19:11
xnoxas you wish19:11
* xnox wonders surely everyones email addresses are predictable =)19:11
juliankxnox: Some people have preferences :)19:13
juliankSome people even commit with different emails when working vs non-working19:13
xnoxsome people have too much time on their hands =)19:13
juliankxnox: I do this for changelog entries, signing the ubuntu ones with juliank@ubuntu.com and the Debian ones with jak@debian.org19:14
juliankThe commits all use my @debian.org, though19:15
juliankgit should really allow me to set up per-branch emails19:15
juliankAuthor: jak@debian.org Committer: juliank@ubuntu.com19:15
* juliank wishes jak@ubuntu.com were an alias for juliank@ubuntu.com19:16
juliankThat account is not even used: https://launchpad.net/~jak19:17
juliank(and how did John Knottenbelt became "jak", where does the a come from ...)19:17
Picimiddle name?19:17
juliankProbably from the email address?19:18
juliankanyway, that was just as unfortunate as jak being blocked on freenode and oftc19:18
sarnoldjuliank?19:18
julianksarnold: yes?19:19
sarnoldjuliank: what do you mean 'blocked on oftc'?19:19
juliankwell, registered by another user19:19
juliankImagine if I had a single nick everywhere19:20
julianksarnold: I almost bought jak.fun recently19:20
* juliank would have been jak@jak.fun19:20
sarnoldjuliank: 'jak' is unused forever; if you want it, I'll drop it for you19:21
juliankI think it's a bit late to change nicks now :)19:21
sarnoldjuliank: 'jak' is unused on freenode for 36 weeks too, they may be willing to drop it here, too19:21
julianksarnold: Maybe on OFTC, but I'm not sure if that does not confuse people who are both in Debian & Ubuntu :)19:24
juliankThat said, highlighting me is not a problem - I react to both :D19:24
juliankOK, I think I can go relax now, and hope I did not break any artfuls19:47
naccsmoser: for orig tarball imports -- what should we do if debian (or ubuntu) switches, for the same upstream, from .gz to .bz2? we correctly don't see the bz2 in `pristine-tar list` but we can't import it, because the same upstream version has already been imported. this happened for src:acl between 2.2.51-1 and 2.2.51-220:48
naccinfinity: slangasek: --^ if you have an opinion/advice on ideal behavior. Basically the importer we have no will use the 'first' orig tarball for a given upstream version as uploaded. But we don't verify if we see a second (different named) tarball for the same upstream that the contents match or anytyhing (currently)20:53
jbichaUbuntu won't allow you to use two different tarballs for the same upstream version20:54
naccjbicha: yes, but debian did20:55
naccwell "different" in that they have different names20:55
infinityjbicha: We sure will.20:55
nacctheir contents might very well be the same20:56
naccoh and actually, i found a case where what you said wasn't true, jbicha :)20:56
nacci'd have to go look it up, but it does confuse LP a bit :)20:56
infinityjbicha: As long as the filename changes, you can change taballs all you want.  People often take advantage of that "feature" to use a cleaner/better upstream tarball by switching file compression.20:56
infinitynacc: So, I'm not sure how the importer should deal with this, but when people switch compression, they almost always do it intentionally to force a new tarball that *does* have different contents, so you certainly can't assume that it's still the same.20:58
infinity(I'm not saying this practice is sane or rational, but it's certainly not uncommon)20:58
naccinfinity: yeah, that's what i was thinking -- i'm not sure gbp-import-orig can handle this, which means i'll need to rethink that part21:01
infinitynacc: The fundamental problem here is that, while VCS workflows (and dpkg/Debian versioning Policy) make a big fuss about upstream version, and tracking an upstream series/branch, the archives (dak and soyuz) literally don't care.  upstream version is meaningless to archive software, all that matters is file consistency.  So, you can't upload foo_1.2.3.orig.tar.gz twice with different contents, but change the filename and all bets are off.21:03
infinitynacc: How you represent that eventuality back in the VCS world where "upstream version" actually means something, I'm not sure.  You might just need a different branch.  1.2.3^xz or something.21:04
naccinfinity: yep, understood -- i just want, if possible, to be able to reproduce a given build's files from our git tree.21:04
naccinfinity: yeah, i'm thinking we'd need to do something like that21:04
infinity(Bonus points for finding a char that's legal in git refs but illegal in dpkg versions)21:04
naccinfinity: i think we can also do something with the version tag21:04
nacc*upstream tag, rather21:04
nacci'll need to play with it a bit to make everything happy21:04
naccinfinity: thanks for the feedback!21:04
hloeungjuliank, xnox, slangasek: thanks for all your work on LP#168647022:55
juliankhloeung: Let's see tomorrow if that actually works or if I messed up :)23:01
hloeungheh23:01
juliankWe should build test cases for the daily script, actually23:05
juliankWell, unit tests23:05
juliankJust gotta mock various tools like apt-get and u-u23:06
Unit193I mock a lot of tools23:07
sarnold:)23:07
Unit193sarnold: I presume there's no update that you know of to 716535?23:11
sarnoldUnit193: correct23:12
sarnoldUnit193: or, rather, none that I've heard23:12
sarnoldoh I see that's explicit in your question. sigh. :)23:12
Unit193Yep, you seem to "know everything" for the sec team, and easy to talk to.23:13
* sarnold blushes23:13
naccmdeslaur: just an fyi, i'll be sru'ing php7.0 7.0.18 shortly (hopefully today, or tmrw)23:14
naccinfinity: ah, it's actually quite a bit easier (for our case), we just need to make the tag names unique, so we can append the extension23:18
naccinfinity: that's the only part that fails for `gbp import-orig`23:18
naccinfinity: in any case, thank you again for the insight!23:23
mdeslaurnacc: ack, thanks23:38

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