| mgz | OOPS-2062AZ5 too late at night... | 01:55 |
|---|---|---|
| mgz | Fourth time lucky, I wonder if putting "lp:" on the start of the prerequisite branch upset anything... | 01:57 |
| lifeless | mgz: bugs.launchpad.net/launchpad-project/+bugs?field.tag=timeout | 01:58 |
| mgz | 77 results being the relevent thing? :) | 01:58 |
| lifeless | no, there is one for +merge-proposal in there ;) | 01:59 |
| mgz | bug 629087 looks likely. | 02:00 |
| ubot5 | Launchpad bug 629087 in Launchpad itself "Branch:+register-merge timeout submitting merge proposal" [Critical,Triaged] https://launchpad.net/bugs/629087 | 02:00 |
| lifeless | yeah thats the one | 02:00 |
| * mgz metoos | 02:00 | |
| === ScottK2 is now known as ScottK | ||
| poolie | hi mgz, scottk | 04:00 |
| ScottK | hello poolie. | 04:07 |
| ScottK | poolie: It's gone midnight, so I'm about to go to bed, but what's up? | 04:07 |
| poolie | just saying hi | 04:07 |
| poolie | thanks for the suggestion | 04:07 |
| poolie | about out of date branches | 04:07 |
| ScottK | Ah. OK. | 04:07 |
| ScottK | Hello back poolie. | 04:08 |
| AuroraBorealis | heya. | 04:26 |
| AuroraBorealis | how do i tell what gets called when you pass commands to bzr? like, bzr add, checkout, merge, etc | 04:28 |
| lifeless | AuroraBorealis: do you mean what python functions run? | 04:29 |
| AuroraBorealis | yeah | 04:29 |
| AuroraBorealis | i see the Command class and that you have to register | 04:29 |
| AuroraBorealis | said commands but i dunno where that gets done o.o | 04:29 |
| lifeless | for plugins during the plugins __init__ | 04:29 |
| lifeless | for core commands by being in builtins.py | 04:30 |
| lifeless | or whatever the file is called :) | 04:30 |
| AuroraBorealis | ah, thank you =) | 04:32 |
| === Topic unset by poolie on #bzr | ||
| AuroraBorealis | oh gee. | 05:29 |
| === poolie changed the topic of #bzr to: Bazaar version control <http://bazaar.canonical.com> | try https://answers.launchpad.net/bzr for more help | http://irclogs.ubuntu.com/ | Patch pilot: JAM | ||
| vila | hi all ! | 06:18 |
| AuroraBorealis | hi... | 06:20 |
| AuroraBorealis | where does the InventoryEntry object store its 'kind' variable? | 06:20 |
| AuroraBorealis | i keep seeing it referenced but i never see it created, its not created in __init__ and ctrl+f shows its not created anywhere else in inventory.py.. | 06:21 |
| AuroraBorealis | never mind i'm stupid, its in the subclasses. although said methods are calling that variable in the super class so they would throw errors if you called said method on the superclass o.o | 06:25 |
| ChrisCauser | Hi Everyone, quick question. I'm reading up a lot about why you shouldn't push a feature branch directly upstream. I was just wondering if there was a way to enforce non changing id numbers so the people I work with don't accidentally do this | 08:30 |
| poolie | ChrisCauser: yes, set append_revisions_only | 08:31 |
| ChrisCauser | Brilliant. Many thanks | 08:31 |
| poolie | vila, so if you look at jam's recent get_parent_map work | 08:38 |
| poolie | i think there's a constant in it which he did some empirical work to change | 08:38 |
| vila | yup | 08:38 |
| poolie | i think i suggested it could be configurable and he opted not to | 08:38 |
| vila | indeed | 08:38 |
| vila | for backport reasons | 08:38 |
| poolie | mm | 08:38 |
| vila | which is tangential but could become a concern if we never use new features by fearing backport issues, but that's not the main point here | 08:39 |
| poolie | so, i don't think going back and changing it to be configurable, which would potentially close 832061, would be especially useful in itself | 08:39 |
| vila | I trust jam's work on setting the "best" value here, but that doesn't mean that for some setups this value will be far from optimal | 08:40 |
| poolie | sure | 08:40 |
| vila | if we can't try new values, we'll never know | 08:40 |
| vila | and if the code evolves and the best value is not the best anymore, we'll never know either | 08:41 |
| jam | poolie: I also didn't want to introduce yet-another-read of something that doesn't-really-need-to-be-set | 08:41 |
| poolie | right | 08:41 |
| vila | and if this constant turns out to be easily tweaked based on some (for the sake of the example) branch history property, we can't tweak either | 08:42 |
| poolie | so, basically | 08:42 |
| vila | jam: right, that's a performance issue | 08:42 |
| poolie | i like bugs that correspond to actual problems and where you can tell whether they're fixed or not | 08:42 |
| poolie | if you find a way to tweak that constant, or to dynamically configure it to make things faster, that's great | 08:43 |
| vila | well, not being able to diagnose whether a constant is good or bad *is* the problem I'm referring to | 08:43 |
| jam | vila: so if it was something like "disable it completely" I would consider a config item. If it is just tweaking it, it really depends on how much effect it could have. | 08:43 |
| jam | Since my experiments said "very little" outside of edge cases | 08:43 |
| poolie | so, being able to change a thing when we don't have any actual reason to think we want to change it is Low | 08:44 |
| poolie | there are a lot more important bugs than that | 08:44 |
| vila | well, I won't ask you to extend your experiments to lp hosted code... | 08:44 |
| jam | vila: so in *this particular* case, I did a fair amount of analysis and testing (a couple days at least) to know that the setting was not going to be very optimizable. I don't know that it applies in every case | 08:44 |
| jam | vila: that was tested on lp hosted code | 08:44 |
| vila | *all* lp hosted code | 08:45 |
| jam | vila: I don't think we really care about the perf on lp:bzrtools as long as it is reasonable, and I tested against lp:gcc-linaro lp:mysql and lp:emacs | 08:45 |
| poolie | so, vila, if you had a good reason to suspect that by further tweaking we could get a big improvement, i'd say go for it | 08:45 |
| jam | I suppose I could do lp:launchpad | 08:45 |
| jam | but I did try to do different types of histories, etc. | 08:45 |
| poolie | but that doesn't seem to be where we are | 08:45 |
| jam | Yes, this experiment could be done again in a year, etc. But honestly, unless someone is going to spend the time analyzing the results, it won't help much. | 08:46 |
| vila | poolie: it's not about *me* it's about anybody willing to try with its *own* know code | 08:46 |
| jam | And if they are doing that, they don't need a config setting | 08:46 |
| vila | what do they need then ? Modifying the code ? | 08:46 |
| jam | vila: sure, or a plugin that modifies the constant, or ... | 08:46 |
| poolie | so if someone is going to tune this, it's probably pretty easy for them to make it configurable at that point | 08:46 |
| jam | in the end, you need to write a script | 08:46 |
| jam | and that can fairly easily get at what you need | 08:47 |
| vila | or just a fix for bug #491196... | 08:47 |
| ubot5 | Launchpad bug 491196 in Bazaar "want a way to set configuration options from the command line" [Medium,Confirmed] https://launchpad.net/bugs/491196 | 08:47 |
| poolie | i think adding a command-line option is possibly the thing that would help them most | 08:47 |
| poolie | right, i would say that is far in advance of going back and making every thing configurable | 08:47 |
| poolie | i think too, many things that could be tuned don't want to be tuned by just changing an integer | 08:47 |
| poolie | but possibly changing the algorithm, which is beyond what this can do | 08:47 |
| vila | right, changing algorithms is better done via hooks | 08:48 |
| vila | but hooks can also be configured | 08:48 |
| poolie | or experimenting with an actual patch | 08:48 |
| jam | vila: or you can write a plugin... | 08:48 |
| jam | or monkey patch, or. | 08:48 |
| jam | honestly, if someone wants to experiment with the code, I really don't see a huge difference from them playing with the *code* | 08:49 |
| vila | jam: or set a command-line option, which one is the easiest for someone wanting to experiment without knowing the code base ? | 08:49 |
| jam | vila: someone who wants to tweak this is going to need to know a whole lot more than a command line option | 08:49 |
| jam | note *this* | 08:49 |
| jam | Some other things would be good with a command line/environment var | 08:49 |
| poolie | vila, a command line option would be great | 08:50 |
| poolie | very nice for interactively testing config related stuff, aside from anything else. | 08:50 |
| jam | vila: I agree that having a config item that can be set from a command line (env var, argument passed) is the easiest way to play with bulk behavior. | 08:51 |
| jam | In this case, I think it is YAGNI. | 08:51 |
| jam | You also want to avoid doing the whole fetch | 08:51 |
| jam | so you need to hack the code anyway | 08:51 |
| poolie | vila do you see what i mean about prefering 'know you're done when' bugs? | 08:51 |
| jam | otherwise the 1-5min startup time is lost in the 30min transferring 600MB of data. | 08:51 |
| poolie | vila do you see what i mean about prefering 'know you're done when' bugs, and why this isn't really one at the moment | 08:52 |
| vila | poolie: yes, but we disagree on "it's done" :) | 08:54 |
| poolie | when would it be done? | 08:55 |
| vila | we had many cases in the past where one setting (let's just say 'encoding') wasn't properly found (for whatever reasons) and users were blocked because there was no way to override this setting | 08:55 |
| poolie | yup | 08:56 |
| poolie | i would certainly agree with that | 08:56 |
| poolie | in fact i think i have filed a bug about that particular case, or at least someone has | 08:57 |
| vila | If we can associate that kind of setting with a *config* setting, we can at least say: 'Look, we'll fix that in the latest stable series, in the mean time, just set it to x in this context) | 08:57 |
| vila | I argue that whatever setting (constant or not) should be a config setting to cover these cases and I'd call it done when the know ones are | 08:58 |
| poolie | yep | 08:58 |
| poolie | if it names some specific things that aren't configurable but should be i'd be totally fine with the bug | 08:58 |
| poolie | (bit questionable whether to have one bug or several but the simplest thing would be to start with one) | 08:59 |
| poolie | perhaps better to fix the bugs that are already filed though | 08:59 |
| vila | :) | 08:59 |
| poolie | some of them, like fsenc, are nontrivial to change | 08:59 |
| jam | vila: we've had code bugs in the past that we couldn't fix in the stable release. If we could write some python config values we could say "write this config and it will fix the bug for now". | 09:00 |
| jam | oh, that is a plugin... | 09:00 |
| jam | I'll agree that 'config' seems a little better than a plugin. | 09:00 |
| jam | meaning "lighter for the user" | 09:00 |
| vila | and doesn't require python knowledge | 09:00 |
| vila | I argue that it it was that easy to write a plugin we'll have far more of them today | 09:01 |
| jam | vila: but does require to know the config value to set | 09:01 |
| jam | and if we have to give them something to use | 09:01 |
| jam | we can give them the plugin | 09:01 |
| vila | I'm not saying plugins are bad and should never be written, I'm saying that too few people can write them or even install them | 09:02 |
| poolie | ok... | 09:02 |
| vila | jam: look at bug #320593 and tell me how long the proposed plugin will work | 09:02 |
| ubot5 | Launchpad bug 320593 in Sedot "nggak bisa buat file rrd setelah ganti system" [Undecided,Fix released] https://launchpad.net/bugs/320593 | 09:02 |
| vila | shudder | 09:02 |
| jam | vila: I'm arguing that the set.difference() between set(users_can_set_a_config_we_recommend).difference(set(users_that_can_install_a_plugin_we_recommend_on_a_bug)) is small | 09:03 |
| vila | jam: look at bug #302593 and tell me how long the proposed plugin will work | 09:03 |
| ubot5 | Launchpad bug 302593 in Bazaar "Reading options for a branch from locations.conf fails with bzr+ssh" [Medium,Confirmed] https://launchpad.net/bugs/302593 | 09:03 |
| poolie | jam, could you review or maybe finish off https://code.launchpad.net/~spiv/bzr/faster-stacked-tree-build/+merge/70381 some time? | 09:04 |
| jam | vila: longer than writing a config item for it | 09:04 |
| jam | poolie: it is in my "in progress" queue | 09:04 |
| jam | so yes | 09:04 |
| poolie | cool thanks | 09:04 |
| vila | jam: huh ? Turning a constant into a config item should be a one-liner (if you exclude the help which you need to write anyway) | 09:05 |
| jam | vila: "Reading options for a branch from locations.conf fails with bzr+ssh " | 09:05 |
| jam | are you sure you were on the right bug? | 09:05 |
| poolie | vila, so | 09:05 |
| poolie | let's see if i understand | 09:05 |
| poolie | you think that generally speaking we should have configs-with-defaults rather than magic numbers | 09:05 |
| vila | jam: https://bugs.launchpad.net/bzr/+bug/302593/+attachment/1157814/+files/fixLocationConfigForBzrSsh.py | 09:06 |
| ubot5 | Ubuntu bug 302593 in Bazaar "Reading options for a branch from locations.conf fails with bzr+ssh" [Medium,Confirmed] | 09:06 |
| vila | poolie: yes | 09:06 |
| poolie | i agree with that, though i'd have some concern about the scalability of documentation if it includes lots of stuff most people should never change | 09:06 |
| poolie | some of them should perhaps be debug flags with values | 09:06 |
| jam | vila: but you could never fix that with a config change | 09:06 |
| vila | poolie: which is bug #746993 | 09:06 |
| ubot5 | Launchpad bug 746993 in Bazaar ""bzr help configuration" looks awful" [Medium,Confirmed] https://launchpad.net/bugs/746993 | 09:06 |
| poolie | iiuc you also think we should go back and look for all things that could potentially be config options and change them | 09:06 |
| jam | so yes, the plugin is fragile, it works until you can upgrade to a newer release which has the bug fixed | 09:06 |
| jam | and the failure is so large in scope that it is completely outside of "tweaking a config item" | 09:07 |
| poolie | well, it goes beyond that, into perhaps having an 'internal' or 'hidden' class of options | 09:07 |
| poolie | i don't think that's worth doing until there's actually a motivation to change them | 09:07 |
| jam | poolie, vila: We've also falling into the trap in the past of having configurable settings that people can tweak, but not providing a good default. | 09:07 |
| jam | because we stop at "ok, they can configure it" | 09:08 |
| poolie | that's a risk too | 09:08 |
| jam | and defaults really do matter | 09:08 |
| vila | know risk | 09:08 |
| vila | known risk, but I'm talking about the other end of the spectrum where there is *no* way to change it | 09:08 |
| jam | vila: I feel we got a bit off on the wrong foot, though. I agree that *in general* I'd rather have something exposed to the user easily in case something goes wrong. | 09:09 |
| jam | This particular case, the reward / benefit was not high enough. | 09:09 |
| jam | When we have properly cached configs, that may change. | 09:09 |
| poolie | cool | 09:09 |
| vila | jam: pfew, thanks | 09:09 |
| jam | vila: note that I was clear all along that *this case* was how I handled it. | 09:09 |
| vila | jam: right, I never said you did wrong given the existing code base 1 | 09:10 |
| vila | I'm saying many problems we encountered *could* be far easier to address | 09:11 |
| vila | poolie: yes, this may not be the first thing to work on *today* but I'd like us to keep it in mind and not punting every time | 09:11 |
| poolie | ok, i just read https://bugs.launchpad.net/bzr/+bugs?field.tag=config | 09:12 |
| vila | poolie: in that respect, I feel that 'low' is pretty close to 'wontfix' | 09:12 |
| poolie | i think the ones that are High are plausibly high | 09:12 |
| vila | well, those are the ones I filed yesterday (except for the first one), I didn't touch the others but I took them into account when writing the devnotes/configuration.txt | 09:13 |
| jam | vila: the flip side of the argument is that you've spent a fair amount of time reworking the config stuff, which still isn't quite enough of a change, vs just fixing the bugs we run into that could have been easier with a better config system. | 09:14 |
| jam | (limited human time and all that) | 09:14 |
| jam | I really like where you are pushing config | 09:14 |
| jam | I'm not sure if it was the optimal use of time | 09:14 |
| poolie | jam, yeah, that's why i'm looking at that list | 09:15 |
| poolie | perhaps vila can finish up the High bugs there and any that naturally fall out from doing it and then move on for a bit | 09:16 |
| poolie | perhaps to udd | 09:16 |
| vila | jam: my counter-argument to that is that I didn't spent that much CPU time on it considering the results and that we are now with two designs in flight and that is already a concern | 09:16 |
| jam | vila: sure, I certainly think it has been started, it is in flight, we need a clear landing for it | 09:17 |
| jam | as poolie said, a clear "it is done" | 09:17 |
| jam | but I think you're at a point it can be ratcheted down | 09:17 |
| jam | rather than a primary focus. Perhaps not | 09:18 |
| vila | well, it's certainly not the only thing I'm working on and I fail to see which of the ~10 mps landed in the last 15 days is not on the cirtical path | 09:18 |
| poolie | so, in <http://people.canonical.com/~mbp/kanban/vila-kanban.html> (which may be lossy) | 09:20 |
| poolie | i see 7 for the last month, and 3 are config related (of which 1 is my fdatasync bug, and fairly serious) and 4 are test suite failures | 09:21 |
| vila | poolie: yup, definitely lossy because I didn't file bugs I was fixing (at the some time slicing the features doesn't match filing bugs in my mind but I can change that (which I started yesterday)) | 09:21 |
| poolie | anyhow, i think john and i are just keen to have config infrastructure and also to get your help on other things | 09:26 |
| jam | vila: poke about https://code.launchpad.net/~vila/bzr/missing-stacks/+merge/71918 | 09:42 |
| jam | You said "Remote branches ignore bazaar.conf and locations.conf" care to clarify? | 09:42 |
| jam | it sounds dangerously wrong | 09:43 |
| jam | as locations.conf is provided as a way to *override* settings in branch.conf for remote items. | 09:43 |
| jam | remote branches | 09:43 |
| soren | Hi. I'm hoping someone can help me decipher this error message: | 09:46 |
| soren | bzr: ERROR: Parent not accessible given base "bzr+ssh://bazaar.launchpad.net/%2Bbranch/nova/" and relative path "../../../~soren/nova/trunk/" | 09:46 |
| soren | It's in response to "bzr lp-propose-merge -m 'something'" | 09:46 |
| soren | http://paste.ubuntu.com/673669/ is my ~/.bazaar/locations.conf | 09:46 |
| soren | And I have *no* clue what ../../../~soren/nova/trunk/ is. | 09:47 |
| soren | Oh. It's from Launchpad. | 09:53 |
| soren | :-$ | 09:53 |
| poolie | hi | 09:57 |
| vila | "it sounds dangerously wrong", see revno 5999, I'm not introducing that | 09:57 |
| poolie | oh, so i think your parent_branch config setting is wonky | 09:57 |
| vila | jam: "it sounds dangerously wrong", see revno 5999, I'm not introducing that | 09:58 |
| poolie | hm, or, which branch are you trying to propose? | 09:58 |
| vila | jam: I'm just formalizing the existing code | 09:58 |
| poolie | soren, basically what's happening is that bzr is trying to infer which branch you intend to merge into, and getting an apparently impossible path | 09:58 |
| soren | poolie: Can you try this, please: | 09:58 |
| soren | poolie: bzr info lp:nova | 09:58 |
| jam | vila: that was *only* for reading *just* branch.conf to get the stacking location | 09:58 |
| soren | Just for shits and giggles. | 09:58 |
| jam | not *all* settings | 09:58 |
| jam | if that's what you've done, great. | 09:59 |
| soren | poolie: ...because I get that error doing that as well. ~soren/nova/trunk is something that only ever existed on Launchpad. | 09:59 |
| soren | poolie: Hang on, I'll post the .bzr.log. | 09:59 |
| jam | (we no longer support using locations.conf to override a stacked location, but we still support it for everything else) | 09:59 |
| vila | jam: it's not used yet but the plan is to use it only for this case if that's possible | 09:59 |
| vila | jam: and neither do we for default_stacking_on if I understand correctly | 10:00 |
| soren | poolie: http://paste.openstack.org/show/2269/ | 10:00 |
| jam | vila: I think you're right about default_stacking_on but I have limited experience with it. | 10:00 |
| soren | poolie: It seems to get the ~soren/nova/trunk bit back from lp (no clue why, though) and tries to make sense of it locally. | 10:00 |
| soren | poolie: Or rather, ../../../~soren/nova/trunk which makes even less sense to me. | 10:00 |
| vila | jam: but thanks for the poke, I'll double-check (now that you mentioned it, I may have fall into a trap with that) | 10:01 |
| soren | poolie: This happens even if I explicitly specify the target branch. | 10:01 |
| poolie | vila, can he fix this with bzr config to update the branch conf? | 10:01 |
| vila | jam: which in retrospect was my main concern with revno 5999 (even if I didn't make a funny face at the time ;) | 10:02 |
| soren | poolie: ..and with an empty locations.conf as well, apparently. | 10:02 |
| vila | poolie: in theory yes, in practice there may be bugs with remote branches | 10:02 |
| vila | ... they should be filed and fixed of course ;) | 10:02 |
| poolie | soren, did you perhaps rename this branch from something else to be the trunk? | 10:03 |
| jam | soren: it is a setting in .bzr/branch/branch.conf which is saying that the "parent" branch is ../../../~soren/... | 10:03 |
| jam | I'm not sure why we need the parent | 10:03 |
| jam | ah, lp-propose is failing | 10:03 |
| jam | because it is trying to auto-detect what branch you want to target | 10:03 |
| poolie | yes, that's what i said :) | 10:03 |
| soren | jam: .bzr/branch/branch.conf: http://paste.ubuntu.com/673676/ | 10:04 |
| jam | poolie: yeah, your statements were intermixed around when I was chatting with vila, so I missed bits | 10:04 |
| jam | silly conversation threading | 10:04 |
| poolie | soren i think 'bzr config -d lp:nova --remove parent_location' should fix it | 10:04 |
| soren | poolie: Nope. | 10:04 |
| soren | poolie: I haven't referred to anything as trunk in over a year. | 10:04 |
| jam | soren: it is the remote one that is probably the issues | 10:04 |
| jam | issue | 10:04 |
| soren | I'm happy to try it, but: | 10:05 |
| soren | http://paste.ubuntu.com/673682/ | 10:05 |
| vila | I suspect we have been propagating parent_location from local to remote branches in the past (and we may still do so) | 10:05 |
| jam | soren: the small issue is that launchpad changed how you access branches, from always being 3-parts (~user/project/branch) to sometimes just +branch/project | 10:05 |
| soren | There's no reference to anything called trunk. | 10:05 |
| jam | soren: the issue is lp:nova | 10:05 |
| soren | ~soren/nova/trunk was abandonded well over a year ago. | 10:05 |
| soren | jam: Oh. | 10:05 |
| jam | soren: http://bazaar.launchpad.net/~hudson-openstack/nova/trunk/.bzr/branch/branch.conf | 10:06 |
| soren | jam: That makes even less sense to me: | 10:06 |
| * vila pesters about not being able to decipher /+branch-id/362222 | 10:06 | |
| soren | http://paste.ubuntu.com/673683/ | 10:06 |
| soren | jam: Oh, dear. | 10:06 |
| soren | jam: That is so wrong. | 10:06 |
| jam | soren: I'm guessing you created the branch by branching from soren all those years ago | 10:06 |
| soren | jam: Yes, that's entirely likely. | 10:07 |
| jam | soren: what version of bzr are you using? | 10:07 |
| soren | Now? | 10:07 |
| soren | Bazaar (bzr) 2.4.0 | 10:07 |
| jam | soren: bzr config -d lp:nova "parent_location=" | 10:08 |
| soren | I can't. | 10:08 |
| jam | should get rid of that setting | 10:08 |
| soren | I'm not ~hudson-openstack. | 10:08 |
| jam | ugh | 10:08 |
| soren | ...but I guess I could become him. | 10:08 |
| jam | soren: we can get a sys-admin to do it if you can't | 10:08 |
| jam | but it takes a while | 10:08 |
| * soren does so while whistling innocently. | 10:08 | |
| poolie | you're asking a sysadmin? | 10:09 |
| vila | I think he's becoming ~hudson-openstack and can't answer from there ;) | 10:09 |
| jam | poolie: soren is doing it himself | 10:10 |
| jam | well, at least for now | 10:10 |
| poolie | oh, really | 10:10 |
| jam | if he comes back saying it doesn't work, i'll contact a losa | 10:10 |
| jam | poolie: it sounds like ~hudson-openstack is a bot | 10:10 |
| jam | (hudson and all that) | 10:10 |
| poolie | yeah | 10:10 |
| * vila scratches head... | 10:12 | |
| soren | I'm battling this right now: Unable to copy ownership from "/var/lib/jenkins/.bazaar" to "/var/lib/jenkins/.bazaar/locations.conf". You may want to set it manually. | 10:12 |
| soren | GImme a minute. | 10:12 |
| vila | launchpad redirects lp:nova to https://code.launchpad.net/~hudson-openstack/nova/trunk but 'bzr config -d<xx>' doen't give the same result | 10:13 |
| soren | What does this mean for me? | 10:13 |
| soren | uark! | 10:14 |
| soren | http://paste.ubuntu.com/673691/ | 10:14 |
| soren | That can't be good. | 10:14 |
| poolie | vila ^^ ? | 10:15 |
| soren | lp-propose-merge does seem to be happier, though. | 10:15 |
| vila | poolie: wow, that is so wrong, lp urls are not even supposed to be supported... 'lp:novalp:nova' ... is wrong too but | 10:16 |
| vila | and bug #832653 filed anyway | 10:17 |
| ubot5 | Launchpad bug 832653 in Bazaar "bzr config -d lp:nova gives unexpected results" [Medium,Confirmed] https://launchpad.net/bugs/832653 | 10:17 |
| vila | soren: what's your 'bzr config' output in the branch you ran lp-propose ? | 10:18 |
| soren | vila: http://paste.ubuntu.com/673692/ | 10:19 |
| jam | soren: still looks wrong here: http://bazaar.launchpad.net/~hudson-openstack/nova/trunk/.bzr/branch/branch.conf | 10:21 |
| soren | Yes, ti does. | 10:21 |
| soren | it, even. | 10:21 |
| vila | soren: so you're not under /home/soren/src/openstack/nova ? | 10:21 |
| soren | vila: I am. | 10:22 |
| soren | vila: Oh, sorry. | 10:22 |
| vila | meh | 10:22 |
| vila | ha | 10:22 |
| jam | soren: sftp bazaar.launchpad.net ; cd ~hudson-openstack/nova/trunk/.bzr/branch; rm branch.conf | 10:22 |
| soren | 10:02 < soren> poolie: ..and with an empty locations.conf as well, apparently. | 10:22 |
| soren | vila: I cleared out locations.conf to see if that was causing it. | 10:22 |
| vila | ha, ok, empty locations.conf was the missing bit | 10:22 |
| jam | soren: though you can continue helping vila debug remote config issues if you want. | 10:23 |
| vila | soren: keep in mind that locations.conf *overrides* branch.conf (but seem to act as a default value provider) | 10:23 |
| soren | vila: Wow, really? | 10:23 |
| soren | That's... wow. | 10:23 |
| soren | Ok. | 10:23 |
| soren | Is that by design? | 10:24 |
| jam | soren: Users have access to locations.conf | 10:24 |
| jam | but may not have access to branch.conf | 10:24 |
| soren | How can they not have access to branch.conf? | 10:24 |
| jam | soren: You can't write to *my* branch.conf | 10:24 |
| jam | though you may read it when branching from me | 10:24 |
| jam | so locations.conf was designed as a way to override settings for your personal use | 10:24 |
| jam | but also grew globs | 10:25 |
| vila | soren: see bug #832046 | 10:25 |
| ubot5 | Launchpad bug 832046 in Bazaar "needs a way to specify default values for config options by path" [High,Confirmed] https://launchpad.net/bugs/832046 | 10:25 |
| jam | which meant that it started providing defaults as well as overrides | 10:25 |
| soren | Fascinating. | 10:25 |
| jam | which makes things messy | 10:25 |
| jam | you need 3 layers, basically | 10:25 |
| soren | Defaults, branch, overrides. | 10:25 |
| jam | defaults get overridden by branch local config gets overriden by user specified overriedss | 10:25 |
| soren | Right. | 10:25 |
| soren | That lp:novalp:nova thing... Is that going to cause troupble? | 10:26 |
| soren | Or trouble? | 10:26 |
| vila | tyops welcome ;) | 10:26 |
| fullermd | Rule 1: You always need 1 level more than you have. Rule 2: The cognitive load limit is always 1 level less than you have :p | 10:27 |
| soren | vila: "tyops"? | 10:27 |
| soren | vila: Oh. typos :) | 10:27 |
| vila | well, I would be surprised if it applies to lp:nova urls anyway since urls are supposed to be translated already when dealing with locations.conf | 10:27 |
| soren | vila: orly? | 10:27 |
| vila | soren: yeah, I do less tyops once I invoked the tyop god | 10:28 |
| vila | only | 10:28 |
| vila | but it doesn't work everytime | 10:28 |
| fullermd | Well, maybe this time dog will simile on you. | 10:28 |
| vila | fullermd: smile ? | 10:31 |
| poolie | pqm might be having trouble? | 10:32 |
| poolie | i've asked the sysadmins | 10:32 |
| vila | jam: wow, you sent https://code.launchpad.net/~vila/bzr/824513-fadatasync-options/+merge/72454 to pqm *after* I did (lp is misleading, I did the send after the push) | 10:34 |
| vila | jam: still I didn't get any answer, did you ? | 10:34 |
| jam | vila: I was trying to submit one of mine, I just read the proposal wrong and submitted yours | 10:35 |
| vila | poolie: blackbox.test_branch.TestBranch.test_branch_broken_pack FAIL is spurious, dunno if we have a bug for that though | 10:35 |
| jelmer | we do | 10:35 |
| poolie | yes it's broken; will be fixed soon | 10:35 |
| poolie | o/ jelmer | 10:35 |
| jelmer | hey poolie :) | 10:35 |
| vila | jam: ha, ok, still did you get an answer or is indeed pqm borken, ha right | 10:35 |
| vila | jelmer: hey ! | 10:36 |
| poolie | can you guys re-feed them if necessary when the paper jam is cleared? | 10:36 |
| jam | vila: I didn't get any response and the web page is "0 scripts" for me | 10:36 |
| jam | poolie: certainly | 10:36 |
| vila | jam: ok, same here | 10:36 |
| jam | want us to resubmit your "trivial" as well? | 10:36 |
| poolie | yes thanks, i have two or three approved | 10:37 |
| poolie | i have 10 assigned bugs and i think all but one or two are blocked | 10:37 |
| jam | poolie: in what channel did you ask the sysadmins? (Just trying to follow the thread) | 10:40 |
| jelmer | jam: #is | 10:42 |
| poolie | #is | 10:44 |
| jam | poolie, jelmer: Are we doing the udd standup in 5 min? | 10:55 |
| poolie | no, it's on holiday for augest | 10:55 |
| poolie | *august | 10:55 |
| jam | ah, ok | 10:55 |
| poolie | afaik | 10:55 |
| mgz | afternoon all. | 11:10 |
| * jelmer waves to mgz | 11:12 | |
| vila | mgz: _o/ | 11:19 |
| poolie | hi mgz | 11:21 |
| * poolie fixes a package import, yay | 11:21 | |
| poolie | hm, bug 832052 is interesting, if it was hit on 2.4.0 | 11:28 |
| ubot5 | Launchpad bug 832052 in bzr (Ubuntu) "bzr crashed with ShortReadvError in _seek_and_read(): readv() read 0 bytes rather than 174 bytes at 0 for "469df00e67f848246f876663fb299e10.rix" (dup-of: 413430)" [Undecided,New] https://launchpad.net/bugs/832052 | 11:28 |
| ubot5 | Launchpad bug 413430 in Bazaar "ShortReadvError on index file" [High,Confirmed] https://launchpad.net/bugs/413430 | 11:28 |
| poolie | which should have fdatasync on | 11:28 |
| === nyuszika7h is now known as somebody | ||
| === somebody is now known as nyuszika7h | ||
| jelmer | Riddell: thanks for backporting that i18n fix! | 11:49 |
| Riddell | jelmer: the rt got a reply so we should have access to PQM's 2.4 now | 11:49 |
| jelmer | Riddell: that was quick | 11:50 |
| === idnaria is now known as idnar | ||
| === davi` is now known as Guest70489 | ||
| mgz | jam: ping | 13:05 |
| jam | hi mgz, I was wondering when you would show up | 13:05 |
| mgz | ha, we were both waiting for each other to say something first? | 13:05 |
| jam | I guess so | 13:05 |
| Riddell | anybody object if I just turn on translations for the bzr project in launchpad? I don't see a reason why not | 13:16 |
| vila | Riddell: +1 (I thought it has been done :-/) | 13:16 |
| jelmer | Riddell: go for it | 13:17 |
| vila | Riddell: what series will be impacted ? | 13:17 |
| Riddell | that's the next question, do we want to point people at 2.4 or trunk | 13:17 |
| jelmer | Riddell: perhaps have it export to a different branch rather than directly to trunk | 13:17 |
| jelmer | Riddell: does 2.4 have enough support for translations for translations to actually be usable? | 13:18 |
| vila | Riddell: if there is a way to point to trunk-only first that will allow us to debug the fallouts | 13:18 |
| Riddell | jelmer: yes, for most of the command help text | 13:19 |
| vila | jelmer: only command helps IIRC, so testable but not that useful | 13:19 |
| jelmer | I'd prefer targetting trunk in that case; we don't know what other fallout there may be from translations | 13:19 |
| Riddell | jelmer: what's your worry about exporting to trunk? | 13:30 |
| jelmer | Riddell: worried about a bot committing directly to one of our branches (and perhaps conflicts with pqm) | 13:41 |
| Riddell | yep | 13:43 |
| Riddell | lp:bzr-translations-export ? | 13:44 |
| Riddell | I wonder who should own it | 13:45 |
| Riddell | ~bzr-core I guess | 13:45 |
| jelmer | Riddell: does it have to be a new project? | 13:56 |
| Riddell | jelmer: no, I've set it to lp:~bzr-core/bzr/bzr-translations-export | 13:56 |
| jelmer | Riddell: cool | 13:57 |
| RenatoSilva | is there a lp plugin command for listing all branches of a project or your junk? | 14:48 |
| jelmer | RenatoSilva: I believe someone wrote one, but I'm not sure where you can find it | 14:49 |
| RenatoSilva | well, in the lp plugin, no? | 14:49 |
| RenatoSilva | a separate lp plugin? ok | 14:50 |
| jelmer | RenatoSilva: there isn't anything in the standard lp plugin | 14:51 |
| jelmer | though I think it would be an appropriate place for it | 14:51 |
| RenatoSilva | ok thanks jelmer | 14:53 |
| === jam1 is now known as jam | ||
| vila | jelmer: on jubany, there are uncommitted changes to plugins/builddeb and we miss 41 revisions from lp:builddeb | 15:08 |
| vila | jelmer: shouldn't we at least update a bit ? | 15:08 |
| jelmer | vila: I have to admit, I've never looked at the bzr-builddeb on jubany | 15:08 |
| jelmer | what are the uncommitted changes? | 15:08 |
| vila | jelmer: (the uncommitted changes comment out installing the debian changelog merge hook) | 15:09 |
| jelmer | that sounds reasonable | 15:10 |
| vila | jelmer: and the revno on jubany is 566 (607 on trunk) | 15:10 |
| vila | jelmer: what ? Not installing the hook ? | 15:10 |
| jelmer | with regard to updating, I think it's a good idea but it would probably need some testing with current udd | 15:10 |
| jelmer | vila: yeah, the previous hook didn't very wekll | 15:10 |
| jelmer | *well | 15:10 |
| vila | work well , got that ;) | 15:11 |
| vila | weird, I thought spiv was happy with it... I should have miss some episodes | 15:12 |
| vila | jelmer: what kind of testing ? | 15:12 |
| jelmer | vila: Spiv added a new changelog merge hook | 15:12 |
| jelmer | vila: but I'm pretty sure that was after r566 | 15:12 |
| vila | jelmer: ha ! So it may be worth re-trying the new one right ? | 15:13 |
| vila | jelmer: yeah, looks like revno 596 | 15:13 |
| jelmer | vila: some internal APIs in bzr-builddeb have changed that udd might rely on | 15:15 |
| vila | hmm | 15:16 |
| jam | jelmer, vila: Didn't we switch to the system installed versions? Or was that for python-debian? | 15:25 |
| vila | python-debian | 15:25 |
| vila | jelmer: devscripts is not installed on jubany but you said 'Add dependency on devscripts >= 2.10.59, required now that 'dch -- | 15:26 |
| vila | package' is used. LP: #783122' does that apply to jubany ? | 15:26 |
| vila | ha, crap | 15:28 |
| vila | jelmer: by internals API you meant DistributionBranch ? | 15:28 |
| jelmer | vila: that would apply for the changelog merger | 15:30 |
| jelmer | vila: though I'm surprised the changelog merger matters at all for jubany - where is that used? | 15:31 |
| jelmer | vila: yep | 15:31 |
| vila | the devscripts dep you mean ? It seems to require dpkg-dev instead but that one is installed | 15:32 |
| vila | jelmer: so for DistributionBranch, udd doesn't use kwargs so an update is probably required and also probably in lock-step with bzr-builddeb update :-/ | 15:33 |
| vila | ok, harder than I thought, I'll file a bug | 15:34 |
| vila | bug #833097 | 15:43 |
| ubot5 | Launchpad bug 833097 in Ubuntu Distributed Development "bzr-builddeb needs to be updated on jubany" [High,Confirmed] https://launchpad.net/bugs/833097 | 15:43 |
| === beuno is now known as beuno-lunch | ||
| === beuno-lunch is now known as beuno | ||
| gdoubleu | I'm seeing odd behavior trying to push to an svn repo: I make a change, commit it, and the dpush. A commit gets recorded in the svn repo but it is empty, and my working tree returns to the state prior to the commit (i.e. bzr st/diff shows the file/change I had previously committed). Any ideas? | 18:37 |
| gdoubleu | this is with bzr 2.4.0 and bzr-svn 1.1.0dev | 18:38 |
| gdoubleu | I tried a fresh branch of the svn repo, but seeing the same behavior | 18:38 |
| jelmer | gdoubleu: this is bug 826946 | 18:40 |
| ubot5 | Launchpad bug 826946 in Bazaar Subversion Plugin "commits to http repository are empty" [Critical,In progress] https://launchpad.net/bugs/826946 | 18:40 |
| gdoubleu | ah, I searched for push and missed this one | 18:41 |
| jelmer | gdoubleu: are you using http as well?> | 18:41 |
| gdoubleu | well, https | 18:41 |
| jelmer | this is actually the bug that is blocking the 1.1.0 release | 18:43 |
| gdoubleu | jelmer, is there a previous, working version of bzr-svn that's still compatible with bzr 2.4.0? | 18:46 |
| jelmer | gdoubleu: one of the earlier snapshots of lp:bzr-svn, but I'm not sure which one | 18:53 |
| === med_out is now known as medberry | ||
| gdoubleu | jelmer, fwiw using bzr-svn at revspec date:2011-07-01 works here | 19:33 |
| yofel | hey, is there are reason why bzr dailydeb substitutes {date} while fetching the source from bzr? Thanks to that it downloads the source every day I run it even if I already fetched the source already | 19:55 |
| jelmer | yofel: not sure I follow - if you don't want the date in there, why not use e.g. {revno} ? | 19:56 |
| yofel | jelmer: no, I want the date in there. What I wonder is why if I have '2+git{date}+r{revno}-{revno:packaging}' as version in the recipe. bzr dailydeb makes a stacked branch in like 'work/kdelibs-2+git20110824+r{revno}-{revno:packaging}' when building | 19:58 |
| yofel | why doesn't it leave {date} unsubstituted? | 19:58 |
| yofel | like that I need to fetch the branch every day | 19:59 |
| jelmer | yofel: it rebuilds if the version changes | 19:59 |
| jelmer | yofel: and the date changes every day :) | 19:59 |
| jelmer | it sounds like you want {revdate}, which is the latest bzr-builder that hasn't yet been deployed to lp | 20:00 |
| yofel | jelmer: this is running bzr builder by hand - not on launchpad | 20:00 |
| yofel | since I still need to upload some by hand | 20:00 |
| jelmer | https://bugs.launchpad.net/bzr-builder/+bug/793072 | 20:00 |
| ubot5 | Ubuntu bug 793072 in bzr-builder "Provide deb-version variables for revision commit time/date" [Medium,Fix released] | 20:00 |
| yofel | well, revdate would probably help - if it doesn't substitute it at branch time | 20:01 |
| jelmer | yofel: are you running 0.7.1? | 20:01 |
| yofel | hm, no, my server is on natty - so 0.6. Did that get fixed? | 20:02 |
| jelmer | yofel: no, {date} is still generated at branch construction time | 20:04 |
| jelmer | s/generated/expanded/ | 20:05 |
| jelmer | yofel: on launchpad these directories always get discarded so it doesn't really matter | 20:05 |
| jelmer | yofel: you're welcome to file a bug, and I'll see how hard it is to fix | 20:06 |
| yofel | ok, I've got plenty of bandwidth myself, so it's not serious. Was just wondering why I have to download a few hundred MBs every day if I already have the branches on disk | 20:06 |
| jelmer | yofel: have you considered running it inside of a shared repository? | 20:07 |
| yofel | what's a shared repository? ^^ | 20:08 |
| jelmer | yofel: http://wiki.bazaar.canonical.com/SharedRepositoryTutorial | 20:09 |
| yofel | k, thanks for the pointer | 20:09 |
| === mwhudson__ is now known as mwhudon | ||
| === mwhudon is now known as mwhudson | ||
| poolie | hi all | 22:31 |
| jelmer | poolie! | 22:32 |
| poolie | hi there | 22:33 |
| poolie | that is good news about mrevell | 22:41 |
| === medberry is now known as med_out | ||
| === yofel_ is now known as yofel | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!