Monday, 2015-05-11

*** gamado has quit IRC00:05
*** dimsum__ has joined #openstack-qa00:06
openstackgerritIan Wienand proposed openstack-infra/devstack-gate: [WIP] run centos7 with neutron  https://review.openstack.org/17968900:09
*** zz_dimtruck is now known as dimtruck00:10
*** dimsum__ has quit IRC00:14
*** yamamoto has joined #openstack-qa00:18
*** amotoki has joined #openstack-qa00:19
*** jogo has quit IRC00:25
*** weshay has quit IRC00:28
*** dmorita has joined #openstack-qa00:33
*** dimsum__ has joined #openstack-qa00:39
*** woodster_ has quit IRC00:40
*** dimtruck is now known as zz_dimtruck00:48
*** yamahata has quit IRC00:55
*** zz_dimtruck is now known as dimtruck00:55
*** mriedem has joined #openstack-qa01:02
*** mriedem1 has quit IRC01:04
openstackgerritDavanum Srinivas (dims) proposed openstack/tempest: [WIP] Try logging exceptions  https://review.openstack.org/18137901:13
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: Support for running Nova with oslo.rootwrap daemon  https://review.openstack.org/18099201:27
*** mriedem has quit IRC01:53
*** achanda has joined #openstack-qa01:55
*** mahito has quit IRC01:57
*** dimtruck is now known as zz_dimtruck01:59
*** oomichi has joined #openstack-qa02:01
*** oomichi has quit IRC02:01
*** mahito has joined #openstack-qa02:06
*** dimsum__ has quit IRC02:12
*** gamado has joined #openstack-qa02:20
*** mmedvede_ has quit IRC02:20
*** jogo has joined #openstack-qa02:21
*** dpaterson has joined #openstack-qa02:25
*** zz_dimtruck is now known as dimtruck02:27
*** gamado has quit IRC02:27
*** dimtruck is now known as zz_dimtruck02:40
*** yamamoto has quit IRC02:40
*** yamamoto has joined #openstack-qa02:41
*** amotoki has quit IRC02:43
*** zz_dimtruck is now known as dimtruck02:44
*** sputnik13 has joined #openstack-qa02:46
*** yamamoto_ has joined #openstack-qa02:49
*** mpopow has joined #openstack-qa02:49
*** yamamoto has quit IRC02:52
*** patrickeast has quit IRC02:52
*** mpopow has quit IRC02:54
*** amotoki has joined #openstack-qa02:56
*** boris-42 has quit IRC02:58
*** dimtruck is now known as zz_dimtruck02:59
*** achanda has quit IRC03:01
*** fedexo has quit IRC03:04
*** achanda_ has joined #openstack-qa03:04
*** fedexo has joined #openstack-qa03:05
*** sputnik13 has quit IRC03:07
*** sputnik13 has joined #openstack-qa03:12
*** sputnik13 has quit IRC03:13
*** sputnik13 has joined #openstack-qa03:13
*** achanda_ has quit IRC03:14
*** mahito has quit IRC03:16
*** alex_xu has quit IRC03:20
*** alex_xu has joined #openstack-qa03:21
*** yamamoto_ has quit IRC03:22
*** shashankhegde has joined #openstack-qa03:24
*** mahito has joined #openstack-qa03:24
*** mahito has quit IRC03:29
*** mahito has joined #openstack-qa03:30
*** gfidente has joined #openstack-qa03:37
*** sputnik13 has quit IRC03:40
*** sputnik13 has joined #openstack-qa03:41
*** yamamoto has joined #openstack-qa03:44
*** sputnik13 has quit IRC04:02
*** sputnik13 has joined #openstack-qa04:03
*** mpopow has joined #openstack-qa04:05
*** rushiagr_away is now known as rushiagr04:06
*** gfidente^2nd has joined #openstack-qa04:22
*** coolsvap|afk is now known as coolsvap04:23
*** gfidente has quit IRC04:23
*** sks has joined #openstack-qa04:23
*** yamamoto has quit IRC04:28
*** yamamoto has joined #openstack-qa04:30
*** sputnik13 has quit IRC04:36
*** sputnik13 has joined #openstack-qa04:38
*** gamado has joined #openstack-qa04:45
*** rushiagr is now known as rushiagr_away04:46
*** afazekas has joined #openstack-qa04:55
*** rushiagr_away is now known as rushiagr04:55
* afazekas I have returned04:56
*** gamado has quit IRC04:56
*** yamamoto has quit IRC04:57
*** Madan has joined #openstack-qa05:00
*** sputnik13 has quit IRC05:05
*** sputnik13 has joined #openstack-qa05:08
*** rushiagr is now known as rushiagr_away05:08
*** neeti has joined #openstack-qa05:09
*** gamado has joined #openstack-qa05:10
*** yamamoto has joined #openstack-qa05:17
*** BharatK has joined #openstack-qa05:19
*** woodster_ has joined #openstack-qa05:24
*** sputnik13 has quit IRC05:28
*** rdekel has joined #openstack-qa05:28
*** ibiris_away is now known as ibiris05:29
*** heyongli has quit IRC05:29
*** yamamoto has quit IRC05:30
*** sputnik13 has joined #openstack-qa05:30
*** heyongli has joined #openstack-qa05:31
*** yamamoto has joined #openstack-qa05:33
*** rushiagr_away is now known as rushiagr05:36
*** rushiagr is now known as rushiagr_away05:37
*** achanda has joined #openstack-qa05:37
*** yamamoto has quit IRC05:37
*** jogo has quit IRC05:39
*** jogo has joined #openstack-qa05:39
*** ibiris is now known as ibiris_away05:40
*** anshul has joined #openstack-qa05:41
*** ibiris_away is now known as ibiris05:43
*** yamamoto has joined #openstack-qa05:47
*** shashankhegde has quit IRC05:49
*** mpopow has quit IRC05:51
*** itzikb has joined #openstack-qa05:54
*** mahito has quit IRC05:58
*** shashankhegde has joined #openstack-qa05:59
*** Longgeek_ has joined #openstack-qa05:59
*** Longgeek has quit IRC06:03
*** mahito_ has joined #openstack-qa06:04
*** luqas has joined #openstack-qa06:04
*** yamamoto has quit IRC06:06
*** yamamoto has joined #openstack-qa06:08
*** luqas has quit IRC06:09
*** dmorita has quit IRC06:23
*** dmorita has joined #openstack-qa06:23
*** yfried_ is now known as yfried|afk06:24
*** yfried|afk is now known as yfried_06:24
openstackgerritMerged openstack/tempest: Switch all uses of urllib and HTMLParser to import from six  https://review.openstack.org/17678806:26
*** rushiagr_away is now known as rushiagr06:26
openstackgerritMerged openstack/tempest: Fix comprehension syntax error with python3  https://review.openstack.org/17681406:35
openstackgerritMerged openstack/tempest: Switch all uses of iteritems to use six instead  https://review.openstack.org/17681506:35
openstackgerritMerged openstack/tempest: Switch use of hashing strings to unicode in accounts  https://review.openstack.org/17686206:35
*** yog__ has joined #openstack-qa06:35
*** shardy has joined #openstack-qa06:40
*** shashankhegde has quit IRC06:41
*** yamamoto_ has joined #openstack-qa06:45
*** SridharG has joined #openstack-qa06:46
*** yamamoto has quit IRC06:48
*** mpopow has joined #openstack-qa06:52
*** mpaolino has joined #openstack-qa06:52
*** mpopow has quit IRC06:56
*** safchain has joined #openstack-qa07:00
*** markus_z has joined #openstack-qa07:02
*** sergsh has joined #openstack-qa07:05
*** luqas has joined #openstack-qa07:05
openstackgerritXiaoLiang Hu proposed openstack/tempest: test_verify_server_details fails due to extra info in meta data  https://review.openstack.org/18180607:05
*** luqas has quit IRC07:10
*** k4n0 has joined #openstack-qa07:10
*** apevec has joined #openstack-qa07:13
*** ibiris is now known as ibiris_away07:14
*** fedexo has quit IRC07:14
*** achanda has quit IRC07:15
*** ibiris_away is now known as ibiris07:22
*** yamamoto_ has quit IRC07:33
*** luqas has joined #openstack-qa07:33
*** yfried_ is now known as yfried|afk07:36
*** mahito_ has quit IRC07:38
*** luqas has quit IRC07:38
*** yfried|afk is now known as yfried_07:38
*** woodster_ has quit IRC07:40
*** ihrachyshka has joined #openstack-qa07:43
*** gfidente^2nd is now known as gfidente07:44
*** gfidente has quit IRC07:44
*** gfidente has joined #openstack-qa07:44
*** achanda has joined #openstack-qa07:45
*** mahito has joined #openstack-qa07:46
*** yfried_ has quit IRC07:46
*** jlanoux has joined #openstack-qa07:46
openstackgerritIan Wienand proposed openstack-infra/devstack-gate: [WIP] run centos7 with neutron  https://review.openstack.org/17968907:50
*** mpopow has joined #openstack-qa07:52
*** yamamoto has joined #openstack-qa07:54
*** mpopow has quit IRC07:57
*** achanda has quit IRC08:01
*** jordanP has joined #openstack-qa08:11
*** nithyag_ has quit IRC08:12
*** yfried has joined #openstack-qa08:24
*** ibiris is now known as ibiris_away08:27
*** yog__ has quit IRC08:28
*** rossella_s has quit IRC08:28
*** tkammer has joined #openstack-qa08:30
*** ihrachyshka has quit IRC08:30
andreafafazekas: welcome back :)08:37
*** gilliard_afk is now known as gilliard08:37
afazekasandreaf: :)08:38
andreafafazekas: something for you right away https://review.openstack.org/#/c/149229/08:38
afazekasandreaf, are you ok with that change as it is ?08:41
*** rossella_s has joined #openstack-qa08:41
*** ajo has joined #openstack-qa08:42
*** yamamoto has quit IRC08:45
*** achanda has joined #openstack-qa08:45
*** yog__ has joined #openstack-qa08:45
openstackgerritBenny Kopilov proposed openstack/tempest: To test bootable flag in a cinder volume This commit adds to cinder client bootable_volume capability , change the bootable flag in a cinder volume to True or False. And a testcase for changing the bootable flag from true to false.  https://review.openstack.org/18182708:56
andreafafazekas: yes I'm fine with it - we can still address mtreinish comments in a next patch - I'd really like for this to move forward - but it's a change from HP and +2 by an HP colleagues - so it would be fair to have some non-HP vote on it - besides you reviewed it before and had comments on it08:56
afazekasandreaf: +A08:57
andreafafazekas: thanks!08:57
openstackgerritMahito OGURA proposed openstack-dev/devstack: Fix function and test for 'trueorfalse'.  https://review.openstack.org/18183309:06
*** hichihara has quit IRC09:09
*** yog__ has quit IRC09:12
openstackgerritMerged openstack/tempest: Rework get_network_from_name() function  https://review.openstack.org/17447309:12
openstackgerritMerged openstack/tempest: Fix verify_tempest_config to not leak a tenant  https://review.openstack.org/18156809:12
openstackgerritMerged openstack/tempest: Trim object storage smoke tests to a small set  https://review.openstack.org/17951109:19
*** aix has joined #openstack-qa09:20
*** yog__ has joined #openstack-qa09:23
*** yamamoto has joined #openstack-qa09:24
*** sergsh is now known as sshnaidm09:25
openstackgerritBenny Kopilov proposed openstack/tempest: To test bootable flag in a cinder volume  https://review.openstack.org/18182709:26
openstackgerritMahito OGURA proposed openstack-dev/devstack: Fix function and test for 'trueorfalse'.  https://review.openstack.org/18183309:27
openstackgerritBenny Kopilov proposed openstack/tempest: To test bootable flag in a cinder volume  https://review.openstack.org/18182709:30
*** cdent has joined #openstack-qa09:31
*** tkammer has quit IRC09:31
openstackgerritMerged openstack/tempest-lib: Update setup.cfg metadata to reflect reality  https://review.openstack.org/17834309:37
openstackgerritMerged openstack/tempest: Instance ID was corrected  https://review.openstack.org/17727709:39
*** achanda has quit IRC09:40
*** mpopow has joined #openstack-qa09:41
*** mpopow has quit IRC09:46
*** tkammer has joined #openstack-qa09:49
openstackgerritMerged openstack/tempest: Define validation_resources function for ssh validation  https://review.openstack.org/14922909:49
*** ihrachyshka has joined #openstack-qa10:00
*** rdekel has quit IRC10:02
*** dimsum__ has joined #openstack-qa10:02
*** shardy_ has joined #openstack-qa10:04
*** shardy has quit IRC10:06
*** rdekel has joined #openstack-qa10:08
*** shardy_ has quit IRC10:10
*** shardy has joined #openstack-qa10:10
*** pc_m has joined #openstack-qa10:13
openstackgerritJoseph Lanoux proposed openstack/tempest: Add create_test_server function  https://review.openstack.org/15160110:18
*** gfidente has quit IRC10:19
openstackgerritnithya-ganesan proposed openstack/tempest: Add create_test_server function  https://review.openstack.org/15160110:19
openstackgerritnithya-ganesan proposed openstack/tempest: Create shared ssh verification by extending RemoteClient  https://review.openstack.org/15389910:19
*** NithyaG has joined #openstack-qa10:20
openstackgerritJoseph Lanoux proposed openstack/tempest: Add create_test_server function  https://review.openstack.org/15160110:30
openstackgerritnithya-ganesan proposed openstack/tempest: Create shared ssh verification by extending RemoteClient  https://review.openstack.org/15389910:32
*** samueldmq has joined #openstack-qa10:33
samueldmqmorning10:33
*** mahito has quit IRC10:36
*** mahito has joined #openstack-qa10:36
openstackgerritJoseph Lanoux proposed openstack/tempest: Create shared ssh verification by extending RemoteClient  https://review.openstack.org/15389910:38
openstackgerritJoseph Lanoux proposed openstack/tempest: Migrate compute tests to the ssh-auth-strategy  https://review.openstack.org/16550410:40
openstackgerritJoseph Lanoux proposed openstack/tempest: DO NOT MERGE - test ssh-auth-strategy in the gate  https://review.openstack.org/16920410:41
*** mahito has quit IRC10:44
*** tochi has quit IRC10:44
*** coolsvap is now known as coolsvap|afk10:48
*** gszasz has joined #openstack-qa10:48
*** dmorita has quit IRC10:51
*** gfidente has joined #openstack-qa10:55
*** jhenner has joined #openstack-qa10:59
*** ibiris_away is now known as ibiris11:03
afazekasbkopilov, looks like the 'set-bootable' is part of the 'os-volume-actions' extension, but individually it did not get any identifier11:08
afazekasbkopilov, Probably you should add a tempest Boolean option to the 'volume-feature-enabled' section as `bootable-action`, in order to be able to control whenever run or not run the related test11:10
*** gamado has quit IRC11:11
bkopilovafazekas, ok thanks a lot11:15
*** mpaolino has quit IRC11:15
bkopilovafazekas, How can we add new features in tempest that are not running on old version ? is there any decorator per testcase ?11:16
afazekasmtreinish, Do we plan to support the stable/icehouse branch after the kilo release ?11:16
afazekasbkopilov: It can be a config based skip decorator , which needs to be configured differently at stable/icehouse and at the master branch11:18
afazekasbkopilov: when we stop gating at icheouse it can be default enabled, otherwise initially it needs to be disabled11:20
afazekasbkopilov: Usually good idea to submit a second dependent change to tempest, which just changes the default value from false to true, in order to show the code is working11:22
openstackgerritnithya-ganesan proposed openstack/tempest: Create shared ssh verification by extending RemoteClient  https://review.openstack.org/15389911:26
bkopilovafazekas, ok11:27
afazekasbkopilov:also an iniset should be added to devstack lib/tempest regarding to the new config option.11:27
*** EmilienM|afk is now known as EmilienM11:28
*** mpopow has joined #openstack-qa11:30
*** mmedvede_ has joined #openstack-qa11:30
*** jaypipes has joined #openstack-qa11:32
*** ChanServ sets mode: +o jaypipes11:32
bkopilovafazekas, is there any way to get "cinder version" on a running cloud and use it in test cases ? and use it as filter. this will allow us to add new features easily11:32
*** mpopow has quit IRC11:41
*** SridharG has left #openstack-qa11:42
*** tkammer has quit IRC11:43
openstackgerritJoe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin  https://review.openstack.org/16577511:50
*** masayukig has quit IRC11:53
openstackgerritnithya-ganesan proposed openstack/tempest: Create shared ssh verification by extending RemoteClient  https://review.openstack.org/15389911:53
*** dkranz has joined #openstack-qa11:56
*** Madan has quit IRC11:56
afazekasbkopilov: You can get the list of available extensions from cinder ('cinder list-extensions'), but in this case it is not helpful. Also we would like to avoid runtime feature detection from tempest.   The allowed smartness is to add config checker and helper scripts.11:56
openstackgerritJoseph Lanoux proposed openstack/tempest: Migrate compute tests to the ssh-auth-strategy  https://review.openstack.org/16550411:57
afazekasbkopilov: You may try to create a bug in cider for showing the bootable flag action as an extension.11:57
afazekasbkopilov: IMHO it should be visible there as an extension even if at the code level it just changed an already existing extension.11:58
*** tkammer has joined #openstack-qa11:58
*** masayukig has joined #openstack-qa11:59
*** dimsum__ has quit IRC12:03
*** woodster_ has joined #openstack-qa12:03
*** boris-42 has joined #openstack-qa12:04
*** luqas has joined #openstack-qa12:06
*** BharatK has quit IRC12:07
*** mwagner_lap has quit IRC12:07
openstackgerritSean Dague proposed openstack-dev/grenade: further updates to plugins documentation  https://review.openstack.org/18135812:09
*** weshay has joined #openstack-qa12:12
*** tkammer has quit IRC12:14
*** rfolco has joined #openstack-qa12:16
*** sks has quit IRC12:21
openstackgerritMerged openstack-infra/devstack-gate: Return command status in tsfilter  https://review.openstack.org/17890512:24
openstackgerritMerged openstack-infra/devstack-gate: Add errexit to setup & cleanup functions  https://review.openstack.org/17701712:24
*** tkammer has joined #openstack-qa12:27
*** yfried is now known as yfried|afk12:32
*** aysyd has joined #openstack-qa12:34
openstackrecheckopenstack/requirements change: https://review.openstack.org/181083 failed gate-requirements-pypy in the gate queue with an unrecognized error12:35
*** artom has joined #openstack-qa12:36
*** BharatK has joined #openstack-qa12:43
*** ryansb has left #openstack-qa12:47
*** Longgeek_ has quit IRC12:48
*** Longgeek has joined #openstack-qa12:49
*** mwagner_lap has joined #openstack-qa12:50
openstackgerritRohan Kanade proposed openstack/tempest: Adds Database Instance/Actions/Users API Tests  https://review.openstack.org/18048912:52
*** early has quit IRC12:56
*** early has joined #openstack-qa12:57
openstackgerritJoseph Lanoux proposed openstack/tempest: DO NOT MERGE - test ssh-auth-strategy in the gate  https://review.openstack.org/16920412:59
openstackrecheckopenstack/requirements change: https://review.openstack.org/180925 failed gate-requirements-pypy in the gate queue with an unrecognized error13:00
*** bknudson has quit IRC13:01
*** dkranz has quit IRC13:05
*** k4n0 has quit IRC13:07
*** rushiagr is now known as rushiagr_away13:07
NithyaGandreaf,afazekas,mtrenish: When you have a moment, please review https://review.openstack.org/#/c/153899 the next bit of ssh-auth-strategy13:11
*** Longgeek has quit IRC13:15
*** Longgeek has joined #openstack-qa13:16
*** bknudson has joined #openstack-qa13:22
*** dkranz has joined #openstack-qa13:22
*** dustins has joined #openstack-qa13:24
*** ibiris is now known as ibiris_away13:27
*** jgrimm is now known as zz_jgrimm13:32
*** mpaolino has joined #openstack-qa13:38
*** yamamoto has quit IRC13:39
*** yamamoto has joined #openstack-qa13:39
*** dimsum__ has joined #openstack-qa13:39
*** mriedem has joined #openstack-qa13:42
*** yamamoto has quit IRC13:44
*** yfried|afk has quit IRC13:44
*** yfried|afk has joined #openstack-qa13:45
rahmuhello, I have submitted a patch providing unit tests for javelin2 a while ago. And it's in dire need of some love. Would it be possible to get some attention? https://review.openstack.org/#/c/165775/13:45
rahmumtreinish: ^13:45
rahmuI'm willing to trade reviews if that helps :)13:46
*** yog__ has quit IRC13:47
*** dimsum__ has quit IRC13:53
*** marrusl has joined #openstack-qa13:53
*** tosky has joined #openstack-qa13:54
*** artom has quit IRC13:55
mtreinishafazekas: yeah, we always said we'd support all the current stable branches on master tempest which still includes icehouse13:57
mtreinishso until we the icehouse branch goes eol we've got to keep running things on icehouse13:57
mtreinishrahmu: sure, I'll take a look13:58
*** Yogi has joined #openstack-qa14:00
*** Longgeek has quit IRC14:00
*** Longgeek has joined #openstack-qa14:01
*** ibiris_away is now known as ibiris14:01
*** yamamoto has joined #openstack-qa14:02
*** Longgeek_ has joined #openstack-qa14:03
*** Longgeek has quit IRC14:04
afazekasmtreinish: I asked this question regarding to this patch https://review.openstack.org/#/c/181827/ , you can also see the chatlog with bkopilov14:04
mtreinishafazekas: a flag added in juno without any versioning etc... :(14:04
*** ibiris is now known as ibiris_away14:05
*** bkopilov has quit IRC14:05
afazekasmtreinish: Are you ok with adding config option as described in the log ?14:05
*** pglass has joined #openstack-qa14:05
mtreinishafazekas: yeah, that would be the only way to get it to pass14:05
*** rbak has joined #openstack-qa14:05
mtreinishafazekas: but we should make sure to complain to thingee about this14:05
rahmumtreinish: thanks a bunch14:05
mtreinishbecause adding attrs without a user discoverable mechanism is bad14:06
*** ibiris_away is now known as ibiris14:06
*** openstackgerrit has quit IRC14:06
*** openstackgerrit has joined #openstack-qa14:06
afazekasmtreinish: yeah, but I think we still not have an openstack wide consensus about these things..14:07
mtreinishafazekas: there is, people just forget14:07
mtreinishafazekas: it's in the api change guidlines14:09
*** rossella_s has quit IRC14:09
*** mpaolino has quit IRC14:09
*** rossella_s has joined #openstack-qa14:09
*** neeti has quit IRC14:10
*** yamamoto has quit IRC14:11
*** dboik has joined #openstack-qa14:11
*** zz_dimtruck is now known as dimtruck14:11
*** VanL has joined #openstack-qa14:13
afazekasmtreinish: It just in a referenced in a bug `It needs to be documented as an extension, as the spec will not be updated.`  It is good it just says it needs to be documented as an extension, and technically it does not needs to be an `extension`.   But in pratice it should be listable14:15
afazekasIMHO in practice it should be list-able  as an extension, but I do not see a reference for that14:16
*** dims has joined #openstack-qa14:16
*** openstack has joined #openstack-qa14:20
*** mpaolino has joined #openstack-qa14:25
*** bnemec has joined #openstack-qa14:25
*** miqui has joined #openstack-qa14:30
mriedemanyone know what fixed this? http://logs.openstack.org/10/158210/3/gate/gate-requirements-pypy/7122683/console.html#_2015-05-11_13_01_25_52614:35
mriedemhttp://goo.gl/Lya5bA14:35
*** mtanino has joined #openstack-qa14:35
*** zz_jgrimm is now known as jgrimm14:37
mtreinishmriedem: no idea, I didn't even realize it was broken until I saw the ML this morning14:38
*** mestery has quit IRC14:38
* mriedem goes there14:38
mtreinishmriedem: did the patches to remove the pypy job land?14:38
mriedemhttps://review.openstack.org/#/c/181868/14:39
mriedemi'll e-r query that bug14:39
*** VanL has quit IRC14:40
*** rlrossit has joined #openstack-qa14:40
*** VanL has joined #openstack-qa14:42
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for pypy job bug 1290562  https://review.openstack.org/18191914:42
openstackbug 1290562 in python-ceilometerclient "failing pypy jobs with "option --single-version-externally-managed not recognized"" [Undecided,Fix committed] https://launchpad.net/bugs/1290562 - Assigned to Ildiko Vancsa (ildiko-vancsa)14:42
*** itzikb has quit IRC14:42
*** daniel3_ has joined #openstack-qa14:44
mtreinishmriedem: ^^^ is there a way to throw something about pypy in the query?14:44
mriedemlooking14:44
*** anteaya has joined #openstack-qa14:45
mtreinishif not I'm fine with it as is. I expect that pattern won't pop up on cpython jobs14:45
*** amotoki_ has joined #openstack-qa14:45
mtreinishI guess I was just looking for insurance14:45
mriedemno you're right, found a better one14:46
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for pypy job bug 1290562  https://review.openstack.org/18191914:46
openstackbug 1290562 in python-ceilometerclient "failing pypy jobs with "option --single-version-externally-managed not recognized"" [Undecided,Fix committed] https://launchpad.net/bugs/1290562 - Assigned to Ildiko Vancsa (ildiko-vancsa)14:46
mriedemmmm14:48
mriedemthat restricts to that one job14:48
mriedemmessage:"install-record.txt --single-version-externally-managed" AND tags:"console"14:48
mriedemthat might be best, shows 642 hits14:48
mriedemnvm, not 100% failure14:48
clarkbbuild_name:"gate*pypy" maybe?14:49
*** hemnafk is now known as hemna14:51
mriedemif we want wildcards14:51
mriedemyeah that does hit a lot though14:51
mriedemi'll use that14:51
*** VanL has quit IRC14:52
clarkbwildcards on small fields like that probably dont hurt too bad14:52
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for pypy job bug 1290562  https://review.openstack.org/18191914:52
openstackbug 1290562 in python-ceilometerclient "failing pypy jobs with "option --single-version-externally-managed not recognized"" [Undecided,Fix committed] https://launchpad.net/bugs/1290562 - Assigned to Ildiko Vancsa (ildiko-vancsa)14:52
mtreinishmriedem: heh you could just do status:'failure' AND build_name:"gate*pypy"14:52
*** artom has joined #openstack-qa14:53
openstackgerritJordan Pittier proposed openstack-dev/devstack: Cinder: Set os_privileged_user credentials (for os-assisted-snapshots)  https://review.openstack.org/18192714:55
*** VanL has joined #openstack-qa14:59
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/180582 failed gate-grenade-dsvm in the gate queue with an unrecognized error14:59
*** rdekel has quit IRC14:59
*** mpopow has joined #openstack-qa15:01
*** dkranz has quit IRC15:02
openstackgerritMerged openstack-infra/elastic-recheck: Add query for pypy job bug 1290562  https://review.openstack.org/18191915:04
openstackbug 1290562 in python-ceilometerclient "failing pypy jobs with "option --single-version-externally-managed not recognized"" [Undecided,Fix committed] https://launchpad.net/bugs/1290562 - Assigned to Ildiko Vancsa (ildiko-vancsa)15:04
*** sabeen1 has joined #openstack-qa15:04
*** jecarey has joined #openstack-qa15:05
*** dboik has quit IRC15:05
*** dims has quit IRC15:12
*** aix has quit IRC15:12
*** openstackrecheck has quit IRC15:12
*** openstackrecheck has joined #openstack-qa15:12
*** yamahata has joined #openstack-qa15:14
*** mtanino has quit IRC15:20
*** daniel3_ has quit IRC15:20
*** sabeen3 has joined #openstack-qa15:23
*** sabeen1 has quit IRC15:24
*** bkopilov has joined #openstack-qa15:27
*** sputnik13 has quit IRC15:29
*** rushiagr_away is now known as rushiagr15:29
openstackgerritMatthew Treinish proposed openstack/tempest: Updated from global requirements  https://review.openstack.org/17685115:29
*** sks has joined #openstack-qa15:29
*** mtanino has joined #openstack-qa15:31
*** armax has joined #openstack-qa15:33
openstackgerritBrant Knudson proposed openstack-dev/devstack: Use stevedore for keystone backends  https://review.openstack.org/18193715:34
*** dboik has joined #openstack-qa15:35
*** markus_z has quit IRC15:37
openstackgerritBrant Knudson proposed openstack-dev/devstack: Remove setting nonexistant [ec2] driver option in keystone  https://review.openstack.org/18193915:38
*** yamamoto has joined #openstack-qa15:39
*** Swami has joined #openstack-qa15:40
*** dboik_ has joined #openstack-qa15:42
openstackgerritMerged openstack-dev/devstack: nova: remove allow_migrate_to_same_host config usage  https://review.openstack.org/18139515:43
thingeemtreinish, afazekas: complain to me about what?15:44
*** dboik has quit IRC15:44
*** mpaolino has quit IRC15:44
mtreinishthingee: heh, this: https://review.openstack.org/#/c/181827/ apparently a flag was added to the api in juno without an extension or other discoverability mechanism15:46
mtreinishI didn't deep dive into it, but that's what it was reported as15:46
*** anshul has quit IRC15:46
*** rwsu has joined #openstack-qa15:48
thingeemtreinish: it's optional though. it's not an extension, it's built into the volumes15:48
mtreinishthingee: but how are end users supposed to know whether they can expect that to be there or not?15:49
thingeemtreinish: documentation, changelogs15:50
mtreinishthingee: that doesn't help an end user who just talks to an api with multiple clouds15:50
thingeeapi reference15:50
mtreinishthingee: you're basically saying api consumers need to know what version of cinder is deployed on the cloud they're talking to15:51
thingeemtreinish: I think the thoughts with extensions maybe different from project to project though. I wouldn't expect extensions for the purposes you're mentioning though.15:51
mtreinishthat's not something which is published or a user should need to know15:51
mtreinishespecially when you consider things like clouds which CD15:52
mtreinishthingee: sure extensions aren't the best fit, but just adding attrs to an api response isn't good15:52
openstackgerritOpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements  https://review.openstack.org/17685115:52
mtreinishit's why nova started doing microversions because using extensions for this is terrible15:53
mtreinishbut it was the only mechanism available before microversions15:53
thingeemtreinish: I would recommend if you have these opinions to consider what afazekas has already mentioned, and bring this up cross project. Our api guidelines that we've been following for sometime disagree with your idea.15:54
thingeeNot saying you're wrong, but it's not what we've been following15:54
*** mlavalle has joined #openstack-qa15:54
mtreinishthingee: it's been an established guideline for as long as I've been working on OpenStack15:54
mtreinishhttps://wiki.openstack.org/wiki/APIChangeGuidelines15:54
thingeemtreinish: that's what I'm reading too. Can you be more specific?15:55
thingeeI can change the representation of a resource.15:55
thingeea volume having a bootable flag15:55
thingeeit's optional15:56
thingeemtreinish: and fwiw, nova doing something is a common misconception as cross project.15:57
mtreinishthingee: I'm not sure it's something we can fix at this point, I'm just raising it as an issue.15:57
mtreinishthingee: nova was just an example these are guidelines every project is supposed to have been following15:57
*** e0ne has joined #openstack-qa15:57
mtreinishI used nova because they mvoed off of extensions to microversions15:57
*** mpaolino has joined #openstack-qa15:58
*** e0ne is now known as e0ne_15:58
*** e0ne_ is now known as e0ne15:58
thingeemtreinish: I would disagree with "suppose to have been following"...I think guidelines are guides, not rules. We've established this previous cross projects. Same goes for things like the API working group.15:58
thingeein any case, I agree with the problem you're raising. But it's not necessarily we did something wrong here. There's nothing cross project that has been expressed. Nova has ideas, you have ideas. That's what I see here.16:00
*** VanL has quit IRC16:00
thingeeI would love to get these ideas decided cross project to make things consistent though so we can improve on this aspect.16:00
mtreinishthingee: this was a tc approved thing. It just predates the governance repo: https://wiki.openstack.org/wiki/Governance/Approved/APIStability16:00
thingeemtreinish: the problem though is the guideline is contradicting what you're asking for16:01
thingeeI changed the resource represesentation to something that is optional.16:01
elmikomtreinish, thingee, those api change guidelines are actually under review now for some api-wg changes. i'd love any comments you might have https://review.openstack.org/#/c/180612/16:02
elmikothere is actually some discussion about dropping the extensions language in favor of microversions16:02
elmiko(sorry to interrupt, i just happened to see the convo)16:02
thingeeelmiko: oh neat16:02
sdaguecdent: I'd be curious what you think about - https://review.openstack.org/#/c/181931/ - it might me nice to slice up clarifications we think we want and get them up there16:03
mtreinishthingee: I'm not sure I follow how this contradicts the guidelines, the boot flag was there previously required and you made it optional? That doesn't seem like something that would be allowed under that16:03
thingeemtreinish:16:04
thingeehttps://wiki.openstack.org/wiki/APIChangeGuidelines#Generally_Considered_OK16:04
thingee"Adding an optional property to a resource representation which may be supplied by clients, assuming the API previously would ignore this property"16:04
mtreinishthingee: that's under a conditional with a new extension...16:04
thingeeit's optional and yes, it ignores it16:04
mtreinishit's a sub bullet16:04
*** jlanoux has quit IRC16:05
thingeemtreinish: fair. even still I don't see this under not acceptable =)16:05
thingeeI'm not changing/removing a prop16:05
thingeejust adding16:05
*** safchain has quit IRC16:05
sdaguethingee: adding a property is only non disruptive if there is only 1 instance of the server in the world, when you live with OpenStack clouds that are years old out there, it means that no one can depend on that new property unless it's behind some known versioning, so have to write a ton of conditional client code to work around it16:07
sdaguethe new clients + old servers case16:08
thingeesdague: no one is disagreeing with what you said.16:08
sdaguewhich people tend to forget16:08
sdaguethingee: anyway, I think the old guidelines were a bit vague and sloppy, hopefully we'll get something clearer in the api-wg repo16:09
sdaguehonestly, I expect this would be another tag we'd apply to projects in terms of how strict they are with API compatibility.16:10
*** yfried|afk is now known as yfried16:10
sdaguefor the mechanics of it, if this was done back in juno, it seems the damage is done, and we adjust tests to work around it16:10
mtreinishsdague: sigh, yeah we'll have to add another cinder feature flag16:11
thingeesdague: I agree. I'm suggesting to folks here that instead of complaining about damage, fix the guidelines of what you want. Otherwise we can have this discussion next release by accident. =/16:12
sdaguethingee: yes, agreed, I'm actually trying to get some patches up16:12
sdaguethingee: I'm with you16:12
afazekasmtreinish, thingee:  I do not see it explicitly defined in any of the referenced docs, but if the new version of the software is able to understand (or provide) something what the previous was not, it should be discover-able, without creating a new resource (I mean create volume or vm or whatever)16:12
sdagueI also think the existing guidelines leave out all the reasoning, which make it hard to fill in the gaps when you find one16:13
sdagueso people guess, without existing context16:13
afazekasSo now the `cinder list-extensions` should show me something like `os-volume-actions-bootable` next to the `os-volume-actions`16:14
*** jecarey has quit IRC16:14
thingeesdague: is this the patches you're talking about https://review.openstack.org/#/c/180612/1/guidelines/evaluating_api_changes.rst16:14
afazekasEven if it is not a new extension at the code level16:14
afazekassdague, mtreinish : BTW can you have a look on this: https://review.openstack.org/#/c/180355/16:15
sdaguethingee: I haven't looked at those yet, mostly there have been a bunch of discussions I wanted to capture16:16
*** jogo has quit IRC16:19
mtreinishafazekas: yeah I saw that, I'm not sure it's an issue it's just letting the resolution be variable right?16:25
mtreinishmorganfainberg: do you have any thoughts on: https://review.openstack.org/#/c/180355/16:26
*** pcm_ has joined #openstack-qa16:27
morganfainbergmtreinish: so we specifically convert to utc in keystone so we don't have issues with tz comparisons for token create/expiry. If someone is returning a non-utc tz timestamp, that is wrong.16:28
morganfainbergThere is an issue with microsecond resolution in some cases due to (iirc) a MySQL column and how it stores data16:28
*** bkopilov has quit IRC16:29
*** BharatK has quit IRC16:29
*** pc_m has quit IRC16:29
*** marun has joined #openstack-qa16:29
morganfainbergIn short, the expires field should be very very narrowly maintained.16:29
morganfainbergIn short, the expires field should be very very narrowly maintained.16:29
*** mmedvede_ has quit IRC16:29
morganfainbergIn short the expires should be a very specific format.16:30
morganfainbergThe only fudge factor could be if microseconds are dropped off.16:30
mtreinishmorganfainberg: heh, ok that's what I thought. The bug was openened because rax has microseconds on v2 but tempest wasn't expecting that16:31
morganfainbergIf we (keystone) are issuing inconsistent expires times, that should be fixed in Keystone. Not changed in tempest.16:31
morganfainbergGive a pass on microseconds on v2. But we shouldn't allow any variation besides microseconds.16:31
afazekasmtreinish: a little more than just the precision,  for example using weeks instead of months (with W prefix), punctuation16:32
*** tosky has quit IRC16:32
morganfainbergTbh though, the spec says no microseconds on v216:32
mtreinishmorganfainberg: oh, it's in the spec16:32
mtreinishthen lets enforce it :)16:32
morganfainbergLet me 100% confirm that.16:32
morganfainbergBut I think v2 spec is no microseconds.16:33
morganfainbergCome to think of it.16:33
*** tosky has joined #openstack-qa16:33
mtreinishafazekas: oh, I didn't realize the iso8601 parser allowed all of that16:33
*** Sukhdev has joined #openstack-qa16:35
morganfainbergmtreinish: http://developer.openstack.org/api-ref-identity-v2.html says the expires is 2014-01-31T15:30:58Z16:36
morganfainbergThat is as close to a spec as we have for v216:36
morganfainbergV3 does include microseconds.16:36
morganfainbergAfaicr16:36
mtreinishmorganfainberg: ok, that's what we're currently enforcing in tempest: http://git.openstack.org/cgit/openstack/tempest-lib/tree/tempest_lib/auth.py#n23216:37
*** SridharG has joined #openstack-qa16:37
mtreinishmorganfainberg: can you leave a comment on that review and drop the hammer :)16:37
morganfainbergmtreinish: yep. Let me get breakfast and setup my workspace. 1st thing on the list for today.16:38
*** Nic has joined #openstack-qa16:38
mtreinishmorganfainberg: hehe, thanks16:39
mtreinishoh, I should probably figure out lunch16:39
*** yamamoto has quit IRC16:39
*** SridharG1 has joined #openstack-qa16:41
*** BharatK has joined #openstack-qa16:41
*** mmedvede_ has joined #openstack-qa16:42
*** bkopilov has joined #openstack-qa16:43
*** SridharG has quit IRC16:44
*** mpaolino has quit IRC16:46
*** shashankhegde has joined #openstack-qa16:49
*** mlavalle has quit IRC16:50
*** mlavalle has joined #openstack-qa16:51
*** SridharG1 has left #openstack-qa16:53
*** rushiagr is now known as rushiagr_away16:53
*** afazekas has quit IRC16:55
*** davideagnello has joined #openstack-qa16:58
*** yamahata has quit IRC16:58
*** yamahata has joined #openstack-qa16:58
*** amotoki_ has quit IRC16:59
*** davideagnello has quit IRC17:00
*** davideagnello has joined #openstack-qa17:00
*** sshnaidm has quit IRC17:00
*** dkranz has joined #openstack-qa17:01
*** tosky has quit IRC17:02
*** tosky has joined #openstack-qa17:03
*** shashankhegde has quit IRC17:05
*** e0ne has quit IRC17:13
*** sputnik13 has joined #openstack-qa17:13
*** turul has joined #openstack-qa17:15
*** turul is now known as afazekas17:15
*** cdent has quit IRC17:18
*** jogo has joined #openstack-qa17:19
*** jogo has quit IRC17:19
*** jogo has joined #openstack-qa17:19
*** tkammer has quit IRC17:19
*** VanL has joined #openstack-qa17:19
openstackgerritBenny Kopilov proposed openstack/tempest: To test bootable flag in a cinder volume  https://review.openstack.org/18182717:20
*** rushiagr_away is now known as rushiagr17:21
*** patrickeast has joined #openstack-qa17:24
*** ihrachyshka has quit IRC17:24
*** krtaylor has quit IRC17:24
*** gszasz has quit IRC17:25
*** hitalia has joined #openstack-qa17:29
mriedemsdague: dhellmann: oslo-incubator's versionutils uses pkg_resources which comes from setuptools but we don't have setuptools listed in requirements.txt for projects like nova that are using versionutils.py from o-i,17:30
mriedemwas wondering if there is any reason to not include setuptools in nova's requirements.txt?17:30
*** sks has quit IRC17:31
*** shashankhegde has joined #openstack-qa17:34
openstackgerritSwaminathan Vasudevan proposed openstack/tempest: Check FloatingIP status before connecting to access-points  https://review.openstack.org/16051717:34
*** rushiagr is now known as rushiagr_away17:36
*** krtaylor has joined #openstack-qa17:36
*** yfried has quit IRC17:37
-openstackstatus- NOTICE: We have discovered post-upgrade issues with Gerrit affecting nova (and potentially other projects). Some changes will not appear and some actions, such as queries, may return an error. We are continuing to investigate.17:39
*** ChanServ changes topic to "We have discovered post-upgrade issues with Gerrit affecting nova (and potentially other projects). Some changes will not appear and some actions, such as queries, may return an error. We are continuing to investigate."17:39
*** mestery has joined #openstack-qa17:41
*** Yogi2 has joined #openstack-qa17:41
*** mestery has quit IRC17:44
*** Yogi has quit IRC17:45
*** mestery has joined #openstack-qa17:46
*** jordanP has quit IRC17:52
*** rlrossit has left #openstack-qa17:52
*** pglass has quit IRC17:53
*** achanda has joined #openstack-qa17:54
*** cdent has joined #openstack-qa17:54
openstackgerritBrant Knudson proposed openstack-dev/devstack: Remove KEYSTONE_AUTH_CACHE_DIR  https://review.openstack.org/18200217:54
*** ibiris is now known as ibiris_away17:55
*** mestery has quit IRC17:56
cdentsdague: yeah, gave that a brief glance earlier today and flagged it to look at more closely this evening. seems like a good idea, but I have nit, which I'll point out when I give it a real review17:58
*** daniel3_ has joined #openstack-qa17:59
*** ajmiller has joined #openstack-qa18:02
*** rushiagr_away is now known as rushiagr18:02
sdaguemriedem: is there a reason to?18:05
sdaguelike will it prevent something from working if we don't18:05
sdaguecdent: cool18:06
sdaguecdent: if you want, it might be nice to just etherpad out things that should be written down, divide it up, and create copy18:06
mriedemsdague: well, otherwise you assume python-setuptools will be installed from a distro18:10
sdaguemtreinish: did you put something on the schedule for the ci dashboard?18:10
mriedemand we hit versionutils and pkg_resources isn't there, kaboom18:10
sdaguemriedem: ah, gotcha, incubator18:10
sdagueso, that seems fine to fix. Probably worth circling with dims about what's going on with version utils long term18:11
mriedemdims went awol18:14
*** achanda has quit IRC18:18
*** tsekiyama has joined #openstack-qa18:19
*** yfried has joined #openstack-qa18:19
*** yfried has quit IRC18:19
*** VanL has quit IRC18:22
*** artom_ has joined #openstack-qa18:22
mtreinishsdague: well there is the fishbowl on testing outside the gate18:24
mtreinishthat was on the periodic, experimental stuff18:24
mtreinishwhich includes the dashboard18:24
*** hemna_ has joined #openstack-qa18:24
mtreinishI probably should rename that session to make it more clear18:24
*** dboik_ has quit IRC18:25
*** mpopow has quit IRC18:25
*** bnemec has quit IRC18:25
*** morganfainberg has quit IRC18:25
*** ianw has quit IRC18:25
mtreinishsdague: do you have any suggestions for a better name?18:25
*** samueldmq_ has joined #openstack-qa18:26
*** mjg has joined #openstack-qa18:26
*** gilliard has quit IRC18:26
*** marrusl has quit IRC18:26
*** samueldmq has quit IRC18:26
*** rcarrill` has quit IRC18:26
*** dmellado has quit IRC18:26
*** hemna has quit IRC18:26
*** yfried has joined #openstack-qa18:26
*** artom has quit IRC18:26
*** svasheka has quit IRC18:26
*** chrome0 has quit IRC18:26
*** marrusl has joined #openstack-qa18:26
*** samueldmq_ is now known as samueldmq18:26
*** chrome0 has joined #openstack-qa18:27
*** dboik_ has joined #openstack-qa18:27
*** mpopow has joined #openstack-qa18:27
*** bnemec has joined #openstack-qa18:27
*** morganfainberg has joined #openstack-qa18:27
*** ianw has joined #openstack-qa18:27
*** dmellado has joined #openstack-qa18:27
*** svasheka has joined #openstack-qa18:27
*** rcarrillocruz has joined #openstack-qa18:28
sdagueok, so I saw that as Tempest used for things beyond upstream. Maybe change "outside" to "beyond".18:28
*** yamamoto has joined #openstack-qa18:28
mtreinishsdague: hmm, so "Testing beyond the gate"?18:30
sdagueyeh, maybe?18:32
*** ibiris_away is now known as ibiris18:33
mtreinishit needs a dramatic pause between testing and beyond :)18:33
*** yamamoto has quit IRC18:33
*** apevec has quit IRC18:34
mtreinishwell I suck at naming, so I'll switch it to beyond for now18:34
mtreinishif there are better suggestions we can adjust it again18:34
*** e0ne has joined #openstack-qa18:35
*** Sukhdev has quit IRC18:35
*** yamahata has quit IRC18:40
*** VanL has joined #openstack-qa18:41
*** ibiris is now known as ibiris_away18:41
*** jecarey_ has joined #openstack-qa18:42
*** BharatK has quit IRC18:50
openstackgerritSean Dague proposed openstack-dev/devstack: add network info to the worlddump  https://review.openstack.org/18201418:55
sdaguedansmith: like that &&18:55
sdague^^18:55
dansmithsdague: almost18:56
dansmithsdague: commented18:56
openstackgerritSean Dague proposed openstack-dev/devstack: add network info to the worlddump  https://review.openstack.org/18201418:56
sdagueah, yeh, I did18:57
sdagueupdated18:57
*** mlima has joined #openstack-qa18:57
mlimaHello, how is defnided the test.idempotent_id using as annotation value in tempest test cases?18:58
openstackgerritSean Dague proposed openstack-dev/grenade: wip: make grenade smash resources harder  https://review.openstack.org/18142018:58
morganfainbergmtreinish: so for v2 we don't explicitly use iso8061, so i tried to convey that in the -118:58
openstackgerritSean Dague proposed openstack-dev/grenade: wip: version of smash without extra secgroups  https://review.openstack.org/18154818:58
morganfainbergmtreinish: for v3 we do use iso806118:58
sdagueok, I'm going to stack the grenade changes on that, we'll see what falls out18:58
morganfainbergmtreinish: so it is more acceptible for that change in a v3 context - v2 also have minimal data on the spec and is more just "Examples"18:58
morganfainbergmtreinish: talking w/ dolphm about this as well so we can make sure RAX is not being totally screwed18:59
morganfainbergmtreinish: since there is a way to get microsecond resolution on v2 tokens.18:59
sdaguedansmith: I'll build a 3rd patch which turns off cinder testing as well, but I'd like to leave the existing tests there because the fact that we can repeatably fail is good18:59
dansmithsdague: yeah, for sure, it would just be useful to know if that's a relavant data point19:00
dansmithor relevant even19:00
mtreinishmorganfainberg: yep, I saw the discussion in -keystone19:00
sdaguedansmith: ok, building that change now19:00
mtreinishthe review comments seemed fair to me, we'll wait for the respun patch and go from there19:00
mtreinishmorganfainberg: wait you accept all is8601 on v3?19:01
morganfainbergmtreinish: we specifically say expires_at is iso860119:01
morganfainbergfor v319:01
mtreinishhmm, ok19:01
morganfainbergfor v2 it is iso8601, but uh... non-microsecond and specifically called out as an example that is 8601-like19:01
*** rushiagr is now known as rushiagr_away19:02
morganfainbergso i'd be happier if we try and solve the "it looks like this and may/may not have microseconds"19:02
morganfainbergnot "just use 8601"19:02
morganfainbergfor v2 stuff19:02
openstackgerritSean Dague proposed openstack-dev/grenade: DNM: don't build cinder resources  https://review.openstack.org/18201819:02
*** Rockyg has joined #openstack-qa19:03
mtreinishmorganfainberg: ok, makes sense I guess19:03
*** yamahata has joined #openstack-qa19:03
*** rlrossit has joined #openstack-qa19:05
sdaguedansmith: ok, that test scenario is up19:09
dansmithsdague: cool, thanks19:09
rlrossitHey everyone. It looks like this change (https://github.com/openstack/tempest/commit/f27769e3024adb45ad044e32be0db83555196515) broke our default Tempest behavior for server creation. By default now it will assume you have a floating IP pool. We used fixed IPs when we run so this validation section now tries to create a floating IP pool in setup, so that fails our tests pretty hard.19:12
*** BharatK has joined #openstack-qa19:13
mtreinishrlrossit: did you try setting that validation auth_method config option to 'fixed'?19:14
rlrossityep that was the end result we had to do19:15
rlrossitJust wanted to let you guys know in case something else might have blown up with it from a defaults standpoint19:15
rlrossitI saw that validation section has been there for a while though... was this just not used until this change?19:16
mtreinishrlrossit: ok, thanks for the heads up, could you file a bug about that. We probably shouldn't have changed a default like that19:16
rlrossitsure thing19:16
mtreinishrlrossit: yeah, I think so. I'm not sure what the deal with that was19:16
mtreinishI don't think we should have merged new config options which weren't being used19:16
mtreinishbut that ship has sailed...19:17
rlrossityeah that threw a big red herring because the first place I look is config options to see if something changed recently19:17
rlrossitI think those were added back in like January...19:17
mlimaHello, how is defined the test.idempotent_id used as annotation value in tempest test cases?19:17
mtreinishmlima: what exactly do you need to know?19:20
mtreinishlike how to add that decorator to a test? Or something else?19:20
mtreinishrlrossit: yeahm looks like it's been there for a while19:21
rlrossitit looks like that was part of this? https://blueprints.launchpad.net/tempest/+spec/ssh-auth-strategy19:22
*** daniel3_ has quit IRC19:22
*** e0ne has quit IRC19:22
*** daniel3_ has joined #openstack-qa19:22
mtreinishrlrossit: yeah all these changes are a part of that bp. It's an effort to refactor the ssh code to be more universally consistent between all tests19:23
mtreinishbecause right now it's a real mess19:23
rlrossitmtrienish: but this validation stuff isn't part of ssh right?19:23
rlrossitor at least we don't run with ssh testing but it was setting up all this validation stuff anyways19:24
rlrossitmaybe that's the root of the problem here...19:24
mtreinishrlrossit: hmm, maybe. Is there a global flag to trigger all the validation resource creation?19:25
mtreinishit should probably be opt-in at this point19:25
mtreinishandreaf, jlannox, NithyaG: ^^^19:25
rlrossitwe set run_ssh to False in the compute section, not sure if that's deprecated to move more towards this19:26
mtreinishrlrossit: yeah I think that'll be deprecated as part of this19:26
*** BharatK has quit IRC19:26
mtreinishI think it's just used in the older code path19:27
mtreinishbut we should have an option like that in the validation group to turn off this whole code path19:27
rlrossityeah19:27
rlrossitexactly what I was thinking19:27
mlimamtreinish: I was studying some test cases and all has the annotation test.idempotent_id and I was curious to know how that value is generated19:27
*** BharatK has joined #openstack-qa19:27
mtreinishmlima: oh, it's just a uuid4, there's a script in the tools/ we use to enforce it's presence and uniqueness as well as generate new uuids for tests missing them19:28
mtreinishmlima: http://git.openstack.org/cgit/openstack/tempest/tree/tools/check_uuid.py19:29
mtreinishmlima: it was done as part of this spec: http://specs.openstack.org/openstack/qa-specs/specs/meta-data-and-uuid-for-tests.html19:29
*** daniel3_ has quit IRC19:31
mriedemrlrossit: o/19:31
mriedemway2go19:31
rlrossit\o19:31
*** daniel3_ has joined #openstack-qa19:32
*** Yogi2 has quit IRC19:32
mlimamtreinish: If I wanted to create a new test case, it should have the test.idempotent_id value generated or only after it is approved?19:34
mtreinishmlima: you'll need to generate it before it can be approved19:35
mtreinishthe pep8 job that runs as part of the gate will fail if it doesn't have a uuid set on the test19:35
*** Yogi has joined #openstack-qa19:36
mlimamtreinish: I saw in some test cases that the metadatas does not checked, I think that it can be a new assert.19:39
mtreinishmlima: maybe, I'd have to see a patch or bug with more details to really know one way or the other.19:42
mlimamtreinish: Ok.19:42
mtreinishrlrossit: it looks like gerritbot is dead, but: https://review.openstack.org/18204119:44
mtreinishif you file that bug, I'll put it on the commit msg19:44
*** hitalia has quit IRC19:45
rlrossitgood timing mtreinish: https://bugs.launchpad.net/tempest/+bug/145393619:45
openstackLaunchpad bug 1453936 in tempest "test_create_servers by default creates floating IPs" [Undecided,New]19:45
*** oomichi has joined #openstack-qa19:49
*** pglass has joined #openstack-qa19:50
*** oomichi has quit IRC19:50
*** pglass has quit IRC19:51
*** pglass has joined #openstack-qa19:52
*** dims has joined #openstack-qa20:03
*** sabeen3 has quit IRC20:05
*** mwagner_lap has quit IRC20:07
openstackrecheckopenstack/keystone change: https://review.openstack.org/171906 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1252947 https://bugs.launchpad.net/bugs/145150620:07
openstackLaunchpad bug 1252947 in OpenStack Compute (nova) "libvirtError: Cannot recv data: Connection reset by peer" [Undecided,Confirmed]20:07
openstackLaunchpad bug 1451506 in OpenStack Compute (nova) "spawn failed with "libvirtError: internal error: received hangup / error event on socket" in the gate" [Undecided,Confirmed]20:07
*** hitalia has joined #openstack-qa20:14
*** daniel3_ has quit IRC20:20
*** daniel3_ has joined #openstack-qa20:23
*** daniel3_ has quit IRC20:29
*** tsekiyam_ has joined #openstack-qa20:30
*** tsekiyam_ has quit IRC20:32
*** tosky has quit IRC20:32
*** tsekiyam_ has joined #openstack-qa20:32
*** tosky has joined #openstack-qa20:33
*** pcrews has quit IRC20:33
*** tsekiyama has quit IRC20:33
*** mtanino has quit IRC20:34
*** tsekiyam_ has quit IRC20:41
*** tsekiyama has joined #openstack-qa20:41
openstackgerritSean Dague proposed openstack-dev/grenade: DNM: don't build cinder resources  https://review.openstack.org/18201820:42
openstackgerritSean Dague proposed openstack-dev/grenade: wip: version of smash without extra secgroups  https://review.openstack.org/18154820:42
*** openstackgerrit_ has joined #openstack-qa20:43
*** tsekiyama has quit IRC20:47
*** achanda has joined #openstack-qa20:48
*** Sukhdev has joined #openstack-qa20:52
*** cbader has joined #openstack-qa20:55
*** sks has joined #openstack-qa20:56
openstackgerritChris Dent proposed openstack-dev/devstack: Align ceilometer mod-wsgi choice with general setting  https://review.openstack.org/18208320:59
*** mwagner_lap has joined #openstack-qa21:02
*** sabeen1 has joined #openstack-qa21:04
openstackrecheckopenstack/cinder change: https://review.openstack.org/177517 failed gate-cinder-python27 in the gate queue with an unrecognized error21:05
*** sabeen2 has joined #openstack-qa21:05
*** dustins has quit IRC21:05
*** shardy is now known as shardy_z21:06
*** daniel3_ has joined #openstack-qa21:08
*** sabeen1 has quit IRC21:09
*** mtanino has joined #openstack-qa21:10
*** Yogi has quit IRC21:11
*** pcrews has joined #openstack-qa21:14
*** marrusl has quit IRC21:17
*** sks has quit IRC21:18
*** mpopow has quit IRC21:20
*** mpopow has joined #openstack-qa21:21
*** VanL has quit IRC21:27
*** marrusl has joined #openstack-qa21:29
*** dkranz has quit IRC21:30
openstackgerritMerged openstack-dev/devstack: Use an actual existing nova scheduler in README.md  https://review.openstack.org/18146521:36
*** Rockyg has quit IRC21:39
*** gfidente has quit IRC21:40
*** cdent has quit IRC21:41
*** achanda has quit IRC21:43
*** pcrews has quit IRC21:52
*** mpopow has quit IRC21:52
*** mriedem is now known as mriedem_away21:56
*** artom_ has quit IRC21:57
*** pglass has quit IRC22:00
*** hitalia has quit IRC22:06
*** aysyd has quit IRC22:08
andreafmtreinish, rlrossit: I didn't read your conversation in details yet, it's a bit late in my TZ - I will check tomorrow morning in details22:09
*** rbak has quit IRC22:11
andreafmtreinish, rlrossit: two comments though. The configuration options related to ssh were fairly unusable before - and I don't believe it's any worst now - but it will be more confusing, my bad for not providing interim documention22:11
*** rbak has joined #openstack-qa22:11
andreafmtreinish, rlrossit: and the code for the ssh-auth-bp that has been merged until now has mostly added stuff, it should not have changed any existing behaviour, as it's not used in any test yet really, so I'm surprised it causes any issue - if it does cause changes I think it wasn't meant to, in that case we'll fix it asap22:13
andreafrlrossit: please send me a link to the lp bug if you didn't already22:13
mtreinishandreaf: https://review.openstack.org/18204122:13
rlrossitandreaf: https://bugs.launchpad.net/tempest/+bug/145393622:13
openstackLaunchpad bug 1453936 in tempest "test_create_servers by default creates floating IPs" [High,In progress] - Assigned to Matthew Treinish (treinish)22:13
mtreinishthe issue was the validation code was defaulting to floating ips and it was creating them22:13
rlrossitbasically the problem that it introduced is there's no way to turn it off22:14
rlrossit(and a little bit of what mtreinish said too)22:14
rlrossityeah it looks like run_ssh was defaulted to false before and now it's forced to true, so that is why this introduced problems22:15
rlrossitbecause it looks like the ssh connect method was always floating so technically that part didn't change22:15
rlrossitmtreinish: looking at this more, it looks like a lot of these compute config opts should've been deprecated when they were introduced in the validation section, though I'm not sure how the deprecation process usually works...22:16
mtreinishrlrossit: you just add an attr to the opt definition in config.py to tell it the old opt22:17
andreafrlrossit: we discussed that and decided not to deprecate them until they were actually used22:17
mtreinishthat will allow the old usage (and emit a warning when used) but only show the new option in the samples22:18
andreafrlrossit, mtreinish: the code was meant to be subject to run_ssh but somewhere along the path we lost that bit, sorry about that22:18
rlrossitgotcha. now I understand a little better what's going on here. Thanks andreaf and mtreinish22:18
andreafmtreinish: I'll review your change in details tomorrow, it looks ok at first glance22:20
* andreaf leaving for now22:20
mtreinishandreaf: ok thanks22:20
*** dimtruck is now known as zz_dimtruck22:22
*** pcm_ has quit IRC22:23
*** jgrimm is now known as zz_jgrimm22:24
*** rwsu has quit IRC22:25
*** rlrossit has left #openstack-qa22:25
*** Nic1 has joined #openstack-qa22:29
*** Nic has quit IRC22:29
*** jecarey_ has quit IRC22:31
openstackgerritfumihiko kakuma proposed openstack-infra/devstack-gate: post_test_hook is not executed when the gate test returns an error  https://review.openstack.org/14886322:33
*** marrusl has quit IRC22:35
openstackgerritfumihiko kakuma proposed openstack-infra/devstack-gate: Allow for git base override  https://review.openstack.org/15103422:38
*** pcrews has joined #openstack-qa22:40
*** achanda has joined #openstack-qa22:46
*** mpopow has joined #openstack-qa22:46
*** diegows has joined #openstack-qa22:50
*** mpopow has quit IRC22:51
*** Sukhdev has quit IRC22:52
*** rbak has quit IRC22:54
*** bknudson has quit IRC22:56
*** BadCub is now known as BadCub-Hiding22:56
*** hitalia has joined #openstack-qa22:56
*** flwang has joined #openstack-qa22:56
*** tsekiyama has joined #openstack-qa22:59
*** yamamoto has joined #openstack-qa23:05
flwangmtreinish: ping23:06
mtreinishflwang: pong23:06
flwangmtreinish: as for the patch adding prefix, any idea where to put the wrapper method?23:07
flwangsorry, gerrit is down, so i can't give you the link23:08
mtreinishflwang: heh, I remember the patch :)23:08
mtreinishflwang: probably somewhere in tempest/common, that's where we normally put things like that23:08
mtreinishwe could probably make a tempest.common.data_utils file and do it in there23:09
flwangmtreinish: cool, i had a look but i didn't find a proper file to put it23:11
flwangi will add a new data_utils file after confirmed23:11
mtreinishheh, well the rand_name() call used to live in tempest.common until we migrated the file to tempest-lib. So I think just adding a new module there to add the wrapper would be the easiest23:12
*** dboik_ has quit IRC23:12
flwangmtreinish: heh, seems we're adding something back :)23:13
flwangmtreinish: btw, I had asked the same question but i didn't get a clear answer. the question is, is it possible to run tempest in a specific tenants range?23:14
flwangto avoid it introduce any messing to current env23:15
mtreinisha tenant's range? What do you mean by that?23:15
mtreinishoh do you mean using a pre-existing set of tenants?23:15
*** jhenner has quit IRC23:15
flwangmtreinish: yep23:16
flwangis there any doc about that?23:16
mtreinishflwang: so to do that you can use an accounts.yaml file23:16
mtreinishflwang: yeah there are pieces in the configuration guide: http://docs.openstack.org/developer/tempest/configuration.html#locking-test-accounts-aka-accounts-yaml-or-accounts-file23:17
flwangmtreinish: like i mentioned before, we're going to be brave to leverage tempest in prod23:17
mtreinishthere is also a sample accounts.yaml file in tree: http://git.openstack.org/cgit/openstack/tempest/tree/etc/accounts.yaml.sample23:17
flwangthen do i need any change in tempest.conf?23:18
flwang#test_accounts_file = <None>   ?23:19
mtreinishflwang: yeah you need to turn off tenant isolation23:19
*** hemna_ is now known as hemnafk23:19
mtreinishyep, and point that option to the file you create23:19
flwangmtreinish: seems the doc didn't mention the #allow_tenant_isolation = true should be changed to false23:20
*** Longgeek_ has quit IRC23:20
*** Daviey has quit IRC23:21
mtreinishoh, yeah that looks like a bug in the docs23:21
mtreinishflwang: do you want to push a patch up to fix it :)23:21
*** Daviey has joined #openstack-qa23:21
flwangmtreinish: absolutely yes23:22
flwangmtreinish: cheers man23:22
mtreinishflwang: yep, just checked the code definitely need to set tenant isolation to false: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/credentials.py#n2823:23
flwangawesome23:26
*** tochi has joined #openstack-qa23:27
*** dims has quit IRC23:34
*** hitalia has quit IRC23:37
*** marun has quit IRC23:37
*** dims has joined #openstack-qa23:39
*** dims has quit IRC23:43
*** hitalia has joined #openstack-qa23:44
*** miqui has quit IRC23:45
*** yamamoto has quit IRC23:46
*** tosky has quit IRC23:49
*** gmann has joined #openstack-qa23:50
*** oomichi has joined #openstack-qa23:52
-openstackstatus- NOTICE: Gerrit is going offline while we perform an emergency downgrade to version 2.8.23:54
*** ChanServ changes topic to "Gerrit is going offline while we perform an emergency downgrade to version 2.8."23:55
*** dboik has joined #openstack-qa23:55
*** dboik_ has joined #openstack-qa23:56
*** daniel3_ has quit IRC23:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!