Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2015-02-16

| Channels | #openstack-rally index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:09 oanufriev joined #openstack-rally
00:51 openstackgerrit Mikhail Dubov proposed stackforge/rally: Fixes in docs texts and images  https://review.openstack.org/156093
03:10 jaypipes joined #openstack-rally
04:52 exploreshaifali joined #openstack-rally
04:53 nkhare joined #openstack-rally
05:32 psd_ joined #openstack-rally
05:42 rdas joined #openstack-rally
05:43 Mehrnaz joined #openstack-rally
05:44 anshul joined #openstack-rally
05:58 tfreger joined #openstack-rally
05:59 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Add create and delete routers scenarios for Neutron  https://review.openstack.org/113177
06:26 psd__ joined #openstack-rally
06:59 Mehrnaz_ joined #openstack-rally
07:12 yfried|afk joined #openstack-rally
07:13 andreykurilin_ joined #openstack-rally
07:16 yfried joined #openstack-rally
07:23 rdas joined #openstack-rally
07:32 openstackgerrit Mikhail Dubov proposed stackforge/rally: Fixes in docs texts and images  https://review.openstack.org/156093
07:38 neeti joined #openstack-rally
07:44 psd_ joined #openstack-rally
07:47 msdubov joined #openstack-rally
07:50 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Add create and delete routers scenarios for Neutron  https://review.openstack.org/113177
07:51 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Add create and delete routers scenarios for Neutron  https://review.openstack.org/113177
07:52 nmagnezi joined #openstack-rally
08:36 fhubik joined #openstack-rally
08:36 arxcruz joined #openstack-rally
08:44 paboldin guys, can one please clarify what is the purpose of this? https://review.openstack.org/#/c/155713/
08:44 paboldin and please review this https://review.openstack.org/#/c/151384/10
08:48 oanufriev joined #openstack-rally
08:57 openstackgerrit joined #openstack-rally
09:11 msdubov redixin, seems like paboldin wanted to ping you ^
09:13 psd_ joined #openstack-rally
09:14 kairat_kushaev joined #openstack-rally
09:26 amaretskiy joined #openstack-rally
09:27 wtakase joined #openstack-rally
09:28 wtakase boris-42: Hello.
09:39 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Add create and delete routers scenarios for Neutron  https://review.openstack.org/113177
09:39 psd_ msdubov, ^ :)
09:41 msdubov psd_, Great, thank you!
09:41 msdubov psd_, Why did you need this line?  with mock_clients("neutron").remove_interface_router as rem:
09:42 psd_ msdubov, Couldn't get is under 80 characters otherwise
09:42 psd_ it*
09:42 msdubov psd_, Couldn't you just write mock_clients("neutron").remove_interface_router.assert_called_once_with(...)?
09:42 psd_ very long line
09:42 msdubov psd_, Well, anyway it would be better to split this line than to have this with ... statement
09:42 msdubov psd_, We're generally trying to avoid such things
09:43 psd_ tried breaking it like mock_clients("neutron").remove_interface_router
09:43 psd_ .assert_called_once_with(...) but was getting error
09:44 psd_ how should I split it
09:46 psd_ tox shouts if I use backslash
09:46 psd_ msdubov, ^
09:46 msdubov psd_, Ah really
09:46 msdubov psd_ Okay, than let it keep so :)
09:46 psd_ http://stackoverflow.com/questions/53162/how-can-i-do-a-line-break-line-continuation-in-python
09:47 psd_ msdubov, Okay :)
09:48 msdubov psd_ By the way have you tested it?
09:50 psd_ msdubov, I did, but I am getting error now :(
09:50 psd_ gimme a minute
09:51 psd_ msdubov, is it possible that tox returns different results in consecutive runs?(without changes)
09:52 paboldin psd_: of course. is is only a piece of software and it can have it's own bugs.
09:53 psd_ hm, I will double check
09:58 boris-42 wtakase: hey hey
09:58 boris-42 wtakase: so about EC2API tests
09:58 openstackgerrit Wataru Takase proposed stackforge/rally: Add name pattern filter for resource cleanup  https://review.openstack.org/139643
09:58 boris-42 wtakase: did you see mailing list regarding to deprecation of EC2?
09:59 msdubov psd_, I mean the actual scenario as launched by Rally, not the unit tests :)
09:59 paboldin psd_: i would prefer to use mock_clients.return_value.remove_interface_router ...
09:59 paboldin psd_: because this way you can mock_clients.assert_called_once_with...
10:00 psd msdubov, I did not follow
10:01 wtakase boris-42: is it around beginning of this month?
10:02 msdubov psd I mean just making a benchmark task with your new scenario, then running "rally task start ..." and checking whether the scenario works fine
10:02 msdubov psd Anyway, gates do the same
10:03 psd msdubov, oh no, that I did not do
10:04 boris-42 wtakase: yep
10:04 rvasilets___ joined #openstack-rally
10:04 boris-42 wtakase: and there is another project
10:04 boris-42 wtakase: https://github.com/stackforge/ec2-api
10:04 boris-42 wtakase: that will add ec2-api as a service
10:05 boris-42 wtakase: probably we should align your work with this project as well?
10:07 psd paboldin, could you please elaborate
10:07 psd paboldin, or probably link some example
10:08 paboldin psd: on topic of what?
10:08 psd your above statement, you would prefer to use mock_clients.return_value ...
10:09 wtakase boris-42: yes. I didn't know the project.
10:10 boris-42 wtakase: so as far as I understood from mailing list
10:10 boris-42 wtakase: when that project will be finished it will replace nova ec2 api
10:10 boris-42 wtakase: and as I understood the "API" will be the same just in another place on another port
10:11 boris-42 wtakase: so your benchmarks could be reused
10:13 wtakase boris-42: what should we do? is it better to have tests for old nova ec2 api and new ec2 api?
10:14 boris-42 wtakase: the best is to have one set that works for both
10:15 boris-42 wtakase: but I think that both will use boto as python client
10:15 openstackgerrit Mikhail Dubov proposed stackforge/rally: Fixes in docs texts and images  https://review.openstack.org/156093
10:16 openstackgerrit Mikhail Dubov proposed stackforge/rally: Fix texts and images in docs  https://review.openstack.org/156093
10:20 boris-42 wtakase: so I think I will make ec2-api job
10:20 e0ne joined #openstack-rally
10:20 boris-42 wtakase: so we will be able to test it at least
10:22 wtakase boris-42: you will setup ec2-api environment in jenkins test, right?
10:24 boris-42 wtakase: yep in exeprimental queue
10:24 boris-42 wtakase btw could you split patch
10:24 boris-42 wtakase: it's really too big to make a good review
10:25 boris-42 wtakase: > 500 LOC becomes really hard task for core reviewers
10:26 paboldin psd: well, when you call a mock the call gets recorded. so sometimes you can't intermix assertions
10:26 wtakase boris-42: yes, I will split it to small patches.
10:26 paboldin psd: like if you do mock_osclients("...").assert_called_once_with, you can't then call mock_osclients.assert_called_once_with...
10:26 boris-42 wtakase: thank you
10:28 wtakase boris-42: and I will have a look at the api project and consider how to align my work with the project.
10:29 wtakase boris-42: I think it will be another patch.
10:30 boris-42 wtakase: that will be actually great
10:30 boris-42 wtakase: because it will help ec2-api team to finish their job faster
10:30 boris-42 wtakase: and add better support for ec2 in openstack
10:31 psd paboldin, okay, thanks
10:35 wtakase boris-42: yep, thanks for chatting!
10:36 tfreger joined #openstack-rally
10:38 boris-42 wtakase: btw
10:38 boris-42 wtakase: don't miss this stuff http://boris-42.me/rally-tricks-stop-load-before-your-openstack-goes-wrong/ =)
10:38 psd_ joined #openstack-rally
10:40 wtakase boris-42: thanks!!
10:42 openstackgerrit Sergey Skripnick proposed stackforge/rally: Test "Add floating-IP aware VM boot/delete utils" vs mos do-not-merge  https://review.openstack.org/155713
10:42 openstackgerrit Sergey Skripnick proposed stackforge/rally: Add new script for gate jobs  https://review.openstack.org/144633
10:53 exploreshaifali joined #openstack-rally
11:06 openstackgerrit Merged stackforge/rally: Fix rally task detailed --iterations-data  https://review.openstack.org/155114
11:27 nmagnezi joined #openstack-rally
11:46 openstackgerrit Sergey Skripnick proposed stackforge/rally: Test "Add floating-IP aware VM boot/delete utils" vs mos do-not-merge  https://review.openstack.org/155713
11:46 openstackgerrit Sergey Skripnick proposed stackforge/rally: Add new script for gate jobs  https://review.openstack.org/144633
11:55 psd joined #openstack-rally
11:55 psd msdubov, could you please help me resolve https://review.openstack.org/#/c/113177/10/tests/unit/benchmark/scenarios/neutron/test_utils.py when you find time :)
11:57 e0ne joined #openstack-rally
11:59 andreykurilin_ joined #openstack-rally
12:08 psd_ joined #openstack-rally
12:25 nkhare joined #openstack-rally
12:26 andreykurilin_ joined #openstack-rally
12:42 psd joined #openstack-rally
12:57 cdent joined #openstack-rally
12:59 openstackgerrit Sergey Skripnick proposed stackforge/rally: Test "Add floating-IP aware VM boot/delete utils" vs mos do-not-merge  https://review.openstack.org/155713
12:59 openstackgerrit Sergey Skripnick proposed stackforge/rally: Add new script for gate jobs  https://review.openstack.org/144633
13:16 e0ne joined #openstack-rally
13:29 aix joined #openstack-rally
13:34 msdubov psd, Hmm, I've tried to remove that with-statement that we discussed today and it works
13:35 msdubov psd, So seems like it is not correct to use "with" and mocks in this way
13:35 msdubov psd, Let's see how we can split that big line so that pep does not fail
13:38 msdubov psd, I've left a comment in your patch. Please take a look
13:50 e0ne joined #openstack-rally
13:56 openstackgerrit Wataru Takase proposed stackforge/rally: Add EC2 base for EC2 API benchmarks  https://review.openstack.org/147550
13:56 andreykurilin_ joined #openstack-rally
13:56 paboldin msdubov: can you please point me to the 'with' you are discussing?
14:00 msdubov paboldin, https://review.openstack.org/#/c/113177/10/tests/unit/benchmark/scenarios/neutron/test_utils.py, line 263
14:01 paboldin msdubov: well, this is simply wrong. what this code does is calling __enter__ on the remove_interface_router
14:02 msdubov paboldin, Yes, now I get it, but at first I thought mocks support this kind of stuff
14:03 paboldin msdubov: yes, it supports, but the syntax is different
14:03 paboldin msdubov: https://docs.python.org/3/library/unittest.mock.html#unittest.mock.patch
14:04 msdubov paboldin, Yep, thanks
14:13 psd_ joined #openstack-rally
14:24 openstackgerrit Wataru Takase proposed stackforge/rally: Add EC2 context for EC2 API benchmarks  https://review.openstack.org/156228
14:57 cdent_ joined #openstack-rally
15:40 msdubov joined #openstack-rally
15:43 exploreshaifali joined #openstack-rally
15:54 abhirc joined #openstack-rally
16:01 openstackgerrit Wataru Takase proposed stackforge/rally: Add EC2 API benchmarks  https://review.openstack.org/156259
16:07 anshul joined #openstack-rally
16:24 openstackgerrit Sergey Skripnick proposed stackforge/rally: Test "Add floating-IP aware VM boot/delete utils" vs mos do-not-merge  https://review.openstack.org/155713
16:24 openstackgerrit Sergey Skripnick proposed stackforge/rally: Add new script for gate jobs  https://review.openstack.org/144633
16:37 msdubov joined #openstack-rally
16:38 e0ne joined #openstack-rally
16:46 psd__ joined #openstack-rally
16:47 openstackgerrit Pavel Boldin proposed stackforge/rally: Add FIO disk benchmark for VMs  https://review.openstack.org/152388
16:47 openstackgerrit Pavel Boldin proposed stackforge/rally: Add the generic script-based benchmark for the VMs  https://review.openstack.org/141671
16:47 openstackgerrit Pavel Boldin proposed stackforge/rally: Add floating-IP aware VM boot/delete utils  https://review.openstack.org/151384
16:47 openstackgerrit Pavel Boldin proposed stackforge/rally: Add the context benchmark_image  https://review.openstack.org/138466
16:47 openstackgerrit Pavel Boldin proposed stackforge/rally: Prefixing private atomic actions with `_`  https://review.openstack.org/156275
16:47 paboldin ^^^
16:54 yfried joined #openstack-rally
17:14 openstackgerrit Wataru Takase proposed stackforge/rally: Add name pattern filter for resource cleanup  https://review.openstack.org/139643
17:17 amaretskiy joined #openstack-rally
17:31 anshul joined #openstack-rally
17:31 exploreshaifali joined #openstack-rally
17:31 tellesnobrega joined #openstack-rally
17:31 mkoderer joined #openstack-rally
17:31 redixin joined #openstack-rally
17:31 gugl2 joined #openstack-rally
17:39 psd_ joined #openstack-rally
17:43 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Add create and delete routers scenarios for Neutron  https://review.openstack.org/113177
17:47 tfreger joined #openstack-rally
17:51 openstackgerrit Alexander Maretskiy proposed stackforge/rally: (WIP) Add chunks support and chart `Parallel iterations' to report  https://review.openstack.org/146814
17:55 rook joined #openstack-rally
18:06 psd__ joined #openstack-rally
18:39 exploreshaifali joined #openstack-rally
18:40 rook joined #openstack-rally
18:46 andreykurilin_ joined #openstack-rally
18:56 e0ne joined #openstack-rally
19:15 psd_ joined #openstack-rally
19:17 exploreshaifali_ joined #openstack-rally
19:29 yfried joined #openstack-rally
19:32 exploreshaifali joined #openstack-rally
19:34 rook joined #openstack-rally
19:44 openstackgerrit Merged stackforge/rally: Prefixing private atomic actions with `_`  https://review.openstack.org/156275
19:51 abhirc joined #openstack-rally
20:03 rook joined #openstack-rally
20:21 andreykurilin_ joined #openstack-rally
20:31 andreykurilin_ joined #openstack-rally
21:48 himangi joined #openstack-rally
22:02 aix joined #openstack-rally
22:22 abhirc joined #openstack-rally
22:35 albertom joined #openstack-rally
22:48 openstackgerrit Sergey Skripnick proposed stackforge/rally: Add new script for gate jobs  https://review.openstack.org/144633
22:56 boris-42 redixin: hey there
22:56 boris-42 redixin: there is no need to this recheck bug 1419723
22:56 boris-42 redixin: it doesn't matter
23:50 himangi joined #openstack-rally

| Channels | #openstack-rally index | Today | | Search | Google Search | Plain-Text | summary