Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2014-12-30

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

All times shown according to UTC.

Time Nick Message
00:30 openstackgerrit li,chen proposed stackforge/rally: Use string.ascii_lowercase and string.ascii_uppercase only  https://review.openstack.org/144255
01:56 openstackgerrit Li Yingjun proposed stackforge/rally: Move files to common lib(Part 3)  https://review.openstack.org/143938
02:08 yaguang joined #openstack-rally
03:39 Poornima joined #openstack-rally
04:22 nkhare joined #openstack-rally
04:22 nkhare_ joined #openstack-rally
04:39 rvcehimanshu joined #openstack-rally
04:46 rvcehimanshu joined #openstack-rally
05:11 Poornima joined #openstack-rally
05:26 exploreshaifali joined #openstack-rally
05:32 rvcehimanshu joined #openstack-rally
05:53 prmtl joined #openstack-rally
06:00 Poornima joined #openstack-rally
06:30 exploreshaifali joined #openstack-rally
06:34 neeti joined #openstack-rally
06:36 gema joined #openstack-rally
06:44 yfried_ joined #openstack-rally
06:47 prmtl joined #openstack-rally
07:00 Poornima joined #openstack-rally
07:02 nmagnezi joined #openstack-rally
07:10 exploreshaifali joined #openstack-rally
07:37 prmtl joined #openstack-rally
08:15 tkammer joined #openstack-rally
08:49 Poornima joined #openstack-rally
09:00 andreykurilin joined #openstack-rally
09:11 andreykurilin joined #openstack-rally
09:17 yfried_ amaretskiy: good morning
09:17 yfried_ amaretskiy: re https://review.openstack.org/#/c/143849
09:18 nmagnezi joined #openstack-rally
09:18 yfried_ amaretskiy: I am trying to get a list of all available external networks, but "self._list_networks()" uses nova-client by default
09:19 yfried_ How can I use neutron instead?
09:24 pcaruana joined #openstack-rally
09:31 andreykurilin joined #openstack-rally
09:36 openstackgerrit Merged stackforge/rally: Use string.ascii_lowercase and string.ascii_uppercase only  https://review.openstack.org/144255
09:50 boris-42 joined #openstack-rally
09:56 openstackgerrit Nikita Konovalov proposed stackforge/rally: [Sahara] refactoring _launch_cluster  https://review.openstack.org/143089
09:56 openstackgerrit Nikita Konovalov proposed stackforge/rally: Neutron network context support  https://review.openstack.org/143671
10:06 yfried_ amaretskiy: are you around?
10:06 amaretskiy yfried_ hi
10:07 amaretskiy yfried_, to get list of available networks, use rally.benchmark.wrapper.network.NetworkWrapper.list_networks()
10:08 amaretskiy this wrapper has unified interface - the idea is using one interface both for novanetwork and neutron
10:09 yfried_ amaretskiy: about network context - network are created with default secgroups
10:09 amaretskiy example: https://github.com/stackforge/rally/blob/master/rally/benchmark/context/network.py#L51
10:09 yfried_ amaretskiy: meaning the VMs can't be reached (ie with ping
10:10 yfried_ amaretskiy: would it make sense to add to the context the option to add some scgroup-rules?
10:10 amaretskiy yfried_ this wrapper is not perfect
10:10 amaretskiy we need to improve it
10:10 amaretskiy take a look at https://review.openstack.org/#/c/144337/
10:10 yfried_ amaretskiy: I got around the wrapper, I'm now looking at the network context. tnx
10:10 amaretskiy this patch is WIP
10:11 amaretskiy novanetwork+context is not working
10:12 yfried_ amaretskiy: https://github.com/openstack/tempest/blob/master/tempest/services/network/resources.py
10:12 yfried_ amaretskiy: this is something we did in tempest to auto generate objects out of neutron dict output
10:13 yfried_ amaretskiy: I'm now thinking about neutron with context. I'll worry about nova later
10:13 amaretskiy interesting, thanks
10:13 yfried_ amaretskiy: I could help you with it after I finish the ping thing
10:14 amaretskiy okay!
10:14 yfried_ amaretskiy: so about my question re secgroups - can you think of a reason not to add this OPTION?
10:15 amaretskiy https://github.com/stackforge/rally/blob/master/rally/benchmark/context/secgroup.py
10:15 amaretskiy i'm not sure
10:16 amaretskiy need some time to think :)
10:16 yfried_ amaretskiy: oh, there's already a secgroup context? ok. so just need to allow ping
10:17 amaretskiy give me a time
10:17 amaretskiy not ready to suggest something
10:18 amaretskiy i have same problem in my patch https://review.openstack.org/#/c/144337/
10:18 amaretskiy yes, tehre is a secgroup context
10:19 amaretskiy but that looks strange to use it for ping
10:32 yfried_ amaretskiy: why does it look strange?
10:38 exploreshaifali joined #openstack-rally
10:43 amaretskiy yfried_ let's discuss that a bit later, i need a piece of time to make some things clear
10:43 yfried_ amaretskiy: sure. but I think I'm ok. ping me whenever you want
10:43 amaretskiy ok
10:44 yfried_ amaretskiy: "allow_ssh" context works for me as is.
10:49 nmagnezi joined #openstack-rally
11:05 yfried_ amaretskiy: who is cleaning the servers of nova.utils._boot_server?
11:06 chen joined #openstack-rally
11:07 amaretskiy yfried_ https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/nova/servers.py#L209
11:08 amaretskiy https://github.com/stackforge/rally/blob/master/rally/benchmark/context/cleanup/resources.py#L61
11:33 openstackgerrit Yair Fried proposed stackforge/rally: Moves ping utilities from "vm" to "nova"  https://review.openstack.org/143849
11:34 yfried_ amaretskiy: boris-42: that's a very early WIP but I was able to get VM to ping using neutron. I still have problems with cleanups and I need to add docstring. but I'd like your review so far
11:34 yfried_ ^^
11:35 vkmc joined #openstack-rally
11:35 vkmc joined #openstack-rally
11:35 amaretskiy yfried_ ok, I will review it today
11:35 amaretskiy thanks!
11:38 yfried__ joined #openstack-rally
11:48 chen joined #openstack-rally
11:56 oanufriev joined #openstack-rally
11:57 yfried|afk joined #openstack-rally
11:58 neeti_ joined #openstack-rally
12:13 yfried|afk joined #openstack-rally
12:20 openstackgerrit Mikhail Dubov proposed stackforge/rally: [WIP] Restructure Rally docs  https://review.openstack.org/120018
12:32 tkammer joined #openstack-rally
12:47 pcaruana joined #openstack-rally
13:24 yfried boris-42: can I see how the rally gate is configured? is it created in devstack-gate and/or other openstack-infra repos?
13:31 chen joined #openstack-rally
13:55 psd joined #openstack-rally
13:57 vkmc boris-42, hiii, how are you?
13:57 vkmc boris-42, I wanted to introduce you psd, he would like to start contributing to Rally
13:58 vkmc let me know when you are around
14:00 boris-42 vkmc: hey hey
14:00 boris-42 vkmc: I am around-)
14:00 psd Hi!
14:00 boris-42 psd: nice to meet you
14:00 vkmc cool :)
14:00 boris-42 yfried: I will take a look on that
14:01 boris-42 vkmc: how are you?))
14:01 boris-42 vkmc: haven't seen you for a while=)
14:01 psd boris-42, Good to see you too :)
14:01 vkmc boris-42, good good, in the Trove land
14:01 boris-42 vkmc: ha=) how's that going?
14:01 vkmc dbs everywhere
14:02 vkmc its going I think haha
14:02 boris-42 vkmc in the debs world
14:02 boris-42 dbs*
14:02 boris-42 =)
14:02 vkmc boris-42, and you? :)
14:02 boris-42 vkmc: good good just got new mac book =)
14:02 boris-42 vkmc: setting up everything=)
14:03 vkmc ohhhh, shiny!
14:03 vkmc that's great
14:03 boris-42 vkmc: ya finally 16gb ram!!
14:03 boris-42 =)
14:03 vkmc omg
14:03 vkmc that's a lot haha
14:03 boris-42 vkmc: that's just enough, I was tired waiting for mac book with 32gb ram lol
14:03 boris-42 =0
14:04 boris-42 so bought this one lol
14:04 vkmc that's ok
14:04 vkmc ssd?
14:07 vkmc boris-42, ^
14:07 boris-42 vkmc: better=)
14:07 boris-42 vkmc: it's SSD connected to PCI
14:07 vkmc w00000000t
14:07 boris-42 vkmc: ya copy file 5gb for 4 seconds)
14:08 boris-42 vkmc: and it's quite large 1tb=)
14:08 vkmc boris-42, naaaaaaah
14:08 vkmc boris-42, that's so great
14:09 boris-42 vkmc: yep true geek stuff=)
14:09 psd_ joined #openstack-rally
14:09 bumbleb joined #openstack-rally
14:09 chenli12 joined #openstack-rally
14:10 boris-42 vkmc: btw http://www.youtube.com/watch?v=TZ5JoCDH-cw !!!
14:10 vkmc boris-42, haha totally... :D I want to see some benchmarks of that stuff later
14:11 vkmc boris-42, +2!
14:14 vkmc anyhow psd, here in rally this kind of nerd talks happen everytime
14:15 vkmc as in other projects, bugs are listed in https://bugs.launchpad.net/rally
14:15 vkmc its a great project, the code is super neat and you will have fun, for sure :)
14:16 psd haha!
14:17 psd vkmc, Awesome! I'll go find a bug :D
14:17 vkmc psd, if you need help setting up Rally, let me know
14:18 vkmc they made it easy though, you will be fine
14:18 psd vkmc, Yeah Sure :)
14:20 openstackgerrit Nir Magnezi proposed stackforge/rally: Adds support for instances with IPv6 addresses in VMScenario. Currently, The wait_for uses ping_ip_address for connetivity checks (via ping). This will work only if the instance has an IPv4 address. The ping_ip_address should determine wheter it should us  https://review.openstack.org/144462
14:23 openstackgerrit Nir Magnezi proposed stackforge/rally: Adds support for instances with IPv6 addresses in VMScenario.  https://review.openstack.org/144462
14:30 boris-42 vkmc: psd sorry I am just after workout
14:30 boris-42 vkmc: psd so a bit slow lol
14:30 boris-42 )
14:30 boris-42 vkmc: psd I'll be better soon=)
14:32 yfried boris-42: Exception TypeError: TypeError("'NoneType' object does not support item deletion",) in <Finalize object, dead> ignored
14:32 yfried boris-42: I'm seeing this at the end of the run. where can I see a stacktrace?
14:33 vkmc boris-42, no worries! you are ok :)
14:33 boris-42 yfried: you can try to run "rally -vd "
14:33 boris-42 yfried: then it will print tons of logs
14:33 boris-42 psd: so as far as I understand you would like some simple tasks for start?
14:34 jaypipes joined #openstack-rally
14:35 nmagnezi boris-42, hi, filed a first patch. future patches will handle tests and NeutronScenario (IPv6)
14:38 psd boris-42, np
14:38 psd boris-42, Yeah!
14:43 psd_ joined #openstack-rally
14:44 boris-42 nmagnezi: heh me not good specialist in such IPv6 queistions=)
14:44 boris-42 nmagnezi: but I'll try to help
14:44 nmagnezi boris-42, much appreciated :-)
14:45 nmagnezi boris-42, waiting a unittest for this now
14:45 boris-42 nmagnezi: they are not only unittests
14:45 boris-42 nmagnezi: when you send on review it runs as well functional tests
14:45 rook joined #openstack-rally
14:45 nmagnezi boris-42, mm please explain
14:45 yfried boris-42: floating ips aren't cleaned up, so I'm looking into https://github.com/stackforge/rally/blob/master/rally/benchmark/context/cleanup/resources.py#L90:L146
14:46 yfried boris-42: any pointers on how to get floating ips to cleanup?
14:46 yfried boris-42: do I simply add them to the resources?
14:46 yfried boris-42: I'm running scenario.nova.boot_server
14:47 boris-42 yfried: one sec one sec
14:48 boris-42 yfried: soo boot_server doesn't create floating_Ip as far as I know
14:48 boris-42 yfried: so not sure that I fully get...
14:49 yfried boris-42: https://review.openstack.org/#/c/143849/2 it will soon :)
14:49 boris-42 nmagnezi: when you are sending patch on review, Jenkins is running a lot of different jobs
14:49 boris-42 nmagnezi: there are jobs that runs unit tests and pep8 but there are as well functional tests, like running rally tasks against OpenStack
14:49 boris-42 yfried: lol=)
14:50 nmagnezi boris-42, got it. well, rally currently lacks support for IPv6 subnets (i can use them, but not create them)
14:50 nmagnezi boris-42, I plan to work on that as well, but that's a different patch
14:51 nmagnezi boris-42, that patch has a limited scope (as you can see)
14:51 boris-42 nmagnezi: taht's okay, I am just telling you how our CI works=)
14:51 nmagnezi boris-42, ack. thanks for that
14:51 nmagnezi boris-42, just saying i have no functional tests in mind for this atm
14:51 boris-42 nmagnezi: and that you are wating not for unit tests) that are quite fast, but for other functioanl=)
14:51 boris-42 nmagnezi: you can see messages from Jenkins
14:51 boris-42 nmagnezi: everything that contains -dsvm- is functional
14:52 boris-42 nmagnezi: dsvm (means devstack vm)
14:52 boris-42 nmagnezi: you can read here https://github.com/stackforge/rally/tree/master/tests/ci what it does mean
14:53 boris-42 nmagnezi: so gate-rally-dsvm-* (except gate-rally-dsvm-cli) are rally-gate.sh
14:53 exploreshaifali joined #openstack-rally
14:53 boris-42 nmagnezi: gate-rally-dsvm-cli runs functional https://github.com/stackforge/rally/tree/master/tests/functional
14:53 boris-42 nmagnezi: it's actually that all CLI commands works
14:58 openstackgerrit Nir Magnezi proposed stackforge/rally: Adds support for instances with IPv6 addresses in VMScenario.  https://review.openstack.org/144462
14:59 boris-42 yfried: btw
15:00 boris-42 yfried: I would prefer to keep boot_server name
15:00 boris-42 yfried: instead of boot_vm
15:01 openstackgerrit Nir Magnezi proposed stackforge/rally: Adds support for instances with IPv6 addresses in VMScenario.  https://review.openstack.org/144462
15:03 nmagnezi boris-42, will look into those functional tests you mentioned. i think my code does not touch this due
15:03 nmagnezi boris-42, I added unittests because i was requested to do so in the review
15:03 boris-42 nmagnezi: not sure not sure=)
15:03 boris-42 nmagnezi: yep oru goal is to have 100% coverage by unit tests
15:04 nmagnezi boris-42, fair enough :)
15:05 nmagnezi boris-42, check out the IPv6 address in line #120 https://review.openstack.org/#/c/144462/4/tests/unit/benchmark/scenarios/vm/test_utils.py,cm
15:06 openstackgerrit Roman Vasilets proposed stackforge/rally: Add murano base for murano benchmarks  https://review.openstack.org/137650
15:07 yfried boris-42: so what do I call the outside methods that is actually called by the scenario? changing its name would break ppls jsons and yamls
15:07 openstackgerrit Yair Fried proposed stackforge/rally: (WIP) Moves ping utilities from "vm" to "nova"  https://review.openstack.org/143849
15:07 openstackgerrit Yair Fried proposed stackforge/rally: Adds floating_network to network context  https://review.openstack.org/144472
15:10 yfried boris-42: https://review.openstack.org/#/c/144472/ is a simple addition to network context that auto discovers external network
15:11 yfried boris-42: should make it easier to work with neutron
15:12 boris-42 yfried: oh a lot of patches)
15:12 boris-42 amaretskiy: hey hey
15:12 boris-42 yfried: ^ is changing your code=)
15:12 boris-42 yfried: so about changing name of internal name, a long long ago
15:13 boris-42 yfried: we decide to use "servers" instead of "vm" cause there is no API call VM *
15:13 boris-42 yfried: so for operators it will be simpler to understand servers
15:13 boris-42 yfried: and now even if decide to renamte servers to vms we should do it everywhere
15:13 boris-42 yfried: not only in one single method, cause it makes mess
15:14 amaretskiy boris-42 yfried: I like new improvements in network context and will review this patch
15:18 boris-42 psd: back to you
15:18 boris-42 psd: do you have expirience with contribution to openstack projects?
15:19 boris-42 psd: I mean license + setting up gerrit and so on?
15:19 psd boris-42, nope
15:20 boris-42 psd: oh so that's the hardest part ever=)
15:20 boris-42 psd: so first of all you need to put your signature on selling your soul to OpenStack foundation
15:20 boris-42 psd: and that you are ok with that
15:21 boris-42 psd: take a look here https://wiki.openstack.org/wiki/Rally/Develop#How_to_contribute
15:21 boris-42 psd: for manual
15:21 psd boris-42, I started with that, I am not sure if I have completed the process
15:21 psd okay I will tell you what all have I done
15:21 boris-42 psd: so it's easy to check
15:21 boris-42 psd: clone rally repo
15:22 boris-42 psd: run git review -s
15:22 boris-42 psd: put your launcpad name
15:22 boris-42 psd: if it works => success
15:22 boris-42 =)
15:22 psd boris-42, okay
15:23 yfried boris-42: I think I solved the nameing issue. please see if you agree
15:23 openstackgerrit Yair Fried proposed stackforge/rally: (WIP) Moves ping utilities from "vm" to "nova"  https://review.openstack.org/143849
15:25 boris-42 psd: so if it works we will find some ultra small patch for you
15:25 boris-42 psd: and you'll be able to test full worfklow
15:26 psd boris-42, I did not keep the same launchpad and gerrit usernames will that be a problem ?
15:26 boris-42 psd: I don't think that it will be problems
15:26 boris-42 problem*
15:26 boris-42 psd: just use gerrit username
15:26 psd awesome!
15:26 boris-42 psd: it works?
15:27 amaretskiy yfried: I will review https://review.openstack.org/#/c/144472/ tomorrow
15:31 psd boris-42, I get the following error : http://pastebin.mozilla.org/8147279
15:32 boris-42 psd: interesting
15:33 boris-42 psd: from where do you clone rally repo?
15:34 psd I did a zip clone to desktop. will that create a problem  ?
15:34 boris-42 zip clone?
15:34 boris-42 psd: clone it from stackforge directly
15:34 boris-42 psd: like "git clone https://github.com/stackforge/rally"
15:34 psd I mean the zip download from github/stackforge
15:35 psd Okay
15:35 boris-42 psd: run that command
15:35 boris-42 psd: it may produce issues
15:35 psd boris-42, Alrighty
15:38 psd boris-42, brb
15:41 boris-42 psd: so does it work/
15:42 boris-42 vkmc: so are you working on zaqar?
15:42 boris-42 vkmc: or not anymore?)
15:42 vkmc boris-42, I am! :)
15:42 boris-42 vkmc: ah great=)
15:42 vkmc boris-42, it has been quiet lately... but we are trying to keep things going
15:42 boris-42 vkmc: I know that stuff=)
15:43 boris-42 vkmc: I remember time when Rally lost 5 core contributors in 1 week+)
15:43 boris-42 vkmc: and there was only 2 left=) me and my collegue=)
15:43 vkmc boris-42, that's a bummer
15:43 boris-42 vkmc: that was bad..
15:43 boris-42 vkmc: ya it was bad season =)
15:43 vkmc it is... everything gets harder
15:44 vkmc boris-42, would you try to incubate next release?
15:44 vkmc boris-42, well... I dunno how it works now with the big-tent thing
15:54 psd psd, my ISP doesn't seem to like github and has reading about getting a proxy on with on command line
15:56 vkmc brb
15:56 vkmc psd, are you in the VM?
15:59 psd_ joined #openstack-rally
16:06 openstackgerrit Mikhail Dubov proposed stackforge/rally: [WIP] Restructure Rally docs  https://review.openstack.org/120018
16:12 afazekas joined #openstack-rally
16:19 boris-42 vkmc: oh seems like psd has some issues
16:24 psd joined #openstack-rally
16:27 rvcehimanshu joined #openstack-rally
16:30 psd boris-42, really sorry for the delay
16:30 psd boris-42, I used git clone but with the same error
16:39 boris-42 psd: hm
16:40 boris-42 psd: did you upload in launchpad your ssh key?
16:40 psd in gerrit you mean?
16:40 boris-42 psd: yep sorry*
16:40 psd yes I did
16:41 boris-42 hm
16:41 boris-42 psd: we can try to add manually remotes
16:41 psd okay
16:43 boris-42 https://www.irccloud.com/pastebin/9CPhGFkj
16:43 boris-42 psd: ^
16:43 boris-42 psd: so you'll need to change boris-42 to psd
16:43 boris-42 psd: and create such remote called gerrit
16:44 psd okay
16:49 psd I get a no such file or directory exists error.
16:50 psd I think I am missing out on something stupid :/
16:52 boris-42 psd: could you please run  ->  ssh psd@review.openstack.org -p 29418
16:55 psd boris-42, Permission denied (public key) : error
16:55 boris-42 psd: okay so one thing is wrong
16:55 boris-42 psd: your key or your nick name
16:56 psd okay.
16:56 vkmc psd, you have to make sure that your email in launchpad is the same as the one in gerrit
16:56 psd yeah it is
16:56 openstackgerrit Oleh Anufriiev proposed stackforge/rally: Report generation improvement  https://review.openstack.org/143866
16:57 boris-42 psd: are you sure that you upload id_rsa.pub key?
16:57 vkmc psd, you have to upload id_rsa.pub both to launchpad and gerrit as well
16:57 vkmc and... I saw boris-42 already mentioned it.. the CLA
16:57 psd I have uploaded it just to gerrit
16:57 vkmc oh, upload it to launchpad too
16:58 psd vkmc, boris-42 okay
16:58 vkmc here is the whole process :) https://wiki.openstack.org/wiki/Gerrit_Workflow
17:02 psd I am still getting the same error.going through the doc now
17:08 vkmc ok, we can continue debugging in #openstack-101
17:08 vkmc :)
17:13 psd_ joined #openstack-rally
17:14 yfried joined #openstack-rally
17:16 openstackgerrit Oleh Anufriiev proposed stackforge/rally: Report generation improvement  https://review.openstack.org/143866
17:24 rvasilets left #openstack-rally
17:38 openstackgerrit Roman Vasilets proposed stackforge/rally: Add murano base for murano benchmarks  https://review.openstack.org/137650
17:38 openstackgerrit Roman Vasilets proposed stackforge/rally: Add murano base for murano benchmarks  https://review.openstack.org/137650
17:39 flwang joined #openstack-rally
18:13 harlowja joined #openstack-rally
18:21 openstackgerrit Alexander Maretskiy proposed stackforge/rally: (WIP) Improvements for scenario VMTasks.boot_runcommand_delete  https://review.openstack.org/144337
18:43 psd boris-42, There?
18:43 boris-42 psd: here
18:44 psd boris-42, okay so git review -s returned nothing
18:44 * boris-42 psd: fucking yeah
18:44 boris-42 psd: now you are ready
18:44 psd boris-42, awesome :)
18:44 boris-42 psd: let me find something simple for u
18:45 psd okay :)
18:48 coolsvap joined #openstack-rally
18:50 boris-42 psd: so we have bug in our reports
18:50 boris-42 psd: here http://logs.openstack.org/66/143866/9/check/gate-rally-dsvm-rally/4b83e24/rally-plot/results.html.gz#/CeilometerAlarms.create_alarm/task
18:51 boris-42 psd: input task is not proper formated
18:51 boris-42 psd: after key with name of benchmark we should have [] not {}
18:52 psd Okay
18:54 boris-42 psd: so lemme help you
18:54 psd yeah
18:55 boris-42 psd: https://github.com/stackforge/rally/blob/master/rally/benchmark/processing/plot.py#L310
18:56 boris-42 psd: this line generates data for it
18:56 boris-42 psd: so it should be something like {scenario_name: [kw]}
19:01 psd boris-42, I see it
19:07 psd boris-42, so I should make the changes in my local repo and
19:08 boris-42 psd: make a commit
19:08 boris-42 psd: and run "git review -R"
19:08 psd to make a commit I just say git commit -m 'the message'
19:09 * vkmc lurks
19:17 psd_ joined #openstack-rally
19:20 vkmc boris-42, getting ready for new year?
19:26 psd boris-42, what commit message should I add?
19:26 psd vkmc, ^
19:27 vkmc psd, a title that best describes the problem < 50 chars, a longer description wrapped to 70 chars, the bug that closes (if any)
19:27 vkmc that is the short version
19:28 vkmc the long version is here https://wiki.openstack.org/GitCommitMessages
19:31 psd vkmc, Thanks :
19:31 vkmc make sure that the bug that closes has this syntax "Closes-Bug: #number"
19:32 vkmc I don't see that boris-42 give you a link, so maybe he didn't report it and you don't need to put that line
19:33 psd cool
19:35 psd is something like this okay?
19:35 psd Wrongly formatted input task. After key with name of Benchmark we must use Square brackets [] instead of parantheses {}.
19:35 vkmc sounds good!
19:36 boris-42 vkmc: thanks
19:36 boris-42 vkmc: yep there is no bug for that
19:37 boris-42 vkmc: so we can avoid that bullshit=)
19:37 boris-42 vkmc: with Closes-bug=)
19:37 vkmc hahaha
19:37 boris-42 vkmc: and I am not yet ready
19:37 boris-42 vkmc: for new year lol=)
19:39 vkmc boris-42, I heard that you have like a 12 days holiday? supposing you are in Moscow
19:39 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Wrongly formatted input task. After key with name of Benchmark we must use Square brackets [] instead of parantheses {}.  https://review.openstack.org/144505
19:40 boris-42 psd: you need to add one blank line between first line and body of commit message
19:40 vkmc psd, \o/ happy first contribution
19:40 psd oh
19:40 boris-42 vkmc: no!
19:40 vkmc yup, now you have to amend that
19:40 boris-42 vkmc: it's not yet merged=)
19:40 boris-42 psd: and you should keep Change-iD
19:40 vkmc oh ok,... well, we can celebrate two times
19:40 boris-42 psd: it's important=)
19:41 boris-42 vkmc: okay celebrating to times seems ok=)
19:41 psd boris-42, I should keep what?
19:41 boris-42 psd: you'll see in commit message Change-id: bablbalballgal
19:41 boris-42 psd: don't touch it=)
19:42 psd okay
19:42 vkmc psd, https://wiki.openstack.org/wiki/Gerrit_Workflow#Committing_Changes
19:46 psd when I git commit --amend it, I automatically get the <commit mesage> <blank line> <change id : bla bla>
19:48 vkmc that sounds right
19:48 psd okay so do I insert a blank line at the start?
19:49 vkmc yeah, its title <blank> body <blank> change-id (autogenerated)
19:49 psd okay
19:49 psd and I'll post for review again?
19:50 vkmc exactly
19:50 vkmc git commit -a --amend
19:50 vkmc do the changes
19:50 vkmc and then, git review
19:51 psd The blank line is the front is getting truncated automatically
19:51 psd oh my bad!
19:52 vkmc is the editor probably... by default its vim
19:53 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Wrongly formatted input task.  https://review.openstack.org/144505
19:53 vkmc cool!
19:54 psd :)
20:08 psd boris-42, vkmc  Thanks a lot for helping out today. I am gonna crash now. just message/tag me if you think that some bug/task would be good for me :)
20:09 vkmc psd, no problem and congrats on making your first contribution!
20:09 vkmc psd, feel free to start reading about Rally, how it works, and going through the bug list :)
20:12 psd I will :)
20:17 exploreshaifali joined #openstack-rally
20:20 exploreshaifali joined #openstack-rally
20:30 psd joined #openstack-rally
20:30 Prabhjyot__ joined #openstack-rally
20:30 psd_ joined #openstack-rally
20:41 openstackgerrit Oleh Anufriiev proposed stackforge/rally: Support generation of report from N task  https://review.openstack.org/143866
20:54 nmagnezi joined #openstack-rally
20:54 openstackgerrit Nir Magnezi proposed stackforge/rally: Adds support for instances with IPv6 addresses in VMScenario  https://review.openstack.org/144462
20:57 openstackgerrit Nir Magnezi proposed stackforge/rally: Adds support for instances with IPv6 addresses in VMScenario  https://review.openstack.org/144462
21:03 psd_ joined #openstack-rally
21:26 openstackgerrit Nir Magnezi proposed stackforge/rally: Adds support for instances with IPv6 addresses in VMScenario  https://review.openstack.org/144462
21:32 Prabhjyot__ joined #openstack-rally
21:43 andreykurilin joined #openstack-rally
22:52 andreykurilin joined #openstack-rally
23:11 boris-42 jlk: around?
23:12 jlk yea
23:12 boris-42 jlk: so about your patch that removes crap
23:12 boris-42 jlk: https://review.openstack.org/#/c/144366/
23:12 jlk yes
23:12 boris-42 jlk: we defiantly need to merge it, but I have to give time for Mirantis to fix
23:12 boris-42 their internal crap that requires it
23:12 jlk Did it break Mirantis in a way?
23:13 jlk interesting, do they not use a service catalog?
23:13 boris-42 jlk: so lemme expalin
23:14 boris-42 jlk: admin user is accessable only in case of Admin endpoint
23:14 boris-42 jlk: Admin andpoint is private cloud network
23:15 jlk oh, that sounds similar to rackspace
23:15 boris-42 jlk: so that's why they have open port *
23:15 boris-42 jlk: that allows to use admin user
23:15 jlk and I guess the admin end point can't do everything the regular end point can
23:16 boris-42 I think it can do
23:16 jlk so couldn't they just list the auth_url as the admin end point?
23:16 boris-42 nope
23:17 boris-42 in such case rally can be run only from servers with cloud *
23:17 boris-42 I mean controllers/computes nodes..
23:17 boris-42 so that is the issue..
23:18 * boris-42 ilk so I just would like to say that we will need to wait a bit before merging your patch
23:19 jlk hrm.
23:19 jlk I guess I don't see how what was there helps them.
23:19 jlk if the admin user can only auth through the admin port
23:19 jlk since rally only uses admin users, for tenant creation
23:19 jlk how does that work? quirk of the code that sends it all down the admin port for that stuff?
23:20 jlk then the nova calls happen to whatever the service catalog sent back?
23:21 jlk well, we can run with the patch here, not a huge deal.
23:22 boris-42 jlk: it's very very dirty hack related to FUEL deployments
23:22 boris-42 jlk: they have *special* port that allows them to run admin actions
23:23 boris-42 jlk: so using "admin_port" they can force using this specific port => get ability to run admin actions
23:29 boris-42 jlk: in any case I think that this is very ugly
23:46 jlk yeah seems like it
23:47 jlk and that one port is exposed beyond the controllers/computes right?
23:47 jlk oh
23:47 jlk hrm.

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