johnma | amrith: what peterstac found is a valid bug. I am fixing it right now. will it be too late for rc1 | 00:17 |
---|---|---|
*** tqtran has quit IRC | 00:19 | |
*** johnma has quit IRC | 00:35 | |
*** johnma has joined #openstack-trove | 00:47 | |
*** imandhan has quit IRC | 00:51 | |
*** imandhan has joined #openstack-trove | 00:52 | |
*** imandhan has quit IRC | 00:57 | |
openstackgerrit | mariam john proposed openstack/trove: Handle the exceptions and errors correctly for DB2 https://review.openstack.org/370397 | 00:58 |
amrith | johnma, hello | 01:02 |
amrith | it will *NOT* be too late for RC1 | 01:02 |
amrith | I know the guy driving the bus | 01:02 |
johnma | hi amrith | 01:02 |
amrith | he'll wait ... | 01:02 |
johnma | I fixed that issue and tested it. I am glad peterstac found that | 01:02 |
amrith | https://review.openstack.org/#/c/369823/ is up and all I have to do is update a hash once the change merges | 01:03 |
amrith | and we'll be in good shape. | 01:03 |
amrith | looking at what you changed | 01:03 |
johnma | I am just pushing another one right now . I realised I overwrote the change you made in patchset 2 | 01:03 |
amrith | oops :) | 01:03 |
amrith | sorry | 01:03 |
openstackgerrit | mariam john proposed openstack/trove: Handle the exceptions and errors correctly for DB2 https://review.openstack.org/370397 | 01:04 |
johnma | ok, so thats the final one hopefully. | 01:05 |
johnma | so I am not sure whether that change makes sense but what it was doing initially was to reset a configuration parameter (that supported AUTOMATIC), it was using the command : db2 update dbm cfg using mon_heap_sz automatic(90) | 01:09 |
johnma | instead of 'db2 update dbm cfg using mon_heap_sz 90 automatic' | 01:09 |
amrith | I'm trying to understand it | 01:10 |
amrith | ah, ok | 01:10 |
amrith | let me re-read what the commadn does now | 01:10 |
amrith | i see, you're doing what I'd have normally done with a regexp; find findex and lindex (first and last?) ... makes sense | 01:12 |
amrith | that sure looks like what the code will do | 01:12 |
amrith | now are you sure that AUTOMATIC will be upper-case? | 01:12 |
johnma | yes the command that returns the configuration parameters using the get list dbm configuration returns all the parameters and values in upper case | 01:22 |
*** cdelatte has quit IRC | 01:25 | |
amrith | ok, sounds good johnma | 01:33 |
amrith | i'm betting that peterstac will want to test this | 01:33 |
amrith | peterstac, yt? | 01:33 |
amrith | he usually gets back online later at night | 01:33 |
*** steppp has quit IRC | 01:41 | |
*** steppp has joined #openstack-trove | 01:42 | |
*** imandhan has joined #openstack-trove | 01:51 | |
openstackgerrit | mariam john proposed openstack/trove: Handle the exceptions and errors correctly for DB2 https://review.openstack.org/370397 | 01:53 |
amrith | johnma, I'm going to call it a night. the latest change (5) I'm not sure I understand it. after all for value to become None, wouldn't we just get an exception (out of bounds error) from line 143? | 02:05 |
amrith | otherwise you've got a line (line) on line 138 that reads something like 'foo =' | 02:06 |
amrith | I'm not sure | 02:06 |
amrith | but I'm going to wait to see what the CI says and look again in the morning | 02:06 |
johnma | amrith: it wont throw such an exception. like you said for 'foo=', we split it on '=' | 02:10 |
johnma | so the first would be foo and second string will be empty | 02:11 |
amrith | ok, in that case the code should work | 02:22 |
amrith | i thought split would just make item a one element array | 02:23 |
amrith | in that case | 02:23 |
amrith | no worries | 02:23 |
peterstac | amrith, johnma, taking the new code for a spin now ... | 02:27 |
amrith | put it on gentle, will ya ... | 02:29 |
amrith | hand wash | 02:29 |
amrith | later you can put it in with other colors, and hot water etc., | 02:29 |
amrith | GOOD NIGHT | 02:30 |
peterstac | 'night | 02:40 |
*** imandhan has quit IRC | 02:43 | |
*** imandhan has joined #openstack-trove | 02:43 | |
*** imandhan has quit IRC | 02:58 | |
*** imandhan has joined #openstack-trove | 03:03 | |
peterstac | amrith, johnma, finished testing - everything looks good now so I WF+1'd the change | 03:04 |
peterstac | if the gate's kind it should be done in ~ 2 hours :) | 03:05 |
*** gouthamr has quit IRC | 03:19 | |
*** imandhan has quit IRC | 03:30 | |
*** imandhan has joined #openstack-trove | 03:51 | |
*** imandhan has quit IRC | 03:59 | |
*** imandhan has joined #openstack-trove | 04:00 | |
*** imandhan has quit IRC | 04:23 | |
*** tqtran has joined #openstack-trove | 05:17 | |
*** rcernin has joined #openstack-trove | 05:17 | |
*** tqtran has quit IRC | 05:22 | |
*** gouthamr has joined #openstack-trove | 05:31 | |
*** pcaruana has joined #openstack-trove | 06:04 | |
openstackgerrit | Merged openstack/trove: Handle the exceptions and errors correctly for DB2 https://review.openstack.org/370397 | 06:05 |
*** NBziouech has quit IRC | 06:08 | |
*** pcaruana is now known as pcaruana|afk| | 06:13 | |
*** khushbu_ has joined #openstack-trove | 06:29 | |
openstackgerrit | Cao Xuan Hoang proposed openstack/trove: Remove cfg and logging import unused https://review.openstack.org/361834 | 06:32 |
*** pcaruana|afk| is now known as pcaruana | 06:40 | |
*** gouthamr has quit IRC | 06:41 | |
*** khushbu_ has quit IRC | 06:51 | |
*** khushbu has joined #openstack-trove | 06:53 | |
*** drifterza has joined #openstack-trove | 07:07 | |
*** tqtran has joined #openstack-trove | 07:08 | |
*** Ravikiran_K has joined #openstack-trove | 07:13 | |
*** AJaeger has joined #openstack-trove | 07:16 | |
AJaeger | trove team, amrith : Your coverage post job is failing, see http://logs.openstack.org/47/472159b2d4ada19d6d14cae7f40b2422b59dfd7a/post/trove-coverage-db-ubuntu-xenial/96550c6/console.html | 07:16 |
*** AJaeger has left #openstack-trove | 07:17 | |
*** Ravikiran_K has quit IRC | 08:17 | |
*** khushbu has quit IRC | 08:32 | |
*** khushbu has joined #openstack-trove | 08:38 | |
*** imandhan has joined #openstack-trove | 08:49 | |
*** imandhan has quit IRC | 08:58 | |
*** imandhan has joined #openstack-trove | 09:01 | |
*** imandhan has quit IRC | 09:05 | |
*** tqtran has quit IRC | 09:26 | |
*** khushbu has quit IRC | 09:44 | |
*** tosky has joined #openstack-trove | 09:44 | |
*** khushbu has joined #openstack-trove | 09:45 | |
*** imandhan has joined #openstack-trove | 09:58 | |
*** imandhan has quit IRC | 10:05 | |
*** khushbu has quit IRC | 10:49 | |
*** khushbu has joined #openstack-trove | 11:12 | |
*** khushbu has quit IRC | 11:35 | |
*** imandhan has joined #openstack-trove | 11:45 | |
*** imandhan has quit IRC | 11:49 | |
*** AJaeger has joined #openstack-trove | 11:55 | |
*** AJaeger has left #openstack-trove | 11:55 | |
*** AJaeger has joined #openstack-trove | 12:11 | |
AJaeger | amrith: strange, I cannot reproduce the coverage failure locally as well. Might be a version difference somehow... | 12:18 |
amrith | AJaeger, I checked | 12:19 |
amrith | and the versions are identical on my local machine and in the gate | 12:19 |
AJaeger | amrith: no ideas - just saw it on status.openstack.org/zuul. I suggest you check again the post job after your next merge | 12:19 |
amrith | gist updated with versions | 12:19 |
amrith | AJaeger, wilco | 12:19 |
openstackgerrit | Doug Hellmann proposed openstack/trove-dashboard: Update reno for stable/newton https://review.openstack.org/370753 | 12:22 |
*** pmackinn has joined #openstack-trove | 12:26 | |
openstackgerrit | Doug Hellmann proposed openstack/trove: Update reno for stable/newton https://review.openstack.org/370774 | 12:40 |
openstackgerrit | Merged openstack/trove-dashboard: Update reno for stable/newton https://review.openstack.org/370753 | 12:47 |
*** johnbabers has joined #openstack-trove | 13:18 | |
*** gouthamr has joined #openstack-trove | 13:24 | |
*** hongbin has joined #openstack-trove | 13:52 | |
*** johnbabers has quit IRC | 13:58 | |
*** johnbabers has joined #openstack-trove | 14:01 | |
*** johnbabers has quit IRC | 14:02 | |
*** gouthamr has quit IRC | 14:07 | |
*** drifterza has quit IRC | 14:07 | |
*** NBziouech has joined #openstack-trove | 14:16 | |
*** johnbabers has joined #openstack-trove | 14:20 | |
*** trevormc has joined #openstack-trove | 14:23 | |
*** johnbabers has quit IRC | 14:25 | |
*** johnbabers has joined #openstack-trove | 14:25 | |
mvandijk | amrith: can you confirm that master is now on Ocata? | 14:31 |
trevormc | mvandijk: I think we're waiting on this https://review.openstack.org/#/c/370761/ | 14:34 |
trevormc | or maybe not, I just did a quick search for stable/newton | 14:34 |
mvandijk | yeah it is in github | 14:37 |
amrith | mvandijk, it is ... | 14:52 |
amrith | ocata is now your master | 14:52 |
*** johnbabers has quit IRC | 14:55 | |
*** johnbabers has joined #openstack-trove | 14:58 | |
*** johnbabers has quit IRC | 15:00 | |
*** johnbabers has joined #openstack-trove | 15:02 | |
*** gouthamr has joined #openstack-trove | 15:15 | |
*** gouthamr has quit IRC | 15:17 | |
*** gouthamr has joined #openstack-trove | 15:17 | |
*** pcaruana has quit IRC | 15:19 | |
*** johnbabers has quit IRC | 15:49 | |
*** johnbabers has joined #openstack-trove | 15:52 | |
*** drifterza has joined #openstack-trove | 15:59 | |
openstackgerrit | amrith proposed openstack/trove-dashboard: make trove-dashboard use upper-constraints https://review.openstack.org/370971 | 15:59 |
*** johnbabers has quit IRC | 16:07 | |
openstackgerrit | amrith proposed openstack/trove-dashboard: get trove-dashboard (mitaka) to respect upper-constraints https://review.openstack.org/370978 | 16:11 |
*** aries is now known as aeonic | 16:12 | |
*** trevormc has quit IRC | 16:21 | |
openstackgerrit | Petr Malik proposed openstack/trove: Fix Postgresql configuration test https://review.openstack.org/370982 | 16:26 |
*** rcernin has quit IRC | 16:26 | |
*** johnbabers has joined #openstack-trove | 16:31 | |
*** drifterza has quit IRC | 16:38 | |
*** gouthamr has quit IRC | 16:40 | |
*** gouthamr has joined #openstack-trove | 16:43 | |
openstackgerrit | Merged openstack/trove: Update reno for stable/newton https://review.openstack.org/370774 | 16:53 |
*** tosky has quit IRC | 16:56 | |
*** itlinux has joined #openstack-trove | 17:03 | |
*** NBziouech has quit IRC | 17:07 | |
*** imandhan has joined #openstack-trove | 17:09 | |
*** khushbu_ has joined #openstack-trove | 17:11 | |
*** imandhan has quit IRC | 17:13 | |
*** johnbabers has quit IRC | 17:34 | |
*** imandhan has joined #openstack-trove | 17:36 | |
*** tosky has joined #openstack-trove | 17:39 | |
*** itlinux has quit IRC | 17:43 | |
*** trevormc has joined #openstack-trove | 17:43 | |
*** imandhan has quit IRC | 17:53 | |
*** itlinux has joined #openstack-trove | 17:53 | |
*** johnbabers has joined #openstack-trove | 17:55 | |
*** imandhan has joined #openstack-trove | 17:58 | |
openstackgerrit | Matthew Van Dijk proposed openstack/trove: Improve guestagent datastore models https://review.openstack.org/236082 | 18:06 |
*** khushbu_ has quit IRC | 18:09 | |
*** tqtran has joined #openstack-trove | 18:10 | |
openstackgerrit | Petr Malik proposed openstack/trove: Fix Postgresql configuration test https://review.openstack.org/370982 | 18:10 |
*** khushbu_ has joined #openstack-trove | 18:10 | |
*** imandhan has quit IRC | 18:19 | |
*** imandhan has joined #openstack-trove | 18:20 | |
*** pcaruana has joined #openstack-trove | 18:22 | |
*** _imandhan_ has joined #openstack-trove | 18:24 | |
*** imandhan has quit IRC | 18:27 | |
*** _imandhan_ has quit IRC | 18:38 | |
*** imandhan has joined #openstack-trove | 18:39 | |
*** imandhan has quit IRC | 18:44 | |
*** imandhan has joined #openstack-trove | 18:45 | |
*** khushbu_ has quit IRC | 18:51 | |
*** imandhan has quit IRC | 18:59 | |
*** pcaruana has quit IRC | 19:32 | |
*** itlinux has quit IRC | 19:39 | |
*** david-lyle has quit IRC | 19:40 | |
*** david-lyle has joined #openstack-trove | 19:40 | |
AJaeger | amrith: this coverage job fails as well - http://logs.openstack.org/de/dee522d8b14b1fdf73668e678374a91784b86ba3/ - I'm at wits end ;( | 19:41 |
*** itlinux has joined #openstack-trove | 19:48 | |
*** rcernin has joined #openstack-trove | 19:53 | |
*** imandhan has joined #openstack-trove | 20:13 | |
*** itlinux has quit IRC | 20:13 | |
*** spilla has left #openstack-trove | 20:17 | |
*** spilla has joined #openstack-trove | 20:17 | |
amrith | AJaeger, if you are at your wits end, that is not a good sign. | 20:30 |
amrith | i'm trying to figure out what's up but as far as I can tell, the setups are identical. | 20:30 |
amrith | only thing I can think of is that this is a xenial thing so I'm spinning up a xenial vm | 20:31 |
amrith | I'm still running trusty turd or whatever they call it. | 20:32 |
*** johnbabers has quit IRC | 20:40 | |
*** johnbabers has joined #openstack-trove | 20:41 | |
*** johnbabers has quit IRC | 20:42 | |
*** johnbabers has joined #openstack-trove | 20:43 | |
*** johnbabers has quit IRC | 20:47 | |
*** gouthamr has quit IRC | 20:54 | |
*** arist has quit IRC | 21:00 | |
*** arist has joined #openstack-trove | 21:01 | |
*** gouthamr has joined #openstack-trove | 21:09 | |
*** catintheroof has joined #openstack-trove | 21:15 | |
trevormc | amrith, AJaeger: I was able to reproduce the error in ubuntu 14.04, http://paste.openstack.org/show/577726/ | 21:18 |
trevormc | so not just a xenial thing right? | 21:18 |
*** spilla has quit IRC | 21:27 | |
amrith | well, in that case, yes it isn't a xenial thing | 21:27 |
trevormc | We are one of three openstack projects that use the --no-parallel option in our tox.ini file, the option was added 17 days ago. Is there a reason we need it? | 21:28 |
amrith | because without it, it doesn't work too well :) | 21:28 |
amrith | do the other projects that use it also have the same invocation of setup.py? | 21:28 |
amrith | or do they have --no-parallel in some other sequence on the command line? | 21:28 |
trevormc | I used codesearch http://codesearch.openstack.org/?q=--no-parallel&i=nope&files=&repos= | 21:29 |
trevormc | so to answer your question, no | 21:29 |
trevormc | not sure about the invocation of setup.py | 21:29 |
amrith | does the congress job fail as well? | 21:30 |
trevormc | still investigating | 21:30 |
amrith | i have to take a short break, I'll be back in a bit. | 21:31 |
trevormc | AJaeger: how do you search for gate jobs like that one you provided. | 21:36 |
*** elfosardo has joined #openstack-trove | 21:40 | |
*** cargonza has quit IRC | 21:46 | |
*** AJaeger has quit IRC | 21:46 | |
*** amit213 has quit IRC | 21:47 | |
*** dkehn has quit IRC | 21:47 | |
*** vkmc has quit IRC | 21:47 | |
trevormc | amrith: the congress tox -ebench which uses the --no-parallel option is experiencing the same problem | 21:48 |
*** amit213 has joined #openstack-trove | 21:51 | |
*** cargonza has joined #openstack-trove | 21:52 | |
*** vkmc has joined #openstack-trove | 21:57 | |
*** dkehn has joined #openstack-trove | 21:59 | |
*** AJaeger has joined #openstack-trove | 22:00 | |
trevormc | I think it's a problem with python 2.7 | 22:09 |
*** hongbin has quit IRC | 23:13 | |
*** catintheroof has quit IRC | 23:21 | |
openstackgerrit | Trevor McCasland proposed openstack/trove: Fix error: option --no-parallel not recognized https://review.openstack.org/371218 | 23:30 |
*** trevormc has quit IRC | 23:31 | |
*** hoonetorg has quit IRC | 23:37 | |
*** tqtran has quit IRC | 23:52 | |
*** hoonetorg has joined #openstack-trove | 23:55 | |
*** tqtran has joined #openstack-trove | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!