| *** Sukhdev has quit IRC | 00:01 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable E251,E265 rules ignore H402 https://review.openstack.org/98903 | 00:02 |
|---|---|---|
| mtreinish | sdague: ^^^ | 00:02 |
| clarkb | jogo: can we possible stop merging changes to hacking that enforce really silly things? | 00:03 |
| clarkb | jogo: I feel like maybe we need to yell at reviewers if there is pressure coming from that direction | 00:03 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable H407,H305,E123,H307,E122 rules https://review.openstack.org/98918 | 00:06 |
| *** mmaglana has quit IRC | 00:06 | |
| *** adalbas has joined #openstack-qa | 00:07 | |
| *** marun has quit IRC | 00:11 | |
| *** marun has joined #openstack-qa | 00:11 | |
| *** marun has quit IRC | 00:12 | |
| *** Sukhdev has joined #openstack-qa | 00:13 | |
| *** nati_ueno has quit IRC | 00:14 | |
| *** mwagner_lap has joined #openstack-qa | 00:14 | |
| *** kgriffs is now known as kgriffs|afk | 00:17 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: double up [ for safety https://review.openstack.org/100728 | 00:23 |
| openstackrecheck | Console logs not available after 13:23s for check-grenade-dsvm-partial-ncpu 99772,3,20d91e2 | 00:24 |
| *** dmorita has joined #openstack-qa | 00:29 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable E128 ignore E129 https://review.openstack.org/98909 | 00:31 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable H407,H305,E123,H307,E122 rules https://review.openstack.org/98918 | 00:31 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable E251,E265 rules ignore H402 https://review.openstack.org/98903 | 00:31 |
| openstackrecheck | Console logs not available after 13:25s for gate-nova-pep8 99373,4,ffdba5d | 00:38 |
| *** yamahata has quit IRC | 00:41 | |
| jogo | clarkb: do you have some examples? | 00:44 |
| clarkb | jogo: period at the end of a docstring | 00:44 |
| clarkb | period at the end of commit title | 00:44 |
| clarkb | anything like that | 00:44 |
| jogo | clarkb: AFAIK a bunch came from pep8 1.5 | 00:44 |
| jogo | clarkb: ohh yeah those | 00:46 |
| *** TravT has quit IRC | 00:46 | |
| jogo | so the punctuation one, was there before but the test got better | 00:47 |
| jogo | clarkb: and I don't like it either, I don't make the rules just enforce them | 00:47 |
| jogo | usally if you want to skip a rule on purpose add a comment in tox.ini | 00:47 |
| clarkb | jogo: you do make the rules though | 00:47 |
| jogo | saying skip it | 00:47 |
| jogo | clarkb: they came from hacking.rst | 00:47 |
| clarkb | hacking.rst says put a period at the end of a docstring? | 00:48 |
| jogo | clarkb: yup | 00:48 |
| clarkb | also I think that you don't have to enforce everything in there | 00:48 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/96829 | 00:48 |
| jogo | clarkb: so that is something you can do by saying skip rule and saying your skip was on purpose | 00:48 |
| clarkb | jogo: but then six months later someone tries to fix it all | 00:49 |
| clarkb | and when you release a new hacking I have to ignore new things | 00:49 |
| jogo | thats what the comment is for | 00:49 |
| clarkb | it just leads to not using hacking at all | 00:49 |
| clarkb | which isn't really helpful either | 00:49 |
| jogo | clarkb: yeah so I agree the current situation is non-ideal | 00:49 |
| jogo | one issue is its hard for me to figure out what rules folks universally don't like | 00:49 |
| clarkb | jogo: anything that would make you want to shoot a reviewer | 00:50 |
| clarkb | hacking is basically a reviewer in this case but we seem to have given it a pass on not being a jerk | 00:50 |
| jogo | haha its the jerk on behalf of us | 00:50 |
| clarkb | right | 00:50 |
| clarkb | which isn't a good thing | 00:50 |
| clarkb | it should enforce sensible things that I wouldn't feel stupid for -1ing on | 00:51 |
| jogo | clarkb: so I think the next step in hacking is to make it an autocorrect tool too | 00:51 |
| clarkb | jogo: no | 00:51 |
| clarkb | please no | 00:51 |
| jogo | not server side | 00:51 |
| jogo | but locally | 00:51 |
| clarkb | because again ^ | 00:51 |
| openstackrecheck | Console logs not available after 13:33s for check-tempest-dsvm-full 100728,1,f13184d | 00:51 |
| jogo | clarkb: I feel stupid -1ing on whitespace | 00:52 |
| jogo | but we all agree its important | 00:52 |
| clarkb | jogo: whitespace is slightly different. python is whitespace sensitive and any decent editor makes it a non issue | 00:52 |
| jogo | so IMHO hacking helps for bigger repos | 00:53 |
| jogo | bigger == more contributors / patches | 00:53 |
| clarkb | jogo: yup and I think those are the cases where nitpicking a '.' is dumb | 00:54 |
| clarkb | it creates way more churn | 00:54 |
| jogo | clarkb: anyway if you think there are a bunch of rules that we shouldn't enforce by default lets get a list and see what we can do | 00:54 |
| jogo | because I don't exactly disagree with you | 00:55 |
| *** rwsu has quit IRC | 00:58 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: display grenade summary instead of all of it https://review.openstack.org/99333 | 01:05 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: By default enable memcache for Keystone https://review.openstack.org/100738 | 01:06 |
| *** flwang has quit IRC | 01:10 | |
| *** matsuhashi has joined #openstack-qa | 01:17 | |
| jogo | sdague: hrm partial-ncpus keeps failing | 01:18 |
| jogo | n-cond log isn't there | 01:18 |
| openstackrecheck | Console logs not available after 13:21s for check-tempest-dsvm-postgres-full 100420,3,2db3f56 | 01:19 |
| jogo | sdague: also why is console.html 3.4 MB here | 01:19 |
| jogo | http://logs.openstack.org/72/99772/3/check/check-grenade-dsvm-partial-ncpu/20d91e2/ | 01:19 |
| jogo | clarkb: ^ that probably isn't good for ES | 01:19 |
| clarkb | no probably not | 01:21 |
| jogo | its not good for chrome either | 01:22 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: double up [ for safety https://review.openstack.org/100728 | 01:22 |
| jogo | clarkb: not sure what triggered that | 01:22 |
| jogo | maybe mtreinish or sdague know | 01:22 |
| *** salv-orlando has quit IRC | 01:25 | |
| *** yamahata has joined #openstack-qa | 01:26 | |
| jogo | sdague: I think "add a more comprehensive check-sanity | 01:29 |
| jogo | " should be reverted | 01:30 |
| sdague | jogo: why? | 01:30 |
| jogo | sdague: message:"Couldn't find log for " AND tags:"console.html" | 01:30 |
| jogo | http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiQ291bGRuJ3QgZmluZCBsb2cgZm9yIFwiIEFORCB0YWdzOlwiY29uc29sZS5odG1sXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjE3MjgwMCIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJ1c2VyX2ludGVydmFsIjowfSwic3RhbXAiOjE0MDMwNTQ2NzMwOTcsIm1vZGUiOiIiLCJhbmFseXplX2ZpZWxkIjoiIn0= | 01:31 |
| *** nosnos has joined #openstack-qa | 01:31 | |
| openstackrecheck | Console logs not available after 13:21s for gate-grenade-dsvm-partial-ncpu 69731,8,49da448 | 01:32 |
| sdague | jogo: right, that's intentional | 01:32 |
| sdague | now instead of some mysterious failure late in tempest when new side services don't stop | 01:32 |
| sdague | we actually stop | 01:32 |
| sdague | because new side services didn't start | 01:32 |
| jogo | so http://logs.openstack.org/72/99772/3/check/check-grenade-dsvm-partial-ncpu/20d91e2/console.html | 01:33 |
| sdague | what that actually does it take a whole bunch of hard to fingerprint fails and move them to one very specific fail point | 01:33 |
| jogo | it has a missing n-cpu\ | 01:33 |
| sdague | jogo: yes, what about it | 01:33 |
| jogo | err n-cond | 01:33 |
| sdague | right conductor didn't start | 01:33 |
| sdague | and if you look at the collected artifacts, you see that conductor didn't start | 01:34 |
| openstackgerrit | A change was merged to openstack/tempest: Enable E113,E111,H302,F812,E713 rules https://review.openstack.org/98899 | 01:34 |
| jogo | but nova-conductor is in the running process list | 01:35 |
| sdague | old conductor | 01:35 |
| jogo | how do you know? | 01:35 |
| sdague | we don't, which is part of the problem of entry points | 01:35 |
| jogo | well based on the start time | 01:36 |
| jogo | its the same start time as nova-api | 01:36 |
| sdague | anyway, I have to go. | 01:37 |
| jogo | and new n-api started | 01:37 |
| jogo | I think there is a bug with log files vanishing | 01:37 |
| *** Sukhdev has quit IRC | 01:37 | |
| *** weshay has joined #openstack-qa | 01:38 | |
| *** yjiang|2 has joined #openstack-qa | 01:38 | |
| jogo | https://bugs.launchpad.net/grenade/+bug/1331274 | 01:43 |
| *** yjiang|2 has quit IRC | 01:43 | |
| *** giulivo has quit IRC | 01:44 | |
| openstackrecheck | Console logs not available after 13:22s for check-grenade-dsvm 97980,10,6a4e1bf | 01:45 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for grenade bug 1331274 https://review.openstack.org/100743 | 01:47 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Enable some volumes v2 tests by sharing codes https://review.openstack.org/100193 | 01:48 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Sharing codes for cinder v1 and v2 tests https://review.openstack.org/96674 | 01:48 |
| *** dims has quit IRC | 01:58 | |
| openstackrecheck | Console logs not available after 13:21s for check-tempest-dsvm-full 100728,2,d82af54 | 01:59 |
| *** weshay has quit IRC | 02:01 | |
| *** adalbas has quit IRC | 02:02 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-infra/devstack-gate: Use the DEVSTACK_GATE_USE_APACHE flag to disable https://review.openstack.org/100744 | 02:04 |
| *** rbak has joined #openstack-qa | 02:05 | |
| *** 77CAADSW3 has quit IRC | 02:06 | |
| *** GMann has joined #openstack-qa | 02:07 | |
| openstackrecheck | Console logs not available after 13:21s for gate-glance-python26 88719,25,ed228ff | 02:12 |
| *** rbak has quit IRC | 02:14 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: By default enable memcache for Keystone https://review.openstack.org/100738 | 02:16 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Enable Keystone under Apache by default https://review.openstack.org/100747 | 02:16 |
| openstackgerrit | A change was merged to openstack/tempest: Adds test for Floating Ips bulk Nova V2 API https://review.openstack.org/86824 | 02:18 |
| *** mestery has joined #openstack-qa | 02:19 | |
| *** dims has joined #openstack-qa | 02:24 | |
| *** coolsvap|afk is now known as coolsvap | 02:25 | |
| openstackrecheck | Console logs not available after 13:32s for check-tempest-dsvm-neutron 99092,7,e7c7b45 | 02:26 |
| *** dims has quit IRC | 02:28 | |
| *** matsuhashi has quit IRC | 02:29 | |
| *** matsuhashi has joined #openstack-qa | 02:32 | |
| *** Sukhdev has joined #openstack-qa | 02:38 | |
| openstackrecheck | Console logs not available after 13:21s for check-tempest-dsvm-neutron 96275,18,6cc60ab | 02:39 |
| *** ekhugen has quit IRC | 02:43 | |
| *** mmaglana has joined #openstack-qa | 02:48 | |
| *** harlowja is now known as harlowja_away | 02:48 | |
| openstackrecheck | Console logs not available after 13:22s for check-grenade-dsvm 100054,2,09a92c6 | 02:53 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable E128 ignore E129 https://review.openstack.org/98909 | 02:59 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable H407,H305,E123,H307,E122 rules https://review.openstack.org/98918 | 02:59 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable E251,E265 rules ignore H402 https://review.openstack.org/98903 | 03:00 |
| *** matsuhashi has quit IRC | 03:00 | |
| openstackrecheck | Console logs not available after 13:23s for gate-nova-python27 56812,19,8e5687d | 03:15 |
| *** nosnos has quit IRC | 03:27 | |
| openstackrecheck | Console logs not available after 13:29s for check-grenade-dsvm 100747,1,f8e021f | 03:29 |
| *** flwang has joined #openstack-qa | 03:35 | |
| *** SridharG has joined #openstack-qa | 03:37 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/grenade: Bump CIRROS_IMAGE to 0.3.2 (trunk version). https://review.openstack.org/100757 | 03:40 |
| openstackrecheck | Console logs not available after 13:24s for check-grenade-dsvm-partial-ncpu 100738,2,7e4ae44 | 03:42 |
| openstackgerrit | Fei Long Wang proposed a change to openstack/tempest: Add scenario test for image service https://review.openstack.org/100140 | 03:43 |
| *** matsuhashi has joined #openstack-qa | 03:53 | |
| openstackrecheck | Console logs not available after 13:26s for gate-tempest-dsvm-full 69732,8,97ab040 | 03:55 |
| *** mestery has quit IRC | 04:00 | |
| *** yfried_ has quit IRC | 04:02 | |
| *** mestery has joined #openstack-qa | 04:02 | |
| *** mestery has quit IRC | 04:04 | |
| *** mike77 has joined #openstack-qa | 04:05 | |
| *** mike77 has joined #openstack-qa | 04:05 | |
| *** nosnos has joined #openstack-qa | 04:07 | |
| openstackrecheck | Console logs not available after 13:24s for check-tempest-dsvm-full 99616,2,9debde3 | 04:09 |
| openstackgerrit | ChangBo Guo(gcb) proposed a change to openstack/tempest: Fix calls to mock.assert_not_called() https://review.openstack.org/93534 | 04:10 |
| *** matsuhashi has quit IRC | 04:12 | |
| *** amotoki has joined #openstack-qa | 04:12 | |
| *** matsuhashi has joined #openstack-qa | 04:12 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/85666 failed because of: gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1286818 | 04:13 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/98991 failed gate-grenade-dsvm in the gate queue with an unrecognized error | 04:16 |
| *** matsuhashi has quit IRC | 04:17 | |
| *** jgrimm has joined #openstack-qa | 04:19 | |
| *** matsuhashi has joined #openstack-qa | 04:21 | |
| *** Longgeek has joined #openstack-qa | 04:26 | |
| *** dims has joined #openstack-qa | 04:27 | |
| *** dims has quit IRC | 04:32 | |
| *** mmaglana has quit IRC | 04:32 | |
| *** matsuhashi has quit IRC | 04:36 | |
| *** matsuhashi has joined #openstack-qa | 04:36 | |
| *** Sukhdev has quit IRC | 04:38 | |
| *** matsuhashi has quit IRC | 04:39 | |
| *** matsuhashi has joined #openstack-qa | 04:39 | |
| *** yfried_ has joined #openstack-qa | 04:39 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/99772 failed because of: gate-grenade-dsvm-partial-ncpu: https://bugs.launchpad.net/bugs/1298472 | 04:41 |
| *** psharma has joined #openstack-qa | 04:45 | |
| *** dims has joined #openstack-qa | 04:46 | |
| *** mmaglana has joined #openstack-qa | 04:47 | |
| *** dims has quit IRC | 04:51 | |
| *** mike77_ has joined #openstack-qa | 04:54 | |
| *** Poornima|mtg has joined #openstack-qa | 04:56 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Mising CLI test for Heat https://review.openstack.org/100142 | 04:56 |
| *** mike77 has quit IRC | 04:58 | |
| *** mmaglana has quit IRC | 05:01 | |
| *** ildikov has quit IRC | 05:09 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/grenade: Suppot keystone eventlet to mod_wsgi (icehouse -> juno) https://review.openstack.org/100764 | 05:10 |
| *** matsuhashi has quit IRC | 05:11 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/grenade: Suppot keystone eventlet to mod_wsgi (icehouse -> juno) https://review.openstack.org/100764 | 05:11 |
| *** matsuhashi has joined #openstack-qa | 05:11 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Enable Keystone under Apache by default https://review.openstack.org/100747 | 05:15 |
| *** matsuhashi has quit IRC | 05:16 | |
| flwang | zhikunliu: ping | 05:16 |
| zhikunliu | flwang: pong | 05:16 |
| flwang | zhikunliu: could you please help take a look at this patch https://review.openstack.org/#/c/100140/ ? | 05:16 |
| *** matsuhashi has joined #openstack-qa | 05:17 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/grenade: Support keystone eventlet to mod_wsgi (icehouse -> juno) https://review.openstack.org/100764 | 05:17 |
| flwang | zhikunliu: i run it perfectly on my local env and seems jenkins can pass some py27, dsvm-xx as well | 05:17 |
| flwang | zhikunliu: but it can't pass all gates | 05:17 |
| flwang | zhikunliu: any idea? | 05:17 |
| zhikunliu | flwang:ok, I'll have a look | 05:18 |
| flwang | zhikunliu: thanks, really appreciate it | 05:18 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: By default enable memcache for Keystone https://review.openstack.org/100738 | 05:19 |
| *** matsuhashi has quit IRC | 05:24 | |
| *** matsuhashi has joined #openstack-qa | 05:24 | |
| raies | jogo: hi | 05:25 |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/100297 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1330776 | 05:26 |
| raies | mtreinish: hi I need your suggestion on this https://review.openstack.org/#/c/89472/ | 05:28 |
| jogo | raies: pong | 05:33 |
| raies | jogo: have you seen https://review.openstack.org/#/c/98974/ | 05:39 |
| jogo | raies: what about it? | 05:40 |
| jogo | tempest should be runnable against any enviornment not just a devstack one | 05:40 |
| raies | that mean 404 check should not be there ?? | 05:41 |
| raies | in any deployment if there is actually no fwaas enabled then there will be such error 404. which should not be treated as fail | 05:42 |
| raies | jogo: ^^ | 05:42 |
| jogo | in general tempest doesn't do simple auto discovery as that leads to confusion | 05:44 |
| jogo | so you shouldn't try to detect of a service is there or not, that comes from a config. but the config can be autogenerated | 05:45 |
| raies | hmm :) that mean this 404 not found check is invalid. | 05:46 |
| raies | ok thanks for your suggestion | 05:47 |
| *** dims has joined #openstack-qa | 05:47 | |
| *** dims has quit IRC | 05:51 | |
| *** afazekas_ has joined #openstack-qa | 05:54 | |
| *** yjiang|2 has joined #openstack-qa | 05:55 | |
| *** SridharG has quit IRC | 06:02 | |
| openstackgerrit | Zhenguo Niu proposed a change to openstack-dev/devstack: Add support for django_openstack_auth https://review.openstack.org/79881 | 06:04 |
| *** wendar has quit IRC | 06:04 | |
| *** wendar_ has joined #openstack-qa | 06:04 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/99138 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/921858 | 06:09 |
| *** k4n0 has joined #openstack-qa | 06:11 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Missing Neutron CLI tests https://review.openstack.org/98974 | 06:17 |
| *** k4n0 has quit IRC | 06:19 | |
| *** psharma has quit IRC | 06:19 | |
| *** GMann has quit IRC | 06:19 | |
| *** pcrews has quit IRC | 06:19 | |
| *** openstackgerrit has quit IRC | 06:19 | |
| *** mrodden has quit IRC | 06:19 | |
| *** k4n0 has joined #openstack-qa | 06:23 | |
| *** psharma has joined #openstack-qa | 06:23 | |
| *** GMann has joined #openstack-qa | 06:23 | |
| *** pcrews has joined #openstack-qa | 06:23 | |
| *** openstackgerrit has joined #openstack-qa | 06:23 | |
| *** mrodden has joined #openstack-qa | 06:23 | |
| *** k4n0 has quit IRC | 06:24 | |
| *** k4n0 has joined #openstack-qa | 06:24 | |
| openstackgerrit | garyk proposed a change to openstack-dev/devstack: Neutron: start to use nova's neutron configuration section https://review.openstack.org/100779 | 06:26 |
| *** ildikov has joined #openstack-qa | 06:29 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Missing Neutron CLI tests https://review.openstack.org/98974 | 06:34 |
| *** tkammer has joined #openstack-qa | 06:34 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Added missing ceilometer CLI tests https://review.openstack.org/100190 | 06:37 |
| openstackgerrit | ajay proposed a change to openstack/tempest: Add three new CLI tests of sahara https://review.openstack.org/100784 | 06:40 |
| *** Manishanker has quit IRC | 06:43 | |
| *** GMann_ has joined #openstack-qa | 06:44 | |
| *** GMann has quit IRC | 06:45 | |
| *** dims_ has joined #openstack-qa | 06:47 | |
| *** dims_ has quit IRC | 06:53 | |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/100581 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 06:56 |
| *** samuelbercovici has joined #openstack-qa | 06:56 | |
| *** Poornima|mtg is now known as Poornima | 06:56 | |
| *** [1]evgenyf has joined #openstack-qa | 06:58 | |
| *** samuelbercovici1 has joined #openstack-qa | 07:05 | |
| *** matsuhashi has quit IRC | 07:08 | |
| *** matsuhas_ has joined #openstack-qa | 07:08 | |
| *** samuelbercovici has quit IRC | 07:09 | |
| *** samuelbercovici1 is now known as samuelbercovici | 07:09 | |
| *** chandan_kumar has quit IRC | 07:17 | |
| *** jordanP has joined #openstack-qa | 07:20 | |
| *** salv-orlando has joined #openstack-qa | 07:21 | |
| *** mike77_ has quit IRC | 07:24 | |
| *** samuelbercovici1 has joined #openstack-qa | 07:31 | |
| *** jhenner has quit IRC | 07:33 | |
| *** samuelbercovici has quit IRC | 07:34 | |
| *** samuelbercovici1 is now known as samuelbercovici | 07:35 | |
| *** luqas has joined #openstack-qa | 07:35 | |
| *** jhenner has joined #openstack-qa | 07:37 | |
| *** jlibosva has joined #openstack-qa | 07:38 | |
| *** yjiang|2 has quit IRC | 07:42 | |
| *** coolsvap is now known as coolsvap|afk | 07:55 | |
| *** jhenner has quit IRC | 07:55 | |
| *** jpich has joined #openstack-qa | 07:57 | |
| openstackgerrit | A change was merged to openstack/tempest: Verify list_instance_action attributes of Nova API https://review.openstack.org/85666 | 07:59 |
| *** jhenner has joined #openstack-qa | 08:01 | |
| *** leseb has joined #openstack-qa | 08:02 | |
| *** safchain has joined #openstack-qa | 08:02 | |
| *** yamahata has quit IRC | 08:05 | |
| *** yamahata has joined #openstack-qa | 08:05 | |
| *** GMann has joined #openstack-qa | 08:06 | |
| *** leseb has quit IRC | 08:06 | |
| *** leseb has joined #openstack-qa | 08:07 | |
| *** GMann_ has quit IRC | 08:07 | |
| *** jordanP has quit IRC | 08:09 | |
| *** jhenner has quit IRC | 08:11 | |
| *** jordanP has joined #openstack-qa | 08:11 | |
| *** andreaf_ has quit IRC | 08:11 | |
| *** amotoki has quit IRC | 08:14 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack-dev/grenade: Test: enable verbosity in tempest https://review.openstack.org/100302 | 08:15 |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Check create-Floating-IP-Bulk Nova API attributes https://review.openstack.org/100816 | 08:15 |
| *** samuelbercovici1 has joined #openstack-qa | 08:18 | |
| *** samuelbercovici has quit IRC | 08:21 | |
| *** tkammer has quit IRC | 08:21 | |
| *** samuelbercovici1 is now known as samuelbercovici | 08:22 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify V2 list server_group API attributes https://review.openstack.org/95065 | 08:26 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/98937 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1327218 https://bugs.launchpad.net/bugs/1302774 | 08:30 |
| *** amotoki has joined #openstack-qa | 08:32 | |
| *** luqas has quit IRC | 08:33 | |
| *** samuelbercovici1 has joined #openstack-qa | 08:35 | |
| *** jhenner has joined #openstack-qa | 08:35 | |
| *** tkammer has joined #openstack-qa | 08:37 | |
| *** samuelbercovici has quit IRC | 08:39 | |
| *** samuelbercovici1 is now known as samuelbercovici | 08:39 | |
| *** jhenner has quit IRC | 08:40 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Mising CLI test for Heat https://review.openstack.org/100142 | 08:41 |
| *** eezhova has joined #openstack-qa | 08:42 | |
| *** gszasz has joined #openstack-qa | 08:45 | |
| *** Longgeek has quit IRC | 08:46 | |
| *** Longgeek has joined #openstack-qa | 08:46 | |
| *** Longgeek has quit IRC | 08:48 | |
| *** Longgeek has joined #openstack-qa | 08:49 | |
| *** jhenner has joined #openstack-qa | 08:49 | |
| *** giulivo has joined #openstack-qa | 08:50 | |
| *** dims_ has joined #openstack-qa | 08:51 | |
| *** Longgeek has quit IRC | 08:53 | |
| *** Longgeek_ has joined #openstack-qa | 08:53 | |
| *** mkollaro has joined #openstack-qa | 08:53 | |
| *** leseb has quit IRC | 08:54 | |
| *** jhenner has quit IRC | 08:56 | |
| *** dims_ has quit IRC | 08:56 | |
| *** andreaf has joined #openstack-qa | 08:57 | |
| *** jlanoux has joined #openstack-qa | 08:57 | |
| *** ghyoc has joined #openstack-qa | 08:58 | |
| *** andreaf_ has joined #openstack-qa | 08:59 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Added missing ceilometer CLI tests https://review.openstack.org/100190 | 09:01 |
| *** matsuhas_ has quit IRC | 09:01 | |
| *** andreaf has quit IRC | 09:01 | |
| *** luqas has joined #openstack-qa | 09:03 | |
| *** matsuhashi has joined #openstack-qa | 09:09 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Missing Neutron CLI tests https://review.openstack.org/98974 | 09:10 |
| *** samuelbercovici1 has joined #openstack-qa | 09:11 | |
| *** samuelbercovici has quit IRC | 09:14 | |
| *** samuelbercovici1 is now known as samuelbercovici | 09:14 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/91976 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1330776 | 09:17 |
| *** Anju has joined #openstack-qa | 09:19 | |
| *** samuelbercovici1 has joined #openstack-qa | 09:19 | |
| *** samuelbercovici has quit IRC | 09:22 | |
| *** samuelbercovici1 is now known as samuelbercovici | 09:22 | |
| *** jhenner has joined #openstack-qa | 09:24 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/tempest: Remove skipping test for get_vifs_by_instance https://review.openstack.org/100834 | 09:27 |
| *** ilyashakhat has joined #openstack-qa | 09:27 | |
| *** jhenner has quit IRC | 09:29 | |
| *** mkollaro has quit IRC | 09:33 | |
| *** matsuhashi has quit IRC | 09:38 | |
| *** samuelbercovici1 has joined #openstack-qa | 09:39 | |
| *** mkollaro has joined #openstack-qa | 09:41 | |
| *** mkoderer has quit IRC | 09:42 | |
| *** samuelbercovici has quit IRC | 09:42 | |
| *** samuelbercovici1 is now known as samuelbercovici | 09:42 | |
| *** SridharG has joined #openstack-qa | 09:43 | |
| *** qba73 has joined #openstack-qa | 09:50 | |
| *** dims_ has joined #openstack-qa | 09:51 | |
| *** tosky has joined #openstack-qa | 09:55 | |
| *** jhenner has joined #openstack-qa | 09:55 | |
| *** rodrigods has joined #openstack-qa | 09:55 | |
| *** rodrigods has joined #openstack-qa | 09:55 | |
| *** dims_ has quit IRC | 09:56 | |
| *** leseb has joined #openstack-qa | 09:58 | |
| *** rodrigods has quit IRC | 09:59 | |
| *** matsuhashi has joined #openstack-qa | 10:01 | |
| *** jordanP has quit IRC | 10:04 | |
| *** Anju has quit IRC | 10:05 | |
| *** jhenner has quit IRC | 10:05 | |
| *** matsuhashi has quit IRC | 10:06 | |
| *** giulivo is now known as gfidente | 10:07 | |
| *** matsuhashi has joined #openstack-qa | 10:08 | |
| *** leseb has quit IRC | 10:13 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: Add fingerprint for grenade bug 1331274 https://review.openstack.org/100743 | 10:14 |
| *** Manishanker has joined #openstack-qa | 10:14 | |
| *** Manishanker has quit IRC | 10:14 | |
| *** matsuhashi has quit IRC | 10:17 | |
| *** matsuhashi has joined #openstack-qa | 10:18 | |
| *** dmorita has quit IRC | 10:25 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add fingerprint for grenade bug 1331274 https://review.openstack.org/100743 | 10:29 |
| *** 17SAAEGGD has quit IRC | 10:29 | |
| *** yamahata_ has joined #openstack-qa | 10:31 | |
| *** SridharG has quit IRC | 10:37 | |
| *** openstackrecheck has quit IRC | 10:37 | |
| *** openstackrecheck has joined #openstack-qa | 10:37 | |
| *** dims_ has joined #openstack-qa | 10:43 | |
| *** samuelbercovici1 has joined #openstack-qa | 10:49 | |
| *** leseb has joined #openstack-qa | 10:51 | |
| *** jpich has quit IRC | 10:51 | |
| *** samuelbercovici has quit IRC | 10:52 | |
| *** samuelbercovici1 is now known as samuelbercovici | 10:52 | |
| *** samuelbercovici1 has joined #openstack-qa | 10:54 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: Now that grenade logs are separated, use console https://review.openstack.org/100851 | 10:56 |
| *** leseb has quit IRC | 10:56 | |
| *** samuelbercovici has quit IRC | 10:56 | |
| *** samuelbercovici1 is now known as samuelbercovici | 10:57 | |
| *** Longgeek_ has quit IRC | 10:58 | |
| *** raies_ has joined #openstack-qa | 10:58 | |
| *** Longgeek has joined #openstack-qa | 10:59 | |
| *** yamahata has quit IRC | 10:59 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Now that grenade logs are separated, use console https://review.openstack.org/100851 | 10:59 |
| *** raies has quit IRC | 11:00 | |
| *** ildikov has quit IRC | 11:01 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: move console.html -> console https://review.openstack.org/100854 | 11:01 |
| *** GMann has quit IRC | 11:01 | |
| openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Make the #1235437 query time limit change resitent https://review.openstack.org/94079 | 11:02 |
| *** GMann has joined #openstack-qa | 11:02 | |
| *** jhenner has joined #openstack-qa | 11:02 | |
| *** ildikov_ has joined #openstack-qa | 11:03 | |
| *** mpavlase has joined #openstack-qa | 11:04 | |
| *** jpich has joined #openstack-qa | 11:04 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: move console.html -> console https://review.openstack.org/100854 | 11:05 |
| *** openstackrecheck has quit IRC | 11:07 | |
| *** openstackrecheck has joined #openstack-qa | 11:07 | |
| *** jhenner has quit IRC | 11:09 | |
| *** samuelbercovici1 has joined #openstack-qa | 11:12 | |
| *** Longgeek has quit IRC | 11:13 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Wait for OVS to reconfigure when creating PUBLIC_BRIDGE https://review.openstack.org/99414 | 11:14 |
| *** Longgeek has joined #openstack-qa | 11:14 | |
| *** samuelbercovici has quit IRC | 11:15 | |
| *** samuelbercovici1 is now known as samuelbercovici | 11:15 | |
| *** vponomaryov has quit IRC | 11:16 | |
| *** dims_ has quit IRC | 11:16 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix unsubstituted filename creation https://review.openstack.org/85120 | 11:16 |
| *** Longgeek has quit IRC | 11:19 | |
| *** Longgeek has joined #openstack-qa | 11:19 | |
| *** tkammer has quit IRC | 11:22 | |
| *** samuelbercovici1 has joined #openstack-qa | 11:24 | |
| *** vponomaryov has joined #openstack-qa | 11:26 | |
| *** samuelbercovici has quit IRC | 11:27 | |
| *** samuelbercovici1 is now known as samuelbercovici | 11:27 | |
| *** samuelbercovici1 has joined #openstack-qa | 11:29 | |
| *** jpich has quit IRC | 11:29 | |
| *** jhenner has joined #openstack-qa | 11:30 | |
| *** samuelbercovici has quit IRC | 11:32 | |
| *** samuelbercovici1 is now known as samuelbercovici | 11:32 | |
| *** GMann has quit IRC | 11:35 | |
| *** jhenner has quit IRC | 11:37 | |
| openstackgerrit | A change was merged to openstack/tempest: Enable E251,E265 rules ignore H402 https://review.openstack.org/98903 | 11:40 |
| *** jpich has joined #openstack-qa | 11:42 | |
| sdague | devananda / adam_g / Shrews: opinions on - https://review.openstack.org/#/c/98317 | 11:45 |
| *** samuelbercovici1 has joined #openstack-qa | 11:47 | |
| *** matsuhashi has quit IRC | 11:48 | |
| *** jpich has quit IRC | 11:49 | |
| *** samuelbercovici has quit IRC | 11:50 | |
| *** samuelbercovici1 is now known as samuelbercovici | 11:50 | |
| *** matsuhashi has joined #openstack-qa | 11:51 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Make the #1235437 query time limit change resitent https://review.openstack.org/94079 | 11:51 |
| *** openstackrecheck has quit IRC | 11:52 | |
| *** leseb has joined #openstack-qa | 11:52 | |
| *** openstackrecheck has joined #openstack-qa | 11:52 | |
| *** luqas has quit IRC | 11:52 | |
| sdague | jogo: https://review.openstack.org/#/c/100680/ - I think something is odd with the requirements update process, as you just got pep8 proposes in | 11:53 |
| *** matsuhashi has quit IRC | 11:56 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Convert scenario test tearDown to addCleanup https://review.openstack.org/62101 | 11:56 |
| *** matsuhashi has joined #openstack-qa | 11:56 | |
| *** jhenner has joined #openstack-qa | 11:57 | |
| *** leseb has quit IRC | 11:57 | |
| *** Longgeek has quit IRC | 11:57 | |
| *** weshay has joined #openstack-qa | 11:58 | |
| *** Longgeek has joined #openstack-qa | 11:58 | |
| *** tkammer has joined #openstack-qa | 11:58 | |
| *** matsuhashi has quit IRC | 12:01 | |
| *** jpich has joined #openstack-qa | 12:02 | |
| *** gszasz has quit IRC | 12:03 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Convert scenario test tearDown to addCleanup https://review.openstack.org/62101 | 12:04 |
| *** jpich_ has joined #openstack-qa | 12:05 | |
| *** jpich has quit IRC | 12:06 | |
| *** vponomaryov has quit IRC | 12:08 | |
| *** mwagner_lap has quit IRC | 12:10 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: double up [ for safety https://review.openstack.org/100728 | 12:10 |
| *** yamahata has joined #openstack-qa | 12:11 | |
| BobBall | What's the purpose of devstack's exercise.sh? | 12:16 |
| *** jpich_ is now known as jpich | 12:16 | |
| BobBall | If it's to verify a base level of functions are working then I think it should be gating... | 12:16 |
| BobBall | because I think it's currently broken in aggregates | 12:16 |
| *** adalbas has joined #openstack-qa | 12:17 | |
| *** samuelbercovici1 has joined #openstack-qa | 12:19 | |
| *** samuelbercovici has quit IRC | 12:21 | |
| *** samuelbercovici1 is now known as samuelbercovici | 12:21 | |
| sdague | honestly, my preference is to delete them | 12:22 |
| sdague | I'm not sure how dtroyer is currently feeling on that front | 12:22 |
| sdague | they are really fragile | 12:22 |
| sdague | and hard to debug | 12:23 |
| BobBall | definitely hard to debug... but I'd argue against fragile - they work very consistently in our CI | 12:23 |
| *** vponomaryov has joined #openstack-qa | 12:23 | |
| BobBall | we use them as a sanity check before running tempest which is useful | 12:23 |
| sdague | they don't really do much | 12:24 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Convert scenario test tearDown to addCleanup https://review.openstack.org/62101 | 12:24 |
| sdague | yeh, we used to, but found it just added a bunch of time, and when it failed, it was really not easy to get to a failure | 12:25 |
| BobBall | Agreed - we could easily have our own sanity test that just boots a VM then kills it | 12:25 |
| *** johnthetubaguy has joined #openstack-qa | 12:26 | |
| sdague | what about just running the basic_ops test in tempest instead? | 12:26 |
| BobBall | sdague was just saying he'd lean towards deleting exercise.sh but dtroyer may have other views | 12:26 |
| *** ylobankov has joined #openstack-qa | 12:26 | |
| *** ylobankov has quit IRC | 12:26 | |
| BobBall | That'd work | 12:26 |
| *** ylobankov has joined #openstack-qa | 12:26 | |
| sdague | test_server_basic_ops.py probably does what you want | 12:27 |
| sdague | def test_server_basicops(self): | 12:27 |
| sdague | self.add_keypair() | 12:27 |
| sdague | self.create_security_group() | 12:27 |
| sdague | self.boot_instance() | 12:27 |
| sdague | self.verify_ssh() | 12:27 |
| sdague | self.terminate_instance() | 12:27 |
| sdague | though only if run_ssh is enabled for that run | 12:28 |
| BobBall | Indeed - I'll happily fix our CI to use that - but the question of whether to keep them (and fix them when they break) or delete them stands | 12:28 |
| BobBall | run_ssh? Is that not normally enabled? | 12:28 |
| *** jordanP has joined #openstack-qa | 12:31 | |
| sdague | it isn't | 12:31 |
| sdague | because it fails a loot | 12:31 |
| BobBall | ah ok | 12:31 |
| sdague | basically, we remain in this place where you can't consistently boot a compute that has a working network | 12:32 |
| sdague | realistically we need to make every compute get checked for a working network | 12:32 |
| sdague | and just get to the bottom of it | 12:32 |
| sdague | today we just only check a few of them | 12:33 |
| sdague | and the statistics mean we mostly pass | 12:33 |
| *** mpavlase has quit IRC | 12:33 | |
| *** mpavlase has joined #openstack-qa | 12:33 | |
| sdague | but my view remains that we have a validation toolchain, it's setup on every devstack pull by default, we should use that :) | 12:34 |
| *** jordanP has quit IRC | 12:36 | |
| *** andreaf_ has quit IRC | 12:37 | |
| *** andreaf_ has joined #openstack-qa | 12:37 | |
| *** leseb has joined #openstack-qa | 12:38 | |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/100723 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1330776 | 12:39 |
| *** andreaf_ is now known as andreaf | 12:40 | |
| *** zhikunliu has quit IRC | 12:42 | |
| *** samuelbercovici1 has joined #openstack-qa | 12:42 | |
| openstackgerrit | Zu Qiang proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy https://review.openstack.org/94988 | 12:44 |
| *** luqas has joined #openstack-qa | 12:44 | |
| *** samuelbercovici has quit IRC | 12:46 | |
| *** samuelbercovici1 is now known as samuelbercovici | 12:46 | |
| *** GMann has joined #openstack-qa | 12:47 | |
| *** GMann has quit IRC | 12:47 | |
| *** jordanP has joined #openstack-qa | 12:48 | |
| *** Longgeek has quit IRC | 12:48 | |
| *** ekhugen has joined #openstack-qa | 12:50 | |
| *** Obulpathi has joined #openstack-qa | 12:51 | |
| *** Obulpathi has quit IRC | 12:52 | |
| *** ildikov_ has quit IRC | 12:53 | |
| openstackgerrit | Andrey Pavlov proposed a change to openstack/tempest: Add 'Member' role to all created users https://review.openstack.org/96460 | 12:54 |
| *** dustins has joined #openstack-qa | 12:56 | |
| *** samuelbercovici has quit IRC | 12:57 | |
| *** Longgeek has joined #openstack-qa | 12:57 | |
| *** dkranz has joined #openstack-qa | 12:58 | |
| *** SridharG has joined #openstack-qa | 12:58 | |
| *** ildikov has joined #openstack-qa | 13:00 | |
| *** yfried_ has quit IRC | 13:01 | |
| *** samuelbercovici has joined #openstack-qa | 13:01 | |
| *** dims_ has joined #openstack-qa | 13:04 | |
| *** yamahata has quit IRC | 13:07 | |
| *** yamahata has joined #openstack-qa | 13:07 | |
| *** samuelbercovici1 has joined #openstack-qa | 13:08 | |
| *** samuelbercovici has quit IRC | 13:11 | |
| *** cbader has joined #openstack-qa | 13:11 | |
| *** samuelbercovici1 is now known as samuelbercovici | 13:11 | |
| *** Poornima|mtg has joined #openstack-qa | 13:13 | |
| *** dims_ has quit IRC | 13:13 | |
| *** psharma_ has joined #openstack-qa | 13:14 | |
| openstackgerrit | ajay proposed a change to openstack/tempest: Add three new CLI tests of sahara https://review.openstack.org/100784 | 13:15 |
| *** leseb has quit IRC | 13:15 | |
| *** leseb has joined #openstack-qa | 13:16 | |
| *** Poornima has quit IRC | 13:17 | |
| *** k4n0 has quit IRC | 13:17 | |
| *** psharma has quit IRC | 13:17 | |
| *** dims_ has joined #openstack-qa | 13:17 | |
| *** k4n0 has joined #openstack-qa | 13:18 | |
| *** psharma__ has joined #openstack-qa | 13:19 | |
| *** Poornima|afk has joined #openstack-qa | 13:19 | |
| *** mriedem has joined #openstack-qa | 13:22 | |
| *** psharma_ has quit IRC | 13:22 | |
| *** psharma__ has quit IRC | 13:22 | |
| *** k4n0 has quit IRC | 13:22 | |
| *** Poornima|mtg has quit IRC | 13:22 | |
| *** k4n0 has joined #openstack-qa | 13:23 | |
| *** samuelbercovici1 has joined #openstack-qa | 13:26 | |
| *** samuelbercovici1 has quit IRC | 13:27 | |
| *** dims_ has quit IRC | 13:28 | |
| *** samuelbercovici has quit IRC | 13:29 | |
| *** mestery has joined #openstack-qa | 13:32 | |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack-infra/devstack-gate: Optional Ceph backend for Cinder https://review.openstack.org/100905 | 13:36 |
| *** leseb has quit IRC | 13:38 | |
| openstackgerrit | A change was merged to openstack-dev/grenade: Bump CIRROS_IMAGE to 0.3.2 (trunk version). https://review.openstack.org/100757 | 13:38 |
| *** ghyoc has quit IRC | 13:38 | |
| *** ghyoc has joined #openstack-qa | 13:39 | |
| mriedem | sdague: your email has me thinking about trenches and mustard gas | 13:42 |
| mriedem | and ernest borgnine | 13:42 |
| *** thomasbiege has joined #openstack-qa | 13:43 | |
| *** malini1 has joined #openstack-qa | 13:44 | |
| *** mestery has quit IRC | 13:44 | |
| *** Manishanker has joined #openstack-qa | 13:44 | |
| sdague | mriedem: oh dear, which email is this? | 13:45 |
| mriedem | gate | 13:46 |
| sdague | heh | 13:46 |
| sdague | oh, while I'm thinking about it, can you send your workers fixes straight back to cinder and glance instead of in devstack? | 13:46 |
| mriedem | yeah...............not really in a hurry | 13:47 |
| mriedem | glance would be first | 13:47 |
| sdague | ok | 13:47 |
| *** thomasbiege has quit IRC | 13:47 | |
| mriedem | when i talked about the cinder change in the cinder room last week asking if anyone ever set that, there were crickets kind of | 13:47 |
| jgriffith | mriedem: not kind of | 13:47 |
| *** mwagner_lap has joined #openstack-qa | 13:47 | |
| jgriffith | mriedem: it was in fact text-book | 13:47 |
| sdague | swarm of crickets? | 13:48 |
| jgriffith | :) | 13:48 |
| mriedem | jgriffith: ha, would you be opposed to changing the default? | 13:48 |
| *** malini1 has quit IRC | 13:48 | |
| *** malini1 has joined #openstack-qa | 13:48 | |
| jgriffith | mriedem: I would not | 13:48 |
| mriedem | ok, good enough for me | 13:48 |
| jgriffith | mriedem: I ran some tests after you pointed the change out and it seems okie-dokie to me | 13:48 |
| mriedem | just didn't want to spend a bunch of time on something that was going to lead to bike shedding | 13:48 |
| mriedem | ok | 13:48 |
| *** salv-orlando has quit IRC | 13:49 | |
| *** salv-orlando_ has joined #openstack-qa | 13:49 | |
| sdague | honestly, I don't think this will generate bike shedding, I think it's just not something people think about | 13:49 |
| jgriffith | mriedem: I don't see why it should | 13:49 |
| afazekas_ | sdague: how much time we need for an image pre-cache ? PRECACHE_IMAGES @ devstack | 13:49 |
| jgriffith | sdague: +1 | 13:49 |
| * jgriffith never did | 13:49 | |
| sdague | afazekas_: I'm not sure what the question is | 13:49 |
| *** tkammer has quit IRC | 13:49 | |
| sdague | oh, for nodepool? | 13:49 |
| sdague | that's always tough to answer | 13:50 |
| sdague | I've been going with a week | 13:50 |
| sdague | though the fedora images have such a high failure rate, I don't know how long it takes for them to actually manage to build | 13:50 |
| sdague | I think 1 in 10 attempts to download the image from the fedora mirrors fails | 13:50 |
| afazekas_ | sdague: how this part expected to be used: https://github.com/openstack-dev/devstack/blob/master/stackrc#L365 | 13:51 |
| sdague | afazekas: so in an ideal world, that gives nodepool a chance to cache the image | 13:51 |
| sdague | and we'd be good on the next day | 13:51 |
| sdague | however, that's only true if the image has a good download rate | 13:52 |
| sdague | which the fedora images do not | 13:52 |
| sdague | the fedora mirror system is pretty flakey | 13:52 |
| afazekas_ | sdague: wouldn't be better to move the image to static.openstack.org ? | 13:52 |
| sdague | that's possible | 13:52 |
| sdague | you'd have to ask infra folks if they are good with that | 13:53 |
| *** sballe has joined #openstack-qa | 13:54 | |
| afazekas_ | let's see can we have no_timer_check version of the image (request sent) | 13:54 |
| *** amotoki has quit IRC | 13:55 | |
| *** Obulpathi has joined #openstack-qa | 13:56 | |
| openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1331476 https://review.openstack.org/100912 | 13:56 |
| *** Obulpathi has quit IRC | 13:56 | |
| *** Obulpathi has joined #openstack-qa | 13:57 | |
| *** Longgeek_ has joined #openstack-qa | 13:57 | |
| *** yfried_ has joined #openstack-qa | 13:59 | |
| *** Longgeek has quit IRC | 13:59 | |
| SridharG | dkranz: Hello. Can you please have a look at https://review.openstack.org/#/c/100134/. You approved this earlier, but I had to do some minor alignment issues. Thank you. | 14:00 |
| *** mrodden has quit IRC | 14:00 | |
| *** sdague has quit IRC | 14:02 | |
| *** nosnos has quit IRC | 14:03 | |
| openstackgerrit | A change was merged to openstack-dev/grenade: Fix errors in stop-services from missing includes https://review.openstack.org/95072 | 14:04 |
| *** rbak has joined #openstack-qa | 14:08 | |
| *** sdague has joined #openstack-qa | 14:08 | |
| *** atiwari has joined #openstack-qa | 14:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/96829 | 14:11 |
| mriedem | sdague: methinks hacking/pep8 check for using tags:"console.html" in query files https://review.openstack.org/#/c/100854/ | 14:11 |
| mriedem | i'm not sure how easy it is to hook that in to e-r, haven't done a custom hacking check in nova before | 14:12 |
| sdague | mriedem: could be | 14:12 |
| Shrews | sdague: on 98317... I don't understand the benefit of doing that. Will need devananda to chime in on it. | 14:12 |
| mriedem | i'll write a todo | 14:12 |
| sdague | realistically we need a style parser for the queries to ferrit out some common issues | 14:12 |
| mriedem | like wildcards | 14:13 |
| sdague | yep | 14:13 |
| mriedem | and probably use of filename | 14:13 |
| mriedem | although you don't see that much anymore | 14:13 |
| sdague | yeh, a good query should include a tag or filename | 14:13 |
| sdague | honestly, I was going to write up a REVIEWING thing, maybe I'll do that now before my eye dr appointment | 14:14 |
| *** [1]evgenyf has quit IRC | 14:14 | |
| sdague | Shrews: ok. Would be good if ironic team members could keep an eye on ironic changes in devstack as well. | 14:14 |
| *** devananda has quit IRC | 14:14 | |
| mriedem | reviewing? we have some good content in the readme | 14:15 |
| *** evgenyf has joined #openstack-qa | 14:15 | |
| mriedem | but probably not completely up to date | 14:15 |
| *** devananda has joined #openstack-qa | 14:15 | |
| Shrews | sdague: yep, which is why i created this after our discussion yesterday: http://bit.ly/1ikKVFA | 14:15 |
| EmilienM | sdague: fyi, still investigating the issue in running javelin2 on my env | 14:15 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Convert scenario test tearDown to addCleanup https://review.openstack.org/62101 | 14:16 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/grenade: Ensure upgrade testing tests same deployment methods https://review.openstack.org/100764 | 14:16 |
| *** mestery has joined #openstack-qa | 14:16 | |
| yfried_ | mtreinish: updated https://review.openstack.org/#/c/62101/ | 14:17 |
| yfried_ | mtreinish: this requires careful review to make sure all test still behave the same | 14:17 |
| *** markmcclain has joined #openstack-qa | 14:18 | |
| afazekas_ | sdague: looks like we can have unofficial f20 spin with no_timer_check | 14:19 |
| *** Manishanker has quit IRC | 14:19 | |
| *** mlavalle has joined #openstack-qa | 14:21 | |
| Shrews | sdague: also on 98317, all of the ironic tests failed, so that at least is worthy of -1. i just voted based on that | 14:21 |
| *** gszasz has joined #openstack-qa | 14:21 | |
| *** esmute has quit IRC | 14:22 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: additional review guidelines https://review.openstack.org/100916 | 14:22 |
| sdague | afazekas_: I'm concerned about us having unofficial content here | 14:23 |
| sdague | EmilienM: ok cool. I'm going to be out for much of the afternoon because eye dr appointment, and not being able to read a screen for a few hours after. I'll take some time to dig in with you on this tomorrow if things remain stuck. | 14:23 |
| sdague | mriedem: how about those additional comments? | 14:24 |
| EmilienM | sdague: oh, good luck with that | 14:24 |
| *** esmute has joined #openstack-qa | 14:25 | |
| sdague | EmilienM: yeh, it's just been a long time since they did the dialation thing, so they like to do it. :) | 14:25 |
| EmilienM | :( | 14:25 |
| *** dims_ has joined #openstack-qa | 14:27 | |
| *** leseb has joined #openstack-qa | 14:27 | |
| *** kashyap has joined #openstack-qa | 14:29 | |
| *** rwsu has joined #openstack-qa | 14:29 | |
| kashyap | afazekas, Hi, should we continue the discussion from #fedora-cloud here? | 14:30 |
| afazekas_ | sdague: what is your concern about an unofficial spin ? | 14:31 |
| sdague | because if it breaks, who's fixing it? | 14:31 |
| sdague | putting something into the gate means we're going to block people's code on it | 14:32 |
| sdague | if it breaks | 14:32 |
| kashyap | sdague, Fedora cloud image building folks are very proactive/responsive, and can fix issues quite quickly | 14:32 |
| sdague | kashyap: are they in this channel? | 14:33 |
| kashyap | sdague, I can ask them to join here. But the release engineer doing the builds is off to drop his kids at school | 14:33 |
| sdague | right, but that's sort of part of the problem. If it's one person, that's not going to be very easy to get support out of it if that person goes on vacation. | 14:34 |
| openstackgerrit | A change was merged to openstack/tempest: Sync versionutils from oslo-incubator https://review.openstack.org/100030 | 14:34 |
| openstackgerrit | A change was merged to openstack/tempest: Don't include boto tests as part of smoke tests https://review.openstack.org/100717 | 14:34 |
| sdague | anyway, we've got a couple of different issues here | 14:34 |
| openstackgerrit | A change was merged to openstack/tempest: Sharing codes for cinder v1 and v2 tests https://review.openstack.org/96674 | 14:34 |
| kashyap | sdague, It's just one, there's 2-3 folks working on it | 14:34 |
| kashyap | It's not* | 14:35 |
| afazekas_ | sdague: The image itself does not change when it is built. | 14:35 |
| sdague | #1 - it's not possible to download from the fedora infrastructure reliably - 10% failure rate from our cloud providers | 14:35 |
| afazekas_ | If it required even I can rebuild it | 14:35 |
| *** mattdm has joined #openstack-qa | 14:35 | |
| kashyap | sdague, Sure, please discuss it here, if they can be resolved by working w/ fedora rel. eng, I'd be happy to help | 14:35 |
| mattdm | hi all | 14:35 |
| sdague | #2 - there are possibly issues with the normal upstream fedora image that could be fixed with custom respin | 14:35 |
| kashyap | sdague, mattdm is here, he's Fedora project leader, and cloud sig technical lead too | 14:35 |
| kashyap | mattdm, sdague was summarizing two issues currently w/ Fedora images. . . | 14:36 |
| kashyap | Just repasting: | 14:36 |
| sdague | #3 - having a custom respin means that we're off the main path for testing of the image | 14:36 |
| kashyap | sdague> #1 - it's not possible to download from the fedora infrastructure reliably - 10% failure rate from our cloud providers | 14:36 |
| kashyap | sdague, Thanks for clearly writing them, let's give Matt a few minutes to catch up on scroll | 14:37 |
| sdague | yep, no prob | 14:37 |
| mattdm | sdague We'd be off the main path until f21... since respinning the image is currently a very heavyweight proceedure, having it unofficial lets us address problems more quickly | 14:38 |
| mattdm | is the failure in hitting the fedora mirrors or fedora core infrastructure? | 14:38 |
| mattdm | I *would* like to be on the main path, obviously | 14:38 |
| *** larsks has joined #openstack-qa | 14:39 | |
| EmilienM | sdague: I still have the issue with the glance image, maybe you know the fix | 14:39 |
| sdague | mattdm: I don't fully know, I think going through the url we are using we get bounced to mirrors | 14:39 |
| EmilienM | sdague: http://paste.openstack.org/show/84399/ | 14:39 |
| mattdm | sdague okay, yeah, in that case, yes. 10% is an awfully high failure rate though | 14:39 |
| kashyap | mattdm, Yes - we certainly have to make re=spinning images a bit more lighter and decentralized. FWIW, personally, I'm using 'virt-builder' to create updated-images in my test env. trivially in a scripted manner | 14:40 |
| mattdm | I would, however, not be terribly surprised with a 1% rate -- it's a really big network run by volunteers. software should silently retry at least once. | 14:40 |
| mattdm | kashyap there's a balance -- the centralized approach means that they're transparent, repeatable, and trackable. | 14:41 |
| kashyap | mattdm, Oh sure -- I see that point voiced by dgilmore as well on #fedora-cloud | 14:41 |
| kashyap | I'm all for accurate/trackable images | 14:41 |
| sdague | I also don't really want us to be testing images for the first time in our CI, because then our CI isn't testing openstack, it's testing fedora | 14:41 |
| *** sabeen has joined #openstack-qa | 14:41 | |
| sdague | it's really important to remember these images will get 4000 test runs a week on them | 14:42 |
| kashyap | Fair point. | 14:42 |
| sdague | so if there is even a 1% bug | 14:42 |
| mattdm | sdague yeah, so that' exactly the catch. our QA process is currently _only_ targetted at the offical releases | 14:43 |
| sdague | that's 40 patches we failed | 14:43 |
| mattdm | we're changing that for F21, but weren't planning to backport that process to f20 | 14:43 |
| sdague | mattdm: right, which is why I'm concerned about doing a custom spin | 14:43 |
| *** Longgeek_ has quit IRC | 14:43 | |
| kashyap | mattdm, Quick note - we should use tools like `qemu-sanity-check` to automate minimal testing of non-official images, there by removing humans from the chore | 14:43 |
| kashyap | But that's a separate discussion, don't want to divert topic | 14:44 |
| *** Longgeek has joined #openstack-qa | 14:44 | |
| mattdm | sdague: yeah but I'm not sure what the other options are. an official respin would not, at this point, get heavy qa _either_ | 14:44 |
| sdague | ok, well if people think this will help, we could give it a try | 14:45 |
| kashyap | mattdm, sdague: So, to summarize an action item here, even for future, -- a Fedora-QA (mostly auto) tested, custom image is acceptable for Gate? | 14:45 |
| *** Longgeek has quit IRC | 14:45 | |
| sdague | kashyap: honestly, we've never gotten enough reliability out of any fedora image to really say it was acceptable for the gate | 14:46 |
| sdague | which is why the heat job is currently non-voting | 14:46 |
| sdague | so if you think this will help, cool | 14:46 |
| sdague | we can try | 14:46 |
| kashyap | Okay, FWIW, I'm doing my level best to do all my testing w/ Fedora images | 14:46 |
| sdague | kashyap: sure, but you probably aren't doing 4000 runs a week :) | 14:47 |
| kashyap | sdague, True, sir | 14:47 |
| sdague | it's in the volume that we see the issue | 14:47 |
| sdague | our tollerance for even small failure rates is pretty low | 14:47 |
| mattdm | sdague what is the issue? | 14:47 |
| *** Longgeek has joined #openstack-qa | 14:47 | |
| sdague | mattdm: honestly, I disabled those tests from voting months ago, so I don't really know any more. It was a lot of the image failing too boot, and not really clear why. | 14:49 |
| * afazekas_ the no_timer_check is not fedora specific, we had timer check issues with cirros as well. | 14:49 | |
| * mattdm can't fix these things if there aren't bug reports | 14:49 | |
| kashyap | Yeah, some specifics would be useful :-( from those more in the CI trenches | 14:50 |
| sdague | kashyap: well this is actually part of my concern about even having the fedora image in the mix | 14:50 |
| sdague | there aren't more people to dig into the fails | 14:50 |
| sdague | and it's another variable | 14:50 |
| mattdm | for the glorious future of f21, we will be doing more of this before it gets to you. but again, doesn't help now :( | 14:51 |
| sdague | so probably step #0 is to get some folks to fully categorize what's happening | 14:51 |
| mattdm | +1 to step #0 | 14:51 |
| sdague | but that actually requires more people that have interest here | 14:51 |
| sdague | because, you know, we've got a few other uncategorized fails - http://status.openstack.org/elastic-recheck/data/uncategorized.html :) | 14:51 |
| kashyap | sdague, Okay, noted. IIRC, ianw is steadfastly working on Fedora in Gate as well | 14:52 |
| sdague | kashyap: well that's actually a different thing | 14:52 |
| sdague | because that's fedora as host | 14:53 |
| sdague | and not fedora as 2nd level guest | 14:53 |
| sdague | in heat tests | 14:53 |
| kashyap | I'll try to help whatever I can by testing these cloud images more often, but my knowledge of OpenStack CI is limited to null | 14:53 |
| kashyap | sdague, Hmm, okay, FWIW, I run nested KVM in all my fedora env running OpenStack | 14:53 |
| kashyap | So, all the Nova instances are nested KVM guests | 14:54 |
| afazekas_ | kashyap: With nested it is working | 14:54 |
| sdague | we're not actually nested, we're in qemu only | 14:54 |
| kashyap | Yeah, pure emulation is the issue I see | 14:54 |
| sdague | anyway, need to run | 14:54 |
| sdague | later folks | 14:54 |
| *** mrodden has joined #openstack-qa | 14:54 | |
| kashyap | afazekas_, If the host supports nested KVM, shouldn't the nodepool or whatever is picking the guests enable it by default? | 14:55 |
| afazekas_ | kashyap: I do not think is it enabled one any of the hosts. AFAIK the nesting is not really working (supported?) on anything else than f20.. | 14:58 |
| kashyap | afazekas_, Hmm, it's embarrassingly simple to do so: | 14:58 |
| kashyap | If this is 'Y' | 14:59 |
| kashyap | $ cat /sys/module/kvm_intel/parameters/nested | 14:59 |
| kashyap | then, enable host passthrough in L1 to expose /dev/kvm: | 14:59 |
| kashyap | $ virt-xml compute --edit --cpu host-passthrough,clearxml=ye | 14:59 |
| openstackgerrit | David Paterson proposed a change to openstack/qa-specs: Add spec for bp:post-run-cleanup https://review.openstack.org/91725 | 14:59 |
| kashyap | $ virt-xml compute --edit --cpu host-passthrough,clearxml=yes | 14:59 |
| openstackgerrit | David Paterson proposed a change to openstack/qa-specs: Made changes according to comments I received from the community. https://review.openstack.org/100932 | 14:59 |
| kashyap | afazekas_, Anyway, let's see how it goes | 14:59 |
| *** Poornima|afk has quit IRC | 14:59 | |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/92809 failed gate-grenade-dsvm in the gate queue with an unrecognized error | 15:01 |
| afazekas_ | kashyap: see you, BTW: host-passthrough can be set in the nova.conf (libvirt_cpu_mode=host-passthroug) | 15:03 |
| afazekas_ | h | 15:03 |
| afazekas_ | bye | 15:03 |
| *** dkranz has quit IRC | 15:04 | |
| *** TravT has joined #openstack-qa | 15:05 | |
| *** kgriffs|afk is now known as kgriffs | 15:07 | |
| *** dims_ has quit IRC | 15:07 | |
| kashyap | afazekas_, Cool, thanks | 15:07 |
| *** dpaterson has joined #openstack-qa | 15:09 | |
| *** afazekas_ has quit IRC | 15:10 | |
| *** yamahata has quit IRC | 15:13 | |
| openstackgerrit | David Paterson proposed a change to openstack/qa-specs: Made changes according to comments provided by community https://review.openstack.org/100938 | 15:13 |
| *** adalbas has quit IRC | 15:14 | |
| *** imcleod has joined #openstack-qa | 15:15 | |
| *** davpat2112 has joined #openstack-qa | 15:16 | |
| davpat2112 | Running into issue trying to update a spec by following the gerrit workflow. I made a change. Do git commit -a and add new commit message, then git review I am getting a new changeId, which is inccorect from what I understand. | 15:17 |
| davpat2112 | But if I put the changeId in the commit msg and try git review I receive an error about a duplicate request. | 15:18 |
| davpat2112 | Can someone please tell me the proper flow for updating an existing spec? | 15:18 |
| *** dpaterson has quit IRC | 15:19 | |
| *** dkranz has joined #openstack-qa | 15:19 | |
| *** Longgeek has quit IRC | 15:20 | |
| *** jpich has quit IRC | 15:22 | |
| *** Picoviski has quit IRC | 15:22 | |
| *** shardy is now known as shardy_afk | 15:23 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add tox target for building docs https://review.openstack.org/100940 | 15:25 |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Update readme with more guidelines for writing queries https://review.openstack.org/100941 | 15:25 |
| *** adalbas has joined #openstack-qa | 15:27 | |
| *** dpaterson has joined #openstack-qa | 15:28 | |
| *** davpat2112 has quit IRC | 15:31 | |
| mtreinish | davpat2112: you need to use git commit --amend, see: https://wiki.openstack.org/wiki/Gerrit_Workflow#Updating_a_Change | 15:32 |
| *** dims_ has joined #openstack-qa | 15:32 | |
| openstackgerrit | Malini Kamalambal proposed a change to openstack/tempest: Add Tests for Message & Claim APIs https://review.openstack.org/94944 | 15:38 |
| *** jpich has joined #openstack-qa | 15:39 | |
| *** mkoderer has joined #openstack-qa | 15:42 | |
| *** dims_ has quit IRC | 15:42 | |
| *** dims_ has joined #openstack-qa | 15:44 | |
| *** malini1 has quit IRC | 15:44 | |
| *** mpavlase has left #openstack-qa | 15:45 | |
| *** Sukhdev has joined #openstack-qa | 15:45 | |
| *** ildikov has quit IRC | 15:48 | |
| openstackgerrit | A change was merged to openstack/tempest: Use known flavor ID as marker https://review.openstack.org/100433 | 15:48 |
| *** sballe_ has joined #openstack-qa | 15:53 | |
| *** SridharG has quit IRC | 15:55 | |
| *** jpich has quit IRC | 15:55 | |
| *** sballe has quit IRC | 15:56 | |
| EmilienM | sdague: i found out the issue with glance images in javelin | 15:57 |
| *** dims_ has quit IRC | 15:57 | |
| *** dkranz has quit IRC | 16:00 | |
| *** salv-orlando_ has quit IRC | 16:00 | |
| *** gszasz has quit IRC | 16:00 | |
| *** leseb has quit IRC | 16:01 | |
| *** leseb has joined #openstack-qa | 16:01 | |
| *** amotoki has joined #openstack-qa | 16:02 | |
| *** jlanoux has quit IRC | 16:02 | |
| *** jlibosva has quit IRC | 16:03 | |
| *** sabeen1 has joined #openstack-qa | 16:03 | |
| *** sabeen has quit IRC | 16:05 | |
| *** leseb has quit IRC | 16:06 | |
| jogo | sdague: hacking getting the pep8 patch makes sense | 16:07 |
| jogo | its supposed to be the only repo that uses it directly | 16:07 |
| *** aimon_ has quit IRC | 16:07 | |
| *** evgenyf has quit IRC | 16:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/96829 | 16:12 |
| *** dkranz has joined #openstack-qa | 16:12 | |
| mriedem | dims might have been onto something with the nova-network ssh timeouts and boto tests given a recent change in boto https://github.com/boto/boto/commit/fb3a7b407488c8b2374502d10a90d431daf0aef9 | 16:14 |
| *** dpaterson has quit IRC | 16:14 | |
| mriedem | re connection pooling | 16:14 |
| mriedem | on a side note, i guess boto doesn't care much about descriptive commit messages | 16:15 |
| yfried_ | mtreinish: ping | 16:15 |
| *** hemna_ is now known as hemna | 16:15 | |
| *** chandan_kumar has joined #openstack-qa | 16:18 | |
| rbak | Does anyone know if Tempest can run in existing projects rather than creating new ones for each test? | 16:19 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add tox target for building docs https://review.openstack.org/100940 | 16:21 |
| *** openstackrecheck has quit IRC | 16:22 | |
| *** openstackrecheck has joined #openstack-qa | 16:22 | |
| yfried_ | rbak: what do you mean existing project? point it to an existing openstack cloud? this should work | 16:26 |
| yfried_ | rbak: if you mean work on a specific project(=tenant) without admin creds, then I don't think so | 16:27 |
| rbak | yfried_: I do mean a tenant, and it could have admin creds, but currently the admin appears to create a separate tenant for every test. | 16:29 |
| mrodden | sigh https://bugs.launchpad.net/bash8/+bug/1331548 | 16:29 |
| yfried_ | rbak: if you set allow_tenant_isolation=False it should avoid that | 16:29 |
| rbak | yfried_: Thanks, I'll try that | 16:29 |
| yfried_ | rbak: instead it will expect 3 tenants (admin, demo, alt_demo) | 16:30 |
| yfried_ | rbak: you should provide those | 16:30 |
| yfried_ | rbak: but this means no parallel execution of tests | 16:30 |
| rbak | yfried_: I already had those provided, I just didn't know how to force Tempest to use those instead of creating new ones. | 16:31 |
| *** wendar_ is now known as wendar | 16:31 | |
| rbak | yfried_: And that's good to know about the parallel execution | 16:31 |
| yfried_ | rbak: enjoy. | 16:31 |
| *** malini1 has joined #openstack-qa | 16:31 | |
| *** leseb has joined #openstack-qa | 16:32 | |
| yfried_ | rbak: I'm curious - may I ask why do you want to work this way? | 16:32 |
| rbak | yfried_: With Tempest creating extra tenants all the time it makes it hard to keep track cloud usage | 16:33 |
| rbak | yfried_: The tenants also are getting left behind, so we have to clean them up with separate scripts. | 16:34 |
| yfried_ | rbak: is this a "production" cloud? are you running tempest on a cloud that's being used for stuff other than testing openstack | 16:34 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Add min_client_version decorator for CLI tests https://review.openstack.org/100031 | 16:35 |
| mtreinish | rbak: the tenants shouldn't get left behind | 16:35 |
| mtreinish | they should be cleaned up after the test class finishes | 16:35 |
| yfried_ | rbak: ^ what he said :) | 16:35 |
| mtreinish | otherwise it's a bug | 16:35 |
| rbak | yfried_: Yes, this is production. We're trying to use Tempest to compliment the monitoring by running every few hours for more in depth testing | 16:35 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Add min_client_version decorator for CLI tests https://review.openstack.org/100031 | 16:36 |
| rbak | yfried_: mtreinish: Any ideas what would cause them to get left behind? I'm using the most current version. | 16:36 |
| *** qba73 has quit IRC | 16:37 | |
| *** leseb has quit IRC | 16:37 | |
| *** k4n0 has quit IRC | 16:37 | |
| *** leseb has joined #openstack-qa | 16:37 | |
| mtreinish | rbak: I'd have to see some logs and exactly what is getting left behind. But things don't tearDown correctly sometimes if there is a test failure. | 16:38 |
| yfried_ | rbak: usually tests that fail on clean up. if a test passed, then it shouldn't leave anything behind (otherwise that's a bug in tempest) | 16:38 |
| rbak | yfried_: mtreinish: The failed test behavior I expect, but even on successful tests tenants are getting left behind. I'll look into and see if I can figure out where the problem is. | 16:39 |
| mtreinish | rbak: for reference: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/isolated_creds.py#n430 | 16:40 |
| rbak | mtreinish: Great, thanks | 16:40 |
| mtreinish | the most common bug I can think of is that isn't being called for a test class | 16:40 |
| yfried_ | rbak: could it be skipped tests? I've seen this happen, but didn't have time to debug this | 16:41 |
| *** jordanP has quit IRC | 16:41 | |
| yfried_ | rbak: try to isolate the odd tenants, post a bug. I would like to hear more about that | 16:41 |
| rbak | yfried_: I don't think it's the skipped tests. There is far more tenants left behind than there are skipped tests. | 16:42 |
| yfried_ | rbak: are you running full tempest, or just scenarios? | 16:42 |
| yfried_ | rbak: as mtreinish said - test logs would be helpful | 16:42 |
| rbak | yfried_: Specific test sets for the moment. They mostly consist of the smoke api tests. | 16:43 |
| rbak | yfried_: And I'll get some logs together to see if I can find what's going on | 16:44 |
| rbak | yfried_: I had been assuming this was expected behavior until now | 16:44 |
| *** leseb has quit IRC | 16:50 | |
| *** leseb has joined #openstack-qa | 16:50 | |
| yfried_ | rbak: I wonder what kind of issues these periodic tests reveal. could you elaborate? | 16:52 |
| rbak | yfried_: Do you mean what we're trying to accomplish by running the tests every few hours? | 16:54 |
| *** leseb has quit IRC | 16:54 | |
| *** malini1 has quit IRC | 16:54 | |
| yfried_ | rbak: I assume it gives you data about the status of your cloud | 16:54 |
| yfried_ | rbak: I wonder what specific issues you have seen | 16:55 |
| *** malini1 has joined #openstack-qa | 16:55 | |
| yfried_ | rbak: are you running master branch? I assume your cloud isn't Juno | 16:56 |
| rbak | yfried_: This is all still being tested, so we don't have a lot of data yet, but this has helped uncover some problems like bugs in Keystone, or services that have hung but still report as up with the monitoring. | 16:57 |
| rbak | yfried_: And yes, we're using the master branch running on an Icehouse cloud | 16:57 |
| *** dims_ has joined #openstack-qa | 16:59 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 17:00 | |
| *** adalbas has quit IRC | 17:01 | |
| yfried_ | rbak: that's a great idea. I think we will start doing the same | 17:03 |
| *** harlowja_away is now known as harlowja | 17:06 | |
| *** amotoki has quit IRC | 17:07 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/98937 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 17:07 |
| *** Manishanker has joined #openstack-qa | 17:10 | |
| *** salv-orlando has joined #openstack-qa | 17:11 | |
| *** jhenner has quit IRC | 17:15 | |
| *** dpaterson has joined #openstack-qa | 17:15 | |
| *** jhenner has joined #openstack-qa | 17:15 | |
| *** adalbas has joined #openstack-qa | 17:16 | |
| *** Manishanker has quit IRC | 17:16 | |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack-infra/devstack-gate: Optional Ceph backend for Cinder https://review.openstack.org/100905 | 17:16 |
| *** sballe__ has joined #openstack-qa | 17:19 | |
| *** aimon__ has joined #openstack-qa | 17:20 | |
| *** luqas has quit IRC | 17:22 | |
| *** davpat2112 has joined #openstack-qa | 17:22 | |
| *** sballe_ has quit IRC | 17:23 | |
| *** wramthun has quit IRC | 17:23 | |
| *** wramthun has joined #openstack-qa | 17:24 | |
| devananda | sdague: hi! I don't understand your question on https://review.openstack.org/#/c/71476 -- can you explain what it is about create_nodes that you don't understand? | 17:26 |
| *** dpaterson has quit IRC | 17:26 | |
| *** pcrews has quit IRC | 17:27 | |
| sdague | devananda: https://review.openstack.org/#/c/71476/17/tempest/api/baremetal/test_nodes.py - L53 - L58 | 17:29 |
| sdague | you will be able to run that on any ironic installation anywhere? | 17:29 |
| devananda | sdague: if the client has access to the ironic API (eg, has admin credentials with the cloud, and that API endpoint is exposed) -- yes. | 17:33 |
| sdague | even if the host is arm? | 17:34 |
| devananda | sdague: that is registering a new node, then asserting that a node UUID was returned by creating it, and that a subsequent GET of that resource returns the supplied parameters | 17:34 |
| devananda | sdague: doesn't matter what the host is -- this isn't a real host | 17:34 |
| sdague | ok | 17:34 |
| devananda | sdague: it's a fake node | 17:34 |
| devananda | just an API exercise | 17:34 |
| devananda | however... | 17:35 |
| sdague | so if you pointed that at a real ironic installation? | 17:35 |
| devananda | it's missing an assertion of the availability of the 'fake' driver | 17:35 |
| sdague | we have to separate the specific configuration of the gate | 17:35 |
| devananda | if this were run against the pxe_ipmitool driver, for example, things would NOT work | 17:35 |
| sdague | right | 17:35 |
| devananda | this particular test might pass -- but the actual ironic service would start spewing errors | 17:35 |
| sdague | right | 17:36 |
| openstackgerrit | Malini Kamalambal proposed a change to openstack/tempest: Add Tests for Message & Claim APIs https://review.openstack.org/94944 | 17:36 |
| sdague | so this might lead us down the path of where there is a disconnected | 17:36 |
| sdague | disconnect in understanding between us | 17:36 |
| sdague | tempest is a test suite for clouds, which tests clouds | 17:36 |
| sdague | and can happen to test the clouds we stand up in the gate | 17:36 |
| sdague | but tests that don't work on all clouds that expose the openstack API, need to not be there | 17:37 |
| devananda | ironic is not a service which should (at this point in time) ever be exposed to end-users | 17:37 |
| devananda | yet we need to test it | 17:37 |
| devananda | it SHOULD be exposed within the cloud -- both to operators of that cloud, and to other services (eg nova) | 17:37 |
| sdague | so that's a new kind of beast | 17:38 |
| devananda | right | 17:38 |
| devananda | it doesn't make any sense to think of pointing this test at an HPCloud or rackspace *from outside* | 17:38 |
| devananda | even the rackspace dedicated hosting guys working on the ironic-python-agent project | 17:38 |
| devananda | will never expose the ironic API to their customers | 17:38 |
| devananda | customers will use Nova | 17:38 |
| devananda | their ops team will use ironic | 17:39 |
| sdague | hmmm | 17:39 |
| sdague | I think that's a distinction that might not be clear to people | 17:39 |
| devananda | heh | 17:39 |
| sdague | that ironic has *no* user API | 17:39 |
| sdague | only an admin API | 17:39 |
| devananda | it's clear to the folks that are working on ironic | 17:40 |
| sdague | right, but not clear to the rest of OpenStack | 17:40 |
| clarkb | wait | 17:40 |
| clarkb | devananda: I thought you explicitly stated users could/would use ironic directly without nova | 17:40 |
| devananda | clarkb: the term "user" is highly overloaded | 17:41 |
| sdague | so knowing this, the next change that someone needs to propose to tempest is to move all the tests into api/baremetal/admin | 17:41 |
| *** nati_ueno has joined #openstack-qa | 17:41 | |
| devananda | clarkb: an operator is a user | 17:41 |
| clarkb | devananda: well we were specifically discussing it in this context though | 17:42 |
| clarkb | devananda: because i was trying to frame ironic as a nova driver where end users consume it via nova | 17:42 |
| sdague | devananda: not really in openstack terms | 17:42 |
| clarkb | but I was told that this is not the case | 17:42 |
| devananda | clarkb: if someone is running openstackw ith ironic, the *user of the cloud* should talk to nova to provision instances | 17:42 |
| clarkb | ok thats what I was arguing for :) | 17:43 |
| clarkb | maybe we were just talking past each other | 17:43 |
| devananda | clarkb: in that same environment, cloud operations staff will talk to ironic eg, to manage the hardware made available to nova | 17:43 |
| sdague | right, users != operators, user api is for users, admin api is for operators | 17:43 |
| clarkb | devananda: specifically you argued that you expected people to use ironic without nova | 17:43 |
| devananda | clarkb: the confusion comes from "users of tripleo" | 17:43 |
| clarkb | as normal operations | 17:43 |
| devananda | clarkb: who are really operators | 17:43 |
| *** ildikov has joined #openstack-qa | 17:44 | |
| sdague | users of tripleo is a terrible confusion then :) | 17:44 |
| sdague | stop using that phrase | 17:44 |
| devananda | clarkb: anyone who's installing an operating system on a physical server a) is not an end-user and b) should use ironic (whether they ALSO use nova or not is up to them) | 17:44 |
| devananda | sdague: ++ | 17:44 |
| clarkb | they are an end user though | 17:44 |
| devananda | gah! | 17:44 |
| clarkb | they are the end user of your tool | 17:44 |
| devananda | words words words | 17:44 |
| sdague | clarkb: don't care | 17:44 |
| sdague | users are api consumers of non admin apis | 17:45 |
| devananda | sdague: moving all of the ironic tests into /admin/ sounds like a GREAT idea | 17:45 |
| sdague | if we diverge from that | 17:45 |
| *** Sukhdev has quit IRC | 17:45 | |
| sdague | we get confused | 17:45 |
| devananda | sdague: as it is, ironic enforces an admin check in our API before doing anything besides exposing the API endpoint | 17:45 |
| devananda | sdague: and we dont have any short-term plans of changing that | 17:45 |
| sdague | devananda: ok, so yeh, we should restructure the tempest tree then, because that's part of the confusion | 17:46 |
| devananda | adam_g: ^ | 17:46 |
| sdague | and probably put a README in the baremetal subtree explaining that | 17:46 |
| devananda | sdague: is it OK in tempest to assert something about a service prior to running tests? | 17:46 |
| devananda | sdague: eg, assert that the fake driver is available | 17:46 |
| devananda | sdague: it's discoverable via GET /v1/drivers | 17:47 |
| sdague | devananda: typically we want that to be set in the config file | 17:47 |
| devananda | if "fake" isn't present, I dont think the /api/ tests should run | 17:47 |
| *** safchain has quit IRC | 17:47 | |
| sdague | so that you don't have autoskipping and passing, when really your service returned garbage | 17:47 |
| devananda | Shrews: also pinging you here -- both you and adam should read the scrollback, when you're back | 17:48 |
| *** malini1 has quit IRC | 17:48 | |
| openstackgerrit | Vishal kumar mahajan proposed a change to openstack/tempest: Added swift CLI test cases https://review.openstack.org/96386 | 17:48 |
| adam_g | moving the baremetal API tests to /admin/ makes sense to me | 17:49 |
| devananda | sdague: fwiw, our docs have said for the last year "A RESTful API service, by which operators and other services may..." -- think that needs to be worded stronger? | 17:52 |
| * devananda tosses up a doc patch | 17:52 | |
| *** malini1 has joined #openstack-qa | 17:52 | |
| *** SridharG has joined #openstack-qa | 17:53 | |
| *** nati_ueno has quit IRC | 17:53 | |
| sdague | devananda: sure, but there are lots of docs in openstack world :) | 17:53 |
| *** dims_ has quit IRC | 17:53 | |
| sdague | part of the problem is the fire hose is so furious you probably need to provide extra signaling in the way this is not like other openstack services | 17:54 |
| *** nati_ueno has joined #openstack-qa | 17:54 | |
| sdague | sort of like the same surprise we call got when we figured out how marconi worked, because it wasn't what people were expected | 17:54 |
| devananda | sdague: feel free to point me at any that say anything otherwise. folks write stuff about ironic nwo that I never see ... it's kinda cool | 17:54 |
| sdague | devananda: I'm not sure anyone said anything otherwise | 17:54 |
| devananda | ah, gotcha | 17:55 |
| sdague | I just have the idea that openstack services provide an end user api | 17:55 |
| sdague | and a service that doesn't... is new | 17:55 |
| devananda | *nod* | 17:55 |
| yfried_ | mtreinish: I hate to nag, but I would like to have your general opinion about https://review.openstack.org/62101 in general. since we are in oposite timezones, I don't want to miss tomorrow (it's a hell to rebase...) | 17:58 |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/98844 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/921858 | 17:59 |
| *** salv-orlando has quit IRC | 18:00 | |
| *** SridharG has quit IRC | 18:00 | |
| *** dims_ has joined #openstack-qa | 18:01 | |
| mtreinish | yfried_: I haven't had a chance to really look at it yet. I'll take a look after I eat some lunch | 18:02 |
| yfried_ | mtreinish: enjoy your lunch. no pressure. just needed to know you are aware of that. I would appriaicate your help with pushing this in this weeks meeting. (not sure if I can attend it yet) | 18:03 |
| devananda | adam_g: what do you think of adding a check to tempest/api/baremetal so that those only run when the "fake" driver is available (or some config option which defaults to the "fake" driver) | 18:06 |
| devananda | adam_g: as I think trying to run all the CRUD tests against any other driver is going to go sideways .... | 18:06 |
| *** torandu_ has quit IRC | 18:07 | |
| *** torandu_ has joined #openstack-qa | 18:13 | |
| adam_g | devananda, makes sense for that to be configurable. we'll likely want to expose driver info via config once we have IPA | 18:15 |
| *** jlibosva has joined #openstack-qa | 18:22 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Make test_snapshot_pattern ssh error path work https://review.openstack.org/98446 | 18:33 |
| sdague | oh... that looks interesting | 18:33 |
| *** dims_ has quit IRC | 18:36 | |
| *** jaypipes has joined #openstack-qa | 18:37 | |
| *** ChanServ sets mode: +o jaypipes | 18:37 | |
| mtreinish | sdague: heh, something about code paths not getting exercised not working... | 18:37 |
| openstackgerrit | Vishal kumar mahajan proposed a change to openstack/tempest: Added swift CLI test cases https://review.openstack.org/96386 | 18:38 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Move baremetal API tests to an /admin subdir https://review.openstack.org/100989 | 18:38 |
| *** samuelbercovici1 has joined #openstack-qa | 18:39 | |
| *** samuelbercovici1 is now known as samuelbercovici | 18:39 | |
| *** dpaterson has joined #openstack-qa | 18:40 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack-dev/devstack: Install oslo.db from source https://review.openstack.org/100991 | 18:41 |
| *** kgriffs is now known as kgriffs|afk | 18:42 | |
| *** davpat2112 has quit IRC | 18:43 | |
| sdague | jogo: so I think I found the crux of the moved race with screen | 18:44 |
| sdague | https://github.com/openstack-dev/devstack/blob/master/functions-common#L1076 | 18:45 |
| sdague | I think screen is actually dropping this bit | 18:45 |
| sdague | because it's still spawning | 18:45 |
| *** davpat2112 has joined #openstack-qa | 18:49 | |
| *** dpaterson has quit IRC | 18:51 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Neutron: start to use nova's neutron configuration section https://review.openstack.org/100779 | 18:55 |
| *** BobBall is now known as BobBall_Away | 18:55 | |
| openstackgerrit | Kaitlin Farr proposed a change to openstack/tempest: Adds scenario tests for volume encryption https://review.openstack.org/58018 | 18:57 |
| *** tosky has quit IRC | 19:00 | |
| *** dims_ has joined #openstack-qa | 19:00 | |
| *** TravT has quit IRC | 19:01 | |
| *** openstackgerrit has quit IRC | 19:06 | |
| *** openstackgerrit_ has joined #openstack-qa | 19:07 | |
| *** openstackgerrit_ is now known as openstackgerrit | 19:08 | |
| *** jaypipes has quit IRC | 19:09 | |
| *** dims_ has quit IRC | 19:09 | |
| *** jlibosva has quit IRC | 19:10 | |
| *** kgriffs|afk is now known as kgriffs | 19:18 | |
| *** SridharG has joined #openstack-qa | 19:18 | |
| jogo | sdague: can we just stop using screen in grenade? | 19:24 |
| sdague | jogo: it would be some unspecified amount of work | 19:24 |
| *** jaypipes has joined #openstack-qa | 19:24 | |
| *** ChanServ sets mode: +o jaypipes | 19:24 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: double up [ for safety https://review.openstack.org/100728 | 19:27 |
| *** mriedem has quit IRC | 19:33 | |
| *** mriedem has joined #openstack-qa | 19:38 | |
| openstackrecheck | openstack/ironic change: https://review.openstack.org/99518 failed because of: gate-ironic-python26: https://bugs.launchpad.net/bugs/1282795, gate-tempest-dsvm-ironic: unrecognized error | 19:42 |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: Build retry loop for screen sessions https://review.openstack.org/101004 | 19:42 |
| sdague | so that will be interesting | 19:42 |
| sdague | or possibly explode horribly | 19:42 |
| openstackgerrit | Sabeen proposed a change to openstack/tempest: Create and delete a stack using a HOT template https://review.openstack.org/97257 | 19:42 |
| mtreinish | sdague: http://logs.openstack.org/89/100989/1/check/check-tempest-dsvm-full/ce20500/logs/devstacklog.txt.gz a package version issue? | 19:43 |
| mtreinish | I don't really have a lot of apt experience | 19:43 |
| *** malini1 has quit IRC | 19:43 | |
| sdague | mtreinish: nope | 19:43 |
| sdague | it's the mirror isn't accessible | 19:43 |
| mtreinish | ahh ok | 19:44 |
| sdague | http://logs.openstack.org/89/100989/1/check/check-tempest-dsvm-full/ce20500/logs/devstacklog.txt.gz#_2014-06-18_18_44_16_453 | 19:44 |
| mtreinish | heh, I just scrolled to the bottom and missed that | 19:44 |
| sdague | Bug 1286818 - Ubuntu mirror periodically inconsistent causing gate build failures | 19:44 |
| sdague | I think that's the issue | 19:44 |
| *** TravT has joined #openstack-qa | 19:45 | |
| *** mmaglana has joined #openstack-qa | 19:46 | |
| *** dustins has quit IRC | 19:49 | |
| *** Obulpath_ has joined #openstack-qa | 19:57 | |
| *** Obulpathi has quit IRC | 19:57 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/100666 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1298472 | 19:58 |
| *** leseb has joined #openstack-qa | 20:02 | |
| mtreinish | sdague: did you see my comments on: https://review.openstack.org/#/c/96445 | 20:03 |
| mtreinish | if you add the unit test work item I'll push that bp through | 20:03 |
| sdague | I had not | 20:04 |
| sdague | sure, I'll spin around on it | 20:04 |
| *** SridharG has quit IRC | 20:04 | |
| *** Obulpath_ has quit IRC | 20:05 | |
| *** Obulpathi has joined #openstack-qa | 20:05 | |
| mtreinish | flwang: on https://review.openstack.org/#/c/99857/ is that a general glance scenario testing bp or just for that one test? | 20:05 |
| sdague | mriedem: if you want to wordsmith them together go for it :) | 20:08 |
| mriedem | sdague: ok, just throw them into mine? | 20:09 |
| sdague | mriedem: yeh | 20:09 |
| mriedem | will do | 20:09 |
| *** jordanP has joined #openstack-qa | 20:09 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Add min_client_version decorator for CLI tests https://review.openstack.org/100031 | 20:09 |
| sdague | you were there first... even if it was with a woosy WIP patch :) | 20:09 |
| sdague | commit man | 20:09 |
| mriedem | woosy? wussy? | 20:09 |
| sdague | or that | 20:09 |
| mriedem | ha | 20:10 |
| sdague | I was alliterating for effect | 20:10 |
| sdague | jogo: 97 commits today - https://github.com/openstack/openstack/graphs/commit-activity - it's some times really nice to enjoy the throughput of the system when it works | 20:12 |
| openstackgerrit | A change was merged to openstack/qa-specs: Improve wording in nova-api-test-inheritance https://review.openstack.org/92272 | 20:12 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Add min_client_version decorator for CLI tests https://review.openstack.org/100031 | 20:12 |
| *** jordanP has quit IRC | 20:14 | |
| jogo | sdague: nice | 20:15 |
| jogo | sdague: and 4 hours to go | 20:15 |
| *** jaypipes has quit IRC | 20:15 | |
| sdague | mtreinish: what about this one - https://review.openstack.org/#/c/98819/ | 20:17 |
| mtreinish | sdague: +2 | 20:18 |
| sdague | I'm trying to cull my outbound queue through either getting things moving forward, or abandoned :) | 20:19 |
| sdague | dkranz: you around? | 20:19 |
| dkranz | sdague: Yup | 20:19 |
| sdague | https://review.openstack.org/#/c/98505/ exposed the fact that your DEBUG path doesn't work all the time | 20:19 |
| sdague | anything thoughts on that | 20:19 |
| sdague | s/anything/any/ | 20:20 |
| dkranz | sdague: Hmm. It worked for me. I'll take a look. | 20:20 |
| sdague | I think the slicing on body blows up some times | 20:21 |
| dkranz | sdague: seems like there is a non-asciii character issue | 20:22 |
| sdague | right, but the body doesn't have to be ascii | 20:22 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Move baremetal API tests to an /admin subdir https://review.openstack.org/100989 | 20:23 |
| dkranz | sdague: I've never quite understood the unicode lossage in py2 | 20:25 |
| dkranz | sdague: so "str" does not just render non-ascii as \xnn ? | 20:26 |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: Build retry loop for screen sessions https://review.openstack.org/101004 | 20:26 |
| sdague | dkranz: honestly, this is one of those areas where I'm grey on how that all works | 20:27 |
| dkranz | sdague: ok, I'll poke a bit | 20:27 |
| *** safchain has joined #openstack-qa | 20:29 | |
| dkranz | sdague: ok, I have a fix. Do you want to include it in your patch or should I submit a separate patch? | 20:38 |
| sdague | dkranz: actually, my patch was really an idea up for debate. But I think we should fix the underlying issue | 20:39 |
| sdague | so I'd say do your patch separately | 20:40 |
| dkranz | sdague: Of course. | 20:40 |
| dkranz | sdague: ok | 20:40 |
| sdague | cool, thankis | 20:40 |
| *** wramthun has quit IRC | 20:43 | |
| *** wramthun has joined #openstack-qa | 20:43 | |
| *** danfinch has quit IRC | 20:54 | |
| *** mwagner_lap has quit IRC | 21:00 | |
| *** yamahata has joined #openstack-qa | 21:00 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Update readme with more guidelines for writing queries https://review.openstack.org/100941 | 21:05 |
| mriedem | sdague: merged ^ | 21:05 |
| *** dims_ has joined #openstack-qa | 21:06 | |
| sdague | mriedem: +2 | 21:06 |
| *** jamielennox is now known as jamielennox|away | 21:08 | |
| mrodden | well i found a way to use tox in devstack... | 21:08 |
| sdague | mrodden: oh... ? | 21:10 |
| mrodden | skipsdist = True | 21:10 |
| mrodden | no setup.py needed | 21:10 |
| mrodden | you can use it to build venv.s, install deps, and run commands | 21:11 |
| mrodden | kind of handy | 21:11 |
| sdague | nice | 21:11 |
| sdague | that sounds like a win | 21:11 |
| mrodden | yeah | 21:12 |
| sdague | so you going to do the rename first? | 21:12 |
| mrodden | not sure | 21:12 |
| sdague | then we can pull this down into devstack and grenade | 21:12 |
| mrodden | yeah | 21:12 |
| mrodden | i was going to poke the infra guys today and ask how much needs to change and how hard it is | 21:12 |
| mrodden | like the pypi namespace | 21:12 |
| mrodden | and the gerrit project name | 21:13 |
| sdague | yeh, we should bring it up now, because they do friday renames | 21:13 |
| sdague | and get into the queue | 21:13 |
| mrodden | yeah good point... | 21:13 |
| sdague | I'll let you open the conversation, I'll play backup | 21:13 |
| mrodden | sure | 21:13 |
| mrodden | but first are we sure we dont want to rename it bikeshed8 ? | 21:14 |
| mrodden | :) | 21:14 |
| sdague | heh | 21:14 |
| sdague | flame8 | 21:14 |
| mrodden | +2 | 21:14 |
| *** samuelbercovici has quit IRC | 21:17 | |
| *** yamahata has quit IRC | 21:18 | |
| *** dkranz has quit IRC | 21:18 | |
| *** yamahata has joined #openstack-qa | 21:18 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add a config option to select compute xml api https://review.openstack.org/99737 | 21:18 |
| *** mmaglana has quit IRC | 21:18 | |
| sabeen1 | Hi All - I'm trying to change the port for the keystone endpoint in devstack, but have not been successful so far.. i've tried changing the port # in openrc and in keystone.conf but it still remains at 5000 .. does anyone know how to do that? | 21:20 |
| *** dims_ has quit IRC | 21:20 | |
| *** dims_ has joined #openstack-qa | 21:21 | |
| *** ekhugen has quit IRC | 21:23 | |
| *** cbader has quit IRC | 21:24 | |
| flwang | mtreinish: ping | 21:25 |
| *** weshay has quit IRC | 21:25 | |
| mtreinish | flwang: pong | 21:25 |
| flwang | mtreinish: I saw you mentioned my glance scenario bp, but seems no conclusion, right? | 21:26 |
| *** dims_ has quit IRC | 21:26 | |
| flwang | mtreinish: you can refer this spec https://review.openstack.org/#/c/99857/ | 21:26 |
| mtreinish | flwang: no I was asking you is that just for one test? | 21:26 |
| flwang | mtreinish: I'm the glance core and it's reviewed by flaper87, another glance core | 21:26 |
| flwang | mtreinish: it's for scenario test, create image, and share it | 21:27 |
| flwang | even public it | 21:27 |
| mtreinish | flwang: no I understand it's a new test case and what it would entail. I'm trying to ascertain the scope of that spec proposal | 21:28 |
| *** safchain has quit IRC | 21:29 | |
| mtreinish | because specs/bps are only needed for larer test development efforts | 21:29 |
| mtreinish | not individual tests | 21:29 |
| mtreinish | *larger | 21:29 |
| flwang | mtreinish: ah, I think it's not large effort, I just want fill the image scenario test gap | 21:30 |
| flwang | if that's what you want to know | 21:30 |
| flwang | mtreinish: so only one patch for this as you see | 21:30 |
| mtreinish | flwang: yeah if you just working on a new scenario test feel free to push that without a spec. | 21:30 |
| mtreinish | but if you were working on several glance tests then having a bp to track that effort is useful | 21:31 |
| mtreinish | and that would need a spec | 21:31 |
| flwang | mtreinish: ah, I saw your mail in the openstack-dev :) and I'm trying to follow up | 21:31 |
| flwang | sorry for the trouble | 21:31 |
| *** davpat2112 has quit IRC | 21:31 | |
| mtreinish | flwang: sure, no worries | 21:33 |
| flwang | mtreinish: cheers | 21:33 |
| *** luqas has joined #openstack-qa | 21:34 | |
| *** mmaglana has joined #openstack-qa | 21:36 | |
| *** mmaglana has quit IRC | 21:36 | |
| *** mmaglana has joined #openstack-qa | 21:37 | |
| flwang | mtreinish: btw, it would be nice if you can take a loot at this patch https://review.openstack.org/#/c/99514/ when you're most convenient, thanks | 21:42 |
| *** andreaf has quit IRC | 21:48 | |
| *** luqas has quit IRC | 21:48 | |
| *** danfinch has joined #openstack-qa | 21:53 | |
| *** mriedem has quit IRC | 21:59 | |
| *** mkoderer has quit IRC | 22:02 | |
| *** Obulpathi has quit IRC | 22:02 | |
| *** Sukhdev has joined #openstack-qa | 22:06 | |
| *** ghyoc has quit IRC | 22:13 | |
| *** dims_ has joined #openstack-qa | 22:16 | |
| *** dims_ has quit IRC | 22:19 | |
| *** dims_ has joined #openstack-qa | 22:20 | |
| *** yamahata has quit IRC | 22:28 | |
| mrodden | sdague: if you are still around https://review.openstack.org/#/c/101042/ | 22:35 |
| *** leseb has quit IRC | 22:42 | |
| *** leseb has joined #openstack-qa | 22:43 | |
| *** SpamapS has joined #openstack-qa | 22:44 | |
| SpamapS | If I were to make tempest depend on os-collect-config in a test, would there be any issue with that? | 22:45 |
| mtreinish | SpamapS: it's in global req right? I don't think there would be a problem with that being a dependency. | 22:45 |
| jogo | sdague: 108 | 22:45 |
| mtreinish | but what would you be using it for? | 22:46 |
| mtreinish | SpamapS: oh wait do we install that from trunk in devstack? | 22:46 |
| SpamapS | mtreinish: https://git.openstack.org/cgit/openstack/os-collect-config/tree/os_collect_config/cfn.py | 22:47 |
| *** mwagner_lap has joined #openstack-qa | 22:47 | |
| SpamapS | mtreinish: I want to make sure that the "Collector" class there works. | 22:47 |
| *** leseb has quit IRC | 22:47 | |
| SpamapS | https://git.openstack.org/cgit/openstack/os-collect-config/tree/os_collect_config/heat.py is also important. :) | 22:47 |
| SpamapS | mtreinish: They're public API's, so we don't have to shell out to call os-collect-config or anything.. just need to setup cfg.CONF a little and then we can import/call them directly. | 22:48 |
| mtreinish | SpamapS: ok so this seems like it would probably be a decent heat scenario test | 22:49 |
| mtreinish | config with tempest gets a bit hairy though, it's not exactly the same as the other projects | 22:50 |
| mtreinish | because eveything in tempest gets executed from inside a test runner | 22:50 |
| *** leseb has joined #openstack-qa | 22:50 | |
| mtreinish | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py | 22:50 |
| mtreinish | but we can probably find a way around that | 22:50 |
| SpamapS | mtreinish: what makes it a scenario? | 22:51 |
| SpamapS | I am a bit tempest-stupid. | 22:51 |
| *** mestery has quit IRC | 22:51 | |
| mtreinish | SpamapS: well there are 4 classes of tests. API, scenario, cli, and third party (excluding the stress suite and unit tests) | 22:52 |
| *** rbak has quit IRC | 22:52 | |
| *** sballe__ has quit IRC | 22:52 | |
| mtreinish | api tests are about rest api functional validation and doesn't use the official clients | 22:52 |
| mtreinish | the scenario tests are about simulating end user workflows | 22:53 |
| mtreinish | which this feels like | 22:53 |
| SpamapS | Ah ok so that distinction does apply. | 22:53 |
| * mrodden & | 22:53 | |
| SpamapS | mtreinish: ok I think I'll look and see how to isolate cfg.CONF | 22:54 |
| SpamapS | mtreinish: I believe I can replace the module's copy | 22:54 |
| SpamapS | "monkey patch" I mean. | 22:54 |
| openstackgerrit | Fei Long Wang proposed a change to openstack/tempest: Add scenario test for image service https://review.openstack.org/100140 | 22:54 |
| devananda | stable branch question for folks -- what happens if we add a tempest test that works with a feature in master, but the feature wasn't present in, say,the last stable branch | 22:55 |
| devananda | do we use feature flags for that? | 22:55 |
| *** leseb has quit IRC | 22:55 | |
| mtreinish | SpamapS: yeah I'm sure there is a way we can work around the config side of things, we probably would need to solve it eventually :) | 22:55 |
| mtreinish | SpamapS: also http://docs.openstack.org/developer/tempest/#field-guides might be useful | 22:55 |
| devananda | Shrews: I'm thinking of the baremetal advanced scenario, testing preserve ephemeral, here | 22:55 |
| mtreinish | devananda: yes you need to have a config flag for new features that will skip the test if the feature isn't available | 22:56 |
| mtreinish | devananda: most of the projects also api extesion list opts which are used for specifying enabled extensions | 22:56 |
| *** markmcclain has quit IRC | 22:56 | |
| mtreinish | which can be used if the feature corresponds to an optional api | 22:56 |
| devananda | mtreinish: gotcha. we dont have any tempest tests yet for optional things | 22:58 |
| mtreinish | devananda: https://review.openstack.org/#/c/97834/3/README.rst | 22:58 |
| devananda | mtreinish: nice, thanks | 22:59 |
| Shrews | devananda: hrm, ok. making dinner now, but will WIP that change and look into doing that tomorrow | 23:00 |
| devananda | mtreinish: is there a way to trigger a test run of eg, stable/icehouse for a new tempest patch? | 23:00 |
| devananda | Shrews: we had landed support for preserve ephemeral in Icehouse but IIRC there were some bugs | 23:00 |
| mtreinish | devananda: tempest runs against stable icehouse on every patch already | 23:01 |
| devananda | Shrews: thanks. enjoy dinner! | 23:01 |
| mtreinish | to ensure we don't break things | 23:01 |
| mtreinish | I'm not sure about stable icehouse with ironic though | 23:01 |
| devananda | mtreinish: right, but ironic is only enabled in the tempest-dsvm-virtual-ironic job | 23:01 |
| devananda | *the nova.virt.ironic driver is ony ... | 23:01 |
| devananda | which is also non-voting :( | 23:02 |
| devananda | even though it's stable/reliable and voting on us | 23:02 |
| devananda | (yay asymmetry!) | 23:02 |
| mtreinish | devananda: I see 3 non voting tempest jobs run on tempest patches | 23:02 |
| mtreinish | yeah that's the incubated vs integrated thing... | 23:03 |
| devananda | right | 23:03 |
| devananda | mtreinish: afaik, none of the ironic jobs run against tempest changes are checking stable/icehouse -- because if they were, they would have been failing before today | 23:03 |
| devananda | mtreinish: hmm.. i think my brain just found a bigger issue | 23:05 |
| devananda | thanks | 23:05 |
| mtreinish | devananda: so you can propose adding an icehouse ironic tempest job. Although that would still be nonvoting | 23:05 |
| mtreinish | devananda: sure np | 23:05 |
| *** mrodden has quit IRC | 23:07 | |
| *** sabeen1 has quit IRC | 23:11 | |
| *** mlavalle has quit IRC | 23:12 | |
| *** leseb has joined #openstack-qa | 23:13 | |
| *** gfidente has quit IRC | 23:14 | |
| *** leseb has quit IRC | 23:14 | |
| *** leseb has joined #openstack-qa | 23:15 | |
| *** dims_ has quit IRC | 23:17 | |
| *** jamielennox|away is now known as jamielennox | 23:19 | |
| *** leseb has quit IRC | 23:19 | |
| *** mordred has quit IRC | 23:24 | |
| *** mordred has joined #openstack-qa | 23:27 | |
| *** mordred has quit IRC | 23:30 | |
| *** mordred has joined #openstack-qa | 23:30 | |
| *** dpaterson has joined #openstack-qa | 23:44 | |
| *** aimon__ has quit IRC | 23:45 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/56812 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 23:47 |
| *** mmaglana has quit IRC | 23:50 | |
| *** kgriffs is now known as kgriffs|afk | 23:52 | |
| *** dims_ has joined #openstack-qa | 23:56 | |
| flwang | maurosr: ping | 23:56 |
| maurosr | flwang: hey | 23:57 |
| flwang | maurosr: just found replied your comments on my patch, https://review.openstack.org/#/c/99514/4/tempest/scenario/test_swift_basic_ops.py | 23:58 |
| * maurosr checking | 23:58 | |
| flwang | maurosr: would you mind revisiting it to check if my reply is reasonable? | 23:58 |
| flwang | maurosr: awesome | 23:58 |
| flwang | maurosr: I'd like to submit a new ps to address your concern but i'd like to confirm if i got your suggestion completely | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!