*** rongze has quit IRC | 00:00 | |
*** tanisdl has quit IRC | 00:04 | |
*** tanisdl has joined #openstack-trove | 00:05 | |
*** Barker has quit IRC | 00:07 | |
*** datsun_F40PH has quit IRC | 00:08 | |
*** jmontemayor has quit IRC | 00:12 | |
*** rnirmal has quit IRC | 00:27 | |
*** matsuhashi has joined #openstack-trove | 00:29 | |
*** matsuhashi has quit IRC | 00:50 | |
*** matsuhashi has joined #openstack-trove | 00:50 | |
*** yogesh has joined #openstack-trove | 00:53 | |
*** matsuhas_ has joined #openstack-trove | 00:53 | |
*** matsuhashi has quit IRC | 00:54 | |
*** Barker has joined #openstack-trove | 00:54 | |
*** yogesh has quit IRC | 00:56 | |
*** yogesh_ has joined #openstack-trove | 00:59 | |
*** yogesh_ has quit IRC | 01:04 | |
*** yogesh has joined #openstack-trove | 01:05 | |
*** yogesh has quit IRC | 01:05 | |
*** yogesh has joined #openstack-trove | 01:05 | |
*** yogesh_ has joined #openstack-trove | 01:09 | |
*** yogesh has quit IRC | 01:11 | |
*** plodronio has quit IRC | 01:16 | |
*** nosnos has joined #openstack-trove | 01:26 | |
*** rongze has joined #openstack-trove | 01:27 | |
*** rongze has quit IRC | 01:31 | |
*** yidclare has quit IRC | 01:35 | |
*** jcooley_ has joined #openstack-trove | 01:37 | |
*** yogesh has joined #openstack-trove | 01:39 | |
*** yogesh_ has quit IRC | 01:42 | |
*** yogesh has quit IRC | 01:43 | |
*** jcooley_ has quit IRC | 01:45 | |
openstackgerrit | SlickNik proposed a change to openstack/trove: Added ability to pass custom command-line options to backup runner https://review.openstack.org/58062 | 01:59 |
---|---|---|
*** jcooley_ has joined #openstack-trove | 02:01 | |
*** jcooley_ has quit IRC | 02:02 | |
*** rongze has joined #openstack-trove | 02:06 | |
*** esp has left #openstack-trove | 02:23 | |
*** matsuhas_ has quit IRC | 02:31 | |
*** matsuhashi has joined #openstack-trove | 02:31 | |
*** achampion has joined #openstack-trove | 02:35 | |
*** erkules_ has joined #openstack-trove | 02:49 | |
*** erkules has quit IRC | 02:49 | |
*** tanisdl has quit IRC | 02:51 | |
*** nosnos has quit IRC | 02:58 | |
*** nosnos has joined #openstack-trove | 02:59 | |
*** sbisbee has left #openstack-trove | 03:15 | |
*** mmcdaris has quit IRC | 03:17 | |
*** amcrn has quit IRC | 03:19 | |
*** jcooley_ has joined #openstack-trove | 03:29 | |
*** SushilKM has joined #openstack-trove | 03:39 | |
*** mrsnivvel has joined #openstack-trove | 03:41 | |
*** matsuhashi has quit IRC | 03:42 | |
*** SushilKM has quit IRC | 03:48 | |
openstackgerrit | Sushil Kumar proposed a change to openstack/python-troveclient: Updates README https://review.openstack.org/58834 | 04:01 |
*** SushilKM has joined #openstack-trove | 04:10 | |
*** SushilKM has quit IRC | 04:20 | |
*** SushilKM has joined #openstack-trove | 04:20 | |
*** matsuhashi has joined #openstack-trove | 04:25 | |
*** rongze has quit IRC | 04:31 | |
*** yidclare has joined #openstack-trove | 04:56 | |
*** matsuhashi has quit IRC | 05:00 | |
*** SushilKM has quit IRC | 05:00 | |
*** matsuhashi has joined #openstack-trove | 05:08 | |
*** amytron has joined #openstack-trove | 05:24 | |
*** matsuhashi has quit IRC | 05:27 | |
*** matsuhashi has joined #openstack-trove | 05:59 | |
*** SushilKM has joined #openstack-trove | 06:38 | |
*** yogesh has joined #openstack-trove | 06:46 | |
*** denis_makogon has joined #openstack-trove | 06:48 | |
*** yogesh has quit IRC | 06:56 | |
*** yogesh has joined #openstack-trove | 06:56 | |
*** SushilKM has quit IRC | 06:57 | |
*** yogesh has quit IRC | 07:01 | |
*** debasish has joined #openstack-trove | 07:23 | |
*** vipul has quit IRC | 07:32 | |
*** vipul has joined #openstack-trove | 07:34 | |
*** jcooley_ has quit IRC | 07:35 | |
*** amytron has quit IRC | 07:35 | |
*** jcooley_ has joined #openstack-trove | 07:36 | |
*** yogesh has joined #openstack-trove | 07:46 | |
*** yogesh has quit IRC | 07:50 | |
*** yogesh has joined #openstack-trove | 07:50 | |
*** yogeshmehra has joined #openstack-trove | 07:52 | |
*** yogesh has quit IRC | 07:55 | |
*** flaper87|afk is now known as flaper87 | 08:03 | |
openstackgerrit | Olga Kopylova proposed a change to openstack/trove: db: make compatible with SQLAlchemy 0.8 https://review.openstack.org/59157 | 08:19 |
*** jcooley_ has quit IRC | 08:36 | |
*** jcooley_ has joined #openstack-trove | 08:38 | |
*** denis_makogon has quit IRC | 08:42 | |
*** ikhudoshyn has quit IRC | 09:00 | |
*** yogesh has joined #openstack-trove | 09:01 | |
*** ikhudoshyn has joined #openstack-trove | 09:02 | |
*** yogesh has quit IRC | 09:05 | |
*** jcooley_ has quit IRC | 09:34 | |
*** matsuhashi has quit IRC | 09:51 | |
*** matsuhashi has joined #openstack-trove | 09:54 | |
openstackgerrit | Denis M. proposed a change to openstack/trove: Disable redundant DB initialization on guesagent's start https://review.openstack.org/59078 | 09:54 |
openstackgerrit | Denis M. proposed a change to openstack/trove: RootReport through Conductor https://review.openstack.org/59410 | 09:54 |
*** jcooley_ has joined #openstack-trove | 10:05 | |
*** jcooley_ has quit IRC | 10:12 | |
*** yogeshme_ has joined #openstack-trove | 10:12 | |
*** yogeshmehra has quit IRC | 10:14 | |
*** rongze has joined #openstack-trove | 10:28 | |
*** yogeshme_ has quit IRC | 10:32 | |
openstackgerrit | Denis M. proposed a change to openstack/trove-integration: Trove-guestagent code tests in integration https://review.openstack.org/59661 | 10:33 |
*** rongze has quit IRC | 10:46 | |
*** erkules_ is now known as erkules | 11:03 | |
*** rongze has joined #openstack-trove | 11:03 | |
*** jcooley_ has joined #openstack-trove | 11:05 | |
*** jcooley_ has quit IRC | 11:10 | |
openstackgerrit | Denis M. proposed a change to openstack/trove-integration: Trove-guestagent code tests in integration https://review.openstack.org/59661 | 11:13 |
*** achampion has quit IRC | 11:34 | |
*** rongze has quit IRC | 11:39 | |
*** jcooley_ has joined #openstack-trove | 11:51 | |
*** rongze_ has joined #openstack-trove | 11:56 | |
*** debasish has quit IRC | 12:18 | |
*** jcooley_ has quit IRC | 12:25 | |
*** PradeepChandani has joined #openstack-trove | 12:31 | |
openstackgerrit | Sushil Kumar proposed a change to openstack/trove: Removes unused import statements https://review.openstack.org/59696 | 12:35 |
*** pdmars has joined #openstack-trove | 12:55 | |
*** jcooley_ has joined #openstack-trove | 13:08 | |
*** jcooley_ has quit IRC | 13:13 | |
*** jcooley_ has joined #openstack-trove | 13:23 | |
*** jcooley_ has quit IRC | 13:28 | |
*** Barker has quit IRC | 13:34 | |
*** matsuhashi has quit IRC | 13:40 | |
openstackgerrit | Sushil Kumar proposed a change to openstack/trove: Updates tox.ini to remove suppressed H401. https://review.openstack.org/59722 | 13:54 |
*** SushilKM has joined #openstack-trove | 14:06 | |
*** jcooley_ has joined #openstack-trove | 14:17 | |
*** nosnos has quit IRC | 14:20 | |
*** jcooley_ has quit IRC | 14:21 | |
*** rongze_ has quit IRC | 14:27 | |
*** radez_g0n3 is now known as radez | 14:37 | |
*** rongze has joined #openstack-trove | 14:38 | |
openstackgerrit | Sushil Kumar proposed a change to openstack/trove: Removes unused import statements https://review.openstack.org/59696 | 14:38 |
*** SushilKM has quit IRC | 14:44 | |
*** jcru has joined #openstack-trove | 14:47 | |
*** ikhudoshyn_ has joined #openstack-trove | 14:55 | |
*** amytron has joined #openstack-trove | 15:00 | |
*** Barker has joined #openstack-trove | 15:00 | |
*** robertmyers has joined #openstack-trove | 15:03 | |
*** jcooley_ has joined #openstack-trove | 15:17 | |
*** jcooley_ has quit IRC | 15:21 | |
*** Barker has quit IRC | 15:28 | |
*** radez is now known as radez_g0n3 | 15:32 | |
*** Barker has joined #openstack-trove | 15:33 | |
*** datsun180b has joined #openstack-trove | 15:35 | |
*** datsun180b is now known as datsun_F40PH | 15:47 | |
*** datsun_F40PH is now known as datsun180b | 15:48 | |
openstackgerrit | Robert Myers proposed a change to openstack/trove: Moving storage logic out of backup runner class https://review.openstack.org/56702 | 15:53 |
*** jmontemayor has joined #openstack-trove | 15:53 | |
*** Barker has quit IRC | 16:03 | |
*** jcooley_ has joined #openstack-trove | 16:04 | |
*** grapex has joined #openstack-trove | 16:05 | |
*** SushilKM has joined #openstack-trove | 16:07 | |
*** rnirmal has joined #openstack-trove | 16:08 | |
*** plodronio has joined #openstack-trove | 16:10 | |
*** jcooley_ has quit IRC | 16:19 | |
*** jcooley_ has joined #openstack-trove | 16:19 | |
*** jcooley_ has quit IRC | 16:20 | |
*** tanisdl has joined #openstack-trove | 16:22 | |
*** jcooley_ has joined #openstack-trove | 16:35 | |
*** achampion has joined #openstack-trove | 16:36 | |
*** ikhudoshyn_ has quit IRC | 16:36 | |
*** ikhudoshyn_ has joined #openstack-trove | 16:38 | |
*** radez_g0n3 is now known as radez | 16:38 | |
dmakogon_ | hub_cap, i've added some unit-tests to agent https://github.com/denismakogon/trove-guestagent | 16:44 |
*** jcooley_ has quit IRC | 16:47 | |
openstackgerrit | Robert Myers proposed a change to openstack/trove: Simplify swift storage load logic https://review.openstack.org/57796 | 16:48 |
*** grapex_ has joined #openstack-trove | 16:48 | |
*** SushilKM has quit IRC | 16:51 | |
*** grapex has quit IRC | 16:52 | |
hub_cap | dmakogon_: sweet. how come youre not denis_makogon? | 16:53 |
hub_cap | hey just so everyone knows, i1 gets cut today | 16:55 |
hub_cap | wait i lie, its cut in 2 days but the cutoff for merge is today | 16:55 |
datsun180b | fun | 16:59 |
openstackgerrit | Sushil Kumar proposed a change to openstack/trove: Removes unused import statements https://review.openstack.org/59696 | 16:59 |
dmakogon_ | hub_cap, any plans on changing integration to allow usage of external agent ? | 17:00 |
hub_cap | -integration? | 17:00 |
hub_cap | i think we need to ya | 17:01 |
dmakogon_ | could we start use cloud-init scripts in integration & | 17:02 |
dmakogon_ | *start to | 17:02 |
*** grapex_ has quit IRC | 17:03 | |
dmakogon_ | also, i think we would need another gate for agent tho | 17:03 |
*** grapex has joined #openstack-trove | 17:03 | |
openstackgerrit | Robert Myers proposed a change to openstack/trove: Properly mark a backup as FAILED if the process failed https://review.openstack.org/59283 | 17:07 |
*** jcooley_ has joined #openstack-trove | 17:12 | |
dmakogon_ | hub_cap, i almost done with units, having problem with tests which requires db | 17:14 |
robertmyers | dammit: AssertionError: 2013-12-03T17:11:01Z != 2013-12-03T17:11:00Z | 17:14 |
hub_cap | robertmyers: fix that lol | 17:14 |
hub_cap | i just had to explain to ttx why it was ok to move a critical bug to i2 | 17:14 |
robertmyers | I can't do the math | 17:14 |
*** jcooley_ has quit IRC | 17:15 | |
*** jcooley_ has joined #openstack-trove | 17:16 | |
robertmyers | hub_cap: lets just stop testing it ;) | 17:16 |
hub_cap | robertmyers: HAH | 17:17 |
*** amcrn has joined #openstack-trove | 17:17 | |
*** tanisdl has quit IRC | 17:17 | |
robertmyers | well, I think I cleaned up *my* mess from yesterday lets hope the robots agree | 17:18 |
imsplitbit | got it, we are no longer doing testing! | 17:19 |
robertmyers | done and done | 17:19 |
hub_cap | imsplitbit: exactly, specially for redis and clustering | 17:19 |
imsplitbit | sweeeeeeeeeeeeeeeeeeeeeeeeeeeeeet! | 17:19 |
hub_cap | u know, we always joke about testing | 17:19 |
imsplitbit | I like where your head is at | 17:19 |
imsplitbit | just not whats on it | 17:19 |
hub_cap | like "we dont need to test, our code is awesome" right? | 17:19 |
imsplitbit | exactly | 17:19 |
hub_cap | why dont we just not code? | 17:19 |
hub_cap | u _never_ have to write for a test if the code doesnt exist | 17:20 |
imsplitbit | just like you don't need to write documentation if you have access to the code you have all the documentation you need | 17:20 |
hub_cap | well duh | 17:20 |
*** tanisdl has joined #openstack-trove | 17:20 | |
*** jcooley_ has quit IRC | 17:20 | |
imsplitbit | done and done! | 17:20 |
ikhudoshyn_ | hub_cap: hi, could u pls look at https://review.openstack.org/#/c/57189/ its 'bout mount_point. i made it to be taken from my.cnf | 17:20 |
imsplitbit | I like this new direction we're goin in | 17:20 |
hub_cap | i mean, wjat is docuementation but stupid peoples version of tje code itself | 17:21 |
hub_cap | ikhudoshyn: GREAT!!!! ikhudoshyn_ | 17:21 |
imsplitbit | and what is a j when it could be an h? | 17:21 |
imsplitbit | you feel me?? | 17:21 |
dmakogon_ | hub_cap, would you mind https://review.openstack.org/#/c/59410/ https://review.openstack.org/#/c/59078/ ?))) | 17:21 |
*** yidclare has quit IRC | 17:22 | |
ikhudoshyn_ | dmakogon_: ^ ^ I guesss it woth to ask SlickNik to look @ it | 17:22 |
hub_cap | jmspljtbjt: unless jt can be a i | 17:22 |
dmakogon_ | ikhudoshyn, hub_cap approved mine yesterday ^^ | 17:22 |
ikhudoshyn_ | s/woth/worth/ | 17:22 |
*** jcooley_ has joined #openstack-trove | 17:23 | |
ikhudoshyn_ | nice, id like mine to be approved too )) hub_cap? | 17:23 |
*** esp has joined #openstack-trove | 17:27 | |
hub_cap | ikhudoshyn_: u are close to approval | 17:31 |
ikhudoshyn_ | good to know, thanks | 17:32 |
hub_cap | im slightly afraid to do file operations in the guest... every time we have tried to do file.open stuff, we end up w/ issues because of read/write issues | 17:32 |
dmakogon_ | hub_cap, why is that so ? | 17:33 |
robertmyers | hub_cap: subprocess.Popen('sudo chmod 777 /', shell=True) | 17:34 |
robertmyers | done | 17:34 |
kevinconway | robertmyers: you forgot the sticky bit and recursive flag | 17:34 |
robertmyers | doh | 17:34 |
kevinconway | also chown nobody | 17:34 |
robertmyers | let me fix up my review | 17:35 |
dmakogon_ | hub_cap, guest works with 100% existing files, all operation could be performed with sudo user rights | 17:35 |
kevinconway | dmakogon_: i think he's saying when the guest doesn't run as root | 17:35 |
kevinconway | the open() only works if the process user has permission to read | 17:35 |
dmakogon_ | kevinconway, i got it | 17:35 |
kevinconway | but you could Popen('sudo cat <myfile', stdout=PIPE) | 17:36 |
kevinconway | and then read from it into the process | 17:36 |
dmakogon_ | yes | 17:36 |
robertmyers | we *could* run the guest as mysql user | 17:36 |
dmakogon_ | robertmyers, we *might be able" run the guest as mysql user | 17:36 |
hub_cap | /slap robertmyers | 17:37 |
hub_cap | /slap dmakogon_ | 17:37 |
dmakogon_ | =) | 17:37 |
ikhudoshyn_ | my client does not show any smilies, does "/slap" means "fsck u and shut up"? ))) | 17:38 |
dmakogon_ | hub_cap, about grep xxxx - what if parameter structure is more complicated then "key = value" (in a single line) | 17:38 |
dmakogon_ | ikhudoshyn_, slap means "please, i suggest you to stop talking different strange things" | 17:39 |
kevinconway | dmakogon_: if you PIPE the file into your agent then you can use the re module to get what you want | 17:39 |
ikhudoshyn_ | that's what i thought | 17:40 |
*** grapex has quit IRC | 17:40 | |
dmakogon_ | kevinconway, lol, try to use it with complicated XML/YAML/JSON | 17:40 |
*** SushilKM has joined #openstack-trove | 17:40 | |
dmakogon_ | i think we could copy any file we want | 17:41 |
kevinconway | dmakogon_: you can still use parser libs | 17:42 |
kevinconway | i think the issue was read permissions on the file might not match what you have | 17:42 |
datsun180b | like awk | 17:42 |
kevinconway | so if you use PIPE instead of open() you can still use the same logic | 17:42 |
kevinconway | just change the file read part | 17:43 |
dmakogon_ | kevinconway, yes, that is what i want | 17:43 |
openstackgerrit | Sushil Kumar proposed a change to openstack/trove: Removes unused import statements https://review.openstack.org/59696 | 17:43 |
dmakogon_ | cp16net 's review (configurations) has perfect example of python built-in parsers usage | 17:43 |
dmakogon_ | https://review.openstack.org/#/c/53168/10/trove/common/configurations.py | 17:45 |
hub_cap | LOL ikhudoshyn_ | 17:45 |
hub_cap | dmakogon_: we can cross that when we need to i think | 17:45 |
hub_cap | *when we need to, i think* | 17:45 |
* hub_cap needs to get better at using commas | 17:46 | |
dmakogon_ | hub_cap, i think it's time | 17:46 |
hub_cap | ? | 17:46 |
hub_cap | to do more than grep this file? | 17:46 |
dmakogon_ | hub_cap, to use six parsers | 17:46 |
dmakogon_ | hub_cap, yes | 17:46 |
hub_cap | dmakogon_: do we have a need for it? | 17:47 |
hub_cap | remember this is the mysql implementation we are talking about, not "every" impl with more than one mount point etc.. | 17:47 |
dmakogon_ | hub_cap, yes, guest doing pull of mysql adming password for my.cnf | 17:47 |
dmakogon_ | hub_cap, now we need datadir | 17:47 |
hub_cap | right | 17:47 |
hub_cap | grep | 17:48 |
dmakogon_ | hub_cap, grep until it's more complicated then one line | 17:48 |
hub_cap | correct | 17:49 |
hub_cap | thats what i said | 17:49 |
dmakogon_ | ok =) | 17:49 |
hub_cap | u said 'i think its time', which, to me, assumed you wanted to use six for finding mysql lib... which we dont need (yet) | 17:49 |
dmakogon_ | when i was working on cassandra review, i used python yaml to parse and edit config file | 17:49 |
dmakogon_ | hub_cap, i mean "it's time to use six for usage of from six.moves import configparser | 17:50 |
hub_cap | Ah | 17:51 |
dmakogon_ | hub_cap, cp16net used it in configuration review | 17:51 |
dmakogon_ | hub_cap, and he, i suppose, succeeded | 17:51 |
*** cweid has joined #openstack-trove | 17:52 | |
*** yidclare has joined #openstack-trove | 17:57 | |
*** rongze has quit IRC | 17:58 | |
cp16net | yeah i think i needed to because of the changes to make for parsing | 18:01 |
*** SushilKM has quit IRC | 18:02 | |
dmakogon_ | cp16net, yup =) | 18:04 |
dmakogon_ | cp16net, be ready to "fight" tomorrow at meeting | 18:04 |
dmakogon_ | cp16net, )) | 18:04 |
hub_cap | HAH | 18:07 |
*** SushilKM has joined #openstack-trove | 18:08 | |
dmakogon_ | hub_cap, we had discussed configuration delivery on guest side more than once, even twice. | 18:09 |
dmakogon_ | hub_cap, our vision is different | 18:10 |
hub_cap | dmakogon_: hehe, sounds like itll be a fight then!!! | 18:10 |
hub_cap | TO THE DEATH | 18:10 |
dmakogon_ | that is why i have set topic for tomorrow meeting | 18:10 |
hub_cap | ok ill add a word to your meeting topic dmakogon_ | 18:11 |
hub_cap | dmakogon_: cp16net done. https://wiki.openstack.org/wiki/Meetings/TroveMeeting#Agenda_for_the_next_meeting | 18:12 |
*** SushilKM__ has joined #openstack-trove | 18:12 | |
dmakogon_ | hub_cap, lol | 18:12 |
dmakogon_ | lmfao | 18:12 |
dmakogon_ | cp16net, i suppose, we will fight, DEATHMATCH | 18:13 |
*** SushilKM has quit IRC | 18:13 | |
*** vipul is now known as vipul-away | 18:15 | |
hub_cap | DEATHMATCH | 18:16 |
amcrn | no more changes to configurations, i want that bad boy to merge | 18:19 |
amcrn | and there's no way cp16net wants it less than i do :P | 18:20 |
dmakogon_ | amcrn, we need it to discuss, involving everybody | 18:21 |
amcrn | discussion is so 2012 ;) | 18:22 |
dmakogon_ | amcrn, there are several problems with that, even with current implementation of mysql support | 18:22 |
*** SushilKM__ has quit IRC | 18:23 | |
dmakogon_ | i'd like to discover then only tomorrow, at the meeting, until that i would review current patchset | 18:23 |
amcrn | i was being sarcastic; i'll apply judicious "/s" more often, my humor is perhaps a bit too dry | 18:24 |
dmakogon_ | i saw that cp16net didn't took into accout any of my suggestions =( | 18:24 |
*** datsun180b is now known as datsun_F40PH | 18:25 | |
datsun_F40PH | that should be automatic by now, wtf colloquy | 18:25 |
*** SushilKM has joined #openstack-trove | 18:27 | |
* dmakogon_ gone out for a while | 18:27 | |
datsun_F40PH | grapex's hdd went kablooey again so he's going to be recovering that for a while | 18:31 |
*** vipul-away is now known as vipul | 18:32 | |
*** yogesh has joined #openstack-trove | 18:32 | |
*** openstackgerrit has quit IRC | 18:33 | |
*** openstackgerrit has joined #openstack-trove | 18:33 | |
openstackgerrit | A change was merged to openstack/trove-integration: Updates .gitignore https://review.openstack.org/58725 | 18:38 |
openstackgerrit | Illia Khudoshyn proposed a change to openstack/trove: Remove 'mount_point' from guestagent RPC API https://review.openstack.org/57189 | 18:46 |
*** rongze has joined #openstack-trove | 18:49 | |
hub_cap | dmakogon_: nice blog on trove | 18:50 |
hub_cap | ikhudoshyn_: ikhudoshyn nice update to your review. i will put a note saying i requested it and +2 it now | 18:51 |
ikhudoshyn_ | hub_cap: thanks but.. do u mean "grep xxx" update? | 18:51 |
ikhudoshyn_ | if it is, then let's wait for robots | 18:52 |
hub_cap | yes ikhudoshyn_ | 18:52 |
hub_cap | and ya i wait for robots :) | 18:52 |
hub_cap | i had one question though, i just saw it | 18:52 |
hub_cap | https://review.openstack.org/#/c/57189/9/trove/taskmanager/models.py | 18:52 |
hub_cap | line 337 | 18:52 |
hub_cap | do you think we still need volume_info = {}, or can we just return the device_path as a string ? | 18:53 |
ikhudoshyn_ | well, i looked at it and saw volume_info used as a dict many times | 18:53 |
ikhudoshyn_ | but i cant say for sure we need a dict here | 18:54 |
*** rongze has quit IRC | 18:54 | |
ikhudoshyn_ | i mean in other places some other entiries exist in volume-info | 18:54 |
hub_cap | right.. but here its not volume_info anymore, its just device_path | 18:55 |
hub_cap | how much work would it be to change that return to device_path and not use it like a dict | 18:55 |
ikhudoshyn_ | thats what im talking about. That return seem to be treated in many places as a dict, not a simple value. There is a family of such functions and they return dict.. | 19:00 |
ikhudoshyn_ | It would be necessary to digg thu all the code.. | 19:00 |
ikhudoshyn_ | https://review.openstack.org/#/c/57189/9/trove/taskmanager/models.py @ line 177 and below | 19:03 |
ikhudoshyn_ | volume_info is retrieved from several functions as a dict, I would prefer having them all as dicts | 19:04 |
hub_cap | sure and in all the cases where you woudl expect > 1 item i would agree | 19:05 |
ikhudoshyn_ | ?? but in this case.. | 19:06 |
ikhudoshyn_ | do u still want plain value? | 19:06 |
ikhudoshyn_ | another piece, https://review.openstack.org/#/c/59078/ pls)) | 19:07 |
hub_cap | i think its too much work for this review though ikhudoshyn_ | 19:08 |
openstackgerrit | Dan Nguyen proposed a change to openstack/trove: Validate databases for user grants https://review.openstack.org/59529 | 19:11 |
hub_cap | ikhudoshyn_: where else do we call that function? iirc, that init's the _ENGINE for sqlalchemy w/ the the models | 19:12 |
ikhudoshyn_ | hub_cap: sry, what function? | 19:13 |
hub_cap | the last review you sent me ikhudoshyn_ | 19:13 |
hub_cap | configure_db | 19:13 |
ikhudoshyn_ | ah.. | 19:13 |
hub_cap | ohhhhhhh the guest | 19:13 |
hub_cap | it doesnt need to talk to db anymore | 19:13 |
* hub_cap sees the lightbulb flicker on and then crack above his head | 19:14 | |
* datsun_F40PH hilarious trombone tune | 19:14 | |
hub_cap | nice datsun_F40PH | 19:14 |
hub_cap | gees ikhudoshyn_ maybe i shouldve just read your commit msg.................. | 19:14 |
hub_cap | sorry i just came in from lookin at a big bird that was in my backyard, and no he was not yellow | 19:15 |
SlickNik | Who reads commit messages? | 19:15 |
hub_cap | SlickNik: exactly | 19:15 |
ikhudoshyn_ | hub_cap: thats ok)) | 19:16 |
hub_cap | looked like some sort of hawk, i only saw it from far away | 19:16 |
ikhudoshyn_ | cool, I only can see crows and sparrows around here | 19:17 |
ikhudoshyn_ | ( | 19:17 |
hub_cap | ikhudoshyn_: :( | 19:17 |
ikhudoshyn_ | well, we have more in zoo)) | 19:18 |
datsun_F40PH | hub_cap: what color was it | 19:18 |
hub_cap | datsun_F40PH: grey/brown chest, grey/blue wings and a grey face... i think | 19:19 |
datsun_F40PH | sounds like a condor or vulture | 19:19 |
datsun_F40PH | http://en.wikipedia.org/wiki/List_of_birds_of_California#Hawks.2C_Kites.2C_and_Eagles get your Audubon on | 19:19 |
esp | hub_cap: glad to see you get to enjoy nature in your office | 19:20 |
hub_cap | esp: :) | 19:20 |
hub_cap | datsun_F40PH: ya it sucks i didnt get to see the tail | 19:21 |
datsun_F40PH | http://en.wikipedia.org/wiki/American_Black_Vulture ? | 19:22 |
openstackgerrit | Sushil Kumar proposed a change to openstack/trove: Removes unused import statements https://review.openstack.org/59696 | 19:22 |
hub_cap | naw datsun_F40PH.. i saw a "picture guide to hawks in california" and i couldnt recognise it | 19:23 |
hub_cap | might not have been a hawk but it was a BIG bird | 19:24 |
datsun_F40PH | was it a helicopter | 19:24 |
datsun_F40PH | those can get pretty big | 19:24 |
datsun_F40PH | they make a lot of noise though | 19:24 |
openstackgerrit | SlickNik proposed a change to openstack/trove: Added ability to pass custom command-line options to backup runner https://review.openstack.org/58062 | 19:24 |
hub_cap | datsun_F40PH: thx for the heads up, i found out what species the bird was | 19:25 |
hub_cap | http://upload.wikimedia.org/wikipedia/commons/6/66/AH-64D_Apache_Longbow.jpg | 19:25 |
datsun_F40PH | aww it's got babies | 19:25 |
SlickNik | lol, the first thing that I thought of when I saw that link was: "What's the longbow project? haven't really heard of that one…" | 19:27 |
hub_cap | lol SlickNik | 19:27 |
hub_cap | and lol datsun_F40PH | 19:27 |
openstackgerrit | Debasish Chowdhury proposed a change to openstack/trove: Unittest Case for get_user https://review.openstack.org/48201 | 19:32 |
*** vipul is now known as vipul-away | 19:33 | |
*** vipul-away is now known as vipul | 19:33 | |
*** jcooley_ has quit IRC | 19:37 | |
*** jcooley_ has joined #openstack-trove | 19:37 | |
*** denis_makogon has joined #openstack-trove | 19:39 | |
* denis_makogon back | 19:39 | |
denis_makogon | hub_cap, did you reviewed Trove Heat Resource ? | 19:39 |
hub_cap | denis_makogon: i havent got to it yet | 19:41 |
*** jcooley_ has quit IRC | 19:41 | |
*** jcooley_ has joined #openstack-trove | 19:42 | |
*** yogesh has quit IRC | 19:45 | |
*** yogesh has joined #openstack-trove | 19:45 | |
datsun_F40PH | hub_cap: i hope adrian otto is bothering you about #solum | 19:46 |
datsun_F40PH | i can only answer so many questions about our gerrit setup before my answers all default to shrugs and "a wizard did it" | 19:47 |
*** jcooley_ has quit IRC | 19:47 | |
*** jcooley_ has joined #openstack-trove | 19:47 | |
denis_makogon | lol, solum community bigger then any other not-even-incubated projects has | 19:47 |
hub_cap | datsun_F40PH: we are talking yes | 19:48 |
denis_makogon | datsun_F40PH, ashestakov, cp16net, vipul, SlickNik, grapex, esmute, ikhudoshyn, isviridov - guys now it's time to review Trove heat resource -https://review.openstack.org/#/c/59231/4 | 19:49 |
datsun_F40PH | Now ist ze time in Shprockets vhen ve dahnse! | 19:50 |
*** yogesh has quit IRC | 19:50 | |
kevinconway | datsun_F40PH: hey… that's not german... | 19:51 |
*** rongze has joined #openstack-trove | 19:51 | |
*** jcooley_ has quit IRC | 19:51 | |
denis_makogon | cp16net, hub_cap - how about that kind of deathmatch - http://www.mygamesix.com/uploads/posts/2010-10/1287725393_kadr00.jpg | 19:51 |
*** jcooley_ has joined #openstack-trove | 19:52 | |
*** vipul is now known as vipul-away | 19:53 | |
*** rongze has quit IRC | 19:56 | |
openstackgerrit | Illia Khudoshyn proposed a change to openstack/trove: Remove 'mount_point' from guestagent RPC API https://review.openstack.org/57189 | 20:00 |
*** jcru has quit IRC | 20:01 | |
*** kevinconway has quit IRC | 20:01 | |
*** kevinconway has joined #openstack-trove | 20:02 | |
*** vipul-away is now known as vipul | 20:04 | |
*** jcooley_ has quit IRC | 20:06 | |
*** pdmars_ has joined #openstack-trove | 20:07 | |
*** pdmars_ has quit IRC | 20:07 | |
*** pdmars_ has joined #openstack-trove | 20:08 | |
*** kevinconway has quit IRC | 20:08 | |
*** demorris has joined #openstack-trove | 20:08 | |
*** pdmars has quit IRC | 20:09 | |
*** kevinconway has joined #openstack-trove | 20:11 | |
*** datsun180b has joined #openstack-trove | 20:11 | |
*** vipul is now known as vipul-away | 20:11 | |
*** vipul-away is now known as vipul | 20:12 | |
*** jcru has joined #openstack-trove | 20:12 | |
*** datsun_F40PH has quit IRC | 20:13 | |
*** datsun180b has quit IRC | 20:14 | |
*** datsun180b has joined #openstack-trove | 20:14 | |
denis_makogon | SlickNik, could you please take a look at https://review.openstack.org/#/c/59410/3 | 20:23 |
*** pdmars has joined #openstack-trove | 20:24 | |
*** pdmars_ has quit IRC | 20:25 | |
*** vipul is now known as vipul-away | 20:27 | |
*** SushilKM has quit IRC | 20:27 | |
*** plodronio has quit IRC | 20:28 | |
*** pdmars has quit IRC | 20:29 | |
*** ikhudoshyn_ has quit IRC | 20:30 | |
*** vipul-away is now known as vipul | 20:45 | |
*** achampion has quit IRC | 20:46 | |
*** achampion has joined #openstack-trove | 20:47 | |
*** jmontemayor_ has joined #openstack-trove | 20:50 | |
*** jcooley_ has joined #openstack-trove | 20:51 | |
*** jcooley_ has quit IRC | 20:53 | |
*** rongze has joined #openstack-trove | 20:53 | |
*** yogesh has joined #openstack-trove | 20:56 | |
*** rongze has quit IRC | 20:58 | |
*** kevinconway has quit IRC | 20:59 | |
*** jmontemayor_ has quit IRC | 20:59 | |
*** pdmars has joined #openstack-trove | 21:00 | |
*** datsun_F40PH has joined #openstack-trove | 21:00 | |
*** jcru has quit IRC | 21:00 | |
*** vipul is now known as vipul-away | 21:00 | |
*** kevinconway has joined #openstack-trove | 21:01 | |
*** datsun180b has quit IRC | 21:03 | |
*** datsun_F40PH is now known as datsun180b | 21:03 | |
hub_cap | hey dmakogon_ around? | 21:04 |
*** yogesh has quit IRC | 21:06 | |
*** yogesh has joined #openstack-trove | 21:08 | |
*** achampion has quit IRC | 21:08 | |
*** achampion has joined #openstack-trove | 21:09 | |
*** jcru has joined #openstack-trove | 21:09 | |
*** vipul-away is now known as vipul | 21:19 | |
*** timfreund has quit IRC | 21:20 | |
*** radez is now known as radez_g0n3 | 21:23 | |
*** yogesh has quit IRC | 21:24 | |
*** jcooley_ has joined #openstack-trove | 21:24 | |
*** yogesh has joined #openstack-trove | 21:25 | |
denis_makogon | hub_cap, yes | 21:27 |
hub_cap | denis_makogon: hey i made a comment on your conductor work | 21:28 |
hub_cap | we need a new blueprint or maybe even a bug report since it was missed | 21:28 |
hub_cap | it needs to be independent and il be backporting it to i1 (i1 is being cut today) | 21:28 |
*** yogesh has quit IRC | 21:29 | |
denis_makogon | hub_cap, i saw, i'll do that | 21:29 |
denis_makogon | hub_cap, btw, -2 was vary unexpected =( | 21:29 |
*** yogesh has joined #openstack-trove | 21:31 | |
denis_makogon | hub_cap, which option is more acceptable: BP or Bug-report ? | 21:31 |
denis_makogon | i'll file a bug | 21:31 |
*** plodronio has joined #openstack-trove | 21:32 | |
denis_makogon | hub_cap, could you please set Importance https://bugs.launchpad.net/trove/+bug/1257489 ? | 21:35 |
*** vipul is now known as vipul-away | 21:39 | |
*** vipul-away is now known as vipul | 21:39 | |
robertmyers | hub_cap: this should also be in icehouse-1 (low) priority https://bugs.launchpad.net/trove/+bug/1234869 | 21:42 |
denis_makogon | robertmyers, i would start working on dashboard again, if you don't mind | 21:43 |
robertmyers | denis_makogon: fine by me | 21:43 |
denis_makogon | robertmyers, great =) | 21:43 |
denis_makogon | which client version available at pypi | 21:44 |
denis_makogon | ? | 21:44 |
robertmyers | denis_makogon: Its all open source anyway... so I can't stop you either :) | 21:44 |
robertmyers | it uses >= 1.0.0 | 21:44 |
denis_makogon | robertmyers, what does OS requirements says ? | 21:45 |
robertmyers | so should work with the latest, the problem is the gate uses openstack pypi | 21:45 |
robertmyers | denis_makogon: >=1.0.0 | 21:45 |
denis_makogon | robertmyers, i'm asking you because i want to keep you tuned | 21:46 |
robertmyers | feel free to add me to the reviews | 21:46 |
denis_makogon | robertmyers, does openstack pypi have lates version ? | 21:47 |
robertmyers | denis_makogon: yeah, at the time (a couple weeks ago) they only had 1.0.0 not 1.0.3 | 21:47 |
robertmyers | Not sure who updates that | 21:48 |
hub_cap | denis_makogon: the reason i -2 things is because "I want to re-review it" | 21:48 |
hub_cap | not because of anythign else but that | 21:48 |
denis_makogon | hub_cap, does -infra could answer question about client version ? | 21:49 |
hub_cap | _why_ i want to re-review something will be different every time, so dont think of a -2 as anything more than "i'd like to view the changes" | 21:49 |
hub_cap | possibly but i migth be able to answer it, sec | 21:49 |
hub_cap | what is the question denis_makogon ? | 21:50 |
denis_makogon | hub_cap, me and robertmyers wondering about troveclient version in openstack pypi | 21:51 |
hub_cap | denis_makogon: im sure automation does that | 21:52 |
hub_cap | http://pypi.openstack.org/openstack/python-troveclient/ | 21:52 |
hub_cap | we have all our versions in it | 21:52 |
denis_makogon | hub_cap, i'm ok with -2, i understood the reasons, i'm not so happy, because i've got to many of "do not merge", my gerrit page now has lots of red colors, i don't like red | 21:52 |
hub_cap | denis_makogon: red is a natural ocurring element in openstack | 21:52 |
hub_cap | :) | 21:52 |
denis_makogon | hub_cap, i like olive color | 21:53 |
kevinconway | denis_makogon: black? | 21:53 |
denis_makogon | hub_cap, orange | 21:53 |
amcrn | denis_makogon: if you were red/green color blind, this wouldn't have been a problem | 21:53 |
openstackgerrit | A change was merged to openstack/trove: Removes unused import statements https://review.openstack.org/59696 | 21:54 |
denis_makogon | if it would be red/gree - it would remind me a christmas every time, i like christmas | 21:54 |
hub_cap | amcrn: HAH | 21:54 |
*** rongze has joined #openstack-trove | 21:55 | |
amcrn | the introduction of this conductor means i have to write more puppet… *grumble grumble* | 21:57 |
*** timfreund has joined #openstack-trove | 21:59 | |
*** rongze has quit IRC | 22:00 | |
hub_cap | amcrn: lulz | 22:00 |
hub_cap | i hear salt will recognise new stuff and write it for u | 22:01 |
hub_cap | right, salt is magic? | 22:01 |
amcrn | so i've heard, or maybe it's ansible, or chef, or <newest flavor of the month> | 22:02 |
hub_cap | oh right ansible | 22:02 |
datsun180b | don't be ridiculous | 22:02 |
*** pdmars has quit IRC | 22:02 | |
datsun180b | it's pecan that solves all of our problems and can turn lead into gold and water into blood | 22:03 |
*** demorris has quit IRC | 22:03 | |
hub_cap | youre right datsun180b , by the time amcrn re-impls it, salt will be old and something else will be shiny | 22:03 |
amcrn | the arguments you see on mailing lists about these tools are reminiscent of vi vs. emacs conversations of old | 22:03 |
hub_cap | amcrn: for real... datsun180b and i know which tool is the best for the job | 22:03 |
amcrn | undocumented shell commands? | 22:04 |
amcrn | ;) | 22:04 |
*** plodronio has quit IRC | 22:04 | |
*** plodronio_ has joined #openstack-trove | 22:04 | |
datsun180b | amcrn: cat `which esoteric-shell-command` is all the docs you need | 22:04 |
kevinconway | i have a bash agent that listens on a rabbit queue | 22:04 |
kevinconway | you could use that | 22:04 |
amcrn | never change kevinconway, never change | 22:05 |
amcrn | i find myself laughing at nearly everyone of your sarcastic git commit comments | 22:05 |
*** tanisdl has quit IRC | 22:06 | |
amcrn | on a different note, how is rackspace planning on deploying conductor? will api + taskmgr + conductor be on a single box, and have an active ha pair? | 22:08 |
hub_cap | amcrn: speaking from someone whos not deploying it, id guess HA... if we built a service that wasnt naturally HA then we'd have a problem | 22:10 |
amcrn | well, i was more looking for if someone found a reasonable reason to split the conductor into a separate instance/box | 22:10 |
amcrn | i'll just assume the triple-up with a pair or two for ha | 22:12 |
amcrn | makes deployment easier | 22:12 |
hub_cap | oh ya i think so for now | 22:15 |
hub_cap | but i know that some people have seen deployment issues for very large nova instances w/ a single conductor | 22:15 |
hub_cap | so thye end up putting it on the local box of every compute | 22:15 |
amcrn | interesting, i'll keep that in mind, thanks :) | 22:18 |
denis_makogon | yes, something like HA mode for conductor | 22:18 |
hub_cap | im running to the store.. ill be back in ~30 min | 22:19 |
*** tanisdl has joined #openstack-trove | 22:24 | |
*** denis_makogon has quit IRC | 22:34 | |
*** cweid has quit IRC | 22:35 | |
*** jmontemayor_ has joined #openstack-trove | 22:38 | |
*** jmontemayor_ has quit IRC | 22:39 | |
*** jmontemayor has quit IRC | 22:40 | |
*** achampio1 has joined #openstack-trove | 22:54 | |
*** achampion has quit IRC | 22:57 | |
*** rongze has joined #openstack-trove | 22:57 | |
*** rongze has quit IRC | 23:02 | |
amcrn | hrm, was there a discussion nullifying the desire for datastore_update from trove-manage to do an upsert? | 23:03 |
amcrn | (i remember quite a few folks agreeing that was a miss in the initial change-set, but i don't see any open reviews rectifying it) | 23:04 |
*** robertmyers has quit IRC | 23:05 | |
amcrn | https://github.com/openstack/trove/blob/d488e37a698d6f1727d3abeb31d7c6a7699af778/trove/datastore/models.py#L160-L164 looks like it handles it, so... | 23:12 |
*** yogesh has quit IRC | 23:17 | |
*** yogesh has joined #openstack-trove | 23:17 | |
*** jcooley_ has quit IRC | 23:18 | |
*** yogesh has quit IRC | 23:22 | |
hub_cap | amcrn: im not sure what yer talkin about | 23:25 |
*** _kpom_ has quit IRC | 23:25 | |
amcrn | there was some commentary earlier last week about being forced to manually insert a datastore because the code only updated, not upserted | 23:26 |
amcrn | or, maybe i misinterpreted the situation at hand | 23:26 |
hub_cap | amcrn: can u put it on the agenda so we can talk about it? there were a few _to be updated_ things that were missed in v1 of the datastores | 23:27 |
hub_cap | brb amcrn restarting | 23:28 |
amcrn | i'm re-testing a full blown install to confirm (before adding to agenda) | 23:28 |
amcrn | ok, cool | 23:28 |
hub_cap | ugh, xfce is way nicer than gnome to look at | 23:34 |
hub_cap | aww ya and i got my monaco back... god i love this font | 23:40 |
*** achampio1 has quit IRC | 23:44 | |
*** rnirmal has quit IRC | 23:45 | |
openstackgerrit | SlickNik proposed a change to openstack/python-troveclient: Fixed misspellings of common words https://review.openstack.org/59876 | 23:50 |
hub_cap | sweet! xfce honors my .Xmodmap settings horray | 23:51 |
*** tanisdl_ has joined #openstack-trove | 23:51 | |
*** tanisdl has quit IRC | 23:52 | |
*** tanisdl_ is now known as tanisdl | 23:52 | |
openstackgerrit | SlickNik proposed a change to openstack/trove: Fixed misspellings of common words https://review.openstack.org/59877 | 23:52 |
*** openstackgerrit has quit IRC | 23:56 | |
*** openstackgerrit has joined #openstack-trove | 23:56 | |
*** rongze has joined #openstack-trove | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!