Friday, 2018-01-12

*** bigdogstl has quit IRC00:11
*** bigdogstl has joined #congress00:11
*** bigdogstl has quit IRC00:21
*** bigdogstl has joined #congress00:27
*** bigdogstl has quit IRC00:31
*** bigdogstl has joined #congress00:41
*** bigdogstl has quit IRC00:46
*** ekcs has quit IRC00:47
*** ekcs has joined #congress00:49
*** bigdogstl has joined #congress00:50
openstackgerritEric Kao proposed openstack/congress master: Update cfgvalidator_driver for oslo.config 5.2  https://review.openstack.org/53259301:17
openstackgerritEric Kao proposed openstack/congress master: Mistral driver  https://review.openstack.org/52942701:17
*** masahito has joined #congress02:26
*** ramineni_ has joined #congress02:51
*** bigdogstl has quit IRC03:14
*** bigdogstl has joined #congress03:18
*** ekcs has quit IRC03:28
*** ekcs has joined #congress03:28
*** bigdogstl has quit IRC03:33
*** masahito has quit IRC03:36
*** bigdogstl has joined #congress03:48
*** bigdogstl has quit IRC03:57
*** bigdogstl has joined #congress03:58
*** bigdogstl has quit IRC04:05
*** bigdogstl has joined #congress04:07
*** bigdogstl has quit IRC04:12
*** bigdogstl has joined #congress04:32
*** bigdogstl has quit IRC04:37
ramineniekcs: ekcs_ : hi04:48
*** bigdogstl has joined #congress05:04
*** bigdogstl has quit IRC05:15
openstackgerritMerged openstack/congress-tempest-plugin master: Updated from global requirements  https://review.openstack.org/53295205:21
*** absubram has joined #congress05:23
*** absubram has quit IRC05:23
*** ramineni_ has quit IRC06:11
*** bigdogstl has joined #congress06:17
*** bigdogstl has quit IRC06:22
*** bigdogstl has joined #congress06:23
ekcshi ramineni06:25
ramineniekcs: hijust a small question06:26
ramineniin bumping version python-congressclient>=1.9.0,<200006:26
ramineniwhat does <2000 mean06:26
raminenijust curious06:26
*** absubram has joined #congress06:26
ekcsI think initially there was some reason why they didn’t like having a completely unbounded requirement.06:27
ekcsI don’t know all the details.06:27
ekcsbut I remember reading it somewhere.06:27
ramineniekcs: ohok06:27
ekcsnot sure it’s still needed now with upper-constraints in place, but doesn’t really matter for now.06:28
ekcshey when you get a chance could you merge this patch so others can rebase on it and proceed?06:29
ekcshttps://review.openstack.org/#/c/532593/306:29
ramineniekcswait a sec06:30
ramineniekcs: I don't see pierre comments addressed , or im missing something06:30
ekcsI updated the patch to do basically what he suggested.06:33
ramineniekcs: or ill do if its late for you06:33
ramineniekcs: choices = list(cfg_type.choices.keys())06:33
raminenithis should be enough, without lambda function06:33
ekcsyea but it’s doing the same thing. simpler is better, but i’d rather get this in asap so things don’t get held up. then if it’s important a separate patch can change it.06:34
ekcsnew patch + check can take 3 hours.06:34
raminenihaha06:35
ekcsi didn’t look at it carefully enough earlier, cuz I only paid attention to the need to add list(..) based on what he said.06:35
ramineniekcs: ill take care of it, is there anyurgent patches to be merged?06:35
ekcsit i had read more carefully i would’ve done it, but at this point i’d rather just merge.06:36
ramineniill merge them as well after this06:36
ekcscuz it’s a cosmetic difference not an important difference.06:36
ekcswell as soon as you find the mistral patch is ready we can merge. but this is the main one cuz anything else someone does would fail and need rebase and recheck later (3 hours delay lol)06:38
ekcsoh yea and if gate fails then it’s another 3 hr check + 1 hr gate. so really want to get on with it for something holding up the gate and interfering with other work.06:39
*** bigdogstl has quit IRC06:41
ekcsif this one looks ready to you please go ahead and merge it: https://review.openstack.org/#/c/529427/06:42
ramineniekcs: for mistral driver , can we wait till tempest tests pass06:43
ramineniI would prefer to merge as complete feature06:43
ramineniekcs: as I have seen , api name is wrong in last patch, so that we can avoid such things, if tempest tests pass06:44
ekcsok. let me see if I can update the tempest stuff now and hopefully we can merge before the weekend. is it ok if we see tempest pass and merge the driver even if the full tempest test is not ready to merge?06:48
ekcsbecause for sake of time I disable the OOO tests in those patches where I’m not sure it’ll suceed (because of skip checking or other reasons).06:49
ekcsthen in order for the tempest patch to be ready to merge the OOO tests need to be turned back on (another 3+ hrs wait) and then it can be ready.06:50
ekcsbut if I update the tempest test now with OOO disabled we can get result in 30=40 minutes.06:50
ramineniekcs: is it that urgent for you06:52
ramineniis it not ok by next week?06:52
ramineniekcs: let the gate / tempest tests stabilize ,we could merge mistral on priority06:53
ramineniekcs: zuul also slow now a days (today seems faster though), it would take time for you06:53
ekcsI guess that’s fine.06:56
ekcsI just really hate dealing with patch conflicts. because the driver changes zuul settings.06:56
ekcsand other changes change zuul settings we get conflicts.06:57
ekcsand so it mean each time a patch merges other patches need to resolve conflict and wait 3+ hrs lol before ready to merge.06:57
ekcsall the extra waiting time can really add up when trying to get pieces done by queens-3.06:59
ekcsbut hopefully it’ll be fine.07:00
ramineniekcs: ok07:01
ekcshey on this patch: https://review.openstack.org/#/c/531534/07:02
ekcswhy do you say it’s not required anymore?07:03
ekcsi’m pretty sure the other patch you referenced won’t prevent the skip.07:03
ekcsoh nvm i’m kind of confused.07:05
ekcsok nvm what I said. i was confused between the two different tempest things.07:07
ekcsbut that reminded me of something: ramineni could we hold off on switching congress check/gate to congress plugin until after queens-3?07:07
ekcsit’s not required to be done before feature freeze, and I’m thinking it’s safest to continue using existing check/gate while cherry-picking each change to the new repo.07:08
ekcs< 2 weeks before queens-3 just a bad time to risk holding up patches.07:09
*** ekcs has quit IRC07:10
*** ekcs has joined #congress07:11
ekcsoops sorry ramineni I got disconnected. not sure how much of what I said got through.07:11
*** bigdogstl has joined #congress07:16
*** ekcs has quit IRC07:19
*** ekcs has joined #congress07:20
*** bigdogstl has quit IRC07:20
openstackgerritEric Kao proposed openstack/congress master: Mistral driver  https://review.openstack.org/52942707:42
openstackgerritEric Kao proposed openstack/congress master: simplify dict-to-list conversion  https://review.openstack.org/53311507:46
*** bigdogstl has joined #congress07:46
*** bigdogstl has quit IRC07:57
*** bigdogstl has joined #congress08:12
*** bigdogstl has quit IRC08:16
ramineniekcs: we need to swicth, otherwise other tempest plugin tests doesn't pass , they get skipped08:16
*** AlexeyAbashkin has joined #congress08:23
openstackgerritShangXiao proposed openstack/congress master: [Trivialfix]Fix typos in congress  https://review.openstack.org/49196508:38
openstackgerritAnusha Ramineni proposed openstack/congress master: Enable skipped heat, murano, aodh tests  https://review.openstack.org/53313308:58
*** bigdogstl has joined #congress08:58
openstackgerritAnusha Ramineni proposed openstack/congress master: Enable skipped heat, murano, aodh tests  https://review.openstack.org/53313309:01
*** bigdogstl has quit IRC09:10
*** bigdogstl has joined #congress09:28
*** bigdogstl has quit IRC09:33
*** bigdogstl has joined #congress10:00
*** bigdogstl has quit IRC10:12
openstackgerritAnusha Ramineni proposed openstack/congress-tempest-plugin master: Remove cwd  https://review.openstack.org/53278710:30
*** AlexeyAbashkin has quit IRC10:57
*** AlexeyAbashkin has joined #congress11:12
*** bigdogstl has joined #congress11:16
openstackgerritMerged openstack/congress master: Update cfgvalidator_driver for oslo.config 5.2  https://review.openstack.org/53259311:25
*** bigdogstl has quit IRC11:28
openstackgerritAnusha Ramineni proposed openstack/congress master: Enable skipped heat, murano, aodh tests  https://review.openstack.org/53313311:46
openstackgerritMerged openstack/congress-tempest-plugin master: Use telemetry-tempest-plugin for aodh tests  https://review.openstack.org/53246411:48
*** bigdogstl has joined #congress12:16
*** bigdogstl has quit IRC12:30
*** bigdogstl has joined #congress13:02
*** absubram has quit IRC13:11
*** bigdogstl has quit IRC13:19
*** bigdogstl has joined #congress13:19
*** bigdogstl has quit IRC13:40
*** bigdogstl has joined #congress14:10
*** bigdogstl has quit IRC14:15
*** bigdogstl has joined #congress14:26
*** bigdogstl has quit IRC15:09
*** bigdogstl has joined #congress15:10
*** AlexeyAbashkin has quit IRC15:34
-openstackstatus- NOTICE: Zuul has been restarted and lost queue information; changes in progress will need to be rechecked.16:43
*** absubram has joined #congress17:00
*** AlexeyAbashkin has joined #congress17:12
*** AlexeyAbashkin has quit IRC17:17
openstackgerritPierre Crégut proposed openstack/congress master: Fix serialization of choices and reenable congress-agent.  https://review.openstack.org/53324817:19
*** AlexeyAbashkin has joined #congress17:21
*** AlexeyAbashkin has quit IRC17:25
*** absubram has quit IRC17:34
*** AlexeyAbashkin has joined #congress17:42
*** AlexeyAbashkin has quit IRC17:44
*** AlexeyAbashkin has joined #congress19:25
*** AlexeyAbashkin has quit IRC19:42
*** bigdogstl has quit IRC19:54
*** bigdogstl has joined #congress20:13
*** bigdogstl has quit IRC20:19
*** bigdogstl has joined #congress20:25
*** bigdogstl has quit IRC20:30
*** bigdogstl has joined #congress20:33
*** openstackstatus has quit IRC20:46
*** openstack has joined #congress20:48
*** ChanServ sets mode: +o openstack20:48
*** AlexeyAbashkin has joined #congress22:54
*** AlexeyAbashkin has quit IRC22:59

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