| *** prashantS has joined #openstack-rally | 00:04 | |
| *** prashantS has quit IRC | 00:11 | |
| *** prashantS has joined #openstack-rally | 00:12 | |
| *** prashantS has quit IRC | 00:16 | |
| *** ilbot3 has quit IRC | 01:45 | |
| *** ilbot3 has joined #openstack-rally | 01:48 | |
| *** boris-42 has quit IRC | 02:28 | |
| *** boris-42 has joined #openstack-rally | 02:35 | |
| openstackgerrit | Boris Pavlovic proposed openstack/rally: Print Rally debug logs in Rally dsvm gate jobs https://review.openstack.org/179580 | 02:47 |
|---|---|---|
| *** tfreger has joined #openstack-rally | 04:08 | |
| openstackgerrit | Merged openstack/rally: Print Rally debug logs in Rally dsvm gate jobs https://review.openstack.org/179580 | 05:00 |
| *** boris-42 has quit IRC | 05:38 | |
| *** anshul has joined #openstack-rally | 06:23 | |
| *** yfried has joined #openstack-rally | 06:33 | |
| openstackgerrit | Yair Fried proposed openstack/rally: Add test for deployment check https://review.openstack.org/170342 | 06:35 |
| *** yfried has quit IRC | 06:59 | |
| *** Miouge has joined #openstack-rally | 07:12 | |
| *** exploreshaifali has joined #openstack-rally | 07:23 | |
| *** exploreshaifali has quit IRC | 07:30 | |
| *** anshul has quit IRC | 07:38 | |
| *** boris-42 has joined #openstack-rally | 07:50 | |
| boris-42 | kun_huang: ping | 07:50 |
| openstackgerrit | Merged openstack/rally: Make vmtasks network_wrapper code less wonky https://review.openstack.org/179425 | 08:05 |
| *** subscope has joined #openstack-rally | 08:21 | |
| *** yfried has joined #openstack-rally | 08:43 | |
| boris-42 | yfried: hi hi | 08:50 |
| yfried | b | 08:50 |
| boris-42 | yfried: how are you? | 08:51 |
| yfried | boris-42: can we talk in 90min? | 08:51 |
| boris-42 | yfried: I hope so | 08:51 |
| boris-42 | yfried: take a look here goo.gl/YuCJfl | 08:51 |
| boris-42 | yfried: it's Rally Gerrit dashboard simplifies a lot life | 08:51 |
| *** yfried is now known as yfried|afk | 09:13 | |
| *** anshul has joined #openstack-rally | 09:20 | |
| openstackgerrit | Merged openstack/rally: Add test for deployment check https://review.openstack.org/170342 | 09:21 |
| *** yfried|afk is now known as yfried | 09:33 | |
| *** yfried is now known as yfried|afk | 09:43 | |
| *** subscope has quit IRC | 09:46 | |
| *** yfried|afk is now known as yfried | 09:53 | |
| yfried | boris-42: nice | 09:54 |
| yfried | boris-42: what's "important"? | 09:54 |
| boris-42 | yfried: any patch that is stared by core team | 09:56 |
| boris-42 | yfried: but it is not stared by me + msdubov | 09:57 |
| *** anshul has quit IRC | 09:57 | |
| boris-42 | yfried: then it's critical for next release | 09:57 |
| boris-42 | yfried: I wrote here description http://boris-42.me/rally-review-dashboard/ | 09:57 |
| yfried | boris-42: I'm seeing no patches in "Critical for next release" | 09:58 |
| boris-42 | yfried: https://review.openstack.org/#/c/170491/ | 09:58 |
| boris-42 | yfried: holidays | 09:58 |
| boris-42 | yfried: I can't rich msdubov | 09:58 |
| yfried | boris-42: how do you "star" a patch? | 09:59 |
| yfried | boris-42: oh, saw it | 09:59 |
| boris-42 | yfried: yep | 09:59 |
| yfried | boris-42: I see you also add it to topic | 10:00 |
| yfried | boris-42: btw, I need your help with https://review.openstack.org/#/c/172831/ | 10:01 |
| boris-42 | yfried: sure | 10:01 |
| boris-42 | yfried: I have more global idea | 10:01 |
| boris-42 | yfried: that is related to that work it will be nice if you will work on it as well | 10:01 |
| yfried | boris-42: sure. | 10:01 |
| boris-42 | yfried: so the global idea is to make Rally framework independent from OpenStack | 10:02 |
| boris-42 | yfried: and split it 2 repos | 10:02 |
| boris-42 | yfried: 1 is rally 2 is rally plugins that are only openstack related | 10:02 |
| boris-42 | yfried: it's very very huge task but it worth | 10:02 |
| boris-42 | yfried: one of the steps is to move all plugins under rally/plugins/ dir | 10:03 |
| boris-42 | yfried: e.g. this one http://paste.openstack.org/show/214778/ | 10:04 |
| boris-42 | yfried: so it makes sense to move services directory under rally/plugins/openstack/services direclty | 10:05 |
| boris-42 | yfried: and move whole the stuff to rally/plugins/oopenstack/* | 10:05 |
| yfried | boris-42: will it still be under openstack's namespace? | 10:06 |
| boris-42 | yfried: yep | 10:06 |
| boris-42 | yfried: the goal is to keep everything simple (or simpler) than now | 10:07 |
| boris-42 | yfried: for benchmarking openstack | 10:07 |
| boris-42 | yfried: but with better arch | 10:07 |
| yfried | boris-42: also, I understand that there are many other tools that do benchmarking. if you move away from openstack, won't Rally loose it's main value? | 10:07 |
| boris-42 | yfried: one more time | 10:07 |
| boris-42 | yfried: goal is to keep simplicity regarding to OpenStack | 10:08 |
| boris-42 | yfried: but have better architecture | 10:08 |
| boris-42 | yfried: plus it's very very hard for newbies to understand rally code base | 10:09 |
| boris-42 | yfried: because framework is mixed with plugins | 10:09 |
| yfried | boris-42: ok. I'm good with whatever new arch. but you'd have to draft a good bp so I can help you with your vision | 10:09 |
| boris-42 | yfried: so the thing that I would like from you is very simple | 10:10 |
| boris-42 | yfried: just move plugins | 10:10 |
| boris-42 | yfried: from one place to another | 10:10 |
| yfried | boris-42: right now? | 10:10 |
| *** anshul has joined #openstack-rally | 10:10 | |
| boris-42 | yfried: we can do it even now | 10:11 |
| yfried | ok. where to? | 10:11 |
| boris-42 | yfried: take a look at my paste | 10:11 |
| boris-42 | yfried: move under rally/plugins directory like here http://paste.openstack.org/show/214778/ | 10:11 |
| boris-42 | yfried: I even started | 10:11 |
| yfried | boris-42: what should go in common and what should go in openstack? | 10:11 |
| boris-42 | yfried: https://review.openstack.org/#/c/176092/ | 10:12 |
| boris-42 | yfried: everything that is specific for OpenStack in OpenStack=) | 10:12 |
| boris-42 | yfried: we don't have SLA and Runners that are specific for OpenStack | 10:12 |
| yfried | boris-42: shouldn't this wait for the unified plugin base? | 10:12 |
| boris-42 | yfried: but we have few benchmarks scenarios that are not OpenStack related | 10:12 |
| boris-42 | yfried: yep it can wait | 10:12 |
| boris-42 | yfried: at least mark it in your spec | 10:13 |
| boris-42 | yfried: that will be enough for now | 10:13 |
| boris-42 | yfried: btw I finished it almost | 10:13 |
| yfried | boris-42: so give https://review.openstack.org/#/q/status:open+project:openstack/rally+branch:master+topic:refactor_a_bit_plugins,n,z a push and we can start working on the rest of the stuff | 10:13 |
| boris-42 | yfried: yep so don't worry about splitting stuff it will be very very smooth | 10:14 |
| boris-42 | yfried: e.g. first we will make benchmark enigne indepent from OpenStack and split plugins to separated directory | 10:15 |
| yfried | boris-42: ok | 10:15 |
| boris-42 | yfried: after that we will move plugins outside of rally tree to seprated repo (and add this project to rally requiremnts) | 10:15 |
| yfried | boris-42: so, I'm waiting for your plugin work first. | 10:15 |
| boris-42 | yfried: this step will allow us to add more people to core reviews that are reviewing only plugins* | 10:15 |
| boris-42 | and it will block everybody to change plugins & framework in one step | 10:16 |
| yfried | boris-42: re my spec, should it mentions this split? would be better if this split was documented in another spec and mine could just mention it as requirements | 10:37 |
| boris-42 | yfried: so if you would like you can make one more spec | 10:39 |
| yfried | boris-42: better you do it. I'm still unclear about the details, and you already have that in mind | 10:39 |
| boris-42 | yfried: that just expalins that we should move all plugins to under rally/plugins/* to simplify understanding of rally code base | 10:39 |
| boris-42 | yfried: there is no details except this one sentence ^ | 10:40 |
| boris-42 | yfried: it's is just few times calling "git mv" command and changing imports | 10:40 |
| yfried | boris-42: but we need to also need to ID code that should go in common. | 10:40 |
| boris-42 | yfried: it's obvious.. | 10:40 |
| boris-42 | yfried: it's really obvious | 10:41 |
| yfried | boris-42: ok, I'll try. but my spec also started as simple and obvious... | 10:41 |
| boris-42 | yfried: if it use openstack clients or requires openstack -> openstack | 10:41 |
| boris-42 | yfried: nope scenario utils was never simple for me | 10:41 |
| boris-42 | yfried: I even rejected already amaretsky propsal (that was about 2-3 months ago done) | 10:42 |
| boris-42 | yfried: that is terrible hard task that you are trying to work on | 10:42 |
| boris-42 | yfried: I was trying to delay it as much as possible | 10:42 |
| boris-42 | but more and more people is facing the issues with current organization of plugins utils code | 10:42 |
| boris-42 | so we have to do something | 10:43 |
| yfried | boris-42: it's still blocked by plugin unification. what's your ETA on that? | 10:43 |
| yfried | boris-42: no repo split in this spec yet, right? that comes later? | 10:44 |
| boris-42 | yfried: so please write short spec regarding to moving all plugins* | 10:46 |
| boris-42 | yfried: mention this in your spec (current) and say something like "regarding to plugins moving let's do services under rally/plugins/openstack/services/*) | 10:46 |
| boris-42 | yfried: that is what I want | 10:46 |
| boris-42 | yfried: about ETA I hope that we will get plugin base merged by Tuesday | 10:47 |
| *** Miouge has quit IRC | 10:52 | |
| openstackgerrit | Yair Fried proposed openstack/rally: [spec] Refactoring scenario utils https://review.openstack.org/172831 | 11:02 |
| openstackgerrit | Yair Fried proposed openstack/rally: [Spec] Reorder plugins https://review.openstack.org/179644 | 11:02 |
| yfried | boris-42: https://review.openstack.org/#/c/179644/ | 11:03 |
| *** yfried is now known as yfried|afk | 11:13 | |
| boris-42 | yfried|afk: https://review.openstack.org/#/c/179644/ +2ed | 11:14 |
| boris-42 | нyfried|afk btw did you see this https://twitter.com/PavlovicBoris/status/594097866009944064 ? | 11:17 |
| *** yfried|afk is now known as yfried | 11:17 | |
| yfried | boris-42: won't be able to attend the summit | 11:18 |
| boris-42 | yfried: sad =( | 11:18 |
| yfried | boris-42: but my manager will - tkammer | 11:18 |
| boris-42 | yfried: we have 3 design sessions.=( | 11:18 |
| yfried | boris-42: :( | 11:18 |
| boris-42 | yfried: I hope next time as you are core of ofiicial OpenStack project they will send you to summit | 11:20 |
| yfried | boris-42: next time == Tokyo. | 11:20 |
| boris-42 | yfried: yep | 11:20 |
| yfried | boris-42: don't count on it... | 11:20 |
| boris-42 | yfried: why so ? | 11:20 |
| yfried | boris-42: so prices - it's a really long and costly flight for us. | 11:21 |
| yfried | boris-42: and also, assuming the company would agree to send me, my wife still has veto option :( | 11:21 |
| boris-42 | yfried: oh | 11:22 |
| yfried | boris-42: she's not wild about staying alone with 2 kids | 11:22 |
| boris-42 | yfried: so maybe I will attend some OpenStack not-official summit somewhere in Europe | 11:22 |
| yfried | boris-42: I know tempest team and a mid-cycle session once | 11:23 |
| yfried | boris-42: it is more of actual coding and not a lot of discussions | 11:23 |
| boris-42 | yfried: heh | 11:23 |
| boris-42 | yfried: it depends on the mid-cycle lead | 11:23 |
| yfried | boris-42: anyway, I still need your input on my spec with some comments/arguments made by amaretskiy and meteorfox | 11:25 |
| boris-42 | yfried: will do it now | 11:26 |
| boris-42 | yfried: seems like your spec doesn't pass tox | 11:26 |
| boris-42 | yfried: could you please check it locally and fix | 11:26 |
| yfried | boris-42: working on it | 11:26 |
| openstackgerrit | Yair Fried proposed openstack/rally: [spec] Refactoring scenario utils https://review.openstack.org/172831 | 11:50 |
| openstackgerrit | Yair Fried proposed openstack/rally: [Spec] Reorder plugins https://review.openstack.org/179644 | 11:50 |
| boris-42 | yfried: so not enough description for me at least to see whole picture | 11:58 |
| yfried | boris-42: what are we talkin about now? | 11:58 |
| boris-42 | yfried: I am really interested in details regarding to atomic_actions and how it can be reused in both | 11:58 |
| boris-42 | yfried: your spec regarding to utils | 11:58 |
| boris-42 | yfried: could you make POC patch | 11:59 |
| boris-42 | yfried: that will switch single function like image upload | 11:59 |
| *** arcimboldo has joined #openstack-rally | 11:59 | |
| boris-42 | yfried: and shows how context ("images") and at least 1 image scenario how will look like | 11:59 |
| *** arcimboldo has left #openstack-rally | 11:59 | |
| yfried | boris-42: re atomic actions - https://review.openstack.org/#/c/148637/ | 12:00 |
| boris-42 | yfried: there are 2 projects that are using rally in their gates | 12:00 |
| boris-42 | yfried: if we merge this we will break theirs gate | 12:00 |
| boris-42 | yfried: we need to implement stuff in another way | 12:01 |
| yfried | boris-42: not sure. why would it break gates? | 12:03 |
| boris-42 | https://www.irccloud.com/pastebin/ijf19jtF | 12:04 |
| boris-42 | yfried: ^ | 12:04 |
| boris-42 | yfried: because you are changing scenarios | 12:04 |
| boris-42 | yfried: in not backward compable way | 12:04 |
| boris-42 | yfried: they have own plugins * | 12:05 |
| boris-42 | yfried: that are not in rally tree | 12:05 |
| boris-42 | yfried: and this won't work https://review.openstack.org/#/c/148637/6/rally/benchmark/scenarios/cinder/utils.py for them | 12:05 |
| boris-42 | yfried: btw put my paste to work items in your spec | 12:05 |
| boris-42 | yfried: be back after launch | 12:05 |
| yfried | boris-42: making it work with a deprecation flow is implementation detail. obviously this was done before plugin unification existed. | 12:07 |
| *** anshul has quit IRC | 12:13 | |
| openstackgerrit | Yair Fried proposed openstack/rally: [spec] Refactoring scenario utils https://review.openstack.org/172831 | 12:27 |
| yfried | boris-42: ^ | 12:28 |
| *** yfried has quit IRC | 12:37 | |
| *** tfreger has quit IRC | 13:22 | |
| *** yfried has joined #openstack-rally | 13:31 | |
| boris-42 | yfried: so we need to provide in that spec more details | 13:35 |
| boris-42 | yfried: words/code/samples | 13:35 |
| boris-42 | currently it's not enough clear | 13:35 |
| yfried | I'm not sure about that. I don't want to start implementing stuff without a clear idea. | 13:36 |
| vaidy | boris-42: hi | 13:39 |
| boris-42 | yfried: imho | 13:40 |
| boris-42 | yfried: I like more code than specs | 13:40 |
| boris-42 | yfried: from your spec it's not clear what you are going to implement | 13:41 |
| boris-42 | yfried: the simplest way to show is it implement some very base + port part of functionallity to it | 13:41 |
| yfried | boris-42: I'll try that tomorrow | 13:41 |
| boris-42 | yfried: so what I am asking not too much | 13:42 |
| boris-42 | yfried: it's just to move for example 2 methods (create/delete image) | 13:42 |
| boris-42 | yfried: and port context and image scenario to it | 13:42 |
| boris-42 | yfried: even without unit/functional tests and it even can get -1 from Jenkins | 13:42 |
| boris-42 | yfried: just to see HOW it will look in REAL life | 13:42 |
| boris-42 | yfried: because the goal is to make things simpler not just to refactor | 13:43 |
| boris-42 | vaidy: hi hi | 13:43 |
| vaidy | boris-42: hope himanshu is been interacting with you regarding the involvement of team in rally contribution | 13:46 |
| vaidy | rvce-himansu | 13:47 |
| vaidy | boris-42: we are planning to have one dedicated resource to continuously work on rally | 13:47 |
| boris-42 | vaidy: that sounds great | 13:48 |
| vaidy | boris-42: and that would be me to contribute upstream | 13:48 |
| boris-42 | vaidy: ok great | 13:48 |
| boris-42 | vaidy: so what you are going to work on? | 13:48 |
| vaidy | boris-42: himanshu was saying we already have many developers and we would require reviewers and would like to start there | 13:49 |
| vaidy | boris-42: what do you say? | 13:49 |
| boris-42 | vaidy: we need reviewers | 13:49 |
| boris-42 | vaidy: as many as possible=) | 13:49 |
| boris-42 | vaidy: here is rally dashboard goo.gl/04aT6p that simplifies reviewing process | 13:49 |
| vaidy | boris-42: cool that should be ok with me too.. will start with reviewing the code.. but will require some development too to be in touch. | 13:50 |
| boris-42 | vaidy: here is description of what you see there http://boris-42.me/rally-review-dashboard/ | 13:50 |
| boris-42 | vaidy: sure | 13:50 |
| boris-42 | vaidy: so we have bunch of open bugs | 13:50 |
| boris-42 | vaidy: some of them are not too complicated | 13:50 |
| vaidy | boris-42: yeah saw that in your linkedin profile oo | 13:50 |
| vaidy | too | 13:51 |
| vaidy | :) | 13:51 |
| boris-42 | vaidy: e.g this one https://bugs.launchpad.net/rally/+bug/1449760 | 13:51 |
| openstack | Launchpad bug 1449760 in Rally "Rally deployment check doesn't check "users" credentials " [Medium,Triaged] | 13:51 |
| boris-42 | vaidy: =) | 13:51 |
| *** mwagner_lap has joined #openstack-rally | 13:51 | |
| vaidy | boris-42: will start with small ones to get used to.. and once comfortable will take up the complex ones | 13:51 |
| vaidy | boris-42: hope you are ok with that | 13:51 |
| boris-42 | vaidy: yep you can take that bug | 13:53 |
| boris-42 | vaidy: it's not hard for fixing | 13:53 |
| boris-42 | vaidy: this methods requires some tuning https://github.com/openstack/rally/blob/master/rally/cmd/commands/deployment.py#L234-L259 | 13:53 |
| boris-42 | vaidy: to check users as well | 13:53 |
| vaidy | boris-42: sure will pick up this bug and will work on this | 13:57 |
| boris-42 | vaidy: great thanks | 13:58 |
| vaidy | boris-42: welcome | 14:01 |
| boris-42 | vaidy: have to go | 14:04 |
| boris-42 | vaidy: be tomorrow back | 14:04 |
| openstackgerrit | Roman Vasilets proposed openstack/rally: Shifting _generate_random_name() call to utils https://review.openstack.org/159719 | 14:07 |
| *** Miouge has joined #openstack-rally | 14:09 | |
| openstackgerrit | Jens Rosenboom proposed openstack/rally: Fix security groups cleanup https://review.openstack.org/179428 | 14:16 |
| *** ibiris_away has quit IRC | 14:57 | |
| *** ibiris_away has joined #openstack-rally | 14:57 | |
| *** ibiris_away is now known as ibiris | 14:57 | |
| *** ibiris is now known as ibiris_away | 15:08 | |
| *** yfried is now known as yfried|afk | 16:01 | |
| *** yfried|afk is now known as yfried | 16:12 | |
| *** kiran-r has joined #openstack-rally | 16:23 | |
| *** Miouge has quit IRC | 16:29 | |
| *** yfried is now known as yfried|afk | 17:04 | |
| *** yfried|afk is now known as yfried | 17:14 | |
| *** yfried is now known as yfried|afk | 17:37 | |
| *** yfried|afk is now known as yfried | 17:38 | |
| *** Miouge has joined #openstack-rally | 17:42 | |
| *** shakamunyi has joined #openstack-rally | 17:48 | |
| *** mwagner_lap has quit IRC | 18:31 | |
| *** e0ne has joined #openstack-rally | 18:37 | |
| *** yfried is now known as yfried|afk | 18:46 | |
| *** mwagner_lap has joined #openstack-rally | 18:48 | |
| *** exploreshaifali has joined #openstack-rally | 19:05 | |
| *** yfried|afk is now known as yfried | 19:06 | |
| *** yfried is now known as yfried|afk | 19:16 | |
| *** e0ne has quit IRC | 19:42 | |
| *** exploreshaifali has quit IRC | 19:51 | |
| *** e0ne has joined #openstack-rally | 20:13 | |
| *** neeti has joined #openstack-rally | 20:18 | |
| *** neeti has quit IRC | 20:28 | |
| *** boris-42 has quit IRC | 20:28 | |
| *** e0ne has quit IRC | 20:40 | |
| *** Miouge has quit IRC | 21:13 | |
| openstackgerrit | Roman Vasilets proposed openstack/rally: Shifting _generate_random_name() call to utils https://review.openstack.org/159719 | 22:17 |
| *** oanufriev has joined #openstack-rally | 23:18 | |
| *** harlowja has quit IRC | 23:23 | |
| *** mwagner_lap has quit IRC | 23:35 | |
| *** mwagner_lap has joined #openstack-rally | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!