| *** pdmars has quit IRC | 00:01 | |
| *** rueben has joined #openstack-trove | 00:02 | |
| *** rueben has quit IRC | 00:02 | |
| *** achampion has quit IRC | 00:12 | |
| *** michael-yu has quit IRC | 00:35 | |
| *** achampion has joined #openstack-trove | 00:38 | |
| *** achampion has quit IRC | 00:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-troveclient: Updated from global requirements https://review.openstack.org/99095 | 00:47 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/96275 | 00:48 |
| *** ramashri has quit IRC | 00:52 | |
| *** makkie has joined #openstack-trove | 00:53 | |
| makkie | hi | 00:57 |
| makkie | unfortunately, i encountered non-deterministic failures many times in trove-gate... | 00:57 |
| makkie | https://review.openstack.org/#/c/99001/ | 00:57 |
| makkie | can i request trove-gate re-check here? | 00:57 |
| *** rwsu has quit IRC | 00:58 | |
| *** kevinconway has quit IRC | 01:07 | |
| *** mattgriffin has quit IRC | 01:11 | |
| *** grapex has joined #openstack-trove | 01:15 | |
| *** grapex has joined #openstack-trove | 01:15 | |
| *** matsuhashi has joined #openstack-trove | 01:17 | |
| *** mattgriffin has joined #openstack-trove | 01:22 | |
| *** mattgriffin has quit IRC | 01:22 | |
| *** saurabhs has left #openstack-trove | 01:22 | |
| *** michael-yu has joined #openstack-trove | 01:23 | |
| *** tvoran has joined #openstack-trove | 01:25 | |
| *** eghobo has quit IRC | 01:28 | |
| *** michael-yu has quit IRC | 01:30 | |
| *** nosnos has joined #openstack-trove | 01:31 | |
| SlickNik | makkie: Sure thing. | 01:32 |
| SlickNik | makkie: are you on launchpad? | 01:32 |
| makkie | SlickNik: thank you! | 01:35 |
| makkie | SlickNik: i'm shuichiro-makigaki on launchpad | 01:35 |
| *** ViswaV_ has joined #openstack-trove | 01:35 | |
| SlickNik | makkie: I'm going to add you to the trove-developers group, so you will be able to kick-off the gate checks yourself. | 01:36 |
| makkie | SlickNik: wow, sounds good, thank you very much! | 01:37 |
| SlickNik | No problem at all. Thank you for taking the time to make trove better! :) | 01:38 |
| *** ViswaV has quit IRC | 01:38 | |
| SlickNik | Done | 01:38 |
| makkie | i'm interested in DBaaS as database administrator, and want to introduce trove to my company | 01:41 |
| makkie | anyway, thank you! | 01:41 |
| SlickNik | You're welcome. That's awesome! We're here to help if you need any assistance with Trove-related questions. | 01:44 |
| SlickNik | If you don't mind me asking, which company are you at? | 01:45 |
| *** ramishra has joined #openstack-trove | 01:50 | |
| *** tvoran has quit IRC | 01:50 | |
| *** annashe__ has quit IRC | 01:54 | |
| makkie | Rakuten, which is japanese e-commerce web site and parent company of buy.com, kobo, priceminister and so on | 01:55 |
| makkie | i got emails from launchpad and gerrit. i'm happy to join trove developers group :-) | 01:57 |
| fifieldt | Rakuten! すごい :) | 02:05 |
| *** tvoran has joined #openstack-trove | 02:05 | |
| *** tvoran has quit IRC | 02:11 | |
| *** tvoran has joined #openstack-trove | 02:11 | |
| *** mattgriffin has joined #openstack-trove | 02:13 | |
| makkie | hi, tom! basically it's a good company, but database infrastructure and management style are sometimes too legacy... | 02:19 |
| makkie | then, i believe trove will save me | 02:20 |
| *** coolsvap|afk is now known as coolsvap | 02:25 | |
| *** matsuhashi has quit IRC | 02:29 | |
| *** matsuhashi has joined #openstack-trove | 02:32 | |
| *** haomai___ has quit IRC | 02:36 | |
| *** achampion has joined #openstack-trove | 02:44 | |
| *** haomaiwa_ has joined #openstack-trove | 02:45 | |
| *** achampion has quit IRC | 02:54 | |
| *** matsuhashi has quit IRC | 03:00 | |
| *** mattgriffin has quit IRC | 03:14 | |
| *** mattgriffin has joined #openstack-trove | 03:16 | |
| *** nosnos has quit IRC | 03:27 | |
| *** michael-yu has joined #openstack-trove | 03:41 | |
| *** michael-yu has quit IRC | 03:44 | |
| *** annashe__ has joined #openstack-trove | 03:46 | |
| *** tvoran has quit IRC | 03:49 | |
| *** eghobo has joined #openstack-trove | 03:51 | |
| *** mattgriffin has quit IRC | 03:52 | |
| *** matsuhashi has joined #openstack-trove | 03:53 | |
| makkie | hello again! | 03:55 |
| makkie | thank you for your support. it took long time, but i can pass trove-gate now. | 03:55 |
| makkie | could you please review my changes? | 03:55 |
| makkie | https://review.openstack.org/#/c/99001/ | 03:55 |
| *** juantwo has quit IRC | 04:00 | |
| *** nosnos has joined #openstack-trove | 04:07 | |
| *** matsuhashi has quit IRC | 04:12 | |
| *** matsuhashi has joined #openstack-trove | 04:12 | |
| *** sgotliv has joined #openstack-trove | 04:15 | |
| *** matsuhashi has quit IRC | 04:17 | |
| *** haomaiwa_ has quit IRC | 04:17 | |
| *** haomaiwang has joined #openstack-trove | 04:18 | |
| *** matsuhashi has joined #openstack-trove | 04:21 | |
| *** Longgeek has joined #openstack-trove | 04:26 | |
| *** haomai___ has joined #openstack-trove | 04:27 | |
| *** haomaiwang has quit IRC | 04:31 | |
| *** denis_makogon has joined #openstack-trove | 04:35 | |
| *** matsuhashi has quit IRC | 04:36 | |
| *** matsuhashi has joined #openstack-trove | 04:36 | |
| *** rhodgin has quit IRC | 04:38 | |
| *** mattgriffin has joined #openstack-trove | 04:38 | |
| *** matsuhashi has quit IRC | 04:39 | |
| *** eghobo has quit IRC | 04:39 | |
| *** matsuhashi has joined #openstack-trove | 04:39 | |
| *** eghobo has joined #openstack-trove | 04:40 | |
| *** achampion has joined #openstack-trove | 04:44 | |
| *** achampion has quit IRC | 04:53 | |
| *** ramishra has quit IRC | 05:00 | |
| *** ramishra has joined #openstack-trove | 05:00 | |
| *** ramishra has quit IRC | 05:03 | |
| *** ramishra has joined #openstack-trove | 05:03 | |
| *** matsuhashi has quit IRC | 05:11 | |
| *** matsuhashi has joined #openstack-trove | 05:11 | |
| *** matsuhashi has quit IRC | 05:16 | |
| *** michael-yu has joined #openstack-trove | 05:16 | |
| *** matsuhashi has joined #openstack-trove | 05:17 | |
| *** matsuhashi has quit IRC | 05:24 | |
| *** matsuhashi has joined #openstack-trove | 05:24 | |
| *** grapex has quit IRC | 05:39 | |
| *** juantwo has joined #openstack-trove | 05:42 | |
| *** achampion has joined #openstack-trove | 05:42 | |
| *** juantwo has joined #openstack-trove | 05:42 | |
| *** achampion has quit IRC | 05:58 | |
| *** michael-yu has quit IRC | 06:03 | |
| *** michael-yu has joined #openstack-trove | 06:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Imported Translations from Transifex https://review.openstack.org/100083 | 06:07 |
| *** michael-yu has quit IRC | 06:10 | |
| *** denis_makogon has quit IRC | 06:19 | |
| *** openstackgerrit has quit IRC | 06:19 | |
| *** cp16net has quit IRC | 06:19 | |
| *** denis_makogon has joined #openstack-trove | 06:23 | |
| *** openstackgerrit has joined #openstack-trove | 06:23 | |
| *** cp16net has joined #openstack-trove | 06:23 | |
| *** mattgriffin has quit IRC | 06:34 | |
| *** miqui has quit IRC | 06:36 | |
| *** grapex has joined #openstack-trove | 06:40 | |
| *** sgotliv has quit IRC | 06:41 | |
| *** juantwo_ has joined #openstack-trove | 06:48 | |
| *** ramishra has quit IRC | 06:48 | |
| *** grapex has quit IRC | 06:49 | |
| *** ramishra has joined #openstack-trove | 06:51 | |
| *** juantwo has quit IRC | 06:52 | |
| *** sgotliv has joined #openstack-trove | 07:00 | |
| *** juantwo has joined #openstack-trove | 07:06 | |
| *** matsuhashi has quit IRC | 07:08 | |
| *** matsuhas_ has joined #openstack-trove | 07:08 | |
| *** eguz has joined #openstack-trove | 07:08 | |
| *** juantwo_ has quit IRC | 07:09 | |
| *** sgotliv has quit IRC | 07:10 | |
| *** sgotliv has joined #openstack-trove | 07:10 | |
| *** eguz has quit IRC | 07:11 | |
| *** eghobo has quit IRC | 07:12 | |
| *** ramishra has quit IRC | 07:18 | |
| *** annashe__ has quit IRC | 07:18 | |
| openstackgerrit | Sergey Gotliv proposed a change to openstack/trove: Fix enable on boot when working with systemd and symlinked units https://review.openstack.org/100504 | 07:22 |
| *** julienvey has joined #openstack-trove | 07:25 | |
| *** flaper87|afk is now known as flaper87 | 07:33 | |
| *** amytron has quit IRC | 07:38 | |
| *** grapex has joined #openstack-trove | 07:40 | |
| *** grapex has quit IRC | 07:45 | |
| *** denis_makogon has quit IRC | 07:51 | |
| *** coolsvap is now known as coolsvap|afk | 07:55 | |
| *** julienvey has quit IRC | 08:18 | |
| *** ramishra has joined #openstack-trove | 08:18 | |
| *** julienvey has joined #openstack-trove | 08:18 | |
| *** haomai___ has quit IRC | 08:28 | |
| makkie | SlickNik: thank you for review! | 08:36 |
| *** boden has joined #openstack-trove | 08:36 | |
| *** denis_makogon has joined #openstack-trove | 08:38 | |
| *** juantwo has quit IRC | 08:39 | |
| *** julienve_ has joined #openstack-trove | 08:45 | |
| *** Longgeek has quit IRC | 08:46 | |
| *** Longgeek has joined #openstack-trove | 08:46 | |
| *** Longgeek has quit IRC | 08:48 | |
| *** Longgeek has joined #openstack-trove | 08:49 | |
| *** julienvey has quit IRC | 08:49 | |
| *** Longgeek has quit IRC | 08:53 | |
| *** Longgeek_ has joined #openstack-trove | 08:53 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Fix poll_until retriever when heat provisioning is enabled https://review.openstack.org/97194 | 08:58 |
| openstackgerrit | Denis M. proposed a change to openstack/trove-integration: Add redstack command that enables heat support https://review.openstack.org/100598 | 09:01 |
| *** matsuhas_ has quit IRC | 09:01 | |
| *** sgotliv has quit IRC | 09:05 | |
| *** matsuhashi has joined #openstack-trove | 09:09 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add datatore/version name into configuration response https://review.openstack.org/98985 | 09:11 |
| *** isviridov|away is now known as isviridov | 09:13 | |
| *** ramishra has quit IRC | 09:16 | |
| *** matsuhashi has quit IRC | 09:38 | |
| *** julienve_ has quit IRC | 09:38 | |
| *** julienvey has joined #openstack-trove | 09:39 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Added manager attribute to datastove-version-show response https://review.openstack.org/100836 | 09:39 |
| *** grapex has joined #openstack-trove | 09:42 | |
| *** grapex has quit IRC | 09:46 | |
| *** ramishra has joined #openstack-trove | 09:57 | |
| *** matsuhashi has joined #openstack-trove | 10:01 | |
| *** matsuhashi has quit IRC | 10:06 | |
| *** annashe__ has joined #openstack-trove | 10:07 | |
| *** matsuhashi has joined #openstack-trove | 10:08 | |
| *** sgotliv has joined #openstack-trove | 10:08 | |
| *** ramishra has quit IRC | 10:09 | |
| *** matsuhashi has quit IRC | 10:17 | |
| *** matsuhashi has joined #openstack-trove | 10:18 | |
| openstackgerrit | A change was merged to openstack/trove: Reverting deleted tests https://review.openstack.org/99353 | 10:22 |
| *** makkie_ has joined #openstack-trove | 10:29 | |
| *** makkie has quit IRC | 10:29 | |
| openstackgerrit | A change was merged to openstack/trove: Fix inheritance for ConfigurationParameterDeleted https://review.openstack.org/99391 | 10:34 |
| *** Longgeek_ has quit IRC | 10:58 | |
| *** Longgeek has joined #openstack-trove | 10:59 | |
| *** ramishra has joined #openstack-trove | 11:10 | |
| *** Longgeek has quit IRC | 11:13 | |
| *** Longgeek has joined #openstack-trove | 11:14 | |
| *** ramishra has quit IRC | 11:14 | |
| *** denis_makogon has quit IRC | 11:17 | |
| *** setho2 has quit IRC | 11:17 | |
| *** Longgeek has quit IRC | 11:19 | |
| *** Longgeek has joined #openstack-trove | 11:19 | |
| *** sgotliv has quit IRC | 11:22 | |
| *** setho has joined #openstack-trove | 11:25 | |
| *** denis_makogon has joined #openstack-trove | 11:25 | |
| *** number80 has quit IRC | 11:32 | |
| *** annashe__ has quit IRC | 11:40 | |
| *** grapex has joined #openstack-trove | 11:43 | |
| *** grapex has quit IRC | 11:47 | |
| *** matsuhashi has quit IRC | 11:48 | |
| *** matsuhashi has joined #openstack-trove | 11:51 | |
| *** matsuhashi has quit IRC | 11:56 | |
| *** sgotliv has joined #openstack-trove | 11:56 | |
| *** matsuhashi has joined #openstack-trove | 11:56 | |
| *** Longgeek has quit IRC | 11:57 | |
| *** Longgeek has joined #openstack-trove | 11:58 | |
| *** matsuhashi has quit IRC | 12:01 | |
| *** setho2 has joined #openstack-trove | 12:02 | |
| *** setho has quit IRC | 12:03 | |
| *** setho2 has quit IRC | 12:07 | |
| *** denis_makogon has quit IRC | 12:08 | |
| *** ramishra has joined #openstack-trove | 12:11 | |
| *** setho has joined #openstack-trove | 12:11 | |
| *** denis_makogon has joined #openstack-trove | 12:12 | |
| *** ramishra has quit IRC | 12:15 | |
| *** grapex has joined #openstack-trove | 12:43 | |
| *** grapex has quit IRC | 12:47 | |
| *** pdmars has joined #openstack-trove | 12:48 | |
| *** Longgeek has quit IRC | 12:48 | |
| *** pdmars has quit IRC | 12:50 | |
| *** pdmars has joined #openstack-trove | 12:50 | |
| *** tvoran has joined #openstack-trove | 12:52 | |
| *** Longgeek has joined #openstack-trove | 12:57 | |
| openstackgerrit | Theron Voran proposed a change to openstack/trove: Corrects typo in instance models and service https://review.openstack.org/100654 | 13:01 |
| *** makkie_ has quit IRC | 13:10 | |
| *** makkie has joined #openstack-trove | 13:24 | |
| *** jcru has joined #openstack-trove | 13:28 | |
| *** amytron has joined #openstack-trove | 13:29 | |
| *** jcru has quit IRC | 13:34 | |
| openstackgerrit | Julien Vey proposed a change to openstack/trove: Delete undeclared variable in guest-agent API class https://review.openstack.org/98766 | 13:34 |
| openstackgerrit | Julien Vey proposed a change to openstack/trove: Delete undeclared variable in guest-agent API class https://review.openstack.org/98766 | 13:35 |
| makkie | hi | 13:43 |
| peterstac | Any core interested in reviewing the replication guestagent APIs? | 13:43 |
| peterstac | https://review.openstack.org/#/c/91904/ | 13:43 |
| peterstac | #link https://review.openstack.org/#/c/91904/ | 13:43 |
| makkie | I also need one more core review +2. | 13:43 |
| makkie | https://review.openstack.org/#/c/99001/ | 13:43 |
| makkie | could you review my changes? thanks! | 13:43 |
| makkie | with peterstac ;-) | 13:44 |
| peterstac | makkie: Oh, it's not my review - just noticed it's been sitting there for almost a week | 13:45 |
| *** tvoran has quit IRC | 13:45 | |
| peterstac | makkie: And I thought replication was needed quickly ;) | 13:45 |
| *** miqui has joined #openstack-trove | 13:46 | |
| *** jcru has joined #openstack-trove | 13:46 | |
| *** ramishra has joined #openstack-trove | 13:51 | |
| makkie | peterstac: ah, i see. many need core reviewers... they are busy... | 13:52 |
| *** ramishra has quit IRC | 13:56 | |
| *** Longgeek_ has joined #openstack-trove | 13:57 | |
| *** Longgeek has quit IRC | 13:59 | |
| *** edmund has joined #openstack-trove | 14:02 | |
| *** nosnos has quit IRC | 14:03 | |
| *** mattgriffin has joined #openstack-trove | 14:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/96275 | 14:11 |
| *** grapex has joined #openstack-trove | 14:18 | |
| *** rhodgin has joined #openstack-trove | 14:21 | |
| *** esmute has quit IRC | 14:22 | |
| *** haomaiwang has joined #openstack-trove | 14:23 | |
| *** esmute has joined #openstack-trove | 14:25 | |
| *** grapex has quit IRC | 14:25 | |
| *** tvoran has joined #openstack-trove | 14:25 | |
| *** grapex has joined #openstack-trove | 14:26 | |
| *** robertmyers has joined #openstack-trove | 14:26 | |
| *** rwsu has joined #openstack-trove | 14:29 | |
| *** ramishra has joined #openstack-trove | 14:32 | |
| *** kevinconway has joined #openstack-trove | 14:39 | |
| *** Longgeek_ has quit IRC | 14:43 | |
| *** Longgeek has joined #openstack-trove | 14:44 | |
| *** Longgeek has quit IRC | 14:45 | |
| *** Longgeek has joined #openstack-trove | 14:47 | |
| *** ramishra has quit IRC | 14:54 | |
| *** denis_makogon_ has joined #openstack-trove | 15:01 | |
| *** makkie has quit IRC | 15:03 | |
| *** juantwo has joined #openstack-trove | 15:12 | |
| *** annashe__ has joined #openstack-trove | 15:20 | |
| *** Longgeek has quit IRC | 15:20 | |
| *** denis_makogon has quit IRC | 15:28 | |
| *** denis_makogon_ is now known as denis_makogon | 15:28 | |
| denis_makogon | amrith, ping | 15:28 |
| *** dmakogon_ has joined #openstack-trove | 15:28 | |
| denis_makogon | amrith, need to talk about https://review.openstack.org/#/c/99682/ | 15:29 |
| *** juantwo has quit IRC | 15:41 | |
| *** juantwo has joined #openstack-trove | 15:42 | |
| *** ViswaV has joined #openstack-trove | 15:45 | |
| *** ViswaV_ has quit IRC | 15:47 | |
| *** haomaiwang has quit IRC | 15:51 | |
| *** eghobo has joined #openstack-trove | 15:52 | |
| *** haomaiwang has joined #openstack-trove | 15:52 | |
| *** ViswaV has quit IRC | 16:07 | |
| *** haomaiwang has quit IRC | 16:08 | |
| *** ViswaV has joined #openstack-trove | 16:11 | |
| *** sgotliv has quit IRC | 16:12 | |
| amrith | denis_makogon, hi ... | 16:17 |
| *** eghobo has quit IRC | 16:18 | |
| amrith | also https://review.openstack.org/#/c/100598/ | 16:18 |
| amrith | wassup? | 16:18 |
| *** eghobo has joined #openstack-trove | 16:18 | |
| *** jmontemayor has joined #openstack-trove | 16:23 | |
| *** jmontemayor has quit IRC | 16:23 | |
| denis_makogon | amrith, those variables are shared but not the part of test config, thats why i left them there, but out of CONFIG object | 16:25 |
| amrith | so, I guess my question is this. the variable you are accessing are the same. The change really doesn't change anything | 16:26 |
| amrith | config.CONFIG is really the same as config | 16:27 |
| amrith | yes? | 16:27 |
| denis_makogon | and i agree, i can change name of method, https://review.openstack.org/#/c/100598/, but i disagree about disabling heat support seems strange because if user will have any instances, he should delete them before restaring services | 16:27 |
| amrith | let's talk about one review at a time. | 16:27 |
| denis_makogon | amrith, config.CONFIG can be modified through test_begin.conf | 16:27 |
| denis_makogon | ok | 16:27 |
| amrith | effectively, the entire change to run_tests.py changes nothing | 16:28 |
| amrith | yes? | 16:28 |
| denis_makogon | we need to split up config for fake tests and real tests | 16:28 |
| amrith | so, trove/tests/config.py | 16:28 |
| denis_makogon | run_tests.py was changed because config.py was changed | 16:29 |
| denis_makogon | that's all | 16:29 |
| amrith | the last line | 16:30 |
| amrith | or almost the last line | 16:30 |
| amrith | CONFIG = TestConfig() | 16:30 |
| amrith | so, in reality, config.CONFIG is the same as config, yes? | 16:30 |
| denis_makogon | no | 16:30 |
| denis_makogon | config.py contains CONFIG object that represents test config that | 16:31 |
| denis_makogon | config != config.CONFIG | 16:31 |
| denis_makogon | i guess we have mismatch in terms for now, what do you mean by telling "config" ? | 16:32 |
| denis_makogon | is it service configuration file or test config ? | 16:32 |
| denis_makogon | amrith, are you there ? | 16:33 |
| amrith | yes I am | 16:33 |
| denis_makogon | so, config.py, as you can see, contains a CONFIG object that represents a configuration file, config module is a placeholder for that object and other things | 16:35 |
| amrith | so what you are changing in config.py is to move things from TestConfig.dbaas_datastore and make it just config.dbaas_datastore? | 16:35 |
| denis_makogon | amrith, that's why i left things like "dbaas_datastore_version_id" there as globals | 16:35 |
| denis_makogon | amrith, yes | 16:36 |
| denis_makogon | amrith, ther reason is to make tests datastore-independent | 16:36 |
| denis_makogon | amrith, and those variables are meaningful only for fake-mode tests | 16:36 |
| amrith | I saw that but assumed that this wasn't what you were doing because ... | 16:36 |
| amrith | instance_create_time, I'd argue that it should be global as well | 16:37 |
| amrith | it does depend on the datastore | 16:37 |
| amrith | some are fast, others are faster | 16:37 |
| amrith | or mysql_connection_method ... | 16:37 |
| amrith | etc., | 16:37 |
| amrith | no? | 16:37 |
| denis_makogon | amrith, as you know, integration tests are made in the next way, we're not running all test cases for each datastore, those test groups that was proposed and merged for different datastores are not including those things that you mantioned | 16:38 |
| denis_makogon | amrith, | 16:39 |
| denis_makogon | https://github.com/openstack/trove-integration/blob/master/tests/integration/int_tests.py#L223-L248 | 16:39 |
| amrith | ok, I'll go and read it again | 16:40 |
| amrith | I don't quite understand in that case what you are trying to do in this change but I'll read it again. | 16:41 |
| denis_makogon | amrith, integration tests are also relaying on CONFIG object, that's why i removed those variables out of there, and proposed datastores/version API calls to retrieve datastores/versions, and it'll work for other datastores | 16:41 |
| denis_makogon | amrith, the main goal to break dependency between int-tests and config-defined datastores | 16:42 |
| denis_makogon | amrith, to be able to run tests with different datastores | 16:42 |
| amrith | ok, what about https://review.openstack.org/#/c/100598/ | 16:43 |
| *** haomaiwa_ has joined #openstack-trove | 16:43 | |
| denis_makogon | amrith, agreed with all comments, but how to act if user will have instances provisioned through heat ? | 16:44 |
| denis_makogon | amrith, or just say, no matter, let's disable heat support ? | 16:44 |
| denis_makogon | if this is the case, i'm fine with that | 16:45 |
| amrith | what's the question? | 16:45 |
| *** ramashri has joined #openstack-trove | 16:45 | |
| amrith | right now it appears that with heat the claim is that we should do more than just check instance status; we should check stack status. I'm merely pointing out that the code doesn't do that. | 16:45 |
| denis_makogon | amrith, why is that ? | 16:46 |
| denis_makogon | amrith, have you looked at heat codepath? | 16:46 |
| amrith | where? | 16:46 |
| denis_makogon | amrith, | 16:47 |
| denis_makogon | https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L418-L422 | 16:47 |
| denis_makogon | this poll method will wait until stack is 100% ready | 16:47 |
| amrith | huh? | 16:48 |
| amrith | are you looking at https://review.openstack.org/#/c/100598/ or something else? | 16:48 |
| *** haomaiwa_ has quit IRC | 16:48 | |
| amrith | you seem to be talking about https://review.openstack.org/#/c/97194/ | 16:48 |
| amrith | ok, let's talk about that | 16:49 |
| amrith | my comment earlier relates to this | 16:49 |
| amrith | right now it appears that with heat the claim is that we should do more than just check instance status; we should check stack status. I'm merely pointing out that the code doesn't do that. | 16:49 |
| amrith | but I thought we were talking about something else | 16:49 |
| amrith | so ...https://review.openstack.org/#/c/97194/ | 16:49 |
| amrith | if we use heat | 16:49 |
| amrith | then we call _check_trove_instance_is_active | 16:49 |
| amrith | do you agree with the name change? | 16:50 |
| denis_makogon | amrith, status BUILD also valid | 16:51 |
| denis_makogon | amrith, ACTIVE is final status | 16:51 |
| denis_makogon | amrith, method name is 100% valid | 16:52 |
| amrith | I disagree | 16:52 |
| denis_makogon | amrith, why should it be changed? | 16:52 |
| amrith | you ask, why should it be changed. I think it should be changed because I believe the name does not reflect what the method is doing. the method is checking whether the trove instance is in an active state or a running state. it returns true if it is in a running state | 16:53 |
| amrith | for example, | 16:53 |
| amrith | the function check_if_a_number_is_even | 16:53 |
| denis_makogon | amrith, i don't see too much difference between valid/active word in function name | 16:53 |
| amrith | should (one would assume) return true if the number is even | 16:53 |
| amrith | and false if the number is not even | 16:54 |
| denis_makogon | amrith, it also checks if instance is not ERROR/FAILE as well | 16:54 |
| amrith | why would I call this check_if_a_number_is_valid? | 16:54 |
| amrith | actually it does't. It checkes NOT NEW or BUILDING. | 16:54 |
| amrith | but, be that as it might | 16:55 |
| *** ramishra has joined #openstack-trove | 16:55 | |
| denis_makogon | amrith, same words, but you talking nothing but Not New or Building, and avoiding other statuses | 16:55 |
| amrith | what do you mean by "same words"? | 16:55 |
| *** isviridov is now known as isviridov|away | 16:56 | |
| denis_makogon | amrith, you said "It checkes NOT NEW or BUILDING.". I said - it checks that instance is not in ERROR/FAILED | 16:56 |
| denis_makogon | stick with two ends | 16:57 |
| denis_makogon | but in general - same thing | 16:57 |
| amrith | sorry, error or failed is the check on the nova status. not new or building is the check for trove status. | 16:57 |
| amrith | I don't know what the "stick with two ends" means? | 16:58 |
| denis_makogon | for short period of time, those method (..._is_valid) checks if instance is in one of three states not New, BUILD, ACTIVE | 16:58 |
| amrith | at any rate, the function_check_trove_instance_is_in_valid_state() is not named correctly; I think. | 16:58 |
| *** ramishra has quit IRC | 16:59 | |
| denis_makogon | from my perspective, that method checks if instance is not in invalid state(ERROR/FAILED) | 16:59 |
| amrith | ok, we disagree. | 17:00 |
| denis_makogon | amrith, it checks if instance is not corrupted, it means instance is in one of the valid states (NEW, BUILD, ACTIVE) | 17:00 |
| denis_makogon | that's why it called ..._is_valid | 17:00 |
| amrith | like I said, we disagree ;) | 17:01 |
| denis_makogon | ok, we need to bring it up on a meeting | 17:01 |
| denis_makogon | in open discussion | 17:01 |
| amrith | or someone else can post comments on the review | 17:01 |
| denis_makogon | too long to wait | 17:01 |
| denis_makogon | need to grab cores and let them decide | 17:02 |
| amrith | ok, sure. | 17:02 |
| denis_makogon | SlickNik, cp16net, grapex, hub_cap, cp16net, amcrn, guys are you around? | 17:02 |
| amrith | I have no idea when they'll be around so I'll post my comments on the bug and they can read them there ... | 17:03 |
| denis_makogon | vipul, hey, are you the? | 17:03 |
| vipul | hey denis_makogon what's up | 17:03 |
| denis_makogon | vipul, have you got like 5 mins to review small patch? | 17:03 |
| vipul | sure shoot me the link | 17:03 |
| amrith | https://review.openstack.org/#/c/97194/3 | 17:04 |
| denis_makogon | vipul, we(i and amrith ) have small disagreement in method naming, https://review.openstack.org/#/c/97194/ | 17:04 |
| amrith | vipul, ... https://review.openstack.org/#/c/97194/3 | 17:04 |
| vipul | yes please | 17:06 |
| vipul | whoops wrong window | 17:06 |
| denis_makogon | amrith, https://review.openstack.org/#/c/98985/, let's talk about this one | 17:09 |
| amrith | ok, let's | 17:11 |
| denis_makogon | amrith, do you agree about not asking backend for another copy of a configuration group? | 17:12 |
| denis_makogon | amrith, and just passing already existing copy | 17:13 |
| amrith | are you talking about models.py@128 | 17:13 |
| denis_makogon | yes | 17:14 |
| amrith | what's the first parameter "context" being used for? | 17:14 |
| amrith | in this method? | 17:14 |
| denis_makogon | it's common for models methods | 17:14 |
| denis_makogon | so, i decided to keep it as it done in lots of models | 17:15 |
| amrith | is context used in load_configuration_datastore_and_version? | 17:16 |
| amrith | you pass it as parameter 1 | 17:16 |
| amrith | I'm asking why | 17:16 |
| amrith | am I missing something that needs context in that case? | 17:16 |
| *** thedodd has joined #openstack-trove | 17:16 | |
| denis_makogon | it's not used | 17:16 |
| denis_makogon | ok, i'll remove it | 17:16 |
| amrith | no, not so fst | 17:17 |
| amrith | s/fst/fast/ | 17:17 |
| amrith | why not make load_configuration_datastore_version and and the other have simialr signatures | 17:17 |
| amrith | and you can even reuse some code | 17:17 |
| amrith | if you pass context and id, you can make the code a little cleaner | 17:17 |
| amrith | either would work for me. | 17:18 |
| amrith | I just didn't understand why someone would pass a parameter which isn't used; that's all. | 17:18 |
| denis_makogon | i would remove context since it's not used, but i woudn't change configuration into it's id | 17:18 |
| amrith | ok, as you wish | 17:19 |
| denis_makogon | amrith, because i really don't like the idea of double asking backend for the same configuration object | 17:19 |
| amrith | don't understand why but that's fine. | 17:19 |
| *** saurabhs has joined #openstack-trove | 17:19 | |
| amrith | as you wish | 17:20 |
| denis_makogon | amrith, same thing for services.py#50 | 17:20 |
| denis_makogon | dougshelley66, ping | 17:21 |
| amrith | like I said, if you really didn't want to ask for config twice, there's a way to simplify the code. but that's your call. | 17:21 |
| amrith | are we all set? or should we wait for Vipul? | 17:22 |
| denis_makogon | amrith, i'll ping him later | 17:22 |
| vipul | So amrith, denis_makogon -- In terms of logic change, it looks like the change is really to fast-fail the poll_until if the nova server status is already in a failed state | 17:23 |
| dougshelley66 | denis_makogon: hi | 17:23 |
| denis_makogon | amrith, code simplification in this case will hurt perfomance API service | 17:23 |
| amrith | vipul, I believe that is an unintended consequence | 17:23 |
| vipul | which i think is fine .. right now we'll just be waiting longer for the timeout | 17:23 |
| vipul | i think that part is a good change | 17:23 |
| denis_makogon | dougshelley66, can we talk about https://review.openstack.org/#/c/98985/6/trove/configuration/views.py,cm ? | 17:24 |
| vipul | i do agree with amrith on the verbiage aorund the method name | 17:24 |
| vipul | it shouldn't be 'valid' | 17:24 |
| dougshelley66 | denis_makogon: sure | 17:24 |
| dougshelley66 | the bug report was pretty cryptic | 17:24 |
| denis_makogon | amrith, vipul, ok | 17:24 |
| dougshelley66 | maybe we should fix that first | 17:24 |
| dougshelley66 | it read it to mean we wanted those 2 pieces of data on list and show | 17:24 |
| denis_makogon | dougshelley66, bug report tells to change only two types of views | 17:25 |
| vipul | the goal is to find an active instance, so we should rename that to trove_instance_is_active or something | 17:25 |
| dougshelley66 | denis_makogon, ok so i misunderstood what the second view was for | 17:25 |
| denis_makogon | vipul, i get that, i'll update it | 17:25 |
| amrith | vipul, do you believe the change in order or testing nova and trove is material? I wasn't sure so I just highlighted it. it could change the user visible experience. | 17:25 |
| dougshelley66 | denis_makogon, however, the column headers did get chnaged on list | 17:25 |
| dougshelley66 | denis_makogon, but contain no data | 17:25 |
| SlickNik | vipul / amrith / denis_makogon: Isn't the fast poll something that we could use for the non-heat case as well? I'm still looking at the code but is there a need to fork the logic for the two cases? (heat vs. non-heat) | 17:26 |
| *** yidclare has joined #openstack-trove | 17:26 | |
| vipul | amrith: that's a good point.. the main thing is the user will see thier instance go to Error with reason being BUILD_TIMEOUT | 17:26 |
| amrith | SlickNik, I don't know. One of my comments on the review is that the comments on the change imply that we'll be doing something more for heat but I don't see code for that. | 17:27 |
| vipul | even when the reason really isn't because we timed out waiting for it go out of BUILDING.. it may have failed due to Nova failure first | 17:27 |
| denis_makogon | SlickNik, yes, before that poll_until method stack already completely ACTIVE | 17:27 |
| amrith | vipul, yes. but in the past the error they'd have got was that the SERVICE was in a bad state. Now it will say the SERVER is in a bad state | 17:28 |
| amrith | or some such | 17:28 |
| denis_makogon | SlickNik, no need to ask nova to verify server status | 17:28 |
| amrith | notice that the two throw's have subtly different words. | 17:28 |
| vipul | SlickNik: yea according to the commit msg denis_makogon says it will already wait for stack to be completed, still need to verify that | 17:28 |
| amrith | I don't know if that could change for someone who was expecting a particular failure | 17:28 |
| amrith | like, maybe a test? | 17:28 |
| vipul | amrith: this code is async, so that message wont' actually be sent back | 17:28 |
| denis_makogon | SlickNik, https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L418-L423 | 17:28 |
| vipul | it's really a loop the taskmanager does after it's already responded to the request | 17:28 |
| vipul | the change would impact how quickly a instance changes to ERROR based on guest timeout | 17:29 |
| amrith | vipul, OK, I'll go read the code again and try to understand that. | 17:29 |
| denis_makogon | dougshelley66, https://review.openstack.org/#/c/98988/2/troveclient/v1/shell.py,cm i guess you tested it with this patch ? | 17:31 |
| denis_makogon | dougshelley66, if that's so, it means that i should revert change for list command | 17:31 |
| dougshelley66 | denis_makogon, yes i believe i applied both patches and tried it | 17:31 |
| denis_makogon | dougshelley66, [id, name, description] is enough for list | 17:31 |
| dougshelley66 | denis_makogon, if you do configuration list what do you see for the result set? | 17:31 |
| dougshelley66 | denis_makogon, before the patch is has those items + ds version id | 17:32 |
| denis_makogon | dougshelley66, i should see the same attributes | 17:32 |
| dougshelley66 | denis_makogon, after the patch is has name and version | 17:32 |
| denis_makogon | dougshelley66, yes, i forgot about that | 17:32 |
| dougshelley66 | denis_makogon, but they aren't populated | 17:32 |
| *** michael-yu has joined #openstack-trove | 17:32 | |
| denis_makogon | dougshelley66, version id would be there | 17:33 |
| denis_makogon | dougshelley66, i get that, just need to fix client code | 17:33 |
| denis_makogon | dougshelley66, thanks for pointing out | 17:33 |
| dougshelley66 | denis_makogon, ok - maybe the bug report should include a description of exactly what is expected for the result sets from the 2 commands? | 17:33 |
| denis_makogon | dougshelley66, i guess no, since each view is pinned to exact shell command | 17:34 |
| vipul | denis_makogon, amrith: i think a better thing to do here is throw a different exception when it's a Nova fast-fail Error, vs when it's a Trove Service status Error. When you update the instance status, if it's a nova error, then we need to set it to FAILED = ServiceStatus(0x08, 'failed to spawn', 'FAILED') | 17:34 |
| denis_makogon | vipul, it's already done | 17:35 |
| vipul | where | 17:35 |
| denis_makogon | vipul, https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L264 | 17:35 |
| dougshelley66 | denis_makogon, so I assume that bug report and the resulting patch would confuse others. All I'm saying is we should clarify the intent/requirements in the bug report | 17:35 |
| denis_makogon | poll_until will fail, and instance models will be updated | 17:35 |
| vipul | denis_makogon: right -- that method assumes that we waiting 10 minutes and the error is because of a GUEST Timeout | 17:36 |
| denis_makogon | dougshelley66, aren't they already there ? | 17:36 |
| vipul | s/waiting/waited | 17:36 |
| denis_makogon | vipul, poll_untill will fail faster if nova instance is in ERROR/FAILED state | 17:36 |
| dougshelley66 | denis_makogon, so a bug report that starts out by dumping code fragments is misguided from the beginning | 17:37 |
| denis_makogon | it would not wait for guest | 17:37 |
| vipul | denis_makogon: but the error message won't indicate that it failed because of Nova error -- it will only say that it failed because guest timed out | 17:37 |
| dougshelley66 | denis_makogon, i would assume it should says something like "when we run trove configuration show" the output should look like this: etc | 17:37 |
| denis_makogon | dougshelley66, let's ask SlickNik to update bug report | 17:37 |
| dougshelley66 | denis_makogon, fine | 17:37 |
| denis_makogon | dougshelley66, ok, i'm fine with that too | 17:37 |
| denis_makogon | vipul, it would be enough to add logging message here https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L317 | 17:38 |
| denis_makogon | vipul, and if it's not a guest timeout exception would be handled by https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L266-L267 | 17:39 |
| denis_makogon | vipul, so, i guess it'll be enough to add certain log message inside _service_is_active for that case if nova instance was corrupted | 17:40 |
| denis_makogon | vipul, and of course, self.update_statuses_on_time_out() would be added there too | 17:41 |
| *** jmontemayor has joined #openstack-trove | 17:41 | |
| vipul | denis_makogon: it's not enough just to add a logging message.. you need to catch a diffrent exception, and based on the exception caught set the appropriate reason for failure | 17:41 |
| *** ramashri has quit IRC | 17:42 | |
| *** cweidenkeller has joined #openstack-trove | 17:42 | |
| *** jmontemayor_ has joined #openstack-trove | 17:42 | |
| denis_makogon | vipul, exception that is being threw is a TroveError, that is a parent of PollTimeout exception | 17:43 |
| denis_makogon | vipul, i agree that it should be fixed, but it's fit into given bug-report | 17:43 |
| denis_makogon | vipul, amrith or someone else can file a bug-report and fix described issue | 17:43 |
| *** jcru has quit IRC | 17:44 | |
| denis_makogon | vipul, do you agree? | 17:44 |
| amrith | huh? | 17:44 |
| *** haomaiwa_ has joined #openstack-trove | 17:44 | |
| denis_makogon | amrith, if you don't want to do that, then ok | 17:44 |
| SlickNik | denis_makogon: So, what's wrong with querying nova to ensure that the server status is ACTIVE even in the heat case (i.e. leaving the code as is)? I get that it will pass since the stack is ACTIVE. | 17:45 |
| SlickNik | But there's an argument to be made for simplicity of code, and maintainability by not introducing a fork to do something different for the heat case when it's not absolutely necessary. | 17:45 |
| amrith | denis_makogon, I believe that what vipul gave you was feedback on your fix and potentially the reason he would -1 the fix | 17:45 |
| amrith | this isn't a different bug that he's suggesting. | 17:45 |
| *** jcru has joined #openstack-trove | 17:45 | |
| denis_makogon | amrith, given issue doesn't fit into proposed bug report | 17:45 |
| amrith | I believe SlickNik's comments are in the same vein | 17:45 |
| *** jcru has quit IRC | 17:46 | |
| *** jmontemayor has quit IRC | 17:46 | |
| amrith | that he is proposing that your fix is not the right way (maybe). he's right here so I'll let him put words in his own mouth ;) | 17:46 |
| *** jcru has joined #openstack-trove | 17:46 | |
| denis_makogon | SlickNik, those nova calls for heat-provisioned resources are redurand | 17:46 |
| denis_makogon | SlickNik, because they are not needed since stack is already in ACTIVE state | 17:47 |
| *** jmontemayor has joined #openstack-trove | 17:47 | |
| denis_makogon | SlickNik, and it means that all resources are in ACTIVE/In-use state | 17:47 |
| *** michael-yu has quit IRC | 17:47 | |
| denis_makogon | SlickNik, taskmanager will do {usage_timeout}/sleet_timeout redurand calls to nova | 17:48 |
| denis_makogon | SlickNik, we should avoid such things | 17:49 |
| *** haomaiwa_ has quit IRC | 17:49 | |
| *** jmontemayor_ has quit IRC | 17:49 | |
| *** grapex has quit IRC | 17:50 | |
| *** grapex has joined #openstack-trove | 17:51 | |
| amrith | denis_makogon, by that logic, we should not check nova state each time for non-heat instances either. just on the first time, maybe? | 17:51 |
| amrith | in other words, the calls to check nova status are redumbnant. | 17:52 |
| vipul | lol @ amrith | 17:52 |
| denis_makogon | amrith, in this case we need to separate pollers | 17:52 |
| denis_makogon | *two | 17:52 |
| denis_makogon | one will finish as soon as nova instance is ACTIVE | 17:53 |
| SlickNik | denis_makogon: Yes, I get that we're now making an extra call to nova to verify that the server is in an ACTIVE state (for the heat case). | 17:53 |
| SlickNik | From a code maintainability perspective though, the more branches your code has, the harder it gets to reason about it / maintain. | 17:53 |
| SlickNik | So that tradeoff makes this a bit of a six-of-one, half-a-dozen of another issue to me. Therefore, my inclination is to preserve the status quo. | 17:53 |
| *** kevinconway_ has joined #openstack-trove | 17:53 | |
| denis_makogon | and other one will wait until guest is ACTIVE | 17:53 |
| *** kevinconway has quit IRC | 17:53 | |
| *** kevinconway_ is now known as kevinconway | 17:53 | |
| SlickNik | However, vipul does bring up a good point about the fail fast option, which I think would be beneficial to both the heat / non-heat case. | 17:53 |
| SlickNik | So I would not be opposed to fixing that (albeit as a separate bugfix). | 17:54 |
| denis_makogon | SlickNik, i get that, and will fix that too | 17:54 |
| *** ramashri has joined #openstack-trove | 17:55 | |
| denis_makogon | SlickNik, but i really think that we need to split out guest poller and use it while heat support is enabled | 17:55 |
| vipul | SlickNik: +1 if we can combine the two code paths into one i'm all for that | 17:55 |
| *** ramishra has joined #openstack-trove | 17:56 | |
| amrith | vipul, that's how the code was earlier | 17:56 |
| amrith | I'm getting the feeling that there is a question of whether we should split this code out in the heat case | 17:56 |
| amrith | that's above my level of understanding of the code/product so I'll put my nose elsewhere. | 17:56 |
| vipul | then let's not do this change if we are unnecessarily optimizing | 17:57 |
| vipul | i think this change can morph into fast-fail if nova is already failed | 17:57 |
| denis_makogon | vipul, stack poller will fail even before guest poller | 17:58 |
| vipul | yea I get that. that's already in place | 17:58 |
| denis_makogon | vipul, so, i still thinks that we should not talk to nova inside poll method is heat is enabled | 17:58 |
| *** NehaV3 has joined #openstack-trove | 17:59 | |
| *** ramishra has quit IRC | 18:00 | |
| vipul | denis_makogon: Yea i kinda of agree with that it's unnecessary to query Nova if the instance was provisioned by Heat -- but i think as SlickNik mentioned it introduces yet another code path.. so if it's not a big deal to query nova even in Heat case then we should keep doing it for simplicity | 18:04 |
| denis_makogon | vipul, what if we would split-up poller into to pollers, and move nova instance poller under _create_serve_... methods? | 18:10 |
| denis_makogon | vipul, than we would have an ability to fast-fail even before guest poller ? | 18:11 |
| *** shayneburgess has joined #openstack-trove | 18:11 | |
| *** michael-yu has joined #openstack-trove | 18:11 | |
| denis_makogon | vipul, lets say, poller will be placed at _create_server(...) method | 18:12 |
| *** jasonb365 has joined #openstack-trove | 18:12 | |
| SlickNik | glucas / vgnbkr / peterstac: Thanks for pushing up replication patches. | 18:12 |
| denis_makogon | vipul, and in this case we would not need to maintain two code paths | 18:13 |
| SlickNik | FYI I'm going to be taking it for a spin this week, and stepping up reviews on it. | 18:13 |
| SlickNik | Will keep you in the loop. | 18:13 |
| SlickNik | ^ also dougshelley66 | 18:13 |
| dougshelley66 | Thanks SlickNik - that will be helpful | 18:14 |
| glucas | SlickNik: great, thanks! | 18:14 |
| *** sgotliv has joined #openstack-trove | 18:21 | |
| peterstac | SlickNik: Thx! (although I was just pushing the review, not actual patches ;) ) | 18:23 |
| *** ramishra has joined #openstack-trove | 18:23 | |
| *** jmontemayor has quit IRC | 18:24 | |
| SlickNik | peterstac: I know. Included you on the list, since you mentioned it earlier this morning ;) | 18:24 |
| peterstac | SlickNik: np | 18:25 |
| *** ramishra has quit IRC | 18:27 | |
| vipul | denis_makogon: It's still two code paths, but push up an update to that review.. we can see how that looks | 18:28 |
| denis_makogon | vipul, of course | 18:31 |
| *** yidclare has quit IRC | 18:31 | |
| *** NehaV3 has quit IRC | 18:31 | |
| amrith | denis_makogon, are we all set? you are going to push new code, yes? | 18:40 |
| denis_makogon | amrith, yes | 18:43 |
| amrith | thx | 18:43 |
| denis_makogon | amrith, thanks to you too | 18:50 |
| amrith | denis_makogon, np. | 18:51 |
| *** ViswaV has quit IRC | 19:05 | |
| openstackgerrit | Michael Yu proposed a change to openstack/trove: Adds backup/restore support for couchbase https://review.openstack.org/86731 | 19:06 |
| *** openstackgerrit has quit IRC | 19:06 | |
| *** openstackgerrit_ has joined #openstack-trove | 19:07 | |
| *** openstackgerrit_ is now known as openstackgerrit | 19:08 | |
| openstackgerrit | Michael Yu proposed a change to openstack/trove: Adds backup/restore support for Couchbase https://review.openstack.org/86731 | 19:10 |
| *** yidclare has joined #openstack-trove | 19:11 | |
| *** ViswaV has joined #openstack-trove | 19:22 | |
| *** yidclare has quit IRC | 19:23 | |
| *** ramishra has joined #openstack-trove | 19:24 | |
| *** ramishra has quit IRC | 19:29 | |
| *** yidclare has joined #openstack-trove | 19:36 | |
| *** ramashri has quit IRC | 19:45 | |
| *** yidclare has quit IRC | 19:47 | |
| *** pdmars has quit IRC | 19:50 | |
| *** pdmars has joined #openstack-trove | 19:55 | |
| *** shayneburgess has quit IRC | 19:57 | |
| *** julienvey has quit IRC | 19:57 | |
| *** shayneburgess has joined #openstack-trove | 19:59 | |
| *** michael-yu has quit IRC | 20:05 | |
| *** boden has quit IRC | 20:15 | |
| *** michael-yu has joined #openstack-trove | 20:19 | |
| *** ramishra has joined #openstack-trove | 20:25 | |
| *** michael-yu has quit IRC | 20:27 | |
| *** julienvey has joined #openstack-trove | 20:28 | |
| *** ramishra has quit IRC | 20:29 | |
| *** michael-yu has joined #openstack-trove | 20:29 | |
| *** julienve_ has joined #openstack-trove | 20:29 | |
| *** ramashri has joined #openstack-trove | 20:30 | |
| *** jasonb365 has quit IRC | 20:30 | |
| *** julienvey has quit IRC | 20:32 | |
| *** julienve_ has quit IRC | 20:35 | |
| *** ramashri has quit IRC | 20:35 | |
| *** shayneburgess_ has joined #openstack-trove | 20:36 | |
| *** ramashri has joined #openstack-trove | 20:36 | |
| *** shayneburgess has quit IRC | 20:36 | |
| *** shayneburgess_ is now known as shayneburgess | 20:36 | |
| *** ramashri has quit IRC | 20:37 | |
| *** ramashri has joined #openstack-trove | 20:37 | |
| *** edmund has quit IRC | 20:38 | |
| *** edmund has joined #openstack-trove | 20:40 | |
| *** yidclare has joined #openstack-trove | 20:41 | |
| *** yidclare has quit IRC | 20:42 | |
| *** juantwo has quit IRC | 20:46 | |
| *** juantwo has joined #openstack-trove | 20:52 | |
| *** miqui has quit IRC | 20:54 | |
| *** juantwo has quit IRC | 20:56 | |
| *** juantwo has joined #openstack-trove | 20:57 | |
| *** juantwo has quit IRC | 21:01 | |
| *** jmontemayor has joined #openstack-trove | 21:04 | |
| *** jmontemayor has quit IRC | 21:06 | |
| *** denis_makogon has quit IRC | 21:06 | |
| *** jmontemayor has joined #openstack-trove | 21:07 | |
| *** ramishra has joined #openstack-trove | 21:25 | |
| *** fifieldt has quit IRC | 21:27 | |
| *** mattgriffin has quit IRC | 21:28 | |
| *** ramishra has quit IRC | 21:30 | |
| *** julienvey has joined #openstack-trove | 21:30 | |
| *** julienvey has quit IRC | 21:31 | |
| *** julienvey has joined #openstack-trove | 21:31 | |
| *** dkehn_ has joined #openstack-trove | 21:33 | |
| *** dkehnx has quit IRC | 21:34 | |
| *** jmontemayor has quit IRC | 21:34 | |
| *** julienvey has quit IRC | 21:36 | |
| *** fifieldt has joined #openstack-trove | 21:40 | |
| *** yidclare has joined #openstack-trove | 21:51 | |
| *** ranjitha has joined #openstack-trove | 21:59 | |
| *** shayneburgess has quit IRC | 22:10 | |
| *** robertmy_ has joined #openstack-trove | 22:12 | |
| *** robertmy_ has quit IRC | 22:12 | |
| *** michael-yu has quit IRC | 22:12 | |
| *** robertmyers has quit IRC | 22:13 | |
| *** robertmy_ has joined #openstack-trove | 22:13 | |
| *** ramashri has quit IRC | 22:15 | |
| *** ramashri has joined #openstack-trove | 22:15 | |
| *** dkehn_ is now known as dkehnx | 22:17 | |
| *** robertmy_ has quit IRC | 22:18 | |
| *** mattgriffin has joined #openstack-trove | 22:20 | |
| *** juantwo has joined #openstack-trove | 22:23 | |
| *** juantwo has quit IRC | 22:23 | |
| *** juantwo has joined #openstack-trove | 22:24 | |
| *** ramishra has joined #openstack-trove | 22:26 | |
| *** kevinconway has quit IRC | 22:27 | |
| *** michael-yu has joined #openstack-trove | 22:28 | |
| *** thedodd has quit IRC | 22:29 | |
| *** ramishra has quit IRC | 22:31 | |
| *** shayneburgess has joined #openstack-trove | 22:33 | |
| *** edmund has quit IRC | 22:35 | |
| *** flaper87 is now known as flaper87|afk | 22:41 | |
| *** cweidenkeller has quit IRC | 22:44 | |
| *** sgotliv has quit IRC | 22:44 | |
| *** rhodgin has quit IRC | 22:54 | |
| *** mattgriffin has quit IRC | 23:02 | |
| *** jmontemayor has joined #openstack-trove | 23:05 | |
| *** juantwo_ has joined #openstack-trove | 23:06 | |
| *** juantwo_ has joined #openstack-trove | 23:06 | |
| *** saurabhs has left #openstack-trove | 23:08 | |
| *** juantwo has quit IRC | 23:09 | |
| *** pdmars has quit IRC | 23:12 | |
| *** jcru has quit IRC | 23:14 | |
| *** Isotopp has quit IRC | 23:17 | |
| *** tvoran has quit IRC | 23:18 | |
| *** mordred has quit IRC | 23:24 | |
| *** Isotopp has joined #openstack-trove | 23:25 | |
| *** Isotopp has joined #openstack-trove | 23:25 | |
| *** ramishra has joined #openstack-trove | 23:27 | |
| *** mordred has joined #openstack-trove | 23:27 | |
| *** mordred has quit IRC | 23:30 | |
| *** mordred has joined #openstack-trove | 23:30 | |
| *** ramishra has quit IRC | 23:31 | |
| *** michael-yu has quit IRC | 23:33 | |
| *** shayneburgess has quit IRC | 23:44 | |
| *** ranjitha has quit IRC | 23:48 | |
| *** ranjitha has joined #openstack-trove | 23:51 | |
| *** amytron has quit IRC | 23:52 | |
| *** jmontemayor has quit IRC | 23:54 | |
| *** achampion has joined #openstack-trove | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!