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

=== maclin1 is now known as maclin
=== ahoneybun is now known as Guest36814
=== dax is now known as hax
=== hax is now known as dax
=== maclin1 is now known as maclin
pittiLaney: I think http://autopkgtest.ubuntu.com/running has been stuck for a while; this page should grow a time stamp at the bottom06:43
=== klebers_ is now known as klebers
Laneypitti: hmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmm!08:02
pittiLaney: amqp-status-collector might have crashed?08:03
pittiLaney: and good morning!08:03
pittiLaney: there is a cronjob which is supposed to restart it, but IIRC that didn't work for some reason08:04
Laneyyes I'm in there08:04
Laneyit crashed a little while ago08:05
Laneypitti: i restarted it manually & apw is checking why the cron thing didn't work08:16
pittiLaney: cheers08:16
* Laney stabs tmux08:17
davmor2Laney: why what did tmux ever do to you...and use byobu instead :P08:19
Laneydavmor2: it is via byobu08:20
Laneyah, F6 worked08:20
davmor2Laney: I could of told you that :P08:20
apwF6 ... jez08:29
bdrung_workpitti, i tested apport-retrace. why is the source package needed?09:27
pittibdrung_work: it creates a StacktraceSource: field; but that's not really essential, it could be made optinal09:56
bdrung_workpitti, the logic in _search_contents (packaging_impl.py) is broken.10:09
bdrung_workpitti, you assume the presence of specific Content-%s.gz files10:10
bdrung_workpitti, e.g. you try http://ftp.de.debian.org/debian/dists/jessie/Contents-amd64.gz, but that exists in the main, contrib, or non-free subdirectory10:13
pittibdrung_work: ah, could very well be that this needs adjustment for debian's layout; for Ubuntu's, Contents is in that location (http://archive.ubuntu.com/ubuntu/dists/xenial/)10:14
pittiand not in the component/ dir10:14
bdrung_workthat could be derived from the Release file10:15
bdrung_workor does apt provide this mapping information somewhere?10:15
pittiright, looking at Release sounds good10:21
bdrung_workpitti, i stumbled over another issue: 'apport.packaging.get_file_package(report[path]' make_sandbox() returns the wrong package name (since two different packages provide the same binary file)10:37
pittibdrung_work: I suggest filing them as bugs, so that someone interested can work on those; I can help with reviewing MPs, but I can't/won't work on those myself10:39
bdrung_workpitti, any chance to get apport switched from bzr to git?12:07
pittibdmurray: ^ any objections? I'm fine with that12:08
bdrung_workpitti, the naming of the config path is not very good12:14
bdrung_workexample: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=84520812:14
ubottuDebian bug 845208 in apport-retrace "apport-retrace crashed with IOError in _get_primary_mirror_from_apt_sources(): [Errno 2] No such file or directory: u'Debian GNU/Debian testing/sources.list'" [Normal,Open]12:14
bdrung_workor in my case "Debian GNU/Linux 8/sources.list"12:18
slashdrbasak : good morning, do you have any update about the isc-dhcp (split into 2 binary packages) LP: #1176046 ?12:36
ubottuLaunchpad bug 1176046 in isc-dhcp (Ubuntu Trusty) "isc-dhcp dhclient listens on extra random ports" [Wishlist,In progress] https://launchpad.net/bugs/117604612:36
rbasakslashd: see https://bugs.launchpad.net/ubuntu/+source/isc-dhcp/+bug/1176046/comments/46. I did try to ping you about that too.12:44
ubottuLaunchpad bug 1176046 in isc-dhcp (Ubuntu Trusty) "isc-dhcp dhclient listens on extra random ports" [Wishlist,In progress]12:44
slashdrbasak, reading12:45
slashdrbasak, will update the lp bug in a few minutes12:46
=== bregma__ is now known as bregma
=== slashd- is now known as slashd
slashdrbasak, done12:58
slashdrbasak, sorry for missing your update on Apr 11.13:00
=== caribou_ is now known as caribou
bdrung_workpitti, first one: https://bugs.launchpad.net/apport/+bug/168411713:03
ubottuLaunchpad bug 1684117 in Apport "apport-retrace determines wrong package providing a file" [Undecided,New]13:03
rbasakslashd: thanks. Just glancing at it now.13:06
rbasakslashd: in verif-xenial.debdiff, shouldn't the Replaces line be unversioned?13:06
rbasakslashd: currently I don't think it'll have any effect?13:07
rbasakslashd: if you do end up editing it, there are a couple of typos in d/changelog that you might as well fix while you're there.13:07
pittibdrung_work: followed up on that one, please confirm13:07
rbasakslashd: also the xenial changelog needs a bug reference please.13:07
pittibdrung_work: oh, nevermind13:08
bdrung_workpitti, do you still need more information?13:09
bdrung_worki can't give away my test case since that involves inhouse packages which i do not want to name13:10
rbasakslashd: also, please could you detail some upgrade tests in the bug description? Some variations of what to do on Trusty, upgrade to Xenial, and then what to check that the upgrade path worked as expected. Including things like not doing anything special on Trusty and installing -noddns on Trusty, and so on. I think these should be documented in the bug and explicitly checked during SRU verification.13:10
bdrung_workpitti, do you run any checks on the code? flake8? pylint?13:11
rbasakslashd: this is in respect of the packaging changes specifically, to check that the metadata does what we want it to do. Checking the binary behaves as expected needs verifying too, of course.13:11
pittibdrung_work: nope, it's clear, I just missed the param; yes, test/run runs pyflakes and pycodestyle (and the tests of course)13:12
slashdrbasak, the upgrade worked on caribou and I test as noted on comment #42 --> https://bugs.launchpad.net/ubuntu/trusty/+source/isc-dhcp/+bug/1176046/comments/42 ... but I'll detailed more if needed #2 for the typo, I'll review it and do the necessary modification #3 I'll add bug reference13:12
ubottuLaunchpad bug 1176046 in isc-dhcp (Ubuntu Trusty) "isc-dhcp dhclient listens on extra random ports" [Wishlist,In progress]13:12
rbasakslashd: I appreciate that you did it in comment 42. What I want is to make sure that it's done for SRU verification too, so the test plan should really be in the SRU information.13:13
bdrung_workpitti, and max line length = 99 chars?13:13
slashdrbasak, sure13:13
rbasakslashd: and of course if we change the Replaces, it'll need retesting.13:13
pittibdrung_work: not enforced (see invocation in test/run), it just ignores the line len error (historic reason, would be fine to use --max-line-length)13:14
slashdrbasak, yep I'll give it a with a unversion Replaces:13:16
slashdrbasak, ok tks for your input13:16
rbasakjbicha: around?14:02
rbasakjbicha: in graphviz in the trusty queue...14:02
rbasak...shouldn't the new dependency be (= ${binary:Version}) like the others?14:02
rbasakLooks like that's what you did for Xenial, and the binary is created from the same source?14:03
rbasak(and matches the others)14:03
jbicharbasak: yes, are you to able to fix that? I don't have upload rights for graphviz/trusty and that's a very old SRU candidate now14:07
rbasakjbicha: sure, I'll fix up. Are you planning on doing SRU verification? As long as we have some volunteer.14:09
jbichayes, the test case is pretty easy14:15
rbasakOK, thanks!14:15
pittibdrung_work: btw, bug 1639427 might be your contents.gz  issue?14:28
ubottubug 1639427 in apport (Debian) "cannot determine package name from file" [Unknown,Confirmed] https://launchpad.net/bugs/163942714:28
bdrung_workpitti, nope. i patched that contents part14:30
bdrung_workoh, wait14:30
bdrung_worklet me read that bug report14:31
bdrung_workpitti, that might be my contents.gz issue (but that manifested in a backtrace in my case)14:32
bdrung_workpitti, i have patches for the contents.gz issue in my git repository14:43
bdrung_workpitti, what do you prefer. Having CoreDumpFile which contains a string pointing to the core dump file or allow to specify a file in CoreDump?15:38
pittibdrung_work: you can already have a file pointer in a Report object15:39
pittibdrung_work: ... by assigning a tuple; see http://bazaar.launchpad.net/~apport-hackers/apport/trunk/view/head:/problem_report.py#L315 for the format15:40
bdrung_workpitti, the thing is that i need to put the path into the file on disk15:40
pittinot a fan, I must say -- the point of .crash files is to be self-contained, and people copy them around (from a locked-down server to a desktop) to report them, etc.15:41
pittiapport-retrace has an option for specifying an external core dump15:41
pittibut if I have to pick, CoreDumpFile: for sure -- doing guesswork on a key is bad15:42
naccrbasak: re: LP: #1684112, i think it'd actually be pretty straightforward15:42
ubottuLaunchpad bug 1684112 in usd-importer "Cannot restrict import to supported releases only" [Wishlist,New] https://launchpad.net/bugs/168411215:42
rbasaknacc: thanks. But really, not urgent. It'll become moot when we import everything.15:43
naccrbasak: yeah, understood, but also in the meanwhile it'd be nice )15:43
rbasak:)15:43
slashdrbasak, I did the necessary modification and tests (re: LP: #1176046)15:45
ubottuLaunchpad bug 1176046 in isc-dhcp (Ubuntu Trusty) "isc-dhcp dhclient listens on extra random ports" [Wishlist,In progress] https://launchpad.net/bugs/117604615:45
slashdand update the lp bug accordingly15:45
rbasakslashd: thanks. I'll add it to my list15:45
slashdrbasak, thanks15:45
pittibdrung_work: need to leave, cu tomorrow!15:47
bdrung_workcu15:48
rbasakslashd: I didn't mean to ask you to test the upgrade paths now. What I want is the SRU information to make it clear that they will be tested from the proposed pockets during SRU verification.16:19
slashdrbasak, ok but it's still worth it to test before anyway as an extra security and obviously I'll test again once in -proposed16:27
rbasakslashd: sure, but can you add this plan to the SRU information please?16:28
rbasakslashd: since some other ~ubuntu-sru may process this after verification is marked done, it should be made explicit in the bug exactly what we agreed needs to be tested.16:29
slashdrbasak sure16:29
rbasakThanks!16:29
slashdrbasak, done16:31
JonelethIrenicuswhen trying to build apps what is init tool merge?16:42
JonelethIrenicusgetting not found error16:42
naccJonelethIrenicus: context? "build apps"?16:42
JonelethIrenicusnacc: build ubuntu unity 8 apps16:42
naccJonelethIrenicus: ok, you might ask in a unity channel? (see /topic)16:45
JonelethIrenicusunity channel on freenode?16:47
nacc!alis | JonelethIrenicus16:48
ubottuJonelethIrenicus: Alis is an IRC service to help you find channels. For help on using it, see "/msg Alis help list" or ask in #freenode. Example usage: "/msg Alis list http"16:48
JonelethIrenicusdoes ubuntu still have an sdk channel for app development16:49
JonelethIrenicusnacc: any idea?16:54
naccJonelethIrenicus: did you look in alis?17:01
JonelethIrenicusnacc: the giant list of channels?17:02
naccJonelethIrenicus: you can search alis, did you read the faq above?17:03
JonelethIrenicusnacc: its in the topic17:04
JonelethIrenicus#ubuntu-app-devel17:04
JonelethIrenicusdamn dude that was difficult17:04
JonelethIrenicushaha17:04
=== _salem is now known as salem_
=== salem_ is now known as _salem
vtapiaping rbasak17:20
=== JanC_ is now known as JanC
naccrharper: ooh, i think i finally got ppa imports working :)19:10
rharpernice!19:10
=== jasondotstar_ is now known as jasondotstar
naccrbasak: should we have patches-applied -devel branches?22:08
naccrbasak: i can't see a reason not to, at least currently, but we don't22:10
naccrharper: yep, it's working, i'm fixing up my commits and pushing22:34
nacconly about 273 insertions to add the new subcommand22:34
naccsome/most of which can be abstracted back out to common functions, tbh22:34
rbasaknacc: I think we absolutely should in order to meet the use case for patches-applied branches for drive-by contributors22:59
rbasaknacc: not sure about namespacing though22:59
naccrbasak: yep, i can fix that now22:59
naccrbasak: i think it'd fit with the rest of them, as-is22:59
naccbut i need to make sure the code can do it :)22:59
rbasakOK :)22:59
naccrbasak: if around, have time for another quick question?23:43
rbasaknacc: o/23:46
naccrbasak: just so i don't forget, we need to figure out namespaces for import-ppa. Right now, push is disabled for it23:47
naccbut if it's meant to be say, ppa_nacc/.... in the lpusip repository (potentially) or lpmep (obviously possible)23:47
naccthat starts to leak into `usd clone` which brings in all the remote branches and tags23:47
nacci think it's an argument for gitnamespaces so they don't collide23:48
naccthey will stitll share objects23:48
rbasaknacc: I've been swinging my opinion the other way and away from gitnamespaces as it happens. We should figure out the use cases and then see if we can come up with a single namespace plan that can encompass them23:50
naccrbasak: yeah, the alternative it to not do the remap we do now23:50
naccwhere the 'lpusip' remote gets the branches put into refs/heads23:50
nacci'm not sure23:50
rbasaknacc: we might be able to adjust things with sensible push and fetch refspecs23:50
naccrbasak: yeah, that's what i'm struggling to come up with23:51
nacci'm pushing what i have now, as it doesn't change anything23:51
rbasaknacc: HO tomorrow?23:51
naccwell, with regards to this issue23:51
naccbut maybe ..23:51
naccyeah23:51
nacc:)23:51
naccand we can just hammer it out23:51
nacci think the primary issue is we have too many special cases :)23:51
naccand so we just need a 'policy' for what the git refspec should look like generally that's unambiguous to determine where a given refs/head or refs/tags comes from23:52
rbasakMore generally, it should be what "owns" the name, rather than where it comes from.23:52
naccfwiw, with the latest push, we have the 'namespace' stored in one place in antying that deals with a git repository now23:53
rbasakI think.23:53
naccso we have a better means of doing something with that value easier (chagnes/improvements)23:53
naccrbasak: err, yes 'owns' in the process sense, yep23:53
naccrbasak: the primary thing i was getting mixed up on is that both lpusip and lpmep can have, say, ubuntu/devel branches23:53
naccand we don't want them to accidentally mix23:53
rbasakhttp://pad.ubuntu.com/git-importer-namespaces23:54

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