Thursday, 2013-12-12

*** vipul-away is now known as vipul00:00
*** denis_makogon has quit IRC00:07
*** achampion has joined #openstack-trove00:12
openstackgerritSteve Leon proposed a change to openstack/trove: Make use of IP filtering when creating DNS records  https://review.openstack.org/6158900:14
SlickNikheh, the fix for the deleted_at timestamp issue hit the same deleted_at timestamp issue while trying to merge.00:19
SlickNikhttps://jenkins02.openstack.org/job/gate-trove-python27/698/consoleFull00:20
SlickNik^^^ FYI robertmyers / juice00:21
*** dmitry_iakunchyk has joined #openstack-trove00:21
hub_capSlickNik: thats AWESOME00:22
hub_capreverify self00:22
*** dmitry_iakunchyk has quit IRC00:24
SlickNiklol00:25
SlickNikActually it looks like a very similar error in a different test.00:30
SlickNikJust spoke to juice and he's on it.00:30
hub_capheh sweet00:34
*** vipul is now known as vipul-away00:39
*** yidclare has quit IRC00:42
hub_capwell SlickNik im out.. ill come back tonight to see how thigns are going (id like to make sure we are stable again before our workday starts tomrrow)00:43
*** yogesh has joined #openstack-trove00:52
*** jasonb365 has joined #openstack-trove01:07
*** vipul-away is now known as vipul01:10
*** jcooley_ has quit IRC01:12
*** Barker has joined #openstack-trove01:18
*** jasonb365 has quit IRC01:20
*** jcooley_ has joined #openstack-trove01:33
cp16nethub_cap: pong01:35
*** yogesh has quit IRC01:35
*** nosnos has joined #openstack-trove01:35
*** jmontemayor has quit IRC01:37
hub_capcp16net: did i ping u? like hrs ago?01:39
cp16neti dunno01:40
hub_capmmmm im pretty sure im addicted to the tikka masala at my local indian joint01:40
cp16neti just got on01:40
hub_capyour bouncer alerted u i guess?01:40
cp16net[23:30:42] cp16net: around?01:40
hub_capya i cant remember.............01:40
cp16neti think thats gmt01:40
cp16netyup01:40
hub_capi think i was gonna ask u something blueprint related01:42
hub_capoooooowell01:42
cp16netyou were triaging some bps01:42
cp16neti saw01:42
cp16netapi?01:42
cp16netroutes?01:42
hub_capwell that did confuse me (the route thing)01:42
hub_capbut i cant really talk now cuz my wife is supposed to be eating dinner01:43
hub_capand im supposed to be watching boy hehe01:43
hub_capill hit u up in a while cp16net01:43
cp16netword01:43
cp16netok01:43
cp16neti'll be playing my ps4 :-P01:43
cp16netand waiting for tests in the background01:43
juicecp16net: which games do you have?01:45
cp16neti just got killzone01:45
cp16netsystem update required...01:45
juiceis that the only one you have?01:46
juicehow is it?01:46
cp16netsince my wife wasnt happy that i was going to buy it for myself i held back01:46
cp16netyeah its the only one i have01:46
cp16netits pretty neat01:46
cp16netalthough last night i roamed around chapter 3 for about 2 hours and couldnt figure out what to do01:46
juiceare the graphics greatly improved01:46
cp16neti feel like the most improvement i see is that there are reallish looking shadows everywhere01:47
cp16netdepending on your movement you see shadows alot01:47
*** Barker has quit IRC01:47
cp16neti think thats something they almost over did really01:47
cp16netthe interface is greatly improved01:48
cp16netjuice: you get one?01:48
hub_capi got a can of these baked beans01:50
juiceyes sir01:50
juicei only have battlefield01:50
juicei suck at it but still like to play sometimes01:50
juiceI tried out Knack the other day - pretty fun game01:51
*** rongze has joined #openstack-trove01:52
cp16neti heard that knack has the worst score out of all the release titles01:52
amcrnhub_cap: i've got a LOG! from Blammo01:53
hub_capamcrn: im pretty sure i called soemthign that made changelogs a long while ago log-blammo01:55
hub_capcp16net: remember that?01:55
cp16netoh yeah01:55
*** demorris has joined #openstack-trove01:55
amcrnniiice01:55
hub_capugh stupid emacs recognises meta as my alt key, but wont recognise M-x unless i hit the space key01:56
hub_capSlickNik: ^ ^ Sound familiar?01:56
hub_caps/space/escape01:56
hub_capits killin me... oh the glories of setting everything up from scratch01:57
cp16nethah01:57
hub_capthis time though i can save all my files in github01:57
hub_capneed custom keymappings? .Xmodmap01:57
amcrni'm out, night fellas01:59
amcrncp16net: i'll reply to your ML sometime tomorrow, I think I have a semi-clever workaround02:00
cp16netcool stuff amcrn02:01
cp16netit might be something i've been thinking as well02:01
*** demorris has quit IRC02:01
amcrncp16net: the general premise is that if a "clone" operation exists, then the ease of porting a configuration-group to another version bump is trivial02:02
amcrnif a cloud provider is planning on having tons of minor version bumps, they'll use a single version of "5.5" and just update the pkgs secretly behind the scenes anyway02:02
amcrnbut if a provider wants to have tons of visible versions, and that happens to include tons of "compatible" minor versions, then as long as a user can clone an existing, the issue becomes fairly minor02:03
cp16netumm well define what is a "clone"02:04
cp16netof an instance? config group?02:04
amcrngiven configuration-id "12345", create a copy of it with a target datasource-version-id, resulting in configuration-id "12346"02:05
amcrn(deep copy, not shallow)02:05
cp16nethrm..02:06
cp16netthats not a bad idea02:06
* cp16net wasnt thinking that02:06
amcrnhaven't thought it all the way through, so there might be some holes, but it was an interesting concept that crossed my synapses02:06
amcrnwhat's yours?02:07
cp16netincluding the ability to have a reference to both a type and version optionally02:08
cp16netif you set the version its obviouslly tied to a type02:08
amcrntype as a required field, version as optional?02:08
cp16netin terms of the database record yes02:09
cp16netin terms of the api a datastore would be required be it from a version or type02:09
cp16netthis would allow as much flexibility as the datastores have now02:10
cp16netdepending on how providers choose to set them up02:10
amcrnhrm, i think i like your idea better02:10
amcrnthe only snafu i can think of is if the datastore maintainers don't follow proper versioning02:11
amcrnit might be difficult for the provider to make sure they're not accidentally allowing a version 'x' configuration-group masquerade as a version 'y' because they're incompatible02:11
amcrnbut that might not be an issue, i'd have to work through a couple of scenarios on paper/pad02:11
cp16netwell heres more food for thought...02:12
cp16neta configuration that you have has the edit/update property where it would be on the customer to update config group's datastore if they chose to update02:13
cp16netbut that might get really weird when you have it already applied to different instacne types02:13
cp16netand they you edit the CG (config group)02:13
cp16netyou sould not be allowed to edit the datastore of the CG at that time02:14
amcrnhrm interesting.02:14
openstackgerritA change was merged to openstack/trove: Fix deleted_at timestamp in usage tests  https://review.openstack.org/6159002:15
amcrnthat is food for thought, let me chew on it a bit and send an "official" reply to the ML thread; there's definitely quite a few ways to do it.02:16
amcrntime to procure sustenance and hit the road02:16
amcrng'night :)02:17
*** amcrn has quit IRC02:17
cp16netdont choke on it :-P02:18
cp16netdamn he left02:18
*** amytron has joined #openstack-trove02:20
*** erkules_ has joined #openstack-trove02:24
*** jcooley_ has quit IRC02:24
*** erkules has quit IRC02:26
*** rongze has quit IRC02:28
*** rongze has joined #openstack-trove02:29
*** vgnbkr has joined #openstack-trove02:40
*** SnowDust has joined #openstack-trove02:41
openstackgerritA change was merged to openstack/trove: Extract suffix from req URL to avoid escaping dots  https://review.openstack.org/5696602:48
openstackgerritA change was merged to openstack/trove: db: make compatible with SQLAlchemy 0.8  https://review.openstack.org/5915702:51
hub_capjuice: around?02:53
SlickNikhub_cap: he just left not too long ago. Said he'd be online later, I think.02:55
hub_capSlickNik: do u know if the bug that merged (deleted_at timestamp) endedup having a sublte, slightly different, bug as well?02:57
hub_capalso, do u think we aer at a point where we can rock some rechecks on all the -1's02:58
SlickNikhub_cap: Nope, I dug a bit more into it. The fix for that was good. A similar issue exists in another test and the gate is sometimes hitting that other issue.02:58
hub_capok SlickNik02:59
hub_capi dont see your requirements change yet SlickNik02:59
hub_caphas it not landed?02:59
SlickNikhub_cap: I updated the bug with details of the other issue that the gate is sometimes hitting. Let me find you a link.02:59
SlickNikIt's verifying.02:59
hub_capsweet SlickNik03:00
SlickNikgonna re-run rechecks on some of the −1's after it lands.03:00
hub_capi called it03:00
hub_capi said itl be 8pm here time03:00
hub_capi cant believe it lol03:00
SlickNiklol03:00
openstackgerritA change was merged to openstack/trove: Updated from global requirements  https://review.openstack.org/6125503:01
hub_capWOO03:01
* hub_cap is so happy03:01
SlickNikThat was the last of the +2, Approved that were stuck in the gate.03:01
hub_capso once dmakogon_ updates to not use mockito we wil be good to go ya03:01
SlickNikWell, here's details for that other gate issue I was mentioning:03:02
SlickNikhttps://bugs.launchpad.net/trove/+bug/123662003:02
SlickNikrobertmyers patch fixed the delete event, but there's a resize event that has a similar issue (i.e. assert on a time field that's off by a second sometimes)03:03
SlickNikI know juice is working on a fix for that one, hopefully he'll have something tomorrow.03:04
*** amytron has quit IRC03:05
*** SushilKM has joined #openstack-trove03:06
hub_capok cool SlickNik. im glad we are fixing these properly too03:09
SlickNikYeah, me too.03:10
hub_capSlickNik: https://review.openstack.org/#/c/57780/03:11
hub_capwe need to get this merged too, otherwise its kinda annoying to test the new services03:11
*** diakunchikov__ has quit IRC03:12
*** ikhudoshyn has quit IRC03:12
SlickNiklookinh03:12
SlickNiklooking*03:12
*** ikhudoshyn has joined #openstack-trove03:12
hub_capive restarted the int-tests for it03:12
*** diakunchikov__ has joined #openstack-trove03:13
hub_capid really like to get the datastores merged in, its taking too long hehe03:13
hub_capok this irssi server is terrible, im building a new one.. ugh03:15
SlickNik57780 looks good, the only comment I have is that if you run set_datastore multiple times, it _always_ makes the last run one the default type.03:17
SlickNikYou probably want to pass in a flag indicating whether to set the newly created datastore as default or not.03:18
*** vgnbkr has quit IRC03:19
hub_capgood point03:19
hub_capbut this is gonna go away anyway right :)03:19
hub_capeventually this will be shared between tempest and devstack03:19
*** jcooley_ has joined #openstack-trove03:20
SlickNikYeah, we can fix it then. I'm okay with merging it since it's blocking datastore testing atm.03:20
SlickNikLet the robot's come back. I'll approve.03:21
SlickNikrobots*03:21
hub_capword SlickNik03:23
hub_capgod my latency to my rax servers is so annoying03:23
hub_capitll be like consistant ping ~60ms03:24
hub_capthen ill get 400ms03:24
SlickNikstrangeness.03:24
SlickNiktraceroute that and find the weakest link.03:24
hub_capwell waht sux is that its completely random03:25
hub_capand its happening on > 1 server in different dc's03:25
*** jcooley_ has quit IRC03:26
SlickNikugh that sounds bizzaro.03:26
hub_capya it sux03:27
hub_capi deal w/ it03:27
hub_capour dc's are not close to me03:27
hub_cap@all03:27
hub_capdfw, iad, hk, lon...03:27
SlickNikWhere they at? TX?03:27
SlickNikahhh03:27
hub_capiad, dfw, syd, hkg, ord03:28
SlickNikord?03:28
hub_capchicago ohare03:28
SlickNikwow I didn't know that was ord03:28
*** rongze_ has joined #openstack-trove03:33
SlickNikOkay dude, I gotta head out. I'll be back on later to check on some of the reviews (incl 57780).03:35
*** rongze has quit IRC03:36
*** yogesh has joined #openstack-trove03:45
*** haomaiwang has joined #openstack-trove03:48
*** haomaiwang has quit IRC03:50
*** haomaiwang has joined #openstack-trove03:51
*** haomaiwang has quit IRC03:58
*** yogesh has quit IRC03:59
*** yogesh has joined #openstack-trove04:06
*** yogesh has quit IRC04:07
*** amytron has joined #openstack-trove04:07
*** haomaiwang has joined #openstack-trove04:07
SushilKMneeded help to re-run redwarf for https://review.openstack.org/#/c/60662/04:07
SushilKM its somehow failed, probably not because of my code :(04:08
*** yogesh has joined #openstack-trove04:08
*** yogesh has quit IRC04:11
*** yogesh has joined #openstack-trove04:13
*** yogesh has quit IRC04:14
*** SushilKM has quit IRC04:17
*** yogesh has joined #openstack-trove04:18
*** yogesh has quit IRC04:20
*** yogesh has joined #openstack-trove04:21
*** yogesh has quit IRC04:21
*** yogesh has joined #openstack-trove04:28
*** yogesh has quit IRC04:30
*** ashestakov_phone has quit IRC04:30
*** yogesh_ has joined #openstack-trove04:32
*** yogesh_ has quit IRC04:33
openstackgerritnilakhya proposed a change to openstack/trove-integration: Remove erroneous logfile in script directory  https://review.openstack.org/6165404:41
*** yogesh has joined #openstack-trove04:44
*** yogesh has quit IRC04:44
*** yogesh has joined #openstack-trove04:47
*** yogesh has quit IRC04:48
openstackgerritnilakhya proposed a change to openstack/trove-integration: Remove erroneous logfile in scripts directory  https://review.openstack.org/6165404:49
*** rongze_ has quit IRC04:52
openstackgerritnilakhya proposed a change to openstack/trove-integration: Remove erroneous logfile in scripts directory  https://review.openstack.org/6165405:01
*** SergeyLukjanov has joined #openstack-trove05:17
*** achampion has quit IRC05:22
*** rongze has joined #openstack-trove05:22
*** rongze has quit IRC05:27
*** SergeyLukjanov is now known as _SergeyLukjanov05:33
*** _SergeyLukjanov has quit IRC05:33
*** dougshelley66 has joined #openstack-trove05:33
*** dougshelley66 has quit IRC05:44
*** achampion has joined #openstack-trove05:44
*** yogesh has joined #openstack-trove05:49
*** haomaiwa_ has joined #openstack-trove05:49
*** haomaiwang has quit IRC05:51
*** SergeyLukjanov has joined #openstack-trove05:51
*** yogesh has quit IRC05:59
*** yogesh_ has joined #openstack-trove06:00
*** jcooley_ has joined #openstack-trove06:06
*** yogesh has joined #openstack-trove06:08
*** yogesh_ has quit IRC06:10
*** yogesh_ has joined #openstack-trove06:12
*** yogesh has quit IRC06:12
*** jaishanker has joined #openstack-trove06:12
*** SergeyLukjanov has quit IRC06:17
*** rongze has joined #openstack-trove06:20
*** vipul has quit IRC06:22
*** vipul has joined #openstack-trove06:22
*** yogesh_ has quit IRC06:28
*** jcooley_ has quit IRC06:30
*** jcooley_ has joined #openstack-trove06:30
*** jcooley_ has quit IRC06:31
*** jcooley_ has joined #openstack-trove06:31
*** amytron has quit IRC06:33
*** SushilKM has joined #openstack-trove06:38
*** shalini has joined #openstack-trove06:42
*** PradeepChandani_ has quit IRC06:46
*** yogesh has joined #openstack-trove06:46
*** lifeless has quit IRC06:48
*** jaishanker_ has joined #openstack-trove06:48
*** PradeepChandani has joined #openstack-trove06:48
*** jaishanker has quit IRC06:49
*** dougshelley66 has joined #openstack-trove06:51
*** yogesh_ has joined #openstack-trove06:51
*** yogesh has quit IRC06:51
*** yogesh_ has quit IRC06:56
*** yogesh has joined #openstack-trove06:56
*** yogesh_ has joined #openstack-trove06:58
*** yogesh has quit IRC07:02
*** yogesh_ has quit IRC07:08
*** SnowDust has quit IRC07:08
*** lifeless has joined #openstack-trove07:15
*** jcooley_ has quit IRC07:18
*** haomaiwa_ has quit IRC07:21
*** jcooley_ has joined #openstack-trove07:22
*** yogesh has joined #openstack-trove07:32
*** yogesh has quit IRC07:40
*** jcooley_ has quit IRC07:46
*** yogesh has joined #openstack-trove07:46
*** yogesh has quit IRC07:48
*** yogesh has joined #openstack-trove07:48
*** jcooley_ has joined #openstack-trove07:50
*** jcooley_ has quit IRC07:58
*** erkules_ is now known as erkules08:00
*** yogesh has quit IRC08:02
*** rongze has quit IRC08:02
*** jaishanker_ has quit IRC08:05
*** rongze has joined #openstack-trove08:06
*** rongze has quit IRC08:11
openstackgerritA change was merged to openstack/trove: Fixed typos in files in trove/guestagent.  https://review.openstack.org/6066208:16
*** denis_makogon has joined #openstack-trove08:19
*** yogesh has joined #openstack-trove08:19
SushilKMhttps://review.openstack.org/#/c/60564/ is ready for merge all builds and tests completed :)08:21
*** haomaiwang has joined #openstack-trove08:34
SushilKMduring merge of https://review.openstack.org/60564 got jenkins failing09:10
SushilKMgot the comment from jenkins09:10
SushilKMThis change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset.09:10
denis_makogonSushilKM, please prebase you review09:20
SushilKMoh my bad again all approvals09:21
denis_makogonSushilKM, yup =/09:21
SushilKManyways, i wud proceed thats the only way ahead09:21
SushilKM:(09:21
denis_makogonSlickNik, stay here, with us =)09:21
*** haomaiwang has quit IRC09:22
SushilKMthere was needed one change09:24
SushilKMin09:24
SushilKMtrove/guestagent/datastore/mysql/service.py09:24
SushilKMi m running tox09:24
SushilKMthen wud send it on09:24
denis_makogonщл09:26
denis_makogonok09:26
*** SushilKM has quit IRC09:37
*** yogesh has quit IRC09:39
*** flaper87|afk is now known as flaper8709:40
openstackgerritSushil Kumar proposed a change to openstack/trove: Enabling H403, H702 rules  https://review.openstack.org/6056409:41
*** SushilKM has joined #openstack-trove09:43
SushilKMI have rebased the change, but for approvals I think I need to wait until tonight(my time zone)..... :)09:43
*** denis_makogon has quit IRC09:44
*** haomaiwang has joined #openstack-trove10:15
*** nosnos_ has joined #openstack-trove10:15
*** nosnos_ has quit IRC10:16
*** nosnos_ has joined #openstack-trove10:16
*** nosnos has quit IRC10:17
*** SergeyLukjanov has joined #openstack-trove10:17
*** nosnos_ has quit IRC10:20
*** diakunchikov__ has quit IRC10:28
*** SnowDust has joined #openstack-trove10:31
*** dmakogon_ is now known as denis_makogon10:33
denis_makogonSushilKM, i'll ask -cores to re-approve it, since new patch is manually rebased and nothing else10:36
SushilKMmany thanks @denis_makogon10:37
denis_makogonSushilKM, you are welcome10:38
SushilKMactually, i wud look for sooner approval, coz if something else gets merged earlier, i may need to rebase it again, and again approvals ....10:39
*** diakunchikov has joined #openstack-trove10:39
*** rongze has joined #openstack-trove10:41
SushilKMthankfully jenkins and redwarf are passed in one go ... :D10:43
*** rongze has quit IRC10:46
denis_makogonSushilKM, i see, great10:50
*** yogesh has joined #openstack-trove10:50
*** debasish has quit IRC10:50
*** yogesh has quit IRC10:55
openstackgerritIonut Artarisi proposed a change to openstack/trove: add trove-cli to list of scripts so it gets installed in bindir  https://review.openstack.org/6170411:09
*** SnowDust has quit IRC11:19
openstackgerritDenis M. proposed a change to openstack/trove: Status change on missing template issue  https://review.openstack.org/5884511:58
*** shalini has quit IRC12:03
*** SnowDust has joined #openstack-trove12:03
openstackgerritSushil Kumar proposed a change to openstack/trove: Updates trove-guestagent.conf.sample  https://review.openstack.org/6171312:10
openstackgerritDenis M. proposed a change to openstack/trove: Use testtools as base test class.  https://review.openstack.org/6116912:12
*** achampion has quit IRC12:35
*** rongze has joined #openstack-trove12:37
*** rongze has quit IRC12:41
openstackgerritDenis M. proposed a change to openstack/trove: Use testtools as base test class  https://review.openstack.org/6116912:44
openstackgerritDenis M. proposed a change to openstack/trove: Fix unit tests assertions  https://review.openstack.org/6172512:53
*** SergeyLukjanov is now known as _SergeyLukjanov12:54
*** _SergeyLukjanov has quit IRC12:54
*** debasish has joined #openstack-trove13:01
openstackgerritKevin Conway proposed a change to openstack/trove-integration: Add support for kick-starting new datastores  https://review.openstack.org/5778013:02
*** pdmars has joined #openstack-trove13:05
*** pdmars has quit IRC13:07
*** pdmars has joined #openstack-trove13:11
*** rongze has joined #openstack-trove13:12
openstackgerritDenis M. proposed a change to openstack/trove-integration: Initial support for single instance Cassandra Database  https://review.openstack.org/5266613:13
openstackgerritDenis M. proposed a change to openstack/trove: Fix unit tests assertions  https://review.openstack.org/6172513:16
*** SergeyLukjanov has joined #openstack-trove13:23
openstackgerritDenis M. proposed a change to openstack/trove-integration: Initial support for single instance Cassandra Database  https://review.openstack.org/5266613:24
*** debasish has quit IRC13:26
openstackgerritDenis M. proposed a change to openstack/trove-integration: Initial support for single instance Cassandra Database  https://review.openstack.org/5266613:31
*** SnowDust has quit IRC13:32
openstackgerritDenis M. proposed a change to openstack/trove-integration: Add support for minimal MongoDB testing  https://review.openstack.org/5337813:34
*** achampion has joined #openstack-trove13:37
*** kevinconway has joined #openstack-trove13:41
*** dougshelley66 has quit IRC13:53
*** abramley has joined #openstack-trove13:53
*** rongze_ has joined #openstack-trove14:01
*** rongze has quit IRC14:04
*** dougshelley66 has joined #openstack-trove14:11
*** rongze has joined #openstack-trove14:20
*** rongze_ has quit IRC14:24
openstackgerritSushil Kumar proposed a change to openstack/python-troveclient: Fixes trove help command  https://review.openstack.org/6175614:25
*** SushilKM has quit IRC14:35
*** rongze_ has joined #openstack-trove14:43
*** rongze has quit IRC14:44
*** demorris has joined #openstack-trove14:53
*** glucas has joined #openstack-trove15:02
*** amytron has joined #openstack-trove15:02
*** robertmyers has joined #openstack-trove15:03
*** jcooley_ has joined #openstack-trove15:05
*** robertmyers has quit IRC15:05
*** robertmyers has joined #openstack-trove15:05
*** Barker has joined #openstack-trove15:09
*** jcru has joined #openstack-trove15:10
*** jasonb365 has joined #openstack-trove15:22
*** Flying_Bond has joined #openstack-trove15:35
*** Flying_Bond has quit IRC15:35
*** Flying_Bond has joined #openstack-trove15:36
*** haomaiwa_ has joined #openstack-trove15:36
*** Flying_Bond has quit IRC15:36
*** Flying_Bond has joined #openstack-trove15:37
*** Flying_Bond has quit IRC15:37
*** haomaiwang has quit IRC15:37
*** datsun180b has joined #openstack-trove15:39
*** Flying_Bond has joined #openstack-trove15:41
*** rnirmal has joined #openstack-trove15:41
*** Flying_Bond has quit IRC15:44
*** grapex has joined #openstack-trove15:48
denis_makogonhub_cap, wow, why does fixing/correcting tests is bad idea ?15:55
hub_capdenis_makogon: is tht not the review we tlked about in the meeting yesterday?15:56
hub_capyou arent _fixing_ anything15:56
hub_capif they were broken they wouldnt run, and we woudlnt have code merged15:57
hub_capor there would be a bug like w/ the timing issue for notification15:57
hub_capchanging (a,b) to (b,a) does not fix any bug15:57
*** demorris_ has joined #openstack-trove15:57
hub_capdenis_makogon: if it is different from the decision we _made_ yesterday, then ill look it over15:57
denis_makogonhub_cap, there's not only (a,b) -> (b,a)15:57
hub_capwell remove all of that15:57
hub_capand ill look it over15:57
hub_capi want to see there is _no_ (a,b) -> (b,a)15:58
hub_capzero15:58
denis_makogonok15:58
denis_makogonbut it seems like ignoring testtools API15:58
hub_capok ill let them scream at me :)15:58
*** SushilKM has joined #openstack-trove15:59
*** demorris has quit IRC15:59
*** demorris_ is now known as demorris15:59
kevinconwayi vote we switch to shunit2. it is the best framework.16:00
kevinconwayjust like python unit test except it is bash.16:00
SushilKMHi Michael, Denis r u around16:01
SushilKMregarding https://review.openstack.org/#/c/61756/16:01
SushilKMi saw ur comment about updating the readme16:01
hub_capkevinconway: lol16:01
hub_capSushilKM: yes16:01
SushilKMbut readme already has the correct trove help output16:01
hub_capit does lol?16:02
hub_capso someone _hand edited_ it ?16:02
SushilKMsee here16:02
SushilKMhttps://github.com/openstack/python-troveclient/blob/master/README.rst16:02
hub_capya i do see that16:02
hub_capso does it work too when u do trove help blah16:02
hub_capinstead of just trove help?16:02
SushilKMthe one i implemented16:03
SushilKMworks16:03
SushilKMi have tested it16:03
SushilKMlet me show u ...16:03
SushilKMjst a se16:03
SushilKMsec16:03
hub_capk. ill also pull it down real quick16:03
SushilKMsure16:04
*** greghill has joined #openstack-trove16:04
hub_capSushilKM: is that supposed to make sure <subommand> comes after? cuz it already does in my client... odd16:04
SushilKMbtw chl this16:04
SushilKMhttps://gist.github.com/sushilkm/793044216:04
hub_capnmnm SushilKM16:05
hub_capi was doing nova list16:05
SushilKMi have listed the difference in the bug :)16:05
SushilKMhttps://bugs.launchpad.net/trove/+bug/126033216:05
datsun180bSlickNik: hey thanks for reverifying my bug after i'd gone home last night16:06
*** greghill has quit IRC16:06
*** yidclare has joined #openstack-trove16:06
*** greghill has joined #openstack-trove16:06
SushilKMso hub_cap did u see that gist :)16:09
datsun180bthat's great16:10
*** Barker has quit IRC16:13
*** Barker has joined #openstack-trove16:15
hub_capSushilKM: wonderful16:16
*** Flying_Bond has joined #openstack-trove16:18
SushilKMso +116:18
SushilKM:D16:18
SushilKMthe kind gesture16:20
*** SergeyLukjanov has quit IRC16:21
*** SushilKM has quit IRC16:25
*** SushilKM has joined #openstack-trove16:26
greghillif anyone gets a chance https://review.openstack.org/#/c/61505/ now passes the automated stuff and needs a re-review16:27
hub_caphow sweet, greghill is new.. "if anyone gets a chance"16:29
hub_capgreghill: be like "REVIEW MY SHIT XXXX"16:29
hub_capNOW OR DIE16:29
greghillwell, it's not like a critical bug or anything and I know more important things are afoot16:30
datsun180bmodesty will get you nowhere16:30
hub_caphehe16:30
hub_capexactly datsun180b16:30
robertmyersgreghill: everything is important16:30
datsun180bspeaking from experience16:31
hub_caprobertmyers: my stuff is more important though16:31
* hub_cap falls out of chair from huge ego16:31
robertmyershub_cap: no mine ;)16:31
hub_caphttp://www.youtube.com/watch?v=p-3e0EkvIEM16:31
greghillif you don't review my bug, you will not be able to attend a summit without looking over your shoulder, wondering where I might be16:32
greghillis that better?16:32
SushilKMhub_cap  thanks16:32
SushilKM:D16:32
hub_capgreghill: yes16:33
hub_capkinda16:33
SushilKMbtw hub_cap can u also update the importance factor of the bug ... :D16:33
hub_capno curse words16:33
hub_capno caps16:33
hub_capSushilKM: yes16:33
SushilKMmany thanks16:33
greghillI'm such a n00b16:33
cp16netlol16:33
SushilKMand one more thing @hub_cap16:33
SushilKMhttps://review.openstack.org/#/c/60564/ failed during merging16:33
SushilKMfor it needed a rebase16:34
SushilKMi had to manually do that, so once again need approvals on that16:34
SushilKM:D16:34
hub_capgreghill: lol at my bug report "Updating a users host in update_attributes to say, poopoo, is invalid."16:34
cp16netmine...16:34
*** Flying_Bond has quit IRC16:34
hub_cap"Old version history" is worthless on gerrit16:35
datsun180bhub_cap: poopoo is a valid hostname though, just not a valid ipv4 adress or fqdn16:35
greghillI read through the lines and realized it meant if the ipv4 only setting was enabled (as is default)16:36
datsun180bseeing fqdn reminds me of fhqwhgads16:36
greghillcome on, fhqwhgads16:37
hub_capdatsun180b: LEFT FIELD16:38
hub_capCOME ON FW...DS16:38
hub_cap ^^ didnt feel like typing16:38
hub_capfor those who dont know what datsun180b and dont know how to google16:38
hub_caphttp://www.homestarrunner.com/fhqwhgads.html16:38
kevinconwayhub_cap: wow… is that still a thing?16:38
hub_capkevinconway: bits dont die :)16:39
datsun180bbut they can rot16:39
hub_capheh nice16:39
hub_capi catn stop listening to this song now16:39
hub_capgeesus16:39
hub_capwhy datsun180b why16:39
datsun180bthe ride never ends16:40
cp16netcome on fhqwhgads16:40
*** Flying_Bond has joined #openstack-trove16:41
hub_capmy son likes that song lol16:41
hub_capbut then again, his 1yr old brain is far from fully formed :)16:41
greghillI actually bought the CD16:41
cp16netwill you buy me a pizza?16:42
greghillonly if you sing that song for the whole company16:42
hub_capcp16net: DO IT16:42
*** flaper87 is now known as flaper87|afk16:42
hub_capcuz greghill will have to buy like 50 pizzas to feed everyone16:42
cp16netlol16:43
datsun180bremember when h*r would be a nearly full-screen experience16:43
hub_capgreghill: new guy becomes broke guy who decided a first week pizza bet was a good idea16:43
hub_capdatsun180b: huuuuuuuuuh?16:43
cp16nethe clearly doesnt know who hes talking to....16:43
cp16net:-P16:43
datsun180bhub_cap: our screens got bigger, but homestar didn't16:43
cp16net480x64016:44
greghillI did not generalize that bet, unfortunately16:45
openstackgerritRobert Myers proposed a change to openstack/trove: Simplify swift storage load logic  https://review.openstack.org/5779616:45
openstackgerritRobert Myers proposed a change to openstack/trove: Moving storage logic out of backup runner class  https://review.openstack.org/5670216:45
hub_capohhhhhhhh dude ok okokok i gets it datsun180b16:46
hub_capgreghill: cp16net :)16:46
hub_caphttps://github.com/openstack/trove-integration/blob/master/scripts/run_tests.log <--- hilarious commit16:46
datsun180bon the upshot at least logging works16:47
hub_capcan people not target their own bugs / blueprints ?16:47
datsun180bsmall victories16:47
hub_capdatsun180b: LULZ16:47
datsun180bi prefer having verbose failures inst16:48
datsun180bsegmentation fault16:48
hub_capi keep having to target peoples bugs and its anoying!16:48
greghillmaybe we weren't aiming for a target? :D  (sorry)16:49
datsun180bhub_cap: we may actually not be allowed to do that16:50
datsun180bsomething something leadership decision16:50
cp16nethub_cap: you using them for target practice?16:50
cp16net:-P16:50
pdmarshub_cap: that's the milestone? i can't change it on a bug i created16:50
greghillare there any bugs that everyone's like "man, I wish someone would fix that stupid thing"?  kinda looking for something to work on next, but I have no context16:50
datsun180bimagine if just any walk-in could start declaring what was and wasn't going into our i316:51
hub_cappdmars: srsly?!16:55
openstackgerritDenis M. proposed a change to openstack/trove: Updating unit test assertions  https://review.openstack.org/6172516:55
hub_cappdmars: i assume you are also mucking w a bug in /trove not the other 2 in launchpad right?16:56
pdmarsyeah16:56
pdmarshttps://bugs.launchpad.net/trove/+bug/125997616:56
hub_capcool... i wanted to make sure i was telling the guy the right thing :)16:56
*** haomaiwa_ has quit IRC16:58
*** demorris has quit IRC16:58
*** rongze_ has quit IRC17:01
datsun180bhub_cap / greghill https://bugs.launchpad.net/trove/+bug/1246599 looks like it's straightforward, i bet it's just the client not recognizing pagination though i'm not certain why the result is 8 and not 10 items17:02
*** dukhlov_ has quit IRC17:02
openstackgerritRobert Myers proposed a change to openstack/trove: Moving storage logic out of backup runner class  https://review.openstack.org/5670217:03
greghilldatsun180b: that link doesn't work for me17:03
datsun180bgreghill: probably because it has some protected info and you may not be part of -drivers yet17:03
greghillah17:04
hub_capdatsun180b: trove database-list is supposed to paginate for u17:04
datsun180bright17:04
datsun180bso the first trick is to try to validate this bug and see why it's 8 and not 10 items for one17:04
Flying_Bondwho is  working on mongodb or cassandra??17:05
hub_capFlying_Bond: mongo = ikhudoshyn, c* = denis_makogon17:05
denis_makogonFlying_Bond, whazap17:06
denis_makogondid someone know how to enable vnc at devstack /17:08
datsun180bhub_cap: if that bug's no bueno for greg i might throw my granular-privileges BP at him barring more important work for him17:08
hub_capdatsun180b: The information on this page is private.17:08
hub_capwtf is that... thats teh bug listing17:08
hub_capdenis_makogon: not in here hehehe17:08
cp16netdenis_makogon: add vnc to the service list17:08
hub_capmaybe in openstack-dev ?17:08
datsun180bhub_cap: yeah, bug smitty include a snapshot of his client interaction and included some tokens17:08
hub_capsrsly cp16net its that easy?17:08
cp16netyeah17:08
hub_capdatsun180b: HAH17:09
cp16neti think thats the case for all services17:09
hub_captoken changin time17:09
cp16netjust like adding trove17:09
cp16netchange it in your localrc files17:09
hub_capcp16net: oh im dumb..17:09
hub_capdevstack17:09
cp16netyeah... but dont quote me on "vnc" being the name17:09
cp16neti think its n-vnc17:10
cp16netnot sure17:10
robertmyershub_cap: can you please look at this review: https://review.openstack.org/#/c/60612/17:10
robertmyerstox tests are killing me17:11
robertmyersbuffer overflow17:11
hub_capmy kid smells like fruit rollups17:11
datsun180bhub_cap: this should be alarming only if he hasn't been near any recently17:12
hub_caphe hasnt, hes not old enough yet17:12
openstackgerritRobert Myers proposed a change to openstack/trove: Simplify swift storage load logic  https://review.openstack.org/5779617:12
hub_capbut hes got these strawberry yogurt wafers17:12
hub_capso i think its that datsun180b17:12
datsun180bguess so17:12
datsun180bso unless you or amytron stop me i'm going to throw that granular-privs BP at greg and we're going to work through it17:13
datsun180byou have fifteen seconds to comply17:13
hub_capSlickNik: https://review.openstack.org/#/c/57780/17:13
*** SergeyLukjanov has joined #openstack-trove17:14
hub_capdatsun180b: NOOOO17:15
hub_capjk im impartial17:15
*** demorris has joined #openstack-trove17:16
SushilKMmichael can u check this one .... https://review.openstack.org/#/c/60668/17:16
SushilKMcleans up the gitignore for editor related files :)17:16
hub_capvipul: has anyone picked up the "Default datastore" yet?17:19
denis_makogonhub_cap, no17:19
denis_makogonbug is still Unassigned17:19
denis_makogonhub_cap, could you please take a look at it again https://review.openstack.org/#/c/61169/ ?17:21
*** decede has quit IRC17:23
*** Flying_Bond has quit IRC17:28
*** amytron has quit IRC17:28
*** amytron has joined #openstack-trove17:29
*** amcrn has joined #openstack-trove17:30
*** doug_shelley66 has joined #openstack-trove17:32
*** dougshelley66 has quit IRC17:33
*** Flying_Bond has joined #openstack-trove17:34
openstackgerritRobert Myers proposed a change to openstack/trove: Moving storage logic out of backup runner class  https://review.openstack.org/5670217:37
openstackgerritRobert Myers proposed a change to openstack/trove: Simplify swift storage load logic  https://review.openstack.org/5779617:41
*** ashestakov has joined #openstack-trove17:50
*** decede has joined #openstack-trove17:51
openstackgerritA change was merged to openstack/trove: use_stderr should be False for tox tests  https://review.openstack.org/6061218:04
*** achampion has quit IRC18:08
openstackgerritA change was merged to openstack/python-troveclient: Add -U to pip install command in tox.ini  https://review.openstack.org/6151018:09
openstackgerritA change was merged to openstack/python-troveclient: Updated from global requirements  https://review.openstack.org/6125218:09
SlickNikhub_cap: sushilkm has a valid comment on that patch (https://review.openstack.org/#/c/57780/)18:10
openstackgerritA change was merged to openstack/trove: Add -U to pip install command in tox.ini  https://review.openstack.org/6151518:10
SlickNikIs kevinconway around  to fix it? If not, I can upload a new change-set to unblock.18:11
*** decede has quit IRC18:11
hub_capkevinconway: around? ^ ^18:18
denis_makogonif noone wouldn't mind, i'll take this one https://bugs.launchpad.net/trove/+bug/122098918:20
denis_makogonhub_cap, any luck with testing cassandra ?18:21
denis_makogondatsun180b, around ?18:22
denis_makogonguys, i'd like to hear your thoughts about logging in conductor18:23
denis_makogonwould it be better to avoid logging whole payload18:23
denis_makogoni suggest log only instance id and DBInstance status18:24
denis_makogonit would be shorter18:24
*** decede has joined #openstack-trove18:25
SushilKMslicknik: have rebased https://review.openstack.org/#/c/60564/18:25
SushilKMi have*18:25
hub_capdenis_makogon: not yet, id like to get a review merged that makes datastores easier to deal with18:25
hub_capin -integration18:25
hub_capthen u will need to update your -integration review18:26
datsun180bdenis_makogon: you don't have to ask me to modify conductor18:26
datsun180byou just have to ask all of us18:26
hub_capLOL datsun180b18:27
hub_capim not concerned about logging until someone notices a production issue18:27
denis_makogonhub_cap, oh, i see, you mean kevinconway 's review, ok18:27
hub_capdenis_makogon: oh yes, sorry..18:27
hub_capi shouldve posted the ### to u18:28
denis_makogonnp, i'll re-do it18:28
hub_capbut i think its lunch time for the rax crew18:28
datsun180bnoon thirty here18:28
hub_capdenis_makogon: u can probablyjust make your review depend on that, and modify it18:28
datsun180bfunny thing about noon, it's from latin for "ninth hour"18:28
datsun180bof course these are the same guys that call our tenth month "october" so what can you expect18:28
hub_caponce your code is rebased / dependency i can run the tests18:29
denis_makogonhub_cap, i thinks kevinconway 's review would be updated, i'll just wait18:29
hub_caplunchtober hour18:29
denis_makogondatsun180b, so, you are ok with changing logging in conductor ?18:29
openstackgerritA change was merged to openstack/python-troveclient: Fixed misspellings of common words  https://review.openstack.org/5987618:29
openstackgerritA change was merged to openstack/trove: Fixed misspellings of common words  https://review.openstack.org/5987718:29
datsun180bdenis_makogon: it's not up to me18:29
denis_makogondatsun180b, ok18:29
*** decede has quit IRC18:32
hub_capdenis_makogon: like i said, if there is a reason to fix it, fix it18:32
hub_capbut "i dont like logging the payload" is not a good reason imo18:32
hub_capif "after X nodes in a production cluster this logging causes the infrastructure to go down", then sure18:33
hub_capchange it18:33
denis_makogonhub_cap, i mean payload it too big, after day of running conductor i got huge log, and it weights a lot18:33
denis_makogoni saw that biggest messages were payloads18:33
hub_capdefine _huge_ log18:33
denis_makogon100M18:34
datsun180bisn't logrotate a thing18:34
hub_capheh im not sure 100m is a big log18:34
denis_makogonit's only dev. thing18:34
denis_makogoni researched and saw that instance status is really  matters18:35
denis_makogonnow i ran conductor with 1>/dev/null 2>/dev/null18:35
hub_cap:)18:36
* datsun180b dusts hands off18:36
datsun180blooks like you're done18:36
denis_makogonmaybe =)18:37
kevinconwayhub_cap: hey, problem with the integration commit i saw18:37
denis_makogonbut really, would it be better to log id and status ?18:37
kevinconwayi'll check it out18:37
datsun180bdenis_makogon: probably still want to mention the method being called if you really want to whittle it down18:37
datsun180bthough log.debug will give you that anyway18:37
hub_capdenis_makogon: it would be best to code payloads in a debug status, and if u dont care about them, set the log to info18:38
hub_capthere is a way to do it in configs18:38
denis_makogoni know18:38
hub_capwouldnt that be cleaner than 1>/dev/null ?18:38
* hub_cap assumes that line is a debug stmt18:39
*** achampion has joined #openstack-trove18:39
datsun180bconductor does already consume its own config file18:39
datsun180bYOU'RE WELCOME18:39
hub_caplol datsun180b18:39
hub_capdefault_log_levels=datsun180b=ERROR18:40
openstackgerritKevin Conway proposed a change to openstack/trove-integration: Add support for kick-starting new datastores  https://review.openstack.org/5778018:40
hub_capkevinconway: thx. lets get this +2'd and merged so that i can test redis and cassandra :D18:40
hub_capSlickNik: ^ ^ (after robots of course)18:41
kevinconwaylooks like it was an oversight on naming the variables18:41
SlickNikyup, keeping an eye on it.18:41
SlickNikThanks for the fix, kevinconway.18:41
denis_makogondatsun180b, verbose, debug paremeters are not in conductor.sample, review hadn't been landed yet =)18:41
*** yogesh has joined #openstack-trove18:42
denis_makogondatsun180b, https://review.openstack.org/#/c/59890/18:42
*** Flying_Bond has quit IRC18:43
datsun180bi don't understand, do you want it verbose or don't you18:44
denis_makogondatsun180b, i want to make it configurable, that is what amcrn done with this patch18:45
*** Flying_Bond has joined #openstack-trove18:45
datsun180bbut it is configurable18:45
datsun180bsee how he's just updating the config file18:45
datsun180bthe one that's already consumed by conductor18:45
*** decede has joined #openstack-trove18:46
hub_capSlickNik: kevinconway http://tinyurl.com/lp4bz7u18:46
*** roadnick has joined #openstack-trove18:48
roadnickis there someone here who can help me with an installation question?18:49
hub_caproadnick: yup ask away18:49
roadnickcool, thanks.  I'm trying to install on Ubuntu 12.04 using Devstack, but not on HPCloud or RAX.18:49
roadnickshould that be possible?18:50
datsun180bdon't see why not, my vm's 12.0418:50
hub_capsure18:50
hub_capmost of the guys use local vms to dev on18:50
hub_capand those are just (they start as) vanilla lts18:51
roadnickI'm getting errors where it can't find two plexapp.com packages18:51
datsun180bi've got a vagrant file and everything if you like18:51
denis_makogongrapex, ping18:51
hub_capplexapp?18:51
hub_capthe plex media server?18:51
roadnickW:  Failed to fetch http://www.plexapp.com/repo/dists/precise/main/binary-amd64/Packages 404 Not Found18:51
*** decede has quit IRC18:51
hub_capwha?! my guess is that you have set up your apt repo to grab from there18:52
roadnickIt ends with E: Some index files failed to download. They ahve been ignored, or old ones used instead.18:52
hub_capand likely set it up improperly18:52
datsun180byeah, that doesn't look like the defaults18:52
hub_capjust do a sudo apt-get update18:52
hub_capsee if it fails18:52
roadnickchecking...18:52
hub_capif so your /etc/apt/sources.list (or something in the sources.list.d/*.list) has that plexapp line in it18:52
roadnicksame error.  OK, so it's not a trove issue. :(18:53
hub_capu will find plenty of trove issues18:53
roadnickright, ok, so I'll edit that18:53
hub_capbut this is not a trove issue :P18:53
roadnick:)18:53
hub_capand welcome18:53
roadnicksave me some time:  is it possible to get this to installa nd work?18:53
hub_capto the trove club18:53
roadnickha!  thanks. :)18:53
hub_capsure we do it all day long18:53
denis_makogongrapex, https://rdjenkins.dyndns.org/job/Trove-Gate/1810/consoleFull18:53
hub_capu might need a fresh vm18:53
denis_makogongrapex, 13:01:06 trove.tests.api.instances.CreateInstanceQuotaTest.test_update_quota_volume_should_fail_volume_not_supportedSKIP: Volume support needs to be disabled18:54
hub_capor u can use this roadnick18:54
datsun180boh it's so refreshing to be met with that cert warning every time18:54
hub_caphttps://github.com/openstack/trove-integration18:54
hub_capthats how the devs set up their environment.. it just does set up devstack18:54
* datsun180b coughcoughVAGRANTcough18:54
hub_capand some other stuff for testing18:54
hub_capoh even better18:54
cp16nethttp://gifs.gifbin.com/032011/reverse-1298985833_maze-scare-reaction-punching-hole-through-monitor.gif18:54
hub_capdatsun180b: u can take it from here18:54
cp16netthats how i setup my env18:54
datsun180bwell vagrant costs money18:54
hub_caplol cp16net18:55
roadnickthat does seem to have solved that problem, thanks so much.18:55
hub_caproadnick: coolness18:55
roadnicknow we'll see how far I get otherwise.18:55
hub_caproadnick: just so u know18:55
cp16netroadnick: welcome18:55
roadnickThanks so much, all!18:55
cp16net:-P18:55
datsun180bso i don't know if the right way to start is "Welcome to Qarth^H^H^H^H^HTrove! Please insert $80 to continue."18:55
hub_capthere is some stuff in trove-integration thats not in devstack proper18:55
hub_capdatsun180b: nice ;)18:55
roadnickhub_cap:  thanks for the warning.18:55
datsun180bLUCKY for you I also have a wiki article detailing exactly how I build my VMs18:56
hub_capso u will get 80% there w/o trove-integration scripts, and still have th build a image and set up datastores using trove-manage commands18:56
hub_capdatsun180b: yes link that plz18:56
datsun180bhttps://wiki.openstack.org/wiki/Trove-with-VMF5 that's the one18:56
roadnickyes, please18:56
hub_capquestions welcome18:56
datsun180bshort version is: /opt/stack/trove; /opt/stack/python-troveclient; /home/you/trove-integration/scripts; cd int*/scripts; ./redstack install18:57
amcrnsometime pre-2014 i need to publicly publish these trove puppet scripts18:58
roadnickok, well, filled up my drive so I guess I will be back later. :)18:59
roadnickthanks all!18:59
cp16nethub_cap: cats on everything?18:59
datsun180broadnick: let us know if you have any problems or questions18:59
roadnickwill do.  thanks again!19:00
*** roadnick has quit IRC19:00
cp16nethttp://2.bp.blogspot.com/_qEPgXPhoEEs/THJ4d2k6XrI/AAAAAAAAAnI/4pVfNMrYtg8/s320/animated-cat.gif19:00
pdmarsoh man, now i have that in my irc19:00
denis_makogongrapex, https://bugs.launchpad.net/trove/+bug/117900919:03
*** Flying_Bond has quit IRC19:05
*** decede has joined #openstack-trove19:09
*** freyes has joined #openstack-trove19:11
hub_capthat animated cat is amazing19:16
cp16net:)19:17
hub_caphttp://2.bp.blogspot.com/_qEPgXPhoEEs/THJ4d2k6XrI/AAAAAAAAAnI/4pVfNMrYtg8/s320/animated-cat.gif19:17
pdmarsmind blowing, and i mean that literally19:17
pdmarsdamnit hub_cap19:17
hub_capbetter pdmars ? http://2.bp.blogspot.com/_qEPgXPhoEEs/THJ4d2k6XrI/AAAAAAAAAnI/4pVfNMrYtg8/s320/animated-cat.gif19:17
cp16netjust stare at it for the rest of the day19:17
pdmarshub_cap: i hate you19:17
hub_capheh19:17
openstackgerritRobert Myers proposed a change to openstack/trove: Paginate backup list api  https://review.openstack.org/6035819:20
cp16nethttp://www.groonk.net/blog/wp-content/uploads/2011/03/tumblr_li3yehH0WN1qz6vfao1_500.gif RUN!19:20
openstackgerritA change was merged to openstack/trove-integration: Remove erroneous logfile in scripts directory  https://review.openstack.org/6165419:20
robertmyershub_cap: did you wash that cat before you put it in the channel?19:20
SushilKMregarding Denis checkin https://review.openstack.org/#/c/61169/19:21
hub_caphell yes i did robertmyers, its hostname was poopoo, and it was invalid19:21
cp16netit takes a day to wash the cat...19:21
SushilKMthere is change only in 1 file dont we need to use testtools on all the tests19:21
hub_capcp16net: that was like 1 hr dude whatever19:21
hub_capi BS'd the other 7hrs19:21
cp16netthat cat must have been really clean....19:22
cp16nethttp://lh3.ggpht.com/lubos.motl/SNzHOSzB_JI/AAAAAAAABCo/hQHOSzwUfcY/animated-quartic.gif19:23
hub_capthat cat hurled yesterday cp16net :/19:23
* cp16net mind blown19:23
hub_capforeal thats cool19:24
cp16neti had to find some gifs for my PR i made and found these kinda cool ;)19:25
openstackgerritA change was merged to openstack/trove: Conductor Does Not Default to Verbose/Debug Logs  https://review.openstack.org/5989019:28
*** vipul is now known as vipul-away19:30
openstackgerritA change was merged to openstack/python-troveclient: Fixes trove help command  https://review.openstack.org/6175619:31
openstackgerritA change was merged to openstack/trove: Enabling H403, H702 rules  https://review.openstack.org/6056419:32
greghillcan a non-rackspace core person give a second look at https://review.openstack.org/#/c/61505/ ?  (insert swearing and yelling)19:32
openstackgerritA change was merged to openstack/trove: Updates trove-guestagent.conf.sample  https://review.openstack.org/6171319:32
openstackgerritA change was merged to openstack/trove: Removing IDE specific ignores  https://review.openstack.org/6066819:33
openstackgerritSushil Kumar proposed a change to openstack/trove-integration: Fixes .gitignore  https://review.openstack.org/6185319:35
SushilKMhey hub_cap, there are multiple gitignore files in trov-integration project19:36
SushilKMhave pushed the patchset to move all of them into one single https://review.openstack.org/#/c/61853/19:37
openstackgerritA change was merged to openstack/trove: Fix mysqldump backup and restore  https://review.openstack.org/5923419:42
*** ashestakov has quit IRC19:42
openstackgerritSushil Kumar proposed a change to openstack/python-troveclient: Updates README  https://review.openstack.org/5883419:43
*** SushilKM__ has joined #openstack-trove19:44
greghillis there a precedent for migrating data from one table to another in a migrate_repo script?  I don't see any prior examples but the ticket I'm on suggests doing so, so I wondered if that was kosher or not19:44
*** vipul-away is now known as vipul19:44
*** SushilKM has quit IRC19:46
*** SergeyLukjanov has quit IRC19:46
*** SergeyLukjanov_ has joined #openstack-trove19:47
*** SergeyLukjanov_ has quit IRC19:48
*** SergeyLukjanov has joined #openstack-trove19:48
*** amrith has joined #openstack-trove19:54
greghilllooking at it more, the data is going to be vendor-specific, so automatic migration doesn't seem to be a sane approach as we can't possibly know what every vendor populated in this table19:56
*** Barker has quit IRC19:57
hub_capgreghill: thats a good point greghill. vipul was the original bug reporter, maybe he has more infos19:59
*** Barker has joined #openstack-trove20:00
vipulgreghill: are you referring to the datastore_id fkey issue?20:00
vipulSo the problem can be solved in a couple of ways.. either the code needs to allow a NULL to come back for datastore when fetching an Instance20:01
vipulor if it is a required thing, then it better have some 'default' value that gets set during a db migration20:01
kevinconwayshould it default to the default datastore?20:02
vipulwell a default datastore may not exist, since it's a manual step today -- but the code could handle it that way20:03
greghillI'm leaning toward not making it required since each vendor will have to figure out how they want to populate those tables20:03
vipulgreghill: yea i'd be ok with that.. so only 'new' instances after the upgrade would have a datastore returned20:04
greghillI'm not sure how that might impact consumers of the API or not, so I'm digging more20:04
*** jasonb365 has quit IRC20:04
greghillright, or each vendor can figure out how to populate the data for old instances20:04
greghillI don't see any code referencing those fields at all, but maybe my grep-fu isn't strong enough yet20:05
*** SnowDust has joined #openstack-trove20:06
*** denis_makogon_ has joined #openstack-trove20:08
vipuli recall failure being around here : https://github.com/openstack/trove/blob/master/trove/instance/models.py#L12720:08
*** plodronio has left #openstack-trove20:10
*** denis_makogon has quit IRC20:10
*** denis_makogon_ is now known as denis_makogon20:10
*** dmakogon_ has joined #openstack-trove20:11
*** vipul is now known as vipul-away20:13
*** vipul-away is now known as vipul20:13
*** SnowDust has quit IRC20:14
greghillyep, I have a change in place locally, just need to test it20:14
SushilKM__hey hub_cap and slicknik, please review https://review.openstack.org/#/c/58834/20:18
denis_makogonhi, vipul, could you please add desription of it to bug-report ?20:18
denis_makogonvipul, also would be nice to hear how did you found where bug occures20:20
SushilKM__vipul, denis u too invited, please :)20:21
denis_makogonvipul, if bug would be fixed, we would need to make backport to stabe/havana release20:27
*** vipul is now known as vipul-away20:28
*** yogesh has quit IRC20:30
openstackgerritRobert Myers proposed a change to openstack/trove: Moving storage logic out of backup runner class  https://review.openstack.org/5670220:30
*** SushilKM__ has quit IRC20:33
openstackgerritRobert Myers proposed a change to openstack/trove: Properly mark a backup as FAILED if the process failed  https://review.openstack.org/5928320:35
robertmyersfun with rebase20:36
robertmyersI think I'm up todate now let's hope the bots agree20:36
*** jasonb365 has joined #openstack-trove20:38
*** doug_shelley66 has quit IRC20:48
*** dougshelley66 has joined #openstack-trove20:48
*** vipul-away is now known as vipul20:49
*** jcooley_ has quit IRC20:49
*** yogesh has joined #openstack-trove20:49
vipuldenis_makogon: this bug is only related to datastores being introduced20:57
vipuldenis_makogon: I don't think a backport is necessary20:58
denis_makogonvipul, lets talk about that when bug-fix would be released20:58
denis_makogonvipul, could you assign but on me ?20:58
denis_makogonvipul, or give me a link to it20:59
vipuldenis_makogon: I think greghill is already fixing it20:59
denis_makogonvipul, yes ?20:59
denis_makogonvipul, then ok20:59
*** datsun180b has quit IRC20:59
vipulhere it is https://bugs.launchpad.net/trove/+bug/1259642 denis_makogon20:59
*** amrith has quit IRC21:00
*** Barker has quit IRC21:04
*** rwsu has quit IRC21:05
*** amrith has joined #openstack-trove21:05
*** Barker has joined #openstack-trove21:09
*** freyes has quit IRC21:16
greghillI am already working on it21:16
greghillsorting out a test case to cover it at the moment21:17
openstackgerritPaul Marshall proposed a change to openstack/trove: Acknowledge Nova VERIFY_RESIZE as RESIZE state  https://review.openstack.org/6034621:26
*** SergeyLukjanov is now known as _SergeyLukjanov21:31
*** reed has joined #openstack-trove21:33
*** amrith has quit IRC21:35
*** demorris has quit IRC21:36
openstackgerritA change was merged to openstack/trove: check for invalid hostnames on update_attributes  https://review.openstack.org/6150521:40
*** rnirmal has quit IRC21:45
*** SergeyLukjanov has joined #openstack-trove21:45
*** demorris has joined #openstack-trove21:55
*** glucas has quit IRC22:02
*** jasonb365_ has joined #openstack-trove22:04
*** dougshelley66 has quit IRC22:04
cp16netvipul:  you there?22:05
*** jasonb365 has quit IRC22:05
*** jasonb365_ is now known as jasonb36522:05
cp16netor other hp-ers22:05
vipulcp16net: here wassup22:06
cp16netwhat was the purpose of this taskmanager falg?notification_service_id = mysql:2f3ff068-2bfb-4f70-9a9d-a6bb65bc084b22:06
cp16netspecifically the id after the service22:06
hub_capask juice i think he did it22:06
vipuli belive the reason is that diffrent datastores can be different services22:06
hub_capoh right, it can just as easily be22:07
vipulso billing could potentially have different rates based on service_id22:07
cp16netok i was looking at the mongo review and it has the same id22:07
hub_capmysql:mysql22:07
cp16neti figured that really should not be the case22:07
hub_capoh well thats not good!22:07
juicecp16net: that was something we added to uniquely identify a service to a consumer of the message such as a billing system...for example...hypothetically :)22:08
cp16netgotcha :)22:09
*** abramley has quit IRC22:10
*** pdmars has quit IRC22:11
hub_capanother example... the nsa22:11
juiceha ha22:12
*** achampion has quit IRC22:12
*** OpenStack1 has joined #openstack-trove22:31
*** denis_makogon has quit IRC22:33
*** abramley has joined #openstack-trove22:43
*** Barker has quit IRC22:47
*** kevinconway has quit IRC22:48
*** OpenStack1 has quit IRC23:04
*** yogesh has quit IRC23:06
*** robertmyers has quit IRC23:06
*** jcru has quit IRC23:17
*** dougshelley66 has joined #openstack-trove23:26
*** jasonb365 has quit IRC23:26
*** amytron has quit IRC23:30
*** plodronio has joined #openstack-trove23:46
*** greghill has quit IRC23:56
*** SergeyLukjanov has quit IRC23:57

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