Monday, 2014-02-10

*** krow has joined #openstack-trove00:21
*** krow has quit IRC00:51
*** rongze has joined #openstack-trove00:51
*** rongze has quit IRC00:55
*** nosnos has joined #openstack-trove01:12
*** nosnos_ has joined #openstack-trove01:21
*** nosnos_ has quit IRC01:21
*** nosnos has quit IRC01:22
*** nosnos_ has joined #openstack-trove01:22
*** grapex has joined #openstack-trove01:28
*** krow has joined #openstack-trove01:41
*** rongze has joined #openstack-trove01:51
*** rongze has quit IRC01:56
*** Barker has joined #openstack-trove02:01
*** grapex has quit IRC02:22
*** grapex has joined #openstack-trove02:23
*** erkules_ has joined #openstack-trove02:28
*** erkules has quit IRC02:31
*** mattgriffin has joined #openstack-trove02:31
*** Barker has quit IRC02:36
*** krow has quit IRC02:48
openstackgerritJeremy Stanley proposed a change to openstack/database-api: Remove tox locale overrides  https://review.openstack.org/7220502:58
*** krow has joined #openstack-trove03:06
*** edmund1 has joined #openstack-trove03:18
*** edmund has quit IRC03:20
*** harlowja is now known as harlowja_away03:30
*** krow has quit IRC03:34
*** rongze has joined #openstack-trove03:41
*** SushilKM has joined #openstack-trove04:02
*** rongze has quit IRC04:50
*** rongze has joined #openstack-trove04:50
*** rongze has quit IRC04:55
*** krow has joined #openstack-trove04:56
*** rongze has joined #openstack-trove05:22
openstackgerritA change was merged to openstack/database-api: Remove tox locale overrides  https://review.openstack.org/7220505:22
*** rongze has quit IRC05:27
*** nosnos has joined #openstack-trove05:56
*** rongze has joined #openstack-trove05:56
*** nosnos has quit IRC05:57
*** nosnos has joined #openstack-trove05:58
*** nosnos has quit IRC05:58
*** nosnos has joined #openstack-trove05:59
openstackgerritTim Simpson proposed a change to openstack/trove: Update log message to not call guest "nova guest".  https://review.openstack.org/7226805:59
*** nosnos_ has quit IRC05:59
*** nosnos has quit IRC06:47
*** nosnos_ has joined #openstack-trove06:48
*** saju_m has joined #openstack-trove06:53
*** saju_m has quit IRC07:00
*** saju_m has joined #openstack-trove07:16
*** igor has joined #openstack-trove07:16
openstackgerritTim Simpson proposed a change to openstack/trove: Fixes a race condition in resize flavor  https://review.openstack.org/7227907:19
*** igor has quit IRC07:20
*** igor has joined #openstack-trove07:21
*** mattgriffin has quit IRC07:28
*** mattgriffin has joined #openstack-trove07:31
*** grapex has quit IRC07:39
*** krow has quit IRC07:46
*** mattgriffin has quit IRC07:50
*** krow has joined #openstack-trove07:52
*** mattgriffin has joined #openstack-trove07:52
*** krow has quit IRC07:57
*** krow has joined #openstack-trove07:57
*** nosnos_ has quit IRC08:02
*** krow has quit IRC08:02
*** nosnos has joined #openstack-trove08:02
*** krow has joined #openstack-trove08:03
*** SushilKM has quit IRC08:16
*** krow has quit IRC08:20
*** krow has joined #openstack-trove08:23
*** krow has quit IRC08:44
*** flaper87|afk is now known as flaper8708:49
*** krow has joined #openstack-trove08:50
*** krow has quit IRC08:54
*** krow has joined #openstack-trove08:55
*** krow has quit IRC09:00
*** krow has joined #openstack-trove09:01
*** krow has quit IRC09:05
*** krow has joined #openstack-trove09:06
*** krow has quit IRC09:11
*** krow has joined #openstack-trove09:12
*** krow1 has joined #openstack-trove09:15
*** krow has quit IRC09:16
*** krow1 has quit IRC09:26
*** krow has joined #openstack-trove09:26
*** krow has quit IRC09:31
*** krow has joined #openstack-trove09:32
*** krow has quit IRC09:36
*** krow has joined #openstack-trove09:38
*** krow has quit IRC09:42
*** krow has joined #openstack-trove09:43
*** krow has quit IRC09:47
*** krow has joined #openstack-trove09:49
*** krow has quit IRC09:53
*** krow has joined #openstack-trove09:54
*** krow has quit IRC09:59
*** krow has joined #openstack-trove10:00
*** krow has quit IRC10:05
*** krow has joined #openstack-trove10:05
*** krow has quit IRC10:10
*** krow has joined #openstack-trove10:11
*** krow has quit IRC10:16
*** krow has joined #openstack-trove10:17
*** krow1 has joined #openstack-trove10:21
*** krow has quit IRC10:21
*** rongze has quit IRC10:22
*** krow1 has quit IRC10:25
*** krow has joined #openstack-trove10:26
*** krow has quit IRC10:31
*** krow has joined #openstack-trove10:32
*** krow has quit IRC10:36
*** krow has joined #openstack-trove10:37
*** krow has quit IRC10:42
*** krow has joined #openstack-trove10:43
*** rongze has joined #openstack-trove10:46
*** krow has quit IRC10:50
*** krow has joined #openstack-trove10:51
*** erkules_ is now known as erkules11:04
*** krow1 has joined #openstack-trove11:24
*** krow has quit IRC11:25
openstackgerritDenis M. proposed a change to openstack/trove: Add stack_id for future usage  https://review.openstack.org/6787312:02
*** krow1 has quit IRC12:16
*** krow has joined #openstack-trove12:19
*** nosnos_ has joined #openstack-trove12:58
*** nosnos has quit IRC13:01
*** nosnos_ has quit IRC13:03
*** nosnos has joined #openstack-trove13:07
*** freyes has joined #openstack-trove13:10
*** amytron has joined #openstack-trove13:10
*** saju_m has quit IRC13:12
*** pdmars has joined #openstack-trove13:17
*** krow has quit IRC13:36
*** krow has joined #openstack-trove13:37
*** krow has quit IRC13:43
*** achampion has quit IRC13:49
*** rongze has quit IRC13:52
openstackgerritDenis M. proposed a change to openstack/trove: Allow db instance conditional logging  https://review.openstack.org/6378913:56
openstackgerritDenis M. proposed a change to openstack/trove: Allow log files audit  https://review.openstack.org/6430213:56
*** achampion has joined #openstack-trove13:56
openstackgerritDenis M. proposed a change to openstack/trove: Initial support for single instance Cassandra Database  https://review.openstack.org/5188413:58
openstackgerritDenis M. proposed a change to openstack/trove-integration: Initial support for single instance Cassandra Database  https://review.openstack.org/5266614:00
*** achampion has quit IRC14:01
*** robertmyers has quit IRC14:11
*** jcru has joined #openstack-trove14:14
*** haomaiwang has quit IRC14:17
*** jimbobhickville has joined #openstack-trove14:17
*** haomaiwang has joined #openstack-trove14:17
openstackgerritDenis M. proposed a change to openstack/trove-integration: Add support for minimal MongoDB testing  https://review.openstack.org/5337814:22
*** rongze has joined #openstack-trove14:29
*** robertmyers has joined #openstack-trove14:37
*** ViswaV has joined #openstack-trove14:37
*** ViswaV_ has joined #openstack-trove14:38
*** ViswaV has quit IRC14:42
*** edmund1 has quit IRC14:43
*** achampion has joined #openstack-trove14:44
*** pdmars has quit IRC14:45
openstackgerritDenis M. proposed a change to openstack/trove: Initial support for single instance MongoDB support  https://review.openstack.org/5059714:47
*** edmund has joined #openstack-trove14:49
*** matsuhashi has joined #openstack-trove14:49
*** saju_m has joined #openstack-trove15:01
*** saju_m has quit IRC15:03
*** kevinconway has joined #openstack-trove15:04
*** saju_m has joined #openstack-trove15:04
*** Barker has joined #openstack-trove15:04
*** pdmars has joined #openstack-trove15:06
*** ViswaV_ has quit IRC15:07
*** radez_g0n3 is now known as radez15:09
*** rwsu has joined #openstack-trove15:17
*** rongze has quit IRC15:25
*** HoloIRCUser has joined #openstack-trove15:27
*** grapex has joined #openstack-trove15:28
*** grapex has quit IRC15:32
*** grapex has joined #openstack-trove15:33
*** matsuhashi has quit IRC15:37
*** nosnos has quit IRC15:37
*** nosnos has joined #openstack-trove15:37
*** matsuhashi has joined #openstack-trove15:37
*** saju_m has quit IRC15:37
*** freyes has quit IRC15:39
*** saju_m has joined #openstack-trove15:39
*** tanisdl has joined #openstack-trove15:47
*** rongze has joined #openstack-trove15:56
*** haomai___ has joined #openstack-trove15:57
*** haomaiwang has quit IRC16:00
*** jmontemayor has joined #openstack-trove16:00
*** radez is now known as radez_g0n316:02
*** rongze has quit IRC16:06
*** nosnos has quit IRC16:06
*** radez_g0n3 is now known as radez16:07
*** matsuhashi has quit IRC16:07
*** thedodd has joined #openstack-trove16:19
*** tanisdl has quit IRC16:30
*** ViswaV has joined #openstack-trove16:34
*** ViswaV_ has joined #openstack-trove16:36
*** vgnbkr_ has joined #openstack-trove16:37
*** ViswaV has quit IRC16:39
*** ViswaV_ has quit IRC16:45
*** ViswaV has joined #openstack-trove16:46
*** amytron has quit IRC16:47
*** ViswaV has joined #openstack-trove16:47
*** igor has quit IRC16:47
*** igor has joined #openstack-trove16:47
*** tanisdl has joined #openstack-trove16:50
*** ViswaV has quit IRC16:51
*** igor has quit IRC16:52
*** ViswaV has joined #openstack-trove16:52
*** ViswaV has quit IRC16:57
*** ViswaV has joined #openstack-trove16:58
*** rongze has joined #openstack-trove16:58
*** saju_m has quit IRC16:59
*** rongze has quit IRC17:04
*** saju_m has joined #openstack-trove17:04
*** ViswaV has quit IRC17:06
*** ViswaV has joined #openstack-trove17:07
*** ViswaV has quit IRC17:16
*** ViswaV has joined #openstack-trove17:17
*** saju_m has quit IRC17:19
*** ViswaV has quit IRC17:26
*** ViswaV has joined #openstack-trove17:27
*** saju_m has joined #openstack-trove17:32
*** ViswaV has quit IRC17:41
*** ViswaV has joined #openstack-trove17:42
*** ViswaV has quit IRC17:44
*** esp has joined #openstack-trove17:56
*** igor_ has joined #openstack-trove17:58
*** harlowja_away is now known as harlowja17:58
*** amcrn has joined #openstack-trove18:01
*** igor_ has quit IRC18:02
*** datsun180b has joined #openstack-trove18:03
*** SnowDust has joined #openstack-trove18:07
*** saju_m has quit IRC18:12
*** datsun180b has quit IRC18:14
dougshelley66good day fellow Trovesters...wanted to pass along some news18:23
*** ViswaV has joined #openstack-trove18:24
dougshelley66Today ParElastic has been relaunched as Tesora - if you are interested in the details - http://tesora.com/news/press-releases/tesora-launches-today-simplify-delivery-database-service-platform-openstack18:24
vipuldougshelley66: awesome, congrats!18:26
dougshelley66vipul: thanks18:26
dougshelley66looking forward to meeting everyone next week in Austin. Hopefully everyone can attend our Social event at Maggie Mae's on Wednesday evening.18:27
*** saju_m has joined #openstack-trove18:29
*** datsun180b has joined #openstack-trove18:30
hub_capsweet dougshelley66 /me goes to read18:32
dougshelley66hub_cap: thanks - you should read it as you are quoted in it :)18:32
*** yidclare has joined #openstack-trove18:33
hub_capdougshelley66: aye :D18:33
hub_capand yes i think we are all lookin forward to Wed at Maggie Maes!18:34
*** rwsu has quit IRC18:36
*** yogesh has joined #openstack-trove18:37
*** freyes has joined #openstack-trove18:38
*** Ranjitha has joined #openstack-trove18:43
*** zigo has quit IRC18:45
*** khyati has joined #openstack-trove18:47
*** zigo has joined #openstack-trove18:49
*** rwsu has joined #openstack-trove18:51
*** saju_m has quit IRC18:51
*** saju_m has joined #openstack-trove18:52
*** datsun180b has quit IRC18:53
*** saju_m has quit IRC18:58
*** SushilKM has joined #openstack-trove19:01
*** jimbobhickville has quit IRC19:03
*** jimbobhickville has joined #openstack-trove19:03
vgnbkr_Hi all.  Question about int-tests.  When I run them it takes about 400 seconds for the test_instance_created test to run.  Is that normal?  It seems long, and the system doesn't seem to be doing anything for most of the time.19:04
*** datsun180b has joined #openstack-trove19:05
grapexdougshelley66: Wow! Awesome!19:06
grapexvgnbkr_: It is normal. In that time it's loading up the image, creating a new server, and installing the guest agent which then installs MySQL19:06
datsun180bif we had named ourselves Seshat like i'd wanted, they'd have to have chosen a word like Libro19:07
dougshelley66grapex: thanks!19:07
grapexdatsun180b: lol. We should've gone with Juice's name "hyperarchos". I'm still sore about losing that one...19:08
datsun180bi forgot about hyperarchos19:08
vgnbkr_OK, I thought those were already installed in the image an the instance just had to be brought up.  Then again, I guess I didn't not how long it took to bring up a straight nova instance, so I have nothing to compare it to :-)  Thanks.19:08
*** cweid has joined #openstack-trove19:13
*** michael-yu has joined #openstack-trove19:16
*** vgnbkr_ has quit IRC19:18
cweidHey are we having some issues with trove and devstack?19:23
cweidWhen I do a redstack-install and a redstack-kickstart whatever I don't get and trove stuff in the screen session?19:24
datsun180bcweid: i've found recently that issuing a stop and then a start helps19:25
datsun180bthat's install && fix-iptables && kick-start && stop && start19:25
hub_capcweid: is your trove-integration and devstack both up to date?19:25
hub_capor are u on a new vm19:25
cweidAhh I might be running old trove-int19:26
cweidone sec.19:26
hub_capbe sure to pull devstack too if youre on a older vm19:26
hub_capcrap has changed a few times in the last few wks thats just torqued our install19:26
*** thedodd has quit IRC19:30
*** amrith has joined #openstack-trove19:32
vgnbkrSpeaking of which, are there scripts to help bring all the projects up to date, or do you just go into every project and do a pull?19:33
datsun180bi literally have a "for i in (four dirs); do pushd; git fetch --all; git pull origin; popd; done" script19:34
vgnbkrdatsun180b, why the separate fetch?  Doesn't the pull do the same thing?19:35
datsun180bthe four dirs are unrelated and in different repos19:35
datsun180boh context19:36
datsun180bi meant trove, trove-int, python-troveclient, and vagrant19:36
datsun180bnot the other os projects we use19:36
hub_capvgnbkr: there is no script to get devstack up to date, thatd have to exist in redstack and it dont19:38
hub_capbut as for the projects in /opt/stack19:38
hub_captere is a variable to use, i think its RECLONE=yes19:38
hub_capbut i dont use it19:38
hub_capi just sping up a new vm heh19:38
*** saju_m has joined #openstack-trove19:38
vgnbkrThe RECLONE would certainly help.  Thanks.19:39
vgnbkrdatsun180b, I was asking the reason for fetching all of the remotes?  (that's an area of git I'm not too familiar with)19:39
datsun180bto make sure my pull-origins do what they were asked19:40
datsun180bfetch phones home and updates your local copies of the repos19:41
datsun180bpull actually brings those changes from one branch to yours19:41
datsun180bis that what you meant?19:41
vgnbkrdatsun180b, "git pull origin master" would get fetch the changes you are about to merge.  Why are you breaking it apart into a separate "fetch --all"?  (just trying to understand in case I'm missing some subtle point)19:45
datsun180bi am a creature of habit and i don't trust computers19:46
vgnbkrdatsun180b, ah, so you're smart!19:46
vgnbkrOK, thanks.19:46
datsun180balso i haven't read the docs in forever and am too afraid of accidentally throwing my code into a wood chipper with the wrong command19:47
vgnbkrAh, so I see you've used git before?  :-)19:47
datsun180b"Okay, time to commit. But first, let's remove the pyx files... '$ rm *.py' AWWWWW"19:47
*** zigo has quit IRC19:48
vgnbkrI actually just did that.  I hadn't used emacs for a while and wanted to clean up the temp files.  In dired I did, "~Dx" - unfortunately I was sitting on top of the file that had most of my changes :-(19:49
*** datsun180b has quit IRC19:59
*** rongze has joined #openstack-trove19:59
*** saju_m has quit IRC20:01
*** openstack has joined #openstack-trove21:32
cweidHuum when I manually install oslo.config I still can't import it even manually...21:33
cweidls21:33
cweidoops21:33
*** Barker has joined #openstack-trove21:34
*** tanisdl has quit IRC21:35
cweidIs anyone else seeing that issue with a fresh vm?21:37
denis_makogoncweid, good to see you here21:37
cweiddenis_makogon: thanks duder you as well =)21:37
denis_makogoncweid, could you please update redis manager with missinge methods21:37
denis_makogoncweid, also, i would suggest you to use TroveError("Not supported") instead of NotImplemented21:38
cweiddenis_makogon: def planning on it. Have some rackspace stuff that I am working on for the next couple of days21:38
denis_makogoncweid, i could help you, if you are running out free time21:39
denis_makogoncweid, just let me know if i could help you21:39
datsun180byeah the two weeks following this one are going to be fun for us21:39
datsun180bmidcycle next week, for one21:40
denis_makogondatsun180b, and the next one ?21:40
datsun180ba similar event21:40
hub_capdenis_makogon: we have a rax internal conference21:41
datsun180byeah that21:41
hub_capso its back to back crazyness, 2 conferences, 2 wks21:41
hub_cap2 roundtrip flights21:41
datsun180bfor you21:41
denis_makogonhub_cap, thanks, i woried that i missed something21:41
datsun180bsorry, wasn't trying to be evasive21:42
denis_makogonamcrn, i followed your suggestions and added few parameters( by the default) to taskmanager.conf https://review.openstack.org/#/c/71870/21:43
*** amcrn_ has joined #openstack-trove21:44
*** amcrn has quit IRC21:45
*** amcrn_ is now known as amcrn21:48
*** Barker has quit IRC21:50
*** jdbarry_ has joined #openstack-trove21:58
*** edmund has quit IRC22:00
*** edmund has joined #openstack-trove22:00
*** datsun180b has quit IRC22:01
*** pdmars has quit IRC22:03
kalebpomeroywhat hotel are people staying at for the mid-cycle meetup?22:03
dougshelley66The Tesora gang is at the Intercontinental22:05
*** amrith has joined #openstack-trove22:05
SushilKMhey hub_cap, slicknik, vipul22:14
*** shakayumi has joined #openstack-trove22:15
SushilKMheat code path is broken, for might be some changes in heat/heat-client codes22:15
SlickNikSushilKM: Have you identified what the issue is?22:16
SushilKMyep22:16
SushilKMthis is because of https://github.com/openstack/trove/blob/master/trove/common/remote.py#L7422:16
SushilKM%s/%s/ shud be %s/%s22:16
SushilKMthe one I was resolving in https://review.openstack.org/#/c/6817522:16
SushilKMthough this change was abandoned in favor of https://review.openstack.org/#/c/68015/22:17
SushilKMcan we see how can we fix this now22:17
SushilKMi think either of mine or https://review.openstack.org/#/c/68015/ shud be taken into review to fix this up22:17
*** krow has joined #openstack-trove22:18
SushilKMlet me look more into this ...... i wud confirm after I finish up with my tests .....22:19
SlickNikcool, thanks. I'm in meetings, so I won't be able to look into it until later.22:20
*** shakayumi has quit IRC22:21
*** datsun180b has joined #openstack-trove22:21
denis_makogonSushilKM, https://review.openstack.org/#/c/68015/ this one was historicaly first, so it means that it should land first22:22
hub_capthat doesnt mean it has to22:24
hub_capwe have a bug we need to fix22:24
hub_capmat-lowery: can depend his code on the bugfix22:24
denis_makogonhub_cap, Mats patchset covers all cases, but its being marked with X because of I322:25
denis_makogonSushilKM, what do you mean heat code path is broken ?22:26
denis_makogonSushilKM, could you please share logs22:26
hub_capdenis_makogon: yes i know , i -2'd mat-lowery 's review :)22:27
SushilKM@denis I mean try creating a new gues using latest code with heat enabled22:28
SushilKMthere is a bug in place22:28
SushilKMhttps://bugs.launchpad.net/trove/+bug/127845022:28
denis_makogonSushilKM, paste heat logs, please22:29
hub_capdenis_makogon: like whats in the bug listing, or more than that?22:29
hub_capahh from the heat api nm denis_makogon22:30
hub_capis there a file attach option to bugs?22:30
hub_capid much prefer that to actually pasting the logs22:30
SushilKMits a one liner22:30
SushilKMNo route matched for POST /42e5cc53b292435ca831de72ad15982b//stacks22:30
denis_makogon/ is fine22:30
denis_makogon"//"22:31
SushilKMyeah // comes from remote.py22:31
openstackgerritDan Nguyen proposed a change to openstack/trove: Adding missing indexes for trove db  https://review.openstack.org/7203922:32
SushilKMfine i have tested creating the stack using https://review.openstack.org/#/c/68175/22:33
cweiddougshelley66: Party at the intercon then.22:33
SushilKMit gets created22:33
SushilKMhub_cap shall I restore this change22:33
dougshelley66cweid: most definitely!22:33
*** khyati has quit IRC22:33
SushilKMand we can make heat work up again22:33
SushilKMit is needed22:33
*** achampion has quit IRC22:33
hub_capdenis_makogon: are u testing the // ?22:34
denis_makogonSushilKM, i'm going it now22:34
SushilKMfine22:34
SushilKMi m restoring my change, u may test and accordingly review the change22:34
hub_capyea lets let denis_makogon test and report his finding22:34
hub_capand then we can dicsus merging this and changing mat-lowery's commit22:35
denis_makogonhm, seeing same thing ...22:35
denis_makogoni'd personaly pass in %/% first and then Mats review22:37
SushilKMso, shall i restore my change hub_cap, denis22:39
denis_makogonlets ask cores22:39
SushilKMwat say hub_Cap22:39
denis_makogonmat-lowery, are you there ?22:40
mat-lowerydenis_makogon: Here.22:41
hub_capyes lets do what we all agree on :)22:41
hub_capSushilKM: restore22:41
hub_capmat-lowery: after sushil restores, rebase your commit on top of his22:41
mat-loweryhub_cap: Roger.22:41
SushilKMthanks @ hub_ca22:41
hub_capand remove that fixes-bug line from your commit22:41
SushilKMhu b_cap22:41
mat-loweryhub_cap: Got it.22:41
SushilKMso i shall be putting two bug-fixes in my commit message22:42
hub_capSushilKM: if there are 2 bugs then we mark one as a dup22:42
denis_makogonSushilKM, no22:42
hub_capand u onl put one22:42
SushilKMthe one i already have 127112622:42
SushilKMand the other new one is 127845022:42
SushilKMwat say22:42
hub_capSushilKM: full links plz22:44
denis_makogonSushilKM, leave that one which is older22:44
SushilKMjst a sec @hub_Cap22:44
denis_makogonSushilKM, with updating description to keep it full22:44
SushilKMhttps://bugs.launchpad.net/trove/+bug/1278450 and https://bugs.launchpad.net/trove/+bug/127112622:44
hub_capill clean it up22:44
hub_capok done22:45
denis_makogonhub_cap, fast as bolt22:45
SushilKMdid not get that @hub_cap22:45
hub_capdenis_makogon: its all im good for ;)22:46
denis_makogoni've updated a bit header in Mat's bug-report22:47
*** kevinconway has quit IRC22:47
SushilKMhttp://paste.openstack.org/show/64032/ does that looks ok22:48
denis_makogonso, SushilKM, we leaving this bug-report https://bugs.launchpad.net/trove/+bug/1271126, please update commit message22:48
denis_makogonSushilKM, only one Closes-Bug per patch22:49
SushilKMok i saw that 1278450 is duplicated to 127112622:49
mat-lowerydenis_makogon: Sorry I don't understand this: "i've updated a bit header in Mat's bug-report"22:49
denis_makogonmat-lowery, your bug report header contained "novaclient" word, but this issue touches all service urls22:50
*** krow has quit IRC22:50
SushilKMok so the message is already good22:50
SushilKMcan we proceed for reviews22:50
openstackgerritTim Simpson proposed a change to openstack/trove: Fixes a race condition in resize flavor  https://review.openstack.org/7227922:50
denis_makogonSushilKM, http://paste.openstack.org/show/64035/22:50
SushilKM@denis it already has that, thank you22:51
SushilKMkindly review https://review.openstack.org/#/c/68175/22:51
denis_makogonSushilKM, i'd suggest you to write commit description, please22:52
openstackgerritTim Simpson proposed a change to openstack/trove: Update log message to not call guest "nova guest"  https://review.openstack.org/7226822:52
denis_makogonSushilKM, writing commit title is not enough for patchset22:52
denis_makogongrapex, ping22:53
grapexHi denis_makogon22:53
denis_makogongrapex, hi, Tim22:53
denis_makogongrapex, i've got question, is it really necassery to split those patches ?22:54
SushilKMas a requirement commit message is fine ....22:54
openstackgerritTim Simpson proposed a change to openstack/trove: Remove unused admin_context from conductor manager  https://review.openstack.org/7218222:54
denis_makogongrapex, i know that they have different scope, but is it really necassery22:55
SushilKMhttps://wiki.openstack.org/wiki/Gerrit_Workflow#Committing_Changes22:55
grapexdenis_makogon: Which patches?22:55
*** krow has joined #openstack-trove22:55
denis_makogongrapex, i mean https://review.openstack.org/72279 https://review.openstack.org/72268, could this be the same patch ?22:56
denis_makogonSushilKM, your patch contains commit title and empty(!!) commit description22:57
grapexdenis_makogon: I figured people would appreciate it in two patches22:57
*** rongze has joined #openstack-trove22:57
hub_capi dont know what yer talking about22:58
hub_capbut i appreciate it in two patches22:58
grapexPersonally I am not a fan of small patches, but I recognize as the project and number of contributors grow the quickest way to get things done is to have tiny patches22:58
hub_capi am a fan of tiny patches22:58
denis_makogonhub_cap, like 2 lines in LOG calls ?))22:58
hub_capunless its for something like "removing copyright notices"22:58
hub_capif u fix somethign in log lines, fix them everywhere22:58
hub_capdont send me 200 commits w/ 1 fix22:58
hub_capif  uhave 20 __inits__ w headers i dont wanna see 20 commits :P22:59
hub_capdenis_makogon: paste me the review plz22:59
hub_capso i have context22:59
denis_makogonhub_cap,  https://review.openstack.org/72279 https://review.openstack.org/7226822:59
SushilKMhey hub_cap and cores please review https://review.openstack.org/#/c/68175/23:00
*** krow has quit IRC23:00
denis_makogongrapex, i'm ok with small patches, but i really do not see why they cannot be one bigger23:00
grapexhub_cap: You might not like this patch.23:00
grapexI submitted it and thought "Mike might not like this"23:01
hub_capim fine w/ that23:01
grapexthen I thought "eh, why not"23:01
grapexIt is embarassing to see us refer to it as the "Nova Guest"23:01
grapexLike we're talking about the lover who jilted us23:01
*** krow has joined #openstack-trove23:01
grapexback in the days of the fork23:01
grapexI have to go now- talk to you all later!23:01
denis_makogonSushilKM, i -1'd your patch because you hadn't followed rules of commits desription23:02
*** rongze has quit IRC23:02
*** grapex has quit IRC23:02
hub_capsry went afk for a sec23:02
*** yogesh has quit IRC23:02
denis_makogonSushilKM,  ...Long multiline description of the change... - means not _the_ empty body23:02
*** yogesh has joined #openstack-trove23:03
*** jcru has quit IRC23:03
*** krow has quit IRC23:05
*** krow has joined #openstack-trove23:06
*** yogesh has quit IRC23:07
denis_makogoncu guys tomorrow, have a nice day.23:07
* denis_makogon left23:07
openstackgerritSushil Kumar proposed a change to openstack/trove: Corrects service URLs from "%s/%s/" to "%s/%s"  https://review.openstack.org/6817523:09
SushilKMi have updated the commit message too23:10
SushilKMplease proceed for the reviews, Thank you.23:10
*** krow has quit IRC23:12
openstackgerritSushil Kumar proposed a change to openstack/trove: Corrects service URLs from "%s/%s/" to "%s/%s"  https://review.openstack.org/6817523:12
*** krow has joined #openstack-trove23:12
*** thedodd has quit IRC23:13
*** krow has quit IRC23:17
*** krow has joined #openstack-trove23:18
*** Kale_Roberts has quit IRC23:19
*** robertmyers has quit IRC23:19
*** krow1 has joined #openstack-trove23:21
*** krow has quit IRC23:22
*** krow has joined #openstack-trove23:23
*** krow1 has quit IRC23:26
SushilKMoops, we have a long queue waiting on py26 tests ..... :(23:26
*** krow1 has joined #openstack-trove23:29
*** krow has quit IRC23:29
datsun180brobots gonna robot23:32
*** pdmars has joined #openstack-trove23:33
*** krow1 has quit IRC23:33
*** krow has joined #openstack-trove23:34
*** denis_makogon has quit IRC23:36
*** achampion has joined #openstack-trove23:38
*** krow1 has joined #openstack-trove23:38
*** krow has quit IRC23:39
*** datsun180b has quit IRC23:47
*** cweid has quit IRC23:50
*** krow1 has quit IRC23:52
*** HoloIRCUser1 has joined #openstack-trove23:55
*** krow has joined #openstack-trove23:56
*** rongze has joined #openstack-trove23:57
*** HoloIRCUser has quit IRC23:59

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