/srv/irclogs.ubuntu.com/2018/01/24/#ubuntu-devel.txt

JonelethIrenicusI have some remaining library I built myself and had to manually delete it from my system, but for some reason I think it still may be linked how can i get rid of it?02:44
cpaelzerahasenack: thanks for spotting my wrong targetting of released on the spamassassin SRU07:13
cpaelzerI really think it is artful only07:14
ricotzchrisccoulson, hi, please push your thunderbird branches07:43
juliankrbasak, cpaelzer: I think my multipath-tools upload is almost ready, I dropped the uds importer merge because I was waiting for some other stuff. Can I open a new merge so one of you importer guys can merge that upload before the importer runs so we keep the log?09:53
* juliank does not want to lose that, there's quite a bit of stuff in there :)09:53
cpaelzerjuliank: yeah09:54
cpaelzerjuliank: so you don't want an actual review now, but instead just to save your history ?09:55
cpaelzerdid I get that right?09:55
juliankHey, if you want to look at it, that's fine too. I'm just cleaning this up a bit now, and will push that out in a few mins.09:55
cpaelzerok, ping me once it is up09:56
cpaelzerand yes, I can do the git magic to let the history be retained09:56
cpaelzerjuliank: looking forward to the review - have you run tests on actual MP HW against the new code?09:59
cpaelzerif not would you mind providing a ppa along the MP so that I can do some (minor) tests?10:00
juliankcpaelzer: No, I don't have any.10:00
juliankSure, I can do that10:00
rbasakI'm not sure I follow, but cpaelzer seems to have understood10:00
cpaelzergreat, post the ppa in the MP comment10:00
cpaelzerjuliank: and mid term get together with xnox and jfh to get you a z/VM guest with a bunch of scsi disks via multipath10:01
cpaelzeror an lpar if you have a spare one left in the team10:01
juliankI have no idea how to actually build a source package with git-ubuntu10:02
juliankI can manually run debuild with the correct options, but the integrations seems broken10:02
juliank$ git ubuntu build-source10:02
juliank01/24/2018 11:01:54 - ERROR:Unable to automatically determine importer branch: Multiple candidate branches found and they do not target the same series: pkg/debian/sid, pkg/debian/buster. Please pass --target-branch.10:02
cpaelzergit ubuntu build-source --verbose --sign --for-merge10:02
juliankIf I do pass a --target-branch argument it just says it does not understand --target-branch10:03
cpaelzerI've never seen the target-banch disambiguation on a build-source so far10:04
juliankOh well, I'll just use debuild :)10:04
cpaelzerrbasak: ^^ did you?10:04
cpaelzerwell let us find the issue and fix it :-)10:04
cpaelzerjuliank: when you have pushed your MP I can try on it10:05
cpaelzerif I can reproduce I can file the issue as needed10:05
juliankcpaelzer: https://code.launchpad.net/~juliank/ubuntu/+source/multipath-tools/+git/multipath-tools/+merge/33652610:07
rbasaknacc: ^10:10
juliankcpaelzer: One thing to note is that I rebased on top of my for-debian branch, essentially, so the commit order might be a bit confusing :)10:21
juliankHopefully that'll get merged soon.10:21
juliankOr rather, I'll just NMU it in delayed/7 or something10:21
cpaelzersoon able to take a look10:21
cpaelzerand be confused then :-)10:21
cpaelzerrbasak: could you check logs why ipxe 1.0.0+git-20161027.b991c67+really20150424.a25a16d-1ubuntu2 seems not to be imported?10:35
cpaelzerzesty-devel is also behind proposed, seems not recently imported at al10:36
cpaelzerbut is in the whitelist10:36
rbasakI'm not sure the logs are working currently :(10:41
rbasakI didn't find any up-to-date ones10:41
cpaelzerok, I'll kick a local import over lunchtime10:41
cpaelzermaybe it pops up something10:41
rbasaknacc: ^10:42
tseliotsil2100: hi, are you ok with the workaround in LP: #1728547 ?10:52
ubottuLaunchpad bug 1728547 in OEM Priority Project "SRU: Add support for keeping the dGPU on in power saving mode" [Medium,Triaged] https://launchpad.net/bugs/172854710:52
sil2100tseliot: oh, didn't see that, I'll take a look at it after lunch but from the comment it sounds okayish11:00
tseliotsil2100: ok, thanks11:09
ahasenackjuliank: hi, saw in irc backlog that you were uploading multipath-tools, is that still the case?11:26
ahasenackrequest.cgi in autopkgtests is broken? I get an internal server error11:30
cpaelzerahasenack: the MP is up of juliank11:32
cpaelzerI was just about to check if I can repro his build error on it11:32
cpaelzerrbasak: ipxe imported fine, new versions there now11:33
cpaelzerrbasak: but almost everything is a forced update now11:33
cpaelzerrbasak: did you commit the import breaking changes to beta already?11:33
cpaelzer(you or nacc)11:33
ahasenackI was going to mention an ibm bug11:34
ahasenackhttps://bugs.launchpad.net/ubuntu/+source/multipath-tools/+bug/171174911:34
ubottuLaunchpad bug 1711749 in multipath-tools (Ubuntu) "[18.04] multipath-tools: Backport 2 patches to Ubuntu 18.04 (NVMe disks are detected as multipath disks)" [Medium,Triaged]11:34
cpaelzerahasenack: the multipath-tools buld of juliank fails with the equivs package just as samba11:34
ahasenacksee if the patches they mention are in this upload, presumably to bionic11:34
cpaelzerit must be something generic11:34
ahasenackcpaelzer: ah, g-u11:35
ahasenackcpaelzer: ok, I have something for that maybe11:35
ahasenacklet me see your paste again, just a sec11:35
cpaelzerahasenack: interesting11:35
ahasenackcpaelzer: line 24: https://paste.ubuntu.com/26449720/11:36
cpaelzeron the multiapth-build it works on try 3/611:36
ahasenackthere is no fix-up patch to create11:36
ahasenackunless that returns empty and is just how the code checks for it11:36
rbasakcpaelzer: I've not touched the importer since nacc got back. I don't think we've bumped beta either.11:36
rbasakI should sync with nacc on this.11:36
ahasenackI noticed that because my build of your spamassassin backtraced with unicodedecode error near that area11:36
ahasenackI wonder if it's related11:36
cpaelzerweird - it seems reproducible for samba, but on multipath works on the integrated retry11:37
cpaelzernacc: could you as a clean slate approach grab both recent merges and run build-source on them?11:38
ahasenackit worked for my clamav build11:38
ahasenackcpaelzer: doesn't line 61 confirm that it finally installed equivs?11:39
ahasenack01/24/2018 08:59:11 - DEBUG:Executing: /snap/bin/lxc exec star-eagle -- apt-get install -y devscripts equivs sudo11:39
ahasenackI think in your case it was just a missing --for-merge, now?11:39
ahasenackbecause samba 4.7.4 is not yet in ubuntu, just in debian11:39
ahasenack01/24/2018 09:00:21 - ERROR:stderr: error: open /var/lib/snapd/hostfs/tmp/review/samba_4.7.4+dfsg.orig.tar.gz: no such file or directory11:39
cpaelzerdidn't work with or without11:40
cpaelzerbut I agree that the equivs might be a red herring11:41
cpaelzerthere also is the " Multiple candidate branches found" that juliank reported11:41
cpaelzerI see that for the samba build as well11:41
cpaelzerline 2611:42
ahasenackhm, that path does look odd11:42
ahasenackit's looking in /var/lib/snapd11:42
ahasenackthat might be a recent change in the snap indeed, there was some shuffling around with the tmp dir where the build happens11:42
cpaelzerTL;DR several things are odd - @ nacc if you could (once around) try building the recent samba and multipath merge - and not related to that see if you'd see why ipxe wasn't continued to be imported11:58
cpaelzerjuliank: now I found what you meant with the commit order :-)12:15
ahasenackcpaelzer: where is this multipath-tools mp, out of curiosity?12:19
cpaelzerhttps://code.launchpad.net/~juliank/ubuntu/+source/multipath-tools/+git/multipath-tools/+merge/33652612:20
cpaelzerI already mentioned the bug you pinged above12:20
cpaelzerbut I have much more and write up the answer atm12:20
ahasenackit's not listed in https://code.launchpad.net/~usd-import-team/+activereviews12:20
ahasenacklet me check against what the mp is12:20
cpaelzer~10 minutes maybe to submit to juliank12:20
cpaelzeragainst me :-)12:20
ahasenackthat's a big delta12:21
cpaelzeroh yeah - I did th elast few merges12:22
cpaelzerwith cyphermox12:22
cpaelzerI knew what I was up to :-)12:22
cpaelzerjuliank: review done, some cleanups requested12:26
cpaelzerjuliank: if you want to overrule my review and upload right away let me know so I can make it known to at least retain history12:27
juliankdamn, I missed notifications12:27
juliankcpaelzer: Thanks for the review, I'll fix that stuff first12:28
juliankTotally forgot the patch headers, ugh.12:28
cpaelzerwell I spotted one of them which I added - so you are in (?good?) company :-)12:30
juliankcpaelzer: I'm not sure why git format-patch adds the weird date from 2001.12:32
juliankbut that's what it does...12:32
cjwatsonit's a fixed magic timestamp used by git format-patch12:34
cjwatsondocumented in git-format-patch(1)12:34
juliankah12:34
cjwatsonnot what I'd have gone for but there you go ...12:34
juliankcjwatson is like an encyclopedia :D12:35
cpaelzerwell if it is magic then keep it juliank12:42
cpaelzerI didn't know it was a special unicorn :-)12:43
Unit193I use format patch all the time, quite useful for IRC and pastebins.12:43
cpaelzerI guess we all do, but who of us have realized that the date after the hash is a magic thing12:44
cjwatsonI do wonder why it's that particular date rather than 681200 seconds earlier12:44
cjwatsonno explanation in git history AFAICS12:47
cpaelzerxnox: so many other things of the chrony change are moving fast (I didn't expect it to go that fast actually), but that makes me ask if you tnhk you'll have some time for bug 1744328?12:48
ubottubug 1744328 in nss (Ubuntu) "libfreebl3.so should be public, not in the nss subdir" [Undecided,New] https://launchpad.net/bugs/174432812:48
cjwatsonhttps://stackoverflow.com/questions/15790120/what-is-the-first-line-of-git-format-patch-output12:48
ricotzchrisccoulson, you can find tb 52.6 tarball here https://launchpad.net/~mozillateam/+archive/ubuntu/ppa/+sourcepub/8739132/+listing-archive-extra15:53
jibelcould someone review the patch attached to bug 174472216:51
ubottubug 1744722 in ubuntu-release-upgrader (Ubuntu) "Unknown bad source brings up during 'zesty' to 'artful' upgrade and It break the process" [Critical,In progress] https://launchpad.net/bugs/174472216:51
jibel?16:51
naccjuliank: cpaelzer: that is due to an issue with the fetch_orig logic, i believe. Feel free to open a bug (--target-branch).17:07
juliankjibel: I can only say that it sounds ok, but I don't have any close insight into that part17:08
juliankjibel: If I understand things correctly we could just upload it to bionic and see if upgrades from zesty to it break17:10
juliankI mean, the worst thing that could happen is a few updates to bionic failing for a day or so17:12
naccrbasak: pign17:48
nacc*ping, rather :)17:48
nacccpaelzer: ahasenack: samba will onnly build with -propsoed enabled, afaict?18:01
ahasenacknacc: yes, it needs libldb that is in proposed18:01
naccwe don't currently expose that option (you could it manually)18:01
naccor pass the correct flags to dpkg-buildpackage (-- -d)18:02
ahasenackwhich option?18:02
naccahasenack: is there something else i['m missing in the ping on that?18:02
ahasenackI think the error was the file-not-found when fetching the orig file, was it not?18:02
naccahasenack: what file not found?18:03
ahasenackthe orig tarball18:03
naccahasenack: you need to use edge to work aroudn that18:03
ahasenacklet me scroll up18:03
naccahasenack: i got too many pings to follow, and no bugs to read, so it's a bit tricky :)18:03
ahasenack<cpaelzer> ahasenack: the multipath-tools buld of juliank fails with the equivs package just as samba18:03
ahasenackdid you also check multipath-tools?18:03
rbasaknacc: pong18:03
naccmultipath-tools fails for a different reason18:04
ahasenackhttps://code.launchpad.net/~juliank/ubuntu/+source/multipath-tools/+git/multipath-tools/+merge/33652618:04
naccrbasak: do you have time to sync still today? or want to put something on the cal?18:04
rbasaknacc: can sync now18:04
rbasakstandup HO?18:04
naccrbasak: ok, joining yep18:04
naccahasenack: oh you could build samba with a lxd profile probably18:32
ahasenacknacc: I wasn't trying to, this was cpaelzer reporting to me a problem he saw when he tried it18:34
ahasenackwhich didn't seem to come from the missing build-dep18:34
nacccpaelzer: juliank: filed LP: #1745213 for the extraneous warning19:49
ubottuLaunchpad bug 1745213 in usd-importer "build: drop usage of derive_target_branch" [Undecided,New] https://launchpad.net/bugs/174521319:49
naccwarning(s)19:49
nacccpaelzer: juliank: the multipath-toosl merge builds if i pass --for-merge19:49
julianknacc: strange19:49
nacc(well, it's working now, let me see if it finishes)19:50
naccjuliank: where did it fail for you?19:50
juliankexporting a package19:50
naccjuliank: oh were you using the edge snap?19:50
naccjuliank: or stable?19:50
juliankinstalled:   0.2.2+git11.9fa9149 (291) 110MB classic19:50
juliankstable apparently19:50
naccjuliank: ah ok, we just landed the orig tarball fix to edge yesterday19:51
juliankthat one looks old19:51
nacci ened to do a release (i was out for > month last year)19:51
* juliank switches to edge19:51
naccjuliank: yeah it works with edge (just finnished)19:52
juliankI'll be fixing the needs fixing stuff tomorrow I think. I have to rebase the patches for debian and then on top of that. It's a bit annoying right now, but hopefully this gets merged there eventually.19:55
juliankAnd I test both states, the for-debian one and the one supposed for merging in autopkgtest. Doing it right :)19:56
juliankThe worst part is that these are two separate git repos :)19:59
naccheh19:59
juliankLocutusOfBorg: you on curl merge? Seems trivial. If not, I'd be happy to.20:01
juliankIt's silly merge season!20:01
juliankExcept well, it's not silly20:01
juliankAnd it got so much easier now that the tarballs grab-merge fetches do not have partially applied patches in them20:02
jbichanacc: could you try importing moon-buggy into git-ubuntu? I'm curious to see how it handles the brokenness from Debian bug 88774020:17
ubottuDebian bug 887740 in src:moon-buggy "moon-buggy: Please bump the Debian part of the version number" [Normal,Open] http://bugs.debian.org/88774020:17
jbichathere's a new reply on the bug that hasn't been posted yet where he says it's only a LP bug that the pkgs is not syncable20:18
smoserdoes every upload of grub2 require a grub2-signed upload ?21:07
LocutusOfBorgjuliank, feel free to steal :)21:26
juliank:)21:31
juliankI wonder if we should have build profile ubuntu for stuff like curl21:39
julianklibssh2-1-dev <!ubuntu>21:39
juliankbuild-depends would allow us to sync that stuff21:39
juliankinfinity: ^21:39
* juliank thinks curl has a weirfd test suite21:40
juliankbut still better than (IIRC) coreutils21:41
juliankthat one fails if you have a bind mount on your system21:41
Unit193juliank: Last time I thought that'd be a cool idea, someone said "Build profiles aren't gentoo use flags" (which, is fun because at the time I'd never used gentoo. :P )21:47
infinityjuliank: See above.21:47
juliankThey sort of are21:47
infinityjuliank: The default behaviour of buildds should be to build without profiles, it's mind-numbingly confusing if we make that not true.21:47
Unit193I believe I had asked either in terms of the indicators specifically, or whether PPAs could parse the *.changes file and build with the appropriate profile.21:50
juliankinfinity: Well, allright, I just thought it would save some work :D21:50
Unit193(And someone = cjw of course. :P )21:50
Unit193juliank: Yes, as far as I could tell the only way to do it was hacks that'd have #21:51
Unit193$universe package | package-in-main-but-not-installed, but now you can do build-depends inj uni.21:51
infinityjuliank: I don't think it actually does save work.  If we built Ubuntu with a different profile, people would take advantage of that to fork packaging more often (and, sure, push that fork into more confusing rules/control in Debian, but ew) instead of either communicating with Debian about if certain deps are needed or MIRing where appropriate.21:52
infinityjuliank: libssh2 is a bit of an outlier there (and it may also be time to revisit if we still hate it)21:52
juliankIt would be interesting to see curl with libssh-2 and how that works21:53
juliankI tried to use curl for dput sftp but obviously failed since we have no ssh support.21:53
Unit193Yes the biggest issue for depends is annoying main packages, can say.  And while syncing would certainly be a nice gain, I do agree that having "hidden delta", that is where the version is the same but the build results aren't, isn't really my favorite idea.21:54
juliankcurl tests are almost complete!!!21:55
juliankoh crap, it's doing the third variant now21:56
julianktotally forgot about nss21:56
juliankwhy do we spent 15 or 20 or 30 minutes running tests for all three variants and ignore the result. seems a bit pointless.22:04
juliankbut oh well, not going to improve that today22:04
juliankmerged22:05
juliankI still have what, 25 merges or so stuck in autopkgtests. oh the fun.22:07
* juliank did not count, only guessed :)22:07
* juliank did 40 uploads in 2 weeks and 3 days. that's insane.22:10
Unit193juliank: Merge dput?22:10
juliankUnit193: dput 1.0.1 is in -proposed22:11
Unit193Woah, good job.22:11
cjwatsoninfinity: I'm not opposed to maybe having a way to configure PPAs to build with a profile (could be handy for bootstrapping, say), but I agree that it would be best not to build Ubuntu proper with one.23:48
cjwatsonProbably not even that hard these days; just needs changes at three or four different levels of the stack.23:49

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