| Chipaca | popey: BUUUUUG | 00:18 |
|---|---|---|
| === lifeless_ is now known as lifeless | ||
| === zumbi is now known as Guest40451 | ||
| karni | Night guys | 02:46 |
| === smb` is now known as smb | ||
| Chipaca | *yawn* | 08:38 |
| Chipaca | morning, all | 08:38 |
| mandel | morning all! | 09:36 |
| JamesTait | Good morning all! :D | 10:07 |
| === Chipaca_ is now known as Chipaca | ||
| nessita | buenos días! | 11:31 |
| mandel | nessita, morning! | 11:45 |
| mandel | nessita, I have an interesting question for you, got time? | 11:46 |
| nessita | mandel: sure! | 11:46 |
| mandel | nessita, I've been looking at writing integration tests with squid + ssl so that we can test the cases in which we have a selfsigned and get an error | 11:47 |
| mandel | nessita, what happens is that due to license issues the package of squid3 that ubuntu ships does not use the --enable-ssl flag, which means that it does not support that | 11:47 |
| * Daviey wonders if it turned out not infact to be interesting, if nessita could redact her offer of time. | 11:47 | |
| mandel | nessita, with some help from #ubuntu-servers I've got a ppa with squid3 + ssl for the tests.. but, how do we manage this? | 11:48 |
| mandel | nessita, should I forget squid and look into a diff path? mock the ssl issues for example? | 11:48 |
| nessita | mandel: guh... question, what --enable-ssl provides? | 11:49 |
| Daviey | mandel: Do you need a full proxy, or would a stub https server be enough? | 11:49 |
| mandel | nessita, http://www1.it.squid-cache.org/Versions/v3/3.HEAD/cfgman/https_port.html | 11:49 |
| mandel | Daviey, at the moment we are using a full proxy to try and have the integration tests as real as possible, if not I think I can do mock | 11:49 |
| mandel | nessita, I can also do squid3 -v check if the flag was used and skip the ssl tests if it wasn't | 11:50 |
| Daviey | mandel: it seems to me that you only need to test as far as the handshake.. | 11:50 |
| Daviey | we simply cannot enable redistribution of squid linking with openssl. | 11:50 |
| mandel | Daviey, I know.. licenses.. | 11:51 |
| nessita | mandel: would you please tell me what does --enable-ssl provides (without using a link for the answer)? :-D | 11:52 |
| mandel | nessita, ppa:lynxman/squid-ssl already compiled for amd64 http://people.canonical.com/~lynxman/squid3/ | 11:52 |
| nessita | mandel: let's go step by step, I'm trying to understand your issue before understanding one of the solutions (the PPA) | 11:52 |
| mandel | nessita, it allows squid to listen connections over https | 11:52 |
| Daviey | mandel: if you want to add gnutls support for squid, i'd be over the moon :) | 11:52 |
| nessita | mandel: ok, so, without that, you can not set a proxy test to be https://localhost:foo | 11:53 |
| mandel | Daviey, no no no, I don't have the time to fix the world :) | 11:53 |
| mandel | nessita, yes, that is the issue | 11:53 |
| * nessita thinks | 11:53 | |
| nessita | mandel: if you have the patch file that adds that support, we could add that to our PPA, though I'm not thrilled to do that | 11:54 |
| nessita | Daviey: hola! | 11:54 |
| mandel | nessita, yes, I'm not happy either.. | 11:55 |
| nessita | Daviey: what's the licensing issue with that part of squid? | 11:55 |
| nessita | Daviey: ping? | 11:58 |
| Daviey | nessita: sorry, it's BSD and GPL linking | 12:01 |
| Daviey | YOU can do it, WE just can't redistribute it. | 12:01 |
| nessita | mandel: I guess a pacthed squid in our ppa can work. We just need the patch to apply, and we can upload the new squid to ppa:ubuntuone/nightlies | 12:03 |
| mandel | nessita, is that an ok solution, I mean, I don't want to screw people because of this | 12:03 |
| nessita | mandel: is not a problem, as far as I can see, we can check with dobey if he sees any issue, but I don't see one | 12:04 |
| nessita | Daviey: thanks! | 12:04 |
| mandel | nessita, cool :D | 12:04 |
| nessita | mandel: I read your comments to my review. ABout the fakes... why you need to fake the arrow? why not juts letting the code use the real one (since is there in the pythonpath)? | 12:06 |
| mandel | nessita, I don't mind using the real one, yet a prefer tests not to relay in real code, I mean, is the arrow is broken, the arrow tests should fail but not the expander ones, but if you prefer not to do that I'm happy to remove it :) | 12:08 |
| mandel | nessita, I also misunderstood what you meant with the expanded arg, fixed that, you where right, I called the set_expanded with it rather than store it as false | 12:08 |
| mandel | nessita, and removed the @property to use property() to make the code cleaner | 12:08 |
| nessita | mandel: so, you're right about that when it comes to 100% pure unit tests. But I personally think that if we *only* build 100% unit tests, we're missing a lot of things. And since we don't have the time to write 100% unit tests and thin layers of integrating modules, I advice to have a test suite that runs relatively fast, and to mock only the strictly necessary | 12:09 |
| nessita | mandel: so, unless something is a pain (such as using a socket, for example, or an unpatchable class, or sys.modules, or something like that) I would advice not doing mocking | 12:10 |
| mandel | nessita, sure, no problem, what ever is best for the team, I'll remove those, but.. any ideas on how to assert that the correct methods have been called? or should I just test that the get returns the expected value | 12:10 |
| mandel | ?? | 12:10 |
| nessita | mandel: the latter, since testing that methods 'a', 'b', 'c' were called in that order will break the test when we change the method 'a' to be 'z' which does the same as 'a' but is better, for example | 12:11 |
| duanedesign | o/ | 12:11 |
| mandel | nessita, ok, should be a simple change, let me get down to it | 12:12 |
| nessita | mandel: or sometimes you test 'a', 'b', 'c' were called, and the we move some stuff around in our internal implementation due to refactoring, and that will break the test (and, depending on the case, shouldn't) | 12:12 |
| nessita | hola duanedesign | 12:12 |
| mandel | nessita, I think I was just stating that the method was called with the args, but you are right, I'll just assert that the expected value is present | 12:13 |
| mandel | nessita, and is easier to remove lines than to add them :) | 12:13 |
| nessita | mandel: right. Ack to the setter and _expanded prop | 12:18 |
| nessita | alecu: hola! I need some help from your end | 13:26 |
| nessita | alecu: would you have some minutes? | 13:27 |
| ralsina | good late morning! | 13:33 |
| dobey | Daviey: because squid is BSD and openssl is GPL? | 13:36 |
| karni | Hey, JAVA devs out there - We're in the review stage of Ubuntu One Files Java Client, which is a library that allows to talk to U1, manipulate files, up/download, etc. If you'd like to give it a go, this is the last branch: | 13:39 |
| karni | lp:~karni/ubuntuone-files-java-client/interactive-cli-example | 13:39 |
| karni | If you like U1 and code for Android, you may want to find it of interest, too :) | 13:39 |
| Daviey | dobey: other way around, but openssl is bsd-like | 13:41 |
| mandel | ok, I'm off to have lunch | 13:41 |
| * mandel lunch | 13:41 | |
| dobey | Daviey: because openssl has advert clause then? | 13:43 |
| nessita | alecu: ping? | 13:44 |
| Daviey | dobey: http://www.squid-cache.org/mail-archive/squid-dev/200406/0011.html | 13:45 |
| karni | aquarius: If anyone asks about U1 Java library, I just shouted out we are happy to have initial public preview, as stated few lines above. I'd like to weed out any rough edges with whoever uses it first (including me ;)) | 13:46 |
| karni | joshuahoover: ↑ | 13:46 |
| aquarius | karni, cool! | 13:46 |
| ralsina | mandel, dobey, Daviey: that probably means noone ever uses squid with ssl, so we canjust skip those tests, I say | 13:46 |
| nessita | ralsina: squid with ssl is a way to have a proxy for https, which a lot of people may have (even though is not squid behind that) | 13:47 |
| aquarius | karni, we should chat with roberta about it; she's at MWC this week though | 13:47 |
| nessita | ralsina: so we need that to build the integration tests | 13:47 |
| ralsina | nessita: yes, but we are not going to work around this by packaging an illegal squid for ourselves, are we? | 13:48 |
| karni | aquarius: Agreed. Although I prefer to treat it like u1db 'preview' - once we know it's prime time ready, we can announce it. | 13:48 |
| nessita | ralsina: is not an illegal squid, as far as I understood it | 13:48 |
| ralsina | nessita: it's not distributable | 13:48 |
| karni | aquarius: I have a dead simple CLI example, and will probably quickly write a UI-enabled example. | 13:48 |
| ralsina | nessita: but I may have missed some licensing subtlety | 13:49 |
| dobey | nessita: we would have to each build it ourselves locally | 13:49 |
| dobey | nessita: it couldn't be in the PPA | 13:49 |
| nessita | dobey: ok, so we can build it in those system running the squid tests, no? | 13:49 |
| aquarius | karni, definitely, that's why I want to talk to roberta :) | 13:49 |
| dobey | well, unless we used gnutls instead | 13:49 |
| karni | aquarius: ah, cool :) | 13:49 |
| aquarius | karni, and you and I should chat about it; a blog post by you would be cool, I think | 13:49 |
| karni | aquarius: That would be nice :) | 13:50 |
| nessita | alecu: you around? | 13:51 |
| dobey | nessita: i'm not sure testing that works against squid, will tell us anything useful | 13:52 |
| karni | aquarius: Also, this is a silly detail, but I've grown to think ubuntuone-java-files-*client* is not a great name for a library | 13:53 |
| karni | aquarius: What do you think? | 13:53 |
| nessita | dobey: why not? | 13:54 |
| aquarius | karni, well... it isn't, really, but I don't know how Android and Java stuff like this is really named | 13:54 |
| karni | aquarius: Same goes for ubuntu-sso-java-*client* - it is a client of the API, but a library to the software that will use it | 13:54 |
| karni | aquarius: say, google analytics builds a libGoogleAnalytics.jar | 13:54 |
| ralsina | karni: +1 on removing the client | 13:55 |
| karni | ralsina: ack | 13:55 |
| ralsina | karni: if anyone tries to use it as a server, he will figure it out soon enough ;-) | 13:55 |
| karni | aquarius: none of the libraries we use in the u1-files-java-client (we use like 13) have "lib" or "library" in their names | 13:55 |
| karni | ralsina: haha | 13:55 |
| dobey | nessita: because we're testing against a configuration which noone should actually have anyway. there's no guarantee that squid and IIS and whatever else, all do ssl proxies the same way. we need to test against actual server configurations people will be using | 13:55 |
| aquarius | karni, do java things normally get called a "library"? It'd be useful to see what similar stuff is called | 13:56 |
| dobey | nessita: also, have you seen https://bugs.launchpad.net/ubuntu/+source/ubuntuone-client/+bug/942312 ? | 13:56 |
| karni | aquarius: I'll google | 13:56 |
| karni | aquarius: not really. they just call by names, like commons-logging-1.1.1.jar, jackson-mini-1.9.4.jar, signpost-commonshttp4-1.2.1.1.jar | 13:57 |
| dobey | aquarius: things that are libraries do; though people generally refer to them by name as in any other language | 13:58 |
| aquarius | dobey, this is my point: I'm not sure the Java world talks about "libraries" in the same way we do | 13:58 |
| dobey | aquarius: what way do we talk about libraries? | 13:59 |
| nessita | dobey: regarding the first comment, I agree, though since having real configs is very hard (for now at least, on every day testing envs), I would like to have at least one way to test the SSL cert thing | 13:59 |
| aquarius | dobey, a block of reusable code, packaged up on its own with the intention of being included in another project wholesale, is called a "library"; in C it's normally called libsomething. Java... I'm not sure they'd call that a library. I think they might call it a jar, for example. | 14:00 |
| nessita | dobey: yes to the second, was pinging alecu about it | 14:00 |
| dobey | aquarius: in java, everything is a jar. in C/C++ we only name the binaries libfoo.so out of tradition. and to make the compiler command line look nicer. there really isn't any technical requirement that it be that way. in fact, the correct technical term for anything with an .so in *nix is "shared object" | 14:01 |
| aquarius | dobey, exactly my point. If a Java person is looking for a shared object to help them talk to U1, would they google for "u1 java library", or something else? I don't know, which is why I asked karni :) | 14:02 |
| * karni reads, was googling around | 14:03 | |
| dobey | aquarius: they would probably search for "ubuntu one java" and see what comes up. they might search also for "class library" or "implementation" or other technically useful terms | 14:03 |
| dobey | aquarius: i don't ever search for "library" really when i am looking for some implementation in C or python or whatever | 14:03 |
| mandel | karni, call it jar jar binks! | 14:04 |
| dobey | aquarius: it's a term that is a bit too general and will often dilute results | 14:04 |
| karni | mandel: heh :) | 14:04 |
| pedronis | aquarius: they both use library or package, library to do X or package to do X | 14:05 |
| karni | aquarius: dobey: I also stumbled upon this http://www.debian.org/doc/packaging-manuals/java-policy/x104.html | 14:05 |
| karni | and this http://stackoverflow.com/questions/4177762/naming-java-library-how-to | 14:06 |
| karni | Candidates would be: jUbuntuOneFiles.jar , libUbuntuOneFiles.jar | 14:06 |
| karni | like jMock.jar, libGoogleAnalytics.jar | 14:07 |
| aquarius | libUbuntuOneFiles.jar sounds sensible to me | 14:08 |
| karni | aquarius: The library jar name is not such a problem, the project name is. Clearly, as I've stated, I have mixed feelings towards "Ubuntu One Files Java Client" - if we drop client, it just sounds funny :) | 14:09 |
| dobey | nessita: what do we need to test with the "ssl cert thing"? we can probably fake it, if it's just to test the dialog and parsing or something? | 14:09 |
| karni | aquarius: maybe the project name could contain 'api' keyword, then when we publish Java music library, it have a consistent name | 14:09 |
| pedronis | well I wouldn't be surprised by ubuntuone-files-client.jar either | 14:09 |
| karni | aquarius: The project name, that is. | 14:09 |
| aquarius | "Ubuntu One Files Java library" seems OK to me? | 14:10 |
| pedronis | karni: what's the package name: com.ubuntu.one....? | 14:10 |
| nessita | dobey: we have those (unit) tests in place, mandel was hoping to also have integration tests like with the rest of the proxy support work | 14:10 |
| dobey | karni: are you just wanting to rename the project? | 14:10 |
| karni | pedronis: That's also the question. Currently it's com.ubuntu.one.api.files | 14:10 |
| karni | pedronis: We're not sure if it should be com.ubuntuone.api.files | 14:10 |
| dobey | nessita: and what would it do? ensure that squid serves an SSL cert to us? | 14:11 |
| pedronis | karni: I think one.ubuntu.com is the more official url, but not sure | 14:11 |
| karni | pedronis: Our U1F Android app is com.ubuntuone.android.files, SSO lib is com.ubuntu.sso | 14:11 |
| nessita | dobey: no, to make squid returns an expired cert, or a self signed cert | 14:11 |
| pedronis | karni: ah so we used com.ubuntuone for something already, better stick to it then | 14:12 |
| pedronis | we own both anyway | 14:12 |
| karni | dobey: I'm looking for most sensible name for the project name - I think we have just found a nice name for the jar files. | 14:12 |
| dobey | nessita: can we not just shove those types of certs directly into the tests without squid? | 14:12 |
| karni | pedronis: nessita: mandel: Do desktop apps use 'ubuntuone' in their package name? (As opposed to com.ubuntu.one...) | 14:13 |
| dobey | karni: yes | 14:13 |
| karni | In that case we should stick with ubuntuone it seems. | 14:13 |
| dobey | ubuntuone-client, ubuntu-sso-client, ubuntuone-control-panel, libubuntuone, rhythmbox-ubuntuone, ubuntuone-storage-protocol | 14:13 |
| karni | dobey: Oh, thank you | 14:14 |
| nessita | dobey: as far as I understand, no, I think we need to have squid returning what we need for the test. I guess we could patch what cert to return, but I'm not sure we can patch squid so it handled https url using the whole ssl "stack" | 14:14 |
| dobey | Album Savings: $57.20 compared to buying all songs | 14:14 |
| dobey | nice | 14:14 |
| Yancho_ | guys after opening ubuntuone-control-panel-gtk i am getting: File Sync error (org.freedesktop.DBUs.Error.NoReply: Did not receive a reply .. any idea from where I can start debugging please? | 14:15 |
| dobey | nessita: i'm not sure why squid has to do anything here. | 14:15 |
| dobey | Yancho_: ~/.cache/ubuntuone/log/syncdaemon-exceptions.log is probably the first place to look. other log files in same directory would be second | 14:16 |
| karni | Thanks for the brainstorm everyone! | 14:16 |
| nessita | dobey: as far as I know, we have the test suite X, that we run without having a proxy, and with a proxy setup using squid. The same test suite X has to pass in both scenarios. We also have some specific test suite to be run when squid is setup, and those test suite depend on the "proxy" to have specific settings/values. mandel is trying to add a new suite where the "proxy" is an https url, and thus with that he expects squid to serve all the ssl bits a | 14:17 |
| nessita | mandel: correct me if I misunderstand something | 14:17 |
| Yancho_ | tnx dobey | 14:17 |
| dobey | nessita: i understand what is trying to be done. what i don't understand is *why* it has to be that way. | 14:18 |
| nessita | dobey: can you please repharse your counterproposal then? | 14:18 |
| dobey | nessita: why do we need integration tests with squid ssl support, when squid with ssl support is so unlikely to be the real world situation? | 14:19 |
| nessita | dobey: we need a test case with a setup where the proxy is an https url and the proxy will give ceratin ssl certs to perform different tests. We're doing it with squid since we already have the testscases to do it, but we could do it with another proxy setup (no idea which one though) | 14:20 |
| dobey | nessita: *WHY* ? :) | 14:20 |
| nessita | dobey: because we need integration tests for the https proxy | 14:21 |
| nessita | dobey: how would you do that ^, without having squid configured to provide a https proxy? | 14:22 |
| dobey | we need integration tests with real world proxies. i don't think we need those integration tests to be in the normal test suite, but rather something we run in jenkins against actual real-world proxy servers also set up somewhere | 14:22 |
| nessita | dobey: I think that's a separated discussion (and it makes sense, but is out of the scope of this talk IMHO) | 14:23 |
| dobey | nessita: for now, the way to test that is probably through manual testing | 14:23 |
| nessita | dobey: if we can have some sort of automation for this, we should do it. Manual testing does not scale... | 14:23 |
| dobey | nessita: then the scope of this talk is squid, and i think we don't need to have integration tests for something nobody is using. | 14:23 |
| dobey | nessita: yes, which is why i said it should be in jenkins, and we should have manual tests being run by qa or whoever, until we can automate it | 14:24 |
| nessita | dobey: the fact that squid is behind those tests is "coincidence" (in the sense that is the thing we can setup automatically). If you can propose a way to have another proxy setup differently, let's talk about that. And I propose this separated from the fact that we should run the proxy suite for each possible proxy setup we want to support. | 14:25 |
| nessita | ralsina: your thoughts? ^ | 14:26 |
| dobey | i think we should not do integration tests at the moment then, as we haven't the means to do them. | 14:26 |
| ralsina | nessita: I did some minor research right now, and there is no free proxy we can use for this | 14:26 |
| nessita | ralsina: besides squid, you mean? | 14:26 |
| ralsina | and we can't block on it. So I propose we find a way to set one of these on jenkins | 14:26 |
| ralsina | nessita: I propose setting up a ssl-squid in jenkins and do a job running the tests there | 14:27 |
| nessita | ralsina: yes, that's my proposal as well | 14:27 |
| ralsina | nessita: but we can't distribute the squid-ssl package | 14:27 |
| nessita | ack to that | 14:27 |
| ralsina | nessita: ok then, we agree ;-) | 14:27 |
| nessita | ralsina: agree with dobey now :-P | 14:28 |
| ralsina | nessita: OTOH, we can't get blocked with this | 14:28 |
| ralsina | THERE ;-) | 14:28 |
| ralsina | dobey: the point of squid-ssl tests is that there are certain things we can test there, like whether we are rejecting invalid certificates and so on | 14:29 |
| ralsina | dobey: I have no idea whatsoever on how to fake that, unless we recorded a session or something like that | 14:29 |
| dobey | ralsina: that seems like something that belongs in a unit test, and not an integration test | 14:30 |
| nessita | ralsina: one of the goals is not to fake that (just FYI) | 14:30 |
| ralsina | dobey: it can't be a unit test if it depends on the behaviour of other software. That's the whole point of integratiokn tests. | 14:30 |
| ralsina | nessita: I know, I was the one that pushed for integration tests for proxys in the first place :-) | 14:31 |
| dobey | ralsina: the problem is that tests depending on other software, that nobody is using, isn't useful though. and i feel it's a waste of time to do all the work to make it work for squid, and then we say it works and ship it, and ssl proxies doesn't work for anyone in the real world. | 14:32 |
| dobey | ralsina: we effectively have the same statistics as only testing the unit side of it. | 14:32 |
| ralsina | dobey: no, because the protocol for SSL proxys is a standardprotocol | 14:32 |
| ralsina | dobey: it's like saying that testing agains IIS is not testing HTTP | 14:33 |
| dobey | ralsina: IMAP is a standard too. but we all know how well that worked out. | 14:33 |
| dobey | ralsina: and it would be true. :) | 14:33 |
| ralsina | dobey: yes. This may be more like HTTP than like IMAP, though | 14:33 |
| dobey | you will be surprised then i guess :) | 14:33 |
| ralsina | dobey: ha | 14:35 |
| ralsina | So, nessita, let's think about getting this on jenkins | 14:36 |
| === m_conley_away is now known as m_conley | ||
| dobey | i'd recommend poking the platform/server/debian people to ship squid built against gnutls | 14:37 |
| ralsina | dobey: well, that's a great solution to have thought about 3 months ago ;-) | 14:37 |
| ralsina | dobey: does it even *have* a gnutls build option? | 14:38 |
| nessita | dobey: mandel poke Daviey and apparently they are not doing it... | 14:39 |
| ralsina | nessita: whenyou have a slot, I changed the spacing in the argparse thing. (damn, I was so close to get that one through without a needsfixing ;-) | 14:40 |
| dobey | ralsina: --enable-ssl; gnutls has compatibility stuff that replaces openssl headers and uses gnutls compat lib instead. don't know if it "just works" though | 14:40 |
| nessita | ralsina: ack! | 14:41 |
| dobey | nessita: as i understand it, it's just that openssl won't (can't) be done. | 14:43 |
| nessita | dobey: sorry, I already contextswitched my brain. Would please give me more context for that? | 14:43 |
| dobey | let's wait for mandel | 14:44 |
| dobey | ugh, i think sinuses are making my lungs hurt | 14:49 |
| mandel | dobey, I'm back, let read the backlog | 14:52 |
| mandel | dobey, ralsina, nessita so, yes, the ssl enabled squid stuff is a very strange scenario.. but we don't have numbers about that. If it was not used they wouln'd provide the --enable-ssl flag | 14:57 |
| mandel | also, we cannot distribute it, so its a matter of having that in jenkins | 14:57 |
| mandel | what I propose if to have the SquidTestCase to check if the --enable-ssl flag was used, and if it was enable the https ports, otherwise skip it | 14:58 |
| dobey | mandel: where did you actually discuss the license issues outside of #ubuntuone? did you discuss the possibility of building it with gnutls instead? | 14:58 |
| mandel | dobey, it was mentioned.. but was just a proposal. I have not looked into that. I did most of my talks in private with one of our guys from ubuntu-server | 14:59 |
| mandel | dobey, there is a bug about that with a won't fix due to license issues (open-ssl) version, I did not look more into using gnutls | 15:00 |
| nessita | dobey, mandel: for now, I would discourage trying to build with gnutls, giving everyone's time constraints | 15:00 |
| nessita | me | 15:00 |
| mandel | nessita, +10000000 ** 100 | 15:00 |
| mandel | me | 15:00 |
| nessita | dobey, ralsina, alecu, briancurtin, standup? | 15:02 |
| briancurtin | me | 15:02 |
| dobey | not me | 15:02 |
| alecu | me | 15:04 |
| urbanape | me | 15:04 |
| mandel | nessita, remember we have the mac boy ( urbanape) with us :) | 15:05 |
| alecu | mandel, after the standup, can you give me a summary update on the SSL issue? | 15:08 |
| mandel | alecu, sure, is an easy one :) | 15:09 |
| dobey | nessita: are you going to go with the standup, or we will wait forever for ralsina ? | 15:09 |
| alecu | mandel, (btw: I'm adding some SSL integration tests to the SD proxy stuff, so I'm adding a dummy certificate and key and some code to set that up) | 15:10 |
| ralsina | please go ahead | 15:10 |
| briancurtin | while not ralsina: time.sleep(1) | 15:10 |
| nessita | DONE: more on bug #933576, reviews, holiday on Monday | 15:10 |
| nessita | TODO: the same! :-) also, start conversation about Control Panel's new look on Ubuntu | 15:10 |
| nessita | BLOCKED: nopes | 15:10 |
| nessita | NEXT: mandel | 15:10 |
| mandel | DONE: looked at setting tests to use https with squid. --enable-ssl is not there, we need to agree (I think we already did) in the approach. Removed most of the fakes from my ss-dialog branch. | 15:10 |
| mandel | TODO: Remove the rest of the fakes, ping nessita +1 to review it. Implement what we agree on about squid + ssl. | 15:10 |
| mandel | BLOCKED: not more than usual. | 15:10 |
| mandel | next: briancurtin | 15:10 |
| * ralsina is currently on his 112th minute on the phone, thanks | 15:10 | |
| briancurtin | DONE: got started with urbanape, adjusted some tests in the read_link issue to work with new NTFS hardlink impl | 15:10 |
| briancurtin | TODO: continue working with urbanape over ssh/screen/mumble on getting the buildout to work, receive my OS X install CD from FedEx and...install it | 15:10 |
| briancurtin | BLOCKED: none | 15:10 |
| briancurtin | NEXT: not dobey | 15:10 |
| dobey | λ DONE: rb nightlies fixes, bug #934235, | 15:10 |
| dobey | λ TODO: rb extension bugs, look at twisted g/gtkapplication support/hanging issue | 15:10 |
| dobey | λ BLCK: none. | 15:10 |
| dobey | alecu | 15:10 |
| alecu | DONE: cleanups on disconnections for bug #929207, debugging, fixing tests, natholiday on monday | 15:10 |
| alecu | TODO: rework bug descriptions for FFE, finish and submit branches | 15:10 |
| alecu | BLOCKED: no | 15:10 |
| alecu | NEXT: urbanape | 15:10 |
| urbanape | DONE: Compiled PyQt ALL DAY LONG. | 15:10 |
| urbanape | TODO: Hacking on the buildout with briancurtin | 15:10 |
| urbanape | BLOCK: Nothing, so far | 15:10 |
| urbanape | end? | 15:11 |
| nessita | any comments anyone? | 15:11 |
| ralsina | urbanape: compiling Qt is a rite of passage. Wait for PyQt ;) | 15:11 |
| mandel | alecu, how are you writing those tests? | 15:12 |
| nessita | if not, I guess is eom! | 15:12 |
| urbanape | ralsina: did PyQt | 15:12 |
| alecu | mandel, let me summarize what these tests are testing first: | 15:12 |
| urbanape | DOUBLE RITE | 15:12 |
| ralsina | urbanape: brave man! | 15:12 |
| alecu | mandel, SD needs to connect to the sync servers going thru the proxy tunnel. The tunnel listens for a few lines telling it where to connect, and then it forwards all traffic transparently. | 15:13 |
| alecu | mandel, but SD needs a "client" to connect to the tunnel and send the connection command (in plaintext) and then it needs that client to switch from the tunnel protocol to the ssl used to transport the SD traffic | 15:15 |
| alecu | mandel, so my tests are testing that transition: from plaintext (used to speak to the tunnel server) to SD protocol inside a SSL encrypted channel. | 15:16 |
| alecu | mandel, that's why I need to run a fake SSL server. And that SSL server needs a dummy certificate file and a dummy key file, that I generated manually and I'll be including in our tree. | 15:17 |
| mandel | alecu, ok, so that is a diff ssl scenario, I wanted to write tests for when the proxy that we are listening to is in a https port. I wanted to ensure that when we go over https and have a cert that is no vlaid we show the dialogs etc.. | 15:17 |
| rye | wait, so we are going with an external proxy-handling process? Why don't we use some existing one? | 15:18 |
| mandel | alecu, I can provide you with code that will generate a random cert instead, is very easy with open ssl | 15:18 |
| alecu | mandel, cool, but I'm not sure if we'll have all the ssl cmdline tools in other platforms. | 15:19 |
| nessita | ralsina: can I have a review please? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/work-on-windows/+merge/94980 | 15:19 |
| mandel | alecu, we have the library :) | 15:19 |
| alecu | rye, what existing proxy handling process do you propose? | 15:19 |
| mandel | alecu, http://paste.ubuntu.com/860638/ | 15:20 |
| mandel | alecu, some imports are missing, but they are from os, so its easy :) | 15:20 |
| ralsina | nessita: sure, in about 10' | 15:21 |
| alecu | rye, the idea to build one was so that it uses the same library (QtNetwork) as the rest of our tools, so we have the same proxy support everywhere. | 15:21 |
| alecu | mandel, oh, that's lovely. | 15:21 |
| rye | alecu, ah | 15:21 |
| rye | https://wiki.edubuntu.org/RomanYepishev/UbuntuOne/ProxySupport :) | 15:21 |
| rye | why is edubuntu wiki ranked higher in google searches than wiki.ubuntu.com ? | 15:22 |
| mandel | rye, because they are educated.. hehehehehe | 15:22 |
| dobey | rye: also, the last 2 points on that page are quite valid. and i suspect those tools don't work so well on windows? | 15:26 |
| nessita | brb | 15:33 |
| alecu | rye, proxytunnel is interesting. I evaluated using similar tools like cntlm, but each tool supported a different subset of options (ie, ntlm authentication, or socks proxies, etc) and supported different platforms, so we would end up depending on a handful of tools. | 15:36 |
| alecu | mandel, re: your generate_self_signed_cert() function; is it already on our sourcetree? | 15:39 |
| mandel | alecu, no, is work in progress of ubuntuone-dev-tools, feel free to use it and I'll deal with cleaning the duplication :) | 15:40 |
| alecu | mandel, great, thanks. | 15:41 |
| alecu | mandel, I think for now I'll use the certificates made by hand, but when this landsw on u1-dev-tools I'll switch to it, because my certificate will expire in 10 years :-) | 15:42 |
| mandel | alecu, hehe | 15:42 |
| mandel | alecu, I'll let you know when it does | 15:43 |
| ralsina | nessita: +1 on work-on-windows | 15:44 |
| dobey | nessita: did you file a bug, or know of one, about the messaging menu opening the gtk control panel? | 15:45 |
| alecu | dobey, I saw it filed under bug #735464 | 15:48 |
| dobey | alecu: that is a separate bug, though people are commenting on it because they see similar behavior in 12.04 now | 15:49 |
| dobey | alecu: but it was originally filed on 11.04, and i suspect a different issue | 15:49 |
| alecu | right | 15:50 |
| alecu | popey, ping | 15:50 |
| alecu | popey, I'm trying to understand the bug with proxy hostnames: https://bugs.launchpad.net/ubuntu/+source/ubuntuone-client/+bug/942312 | 15:50 |
| alecu | popey, did you have proxies enabled on the system settings while this happened? | 15:51 |
| popey | alecu: ooh, i think i see the problem. the dialog where you put your proxy.. if you have it set to "manual" you can leave the other fields blank and it still lets you save that | 15:53 |
| popey | alecu: which IMO it shouldn't let you do | 15:53 |
| alecu | popey, well, each app could tell if the "hostname" was empty, and ignore the settings | 15:54 |
| alecu | popey, in this case it looks like libsoup is the bit where it's failing. | 15:54 |
| popey | that shouldn't be necessary really | 15:54 |
| === yofel_ is now known as yofel | ||
| popey | I'll file a bug against the networky bit | 15:55 |
| popey | also, unrelated to that | 15:55 |
| popey | in rb I keep getting a prompt that "Mp3 playback support is not available" | 15:55 |
| nessita | dobey: the one about having -gtk mixed with -qt? I did not file it, nor I recall one about it | 15:55 |
| popey | i click install and it goes off and does stuff but i still cant play music | 15:55 |
| popey | (from the store) | 15:55 |
| popey | and when i restart it says the same thing | 15:55 |
| popey | is this a known issue? | 15:55 |
| dobey | nessita: yeah, the messaging indicator opening the gtk one when you click on it, and using the dbus interface to open the volumes tab | 15:56 |
| nessita | dobey: no bug in my radar for that | 15:56 |
| alecu | popey, no idea about that rb bug. Let's ask dobey. | 15:56 |
| dobey | alecu: no idea. we don't do any magic stuff for proxies there. if the same error comes up there, it could be coming from sso | 15:58 |
| alecu | dobey, no: I meant the "Mp3 playback support is not available" bug. | 15:59 |
| dobey | nessita: right, i have a fix; though it doesn't open to the "folders" section for the folders, as it just runs the installer instead (which in turn runs the qt control panel if installed) | 15:59 |
| alecu | dobey, the issue with proxies is when the user sets the proxies to "Manual", but leaves the proxy host field empty. | 16:00 |
| dobey | nessita: but i was looking for a bug to --fixes= for | 16:00 |
| dobey | alecu: ok | 16:00 |
| dobey | alecu: the mp3 bug is known/filed. i have to debug it still though | 16:00 |
| alecu | popey, ^ | 16:00 |
| popey | ok | 16:00 |
| popey | whilst I'm in rb i also have duplicates of all my u1ms music | 16:01 |
| popey | i mentioned this previously and someone asked if i could get some info from my machine | 16:01 |
| popey | but I dont know what I need to get | 16:01 |
| dobey | popey: yes, that bug is filed | 16:01 |
| nessita | ralsina: your opt-branch approved! | 16:01 |
| ralsina | dobey, nessita: the u1sdtool bug is probably gi/twisted: it blocks waiting for data in a pipe (apparently) | 16:01 |
| ralsina | nessita: cool, thanks | 16:01 |
| dobey | i know what it is. will probably fix it today, and fix will appear in ubuntu next week | 16:01 |
| nessita | ralsina: yeap | 16:01 |
| dobey | ralsina: nope. i have similar issues with other non-twisted things doing it as well | 16:02 |
| ralsina | nessita: if you have a small bug or two, I may take a look while I do other stuff | 16:02 |
| nessita | dobey: I read what ralsina said as 'gi or twisted' | 16:02 |
| ralsina | yes, one or the other :-) | 16:02 |
| nessita | ralsina: let me look | 16:03 |
| ralsina | although it locks inside twisted, it could be caused by the interaction with gi | 16:03 |
| dobey | nessita: not sure. control panel qt has same issue | 16:03 |
| nessita | ralsina: we have this same behavior on other apps not using twisted | 16:03 |
| ralsina | nessita: interesting | 16:03 |
| dobey | i think the qt control panel issue is probably easy to fix though | 16:06 |
| joshuahoover | alecu, ralsina: for bugs #929207 #929208 #929212 we need to answer pitti's concerns (listed on 929207 & probably consolidate these into one u1 proxy support bug that we land the necessary branches against to make life easier for everyone | 16:16 |
| ralsina | joshuahoover: got it, looking | 16:18 |
| nessita | joshuahoover: also bug #933729, no? | 16:18 |
| nessita | ralsina, mandel: ^ | 16:18 |
| joshuahoover | nessita: yes! you beat me to my next one :) | 16:19 |
| nessita | :-) | 16:19 |
| joshuahoover | mandel, ralsina: bug #933729 (http://pad.lv/933729 ) has a comment from pitti w/ a concern he has about handling ssl mismatches that we need to answer | 16:19 |
| ralsina | joshuahoover, alecu, nessita: I guess the answers are "no", "because syncdaemon can't depend on (what?)" and "the system, except for authentication which is not standard"? | 16:20 |
| mandel | joshuahoover, well.. his concerns are right, we are going to let the user be stupid | 16:20 |
| nessita | ralsina: regarding the tunnel process, I know alecu will answer during the day | 16:20 |
| ralsina | nessita: cool | 16:20 |
| mandel | ralsina, right? | 16:20 |
| alecu | joshuahoover, I agree about consolidating some of the proxy bugs | 16:21 |
| joshuahoover | nessita, ralsina: those are the FEs i found that needed some additional info/help...the others look like they'll be ok assuming we get them landed soon...i need to follow up on one other http://pad.lv/933010 which is a uife that i didn't notify the doc and translation lists about yet | 16:21 |
| ralsina | The only possible source for proxy ssl mismatches is that the user misconfigures the proxy name | 16:21 |
| mandel | ralsina, or the it people that set up the proxy certs are stupid, I have worked in one of those.. | 16:22 |
| ralsina | But for example, you will get a mismatch if you configure the proxy's IP address instead of the canonical name, so there *may* be benign mismatches :-( | 16:22 |
| ralsina | mandel, alecu, joshuahoover: So, basically, I don't care as long as we give a good explanation about the mismatch. Supporting misconfigured setups is not a priority. | 16:25 |
| nessita | ralsina: what id you tackle this on you reported? bug #942355 | 16:25 |
| nessita | what if* | 16:25 |
| nessita | (ralsina: speaking of what you asked beore about doing some bugs fixes on the side) | 16:25 |
| ralsina | nessita: ok | 16:25 |
| ralsina | nessita: should not be terribly hard :-) | 16:25 |
| nessita | ralsina: I'll assign that one and another one, JIC | 16:26 |
| ralsina | nessita: cool, thx | 16:26 |
| nessita | ralsina: though... this one requires a UIFe https://bugs.launchpad.net/ubuntuone-control-panel/+bug/942025 :-/ | 16:26 |
| dobey | joshuahoover: we're only supposed to notify docs/translations (probably only translations for this one), after approved though, right? | 16:26 |
| nessita | so, no, that one no | 16:26 |
| ralsina | nessita: I can do the branch then we decide if we ask for it or not | 16:27 |
| joshuahoover | dobey: we have to do it before approval for all UIFEs...to let them know about it and allow them to express their concerns | 16:27 |
| ralsina | ok, not that one :-) | 16:27 |
| nessita | ralsina: I'm talking about another bug (not the one about the scrolling) | 16:27 |
| ralsina | nessita: the email address font size one? | 16:27 |
| nessita | yes | 16:27 |
| nessita | so, the scrolling is bug fix | 16:28 |
| nessita | the font-too-small is UIFe | 16:28 |
| ralsina | ack | 16:28 |
| nessita | ralsina: I assigned the scroll-fix to you | 16:28 |
| joshuahoover | nessita, ralsina: i think http://pad.lv/942025 for the small font size is ok to fix w/o a ui exception but that's just my opinion and definitely not following all the rules ;) | 16:29 |
| nessita | joshuahoover: hum, how do you justify it does not require a UIFe? | 16:29 |
| nessita | joshuahoover: is changing a "screenshot" of that tab | 16:29 |
| dobey | small? | 16:29 |
| joshuahoover | dobey: right, "small" :) | 16:29 |
| dobey | all the fonts in that app are freakin huge | 16:29 |
| ralsina | trivial and obviously wrong and of no consequence? ;-) | 16:29 |
| dobey | and, for some reason, bold. | 16:30 |
| dobey | oh well, i should get some lunch | 16:30 |
| dobey | bbiab | 16:30 |
| joshuahoover | nessita: i guess if we're going to adjust that font size we should have design review it and make sure there aren't any others that need to be adjusted...we're hammering pitti and crew with all these freeze exceptions | 16:30 |
| nessita | joshuahoover: +10 | 16:31 |
| nessita | joshuahoover: I'm putting together an email about the control panel looks | 16:31 |
| joshuahoover | nessita: thank you :) | 16:31 |
| ralsina | maybe we can do a global "make it fit better" UiFe | 16:31 |
| nessita | joshuahoover: so can you please analyze it in detail, when is finished? I have several things I would like to see fixed | 16:31 |
| joshuahoover | nessita: yes, i would be happy to | 16:32 |
| nessita | thanks! | 16:32 |
| nessita | joshuahoover: should be sent today | 16:32 |
| joshuahoover | nessita: cool, i'll let rtgrant know it's coming and will need some design attention | 16:32 |
| nessita | ralsina: oops, text conflict in https://code.launchpad.net/~ralsina/ubuntuone-control-panel/opt-parsing/+merge/94599 | 16:33 |
| ralsina | nessita: oops indeed! looking... | 16:39 |
| mandel | I'll going to grab a coffee, my brain is failing.. | 16:41 |
| mandel | will be back in few mins | 16:41 |
| ralsina | nessita: should we really import the reactor in the bin script? | 16:42 |
| ralsina | nessita: oh, yes we should. Got it. | 16:42 |
| nessita | ralsina: yes | 16:43 |
| nessita | lunchtime for mw | 16:48 |
| nessita | me* | 16:48 |
| === briancurtin2 is now known as briancurtin | ||
| mandel | ralsina, I'm leaving a little earlier, I'm not feeling very well.. | 17:14 |
| mandel | ralsina, coffee went out as fast as it went in.. | 17:15 |
| mandel | I'm off, no feeling well, c u tom | 17:29 |
| nessita | bye mandel, get better | 17:30 |
| mandel | nessita, thx, really appreciate it | 17:30 |
| ralsina | mandel: go take care! | 17:45 |
| ralsina | reviews for a very simple branch please? https://code.launchpad.net/~ralsina/ubuntuone-control-panel/fix-scroll/+merge/95006 | 17:52 |
| dobey | come on and finish already server rescan | 17:56 |
| * briancurtin lunch | 18:04 | |
| dobey | meh; why oh why is the indicator not turning blue | 18:18 |
| ralsina | interesting. Qt suggests localizing the shortcuts like this: _("Ctrl+W") ... let's not do that instead. | 18:24 |
| dobey | srsly? | 18:26 |
| dobey | le sigh | 18:26 |
| dobey | hrmm | 18:27 |
| dobey | i'm pretty sure the album art for "Nine Inch Nails - The Downward Spiral" does in fact, not involve Three 6 Mafia | 18:27 |
| ralsina | dobey,nessita: very simple review -- https://code.launchpad.net/~ralsina/ubuntuone-control-panel/quit-quit-quit/+merge/95020 | 19:00 |
| * dobey wonders how to make the indicator think he has a new folder available | 19:00 | |
| urbanape | should dbus-python pick up qt availability automagically on configure? | 19:06 |
| urbanape | would you think? | 19:06 |
| dobey | it doesn't have the qt bits in it directly does it? | 19:07 |
| ralsina | urbanape: on mac? I doubt it | 19:07 |
| urbanape | dobey: it doesn't seem to, no | 19:07 |
| dobey | the dbus-pythin qt bits are in pyqt no? | 19:07 |
| ralsina | urbanape: you probably need to hack the qt configuration to make it build the dbus mainloop | 19:08 |
| urbanape | trying to untangle the order in which to get all these installed. | 19:08 |
| dobey | maybe you need to build pyqt again after installing dbus-python | 19:08 |
| urbanape | yeah | 19:08 |
| ralsina | urbanape: probably need to build qt first so that it has the dbus support configured, then dbus-python, then pyqt | 19:08 |
| urbanape | so, Amber once traveled to Russia (before all the Westernization). She had to have a visa to go, but in order to get a visa, she had to have a hotel reservation. Before she could get a hotel reservation, she had to have an airline itinerary. Before she could book her airfare, she needed a visa. | 19:09 |
| dobey | urbanape: are we going to try to use dbus there, instead of the mac thing? | 19:09 |
| ralsina | urbanape: OTOH, if we go for the TCP IPC, you don't *need* dbus | 19:09 |
| urbanape | well, since homebrew has a dbus port already, we wanted to see how far we could get without any particular platform changes. | 19:10 |
| nessita | dobey: you can ask thisfred, he did that for the -gtk controlpanel | 19:10 |
| urbanape | fwiw, I'd rather have something working and then change it rather than start with a whole other set of assumptions and not know what we might be missing. | 19:11 |
| nessita | ralsina: may I ask for a favor when building commit messages? would you please write them as (note the dash, the ending dot, the capitalization, the past tense): | 19:11 |
| nessita | - Fixed foo bar baz, yadda yadda doo (LP: #123456). | 19:11 |
| ralsina | nessita: sure | 19:11 |
| nessita | ralsina: thanks! it makes building the changelo much easier :-) | 19:12 |
| nessita | (otherwise I edit every commit message by hand) | 19:12 |
| ralsina | nessita: changing my pending branches to that style | 19:12 |
| nessita | thanks! | 19:12 |
| dobey | nessita: why the -? | 19:13 |
| dobey | also you already know i hate the (LP: #) in there :) | 19:14 |
| nessita | dobey: yes, that's why I don't ask *you* to do it :-P | 19:14 |
| nessita | dobey: the - ease the reading of long changelogs, let me point you at an example | 19:14 |
| dobey | it looks weird in the commit log | 19:15 |
| nessita | dobey: see the changelog in here https://launchpad.net/ubuntuone-control-panel/trunk/2.0.0 | 19:16 |
| nessita | ralsina: you too, if you feel curious | 19:16 |
| dobey | eh | 19:16 |
| nessita | dobey: I think is very important to have the LP: # in there so we can very easily click on the LP link and see what was the bug being fixed | 19:16 |
| dobey | nessita: "there" is the changelog or the commit message? | 19:17 |
| nessita | dobey: changelog, which is veru easily built if commit messages use that format | 19:17 |
| nessita | very* | 19:17 |
| nessita | if not, is a pain to write such changelogs | 19:17 |
| dobey | how is it being built? | 19:17 |
| dobey | some tool is generating it? | 19:18 |
| nessita | dobey: I have a tool that generates, but is a little buggy right now so I use it and then clean a bit by hand | 19:18 |
| dobey | yeah it's buggy. it requires commit messages to be put in like that :) | 19:18 |
| nessita | ralsina: this is great! "- Preserved the scrollbar position when clearing/filling the folder list. (LP: #942355).", just a tiny note, no dot between "list" and (LP: # | 19:19 |
| nessita | dobey: you're too mean sometimes :-) | 19:19 |
| dobey | the tool should be formatting it and reading bugs out of the branch metadata | 19:19 |
| nessita | dobey: perhaps, but I never had (nor wil have, I presume) to make it that good | 19:19 |
| nessita | so, for now, mostly bzr log and hand work | 19:19 |
| dobey | :( | 19:20 |
| dobey | or we can just not make changelog things on the releases :) | 19:20 |
| nessita | dobey: I'm -10 to that | 19:20 |
| nessita | changelog are very important, from my POV | 19:20 |
| thisfred | did we ask the udd people if they have ways of autogenerating sensible changelogs from launchpad information? | 19:21 |
| nessita | thisfred: I did not. What does udd stand for? | 19:21 |
| dobey | hrmm, i guess the indicator only shows for shares from other people | 19:22 |
| dobey | and not new UDFs | 19:22 |
| thisfred | nessita: Ubuntu distributed development | 19:22 |
| nessita | thisfred: no idea that existed :-) | 19:22 |
| thisfred | all the new tools that Elliot showed us a long time ago | 19:22 |
| thisfred | so not so new now ;) | 19:23 |
| dobey | thisfred: i don't think so, no | 19:23 |
| thisfred | nessita: http://developer.ubuntu.com/packaging/html/udd-intro.html | 19:23 |
| dobey | but it's really pretty simple to write a tool that does exactly this | 19:23 |
| thisfred | it's all text | 19:24 |
| nessita | ralsina: why quit is QtCore.QCoreApplication.instance().quit() instead of self.close()? | 19:48 |
| ralsina | nessita: because it's guaranteed to work | 19:54 |
| ralsina | nessita: if the application has a certain flag, it doesn't exit on window closing | 19:55 |
| ralsina | nessita: but I am happy to change it to close() | 19:55 |
| nessita | ralsina: but that's ok, no? I'm not sure we want to force quit if the app has something setup to not quit | 19:55 |
| ralsina | nessita: reasonable | 19:55 |
| ralsina | nessita: changing it | 19:55 |
| nessita | ack | 19:55 |
| alecu | nessita, did you find the cause for the click handler that seemed to make the ping webcalls twice? | 20:00 |
| nessita | alecu: not yet (did not try that much yet) | 20:01 |
| nessita | alecu: are you hitting it? | 20:09 |
| ralsina | nessita: I am slightly surprised that ControlPanel is not a top-level window. Closing it, therefore, requires a small hack, so don't worry about slightly strange code in the next revno ;-) | 20:09 |
| nessita | ralsina: why not make it top-level window instead??? | 20:09 |
| ralsina | nessita: because that means changing a bunch of other things | 20:10 |
| ralsina | nessita: the change seems to be much larger | 20:10 |
| nessita | ralsina: such as? (really don't know) but seems more correct, no? | 20:10 |
| nessita | ralsina: this smells to me like "this bag has a hole in it. Let's attach a container below it so we gather the leaks..." (instead of fixing the hole!) :-) | 20:11 |
| alecu | nessita, lol | 20:11 |
| ralsina | nessita: let me research it a bit then | 20:11 |
| nessita | ralsina: thanks (if you have time, of course) | 20:11 |
| alecu | nessita, no, I'm not hitting the issue with the ping | 20:11 |
| ralsina | nessita: sure | 20:12 |
| ralsina | nessita: here's a good reason: making it a top level window may change something visible, because the top level is a QMainWindow | 20:13 |
| ralsina | nessita: and ControlPanel is not a QMainWindow but a QWidget | 20:14 |
| nessita | ralsina: wait, now I'm confused | 20:14 |
| nessita | the ControlPanel class is supposed to be QWIdget, which is added insise a MainWindow | 20:14 |
| ralsina | nessita: there is a QMainWindow, that has a CentralWidget, which is the ControlPanel class | 20:14 |
| ralsina | nessita: yes. So i you close ControlPanel, the MainWindow is still there :-) | 20:15 |
| nessita | ralsina: then the shortcuts should be added to the QMainWIndow, no? | 20:15 |
| nessita | ralsina: I did not notice this in the diff, but the ctrl+foo should go in the main window, from my POV | 20:15 |
| ralsina | nessita: yes, which is why this was surprising, now that I see it ;-) | 20:16 |
| nessita | ralsina: heh... so we have a plan? | 20:16 |
| ralsina | nessita: yep | 20:16 |
| nessita | (it sounded weird that we did not have a mainwindow, but honestly did not go to check) | 20:16 |
| ralsina | nessita: there is really no need to have a MainWindow for this app. It's more of a QDialog | 20:16 |
| nessita | ralsina: nooooooooooo :-) | 20:17 |
| ralsina | nessita: but it's not important | 20:17 |
| ralsina | nessita: Main Windows have menus! ;-) | 20:17 |
| nessita | ralsina: we could have menus ;-) | 20:17 |
| nessita | but don't say it out loud :-D | 20:17 |
| ralsina | nessita: we have a bug for that,and dobey and I marked it wontfix ;-) | 20:17 |
| dobey | heh | 20:18 |
| ralsina | argh, my mouse pointer just froze. So I am stuck in this channel. Hopefully noone mentions me in another one. | 20:24 |
| dobey | :) | 20:26 |
| ralsina | dobey: quassel has a handy "monitor window" where I see my mentions. Shame on you! ;-) | 20:26 |
| ralsina | nessita: pushed with the action on the right widget, works and everything. | 20:28 |
| dobey | heh | 20:28 |
| nessita | YEAH | 20:28 |
| ralsina | Not that it didn't work the last two times, mind you ;-) | 20:28 |
| ralsina | And thus we win the xmonad market! \o/ | 20:29 |
| urbanape | briancurtin: well, no matter what order I try to build these, I'm not getting the qt dbus mainloop created. | 20:34 |
| briancurtin | urbanape: hmm. i have no idea where to go from there, so i guess its time to google away | 20:36 |
| ralsina | urbanape: you probably need to hack qt's config file to build it | 20:36 |
| ralsina | urbanape: let me check | 20:36 |
| ralsina | urbanape: are you using clang? | 20:37 |
| urbanape | yup | 20:38 |
| urbanape | uh, no | 20:38 |
| urbanape | LLVM gcc | 20:38 |
| dobey | urbanape: you might want to see what debian/rules is doing for the python-qt4 source package in ubuntu | 20:39 |
| ralsina | dobey: qt builds with dbus support by default on linux | 20:39 |
| urbanape | and I've build Qt with dbus support. It's one of the options for homebrew | 20:40 |
| ralsina | urbanape: in the resulting binaries, do you have a dbusmainloop file? | 20:40 |
| nessita | ralsina: need to change location and computer, will be back to finish reviews | 20:40 |
| * ralsina doublechekcs the name | 20:40 | |
| ralsina | nessita: cool, np | 20:41 |
| * nessita -> will be back | 20:41 | |
| dobey | ralsina: stop confusing qt and pyqt4 :) | 20:41 |
| ralsina | dobey: haha | 20:42 |
| balloons | afternoon everyone.. I wondering if anyone can tell me how to logout of ubuntu one (the deskop client).. I'm running the up to date version in precise and I can't seem to switch to a different account | 20:42 |
| ralsina | ok, there is on one hand, qt4 dbus support, which provides a libQtDbus (modulo operating system naming) | 20:43 |
| dobey | balloons: you relaly shouldn't try to use 2 different accounts, as the same user on the system | 20:43 |
| dobey | really even | 20:44 |
| ralsina | OTOH, there is a dbus mainloop that comes from PyQt | 20:44 |
| dobey | ralsina: yes. | 20:44 |
| ralsina | urbanape: you may have to edit PyQt's configure.py | 20:46 |
| ralsina | So, dbus->Qt->python-dbus->PyQt(with hacked configure.py) | 20:47 |
| urbanape | that seems straigtforwardly messy. Like uncooked spaghetti. | 20:48 |
| ralsina | urbanape: good news are, we can do a tarball of the whole thing and plop it somewhere else | 20:50 |
| ralsina | urbanape: so yes, just like spaghetti | 20:50 |
| urbanape | oh, yeah, we'll end up bundling the whole thing together, no doubt. | 20:50 |
| urbanape | assuming we don't find the right place to slice it and do away with all that in the end. | 20:51 |
| balloons | dobey, I'm attempting to write some testcases ala, http://www.theorangenotebook.com/2012/02/opportunity-manual-application-testing.html.. I'm *that* guy :-) I'm trying to logout in order to understand how ubuntu one works for day one.. perhaps I'm missing something | 20:51 |
| dobey | balloons: do you want to log back in with the same u1 account, or with a different u1 account? | 20:52 |
| balloons | the first run seems to install the client, and then prompt you to login via ubuntu sso.. it must store this info somewhere.. I'm guessing I can just purge and reinstall the package | 20:52 |
| balloons | login with the same.. I'm just making sure my tests all make sense | 20:52 |
| ralsina | balloons: basically, logging out and in as another user is strongly discouraged because it DOES NOT WORK ;-) | 20:52 |
| ralsina | balloons: but if you must try, you can logout by removing your current device from control panel | 20:52 |
| dobey | balloons: ok, you open the control panel, go to the "devices" tab, and remove the current device (the one with the bandwidth/etc config under it) | 20:52 |
| balloons | ahhh yea that did it | 20:53 |
| balloons | interesting.. so logging in / out isn't really recommended/encouraged etc | 20:53 |
| balloons | gotcha | 20:53 |
| dobey | balloons: it's fine as long as it's the same account you're using | 20:54 |
| dobey | balloons: but trying to use a different account, when you've already configured u1, as that local user, will cause a big bucket of problems :) | 20:54 |
| ralsina | balloons: suppose you logout as userA, change something, then login as userB. What should happen to userA's changes on disk? Deleted? Uploaded to userB's account? Error? | 20:54 |
| dobey | and won't work | 20:54 |
| balloons | I get it.. ubuntu one is a bit more than just a cloud folder | 20:55 |
| balloons | ok, that worked fine.. So, any other basic tests you can think of to include? I've basically just got a test to install, login and add a file | 20:55 |
| dobey | it's not a remote filesystem. it's a file synchronization service. | 20:55 |
| dobey | balloons: talk to elopio and rmcbride about u1 client testing of that sort | 20:56 |
| balloons | dobey, ok... are they about / in channel ? :-) | 20:57 |
| balloons | if you check out that post, i have a bzr branch.. I'm happy to take merge requests. .I would love to see some from the u1 team | 20:57 |
| balloons | thanks for your help dobey .. i'll idle in here if anyone has questions | 20:58 |
| dobey | balloons: actually, we have a branch for our tests stuff, with a goal to have all the currently fairly manual bits to be automated in jenkins at some point | 21:01 |
| balloons | dobey, hmm.. are these QA tests or unit? | 21:01 |
| dobey | yes | 21:01 |
| balloons | :-0 | 21:01 |
| balloons | I'm after tests for end-users.. all manual, and focused on pure QA | 21:02 |
| ralsina | balloons: QA | 21:02 |
| dobey | the client stuff is manual tests right now. but the idea is to automate with mago and such, and run them all the time | 21:03 |
| dobey | balloons: elopio and rmcbride are our QA guys | 21:03 |
| balloons | gotcha.. guess I should go have a look @ that branch and have a chat with those guys | 21:03 |
| rmcbride | balloons: sorry, I've been working in another session. Our current test info is at http://testcases.qa.ubuntu.com/Applications/UbuntuOne/11.10?action=show&redirect=Applications%2FUbuntuOne | 21:08 |
| rmcbride | balloons: for the manual stuff | 21:08 |
| rmcbride | balloons: the automation efforts focuss around getting those to run in a jenkins instance using various test tools. Mainly we're just automating the web page tests right now, though elopio is engaged in automating the client tests. | 21:09 |
| rmcbride | balloons: we follow the stuff int he testcased wiki for our manual testing | 21:10 |
| nessita | ralsina: there is a leftover class instance in the ControlPanel widget... could you please remove? | 21:21 |
| dobey | i guess i'll have to file a bug | 21:51 |
| balloons | dobey, more randomness today.. what's ubuntu one doing with case conductor? | 21:56 |
| dobey | with what? | 21:57 |
| rmcbride | balloons: elopio has just started setting up an instance for us to start looking at | 21:57 |
| balloons | rmcbride, hello.. well I think we should have a chat then.. :-) we (as in the ubuntu qa community) have been looking at it for some time | 21:59 |
| elopio | balloons, dobey: we are playing with case conductor at http://www.elopio.net:8000 | 21:59 |
| elopio | oh wati | 22:00 |
| balloons | elopio, small world | 22:00 |
| elopio | *wait | 22:00 |
| * dobey has no idea what that even is :) | 22:00 | |
| balloons | caseconductor is a test case management tool ;-) | 22:00 |
| balloons | it's being built by the mozilla folks as the successor to litmus | 22:01 |
| dobey | oh | 22:01 |
| elopio | there it is now. I think... | 22:01 |
| balloons | elopio, trying ;-) | 22:01 |
| rmcbride | elopio: have you worked out the account activation? | 22:02 |
| elopio | rmcbride: nop. I just have to fill the email server information on the config, but I started filling the tests information instead. | 22:03 |
| rmcbride | elopio: ah, heh. OK | 22:04 |
| balloons | elopio, rmcbride this is great stuff.. i'm going to cc you on an email and send you some background info on our efforts with caseconductor | 22:04 |
| balloons | I think we should coordinate our efforts on this.. | 22:04 |
| rmcbride | balloons: excellent, thanks much, and agreed | 22:04 |
| elopio | balloons: I agree too. | 22:05 |
| dobey | well at least i accomplished that today | 22:05 |
| elopio | I'm going to have lunch. | 22:08 |
| elopio | balloons: I'll wait for | 22:08 |
| elopio | your email and talk to you again. | 22:08 |
| balloons | elopio, yes.. we'll setup a voice chat soon | 22:08 |
| balloons | you'll see a mail from me | 22:08 |
| balloons | this is exciting to see | 22:08 |
| balloons | I suppose I should introduce myself a bit.. I'm the qa community coordinator on jono's team | 22:09 |
| balloons | introductions always happen so backwards :-) enjoy lunch | 22:09 |
| rmcbride | balloons: I knew someone was incoming in that position. Congratulations and welcome | 22:09 |
| balloons | yea.. at some point you figured we'd bump heads eh? ;-) | 22:10 |
| === m_conley is now known as m_conley_away | ||
| dobey | later all | 22:18 |
| * nessita -> eod | 22:32 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!