*** ianw has joined #openstack-qa | 00:09 | |
*** annegentle has quit IRC | 00:10 | |
*** annegentle has joined #openstack-qa | 00:12 | |
*** notmyname_ has joined #openstack-qa | 00:19 | |
*** fbo_away has quit IRC | 00:23 | |
*** jhenner has quit IRC | 00:23 | |
*** ekarlso has quit IRC | 00:23 | |
*** bearhands has quit IRC | 00:23 | |
*** notmyname has quit IRC | 00:23 | |
*** notmyname_ is now known as notmyname | 00:24 | |
*** bearhands has joined #openstack-qa | 00:27 | |
*** ekarlso has joined #openstack-qa | 00:29 | |
*** annegentle has quit IRC | 00:43 | |
*** annegentle has joined #openstack-qa | 00:44 | |
openstackgerrit | Christopher Yeoh proposed a change to openstack/tempest: Adds test_update_all_metadata_field_not_included negative test https://review.openstack.org/46941 | 01:34 |
---|---|---|
*** zhikunliu has joined #openstack-qa | 01:56 | |
*** bdperkin_gone is now known as bdperkin | 02:07 | |
*** fbo_away has joined #openstack-qa | 02:17 | |
*** jhenner has joined #openstack-qa | 02:17 | |
*** sungju has joined #openstack-qa | 02:18 | |
*** sourabh has joined #openstack-qa | 02:44 | |
*** annegentle has quit IRC | 03:05 | |
*** dims has quit IRC | 03:06 | |
*** SergeyLukjanov has joined #openstack-qa | 03:06 | |
*** annegentle has joined #openstack-qa | 03:08 | |
*** SergeyLukjanov has quit IRC | 04:17 | |
*** madan has joined #openstack-qa | 04:39 | |
*** raies has joined #openstack-qa | 04:43 | |
*** annegentle has quit IRC | 04:49 | |
*** annegentle has joined #openstack-qa | 04:53 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas vpnservice test cases https://review.openstack.org/47339 | 05:01 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests https://review.openstack.org/47348 | 05:01 |
*** afazekas has joined #openstack-qa | 05:01 | |
Anju | afazekas: ping | 05:06 |
afazekas | Anju: pong | 05:06 |
Anju | afazekas: hii .. https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_server_rescue.py#L48 | 05:07 |
Anju | can we add a single volume create method here | 05:07 |
Anju | afazekas: ^^^ | 05:09 |
afazekas | looks safe to to me, the detach wait is in the code | 05:11 |
Anju | yes | 05:11 |
afazekas | LGTM | 05:12 |
Anju | afazekas: thanku | 05:12 |
Anju | :) | 05:12 |
openstackgerrit | nayna-patel proposed a change to openstack/tempest: Added test to check list/show extensions-neutron https://review.openstack.org/45030 | 05:19 |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add volume list tests https://review.openstack.org/47508 | 05:23 |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Enable q-vpn service https://review.openstack.org/47335 | 05:24 |
*** Madan1 has joined #openstack-qa | 05:30 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas vpnservice test cases https://review.openstack.org/47339 | 05:34 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests https://review.openstack.org/47348 | 05:34 |
*** bearhands is now known as comstud | 05:35 | |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add negative volume tests https://review.openstack.org/44610 | 05:37 |
*** prekarat has joined #openstack-qa | 05:46 | |
raies | afazekas: hi gm :) | 05:48 |
afazekas | hi | 05:49 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests https://review.openstack.org/47348 | 05:59 |
raies | afazekas: please have a look on https://review.openstack.org/#/c/47339/ | 06:04 |
afazekas | https://review.openstack.org/#/c/47339/6/tempest/api/network/base.py L71 add an exception handler with logging | 06:09 |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Added Negative tests for image member https://review.openstack.org/47167 | 06:10 |
afazekas | if the interface removal not success the router deletion should be attempted , it is possible it failed on the creation | 06:10 |
afazekas | create_router really needs to return with a in interface ? it is little bit confusing | 06:11 |
raies | 1. if the interface removal not success the router deletion should be attempted ?? | 06:12 |
raies | I think , if the interface removal not success the router deletion should NOT be attempted | 06:13 |
raies | what do you think ? | 06:13 |
afazekas | the router deletion contains two operation | 06:13 |
raies | yes 1. interface removal and 2. deletion | 06:14 |
afazekas | the first can fail because on delete just because it not added on create, so it could lead to router leakage | 06:14 |
* afazekas I would enhance or neutron deletion function to handle correctly if one deletion in the loop fails, but I think it is out of scope in a vpn test case change | 06:16 | |
raies | then what I should do here ? | 06:17 |
afazekas | https://github.com/openstack/tempest/blob/master/tempest/thirdparty/boto/test.py#L247 it is more complex cleanup | 06:17 |
afazekas | try: interface removal except Exception ; LOG.exception(exc) | 06:18 |
afazekas | change a variable of an exception is logged | 06:19 |
afazekas | and raise an exception at the end of the tearDownClass if any exception occurred | 06:19 |
raies | ok I will see it | 06:19 |
raies | :w | 06:21 |
afazekas | BTW: If you have time to investigate why this https://review.openstack.org/#/c/45578/ , https://review.openstack.org/#/c/46330/ not working would be great | 06:21 |
raies | ok | 06:22 |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Added Negative tests for image member https://review.openstack.org/47167 | 06:22 |
afazekas | raies: https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L88 | 06:30 |
afazekas | this is little bit more aggressive delete, but does not logs the exception | 06:30 |
raies | ok | 06:30 |
raies | afazekas: it seems better. I will preffer it. and will try to incorporate with a message | 06:31 |
afazekas | If add log checker for ERROR messages in the tempest log, the 2th one with exception logging can be sufficient and simpler. so I am ok with that | 06:35 |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Calling create_volume once for attach_detach https://review.openstack.org/47785 | 06:44 |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Calling create_volume once for attach_detach https://review.openstack.org/47785 | 06:45 |
afazekas | Anju: https://review.openstack.org/#/c/47785/2/tempest/api/compute/servers/test_server_rescue.py replace L57 with L53 ( no comment is required at L53) | 06:55 |
Anju | both need to br deleted ? | 06:55 |
afazekas | do not forget to update the test cases :) | 06:55 |
afazekas | A volume is ready to attach when it is in the available state | 06:56 |
Anju | are you saying for this patch only | 06:56 |
afazekas | it is in that state after waiting | 06:56 |
afazekas | '# Ready for detach' after wait for available is illogical | 06:56 |
Anju | afazekas: # readuy for attach ? | 06:57 |
Anju | afazekas: # ready for attach ? | 06:58 |
afazekas | at L57 it describes the situation correctly | 06:58 |
*** tkammer has joined #openstack-qa | 07:02 | |
*** fbo_away is now known as fbo | 07:12 | |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Calling create_volume once for attach_detach https://review.openstack.org/47785 | 07:16 |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Calling create_volume once for attach_detach https://review.openstack.org/47785 | 07:17 |
Anju | afazekas: which test cases you are talking about to update | 07:18 |
*** SergeyLukjanov has joined #openstack-qa | 07:32 | |
afazekas | Anju: We do not need to variable for the same value, cls.volume_to_detach should be removed | 07:41 |
*** adam_g has quit IRC | 07:42 | |
*** bashok has joined #openstack-qa | 07:42 | |
*** adam_g has joined #openstack-qa | 07:42 | |
*** adam_g has joined #openstack-qa | 07:42 | |
afazekas | The license spacing change seams unrelated | 07:42 |
afazekas | The license comments using mere than one space as it was originally | 07:43 |
Anju | afazekas: same value, cls.volume_to_detach should be removed ????? | 07:44 |
afazekas | If you are planning to use the same volume in two test case you should not make an alias in the setupclass it makes more difficult to understand the code | 07:45 |
afazekas | you may rename it to just 'volome' , but imho the volume_to_attach is ok as well. | 07:46 |
Anju | afazekas: okkk | 07:48 |
Anju | afazekas: using this the other cahnges will be atffected | 07:51 |
Anju | my concern is only make only one volume | 07:51 |
Anju | and use the same volume with attach and detach | 07:52 |
Anju | if i use the same name volume_to_attach | 07:52 |
*** giulivo has joined #openstack-qa | 08:02 | |
*** yassine has joined #openstack-qa | 08:04 | |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Calling create_volume once for attach_detach https://review.openstack.org/47785 | 08:13 |
*** bashok has quit IRC | 08:26 | |
openstackgerrit | Xiang Hui proposed a change to openstack/tempest: Add positive tests for os-floating-ip-pools https://review.openstack.org/47794 | 08:32 |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add server resizing tests https://review.openstack.org/44732 | 08:38 |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add server resizing tests https://review.openstack.org/44732 | 08:41 |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add server resizing tests https://review.openstack.org/44732 | 08:43 |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add server resizing tests https://review.openstack.org/44732 | 08:44 |
raies | afazekas: hello | 09:12 |
raies | as you have suggested to raise exception in https://review.openstack.org/#/c/47339/, I have amended a bit here like http://pastebin.com/2KEjEknE | 09:13 |
raies | will you please look this and revert | 09:13 |
openstackgerrit | Xiang Hui proposed a change to openstack/tempest: Add positive tests for os-floating-ip-pools https://review.openstack.org/47794 | 09:13 |
raies | afazekas: ^^ | 09:13 |
raies | afazekas: are you there ? | 09:34 |
afazekas | the base.py looks like as it was before | 09:36 |
raies | then amendment is required please let me know one more | 09:39 |
raies | it may be that I misunderstood | 09:39 |
raies | to delete router 1. detach it 2. then delete it | 09:40 |
raies | without detach deletion can't be done | 09:40 |
raies | when what is required ? | 09:40 |
raies | only raise exception or do not do to delete if 1 is failed ? | 09:41 |
*** Madan2 has joined #openstack-qa | 09:45 | |
*** Madan1 has quit IRC | 09:46 | |
*** Madan2 has quit IRC | 09:50 | |
*** Madan1 has joined #openstack-qa | 09:53 | |
afazekas | raies: the goal would be try to continue the resource deletion even if one operation failed, and do not do it silently | 10:01 |
raies | I understood it as - (for of router) --> has_ex = F --> try ---> all of try ---> except ---> if exception ---> has_ex =T ---> if has_ex : ---> TearDownException() | 10:02 |
raies | :) | 10:02 |
*** zigo_ has joined #openstack-qa | 10:04 | |
*** zigo has quit IRC | 10:05 | |
*** bashok has joined #openstack-qa | 10:20 | |
raies | afazekas: I have implemented as : http://pastebin.com/0rCcDigB please have a look | 10:22 |
afazekas | if has_exception: | 10:24 |
afazekas | raise TearDownException() | 10:24 |
afazekas | just at the end of the tearDown class | 10:24 |
raies | ok then I have to put has_exception at the top of the function tearDown right ? | 10:24 |
afazekas | yes | 10:25 |
raies | ok | 10:25 |
raies | will return after a moment | 10:25 |
afazekas | if has_exception: | 10:25 |
afazekas | raise TearDownException() is outside to all for loop | 10:25 |
afazekas | and only once at the end | 10:25 |
raies | afazekas: http://pastebin.com/2yLdFGaZ | 10:29 |
raies | afazekas: u there ? | 10:32 |
*** SergeyLukjanov has quit IRC | 10:36 | |
raies | afazekas: ?? | 10:36 |
openstackgerrit | Eoghan Glynn proposed a change to openstack-dev/devstack: Modified ceilometer alarm evaluator console script https://review.openstack.org/47637 | 10:41 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas vpnservice test cases https://review.openstack.org/47339 | 10:41 |
afazekas | re | 10:44 |
afazekas | raies: LGTM | 10:45 |
* afazekas waiting for jenkins | 10:45 | |
raies | ok | 10:47 |
raies | thanks for great input :) | 10:47 |
afazekas | sorry, the L81 and L82 two statement and should have 2 separated try: | 10:50 |
openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: port test_flavors into v3 part1 https://review.openstack.org/47811 | 10:50 |
openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: port test_flavors into v3 part2 https://review.openstack.org/47812 | 10:50 |
*** Madan1 has quit IRC | 10:54 | |
*** madan has quit IRC | 10:59 | |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add server resizing tests https://review.openstack.org/44732 | 11:02 |
*** SergeyLukjanov has joined #openstack-qa | 11:05 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests https://review.openstack.org/47348 | 11:05 |
*** dims has joined #openstack-qa | 11:12 | |
*** mkollaro has joined #openstack-qa | 11:15 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas vpnservice test cases https://review.openstack.org/47339 | 11:17 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests https://review.openstack.org/47348 | 11:18 |
*** toanster_ has quit IRC | 11:23 | |
tkammer | afazekas, I'm trying to figure out something regarding the way we work in tempest | 11:30 |
tkammer | afazekas, I see a lot of code that uses DEFAULTS from env variables | 11:30 |
afazekas | 'DEFAULTS' ?? | 11:31 |
tkammer | afazekas, I'm trying to figure out why we use this if we already provide CLI support | 11:31 |
tkammer | I mean os.environ | 11:31 |
tkammer | DEFAULTS = os.environ variables | 11:31 |
tkammer | to be able to change | 11:32 |
afazekas | TEMPEST_CONFIG_DIR+TEMPEST_CONFIG is the config file location | 11:32 |
afazekas | the others usually set by tox | 11:32 |
afazekas | check the tox.ini | 11:32 |
tkammer | yeah, why do we need it if we can simple send it as params? | 11:32 |
afazekas | we cannot use params in unittest | 11:33 |
tkammer | not in the unitest | 11:33 |
tkammer | I mean, for example, the auto configuration script | 11:33 |
tkammer | I'm writing | 11:33 |
tkammer | if I'm going to support CLI, I have no reason to implement the env variables correct? | 11:34 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests https://review.openstack.org/47816 | 11:38 |
afazekas | IMHO the environment variables which are default available in the openrc file should be recognizable as an environment variables (keystone credentials) | 11:38 |
afazekas | BTW: sooner or later the config script should be usable in this use case: https://etherpad.openstack.org/RefStackBlueprint (no admin credentials) | 11:40 |
*** hootjr has joined #openstack-qa | 11:40 | |
afazekas | http://<horizon_service>/dashboard/project/access_and_security/api_access/openrc/ basically it is the same as the keystonerc_foo created by packstack | 11:42 |
*** zhikunliu has quit IRC | 11:44 | |
*** weshay has joined #openstack-qa | 11:49 | |
tkammer | afazekas, I agree with you, I believe the only env variables needed are the official Openstack env variables, I just ask because I would like to refactor a little the TempestConfig and want to remove the "TEMPEST_CONFIG_DIR" and "TEMPEST_CONFIG" | 11:52 |
afazekas | tkammer: and what would like to use instead of them ? | 11:53 |
tkammer | afazekas, CLI args and the default oslo config features | 11:54 |
afazekas | so you wild like to remove them form the https://review.openstack.org/#/c/42920/6/tools/tempest_auto_config.py , and not from the https://github.com/openstack/tempest/blob/master/tempest/config.py#L658 | 11:56 |
tkammer | no, I would like to remove them completely, from my script *and* the config.py | 11:56 |
tkammer | basically, what tempest is doing is "double the work" | 11:57 |
afazekas | it will not work | 11:57 |
tkammer | the oslo config already has a default CLI option --config-dir and --config-file | 11:57 |
tkammer | there is no reason to have the env vars as well | 11:57 |
tkammer | afazekas, why wouldn't it work? | 11:57 |
afazekas | Is the oslo config processes them if they are an env var ? | 11:58 |
tkammer | I don't think so, haven't checked, buy why do we need them if we are running this anyway? | 11:59 |
afazekas | tempest running under a test runner. The only sane way to pass the config file location is the environment variables in this | 11:59 |
tkammer | but you are not passing the config file location, you are passing the config object... or am I missing something? | 12:00 |
afazekas | I am not aware of the any test runner which is passing CLI args to the unittest test cases | 12:00 |
tkammer | afazekas, wait a sec, do you want to tell me that tempest creates the configuration file for *each* test separately? | 12:02 |
tkammer | it doesn't make any sense... :-\ | 12:03 |
afazekas | If I not misunderstand you : you expect you can do the following nosetests --tempest-config /etc/tempest/system_one.config tempest.api | 12:04 |
tkammer | exactly | 12:05 |
afazekas | even if it working not recommended | 12:05 |
afazekas | nosetests: error: no such option: --tempest-config | 12:05 |
afazekas | the test runner will reject (or misunderstand ) the args before your code has the chance to read it | 12:06 |
*** bdperkin is now known as bdperkin_gone | 12:10 | |
tkammer | o.k, now I think I follow what you mean, I need to think about it | 12:11 |
afazekas | probbaly you can change it to use only one env variable instead of 2 | 12:12 |
afazekas | BTW: the whetbox tests are removed, so the db_url configuration is not required anymore | 12:13 |
*** raies_ has joined #openstack-qa | 12:23 | |
raies_ | afazekas: please have a look on https://review.openstack.org/#/c/47339/ , https://review.openstack.org/#/c/47348/ and https://review.openstack.org/#/c/47816/ | 12:25 |
*** rfolco has joined #openstack-qa | 12:27 | |
*** prekarat has quit IRC | 12:28 | |
*** thomasbiege has joined #openstack-qa | 12:31 | |
*** adalbas has joined #openstack-qa | 12:34 | |
*** thomasbiege has quit IRC | 12:35 | |
*** timello has quit IRC | 12:37 | |
*** timello has joined #openstack-qa | 12:39 | |
*** raies_ has quit IRC | 12:40 | |
openstackgerrit | Cyril Roelandt proposed a change to openstack/tempest: Do not review this https://review.openstack.org/47194 | 12:45 |
*** dims has quit IRC | 12:53 | |
*** dims has joined #openstack-qa | 12:54 | |
*** rustlebee is now known as russellb | 13:08 | |
*** mriedem has joined #openstack-qa | 13:17 | |
*** mriedem has quit IRC | 13:17 | |
*** mriedem has joined #openstack-qa | 13:18 | |
*** mriedem has quit IRC | 13:22 | |
*** mriedem has joined #openstack-qa | 13:23 | |
*** SergeyLukjanov has quit IRC | 13:29 | |
*** dkranz has joined #openstack-qa | 13:29 | |
*** malini_afk is now known as malini | 13:33 | |
openstackgerrit | DennyZhang proposed a change to openstack/tempest: Fix typos in tempest/HACKING.rst Edit https://review.openstack.org/47835 | 13:34 |
openstackgerrit | Cyril Roelandt proposed a change to openstack/tempest: Do not review this https://review.openstack.org/47194 | 13:34 |
*** SergeyLukjanov has joined #openstack-qa | 13:34 | |
openstackgerrit | Alex Rudenko proposed a change to openstack-dev/devstack: Additional variable APACHE_SERVER_NAME added https://review.openstack.org/44593 | 13:47 |
openstackgerrit | Kyle Mestery proposed a change to openstack-dev/devstack: Default to the ML2 plugin in Neutron instead of OVS https://review.openstack.org/47837 | 13:54 |
openstackgerrit | Cyril Roelandt proposed a change to openstack/tempest: Do not review this https://review.openstack.org/47194 | 13:57 |
openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: port test_flavor from v2 to v3 part 2 https://review.openstack.org/47839 | 14:00 |
openstackgerrit | minsel proposed a change to openstack/tempest: Handling network resources in tenant isolation https://review.openstack.org/45578 | 14:08 |
*** RecheckWatchBot has quit IRC | 14:08 | |
*** RecheckWatchBot has joined #openstack-qa | 14:09 | |
*** bashok has quit IRC | 14:12 | |
*** bashok_ has joined #openstack-qa | 14:13 | |
openstackgerrit | zhu zhu proposed a change to openstack/tempest: Add test for nova API of VM lock/unlock https://review.openstack.org/47846 | 14:16 |
*** bashok_ has quit IRC | 14:18 | |
*** bashok has joined #openstack-qa | 14:22 | |
RecheckWatchBot | openstack/heat change: https://review.openstack.org/47283 failed tempest because of: https://bugs.launchpad.net/bugs/1221987 | 14:25 |
RecheckWatchBot | openstack-dev/devstack change: https://review.openstack.org/47837 failed tempest because of: https://bugs.launchpad.net/bugs/1221987 | 14:25 |
*** bdperkin_gone is now known as bdperkin | 14:28 | |
*** dkranz has quit IRC | 14:31 | |
Anju | afazekas: ping | 14:42 |
*** dkranz has joined #openstack-qa | 14:43 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Do not merge test of neutron tenant isolation. https://review.openstack.org/46330 | 14:47 |
*** fbo is now known as fbo_away | 14:52 | |
RecheckWatchBot | openstack/tempest change: https://review.openstack.org/47846 failed tempest with an unrecognized error | 14:54 |
*** sthaha has quit IRC | 14:55 | |
RecheckWatchBot | openstack/heat change: https://review.openstack.org/47294 failed tempest because of: https://bugs.launchpad.net/bugs/1221987 | 15:06 |
mtreinish | jog0: ^^^ isn't that the bug you fixed by removing the ceilo thing from nova? | 15:08 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Update docs config https://review.openstack.org/46778 | 15:12 |
openstackgerrit | zhu zhu proposed a change to openstack/tempest: Add test for nova API of VM lock/unlock https://review.openstack.org/47846 | 15:13 |
RecheckWatchBot | openstack/ceilometer change: https://review.openstack.org/47142 failed tempest with an unrecognized error | 15:21 |
sdague | so where is the awesomness that is recheckwatchbot? | 15:23 |
mtreinish | sdague: https://github.com/jogo/elasticRecheck | 15:25 |
mtreinish | it's running on my laptop right now | 15:26 |
sdague | cool | 15:35 |
*** mlavalle has joined #openstack-qa | 15:37 | |
*** yassine has quit IRC | 15:47 | |
RecheckWatchBot | openstack/oslo.config change: https://review.openstack.org/47853 failed tempest because of: https://bugs.launchpad.net/bugs/1221987 | 15:57 |
mtreinish | ^^^ that's fun example of the ES 1/2 hr delay | 15:59 |
RecheckWatchBot | openstack/python-glanceclient change: https://review.openstack.org/42741 failed tempest because of: https://bugs.launchpad.net/bugs/1221987 | 16:00 |
sdague | mtreinish: yeh, well ES is only going to be that fast | 16:01 |
mtreinish | sdague: I get that, I was just commenting on that in that window the patch was approved and passed jenkins from a recheck | 16:02 |
mtreinish | so when you look at the gerrit comments it looks like the bot (or me) is commenting on a passing jenkins run | 16:02 |
sdague | yep | 16:03 |
RecheckWatchBot | openstack/tempest change: https://review.openstack.org/47846 failed tempest because of: https://bugs.launchpad.net/bugs/1211915 | 16:10 |
RecheckWatchBot | openstack-dev/devstack change: https://review.openstack.org/47837 failed tempest with an unrecognized error | 16:11 |
*** dims has quit IRC | 16:14 | |
*** tkammer has quit IRC | 16:19 | |
*** hootjr has quit IRC | 16:19 | |
RecheckWatchBot | openstack/heat change: https://review.openstack.org/47283 failed tempest because of: https://bugs.launchpad.net/bugs/1227915 | 16:25 |
openstackgerrit | A change was merged to openstack-dev/devstack: Remove change_apache_user_group function https://review.openstack.org/46857 | 16:30 |
RecheckWatchBot | openstack/nova change: https://review.openstack.org/43616 failed tempest with an unrecognized error | 16:36 |
*** afazekas has quit IRC | 16:37 | |
*** adam_g has quit IRC | 16:40 | |
RecheckWatchBot | openstack/heat change: https://review.openstack.org/47860 failed tempest with an unrecognized error | 16:41 |
RecheckWatchBot | openstack/neutron change: https://review.openstack.org/40296 failed tempest with an unrecognized error | 16:41 |
*** adam_g has joined #openstack-qa | 16:43 | |
*** adam_g has joined #openstack-qa | 16:43 | |
*** hemnafk is now known as hemna | 16:54 | |
*** SergeyLukjanov has quit IRC | 16:56 | |
RecheckWatchBot | openstack/cinder change: https://review.openstack.org/47864 failed tempest because of: https://bugs.launchpad.net/bugs/1221987 | 16:58 |
openstackgerrit | A change was merged to openstack-dev/devstack: Fix LDAP support for openSUSE https://review.openstack.org/46265 | 17:01 |
openstackgerrit | A change was merged to openstack-dev/devstack: xenapi: Use C locale https://review.openstack.org/47338 | 17:02 |
openstackgerrit | A change was merged to openstack-dev/devstack: docker: ensure socat is installed https://review.openstack.org/47031 | 17:02 |
openstackgerrit | DennyZhang proposed a change to openstack/tempest: README.rst of tempest stress is misleading https://review.openstack.org/47881 | 17:05 |
openstackgerrit | DennyZhang proposed a change to openstack/tempest: README.rst of tempest stress is misleading https://review.openstack.org/47883 | 17:10 |
*** mlavalle has quit IRC | 17:11 | |
openstackgerrit | DennyZhang proposed a change to openstack/tempest: Fix typos in tempest/HACKING.rst Edit https://review.openstack.org/47835 | 17:12 |
*** wramthun has quit IRC | 17:16 | |
RecheckWatchBot | openstack/python-ceilometerclient change: https://review.openstack.org/47331 failed tempest because of: https://bugs.launchpad.net/bugs/1221987 | 17:17 |
*** wramthun has joined #openstack-qa | 17:23 | |
*** SergeyLukjanov has joined #openstack-qa | 17:31 | |
*** giulivo has quit IRC | 17:39 | |
openstackgerrit | Anjan Tiwari proposed a change to openstack/tempest: port test_absolute_limits into v3 part1 https://review.openstack.org/47888 | 17:50 |
*** morganfainberg has quit IRC | 17:59 | |
*** morganfainberg has joined #openstack-qa | 18:02 | |
*** fbo_away is now known as fbo | 18:06 | |
*** morganfainberg has quit IRC | 18:06 | |
*** morganfainberg has joined #openstack-qa | 18:09 | |
RecheckWatchBot | openstack/heat change: https://review.openstack.org/47827 failed tempest because of: https://bugs.launchpad.net/bugs/1221987 | 18:11 |
*** mlavalle has joined #openstack-qa | 18:17 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: print out git references for trees https://review.openstack.org/47895 | 18:20 |
RecheckWatchBot | openstack/heat change: https://review.openstack.org/45462 failed tempest with an unrecognized error | 18:49 |
*** morganfainberg has quit IRC | 18:57 | |
*** morganfainberg has joined #openstack-qa | 18:57 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: XenServer hypervisor plugin https://review.openstack.org/47906 | 19:09 |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: fake hypervisor plugin https://review.openstack.org/47908 | 19:10 |
*** mlavalle has quit IRC | 19:40 | |
*** malini is now known as malini_afk | 19:44 | |
*** fbo is now known as fbo_away | 20:02 | |
*** rockyg has joined #openstack-qa | 20:05 | |
*** jlibosva has joined #openstack-qa | 20:10 | |
*** SergeyLukjanov has quit IRC | 20:30 | |
*** bashok has quit IRC | 20:34 | |
*** morganfainberg has quit IRC | 20:36 | |
*** morganfainberg has joined #openstack-qa | 20:37 | |
*** bdperkin is now known as bdperkin_gone | 20:37 | |
*** DennyZhang has joined #openstack-qa | 20:55 | |
*** dims has joined #openstack-qa | 20:55 | |
DennyZhang | Does anyone know about tempest stress: I've run a tempest stress, how can I get the performance report, like the error rate, througput, latency, etc? | 20:55 |
*** malini_afk is now known as malini | 21:04 | |
openstackgerrit | Doug Hellmann proposed a change to openstack-dev/devstack: Clean up automated changes to requirements https://review.openstack.org/45478 | 21:10 |
DennyZhang | dkranz: From the bp(https://blueprints.launchpad.net/tempest/+spec/stress-tests), I know you're working on tempest refactoring. How is everything going? Besides, would you please educate me how to get the performance report from tempest stress. | 21:12 |
dkranz | DennyZhang: Not sure what you mean by performance report. Most of the recent work on the stress tests was done by mkoderer_ | 21:14 |
openstackgerrit | Doug Hellmann proposed a change to openstack-dev/devstack: Clean up automated changes to requirements https://review.openstack.org/45478 | 21:14 |
DennyZhang | dkranz: thanks. I've run a tempest stress, how can I get the performance report, like the error rate, througput, latency, etc? | 21:15 |
dkranz | DennyZhang: These stress tests are not performance tests. They try to break the system by bashing it in various ways and report success or failure after some specified time. | 21:16 |
*** lifeless has quit IRC | 21:17 | |
DennyZhang | dkranz: I see, thanks. Then do you have any solution for performance tests of openstack? | 21:18 |
*** lifeless has joined #openstack-qa | 21:19 | |
DennyZhang | dkranz: I see, thanks. Then do you know any solution for performance tests of openstack? Like tsung, etc. | 21:19 |
dkranz | DennyZhang: Not really. A lot of companies do their own performance testing but I don't know of any open upstream performance test except for swiftbench which is just for swift. | 21:20 |
DennyZhang | dkranz: I'm new here. Do you happen to know there're any discussion about performance tests? Or where to find the community's opinion about this?what's the community | 21:22 |
dkranz | DennyZhang: Communities like OpenStack don't have opinions in general | 21:23 |
dkranz | DennyZhang: If you want to learn about the community I suggest starting at openstack.org | 21:24 |
dkranz | DennyZhang: For a specific question as to whether there are any community-developed performance tests for OpenStack, you could send a message to openstack-dev@lists.openstack.org | 21:25 |
*** lifeless has quit IRC | 21:25 | |
DennyZhang | dkranz: Thanks for the information. I'll send out the email, after some searching. | 21:26 |
*** jlibosva has quit IRC | 21:29 | |
dkranz | DennyZhang: Your welcome. | 21:31 |
*** mriedem has quit IRC | 21:40 | |
*** dkranz has quit IRC | 21:40 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Move LockFixture to tempest.common https://review.openstack.org/47932 | 21:46 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locking to test_list_hosts_with_zone https://review.openstack.org/47599 | 21:46 |
*** lifeless has joined #openstack-qa | 21:47 | |
*** DennyZhang has quit IRC | 21:47 | |
openstackgerrit | Edward Hope-Morley proposed a change to openstack/tempest: Add cinder-backup functional tests https://review.openstack.org/45465 | 21:49 |
*** weshay has quit IRC | 21:59 | |
*** rockyg has quit IRC | 22:07 | |
*** rockyg has joined #openstack-qa | 22:07 | |
*** rockyg has quit IRC | 22:09 | |
*** rockyg has joined #openstack-qa | 22:09 | |
*** rockyg has quit IRC | 22:10 | |
*** rockyg has joined #openstack-qa | 22:11 | |
*** dosaboy has quit IRC | 22:12 | |
*** dosaboy has joined #openstack-qa | 22:14 | |
*** RecheckWatchBot has quit IRC | 22:18 | |
*** RecheckWatchBot has joined #openstack-qa | 22:19 | |
openstackgerrit | Edward Hope-Morley proposed a change to openstack/tempest: Add cinder-backup functional tests https://review.openstack.org/45465 | 22:21 |
*** adalbas has quit IRC | 22:25 | |
RecheckWatchBot | openstack/nova change: https://review.openstack.org/45819 failed tempest with an unrecognized error | 22:28 |
RecheckWatchBot | openstack/nova change: https://review.openstack.org/47174 failed tempest with an unrecognized error | 22:28 |
RecheckWatchBot | openstack/nova change: https://review.openstack.org/46824 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 22:28 |
openstackgerrit | John Griffith proposed a change to openstack/tempest: Fix json version of bootable check https://review.openstack.org/47935 | 22:30 |
*** openstack has joined #openstack-qa | 22:31 | |
RecheckWatchBot | openstack/python-heatclient change: https://review.openstack.org/47922 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 22:41 |
*** dims has quit IRC | 22:43 | |
RecheckWatchBot | openstack/heat change: https://review.openstack.org/47772 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 22:46 |
*** dims has joined #openstack-qa | 23:00 | |
sdague | mtreinish: ok, I did the bulk update on bugs, hopefully it didn't screw anything up too much | 23:00 |
mtreinish | sdague: cool thanks | 23:00 |
mtreinish | let me take a look | 23:00 |
mtreinish | but I'm sure it's fine | 23:01 |
mtreinish | sdague: there are still ~45 bugs in fix committed state | 23:02 |
sdague | oh, ok, different flags this time | 23:04 |
sdague | now it's getting the rest | 23:04 |
mtreinish | sdague: ok thanks | 23:07 |
sdague | well, it got it down to 33 | 23:09 |
sdague | then ... it's crashing launchpad | 23:09 |
sdague | I'll just keep trying | 23:09 |
mtreinish | too many request? | 23:09 |
RecheckWatchBot | openstack/nova change: https://review.openstack.org/46253 failed tempest with an unrecognized error | 23:20 |
*** hemna is now known as hemnafk | 23:24 | |
RecheckWatchBot | openstack/tempest change: https://review.openstack.org/45465 failed tempest with an unrecognized error | 23:34 |
RecheckWatchBot | openstack/horizon change: https://review.openstack.org/31099 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 23:34 |
*** rfolco has quit IRC | 23:35 | |
*** ken1ohmichi has joined #openstack-qa | 23:38 | |
*** sthaha has joined #openstack-qa | 23:46 | |
*** sthaha has joined #openstack-qa | 23:46 | |
*** matsuhashi has joined #openstack-qa | 23:50 | |
*** malini is now known as malini_afk | 23:50 | |
*** dkranz has joined #openstack-qa | 23:54 | |
RecheckWatchBot | openstack/nova change: https://review.openstack.org/45571 failed tempest with an unrecognized error | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!