Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2016-01-07

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

All times shown according to UTC.

Time Nick Message
00:11 itlinux joined #openstack-rally
00:11 jjmb joined #openstack-rally
00:32 yingjun joined #openstack-rally
01:18 MaxPC joined #openstack-rally
01:27 openstackgerrit Merged openstack/rally: Fix hacking checks  https://review.openstack.org/264322
01:29 jjmb joined #openstack-rally
01:30 jjmb joined #openstack-rally
02:36 harshs joined #openstack-rally
02:48 ilbot3 joined #openstack-rally
02:48 Topic for #openstack-rally is now ☁ Rally RoadMap: http://goo.gl/JZkmwY ☁ Review Dashboard: http://goo.gl/YuCJfl ☁ IRC logs http://irclog.perlgeek.de/openstack-rally ☁ Key persons to ask:  boris-42, yfried, msdubov, rediskin, andreykurilin, amaretskiy  ☁ Documentation: https://rally.readthedocs.org/en/latest/ ☁ To publish changes to Rally:  https://rally.readthedocs.org/en/latest/contribute.html
03:02 boris-42 joined #openstack-rally
03:47 Aish joined #openstack-rally
03:48 Aish left #openstack-rally
04:00 yingjun joined #openstack-rally
04:03 baker joined #openstack-rally
04:08 baker joined #openstack-rally
04:10 baker_ joined #openstack-rally
04:42 harshs joined #openstack-rally
04:54 baker joined #openstack-rally
05:14 kevinbenton joined #openstack-rally
05:20 rdas joined #openstack-rally
05:22 DinaBelova joined #openstack-rally
05:23 bkopilov joined #openstack-rally
05:26 shausy joined #openstack-rally
05:30 shausy joined #openstack-rally
05:50 ahirshbeg_ joined #openstack-rally
06:05 yingjun joined #openstack-rally
06:07 psuriset joined #openstack-rally
06:19 Aish joined #openstack-rally
06:19 Aish left #openstack-rally
06:29 openstackgerrit LiuNanke proposed openstack/rally: Remove openstack-common.conf  https://review.openstack.org/264597
06:30 neeti joined #openstack-rally
06:35 tfreger joined #openstack-rally
06:47 psuriset joined #openstack-rally
06:55 itsuugo joined #openstack-rally
06:55 openstackgerrit Ravikumar Venkatesan proposed openstack/rally: Adds Designate record-update test  https://review.openstack.org/262390
07:29 bochi-michael joined #openstack-rally
07:33 ahirshbeg_ joined #openstack-rally
07:37 ahirshbeg_ left #openstack-rally
07:44 anshul joined #openstack-rally
07:49 yfried_ joined #openstack-rally
08:48 mkoderer_cloud joined #openstack-rally
08:48 arxcruz joined #openstack-rally
08:54 lezbar__ joined #openstack-rally
09:00 yfried joined #openstack-rally
09:04 psd_ joined #openstack-rally
09:06 shausy joined #openstack-rally
09:34 yingjun_ joined #openstack-rally
09:38 openstackgerrit Yatin Kumbhare proposed openstack/rally: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/264100
09:39 boris-42 andreykurilin: ^ hm why hacking check didn't catch this hmmm
09:48 openstackgerrit Bo Chi proposed openstack/rally: Let TempestResourcesContext implements utils.RandomNameGeneratorMixin  https://review.openstack.org/264004
10:26 tosky joined #openstack-rally
10:33 openstackgerrit Boris Pavlovic proposed openstack/rally: [WIP] Integrate Rally & Alembic  https://review.openstack.org/264701
10:57 openstackgerrit Boris Pavlovic proposed openstack/rally: [Reports] Remove class OutputStackedAreaDeprecatedChart  https://review.openstack.org/257990
10:57 openstackgerrit Boris Pavlovic proposed openstack/rally: [Reports] Improve HTML report and add dummy_output scenario  https://review.openstack.org/257886
10:57 openstackgerrit Boris Pavlovic proposed openstack/rally: [Scenario] Add method `Scenario.add_output'  https://review.openstack.org/255336
10:57 openstackgerrit Boris Pavlovic proposed openstack/rally: [Reports] Make HistogramChart.render() suitable for JS  https://review.openstack.org/261542
10:57 ekarlso boris-42: u there ? :p
10:57 boris-42 ekarlso: no=)
10:58 boris-42 ekarlso: actually going to sleep
10:58 ekarlso boris-42: quicky
10:58 boris-42 ekarlso: it's 3.a.m
10:58 boris-42 ekarlso: yep
10:58 boris-42 ekarlso: so?
10:58 ekarlso http://logs.openstack.org/14/249314/7/check/gate-rally-python27/72049b7/console.html < why is that failing at missing resource cleanup ?
10:59 boris-42 ekarlso: yep it just says
11:00 boris-42 ekarlso: that there is no designate cleanup resource managers
11:00 boris-42 ekarlso: https://github.com/openstack/rally/blob/master/rally/plugins/openstack/context/cleanup/resources.py#L468-L495 you see
11:01 boris-42 ekarlso: so you can use designate_1 or designate_2
11:01 boris-42 ekarlso: depending on what you are doing
11:01 boris-42 ekarlso: however If I were taking look at that patch I won't merge it
11:01 ekarlso boris-42: rly ?
11:01 boris-42 ekarlso: because it's much better to have designate resource name
11:02 boris-42 ekarlso: and do the magic inside cleanup resource manager to deal with versions
11:02 ekarlso boris-42: so how do I fix that ?
11:02 boris-42 ekarlso: I already told you
11:02 boris-42 ekarlso: put proper names of managers
11:02 boris-42 ekarlso: designate_v1 and designate_v2 in default context for scenarios
11:03 boris-42 ekarlso: or refactor back desigante_v1 and desigante_v2 to desigante
11:03 ekarlso boris-42: sigh :p
11:05 boris-42 ekarlso: ya some love is required to that piece of code
11:06 ekarlso boris-42: what you suggest ?
11:08 boris-42 ekarlso: rename to "desinate" back for all common resources between version 1 and version 2
11:09 boris-42 ekarlso: and make some logic inside resource manager to make them work with both versions
11:09 boris-42 after that use "designate" + designate_v1 or desiangete_v2 if you have scenarios for specific version
11:09 boris-42 that's all
11:13 boris-42 ekarlso: going to sleep see you tomorrow
11:14 yingjun joined #openstack-rally
12:15 openstackgerrit Bo Chi proposed openstack/rally: Let TempestResourcesContext implements utils.RandomNameGeneratorMixin  https://review.openstack.org/264004
12:36 ekarlso meh, where is the code that does cleanup for designate?
12:53 ekarlso andreykurilin: or rvasilets ?
13:02 openstackgerrit Bo Chi proposed openstack/rally: Let TempestResourcesContext implements utils.RandomNameGeneratorMixin  https://review.openstack.org/264004
13:14 akrzos joined #openstack-rally
13:18 openstackgerrit Endre Karlson proposed openstack/rally: Designate V2 - part 3  https://review.openstack.org/249314
13:19 ekarlso ^ that should be ready for signoff
13:33 jjmb1 joined #openstack-rally
13:35 harshs joined #openstack-rally
13:39 MaxPC joined #openstack-rally
13:49 yingjun joined #openstack-rally
13:50 stpierre joined #openstack-rally
13:55 stpierre anyone know why the meeting log archive isn't being updated? the last logs are from several weeks ago: http://eavesdrop.openstack.org/meetings/rally/2015/
13:56 neeti joined #openstack-rally
13:57 yingjun joined #openstack-rally
14:04 shausy joined #openstack-rally
14:07 psd_ joined #openstack-rally
14:19 anshul joined #openstack-rally
14:29 yingjun joined #openstack-rally
14:36 yingjun joined #openstack-rally
15:06 bkopilov joined #openstack-rally
15:10 diogogmt joined #openstack-rally
15:37 openstackgerrit Tom Patzig proposed openstack/rally: Additional argument support for CinderVolumes  https://review.openstack.org/253659
15:50 itlinux joined #openstack-rally
15:52 rook joined #openstack-rally
15:54 yfried joined #openstack-rally
16:07 itlinux joined #openstack-rally
16:08 lezbar joined #openstack-rally
16:12 shausy joined #openstack-rally
16:19 diogogmt joined #openstack-rally
16:22 diogogmt_ joined #openstack-rally
16:25 itlinux joined #openstack-rally
16:26 baker joined #openstack-rally
16:28 harshs joined #openstack-rally
16:56 Murali joined #openstack-rally
16:57 itlinux joined #openstack-rally
16:59 madhu_ak joined #openstack-rally
17:11 itlinux joined #openstack-rally
17:12 lezbar joined #openstack-rally
17:14 shausy joined #openstack-rally
17:15 davideagnello joined #openstack-rally
17:33 Aish joined #openstack-rally
17:45 itlinux_ joined #openstack-rally
17:53 shausy joined #openstack-rally
18:12 ilbot3 joined #openstack-rally
18:12 Topic for #openstack-rally is now ☁ Rally RoadMap: http://goo.gl/JZkmwY ☁ Review Dashboard: http://goo.gl/YuCJfl ☁ IRC logs http://irclog.perlgeek.de/openstack-rally ☁ Key persons to ask:  boris-42, yfried, msdubov, rediskin, andreykurilin, amaretskiy  ☁ Documentation: https://rally.readthedocs.org/en/latest/ ☁ To publish changes to Rally:  https://rally.readthedocs.org/en/latest/contribute.html
18:14 baker joined #openstack-rally
18:21 rook-desktio joined #openstack-rally
18:23 e0ne joined #openstack-rally
18:30 itlinux joined #openstack-rally
18:32 MaxPC1 joined #openstack-rally
18:36 harlowja joined #openstack-rally
18:50 pvaneck joined #openstack-rally
18:54 itlinux joined #openstack-rally
19:05 jjmb joined #openstack-rally
19:06 baker_ joined #openstack-rally
19:13 itlinux_ joined #openstack-rally
19:25 itlinux joined #openstack-rally
19:29 harshs joined #openstack-rally
19:30 Aish_ joined #openstack-rally
19:38 itlinux joined #openstack-rally
19:42 kbyrne joined #openstack-rally
19:46 harshs joined #openstack-rally
20:03 itlinux joined #openstack-rally
20:05 MaxPC joined #openstack-rally
20:09 MaxPC joined #openstack-rally
20:13 MaxPC1 joined #openstack-rally
20:18 boris-42 stpierre: hi there
20:18 boris-42 stpierre: you should use this link lol http://eavesdrop.openstack.org/meetings/rally/2016/
20:18 boris-42 stpierre: it's 2016 =)
20:20 vkmc is it already?
20:20 vkmc when did this happen?
20:23 stpierre there was at least one more meeting in 2015 that's not there
20:23 stpierre from the 21st
20:23 stpierre i don't know if there was one on the 28th or not
20:24 openstackgerrit joined #openstack-rally
20:31 boris-42 stpierre: I am not sure
20:31 boris-42 stpierre: I believe we skip few meetings.
20:31 boris-42 stpierre: I know that we skip on 28th
20:31 stpierre you told me there was a meeting on the 21st
20:32 boris-42 vkmc: 2016 year ?)
20:32 boris-42 stpierre: hm oh
20:32 vkmc boris-42, yes, I'm kidding :)
20:32 stpierre but i don't see it, even in the raw logs at http://eavesdrop.openstack.org/irclogs/%23openstack-meeting/%23openstack-meeting.2015-12-21.log.html
20:33 boris-42 stpierre: ya seems like it's missing
20:34 boris-42 stpierre: ah I rmember there were some problems with bot those days
20:34 stpierre i was trying to find out what was ever decided about wait_for()/wait_for_status(), and creating (or retaining) a more generic wait routine, since wait_for_status() is pretty seriously tied to openstack objects
20:34 stpierre ah
20:34 stpierre ok
20:34 boris-42 stpierre: I remember that
20:34 boris-42 stpierre: so the thing is that no matter what you are using wait_for() or wait_for_status() you need to pass object to it
20:34 stpierre yeah, the raw logs skip from 9:37 to 15:35
20:35 boris-42 stpierre: having 10 methods wait_for_* is really bad idea
20:35 stpierre i agree
20:35 boris-42 stpierre: wait_for_status is the most new and the best of those methods
20:35 stpierre but it leads to hacks like https://review.openstack.org/#/c/253406/
20:35 boris-42 stpierre: nope
20:35 stpierre nope?
20:35 boris-42 stpierre: this hacks are required for wait_for as well actually
20:36 boris-42 stpierre: there is no difference really no difference
20:36 boris-42 both works in the same way with updates
20:36 stpierre so can we create a new function that doesn't require tons of hacks to wait for something that isn't an openstack object with a status?
20:37 boris-42 stpierre: we were trying LOL for 2.5 years LOL
20:37 stpierre i'll take that as a "yes" :)
20:37 boris-42 stpierre: let's do it in another way
20:37 boris-42 stpierre: we will move everything to wait_for_status
20:38 boris-42 stpierre: remove all wait_for_* methods
20:38 boris-42 and after that we can refactor wait_for_status or make something else
20:38 boris-42 and move to it
20:38 stpierre ok, that sounds reasonable
20:38 stpierre since 99% of our stuff will just use wait_for_status() sanely, there'd be a pretty minimal amount of refactoring to do later
20:52 itlinux joined #openstack-rally
20:56 boris-42 stpierre: yep yep
20:56 boris-42 stpierre: it simpler to refactor one method instead of refactoring bunch of different stuff
20:57 boris-42 stpierre: actually I had to refactor all wait_for methods and I end up with the idea to use everywhere single method
20:57 boris-42 as well it will reduce possibility of mistakes and it will simplify overall code
21:01 openstackgerrit Chris St. Pierre proposed openstack/rally: Add wrapper for Glance API versions  https://review.openstack.org/264964
21:05 boris-42 stpierre: btw wrappers
21:05 boris-42 stpierre: are bad approach=-((
21:05 stpierre hah
21:05 stpierre the only good approach to glance v2 is to never use it, so what's a less-bad approach?
21:06 boris-42 stpierre: ^ lol
21:06 boris-42 stpierre: sooooooo
21:06 boris-42 stpierre: I mean overall wrappers are bad idea
21:06 boris-42 stpierre: we have some spec that is not finisehd
21:07 MaxPC joined #openstack-rally
21:07 boris-42 stpierre: this one actually https://review.openstack.org/#/c/172831/
21:08 stpierre that spec is so old it can practically drink
21:09 boris-42 stpierre: LOL
21:09 stpierre my preference would be to support glance v2 now, and then someone get to work on updating that spec for the 21st century afterward.
21:10 itlinux joined #openstack-rally
21:11 MaxPC1 joined #openstack-rally
21:11 stpierre since it's a *huge* spec, and it doesn't make sense to me to block supporting other API versions until it's further underway
21:11 boris-42 stpierre: so supporting more and more API in wrong way will force us to rewrite more and more code
21:11 stpierre that's true. hopefully it'll make that spec more of a priority
21:11 boris-42 stpierre: so my plan is next
21:11 stpierre but that spec is going to take ages to implement
21:11 boris-42 stpierre: we can work hard on that spec
21:11 boris-42 stpierre: and use new approach at least for new code
21:11 boris-42 stpierre: meanwhile someone will refactor old wrappers
21:11 boris-42 stpierre: so at least we won't add more and more code that requires full rewriting
21:11 stpierre if it's not going to block us on adding Glance v2 or other API version support for very long, it sounds fine to me
21:12 boris-42 stpierre: ya that spec needs some love
21:13 boris-42 stpierre: otherwise it's good
21:13 boris-42 stpierre: btw I am working on adding alembic support to Rally
21:13 boris-42 stpierre: so we will be able to change DB without pain
21:13 boris-42 with less pain*
21:14 stpierre glance v2 support is becoming important to us, and it's tough to maintain locally since the glance utils need to know about it, so i really really want to get this upstream asap. i'll see if i can work on that spec a little, but i can't take on all of the work that's entailed in it
21:14 MaxPC joined #openstack-rally
21:17 boris-42 stpierre: I will work druing this weekend on spec
21:17 openstackgerrit Merged openstack/rally: Let TempestResourcesContext implements utils.RandomNameGeneratorMixin  https://review.openstack.org/264004
21:17 stpierre awesome
21:18 MaxPC1 joined #openstack-rally
21:21 openstackgerrit Boris Pavlovic proposed openstack/rally: [Reports] Remove class OutputStackedAreaDeprecatedChart  https://review.openstack.org/257990
21:21 openstackgerrit Boris Pavlovic proposed openstack/rally: [Reports] Improve HTML report and add dummy_output scenario  https://review.openstack.org/257886
21:21 boris-42 stpierre: ^ btw could you take a look at new HTML output patches
21:21 boris-42 stpierre: I mean scenario output
21:22 stpierre i took a look at a few of them yesterday and tbh they're definitely over my head -- i read through the spec and the overall goal makes sense but i get pretty lost as soon as i get into the code
21:24 harshs joined #openstack-rally
21:25 boris-42 stpierre: =)
21:25 boris-42 stpierre: you catch yesterday good thing
21:32 Aish joined #openstack-rally
21:51 stpierre joined #openstack-rally
21:51 DinaBelova joined #openstack-rally
21:51 boris-42 joined #openstack-rally
21:55 openstackgerrit Merged openstack/rally: Updated from global requirements  https://review.openstack.org/264465
21:57 itlinux joined #openstack-rally
22:09 MaxPC joined #openstack-rally
22:10 stpierre joined #openstack-rally
22:25 openstackgerrit Chris St. Pierre proposed openstack/rally: Create new types.convert()  https://review.openstack.org/249462
22:26 openstackgerrit Chris St. Pierre proposed openstack/rally: Convert ResourceType classes to plugins  https://review.openstack.org/249463
22:26 openstackgerrit Chris St. Pierre proposed openstack/rally: Convert existing scenarios to new types plugins  https://review.openstack.org/249464
22:31 itlinux joined #openstack-rally
22:36 itlinux joined #openstack-rally
22:52 harshs joined #openstack-rally
22:53 openstackgerrit Merged openstack/rally: [Scenario] Add method `Scenario.add_output'  https://review.openstack.org/255336
22:53 openstackgerrit Merged openstack/rally: Remove openstack-common.conf  https://review.openstack.org/264597
23:35 boris-42 zeus: so bug fix for tempest was merged
23:36 boris-42 zeus:  tomorrow we will cut 0.2.0 that should work for u
23:52 openstackgerrit Merged openstack/rally: [VM] refactor wait_for_ping  https://review.openstack.org/253406

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