Tuesday, 2015-12-08

mtreinishtimothyb89: ^^^ try the updated branch it should correctly display no data now00:00
*** rossella_s has quit IRC00:02
*** salv-orl_ has quit IRC00:03
*** jamielennox|away is now known as jamielennox00:05
*** pglass has quit IRC00:08
*** salv-orlando has joined #openstack-qa00:09
mtreinishjhesketh, regXboi, david-lyle: whenever you get a sec: https://review.openstack.org/#/q/status:open+project:openstack/openstack-health+branch:master+topic:HEAD,n,z starts the last page for the per test view00:14
mtreinishjhesketh, regXboi, david-lyle, timothyb89: also: https://review.openstack.org/#/c/253214/ should be quick00:15
*** yamamoto has joined #openstack-qa00:26
*** yamamoto_ has joined #openstack-qa00:29
*** yamamoto has quit IRC00:32
*** yamamot__ has joined #openstack-qa00:32
*** patrickeast has quit IRC00:32
*** mtanino has quit IRC00:33
*** yamamoto_ has quit IRC00:34
*** eil397 has quit IRC00:34
*** dougwig has quit IRC00:35
*** dougwig has joined #openstack-qa00:36
*** apevec has quit IRC00:36
*** yamamot__ has quit IRC00:37
*** Swami has quit IRC00:37
*** zz_dimtruck is now known as dimtruck00:38
*** asselin__ has quit IRC00:39
*** asselin__ has joined #openstack-qa00:39
stevemarmtreinish: thanks for clarifying about how often eventlet is tested00:40
*** jdandrea has joined #openstack-qa00:40
HenryGmtreinish: hi00:42
HenryGmtreinish: I took a look at your model sync failures00:43
HenryGmtreinish: I couldn't figure it out :(00:44
HenryGmtreinish: you might ask akamyshnikova or zzzeek for help00:45
HenryGmtreinish: I would be interested to learn more about this00:46
*** jasonsb has quit IRC00:47
*** sabeen3 has quit IRC00:54
mtreinishstevemar: sure, np. It's one of those things that no one ever notices because it's jammed in the postgres job (just like the nova metadata service)00:56
* mtreinish blames notmorgan for putting it there00:56
mtreinishHenryG: ok, I'll follow up with them tomorrow00:57
mtreinishHenryG: I'm sure that I'm just doing something stupid :)00:57
HenryGmtreinish: no, I think you have some complicated foreign keys that I don't understand.00:58
HenryGOr I should say complicated to me because I don't understand them00:58
mtreinishHenryG: oh, that's probably it, we removed the FKs from the migrations, but left them in the models so the orm could figure out the implicit joins in the db api00:58
mtreinishthat's probably throwing things off00:59
HenryGright00:59
mtreinishI needed to update the db api to use explicit joins anyway so this is probably a good chance to fix that00:59
HenryGwhere you see 'add_fk' it means the diff is saying you need to add that FK to the model to make it the same as the schema built by the migrations01:00
HenryGor the other way around, I can't remember01:01
HenryGalso I don't understand the FK it is suggesting to add01:02
mtreinishHenryG: I think it would probably be the other way around. The fks are missing from the migrations01:02
HenryGyup, just checked, that fixed those01:04
*** hemna is now known as hemnafk01:04
HenryGguess what 'modify_nullable' means :)01:05
mtreinishhah01:07
HenryGmtreinish: so I can fix the models to the point of no diffs01:09
HenryGbut then I get a weird error about "sqlalchemy.exc.InvalidRequestError: Could not find a FROM clause to join from.  Tried joining to <class 'subunit2sql.db.models.TestRun'>, but got: Can't find any foreign key relationships between 'tests' and 'test_runs'."01:09
*** eil397 has joined #openstack-qa01:10
mtreinishHenryG: yeah, that's what I meant about about some of the db api using implicit joins. The FKs were just left in the models to make that migration patch a managable size01:15
mtreinishHenryG: if we're going to remove them now we'll have to update all the joins in the db api to explicitly specify a join key01:16
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Change docstring of compute client  https://review.openstack.org/25404301:16
*** armax has quit IRC01:17
notmorganmtreinish: what?!01:18
notmorganWhT did I do?01:18
notmorganOh ahahaha. Eventlet. Lol01:18
notmorganYeeaaaahhh01:18
openstackgerritJacky_lei_zhang proposed openstack/tempest: Make volume_types_client use **kwargs  https://review.openstack.org/25022801:18
*** pvaneck has quit IRC01:20
*** nithyag_ has joined #openstack-qa01:21
*** NithyaG has quit IRC01:23
openstackgerritKen'ichi Ohmichi proposed openstack/tempest-lib: Migrated floating_ips_client.py from tempest  https://review.openstack.org/25400701:23
*** coolsvap has quit IRC01:23
*** salv-orlando has quit IRC01:25
*** coolsvap has joined #openstack-qa01:25
*** dimtruck is now known as zz_dimtruck01:28
*** patrickeast has joined #openstack-qa01:39
*** akshai has quit IRC01:51
*** hichihara has quit IRC01:55
*** eil397 has quit IRC01:55
*** rfolco_ has quit IRC01:56
*** jasonsb has joined #openstack-qa02:14
openstackgerritJacky_lei_zhang proposed openstack/tempest: [V3] Rename get_policy to show_policy  https://review.openstack.org/25412702:17
openstackgerritJacky_lei_zhang proposed openstack/tempest: [V3] Rename get_region to show_region  https://review.openstack.org/25413302:19
*** yangyape_ has joined #openstack-qa02:22
*** tochi has quit IRC02:38
openstackgerritBin Bin Cong proposed openstack/tempest: Add keystone v3 user negative cases for user authentication  https://review.openstack.org/25416402:41
openstackgerritMerged openstack/openstack-health: Make date window for job pages longer for periodic jobs  https://review.openstack.org/25321402:41
*** hitalia has quit IRC02:45
*** bkopilov has quit IRC02:50
*** yamahata has quit IRC02:51
*** akshai has joined #openstack-qa03:04
*** zz_dimtruck is now known as dimtruck03:05
*** edmondsw has quit IRC03:07
*** shuquan has joined #openstack-qa03:23
*** dimtruck is now known as zz_dimtruck03:24
*** tochi has joined #openstack-qa03:24
*** shuquan has quit IRC03:28
*** binbincong has quit IRC03:28
*** yamamoto has joined #openstack-qa03:29
*** salv-orlando has joined #openstack-qa03:29
*** shuquan has joined #openstack-qa03:31
*** sabeen1 has joined #openstack-qa03:33
*** salv-orlando has quit IRC03:34
*** tochi has quit IRC03:49
*** tochi has joined #openstack-qa03:50
*** binbincong has joined #openstack-qa03:50
*** jecarey has joined #openstack-qa03:51
*** itzikb_ has joined #openstack-qa03:53
*** nicholasgoracke has joined #openstack-qa03:54
*** flwang1 has quit IRC03:55
*** jecarey has quit IRC03:56
*** jecarey has joined #openstack-qa04:04
*** eliqiao has joined #openstack-qa04:06
*** eliqiao_ has quit IRC04:06
*** bkopilov has joined #openstack-qa04:20
*** shuquan has quit IRC04:25
*** shuquan has joined #openstack-qa04:26
openstackgerritIan Wienand proposed openstack-dev/devstack: Only use lsb_release for distro tags  https://review.openstack.org/11710404:32
*** boris-42_ has quit IRC04:43
*** masayukig has quit IRC04:47
openstackgerritIan Wienand proposed openstack-dev/devstack: libvirt: don't repeatedly configure libvirtd logging  https://review.openstack.org/25411104:48
*** shuquan has quit IRC04:50
*** flwang1 has joined #openstack-qa04:52
*** yamahata has joined #openstack-qa04:53
*** masayukig has joined #openstack-qa04:54
*** flwang1 has quit IRC05:13
*** anshul has quit IRC05:14
*** sabeen3 has joined #openstack-qa05:16
*** sabeen1 has quit IRC05:19
*** amotoki has joined #openstack-qa05:20
*** Poornima has joined #openstack-qa05:21
*** akshai has quit IRC05:22
*** vgridnev has joined #openstack-qa05:24
*** shuquan has joined #openstack-qa05:30
*** itzikb_ has quit IRC05:32
*** itzikb_ has joined #openstack-qa05:34
*** itzikb_ has quit IRC05:36
*** hoangcx has joined #openstack-qa05:38
*** hoangcx has left #openstack-qa05:38
*** shuquan has quit IRC05:46
*** shuquan has joined #openstack-qa05:47
*** amotoki has quit IRC05:48
*** jaypipes has joined #openstack-qa05:57
*** ChanServ sets mode: +o jaypipes05:57
*** dims has quit IRC05:57
*** yamahata has quit IRC05:58
*** bochi-michael has joined #openstack-qa05:58
*** yamahata has joined #openstack-qa05:58
*** yamahata has quit IRC06:04
*** yamahata has joined #openstack-qa06:09
*** shuquan has quit IRC06:15
*** gamado has joined #openstack-qa06:16
*** _nadya_ has joined #openstack-qa06:20
*** dims has joined #openstack-qa06:20
*** _nadya_ has quit IRC06:25
*** yangyape_ is now known as Yangyapeng06:28
*** amotoki has joined #openstack-qa06:30
*** nicholasgoracke has quit IRC06:31
openstackgerritAnusha Ramineni proposed openstack-dev/devstack: Add congress to plugin registry  https://review.openstack.org/25457006:35
*** shuquan has joined #openstack-qa06:38
*** e0ne has joined #openstack-qa06:38
*** yfried_ has quit IRC06:42
openstackgerritMerged openstack/os-testr: Fix coverage section in tox.ini  https://review.openstack.org/25340706:43
*** maht has quit IRC06:43
*** e0ne has quit IRC06:45
*** eliqiao has quit IRC06:48
*** salv-orlando has joined #openstack-qa06:49
*** SridharG has joined #openstack-qa06:49
*** e0ne has joined #openstack-qa06:49
*** e0ne has quit IRC06:52
*** salv-orlando has quit IRC06:54
*** sabeen3 has quit IRC06:54
*** e0ne has joined #openstack-qa07:02
*** itzikb_ has joined #openstack-qa07:03
*** hitalia has joined #openstack-qa07:07
*** eliqiao has joined #openstack-qa07:07
*** dims has quit IRC07:11
*** _nadya_ has joined #openstack-qa07:12
*** e0ne has quit IRC07:20
*** e0ne has joined #openstack-qa07:21
*** shuquan has quit IRC07:22
*** yfried_ has joined #openstack-qa07:23
*** anshul has joined #openstack-qa07:25
*** shuquan has joined #openstack-qa07:25
openstackgerritAtsushi SAKAI proposed openstack-dev/devstack: [WIP] Add sleep time measurement  https://review.openstack.org/25459107:30
*** dims has joined #openstack-qa07:31
openstackgerritPiyush Pathak proposed openstack/tempest: Make v3 create_trust  use **kwargs & doc string  https://review.openstack.org/25459607:35
yfried_masayukig: oomichi: gmann: https://review.openstack.org/#/c/186559/ can you please explain why the scenario was removed as well as the API tests?07:37
yfried_afazekas: ^07:37
oomichiyfried_: because neutron-lbaas repo contains the same tests07:39
yfried_oomichi: I couldn't find the scenario there07:39
yfried_oomichi: only the API07:39
yfried_oomichi: also, it was my understanding that scenarios (integration tests) should remain in tempest07:40
*** shardy has joined #openstack-qa07:40
*** flwang1 has joined #openstack-qa07:41
*** jaypipes has quit IRC07:44
*** gsagie has left #openstack-qa07:47
gmannyfried_: it is there - https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/tests/tempest/v2/scenario/test_load_balancer_basic.py07:50
*** dims has quit IRC07:52
*** shuquan has quit IRC07:53
*** shuquan has joined #openstack-qa07:54
*** tochi_ has joined #openstack-qa07:55
*** e0ne has quit IRC07:56
yfried_gmann: really? that's confusing. how come projects are hosting integration tests?07:57
yfried_gmann: and if I have a scenario for FWaaS, where would I push it?07:58
yfried_gmann: also, how can I run this from lbaas repo?07:59
*** tochi_ has quit IRC08:06
gmannyfried_: tox -e scenario08:08
gmannyfried_: I think this way - https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/tests/tempest/README.rst08:08
gmannyfried_: but yea, regarding more scenario testing plan etc, neutron team might have as they own those tests now08:09
yfried_gmann: assuming I don't have the entire lbaas repo, can I somehow pull these tests to my tempest? I believe this was discussed as part of a "tempest plugin"08:11
*** eliqiao_ has joined #openstack-qa08:15
*** flwang1 has quit IRC08:15
gmannyfried_: actually with plugin it is always nice. but if you will try to pull m sure there will be lot of changes you need to make manually08:16
*** _nadya_ has quit IRC08:16
*** eliqiao has quit IRC08:18
*** sree has joined #openstack-qa08:19
*** sshnaidm has joined #openstack-qa08:20
andreafoomichi, gmann: question on https://review.openstack.org/#/c/253133/3 - this is separating the neutron metering label rules client to its own file, but it's not adding docstring for kwargs methods - do you want to keep splitting and renaming patches separated, or should I add on the review that the docstring is missing?08:23
openstackgerritPiyush Pathak proposed openstack/tempest: Make v3 create_trust  use **kwargs & doc string  https://review.openstack.org/25459608:25
oomichiandreaf: nice point. it is easy to implement/review each step with separated patches, I feel.08:26
oomichiandreaf: hard to cover all thing in a single patch for me :(08:26
andreafoomichi: ok - yes it makes sense - I will approve this one then08:26
gmannandreaf: oomichi +108:27
oomichiandreaf: thanks :)08:27
*** sree_ has joined #openstack-qa08:30
*** sree_ is now known as Guest1865308:30
gmannandreaf: regarding https://review.openstack.org/#/c/253380/08:32
gmannandreaf: honestly i feel we should support None also, as i saw people use to uncomment the default value (during bug triaging)08:33
gmannandreaf: but i saw other option not having the same logic. I think it is worth to convert 'None' to None in oslo but that might take time or not sure how that will go08:34
masayukigoomichi: regarding https://review.openstack.org/25404308:34
gmannandreaf: as per your comment, you mean to mention support of 'None' in config.py as help or comment?08:34
*** sree has quit IRC08:34
masayukigoomichi: there's a lot of functions which have **kwargs parameter, actually.08:35
andreafgmann: yes I think it would be nice to do that (add a comment in config.py)08:35
masayukigoomichi: but you don't put the api docs. So that I'd like to know the criteria to put the API url.08:36
openstackgerritSean M. Collins proposed openstack-infra/elastic-recheck: Introduce query for 1523638  https://review.openstack.org/25461008:36
openstackgerritFelixLi proposed openstack/tempest: Remove wrong comment  https://review.openstack.org/25416008:36
gmannandreaf: ok. I will try to push that in oslo later. I saw oslo does convert empty string into None for integer type options08:37
gmannandreaf:  and in str type too None should be ok instead of 'None'08:37
*** dtantsur|afk is now known as dtantsur08:37
oomichimasayukig: basically, we need to put api doc links for POST/PUT methods as http://lists.openstack.org/pipermail/openstack-dev/2015-July/068864.html08:37
oomichimasayukig: and your review extends the scope ;)08:38
oomichimasayukig: for GET also08:38
oomichimasayukig: yesterday08:38
oomichimasayukig: to be honest, I don't have criteria for GET yet08:38
*** _nadya_ has joined #openstack-qa08:39
*** _nadya_ has quit IRC08:39
oomichimasayukig: ideally, it is nice to have some pep8 rule which requires api doc links if method contains "params"08:39
oomichiin its arguments08:40
andreafgmann: well the int, empty is not an int, so it's ok to map it to None - but with string, anything you may put in a config file is a string, so whatever value we take for configuring None, it will be excluded from being an actual value for a string08:41
andreafgmann: but I guess <None> would be fair enough, as it's what None is rendered to by the config file generator08:41
masayukigoomichi: Thanks, I got it :) So I feel the criteria should be in the commit message.08:41
*** yamahata has quit IRC08:43
*** vgridnev has quit IRC08:43
oomichimasayukig: nice idea, please -1 on that ;)08:43
*** ifarkas has joined #openstack-qa08:43
oomichimasayukig: it is easy to know with -1 for fixing it later08:43
masayukigoomichi: and actually, there's a lot of 'action methods', though.08:44
masayukigoomichi: ok08:44
*** jlanoux has joined #openstack-qa08:44
masayukigandreaf: I put a comment on https://review.openstack.org/173334 :)08:44
yfried_gmann: that's why we need plugin, don't we?08:45
andreafmasayukig: thanks08:46
openstackgerritSean M. Collins proposed openstack-infra/elastic-recheck: Introduce query for 1523638  https://review.openstack.org/25461008:47
oomichimasayukig: thanks!08:48
openstackgerritSean M. Collins proposed openstack-infra/elastic-recheck: Introduce query for 1523638  https://review.openstack.org/25461008:49
gmannandreaf: humm, yea i also cannot see 'None' as expected string for any options.08:54
*** eliqiao_ is now known as eliqiao08:54
gmannyfried_: exactly :)08:54
*** sree has joined #openstack-qa09:02
*** e0ne has joined #openstack-qa09:03
*** Guest18653 has quit IRC09:05
openstackgerritGhanshyam Mann proposed openstack/tempest: Handle StrOpt config option of microversion  https://review.openstack.org/25338009:08
gmannandreaf: oomichi ^^ please check. added Notes in config.py09:09
*** safchain has joined #openstack-qa09:09
oomichigmann: I am seeing09:10
gmannoomichi: thanks09:11
andreafgmann: thanks, I would put a comment in the "help" section actually, because that's what someone configuring tempest will see in tempest.conf.sample09:11
oomichigmann: isn't it good to put the NOTE into api_version_requrest.py instead of config.py?09:11
andreafoomichi, gmann: I think we should have in config.py something like "The format is 'X.Y', where 'X' and 'Y' are int values, the string 'latest' or the string 'None'"09:13
andreafoomichi, gmann: the rest of the note I agree can be in api_version_request.py09:14
*** apevec has joined #openstack-qa09:14
*** apevec has quit IRC09:14
*** apevec has joined #openstack-qa09:14
gmannandreaf: oomichi i am affraid if that create confusion for 'None' as something special value than None09:14
gmannandreaf: oomichi yea for api_version_request.py, it makes sense09:14
gmannandreaf: oomichi or adding like the ..."string 'latest' means highest supported version and the string 'None' or None (by default) means No version..."09:16
gmannandreaf: oomichi so that it is clear that None and 'None' are same09:17
andreafok, sounds good to me09:17
gmannandreaf: ok. i will update09:17
*** shuquan has quit IRC09:20
*** sree has quit IRC09:23
*** shuquan has joined #openstack-qa09:23
*** sree has joined #openstack-qa09:23
oomichigmann: I am gussing this problem seems common for the other config items, right?09:23
gmannoomichi: yea if we consider case of having explicitly None in config file09:24
oomichigmann: so I am feeling now we should not pass "None" for these config items if users want to use None like the other configs09:25
oomichigmann: current change can be inconsistent between all configs09:25
oomichigmann: as your NOTE, this kind of thing should be handled on common place like oslo.config09:26
gmannoomichi: yes, but i think we cannot say setting  None explicitly in config is not valid09:27
gmannoomichi: yea, i also feel that oslo should return None not 'None'09:27
oomichigmann: how about writing it on the config "help" ?09:27
gmannoomichi: you mean current Note ?09:28
oomichigmann: no, in cfg.StrOpt(help=<here>)09:29
gmannoomichi: yea i mean putting that NOTE in help right?09:29
oomichigmann: That is too detail as help. so I'd like to consider it more.09:30
gmannoomichi: in help how about ..."string 'latest' means highest supported version and the string 'None' or None (by default) means No version..."09:30
gmannoomichi: andreaf but will anyone mention 'None' ?09:30
gmannoomichi: andreaf that is actually internally return from oslo and we handle that09:31
oomichigmann: how about09:31
oomichigmann: "If both values are not specified, Tempest avoid tests with require a microversion."09:31
oomichigmann: instead of current help09:32
andreafoomichi: +1 :) that's actually the right way of configuring rather than specifying None09:32
gmannoomichi: yup. that looks nice09:32
oomichis/avoid/avoids/ :(09:32
andreafoomichi, gmann: what about if string "None" is specified log a warning09:33
oomichiandreaf: +1 for that. that is very user-friendly :)09:33
gmannandreaf: during config options in config.py ?09:34
gmannandreaf: oomichi yea, but do we have sending some string to oslo and they log or we do in api_versio_request09:35
*** vgridnev has joined #openstack-qa09:35
*** hitalia has quit IRC09:36
andreafgmann, oomichi: I guess in api_version_request?09:36
gmannandreaf: yea, did not find arg in oslo09:37
oomichigmann: andreaf: yeah, nice to put the check in api_version_request09:37
gmannandreaf: oomichi so how about other StrOpt, they can be mentioned as None and it breaks currently09:37
gmannand i feel considering explicit mention of default value as invalid does not looks good even that is None09:38
gmannbut may be i am considering too generic cases :)09:39
andreafgmann, oomichi: I'm actually wondering whether we shouldn't get people used to passing None to StrOpt because that generally does not work with oslo.config, and it will generate confusion09:41
andreafgmann, oomichi: so what I would do is to let if it fail everywhere is 'None' is passed. If you want to be nice we could have a nice error message explaining that pass None you have to skip specifying the option so that default applies09:42
andreafgmann, oomichi: because else if we support 'None' for microversions but not everywhere else, it will be inconsistent09:42
gmannandreaf: yea thats something main issue. either we should support everywhere or no where09:43
andreafgmann: if oslo folks agree to get support for 'None' in, we will change this everywhere once that lands09:43
gmannandreaf: oomichi  thats why as permanent solution i think oslo should handle that09:44
gmannandreaf: you mean to return None not 'None' right09:44
oomichigmann: yeah, I really want to know feedback from oslo team related to this09:45
oomichigmann: this is a very nice point and seems common thing09:45
*** jordanP has joined #openstack-qa09:46
gmannandreaf: oomichi  and i think they can consider  'None' as 'None' as it is not actually None it is string explicitly mentioned as value09:46
oomichigmann: that means sometimes 'None' is different from None from user expectation?09:48
gmannoomichi: :) if that is mentioned explicitly as value in config file09:49
gmannbut m not sure those thing we need to handle which is too much support of all kind of mindset :)09:49
andreaf:)09:49
oomichigmann: yeah ;)09:50
gmannandreaf: oomichi ok so for microversion go ahead, i will update help and add NOTE (log warning) in api_version_request.py09:51
gmannandreaf: oomichi and later i will check with oslo how they thing about supporting 'None'09:51
gmannandreaf: oomichi is it ok?09:51
oomichigmann: good for me09:52
andreafgmann: actually given the consistency discussion, I back to thinking we should not support "None" as a string, until we (perhaps) get support for it in oslo09:52
andreafoomichi: ^^^09:52
oomichiandreaf: gmann: I'd like to take time for re-considering...09:53
andreafgmann, oomichi: so I'm ok checking if "None" was passed and raise an InvalidConfig, but I would not accept it as a valid configuration09:53
oomichiandreaf: because after fixing it in oslo side, the code which handles "None" as valid will be trash. right?09:54
gmannandreaf: so in that case (with current oslo behavior which change None to "None"), uncommenting default None value will be invalid ?09:55
openstackgerritMerged openstack/tempest: Renaming Cinder admin client files  https://review.openstack.org/25243209:55
*** StevenK has quit IRC09:56
*** woodster_ has quit IRC09:56
gmannandreaf: oomichi "None" in config value as value is always invalid and we can through exception but with current situation of oslo None is being returned as "None"09:56
andreafgmann: yes but that's invalid for any StrOpt already anyways09:56
*** StevenK has joined #openstack-qa09:57
andreafgmann, oomichi: and besides the tempest.conf.sample contains <None>, not None, so uncommenting would give the string '<None>'09:57
gmannandreaf: ahh i sea, from config file "None" or None is same as all are string there09:57
gmannhumm09:57
andreafgmann; yes indeed09:58
andreaf#max_microversion = <None>09:59
*** sree_ has joined #openstack-qa10:00
*** sree_ is now known as Guest7752210:01
*** aix has joined #openstack-qa10:01
gmannandreaf: ohh,10:03
*** sree has quit IRC10:03
openstackgerritPiyush Pathak proposed openstack/tempest: Make v3 create_trust  use **kwargs & doc string  https://review.openstack.org/25459610:03
gmannandreaf: yea i also feel now not to support  "None"10:04
*** dtantsur is now known as dtantsur|brb10:08
gmannandreaf: oomichi so with that consideration, oslo should raise exception if None is passed explicitly in StrOpt which has default value as None (as currently they support it) or return None as discussed earlier10:11
gmannand consider None as valid for options having default value other than None10:12
openstackgerritYaroslav Lobankov proposed openstack/tempest: Making names of Cinder client classes consistent  https://review.openstack.org/25246710:12
gmannor empty string ...something.10:13
*** aix has quit IRC10:21
*** salv-orlando has joined #openstack-qa10:25
*** salv-orlando has quit IRC10:30
*** aix has joined #openstack-qa10:33
*** Guest77522 has quit IRC10:35
*** cdent has joined #openstack-qa10:37
*** neeti has joined #openstack-qa10:38
*** sree__ has joined #openstack-qa10:38
*** _nadya_ has joined #openstack-qa10:39
*** sree__ has quit IRC10:42
*** yamamoto has quit IRC10:48
*** neeti has quit IRC10:49
*** hop|2 has joined #openstack-qa10:50
*** shuquan has quit IRC10:51
*** tosky has joined #openstack-qa10:52
*** rfolco_ has joined #openstack-qa10:54
*** Yangyapeng has quit IRC10:58
*** openstackgerrit has quit IRC11:02
*** openstackgerrit has joined #openstack-qa11:02
*** lucas-dinner is now known as lucasagomes11:04
*** salv-orlando has joined #openstack-qa11:06
*** amotoki has quit IRC11:07
*** aysyd has joined #openstack-qa11:09
*** drptbl has joined #openstack-qa11:10
*** bochi-michael has quit IRC11:11
*** salv-orlando has quit IRC11:14
openstackgerritJacky_lei_zhang proposed openstack/tempest: Make volume_types_client use **kwargs  https://review.openstack.org/25022811:17
openstackgerritPiyush Pathak proposed openstack/tempest: Make v2 update_user_password use **kwargs  https://review.openstack.org/25291611:25
*** _nadya_ has quit IRC11:26
*** gszasz has joined #openstack-qa11:27
*** jlanoux has quit IRC11:27
*** jlanoux has joined #openstack-qa11:27
*** dtantsur|brb is now known as dtantsur11:31
*** tochi has quit IRC11:35
*** yamamoto has joined #openstack-qa11:36
*** jlanoux has quit IRC11:40
*** jlanoux has joined #openstack-qa11:41
*** yamamoto has quit IRC11:41
openstackgerritAlexander Gubanov proposed openstack/tempest: WIP: Add tests for API microversion with instance tags  https://review.openstack.org/23317611:44
*** yamamoto has joined #openstack-qa11:44
*** yamamoto has quit IRC11:44
openstackgerritAlexander Gubanov proposed openstack/tempest: WIP: Add tests for API microversion with instance tags  https://review.openstack.org/23317611:49
openstackgerritVincent Hou proposed openstack/tempest: Add tempest tests for volume retype with migration  https://review.openstack.org/19544311:53
*** yamamoto has joined #openstack-qa11:53
*** yamamoto has quit IRC11:53
*** e0ne has quit IRC12:03
*** e0ne has joined #openstack-qa12:11
*** sree has joined #openstack-qa12:14
*** sree has quit IRC12:18
*** Liuqing has joined #openstack-qa12:22
*** _nadya_ has joined #openstack-qa12:28
*** shuquan has joined #openstack-qa12:36
*** shuquan has quit IRC12:41
*** weshay_xchat has joined #openstack-qa12:41
openstackgerritPiyush Pathak proposed openstack/tempest: Make v2 enable_disable_user use **kwargs  https://review.openstack.org/25473412:51
*** yamamoto has joined #openstack-qa12:51
*** bkopilov has quit IRC12:57
*** yfried_ has quit IRC12:58
*** _nadya__ has joined #openstack-qa12:58
*** _nadya_ has quit IRC12:59
*** yangyapeng has joined #openstack-qa12:59
*** gmann_ has joined #openstack-qa13:00
*** _nadya__ has quit IRC13:00
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Remove pre-request-validation from v2 image_client  https://review.openstack.org/25131213:05
*** mwagner_lap has quit IRC13:05
*** mwagner_lap has joined #openstack-qa13:08
*** shardy has quit IRC13:09
*** shardy has joined #openstack-qa13:10
*** sree has joined #openstack-qa13:12
*** amotoki has joined #openstack-qa13:15
*** dims has joined #openstack-qa13:16
*** gmann_ has quit IRC13:23
*** edmondsw has joined #openstack-qa13:31
openstackgerritSteven Hardy proposed openstack-dev/devstack: Heat - revise keystone/trusts config to avoid deprecated options  https://review.openstack.org/25475513:34
*** zz_dimtruck is now known as dimtruck13:34
*** jaypipes has joined #openstack-qa13:35
*** ChanServ sets mode: +o jaypipes13:35
*** Liuqing has quit IRC13:36
*** yfried_ has joined #openstack-qa13:39
*** yamamoto has quit IRC13:40
*** shardy is now known as shardy_afk13:41
*** eliqiao has quit IRC13:45
*** eliqiao has joined #openstack-qa13:46
*** Poornima has quit IRC13:46
*** eliqiao has quit IRC13:46
*** eliqiao has joined #openstack-qa13:46
*** yamamoto has joined #openstack-qa13:46
*** dimtruck is now known as zz_dimtruck13:47
*** regXboi has joined #openstack-qa13:49
*** gamado has quit IRC13:51
*** mquin has quit IRC13:54
*** Liuqing has joined #openstack-qa13:56
*** yamamoto has quit IRC14:01
*** yamamoto has joined #openstack-qa14:02
*** akshai has joined #openstack-qa14:04
*** Liuqing has quit IRC14:11
openstackgerritMerged openstack/tempest: Make add_router_to_l3_agent use **kwargs  https://review.openstack.org/25283914:20
*** sree has quit IRC14:23
*** yamamoto has quit IRC14:24
*** yamamoto has joined #openstack-qa14:27
*** mriedem has joined #openstack-qa14:28
*** ajmiller_ has joined #openstack-qa14:30
*** zhangjn has joined #openstack-qa14:32
*** zhangjn has quit IRC14:32
*** zhangjn has joined #openstack-qa14:32
*** _nadya_ has joined #openstack-qa14:41
*** nicholasgoracke has joined #openstack-qa14:42
*** lucasagomes is now known as lucas-hungry14:42
*** edmondsw has quit IRC14:48
*** nicholasgoracke has quit IRC14:49
*** ajmiller_ has quit IRC14:49
*** armax has joined #openstack-qa14:50
*** bkopilov has joined #openstack-qa14:51
*** dims_ has joined #openstack-qa14:52
*** jecarey has quit IRC14:52
openstackgerritJordan Pittier proposed openstack/tempest: tempest/clients.py: always set the image_client attribute  https://review.openstack.org/24207614:52
*** tnguyen has joined #openstack-qa14:52
*** shardy_afk is now known as shardy14:52
*** dims has quit IRC14:55
*** _nadya__ has joined #openstack-qa14:56
*** pglass has joined #openstack-qa14:56
*** _nadya_ has quit IRC14:56
*** nicholasgoracke has joined #openstack-qa14:57
mtreinishregXboi: if you get a sec I'd appreciate some reviews on: https://review.openstack.org/#/q/status:open+project:openstack/openstack-health+branch:master+topic:HEAD,n,z14:58
mtreinishthat adds the last of the pages from the mockup and makes every link actually clickable :)14:59
*** mfisher_ora has joined #openstack-qa15:00
*** zhangjn has quit IRC15:01
*** dustins|gone is now known as dustins15:01
*** mquin has joined #openstack-qa15:01
*** yangyapeng has quit IRC15:01
*** zz_dimtruck is now known as dimtruck15:02
*** dkranz has joined #openstack-qa15:02
* regXboi looks15:03
regXboimtreinish: the deeds are done - any clean up can be in follow on PS15:06
*** rbrndt has joined #openstack-qa15:06
mtreinishregXboi: cool, thanks15:06
regXboimtreinish: yw15:07
*** artom has joined #openstack-qa15:10
*** edmondsw has joined #openstack-qa15:13
*** TuanNguyen has joined #openstack-qa15:14
*** lucas-hungry is now known as lucasagomes15:17
*** miqui has joined #openstack-qa15:17
openstackgerritMerged openstack/openstack-health: Rework get_test_runs_for_test() rest api  https://review.openstack.org/25375115:19
openstackgerritMerged openstack/openstack-health: Add test detail page  https://review.openstack.org/23128715:20
openstackgerritMerged openstack/openstack-health: Remove the remaining mockups  https://review.openstack.org/25380015:20
andreafyay!! ^^^15:24
*** tosky has quit IRC15:25
mtreinishandreaf: heh, yeah it should get picked up on the next puppet run15:25
*** tosky has joined #openstack-qa15:26
*** gszasz has quit IRC15:27
mtreinisherr, maybe. It looks like the latest runs picked up the changes:15:28
mtreinishhttp://puppetboard.openstack.org/report/health.openstack.org/9dc14b2799c23ee52a5f30290f4567e1a9a4319c15:28
mtreinishhttp://puppetboard.openstack.org/report/status.openstack.org/2c7ffcbcda7cd58be2cf19a2a7bbde6997f9536e15:28
mtreinishbut things didn't get updated:15:29
mtreinishhttp://health.openstack.org/test_runs/tempest.api.volume.test_qos.QosSpecsV2TestJSON.test_get_qos?start_date=2015-11-0115:29
*** gszasz has joined #openstack-qa15:30
*** mtanino has joined #openstack-qa15:33
andreafmtreinish :(15:34
mtreinishI'll wait 1 more puppet interval and then go into debug mode :)15:35
*** SridharG has left #openstack-qa15:42
openstackgerritBenny Kopilov proposed openstack/tempest: Test encrypted volumes for multi-backends  https://review.openstack.org/25169215:46
*** jecarey has joined #openstack-qa15:46
*** patchbot has quit IRC15:48
*** patchbot has joined #openstack-qa15:48
sdaguejgriffith I just noticed that bugs 2 - 6 on ER are all volumes issues15:50
sdaguethose are the reason we have a gate backup right now15:51
jgriffithsdague: I'll take a look15:51
*** tnguyen has left #openstack-qa15:51
sdaguecool15:51
sdagueit's really unclear to me what's going on, other than volumes just not completing15:51
jgriffithsdague: wow... some "old" friends have come back it seems15:52
sdaguehttp://logs.openstack.org/92/253792/7/check/gate-grenade-dsvm-multinode/d52d36a/logs/grenade.sh.txt.gz#_2015-12-07_21_02_47_71715:52
sdagueyeh, it's mostly things don't work15:52
sdaguebut it seems like it's hitting both bootfrom volume attempts on liberty15:53
sdagueat least if they run at the same time15:53
jgriffithsdague: have you looked to see if it's mostly multi-node already?15:54
jgriffithsdague: in other words save me the leg work if I should just start there ;)15:54
jgriffithsdague: NM, looking in logstash now15:55
sdaguehttp://tinyurl.com/pdy7dll15:55
sdagueso, it's super rampant in multinode15:55
sdaguethat's where I've noticed it15:55
sdagueand clearly has started climbing a lot of late15:55
jgriffithsdague: perfect, thanks.  I'll get a setup going and see what's up.  And yes, the trend is fairly obvious15:56
*** vgridnev has quit IRC16:00
*** itzikb_ has quit IRC16:03
*** salv-orlando has joined #openstack-qa16:03
*** salv-orlando has quit IRC16:03
*** salv-orlando has joined #openstack-qa16:03
*** zackf has quit IRC16:04
*** yamamoto has quit IRC16:04
*** hemnafk is now known as hemna16:06
*** aix has quit IRC16:08
*** yamamoto has joined #openstack-qa16:09
*** gszasz has quit IRC16:09
mtreinishjgriffith: you'll like: http://status.openstack.org/openstack-health/#/test/tempest.scenario.test_volume_boot_pattern.TestVolumeBootPatternV2.test_volume_boot_pattern16:13
*** sabeen1 has joined #openstack-qa16:15
jgriffithmtreinish: define "like" :)16:15
jgriffithmtreinish: we seem to have different interpretations of that word16:15
jgriffith:)16:15
mtreinishjgriffith: heh, I should have said you'll like that don't need black magic on the cli to get that graph anymore :)16:16
mtreinishheh, not necessarily the page itself16:16
jgriffithmtreinish: haha!  Ok, yes... I do like the page :)16:17
*** hop|2 has quit IRC16:17
jgriffithmtreinish: ya'll have been busy while I've been away!!  All sorts of new enhancements!16:17
mtreinishjgriffith: heh, that page just landed, like < 1hr ago :)16:18
jgriffithmtreinish: ha! Nice timing16:18
jgriffithmtreinish: BTW, how are things looking with your RBD work?16:21
*** gszasz has joined #openstack-qa16:21
mtreinishjgriffith: my rbd work? Do you mean mriedem? :)16:21
jgriffithmtreinish: GAHHHHH16:21
jgriffithmtreinish: I ALMOST went an entire month without doing that!!16:22
jgriffithmtreinish: of course it's not really fair since I haven't really been around in a few weeks16:22
jgriffithgeesh16:22
mriedemjgriffith: just need to clean that patch up today,16:22
jgriffithmtreinish: would you just change your nick please! :)16:23
mriedembut now looking into these multinode failures16:23
jgriffithI'll pay mtreinish or mriedem , whoever changes first :)16:23
*** mriedem is now known as rodermang16:23
*** rodermang is now known as mriedem16:23
*** sree has joined #openstack-qa16:24
jgriffithmriedem: cool, I"m firing up a fresh mutli-node now to look at some of those as well16:24
mtreinishjgriffith: I think you oh mriedem some money now :)16:24
jgriffithmriedem: let me know if you want to tag team it16:24
*** sabeen1 has quit IRC16:24
jgriffithmtreinish: it appears that may be the case16:24
*** sabeen2 has joined #openstack-qa16:24
*** gszasz has quit IRC16:24
*** salv-orl_ has joined #openstack-qa16:26
* mtreinish can't believe he just wrote 'owe' as 'oh'16:26
* mtreinish hides16:26
jgriffithmtreinish: :)16:26
jgriffithmtreinish: shorthand... IRC slang :)16:27
mriedemi'm glad you caught it before i said something16:27
mriedembecause i wasn't going to let that slide16:27
mtreinishhaha16:27
*** sree has quit IRC16:28
*** salv-orlando has quit IRC16:29
*** yamamoto has quit IRC16:37
*** safchain has quit IRC16:39
*** sabeen2 has quit IRC16:39
*** sabeen has joined #openstack-qa16:39
mriedemjgriffith: so dansmith just noticed that the two hosts have the same hostname, -infra is debugging16:40
mriedemthis was a problem with the multi-node jobs before16:40
*** boris-42_ has joined #openstack-qa16:44
*** armax has quit IRC16:44
*** openstackgerrit has quit IRC16:47
*** openstackgerrit has joined #openstack-qa16:47
*** sshnaidm has quit IRC16:48
*** armax has joined #openstack-qa16:51
openstackgerritJordan Pittier proposed openstack/tempest: Make volume_qos_client use **kwargs  https://review.openstack.org/25486416:52
*** zackf has joined #openstack-qa16:53
*** vgridnev has joined #openstack-qa16:53
*** yamahata has joined #openstack-qa16:58
jgriffithmriedem: there's all sorts of badness going on in some of these17:02
jgriffithmriedem: http://logs.openstack.org/81/209581/28/check/gate-tempest-dsvm-full/0ae8be6/logs/screen-c-api.txt.gz#_2015-12-08_13_43_34_90617:02
mriedemjgriffith: sdague said the newer ovh/bluebox nodes are slower than hp/rax nodes, so quota races might be spiking b/c of that17:02
mriedemnot really sure though17:03
mriedemthis is actually a rax node though http://logs.openstack.org/81/209581/28/check/gate-tempest-dsvm-full/0ae8be6/console.html#_2015-12-08_13_08_50_18117:03
jgriffithmriedem: well, the thing is quota of 1G is never going to work out17:03
mriedemjgriffith: hmm, there was a recent tempest change for volumes quotas17:03
* mriedem looks17:03
*** jlanoux has quit IRC17:04
*** alex_xu has quit IRC17:04
mtreinishmriedem: http://i.imgur.com/oKOjVLc.jpg they are slower17:04
mriedemhttps://review.openstack.org/#/c/250137/17:04
mtreinishmriedem: that seems unlikely to break something17:05
*** dolpher has quit IRC17:05
mriedemjgriffith: 1gb is probably from a negative quota test17:06
*** anshul has quit IRC17:07
jgriffithYeah, it is in fact17:07
*** drptbl has quit IRC17:08
mriedemso...17:08
mriedemthe quota tests should be locked17:08
mriedemlike the compute quota tests17:08
mriedembut that's only per-tenant17:08
mriedemthere is a compute admin test that changes the default quotas for nova, and that's locked out from running with other compute quotas tests so the results aren't messed up17:09
mriedemjgriffith: yeah so the warnings/errors in c-api are just red herrings17:10
jgriffithmriedem: yup17:10
jgriffithmriedem: sorry about that17:10
mriedemnp17:10
jgriffithmriedem: so there's nothing on the Cinder side besides those17:10
mtreinishmriedem: well the default nova quota test is locked17:10
*** alex_xu has joined #openstack-qa17:10
jgriffithNova however is not happy17:10
mtreinishbut the other quota tests create an isolated tenant so there shouldn't be an issue running those together17:11
jgriffithmtreinish: I think they're fine.  I just need to clean up Cinder so it doesn't trace on every one of those17:11
mriedemyeah, definitely not a warning17:11
mriedem*exception trace i mean17:12
jgriffith:)17:12
mriedemoperator doesn't care about quota errors17:12
jgriffithmriedem: I'll just dump them to a warning log message17:12
SpamapSmtreinish: so.. https://review.openstack.org/#/c/230183/ ... has sat for a long time, and the work is nearly done. Am I to believe that this could still be -2'd and the work would be thrown out? This is kind of weird..17:13
mtreinishSpamapS: it has a -117:13
*** amotoki has quit IRC17:14
*** Swami has joined #openstack-qa17:15
SpamapSmtreinish: did you read the -1?17:16
*** jecarey has quit IRC17:16
*** salv-orl_ has quit IRC17:16
jgriffithmriedem: am I missing something here:  http://logs.openstack.org/81/209581/28/check/gate-tempest-dsvm-full/0ae8be6/logs/screen-n-api.txt.gz#_2015-12-08_13_51_53_59417:18
jgriffithmriedem: that's the instance that fails in the test17:18
jgriffithmriedem: the req never seems to show up in n-cpu17:19
jgriffithOHHH17:19
jgriffithwait... let me check the other node :)17:19
mtreinishSpamapS: hmm, it looks like oomichi confused openstack-infra with openstack's internal infrastructure17:19
mtreinishSpamapS: I think we should give oomichi a chance to respond to your replies17:20
openstackgerritMerged openstack/tempest: Split out Neutron metering label rules client  https://review.openstack.org/25313317:20
jgriffithhmm... that's single node17:20
SpamapSmtreinish: that's fine. I just don't want to sit for another month with no feedback.17:21
mriedemjgriffith: i don't see it in n-net either17:22
*** dtantsur is now known as dtantsur|afk17:22
*** rfolco_ has quit IRC17:25
*** ifarkas has quit IRC17:25
*** rfolco_ has joined #openstack-qa17:28
*** austin81 has joined #openstack-qa17:31
*** nicholasgoracke has quit IRC17:35
*** gmann has quit IRC17:38
*** gmann has joined #openstack-qa17:40
*** _nadya__ has quit IRC17:42
*** e0ne has quit IRC17:50
*** hitalia has joined #openstack-qa18:00
*** _nadya_ has joined #openstack-qa18:05
*** _nadya_ has quit IRC18:05
openstackgerritMerged openstack-dev/devstack: Neutron FWaaS: Use generated configuration files if available  https://review.openstack.org/25208718:07
*** dolpher has joined #openstack-qa18:08
*** jasonsb has quit IRC18:17
*** jecarey has joined #openstack-qa18:17
*** salv-orlando has joined #openstack-qa18:17
*** jecarey has quit IRC18:21
*** salv-orlando has quit IRC18:21
jgriffithmriedem: does this mean anything to you:  http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/logs/screen-n-cpu.txt.gz#_2015-11-30_18_20_57_39118:22
jgriffithmriedem: appears to be a common entry in these failures18:22
jgriffithmriedem: I'm unclear on what that actually means, given that the Instance is then created and reported as a volume mounted at dev/vda18:24
jgriffithmriedem: ahh... bdm.device_name is kind of ignored now it seems18:26
jgriffithnever mind18:26
mriedemjgriffith: yeah18:27
mriedemfor the libvirt driver anyway18:27
*** jaypipes has quit IRC18:27
*** eil397 has joined #openstack-qa18:29
*** dims_ has quit IRC18:30
sdaguelifeless: is there any way to make the subunit discovery failure be more verbose. If you have a cruft autosave file in your tree is can permanently kill the discovery, and never tell you why18:31
*** pglass has quit IRC18:33
*** nicholasgoracke has joined #openstack-qa18:36
*** dramalho has joined #openstack-qa18:38
*** dramalho has quit IRC18:38
*** e0ne has joined #openstack-qa18:39
jgriffithmriedem: one last red-herring.... http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/logs/screen-n-cpu.txt.gz#_2015-11-30_18_20_33_94818:39
jgriffithmriedem: appears the Instance shutdown while trying to do the ssh18:40
jgriffithseems to coorelate with a number of failures in logstash18:41
*** cdent has quit IRC18:45
*** sabeen has quit IRC18:46
*** pvaneck has joined #openstack-qa18:48
*** jecarey has joined #openstack-qa18:48
*** david-ly_ has joined #openstack-qa18:50
*** david-ly_ is now known as david-lyle_18:52
*** eliqiao has quit IRC18:52
*** tosky has quit IRC18:52
*** jecarey has quit IRC18:53
*** jecarey has joined #openstack-qa18:53
*** david-lyle has quit IRC18:54
*** austin81 has left #openstack-qa18:54
*** eliqiao has joined #openstack-qa18:55
*** jecarey_ has joined #openstack-qa18:58
*** nicholasgoracke has quit IRC18:59
*** jasonsb has joined #openstack-qa19:00
*** jecarey has quit IRC19:02
*** openstackgerrit has quit IRC19:02
*** openstackgerrit has joined #openstack-qa19:02
*** jecarey has joined #openstack-qa19:03
*** jecarey_ has quit IRC19:03
*** salv-orlando has joined #openstack-qa19:04
*** jasonsb has quit IRC19:06
*** salv-orlando has quit IRC19:11
*** maht has joined #openstack-qa19:13
*** david-lyle_ is now known as david-lyle19:14
*** pglass has joined #openstack-qa19:14
*** rbrndt has quit IRC19:17
*** pglass has quit IRC19:19
*** itzikb_ has joined #openstack-qa19:20
*** maht has quit IRC19:23
*** nicholasgoracke has joined #openstack-qa19:24
*** maht has joined #openstack-qa19:24
*** ngoracke has joined #openstack-qa19:30
sc68calmtreinish: is there an IRC room for openstack-health or is it just here19:30
*** weshay_xchat is now known as weshay19:31
*** pglass has joined #openstack-qa19:31
mtreinishsc68cal: just here19:35
mtreinishit's a qa project19:35
sc68calmtreinish: thanks19:36
*** flwang1 has joined #openstack-qa19:36
*** harlowja has quit IRC19:38
*** harlowja has joined #openstack-qa19:38
*** sshnaidm has joined #openstack-qa19:40
*** rbrndt has joined #openstack-qa19:43
mtreinishregXboi: on https://review.openstack.org/#/c/243703/12 I'm curious what looked odd? is it that he added a line break on L21 there?19:45
*** sabeen has joined #openstack-qa19:48
*** sabeen3 has joined #openstack-qa19:50
mtreinishEmilienM: did you push the patch to rename the puppet jobs to include tempest in the name? All the patches I was watching merged but I still don't see any results in the db19:51
EmilienMmtreinish: not yet, I was waiting for the dependencies before19:52
openstackgerritSean M. Collins proposed openstack/openstack-health: [wip] jobs view - reverse sort of failuresRate  https://review.openstack.org/25492719:52
EmilienMmtreinish: i'm doing it now, thanks for the reminder19:52
EmilienMmtreinish: all deps have been merged19:52
sc68calmtreinish: sort of poking at it now, get my feet wet19:52
*** sabeen has quit IRC19:52
EmilienMmtreinish: I still don't see the subunit file http://logs.openstack.org/18/242718/15/check/gate-puppet-openstack-integration-scenario001-dsvm-trusty/2cc6fc7/logs/19:53
*** _nadya_ has joined #openstack-qa19:54
*** dpaterson has joined #openstack-qa19:54
EmilienMhttp://logs.openstack.org/18/242718/15/check/gate-puppet-openstack-integration-scenario001-dsvm-trusty/2cc6fc7/console.html#_2015-12-08_18_46_25_47219:55
mtreinishsc68cal: apply that to all the lists and I'm +2 on it :)19:55
EmilienMmtreinish: the nodepool image is not updated yet19:56
EmilienMso it does not copy testrepository.subunit yet19:56
mtreinishah ok19:56
*** vgridnev has quit IRC19:56
EmilienMso I can rename the job now but we won't see it yet19:56
EmilienMmtreinish: the name is now gate-puppet-openstack-integration-scenario001-dsvm-trusty19:56
*** _nadya_ has quit IRC19:56
EmilienMwe want gate-tempest-puppet-openstack-integration-scenario001-dsvm-trusty ?19:56
*** _nadya_ has joined #openstack-qa19:57
*** aysyd has quit IRC19:57
mriedemjgriffith: on http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/logs/screen-n-cpu.txt.gz#_2015-11-30_18_20_33_948 - that could just be when tempest times out and starts deleting the instance19:57
mtreinishEmilienM: ok sure, yeah we need that to match: https://github.com/openstack-infra/system-config/blob/master/modules/openstack_project/files/logstash/jenkins-log-client.yaml#L1719:57
mriedemjgriffith: that warning comes from some vm lifecycle event handling code in nova19:57
mtreinishso make it gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty I think19:57
EmilienMmtreinish: perfect19:57
mriedemjgriffith: it does seem weird though19:59
*** _nadya_ has quit IRC20:01
*** jecarey has quit IRC20:01
*** vgridnev has joined #openstack-qa20:02
EmilienMmtreinish: https://review.openstack.org/25493320:02
*** rfolco_ has quit IRC20:03
regXboimtreinish: it wasn't the fact that there was a line break, it was that it was after a line that ran over the gerrit limit20:03
*** pglass has quit IRC20:03
*** pglbutt has joined #openstack-qa20:03
regXboithere were enough other examples (Ls 28, 29, 39, 42, 45, 48, 54, 60) that it got on my nerves20:04
mriedemjgriffith: this is also recently spiking: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message:%5C%22Instance%20shutdown%20by%20itself.%20Calling%20the%20stop%20API.%20Current%20vm_state:%20active,%20current%20task_state:%20None,%20original%20DB%20power_state:%201,%20current%20VM%20power_state:%204%5C%22%20AND%20tags:%5C%22screen-n-cpu.txt%5C%2220:05
mriedemnvm20:05
mriedemthere are older hits20:05
*** pglass_ has joined #openstack-qa20:05
*** artom has quit IRC20:05
mtreinishEmilienM: cool, +120:06
regXboimtreinish: - I explained my annoyance and pulled the -1 on 243703 ... I find it visual noise, but it's not worth a fight20:07
mtreinishregXboi: oh, ok. I was fine with your -1 there, it was just an open question on the review20:08
mtreinishI was just going through the open reviews and saw that20:08
mriedemjgriffith: i think you might be onto something here,20:08
regXboimtreinish: ack - I wasn't clear initially and that's my bad20:08
*** pglbutt has quit IRC20:08
mriedemjgriffith: the timestamps show that before that warning in the n-cpu logs that the instance is unexpectedly stopped in libvirt, we were trying to attach a floating ip20:08
mriedemhttp://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/console.html.gz#_2015-11-30_18_26_19_25120:08
mriedem2015-11-30 18:26:19.251 |     2015-11-30 18:19:45,684 8435 INFO     [tempest_lib.common.rest_client] Request (TestVolumeBootPattern:test_volume_boot_pattern): 202 POST http://127.0.0.1:8774/v2.1/9617451b1b804321a9015e4f4751784d/servers/6eb2b6c7-1033-4af5-bea4-c380138308b8/action 1.324s20:09
mriedemshortly after that, the instance is stopped on the hypervisor http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/logs/screen-n-cpu.txt.gz#_2015-11-30_18_20_33_94820:09
mriedemsdague: ^20:10
mriedemrelated to the test_volume_boot_pattern ssh bugs20:10
mriedemi wonder if libvirt is puking on the interface attach and stopping the vm20:10
*** lucasagomes is now known as lucas-dinner20:13
mriedemsdague: what does this mean to you? the 'failed, not retrying' part: http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/logs/screen-n-net.txt.gz#_2015-11-30_18_19_45_25220:14
mriedemthat's when attaching the floating IP to the instance20:14
*** jecarey has joined #openstack-qa20:14
mtreinishmriedem: well, it's not an error. Maybe the rule already exists?20:17
mtreinishnah, that's unlikely20:17
*** itsuugo_ has joined #openstack-qa20:18
lifelesssdague: can you file a bug on how to reproduce it? on bugs.python.org (cc me)20:20
sdagueyeh, I'll see if I can create a small reproduce on it tomorrow20:20
mriedembingo http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/logs/libvirt/libvirtd.txt.gz#_2015-11-30_18_20_18_16820:21
mriedemjgriffith: ^ that's what made http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/logs/screen-n-cpu.txt.gz#_2015-11-30_18_20_33_94820:21
*** itzikb_ has quit IRC20:31
*** asselin__ has quit IRC20:42
*** pglass_ has quit IRC20:42
*** pglass has joined #openstack-qa20:42
*** pglbutt has joined #openstack-qa20:43
*** pglass has quit IRC20:47
openstackgerritMerged openstack/openstack-health: Update charts to format data correctly  https://review.openstack.org/24370320:49
*** sshnaidm has quit IRC20:52
andreafmtreinish: nice: http://status.openstack.org/openstack-health/#/test/tempest.api.compute.servers.test_server_actions.ServerActionsTestJSON.test_reboot_server_hard20:54
*** jecarey has quit IRC20:54
mtreinishandreaf: heh, yeah it's pretty cool20:55
mtreinishthe only thing is I think I need to shrink the date window on the per test pages20:55
mtreinishit kinda brings firefox to a halt for me. Chrome works fine though20:55
mtreinishthere's a lot of data being graphed20:55
andreafsafari had no issue :D20:56
* mtreinish mutters under his breath20:56
andreafonly on the 2ns graph one I roll over with the mouse, it tells me NaN/NaN/NaN/NaN20:56
andreafactually NaN/NaN/0NaN20:56
mtreinishandreaf: hmm, well that's not right20:57
mtreinishandreaf: that's a good bug for you to fix :)20:57
mtreinishmy guess is I screwed something up in the stacked area graph directive I added for that graph20:58
andreafmtreinish :D I can try ;)20:58
mtreinishandreaf: file a bug on lp first so we can track it20:58
andreafmtreinish: yep will do so - i have to take my son to bed now - nice see the test detail page up, great work on that!20:59
mtreinishthanks20:59
*** _nadya_ has joined #openstack-qa21:01
*** _nadya_ has quit IRC21:06
*** rfolco_ has joined #openstack-qa21:13
*** itzikb_ has joined #openstack-qa21:16
jgriffithmriedem: nice work!21:17
mriedemjgriffith: so i don't know what's causing that, i found an old red hat bugzilla and re-opened since it was the same issue21:18
mriedemguests randomly crashing21:18
jgriffithmriedem: so I looked at a handful of the ssh failures on boot tetss and that seemed to be a common thread21:18
*** rfolco_ has quit IRC21:19
mtreinishmriedem: well guests crashing is a good explanation as to why we can't login to them :)21:20
mriedemmtreinish: the guest crash is actually a bit later, but yeah21:21
mriedemmtreinish: i suspect when we're trying to attach the floating IP to the guest, things go bonkers in libvirt/qemu, something crashes, and we can't ssh in21:21
mriedemand then we get the lifecycle event in n-cpu to stop the vm21:21
mriedembefore tempest actually sends the delete request21:21
*** jasonsb has joined #openstack-qa21:23
*** jasonsb has quit IRC21:23
*** jasonsb has joined #openstack-qa21:24
jgriffithmriedem: mtreinish so small sample does seem to coincide with assign floating ip21:28
jgriffithmriedem: mriedem at least within the minute21:29
*** TuanNguyen has quit IRC21:30
*** dims has joined #openstack-qa21:32
*** mfisher_ora has quit IRC21:44
*** dspano has joined #openstack-qa21:48
*** salv-orlando has joined #openstack-qa21:50
*** salv-orlando has quit IRC21:55
*** dims has quit IRC21:59
*** nichola__ has joined #openstack-qa22:00
*** nicholasgoracke has quit IRC22:04
*** salv-orlando has joined #openstack-qa22:05
boris-42_mtreinish: hi22:05
boris-42_mtreinish: thanks for OSprofiler spec review22:05
boris-42_mtreinish: could you please take a look at my reply22:06
*** jogo has quit IRC22:07
mtreinishboris-42_: your reply makes sense. To have osprofiler be the tracing collection layer and have the backend its pushing to be abstracted22:07
mtreinishboris-42_: but that wasn't entirely clear in my reading of the spec, it might be something to elaborate a bit more on in the next rev22:07
*** itsuugo_ has quit IRC22:08
*** david-ly_ has joined #openstack-qa22:10
*** josdotso has joined #openstack-qa22:11
*** jogo has joined #openstack-qa22:12
*** david-lyle has quit IRC22:13
*** david-ly_ is now known as david-lyle22:13
boris-42_mtreinish: so what you would like to see in spec more? and where?22:18
*** maht has quit IRC22:19
*** maht has joined #openstack-qa22:19
*** BadCub has left #openstack-qa22:20
*** e0ne has quit IRC22:26
*** Swami_ has joined #openstack-qa22:27
*** Swami has quit IRC22:27
*** dustins is now known as dustins|gone22:27
mtreinishboris-42_: the spec is mostly concentrating on how trace points work. Just having prose explaining the exact scope of the lib and how it fits in with other things out there22:30
mtreinishboris-42_: I also think mentioning zipkin in the alternatives section to explain their complementary and osprofiler can be used with a zipkin backend will be good too22:30
andreafmtreinish: https://bugs.launchpad.net/openstack-health/+bug/152408822:32
openstackLaunchpad bug 1524088 in OpenStack Health "NaN on 2nd graph on the test details page" [Undecided,New]22:32
mtreinishandreaf: ooh, look at that a screenshot too :)22:33
*** jordanP has quit IRC22:35
*** shardy has quit IRC22:41
*** pglass has joined #openstack-qa22:42
*** pglbutt has quit IRC22:45
*** _nadya_ has joined #openstack-qa22:50
*** salv-orlando has quit IRC22:53
*** salv-orlando has joined #openstack-qa22:54
*** _nadya_ has quit IRC22:55
mriedemsc68cal: was the linuxbridge job just made voting in the last few days?22:56
*** cp16net has quit IRC22:56
*** samueldmq has quit IRC22:56
*** nichola__ has quit IRC22:56
*** devananda has quit IRC22:57
*** jesusaurus has quit IRC22:57
*** dkranz has quit IRC22:58
*** itzikb_ has quit IRC23:04
openstackgerritMerged openstack/tempest: Fix some Typo in docstrings  https://review.openstack.org/25399723:07
*** josdotso has quit IRC23:07
*** mriedem is now known as mriedem_away23:09
openstackgerritMerged openstack/tempest: Use interfaces_client from tempest-lib  https://review.openstack.org/25274723:12
openstackgerritMerged openstack/tempest: Use interfaces_client from tempest-lib  https://review.openstack.org/25274723:12
openstackgerritMerged openstack/tempest: [V3] Rename get_credential to show_credential  https://review.openstack.org/25410423:12
openstackgerritMerged openstack/tempest: [V3] Rename get_region to show_region  https://review.openstack.org/25413323:12
*** mkoderer has quit IRC23:15
*** mkoderer has joined #openstack-qa23:17
*** regXboi has quit IRC23:18
*** dspano has quit IRC23:18
*** edmondsw has quit IRC23:21
*** yamamoto has joined #openstack-qa23:23
*** pglass has quit IRC23:31
*** tochi has joined #openstack-qa23:36
*** alex_xu has quit IRC23:36
*** yamamoto has quit IRC23:40
*** alex_xu has joined #openstack-qa23:41
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Fix T110 violation for database client  https://review.openstack.org/25401023:42
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Remove unused add_sample()  https://review.openstack.org/25500523:48
*** _dpaterson has joined #openstack-qa23:51
*** sabeen3 has quit IRC23:55

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