| *** sophiap has joined #openstack | 00:02 | |
| *** sophiap has quit IRC | 00:20 | |
| *** ptremblett has quit IRC | 00:24 | |
| *** zheng_li has quit IRC | 00:26 | |
| *** syah has quit IRC | 00:27 | |
| vishy | I'd like to reiterate | 00:34 |
|---|---|---|
| vishy | networking is hard, lets go shopping | 00:34 |
| *** syah has joined #openstack | 00:34 | |
| vishy | is there any way to see the diff between 2 revisions on launchpad? | 00:35 |
| gholt | cw: Yeah, in theory you should be able to change the ring by a power of two and still have the data on the devices they need to be. But... You still need to move all that data to the new paths. | 00:38 |
| gholt | So, in practice, it'd be a pain and helluva impact. | 00:38 |
| zaitcev | I guess you should be able to download and then... diff -urpN -X dontdiff a/ b/ | 00:38 |
| *** gustavomzw has joined #openstack | 00:38 | |
| zaitcev | if everything else fails | 00:39 |
| *** sandywalsh has joined #openstack | 00:41 | |
| gholt | cw: Well, I change my mind. Maybe it wouldn't be that much of an impact. There aren't usually that many partitions on a device so maybe combining/splitting them wouldn't be too bad. | 00:44 |
| gholt | cw: That'd be some fun code to write and test. :) | 00:44 |
| *** gustavomzw has quit IRC | 00:45 | |
| *** jero- has quit IRC | 00:48 | |
| *** maplebed has quit IRC | 00:53 | |
| *** jc_smith has quit IRC | 00:56 | |
| *** jdmaturen has quit IRC | 00:57 | |
| *** sirp1 has quit IRC | 01:02 | |
| *** Glace has quit IRC | 01:25 | |
| *** Glace has joined #openstack | 01:32 | |
| *** dendrobates is now known as dendro-afk | 01:34 | |
| *** rsampaio has joined #openstack | 01:36 | |
| *** rdw has quit IRC | 01:45 | |
| *** jero- has joined #openstack | 01:54 | |
| *** jdmaturen has joined #openstack | 01:54 | |
| jero- | hi | 01:54 |
| notmyname | hi | 01:57 |
| *** sirp1 has joined #openstack | 01:58 | |
| jero- | why is that not your name? | 02:04 |
| *** abecc has quit IRC | 02:10 | |
| notmyname | maybe it is my name ;-) | 02:11 |
| gholt | His parents were enamored with the Liar's Paradox. | 02:12 |
| notmyname | heh | 02:13 |
| *** e1mer has quit IRC | 02:14 | |
| *** Xenith has quit IRC | 02:15 | |
| *** jdmaturen has quit IRC | 02:16 | |
| jero- | :) | 02:17 |
| *** miclorb_ has quit IRC | 02:18 | |
| mtaylor | vishy: yes | 02:19 |
| jero- | anyone here from the nova storage layer ? I'm trying to draw a map of the IO path between guest and storage + the path setup | 02:25 |
| *** jamiew_moviez has quit IRC | 02:26 | |
| *** rsampaio has quit IRC | 02:33 | |
| *** mordred_ has joined #openstack | 02:35 | |
| *** ChanServ sets mode: +v mordred_ | 02:35 | |
| *** mtaylor has quit IRC | 02:37 | |
| *** mordred_ is now known as mtaylor | 02:39 | |
| *** e1mer has joined #openstack | 02:48 | |
| *** almaisan-away is now known as al-maisan | 02:51 | |
| *** RobertLJ has joined #openstack | 02:52 | |
| *** e1mer has quit IRC | 02:53 | |
| *** rajijoom has joined #openstack | 02:55 | |
| *** al-maisan is now known as almaisan-away | 03:02 | |
| *** novas0x2a has joined #openstack | 03:02 | |
| *** sophiap has joined #openstack | 03:05 | |
| *** RobertLJ has quit IRC | 03:08 | |
| *** sophiap has quit IRC | 03:11 | |
| *** e1mer has joined #openstack | 03:20 | |
| *** sophiap has joined #openstack | 03:20 | |
| *** miclorb has joined #openstack | 03:40 | |
| *** mordred_ has joined #openstack | 03:42 | |
| *** ChanServ sets mode: +v mordred_ | 03:42 | |
| *** jero-- has joined #openstack | 03:45 | |
| *** mtaylor has quit IRC | 03:46 | |
| *** jero- has quit IRC | 03:46 | |
| cw | gholt: i'm going to add it to my TODO list to model at some point | 03:48 |
| cw | gholt: i think it's potentially something we might encounter here | 03:48 |
| *** pag has joined #openstack | 04:01 | |
| *** pag has quit IRC | 04:03 | |
| *** sandywalsh has quit IRC | 04:08 | |
| *** Phacops has quit IRC | 04:12 | |
| *** Phacops has joined #openstack | 04:17 | |
| *** justinsheehy has quit IRC | 04:18 | |
| *** justinsheehy has joined #openstack | 04:24 | |
| *** miclorb has quit IRC | 04:30 | |
| *** miclorb has joined #openstack | 04:36 | |
| *** sirp1 has quit IRC | 04:41 | |
| *** kashyapc has joined #openstack | 04:42 | |
| *** zaitcev has quit IRC | 04:50 | |
| *** jdmaturen has joined #openstack | 05:09 | |
| *** jdmaturen has quit IRC | 05:10 | |
| *** silassewell has quit IRC | 05:19 | |
| *** almaisan-away is now known as al-maisan | 05:22 | |
| *** al-maisan has left #openstack | 05:23 | |
| *** guigui has joined #openstack | 05:28 | |
| *** Xenith has joined #openstack | 05:34 | |
| *** rdw has joined #openstack | 05:40 | |
| *** jsgotangco has joined #openstack | 05:49 | |
| *** lmcdowell has quit IRC | 06:26 | |
| *** lmcdowell has joined #openstack | 06:32 | |
| *** ibarrera has joined #openstack | 06:33 | |
| *** justinsheehy has quit IRC | 06:41 | |
| *** allsystemsarego has joined #openstack | 06:47 | |
| *** justinsheehy has joined #openstack | 06:47 | |
| *** brd_from_italy has joined #openstack | 06:48 | |
| *** jsgotangco has quit IRC | 06:49 | |
| *** perestrelka has quit IRC | 06:49 | |
| *** guigui has quit IRC | 06:59 | |
| *** e-vad has quit IRC | 07:03 | |
| *** e-vad has joined #openstack | 07:11 | |
| *** fsoyer has left #openstack | 07:20 | |
| *** kainam has joined #openstack | 07:24 | |
| *** kainam- has quit IRC | 07:24 | |
| *** zheng_li has joined #openstack | 07:45 | |
| *** calavera has joined #openstack | 08:03 | |
| *** miclorb has quit IRC | 08:08 | |
| *** Xenith has quit IRC | 08:09 | |
| *** jpopi has joined #openstack | 08:13 | |
| *** zheng_li has quit IRC | 08:17 | |
| *** brd_from_italy has quit IRC | 08:28 | |
| *** gasbakid has joined #openstack | 08:35 | |
| *** zheng_li has joined #openstack | 08:35 | |
| *** miclorb has joined #openstack | 08:42 | |
| soren | termie: I addressed your concerns in https://code.launchpad.net/~soren/nova/derootification/+merge/32888 | 08:45 |
| *** comstud has quit IRC | 08:46 | |
| *** ChrisAM has quit IRC | 08:46 | |
| termie | soren: approvizzled | 08:48 |
| *** ChrisAM has joined #openstack | 08:49 | |
| *** ChrisAM has quit IRC | 08:54 | |
| *** ChrisAM has joined #openstack | 08:56 | |
| *** ChrisAM has quit IRC | 09:03 | |
| *** sophiap has quit IRC | 09:04 | |
| *** ChrisAM has joined #openstack | 09:07 | |
| soren | termie: \o/ | 09:14 |
| *** perestrelka has joined #openstack | 09:14 | |
| *** rsampaio has joined #openstack | 09:19 | |
| *** miclorb has quit IRC | 09:20 | |
| *** novas0x2a has quit IRC | 09:27 | |
| *** novas0x2a has joined #openstack | 09:28 | |
| *** rsampaio has quit IRC | 09:53 | |
| *** rajijoom has quit IRC | 09:56 | |
| *** miclorb has joined #openstack | 09:57 | |
| *** rnewson has joined #openstack | 10:34 | |
| *** justinsheehy has quit IRC | 10:44 | |
| *** avsm has joined #openstack | 10:45 | |
| *** ibarrera has quit IRC | 10:45 | |
| *** calavera has quit IRC | 10:46 | |
| *** justinsheehy has joined #openstack | 10:56 | |
| *** calavera has joined #openstack | 10:59 | |
| *** ibarrera has joined #openstack | 11:01 | |
| *** Kami__ has joined #openstack | 11:03 | |
| *** Kami_ has quit IRC | 11:03 | |
| *** justinsheehy has quit IRC | 11:08 | |
| *** miclorb has quit IRC | 11:19 | |
| *** sandywalsh has joined #openstack | 11:29 | |
| *** ctennis has quit IRC | 11:36 | |
| *** miclorb_ has joined #openstack | 11:45 | |
| *** jero-- has quit IRC | 11:46 | |
| *** ctennis has joined #openstack | 11:54 | |
| *** justinsheehy has joined #openstack | 11:58 | |
| *** Podilarius has left #openstack | 11:59 | |
| *** londo__ has joined #openstack | 12:03 | |
| *** joschi has joined #openstack | 12:05 | |
| *** londo_ has quit IRC | 12:07 | |
| *** gundlach has joined #openstack | 12:11 | |
| gundlach | morning | 12:11 |
| *** chmouel has quit IRC | 12:15 | |
| *** avsm has quit IRC | 12:16 | |
| *** ambo_ has joined #openstack | 12:22 | |
| soren | gundlach: Maybe your question in revision 244 of https://code.launchpad.net/~hudson-openstack/nova/trunk holds its own answer? | 12:25 |
| * gundlach scrambles to figure out his question | 12:25 | |
| soren | gundlach: Look at r244 on that page, and you'll see. | 12:26 |
| gundlach | ah, yes! thanks! | 12:26 |
| gundlach | ok, next question -- why not pull my description of change into the commit -- or the other way around? | 12:26 |
| soren | gundlach: That's what we're going to do from now on. | 12:26 |
| soren | I'm not sure I like that, though. | 12:27 |
| soren | gundlach: For two reasons: | 12:27 |
| *** abecc has joined #openstack | 12:27 | |
| *** kashyapc has quit IRC | 12:27 | |
| soren | gundlach: a) What you put in the description may no longer be true once the branch has gone through review and maybe has been changed. | 12:27 |
| soren | gundlach: b) I consider them quite different. What I want in the commit log simply isn't necessarily the same as what I want to tell people who intend to review it. | 12:29 |
| soren | b) is a matter of taste more than anything, though. I feel a) applies globally. | 12:29 |
| gundlach | soren: i see on both points. thanks. | 12:31 |
| soren | gundlach: sure thing. | 12:31 |
| gundlach | if we ever did decide to go back to having two messages, perhaps fixing the UI would help -- so that newbies to LP like me would understand the context in which the message was to be read. | 12:31 |
| *** miclorb_ has quit IRC | 12:35 | |
| soren | Yeah, there's clearly room for improvement there. | 12:37 |
| *** lmcdowell has quit IRC | 12:38 | |
| soren | gundlach: Do you think something simple like "Enter a message for the reviewer here" and "Enter the desired commit message here" would have done the trick? | 12:42 |
| gundlach | soren: Probably not, because people coming from another tool would think "OK, my commit message *is* my message for the reviewer" | 12:43 |
| gundlach | maybe more like: | 12:43 |
| gundlach | "Enter your commit message" | 12:43 |
| gundlach | and | 12:43 |
| gundlach | "If you want to leave a special note for the reviewer, do so here. It will only be used during the merge process". | 12:44 |
| gundlach | (or wherever it's used; i already forget because i'm having trouble keeping them straight :) ) | 12:44 |
| soren | bug 619810 | 12:45 |
| uvirtbot | Launchpad bug 619810 in launchpad "Difference between description and commit message in merge proposal is unclear" [Undecided,New] https://launchpad.net/bugs/619810 | 12:45 |
| gundlach | but really, i think more people will be happy with a simpler UI that uses a single message. LP is busy enough that collating ideas is probably a good thing :) | 12:45 |
| *** lmcdowell has joined #openstack | 12:45 | |
| *** jdarcy has joined #openstack | 12:53 | |
| *** pvo has joined #openstack | 12:56 | |
| *** ChanServ sets mode: +v pvo | 12:56 | |
| *** ambo_ has quit IRC | 12:57 | |
| *** gustavomzw has joined #openstack | 12:57 | |
| *** lmcdowell has quit IRC | 13:03 | |
| *** lmcdowell has joined #openstack | 13:09 | |
| *** ambo_ has joined #openstack | 13:11 | |
| *** pvo has quit IRC | 13:14 | |
| *** ratasxy has joined #openstack | 13:23 | |
| *** gasbakid has quit IRC | 13:23 | |
| ratasxy | hello, when it leaves the final version of openstack? | 13:25 |
| gundlach | ratasxy: around october is the first release. | 13:26 |
| gundlach | it will not be the "final" version -- we will keep releasing new versions as we improve it. | 13:26 |
| ratasxy | thanks gundlach | 13:26 |
| *** gasbakid has joined #openstack | 13:26 | |
| ratasxy | gundlach, exist a beta? | 13:27 |
| *** f4m8 has joined #openstack | 13:27 | |
| gundlach | ratasxy: you can get the current code at launchpad.net/openstack. | 13:28 |
| ratasxy | thanks | 13:28 |
| gundlach | there are two parts: "nova", which manages virtualized machines (like EC2), and "swift", which handles storage (like S3) | 13:28 |
| *** Fido has joined #openstack | 13:29 | |
| zul | did the derootification make it it in today? | 13:31 |
| *** guigui has joined #openstack | 13:35 | |
| *** pvo has joined #openstack | 13:39 | |
| *** ChanServ sets mode: +v pvo | 13:39 | |
| *** Fido has quit IRC | 13:40 | |
| *** rajijoom has joined #openstack | 13:46 | |
| *** sophiap has joined #openstack | 13:49 | |
| jero | what does the derootification imply ? | 13:50 |
| *** comstud has joined #openstack | 13:53 | |
| *** ChanServ sets mode: +v comstud | 13:53 | |
| soren | zul: Still needs one more positive review. | 13:55 |
| soren | jero: Currently, everything runs as root. | 13:55 |
| soren | jero: Well, that's not entirely true. | 13:55 |
| soren | jero: they run as the user who starts them. | 13:55 |
| soren | jero: ...but if you run from init scripts, that's root. | 13:55 |
| soren | jero: The derootification branch lets the daemons start as root and drop privileges to a regular user. | 13:56 |
| soren | ...and for the looks for it, it needs more work. Darn it. | 13:57 |
| * soren is having a complete preposition fail today | 13:59 | |
| jero | thanks | 14:00 |
| soren | "/by/ the looks /of/ it"... | 14:00 |
| jero | I thought it was related to some tree-based architecture | 14:00 |
| soren | jero: Oh, no, nothing like that. | 14:00 |
| jero | soren: is that for swift ? | 14:00 |
| soren | jero: No, nova. | 14:00 |
| jero | ok! | 14:00 |
| jero | i definately need a coffee cup. | 14:01 |
| *** kashyapc has joined #openstack | 14:07 | |
| *** Podilarius has joined #openstack | 14:11 | |
| *** hazmat has joined #openstack | 14:11 | |
| *** ppetraki has joined #openstack | 14:25 | |
| *** pcrews has joined #openstack | 14:28 | |
| *** mordred_ is now known as mtaylor | 14:37 | |
| *** sirp1 has joined #openstack | 14:42 | |
| zul | how do you create project network data? http://pastebin.ubuntu.com/479955/ | 14:43 |
| *** ibarrera has quit IRC | 14:43 | |
| *** f4m8 is now known as f4m8_ | 14:46 | |
| *** justinsheehy has quit IRC | 14:49 | |
| *** pcrews is now known as pcrews_bbl | 14:55 | |
| *** tr3buchet has joined #openstack | 14:56 | |
| *** aliguori has joined #openstack | 14:57 | |
| soren | zul: You create a project using "nova-manage project create" | 14:59 |
| soren | zul: Whether that's what it refers to, I'm not sure. | 14:59 |
| *** calavera has quit IRC | 15:07 | |
| *** kashyapc_ has joined #openstack | 15:11 | |
| *** kashyapc has quit IRC | 15:12 | |
| *** asksol has quit IRC | 15:14 | |
| uvirtbot | New bug: #619901 in swift "add debian package dependencies" [Undecided,New] https://launchpad.net/bugs/619901 | 15:15 |
| eday | gundlach: so, does disable-msg not work for you? I'm using latest pylint that ships with lucid and disable= doesn't work for me, which is why I had to switch back | 15:22 |
| gundlach | hmm; i did pip install pylint into my venv and disable-msg didn't work wit 0.21.1 | 15:22 |
| eday | hm, yeah I'm on 0.19 | 15:23 |
| jpipes | eday: me too | 15:23 |
| *** mrayzenoss has joined #openstack | 15:23 | |
| *** mrayzenoss is now known as mray | 15:23 | |
| *** mray has joined #openstack | 15:23 | |
| eday | thats strange they just broke disable-msg, you'd think it would just deprecate it | 15:23 |
| jpipes | eday: but I think you can re-enable disable-msg with some command... | 15:23 |
| gundlach | i'd be fine changing pip-requires to pylint==0.19 until Ubuntu M ships, and discarding my branch | 15:24 |
| gundlach | eday: i think it was deprecated for a while? | 15:24 |
| * jpipes admits there is little documentation on the pylintrc file, though! | 15:24 | |
| gundlach | (was having to read up on it to figure out what i was doing wrong) | 15:24 |
| jpipes | gundlach: yeah, there's not much reading though ;) | 15:24 |
| gundlach | jpipes: maybe i went into the source? can't remember. also, i like your new handle :) | 15:25 |
| eday | jpipes: so, there is a way to do renable disable-msg in 0.21? that might be good for now to transition | 15:25 |
| jpipes | gundlach: heh, didn't even notice...must have had the Internets kick out last night... | 15:28 |
| jpipes | eday: I believe there is, but will have to check the sources...I'd recommend sticking with 0.19 for right now...we can do a mass sweep replace for s/disable-msg/disable/ in a single patch in the future | 15:29 |
| jpipes | gundlach: ^^ | 15:29 |
| jpipes | eday: which would make it nicer for folks merging (see ewan's latest merge conflict stuff from style/pep fixes) | 15:29 |
| gundlach | jpipes: yeah, i did sed -i s/disable-msg/disable/g in my patch. | 15:30 |
| gundlach | jpipes: ok i'll update my patch to change us to 0.19 and change disable to disable-msg throughout. | 15:30 |
| jpipes | gundlach: yeah, I say we table that until Ubuntu M at least, and add 0.19 to the pip-requires. | 15:30 |
| jpipes | gundlach: cheers | 15:30 |
| eday | jpipes, gundlach: cool, sounds good | 15:31 |
| gundlach | jpipes: where are you from? i don't here 'cheers' except from the irish | 15:31 |
| jpipes | gundlach: I'm British | 15:31 |
| jpipes | gundlach: but I've spent most of my life in the U.S. | 15:31 |
| gundlach | jpipes: ah, didn't know it crossed the channel. ok then. | 15:31 |
| jpipes | gundlach: :) | 15:31 |
| eday | gundlach: jpipes is a true international man :) | 15:31 |
| * gundlach lived in ireland for a year, didn't get any british culture | 15:31 | |
| gundlach | eday: ... of mystery | 15:31 |
| eday | gundlach: some would say | 15:32 |
| jpipes | gundlach: heh, I've always wanted to go to Ireland...never been. | 15:32 |
| gundlach | jpipes: don't bother, it's like britain. ;) | 15:32 |
| jpipes | gundlach: lol | 15:32 |
| gundlach | (in terms of weather == rainy and dreary) | 15:32 |
| jpipes | gundlach: my favorite kind of weather :) | 15:32 |
| * jpipes was miserable in Austin... | 15:33 | |
| jpipes | as dendro-afk and eday can attest to | 15:33 |
| * jpipes likes his weather like his beer...cold and wet. | 15:34 | |
| * gundlach grew up in the south, on a lake, so the hotter the better | 15:34 | |
| jpipes | heh | 15:34 |
| justinsb | Actually, it's reasonably nice in London at the moment - warmer than San Francisco | 15:34 |
| gundlach | jpipes: and unceasing | 15:34 |
| jpipes | gundlach: well, that too ;) | 15:34 |
| jpipes | justinsb: yeah? I'll have to go visit my bro then :) | 15:34 |
| jpipes | if I could put up with a 3.5 yr old for more than a day r two... | 15:34 |
| justinsb | jpipes: It's a good time to escape San Fran - not least because it's about to shut down for burning man. I took the opportunity to visit my family! | 15:34 |
| jpipes | justinsb: nice! | 15:35 |
| justinsb | jpipes: Your brother is 3? :-) | 15:35 |
| comstud | haha | 15:35 |
| jpipes | justinsb: lol | 15:35 |
| comstud | where is summer? | 15:35 |
| comstud | it's been damned cold in the easy bay | 15:35 |
| comstud | :( | 15:35 |
| jpipes | justinsb: actually, my wife and I are going to the 2012 games in London...very excited. | 15:35 |
| mtaylor | jpipes: re: pylint==0.19 ... do we need to upload a new pylint package to the ppa? | 15:35 |
| jpipes | mtaylor: hmm, shouldn't need to.. | 15:35 |
| gundlach | comstud: here | 15:35 |
| gundlach | Weather info for northwest woods, Watkinsville, Georgia (updated on 11:36 AM EDT on August 18, 2010); Temperature: 86.9 F / 30.5 C; Humidity: 76%; Dew Point: 78 F / 26 C; Wind: SE at 19.2 mph / 30.9 km/h; Pressure: 30.09 in / 1018.8 hPa (Rising); Conditions: Clear; Visibility: 7.0 miles / 11.3 kilometers; UV: 7 out of 16; Clouds: Clear (CLR) : -; Yesterday's Maximum: 89 F / 31 C; Yesterday's Minimum: 76 F / 24 C; Sunrise: 6:57 AM | 15:35 |
| gundlach | EDT; Sunset: 8:16 PM | 15:35 |
| jpipes | gundlach: poop. | 15:36 |
| mtaylor | jpipes: ok. didn't _really_ read the traceback... was just scanning | 15:36 |
| jpipes | mtaylor: no prob! | 15:36 |
| justinsb | jpipes: You're a brave man... the London travel situation is the same as ever | 15:36 |
| * mtaylor hates flying in to London even more than flying inside the US | 15:36 | |
| jpipes | justinsb: oh, I know :) but we can stay with my brother, who is in Birkhamstead... | 15:36 |
| comstud | gundlach, nice minimum | 15:36 |
| comstud | our highs are in the 80s, but lows in the 50s | 15:36 |
| mtaylor | it's the only place where the security situation is _more_ silly than ours | 15:36 |
| gundlach | jpipes: re ppa -- do we currently have 0.19 in the ppa or 0.21.1? | 15:37 |
| jpipes | mtaylor: ++ | 15:37 |
| mtaylor | gundlach: I don't think we have pylint in the ppa at all | 15:37 |
| jpipes | yeah, we don't | 15:37 |
| *** ratasxy has quit IRC | 15:37 | |
| gundlach | mtaylor: i don't get ppas, but isn't it supposed to have all of its dependencies? or does it just list them for a package manager to fetch? | 15:37 |
| gundlach | s/have/contain/ | 15:37 |
| mtaylor | gundlach: it's just an apt repo you can add... | 15:38 |
| mtaylor | gundlach: the debian package itself lists the depends... but if one of the depends is something that the normal archive can't supply, then we'll add it to the ppa as well as our packages | 15:38 |
| jpipes | gundlach: they're excellent to add packages that your software has specific dependencies on... | 15:38 |
| mtaylor | gundlach: in a perfect world, the ppa would only have our packages | 15:38 |
| gundlach | mtaylor: oh so you're saying we don't have pylint in the repo as its own project. | 15:38 |
| gundlach | mtaylor: got it, i think | 15:38 |
| jpipes | gundlach: no, it's not in the PPA | 15:38 |
| *** dendro-afk is now known as dendrobates | 15:42 | |
| *** avsm has joined #openstack | 15:42 | |
| gundlach | jpipes: just pushed to lp:~gundlach/nova/pylint-version-update if you don't mind reviewing | 15:44 |
| eday | gundlach: do you think we need really need the rackspace/controllers subdirectory? WHat if it were just api/rackspace/images.py (with a Controller class in there). controllers in the path seems a bit redundant | 15:44 |
| jpipes | gundlach: will do | 15:44 |
| gundlach | eday: ask cerberus, he set it up and i merged it in. i'm fine with fewer levels but didn't care enough to change it. | 15:45 |
| gundlach | jpipes: tyvm | 15:45 |
| *** ratasxy has joined #openstack | 15:46 | |
| gundlach | eday: i'd be fine with rackspace.py containing ImageController, ServerController, etc -- Python isn't strict on one class per file | 15:46 |
| eday | _cerberus_: hey! you around? | 15:48 |
| jpipes | gundlach: f=open(pthfile, 'w') | 15:48 |
| jpipes | gundlach: space before and after ;) you fixed the line above it, just not that one :) | 15:48 |
| gundlach | jpipes: fixed in another branch? | 15:48 |
| gundlach | aw, really? smack | 15:48 |
| jpipes | s/before and after/before and after =/ | 15:49 |
| *** JimCurry has joined #openstack | 15:49 | |
| jpipes | gundlach: hehe, I do it all the time :) | 15:49 |
| *** lew has joined #openstack | 15:52 | |
| gundlach | bzr question: | 15:53 |
| gundlach | when i "bzr clone lp:~gundlach/foo foo", why can't I then "bzr push" and have it push to lp:~gundlach/foo ? Am I doing something wrong that I'm required to respecify the obvious default push target? | 15:53 |
| jpipes | gundlach: bzr branch you mean? :) | 15:54 |
| gundlach | jpipes: oh, is that what i'm doing wrong | 15:54 |
| gundlach | what's clone vs branch vs pull? i've tried to read this on the internet and haven't found a clear answer -- probably not looking in the right place | 15:55 |
| jpipes | gundlach: no... you can't push and overwrite a source branch unless you deliberately tell it to (use bzr push --overwrite) | 15:55 |
| gundlach | i'm not trying to overwrite a source branch... i think... i'm trying to merge my changes into a remote repo | 15:56 |
| jpipes | gundlach: bzr branch creates a new branch from another one. bzr pull will pull in any revisions from the source branch that have occurred since branching. | 15:56 |
| jpipes | gundlach: what's the source branch and what's the target branch? local to Launchpad, or local to local? | 15:57 |
| gundlach | so if i need to make a simple fix to a branch on LP, what are the steps i take? i don't want to "bzr branch" because i'm just trying to commit a rev to the current branch on LP. | 15:57 |
| eday | gundlach: I think to answer the branch/push Q, bzr uses different vars for the source and dest (branch and push defaults). SO one doesn't set the other. I think :) | 15:58 |
| jpipes | gundlach: you are thinking that the branch on LP that you branched from is the same as your local...they aren't. they're different. | 15:58 |
| gundlach | jpipes: specifically, i was trying to make the pep8 fix you brought up. I have lp:~gundlach/nova/install-tools-pep8-fixes, and i had deleted my local branch earlier | 15:58 |
| gundlach | jpipes: ah. | 15:58 |
| jpipes | gundlach: if you deleted your local branch, recreate it (bzr branch lp...) | 15:58 |
| jpipes | gundlach: then bzr push --overwrite | 15:58 |
| gundlach | jpipes: i did clone, which is different in what way? | 15:59 |
| jpipes | gundlach: they are the same (right now) | 15:59 |
| comstud | i think clone is just a synonym for branch in bzr | 15:59 |
| eday | jpipes: well, if he's just adding new revs, he won't need --overwrite, correct? Thats only if you uncommit or something | 15:59 |
| gundlach | i did "bzr clone lp:~gundlach/nova/...pep8-fixes; hackety hack; bzr commit -m "typo"; bzr push" | 15:59 |
| gundlach | eday: erm, i actually did do a bzr revert on my local copy, if that counts as an uncommit | 15:59 |
| jpipes | eday: he's not checking out, he's branching...so trying to push to an existing LP location will give an error IIRC | 16:00 |
| jpipes | gundlach: no, that's not an uncommit since it doesn't change anything that bzr cares about | 16:00 |
| eday | jpipes: but if it doesn't change the base revs at all, should be fine | 16:00 |
| comstud | i usually just branch from remote to local | 16:00 |
| comstud | edit local | 16:00 |
| comstud | commit | 16:00 |
| jpipes | eday: ok, you can try it :) | 16:00 |
| comstud | then push to remote name | 16:00 |
| jpipes | comstud: yup. | 16:00 |
| comstud | no need for --overwrite | 16:01 |
| *** pvo_ has joined #openstack | 16:01 | |
| *** pvo_ has joined #openstack | 16:01 | |
| *** ChanServ sets mode: +v pvo_ | 16:01 | |
| gundlach | comstud: so you have to specify 'bzr push lp:~comstud/path/to/remote/branch' ? | 16:01 |
| comstud | unless a new commit hit remote after you branched | 16:01 |
| jpipes | gundlach: only once | 16:01 |
| comstud | in which case you have to figure out how to pull that in | 16:01 |
| comstud | gundlach- yeah, on the first push only | 16:01 |
| comstud | after that, it remembers | 16:01 |
| comstud | and you can just do bzr push | 16:01 |
| gundlach | ok, fine enough. it "felt" like it should have inferred that from the fact that i cloned from there, but i can see how if how 'clone'=='branch' then i haven't really specified a push target for my new branch. | 16:02 |
| jpipes | gundlach: it remembers the push location. if you wish to push to another branch (after already pushing to another) and have bzr remember the new location, use bzr push --remember URL | 16:02 |
| gundlach | jpipes: thanks, i've needed that as well in my few days using bzr | 16:02 |
| gundlach | ok, i think i'm straight; back to getting my pep8 branch re-reviewed :) | 16:02 |
| jpipes | gundlach: yeah...it's a good one to remember. pun intended. ;) | 16:02 |
| gundlach | (thanks, all) | 16:03 |
| jpipes | np | 16:03 |
| jpipes | gundlach: looks like dendrobates had too quick an approval finger ;) | 16:04 |
| gundlach | yep, after i had already pushed my typo fix, so we're set :) | 16:04 |
| *** pvo has quit IRC | 16:04 | |
| *** pvo_ is now known as pvo | 16:04 | |
| jpipes | mtaylor: https://code.launchpad.net/~gundlach/nova/install-tools-pep8-fix/+merge/33002 | 16:05 |
| comstud | jay, you wanted me to review ewan's branch? | 16:05 |
| jpipes | mtaylor: I've seen a number of these things pop up...I'm wondering if the test suite is deterministic enough... :( | 16:05 |
| jpipes | comstud: pls do! | 16:05 |
| comstud | ok, i'll try to get to it in a bit | 16:05 |
| jpipes | comstud: open to all | 16:05 |
| comstud | conf call right now | 16:05 |
| comstud | ok | 16:05 |
| comstud | i see | 16:05 |
| gundlach | comstud: fyi i also thought he asked me, specifically, to review it :) the emails being sent straight to us, rather than to a list, is confusing | 16:06 |
| mtaylor | jpipes: yes. I have too... I think those should be investigated... flaky tests are worse than no tests | 16:06 |
| jpipes | mtaylor: ++ | 16:06 |
| *** pcrews_bbl is now known as pcrews | 16:06 | |
| comstud | gundlach, yeah :) | 16:06 |
| jpipes | gundlach, comstud: it's just LP's "personal touch" :P | 16:07 |
| comstud | i felt special. | 16:07 |
| comstud | now I don't. | 16:07 |
| comstud | :( | 16:07 |
| * mtaylor goes to file bug | 16:08 | |
| jpipes | hehe | 16:08 |
| gundlach | Launchpad bug 45823: comstud doesn't feel special. | 16:09 |
| uvirtbot | Launchpad bug 45823 in xfonts-core "ISO8859-15 fonts are very small (dup-of: 39560)" [Medium,New] https://launchpad.net/bugs/45823 | 16:09 |
| uvirtbot | Launchpad bug 39560 in xfonts-core "The xfonts-*-transcoded packages are missing from Breezy onwards" [Medium,Fix released] https://launchpad.net/bugs/39560 | 16:09 |
| gundlach | thanks, uvirtbot | 16:09 |
| comstud | haha | 16:09 |
| *** jdarcy has quit IRC | 16:10 | |
| *** sophiap has quit IRC | 16:10 | |
| gundlach | eday: thx for the approve on pylint-version-update, but i missed the pylintrc in my sed sweep. is there some more appropriate way than IRC to inform the reviewer that the review and a branch update missed each other like ships in the night? | 16:11 |
| gundlach | ps: your review missed my branch update, like a ship in the night. | 16:11 |
| mtaylor | Launchpad bug 619954 | 16:12 |
| uvirtbot | Launchpad bug 619954 in launchpad "merge request review request of a team should indicate such" [Undecided,New] https://launchpad.net/bugs/619954 | 16:12 |
| mtaylor | comstud: ^^ | 16:12 |
| mtaylor | :) | 16:12 |
| comstud | thnx:) | 16:12 |
| comstud | i don't want to feel special when I shouldn't. | 16:12 |
| mtaylor | I agree | 16:14 |
| *** sophiap has joined #openstack | 16:14 | |
| mtaylor | I had the same reaction to that email this morning... | 16:14 |
| alekibango | everyone is so special today... | 16:15 |
| alekibango | :D | 16:16 |
| *** guigui has quit IRC | 16:16 | |
| gundlach | mtaylor: while we're on the topic: is it a separate bug that it's hard for me to automatically filter merge proposals into a gmail label? The only way i can detect them is that the reply-to is set to "mp+<some_number>@code.launchpad.net" -- so i have to make my filter be (reply-to:mp AND reply-to:code.launchpad.net). Kinda clunky. | 16:16 |
| mtaylor | gundlach: yup. lemme file that one too | 16:18 |
| comstud | can't filtering on ^Subject: \[Merge\] ? | 16:18 |
| comstud | filter | 16:18 |
| comstud | i suppose that hits more than you want | 16:18 |
| gundlach | comstud: i'm in gmail, so i don't think i can do "starts-with" filtering; and yeah, false positives | 16:18 |
| mtaylor | gundlach: does google support filtering on arbitrary headers now? | 16:18 |
| *** chmouel has joined #openstack | 16:18 | |
| gundlach | mtaylor: not aware, sorry | 16:18 |
| comstud | they come to my gmail too, tho i'm pulling down my gmail into mail.app | 16:19 |
| comstud | hehe | 16:19 |
| * comstud pays attention to this call he's on | 16:19 | |
| gundlach | sure he does | 16:19 |
| alekibango | for me, google is not trustable. they ARE evil. | 16:20 |
| alekibango | trustworthy* | 16:20 |
| * gundlach goes ahead and marks his pylint branch as Approved | 16:21 | |
| eday | gundlach: I got the pylintrc change in my branch already | 16:27 |
| gundlach | eday: cool; i imagine bzr won't complain when you merge into trunk if you made the same changes i did, right? | 16:28 |
| eday | gundlach: it will reject with a conflict | 16:29 |
| eday | gundlach: I'll merge trunk into mine :) | 16:29 |
| gundlach | eday: if branches A and B both make the same change and try to merge into C, there's a conflict, but if B merges to trunk, then A merges B and then merges to trunk it's OK? | 16:29 |
| gundlach | s/trunk/C/g | 16:29 |
| *** maplebed has joined #openstack | 16:30 | |
| eday | gundlach: also, when a merge prop changes (like you add a new rev) you can set the status from needs review to work-in-progress and then back to needs review, it will trigger notifications. can also just add a comment asking folks to review new rev (your Q about how to catch updates) | 16:30 |
| eday | gundlach: correct | 16:30 |
| gundlach | eday: ah, ok, thanks | 16:30 |
| eday | gundlach: so, I can either merge trunk into mine once it lands, or just merge yours directly | 16:31 |
| eday | every rev has a unique ID on it, so it knows what is/isn't already applied | 16:31 |
| gundlach | eday: i feel like in svn, if two branches are merged into trunk that both do the *same* thing to a given line, the second merge will go without a hitch. am i wrong? and if i'm not, why can't bzr do this? | 16:32 |
| gundlach | (sorry that i probably don't grok dvcs enough to understand why my question is dumb :) ) | 16:32 |
| eday | gundlach: I don't think I've tried for *exact* changes before, would need to test. I think it would throw a conflict though | 16:34 |
| *** sparky_ has joined #openstack | 16:34 | |
| eday | gundlach: one thing (which I have fixed in my branch) is that pylintrc disable-msg=... needs to be one line (comma separated). multiple lines keep overwriting, so only the last sticks | 16:35 |
| gundlach | eday: ah hah! my mistake. | 16:36 |
| gundlach | looks like mine's already merged, so shall i just hold off and let your merge fix that? | 16:36 |
| eday | gundlach: yeah, I already have it fixed | 16:38 |
| gundlach | k | 16:38 |
| *** rajijoom has quit IRC | 16:40 | |
| *** jdmaturen has joined #openstack | 16:46 | |
| *** jdmaturen has joined #openstack | 16:46 | |
| *** zheng_li has quit IRC | 16:49 | |
| JimCurry | happy wednesday openstack! | 16:51 |
| eday | JimCurry: g'morning :) | 16:52 |
| *** pvo has quit IRC | 16:53 | |
| eday | JimCurry: when is it you are coming back to portland? | 16:53 |
| JimCurry | next week! working on a babysitter so we can go to dinner. no candidates so far though…. | 16:54 |
| eday | My wife and I were trying to think of someone we know, couldn't come up with anyone who's around/doesn't already have kids :) | 16:56 |
| *** jamiew has joined #openstack | 16:56 | |
| *** pvo has joined #openstack | 16:59 | |
| *** ChanServ sets mode: +v pvo | 16:59 | |
| *** jamiew has left #openstack | 17:01 | |
| *** littleidea has joined #openstack | 17:02 | |
| *** zheng_li has joined #openstack | 17:09 | |
| jpipes | JimCurry: happy Wednesday to you :) | 17:11 |
| *** mray has quit IRC | 17:14 | |
| JimCurry | I bet Portland and Columbus are better places to be today than 105 degree Austin | 17:14 |
| *** mrayzenoss has joined #openstack | 17:21 | |
| *** mrayzenoss is now known as mray | 17:21 | |
| *** mray has joined #openstack | 17:21 | |
| *** sirp1 has quit IRC | 17:22 | |
| jero | 105! | 17:25 |
| jero | justinsb: are you around ? | 17:26 |
| justinsb | jero: Hi! | 17:27 |
| jero | hi ! | 17:27 |
| jero | justinsb: I am figuring all the i/o paths in nova, and I was wondering what are the main obstacles with the iscsi support implementation | 17:28 |
| justinsb | jero: Main obstacle is trying to figure out what it should do :-) | 17:29 |
| justinsb | jero: There's a blueprint, but that seems to be a bit of a black hole where ideas go to die | 17:29 |
| justinsb | jero: It was working when it was first written - I expect it now needs some tweaking to merge | 17:29 |
| justinsb | jero: But it's only Linux's OpeniSCSI initiator & target | 17:30 |
| justinsb | jero: And I don't think that's really the most interesting iSCSI use case | 17:30 |
| justinsb | Blueprint here: https://blueprints.launchpad.net/nova/+spec/iscsi-support | 17:31 |
| justinsb | And actual 'discussion' here: http://wiki.openstack.org/IscsiBlueprint | 17:32 |
| jero | justinsb: is it based on the same code as the aoe stuff ? | 17:32 |
| justinsb | jero: Yes. There's a flag which says whether to use iSCSI or AoE in the volume service | 17:34 |
| justinsb | jero: And then that writes different data which the compute service then uses to determine how to mount the volume | 17:34 |
| jero | justinsb: so it could potentially be extended to use fiber channel and others | 17:34 |
| justinsb | jero: I'm sure it could. However, I think we've now established a 'standard' way to do plug-in extension points, so I'd need to check whether this had been done for the volume service yet | 17:35 |
| justinsb | jero: The bulk of the code is here http://bazaar.launchpad.net/~justin-fathomdb/nova/iscsi-volumes/revision/141 and here http://bazaar.launchpad.net/~justin-fathomdb/nova/iscsi-volumes/revision/146 | 17:36 |
| *** rnewson has quit IRC | 17:36 | |
| jero | thanks justinsb ! | 17:37 |
| justinsb | jero: No problem. I just checked - it looks like the architecture of the volume service hasn't changed much, so the code should still basically apply. | 17:38 |
| jero | do you know how volumes are mapped in big cloud infras ? do they use iscsi or FC luns | 17:38 |
| justinsb | If by 'big cloud infras' you mean EC2, then I don't know. I think EBS is a home-brew system. | 17:39 |
| jero | rackspacers any hint ? | 17:39 |
| justinsb | I think we have the opportunity to learn from EC2's mistakes here ... EBS doesn't seem to offer compelling advantages over iSCSI. | 17:40 |
| jero | yeah | 17:42 |
| *** Xenith has joined #openstack | 17:53 | |
| *** kashyapc_ has quit IRC | 17:59 | |
| *** zaitcev has joined #openstack | 18:03 | |
| *** Glace has quit IRC | 18:07 | |
| cw | gholt: awake yet? | 18:09 |
| cw | wrt to swift, when adding capacity and rebalancing the ring, i assume care is taken to make sure that no partition has all three physical stores moved? | 18:12 |
| cw | in fact i assume you just do RR across each one so you orphan 1/3 of the data at most? | 18:12 |
| cw | (assuming 3 copies here) | 18:12 |
| creiht | cw: correct | 18:15 |
| creiht | No more than 1 replica of a partition will be moved | 18:16 |
| cw | one thing that occurs to me, @ 80% utilization ... a rebalance is going to cause a large amount of bi-sectional work amongst the object stores | 18:18 |
| cw | it would be good to have the communications between the stores on a different subnet to the proxy traffic | 18:18 |
| gholt | Yeah, that was a consideration here as well, though we also run against aggr port limitations and buying extra ethernet cards, etc. It's a fun balance. | 18:19 |
| gholt | To get multi-net functionality into swift as it is now should not be hard. Just a bit more dat ain the ring. | 18:19 |
| cw | well, 1G out of the nodes @ dual 10G agg isn't that bad | 18:20 |
| cw | yeah, jc was telling me it would need some ring work ... i'm just busy at a high level driking coffee and draw lies on a whiteboard | 18:20 |
| gholt | The only reason it isn't in the code already is that we didn't need it (at this point). | 18:20 |
| cw | ring sizing also requires a bit of thought | 18:20 |
| gholt | Yeah, my best guess right now is that ring sizing would require a bit of 'read-only' time. But that there might be a way to avoid that. | 18:21 |
| cw | when a spindle has a non-unity weight ... does it just take up more space in the ring? | 18:21 |
| cw | gholt: i'm just talking about picking an initial ring size ... 2^19 seems about right planning ahead somewhat | 18:22 |
| gholt | You mean if a device has a weight of 100 and another has 200? The 200 will get twice the partitions assigned to it. | 18:22 |
| cw | right | 18:22 |
| cw | i have 2TiB drives now ... but 3TiB will be out before year end | 18:22 |
| cw | that's enough of a delta it makes sense to consider it for ring sizing | 18:23 |
| gholt | Yeah, that's how that works. One easy way is to just set the weight to the number of Gs. | 18:23 |
| notmyname | one simple thing you could do is assign a weight that is equal to the number of GB on a drive | 18:23 |
| creiht | haha | 18:23 |
| cw | i did some rough math on the whiteboard and found out it would take 150 days to fill up | 18:23 |
| cw | which felt very wrong | 18:24 |
| gholt | :) | 18:24 |
| cw | i moved a decimal pt ... real math shows 14 days | 18:24 |
| gholt | I'm horrible at both kinds of math. | 18:24 |
| notmyname | two weeks to fill up what? your cluster? | 18:25 |
| cw | oh, this is 'astrophysics math' when i whiteboard, you round some up and some down and go for powers of 10 boundaries | 18:25 |
| cw | mostly it works ok | 18:25 |
| cw | notmyname: potentially yes | 18:25 |
| cw | that's assuming large objects to get good throughput ... smaller objects would take a lot longer | 18:26 |
| cw | how do you deal with disk failures? | 18:28 |
| cw | replace the spindle, remount and let swift deal? | 18:28 |
| gholt | Mostly yes. | 18:29 |
| cw | or do you explicitly mess w/ the ring? | 18:29 |
| cw | is the size was the same ... it seems you can [umount], replace disks, mkfs, remount | 18:29 |
| gholt | Ring changes really only happen if something can be fixed in a "reasonable" amount of time. | 18:29 |
| cw | and it should JustWork | 18:29 |
| gholt | s/can/can't/ | 18:29 |
| creiht | cw: btw, we've documented some of that in: http://swift.openstack.org/admin_guide.html | 18:30 |
| cw | i know i asked this before ... but do you know the dist of the EA sizes you have in use? | 18:31 |
| creiht | EA? | 18:31 |
| uvirtbot | New bug: #620027 in nova "vgcreate/lvcreate in volume/service.py fail and go undetected" [Undecided,New] https://launchpad.net/bugs/620027 | 18:31 |
| cw | extended attributes | 18:31 |
| cw | creiht: when you unmount a drive ... what stops swift from writing to the underlying fs? | 18:31 |
| cw | lack of base dir structure? | 18:32 |
| creiht | cw: permissions | 18:32 |
| gholt | check_mount calls | 18:32 |
| creiht | cw: and most stuf... yeah what gholt said | 18:32 |
| cw | check_mount is done how often? | 18:32 |
| cw | that sounds expensive ... perms are cheap | 18:32 |
| creiht | cw: Any call that tries to access the dir | 18:33 |
| gholt | Well, os.path.ismount I should say. | 18:33 |
| gholt | It's pretty cheap too. | 18:33 |
| cw | yeah, until it's all up and under load i'm not so sure i care | 18:33 |
| cw | i'll do some 'perf cpu' testing then | 18:33 |
| cw | i was going to slurp wikipedia into it a test but the data dumps are a pain to deal with | 18:34 |
| gholt | The check_mount is our function that does a bit more than ismount. swift.constraints.check_mount | 18:35 |
| *** sandywalsh has quit IRC | 18:36 | |
| gholt | Looking at Python's impl of ismount, it's just a couple lstats, so should be quite fast. | 18:39 |
| *** jc_smith has joined #openstack | 18:46 | |
| *** Xenith has quit IRC | 18:51 | |
| zaitcev | What's the equivalent of "cvs update -d" or "git pull" in bzr? | 18:55 |
| gholt | afaik bzr pull | 18:56 |
| *** sandywalsh has joined #openstack | 18:56 | |
| *** thesandywalsh has joined #openstack | 18:56 | |
| zaitcev | gholt: Thanks, that worked. | 18:56 |
| *** thesandywalsh has quit IRC | 18:57 | |
| rbergeron | zaitcev :) | 19:02 |
| zaitcev | Where are my RPMS Robyn | 19:02 |
| rbergeron | yes, i am going to fudcon - I will still let you do all the talking :) | 19:03 |
| zaitcev | I'm installing by cp -a into /usr/local | 19:03 |
| *** pvo has quit IRC | 19:03 | |
| zaitcev | feeling caveman... with Python! | 19:03 |
| rbergeron | zaitcev: come to #fedora-cloud ;) | 19:03 |
| rbergeron | zaitcev: which rpms are you speaking of specifically | 19:05 |
| uvirtbot | New bug: #620042 in swift "Add documentation for Debian Packages" [Undecided,New] https://launchpad.net/bugs/620042 | 19:06 |
| zaitcev | rbergeron: I thought you were going to package Swift and Nova for Fedora 15 at least. | 19:06 |
| *** justinsheehy has joined #openstack | 19:07 | |
| rbergeron | me personally? nooooooo.... | 19:07 |
| zaitcev | oh | 19:07 |
| rbergeron | swift is almost done - https://bugzilla.redhat.com/show_bug.cgi?id=617632 | 19:08 |
| uvirtbot | bugzilla.redhat.com bug 617632 in Package Review "Review Request: openstack-swift - OpenStack Object Storage (swift)" [Medium,Assigned] | 19:08 |
| rbergeron | Nova .... is going to be a little stickier, I think, with the dependencies | 19:08 |
| *** pvo has joined #openstack | 19:13 | |
| *** pvo has joined #openstack | 19:13 | |
| *** ChanServ sets mode: +v pvo | 19:13 | |
| *** pvo has quit IRC | 19:13 | |
| eday | _cerberus_: https://code.edge.launchpad.net/~eday/nova/api-port/+merge/32940 has the prereqs to the api-tests branch, would like to hear your thoughts before I mark to merge | 19:14 |
| *** pvo has joined #openstack | 19:18 | |
| *** pvo has joined #openstack | 19:18 | |
| *** ChanServ sets mode: +v pvo | 19:18 | |
| *** sirp1 has joined #openstack | 19:28 | |
| _cerberus_ | eday: looking | 19:29 |
| *** binaryWarrior has joined #openstack | 19:31 | |
| _cerberus_ | eday: are list comprehensions considered bad? | 19:33 |
| _cerberus_ | I see you refactored one | 19:33 |
| eday | _cerberus_: nope, but long lines are (80 chars or less for pep8) | 19:36 |
| _cerberus_ | eday: Ahh yeah, of course | 19:36 |
| _cerberus_ | I commented and reviewed as approve | 19:37 |
| eday | I guess I could have just split the comp. into two lines, but I thought this way was a bit more readable | 19:37 |
| _cerberus_ | Yes | 19:37 |
| eday | _cerberus_: awesome, thanks! | 19:37 |
| *** avsm has quit IRC | 19:39 | |
| *** justinsb has quit IRC | 19:48 | |
| *** pvo has quit IRC | 19:53 | |
| *** binaryWarrior has quit IRC | 19:59 | |
| letterj | mtaylor: there is a merge proposal in the debian branch I would like for you to look at | 20:00 |
| *** rdw has quit IRC | 20:02 | |
| *** mray has quit IRC | 20:03 | |
| *** mrayzenoss has joined #openstack | 20:03 | |
| *** mrayzenoss is now known as mray | 20:03 | |
| *** mray has joined #openstack | 20:03 | |
| vishy | ghetto_locking | 20:13 |
| *** thedot has joined #openstack | 20:16 | |
| eday | gundlach: duh, obvious way to remove the self.called stuff in tests... just check the result body :) | 20:25 |
| *** pvo has joined #openstack | 20:28 | |
| *** ChanServ sets mode: +v pvo | 20:28 | |
| gundlach | eday: i was wondering if that would suffice for you -- i thought some of them didn't return responses, or maybe you had some bigger plan in mind | 20:28 |
| gundlach | also whenever i started wondering down that path (sic) I kept getting diverted into trying to figure out a generic class that could do this testing for us, and raise an assert error if something went wrong... and it seemed too complex and i'd throw it out. probably happened 3 times while reviewing your code :) | 20:29 |
| eday | hehe | 20:29 |
| eday | I'm doing this now: environ = webob.Request.blank('/test').environ result = Router()(environ, start_response) self.assertEqual(''.join(result), "Router result") | 20:29 |
| eday | err, bad paste, insert two newlines in there :) | 20:29 |
| *** sirp1 has quit IRC | 20:30 | |
| gundlach | you could also do self.assertEqual(webob.Request.blank('/test').get_response(Router()).body, "Router result") | 20:30 |
| gundlach | broken up as needed | 20:31 |
| vishy | i have a lot of the tests passing | 20:32 |
| *** sirp1 has joined #openstack | 20:33 | |
| vishy | in my orm branch | 20:33 |
| gundlach | i made some whitespace changes in tools/install_venv.py and now volume tests are failing. are the tests broken? https://code.launchpad.net/~gundlach/nova/install-tools-pep8-fix/+merge/33002 | 20:34 |
| *** ctennis has quit IRC | 20:35 | |
| eday | gundlach: ahh, yeah, I need to read up more on all that is in webob. | 20:43 |
| *** codejunkie has quit IRC | 20:44 | |
| *** ctennis has joined #openstack | 20:47 | |
| *** londo_ has joined #openstack | 20:56 | |
| *** wreese_ has joined #openstack | 20:56 | |
| *** larsemil_ has joined #openstack | 20:57 | |
| *** zul_ has joined #openstack | 20:57 | |
| *** londo__ has quit IRC | 20:57 | |
| *** larsemil has quit IRC | 20:57 | |
| *** Robi_ has quit IRC | 20:57 | |
| *** thedot has quit IRC | 20:57 | |
| *** jc_smith has quit IRC | 20:57 | |
| *** creiht has quit IRC | 20:57 | |
| *** wreese has quit IRC | 20:57 | |
| *** zul has quit IRC | 20:57 | |
| *** thedot has joined #openstack | 20:57 | |
| *** jc_smith has joined #openstack | 20:58 | |
| *** creiht has joined #openstack | 20:58 | |
| *** ChanServ sets mode: +v creiht | 20:58 | |
| *** Robi_ has joined #openstack | 20:58 | |
| *** allsystemsarego has quit IRC | 20:59 | |
| *** thedot has quit IRC | 21:03 | |
| *** rdw has joined #openstack | 21:06 | |
| *** Podilarius has quit IRC | 21:11 | |
| *** sparky_ has quit IRC | 21:20 | |
| *** ppetraki has quit IRC | 21:20 | |
| *** sirp2 has joined #openstack | 21:26 | |
| *** sirp1 has quit IRC | 21:27 | |
| _cerberus_ | eday: Where can I find docs for stubout? | 21:32 |
| eday | I just read: pydoc stubout, didn't really look online | 21:34 |
| *** rnewson has joined #openstack | 21:34 | |
| eday | _cerberus_: this has some info too: http://code.google.com/p/pymox/wiki/MoxDocumentation#Stub_Out | 21:35 |
| eday | (it's part of python mox module) | 21:35 |
| _cerberus_ | Ahh, thanks | 21:36 |
| eday | oh, wait... thats using stubout through mox, not directly | 21:36 |
| eday | hmm | 21:36 |
| *** justinsb has joined #openstack | 21:43 | |
| justinsb | Hi - Launchpad question... I fixed the merge problems on https://code.launchpad.net/~justin-fathomdb/nova/check-subprocess-exit-code/+merge/30707 and pushed a new version; how do I request a re-review without starting a new merge request (which would effectively lose all the context)? | 21:44 |
| *** rnewson has quit IRC | 21:46 | |
| *** e1mer has quit IRC | 21:47 | |
| eday | justinsb: mark as "work in progress" while fixing, and then back to "needs review" once you push | 21:48 |
| eday | justinsb: can also just leave a comment asking for folks to look at it again | 21:48 |
| *** JimCurry has left #openstack | 21:50 | |
| eday | jpipes: did you see you have text conflicts in your two proposed branches? :) | 21:50 |
| *** Xenith has joined #openstack | 21:50 | |
| _cerberus_ | eday: The testing code you have out there so far looks good. Getting pedantic for a second, what you've done so far is effectively mocking, using stubout, since you're ensuring something gets called. I think it's a great start though | 21:52 |
| mtaylor | letterj: looking | 21:54 |
| mtaylor | justinsb: also, you can click the "resubmit proposal" link, which will create a new merge prop but keep all of the old context | 21:54 |
| mtaylor | justinsb: sort of depends on how extreme you want to communicate that the new prop is - a resubmit resets any previous votes that were there but keeps the comments. just seting back to "needs review" is a less extreme thing and most of the time what you want | 21:55 |
| justinsb | eday, mtaylor: Thanks | 21:56 |
| justinsb | eday, mtaylor: I tried just toggling the state, but it's a minor merge, so I'll leave a comment as I don't think the toggle did anything (because I'd already pushed) | 21:56 |
| eday | _cerberus_: yeah, I have another rev I'm about to push that cleans up the mocking stuff a bit. definitely want to hear what you think can be improved her | 21:58 |
| _cerberus_ | eday: I'll do my best | 21:58 |
| eday | _cerberus_: I may not be taking advantage of mox yet in some way... just not sure how it will help with this | 21:59 |
| eday | _cerberus_: I just pushed another rev to api-tests, pull those down and take a look | 22:00 |
| _cerberus_ | eday: there's a lot of philosophical arguing over whether or not you really need to do mocking, since they tend to be very brittle. Any changes in method calls breaks the whole test in what most would argue are useless ways | 22:00 |
| *** rnewson has joined #openstack | 22:01 | |
| eday | _cerberus_: yeah.. I'm not quite sure where I sit for these types of apps yet... | 22:01 |
| justinsb | Another question... if I were to remove Twisted from the Compute service (replacing with threads) and submitted that, would it be merged? Should I post this as a Launchpad Blueprint/Bug/Question? | 22:02 |
| eday | justinsb: if the work is already done, I'd just push it and let folks poke at it :) | 22:02 |
| mtaylor | justinsb: I'd submit that as a bug or as an email to the mailing list - seems like one of those things that wants discussion | 22:03 |
| mtaylor | justinsb: OR - do what eday said if it's already done | 22:03 |
| _0x44 | eday: Early on we made pretty heavy use of mocks in our current codebase, and that ended up causing us a lot of problems with testing when things would change. | 22:03 |
| mtaylor | letterj: looks good in general. This: rsync (>=3.0.3-2) ... seems a little specific. I believe debian policy is usually to not depend on a specific deb release - so at least make is rsync (>3.0.3) ... unless you _actually_ need debian release -2 for it to work | 22:04 |
| eday | justinsb, mtaylor: But given the previous discussion on the mailing list re: twisted/threads for these types of workers, it didn't sound like anyone was really against the idea.. certainly would simplify things | 22:04 |
| justinsb | eday, mtaylor: I have a very basic version which is fine for poking at; but I'm sure it won't merge any more... I'll update it, propose it, and see how long it survives the flames... | 22:04 |
| eday | _0x44: ahh, good to know | 22:04 |
| mtaylor | justinsb: sounds like a good plan to me! | 22:04 |
| *** mray has quit IRC | 22:04 | |
| mtaylor | letterj: also, you have the override_dh_installdirs but don't override it with anything other than dh_installdirs :) | 22:05 |
| _0x44 | eday: Now if someone uses a mock (even inadvertantly) armmer crashes through the wall and crushes them | 22:05 |
| eday | haha | 22:05 |
| *** dendrobates is now known as dendro-afk | 22:06 | |
| eday | _0x44: do you get kool-aid with the crushing at least? | 22:06 |
| letterj | mtaylor: overriding the dh_installdirs is the only way I could get it to run | 22:07 |
| _cerberus_ | eday: only if he tips over | 22:07 |
| _0x44 | He makes you drink the kool-aid | 22:07 |
| _0x44 | :) | 22:07 |
| _cerberus_ | And then you just need a towel | 22:07 |
| *** e1mer has joined #openstack | 22:08 | |
| mtaylor | letterj: fascinating. | 22:08 |
| *** rbright has joined #openstack | 22:09 | |
| *** gundlach has quit IRC | 22:10 | |
| letterj | if you look at a log of the package build (even with *.dirs files in the debian directory) dh_installdirs is never run. It is strange. :-) | 22:10 |
| letterj | mtaylor: I'll make that change and push it up. | 22:11 |
| mtaylor | hrm. well I haven't used .dirs files in a long time since the .install files came into being | 22:11 |
| mtaylor | I'm wondering if there's something we're missing here | 22:11 |
| justinsb | mtaylor, eday: Crud... I can't realistically merge it, because it incorporates some significant other branches (e.g. raw disk images), which have themselves been blocked behind exit code checking :-( Looks like I'll have to try to get raw disk images through first | 22:15 |
| eday | _cerberus_: do you think the new api-tests changes look less like mocking? | 22:15 |
| _cerberus_ | eday: yeah. Like I said, I was just being nit-picky. Verifying that things are called in a specific order *can* be useful but it's usually not. | 22:16 |
| eday | _cerberus_: yeah, seems like all the matters is that the result is correct, not how you got it. the full mock/record/replay thing seems like extra work :) | 22:19 |
| *** rbright has left #openstack | 22:20 | |
| _cerberus_ | eday: exactly | 22:21 |
| *** rnewson has quit IRC | 22:27 | |
| *** spackest has joined #openstack | 22:41 | |
| spackest | so for whatever reason my company is rather in love with san and far from in love with locally attached spinning disk | 22:41 |
| * cw likes local disk very much | 22:42 | |
| spackest | wondering if anyone could share some implementation details of their swift setup and how perhaps locally attached spinning disk might actually work for petabytes of data | 22:42 |
| cw | spackest: you have lots of them | 22:42 |
| cw | spackest: if some fail, others have the data | 22:42 |
| spackest | think we're about to spend (waste :) ) millions of dollars to avoid local disk | 22:42 |
| spackest | any rackspace folks here that can share details? we could sign ndas if needed | 22:43 |
| spackest | or any other folks with large swift installs? | 22:43 |
| cw | we're working on a couple here | 22:43 |
| cw | starting small though | 22:43 |
| cw | i can share concrete details, pics, in the future | 22:43 |
| spackest | think the thought is that sata fails too often and it is really, really hard to manage replacements | 22:44 |
| spackest | cw: as details come that would be awesome | 22:44 |
| cw | spackest: it's otlerant to failures | 22:44 |
| cw | spackest: what data size do you want to start at? | 22:44 |
| zaitcev | spackest: Have you ever heard of Google? They would have NEVER gotten where they are now if they used SAN. | 22:45 |
| spackest | almost feels like they think 10% of sata drives fail each day and would require a full team of very technical folks to replace them | 22:45 |
| spackest | eventually 100 PB | 22:45 |
| cw | it's not that bad | 22:45 |
| cw | from experience with roughly 30k spindles you get about 1 a day | 22:45 |
| cw | sometimes a little more | 22:46 |
| cw | it's oddly a lot better than i would have expected | 22:46 |
| *** spackest1 has joined #openstack | 22:46 | |
| spackest1 | just would like some real data to respond with | 22:46 |
| *** zheng_li has quit IRC | 22:47 | |
| cw | @100PB i can only make up lies | 22:47 |
| larissa | cw: Error: "100PB" is not a valid command. | 22:47 |
| cw | lol | 22:47 |
| spackest1 | we're pushing 10 pbs I think but want to grow it | 22:47 |
| cw | at 100PB i can extrapolate for swift on what we've done so far | 22:47 |
| zaitcev | There are papers presented at FAST that have real failure data (basically about 3 times worse than claimed MTBF). I don't know why would anyone expect anything different. | 22:47 |
| cw | zaitcev: i want to get some 5900 RPM LP desktop drives and see how those hold up | 22:48 |
| cw | i've been warned off them for now and don't have time to really push it | 22:48 |
| spackest1 | just wondering how manageable such a system would be | 22:48 |
| zaitcev | And the real trick anyway is not to bother replacing drives. Just leave dead drives there and swap servers when they completely kaput, in a routine sweep. | 22:48 |
| *** zheng_li has joined #openstack | 22:49 | |
| cw | that sorta sucks | 22:49 |
| cw | you have parts of the partition that are suboptimal then | 22:49 |
| *** spackest has quit IRC | 22:50 | |
| spackest1 | zaitcev: do you have links? | 22:50 |
| gholt | That actually was a consideration here too. If a drive fails, take it out of the ring. Once a server gets "bad" enough, take it out of the ring, reset everything to new and put it back online. | 22:51 |
| gholt | Depends on how much you trust your drive replacement staff. :) | 22:51 |
| gholt | We ended up going the replace it sooner just because that makes better use of the resources out there. Sucks to have a beefy machine with half its drives gone. | 22:52 |
| *** zheng_li has quit IRC | 22:53 | |
| *** zaitcev has quit IRC | 22:57 | |
| *** jdmaturen has left #openstack | 23:05 | |
| cw | gholt: our customers have staff on hand in their DCs so the recommendation will be replace it when possible | 23:05 |
| cw | i guess if they drag it out for too long we'll just remove them from the ring | 23:05 |
| cw | i'm a little scared to automate that heavily initially ... in case it blows up :) | 23:06 |
| gholt | Same here. :) | 23:06 |
| *** miclorb_ has joined #openstack | 23:07 | |
| cw | gholt: failure rates increase somewhat as disks age ... but with product growth that ends up being less of an issue hopefully | 23:08 |
| cw | ie. most of the stuff you care about is new and shiny | 23:08 |
| gholt | Yeah, there seems to be a flurry of just-installed failures as well, even with "proper" burn in. That's make me sad, but it is what it is. | 23:09 |
| *** pcrews has quit IRC | 23:09 | |
| cw | yeah, there are those ... i have someone trying to sell me 1000s of reconditions drives because of that | 23:10 |
| *** spackest has joined #openstack | 23:10 | |
| cw | they insist they are MORE AWESOME because they have worn down bits or something | 23:10 |
| *** zaitcev has joined #openstack | 23:10 | |
| *** dendro-afk is now known as dendrobates | 23:13 | |
| *** spackest1 has quit IRC | 23:13 | |
| gholt | I have no idea what the lifespan of SSDs would be. Could be interesting. | 23:14 |
| mtaylor | gholt: no one does really | 23:14 |
| cw | the vendors who have access to the raw bits could probably tell at this stage | 23:15 |
| cw | what users see has all the bits corrected ... we can't see how much 'decay' there is inside | 23:15 |
| cw | with some idea of how that grows over time we could model it to some extent (flash is pretty well understood at this point) | 23:15 |
| *** sirp2 has quit IRC | 23:17 | |
| *** Kami__ has quit IRC | 23:20 | |
| *** Kami_ has joined #openstack | 23:21 | |
| *** cory_ has quit IRC | 23:21 | |
| *** gasbakid has quit IRC | 23:24 | |
| *** spackest1 has joined #openstack | 23:32 | |
| *** sandywalsh has quit IRC | 23:33 | |
| *** lew has quit IRC | 23:33 | |
| *** littleidea has quit IRC | 23:34 | |
| *** spackest has quit IRC | 23:35 | |
| *** spackest1 has quit IRC | 23:43 | |
| *** littleidea has joined #openstack | 23:45 | |
| eday | vishy: any idea what the 'cloud_topic' was used for? It's referenced in the server/unittests, but it looks unused (flag commented out, nothing in nova source sending to it). Am thinking of just removing it as it looks like dead code | 23:53 |
| *** sirp1 has joined #openstack | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!