Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2015-06-29

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

All times shown according to UTC.

Time Nick Message
00:38 PrashantS joined #openstack-rally
00:47 dmorita joined #openstack-rally
02:09 openstackgerrit Boris Pavlovic proposed openstack/rally: Optimize Rally imports to reduce start time  https://review.openstack.org/196493
02:32 openstackgerrit Boris Pavlovic proposed openstack/rally: Optimize Rally imports to reduce start time  https://review.openstack.org/196493
02:43 mwagner_afk joined #openstack-rally
03:51 davideagnello joined #openstack-rally
04:23 kiran-r joined #openstack-rally
04:41 tfreger joined #openstack-rally
04:43 yfried joined #openstack-rally
04:45 nihilifer joined #openstack-rally
04:53 kiran-r kun_huang: Hi!
04:57 davideagnello joined #openstack-rally
05:03 davideagnello joined #openstack-rally
05:04 klindgren_ joined #openstack-rally
05:07 rdas joined #openstack-rally
05:32 yfried joined #openstack-rally
05:35 yfried_ joined #openstack-rally
06:10 agarciam joined #openstack-rally
06:34 adiantum joined #openstack-rally
06:52 davideagnello joined #openstack-rally
06:54 anshul joined #openstack-rally
06:55 rdas joined #openstack-rally
07:06 anshul joined #openstack-rally
07:07 nkhare joined #openstack-rally
07:08 exploreshaifali joined #openstack-rally
07:14 fhubik joined #openstack-rally
07:25 arxcruz joined #openstack-rally
07:33 albertw1 joined #openstack-rally
07:41 cdent joined #openstack-rally
07:42 kiran-r joined #openstack-rally
07:44 neeti joined #openstack-rally
07:47 kun_huang kiran-r: hi
07:47 kun_huang kiran-r: I'm on bus this morning
07:47 kun_huang got your request now
07:49 kiran-r kun_huang: Please check the 2 patches, dependent to each other
07:50 kun_huang these two are ready? https://review.openstack.org/#/c/193432/16 https://review.openstack.org/#/c/193527/ ?
07:53 dmorita joined #openstack-rally
07:54 kun_huang kiran-r:
07:54 kun_huang ping
07:54 kiran-r kun_huang: Yes
07:54 kun_huang have test this line? supports_extension("lbaas")
07:54 kun_huang I remember that lbaas is an agent in neutron not extension
07:55 openstackgerrit Kiran proposed openstack/rally: Add ability to create pool in Neutron context  https://review.openstack.org/193432
08:05 kiran-r kun_huang: It also shows up in neutron ext-list http://paste.openstack.org/show/324508/
08:08 fhubik joined #openstack-rally
08:09 fhubik_afk joined #openstack-rally
08:28 kiran-r yfried_ : ping
08:41 kun_huang kiran-r: we are now facing new upstream issue: fixtures
08:41 davideagnello joined #openstack-rally
08:41 kun_huang kiran-r: it is uploaded, TODAY
08:41 kiran-r kun_huang: Yes, like this one http://logs.openstack.org/32/193432/17/check/gate-rally-python26/5ec974d/console.html
08:41 kun_huang at least rally's unit tests are blocked
08:42 kun_huang let's see response of other community first
08:53 kbyrne joined #openstack-rally
08:54 kir_ joined #openstack-rally
08:55 kir_ redixin: https://review.openstack.org/#/c/188847/ still has hard time with mirantis CI
09:01 kun_huang ping kiran-r
09:02 kiran-r kun_huang: Yes
09:02 kun_huang find a workaround
09:03 kun_huang but I'm but sure...
09:03 kiran-r kun_huang: it is not a problem in the rally code.
09:04 kun_huang maybe
09:04 kiran-r yfried_: Please look into this ^
09:04 kun_huang I need go home and have a test
09:04 kun_huang there is no openstack cloud on hand
09:08 tab___ joined #openstack-rally
09:17 kun_huang https://github.com/openstack/rally/blob/master/tests/unit/plugins/openstack/scenarios/glance/test_utils.py#L50
09:17 kun_huang https://github.com/openstack/rally/blob/master/tests/unit/plugins/openstack/scenarios/nova/test_utils.py#L54
09:19 kiran-r kun_huang I filed a bug on this https://bugs.launchpad.net/rally/+bug/1469644
09:19 openstack Launchpad bug 1469644 in Rally "Rally jenkins fails due to fixtures 1.3.0" [Undecided,New]
09:19 tosky joined #openstack-rally
09:22 yfried_ kiran-r: look into what?
09:24 fhubik_afk joined #openstack-rally
09:24 fhubik joined #openstack-rally
09:25 fhubik joined #openstack-rally
09:26 kiran-r yfried_ : Jenkins is failing with fixtures 1.3.0
09:27 kbaikov joined #openstack-rally
09:53 aix joined #openstack-rally
09:53 openstackgerrit joined #openstack-rally
10:01 karimb joined #openstack-rally
10:03 uschreiber_ joined #openstack-rally
10:30 davideagnello joined #openstack-rally
10:38 openstackgerrit Kiran proposed openstack/rally: Add the right version of fixture in requirements  https://review.openstack.org/196623
10:42 kiran-r kun_huang ^
10:42 kiran-r kun_huang ping
10:49 tosky_ joined #openstack-rally
10:49 fhubik_lunch joined #openstack-rally
10:49 acruz joined #openstack-rally
10:52 tosky__ joined #openstack-rally
10:52 karmatronic joined #openstack-rally
10:53 karmatronic joined #openstack-rally
10:54 fhubik_meeting joined #openstack-rally
10:57 openstackgerrit nayna-patel proposed openstack/rally: Add context for Designate Domains  https://review.openstack.org/195039
11:07 acruz joined #openstack-rally
12:11 MaxPC1 joined #openstack-rally
12:18 davideagnello joined #openstack-rally
12:25 yfried_ joined #openstack-rally
12:52 exploreshaifali joined #openstack-rally
13:04 yfried|afk joined #openstack-rally
13:05 yfried|afk joined #openstack-rally
13:06 yfried|afk andreykurilin: mtg today?
13:18 rvasilets_ joined #openstack-rally
13:19 openstackgerrit Albert White proposed openstack/rally: find should use portable options  https://review.openstack.org/196086
13:22 oanufriev joined #openstack-rally
13:45 andreykurilin__ joined #openstack-rally
13:50 kun_huang kir_: pong
13:57 stpierre joined #openstack-rally
13:59 boris-42 joined #openstack-rally
13:59 karimb joined #openstack-rally
14:01 albertw joined #openstack-rally
14:02 boris-42 yfried|afk: ping
14:02 boris-42 kun_huang: ping
14:02 kun_huang boris-42: pong
14:02 boris-42 kun_huang: it's rally meeting time=)
14:02 boris-42 meteorfox: ping
14:02 boris-42 stpierre: ping
14:07 davideagnello joined #openstack-rally
14:25 kir_ kun_huang: ping
14:26 kun_huang kir_: coming to #openstack-meeting
14:26 kun_huang if having any trouble, you could ask for help there
14:29 kir_ open #openstack-meeting
14:29 kir_ joined #openstack-rally
14:38 gubouvier joined #openstack-rally
14:42 nihilifer joined #openstack-rally
15:00 boris-42 kir_: hi
15:01 kiran-r joined #openstack-rally
15:01 kiran-r boris-42: ping
15:01 kiran-r boris-42: Hi! :)
15:02 taturn joined #openstack-rally
15:02 taturn hi
15:02 andreykurilin__ taturn: hi there_
15:04 taturn cool
15:07 stpierre boris-42: i know the meeting's over, but i feel like we might be able to get a middle ground if our supported release policy a) guaranteed support on all upstream supported releases; b) discouraged breaking compatibility with unsupported releases without reason; and c) noted that unsupported releases do not undergo CI testing, so may have issues. that still means that we're testing four releases in gate (which will raise plenty of issues
15:07 stpierre as i'm sure you know), but our scope won't grow without bound, and we won't go out of our way to break crap.
15:07 boris-42 stpierre: So I am not saying
15:08 boris-42 stpierre: that we should gurantee that Rally will work against Diablo
15:08 boris-42 stpierre: I am saying that it will be nice if it will work against it
15:08 stpierre yes, i agree
15:08 boris-42 stpierre: or at least part of it
15:08 stpierre but if it doesn't, i don't think that should keep us awake at night
15:08 boris-42 stpierre: and we should extend our gates to support or stable releases
15:08 stpierre definitely
15:09 boris-42 stpierre: yep true but if somebody is proposing fixes we should review / merge
15:10 stpierre i'm not sure i agree with that universally. our willingness to merge should consider how much code complexity it adds. IMO it's sort of like albertw's find patch. that's a fine patch because it changes one line in a trivial way, and makes install_rally.sh better. a patch to make all of install_rally.sh work on old broken-ass Solaris 8 would probably be something terrible like -200, +600, and i think we should reject the hell out it.
15:10 stpierre we have to limit our scope, or go insane. those are the two options.
15:11 stpierre so if you have a minor tweak to a scenario and it works on grizzly, hey, great. but if you need to add a ton of abstraction in order to support a substantially different older API, then that's not so great. that reduces long-term code maintainability, and it's a fundamentally untestable change
15:12 kir_ boris-42: hi
15:22 taturn left #openstack-rally
15:23 e0ne joined #openstack-rally
15:29 psd joined #openstack-rally
15:31 davideagnello joined #openstack-rally
15:38 kiran-r joined #openstack-rally
15:44 PrashantS joined #openstack-rally
15:45 PrashantS joined #openstack-rally
15:48 e0ne joined #openstack-rally
15:50 boris-42 stpierre: heh
15:50 boris-42 stpierre: agree
15:53 davideagnello joined #openstack-rally
15:54 albertw1 joined #openstack-rally
16:03 kir_ boris-42: plz review https://review.openstack.org/#/c/188847/ - I do understand that without cobbler server it's untestable, so not sure how to proceed
16:07 kiran-r joined #openstack-rally
16:07 openstackgerrit Pavel Boldin proposed openstack/rally: Add command-dict option to specify command args  https://review.openstack.org/177016
16:07 openstackgerrit Pavel Boldin proposed openstack/rally: Add command_customizer context  https://review.openstack.org/183227
16:07 openstackgerrit Pavel Boldin proposed openstack/rally: Add boot_runcommand_delete_custom_image scenario  https://review.openstack.org/183228
16:07 openstackgerrit Pavel Boldin proposed openstack/rally: Make `boot_runcommand_delete' accept command-dict  https://review.openstack.org/177014
16:07 openstackgerrit Pavel Boldin proposed openstack/rally: Add command-dict option to upload a local command  https://review.openstack.org/177015
16:07 boris-42 kir_: so we have 2 ways to do the things
16:08 boris-42 kir_: *we* will belive you that it works
16:08 boris-42 kir_: and merge if it looks ok
16:08 boris-42 kir_: or you can help us to build CI for it
16:10 kir_ boris-42: who provides hardware for this CI?
16:11 boris-42 kir_: so we have a bit
16:11 boris-42 kir_: just 2 servers
16:14 kir_ boris-42: it's enough for functional testing
16:17 kir_ boris-42: provided that servers are smtp-rebootable
16:18 kir_ boris-42: c/smpt/snmp
16:25 kiran-r joined #openstack-rally
16:26 boris-42 kir_: we will need to talk with redixin
16:26 boris-42 kir_: unfortuntally today is holiday in Ukraine
16:27 boris-42 kir_: so I will review today your patch and we will ping tomorrow redixin and discuss functional testing of this patch
16:27 kir_ boris-42: ok
16:30 kiran-r joined #openstack-rally
16:35 _kiran_ joined #openstack-rally
16:45 oanufriev joined #openstack-rally
17:07 davideagnello joined #openstack-rally
17:14 temujin_ joined #openstack-rally
17:15 oanufriev joined #openstack-rally
17:16 exploreshaifali joined #openstack-rally
17:18 PrashantS joined #openstack-rally
17:20 openstackgerrit Kun Huang proposed openstack/rally: Fix gate: move test case to avoid latest fixtures  https://review.openstack.org/196746
17:20 temujin joined #openstack-rally
17:22 temujin joined #openstack-rally
17:30 temujin joined #openstack-rally
17:32 dontalton joined #openstack-rally
17:41 kun_huang boris-42: ping
17:43 kun_huang ping andreykurilin
17:49 PrashantS joined #openstack-rally
18:02 mwagner_afk joined #openstack-rally
18:08 exploreshaifali joined #openstack-rally
18:12 andreykurilin__ kun_huang: pong
18:18 kun_huang https://review.openstack.org/#/c/196746/
18:18 kun_huang fixtures fixed andreykurilin__
18:18 kun_huang bug coverage failed
18:18 kun_huang not about coverage itself
18:19 kun_huang andreykurilin__:
18:19 kun_huang coverage is to compare master and HEAD
18:19 kun_huang because of fixtures issue, any coverage test on master failed...
18:19 kun_huang so we need disable coverage first...
18:21 andreykurilin__ We can do it just modifying code in our repo
18:22 temujin joined #openstack-rally
18:23 kun_huang andreykurilin__: push instead of git review?
18:25 andreykurilin__ kun_huang: No, I mean we can modify script tests/ci/coverage.sh and disable coverage check(return success)
18:28 kun_huang andreykurilin__: besides you, is another core reviewer online?
18:30 openstackgerrit joined #openstack-rally
18:36 andreykurilin__ kun_huang: My internet connection is not stable now, so I can be offline in near future :( boris-42 should be online
18:38 kun_huang ok, I'll submit one and wait him
18:43 openstackgerrit Kun Huang proposed openstack/rally: disable coverage test temporarily  https://review.openstack.org/196795
18:47 openstackgerrit Kun Huang proposed openstack/rally: Fix gate: move test case to avoid latest fixtures  https://review.openstack.org/196746
18:48 kun_huang updated
18:48 kun_huang go to ben
18:48 kun_huang bed
18:57 openstackgerrit Chris St. Pierre proposed openstack/rally: Fix gate: move test case to avoid latest fixtures  https://review.openstack.org/196746
18:57 openstackgerrit Chris St. Pierre proposed openstack/rally: Revert "disable coverage test temporarily"  https://review.openstack.org/196799
18:57 stpierre kun_huang: i rebased your changes to be a series so that we merge these in order
18:57 stpierre and added a revert for the coverage test changes so we don't accidentally leave that disabled :)
19:54 temujin joined #openstack-rally
20:20 oanufriev__ joined #openstack-rally
20:20 nihilifer joined #openstack-rally
20:31 boris-42 joined #openstack-rally
20:33 Prashant_ joined #openstack-rally
20:37 boris-42 kun_huang: are you around?
20:39 openstackgerrit Chris St. Pierre proposed openstack/rally: Fix gate: move test case to avoid latest fixtures  https://review.openstack.org/196746
20:39 openstackgerrit Chris St. Pierre proposed openstack/rally: Revert "disable coverage test temporarily"  https://review.openstack.org/196799
20:39 stpierre boris-42: i squashed them together
20:40 stpierre so hopefully now it'll pass gate
20:40 boris-42 stpierre: btw https://review.openstack.org/#/c/196746/
20:41 boris-42 stpierre: take a look at my comment
20:41 stpierre i don't understand the change well enough to know the answer to that
20:41 stpierre i assumed that the cleanUp was necessary because the class included the fixture objects
20:41 stpierre so you could get rid of it precisely because the test was moved into another class
20:41 boris-42 stpierre: not sure.. really not sure..
20:41 stpierre but i guess i can TIAS
20:48 stpierre honestly, i think we can just get rid of these two tests completely. they don't test anything about nova or glance, they test rally.benchmark.utils.get_from_manager()
20:48 stpierre and it has its own unit tests
20:49 echoingumesh joined #openstack-rally
20:50 boris-42 stpierre: +2
20:50 boris-42 stpierre: let's just get rid of them
20:52 openstackgerrit Chris St. Pierre proposed openstack/rally: Remove extraneous failed server/image tests  https://review.openstack.org/196855
20:52 boris-42 stpierre: ^ not sure that this will work without disabling coverage job
20:52 temujin joined #openstack-rally
20:53 boris-42 stpierre: ^ btw it will be nice to add kun as co-author
20:53 stpierre good call
20:53 boris-42 stpierre: he participate as well in fixing this=)
20:54 openstackgerrit Chris St. Pierre proposed openstack/rally: Remove extraneous failed server/image tests  https://review.openstack.org/196855
20:54 stpierre i missed the reasoning behind disabling coverage -- why can't we just add test(s) as necessary to restore the coverage?
20:56 boris-42 stpierre: so coverage will test old code that doesn't work
20:56 boris-42 stpierre: and it will fail
20:56 boris-42 stpierre: if you disable coverage it won't care that old (master) code is failing
20:56 stpierre oooooooooooooooooooh
20:56 stpierre i get it
20:56 boris-42 stpierre: ya it runs 2 times coverage job
20:56 stpierre so the coverage isn't changing, it's just that the coverage job expects tests against master to succeed
20:56 boris-42 stpierre: to understand did you change everything
20:57 boris-42 stpierre: master and master + yourpatch
20:58 openstackgerrit Chris St. Pierre proposed openstack/rally: Restore coverage test functionality  https://review.openstack.org/196799
20:58 openstackgerrit Chris St. Pierre proposed openstack/rally: Remove extraneous failed server/image tests  https://review.openstack.org/196855
21:06 stpierre i would shed no tears if fakes.py went the way of the dodo
21:07 stpierre maybe that'll be the next windmill i tilt at after my scenario fixture change gets merged
21:09 PrashantS joined #openstack-rally
21:26 mwagner_lap joined #openstack-rally
21:33 temujin joined #openstack-rally
21:39 oanufriev__ joined #openstack-rally
22:23 exploreshaifali joined #openstack-rally
22:27 oanufriev__ joined #openstack-rally
22:46 briandowns joined #openstack-rally
23:04 oanufriev__ joined #openstack-rally
23:22 echoingumesh joined #openstack-rally

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