Monday, 2017-10-23

*** LindaWang has joined #openstack-rally01:06
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #openstack-rally01:56
*** psuriset has joined #openstack-rally02:10
*** dave-mccowan has quit IRC03:05
*** aojea has joined #openstack-rally03:15
*** aojea has quit IRC03:19
*** psuriset has quit IRC03:32
*** psuriset has joined #openstack-rally03:49
*** yamamoto has joined #openstack-rally04:16
*** Dinesh_Bhor has joined #openstack-rally04:19
*** spectr has joined #openstack-rally05:52
*** yamamoto has quit IRC06:33
*** yamamoto has joined #openstack-rally06:52
*** pcaruana has joined #openstack-rally07:00
*** serlex has joined #openstack-rally07:02
*** yamamoto_ has joined #openstack-rally07:11
*** yamamoto has quit IRC07:13
*** aojea has joined #openstack-rally07:16
*** aojea has quit IRC07:20
*** tesseract has joined #openstack-rally07:27
*** yamamoto_ has quit IRC07:29
*** yamamoto has joined #openstack-rally07:29
*** yamamoto has quit IRC07:43
*** yamamoto has joined #openstack-rally07:57
*** yamamoto has quit IRC08:03
*** mrch has joined #openstack-rally08:11
*** e0ne has joined #openstack-rally08:13
*** spectr has quit IRC08:35
*** yamamoto has joined #openstack-rally08:36
*** yamamoto has quit IRC08:38
*** ushkalim has joined #openstack-rally08:49
*** spectr has joined #openstack-rally08:49
*** yamamoto has joined #openstack-rally08:50
*** tosky has joined #openstack-rally08:54
*** yamamoto has quit IRC09:03
*** yamamoto has joined #openstack-rally09:13
*** mvk has quit IRC09:27
*** spectr has quit IRC09:31
*** spectr has joined #openstack-rally09:31
*** LindaWang has quit IRC09:45
*** yamamoto has quit IRC09:58
*** yamamoto has joined #openstack-rally09:59
*** mvk has joined #openstack-rally10:02
*** spectr has quit IRC10:04
*** yamamoto has quit IRC10:06
*** yamamoto has joined #openstack-rally10:06
openstackgerritBoris Pavlovic proposed openstack/rally master: [wip] Introduce Envs and Platforms  https://review.openstack.org/46659810:10
*** oanufrii has joined #openstack-rally10:14
*** aojea has joined #openstack-rally10:16
*** aojea has quit IRC10:20
*** spectr has joined #openstack-rally10:21
*** yamamoto_ has joined #openstack-rally10:37
*** yamamoto has quit IRC10:40
*** spectr has quit IRC10:45
*** spectr has joined #openstack-rally10:47
*** yamamoto_ has quit IRC10:48
*** yamamoto has joined #openstack-rally10:50
*** yamamoto has quit IRC10:55
*** yamamoto has joined #openstack-rally10:55
*** dave-mccowan has joined #openstack-rally11:04
*** stpierre has joined #openstack-rally11:04
*** jkilpatr has joined #openstack-rally11:19
*** yamamoto_ has joined #openstack-rally11:30
*** LindaWang has joined #openstack-rally11:31
*** yamamoto_ has quit IRC11:31
*** yamamoto has quit IRC11:32
*** yamamoto has joined #openstack-rally11:32
*** lpetrut has joined #openstack-rally11:39
*** yamamoto has quit IRC11:52
*** yamamoto has joined #openstack-rally11:53
*** LindaWang has quit IRC11:57
*** yamamoto has quit IRC11:58
*** jlabarre has joined #openstack-rally12:04
*** LindaWang has joined #openstack-rally12:07
*** e0ne has quit IRC12:25
*** yamamoto has joined #openstack-rally12:36
a1C2LS54O[From Gitter] shihai1991 : @12:37
a1C2LS54O[From Gitter] shihai1991 : @cxhuawei I would review your patch tonight.12:38
a1C2LS54O[From Gitter] andreykurilin : I already started, but need to setup vm with a devstack to experiment12:40
a1C2LS54O[From Gitter] shihai1991 : @andreykurilin Cool:)12:40
*** pcaruana has quit IRC12:41
a1C2LS54O[From Gitter] andreykurilin : yup, after cutting release(finally), I have more free time :)12:41
a1C2LS54O[From Gitter] shihai1991 : Looks you have merged a lot patches.12:43
a1C2LS54O[From Gitter] andreykurilin : we had issues with functional tests:(12:43
a1C2LS54O[From Gitter] andreykurilin : so it was hard 2 weeks fixing them12:44
a1C2LS54O[From Gitter] shihai1991 : What's the problem?12:45
a1C2LS54O[From Gitter] andreykurilin : let's discuss it in 15 minutes at our weekly meeting:)12:45
*** yamamoto has quit IRC12:46
a1C2LS54O[From Gitter] andreykurilin : @chenhb-zte @cxhuawei : notification: we have a weekly meeting in 15 minutes12:46
a1C2LS54O[From Gitter] shihai1991 : OK. andrey, I have a small question. Could I throw a sys.exit(1) if the task failed.12:46
a1C2LS54O[From Gitter] andreykurilin : it is also a good topic to discuss :)12:50
*** pcaruana has joined #openstack-rally12:57
*** pcaruana has quit IRC13:01
*** yamamoto has joined #openstack-rally13:02
a1C2LS54O[From Gitter] andreykurilin : @shihai1991 @chenhb-zte @chenhb-zte meeting!13:05
a1C2LS54O[From Gitter] andreykurilin : it is a new time:)13:05
a1C2LS54O[From Gitter] andreykurilin : as we discussed previously13:05
a1C2LS54O[From Gitter] chenhb-zte : ping13:07
a1C2LS54O[From Gitter] andreykurilin : ok, let's start13:07
a1C2LS54O[From Gitter] andreykurilin : First of all, we finally released 0.10.013:07
a1C2LS54O[From Gitter] andreykurilin : it was delayed fo almost 6 month13:08
a1C2LS54O[From Gitter] andreykurilin : :(13:08
a1C2LS54O[From Gitter] andreykurilin : but we made a good progres s of refactoring inner stuff13:08
a1C2LS54O[From Gitter] andreykurilin : hope, the next release will be in less then a month13:08
a1C2LS54O[From Gitter] andreykurilin : so congrats and thank you!13:08
a1C2LS54O[From Gitter] chenhb-zte : :smile:13:09
a1C2LS54O[From Gitter] andreykurilin : There is one important note which I want to mention13:09
a1C2LS54O[From Gitter] andreykurilin : Please, check sometimes results of the jobs even with success status13:10
a1C2LS54O[From Gitter] andreykurilin : checking logs is also quite important13:10
a1C2LS54O[From Gitter] andreykurilin : https://github.com/openstack/rally/commit/1194aa4311b8498a7a7ff19db8ed64e22905bd3613:10
a1C2LS54O[From Gitter] andreykurilin : Starting from augest (my patch) we ignored results of functional job13:11
a1C2LS54O[From Gitter] andreykurilin : and before the release we had ~20 failing tests13:11
a1C2LS54O[From Gitter] andreykurilin : despite the fact that most of them relate to out-dated testing code, there were some critical bugs ;(13:11
a1C2LS54O[From Gitter] andreykurilin : the good news: after small refactoring, functional tests do not require openstack and can be launched locally in quite simple way13:12
a1C2LS54O[From Gitter] andreykurilin : `tox -efunctional`13:12
a1C2LS54O[From Gitter] chenhb-zte : got13:13
a1C2LS54O[From Gitter] andreykurilin : also, particual test can be launched too `tox -e functional -- tests/functional/test_cli_task.py::TaskTestCase::test_status`13:13
a1C2LS54O[From Gitter] andreykurilin : @shihai1991 this is the issue I mentioned earlier ^13:14
a1C2LS54O[From Gitter] shihai1991 : ok13:15
a1C2LS54O[From Gitter] chenhb-zte : i like this improvement13:15
a1C2LS54O[From Gitter] andreykurilin : ok, the next topic throwing not zero exit code in `task start` in case of failures13:16
a1C2LS54O[From Gitter] shihai1991 : yep. This is my issue.13:17
a1C2LS54O[From Gitter] andreykurilin : I think it can be a good improvement which can simplify automatization scripts13:17
a1C2LS54O[From Gitter] andreykurilin : about implentation: sys.exit is used in cliutils.run method, so the start method itself should just return a right value13:18
a1C2LS54O[From Gitter] andreykurilin : Also, it think we should leave 1 value for exceptions (in most cases they relate to unexpected things) and use 2 or 3 (just another not-zero value)13:19
a1C2LS54O[From Gitter] andreykurilin : thoughts ?13:19
a1C2LS54O[From Gitter] chenhb-zte : loading task, context, scenario13:21
a1C2LS54O[From Gitter] shihai1991 : Ok, i could return it from self.details()13:22
a1C2LS54O[From Gitter] andreykurilin : so it will influence `task details` command itself. not sure that it should return exit code depend on task status. but making an external call at start method is not a solution as well..13:25
a1C2LS54O[From Gitter] chenhb-zte : use some exception codes?  like http code13:27
a1C2LS54O[From Gitter] andreykurilin : ideally yes, but it will limit us to use custom exceptions for one place usage13:28
a1C2LS54O[From Gitter] shihai1991 : This is a probleam.13:29
a1C2LS54O[From Gitter] chenhb-zte : i did not understand what your meaning.13:32
a1C2LS54O[From Gitter] andreykurilin : what exactly?)13:32
a1C2LS54O[From Gitter] chenhb-zte : why it limits us to use custom exception?13:33
a1C2LS54O[From Gitter] shihai1991 : it would coverage the other errors?13:34
a1C2LS54O[From Gitter] andreykurilin : @chenhb-zte as for the case from which the topic started, we need a separate code for SLA failures. It sounds a bit strange to create SLAFailedException just to be called once in rally.cli.commands.Task.start method13:35
a1C2LS54O[From Gitter] andreykurilin : it can help with documentation, but I see no problem to use exit code X in different commands with different meanings. We can document the meaning in docstrings for each method13:37
a1C2LS54O[From Gitter] chenhb-zte : Actually, only the question is whether we need these codes for every exception.13:41
a1C2LS54O[From Gitter] andreykurilin : I suppose no. We can do like:13:43
a1C2LS54O[From Gitter] chenhb-zte : Maybe it is enough that we just return 1. :smile:13:44
a1C2LS54O[From Gitter] andreykurilin : maybe it will simplify the case... but usually 1 means that something is broken..13:45
a1C2LS54O[From Gitter] shihai1991 : Do we have define this error?13:45
a1C2LS54O[From Gitter] andreykurilin : what do you mean?13:46
a1C2LS54O[From Gitter] shihai1991 : ^define the 1's error13:47
a1C2LS54O[From Gitter] andreykurilin : 1 error usually represents any error)13:47
a1C2LS54O[From Gitter] andreykurilin : ok, let's return 1 as for now... I do not have a good solution..13:48
a1C2LS54O[From Gitter] shihai1991 : OK. got it.13:48
a1C2LS54O[From Gitter] chenhb-zte : i want we could discuss this detailedly if more users request this feature.13:50
a1C2LS54O[From Gitter] shihai1991 : so we can just use 1 temporarily and write some notes for it.13:51
a1C2LS54O[From Gitter] andreykurilin : ok13:55
a1C2LS54O[From Gitter] andreykurilin : anything else to discuss?)13:55
a1C2LS54O[From Gitter] chenhb-zte : nothing from me. by the way,  i did not find v0.10.0 release not at readthedocs.13:56
a1C2LS54O[From Gitter] chenhb-zte : *note13:57
a1C2LS54O[From Gitter] shihai1991 : from my experience now, rally can develop to devops.13:57
a1C2LS54O[From Gitter] andreykurilin : :)13:57
a1C2LS54O[From Gitter] andreykurilin : @chenhb-zte strange. need to check13:57
a1C2LS54O[From Gitter] shihai1991 : i have try to save some benchmark datas and hope nova's ptl could get some idea from it.13:58
a1C2LS54O[From Gitter] chenhb-zte : :smile:  we have done that.13:58
a1C2LS54O[From Gitter] shihai1991 : which one?13:58
a1C2LS54O[From Gitter] andreykurilin : @chenhb-zte it looks like after infra's team finished migration to zuul v3, posting job to rtd doesn't trigger rebuilding13:59
a1C2LS54O[From Gitter] chenhb-zte : develop rally to devops13:59
a1C2LS54O[From Gitter] shihai1991 : cools. Could you share some core ideas?14:00
a1C2LS54O[From Gitter] shihai1991 : I found some interested project from seecloud. like https://github.com/seecloud/performance.14:01
a1C2LS54O[From Gitter] shihai1991 : I have any other issue to discuss:)14:02
a1C2LS54O[From Gitter] shihai1991 : ^no other issue14:03
a1C2LS54O[From Gitter] andreykurilin : ok)14:03
a1C2LS54O[From Gitter] chenhb-zte : @shihai1991 sure.14:03
a1C2LS54O[From Gitter] cxhuawei : ehh, I forgot the meeting. :worried:14:38
*** tosky has quit IRC14:46
*** e0ne has joined #openstack-rally15:10
*** LindaWang has quit IRC15:24
*** ushkalim has quit IRC15:24
*** mrch has quit IRC15:34
*** oanufrii has quit IRC15:43
openstackgerritchenxu proposed openstack/rally master: OSProfiler Intergration  https://review.openstack.org/50044815:49
*** jkilpatr has quit IRC15:55
*** psuriset has quit IRC16:01
*** psuriset has joined #openstack-rally16:18
*** itlinux has joined #openstack-rally16:18
*** tosky has joined #openstack-rally16:28
*** lpetrut has quit IRC16:30
*** jkilpatr has joined #openstack-rally16:32
*** mvk has quit IRC16:49
*** serlex has quit IRC16:58
*** lpetrut has joined #openstack-rally17:18
*** tesseract has quit IRC17:18
*** lpetrut has quit IRC17:29
*** mvk has joined #openstack-rally17:32
*** e0ne has quit IRC18:06
*** e0ne has joined #openstack-rally18:08
*** e0ne has quit IRC18:10
*** aojea has joined #openstack-rally18:24
*** e0ne has joined #openstack-rally18:47
*** e0ne has quit IRC18:52
*** itlinux has quit IRC18:52
*** harlowja has joined #openstack-rally18:53
*** e0ne has joined #openstack-rally18:55
*** itlinux has joined #openstack-rally18:56
*** e0ne has quit IRC19:00
*** aojea has quit IRC19:16
*** e0ne has joined #openstack-rally19:20
*** e0ne has quit IRC19:25
*** aojea has joined #openstack-rally19:31
*** aojea has quit IRC19:33
*** e0ne has joined #openstack-rally19:33
*** aojea has joined #openstack-rally19:34
*** e0ne has quit IRC19:38
*** itlinux has quit IRC19:39
*** aojea has quit IRC19:43
*** e0ne has joined #openstack-rally19:47
*** psuriset has quit IRC19:50
*** e0ne has quit IRC19:52
*** e0ne has joined #openstack-rally19:59
*** e0ne has quit IRC20:04
*** e0ne has joined #openstack-rally20:11
*** e0ne has quit IRC20:16
*** e0ne has joined #openstack-rally20:23
*** e0ne has quit IRC20:28
*** e0ne has joined #openstack-rally20:31
*** e0ne has quit IRC20:35
*** e0ne has joined #openstack-rally20:45
*** lpetrut has joined #openstack-rally20:45
*** e0ne has quit IRC20:50
*** e0ne has joined #openstack-rally20:52
*** e0ne has quit IRC20:57
*** dave-mccowan has quit IRC21:28
*** MasterOfBugs has joined #openstack-rally21:34
*** pramodrj07 has joined #openstack-rally21:34
*** jlabarre has quit IRC21:44
*** lpetrut has quit IRC21:55
*** dave-mccowan has joined #openstack-rally22:49
*** dave-mccowan has quit IRC23:27
*** tosky has quit IRC23:34

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