*** tosky has quit IRC | 00:09 | |
*** masayukig has quit IRC | 00:11 | |
*** matsuhashi has quit IRC | 00:13 | |
*** rbak has joined #openstack-qa | 00:13 | |
*** levi_b has joined #openstack-qa | 00:16 | |
*** nati_uen_ has quit IRC | 00:16 | |
*** nati_ueno has joined #openstack-qa | 00:17 | |
*** masayukig has joined #openstack-qa | 00:17 | |
*** TravT has quit IRC | 00:18 | |
*** levi_b has quit IRC | 00:20 | |
*** sthaha has quit IRC | 00:21 | |
*** nati_ueno has quit IRC | 00:21 | |
*** safchain_ has quit IRC | 00:22 | |
*** leseb has joined #openstack-qa | 00:22 | |
*** ekhugen has joined #openstack-qa | 00:25 | |
*** leseb has quit IRC | 00:27 | |
*** aimon has joined #openstack-qa | 00:29 | |
*** sthaha has joined #openstack-qa | 00:31 | |
*** sthaha has quit IRC | 00:31 | |
*** sthaha has joined #openstack-qa | 00:31 | |
*** nati_ueno has joined #openstack-qa | 00:32 | |
*** dmorita has joined #openstack-qa | 00:37 | |
*** rbak has quit IRC | 00:40 | |
openstackgerrit | Huruifeng proposed a change to openstack/tempest: Add tests for neutron firewall resource https://review.openstack.org/94590 | 00:42 |
---|---|---|
*** nati_ueno has quit IRC | 00:46 | |
*** nati_ueno has joined #openstack-qa | 00:47 | |
*** nati_ueno has quit IRC | 00:51 | |
*** jaypipes has joined #openstack-qa | 01:01 | |
*** ChanServ sets mode: +o jaypipes | 01:01 | |
*** GMann has quit IRC | 01:03 | |
*** oomichi has quit IRC | 01:04 | |
*** nati_ueno has joined #openstack-qa | 01:07 | |
*** rbak has joined #openstack-qa | 01:07 | |
*** jaypipes has quit IRC | 01:09 | |
*** nosnos has joined #openstack-qa | 01:09 | |
*** rbak has quit IRC | 01:11 | |
*** levi_b has joined #openstack-qa | 01:11 | |
*** rbak has joined #openstack-qa | 01:11 | |
*** levi_b has quit IRC | 01:15 | |
*** torandu has quit IRC | 01:15 | |
*** torandu has joined #openstack-qa | 01:16 | |
*** davlaps has joined #openstack-qa | 01:16 | |
*** arosen has quit IRC | 01:19 | |
*** leseb has joined #openstack-qa | 01:21 | |
*** arosen has joined #openstack-qa | 01:23 | |
*** nati_ueno has quit IRC | 01:23 | |
*** nati_ueno has joined #openstack-qa | 01:24 | |
*** nati_ueno has quit IRC | 01:24 | |
*** nati_ueno has joined #openstack-qa | 01:25 | |
*** nati_uen_ has joined #openstack-qa | 01:26 | |
*** leseb has quit IRC | 01:26 | |
*** nati_uen_ has quit IRC | 01:26 | |
*** nati_uen_ has joined #openstack-qa | 01:26 | |
*** nati_uen_ has quit IRC | 01:26 | |
*** nati_ueno has quit IRC | 01:29 | |
*** rbak has quit IRC | 01:29 | |
*** morganfainberg is now known as morganfainberg_Z | 01:41 | |
openstackgerrit | Huruifeng proposed a change to openstack/tempest: Add tests for neutron firewall resource https://review.openstack.org/94590 | 01:48 |
*** Sukhdev has joined #openstack-qa | 01:51 | |
*** dims has quit IRC | 01:59 | |
*** yamahata has joined #openstack-qa | 02:02 | |
*** Poornima has joined #openstack-qa | 02:09 | |
*** oomichi has joined #openstack-qa | 02:11 | |
*** malini_afk is now known as malini | 02:12 | |
*** GMann has joined #openstack-qa | 02:14 | |
*** davlaps has quit IRC | 02:16 | |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify list_instance_action attributes of Nova API https://review.openstack.org/85666 | 02:16 |
*** Sukhdev has quit IRC | 02:17 | |
*** ekhugen has quit IRC | 02:21 | |
*** leseb has joined #openstack-qa | 02:22 | |
*** leseb has quit IRC | 02:27 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/94812 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1273908, gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1323658, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254238 | 02:47 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/92519 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1254890, gate-tempest-dsvm-neutron: unrecognized error, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254238 | 02:48 |
openstackrecheck | openstack/nova change: https://review.openstack.org/97164 failed because of: gate-tempest-dsvm-neutron: unrecognized error, gate-nova-docs: https://bugs.launchpad.net/bugs/1270382 | 02:48 |
openstackrecheck | openstack/ceilometer change: https://review.openstack.org/96781 failed gate-ceilometer-python27 in the gate queue with an unrecognized error | 02:48 |
openstackrecheck | openstack/nova change: https://review.openstack.org/94713 failed gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 02:49 |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Adds test for list/get volume attachments V2 APIs https://review.openstack.org/87191 | 02:52 |
*** GMann has quit IRC | 03:02 | |
*** GMann has joined #openstack-qa | 03:03 | |
*** Longgeek has joined #openstack-qa | 03:13 | |
*** coolsvap|afk is now known as coolsvap | 03:13 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Fix the way Apache site configuration files are used, to improve OS portability https://review.openstack.org/95977 | 03:16 |
*** Longgeek has quit IRC | 03:17 | |
*** nosnos has quit IRC | 03:20 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/96293 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1254238, gate-tempest-dsvm-full: unrecognized error | 03:22 |
*** leseb has joined #openstack-qa | 03:23 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/76829 failed because of: gate-nova-python27: https://bugs.launchpad.net/bugs/1266611 | 03:23 |
*** sthaha has quit IRC | 03:23 | |
*** leseb has quit IRC | 03:24 | |
*** leseb has joined #openstack-qa | 03:25 | |
*** leseb has quit IRC | 03:29 | |
openstackgerrit | Sridhar Gaddam proposed a change to openstack-dev/devstack: Set default value for IPv6 subnet attribute flag https://review.openstack.org/97383 | 03:35 |
*** sthaha has joined #openstack-qa | 03:38 | |
*** levi_b has joined #openstack-qa | 03:38 | |
*** levi_b has quit IRC | 03:43 | |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Cleanup for local variable in test_attach_volume https://review.openstack.org/97385 | 03:50 |
*** nosnos has joined #openstack-qa | 04:00 | |
*** k4n0 has joined #openstack-qa | 04:11 | |
*** arosen has quit IRC | 04:12 | |
*** torandu has quit IRC | 04:12 | |
*** Longgeek has joined #openstack-qa | 04:14 | |
*** Longgeek has quit IRC | 04:18 | |
*** raies has joined #openstack-qa | 04:21 | |
raies | ylobankov: hi | 04:22 |
*** leseb has joined #openstack-qa | 04:26 | |
*** leseb has quit IRC | 04:30 | |
*** psharma has joined #openstack-qa | 04:32 | |
*** levi_b has joined #openstack-qa | 04:38 | |
raies | GMann: hi | 04:39 |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Allow to configure git base URL https://review.openstack.org/95901 | 04:39 |
*** raies has quit IRC | 04:41 | |
*** raies has joined #openstack-qa | 04:42 | |
*** levi_b has quit IRC | 04:42 | |
*** harlowja is now known as harlowja_away | 04:48 | |
*** Abhijeet has joined #openstack-qa | 04:52 | |
raies | ylobankov: hi | 04:56 |
openstackgerrit | ajay proposed a change to openstack/tempest: Added cinder cli tests https://review.openstack.org/96072 | 04:56 |
*** Madan has joined #openstack-qa | 05:02 | |
*** malini is now known as malini_afk | 05:07 | |
*** Longgeek has joined #openstack-qa | 05:14 | |
*** Longgeek has quit IRC | 05:19 | |
*** leseb has joined #openstack-qa | 05:26 | |
*** rdekel has joined #openstack-qa | 05:27 | |
*** arosen has joined #openstack-qa | 05:28 | |
*** arosen has quit IRC | 05:29 | |
*** leseb has quit IRC | 05:30 | |
*** jhenner has quit IRC | 05:35 | |
*** Madan has quit IRC | 05:35 | |
*** Madan has joined #openstack-qa | 05:39 | |
*** Madan has quit IRC | 05:43 | |
*** Madan has joined #openstack-qa | 05:46 | |
*** rdekel has quit IRC | 05:53 | |
*** coolsvap is now known as coolsvap|afk | 05:56 | |
*** coolsvap|afk is now known as coolsvap | 06:01 | |
*** Madan has quit IRC | 06:15 | |
*** Longgeek has joined #openstack-qa | 06:15 | |
*** ilyashakhat_ has quit IRC | 06:17 | |
openstackgerrit | lifeless proposed a change to openstack-infra/elastic-recheck: Add rule for bug 1325815 https://review.openstack.org/97409 | 06:19 |
*** Longgeek has quit IRC | 06:20 | |
openstackrecheck | openstack/ceilometer change: https://review.openstack.org/83369 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1323658 | 06:24 |
*** levi_b has joined #openstack-qa | 06:26 | |
*** Madan has joined #openstack-qa | 06:27 | |
*** Longgeek has joined #openstack-qa | 06:30 | |
*** levi_b has quit IRC | 06:31 | |
*** Longgeek has quit IRC | 06:31 | |
*** rdekel has joined #openstack-qa | 06:32 | |
*** luqas has joined #openstack-qa | 06:34 | |
*** nati_ueno has joined #openstack-qa | 06:38 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/elastic-recheck: Ignore non-voting jobs in gerrit https://review.openstack.org/97369 | 06:49 |
*** coolsvap is now known as coolsvap|afk | 06:57 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Compute "cells" basic API test https://review.openstack.org/90337 | 06:59 |
*** tkammer has joined #openstack-qa | 07:00 | |
*** Longgeek has joined #openstack-qa | 07:01 | |
*** coolsvap|afk is now known as coolsvap | 07:02 | |
*** ildikov has joined #openstack-qa | 07:03 | |
*** Longgeek has quit IRC | 07:07 | |
*** Longgeek has joined #openstack-qa | 07:08 | |
openstackgerrit | Longgeek proposed a change to openstack-dev/devstack: Add missing config option os_auth_url https://review.openstack.org/97107 | 07:12 |
ylobankov | raies: hi | 07:12 |
*** afazekas has joined #openstack-qa | 07:14 | |
openstackgerrit | Ian Wienand proposed a change to openstack-dev/devstack: Add basic echo_* functions for grenade https://review.openstack.org/97418 | 07:17 |
raies | ylobankov: please check http://pastebin.com/iKRkYBuc regarding https://review.openstack.org/#/c/47816/ I think it will do well | 07:17 |
*** dkorn has joined #openstack-qa | 07:18 | |
*** Albert__13 has joined #openstack-qa | 07:18 | |
ylobankov | raies: OK. I will take a look at it right now | 07:18 |
eezhova | @afazekas: Hi! Could you please take a look at my comment regarding https://review.openstack.org/#/c/88579/10 ? | 07:19 |
raies | ylobankov: LOG.exception('Exception raised deleting policy %s', ipsec_policy_id) | 07:20 |
raies | cell was written by mistake | 07:21 |
*** dkranz has quit IRC | 07:21 | |
afazekas | eezhova: no, sorry I was blind | 07:21 |
*** andreaf has joined #openstack-qa | 07:22 | |
eezhova | afazekas: So is it ready to merge now or it needs some additional testing? | 07:23 |
raies | ylobankov: have u checked ?? | 07:23 |
ylobankov | raies: which cell? | 07:23 |
eezhova | afazekas: thanks | 07:24 |
raies | in http://pastebin.com/iKRkYBuc in line 6 | 07:24 |
raies | ylobankov: ^^ | 07:24 |
*** salv-orlando has joined #openstack-qa | 07:26 | |
*** andreaf has quit IRC | 07:26 | |
ylobankov | raies: ah, got it | 07:26 |
raies | ylobankov: I think this code will be fine against your comment in https://review.openstack.org/#/c/47816/ what you say ?? | 07:27 |
ylobankov | raies: I suspect that you don't try/except block | 07:27 |
ylobankov | raies: I suspect that you don't need try/except block | 07:28 |
ylobankov | raies: You create policy and then you just add policy to self.addCleanup method. Why do you need try/except block? | 07:30 |
raies | as per Jordan Pittiersuggestion | 07:31 |
raies | ylobankov: try delete, if exception is not found make it pass and if any other reason is there then log that thats seem to be good for me what you say | 07:32 |
raies | firstly I had used list/filter methos | 07:32 |
raies | **method | 07:32 |
raies | ylobankov: what you say ?? | 07:36 |
ylobankov | raies: You create policy and immediately add policy to self.addCleanup method to delete. Why do you need to catch NotFound exception if you created policy a moment go? | 07:38 |
ylobankov | raies: *ago | 07:38 |
ylobankov | raies: I will write comment to your patch | 07:39 |
raies | actually test is like this create --> update --->> delete | 07:40 |
raies | ylobankov: I have removed delete but that should not be removed from test | 07:40 |
raies | ylobankov: so if policy is delete in the test phase then there is no need to delete from addCleanup and it will throw nowfound exception | 07:41 |
raies | so that is handled | 07:41 |
ylobankov | raies: Yes, but you don't delete policy in the test phase :) You delete policy trough addCleanup | 07:43 |
raies | ylobankov: as delete is part of test so it is being deleted in test | 07:44 |
raies | ylobankov: I think same flow is follwed by all other | 07:44 |
raies | ylobankov: cleanup does not indicate any test coverage, that why delete should be put in test. | 07:46 |
raies | ylobankov: if we are talking that this test is create_update_delete then I think delete should also go in with test method | 07:49 |
ylobankov | raies: Yes, I think deleting should be in test. Could you please submit new patch set? | 07:49 |
raies | ylobankov: thanks :) | 07:50 |
*** ildikov has quit IRC | 07:51 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests https://review.openstack.org/47816 | 07:51 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests https://review.openstack.org/47816 | 07:53 |
*** jpich has joined #openstack-qa | 07:53 | |
*** jordanP has joined #openstack-qa | 07:53 | |
*** jlibosva has joined #openstack-qa | 07:56 | |
*** salv-orlando has quit IRC | 08:02 | |
*** safchain has joined #openstack-qa | 08:05 | |
*** jlibosva has quit IRC | 08:08 | |
*** jlibosva has joined #openstack-qa | 08:10 | |
*** levi_b has joined #openstack-qa | 08:14 | |
*** jlibosva has quit IRC | 08:15 | |
*** jlibosva has joined #openstack-qa | 08:15 | |
*** yrabl has joined #openstack-qa | 08:17 | |
*** levi_b has quit IRC | 08:19 | |
*** nati_ueno has quit IRC | 08:21 | |
*** cdent has joined #openstack-qa | 08:22 | |
*** SridharG has joined #openstack-qa | 08:24 | |
*** coolsvap is now known as coolsvap|afk | 08:24 | |
*** fbo_away is now known as fbo | 08:25 | |
*** luqas has quit IRC | 08:26 | |
*** andreaf has joined #openstack-qa | 08:28 | |
*** oomichi has quit IRC | 08:29 | |
*** mpavlase has joined #openstack-qa | 08:32 | |
*** ildikov has joined #openstack-qa | 08:33 | |
*** oomichi has joined #openstack-qa | 08:35 | |
*** cdent_ has joined #openstack-qa | 08:42 | |
*** cdent has quit IRC | 08:43 | |
*** cdent_ is now known as cdent | 08:43 | |
*** coolsvap|afk is now known as coolsvap | 08:46 | |
raies | afazekas: hi can you please have a look on https://review.openstack.org/#/c/89472/ | 08:47 |
*** tteggel has joined #openstack-qa | 08:54 | |
*** luqas has joined #openstack-qa | 08:56 | |
*** salv-orlando has joined #openstack-qa | 08:58 | |
*** vrovachev has joined #openstack-qa | 09:03 | |
*** cdent has quit IRC | 09:03 | |
*** cdent has joined #openstack-qa | 09:05 | |
*** cdent has quit IRC | 09:10 | |
*** cdent has joined #openstack-qa | 09:11 | |
*** cdent has quit IRC | 09:11 | |
*** cdent has joined #openstack-qa | 09:12 | |
*** cdent_ has joined #openstack-qa | 09:13 | |
afazekas | raies: can you ping dwalleck about this ? | 09:15 |
*** cdent has quit IRC | 09:17 | |
*** cdent_ is now known as cdent | 09:17 | |
raies | afazekas: I just wanted your view on his comment | 09:17 |
afazekas | raies: both way works for me. | 09:18 |
raies | So if you will give your opinion there then it will make an impact | 09:18 |
raies | please give you opinion there on the patch so that he could evaluate his mark | 09:21 |
*** salv-orlando has quit IRC | 09:22 | |
*** jhenner has joined #openstack-qa | 09:30 | |
*** cdent_ has joined #openstack-qa | 09:32 | |
*** gszasz has joined #openstack-qa | 09:33 | |
*** cdent has quit IRC | 09:35 | |
*** cdent_ is now known as cdent | 09:35 | |
*** gszasz is now known as gszasz_lunch | 09:35 | |
openstackrecheck | openstack/heat change: https://review.openstack.org/92498 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1262566 https://bugs.launchpad.net/bugs/1254890 | 09:41 |
openstackgerrit | Vadim Rovachev proposed a change to openstack-dev/devstack: Add flag for ceilometer tests https://review.openstack.org/96484 | 09:41 |
GMann | hi cyeoh | 09:43 |
GMann | cyeoh: regarding https://review.openstack.org/#/c/95409/1//COMMIT_MSG | 09:43 |
GMann | cyeoh: Please check my reply in the review. and please let me kow your opinion | 09:44 |
openstackgerrit | Vadim Rovachev proposed a change to openstack-dev/devstack: Add flag for ceilometer tests https://review.openstack.org/96484 | 09:44 |
*** sthaha has quit IRC | 09:46 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/grenade: first pass of integrating javelin2 to grenade https://review.openstack.org/97317 | 09:47 |
*** giulivo has joined #openstack-qa | 09:49 | |
openstackgerrit | Vadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer https://review.openstack.org/64136 | 09:50 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests https://review.openstack.org/47816 | 09:52 |
*** amotoki has joined #openstack-qa | 09:57 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: make mysql run in strict mode https://review.openstack.org/97442 | 09:58 |
*** levi_b has joined #openstack-qa | 10:02 | |
*** gabriel-bezerra has quit IRC | 10:07 | |
*** levi_b has quit IRC | 10:07 | |
*** samuelmz has quit IRC | 10:07 | |
*** rodrigods has quit IRC | 10:07 | |
*** gabriel-bezerra has joined #openstack-qa | 10:09 | |
*** samuelmz has joined #openstack-qa | 10:09 | |
*** evgenyf has joined #openstack-qa | 10:10 | |
*** rodrigods has joined #openstack-qa | 10:10 | |
*** tosky has joined #openstack-qa | 10:20 | |
*** coolsvap is now known as coolsvap|afk | 10:24 | |
*** Longgeek_ has joined #openstack-qa | 10:27 | |
*** Longgeek has quit IRC | 10:30 | |
raies | jordanP: hi | 10:32 |
jordanP | raies, hi | 10:32 |
raies | regarding your comment in https://review.openstack.org/#/c/47816/28/tempest/api/network/test_vpnaas_extensions.py #L84 | 10:32 |
raies | please do it a bit clear | 10:33 |
jordanP | raies, in Tempest and specifically in cleanup functions we don't catch exceptions like this | 10:33 |
jordanP | let just the exception propagates | 10:34 |
jordanP | just catch the exception you "expect" and the exceptions that are not harmful | 10:34 |
raies | jordanP: ok | 10:35 |
*** rdekel has quit IRC | 10:35 | |
raies | jordanP: but in my case if cleanup is failed | 10:35 |
raies | i.e. delete is failed to some problem them how to show the exception ? | 10:36 |
raies | then should not be use except Exception: | 10:36 |
raies | The link you gave I read f you really need to use a try block, please ensure the original exception at least logged. When the exception is logged you usually need to raise the same or a different exception anyway. | 10:36 |
jordanP | if delete fails (and the exception is not exceptions.NotFound) then something wrong happen and it would need further investigation, so it's good to let the exception propagates so that the caller is informed | 10:37 |
raies | ok | 10:38 |
raies | so in my case http://pastebin.com/VS8gS3Dd this piece of code is sufficient right ? | 10:40 |
raies | jordanP: ^^ | 10:40 |
jordanP | exactly | 10:40 |
raies | jordanP: ok thanks :) | 10:40 |
jordanP | np | 10:40 |
*** evgenyf has quit IRC | 10:44 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests https://review.openstack.org/47816 | 10:45 |
*** yamahata has quit IRC | 10:52 | |
*** gszasz_lunch is now known as gszasz | 10:54 | |
*** tkammer has quit IRC | 10:57 | |
*** salv-orlando has joined #openstack-qa | 10:58 | |
*** k4n0 has quit IRC | 11:02 | |
*** mkollaro has joined #openstack-qa | 11:09 | |
*** mkollaro has quit IRC | 11:10 | |
*** mkollaro has joined #openstack-qa | 11:11 | |
raies | smurashov: hi | 11:11 |
smurashov | raies: hi | 11:11 |
raies | smurashov: as per your comment in https://review.openstack.org/#/c/86002/8/tempest/api/baremetal/test_nodestates.py | 11:11 |
raies | break should be replaced with return | 11:12 |
smurashov | raies: yes, can i help? | 11:12 |
raies | in this case if raise_error: message will go in testcase itself | 11:12 |
raies | what you say I wthink that will not be good | 11:12 |
raies | smurashov: ye ofcource | 11:13 |
raies | **yes of course | 11:13 |
*** dims_ has joined #openstack-qa | 11:14 | |
*** Madan has quit IRC | 11:14 | |
raies | smurashov: u there ?? | 11:14 |
smurashov | raies: yes, why do you think it not be good? | 11:15 |
raies | smurashov: got your point please check it in next patch :) | 11:16 |
raies | thanks | 11:16 |
smurashov | raies: ok | 11:16 |
smurashov | :) | 11:16 |
raies | smurashov: still please check http://pastebin.com/AfmT271d for validation | 11:17 |
smurashov | raies: it's seems good:) | 11:18 |
smurashov | raies: but why do you return False? | 11:18 |
smurashov | raies: may be would be better return None? or you expect False? | 11:18 |
raies | smurashov: yes this was good approach thanks :) | 11:18 |
smurashov | raies: :) | 11:18 |
*** Madan has joined #openstack-qa | 11:19 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Nodestate API test for power state set https://review.openstack.org/86002 | 11:19 |
openstackrecheck | openstack/oslo.db change: https://review.openstack.org/94552 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 11:20 |
*** sthaha has joined #openstack-qa | 11:21 | |
*** dmorita has quit IRC | 11:21 | |
*** luqas has quit IRC | 11:22 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Nodestate API test for power state set https://review.openstack.org/86002 | 11:23 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Baremetal nodestate APIs https://review.openstack.org/89472 | 11:23 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Nodestate API test for power state set https://review.openstack.org/86002 | 11:31 |
*** Abhijeet has quit IRC | 11:32 | |
*** Madan has quit IRC | 11:33 | |
*** matsuhashi has joined #openstack-qa | 11:34 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Baremetal nodestate APIs https://review.openstack.org/89472 | 11:34 |
*** Poornima has quit IRC | 11:37 | |
*** safchain has quit IRC | 11:37 | |
*** yamahata has joined #openstack-qa | 11:38 | |
*** jordanP has quit IRC | 11:45 | |
*** rbak has joined #openstack-qa | 11:47 | |
*** weshay has joined #openstack-qa | 11:49 | |
*** gszasz has quit IRC | 11:50 | |
*** rbak has quit IRC | 11:50 | |
*** levi_b has joined #openstack-qa | 11:50 | |
*** mwagner_lap has quit IRC | 11:53 | |
*** safchain has joined #openstack-qa | 11:54 | |
*** levi_b has quit IRC | 11:55 | |
*** jordanP has joined #openstack-qa | 12:01 | |
*** jhenner1 has joined #openstack-qa | 12:01 | |
*** mpavlase1 has joined #openstack-qa | 12:01 | |
*** dustins has joined #openstack-qa | 12:02 | |
*** mkollaro has quit IRC | 12:02 | |
*** jhenner has quit IRC | 12:02 | |
*** mpavlase has quit IRC | 12:03 | |
*** mkollaro has joined #openstack-qa | 12:03 | |
*** ildikov_ has joined #openstack-qa | 12:05 | |
*** amotoki has quit IRC | 12:05 | |
*** ildikov has quit IRC | 12:05 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Ignore non-voting jobs in gerrit https://review.openstack.org/97369 | 12:13 |
*** evgenyf has joined #openstack-qa | 12:14 | |
*** SridharG has quit IRC | 12:15 | |
*** levi_b has joined #openstack-qa | 12:16 | |
*** SridharG has joined #openstack-qa | 12:17 | |
*** adalbas has joined #openstack-qa | 12:19 | |
*** openstackrecheck has quit IRC | 12:23 | |
*** openstackrecheck has joined #openstack-qa | 12:23 | |
openstackgerrit | A change was merged to openstack/tempest: Upload public image requires admin role by default https://review.openstack.org/94435 | 12:25 |
*** psharma has quit IRC | 12:30 | |
*** dims_ has quit IRC | 12:31 | |
*** dims_ has joined #openstack-qa | 12:32 | |
*** oomichi has quit IRC | 12:32 | |
*** weshay has quit IRC | 12:36 | |
*** weshay has joined #openstack-qa | 12:37 | |
*** aysyd has joined #openstack-qa | 12:38 | |
*** bauzas has quit IRC | 12:38 | |
*** mpavlase1 has quit IRC | 12:39 | |
*** mkollaro has quit IRC | 12:41 | |
*** mkollaro1 has joined #openstack-qa | 12:41 | |
*** jhenner1 has quit IRC | 12:43 | |
*** mpavlase has joined #openstack-qa | 12:43 | |
*** salv-orlando has quit IRC | 12:43 | |
*** cdent has quit IRC | 12:44 | |
*** krtaylor has quit IRC | 12:45 | |
*** Longgeek_ has quit IRC | 12:48 | |
*** luqas has joined #openstack-qa | 12:49 | |
*** salv-orlando has joined #openstack-qa | 12:50 | |
*** ekhugen has joined #openstack-qa | 12:51 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/96829 | 12:52 |
*** gszasz has joined #openstack-qa | 12:52 | |
*** salv-orlando has quit IRC | 12:53 | |
*** salv-orlando has joined #openstack-qa | 12:55 | |
*** bauzas has joined #openstack-qa | 12:55 | |
*** GMann has quit IRC | 12:58 | |
*** jchai has joined #openstack-qa | 12:59 | |
*** jhenner has joined #openstack-qa | 12:59 | |
*** mriedem has joined #openstack-qa | 13:03 | |
*** jaypipes has joined #openstack-qa | 13:03 | |
*** ChanServ sets mode: +o jaypipes | 13:03 | |
*** ildikov_ has quit IRC | 13:05 | |
*** openstackgerrit has quit IRC | 13:06 | |
*** openstackgerrit has joined #openstack-qa | 13:07 | |
*** ildikov has joined #openstack-qa | 13:08 | |
*** rbak has joined #openstack-qa | 13:08 | |
*** matsuhashi has quit IRC | 13:12 | |
*** bauzas has quit IRC | 13:12 | |
*** nosnos has quit IRC | 13:12 | |
*** jchai is now known as jchai_afk | 13:14 | |
openstackgerrit | Ravikumar Venkatesan proposed a change to openstack/tempest: Test cases for database limits https://review.openstack.org/85320 | 13:15 |
*** jchai_afk is now known as jchai | 13:16 | |
*** bauzas has joined #openstack-qa | 13:25 | |
*** asvechnikov has joined #openstack-qa | 13:26 | |
*** malini_afk is now known as malini | 13:26 | |
*** nati_ueno has joined #openstack-qa | 13:29 | |
*** malini is now known as malini_afk | 13:29 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Wait for server activations for the updated servers https://review.openstack.org/97496 | 13:30 |
openstackgerrit | ajay proposed a change to openstack/tempest: Added nova cli test https://review.openstack.org/97497 | 13:31 |
*** malini_afk is now known as malini | 13:31 | |
*** Obulpathi has joined #openstack-qa | 13:32 | |
*** Obulpathi has joined #openstack-qa | 13:32 | |
*** jgrimm has joined #openstack-qa | 13:33 | |
andreaf | mtreinish: ping | 13:35 |
mtreinish | andreaf: pong | 13:35 |
andreaf | re https://blueprints.launchpad.net/tempest/+spec/cross-domain-testing - any reason for it being superseded rather than review? | 13:36 |
mtreinish | oops, clicked the wrong one | 13:36 |
mtreinish | let me fix that | 13:36 |
mtreinish | sry | 13:36 |
andreaf | mtreinish: np | 13:37 |
mtreinish | it was muscle memory after doing so many superseeded in a row | 13:37 |
andreaf | mtreinish: with storyboard hopefully we'll be able to automate more this kind of tasks ^_^ | 13:38 |
mtreinish | andreaf: yeah hopefully, the lp api around bps kind of sucks, otherwise I would have written a script to do this | 13:39 |
*** pafuent has joined #openstack-qa | 13:39 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Wait for server activations for the updated servers https://review.openstack.org/97496 | 13:39 |
mtreinish | SlickNik, GMann: for https://review.openstack.org/#/c/95639/ which bp should I leave open for it? https://blueprints.launchpad.net/tempest/+spec/tempest-guest-tests or https://blueprints.launchpad.net/tempest/+spec/trove-api-test-coverage-tempest ? | 13:39 |
mtreinish | both list the spec review | 13:40 |
pafuent | Hi. I'm from Blazar, and I'm interested in running our Tempest test in our gate job. I will like to know which is the recommended way to achieve this goal. | 13:40 |
*** oomichi has joined #openstack-qa | 13:44 | |
*** oomichi has quit IRC | 13:47 | |
*** rbak has quit IRC | 13:53 | |
mkoderer | pafuent: do you mean something like this: http://www.joinfu.com/2014/02/setting-up-an-external-openstack-testing-system/ ? | 14:01 |
pafuent | mkoderer: No. Just to run the Tempest test in our contrib directory inside a devstack jenkins gate job | 14:02 |
pafuent | pafuent: We do locally copying our files inside tempest and adding an entry to tox.ini | 14:02 |
pafuent | mkoderer: The idea is to get this running for each commit | 14:03 |
mkoderer | pafuent: triggered by gerrit? | 14:03 |
pafuent | mkoderer: Yes} | 14:03 |
pafuent | mkoderer: For devstack we copy some files of our contrib dir and devstack starts blazar as a service | 14:04 |
pafuent | mkoderer: There is something similar for Tempest? In other words which is the recommended way to do this. | 14:05 |
*** GMann has joined #openstack-qa | 14:05 | |
mkoderer | pafuent: is it review.openstack.org or an in-house gerrit installation? | 14:06 |
pafuent | mkoderer: Is review.openstack.org | 14:06 |
mkoderer | pafuent: so IMHO jay's blog post can be the starting point | 14:07 |
pafuent | mkoderer: OK. I'll check that blog post deeper. Thanks. | 14:07 |
mkoderer | I never heard about blazar btw | 14:07 |
pafuent | mkoderer: Is Climate | 14:08 |
mkoderer | k | 14:08 |
pafuent | mkoderer: We change the name a few weeks ago (during the gerrit downtime) | 14:08 |
*** adalbas has quit IRC | 14:08 | |
jordanP | mtreinish, sdague hi. The backport for nova/havana was merged yesterday (related to instance shelving). I'd like to retrigger jenkins for my tempest patch here : https://review.openstack.org/#/c/96165/ "recheck no bug" doesn't work anymore, what's the best way, for my information, to retrigger jenkins ? | 14:09 |
mtreinish | jordanP: just use the bug number for the shelving bug | 14:10 |
mtreinish | because that was the reason you need to recheck | 14:10 |
jordanP | cool thanks | 14:10 |
jordanP | I forgot the bug number didn't have to be a bug in the tempest project | 14:10 |
*** adalbas has joined #openstack-qa | 14:10 | |
*** kgriffs|afk is now known as kgriffs | 14:11 | |
mkoderer | pafuent: ahh so it's about a stackforge project? | 14:11 |
*** levi_b has quit IRC | 14:11 | |
pafuent | mkoderer: Yes | 14:12 |
*** GMann has quit IRC | 14:12 | |
mkoderer | pafuent: ok forget what I said :) I misread it | 14:12 |
pafuent | mkoderer: No problem | 14:13 |
mkoderer | pafuent: I think the infra ppl can help you with that. AFAIK you need to define the needed jobs in the jobs builder | 14:15 |
mkoderer | pafuent: http://ci.openstack.org/jjb.html#jjb | 14:15 |
mkoderer | but maybe I still didn't get the question :) | 14:15 |
pafuent | mkoderer: I think you got the idea. I'll chat with the infra team. Thanks. | 14:16 |
*** mwagner_lap has joined #openstack-qa | 14:21 | |
openstackgerrit | A change was merged to openstack/tempest: Relax security group rules cleanup https://review.openstack.org/88317 | 14:25 |
openstackrecheck | openstack/trove change: https://review.openstack.org/97087 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 14:25 |
*** dkorn has quit IRC | 14:26 | |
*** andreaf has quit IRC | 14:29 | |
*** nati_ueno has quit IRC | 14:29 | |
*** TravT has joined #openstack-qa | 14:39 | |
*** rodrigods has quit IRC | 14:47 | |
*** dims_ has quit IRC | 14:49 | |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Enable horizon on Fedoras https://review.openstack.org/97532 | 15:03 |
afazekas | sdague: https://review.openstack.org/97532 | 15:03 |
*** safchain has quit IRC | 15:03 | |
*** safchain has joined #openstack-qa | 15:04 | |
afazekas | chmouel: ^^ | 15:05 |
*** mlavalle has joined #openstack-qa | 15:09 | |
*** rbak has joined #openstack-qa | 15:12 | |
*** jchai is now known as jchai_afk | 15:13 | |
* afazekas grr.., I would't be surprised if another team request a gerrit service account soon.. | 15:14 | |
sdague | afazekas: +2 | 15:15 |
afazekas | sdague: thx | 15:15 |
*** afazekas has quit IRC | 15:20 | |
*** jchai_afk is now known as jchai | 15:21 | |
*** vrovachev has quit IRC | 15:22 | |
*** davlaps has joined #openstack-qa | 15:23 | |
*** malini is now known as malini_afk | 15:30 | |
openstackgerrit | A change was merged to openstack/tempest: Switch back to nc in test_load_balancer_basic https://review.openstack.org/88579 | 15:31 |
chmouel | i don't think "recheck bug 1" is a good thing to do in review https://review.openstack.org/97532 | 15:33 |
*** tmcpeak has joined #openstack-qa | 15:35 | |
*** morganfainberg_Z is now known as morganfainberg | 15:43 | |
mtreinish | mlavalle: which bp is attached to the spec in https://review.openstack.org/95600 ? https://blueprints.launchpad.net/tempest/+spec/neutron-advanced-scenarios or https://blueprints.launchpad.net/tempest/+spec/neutron-scenario-tests ? | 15:45 |
*** andreaf has joined #openstack-qa | 15:46 | |
*** nati_ueno has joined #openstack-qa | 15:47 | |
mlavalle | mtreinish: the good one is https://blueprints.launchpad.net/tempest/+spec/neutron-advanced-scenarios | 15:50 |
mtreinish | mlavalle: ok then I'll close out neutron-scenario tests | 15:50 |
mlavalle | Yes, please | 15:50 |
*** evgenyf has quit IRC | 15:50 | |
mlavalle | thanks :-) | 15:51 |
*** jlibosva has quit IRC | 15:52 | |
*** nati_uen_ has joined #openstack-qa | 15:52 | |
*** nati_ue__ has joined #openstack-qa | 15:53 | |
sdague | ianw: what's up with the check experimental from Red Hat CI? | 15:53 |
*** nati_ue__ has quit IRC | 15:53 | |
*** nati_ue__ has joined #openstack-qa | 15:54 | |
*** nati_ueno has quit IRC | 15:55 | |
*** nati_uen_ has quit IRC | 15:57 | |
*** gszasz has quit IRC | 15:57 | |
*** amotoki has joined #openstack-qa | 16:00 | |
*** torandu has joined #openstack-qa | 16:01 | |
*** jhenner1 has joined #openstack-qa | 16:01 | |
*** afazekas has joined #openstack-qa | 16:04 | |
*** jhenner has quit IRC | 16:04 | |
*** torandu has quit IRC | 16:05 | |
*** torandu has joined #openstack-qa | 16:06 | |
SlickNik | mtreinish / gmann: The one at https://blueprints.launchpad.net/tempest/+spec/tempest-guest-tests seems to have more information in it. Let's use that one. | 16:08 |
asselin_ | mtreinish, what does supeseded mean here? https://blueprints.launchpad.net/tempest/+spec/stress-api-tracking | 16:09 |
mtreinish | asselin_: see: http://lists.openstack.org/pipermail/openstack-dev/2014-May/035714.html | 16:09 |
mtreinish | asselin_: it's just a status to mark it as closed it was that or obsolete to get it off the list | 16:10 |
*** aimon has quit IRC | 16:10 | |
asselin_ | mtreinish, ok thanks will follow the new process | 16:10 |
mtreinish | SlickNik: that works for me I'll close the other one | 16:11 |
SlickNik | mtreinish: Sounds good. Thanks! | 16:11 |
*** jordanP has quit IRC | 16:12 | |
*** mkollaro1 has quit IRC | 16:13 | |
*** jhenner has joined #openstack-qa | 16:13 | |
*** yfried has quit IRC | 16:15 | |
*** dims_ has joined #openstack-qa | 16:16 | |
*** yamahata has quit IRC | 16:16 | |
*** jhenner1 has quit IRC | 16:17 | |
tmcpeak | hey, what's the best way to run a certain subset of tests | 16:18 |
tmcpeak | I know I can use the regex | 16:18 |
tmcpeak | is there anything more selective than that? | 16:18 |
tmcpeak | for example, if I want to run some API tests, but exclude a few | 16:19 |
*** dims_ has quit IRC | 16:21 | |
mtreinish | tmcpeak: with testr its only a regex right now | 16:21 |
mtreinish | you can do that with a regex it's just not that pretty | 16:22 |
mtreinish | look at the tox.ini for examples | 16:22 |
*** markmcclain has joined #openstack-qa | 16:24 | |
*** SridharG has quit IRC | 16:25 | |
*** cdent has joined #openstack-qa | 16:31 | |
Shrews | sdague: re your comment on 94439... i'm pretty new to tempest. care to give me a pointer to the regex i need to change? | 16:33 |
sdague | Shrews: sure, so the ironic tempest job doesn't use a normal tempest target | 16:33 |
tmcpeak | mtreinish: back | 16:34 |
tmcpeak | mtreinish: ok cool | 16:34 |
tmcpeak | mtreinish: thank you | 16:35 |
sdague | Shrews: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/devstack-gate.yaml#L587-L602 | 16:35 |
sdague | which means adding tests doesn't mean they run | 16:35 |
sdague | so that regex is going to need to be expanded | 16:36 |
Shrews | sdague: ah, now i see. thanks for the review and learnin' me good :) | 16:36 |
sdague | it would also be interesting to understand if that's still needed | 16:37 |
sdague | it seemed like a lot more of this is working now | 16:37 |
Shrews | sdague: hrm, yeah. looks like that needs some review now. it skips the scenario tests, but then later adds a scenario test. weird | 16:39 |
*** aimon has joined #openstack-qa | 16:44 | |
*** harlowja_away is now known as harlowja | 16:45 | |
*** ylobankov has quit IRC | 16:45 | |
*** nati_ue__ has quit IRC | 16:46 | |
*** Albert__13 has quit IRC | 17:00 | |
*** fbo is now known as fbo_away | 17:02 | |
*** jpich has quit IRC | 17:04 | |
*** ashishsnigam has joined #openstack-qa | 17:13 | |
*** luqas has quit IRC | 17:14 | |
ashishsnigam | Need some core reviews on https://review.openstack.org/#/c/94599/ | 17:14 |
*** kgriffs is now known as kgriffs|afk | 17:15 | |
*** Obulpathi has quit IRC | 17:15 | |
*** Obulpathi has joined #openstack-qa | 17:17 | |
*** dims_ has joined #openstack-qa | 17:17 | |
anteaya | sdague: are you going to be available for the -infra meeting today? | 17:20 |
anteaya | I am noticing there appears to be a conflict with that and picking up your vegetable basket | 17:20 |
*** Obulpathi has quit IRC | 17:20 | |
*** cdent_ has joined #openstack-qa | 17:21 | |
anteaya | I have an item on the agenda to discuss consistency when reviewing acl file changes | 17:21 |
*** Obulpathi has joined #openstack-qa | 17:21 | |
*** markmcclain has quit IRC | 17:21 | |
*** cdent has quit IRC | 17:21 | |
*** cdent_ is now known as cdent | 17:21 | |
*** giulivo has quit IRC | 17:26 | |
*** levi_b has joined #openstack-qa | 17:28 | |
*** masayukig has quit IRC | 17:29 | |
jogo | whoa the gate queue is 49 deep and less then two hours | 17:35 |
jogo | oh never mind | 17:35 |
*** masayukig has joined #openstack-qa | 17:36 | |
*** nati_ueno has joined #openstack-qa | 17:36 | |
*** mmaglana has joined #openstack-qa | 17:41 | |
*** kgriffs|afk is now known as kgriffs | 17:45 | |
*** safchain has quit IRC | 17:46 | |
anteaya | jogo: :D | 17:46 |
anteaya | jogo: do you want to know the history of why that is? | 17:47 |
jogo | anteaya: I do | 17:47 |
anteaya | let's start off with a setuptools release sunday night that broke the world | 17:48 |
anteaya | I don't have the link | 17:48 |
anteaya | that was fixed about 10 or 12 hours later, but long enough that when the rechecks happened it was a fair pileon | 17:48 |
anteaya | add in hpcloud decommisioning 1.0 | 17:49 |
anteaya | and 1.1 having serious outage issues last week | 17:49 |
anteaya | various reasons | 17:49 |
anteaya | now we have 2 hpcloud techs in -infra with us and we are figuring out what is going on | 17:49 |
jogo | fun | 17:49 |
anteaya | right now it takes about 40 minutes for nodepool to see a node is built | 17:50 |
anteaya | and by that time the time is run out and it is scheduled for deletion | 17:50 |
anteaya | so any tests actually running now are all on rax since hpcloud is just building and deleting | 17:50 |
anteaya | the cores are figuring out what to do on our end to have actual tests running | 17:51 |
anteaya | and neutron is working on an ssh bug, which I don't know the details | 17:51 |
anteaya | which is playing a role, but I don't know how large | 17:51 |
anteaya | that is what I know | 17:51 |
jogo | anteaya: I noticed the ssh bug on http://status.openstack.org/elastic-recheck/gate.html | 17:52 |
anteaya | yup | 17:52 |
jogo | anteaya: so that explains it then | 17:52 |
jogo | wow | 17:52 |
anteaya | which is affecting our gate and may also be affecting hpcloud | 17:52 |
anteaya | I don't know if it is or not | 17:52 |
anteaya | so yeah | 17:53 |
anteaya | and as far as I can tell check isn't moving | 17:53 |
anteaya | that horizon job at the top of the gate is 9 hours old | 17:54 |
anteaya | waiting on a free python26 node | 17:54 |
jogo | wow | 17:54 |
anteaya | woooo | 17:54 |
jogo | I don't think I have ever seen so many jobs backed up | 17:55 |
anteaya | yup | 17:55 |
*** kgriffs is now known as kgriffs|afk | 17:55 | |
anteaya | my role is to not review anything and not make things worse than they are | 17:55 |
jogo | anteaya: I'll make sure to not review that much either heh | 17:57 |
anteaya | no point really | 17:57 |
*** jhenner has quit IRC | 17:57 | |
jogo | yeah, good news is I have plenty of patches to work on | 17:58 |
jogo | trying to think on what I can do to help, but not really sure | 17:58 |
*** jhenner has joined #openstack-qa | 17:58 | |
*** nati_ueno has quit IRC | 18:02 | |
*** nati_uen_ has joined #openstack-qa | 18:02 | |
*** SridharG has joined #openstack-qa | 18:06 | |
sdague | anteaya: I will probably not be at the infra meeting, or at least not for most of it. Because of TC and Project meetings, that's the only timeblock that I can go do CSA pickup. | 18:06 |
clarkb | sdague: you need delivery | 18:07 |
sdague | clarkb: going to the farm is part of the whole experience :) | 18:07 |
sdague | and that's just the way it works around here | 18:07 |
jogo | sdague: have a moment? https://review.openstack.org/#/c/95350/2 | 18:08 |
anteaya | sdague: I was hoping we could come to an agreement around the acl reviewing | 18:08 |
sdague | jogo: shoot | 18:08 |
jogo | sdague: I responded in the patch, but I am not sure if the direction I am going is the right one | 18:09 |
jogo | Every so often a valid pep8 failure happens in the gate, so my thinking was we should count those failures as classified for the sake of http://status.openstack.org/elastic-recheck/data/uncategorized.html | 18:09 |
jogo | ex: http://logs.openstack.org/13/94813/6/gate/gate-nova-pep8/96604c4/console.html | 18:09 |
sdague | jogo: that a bad merge? | 18:09 |
jogo | yes | 18:09 |
sdague | are you sure you want to supress it? | 18:10 |
sdague | I agree we should fingerprint | 18:10 |
jogo | sdague: and for some reason those are fairly common (weekly) | 18:10 |
jogo | sdague: well supress from gerrit side | 18:10 |
jogo | so folks don't just run recheck | 18:10 |
jogo | maybe we don't want to supress from the graph side | 18:11 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: [Fedora] Apache double enable/disable apache site https://review.openstack.org/97585 | 18:11 |
sdague | I think I'd be ok if we suppress it from the bot side | 18:11 |
sdague | maybe we should add 2 vars to the yaml | 18:11 |
sdague | report: true/false | 18:11 |
sdague | graph: true/false | 18:11 |
sdague | defaulting to true | 18:12 |
jogo | sdague: thats what I meant to do somehow that patch didn't get pushed | 18:12 |
sdague | there is a patch under it, but it's a little odd - https://review.openstack.org/#/c/97354/1/elastic_recheck/elasticRecheck.py | 18:13 |
sdague | would it be better to just filter those out at load time/ | 18:13 |
sdague | ? | 18:13 |
jogo | we use the loader in other places though | 18:13 |
sdague | I think 97354 does it in all cases right | 18:13 |
jogo | classify is only used by the bot | 18:14 |
*** rbak has quit IRC | 18:14 | |
sdague | ok, so then loader.load(queries, drop_no_report=false, drop_no_graph=false) | 18:16 |
sdague | it just seems odd to constantly hit the if there instead of adding a couple flags to the loader to be able to skip things when it loads | 18:16 |
jogo | sdague: yeah, makes sense to me. new patch comming soon | 18:17 |
sdague | coolio | 18:17 |
afazekas | sdague: can you stop this: https://review.openstack.org/#/c/97532/ | 18:19 |
afazekas | something is still wrong | 18:19 |
sdague | afazekas: sure, but why? | 18:19 |
sdague | afazekas: you can also WorkFlow -1 it | 18:20 |
sdague | and it won't merge | 18:20 |
*** ildikov has quit IRC | 18:20 | |
*** Sukhdev has joined #openstack-qa | 18:22 | |
openstackgerrit | Ramy Asselin proposed a change to openstack/qa-specs: Add qa spec for bp:stress-api-tracking https://review.openstack.org/97589 | 18:26 |
*** ashishsnigam has quit IRC | 18:27 | |
tmcpeak | when I run Tempest tests for components which I don't have (like tempest.api.baremetal) it knows to skip | 18:33 |
tmcpeak | how does that work? | 18:33 |
maurosr | tmcpeak: it happens during class setup https://github.com/openstack/tempest/blob/master/tempest/api/baremetal/base.py#L48 | 18:34 |
*** kgriffs|afk is now known as kgriffs | 18:34 | |
tmcpeak | maurosr: awesome, I'll check this out, thanks! | 18:34 |
maurosr | it checks CONF.service_available.<component> | 18:34 |
maurosr | yw | 18:34 |
maurosr | mtreinish: you around? | 18:35 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Enable horizon on Fedoras https://review.openstack.org/97532 | 18:35 |
*** ajayaa has joined #openstack-qa | 18:38 | |
maurosr | qq, v3 and v2 will converge into v2.1, so patches to attribute tests are also in hold? specially something that is only implemented on v3 | 18:38 |
maurosr | cause I think it will endup being useful into v2.1 since we want the new functionalities | 18:38 |
*** nati_uen_ has quit IRC | 18:39 | |
*** nati_ueno has joined #openstack-qa | 18:39 | |
maurosr | so would be welcome anyway | 18:40 |
*** ajo|mac has joined #openstack-qa | 18:42 | |
*** kgriffs is now known as kgriffs|afk | 18:44 | |
*** SridharG has quit IRC | 18:47 | |
*** tmcpeak has quit IRC | 18:48 | |
*** SridharG has joined #openstack-qa | 18:49 | |
ajayaa | Hi. Is it advisable to write tempest tests for custom policies? | 18:50 |
sdague | ajayaa: an example? | 18:53 |
*** nati_ueno has quit IRC | 18:53 | |
*** kgriffs|afk is now known as kgriffs | 18:54 | |
*** nati_uen_ has joined #openstack-qa | 18:55 | |
*** cdent has quit IRC | 18:58 | |
*** mlavalle has quit IRC | 19:00 | |
ajayaa | sdague: For e.g. I could define a new role called project_admin and give him privilege just to add users in his project. | 19:00 |
*** mlavalle has joined #openstack-qa | 19:00 | |
*** bauzas has quit IRC | 19:01 | |
ianw | sdague: that's running the f20 job for devstack changes | 19:01 |
ajayaa | sdauge: He won't be able to add a role for an user in different project other than his own. | 19:01 |
sdague | ajayaa: this is with keystone v3? | 19:01 |
ajayaa | sdague: yes. | 19:01 |
sdague | ianw: but why the 2nd check experimental immediately after? | 19:01 |
*** ildikov has joined #openstack-qa | 19:02 | |
sdague | ajayaa: so I believe keystone v3 tests like that are in process. andreaf was working on something for that | 19:02 |
*** Obulpathi has quit IRC | 19:04 | |
ajayaa | sdauge: thanks. andreaf, ping! | 19:04 |
ajayaa | sdague: sorry! different keyboard. | 19:05 |
ianw | sdague: which change did that happen on? | 19:05 |
*** Obulpathi has joined #openstack-qa | 19:05 | |
sdague | ianw: https://review.openstack.org/#/c/97442/ | 19:07 |
sdague | maybe it happens first, and gerrit changes order? | 19:07 |
ianw | sdague: i think that' | 19:09 |
andreaf | ajayaa: I'm going to have dinner now I may be online later or tomorrow - if you write your question I'll answer later | 19:09 |
ianw | sdague: i think that's working as intended; it posts the results of our check (all failing, unfortunately due to known issues) and then asks for the experimental run | 19:10 |
sdague | ah | 19:10 |
ajayaa | andreaf, I will send an email to openstack-qa. | 19:10 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Support suppressing bugs in bot https://review.openstack.org/97354 | 19:14 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add suppressed fingerprint for pep8 failures in gate https://review.openstack.org/95350 | 19:14 |
*** Obulpathi has quit IRC | 19:14 | |
mtreinish | maurosr: yeah, what's up | 19:14 |
*** Obulpathi has joined #openstack-qa | 19:15 | |
maurosr | qq, v3 and v2 will converge into v2.1, so patches to attribute tests are also in hold? specially something that is only implemented on v3 | 19:15 |
maurosr | cause I think it will endup being useful into v2.1 since we want the new functionalities | 19:15 |
maurosr | so would be welcome anyway | 19:15 |
maurosr | mtreinish: ^ | 19:15 |
mtreinish | maurosr: yeah I interpreted it as everything v3 specific in tempest was on hold | 19:16 |
mtreinish | but things that also apply to v2.1 are still valide | 19:16 |
maurosr | ok nice =) | 19:17 |
*** bauzas has joined #openstack-qa | 19:17 | |
maurosr | tks | 19:17 |
*** masayukig has quit IRC | 19:22 | |
*** adalbas has quit IRC | 19:25 | |
*** masayukig has joined #openstack-qa | 19:29 | |
*** Obulpathi has quit IRC | 19:30 | |
*** Obulpathi has joined #openstack-qa | 19:31 | |
*** Obulpathi has quit IRC | 19:33 | |
*** Obulpathi has joined #openstack-qa | 19:33 | |
*** adalbas has joined #openstack-qa | 19:38 | |
*** mkoderer has quit IRC | 19:42 | |
*** SridharG has quit IRC | 19:47 | |
EmilienM | Hi there, I would appreciate a review on ceilometer support in Grenade: https://review.openstack.org/#/c/94468/ - Thank you! | 19:53 |
EmilienM | sdague: since the patch in devstack-gate has been merged, I think the one in grenade is ready for review. jogo already +1 | 19:54 |
mtreinish | lifeless: so I was looking at the unittest api and the tb is already stored in the returned testSuite class. So I think the only unittest api change is to add a public method to testSuite that either returns the methodname or None | 19:56 |
mtreinish | something like http://paste.openstack.org/show/82671/ | 19:56 |
mtreinish | then on the testtools side we do something like https://github.com/mtreinish/testtools/commit/430cec9321f0a37cca801797ffdb205f503c911f and just add a call to that new public method when it gets added to unittest | 19:57 |
mtreinish | does that sound sane, or am I missing something (which is probably the case) | 19:57 |
lifeless | mtreinish: I'd quibble about the name I think, we don't want this to look like 'and it ran tests' | 19:58 |
lifeless | mtreinish: in fact, it might be better as a new loader return value kind of things - separate errors and successes | 19:58 |
mtreinish | lifeless: sure, I'm not attached to that name | 19:58 |
*** pbrooko has joined #openstack-qa | 19:58 | |
lifeless | but would that work? not as a it stands - needs to return an interable, because suites might not be leaf objects | 19:59 |
lifeless | there may be many errors | 19:59 |
lifeless | and you'd then need a composite method (like run() or debug()) to query for instantiation errors across the whole composite structure | 19:59 |
mtreinish | oh, I see what your saying the Suite isn't necessarily the suite that failed on import it could be the parent suite too | 20:00 |
mtreinish | ok I'll look into that, I was thinking of it strictly from the testtools use case and in that case it's already the leaf object | 20:01 |
*** davlaps has quit IRC | 20:03 | |
mtreinish | lifeless: in the meantime what about that testtools patch I referenced? do you think we could merge something like that in the short term. It probably won't really need to change much for python3.5 (assuming the new method is added for 3.5) | 20:03 |
lifeless | I've found previously that merging things before upstream work is done generally leads to upstream work not being done | 20:04 |
lifeless | so I'm hesitant about that | 20:04 |
*** tmcpeak has joined #openstack-qa | 20:04 | |
*** ajayaa has quit IRC | 20:06 | |
mtreinish | yeah but that patch is for supporting < python 3.5. Unless we can add the new api to previous releases we're still going to have to guess the failed module name (or use a private attr), and the method of getting the error won't change. | 20:06 |
*** weshay has quit IRC | 20:10 | |
lifeless | mtreinish: right but we'll know that there is a path to upstream support, and can make our local code look like that | 20:14 |
openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: Enable Horizon apache file on all platforms https://review.openstack.org/97614 | 20:16 |
*** mriedem has quit IRC | 20:19 | |
*** mriedem has joined #openstack-qa | 20:21 | |
*** Sukhdev has quit IRC | 20:22 | |
*** weshay has joined #openstack-qa | 20:25 | |
openstackgerrit | Grishkin proposed a change to openstack/tempest: Reworked scenario tests to use their own non-default security groups https://review.openstack.org/86049 | 20:27 |
*** nati_uen_ has quit IRC | 20:28 | |
*** mriedem has quit IRC | 20:33 | |
*** mriedem has joined #openstack-qa | 20:34 | |
*** mpavlase has quit IRC | 20:42 | |
*** ekhugen has quit IRC | 20:47 | |
*** nati_ueno has joined #openstack-qa | 20:47 | |
*** aysyd has quit IRC | 20:52 | |
*** nati_uen_ has joined #openstack-qa | 21:00 | |
*** nati_ueno has quit IRC | 21:00 | |
*** kgriffs is now known as kgriffs|afk | 21:00 | |
*** kgriffs|afk is now known as kgriffs | 21:05 | |
*** dhellmann has quit IRC | 21:11 | |
*** dims__ has joined #openstack-qa | 21:13 | |
*** dhellmann has joined #openstack-qa | 21:13 | |
*** dims_ has quit IRC | 21:15 | |
*** adalbas has quit IRC | 21:22 | |
*** safchain has joined #openstack-qa | 21:25 | |
*** jchai has quit IRC | 21:26 | |
*** Obulpathi has quit IRC | 21:30 | |
*** eglynn has joined #openstack-qa | 21:30 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add missing compute service tags to volume API tests https://review.openstack.org/97630 | 21:30 |
*** pafuent has left #openstack-qa | 21:31 | |
eglynn | mtreinish: so just continuing on that skipping-tests-in-branchless-tempest theme from the project/release meeting, if you got a minute? | 21:34 |
*** weshay has quit IRC | 21:34 | |
mtreinish | eglynn: sure | 21:35 |
eglynn | mtreinish: ... so I was a bit confused by the discussion with DinaBelova yesterday evening | 21:35 |
eglynn | (... we re-hashed it on the openstack-ceilo channel this morning) | 21:36 |
*** levi_b has quit IRC | 21:36 | |
eglynn | specifically this notion ... "the thing is if there is an api bug we could end up skipping a whole bunch of things because the api is misbehaving" | 21:36 |
*** levi_b has joined #openstack-qa | 21:36 | |
eglynn | i.e. that we need config option as well as a discoverable API ... because the API may be buggy | 21:36 |
openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: An example of Certificate Generation with devstack https://review.openstack.org/47526 | 21:37 |
openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: SSL protected cinder endpoints https://review.openstack.org/97632 | 21:37 |
eglynn | mtreinish: ... but surely the API call would generally be used to set the config option, in the case where branchless tempest is run against public cloud X? | 21:37 |
mtreinish | eglynn: yes but it's an out of band tool | 21:38 |
mtreinish | eglynn: see http://git.openstack.org/cgit/openstack/tempest/tree/tempest/cmd/verify_tempest_config.py | 21:38 |
eglynn | mtreinish: out-of-band? ... in what sense? | 21:38 |
mtreinish | eglynn: it's not integrated into running the tests you have to do it separately | 21:38 |
eglynn | mtreinish: sure ... so I wanted to run the test against an unknown public cloud, I need to follow two distinct steps | 21:39 |
eglynn | 1. create config file with the correct settings for that cloud | 21:39 |
eglynn | 2. run actual tempest tests | 21:40 |
mtreinish | yeah that's the basic idea | 21:40 |
eglynn | mtreinish: in order to accomplish step #1 for a public cloud, I'd need to manually call out to the discoverable API, no? | 21:40 |
*** marun is now known as marun_afk | 21:40 | |
*** levi_b has quit IRC | 21:41 | |
mtreinish | or use a tool to help write the config file, either that script I mentioned before and there is a spec on writing a tool to help generate one from scratch | 21:41 |
eglynn | ... but we appear to be saying that we don't fully "trust" that API | 21:41 |
mtreinish | eglynn: from the tempest perspective we don't because it's all about verifying it | 21:41 |
mtreinish | we did autodiscovery of features and services before | 21:41 |
mtreinish | and we started skipping broken things in the gate | 21:42 |
mtreinish | because of the number of tests it was hard to notice a few new skips | 21:42 |
*** dims__ has quit IRC | 21:42 | |
mtreinish | so that's why we have everything like this be explict in the config file (or at least try to) | 21:42 |
eglynn | mtreinish: would it be fair to say that we're still just as likely to skip broken things in the tempest-against-public-cloud-X case? | 21:43 |
eglynn | mtreinish: (... seeing as the only way to populate the config file in that case IIUC is to rely on the API) | 21:43 |
mtreinish | eglynn: well if you're using the api to discover what features are available then probably. but if you don't know what your running against there's not much you can do about that | 21:43 |
eglynn | mtreinish: OK, that question was kind of the root of my confusion | 21:44 |
eglynn | mtreinish: ... as I thought from the original discussion last week that the tempest-against-public-cloud-X case was the major concern | 21:44 |
eglynn | mtreinish: ... but seems like we can live with some fuzziness in that case | 21:45 |
eglynn | mtreinish: ... as long as the gate is locked down with static config for each branch | 21:45 |
mtreinish | yeah, having the api is important for the end user case (which is greater than just tempest's concern) but the config option is a neccesity from tempest perspective | 21:45 |
mtreinish | eglynn: yeah exactly, which we get but using devstack to generate them | 21:46 |
mtreinish | s/but/by/ | 21:46 |
eglynn | mtreinish: yep ... static config for each branch in the (still branch-ful) *devstack* as opposed to the (newly branch-less) tempest | 21:46 |
mtreinish | which is honestly the best way to generate a tempest config by getting whatever deployment tooling to generate it | 21:46 |
mtreinish | eglynn: yeah I'll add something about having feature flags to the main readme for tempest at some point today or tomorrow | 21:48 |
eglynn | mtreinish: so to summarize ... | 21:48 |
eglynn | mtreinish: we need *both* a discoverable API and config-driven test exclusion | 21:49 |
eglynn | mtreinish: ... with the former used (indirectly) for public cloud testing, but *only* the latter relied upon in branchless tempest runs | 21:49 |
mtreinish | yeah, that's the best way to do it. Although the api is also useful for everyone using ceilo to understand what they can expect to work | 21:50 |
*** kgriffs is now known as kgriffs|afk | 21:51 | |
mtreinish | because running tempest against an unknown deployment (like a public cloud) is kind of like being an end user of the api | 21:51 |
mtreinish | eglynn: btw, did you see?: http://lists.openstack.org/pipermail/openstack-dev/2014-May/036262.html | 21:51 |
* eglynn looks | 21:51 | |
mtreinish | I'm trying to figure out what to do for the qa day, and I'm wondering if some combined effort on ceilo and tempest might be a good idea | 21:52 |
*** masayukig has quit IRC | 21:52 | |
*** Mike77 has joined #openstack-qa | 21:52 | |
*** mriedem has quit IRC | 21:53 | |
eglynn | mtreinish: I personally would love to attend something like that | 21:54 |
eglynn | mtreinish: ... BUT we're planning a ceilo meetup in Paris from July 2nd thru' 4th | 21:54 |
*** nati_uen_ has quit IRC | 21:54 | |
eglynn | mtreinish: ... and my boss is likely to shoot me if come back seeking more travel budget :( | 21:55 |
eglynn | mtreinish: ... remote participation not a realistic option, I assume? | 21:55 |
mtreinish | eglynn: ok, I can fully understand that. But, you can give it a try anyway right? :) | 21:55 |
mtreinish | yeah remote participation probably won't work too well | 21:56 |
eglynn | mtreinish: yep, it never does for those kind of events | 21:56 |
*** masayukig has joined #openstack-qa | 21:57 | |
eglynn | mtreinish: ... so is that proposed July 18th date anticipated as a full day? | 21:58 |
eglynn | mtreinish: (or seeing as its a Friday, projected to finish at lunchtime ... to allow folks who've been there all week start travelling home?) | 21:58 |
* eglynn just wondering about possible travel logistics before going begging for budget ... | 21:58 | |
mtreinish | yeah the intent was for the full day. But we haven't locked down the agenda yet. | 21:59 |
mtreinish | I hadn't even considered the half day because of travel | 21:59 |
mtreinish | maybe we'll need to do some shuffling | 21:59 |
eglynn | mtreinish: cool enough, I look at the travel options and see if I can get my hands on some budget | 22:00 |
mtreinish | eglynn: ok cool | 22:00 |
eglynn | mtreinish: ... thanks for the heads-up! | 22:00 |
*** mwagner_lap has quit IRC | 22:02 | |
*** nati_ueno has joined #openstack-qa | 22:02 | |
*** dims has joined #openstack-qa | 22:12 | |
*** dustins has quit IRC | 22:13 | |
*** nati_ueno has quit IRC | 22:24 | |
*** bauzas has quit IRC | 22:25 | |
*** eglynn has quit IRC | 22:28 | |
*** nati_ueno has joined #openstack-qa | 22:32 | |
*** tosky has quit IRC | 22:38 | |
*** aimon has quit IRC | 22:43 | |
openstackgerrit | Andrea Frittoli proposed a change to openstack/qa-specs: Spec for running tempest as non-admin https://review.openstack.org/86967 | 22:44 |
*** jgrimm has quit IRC | 22:44 | |
*** vchernomor has quit IRC | 22:45 | |
*** danfinch has quit IRC | 22:46 | |
*** danfinch has joined #openstack-qa | 22:47 | |
*** nati_uen_ has joined #openstack-qa | 22:47 | |
*** aimon has joined #openstack-qa | 22:48 | |
*** aimon has quit IRC | 22:49 | |
*** nati_ueno has quit IRC | 22:50 | |
*** pbrooko has quit IRC | 22:51 | |
mtreinish | andreaf: you missed one advantage of tenant isolation in that comment. It doesn't constrain the number of workers that we can use when running in parallel | 22:58 |
mtreinish | but was the spec actually proposing we remove it, I thought the proposal was to add the option to use a list of creds if admin wasn't provided | 22:59 |
andreaf | mtreinish: my comment was meant as a reply to sdague's comment on the same patch | 22:59 |
*** TravT has quit IRC | 22:59 | |
*** levi_b has joined #openstack-qa | 23:01 | |
mtreinish | oh, I didn't read that as saying the current model goes away, just that we have an alternative which removes the dependence on admin | 23:01 |
mtreinish | because without that what's the point of running tempest without admin if it has to be serial | 23:02 |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Copy devstacklog.txt https://review.openstack.org/97251 | 23:02 |
andreaf | mtreinish: ok - well it seems that we agree tenant isolation should not go away | 23:05 |
*** levi_b has quit IRC | 23:06 | |
andreaf | mtreinish: it would nice though to refactor things a bit (doh, not again :P) and have something like a credentials provider, which serves credentials to client managers / tests, and worries about the logic around tenant isolation or not... now we have too many if spread around checking if tenant isolation is on or not | 23:07 |
mtreinish | andreaf: that's kinda of what tenant isolation does now, you're just talking about bringing the logic up one level out of the individual test base classes. | 23:09 |
mtreinish | I don't think it would be too bad of a refactor, you can bring that logic into test.py as a first stab at it | 23:09 |
mtreinish | just to unify the credential selection logic | 23:09 |
mtreinish | and store the cred object in a canonical location in the test class which can be referenced for whatever the subclasses need | 23:13 |
mtreinish | that way we don't have to break it off into another external thing which I think would be a bit more involved | 23:13 |
*** eglynn has joined #openstack-qa | 23:15 | |
*** mmaglana has quit IRC | 23:19 | |
*** safchain has quit IRC | 23:19 | |
*** EmilienM_ has joined #openstack-qa | 23:20 | |
*** bdperkin_ has joined #openstack-qa | 23:20 | |
*** DinaBelova2 has joined #openstack-qa | 23:20 | |
*** salv-orlando has quit IRC | 23:20 | |
*** bdperkin has quit IRC | 23:20 | |
*** andreaf has quit IRC | 23:20 | |
*** ruhe has quit IRC | 23:20 | |
*** DinaBelova has quit IRC | 23:20 | |
*** EmilienM has quit IRC | 23:20 | |
*** DinaBelova2 is now known as DinaBelova | 23:20 | |
*** EmilienM_ is now known as EmilienM | 23:20 | |
*** rossella_s has quit IRC | 23:20 | |
*** ruhe2 has joined #openstack-qa | 23:20 | |
*** rossella has joined #openstack-qa | 23:20 | |
*** andreaf has joined #openstack-qa | 23:20 | |
*** rossella is now known as rossella_s | 23:20 | |
*** ruhe2 is now known as ruhe | 23:20 | |
*** bdperkin_ is now known as bdperkin | 23:21 | |
*** bdperkin has quit IRC | 23:21 | |
*** bdperkin has joined #openstack-qa | 23:21 | |
andreaf | mtreinish: I lost a bit of what you wrote - got disconnected - but I think I understand what you mean | 23:22 |
mtreinish | andreaf: ok cool. Because I'm not sure I would understand just from what I wrote, even complete :) | 23:24 |
*** eglynn-regus has quit IRC | 23:24 | |
andreaf | mtreinish: :D | 23:24 |
openstackrecheck | openstack/heat change: https://review.openstack.org/94755 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1262566, gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1323658 | 23:25 |
openstackgerrit | Miguel Lavalle proposed a change to openstack/qa-specs: Add spec for bp:/neutron-advanced-scenarios https://review.openstack.org/95600 | 23:39 |
openstackgerrit | Miguel Lavalle proposed a change to openstack/qa-specs: Add spec for bp:/neutron-advanced-scenarios https://review.openstack.org/95600 | 23:41 |
*** andreaf has quit IRC | 23:42 | |
*** GMann has joined #openstack-qa | 23:43 | |
*** oomichi has joined #openstack-qa | 23:48 | |
oomichi | hi cyeoh | 23:49 |
*** tmcpeak has quit IRC | 23:51 | |
*** tmcpeak has joined #openstack-qa | 23:52 | |
*** GMann has quit IRC | 23:52 | |
*** tmcpeak has quit IRC | 23:56 | |
*** aimon has joined #openstack-qa | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!