/srv/irclogs.ubuntu.com/2013/12/05/#ubuntu-devel.txt

jtaylorzyga: if you are dealing with memory corruption valgrind --db-attach=yes is useful00:01
jtaylorafter attaching py-list and see what line of python code caused it00:01
jtaylorbut unfortunately you need a large amount of suppressions as python is very valgrind noise00:01
jtaylor< offline00:03
=== achernya_ is now known as achernya
=== _salem is now known as salem_
=== shuduo_afk is now known as shuduo
=== brainwash_ is now known as brainwash
=== salem_ is now known as _salem
=== shuduo is now known as shuduo_afk
=== shuduo_afk is now known as shuduo
=== shuduo is now known as shuduo_afk
geofftkees: ping?04:39
=== shuduo_afk is now known as shuduo
pittiGood morning05:00
pittixnox: syslog> I'd rather add it as an autopkgtest so that it blocks regressions in -proposed, rather than seeing them in the install smoketests once the damage is already done; but can't hurt to test it in both places, of course05:02
=== dspiteri is now known as darrenS
=== darrenS is now known as DarrenS
dholbachgood morning07:52
=== iahmad is now known as iahmad|brb
=== iahmad|brb is now known as iahmad
penghuancjwatson:i have updated my merge proposal about https://code.launchpad.net/~penghuanmail/ubiquity/lp.1197220/+merge/195712, hope you can take some time have a look at it, thanks!08:10
BrotherBrickmorning08:17
=== iahmad is now known as iahmad|afk
pittidoko_, barry: FTR, haveged didn't help after all, py2.7/3.3 still fail the UUID tests very often08:44
pittidoko_, barry: so it's not missing entropy, but some other assumption that UUID makes; 23 collisions (i. e. identical UUID) out of 1000 is quite much indeed08:44
pittidoko_, barry: so I'm not sure whether this is a design problem of UUID1, or the Python test which assumes that it can generate 1000 time-based UUID1s at the same time and not get any collisions08:45
pittibut if it is, it might perhaps make sense to xfail/disable that test in autopkgtest? can this be done easily, or would that require patching?08:46
pittidoko_, barry: note that the documentation says it's going to use a 14 bit random sequence number, which is 16384 possible starting values08:48
pittidoko_, barry: so one would expect 61 collisions amongst 1.000 UUIDs if it was only that; the actual number is lower as it also takes the current time into account, but in the VM we might not have the microsecond precision that this test assumes08:49
pittijibel: ^08:49
jibelpitti, IIRC you can exclude tests for autopkgtest only directly from debian/tests/<testscript>08:51
pittierr, probably not 61 (birthday paradoxon); too lazy to do the exact calculations, but one certainly can't expect no collisions at all for 1000 out of 1638408:51
pittijibel: yes, that's what we should do; I meant, if there's some easy "$SKIP_TESTS" or if it needs some seddery08:52
jibeldoko_, BTW, python3.4 autopkgtest explicitly calls python3.308:54
=== iahmad|afk is now known as iahmad
=== doko_ is now known as doko
dokopitti, jibel: fixed for the next uploads09:28
pittidoko: you mean skipping the tests?09:28
dokoyes. just wondering why they don't fail on the buildds :-/09:28
pittidoko: they aren't VMs; I guess they have a higher clock resolution there?09:29
dokoyes, no vm's09:30
jibeldoko, did you fix python3.3 -> 3.4 in debian/tests/* too ?09:30
dokojibel, yes09:30
jibeldoko, thanks09:30
dokojibel, however this one didn't reach trusty for arm64 yet ...09:31
pittididrocks: sorry, I got a bit confused about notify-osd vs. the new -icons; why are these on manual landing?09:45
pittididrocks: I thought we defaulted to auto-landing for stuff which isn't critical on the phone, to avoid too much manual work?09:46
didrockspitti: hey, cu2d wasn't done for having this manual landing forever09:49
didrocksthat was forced upon us09:49
didrocksso it would mean dividing the stacks between desktop-only and the rest09:50
didrocks(so that the desktop-only stacks can land automatically)09:50
=== shuduo is now known as shuduo_afk
didrocksand TBH, don't really have the time to recreate the views and so on09:51
didrockshence the landing spreadsheet, so that we can force the publication on those (if tests pass)09:51
didrocksin addition, that won't help us much, because most of the indicators or hud are both desktop and touch09:51
didrocksso we can't land unity7 automatically, because it was built and tests against a version in the ppa which won't be released09:52
didrocksand so, we can have unseen ABI breakages and so on09:52
didrocks(cu2d for that put the stack in manual mode automatically, but it would mean that in the end, it will always be manual as the hud and indicators are in manual mode)09:52
=== shuduo_afk is now known as shuduo
=== shuduo is now known as shuduo_afk
=== _salem is now known as salem_
=== sil2100_ is now known as sil2100
* cjwatson fixes auto-sync rashing11:12
cjwatson*crashing11:12
apacheloggerev: how do whoopsie-preferences' crashes and autoreportcrashes setting relate to whoopsie and apport ... i.e. when does whoopsie actually send the reports and when does apport and when does neither?11:14
apacheloggeralso, whoopsie is supposed to get adopted in kubuntu this cycle; finally ^^11:14
pittiapachelogger: note we still have this bug where whoopsie doesn't pick up the newly created *.upload stamps in /var/crash; "sudo restart whoopsie" usually helps11:16
pittiso at least for me, whoopsie seldomly actually uploads something :/11:16
apacheloggerit seemed to me it always uploads on start, but yeah I noticed the issue11:16
pittiapachelogger: at the moment, apport doesn't send crashes to Launchpad, only to whoopsie (we'll turn that back on soon, though, for the dev release)11:16
pittiapachelogger: whoopsie is supposed to upload as soon as you click "yes, report" in the apport UI, i. e. when the *.upload stamp is created11:17
pitti(always, also in stable releases)11:17
apacheloggerhm, I'll likely have to patch some KDE stuff then, since we do not use apport but KDE's tool11:19
apacheloggerRiddell: ^11:19
pittiapachelogger: ah, you don't use apport-kde any more? so we could drop this?11:19
pitti(it keeps causing issues from time to time, when the sip API changes again)11:19
pittiABI11:19
apacheloggerpitti: I have yet to figure out a plan because it's actually unmaintained from our side11:20
pittiand it pullls in a gazillion build deps11:20
pittiapachelogger: it's covered by tests, so it should generally work; it's not a high cost to keep it, but if you say you don't and won't use it any more, I'd rather mothball it11:20
pittiapachelogger: i. e. you just don't use apport-kde, or not apport at all?11:21
apacheloggerpitti: it's complicated ;)11:22
pittiapachelogger: without the apport libs it's probably quite some effort to craft the input files for whoopsie?11:22
pittiapachelogger: heh, it always is :)11:22
apacheloggerin dev releases we are supposed to use apport & apport-kde11:22
apacheloggerin stable releases we use upstream's tool which has absolutely no ties to apport at this point11:22
pittiapachelogger: ah, ok; so I suppose you'd use whoopsie in dev releases, but not stables, too11:23
apacheloggerpitti: actually we'd want whoopsie regardless, for the metrics11:24
apacheloggerso it seems we need to tie the kde tool to whoopsie/apport such that it can tell whoopsie to send a report11:24
pittiapachelogger: it will need to create a /var/crash/...*.crash with the expected format, and then create the .upload stamp11:25
pittiapachelogger: whoopsie only has a relatively small and fixed set of keys it looks at, so most of the extra apport fields (from hooks, etc.) aren't necessray11:25
pittiapachelogger: it's mainly Executable, Package, Dependencies, Signal, Stacktrace, StacktraceAddressSignature (and perhaps a few other easy ones)11:26
pittiapachelogger: ah, it doesn't need Stacktrace, but CoreDump11:26
pittiit mainly uses StacktraceAddressSignature for the bucketing11:27
pitti(perhaps you do want to consider using python3-apport for generating those..)11:27
apacheloggeryeah, was thinking that just now11:28
pittiit only needs python3-apt and the lightweight python3-problem-report, so its deps shouldn't be too heavy11:28
evpitti, apachelogger: acceptable fields: http://bazaar.launchpad.net/~daisy-pluckers/whoopsie/trunk/view/head:/src/whoopsie.c#L9311:30
pittiev: cheers11:31
apacheloggerthx11:32
pittiso, StacktraceAddressSignature and Package/Dependencies (with all the "modified" checks) are certainly the most complicated ones11:32
apacheloggerwe definitely don't want to reinvent the wheel, so using python3-apport seems certainly most sensible11:34
apacheloggerpitti: I also put down a todo to check where we want to go with apport-kde11:34
diwicpitti, hi, do you have a moment for two questions?11:39
pittidiwic: hey, how are you? shoot11:39
diwicpitti, hi, sprinting in Taipei actually :-) how are you?11:39
pittidiwic: and thanks for not treating me like an IP socket by saying "ping" :-P (SCNR)11:40
pittidiwic: learning the "joys" *cough* of C++.. but quite well11:40
diwicpitti, so first question was if you were planning to SRU the xdg_runtime_dir fix in systemd to saucy or not11:40
pittidiwic: ah, it fell off my radar for a bit; upstream committed a different patch, but that might take some time to backport, so we might just as well go with our current one11:41
diwicpitti, aha, so Lennart actually ended up fixing it?11:41
pittidiwic: preparing SRUs shouldn't be blocked on me personally, but if noone else wants to touch it I can add it to my TODO11:42
pittidiwic: yes (I thought I tossed you the commit the other day)11:42
diwicpitti, nice. Can't find any email from you about it if you sent one, but glad it's fixed anyhow. So let's just see who prepares the SRU first of you and me then. But you'd still have to sponsor/SRU it I presume11:43
diwicpitti, second question. So my boss wants me to deliver a feature in OSP2, which we ship in preinstalls. OSP2 is based on 12.04.4. The feature needs a backend module in PulseAudio, but the pulseaudio won't be loaded unless you run OSP2, so the regression risk is minimal for standard Ubuntu.11:46
pittidiwic: http://cgit.freedesktop.org/systemd/systemd/commit/?id=954449b82df7f FYI11:47
pittidiwic: sounds ok to me, but as I'm not in ~ubuntu-sru I can't give a formal approval11:47
diwicpitti, really? I was sure you were. :-)11:48
pittiI had been for a long time, yes11:48
* pitti adds ffox tab for bug 1197395 as a reminder11:49
ubottubug 1197395 in systemd (Ubuntu Saucy) "/run/user/$ID/pulse owned by root and not by the user" [Undecided,Confirmed] https://launchpad.net/bugs/119739511:49
diwicpitti, ok, so maybe I should ask one of the 10 ubuntu-sru members instead, any special one you'd recommend? :-)11:49
pittidiwic: TBH I'd just prepare the bug (testing, regression info), upload it, and see who bites :)11:50
pittiwe don't seem to have a "primary" SRU team member these days11:50
diwicRAOF, hi, want to discuss an SRU before I prepare so I know I'm doing things the right way?11:52
tseliotpitti: can you approve bug #1255583 please? I'd like to upload a new nvidia-prime that depends on bbswitch11:52
ubottubug 1255583 in bbswitch (Ubuntu) " [MIR] Main inclusion request for bbswitch" [Medium,Triaged] https://launchpad.net/bugs/125558311:52
pittitseliot: I'm not in ~ubuntu-mir11:53
tseliotoh11:53
tseliotdidrocks: please ^11:53
dholbachcould somebody please approved my mail in u-d-a?11:57
pittidholbach: done11:57
didrockstseliot: hum, module-assistant is recommended by in universe11:57
dholbachpitti, gracias!11:57
tseliotdidrocks: but m-a is in main11:57
didrockstseliot:  module-assistant | 0.11.6        | trusty/universe  | source, all11:58
didrocksis it?11:58
pittiwe really don't want to put that on users11:58
pittiwe should stick to DKMS IMHO11:59
tseliotdidrocks: oh, right11:59
didrockspitti: agreed :)11:59
tseliotshall I disable the module-assistant bit?11:59
didrocksyep11:59
didrockstseliot: FYI, the MIR team is small enough to normally not treat urgent issues like that. So ensure first that the MIR really match requirements please :)11:59
tseliotthe debian maintainer probably won't be pleased11:59
didrockstseliot: check-mir is a good one for that11:59
tseliotdidrocks: the point I'm going to upload the same thing in Precise for 12.04.4 for HWE12:00
didrockstseliot: yeah, that's why I'm treating that now, but we have a huge list and a very small team, so preparing that in advance would help to not sidetrack the team (and ensuring if it's urgent that the basic checks pass ;))12:01
didrockstseliot: I'm continuing the review meanwhile, fortunately enough, it's a simple one12:01
tseliotdidrocks: right, I understand. I was pretty sure module-assistant was in main, I don't know why. Sorry about that12:02
didrockstseliot: check-mir FTW! :)12:02
didrocks(in ubuntu-dev-tools)12:02
tseliotdidrocks: from now on I'll use that, thanks :)12:02
didrocksyw!12:03
didrocks(quite impressive the amount of override we have to make in dh_auto_install for dkms modules and that dh_dkms doesn't do it for us)12:04
RAOFdiwic: Sure, what are you after? My bandwidth might be limited though.12:07
diwicRAOF, well, my boss wants me to deliver a feature for OSP2, which we ship on preinstalls and based on 12.04.4.12:08
diwicRAOF, the feature includes a backend written as a pulseaudio module. So PulseAudio needs to be SRUed, but the module won't be loaded for normal Ubuntu users, so minimal regression risk.12:09
ogra_diwic, hmm, did pulse recently change in precise ? i cant use my BT headset anymore since the last upgrade12:10
diwicogra_, when did you upgrade? The last upgrade only concerned 3.5 mm headset jacks IIRC.12:10
ogra_some time last week12:11
diwicRAOF, so; the easiest way for me right now would be to sidestep SRU rules and just put something into 12.04.4 without caring about future releases12:11
diwicogra_, unless someone else has SRUed PulseAudio - the 15.4 upgrade was several weeks ago if not months12:11
diwicogra_, maybe kernel regression?12:11
ogra_well, i surely didnt upgrade for a month or so12:12
ogra_yeah, probably12:12
ogra_A2DP still works fine ...12:12
diwicogra_, pulseaudio ...15.4 was released into -updates 2013-10-0312:13
ogra_ah, yeah, i surely upgraded that before then12:13
diwicand that's the latest12:13
diwicRAOF, but I'm not sure that's something the SRU team would agree on. I'm not sure what the solution for 14.04 would be at this point; my deadline is for 12.04.4 and upstream has kind of said no to my current way of doing the backend.12:14
RAOFdiwic: Does the core pulseaudio *code* change?12:15
RAOFdiwic: Or are you just adding an additional module without any other changes?12:15
RAOFAlso, what sort of guarantees do we have that the module isn't going to be loaded.12:15
diwicRAOF, not core pulseaudio code. Only code changes in the module (which isn't loaded).12:15
diwicRAOF, the guarantee is "the module isn't listed in /etc/default.pa".12:16
diwicRAOF, the module would still ship in the Pulseaudio package though. Or possibly in a new binary package, if you prefer.12:16
RAOFNo, that's fine.12:16
RAOFAs long as it's not loaded by default that seems fine.12:17
diwicRAOF, ok, thanks - would it be okay to shoot you an email when I've uploaded the SRU (which would then contain the new unused module), to have you releasing it into -updates?12:19
mlankhorstRAOF: hey do you have some time next week?12:19
mlankhorstI want to prepare lts-saucy12:19
RAOFmlankhorst: Yeah, after Monday I'll be back not-sprinting.12:19
mlankhorstperfect, I'll prepare everything for upload then12:20
RAOFdiwic: Wait - I can release it into -proposed; do you want to skip -proposed entirely?12:20
diwicRAOF, sure, I can do -proposed verification, if somebody after that releases it into -updates12:21
RAOFdiwic: Right, that's fine.12:21
diwicRAOF, is that you as well? I don't know the details of the shepherding here, I mostly know that I have a deadline ;-)12:22
RAOFdiwic: Yeah, that's fine.12:22
diwicRAOF, ok, thanks a lot! Hopefully I'll finish the SRU some time next week12:22
didrockstseliot: everything is good otherwise, just tell me once the fixed version is in the release pocket12:44
tseliotdidrocks: sure, thanks. I've sent a message to the maintainer to let him know (trying not upset anybody)12:45
didrocksthanks12:48
=== FourDollars_ is now known as FourDollars
mardyMirv: hi! So, it looks like we'll have to cherry-pick the fix for https://bugreports.qt-project.org/browse/QTBUG-35233 into Qt 5.2.014:25
mardyseb128: FYI ^14:25
seb128mardy, thanks14:25
seb128does anyone knows in which condition apt lists packages with no "candidate"?14:27
tedgcjwatson, Looking at upstart-app-launch's usage of click.  It seems like we need two values, the package directory and the path to the manifest.  Would it be reasonable to have a "libclick" that did those?14:34
tedgcjwatson, i.e., still keep the logic in the click package, but just duplicate it in C.14:35
cjwatsontedg: I plan to rewrite click in C, hopefully this cycle14:35
cjwatsontedg: until then let's not duplicate14:35
tedgcjwatson, Okay, perhaps a fallback if you find time running short.14:36
josephtstgraber: is there a preferred way to install a set of packages when creating a container?14:46
pittiI'd appreciate if some SRU team member could process bug 1257211 today; they fix a data loss bug in postgresql; usual MRE/no packaging changes/upstream+distro tests pass story15:04
ubottubug 1257211 in postgresql-9.1 (Ubuntu Trusty) "New upstream microreleases 9.1.11, 8.4.19" [High,In progress] https://launchpad.net/bugs/125721115:04
stgraberjosepht: not really. The next upstream release (1.0 beta1) will introduce a new --packages parameter to the Ubuntu template but until then you're on your own15:20
josephtstgraber: okay, I added a --packages parameter to the ubuntu template :)15:21
jdstrand_cjwatson: hey, I just filed this-- not sure you are aware: bug #125820216:00
ubottubug 1258202 in logrotate (Ubuntu) "recent group permissions change on /var/log cause logrotate errors" [Undecided,New] https://launchpad.net/bugs/125820216:00
=== jdstrand_ is now known as jdstrand
keesgeofft: hola? what's up?16:01
kees!-ubottu16:04
ubottuubottu is <alias> ubotu - added by Pici on 2008-05-08 16:09:51 - last edited by Pici on 2008-05-08 16:09:5716:04
kees!-ubotu16:04
ubottuubotu aliases: yourself, bot, usage, factoid, brain, add, help me, syntax, factoids, everything, me, ubottu, bots, fact, triggers, trigger - added by Seveas on 2006-06-19 12:15:56 - last edited by tsimpson on 2010-09-18 20:14:5016:04
xnoxbarry: Jenkins Notification <devnull@canonical.com>16:04
cjwatsonjdstrand: no, perhaps slangasek can pick this up again since he uploaded the original regression? ;-)16:04
xnoxbarry: that's the emails I used to get jenkins notifications from.16:05
* barry searches his folders16:05
pittibarry: FYI, we used to have notifications for failed autopkgtests, but they've been broken since the DC move16:05
pittibarry: (jibel knows details)16:05
cjwatsonaha!16:05
jdstrandcjwatson: ah :)16:06
=== freeflying is now known as freeflying_away
jdstrandslangasek: fyi, bug #125820216:06
ubottubug 1258202 in logrotate (Ubuntu) "recent group permissions change on /var/log cause logrotate errors" [Undecided,New] https://launchpad.net/bugs/125820216:06
slangasekjdstrand, cjwatson: doh.  How did that merge actually cause the regression, anyway?  We clearly had rsyslog configured to run as non-root before in the Ubuntu delta16:06
barrypitti: ah, okay!  it's a shallow bug then :)16:06
pittibarry: yeah, AFAIK stuck somewhere between firewalling and mail whitelisting16:07
barrypitti, jibel cool.  at least it's on the radar.  thanks16:07
cjwatsonslangasek: upstream rearranged how privilege-dropping worked, so it now irrevocably set[ug]ids near the start and can't retrieve privileges when creating new log files16:08
jdstrandslangasek: don't know. I see that 13.10 livecd has:16:08
jdstrand/var/log                                                 drwxr-xr-x root root16:08
cjwatsonslangasek: I really didn't fancy undoing all that16:08
=== rickspencer3_ is now known as rickspencer3
geofftkees: good morning! I'd like to use LD_AUDIT on a setuid binary :-)16:54
geofftkees: It looks like the disable-ld_audit patch in eglibc is not specifically a CVE fix, but just (worthwhile) paranoia/hardening while upstream got its act together about patching it right?16:55
geofftIs it possible to drop that patch at this point, or is that totally unreasonable?16:56
=== salem_ is now known as _salem
=== _salem is now known as salem_
bdmurraySweetshark: are there any plans to update libreoffice in Trusty? the package version in saucy-proposed is greater than the one in trusty.17:38
Sweetsharkbdmurray: its complicated17:39
Sweetshark;)17:39
Sweetsharkbdmurray: some toolchain updates (likely the usual suspect gcc) made LO 4.1 FTBFS on trusty, so one cant simply bump that. OTOH LibreOffice 4.2 (which we want to have in trusty anyway, but which is still in beta) builds just fine in trusty.17:41
Sweetsharkbdmurray: so I gotta find a way to get 4.1 to build in trusty without wasting too much time on it as nobody will care for that in two weeks anyway. Unfortunately, the breaker seems to be of the nasty kind (its in an external package we build internally in LO, so patching it would mean patching a patch into the build)17:44
Sweetsharkbdmurray: Ill check if we can update mdds to a version building in trusty, and then build LO 4.1 against it (thus not using an internal copy anymore). Using external mdds is would be a good thing anyway (and be worthwhile beyond 4.1) ....17:47
Sweetsharkseb128: ^^ FYI17:47
=== _salem is now known as salem_
bdmurraySweetshark: sounds good, thanks17:52
=== rickspencer3_ is now known as rickspencer3
bdmurraytseliot: the bug number reference in the gnome-desktop3 changelog in the saucy proposed queue is incorrectly formatted which will negatively affect a few things in the SRU process19:11
=== ajmitch_ is now known as ajmitch
tseliotbdmurray: feel free to reject the upload, and I'll re-upload tomorrow19:58
hallynslangasek: stgraber: ok, i'm at wits end trying to get dbus across user namespaces to work.  it could be a simple issue, i don't know.  but my question is:  what is an actual advantage to using dbus versus plain unix sockets?20:01
hallyn(to offset (1) 2-4x slowdown, (2) easy threading at the server, and (3) extra dependencies)20:02
stgraberthe main advantage I was after was the easy availability of client libraries and using something people tend to be familiar with. Suppport for things like introspection and a well defined set of types, method signatures, ...20:03
stgraberthough I'm certainly not against going with our own protocol if DBus simply doesn't cover our needs. As I mentioned, for me backward compatibility is the most important thing, so unfortunately even if DBus could add the features we need, getting that to work on 12.04 will be a pain...20:04
stgraberso I'm personaly fine if we were to go with our own protocol over a raw unix socket. I'd just request that this be a plain text protocol with API versioning support so we don't get into the same kind of problems we had with the lxc monitor protocol (struct mismatch and breakage on upgrades)20:05
stgraberbut it was slangasek who suggested going with DBus, so he may have a few more important reasons :)20:05
hallynstgraber: ok, thanks.  yeah it'll be plaintext and i'll add av ersion #.  plaintext - except the scm cred of course20:12
=== rickspencer3_ is now known as rickspencer3
rbasakhallyn: in general, dbus is nice because it's really easy to map high level APIs down to it. You don't have to do much to get bindings.20:26
hallynrbasak: i got enough koolaid from the tutorial :)20:27
hallynrbasak: as stgraber mentioned, high level bindings will take some upstream work bc we need to send scm_crednetials20:29
bdmurraytseliot: I went ahead and reuploaded them myself20:33
charlieI want to start developing for ubuntu, but was wondering if its okay/suggested to develop in 14.04 daily build installation?20:40
charlieI plan to start all my dev activities on a virtual box having 14.04 daily builds so that my base ubuntu 13.10 stays safe20:41
charlieany ideas/suggesstions?20:41
hallyni think severalppl are using 14.04 daily already20:44
hallyni will be soon20:44
charliecool...thanks20:48
charlieonce installed, am i supposed to update/zsync daily?20:49
slangasekhallyn: the fact that dbus is a structured IPC protocol that the rest of the stack is standardizing on... I think doing your own protocol on top of the socket is going to be harder to sell to the wider community20:50
slangasekhallyn: can you give me details on the problems you're seeing with scm_credentials?  is it because the dbus library is hating on the extra packets?20:50
=== Sweetsha1k is now known as Sweetshark
tseliotbdmurray: excellent, thanks a lot :)20:58
chilicuilcharlie: you can, however if not developing ubuntu itself, I'll recommend you to update once a week on wednesday, or to avoid weekends, canonical employes rest on weekends and you don't want to keep a broken system if you upgrade on saturday or sunday20:59
=== negronjl_ is now known as negronjl
=== Logan__ is now known as Logan_
hallynslangasek: scm-credentials aren't my problem.  I simply can't get a client in a child user namespace to connect to a parent socket in another userns21:21
slangasekah21:21
hallyni haven't figured out why yet,21:21
hallynall I know is that we get just past the AGREE_UNIX_FD,21:21
slangasekand that works with a raw socket?21:21
hallynclient sends BEGIN, server sends some data, then everyone hangs up21:21
hallynthat's nih-dbus21:21
slangasekhmm21:21
slangasekcould you show me the code?21:22
hallynwell for simplicity i was using github.com/stgraber/cgmanager,21:22
hallynbut the problem is'nt there, it's in either nih_main_loop or in dbus_connection somewhere21:22
hallyni tried steppig through the whole process in gdb, all that did was waste 2 hours21:23
hallynltrace doesn't get finegrained enough21:23
hallynslangasek: iiuc the main point of dbus is that it integrates with idiotic junk^W^Wfeatures like policykit21:24
hallynwhich we explicitly don't want.21:24
slangasekno, the main point is to provide a generic IPC mechanism that doesn't require reinventing the wheel21:24
slangasekif you're talking about dbus-daemon, then policykit is an interesting feature; but you shouldn't be talking over dbus-daemon in this context21:25
hallyn'the wheel' is just listen+accept+fire off a thread.  it'll be shorter code than with dbus.21:25
hallyntrue21:25
hallynno bus is involved21:25
hallyndo you know the authentication process well enough to know what should happen next after the AGREE_UNIX_FD?21:25
* hallyn looks back at the 'specification'21:26
slangasekyou still have to design your wire encoding for whatever messages you're passing, which is normally busywork21:26
slangasekno, I'm not familiar with AGREE_UNIX_FD at all21:27
hallynwell I guess it's right after http://dbus.freedesktop.org/doc/dbus-specification.html#auth-command-begin21:27
=== salem_ is now known as _salem
slangasekhallyn: github.com/stgraber/cgmanager seems to only include the server half of the code... what do you have on the client side?  And can you give me a recipe to set up a userns for testing?21:31
stgraberslangasek: my branch contains a "client" in its Makefile, basically just a simple call to dbus-send21:35
slangasekstgraber: ah, ok21:36
hallynslangasek: i also did need to dbus_connection_set_unix_user_function(conn, allow_user, NULL, NULL); where allow_user always returns 0.21:42
hallynsetting up a new vm since last one blew up21:43
hallynstgraber: bug 1258304 looks like your bag :)21:44
ubottubug 1258304 in lxc (Ubuntu) "lxc-create fails on IPv6 address" [Undecided,New] https://launchpad.net/bugs/125830421:44
stgraberhallyn: yeah, I saw it in my e-mails... I'll take a look once I'm done with my current tests21:44
xnoxhallyn: stgraber: libnih API docs http://libnih.la/   so far just the "nih" lib, will update with "nih-dbus" and "nih-dbus-tool"22:19
slangasekhallyn, stgraber: so, trying this same dbus-send command against upstart gives me the same failure; and we know initctl works22:19
hallynxnox: thx22:19
slangasekI think the dbus-send invocation is wrong, though I don't know exactly how22:19
hallynxnox: i'll look at those in awhile when i wanna clean up the code22:20
xnoxhallyn: there still might be mistakes and formatting errors, but most of the things are documented already. + there is libnih tutorial in the upstart cookbook.22:20
stgraberslangasek: against upstart it's probably because we reject from uid > 0. hallyn is actually using a slightly modified version of my branch which allows all uids.22:20
stgraberslangasek: it then works fine as long as you're in the same userns22:20
slangasekhmm22:21
stgraberxnox: yay!22:21
slangasekright, upstart does reject non-root requests on its private socket22:21
slangasektrying to figure out what I'm seeing here, then22:21
hallynlemme push a branch real quick that works for testing,22:22
hallynhm, how do i do that :)22:23
hallynslangasek: i'm testing effectively with github.com/hallyn/cgmanager branch usernstest22:25
hallyni can run the dbus-send as random users, but not from another user ns.22:26
hallynthe credentials during auth phase all come through ok -22:26
slangasekhmm, ok22:27
hallynall right i've got dbgsym packages installed, and gdb is showing me line numbers, but i'd like it to show me code as well so i don't have to keep tabbing back to the vim windows.22:32
slangasekhallyn: 'directory /path/to/source'?22:35
hallynslangasek: and for dbus packages, waht does the path look like?22:35
slangasekhallyn: wherever you have the source unpacked locally :)22:36
hallynslangasek: oh, i thought the dbg packages used to do some magic so i didn'thave to do that22:36
hallynok, thanks22:36

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