| *** trevormc has quit IRC | 00:05 | |
| *** imandhan has quit IRC | 00:08 | |
| openstackgerrit | Merged openstack/trove: Implement full online backups for DB2 https://review.openstack.org/364431 | 00:09 |
|---|---|---|
| *** imandhan has joined #openstack-trove | 00:10 | |
| *** tqtran has quit IRC | 00:25 | |
| *** itlinux has quit IRC | 00:40 | |
| *** itlinux has joined #openstack-trove | 00:41 | |
| *** itlinux has quit IRC | 00:56 | |
| openstackgerrit | liuqing proposed openstack/trove: [install-guide]add more information for trove-guestagent.conf https://review.openstack.org/369125 | 00:57 |
| *** cheneydc has joined #openstack-trove | 00:59 | |
| *** itlinux has joined #openstack-trove | 01:00 | |
| peterstac | pmalik, already +2'd it | 01:03 |
| *** johnbabers has joined #openstack-trove | 01:09 | |
| *** johnbabers has quit IRC | 01:17 | |
| *** gouthamr has quit IRC | 01:17 | |
| *** johnbabers has joined #openstack-trove | 01:26 | |
| *** cheneydc has quit IRC | 01:27 | |
| *** itlinux has quit IRC | 01:39 | |
| *** itlinux has joined #openstack-trove | 01:46 | |
| *** itlinux has quit IRC | 01:56 | |
| *** johnbabers has quit IRC | 02:00 | |
| openstackgerrit | Merged openstack/trove: Allow for invalid packet sequence in keepalive https://review.openstack.org/362347 | 02:38 |
| amrith | peterstac, johnma ^^ | 02:43 |
| peterstac | yep, saw it | 02:46 |
| peterstac | do we want to get https://review.openstack.org/#/c/357293/ in? | 02:46 |
| *** stevemar_ is now known as stevemar | 02:49 | |
| *** itlinux has joined #openstack-trove | 02:57 | |
| *** itlinux has quit IRC | 03:07 | |
| openstackgerrit | Peter Stachowski proposed openstack/trove-integration: Enhance 'redstack clean' regarding modules https://review.openstack.org/364539 | 03:10 |
| amrith | peterstac, I was going to look at it in the morning. I added it to the list for rc2 | 03:12 |
| peterstac | ok, sounds good | 03:12 |
| openstackgerrit | jian.song proposed openstack/trove: Concentrated test_validate_xx_configuration to a new method https://review.openstack.org/369252 | 03:19 |
| *** tqtran has joined #openstack-trove | 03:22 | |
| *** songjian has joined #openstack-trove | 03:25 | |
| *** tqtran has quit IRC | 03:26 | |
| *** imandhan has quit IRC | 03:32 | |
| *** songjian has quit IRC | 03:35 | |
| *** songjian has joined #openstack-trove | 03:36 | |
| openstackgerrit | amrith proposed openstack/trove-integration: DNM: Test kvm-ok within a CI infrastructure https://review.openstack.org/369806 | 03:38 |
| *** himani has joined #openstack-trove | 03:41 | |
| *** khushbu_ has joined #openstack-trove | 03:41 | |
| *** itlinux has joined #openstack-trove | 03:46 | |
| *** itlinux has quit IRC | 03:56 | |
| *** songjian has quit IRC | 04:04 | |
| *** songjian has joined #openstack-trove | 04:04 | |
| *** songjian has quit IRC | 04:10 | |
| *** songjian has joined #openstack-trove | 04:10 | |
| openstackgerrit | Clark Boylan proposed openstack/trove-integration: DNM: hacks to create and upload raw image https://review.openstack.org/369809 | 04:15 |
| *** imandhan has joined #openstack-trove | 04:25 | |
| *** diga has joined #openstack-trove | 04:28 | |
| *** _imandhan_ has joined #openstack-trove | 04:29 | |
| *** songjian has quit IRC | 04:30 | |
| *** songjian has joined #openstack-trove | 04:31 | |
| *** imandhan has quit IRC | 04:32 | |
| *** itlinux has joined #openstack-trove | 04:37 | |
| *** imandhan has joined #openstack-trove | 04:46 | |
| *** itlinux has quit IRC | 04:47 | |
| openstackgerrit | amrith proposed openstack/trove-integration: DNM: check if this causes lxc https://review.openstack.org/369813 | 04:48 |
| *** _imandhan_ has quit IRC | 04:49 | |
| *** khushbu_ has quit IRC | 04:57 | |
| *** imandhan has quit IRC | 04:57 | |
| *** imandhan has joined #openstack-trove | 04:58 | |
| *** khushbu_ has joined #openstack-trove | 04:59 | |
| *** _imandhan_ has joined #openstack-trove | 05:05 | |
| *** imandhan has quit IRC | 05:07 | |
| *** songjian has quit IRC | 05:10 | |
| *** greghaynes has joined #openstack-trove | 05:26 | |
| *** pmackinn has quit IRC | 05:27 | |
| *** rcernin has joined #openstack-trove | 05:33 | |
| *** khushbu_ has quit IRC | 05:37 | |
| *** itlinux has joined #openstack-trove | 05:47 | |
| *** itlinux has quit IRC | 06:01 | |
| openstackgerrit | jian.song proposed openstack/trove: Concentrated test_validate_xx_configuration to a new method https://review.openstack.org/369252 | 06:06 |
| *** songjian has joined #openstack-trove | 06:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/trove-dashboard: Imported Translations from Zanata https://review.openstack.org/369842 | 06:17 |
| *** songjian has quit IRC | 06:26 | |
| *** songjian has joined #openstack-trove | 06:27 | |
| *** pcaruana has joined #openstack-trove | 06:32 | |
| *** itlinux has joined #openstack-trove | 06:36 | |
| *** imandhan has joined #openstack-trove | 06:44 | |
| *** itlinux has quit IRC | 06:46 | |
| *** _imandhan_ has quit IRC | 06:48 | |
| *** tesseract- has joined #openstack-trove | 06:57 | |
| *** _imandhan_ has joined #openstack-trove | 07:01 | |
| *** imandhan has quit IRC | 07:04 | |
| *** drifterza has joined #openstack-trove | 07:14 | |
| *** _imandhan_ has quit IRC | 07:16 | |
| *** imandhan has joined #openstack-trove | 07:17 | |
| *** Ravikiran_K has joined #openstack-trove | 07:19 | |
| *** imandhan has quit IRC | 07:23 | |
| *** imandhan has joined #openstack-trove | 07:24 | |
| *** tqtran has joined #openstack-trove | 07:26 | |
| *** imandhan has quit IRC | 07:30 | |
| *** tqtran has quit IRC | 07:31 | |
| *** imandhan has joined #openstack-trove | 07:31 | |
| *** songjian has quit IRC | 07:39 | |
| openstackgerrit | Janice-Chen proposed openstack/trove: Fix for get volume_type failure for cluster_create in CLI https://review.openstack.org/369873 | 07:46 |
| *** _imandhan_ has joined #openstack-trove | 07:47 | |
| *** imandhan has quit IRC | 07:50 | |
| openstackgerrit | jian.song proposed openstack/trove: Delete some unnecessary initial value = None https://review.openstack.org/369875 | 07:55 |
| *** itlinux has joined #openstack-trove | 07:55 | |
| *** _imandhan_ has quit IRC | 08:01 | |
| *** imandhan has joined #openstack-trove | 08:02 | |
| *** itlinux has quit IRC | 08:05 | |
| *** khushbu_ has joined #openstack-trove | 08:12 | |
| openstackgerrit | jian.song proposed openstack/trove: Delete some unnecessary initial value = None https://review.openstack.org/369875 | 08:22 |
| openstackgerrit | jian.song proposed openstack/trove: Delete some unnecessary initial value = None https://review.openstack.org/369875 | 08:24 |
| *** songjian has joined #openstack-trove | 08:30 | |
| *** songjian has quit IRC | 08:32 | |
| *** imandhan has quit IRC | 08:38 | |
| *** songjian has joined #openstack-trove | 08:38 | |
| *** khushbu_ has quit IRC | 08:43 | |
| *** himani has quit IRC | 09:05 | |
| *** itlinux has joined #openstack-trove | 09:06 | |
| *** himani has joined #openstack-trove | 09:13 | |
| *** itlinux has quit IRC | 09:16 | |
| openstackgerrit | jian.song proposed openstack/trove: Delete some unnecessary initial value = None https://review.openstack.org/369875 | 09:23 |
| openstackgerrit | jian.song proposed openstack/trove: Repeat definition blacklist https://review.openstack.org/369970 | 09:27 |
| *** tqtran has joined #openstack-trove | 09:28 | |
| *** tqtran has quit IRC | 09:32 | |
| himani | Hi core-members, I am new to this project and I want to contribute in this. If you know some bugs to get started with then please let me know. | 09:34 |
| openstackgerrit | jian.song proposed openstack/trove: Delete unused method verify_errors https://review.openstack.org/369978 | 09:40 |
| *** songjian has quit IRC | 09:41 | |
| *** tosky has joined #openstack-trove | 09:41 | |
| *** itlinux has joined #openstack-trove | 10:17 | |
| *** itlinux has quit IRC | 10:28 | |
| *** songjian has joined #openstack-trove | 10:37 | |
| songjian | johnma,hi,I have two commit ,all of them have two +2,could you please help me to review it,thanks a lot...https://review.openstack.org/#/c/362810/ https://review.openstack.org/#/c/363365/ | 10:40 |
| *** NBziouech has joined #openstack-trove | 10:43 | |
| *** khushbu_ has joined #openstack-trove | 10:49 | |
| johnma | songjian: done, thank you | 10:56 |
| *** itlinux has joined #openstack-trove | 10:58 | |
| johnma | himani: Welcome to the group and thank you for your interest in Trove. | 10:58 |
| dougshelley66 | himani welcome! | 10:59 |
| dougshelley66 | himani have you been able to run through redstack setup and complete int-tests? | 11:00 |
| *** cdelatte has joined #openstack-trove | 11:00 | |
| openstackgerrit | Merged openstack/trove: Add support for modules in cluster create/grow https://review.openstack.org/313232 | 11:00 |
| dougshelley66 | that is a good pre-requisite to beginning to work on bugs | 11:00 |
| dougshelley66 | let us know if you need help with that | 11:00 |
| johnma | himani: you can always look for bugs at https://bugs.launchpad.net/trove and look for the ones tagged: Low hanging fruit. They are supposed to be the easy ones but there arent that many. In the coming days, will try to triage some bugs and tag some more | 11:02 |
| *** himani has quit IRC | 11:03 | |
| *** himani has joined #openstack-trove | 11:06 | |
| johnma | himani: also, here is a list of reviews that were abandoned for various reasons: https://review.openstack.org/#/q/project:openstack/trove+status:abandoned. You can go through them and see if there is anything that interests you and restore it and continue to work on it. Of course | 11:06 |
| openstackgerrit | Merged openstack/trove: Cassandra fix lost keyspace(s) after cluster grow https://review.openstack.org/357293 | 11:08 |
| *** itlinux has quit IRC | 11:09 | |
| songjian | johnma,thx | 11:10 |
| himani | johnma: thank you. | 11:12 |
| *** khushbu_ has quit IRC | 11:18 | |
| *** diga has quit IRC | 11:18 | |
| *** himani has quit IRC | 11:27 | |
| *** tqtran has joined #openstack-trove | 11:29 | |
| *** tqtran has quit IRC | 11:34 | |
| *** openstackgerrit has quit IRC | 11:34 | |
| *** openstackgerrit has joined #openstack-trove | 11:34 | |
| *** songjian has quit IRC | 11:48 | |
| *** pmackinn has joined #openstack-trove | 11:50 | |
| *** drifterza has quit IRC | 11:55 | |
| *** drifterza has joined #openstack-trove | 12:07 | |
| *** itlinux has joined #openstack-trove | 12:09 | |
| *** gouthamr has joined #openstack-trove | 12:09 | |
| openstackgerrit | Merged openstack/trove: Update #noqa for out 79 character https://review.openstack.org/363365 | 12:11 |
| amrith | johnma, dougshelley66 g'morning | 12:14 |
| amrith | please see https://review.openstack.org/#/c/369823/ and https://review.openstack.org/#/c/369825/ | 12:15 |
| amrith | dougshelley66, we will have to have speaks | 12:15 |
| *** itlinux has quit IRC | 12:20 | |
| peterstac | amrith, I think we're looking good - maybe we should get https://review.openstack.org/#/c/353699/ in too since the server side merged ... | 12:24 |
| openstackgerrit | Merged openstack/trove-dashboard: Imported Translations from Zanata https://review.openstack.org/369842 | 12:28 |
| amrith | peterstac, johnma ... the releasenotes in the release tagging commit come from the release notes that people put into commits. | 12:33 |
| amrith | if they don't put them on commits, they don't make it on the release commit | 12:34 |
| amrith | also peterstac johnma dougshelley66 if the real fix for the issue https://review.openstack.org/#/c/369873/1 is what this commit proposes, I think we should take it for rc1 | 12:36 |
| amrith | I didn't understand what the guy was talking about in his bug | 12:36 |
| amrith | but the fix sure looks like something we should take for rc1 | 12:36 |
| *** 17WAAMUGO has joined #openstack-trove | 13:00 | |
| *** drifterza has quit IRC | 13:04 | |
| *** 17WAAMUGO has quit IRC | 13:05 | |
| openstackgerrit | amrith proposed openstack/trove: Cluster code looks for wrong field in request for volume type https://review.openstack.org/369873 | 13:14 |
| amrith | peterstac, let's discuss this morning and figure out what we want to do. | 13:16 |
| amrith | johnma ^^ | 13:16 |
| *** itlinux has joined #openstack-trove | 13:20 | |
| *** johnbabers has joined #openstack-trove | 13:31 | |
| *** tqtran has joined #openstack-trove | 13:31 | |
| *** itlinux has quit IRC | 13:34 | |
| *** johnbabers has quit IRC | 13:34 | |
| *** johnbabers has joined #openstack-trove | 13:35 | |
| *** zhugaoxiao has quit IRC | 13:35 | |
| *** tqtran has quit IRC | 13:35 | |
| johnma | amrith, peterstac: sorry just got back. | 13:58 |
| johnma | amrith: regarding what makes it to the release notes, that makes sense. | 14:05 |
| amrith | johnma, saw your updates to the rc1 review | 14:05 |
| amrith | I'll add the release notes | 14:06 |
| amrith | and make the change to online | 14:06 |
| amrith | I went through the trove-dashboard rc1 commit | 14:06 |
| amrith | and realized that not many people tagged release notes for trove-dashboard commits | 14:06 |
| amrith | so I actuall crawled through the list and added some | 14:06 |
| amrith | to make it look 'respectable' for a release :) | 14:06 |
| johnma | :) | 14:06 |
| amrith | so it isn't as though not having release notes means we don't add one ... | 14:07 |
| amrith | if there are others that you think we should reference for trove, please post comments | 14:07 |
| amrith | I'll add them to the highlights section. | 14:07 |
| johnma | well the ones I mentioned in my earlier comments on the trove rc1 release notes | 14:07 |
| johnma | I think pretty much the ones I mentioned dont have release notes - like postgresql incremental backup, scheduled backup | 14:08 |
| amrith | thx johnma | 14:09 |
| amrith | I'm just reading pmalik's comment on https://review.openstack.org/369873 | 14:10 |
| amrith | and I'm not sure I agree | 14:10 |
| amrith | pmalik, yt? | 14:10 |
| *** trevormc has joined #openstack-trove | 14:19 | |
| peterstac | amrith, looks like 369873 is a valid fix (as you and pmalik agreed on in the bug) | 14:28 |
| amrith | peterstac, I'm looking to see if we can add a test | 14:29 |
| amrith | to make sure that this is right, and stays right | 14:29 |
| amrith | but I think that may be overkill for this timeframe | 14:29 |
| peterstac | yeah - also I'm not sure how we can test 'shouldn't pull off the wrong value' as you can always do that :) | 14:30 |
| *** itlinux has joined #openstack-trove | 14:35 | |
| *** spilla has joined #openstack-trove | 14:42 | |
| peterstac | amrith, johnma, Does it makes sense to get https://review.openstack.org/#/c/353699 in too, since the cluster locality stuff merged? | 14:43 |
| johnma | peterstac: I am actually testing it right now | 14:44 |
| peterstac | ah, great thanks! | 14:44 |
| *** johnbabers has quit IRC | 14:47 | |
| *** johnbabers has joined #openstack-trove | 14:47 | |
| amrith | thx peterstac I think we can make sure that it constructs the right nova call but let's leave that for now. | 14:50 |
| amrith | should get this into rc1, as a priority | 14:50 |
| amrith | so once it merges, I'll take the new hash, update the release notes and push the trove thing. once the thing that mariam is testing is done, I'll get the dashboard hash. | 14:51 |
| amrith | sound good? | 14:51 |
| peterstac | sounds good | 14:51 |
| peterstac | on a good note - check out all the green on the test runs here! https://review.openstack.org/#/c/362347/ | 14:53 |
| johnma | the dashboard change is approved. | 15:09 |
| johnma | peterstac: that's pretty neat. | 15:10 |
| openstackgerrit | Merged openstack/trove-dashboard: Add cluster support for affinity/anti-affinity https://review.openstack.org/353699 | 15:12 |
| *** pmackinn has quit IRC | 15:19 | |
| openstackgerrit | Merged openstack/trove-integration: Replace assertEqual(None, *) with assertIsNone https://review.openstack.org/362810 | 15:29 |
| *** rcernin has quit IRC | 15:34 | |
| *** pcaruana has quit IRC | 15:40 | |
| *** johnbabers has quit IRC | 15:41 | |
| *** johnbabers has joined #openstack-trove | 15:42 | |
| *** johnbabers has quit IRC | 15:57 | |
| *** johnbabers has joined #openstack-trove | 15:57 | |
| *** johnbabers has quit IRC | 16:03 | |
| *** johnbabers has joined #openstack-trove | 16:09 | |
| *** johnbabers has quit IRC | 16:12 | |
| *** georgelorch has quit IRC | 16:17 | |
| *** georgelorch has joined #openstack-trove | 16:17 | |
| amrith | peterstac, johnma see above https://review.openstack.org/353699. that's it for dashboard, yes? | 16:21 |
| *** johnbabers has joined #openstack-trove | 16:21 | |
| johnma | I believe so amrith. | 16:21 |
| *** johnbabers has quit IRC | 16:23 | |
| *** johnbabers has joined #openstack-trove | 16:24 | |
| peterstac | the only other outstanding one is https://review.openstack.org/#/c/358693/ | 16:24 |
| peterstac | but I don't know how critical it is (doesn't seem like it is) | 16:25 |
| *** rcernin has joined #openstack-trove | 16:34 | |
| *** johnbabers has quit IRC | 16:34 | |
| *** tesseract- has quit IRC | 16:35 | |
| openstackgerrit | Merged openstack/trove: Cluster code looks for wrong field in request for volume type https://review.openstack.org/369873 | 16:35 |
| *** johnbabers has joined #openstack-trove | 16:36 | |
| amrith | good deal ... | 16:36 |
| *** johnbabers has quit IRC | 16:40 | |
| *** pmackinn has joined #openstack-trove | 16:46 | |
| *** johnbabers has joined #openstack-trove | 16:49 | |
| amrith | peterstac, johnma rc1 commits are back up (still WF-1) till trove meeting | 16:53 |
| johnma | soudns good. thanks amrith | 16:54 |
| amrith | would you both please take a look at https://review.openstack.org/#/c/367640/ | 16:55 |
| amrith | the _ changes in taht commit at least should, I think, make it into newton | 16:55 |
| amrith | the lint stuff is only for test | 16:55 |
| amrith | and I'd like to add a gate job for that if possible | 16:56 |
| *** tosky has quit IRC | 16:56 | |
| *** pmackinn has quit IRC | 17:00 | |
| *** pmackinn has joined #openstack-trove | 17:02 | |
| peterstac | amrith, I'll take a look at that now | 17:10 |
| peterstac | also we might want to get in some of the 'easier' reviews that are left (to avoid backporting them to stable/newton once we cut RC1) | 17:10 |
| peterstac | for example: https://review.openstack.org/#/c/368184/ | 17:11 |
| peterstac | johnma ^^^ | 17:11 |
| johnma | sorry missed the messages. Let me take a look at them now | 17:11 |
| amrith | that won't address your issue because when we cut RC1, the master/newton split will happen | 17:11 |
| amrith | I'm not looking to add more stuff to RC1; is that what you are advocating? | 17:12 |
| peterstac | also: https://review.openstack.org/#/c/367607 | 17:12 |
| amrith | I'm open to the idea if we can get them in there | 17:12 |
| amrith | why not put taht list into the meeting agenda | 17:12 |
| peterstac | amrith, yes I was thinking of getting more stuff into RC1 :) | 17:12 |
| *** pmackinn has quit IRC | 17:16 | |
| *** tqtran has joined #openstack-trove | 17:34 | |
| *** NBziouech has quit IRC | 17:37 | |
| *** tqtran has quit IRC | 17:38 | |
| *** Alexey_Abashkin has joined #openstack-trove | 17:38 | |
| amrith | 7lb of sugar in a 12 oz bag | 17:39 |
| *** itlinux has quit IRC | 17:39 | |
| *** AlexeyAbashkin has quit IRC | 17:41 | |
| amrith | let's talk about it | 17:41 |
| amrith | in 19m | 17:41 |
| amrith | johnma, yt? | 17:48 |
| amrith | peterstac, yt? | 17:49 |
| peterstac | ok - I put any remaining changereqs that seem ready into the meeting agenda - there's only 6 more that might still make it IMO | 17:50 |
| peterstac | amrith, here | 17:50 |
| amrith | peterstac, one issue .. | 17:50 |
| amrith | I tried to get https://review.openstack.org/#/c/367640/ ready for chkin | 17:50 |
| amrith | and address johnma's comment | 17:50 |
| amrith | and in the process, I figured I'd check if anything new got into the system since I build by 'exclusion file' | 17:51 |
| amrith | and the tool picked up this: | 17:51 |
| amrith | ERROR: trove/guestagent/datastore/experimental/db2/service.py 256: E1101 no-member, DB2App._update_dbm_config: Instance of 'str' has no 'is_warning' member | 17:51 |
| amrith | which, I think, is a real problem | 17:51 |
| amrith | 256 is the line number | 17:51 |
| amrith | this is something, potentially in johnma's new code. | 17:51 |
| amrith | err would be a string | 17:51 |
| peterstac | right - I checked out the code instead of cherry-picking it when I tested it | 17:52 |
| amrith | well, same difference | 17:52 |
| amrith | this code is an exception path | 17:52 |
| peterstac | so it worked for me | 17:52 |
| amrith | hmm | 17:52 |
| amrith | wonder how | 17:52 |
| peterstac | I should have rebased it so it would check the latest and greatest | 17:52 |
| peterstac | I did that, and running it again ... | 17:53 |
| peterstac | yep, there are a few more errors | 17:54 |
| *** tqtran has joined #openstack-trove | 17:54 | |
| peterstac | well, we can always fix them once we have a non-voting gate job | 17:55 |
| amrith | yes | 17:55 |
| *** songjian has joined #openstack-trove | 17:56 | |
| amrith | so I've pushed up a change with only a change to the readme for now | 17:56 |
| amrith | we can handle the update to the known errors independently | 17:56 |
| *** hongbin has joined #openstack-trove | 18:03 | |
| *** drifterza has joined #openstack-trove | 18:03 | |
| *** pmackinn has joined #openstack-trove | 18:08 | |
| *** drifterza has quit IRC | 18:13 | |
| *** nshah has left #openstack-trove | 18:13 | |
| *** nshah has joined #openstack-trove | 18:13 | |
| *** nshah has left #openstack-trove | 18:13 | |
| *** NBziouech has joined #openstack-trove | 18:23 | |
| *** imandhan has joined #openstack-trove | 18:42 | |
| openstackgerrit | jian.song proposed openstack/trove: Concentrated test_validate_xx_configuration to a new method https://review.openstack.org/369252 | 18:47 |
| amrith | sorry songjian ... just -1'ed that with a suggested change. | 18:51 |
| amrith | sorry for making you do this many times. | 18:51 |
| *** imandhan has quit IRC | 18:53 | |
| openstackgerrit | jian.song proposed openstack/trove: Concentrated test_validate_xx_configuration to a new method https://review.openstack.org/369252 | 18:54 |
| amrith | johnma, will look for the is_warning() fix and try it out | 19:01 |
| *** songjian has quit IRC | 19:01 | |
| *** imandhan has joined #openstack-trove | 19:02 | |
| *** pcaruana has joined #openstack-trove | 19:07 | |
| *** johnbabers has quit IRC | 19:10 | |
| *** johnbabers has joined #openstack-trove | 19:15 | |
| openstackgerrit | Merged openstack/trove: Insulate TroveContext from oslo.context changes https://review.openstack.org/367607 | 19:17 |
| openstackgerrit | Merged openstack/trove: Separate database and user create in prepare https://review.openstack.org/368184 | 19:19 |
| *** imandhan has quit IRC | 19:24 | |
| *** imandhan has joined #openstack-trove | 19:29 | |
| openstackgerrit | mariam john proposed openstack/trove: Handle the exceptions and errors correctly for DB2 https://review.openstack.org/370397 | 19:33 |
| amrith | johnma, ping ... | 19:34 |
| johnma | yes amrith | 19:34 |
| amrith | just looking at your code, have a question for you. | 19:34 |
| johnma | yes | 19:35 |
| amrith | sorry, looked at it again | 19:36 |
| amrith | looks good | 19:36 |
| amrith | thanks | 19:36 |
| amrith | was wondering whether you should do something wiht out and err | 19:36 |
| amrith | but it looks like in other places we don't | 19:36 |
| amrith | so it looks good | 19:36 |
| amrith | will wait for the tests to finish and +2 | 19:36 |
| johnma | right. there isn't any more processing to be done with those. so it didn't make sense to | 19:37 |
| johnma | thanks amrith | 19:37 |
| johnma | iamrith, peterstac: is there anything else I can help with. I need to step out for a bit but will be back soon. | 19:38 |
| amrith | no worries johnma | 19:38 |
| amrith | I think we're good for now | 19:38 |
| johnma | awesome | 19:39 |
| *** _imandhan_ has joined #openstack-trove | 19:39 | |
| *** imandhan has quit IRC | 19:42 | |
| *** johnbabers has quit IRC | 19:45 | |
| *** hoonetorg has quit IRC | 19:53 | |
| *** can8dnSix has joined #openstack-trove | 19:55 | |
| *** johnbabers has joined #openstack-trove | 19:56 | |
| *** pcaruana has quit IRC | 20:00 | |
| *** gouthamr has quit IRC | 20:07 | |
| *** Ravikiran_K has quit IRC | 20:07 | |
| *** hoonetorg has joined #openstack-trove | 20:08 | |
| *** can8dnSix has quit IRC | 20:08 | |
| *** can8dnSix has joined #openstack-trove | 20:13 | |
| openstackgerrit | Merged openstack/trove: Stop adding ServiceAvailable group option https://review.openstack.org/366963 | 20:32 |
| openstackgerrit | Merged openstack/trove: Deprecate 'guest_log_long_query_time' https://review.openstack.org/311237 | 20:33 |
| *** rcernin has quit IRC | 20:36 | |
| *** miqui has joined #openstack-trove | 20:39 | |
| *** rcernin has joined #openstack-trove | 20:40 | |
| *** rcernin has quit IRC | 20:41 | |
| *** rcernin has joined #openstack-trove | 20:41 | |
| *** rcernin has quit IRC | 20:42 | |
| *** rcernin has joined #openstack-trove | 20:42 | |
| *** gouthamr has joined #openstack-trove | 20:44 | |
| openstackgerrit | Merged openstack/trove: Call GuestError with proper options https://review.openstack.org/363796 | 20:51 |
| openstackgerrit | Peter Stachowski proposed openstack/trove-integration: Enhance 'redstack clean' regarding modules https://review.openstack.org/364539 | 20:55 |
| *** _imandhan_ has quit IRC | 20:58 | |
| *** can8dnSix has quit IRC | 20:59 | |
| *** arist has quit IRC | 20:59 | |
| *** arist has joined #openstack-trove | 21:00 | |
| *** imandhan has joined #openstack-trove | 21:15 | |
| openstackgerrit | Peter Stachowski proposed openstack/trove-integration: Enhance 'redstack clean' regarding modules https://review.openstack.org/364539 | 21:15 |
| openstackgerrit | amrith proposed openstack/trove: Handle the exceptions and errors correctly for DB2 https://review.openstack.org/370397 | 21:19 |
| *** johnbabers has quit IRC | 21:32 | |
| *** pmackinn has quit IRC | 21:43 | |
| *** miqui has quit IRC | 22:05 | |
| openstackgerrit | Merged openstack/trove: initial chkin of pylint https://review.openstack.org/367640 | 22:09 |
| *** trevormc has quit IRC | 22:26 | |
| *** rcernin has quit IRC | 22:43 | |
| *** imandhan has quit IRC | 22:50 | |
| *** imandhan has joined #openstack-trove | 22:51 | |
| *** imandhan has quit IRC | 22:52 | |
| *** imandhan has joined #openstack-trove | 22:52 | |
| *** hongbin has quit IRC | 23:20 | |
| amrith | johnma, peterstac what's the verdict? | 23:52 |
| amrith | got with what we have? | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!