mtreinish | timothyb89: ^^^ try the updated branch it should correctly display no data now | 00:00 |
---|---|---|
*** rossella_s has quit IRC | 00:02 | |
*** salv-orl_ has quit IRC | 00:03 | |
*** jamielennox|away is now known as jamielennox | 00:05 | |
*** pglass has quit IRC | 00:08 | |
*** salv-orlando has joined #openstack-qa | 00:09 | |
mtreinish | jhesketh, 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 view | 00:14 |
mtreinish | jhesketh, regXboi, david-lyle, timothyb89: also: https://review.openstack.org/#/c/253214/ should be quick | 00:15 |
*** yamamoto has joined #openstack-qa | 00:26 | |
*** yamamoto_ has joined #openstack-qa | 00:29 | |
*** yamamoto has quit IRC | 00:32 | |
*** yamamot__ has joined #openstack-qa | 00:32 | |
*** patrickeast has quit IRC | 00:32 | |
*** mtanino has quit IRC | 00:33 | |
*** yamamoto_ has quit IRC | 00:34 | |
*** eil397 has quit IRC | 00:34 | |
*** dougwig has quit IRC | 00:35 | |
*** dougwig has joined #openstack-qa | 00:36 | |
*** apevec has quit IRC | 00:36 | |
*** yamamot__ has quit IRC | 00:37 | |
*** Swami has quit IRC | 00:37 | |
*** zz_dimtruck is now known as dimtruck | 00:38 | |
*** asselin__ has quit IRC | 00:39 | |
*** asselin__ has joined #openstack-qa | 00:39 | |
stevemar | mtreinish: thanks for clarifying about how often eventlet is tested | 00:40 |
*** jdandrea has joined #openstack-qa | 00:40 | |
HenryG | mtreinish: hi | 00:42 |
HenryG | mtreinish: I took a look at your model sync failures | 00:43 |
HenryG | mtreinish: I couldn't figure it out :( | 00:44 |
HenryG | mtreinish: you might ask akamyshnikova or zzzeek for help | 00:45 |
HenryG | mtreinish: I would be interested to learn more about this | 00:46 |
*** jasonsb has quit IRC | 00:47 | |
*** sabeen3 has quit IRC | 00:54 | |
mtreinish | stevemar: 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 there | 00:56 | |
mtreinish | HenryG: ok, I'll follow up with them tomorrow | 00:57 |
mtreinish | HenryG: I'm sure that I'm just doing something stupid :) | 00:57 |
HenryG | mtreinish: no, I think you have some complicated foreign keys that I don't understand. | 00:58 |
HenryG | Or I should say complicated to me because I don't understand them | 00:58 |
mtreinish | HenryG: 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 api | 00:58 |
mtreinish | that's probably throwing things off | 00:59 |
HenryG | right | 00:59 |
mtreinish | I needed to update the db api to use explicit joins anyway so this is probably a good chance to fix that | 00:59 |
HenryG | where 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 migrations | 01:00 |
HenryG | or the other way around, I can't remember | 01:01 |
HenryG | also I don't understand the FK it is suggesting to add | 01:02 |
mtreinish | HenryG: I think it would probably be the other way around. The fks are missing from the migrations | 01:02 |
HenryG | yup, just checked, that fixed those | 01:04 |
*** hemna is now known as hemnafk | 01:04 | |
HenryG | guess what 'modify_nullable' means :) | 01:05 |
mtreinish | hah | 01:07 |
HenryG | mtreinish: so I can fix the models to the point of no diffs | 01:09 |
HenryG | but 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-qa | 01:10 | |
mtreinish | HenryG: 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 size | 01:15 |
mtreinish | HenryG: 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 key | 01:16 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Change docstring of compute client https://review.openstack.org/254043 | 01:16 |
*** armax has quit IRC | 01:17 | |
notmorgan | mtreinish: what?! | 01:18 |
notmorgan | WhT did I do? | 01:18 |
notmorgan | Oh ahahaha. Eventlet. Lol | 01:18 |
notmorgan | Yeeaaaahhh | 01:18 |
openstackgerrit | Jacky_lei_zhang proposed openstack/tempest: Make volume_types_client use **kwargs https://review.openstack.org/250228 | 01:18 |
*** pvaneck has quit IRC | 01:20 | |
*** nithyag_ has joined #openstack-qa | 01:21 | |
*** NithyaG has quit IRC | 01:23 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest-lib: Migrated floating_ips_client.py from tempest https://review.openstack.org/254007 | 01:23 |
*** coolsvap has quit IRC | 01:23 | |
*** salv-orlando has quit IRC | 01:25 | |
*** coolsvap has joined #openstack-qa | 01:25 | |
*** dimtruck is now known as zz_dimtruck | 01:28 | |
*** patrickeast has joined #openstack-qa | 01:39 | |
*** akshai has quit IRC | 01:51 | |
*** hichihara has quit IRC | 01:55 | |
*** eil397 has quit IRC | 01:55 | |
*** rfolco_ has quit IRC | 01:56 | |
*** jasonsb has joined #openstack-qa | 02:14 | |
openstackgerrit | Jacky_lei_zhang proposed openstack/tempest: [V3] Rename get_policy to show_policy https://review.openstack.org/254127 | 02:17 |
openstackgerrit | Jacky_lei_zhang proposed openstack/tempest: [V3] Rename get_region to show_region https://review.openstack.org/254133 | 02:19 |
*** yangyape_ has joined #openstack-qa | 02:22 | |
*** tochi has quit IRC | 02:38 | |
openstackgerrit | Bin Bin Cong proposed openstack/tempest: Add keystone v3 user negative cases for user authentication https://review.openstack.org/254164 | 02:41 |
openstackgerrit | Merged openstack/openstack-health: Make date window for job pages longer for periodic jobs https://review.openstack.org/253214 | 02:41 |
*** hitalia has quit IRC | 02:45 | |
*** bkopilov has quit IRC | 02:50 | |
*** yamahata has quit IRC | 02:51 | |
*** akshai has joined #openstack-qa | 03:04 | |
*** zz_dimtruck is now known as dimtruck | 03:05 | |
*** edmondsw has quit IRC | 03:07 | |
*** shuquan has joined #openstack-qa | 03:23 | |
*** dimtruck is now known as zz_dimtruck | 03:24 | |
*** tochi has joined #openstack-qa | 03:24 | |
*** shuquan has quit IRC | 03:28 | |
*** binbincong has quit IRC | 03:28 | |
*** yamamoto has joined #openstack-qa | 03:29 | |
*** salv-orlando has joined #openstack-qa | 03:29 | |
*** shuquan has joined #openstack-qa | 03:31 | |
*** sabeen1 has joined #openstack-qa | 03:33 | |
*** salv-orlando has quit IRC | 03:34 | |
*** tochi has quit IRC | 03:49 | |
*** tochi has joined #openstack-qa | 03:50 | |
*** binbincong has joined #openstack-qa | 03:50 | |
*** jecarey has joined #openstack-qa | 03:51 | |
*** itzikb_ has joined #openstack-qa | 03:53 | |
*** nicholasgoracke has joined #openstack-qa | 03:54 | |
*** flwang1 has quit IRC | 03:55 | |
*** jecarey has quit IRC | 03:56 | |
*** jecarey has joined #openstack-qa | 04:04 | |
*** eliqiao has joined #openstack-qa | 04:06 | |
*** eliqiao_ has quit IRC | 04:06 | |
*** bkopilov has joined #openstack-qa | 04:20 | |
*** shuquan has quit IRC | 04:25 | |
*** shuquan has joined #openstack-qa | 04:26 | |
openstackgerrit | Ian Wienand proposed openstack-dev/devstack: Only use lsb_release for distro tags https://review.openstack.org/117104 | 04:32 |
*** boris-42_ has quit IRC | 04:43 | |
*** masayukig has quit IRC | 04:47 | |
openstackgerrit | Ian Wienand proposed openstack-dev/devstack: libvirt: don't repeatedly configure libvirtd logging https://review.openstack.org/254111 | 04:48 |
*** shuquan has quit IRC | 04:50 | |
*** flwang1 has joined #openstack-qa | 04:52 | |
*** yamahata has joined #openstack-qa | 04:53 | |
*** masayukig has joined #openstack-qa | 04:54 | |
*** flwang1 has quit IRC | 05:13 | |
*** anshul has quit IRC | 05:14 | |
*** sabeen3 has joined #openstack-qa | 05:16 | |
*** sabeen1 has quit IRC | 05:19 | |
*** amotoki has joined #openstack-qa | 05:20 | |
*** Poornima has joined #openstack-qa | 05:21 | |
*** akshai has quit IRC | 05:22 | |
*** vgridnev has joined #openstack-qa | 05:24 | |
*** shuquan has joined #openstack-qa | 05:30 | |
*** itzikb_ has quit IRC | 05:32 | |
*** itzikb_ has joined #openstack-qa | 05:34 | |
*** itzikb_ has quit IRC | 05:36 | |
*** hoangcx has joined #openstack-qa | 05:38 | |
*** hoangcx has left #openstack-qa | 05:38 | |
*** shuquan has quit IRC | 05:46 | |
*** shuquan has joined #openstack-qa | 05:47 | |
*** amotoki has quit IRC | 05:48 | |
*** jaypipes has joined #openstack-qa | 05:57 | |
*** ChanServ sets mode: +o jaypipes | 05:57 | |
*** dims has quit IRC | 05:57 | |
*** yamahata has quit IRC | 05:58 | |
*** bochi-michael has joined #openstack-qa | 05:58 | |
*** yamahata has joined #openstack-qa | 05:58 | |
*** yamahata has quit IRC | 06:04 | |
*** yamahata has joined #openstack-qa | 06:09 | |
*** shuquan has quit IRC | 06:15 | |
*** gamado has joined #openstack-qa | 06:16 | |
*** _nadya_ has joined #openstack-qa | 06:20 | |
*** dims has joined #openstack-qa | 06:20 | |
*** _nadya_ has quit IRC | 06:25 | |
*** yangyape_ is now known as Yangyapeng | 06:28 | |
*** amotoki has joined #openstack-qa | 06:30 | |
*** nicholasgoracke has quit IRC | 06:31 | |
openstackgerrit | Anusha Ramineni proposed openstack-dev/devstack: Add congress to plugin registry https://review.openstack.org/254570 | 06:35 |
*** shuquan has joined #openstack-qa | 06:38 | |
*** e0ne has joined #openstack-qa | 06:38 | |
*** yfried_ has quit IRC | 06:42 | |
openstackgerrit | Merged openstack/os-testr: Fix coverage section in tox.ini https://review.openstack.org/253407 | 06:43 |
*** maht has quit IRC | 06:43 | |
*** e0ne has quit IRC | 06:45 | |
*** eliqiao has quit IRC | 06:48 | |
*** salv-orlando has joined #openstack-qa | 06:49 | |
*** SridharG has joined #openstack-qa | 06:49 | |
*** e0ne has joined #openstack-qa | 06:49 | |
*** e0ne has quit IRC | 06:52 | |
*** salv-orlando has quit IRC | 06:54 | |
*** sabeen3 has quit IRC | 06:54 | |
*** e0ne has joined #openstack-qa | 07:02 | |
*** itzikb_ has joined #openstack-qa | 07:03 | |
*** hitalia has joined #openstack-qa | 07:07 | |
*** eliqiao has joined #openstack-qa | 07:07 | |
*** dims has quit IRC | 07:11 | |
*** _nadya_ has joined #openstack-qa | 07:12 | |
*** e0ne has quit IRC | 07:20 | |
*** e0ne has joined #openstack-qa | 07:21 | |
*** shuquan has quit IRC | 07:22 | |
*** yfried_ has joined #openstack-qa | 07:23 | |
*** anshul has joined #openstack-qa | 07:25 | |
*** shuquan has joined #openstack-qa | 07:25 | |
openstackgerrit | Atsushi SAKAI proposed openstack-dev/devstack: [WIP] Add sleep time measurement https://review.openstack.org/254591 | 07:30 |
*** dims has joined #openstack-qa | 07:31 | |
openstackgerrit | Piyush Pathak proposed openstack/tempest: Make v3 create_trust use **kwargs & doc string https://review.openstack.org/254596 | 07: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 |
oomichi | yfried_: because neutron-lbaas repo contains the same tests | 07:39 |
yfried_ | oomichi: I couldn't find the scenario there | 07:39 |
yfried_ | oomichi: only the API | 07:39 |
yfried_ | oomichi: also, it was my understanding that scenarios (integration tests) should remain in tempest | 07:40 |
*** shardy has joined #openstack-qa | 07:40 | |
*** flwang1 has joined #openstack-qa | 07:41 | |
*** jaypipes has quit IRC | 07:44 | |
*** gsagie has left #openstack-qa | 07:47 | |
gmann | yfried_: it is there - https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/tests/tempest/v2/scenario/test_load_balancer_basic.py | 07:50 |
*** dims has quit IRC | 07:52 | |
*** shuquan has quit IRC | 07:53 | |
*** shuquan has joined #openstack-qa | 07:54 | |
*** tochi_ has joined #openstack-qa | 07:55 | |
*** e0ne has quit IRC | 07: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 IRC | 08:06 | |
gmann | yfried_: tox -e scenario | 08:08 |
gmann | yfried_: I think this way - https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/tests/tempest/README.rst | 08:08 |
gmann | yfried_: but yea, regarding more scenario testing plan etc, neutron team might have as they own those tests now | 08: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-qa | 08:15 | |
*** flwang1 has quit IRC | 08:15 | |
gmann | yfried_: 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 manually | 08:16 |
*** _nadya_ has quit IRC | 08:16 | |
*** eliqiao has quit IRC | 08:18 | |
*** sree has joined #openstack-qa | 08:19 | |
*** sshnaidm has joined #openstack-qa | 08:20 | |
andreaf | oomichi, 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 |
openstackgerrit | Piyush Pathak proposed openstack/tempest: Make v3 create_trust use **kwargs & doc string https://review.openstack.org/254596 | 08:25 |
oomichi | andreaf: nice point. it is easy to implement/review each step with separated patches, I feel. | 08:26 |
oomichi | andreaf: hard to cover all thing in a single patch for me :( | 08:26 |
andreaf | oomichi: ok - yes it makes sense - I will approve this one then | 08:26 |
gmann | andreaf: oomichi +1 | 08:27 |
oomichi | andreaf: thanks :) | 08:27 |
*** sree_ has joined #openstack-qa | 08:30 | |
*** sree_ is now known as Guest18653 | 08:30 | |
gmann | andreaf: regarding https://review.openstack.org/#/c/253380/ | 08:32 |
gmann | andreaf: honestly i feel we should support None also, as i saw people use to uncomment the default value (during bug triaging) | 08:33 |
gmann | andreaf: 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 go | 08:34 |
masayukig | oomichi: regarding https://review.openstack.org/254043 | 08:34 |
gmann | andreaf: as per your comment, you mean to mention support of 'None' in config.py as help or comment? | 08:34 |
*** sree has quit IRC | 08:34 | |
masayukig | oomichi: there's a lot of functions which have **kwargs parameter, actually. | 08:35 |
andreaf | gmann: yes I think it would be nice to do that (add a comment in config.py) | 08:35 |
masayukig | oomichi: but you don't put the api docs. So that I'd like to know the criteria to put the API url. | 08:36 |
openstackgerrit | Sean M. Collins proposed openstack-infra/elastic-recheck: Introduce query for 1523638 https://review.openstack.org/254610 | 08:36 |
openstackgerrit | FelixLi proposed openstack/tempest: Remove wrong comment https://review.openstack.org/254160 | 08:36 |
gmann | andreaf: ok. I will try to push that in oslo later. I saw oslo does convert empty string into None for integer type options | 08:37 |
gmann | andreaf: and in str type too None should be ok instead of 'None' | 08:37 |
*** dtantsur|afk is now known as dtantsur | 08:37 | |
oomichi | masayukig: basically, we need to put api doc links for POST/PUT methods as http://lists.openstack.org/pipermail/openstack-dev/2015-July/068864.html | 08:37 |
oomichi | masayukig: and your review extends the scope ;) | 08:38 |
oomichi | masayukig: for GET also | 08:38 |
oomichi | masayukig: yesterday | 08:38 |
oomichi | masayukig: to be honest, I don't have criteria for GET yet | 08:38 |
*** _nadya_ has joined #openstack-qa | 08:39 | |
*** _nadya_ has quit IRC | 08:39 | |
oomichi | masayukig: ideally, it is nice to have some pep8 rule which requires api doc links if method contains "params" | 08:39 |
oomichi | in its arguments | 08:40 |
andreaf | gmann: 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 string | 08:41 |
andreaf | gmann: but I guess <None> would be fair enough, as it's what None is rendered to by the config file generator | 08:41 |
masayukig | oomichi: Thanks, I got it :) So I feel the criteria should be in the commit message. | 08:41 |
*** yamahata has quit IRC | 08:43 | |
*** vgridnev has quit IRC | 08:43 | |
oomichi | masayukig: nice idea, please -1 on that ;) | 08:43 |
*** ifarkas has joined #openstack-qa | 08:43 | |
oomichi | masayukig: it is easy to know with -1 for fixing it later | 08:43 |
masayukig | oomichi: and actually, there's a lot of 'action methods', though. | 08:44 |
masayukig | oomichi: ok | 08:44 |
*** jlanoux has joined #openstack-qa | 08:44 | |
masayukig | andreaf: 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 |
andreaf | masayukig: thanks | 08:46 |
openstackgerrit | Sean M. Collins proposed openstack-infra/elastic-recheck: Introduce query for 1523638 https://review.openstack.org/254610 | 08:47 |
oomichi | masayukig: thanks! | 08:48 |
openstackgerrit | Sean M. Collins proposed openstack-infra/elastic-recheck: Introduce query for 1523638 https://review.openstack.org/254610 | 08:49 |
gmann | andreaf: humm, yea i also cannot see 'None' as expected string for any options. | 08:54 |
*** eliqiao_ is now known as eliqiao | 08:54 | |
gmann | yfried_: exactly :) | 08:54 |
*** sree has joined #openstack-qa | 09:02 | |
*** e0ne has joined #openstack-qa | 09:03 | |
*** Guest18653 has quit IRC | 09:05 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Handle StrOpt config option of microversion https://review.openstack.org/253380 | 09:08 |
gmann | andreaf: oomichi ^^ please check. added Notes in config.py | 09:09 |
*** safchain has joined #openstack-qa | 09:09 | |
oomichi | gmann: I am seeing | 09:10 |
gmann | oomichi: thanks | 09:11 |
andreaf | gmann: thanks, I would put a comment in the "help" section actually, because that's what someone configuring tempest will see in tempest.conf.sample | 09:11 |
oomichi | gmann: isn't it good to put the NOTE into api_version_requrest.py instead of config.py? | 09:11 |
andreaf | oomichi, 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 |
andreaf | oomichi, gmann: the rest of the note I agree can be in api_version_request.py | 09:14 |
*** apevec has joined #openstack-qa | 09:14 | |
*** apevec has quit IRC | 09:14 | |
*** apevec has joined #openstack-qa | 09:14 | |
gmann | andreaf: oomichi i am affraid if that create confusion for 'None' as something special value than None | 09:14 |
gmann | andreaf: oomichi yea for api_version_request.py, it makes sense | 09:14 |
gmann | andreaf: oomichi or adding like the ..."string 'latest' means highest supported version and the string 'None' or None (by default) means No version..." | 09:16 |
gmann | andreaf: oomichi so that it is clear that None and 'None' are same | 09:17 |
andreaf | ok, sounds good to me | 09:17 |
gmann | andreaf: ok. i will update | 09:17 |
*** shuquan has quit IRC | 09:20 | |
*** sree has quit IRC | 09:23 | |
*** shuquan has joined #openstack-qa | 09:23 | |
*** sree has joined #openstack-qa | 09:23 | |
oomichi | gmann: I am gussing this problem seems common for the other config items, right? | 09:23 |
gmann | oomichi: yea if we consider case of having explicitly None in config file | 09:24 |
oomichi | gmann: so I am feeling now we should not pass "None" for these config items if users want to use None like the other configs | 09:25 |
oomichi | gmann: current change can be inconsistent between all configs | 09:25 |
oomichi | gmann: as your NOTE, this kind of thing should be handled on common place like oslo.config | 09:26 |
gmann | oomichi: yes, but i think we cannot say setting None explicitly in config is not valid | 09:27 |
gmann | oomichi: yea, i also feel that oslo should return None not 'None' | 09:27 |
oomichi | gmann: how about writing it on the config "help" ? | 09:27 |
gmann | oomichi: you mean current Note ? | 09:28 |
oomichi | gmann: no, in cfg.StrOpt(help=<here>) | 09:29 |
gmann | oomichi: yea i mean putting that NOTE in help right? | 09:29 |
oomichi | gmann: That is too detail as help. so I'd like to consider it more. | 09:30 |
gmann | oomichi: in help how about ..."string 'latest' means highest supported version and the string 'None' or None (by default) means No version..." | 09:30 |
gmann | oomichi: andreaf but will anyone mention 'None' ? | 09:30 |
gmann | oomichi: andreaf that is actually internally return from oslo and we handle that | 09:31 |
oomichi | gmann: how about | 09:31 |
oomichi | gmann: "If both values are not specified, Tempest avoid tests with require a microversion." | 09:31 |
oomichi | gmann: instead of current help | 09:32 |
andreaf | oomichi: +1 :) that's actually the right way of configuring rather than specifying None | 09:32 |
gmann | oomichi: yup. that looks nice | 09:32 |
oomichi | s/avoid/avoids/ :( | 09:32 |
andreaf | oomichi, gmann: what about if string "None" is specified log a warning | 09:33 |
oomichi | andreaf: +1 for that. that is very user-friendly :) | 09:33 |
gmann | andreaf: during config options in config.py ? | 09:34 |
gmann | andreaf: oomichi yea, but do we have sending some string to oslo and they log or we do in api_versio_request | 09:35 |
*** vgridnev has joined #openstack-qa | 09:35 | |
*** hitalia has quit IRC | 09:36 | |
andreaf | gmann, oomichi: I guess in api_version_request? | 09:36 |
gmann | andreaf: yea, did not find arg in oslo | 09:37 |
oomichi | gmann: andreaf: yeah, nice to put the check in api_version_request | 09:37 |
gmann | andreaf: oomichi so how about other StrOpt, they can be mentioned as None and it breaks currently | 09:37 |
gmann | and i feel considering explicit mention of default value as invalid does not looks good even that is None | 09:38 |
gmann | but may be i am considering too generic cases :) | 09:39 |
andreaf | gmann, 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 confusion | 09:41 |
andreaf | gmann, 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 applies | 09:42 |
andreaf | gmann, oomichi: because else if we support 'None' for microversions but not everywhere else, it will be inconsistent | 09:42 |
gmann | andreaf: yea thats something main issue. either we should support everywhere or no where | 09:43 |
andreaf | gmann: if oslo folks agree to get support for 'None' in, we will change this everywhere once that lands | 09:43 |
gmann | andreaf: oomichi thats why as permanent solution i think oslo should handle that | 09:44 |
gmann | andreaf: you mean to return None not 'None' right | 09:44 |
oomichi | gmann: yeah, I really want to know feedback from oslo team related to this | 09:45 |
oomichi | gmann: this is a very nice point and seems common thing | 09:45 |
*** jordanP has joined #openstack-qa | 09:46 | |
gmann | andreaf: oomichi and i think they can consider 'None' as 'None' as it is not actually None it is string explicitly mentioned as value | 09:46 |
oomichi | gmann: that means sometimes 'None' is different from None from user expectation? | 09:48 |
gmann | oomichi: :) if that is mentioned explicitly as value in config file | 09:49 |
gmann | but m not sure those thing we need to handle which is too much support of all kind of mindset :) | 09:49 |
andreaf | :) | 09:49 |
oomichi | gmann: yeah ;) | 09:50 |
gmann | andreaf: oomichi ok so for microversion go ahead, i will update help and add NOTE (log warning) in api_version_request.py | 09:51 |
gmann | andreaf: oomichi and later i will check with oslo how they thing about supporting 'None' | 09:51 |
gmann | andreaf: oomichi is it ok? | 09:51 |
oomichi | gmann: good for me | 09:52 |
andreaf | gmann: 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 oslo | 09:52 |
andreaf | oomichi: ^^^ | 09:52 |
oomichi | andreaf: gmann: I'd like to take time for re-considering... | 09:53 |
andreaf | gmann, oomichi: so I'm ok checking if "None" was passed and raise an InvalidConfig, but I would not accept it as a valid configuration | 09:53 |
oomichi | andreaf: because after fixing it in oslo side, the code which handles "None" as valid will be trash. right? | 09:54 |
gmann | andreaf: so in that case (with current oslo behavior which change None to "None"), uncommenting default None value will be invalid ? | 09:55 |
openstackgerrit | Merged openstack/tempest: Renaming Cinder admin client files https://review.openstack.org/252432 | 09:55 |
*** StevenK has quit IRC | 09:56 | |
*** woodster_ has quit IRC | 09:56 | |
gmann | andreaf: 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 |
andreaf | gmann: yes but that's invalid for any StrOpt already anyways | 09:56 |
*** StevenK has joined #openstack-qa | 09:57 | |
andreaf | gmann, oomichi: and besides the tempest.conf.sample contains <None>, not None, so uncommenting would give the string '<None>' | 09:57 |
gmann | andreaf: ahh i sea, from config file "None" or None is same as all are string there | 09:57 |
gmann | humm | 09:57 |
andreaf | gmann; yes indeed | 09:58 |
andreaf | #max_microversion = <None> | 09:59 |
*** sree_ has joined #openstack-qa | 10:00 | |
*** sree_ is now known as Guest77522 | 10:01 | |
*** aix has joined #openstack-qa | 10:01 | |
gmann | andreaf: ohh, | 10:03 |
*** sree has quit IRC | 10:03 | |
openstackgerrit | Piyush Pathak proposed openstack/tempest: Make v3 create_trust use **kwargs & doc string https://review.openstack.org/254596 | 10:03 |
gmann | andreaf: yea i also feel now not to support "None" | 10:04 |
*** dtantsur is now known as dtantsur|brb | 10:08 | |
gmann | andreaf: 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 earlier | 10:11 |
gmann | and consider None as valid for options having default value other than None | 10:12 |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Making names of Cinder client classes consistent https://review.openstack.org/252467 | 10:12 |
gmann | or empty string ...something. | 10:13 |
*** aix has quit IRC | 10:21 | |
*** salv-orlando has joined #openstack-qa | 10:25 | |
*** salv-orlando has quit IRC | 10:30 | |
*** aix has joined #openstack-qa | 10:33 | |
*** Guest77522 has quit IRC | 10:35 | |
*** cdent has joined #openstack-qa | 10:37 | |
*** neeti has joined #openstack-qa | 10:38 | |
*** sree__ has joined #openstack-qa | 10:38 | |
*** _nadya_ has joined #openstack-qa | 10:39 | |
*** sree__ has quit IRC | 10:42 | |
*** yamamoto has quit IRC | 10:48 | |
*** neeti has quit IRC | 10:49 | |
*** hop|2 has joined #openstack-qa | 10:50 | |
*** shuquan has quit IRC | 10:51 | |
*** tosky has joined #openstack-qa | 10:52 | |
*** rfolco_ has joined #openstack-qa | 10:54 | |
*** Yangyapeng has quit IRC | 10:58 | |
*** openstackgerrit has quit IRC | 11:02 | |
*** openstackgerrit has joined #openstack-qa | 11:02 | |
*** lucas-dinner is now known as lucasagomes | 11:04 | |
*** salv-orlando has joined #openstack-qa | 11:06 | |
*** amotoki has quit IRC | 11:07 | |
*** aysyd has joined #openstack-qa | 11:09 | |
*** drptbl has joined #openstack-qa | 11:10 | |
*** bochi-michael has quit IRC | 11:11 | |
*** salv-orlando has quit IRC | 11:14 | |
openstackgerrit | Jacky_lei_zhang proposed openstack/tempest: Make volume_types_client use **kwargs https://review.openstack.org/250228 | 11:17 |
openstackgerrit | Piyush Pathak proposed openstack/tempest: Make v2 update_user_password use **kwargs https://review.openstack.org/252916 | 11:25 |
*** _nadya_ has quit IRC | 11:26 | |
*** gszasz has joined #openstack-qa | 11:27 | |
*** jlanoux has quit IRC | 11:27 | |
*** jlanoux has joined #openstack-qa | 11:27 | |
*** dtantsur|brb is now known as dtantsur | 11:31 | |
*** tochi has quit IRC | 11:35 | |
*** yamamoto has joined #openstack-qa | 11:36 | |
*** jlanoux has quit IRC | 11:40 | |
*** jlanoux has joined #openstack-qa | 11:41 | |
*** yamamoto has quit IRC | 11:41 | |
openstackgerrit | Alexander Gubanov proposed openstack/tempest: WIP: Add tests for API microversion with instance tags https://review.openstack.org/233176 | 11:44 |
*** yamamoto has joined #openstack-qa | 11:44 | |
*** yamamoto has quit IRC | 11:44 | |
openstackgerrit | Alexander Gubanov proposed openstack/tempest: WIP: Add tests for API microversion with instance tags https://review.openstack.org/233176 | 11:49 |
openstackgerrit | Vincent Hou proposed openstack/tempest: Add tempest tests for volume retype with migration https://review.openstack.org/195443 | 11:53 |
*** yamamoto has joined #openstack-qa | 11:53 | |
*** yamamoto has quit IRC | 11:53 | |
*** e0ne has quit IRC | 12:03 | |
*** e0ne has joined #openstack-qa | 12:11 | |
*** sree has joined #openstack-qa | 12:14 | |
*** sree has quit IRC | 12:18 | |
*** Liuqing has joined #openstack-qa | 12:22 | |
*** _nadya_ has joined #openstack-qa | 12:28 | |
*** shuquan has joined #openstack-qa | 12:36 | |
*** shuquan has quit IRC | 12:41 | |
*** weshay_xchat has joined #openstack-qa | 12:41 | |
openstackgerrit | Piyush Pathak proposed openstack/tempest: Make v2 enable_disable_user use **kwargs https://review.openstack.org/254734 | 12:51 |
*** yamamoto has joined #openstack-qa | 12:51 | |
*** bkopilov has quit IRC | 12:57 | |
*** yfried_ has quit IRC | 12:58 | |
*** _nadya__ has joined #openstack-qa | 12:58 | |
*** _nadya_ has quit IRC | 12:59 | |
*** yangyapeng has joined #openstack-qa | 12:59 | |
*** gmann_ has joined #openstack-qa | 13:00 | |
*** _nadya__ has quit IRC | 13:00 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove pre-request-validation from v2 image_client https://review.openstack.org/251312 | 13:05 |
*** mwagner_lap has quit IRC | 13:05 | |
*** mwagner_lap has joined #openstack-qa | 13:08 | |
*** shardy has quit IRC | 13:09 | |
*** shardy has joined #openstack-qa | 13:10 | |
*** sree has joined #openstack-qa | 13:12 | |
*** amotoki has joined #openstack-qa | 13:15 | |
*** dims has joined #openstack-qa | 13:16 | |
*** gmann_ has quit IRC | 13:23 | |
*** edmondsw has joined #openstack-qa | 13:31 | |
openstackgerrit | Steven Hardy proposed openstack-dev/devstack: Heat - revise keystone/trusts config to avoid deprecated options https://review.openstack.org/254755 | 13:34 |
*** zz_dimtruck is now known as dimtruck | 13:34 | |
*** jaypipes has joined #openstack-qa | 13:35 | |
*** ChanServ sets mode: +o jaypipes | 13:35 | |
*** Liuqing has quit IRC | 13:36 | |
*** yfried_ has joined #openstack-qa | 13:39 | |
*** yamamoto has quit IRC | 13:40 | |
*** shardy is now known as shardy_afk | 13:41 | |
*** eliqiao has quit IRC | 13:45 | |
*** eliqiao has joined #openstack-qa | 13:46 | |
*** Poornima has quit IRC | 13:46 | |
*** eliqiao has quit IRC | 13:46 | |
*** eliqiao has joined #openstack-qa | 13:46 | |
*** yamamoto has joined #openstack-qa | 13:46 | |
*** dimtruck is now known as zz_dimtruck | 13:47 | |
*** regXboi has joined #openstack-qa | 13:49 | |
*** gamado has quit IRC | 13:51 | |
*** mquin has quit IRC | 13:54 | |
*** Liuqing has joined #openstack-qa | 13:56 | |
*** yamamoto has quit IRC | 14:01 | |
*** yamamoto has joined #openstack-qa | 14:02 | |
*** akshai has joined #openstack-qa | 14:04 | |
*** Liuqing has quit IRC | 14:11 | |
openstackgerrit | Merged openstack/tempest: Make add_router_to_l3_agent use **kwargs https://review.openstack.org/252839 | 14:20 |
*** sree has quit IRC | 14:23 | |
*** yamamoto has quit IRC | 14:24 | |
*** yamamoto has joined #openstack-qa | 14:27 | |
*** mriedem has joined #openstack-qa | 14:28 | |
*** ajmiller_ has joined #openstack-qa | 14:30 | |
*** zhangjn has joined #openstack-qa | 14:32 | |
*** zhangjn has quit IRC | 14:32 | |
*** zhangjn has joined #openstack-qa | 14:32 | |
*** _nadya_ has joined #openstack-qa | 14:41 | |
*** nicholasgoracke has joined #openstack-qa | 14:42 | |
*** lucasagomes is now known as lucas-hungry | 14:42 | |
*** edmondsw has quit IRC | 14:48 | |
*** nicholasgoracke has quit IRC | 14:49 | |
*** ajmiller_ has quit IRC | 14:49 | |
*** armax has joined #openstack-qa | 14:50 | |
*** bkopilov has joined #openstack-qa | 14:51 | |
*** dims_ has joined #openstack-qa | 14:52 | |
*** jecarey has quit IRC | 14:52 | |
openstackgerrit | Jordan Pittier proposed openstack/tempest: tempest/clients.py: always set the image_client attribute https://review.openstack.org/242076 | 14:52 |
*** tnguyen has joined #openstack-qa | 14:52 | |
*** shardy_afk is now known as shardy | 14:52 | |
*** dims has quit IRC | 14:55 | |
*** _nadya__ has joined #openstack-qa | 14:56 | |
*** pglass has joined #openstack-qa | 14:56 | |
*** _nadya_ has quit IRC | 14:56 | |
*** nicholasgoracke has joined #openstack-qa | 14:57 | |
mtreinish | regXboi: 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,z | 14:58 |
mtreinish | that adds the last of the pages from the mockup and makes every link actually clickable :) | 14:59 |
*** mfisher_ora has joined #openstack-qa | 15:00 | |
*** zhangjn has quit IRC | 15:01 | |
*** dustins|gone is now known as dustins | 15:01 | |
*** mquin has joined #openstack-qa | 15:01 | |
*** yangyapeng has quit IRC | 15:01 | |
*** zz_dimtruck is now known as dimtruck | 15:02 | |
*** dkranz has joined #openstack-qa | 15:02 | |
* regXboi looks | 15:03 | |
regXboi | mtreinish: the deeds are done - any clean up can be in follow on PS | 15:06 |
*** rbrndt has joined #openstack-qa | 15:06 | |
mtreinish | regXboi: cool, thanks | 15:06 |
regXboi | mtreinish: yw | 15:07 |
*** artom has joined #openstack-qa | 15:10 | |
*** edmondsw has joined #openstack-qa | 15:13 | |
*** TuanNguyen has joined #openstack-qa | 15:14 | |
*** lucas-hungry is now known as lucasagomes | 15:17 | |
*** miqui has joined #openstack-qa | 15:17 | |
openstackgerrit | Merged openstack/openstack-health: Rework get_test_runs_for_test() rest api https://review.openstack.org/253751 | 15:19 |
openstackgerrit | Merged openstack/openstack-health: Add test detail page https://review.openstack.org/231287 | 15:20 |
openstackgerrit | Merged openstack/openstack-health: Remove the remaining mockups https://review.openstack.org/253800 | 15:20 |
andreaf | yay!! ^^^ | 15:24 |
*** tosky has quit IRC | 15:25 | |
mtreinish | andreaf: heh, yeah it should get picked up on the next puppet run | 15:25 |
*** tosky has joined #openstack-qa | 15:26 | |
*** gszasz has quit IRC | 15:27 | |
mtreinish | err, maybe. It looks like the latest runs picked up the changes: | 15:28 |
mtreinish | http://puppetboard.openstack.org/report/health.openstack.org/9dc14b2799c23ee52a5f30290f4567e1a9a4319c | 15:28 |
mtreinish | http://puppetboard.openstack.org/report/status.openstack.org/2c7ffcbcda7cd58be2cf19a2a7bbde6997f9536e | 15:28 |
mtreinish | but things didn't get updated: | 15:29 |
mtreinish | http://health.openstack.org/test_runs/tempest.api.volume.test_qos.QosSpecsV2TestJSON.test_get_qos?start_date=2015-11-01 | 15:29 |
*** gszasz has joined #openstack-qa | 15:30 | |
*** mtanino has joined #openstack-qa | 15:33 | |
andreaf | mtreinish :( | 15:34 |
mtreinish | I'll wait 1 more puppet interval and then go into debug mode :) | 15:35 |
*** SridharG has left #openstack-qa | 15:42 | |
openstackgerrit | Benny Kopilov proposed openstack/tempest: Test encrypted volumes for multi-backends https://review.openstack.org/251692 | 15:46 |
*** jecarey has joined #openstack-qa | 15:46 | |
*** patchbot has quit IRC | 15:48 | |
*** patchbot has joined #openstack-qa | 15:48 | |
sdague | jgriffith I just noticed that bugs 2 - 6 on ER are all volumes issues | 15:50 |
sdague | those are the reason we have a gate backup right now | 15:51 |
jgriffith | sdague: I'll take a look | 15:51 |
*** tnguyen has left #openstack-qa | 15:51 | |
sdague | cool | 15:51 |
sdague | it's really unclear to me what's going on, other than volumes just not completing | 15:51 |
jgriffith | sdague: wow... some "old" friends have come back it seems | 15:52 |
sdague | http://logs.openstack.org/92/253792/7/check/gate-grenade-dsvm-multinode/d52d36a/logs/grenade.sh.txt.gz#_2015-12-07_21_02_47_717 | 15:52 |
sdague | yeh, it's mostly things don't work | 15:52 |
sdague | but it seems like it's hitting both bootfrom volume attempts on liberty | 15:53 |
sdague | at least if they run at the same time | 15:53 |
jgriffith | sdague: have you looked to see if it's mostly multi-node already? | 15:54 |
jgriffith | sdague: in other words save me the leg work if I should just start there ;) | 15:54 |
jgriffith | sdague: NM, looking in logstash now | 15:55 |
sdague | http://tinyurl.com/pdy7dll | 15:55 |
sdague | so, it's super rampant in multinode | 15:55 |
sdague | that's where I've noticed it | 15:55 |
sdague | and clearly has started climbing a lot of late | 15:55 |
jgriffith | sdague: perfect, thanks. I'll get a setup going and see what's up. And yes, the trend is fairly obvious | 15:56 |
*** vgridnev has quit IRC | 16:00 | |
*** itzikb_ has quit IRC | 16:03 | |
*** salv-orlando has joined #openstack-qa | 16:03 | |
*** salv-orlando has quit IRC | 16:03 | |
*** salv-orlando has joined #openstack-qa | 16:03 | |
*** zackf has quit IRC | 16:04 | |
*** yamamoto has quit IRC | 16:04 | |
*** hemnafk is now known as hemna | 16:06 | |
*** aix has quit IRC | 16:08 | |
*** yamamoto has joined #openstack-qa | 16:09 | |
*** gszasz has quit IRC | 16:09 | |
mtreinish | jgriffith: you'll like: http://status.openstack.org/openstack-health/#/test/tempest.scenario.test_volume_boot_pattern.TestVolumeBootPatternV2.test_volume_boot_pattern | 16:13 |
*** sabeen1 has joined #openstack-qa | 16:15 | |
jgriffith | mtreinish: define "like" :) | 16:15 |
jgriffith | mtreinish: we seem to have different interpretations of that word | 16:15 |
jgriffith | :) | 16:15 |
mtreinish | jgriffith: heh, I should have said you'll like that don't need black magic on the cli to get that graph anymore :) | 16:16 |
mtreinish | heh, not necessarily the page itself | 16:16 |
jgriffith | mtreinish: haha! Ok, yes... I do like the page :) | 16:17 |
*** hop|2 has quit IRC | 16:17 | |
jgriffith | mtreinish: ya'll have been busy while I've been away!! All sorts of new enhancements! | 16:17 |
mtreinish | jgriffith: heh, that page just landed, like < 1hr ago :) | 16:18 |
jgriffith | mtreinish: ha! Nice timing | 16:18 |
jgriffith | mtreinish: BTW, how are things looking with your RBD work? | 16:21 |
*** gszasz has joined #openstack-qa | 16:21 | |
mtreinish | jgriffith: my rbd work? Do you mean mriedem? :) | 16:21 |
jgriffith | mtreinish: GAHHHHH | 16:21 |
jgriffith | mtreinish: I ALMOST went an entire month without doing that!! | 16:22 |
jgriffith | mtreinish: of course it's not really fair since I haven't really been around in a few weeks | 16:22 |
jgriffith | geesh | 16:22 |
mriedem | jgriffith: just need to clean that patch up today, | 16:22 |
jgriffith | mtreinish: would you just change your nick please! :) | 16:23 |
mriedem | but now looking into these multinode failures | 16:23 |
jgriffith | I'll pay mtreinish or mriedem , whoever changes first :) | 16:23 |
*** mriedem is now known as rodermang | 16:23 | |
*** rodermang is now known as mriedem | 16:23 | |
*** sree has joined #openstack-qa | 16:24 | |
jgriffith | mriedem: cool, I"m firing up a fresh mutli-node now to look at some of those as well | 16:24 |
mtreinish | jgriffith: I think you oh mriedem some money now :) | 16:24 |
jgriffith | mriedem: let me know if you want to tag team it | 16:24 |
*** sabeen1 has quit IRC | 16:24 | |
jgriffith | mtreinish: it appears that may be the case | 16:24 |
*** sabeen2 has joined #openstack-qa | 16:24 | |
*** gszasz has quit IRC | 16:24 | |
*** salv-orl_ has joined #openstack-qa | 16:26 | |
* mtreinish can't believe he just wrote 'owe' as 'oh' | 16:26 | |
* mtreinish hides | 16:26 | |
jgriffith | mtreinish: :) | 16:26 |
jgriffith | mtreinish: shorthand... IRC slang :) | 16:27 |
mriedem | i'm glad you caught it before i said something | 16:27 |
mriedem | because i wasn't going to let that slide | 16:27 |
mtreinish | haha | 16:27 |
*** sree has quit IRC | 16:28 | |
*** salv-orlando has quit IRC | 16:29 | |
*** yamamoto has quit IRC | 16:37 | |
*** safchain has quit IRC | 16:39 | |
*** sabeen2 has quit IRC | 16:39 | |
*** sabeen has joined #openstack-qa | 16:39 | |
mriedem | jgriffith: so dansmith just noticed that the two hosts have the same hostname, -infra is debugging | 16:40 |
mriedem | this was a problem with the multi-node jobs before | 16:40 |
*** boris-42_ has joined #openstack-qa | 16:44 | |
*** armax has quit IRC | 16:44 | |
*** openstackgerrit has quit IRC | 16:47 | |
*** openstackgerrit has joined #openstack-qa | 16:47 | |
*** sshnaidm has quit IRC | 16:48 | |
*** armax has joined #openstack-qa | 16:51 | |
openstackgerrit | Jordan Pittier proposed openstack/tempest: Make volume_qos_client use **kwargs https://review.openstack.org/254864 | 16:52 |
*** zackf has joined #openstack-qa | 16:53 | |
*** vgridnev has joined #openstack-qa | 16:53 | |
*** yamahata has joined #openstack-qa | 16:58 | |
jgriffith | mriedem: there's all sorts of badness going on in some of these | 17:02 |
jgriffith | mriedem: 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_906 | 17:02 |
mriedem | jgriffith: sdague said the newer ovh/bluebox nodes are slower than hp/rax nodes, so quota races might be spiking b/c of that | 17:02 |
mriedem | not really sure though | 17:03 |
mriedem | this 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_181 | 17:03 |
jgriffith | mriedem: well, the thing is quota of 1G is never going to work out | 17:03 |
mriedem | jgriffith: hmm, there was a recent tempest change for volumes quotas | 17:03 |
* mriedem looks | 17:03 | |
*** jlanoux has quit IRC | 17:04 | |
*** alex_xu has quit IRC | 17:04 | |
mtreinish | mriedem: http://i.imgur.com/oKOjVLc.jpg they are slower | 17:04 |
mriedem | https://review.openstack.org/#/c/250137/ | 17:04 |
mtreinish | mriedem: that seems unlikely to break something | 17:05 |
*** dolpher has quit IRC | 17:05 | |
mriedem | jgriffith: 1gb is probably from a negative quota test | 17:06 |
*** anshul has quit IRC | 17:07 | |
jgriffith | Yeah, it is in fact | 17:07 |
*** drptbl has quit IRC | 17:08 | |
mriedem | so... | 17:08 |
mriedem | the quota tests should be locked | 17:08 |
mriedem | like the compute quota tests | 17:08 |
mriedem | but that's only per-tenant | 17:08 |
mriedem | there 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 up | 17:09 |
mriedem | jgriffith: yeah so the warnings/errors in c-api are just red herrings | 17:10 |
jgriffith | mriedem: yup | 17:10 |
jgriffith | mriedem: sorry about that | 17:10 |
mriedem | np | 17:10 |
jgriffith | mriedem: so there's nothing on the Cinder side besides those | 17:10 |
mtreinish | mriedem: well the default nova quota test is locked | 17:10 |
*** alex_xu has joined #openstack-qa | 17:10 | |
jgriffith | Nova however is not happy | 17:10 |
mtreinish | but the other quota tests create an isolated tenant so there shouldn't be an issue running those together | 17:11 |
jgriffith | mtreinish: I think they're fine. I just need to clean up Cinder so it doesn't trace on every one of those | 17:11 |
mriedem | yeah, definitely not a warning | 17:11 |
mriedem | *exception trace i mean | 17:12 |
jgriffith | :) | 17:12 |
mriedem | operator doesn't care about quota errors | 17:12 |
jgriffith | mriedem: I'll just dump them to a warning log message | 17:12 |
SpamapS | mtreinish: 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 |
mtreinish | SpamapS: it has a -1 | 17:13 |
*** amotoki has quit IRC | 17:14 | |
*** Swami has joined #openstack-qa | 17:15 | |
SpamapS | mtreinish: did you read the -1? | 17:16 |
*** jecarey has quit IRC | 17:16 | |
*** salv-orl_ has quit IRC | 17:16 | |
jgriffith | mriedem: 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_594 | 17:18 |
jgriffith | mriedem: that's the instance that fails in the test | 17:18 |
jgriffith | mriedem: the req never seems to show up in n-cpu | 17:19 |
jgriffith | OHHH | 17:19 |
jgriffith | wait... let me check the other node :) | 17:19 |
mtreinish | SpamapS: hmm, it looks like oomichi confused openstack-infra with openstack's internal infrastructure | 17:19 |
mtreinish | SpamapS: I think we should give oomichi a chance to respond to your replies | 17:20 |
openstackgerrit | Merged openstack/tempest: Split out Neutron metering label rules client https://review.openstack.org/253133 | 17:20 |
jgriffith | hmm... that's single node | 17:20 |
SpamapS | mtreinish: that's fine. I just don't want to sit for another month with no feedback. | 17:21 |
mriedem | jgriffith: i don't see it in n-net either | 17:22 |
*** dtantsur is now known as dtantsur|afk | 17:22 | |
*** rfolco_ has quit IRC | 17:25 | |
*** ifarkas has quit IRC | 17:25 | |
*** rfolco_ has joined #openstack-qa | 17:28 | |
*** austin81 has joined #openstack-qa | 17:31 | |
*** nicholasgoracke has quit IRC | 17:35 | |
*** gmann has quit IRC | 17:38 | |
*** gmann has joined #openstack-qa | 17:40 | |
*** _nadya__ has quit IRC | 17:42 | |
*** e0ne has quit IRC | 17:50 | |
*** hitalia has joined #openstack-qa | 18:00 | |
*** _nadya_ has joined #openstack-qa | 18:05 | |
*** _nadya_ has quit IRC | 18:05 | |
openstackgerrit | Merged openstack-dev/devstack: Neutron FWaaS: Use generated configuration files if available https://review.openstack.org/252087 | 18:07 |
*** dolpher has joined #openstack-qa | 18:08 | |
*** jasonsb has quit IRC | 18:17 | |
*** jecarey has joined #openstack-qa | 18:17 | |
*** salv-orlando has joined #openstack-qa | 18:17 | |
*** jecarey has quit IRC | 18:21 | |
*** salv-orlando has quit IRC | 18:21 | |
jgriffith | mriedem: 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_391 | 18:22 |
jgriffith | mriedem: appears to be a common entry in these failures | 18:22 |
jgriffith | mriedem: I'm unclear on what that actually means, given that the Instance is then created and reported as a volume mounted at dev/vda | 18:24 |
jgriffith | mriedem: ahh... bdm.device_name is kind of ignored now it seems | 18:26 |
jgriffith | never mind | 18:26 |
mriedem | jgriffith: yeah | 18:27 |
mriedem | for the libvirt driver anyway | 18:27 |
*** jaypipes has quit IRC | 18:27 | |
*** eil397 has joined #openstack-qa | 18:29 | |
*** dims_ has quit IRC | 18:30 | |
sdague | lifeless: 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 why | 18:31 |
*** pglass has quit IRC | 18:33 | |
*** nicholasgoracke has joined #openstack-qa | 18:36 | |
*** dramalho has joined #openstack-qa | 18:38 | |
*** dramalho has quit IRC | 18:38 | |
*** e0ne has joined #openstack-qa | 18:39 | |
jgriffith | mriedem: 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_948 | 18:39 |
jgriffith | mriedem: appears the Instance shutdown while trying to do the ssh | 18:40 |
jgriffith | seems to coorelate with a number of failures in logstash | 18:41 |
*** cdent has quit IRC | 18:45 | |
*** sabeen has quit IRC | 18:46 | |
*** pvaneck has joined #openstack-qa | 18:48 | |
*** jecarey has joined #openstack-qa | 18:48 | |
*** david-ly_ has joined #openstack-qa | 18:50 | |
*** david-ly_ is now known as david-lyle_ | 18:52 | |
*** eliqiao has quit IRC | 18:52 | |
*** tosky has quit IRC | 18:52 | |
*** jecarey has quit IRC | 18:53 | |
*** jecarey has joined #openstack-qa | 18:53 | |
*** david-lyle has quit IRC | 18:54 | |
*** austin81 has left #openstack-qa | 18:54 | |
*** eliqiao has joined #openstack-qa | 18:55 | |
*** jecarey_ has joined #openstack-qa | 18:58 | |
*** nicholasgoracke has quit IRC | 18:59 | |
*** jasonsb has joined #openstack-qa | 19:00 | |
*** jecarey has quit IRC | 19:02 | |
*** openstackgerrit has quit IRC | 19:02 | |
*** openstackgerrit has joined #openstack-qa | 19:02 | |
*** jecarey has joined #openstack-qa | 19:03 | |
*** jecarey_ has quit IRC | 19:03 | |
*** salv-orlando has joined #openstack-qa | 19:04 | |
*** jasonsb has quit IRC | 19:06 | |
*** salv-orlando has quit IRC | 19:11 | |
*** maht has joined #openstack-qa | 19:13 | |
*** david-lyle_ is now known as david-lyle | 19:14 | |
*** pglass has joined #openstack-qa | 19:14 | |
*** rbrndt has quit IRC | 19:17 | |
*** pglass has quit IRC | 19:19 | |
*** itzikb_ has joined #openstack-qa | 19:20 | |
*** maht has quit IRC | 19:23 | |
*** nicholasgoracke has joined #openstack-qa | 19:24 | |
*** maht has joined #openstack-qa | 19:24 | |
*** ngoracke has joined #openstack-qa | 19:30 | |
sc68cal | mtreinish: is there an IRC room for openstack-health or is it just here | 19:30 |
*** weshay_xchat is now known as weshay | 19:31 | |
*** pglass has joined #openstack-qa | 19:31 | |
mtreinish | sc68cal: just here | 19:35 |
mtreinish | it's a qa project | 19:35 |
sc68cal | mtreinish: thanks | 19:36 |
*** flwang1 has joined #openstack-qa | 19:36 | |
*** harlowja has quit IRC | 19:38 | |
*** harlowja has joined #openstack-qa | 19:38 | |
*** sshnaidm has joined #openstack-qa | 19:40 | |
*** rbrndt has joined #openstack-qa | 19:43 | |
mtreinish | regXboi: 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-qa | 19:48 | |
*** sabeen3 has joined #openstack-qa | 19:50 | |
mtreinish | EmilienM: 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 db | 19:51 |
EmilienM | mtreinish: not yet, I was waiting for the dependencies before | 19:52 |
openstackgerrit | Sean M. Collins proposed openstack/openstack-health: [wip] jobs view - reverse sort of failuresRate https://review.openstack.org/254927 | 19:52 |
EmilienM | mtreinish: i'm doing it now, thanks for the reminder | 19:52 |
EmilienM | mtreinish: all deps have been merged | 19:52 |
sc68cal | mtreinish: sort of poking at it now, get my feet wet | 19:52 |
*** sabeen has quit IRC | 19:52 | |
EmilienM | mtreinish: 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-qa | 19:54 | |
*** dpaterson has joined #openstack-qa | 19:54 | |
EmilienM | http://logs.openstack.org/18/242718/15/check/gate-puppet-openstack-integration-scenario001-dsvm-trusty/2cc6fc7/console.html#_2015-12-08_18_46_25_472 | 19:55 |
mtreinish | sc68cal: apply that to all the lists and I'm +2 on it :) | 19:55 |
EmilienM | mtreinish: the nodepool image is not updated yet | 19:56 |
EmilienM | so it does not copy testrepository.subunit yet | 19:56 |
mtreinish | ah ok | 19:56 |
*** vgridnev has quit IRC | 19:56 | |
EmilienM | so I can rename the job now but we won't see it yet | 19:56 |
EmilienM | mtreinish: the name is now gate-puppet-openstack-integration-scenario001-dsvm-trusty | 19:56 |
*** _nadya_ has quit IRC | 19:56 | |
EmilienM | we want gate-tempest-puppet-openstack-integration-scenario001-dsvm-trusty ? | 19:56 |
*** _nadya_ has joined #openstack-qa | 19:57 | |
*** aysyd has quit IRC | 19:57 | |
mriedem | jgriffith: 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 instance | 19:57 |
mtreinish | EmilienM: 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#L17 | 19:57 |
mriedem | jgriffith: that warning comes from some vm lifecycle event handling code in nova | 19:57 |
mtreinish | so make it gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty I think | 19:57 |
EmilienM | mtreinish: perfect | 19:57 |
mriedem | jgriffith: it does seem weird though | 19:59 |
*** _nadya_ has quit IRC | 20:01 | |
*** jecarey has quit IRC | 20:01 | |
*** vgridnev has joined #openstack-qa | 20:02 | |
EmilienM | mtreinish: https://review.openstack.org/254933 | 20:02 |
*** rfolco_ has quit IRC | 20:03 | |
regXboi | mtreinish: it wasn't the fact that there was a line break, it was that it was after a line that ran over the gerrit limit | 20:03 |
*** pglass has quit IRC | 20:03 | |
*** pglbutt has joined #openstack-qa | 20:03 | |
regXboi | there were enough other examples (Ls 28, 29, 39, 42, 45, 48, 54, 60) that it got on my nerves | 20:04 |
mriedem | jgriffith: 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%22 | 20:05 |
mriedem | nvm | 20:05 |
mriedem | there are older hits | 20:05 |
*** pglass_ has joined #openstack-qa | 20:05 | |
*** artom has quit IRC | 20:05 | |
mtreinish | EmilienM: cool, +1 | 20:06 |
regXboi | mtreinish: - I explained my annoyance and pulled the -1 on 243703 ... I find it visual noise, but it's not worth a fight | 20:07 |
mtreinish | regXboi: oh, ok. I was fine with your -1 there, it was just an open question on the review | 20:08 |
mtreinish | I was just going through the open reviews and saw that | 20:08 |
mriedem | jgriffith: i think you might be onto something here, | 20:08 |
regXboi | mtreinish: ack - I wasn't clear initially and that's my bad | 20:08 |
*** pglbutt has quit IRC | 20:08 | |
mriedem | jgriffith: 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 ip | 20:08 |
mriedem | http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/console.html.gz#_2015-11-30_18_26_19_251 | 20:08 |
mriedem | 2015-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.324s | 20:09 |
mriedem | shortly 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_948 | 20:09 |
mriedem | sdague: ^ | 20:10 |
mriedem | related to the test_volume_boot_pattern ssh bugs | 20:10 |
mriedem | i wonder if libvirt is puking on the interface attach and stopping the vm | 20:10 |
*** lucasagomes is now known as lucas-dinner | 20:13 | |
mriedem | sdague: 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_252 | 20:14 |
mriedem | that's when attaching the floating IP to the instance | 20:14 |
*** jecarey has joined #openstack-qa | 20:14 | |
mtreinish | mriedem: well, it's not an error. Maybe the rule already exists? | 20:17 |
mtreinish | nah, that's unlikely | 20:17 |
*** itsuugo_ has joined #openstack-qa | 20:18 | |
lifeless | sdague: can you file a bug on how to reproduce it? on bugs.python.org (cc me) | 20:20 |
sdague | yeh, I'll see if I can create a small reproduce on it tomorrow | 20:20 |
mriedem | bingo http://logs.openstack.org/07/251407/2/check/gate-tempest-dsvm-full/144f7fc/logs/libvirt/libvirtd.txt.gz#_2015-11-30_18_20_18_168 | 20:21 |
mriedem | jgriffith: ^ 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_948 | 20:21 |
*** itzikb_ has quit IRC | 20:31 | |
*** asselin__ has quit IRC | 20:42 | |
*** pglass_ has quit IRC | 20:42 | |
*** pglass has joined #openstack-qa | 20:42 | |
*** pglbutt has joined #openstack-qa | 20:43 | |
*** pglass has quit IRC | 20:47 | |
openstackgerrit | Merged openstack/openstack-health: Update charts to format data correctly https://review.openstack.org/243703 | 20:49 |
*** sshnaidm has quit IRC | 20:52 | |
andreaf | mtreinish: nice: http://status.openstack.org/openstack-health/#/test/tempest.api.compute.servers.test_server_actions.ServerActionsTestJSON.test_reboot_server_hard | 20:54 |
*** jecarey has quit IRC | 20:54 | |
mtreinish | andreaf: heh, yeah it's pretty cool | 20:55 |
mtreinish | the only thing is I think I need to shrink the date window on the per test pages | 20:55 |
mtreinish | it kinda brings firefox to a halt for me. Chrome works fine though | 20:55 |
mtreinish | there's a lot of data being graphed | 20:55 |
andreaf | safari had no issue :D | 20:56 |
* mtreinish mutters under his breath | 20:56 | |
andreaf | only on the 2ns graph one I roll over with the mouse, it tells me NaN/NaN/NaN/NaN | 20:56 |
andreaf | actually NaN/NaN/0NaN | 20:56 |
mtreinish | andreaf: hmm, well that's not right | 20:57 |
mtreinish | andreaf: that's a good bug for you to fix :) | 20:57 |
mtreinish | my guess is I screwed something up in the stacked area graph directive I added for that graph | 20:58 |
andreaf | mtreinish :D I can try ;) | 20:58 |
mtreinish | andreaf: file a bug on lp first so we can track it | 20:58 |
andreaf | mtreinish: 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 |
mtreinish | thanks | 20:59 |
*** _nadya_ has joined #openstack-qa | 21:01 | |
*** _nadya_ has quit IRC | 21:06 | |
*** rfolco_ has joined #openstack-qa | 21:13 | |
*** itzikb_ has joined #openstack-qa | 21:16 | |
jgriffith | mriedem: nice work! | 21:17 |
mriedem | jgriffith: so i don't know what's causing that, i found an old red hat bugzilla and re-opened since it was the same issue | 21:18 |
mriedem | guests randomly crashing | 21:18 |
jgriffith | mriedem: so I looked at a handful of the ssh failures on boot tetss and that seemed to be a common thread | 21:18 |
*** rfolco_ has quit IRC | 21:19 | |
mtreinish | mriedem: well guests crashing is a good explanation as to why we can't login to them :) | 21:20 |
mriedem | mtreinish: the guest crash is actually a bit later, but yeah | 21:21 |
mriedem | mtreinish: 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 in | 21:21 |
mriedem | and then we get the lifecycle event in n-cpu to stop the vm | 21:21 |
mriedem | before tempest actually sends the delete request | 21:21 |
*** jasonsb has joined #openstack-qa | 21:23 | |
*** jasonsb has quit IRC | 21:23 | |
*** jasonsb has joined #openstack-qa | 21:24 | |
jgriffith | mriedem: mtreinish so small sample does seem to coincide with assign floating ip | 21:28 |
jgriffith | mriedem: mriedem at least within the minute | 21:29 |
*** TuanNguyen has quit IRC | 21:30 | |
*** dims has joined #openstack-qa | 21:32 | |
*** mfisher_ora has quit IRC | 21:44 | |
*** dspano has joined #openstack-qa | 21:48 | |
*** salv-orlando has joined #openstack-qa | 21:50 | |
*** salv-orlando has quit IRC | 21:55 | |
*** dims has quit IRC | 21:59 | |
*** nichola__ has joined #openstack-qa | 22:00 | |
*** nicholasgoracke has quit IRC | 22:04 | |
*** salv-orlando has joined #openstack-qa | 22:05 | |
boris-42_ | mtreinish: hi | 22:05 |
boris-42_ | mtreinish: thanks for OSprofiler spec review | 22:05 |
boris-42_ | mtreinish: could you please take a look at my reply | 22:06 |
*** jogo has quit IRC | 22:07 | |
mtreinish | boris-42_: your reply makes sense. To have osprofiler be the tracing collection layer and have the backend its pushing to be abstracted | 22:07 |
mtreinish | boris-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 rev | 22:07 |
*** itsuugo_ has quit IRC | 22:08 | |
*** david-ly_ has joined #openstack-qa | 22:10 | |
*** josdotso has joined #openstack-qa | 22:11 | |
*** jogo has joined #openstack-qa | 22:12 | |
*** david-lyle has quit IRC | 22:13 | |
*** david-ly_ is now known as david-lyle | 22:13 | |
boris-42_ | mtreinish: so what you would like to see in spec more? and where? | 22:18 |
*** maht has quit IRC | 22:19 | |
*** maht has joined #openstack-qa | 22:19 | |
*** BadCub has left #openstack-qa | 22:20 | |
*** e0ne has quit IRC | 22:26 | |
*** Swami_ has joined #openstack-qa | 22:27 | |
*** Swami has quit IRC | 22:27 | |
*** dustins is now known as dustins|gone | 22:27 | |
mtreinish | boris-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 there | 22:30 |
mtreinish | boris-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 too | 22:30 |
andreaf | mtreinish: https://bugs.launchpad.net/openstack-health/+bug/1524088 | 22:32 |
openstack | Launchpad bug 1524088 in OpenStack Health "NaN on 2nd graph on the test details page" [Undecided,New] | 22:32 |
mtreinish | andreaf: ooh, look at that a screenshot too :) | 22:33 |
*** jordanP has quit IRC | 22:35 | |
*** shardy has quit IRC | 22:41 | |
*** pglass has joined #openstack-qa | 22:42 | |
*** pglbutt has quit IRC | 22:45 | |
*** _nadya_ has joined #openstack-qa | 22:50 | |
*** salv-orlando has quit IRC | 22:53 | |
*** salv-orlando has joined #openstack-qa | 22:54 | |
*** _nadya_ has quit IRC | 22:55 | |
mriedem | sc68cal: was the linuxbridge job just made voting in the last few days? | 22:56 |
*** cp16net has quit IRC | 22:56 | |
*** samueldmq has quit IRC | 22:56 | |
*** nichola__ has quit IRC | 22:56 | |
*** devananda has quit IRC | 22:57 | |
*** jesusaurus has quit IRC | 22:57 | |
*** dkranz has quit IRC | 22:58 | |
*** itzikb_ has quit IRC | 23:04 | |
openstackgerrit | Merged openstack/tempest: Fix some Typo in docstrings https://review.openstack.org/253997 | 23:07 |
*** josdotso has quit IRC | 23:07 | |
*** mriedem is now known as mriedem_away | 23:09 | |
openstackgerrit | Merged openstack/tempest: Use interfaces_client from tempest-lib https://review.openstack.org/252747 | 23:12 |
openstackgerrit | Merged openstack/tempest: Use interfaces_client from tempest-lib https://review.openstack.org/252747 | 23:12 |
openstackgerrit | Merged openstack/tempest: [V3] Rename get_credential to show_credential https://review.openstack.org/254104 | 23:12 |
openstackgerrit | Merged openstack/tempest: [V3] Rename get_region to show_region https://review.openstack.org/254133 | 23:12 |
*** mkoderer has quit IRC | 23:15 | |
*** mkoderer has joined #openstack-qa | 23:17 | |
*** regXboi has quit IRC | 23:18 | |
*** dspano has quit IRC | 23:18 | |
*** edmondsw has quit IRC | 23:21 | |
*** yamamoto has joined #openstack-qa | 23:23 | |
*** pglass has quit IRC | 23:31 | |
*** tochi has joined #openstack-qa | 23:36 | |
*** alex_xu has quit IRC | 23:36 | |
*** yamamoto has quit IRC | 23:40 | |
*** alex_xu has joined #openstack-qa | 23:41 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Fix T110 violation for database client https://review.openstack.org/254010 | 23:42 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove unused add_sample() https://review.openstack.org/255005 | 23:48 |
*** _dpaterson has joined #openstack-qa | 23:51 | |
*** sabeen3 has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!