Monday, 2013-12-02

*** rongze has joined #openstack-trove00:18
*** matsuhashi has joined #openstack-trove00:19
*** rongze has quit IRC00:23
*** matsuhashi has quit IRC00:30
*** matsuhashi has joined #openstack-trove00:30
*** matsuhashi has quit IRC00:35
*** jcooley_ has quit IRC00:42
*** jcooley_ has joined #openstack-trove00:52
*** jcooley_ has quit IRC00:58
*** jcooley_ has joined #openstack-trove00:59
*** jcooley_ has quit IRC01:04
*** matsuhashi has joined #openstack-trove01:05
*** yogesh_ has quit IRC01:16
*** nosnos has joined #openstack-trove01:33
*** mmcdaris has quit IRC01:49
*** rongze has joined #openstack-trove02:01
*** adrian_otto has joined #openstack-trove02:14
*** amytron has joined #openstack-trove02:26
*** yogeshmehra has joined #openstack-trove02:40
*** erkules_ has quit IRC02:46
*** yogeshmehra has quit IRC02:47
*** erkules_ has joined #openstack-trove02:49
*** erkules has joined #openstack-trove02:50
*** erkules_ has quit IRC02:50
*** matsuhashi has quit IRC02:59
*** matsuhashi has joined #openstack-trove03:02
*** jcooley_ has joined #openstack-trove03:10
*** robertmyers has joined #openstack-trove03:12
*** shakayumi has joined #openstack-trove03:46
*** grapex has joined #openstack-trove03:49
*** grapex has joined #openstack-trove03:49
*** jcooley_ has quit IRC03:59
*** matsuhashi has quit IRC04:02
*** matsuhas_ has joined #openstack-trove04:05
*** grapex has quit IRC04:14
*** adrian_otto has quit IRC04:16
*** robertmyers has quit IRC04:39
*** rongze has quit IRC04:57
*** adrian_otto has joined #openstack-trove04:59
*** jcooley_ has joined #openstack-trove05:26
*** rongze has joined #openstack-trove05:27
*** jcooley_ has quit IRC05:34
*** jcooley_ has joined #openstack-trove05:34
*** nosnos_ has joined #openstack-trove05:35
*** rongze has quit IRC05:36
*** nosnos has quit IRC05:37
*** nosnos_ has quit IRC05:37
*** nosnos has joined #openstack-trove05:38
*** achampion has quit IRC05:50
*** jcooley_ has quit IRC05:54
*** jcooley_ has joined #openstack-trove05:55
*** matsuhas_ has quit IRC05:55
*** matsuhashi has joined #openstack-trove05:56
*** adrian_otto has quit IRC06:00
*** rongze has joined #openstack-trove06:02
*** rongze has quit IRC06:11
*** rongze has joined #openstack-trove06:11
*** achampion has joined #openstack-trove06:15
*** amytron has quit IRC06:29
*** achampion has quit IRC06:31
*** SushilKM has joined #openstack-trove06:45
*** jcooley_ has quit IRC06:48
*** jcooley_ has joined #openstack-trove06:49
*** jcooley_ has quit IRC06:52
*** jcooley_ has joined #openstack-trove06:52
*** yogesh has joined #openstack-trove06:55
*** matsuhashi has quit IRC06:56
*** matsuhashi has joined #openstack-trove06:56
*** matsuhashi has quit IRC06:57
*** matsuhashi has joined #openstack-trove06:57
*** nosnos_ has joined #openstack-trove06:57
*** nosnos has quit IRC06:57
*** jcooley_ has quit IRC07:12
*** jcooley_ has joined #openstack-trove07:12
*** yogesh has quit IRC07:12
*** yogesh has joined #openstack-trove07:13
*** jcooley_ has quit IRC07:17
*** yogesh has quit IRC07:17
*** jcooley_ has joined #openstack-trove07:18
openstackgerritSushil Kumar proposed a change to openstack/python-troveclient: Updates README  https://review.openstack.org/5883407:19
*** SushilKM has quit IRC07:21
openstackgerritSushil Kumar proposed a change to openstack/python-troveclient: Updates README  https://review.openstack.org/5883407:21
*** achampion has joined #openstack-trove07:21
*** jcooley_ has quit IRC07:23
*** jcooley_ has joined #openstack-trove07:23
*** achampion has quit IRC07:26
*** zigo has quit IRC07:27
*** zigo has joined #openstack-trove07:27
*** yogesh has joined #openstack-trove07:32
*** jcooley_ has quit IRC07:32
*** jcooley_ has joined #openstack-trove07:32
*** jcooley_ has quit IRC07:34
*** jcooley_ has joined #openstack-trove07:36
*** yogesh_ has joined #openstack-trove07:36
*** yogesh__ has joined #openstack-trove07:39
*** yogesh has quit IRC07:39
*** yogesh_ has quit IRC07:42
*** yogesh has joined #openstack-trove07:42
*** jcooley_ has quit IRC07:43
*** yogesh_ has joined #openstack-trove07:44
*** jcooley_ has joined #openstack-trove07:44
*** yogesh__ has quit IRC07:45
*** yogesh__ has joined #openstack-trove07:46
*** yogesh has quit IRC07:47
*** yogesh__ has quit IRC07:47
*** yogesh has joined #openstack-trove07:47
*** yogesh has quit IRC07:48
*** yogesh has joined #openstack-trove07:48
*** yogesh_ has quit IRC07:49
*** yogesh has quit IRC07:50
*** yogesh has joined #openstack-trove07:50
*** jcooley_ has quit IRC07:50
*** jcooley_ has joined #openstack-trove07:51
*** yogesh_ has joined #openstack-trove07:52
*** zigo has quit IRC07:55
*** yogesh has quit IRC07:55
*** zigo has joined #openstack-trove07:55
*** jcooley_ has quit IRC07:58
*** jcooley_ has joined #openstack-trove08:00
*** jcooley_ has quit IRC08:08
*** jcooley_ has joined #openstack-trove08:08
*** matsuhashi has quit IRC08:10
*** flaper87|afk is now known as flaper8708:15
*** matsuhashi has joined #openstack-trove08:16
*** yogesh_ has quit IRC08:16
*** yogesh has joined #openstack-trove08:16
*** jcooley_ has quit IRC08:21
*** achampion has joined #openstack-trove08:22
*** achampion has quit IRC08:27
*** nosnos_ has quit IRC08:27
*** yogesh has quit IRC08:27
*** nosnos has joined #openstack-trove08:28
*** yogesh has joined #openstack-trove08:28
*** yogesh_ has joined #openstack-trove08:30
*** yogesh_ has quit IRC08:32
*** yogesh_ has joined #openstack-trove08:32
*** yogesh has quit IRC08:32
*** yogesh_ has quit IRC08:38
*** yogesh has joined #openstack-trove08:38
*** rongze has quit IRC08:44
*** zigo has quit IRC09:06
*** zigo has joined #openstack-trove09:06
*** yogeshmehra has joined #openstack-trove09:13
*** shakayumi has quit IRC09:14
*** matsuhashi has quit IRC09:14
*** denis_makogon has joined #openstack-trove09:24
*** yogesh has quit IRC09:30
*** matsuhashi has joined #openstack-trove09:31
*** yogesh_ has joined #openstack-trove09:34
*** yogesh_ has quit IRC09:42
*** yogesh has joined #openstack-trove09:42
*** yogesh has quit IRC09:47
*** yogeshmehra has quit IRC10:37
*** rongze has joined #openstack-trove10:55
*** matsuhashi has quit IRC11:22
*** zigo_ has joined #openstack-trove11:56
*** zigo has quit IRC11:57
*** nosnos has quit IRC12:22
*** rongze has quit IRC12:23
*** matsuhashi has joined #openstack-trove12:57
*** pdmars has joined #openstack-trove13:05
*** matsuhashi has quit IRC13:06
*** matsuhashi has joined #openstack-trove13:07
*** pdmars has quit IRC13:07
*** pdmars has joined #openstack-trove13:11
*** matsuhashi has quit IRC13:11
*** matsuhashi has joined #openstack-trove13:13
*** matsuhashi has quit IRC13:17
*** rongze has joined #openstack-trove13:25
*** jcru has joined #openstack-trove14:04
*** radez_g0n3 is now known as radez14:22
*** kevinconway has joined #openstack-trove14:51
*** TheRealBill has left #openstack-trove14:52
*** adrian_otto has joined #openstack-trove14:54
*** adrian_otto has quit IRC14:55
*** adrian_otto has joined #openstack-trove14:56
*** jcru_ has joined #openstack-trove14:57
*** robertmy_ has joined #openstack-trove14:57
*** amytron has joined #openstack-trove14:59
*** jcru has quit IRC15:00
*** grapex has joined #openstack-trove15:01
*** grapex has quit IRC15:03
*** grapex has joined #openstack-trove15:03
openstackgerritDenis M. proposed a change to openstack/trove: RootReport through Conductor  https://review.openstack.org/5941015:06
*** Barker has joined #openstack-trove15:06
*** Barker has quit IRC15:08
*** Barker has joined #openstack-trove15:09
openstackgerritDenis M. proposed a change to openstack/trove: RootReport through Conductor  https://review.openstack.org/5941015:10
*** Barker has quit IRC15:13
*** rwsu has joined #openstack-trove15:14
*** Barker has joined #openstack-trove15:15
*** achampion has joined #openstack-trove15:21
*** adrian_otto has quit IRC15:27
*** shakayumi has joined #openstack-trove15:33
kevinconwayhey did any of you folks see https://review.openstack.org/#/c/58848/215:37
*** datsun180b has joined #openstack-trove15:39
*** datsun180b is now known as datsun_F40PH15:40
denis_makogonkevinconway, yeah, i saw15:49
denis_makogonkevinconway, but we are not using tempest15:50
denis_makogonat least now15:50
*** rnirmal has joined #openstack-trove15:55
*** Barker has quit IRC15:58
*** demorris has joined #openstack-trove15:59
*** demorris has quit IRC16:03
*** demorris has joined #openstack-trove16:03
*** jcooley_ has joined #openstack-trove16:04
*** Barker has joined #openstack-trove16:06
*** SushilKM has joined #openstack-trove16:14
*** jcooley_ has quit IRC16:33
*** plodronio has joined #openstack-trove16:34
*** tanisdl has joined #openstack-trove16:35
*** SushilKM has quit IRC16:46
*** haomaiwang has quit IRC16:46
*** haomaiwang has joined #openstack-trove16:47
*** Barker has quit IRC16:52
*** adrian_otto has joined #openstack-trove16:54
*** rongze has quit IRC16:56
*** adrian_otto has quit IRC17:01
*** jmontemayor has joined #openstack-trove17:07
*** Barker has joined #openstack-trove17:12
*** SnowDust has joined #openstack-trove17:19
SnowDustseems its full house today :-)17:19
SnowDusteveryone .. over the line17:20
denis_makogonyup17:20
SnowDusthey hey crazymac .. how r u17:21
*** SushilKM has joined #openstack-trove17:27
SushilKMHi Everyone17:27
SnowDusthi SushilKM17:27
SushilKMwanted to discuss regarding https://review.openstack.org/#/c/57962/17:27
SushilKMI was asked to discuss with Kevin, but it seems that he is in no hurry to get his change working .....17:28
*** rongze has joined #openstack-trove17:28
denis_makogoni'm fine17:30
denis_makogonSushilKM, hit kevinconway17:30
kevinconwaydenis_makogon: i would not recommend hitting me17:30
kevinconwaynot a good way to make friends17:30
SushilKMhey kevin17:31
SushilKMno hitting man17:31
denis_makogon#action SushilKM TODO: catch kevinconway and discuss everything about integration cmd stuff17:31
SushilKMjust pinging17:31
kevinconwaySushilKM: did i not update my review?17:31
kevinconwayi thought i did17:31
*** adrian_otto has joined #openstack-trove17:31
SushilKMbut gate failed on there and no activities since then @kevin17:31
kevinconwaySushilKM: oh i see. was on holiday...17:32
kevinconwaySushilKM: I will update it soon17:33
kevinconwayi see one of my other reviews expired too. need to revive that17:33
kevinconwaywhy doesn't openstack stop working when i stop working?17:33
SushilKMplease work on that kevin, this change is needed to make trove-integration to work for other datastores :)17:33
denis_makogonkevinconway, don't know17:33
SushilKMand u r the owner of this change17:34
hub_capdenis_makogon: those #action things only work when in a meeting :)17:34
denis_makogonSushilKM, new datastore would not get merged until tempest17:34
denis_makogonhub_cap, dog =/17:34
denis_makogon*doh17:34
SnowDust<kevinconway> why doesn't openstack stop working when i stop working ?  BLAME SANTA :D17:34
SnowDustLOL17:34
hub_caplol17:34
denis_makogonhub_cap, could you please take a look at https://review.openstack.org/#/c/59410/17:35
denis_makogonwow, i'm looking through solum meeting - they are insane)))17:35
hub_capsure denis_makogon17:35
hub_capdenis_makogon: what do u mean? brb17:36
SnowDustbrb !17:36
denis_makogonhub_cap, a lot of messages, hard do read and write anything17:36
SushilKMhey hub_cap, i was waiting for ur reviews on https://review.openstack.org/#/c/58203/17:37
openstackgerritCraig Vyvial proposed a change to openstack/trove: adding configuration group support  https://review.openstack.org/5316817:40
hub_capdenis_makogon: ahhhh ok... is it going on right now?17:42
denis_makogonhub_cap, yes17:42
denis_makogon#solum17:42
denis_makogonhub_cap, https://review.openstack.org/#/c/59042/17:43
hub_capdenis_makogon: sweet!!17:44
denis_makogonhub_cap, but noone didn't reviewed this =(17:44
hub_capya you have to ask people17:45
cp16nethub_cap: my turkey was amazing btw :-)17:45
hub_capcp16net: noice. i gotta go buy one now17:45
hub_caphow long did it cook?17:45
denis_makogonhub_cap, https://review.openstack.org/#/c/58845/17:46
denis_makogonhub_cap, https://review.openstack.org/#/c/54900/17:46
openstackgerritA change was merged to openstack/trove: Removes security_groups from _create_server_volume_heat signature  https://review.openstack.org/5820317:47
cp16net20lbs turkey took ~6 hours17:48
cp16neteven after all the coals burned up in about 4 hours17:48
cp16neti read it might take closer to 7 or 8 hours17:49
cp16netbut it was done in about 617:50
cp16neteveryone was impressed how juicy the dark meat was17:50
cp16neti made the claim like so many others say... "its because it was cooked on the green egg"17:51
hub_capdude your coal burned in 4 hrs?17:52
denis_makogoncp16net, i think we could update meeting agenda with Configuration discussion17:53
hub_capdenis_makogon: id like to discuss review 5490017:53
cp16nethub_cap: yeah it was crazy that it all burned up that fast17:54
cp16netmaybe its because it was smaller pieces17:54
cp16netdenis_makogon: we can17:54
hub_capya possible. i always take out of 2 bags of coal for a cook17:54
hub_cap*for a smoke*17:54
cp16netbe aware turkey absorbs lots of smoke17:55
hub_capthe company i buy the coal from has a small and large bag, but not the size of teh bag, its the size of the pieces in17:55
hub_capoya? did yours get oversmoked?17:55
cp16neti only put in like 3 small pieces of hickory for smoke and it was really smokey tasting17:55
hub_capi wouldntve have thought about that17:55
hub_capok do u use chips or pieces for smoke?17:56
cp16neti have pieces of wood i soak17:56
cp16neti did the chips a few times and they are gone fast so i bought a big bag of hickory pieces17:56
hub_capcool i find it hard to find the pieces out here, its all chips... gotta buy em online17:56
cp16netit lasts way longer17:57
hub_capfor sure17:57
cp16netmaybe too long...17:57
hub_capHAH17:57
cp16netat least for turkey17:57
cp16netmy mom wants me to cook a turkey for her when i come for christmas now17:57
openstackgerritSushil Kumar proposed a change to openstack/trove: Ignores swap files generated during file editing  https://review.openstack.org/5869817:57
denis_makogonguy, i haven't dine yet, you're killing me)))18:01
hub_capdenis_makogon: hahaha. i still need breakfast _and_ coffee18:03
hub_capim just drinking a water right now, so sad18:03
denis_makogonlol, cp16net i didn't rebase your patch, you just add them manually ?18:03
* hub_cap is out of coffee beans :)18:03
denis_makogonhub_cap, take a cup of tee18:04
denis_makogon*tea18:04
kevinconwayhub_cap: i gave up coffee a little over a year ago. never looked back.18:04
hub_capcp16net: lol yup. dude u need to do a ham for xmas18:04
denis_makogonkevinconway, what are your drugs ?))18:04
hub_capkevinconway: no way dude. im such a coffee nerd18:04
kevinconwaynah, good night's sleep is all i do now18:04
denis_makogon127 cups of espresso - lethal dose18:05
cp16net126 is how much i need them18:11
cp16net:-P18:11
cp16netoh i bought the stuff to make cafe su da now18:13
cp16neti'm excited about that18:13
cp16neti went to phonatic on saturday with the wife18:13
cp16neti wore my shirt and they didnt give me a discount :(18:13
denis_makogonsad =(18:14
datsun_F40PHwhat's the discount for wearing the shirt?18:14
openstackgerritSushil Kumar proposed a change to openstack/trove-integration: Updates .gitignore  https://review.openstack.org/5872518:15
*** openstackgerrit has quit IRC18:18
*** vipul is now known as vipul-away18:18
*** openstackgerrit has joined #openstack-trove18:18
*** vipul-away is now known as vipul18:18
*** SushilKM__ has joined #openstack-trove18:18
cp16neti thought if i had it on for small business saturday they would give me a discount18:18
cp16netbut that wasnt the case18:18
kevinconwaycp16net: you could probably get a great discount if you worked there18:19
denis_makogondoes any one analyze where else guestagent uses back-end connection ?18:19
cp16netyeah probably18:19
*** SushilKM has quit IRC18:19
*** yogesh has joined #openstack-trove18:19
hub_capcp16net: dude last time i wore the shirt he gave me free egg rolls18:21
hub_captell me how the sua da turns out cp16net18:21
hub_capdatsun_F40PH: there is a conductor review denis_makogon did, id like you to have a looksee plz. denis_makogon can u link it to him?18:22
denis_makogonhub_cap, datsun_F40PH https://review.openstack.org/#/c/59410/18:23
datsun_F40PHhave you ever seen Wrongfully Accused?18:23
datsun_F40PHLeslie Nielsen movie, it's wonderful18:23
datsun_F40PHhttp://www.youtube.com/watch?v=MjbUnn32_zU18:24
datsun_F40PHthat's about how i feel about conductor, especially when the chase takes to the woods18:24
*** adrian_otto1 has joined #openstack-trove18:24
*** adrian_otto has quit IRC18:24
denis_makogondatsun_F40PH, yup, i saw18:24
denis_makogonchase against train ?)))18:25
denis_makogoni could not find any other places where guest uses back-end connection18:26
*** esp has joined #openstack-trove18:27
denis_makogonhttps://review.openstack.org/#/c/59078/ - ikhudoshyn got this review, but int tests failing at WaitForRestore ...18:27
denis_makogoni don't know why18:27
hub_capdatsun_F40PH: hah yes ive seen it18:28
openstackgerritDenis M. proposed a change to openstack/trove: Removing back-end usage from guest code  https://review.openstack.org/5951018:32
denis_makogoni'll try to find out why does it failing, if this review would get two +2 then i'm done with guest and back-end interaction18:33
denis_makogondatsun_F40PH, thanks for reviewing18:34
*** shakayumi has quit IRC18:34
datsun_F40PHyeah that's goofy, i don't see a failure in the jenkins logs18:38
*** yogesh has quit IRC18:38
datsun_F40PHoh i see, it was captured in a weird way by the logs18:39
denis_makogonyes18:41
denis_makogonseems like a problem18:41
*** rongze has quit IRC18:42
denis_makogondatsun_F40PH, could you help me dealing with this bug18:42
*** cweid has joined #openstack-trove18:42
datsun_F40PHi've got to finish porting these changes to sneaky-pete18:42
datsun_F40PHi'd start by investigating the messages passed between taskman and guestagent and make sure they're not hiding any errors18:43
*** adrian_otto1 is now known as adrian_otto18:44
denis_makogondatsun_F40PH, i think this issue happening only while int tests18:46
*** yogesh has joined #openstack-trove18:46
datsun_F40PHall the more reason to look at the guestagent logs18:46
denis_makogonyes, i would try to do this18:46
*** amcrn has joined #openstack-trove18:47
datsun_F40PHsorry to be so terse, i have to get this done as soon as i can18:48
denis_makogondatsun_F40PH, oh, no problem, thanks for helping, anyway =)18:51
datsun_F40PHi did notice this morning that i had a resize test fail because the ib_logfiles were zero, not gone, but in retrying my tests that didn't happen again. must be a timing issue18:51
datsun_F40PHbut that was restart/reboot and not restore18:51
hub_capspeaking of restarts.. ill be back18:57
*** yogesh has quit IRC18:58
*** yogesh has joined #openstack-trove18:58
*** yogesh has quit IRC19:01
*** yogesh has joined #openstack-trove19:01
openstackgerritSteve Leon proposed a change to openstack/trove: Fixes guest-agent when DB connection string is not provided  https://review.openstack.org/5951919:01
*** jcooley_ has joined #openstack-trove19:06
*** rongze has joined #openstack-trove19:12
*** jcooley_ has quit IRC19:13
*** jcooley_ has joined #openstack-trove19:13
*** SushilKM__ has quit IRC19:13
*** jcooley_ has quit IRC19:14
*** yogesh has quit IRC19:14
*** yogesh has joined #openstack-trove19:14
denis_makogonSlickNik, hub_cap datsun_F40PH https://review.openstack.org/59519 - amazingly, tests are passed19:16
*** SushilKM__ has joined #openstack-trove19:17
denis_makogoncould some one re-approve this one https://review.openstack.org/59519 instead of this one ? https://review.openstack.org/#/c/59410/19:17
esmutedenis_makogon: I filed this bug https://bugs.launchpad.net/bugs/125701619:17
esmutei submitted a fixed but i just abandoned it. It seems you have a fix for it soon19:18
openstackgerritSushil Kumar proposed a change to openstack/trove-integration: Updates .gitignore  https://review.openstack.org/5872519:18
denis_makogonesmute, you duplicate  even bug-report19:18
SlickNikdenis_makogon: I think you have the review links pasted wrong. You meant to paste https://review.openstack.org/#/c/59510/, I think?19:19
*** yogesh has quit IRC19:19
denis_makogonesmute, https://bugs.launchpad.net/trove/+bug/125604619:19
denis_makogonSlickNik, yes, you are right)))19:19
esmutecool.. ill mark it as a dup19:19
denis_makogonesmute, ^^19:20
*** rongze has quit IRC19:20
datsun_F40PHSushilKM__: hey you're having problems with .swp files i see19:21
*** SushilKM has joined #openstack-trove19:21
*** SushilKM__ has quit IRC19:21
datsun_F40PHyou can do what i do, make the directories .vim/swap and .vim/backup, and then add "set backupdir=~/.vim/backup" and "set directory=~/.vim/swap"19:22
datsun_F40PHto your vimrc that is19:23
denis_makogondatsun_F40PH, problem gone, tests are working =))))19:23
*** yogesh has joined #openstack-trove19:25
*** SushilKM has quit IRC19:28
denis_makogonhub_cap, vipul, SlickNik, grapex, datsun_F40PH  - tests are completed, now it could be reviewed https://review.openstack.org/#/c/59510/19:34
*** tanisdl has quit IRC19:37
*** rongze has joined #openstack-trove19:49
*** rongze has quit IRC19:53
*** SnowDust has quit IRC19:54
*** rwsu has quit IRC19:55
*** rwsu has joined #openstack-trove19:56
openstackgerritCraig Vyvial proposed a change to openstack/trove-integration: .gitignore some extra generated files and test user creds for cli  https://review.openstack.org/5952719:59
*** Barker has quit IRC20:01
cp16nethub_cap:  its a simple one  :) https://review.openstack.org/#/c/5844120:06
hub_capcp16net: done20:07
openstackgerritDan Nguyen proposed a change to openstack/trove: Validate databases for user grants  https://review.openstack.org/5952920:08
*** tanisdl has joined #openstack-trove20:19
*** radez is now known as radez_g0n320:24
cp16nethub_cap: wow..20:27
cp16netok20:27
openstackgerritCraig Vyvial proposed a change to openstack/trove-integration: fake mode script needs to be updated after changes were merged in trove  https://review.openstack.org/5844120:28
hub_capmuch btr cp16net !20:31
cp16netthx20:32
*** mmcdaris has joined #openstack-trove20:34
*** Barker has joined #openstack-trove20:37
*** yogesh has quit IRC20:42
*** yogesh has joined #openstack-trove20:42
*** denis_makogon_ has joined #openstack-trove20:44
*** denis_makogon has quit IRC20:45
*** yogesh has quit IRC20:45
*** denis_makogon has joined #openstack-trove20:45
*** yogesh has joined #openstack-trove20:46
*** denis_makogon has quit IRC20:46
*** denis_makogon_ is now known as denis_makogon20:46
*** dmakogon_ has joined #openstack-trove20:46
denis_makogonesmute, ping20:46
esmuteHey denis_makogon20:46
denis_makogonesmute, i've read your comment on my patch20:47
denis_makogonesmute, topic is correct, because: 1. There is no correct bug-report for such patch20:47
denis_makogon2. Description covers all changes20:47
denis_makogon3. Commit message describse whole purose of making such changes20:48
esmutedenis_makogon: This one https://bugs.launchpad.net/trove/+bug/125604620:48
denis_makogonesmute, db connection is not a bug20:48
*** yogesh has quit IRC20:49
*** rongze has joined #openstack-trove20:49
esmutei think it is a bug because guest-agent shouldnt be connecting to the DB. And doing that it is causing it to fail20:49
denis_makogonesmute, db connection is feature, which wasn't covered by conductor review20:49
denis_makogonesmute, shouldn't = somehow it connects20:50
denis_makogonesmute, start script has lines of code which create connetion pull20:50
esmuteBut the biggest issue is that you have two patches doing the same thing: https://review.openstack.org/#/c/59510/ and https://review.openstack.org/#/c/59410/20:51
*** yogesh has joined #openstack-trove20:51
denis_makogonesmute, they are all mine20:51
denis_makogoni will abandon those one which not covers all db-connection cases20:52
esmutei'm confused as to why you have two patches for the same fix.20:52
denis_makogonsecond patch was some kind of test20:52
denis_makogonbut it finished successfuly20:52
denis_makogonthat is why i want to leave second one and abandon reviewed20:53
esmuteThat 'test' patch is about to merge. hub_cap +2'd it20:53
denis_makogonesmute, this one https://review.openstack.org/#/c/59510/ was test20:53
denis_makogonhttps://review.openstack.org/#/c/59510/ - it covers all cases20:54
denis_makogonthat is why i want it to leave20:54
*** rongze has quit IRC20:54
esmutedenis_makogon: Ok.. If it is a test or work_in_progress, it should be clearly stated in the commit or in the comment20:54
denis_makogonpatch description covers all changes20:55
denis_makogonplease read20:55
denis_makogonReason:20:55
denis_makogon - guestagent should be free from direct Trove back-end DB connection.20:55
denis_makogonChanges:20:55
denis_makogon - extending conductor API;20:55
denis_makogon - mysql guestaget service now reports through conductor;20:55
denis_makogon - removing from init script db_api initializing;20:55
esmutebut https://bugs.launchpad.net/trove/+bug/1256046 is clearly a bug and needs to be fixed. I suggest to submit a patch only containing the guest-agent change (not the root report feature)20:55
denis_makogon - tests.20:55
*** yogesh has quit IRC20:55
denis_makogonesmute, guestagent fix would not work without report_root20:56
*** yogesh has joined #openstack-trove20:56
denis_makogonbecase report_roor method writed to DB20:56
esmutedenis_makogon: In this case, https://review.openstack.org/#/c/59510/ should be abandon and instead, use https://review.openstack.org/#/c/59410/20:57
esmuteit is just confusing having two patches that address the same thing.20:57
denis_makogonesmute, i will abandon this one https://review.openstack.org/#/c/59410/20:58
esmutealso link the bug number in the commit message: https://bugs.launchpad.net/trove/+bug/125604620:58
denis_makogonesmute, this is not a bug20:58
*** yogesh has quit IRC20:58
*** yogesh has joined #openstack-trove20:58
esmutedenis_makogon: Why not? Guest agent is using conductor. So it doesnt need to connect to the DB anymore. If the guest-agent still tries to connect to the DB, that is a bug21:00
denis_makogonesmute, GA uses conductor for backups and statuses21:00
denis_makogonesmute, please read Ed's review commit description21:01
denis_makogonesmute, Previously, instances updated their status by21:04
denis_makogonupdating the database on the host directly.21:04
denis_makogonNecessarily, each instance would need access to the21:04
denis_makogondatabase to stay updated.21:04
denis_makogonTrove's new conductor service eliminates that need by21:04
denis_makogonworking as a proxy for those instances. By sending a heartbeat21:04
denis_makogonto conductor via RPC, conductor updates the database21:04
denis_makogonon the host on behalf of the instance.21:04
denis_makogonAs backups also made use of the host database, the backup21:04
denis_makogoncode has been refactored to take richer inputs to remove21:04
denis_makogonthe need to query the host database, and now conductor is21:04
denis_makogonalso used to submit updates to backup states.21:04
denis_makogonesmute, do you still think that this is a bug ?21:06
datsun_F40PHi think there's just a matter of a rebase and maybe making one review dependent on the other21:08
*** lifeless has quit IRC21:08
datsun_F40PHimo the db connection should be completely removed, i was just timid about it with conductor in case i'd missed something (you know, like root_history)21:09
denis_makogondatsun_F40PH, in that way i agreed21:10
datsun_F40PHdenis_makogon: looks to me that you've just got the root history changes in two reviews and you might just need to reorganize the two a bit21:11
denis_makogoni will abandon one of them21:11
*** yogesh has quit IRC21:11
*** yogesh has joined #openstack-trove21:12
kevinconwaydenis_makogon: don't forget to reference  a bug in your abandon21:12
denis_makogoni've got another solution21:14
kevinconwaydenis_makogon: does it reference a bug? ))21:14
denis_makogondatsun_F40PH, i'll leave this one https://review.openstack.org/#/c/59410/, and then i would ask ikhudoshyn to rebase his https://review.openstack.org/#/c/59078/21:15
datsun_F40PHmakes sense21:15
denis_makogonSlickNik, https://review.openstack.org/#/c/59078/ could you please remove -2, ikhudoshyn review was first one, historically21:16
*** yogesh has quit IRC21:16
denis_makogonSlickNik, i abandon all my competetive reviews with ikhudoshyn and esmute21:16
datsun_F40PHi think the -2 stands, can't remove the db conn until all the uses are removed21:16
*** achampio1 has joined #openstack-trove21:17
denis_makogondatsun_F40PH, only report_root writes to DB21:17
SlickNikWe need to make sure that we address the other uses of the db conn issues before we take the connection out.21:17
denis_makogonSlickNik, i already tested it21:17
datsun_F40PHand until your change for report_root is merged the other review should sit a while21:18
denis_makogonSlickNik, https://review.openstack.org/#/c/59510/21:18
denis_makogondatsun_F40PH, yes21:18
denis_makogondatsun_F40PH, agreed21:18
SlickNikdenis_makogon: you make the same change in https://review.openstack.org/#/c/59510/21:18
*** jcooley_ has joined #openstack-trove21:19
SlickNikin bin/trove-guestagent21:19
denis_makogonSlickNik, i was testing,21:19
denis_makogonSlickNik, tests are passed, that means, that report_root is last of db-connection dependent problems21:20
denis_makogonSlickNik, now this review is abandoned21:20
*** haomaiwang has quit IRC21:20
*** achampion has quit IRC21:20
denis_makogonSlickNik, that is why i left only this one https://review.openstack.org/#/c/59410/21:20
SlickNikdmakogon_: you should really be creating a changeset dependent on ikhudoshyn's change.21:21
SlickNikGerrit allows you to do that.21:21
SlickNikRather than submitting two changes with similar changes (one for testing).21:22
SlickNikAlso, if you have a change set that is only for testing, please submit it as a draft so that it's not visible to the world.21:23
*** yogesh has joined #openstack-trove21:23
denis_makogonSlickNik, ikhudoshyn should make his review dependent on mine21:23
denis_makogonhttps://review.openstack.org/#/c/59078/ depends on https://review.openstack.org/#/c/59410/21:24
SlickNikyes, that should be the case.21:24
SlickNikI mistyped.21:24
*** yogesh has quit IRC21:24
denis_makogonSlickNik, everything competetive i already removed21:24
denis_makogonSlickNik, sorry for spaming =)21:25
*** jcooley_ has quit IRC21:25
denis_makogonSlickNik, have you seen this http://github.com/denismakogon/trove-guestagent21:25
*** jcooley_ has joined #openstack-trove21:26
SlickNikNo problem. I was confused since I thought the testing change was supposed to be the final patch (submitting both ikhudoshyn's change and yours).21:26
*** yogesh has joined #openstack-trove21:26
*** yogesh has quit IRC21:26
*** jcooley_ has quit IRC21:26
SlickNikWill remove my −2 on his patch and recommend he make it dependent on your change.21:26
denis_makogonok21:27
denis_makogonSlickNik, i already update Wed. meeting with guestagent update topic21:27
SlickNiknope, haven't seen it. Looks interesting ))21:27
denis_makogonSlickNik, i tested it manually, all test-cases are working21:28
datsun_F40PHSlickNik: he's been working on building the ga as its own repo so instances can get it via git21:28
denis_makogondatsun_F40PH, git is only my option21:28
denis_makogondatsun_F40PH, i use git, because it's easier for me, instead of rsync21:29
datsun_F40PHwell that second statement invalidated the first!21:29
denis_makogoni have lots of branches at github21:29
kevinconwaydenis_makogon: have you tried drag-and-drop?21:29
kevinconwayi've found that to be the easiest21:29
datsun_F40PHkevinconway: i just use 2-day shipping, it gets there in time typically21:30
*** yogesh has joined #openstack-trove21:30
denis_makogonkevinconway, yes, VNC don't want to take my stuff21:30
denis_makogonrobertmy_, ping21:42
*** robertmy_ is now known as robertmyers21:50
robertmyersdenis_makogon: pong21:51
denis_makogonrobertmyers, goodday, Robert21:51
*** rongze has joined #openstack-trove21:51
denis_makogonrobertmyers, do we have any plans on horizon and dashboard ?21:51
robertmyersdenis_makogon: haven't looked at it in a while21:52
robertmyersthe client was recently updated tho21:52
denis_makogonrobertmyers, i think it's time to get improve dashboard with new client features21:52
robertmyerssuch as?21:52
denis_makogondatastores21:53
denis_makogonit's huge part of fuctionality21:53
robertmyersah yeah, that will be fun21:53
*** lifeless has joined #openstack-trove21:53
robertmyersI'm guessing after icehouse21:53
denis_makogonrobertmyers, that is why i'm wondering21:53
denis_makogonafter ?21:53
robertmyersyeah, it is pretty close to the deadline21:54
robertmyersDec 5th?21:54
denis_makogonrobertmyers, i thought end of april21:54
denis_makogondeadline I321:54
denis_makogonam i wrong ?21:54
robertmyersno, I'm wrong21:55
robertmyersicehouse 1 is on the 5th21:55
denis_makogonyes21:56
*** yogesh has quit IRC21:56
denis_makogonlol, I1 release is very soon21:56
*** rongze has quit IRC21:57
*** yogesh has joined #openstack-trove21:57
denis_makogonhub_cap, do we have features-to-be-design ?21:57
SlickNikrobertmyers: got a minute?21:57
robertmyersSlickNik: sure21:57
SlickNikwas fixing up https://review.openstack.org/#/c/58062/, and had a clarification to one of your comments.21:58
SlickNikhttps://review.openstack.org/#/c/58062/5/trove/common/cfg.py, you mention making this a dictopt.21:58
robertmyersyes21:58
SlickNikWas the idea to have the keys be the strategy type and the values be the extra_options to be passed in?21:59
robertmyersexactly21:59
SlickNikgotcha, that's what I thought made the most sense while making this a DictOpt, but just wanted to make sure we're on the same page.22:00
robertmyerslike this: backup_runner_options = InnoBackupEx:--no-lock, MySQLDump:--events --routines --triggers22:00
SlickNikYup, exactly. That works splendid. Thanks!22:01
robertmyersno problem22:01
*** yogesh has quit IRC22:01
robertmyersSlickNik: ps, I have a bunch of reviews for backups that need eyes :)22:02
SlickNikrobertmyers: They're on my list to look at today. Just as soon as I upload a new patchset for this review I'm working on.22:03
SlickNik(Which should be within the next hour or so)22:03
robertmyerscool22:04
SlickNikThank you!22:04
*** yidclare has joined #openstack-trove22:05
*** pdmars has quit IRC22:06
*** grapex has quit IRC22:13
hub_capdenis_makogon: what do  u mean?22:19
denis_makogonhub_cap, list of features which should be implemented during I2-I322:20
*** yogesh has joined #openstack-trove22:20
denis_makogonhub_cap, not so long ago (after summit) we've talked about this22:21
denis_makogonhub_cap, i mean correct blueprints, with correct milestones22:21
denis_makogonhub_cap, i think we could provide some kind of responsibilities for incorrect registering of BPs22:22
amcrnlashes?22:23
denis_makogonamcrn, lol, binary lashes)))22:24
*** plodronio_ has joined #openstack-trove22:24
*** plodronio has quit IRC22:25
*** plodronio_ is now known as plodronio22:25
denis_makogoni'm yelling because I1 would be closed at Dec. 5 - we need to document everything that would implemented22:25
denis_makogonmaybe i'm wrong22:27
*** rwsu has quit IRC22:27
denis_makogonmaybe not22:27
denis_makogononly cores and PTL could make such decision22:28
*** demorris has quit IRC22:28
hub_capdenis_makogon: the only person who makes a decision about a feature being in icehouse is a developer working on that feature :)22:29
hub_capdemorris has told he he was going to fill out some i2/3 priorities but has not yet22:29
hub_capif mirantis has some, id love to see them added. same w hp vipul SlickNik :)22:30
denis_makogonhub_cap, would tempest tests land in I2 ?22:32
*** adrian_otto has left #openstack-trove22:32
vipulhub_cap: just mark bugs / bps with the appropriate I tag/22:33
hub_capdenis_makogon: id' love to see it land in i222:38
hub_capvipul: and if you have any "id love to see X or Y done in icehouse" roadmap-y type stuff too22:39
vipulsure.. i can work on getting that.. just wondering where to share that besides launchpad22:40
hub_caplp i think is the _best_ vipul22:40
vipul:p22:40
*** rwsu has joined #openstack-trove22:43
*** rongze has joined #openstack-trove22:53
*** achampio1 has quit IRC22:53
*** rongze has quit IRC22:58
*** amytron has quit IRC22:58
*** flaper87 is now known as flaper87|afk23:00
*** denis_makogon has quit IRC23:00
*** yogeshmehra has joined #openstack-trove23:00
*** denis_makogon has joined #openstack-trove23:01
*** yogeshmehra has quit IRC23:01
*** yogesh has quit IRC23:01
*** yogesh has joined #openstack-trove23:02
denis_makogonvipul, please, could you take a look at https://review.openstack.org/#/c/58845/  and https://review.openstack.org/#/c/59410/ ?23:02
vipuldenis_makogon: sure will do23:02
denis_makogonvipul, thanks for that =)23:03
*** yogesh has quit IRC23:06
*** denis_makogon has quit IRC23:09
*** jcooley_ has joined #openstack-trove23:09
openstackgerritRobert Myers proposed a change to openstack/trove: Adding Incremental Backups  https://review.openstack.org/5956823:10
openstackgerritRobert Myers proposed a change to openstack/trove: Simplify swift storage load logic  https://review.openstack.org/5779623:10
openstackgerritRobert Myers proposed a change to openstack/trove: Properly mark a backup as FAILED if the process failed  https://review.openstack.org/5928323:10
openstackgerritRobert Myers proposed a change to openstack/trove: Moving storage logic out of backup runner class  https://review.openstack.org/5670223:10
robertmyerswhoops, not sure if that was what I wanted :)23:12
robertmyersguess, I'll have to clean up that mess tomorrow23:16
*** robertmyers has quit IRC23:20
*** jcru_ has quit IRC23:21
hub_capLOL23:23
*** cweid has quit IRC23:31
*** jcooley_ has quit IRC23:32
*** rongze has joined #openstack-trove23:55

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