| *** rhsu1 has quit IRC | 00:06 | |
| *** mmaglana has quit IRC | 00:09 | |
| *** Aimon has quit IRC | 00:10 | |
| *** mmaglana has joined #openstack-qa | 00:10 | |
| *** mmaglana has quit IRC | 00:14 | |
| *** openstack has joined #openstack-qa | 00:19 | |
| *** matsuhashi has joined #openstack-qa | 00:26 | |
| *** salv-orlando has joined #openstack-qa | 00:37 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a common class for Nova v2/v3 API tests https://review.openstack.org/78137 | 00:38 |
|---|---|---|
| *** sthaha has joined #openstack-qa | 00:38 | |
| *** zhikunliu has joined #openstack-qa | 00:46 | |
| *** adalbas has quit IRC | 00:47 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/devstack: Set up cgroups for libvirt https://review.openstack.org/82630 | 00:49 |
| jogo | mriedem: if your there was hoping for a quick e-r review https://review.openstack.org/#/c/82245/ | 00:49 |
| mriedem | jogo: +A | 00:51 |
| jogo | mriedem: thanks | 00:52 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add fingerprint for bug 1295876 https://review.openstack.org/82245 | 00:53 |
| *** ken1ohmichi has joined #openstack-qa | 00:54 | |
| *** yamahata has joined #openstack-qa | 00:55 | |
| *** yamahata_ has quit IRC | 00:56 | |
| *** yamahata_ has joined #openstack-qa | 00:56 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify attributes through Nova os-quota-sets API https://review.openstack.org/82681 | 00:59 |
| *** weshay has quit IRC | 00:59 | |
| *** openstackrecheck has quit IRC | 00:59 | |
| *** openstackrecheck has joined #openstack-qa | 01:00 | |
| *** Aimon has joined #openstack-qa | 01:06 | |
| *** rhsu has joined #openstack-qa | 01:07 | |
| *** krtaylor has quit IRC | 01:07 | |
| *** bill_az__ has quit IRC | 01:09 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/81726 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 01:11 |
| *** krtaylor has joined #openstack-qa | 01:11 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a common class for Nova v2/v3 API tests https://review.openstack.org/78137 | 01:15 |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Check reserve/unreserve fixed-ips APIs attributes https://review.openstack.org/82684 | 01:18 |
| *** mlavalle has quit IRC | 01:18 | |
| *** krtaylor has quit IRC | 01:20 | |
| *** mwagner_lap has joined #openstack-qa | 01:23 | |
| *** krtaylor has joined #openstack-qa | 01:23 | |
| *** rwsu has quit IRC | 01:26 | |
| *** nosnos has joined #openstack-qa | 01:29 | |
| *** matsuhashi has quit IRC | 01:30 | |
| *** matsuhashi has joined #openstack-qa | 01:30 | |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Add unit tests for NegativeRestClient class https://review.openstack.org/80876 | 01:31 |
| *** rhsu has quit IRC | 01:39 | |
| *** rhsu1 has joined #openstack-qa | 01:39 | |
| *** jasondotstar has joined #openstack-qa | 01:41 | |
| *** rwsu has joined #openstack-qa | 01:42 | |
| *** yamahata_ has quit IRC | 01:48 | |
| *** yamahata_ has joined #openstack-qa | 01:50 | |
| *** sthaha has quit IRC | 01:50 | |
| *** malini_afk is now known as malini | 01:54 | |
| *** sthaha has joined #openstack-qa | 02:03 | |
| openstackgerrit | Brant Knudson proposed a change to openstack-dev/devstack: Provide a keystone.conf for tempest https://review.openstack.org/82691 | 02:06 |
| openstackgerrit | Brant Knudson proposed a change to openstack-dev/devstack: Provide a keystone.conf for tempest https://review.openstack.org/82358 | 02:06 |
| openstackgerrit | Brant Knudson proposed a change to openstack-dev/devstack: Provide a keystone.conf for tempest https://review.openstack.org/82358 | 02:07 |
| *** HenryG has joined #openstack-qa | 02:29 | |
| *** jasondotstar has quit IRC | 02:30 | |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Add unit tests for NegativeRestClient class https://review.openstack.org/80876 | 02:30 |
| *** mmaglana has joined #openstack-qa | 02:31 | |
| *** mestery has joined #openstack-qa | 02:34 | |
| *** Sukhdev has joined #openstack-qa | 02:40 | |
| *** rfolco has quit IRC | 02:41 | |
| *** mestery has quit IRC | 02:46 | |
| *** mestery has joined #openstack-qa | 02:46 | |
| *** mriedem has quit IRC | 02:48 | |
| *** salv-orlando_ has joined #openstack-qa | 02:49 | |
| *** salv-orlando has quit IRC | 02:50 | |
| *** salv-orlando_ is now known as salv-orlando | 02:50 | |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Add unit tests for cli.output_parser https://review.openstack.org/82700 | 02:53 |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Add a new exception for invalid structure https://review.openstack.org/80927 | 02:53 |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a common class for Nova v2/v3 API tests https://review.openstack.org/78137 | 02:54 |
| *** malini is now known as malini_afk | 03:00 | |
| *** torandu has joined #openstack-qa | 03:08 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify attributes through Nova "list security groups" API https://review.openstack.org/81452 | 03:08 |
| *** matsuhashi has quit IRC | 03:12 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Honor suspend/pause config switches in scenario tests https://review.openstack.org/82566 | 03:13 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-infra/devstack-gate: Sets DEFAULT_INSTANCE_TYPE in localrc for Ironic https://review.openstack.org/82703 | 03:15 |
| *** sweston has joined #openstack-qa | 03:21 | |
| *** torandu has quit IRC | 03:27 | |
| *** torandu has joined #openstack-qa | 03:28 | |
| *** nosnos has quit IRC | 03:29 | |
| *** matsuhashi has joined #openstack-qa | 03:30 | |
| *** chandankumar_ has joined #openstack-qa | 03:32 | |
| *** matsuhashi has quit IRC | 03:36 | |
| *** chandankumar_ has quit IRC | 03:40 | |
| *** mspreitz has joined #openstack-qa | 03:40 | |
| *** Sukhdev has quit IRC | 03:43 | |
| *** mmaglana has quit IRC | 03:47 | |
| *** harlowja_ is now known as harlowja_away | 03:49 | |
| *** GMann has quit IRC | 03:50 | |
| *** torandu has quit IRC | 03:54 | |
| *** chandankumar_ has joined #openstack-qa | 03:55 | |
| *** torandu has joined #openstack-qa | 03:56 | |
| *** GMann has joined #openstack-qa | 03:57 | |
| raies | GMann: Hi | 04:01 |
| raies | GMann: previously we need to add cp /opt/stack/tempest/etc/tempest.conf etc/tempest.conf | 04:02 |
| *** Akshik has joined #openstack-qa | 04:03 | |
| raies | GMann: But now now there occurs some identity error because different username/password is changed to None | 04:03 |
| GMann | raies- hi | 04:03 |
| GMann | yes | 04:03 |
| raies | Now to run tempest separately what extra settings are required | 04:04 |
| GMann | Tempest account has been separated from Devstack now | 04:04 |
| raies | ?? | 04:04 |
| *** torandu has quit IRC | 04:04 | |
| GMann | what you need is- pull the latest master devstack | 04:04 |
| GMann | and run the ./stack.sh | 04:04 |
| GMann | it will set the devstack credentials in tempest. | 04:05 |
| raies | ok, I think I am using old devstack | 04:05 |
| GMann | Basically idea is now tempest accounts are seperated and it can be configured with any enviornment other than devstack | 04:05 |
| raies | thanks GMann san :) | 04:05 |
| Akshik | Are there any Citrix CPBM like tool for OpenStack | 04:08 |
| *** torandu has joined #openstack-qa | 04:11 | |
| *** nati_ueno has quit IRC | 04:13 | |
| openstackrecheck | Console logs not available after 13:22s for gate-python-novaclient-pypy 82443,2,874e901 | 04:15 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/80868 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1286818 | 04:16 |
| openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify list agents attributes of V2/V3 APIs https://review.openstack.org/80923 | 04:17 |
| openstackgerrit | Mauro Rodrigues proposed a change to openstack/tempest: Refactors cleanup resource for scenario https://review.openstack.org/82709 | 04:17 |
| *** torandu has quit IRC | 04:17 | |
| *** torandu has joined #openstack-qa | 04:18 | |
| *** matsuhashi has joined #openstack-qa | 04:23 | |
| *** torandu has quit IRC | 04:23 | |
| *** chandankumar_ has quit IRC | 04:23 | |
| *** torandu has joined #openstack-qa | 04:25 | |
| *** nosnos has joined #openstack-qa | 04:25 | |
| *** Sukhdev has joined #openstack-qa | 04:32 | |
| *** torandu has quit IRC | 04:35 | |
| *** torandu has joined #openstack-qa | 04:36 | |
| *** nati_ueno has joined #openstack-qa | 04:40 | |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Add a test for booting instances at the specified AZs or hosts https://review.openstack.org/66882 | 04:41 |
| *** torandu has quit IRC | 04:46 | |
| openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'get_aggregate' https://review.openstack.org/82712 | 04:48 |
| openstackgerrit | Mauro Rodrigues proposed a change to openstack/tempest: Refactors cleanup resource for scenario https://review.openstack.org/82709 | 04:54 |
| *** rhsu has joined #openstack-qa | 04:54 | |
| *** rhsu1 has quit IRC | 04:54 | |
| marun | sdague: ping | 04:54 |
| marun | sdague: you're -1ing the wrong patch | 04:54 |
| marun | sdague: oops, you +1'd (confused is me) | 04:55 |
| *** yamahata has quit IRC | 04:58 | |
| *** yamahata has joined #openstack-qa | 04:58 | |
| *** yamahata has quit IRC | 04:59 | |
| *** yamahata has joined #openstack-qa | 04:59 | |
| *** masayukig has joined #openstack-qa | 05:04 | |
| *** yfried has quit IRC | 05:10 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify get flavor attributes of Nova V2/V3 APIs https://review.openstack.org/80543 | 05:12 |
| *** matsuhashi has quit IRC | 05:14 | |
| *** matsuhashi has joined #openstack-qa | 05:18 | |
| *** Madan has joined #openstack-qa | 05:22 | |
| openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify certificate API response attributes https://review.openstack.org/81477 | 05:23 |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Validate for list flavor attributes of V2/V3 APIs https://review.openstack.org/78527 | 05:23 |
| marun | clarkb: around? | 05:26 |
| clarkb | marun: sort of | 05:27 |
| clarkb | mostly just watching IRC, whats up? | 05:27 |
| marun | clarkb: I'm contemplating what the neutron functional job should be gating... | 05:27 |
| marun | clarkb: It has a dependency on both devstack and tempest. | 05:28 |
| marun | clarkb: oh wait, just devstack | 05:28 |
| marun | clarkb: to install dependencies. | 05:28 |
| marun | clarkb: so should the job run as part of the devstack gate? | 05:28 |
| marun | clarkb: feel free to divert me for now | 05:28 |
| clarkb | ideally it would, I'm not sure how sdague would feel about it considering it depends on unstable tempest interfaces | 05:29 |
| marun | clarkb: it doesn't | 05:29 |
| clarkb | oh in that case yes definitely | 05:29 |
| marun | clarkb: I commented as much on the review - that's a separate job. | 05:29 |
| marun | clarkb: ok, danke! | 05:29 |
| clarkb | marun: I would double check to see what the swift functional test does | 05:29 |
| clarkb | as that may offer clues to other things that should be done | 05:29 |
| marun | clarkb: ah, the swift tests also rely on devstack to deploy? | 05:30 |
| clarkb | yup | 05:30 |
| marun | ah, gotcha | 05:30 |
| marun | clarkb: I hope to talk at length at summit about how all this stuff works. | 05:32 |
| marun | clarkb: I was talking to someone about how google manages source/ci and I wonder if there isn't room to reduce the amount of duplicative testing in the future. | 05:33 |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify detail_list flavor attributes of V2/V3 APIs https://review.openstack.org/80548 | 05:33 |
| marun | clarkb: but for another time... | 05:33 |
| clarkb | marun: there should be tons of room | 05:33 |
| clarkb | marun: I have an etherpad up but I don't have the link to it on this machine | 05:33 |
| marun | clarkb: in neutron alone, we probably do 10x more testing than we need to. | 05:33 |
| clarkb | and the tl;dr of it is how do we consolidate and be smarter about this | 05:33 |
| clarkb | marun: the biggest issue is we don't have any feature parity anywhere | 05:34 |
| marun | clarkb: if a plugin isn't in the gate, changes to its tree shouldn't trigger 10 tempest runs against the default plugin | 05:34 |
| clarkb | so we have to run a bajillion tests to cover a bunch of corner cases | 05:34 |
| clarkb | neutron vs nova network, kvm vs xen vs containers (andor docker), mysql vs postgres (and mongodb in cases), and so on | 05:35 |
| clarkb | its pretty terrible | 05:35 |
| marun | clarkb: well, we could run them against the code under test 10x. running them for changes that don't affect the gate wouldn't seem to have any value. | 05:35 |
| marun | clarkb: for sure | 05:35 |
| clarkb | marun: the problem with ^ is you ahve no idea what will affect the gate until its too late | 05:35 |
| marun | clarkb: if I touch a tree that doesn't actually execute in the gate, then it's pointless. | 05:35 |
| clarkb | marun: but you don't know when devstack will make a change and completely make the previous assumption wrong | 05:36 |
| marun | clarkb: running the tests more for changes that do affect execution would be a better use of duplicative testing | 05:36 |
| marun | clarkb: we do for neutron | 05:36 |
| clarkb | (this is anotehr problem that needs solving and I think it can be done wif we have some clear cut rules about what we test) | 05:36 |
| marun | clarkb: we can pretty easily demarcate when a change to a plugin has zero effect on a given jenkins job. | 05:36 |
| clarkb | marun: you can't | 05:36 |
| marun | clarkb: yes, you can :) | 05:36 |
| clarkb | because devstack could make your claims false | 05:36 |
| clarkb | without any of your input | 05:36 |
| marun | clarkb: look, sometimes a change will be ambiguous. | 05:37 |
| clarkb | so you may think the opendaylight driver isn't needed in the gate until it is | 05:37 |
| marun | clarkb: I'm talking about unambiguous changes. | 05:37 |
| clarkb | and so on | 05:37 |
| marun | clarkb: it should be possible to detect what changes, compare against devstack config, and abort jobs that are unnecessary | 05:37 |
| marun | clarkb: -> low hanging robot work | 05:37 |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Fliter images with container and disk format https://review.openstack.org/82281 | 05:38 |
| clarkb | marun: right, or we can just come up with "This is what we test" and codify it | 05:38 |
| clarkb | then we don't even need the robots. | 05:38 |
| marun | clarkb: codifying is lazy, and doesn't scale | 05:38 |
| clarkb | marun: it would scale just fine | 05:38 |
| marun | we can do better | 05:38 |
| marun | :) | 05:38 |
| marun | (but we don't have to) | 05:38 |
| clarkb | declare X Y and Z as the things (whatever they are, I actually don't care about details much) then do that | 05:39 |
| clarkb | then if someon breaks the gate because devstack decides to use some new thing thats a bug | 05:39 |
| clarkb | the biggest problem here is we really have no idea what all of the various variables mean | 05:39 |
| marun | clarkb: the classification would be project-specific. we know how to configure neutron, better than devstack. | 05:40 |
| marun | clarkb: we can determine when a change absolutely wouldn't be executed based on a given devstack config. | 05:40 |
| marun | clarkb: if it *could* impact execution, then no question, we run the job. | 05:40 |
| clarkb | marun: right so first step is figuring out what all of the various devstack knobs mean in regards to all the projects | 05:41 |
| clarkb | and figuring out where the overlap between config and tests lies | 05:41 |
| clarkb | this is much much easier if we stop building building into a corner with incompatible interfaces | 05:41 |
| marun | clarkb: right | 05:41 |
| marun | clarkb: cross-project coordination? *gasp* | 05:41 |
| marun | clarkb: so, when is the TC going to have budget to hire some folk to lead this? ;) | 05:42 |
| clarkb | ha, I think sdague intends on pushing this | 05:42 |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Fliter images with container and disk format https://review.openstack.org/82281 | 05:42 |
| marun | I'd definitely get behind that. | 05:42 |
| * clarkb is looking for the etherpad in irc logs and failing | 05:43 | |
| clarkb | marun: https://etherpad.openstack.org/p/juno-test-maxtrices yay for the power of google | 05:44 |
| marun | clarkb: I'm assuming the neutron functional job be enabled as gate for devstack and neutron first for a while before being allowed to vote? | 05:45 |
| marun | clarkb: looking | 05:45 |
| clarkb | marun: if it is non voting it should be in the check queue then we can gate and vote when it is ready | 05:45 |
| marun | clarkb: ok, so add it to devstack's check queue non-voting is the next step? | 05:46 |
| clarkb | marun: yup | 05:46 |
| marun | clarkb: ah, so we can't move to matrix testing until we have parity :/ | 05:47 |
| marun | oy | 05:47 |
| clarkb | marun: exactly | 05:47 |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify flavor extra specs attributes of Nova APIs https://review.openstack.org/82718 | 05:47 |
| *** nati_ueno has quit IRC | 05:47 | |
| marun | clarkb: I don't think that precludes us pursuing a strategy of not running tempest gate jobs that would not be affected by a given patch. | 05:47 |
| clarkb | marun: ya there is probably some work taht can be done in parallel in that space | 05:48 |
| clarkb | I just think it becomes much simpler if we can make feature parity a thing | 05:48 |
| clarkb | and it may not be complete parity but maybe a this covers 80% of stuff for us list | 05:48 |
| *** chandan_kumar has quit IRC | 05:48 | |
| marun | clarkb: I agree that parity is the first priority. | 05:49 |
| marun | (or should be) | 05:49 |
| openstackrecheck | openstack/python-keystoneclient change: https://review.openstack.org/77026 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 05:50 |
| *** nati_ueno has joined #openstack-qa | 05:51 | |
| clarkb | its definitely a "fun" thing to think about. Ideas welcome. I put that in an etherpad so that others could chime in | 05:54 |
| marun | clarkb: cool :) | 05:54 |
| *** otherwiseguy has quit IRC | 05:59 | |
| *** Sukhdev has quit IRC | 06:04 | |
| openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify tenant usages API response attributes https://review.openstack.org/81766 | 06:06 |
| *** mmaglana has joined #openstack-qa | 06:07 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/81044 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1295876 | 06:12 |
| *** andreaf has joined #openstack-qa | 06:14 | |
| *** psharma has joined #openstack-qa | 06:17 | |
| *** mestery has quit IRC | 06:18 | |
| *** mspreitz has quit IRC | 06:26 | |
| *** yfried has joined #openstack-qa | 06:31 | |
| *** rhsu has quit IRC | 06:34 | |
| *** mmaglana has quit IRC | 06:38 | |
| *** nmagnezi has joined #openstack-qa | 06:50 | |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds more verification in list alarms https://review.openstack.org/79942 | 06:58 |
| openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify certificate API response attributes https://review.openstack.org/81477 | 07:01 |
| openstackrecheck | Console logs not available after 13:24s for check-tempest-dsvm-full 82515,3,88d5ace | 07:04 |
| *** jlibosva has joined #openstack-qa | 07:06 | |
| *** andreaf has quit IRC | 07:08 | |
| *** Madan has quit IRC | 07:10 | |
| *** Madan has joined #openstack-qa | 07:10 | |
| openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'get_aggregate' https://review.openstack.org/82712 | 07:15 |
| *** devvesa has joined #openstack-qa | 07:16 | |
| openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify "get version" API response attributes https://review.openstack.org/80936 | 07:16 |
| *** devvesa has left #openstack-qa | 07:16 | |
| *** yrabl has joined #openstack-qa | 07:28 | |
| *** saju_m has joined #openstack-qa | 07:30 | |
| *** tkammer has joined #openstack-qa | 07:33 | |
| *** nati_ueno has quit IRC | 07:37 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify list extensions attributes of V2/V3 APIs https://review.openstack.org/82728 | 07:37 |
| *** luqas has joined #openstack-qa | 07:45 | |
| *** dkorn has joined #openstack-qa | 07:46 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a common class for Nova v2/v3 API tests https://review.openstack.org/78137 | 08:00 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: stop leaking in network tests https://review.openstack.org/82731 | 08:02 |
| openstackrecheck | Console logs not available after 13:26s for check-tempest-dsvm-neutron 82721,1,cd3cbd8 | 08:06 |
| *** dkorn has quit IRC | 08:14 | |
| openstackrecheck | Console logs not available after 13:25s for check-grenade-dsvm 82435,2,2d7acf6 | 08:19 |
| *** dkorn has joined #openstack-qa | 08:25 | |
| *** jasondotstar has joined #openstack-qa | 08:27 | |
| openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'get_aggregate' https://review.openstack.org/82712 | 08:30 |
| openstackrecheck | Console logs not available after 13:38s for gate-neutron-docs 82352,3,670d27f | 08:33 |
| openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify tenant usages API response attributes https://review.openstack.org/81766 | 08:41 |
| *** giulivo has joined #openstack-qa | 08:46 | |
| openstackrecheck | Console logs not available after 13:47s for gate-nova-pep8 81692,4,bc5b417 | 08:49 |
| *** matsuhashi has quit IRC | 08:49 | |
| *** saju_m has quit IRC | 08:50 | |
| *** yassine has joined #openstack-qa | 08:52 | |
| *** saju_m has joined #openstack-qa | 08:52 | |
| *** andreaf has joined #openstack-qa | 08:53 | |
| *** matsuhashi has joined #openstack-qa | 08:56 | |
| *** jordanP has joined #openstack-qa | 08:57 | |
| *** jpich has joined #openstack-qa | 09:02 | |
| *** nmagnezi has quit IRC | 09:02 | |
| *** afazekas has joined #openstack-qa | 09:04 | |
| *** yfried has quit IRC | 09:04 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/81044 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1295876 | 09:08 |
| *** chandan_kumar has joined #openstack-qa | 09:13 | |
| *** matsuhashi has quit IRC | 09:14 | |
| openstackgerrit | Nadya Privalova proposed a change to openstack-dev/devstack: Add configuration for Ceilometer middleware to swift https://review.openstack.org/82131 | 09:14 |
| *** safchain has joined #openstack-qa | 09:15 | |
| *** matsuhashi has joined #openstack-qa | 09:18 | |
| *** nmagnezi has joined #openstack-qa | 09:18 | |
| openstackgerrit | Nadya Privalova proposed a change to openstack-dev/devstack: Add configuration for Ceilometer middleware to swift https://review.openstack.org/82131 | 09:19 |
| *** mkoderer has joined #openstack-qa | 09:21 | |
| *** yfried has joined #openstack-qa | 09:21 | |
| *** luqas has quit IRC | 09:25 | |
| *** GMann has quit IRC | 09:27 | |
| *** dkorn has quit IRC | 09:27 | |
| *** luqas has joined #openstack-qa | 09:29 | |
| openstackgerrit | Julien Leloup proposed a change to openstack/tempest: Stress Test for server availability through SSH https://review.openstack.org/74067 | 09:38 |
| *** saju_m has quit IRC | 09:38 | |
| *** nosnos has quit IRC | 09:39 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Use auth data to fill credentials https://review.openstack.org/82111 | 09:42 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Define V3 Credentials https://review.openstack.org/81872 | 09:42 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Get credential IDs from Credentials class https://review.openstack.org/80246 | 09:43 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Enforces the use of Credentials (part1) https://review.openstack.org/77594 | 09:43 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Enforces the use of Credentials (part2) https://review.openstack.org/82112 | 09:43 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Access credential fields as attributes https://review.openstack.org/82113 | 09:43 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Defines a Credentials class https://review.openstack.org/74387 | 09:43 |
| andreaf | afazekas, giulivo: do you have any time for have a look at those ^^^ | 09:44 |
| andreaf | I rebased them a number of times now, I'd really love get them through and move on with the multi-auth bp | 09:45 |
| *** andreaf has quit IRC | 09:49 | |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Use urandom for heat key generation https://review.openstack.org/82758 | 09:51 |
| *** saju_m has joined #openstack-qa | 09:57 | |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack-dev/devstack: Add workaround for iptables on Fedora https://review.openstack.org/82760 | 09:59 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Move from keystoneclient to openstackclient in eucarc https://review.openstack.org/80475 | 10:00 |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack-dev/devstack: Explicitly open tftpd port in iptables on Fedora https://review.openstack.org/82760 | 10:01 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Network fwaas API test https://review.openstack.org/78857 | 10:01 |
| *** morganfainberg is now known as morganfainberg_Z | 10:05 | |
| *** zhikunliu has quit IRC | 10:05 | |
| *** matsuhashi has quit IRC | 10:08 | |
| *** matsuhashi has joined #openstack-qa | 10:10 | |
| *** yfried has quit IRC | 10:10 | |
| *** tosky has joined #openstack-qa | 10:10 | |
| *** salv-orlando has quit IRC | 10:14 | |
| openstackgerrit | Julien Leloup proposed a change to openstack/tempest: Stress Test for server availability through SSH https://review.openstack.org/74067 | 10:21 |
| *** yfried has joined #openstack-qa | 10:24 | |
| *** masayukig has quit IRC | 10:28 | |
| akamyshnikova | afazekas, sdague Hello, last week this change https://review.openstack.org/71251 was approved, but it based on https://review.openstack.org/63723 and https://review.openstack.org/80928. Could you find some time to take a look at this two changes? | 10:33 |
| *** markmcclain has joined #openstack-qa | 10:39 | |
| *** nmagnezi has quit IRC | 10:40 | |
| mkoderer | akamyshnikova: I will have a look today | 10:40 |
| akamyshnikova | mkoderer, thanks! | 10:40 |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Use OVERRIDE_ZUUL_BRANCH in devstack-vm-gate.sh https://review.openstack.org/82496 | 10:43 |
| *** weshay has joined #openstack-qa | 10:47 | |
| *** jasondotstar has quit IRC | 10:47 | |
| *** andreaf has joined #openstack-qa | 10:47 | |
| openstackgerrit | A change was merged to openstack/tempest: Remove resize-revert workaround for bug 924371 https://review.openstack.org/82095 | 10:52 |
| *** nmagnezi has joined #openstack-qa | 10:57 | |
| *** saju_m has quit IRC | 11:07 | |
| *** saju_m has joined #openstack-qa | 11:08 | |
| *** saju_m has quit IRC | 11:09 | |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds more testcases to test_telemetry_alarming_api https://review.openstack.org/82491 | 11:09 |
| *** mwagner_lap has quit IRC | 11:12 | |
| *** ken1ohmichi has quit IRC | 11:14 | |
| *** saju_m has joined #openstack-qa | 11:15 | |
| *** malini_afk is now known as malini | 11:17 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/tempest: Verify more information in floating ip tests https://review.openstack.org/63723 | 11:20 |
| *** Madan has quit IRC | 11:23 | |
| *** Madan has joined #openstack-qa | 11:30 | |
| afazekas | sdague,mkoderer :https://review.openstack.org/#/c/78345/ | 11:37 |
| *** saju_m has quit IRC | 11:38 | |
| *** saju_m has joined #openstack-qa | 11:38 | |
| sdague | afazekas: so I think this is correct - https://review.openstack.org/#/c/82758/1 but I'm curious where you saw the delays, as I expected to see speed ups in the heat slow jobs with this | 11:44 |
| sdague | which I don't see as something obvious | 11:44 |
| Anju1 | sdague: hii, please look in this https://bugs.launchpad.net/tempest/+bug/1297167 | 11:44 |
| sdague | Anju1: sure, what should I be looking for | 11:45 |
| Anju1 | sdague: I have mentioned all the things there :) | 11:46 |
| Anju1 | please tell me if I am going wrong | 11:46 |
| Anju1 | will join you from home :) bye | 11:46 |
| sdague | sounds reasonable to me | 11:46 |
| Anju1 | ohh thanks :) | 11:46 |
| sdague | so you propose a test delete? | 11:46 |
| cyeoh | would tempest cores mind having a look at: https://review.openstack.org/#/c/81551/ | 12:02 |
| cyeoh | the corresponding nova change has been +2'd https://review.openstack.org/#/c/81356/ | 12:03 |
| cyeoh | (though it needs the tempest patch to merge before it can pass jenkins) | 12:03 |
| *** harlowja_away has quit IRC | 12:03 | |
| cyeoh | this is the set of patches which is required to revert the backwards incompatible change made to flavor manage create | 12:04 |
| *** Akshik has quit IRC | 12:05 | |
| *** Atti_ has joined #openstack-qa | 12:05 | |
| *** Akshik has joined #openstack-qa | 12:05 | |
| *** Akshik has quit IRC | 12:05 | |
| Atti_ | Hi! Guys is it possible to set in a test method CONF option? My not working code example http://pastebin.com/QK9puEAT | 12:07 |
| *** malini is now known as malini_afk | 12:09 | |
| afazekas | sdague: I saw it my VM at devstack setup time | 12:14 |
| sdague | ok | 12:16 |
| afazekas | sdague: The vm has the http://en.wikipedia.org/wiki/RdRand instuction, but does not have hw random device. the Linux just uses this instruction partially | 12:16 |
| sdague | cyeoh: +A | 12:16 |
| *** matsuhashi has quit IRC | 12:16 | |
| cyeoh | sdague: thx! | 12:17 |
| *** dkranz has joined #openstack-qa | 12:17 | |
| *** matsuhashi has joined #openstack-qa | 12:23 | |
| *** markmcclain has quit IRC | 12:24 | |
| *** sthaha has quit IRC | 12:24 | |
| *** markmcclain has joined #openstack-qa | 12:24 | |
| *** sthaha has joined #openstack-qa | 12:25 | |
| *** markmcclain1 has joined #openstack-qa | 12:25 | |
| *** sthaha has quit IRC | 12:27 | |
| *** yamahata has quit IRC | 12:27 | |
| *** sthaha has joined #openstack-qa | 12:27 | |
| *** sthaha has joined #openstack-qa | 12:27 | |
| *** markmcclain2 has joined #openstack-qa | 12:27 | |
| *** markmcclain2 has quit IRC | 12:27 | |
| *** dkranz has quit IRC | 12:28 | |
| *** markmcclain2 has joined #openstack-qa | 12:28 | |
| *** dkranz has joined #openstack-qa | 12:29 | |
| *** markmcclain has quit IRC | 12:29 | |
| *** markmcclain1 has quit IRC | 12:29 | |
| *** markmcclain2 has quit IRC | 12:30 | |
| *** aysyd has joined #openstack-qa | 12:31 | |
| *** markmcclain has joined #openstack-qa | 12:31 | |
| afazekas | sdague: https://review.openstack.org/#/c/81486/3 The non uec image is problematic as I said yesterday. The ec2 image registration test expects uec images, and the uec images can boot faster because sever bios steps are skipped. | 12:32 |
| sdague | afazekas: I get that the EC2 tests will need a uec image | 12:33 |
| sdague | how much time do we lose in booting real images? | 12:33 |
| sdague | because the fact that we don't in the gate I find a little troubling | 12:33 |
| *** rfolco has joined #openstack-qa | 12:34 | |
| afazekas | sdague: sever second, the heat tests using real images anyway | 12:35 |
| sdague | I still think images should be the devstack default, if we want to override it in the gate, that's fine | 12:36 |
| afazekas | The ami image is raw image | 12:38 |
| *** jordanP has quit IRC | 12:38 | |
| *** zul has quit IRC | 12:38 | |
| *** jordanP has joined #openstack-qa | 12:40 | |
| *** ken1ohmichi has joined #openstack-qa | 12:40 | |
| *** zul has joined #openstack-qa | 12:41 | |
| *** saju_m has quit IRC | 12:41 | |
| *** mkollaro has joined #openstack-qa | 12:44 | |
| *** psharma has quit IRC | 12:45 | |
| *** mwagner_lap has joined #openstack-qa | 12:46 | |
| *** tosky has quit IRC | 12:47 | |
| *** adalbas has joined #openstack-qa | 12:48 | |
| *** mrodden has quit IRC | 12:52 | |
| *** mrodden has joined #openstack-qa | 12:52 | |
| *** dims_ has quit IRC | 12:52 | |
| *** luqas has quit IRC | 12:52 | |
| *** jasondotstar has joined #openstack-qa | 12:53 | |
| *** saju_m has joined #openstack-qa | 12:55 | |
| *** saju_m has quit IRC | 12:59 | |
| *** dkranz has quit IRC | 12:59 | |
| *** saju_m has joined #openstack-qa | 13:00 | |
| *** dims_ has joined #openstack-qa | 13:02 | |
| *** jhenner1 has joined #openstack-qa | 13:03 | |
| *** mkollaro has quit IRC | 13:03 | |
| *** mkollaro1 has joined #openstack-qa | 13:03 | |
| *** mkollaro1 is now known as mkollaro | 13:03 | |
| *** matsuhashi has quit IRC | 13:03 | |
| *** jhenner has quit IRC | 13:04 | |
| *** saju_m has quit IRC | 13:04 | |
| *** tkammer has quit IRC | 13:05 | |
| *** saju_m has joined #openstack-qa | 13:05 | |
| *** nmagnezi has quit IRC | 13:06 | |
| *** Atti_ has quit IRC | 13:07 | |
| *** saju_m has quit IRC | 13:09 | |
| *** saju_m has joined #openstack-qa | 13:10 | |
| *** nmagnezi has joined #openstack-qa | 13:19 | |
| *** dkranz has joined #openstack-qa | 13:21 | |
| *** ChanServ changes topic to "the gerrit event stream is currently hung, blocking all testing. troubleshooting is in progress (next update at 14:00 utc)" | 13:21 | |
| *** tkammer has joined #openstack-qa | 13:23 | |
| *** GMann has joined #openstack-qa | 13:24 | |
| openstackgerrit | Mauro Rodrigues proposed a change to openstack/tempest: Refactors cleanup resource for scenario https://review.openstack.org/82709 | 13:26 |
| *** Madan has quit IRC | 13:26 | |
| *** mriedem has joined #openstack-qa | 13:27 | |
| *** mestery has joined #openstack-qa | 13:28 | |
| *** mestery has quit IRC | 13:28 | |
| *** mestery has joined #openstack-qa | 13:29 | |
| *** ChanServ changes topic to "All Things OpenStack QA" | 13:30 | |
| *** markmcclain has quit IRC | 13:34 | |
| *** yamahata has joined #openstack-qa | 13:38 | |
| *** nati_ueno has joined #openstack-qa | 13:40 | |
| *** luqas has joined #openstack-qa | 13:42 | |
| Anju1 | sdague: hii, sry I did not answer at that time | 13:48 |
| Anju1 | sdague: I am proposing a patch to delete that test ..we do not need it | 13:49 |
| sdague | Anju1: yes, that sounds reasonabl | 13:49 |
| sdague | is the review up somewhere yet? | 13:50 |
| *** shwetaap has joined #openstack-qa | 13:50 | |
| Anju1 | today I found that :) and give that the bug to review it :) will do it as soon as possible | 13:51 |
| sdague | Anju1: great | 13:51 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Move pause_available class variable into skip decorators https://review.openstack.org/82535 | 13:58 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Move suspend_available class variable into skip decorators https://review.openstack.org/82534 | 13:58 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Disable other suspend/resume tests if not supported https://review.openstack.org/82410 | 13:58 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Move run_ssh class variable into skip decorators https://review.openstack.org/82537 | 13:58 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Honor suspend/pause config switches in scenario tests https://review.openstack.org/82566 | 13:58 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Move resize_available class variable into skip decorators https://review.openstack.org/82536 | 13:58 |
| mriedem | mtreinish: rebased to put armando's fix before all mine ^ | 13:58 |
| *** nati_ueno has quit IRC | 13:59 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: orchestration API refactor test_list_resources https://review.openstack.org/82828 | 14:02 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: orchestration API add coverage for stack update API https://review.openstack.org/82829 | 14:02 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: orchestration remove unused invalid_template_url https://review.openstack.org/82575 | 14:02 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: orchestration add API tests for list query arguments https://review.openstack.org/82577 | 14:02 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: orchestration add resource limit API test https://review.openstack.org/82576 | 14:02 |
| *** nmagnezi has quit IRC | 14:02 | |
| *** pafuent has joined #openstack-qa | 14:06 | |
| openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Sahara: added new api tests https://review.openstack.org/71592 | 14:06 |
| *** malini_afk is now known as malini | 14:06 | |
| pafuent | Hi. Can someone take a look at this patch https://review.openstack.org/#/c/81909/? | 14:07 |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Removed unnecessary test from account test https://review.openstack.org/82830 | 14:07 |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Fliter images with container and disk format https://review.openstack.org/82281 | 14:07 |
| mtreinish | mriedem: it looks like auto-rebase +2'd it for me | 14:08 |
| *** jhenner1 has quit IRC | 14:08 | |
| *** tosky has joined #openstack-qa | 14:08 | |
| mriedem | mtreinish: yeah, except https://review.openstack.org/#/c/82410/ but was just an fyi as to the big rebase | 14:08 |
| *** akamyshnikova_ has joined #openstack-qa | 14:09 | |
| sdague | mriedem: thank you for super easy to review patches :) | 14:12 |
| sdague | mtreinish: I do have an issue with armando's base patch though, because it does partial skips of a test | 14:12 |
| *** ken1ohmichi has quit IRC | 14:13 | |
| sdague | and I think we need to treat tests as attomic, so you can skip the whole thing if you like, but not parts | 14:13 |
| mtreinish | I don't even think I've looked at it yet | 14:13 |
| sdague | https://review.openstack.org/#/c/82566/4/tempest/scenario/test_server_basic_ops.py | 14:14 |
| sdague | other than that issue, I'm good with the whole series | 14:14 |
| mtreinish | yeah that's wrong | 14:14 |
| mriedem | sdague: i'll take that as a compliment | 14:15 |
| sdague | mriedem: it was meant as one | 14:16 |
| mriedem | i'm still reeling for low-level sarcasm in nova yesterday :) | 14:16 |
| sdague | heh | 14:16 |
| mriedem | *from | 14:16 |
| sdague | I'm getting very tired of reviews that have "Also... " in their commit message | 14:16 |
| sdague | make reviews easy to process, and they will get reviewed much more quickly | 14:17 |
| sdague | honestly, I think we could almost hacking enforce on if you start a paragraph with Also, fail the review | 14:18 |
| sdague | because they are always doing too much | 14:18 |
| mtreinish | sdague: I normally use "In addition" :) | 14:19 |
| *** jasondotstar has quit IRC | 14:21 | |
| *** jhenner has joined #openstack-qa | 14:21 | |
| *** sthaha has quit IRC | 14:22 | |
| *** mestery has quit IRC | 14:24 | |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Removed unnecessary test from account test https://review.openstack.org/82835 | 14:24 |
| *** mkollaro has quit IRC | 14:26 | |
| *** nmagnezi has joined #openstack-qa | 14:29 | |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Removed unnecessary test from account test https://review.openstack.org/82835 | 14:30 |
| *** markmcclain has joined #openstack-qa | 14:32 | |
| *** markmcclain has quit IRC | 14:32 | |
| *** mestery has joined #openstack-qa | 14:33 | |
| *** tkammer has quit IRC | 14:33 | |
| *** markmcclain has joined #openstack-qa | 14:33 | |
| openstackgerrit | Alexander Bochkarev proposed a change to openstack/tempest: Add compute pollsters tests for ceilometer https://review.openstack.org/80965 | 14:35 |
| *** otherwiseguy has joined #openstack-qa | 14:37 | |
| *** mkollaro has joined #openstack-qa | 14:41 | |
| *** tkammer has joined #openstack-qa | 14:46 | |
| *** jasondotstar has joined #openstack-qa | 14:46 | |
| *** kgriffs_afk is now known as kgriffs | 14:59 | |
| *** luqas has quit IRC | 15:03 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/tempest: Verify more information in floating ip tests https://review.openstack.org/63723 | 15:05 |
| andreaf | mtreinish: ping | 15:07 |
| *** adalbas has quit IRC | 15:08 | |
| jaypipes | mtreinish, sdague: core love on https://review.openstack.org/#/c/79942. Looks good to go IMO. | 15:08 |
| *** cmart has joined #openstack-qa | 15:09 | |
| sdague | jaypipes: +A | 15:10 |
| jaypipes | cheerio | 15:10 |
| sdague | jogo: is https://review.openstack.org/#/c/82630/ something you are comfortable works around the libvirt issue? | 15:11 |
| *** saju_m has quit IRC | 15:13 | |
| mtreinish | andreaf: what's up | 15:14 |
| mkoderer | sdague: mtreinish: could we get this approved https://review.openstack.org/#/c/80926/, I would really like to get rid of the dirty hack | 15:16 |
| andreaf | mtreinish: about the config fixture again - if I want to change some config in a test, is it right to do cfg.CONF.set_default() ? | 15:17 |
| andreaf | mtreinish: in the test itself, that is | 15:18 |
| openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Sahara: added tests for job binaries https://review.openstack.org/74668 | 15:18 |
| andreaf | mtreinish: the fixture cleanup shall reset the whole config anyways, so I do not need any additional cleanup I think | 15:19 |
| mtreinish | andreaf: yeah that's the right way | 15:19 |
| mtreinish | the fixture cleanup runs reset() so it will be undone after the test | 15:19 |
| andreaf | mtreinish: ok thanks | 15:20 |
| mtreinish | mkoderer: +A | 15:20 |
| mkoderer | mtreinish: thanks! | 15:20 |
| *** mestery has quit IRC | 15:24 | |
| openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Sahara: added new api tests https://review.openstack.org/71592 | 15:28 |
| openstackgerrit | Longgeek proposed a change to openstack-dev/devstack: Increase in the full path to mkfs.xfs(/sbin/mkfs.xfs). https://review.openstack.org/82459 | 15:28 |
| *** GMann has quit IRC | 15:29 | |
| openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Sahara: added tests for job binaries https://review.openstack.org/74668 | 15:30 |
| *** otherwiseguy has quit IRC | 15:32 | |
| openstackgerrit | A change was merged to openstack/tempest: Renew token before expiry time https://review.openstack.org/82046 | 15:33 |
| andreaf | sdague, mkoderer, afazekas, dkranz: lock support for unit test by mtreinish: https://review.openstack.org/#/c/81896/ could you have a look? | 15:41 |
| *** krtaylor has quit IRC | 15:42 | |
| *** akamyshnikova_ has quit IRC | 15:43 | |
| sdague | mtreinish: is there any cleanup path? | 15:43 |
| sdague | honestly, lacking a cleanup path I'd rather make it just TMPDIR | 15:43 |
| *** rossella_ has quit IRC | 15:44 | |
| mtreinish | so it's an external dir being set it's outside of a test cleanup. I tried using a tmpdir at first but the way fake config is used there is a global state issue | 15:44 |
| mtreinish | the tmpdir would get recreated for each test | 15:44 |
| mtreinish | which doesn't work for locking | 15:44 |
| *** rossella_ has joined #openstack-qa | 15:45 | |
| mtreinish | sdague: the locks should clean themselves up after they're used | 15:45 |
| mtreinish | oh nm you meant a bash tmpdir in the testr conf | 15:46 |
| mtreinish | sdague: yeah that should workj | 15:46 |
| sdague | mtreinish: no I'm concerned about the tempest dir itself | 15:46 |
| sdague | if it's $TMPDIR as a default | 15:47 |
| sdague | that will exist | 15:47 |
| sdague | the only reason we don't use that as a default in oslo is because of possible attacks via tempdir | 15:47 |
| sdague | but I don't think that applies here | 15:47 |
| mtreinish | yeah I don't think there's a sec issue for tempest unit tests | 15:48 |
| andreaf | sdague, mtreinish: so bottom line we could just do os.getenv('TMPDIR') ? | 15:50 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config fixture support to unit tests https://review.openstack.org/81896 | 15:50 |
| *** krtaylor has joined #openstack-qa | 15:50 | |
| sdague | andreaf: well - probably ${TMPDIR:-/tmp} | 15:50 |
| sdague | because TMPDIR is not required to be defined | 15:50 |
| openstackrecheck | Console logs not available after 13:47s for gate-tempest-pep8 82535,3,c53428e | 15:50 |
| *** luqas has joined #openstack-qa | 15:51 | |
| sdague | mtreinish: I think it needs to tweak once more, put in an proposed line | 15:51 |
| mtreinish | ok, you can nest it like that? | 15:51 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/81551 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1262906 | 15:51 |
| sdague | I'm pretty sure | 15:52 |
| sdague | dtroyer might tell me otherwise | 15:52 |
| mtreinish | ok I'll give it a shot | 15:53 |
| mtreinish | should it default to /tmp or /tmp/tempest-locks like I had originally? | 15:53 |
| andreaf | it looks like it works, at least in my version of bash | 15:53 |
| andreaf | mtreinish: I'd say /tmp that's likely to exist | 15:54 |
| andreaf | ^_^ | 15:54 |
| mtreinish | andreaf: well I had the code to create the dir if it didn't exist | 15:54 |
| *** Aimon has quit IRC | 15:55 | |
| *** amotoki_ has joined #openstack-qa | 15:56 | |
| andreaf | mtreinsh: right :P | 15:57 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config fixture support to unit tests https://review.openstack.org/81896 | 15:57 |
| *** yfried has quit IRC | 15:57 | |
| sdague | mtreinish: so my feeling is /tmp | 15:58 |
| sdague | because then you don't leave any crud around | 15:58 |
| sdague | because there is no good way to clean up /tmp/tempest-locks | 15:59 |
| *** mestery has joined #openstack-qa | 15:59 | |
| mtreinish | sdague: ok I assumed that would be your answer. The latest revision uses /tmp with the nested defaults | 16:01 |
| *** Sukhdev has joined #openstack-qa | 16:02 | |
| *** Sukhdev has quit IRC | 16:06 | |
| *** Aimon has joined #openstack-qa | 16:07 | |
| *** mestery has quit IRC | 16:09 | |
| *** mestery has joined #openstack-qa | 16:10 | |
| *** rhsu has joined #openstack-qa | 16:11 | |
| openstackgerrit | A change was merged to openstack/tempest: Adds more verification in list alarms https://review.openstack.org/79942 | 16:15 |
| *** nati_ueno has joined #openstack-qa | 16:17 | |
| psedlak | jhenner: ping | 16:19 |
| jhenner | psedlak: pong | 16:20 |
| *** atiwari has joined #openstack-qa | 16:20 | |
| jhenner | psedlak: makej, musim letět | 16:20 |
| psedlak | jhenner: v tech zmenach pro workaroundy (vnc port, python-suds) prosim vsude pridej LOG.warning('Woraround ...) | 16:20 |
| jhenner | psedlak: njn | 16:21 |
| psedlak | jhenner: a idealne python-suds instaluj pres ten install helper metodu z common | 16:21 |
| jhenner | psedlak: njn | 16:21 |
| openstackgerrit | Jordan Pittier proposed a change to openstack-dev/devstack: Check setup.cfg exists before calling requirements:update.py https://review.openstack.org/82857 | 16:21 |
| *** dkranz has quit IRC | 16:22 | |
| *** nmagnezi has quit IRC | 16:22 | |
| *** yrabl has quit IRC | 16:22 | |
| *** tkammer has quit IRC | 16:25 | |
| *** andreaf has quit IRC | 16:27 | |
| *** mmaglana has joined #openstack-qa | 16:28 | |
| *** nati_ueno has quit IRC | 16:30 | |
| *** catohornet__ has joined #openstack-qa | 16:31 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/67825 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 16:32 |
| *** dkranz has joined #openstack-qa | 16:35 | |
| *** mmaglana_ has joined #openstack-qa | 16:35 | |
| *** mmaglana has quit IRC | 16:38 | |
| *** adalbas has joined #openstack-qa | 16:38 | |
| *** rhsu has quit IRC | 16:41 | |
| *** mmaglana has joined #openstack-qa | 16:41 | |
| *** mmaglana_ has quit IRC | 16:44 | |
| openstackgerrit | Jordan Pittier proposed a change to openstack/tempest: Skip volume snapshot tests if feature is not enabled https://review.openstack.org/81498 | 16:47 |
| *** IzikPenso has joined #openstack-qa | 16:54 | |
| *** torandu has joined #openstack-qa | 16:56 | |
| *** rossella_s has joined #openstack-qa | 16:56 | |
| *** torandu has quit IRC | 16:57 | |
| IzikPenso | Hello, anybody knows if there's a tempest test that tries to configure a instance with more then 1 network ? | 16:57 |
| *** torandu has joined #openstack-qa | 16:57 | |
| *** markmcclain1 has joined #openstack-qa | 17:01 | |
| *** markmcclain1 has quit IRC | 17:01 | |
| *** markmcclain1 has joined #openstack-qa | 17:02 | |
| *** markmcclain has quit IRC | 17:02 | |
| *** nati_ueno has joined #openstack-qa | 17:02 | |
| *** harlowja has joined #openstack-qa | 17:03 | |
| *** rossella_ has quit IRC | 17:04 | |
| *** morganfainberg_Z is now known as morganfainberg | 17:04 | |
| *** mkollaro has quit IRC | 17:07 | |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Commit to test Neutron full job https://review.openstack.org/77905 | 17:08 |
| *** rhsu has joined #openstack-qa | 17:09 | |
| *** kgriffs is now known as kgriffs_afk | 17:10 | |
| *** mkollaro has joined #openstack-qa | 17:10 | |
| *** safchain has quit IRC | 17:11 | |
| openstackgerrit | A change was merged to openstack/tempest: Remove usage of internal library function in basic generator https://review.openstack.org/80963 | 17:16 |
| openstackgerrit | A change was merged to openstack/tempest: Verify the response attributes of 'list_aggregates' https://review.openstack.org/81483 | 17:16 |
| *** jordanP has quit IRC | 17:19 | |
| openstackgerrit | A change was merged to openstack/tempest: Validate get keypair attributes of Nova V2/V3 API https://review.openstack.org/80919 | 17:20 |
| *** mriedem has quit IRC | 17:20 | |
| *** catohornet__ has quit IRC | 17:24 | |
| *** andreaf has joined #openstack-qa | 17:25 | |
| openstackgerrit | Nachi Ueno proposed a change to openstack/tempest: Remove test for obsolated port_filter on Neutron https://review.openstack.org/82880 | 17:25 |
| *** amotoki_ has quit IRC | 17:29 | |
| *** salv-orlando has joined #openstack-qa | 17:30 | |
| *** cmart has quit IRC | 17:32 | |
| *** mitchm has joined #openstack-qa | 17:32 | |
| openstackgerrit | A change was merged to openstack/tempest: Introduce load_tests mechanism for negative tests https://review.openstack.org/80926 | 17:33 |
| openstackrecheck | Console logs not available after 13:27s for gate-keystone-python33 80618,10,5f11cf5 | 17:33 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Improve log level conf for Sahara https://review.openstack.org/78677 | 17:33 |
| nati_ueno | Hi folks. Could you review https://review.openstack.org/#/c/82880/ ? This is needed for fixing critical bug in neutron | 17:35 |
| *** Sukhdev has joined #openstack-qa | 17:35 | |
| mtreinish | nati_ueno: I already reviewed I think you pasted the wrong bug number | 17:35 |
| nati_ueno | my bad.. | 17:35 |
| openstackgerrit | Nachi Ueno proposed a change to openstack/tempest: Remove test for obsolated port_filter on Neutron https://review.openstack.org/82880 | 17:36 |
| *** markmcclain1 has quit IRC | 17:36 | |
| nati_ueno | mtreinish: Thanks! I fixed | 17:36 |
| openstackgerrit | A change was merged to openstack-dev/devstack: VMware: Use multiline for cluster_name opt https://review.openstack.org/82648 | 17:37 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Ironic: Use HOST_IP instead of SERVICE_HOST https://review.openstack.org/82604 | 17:37 |
| sdague | nati_ueno: I continue to not understand the reason for why it's a critical neutron bug | 17:43 |
| sdague | can you enhance the commit message to explain it to me? | 17:43 |
| nati_ueno | sdague: so currently, neutron security group is broken. | 17:43 |
| nati_ueno | sdague: This fix is needed to fix it | 17:44 |
| sdague | how and why? | 17:44 |
| nati_ueno | sdague: so we can't use iptables with OVS | 17:44 |
| nati_ueno | sdague: so we need hybrid vif driver for OVS | 17:44 |
| nati_ueno | sdague: We had a hybrid driver before, but it is removed now | 17:44 |
| nati_ueno | sdague: we have only generic driver which need input from neutron side | 17:45 |
| nati_ueno | sdague: This fix will send a parameter from neturon to nova to decide vif-plug mode | 17:45 |
| sdague | ok, none of that explains how deleting a test fixes neutron | 17:46 |
| *** mriedem has joined #openstack-qa | 17:46 | |
| nati_ueno | sdague: sorry, OK so before we have cap_filter which is totally unused anywhere | 17:46 |
| *** markmcclain has joined #openstack-qa | 17:46 | |
| nati_ueno | In previous discussion, we decided to replace cap_filter to some paramters | 17:47 |
| nati_ueno | In order to fix that bug, we need new set of paramters | 17:47 |
| nati_ueno | However, tempest has a test for cap_filter which will be obsolated | 17:47 |
| sdague | ok, that's more like it. So can you update the commit message with this context, as a couple of paragraphs in there | 17:48 |
| nati_ueno | sdague: sure! | 17:48 |
| sdague | when reviewing changes, it's really important to have the context | 17:48 |
| *** mestery_ has joined #openstack-qa | 17:48 | |
| *** mestery has quit IRC | 17:48 | |
| sdague | otherwise I mostly ignore or -1 the change | 17:48 |
| *** mestery_ is now known as mestery | 17:50 | |
| openstackgerrit | A change was merged to openstack/tempest: Fail a test if stack delete failed https://review.openstack.org/82392 | 17:51 |
| *** torandu_ has joined #openstack-qa | 17:51 | |
| *** torandu_ has quit IRC | 17:51 | |
| *** mlavalle has joined #openstack-qa | 17:51 | |
| openstackgerrit | Nachi Ueno proposed a change to openstack/tempest: Remove test for obsolated port_filter on Neutron https://review.openstack.org/82880 | 17:52 |
| nati_ueno | sdague: I got it. How about new commit message? | 17:52 |
| sdague | nati_ueno: yes, please update with a new commit message with the context, then I'm probably +2 | 17:52 |
| nati_ueno | sdague: I mean i updated the commit message with context https://review.openstack.org/#/c/82880/ | 17:53 |
| *** mitchm has quit IRC | 17:54 | |
| sdague | nati_ueno: also, is there an outstanding neutron change that's related? | 17:54 |
| sdague | that this is blocking | 17:54 |
| *** torandu has quit IRC | 17:54 | |
| nati_ueno | sdague: let me add that review in the commit message | 17:55 |
| sdague | if so, that's good to have here, because normally we require dropping tests like this to have a change on the project | 17:55 |
| sdague | which already has a +2 on it | 17:55 |
| sdague | so we know the project wants to drop that test | 17:55 |
| openstackgerrit | Nachi Ueno proposed a change to openstack/tempest: Remove test for obsolated port_filter on Neutron https://review.openstack.org/82880 | 17:56 |
| nati_ueno | https://review.openstack.org/#/c/21946/ is a neutron test | 17:56 |
| nati_ueno | sorry neutron patch | 17:56 |
| nati_ueno | Which will be -1 from Jenkins due to this tempest test | 17:56 |
| sdague | nati_ueno: yep | 17:57 |
| sdague | nati_ueno: ok, link that | 17:57 |
| sdague | but you also need neutron core +2 on that | 17:57 |
| nati_ueno | I believe this is community decision (please take a look discussion in bug report) | 17:57 |
| *** markmcclain has quit IRC | 17:58 | |
| sdague | if you have a +2 on the neutron patch, we call that good enough on the tempest side, but we do want that as well. | 17:58 |
| nati_ueno | sdague: added https://review.openstack.org/#/c/82880/ | 17:58 |
| *** torandu has joined #openstack-qa | 17:58 | |
| nati_ueno | sdague: hmm this is a dead lock, neutron core wan't see Jenkins's +1.. | 17:58 |
| nati_ueno | sdague: How about let Mark talk with you about this? | 17:58 |
| nati_ueno | sdague: Or I can start discussion in the mailing list | 17:59 |
| sdague | nati_ueno: sure | 17:59 |
| openstackgerrit | A change was merged to openstack/tempest: Refactor create_ and update_ methods for floating ips https://review.openstack.org/80928 | 17:59 |
| sdague | or we can do this in -neutron right now | 17:59 |
| nati_ueno | sdague: sure | 17:59 |
| nati_ueno | sdague: hmm Mark isn't online now | 18:00 |
| nati_ueno | sdague: I'll send a mail for you and mark | 18:00 |
| *** catohornet__ has joined #openstack-qa | 18:01 | |
| *** kgriffs_afk is now known as kgriffs | 18:02 | |
| *** mitchm has joined #openstack-qa | 18:04 | |
| sdague | nati_ueno: on the mailing list please, I largely ignore private emails on stuff like this | 18:04 |
| *** dkranz has quit IRC | 18:05 | |
| nati_ueno | sdague: sure | 18:05 |
| *** Matt2 has left #openstack-qa | 18:06 | |
| *** torandu has quit IRC | 18:07 | |
| *** torandu has joined #openstack-qa | 18:07 | |
| *** yfried has joined #openstack-qa | 18:13 | |
| *** catohornet__ has quit IRC | 18:15 | |
| *** Matt1 has joined #openstack-qa | 18:16 | |
| *** jasondotstar has quit IRC | 18:18 | |
| *** dkranz has joined #openstack-qa | 18:19 | |
| pafuent | Hi. Can someone take a look at this patch https://review.openstack.org/#/c/81909/? | 18:19 |
| *** jasondotstar has joined #openstack-qa | 18:21 | |
| *** catohornet__ has joined #openstack-qa | 18:27 | |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/70816 failed because of: gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1295876 | 18:28 |
| *** yassine has quit IRC | 18:33 | |
| *** IzikPenso has quit IRC | 18:35 | |
| *** mlavalle_ has joined #openstack-qa | 18:36 | |
| *** mlavalle has quit IRC | 18:37 | |
| *** mlavalle_ is now known as mlavalle | 18:37 | |
| *** yassine has joined #openstack-qa | 18:39 | |
| sdague | mtreinish: how do you feel about getting the heat templates out of python and into yaml files? | 18:39 |
| sdague | pafuent: so 2 things about that review | 18:41 |
| sdague | which are mostly preexisting issues with that test | 18:41 |
| sdague | test file | 18:41 |
| sdague | could we extract the template into a yaml file properly | 18:42 |
| pafuent | sdague: Ok, no problem. | 18:42 |
| sdague | the other issue is, all the real work happens in setupClass ? | 18:43 |
| sdague | it's hard to get the timing, because all the tests take 0.002s (give or take) | 18:43 |
| mtreinish | sdague: I'm fine with that my concern is 2 things though: is there inhertance with the yaml files so we can abstract out common chunks | 18:44 |
| sdague | could we instead remove setupClass, put the call in setup and move the tests into a single test function | 18:44 |
| pafuent | sdague: Yes | 18:44 |
| mtreinish | and hardcoded resource names/types in yaml files | 18:44 |
| sdague | mtreinish: heat has it's own variable interpolation | 18:44 |
| sdague | so the answer should be no | 18:44 |
| pafuent | sdague: But all the test will create the same stack | 18:45 |
| mtreinish | sdague: ok yeah if heat will do the variable interpolation then there shouldn't be an issue there | 18:45 |
| mtreinish | but what about duplication between yaml files | 18:45 |
| pafuent | sdague: (The Yes was for all the work is made in setupClass) | 18:45 |
| *** yfried has quit IRC | 18:45 | |
| *** yfried has joined #openstack-qa | 18:45 | |
| sdague | pafuent: any idea how long the setupClass takes? | 18:46 |
| pafuent | sdague: No, but takes time | 18:46 |
| sdague | mtreinish: any idea if we actually get timing results out of setup added to tests ? | 18:46 |
| mtreinish | sdague: I don't think we have separate setup timings | 18:47 |
| mtreinish | we'd have to do that manually or inject it into the subunit | 18:48 |
| mtreinish | or do you mean something else? | 18:48 |
| sdague | mtreinish: I'm trying to figure out if work in setup gets added to the test time | 18:49 |
| *** jpich has quit IRC | 18:49 | |
| sdague | basically if we did setup + 1 test, vs setupclass + 10 tests, if we would have more accuracy in if things are slow or not | 18:49 |
| *** kgriffs is now known as kgriffs_afk | 18:50 | |
| mtreinish | so I think setup might get counted but not setupclass (but that's just my guess) | 18:50 |
| sdague | also, if testr is smart enough to not run setupclass if we regex skip all it's tests | 18:50 |
| mtreinish | lifeless should know the answer though | 18:50 |
| sdague | yeh, I think it's early for lifeless to be on | 18:50 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/82746 failed gate-neutron-python26 in the gate queue with an unrecognized error | 18:52 |
| clarkb | I am pretty sure setupclass is not run if none of the jobs use that class | 18:53 |
| mtreinish | clarkb: yeah I think you're right about that | 18:54 |
| mtreinish | it should be pretty easy to verify too | 18:54 |
| clarkb | the names encode the module path so it is easy for the test runner to not run things when the class isn't used | 18:54 |
| *** tosky has quit IRC | 18:55 | |
| sdague | cool | 18:55 |
| sdague | so in looking at the existing heat tests in tempest, I think we can simplify a lot of them | 18:55 |
| *** dkorn has joined #openstack-qa | 18:56 | |
| sdague | folks were doing a lot of copy / paste from some of the other tests that really lean hard on setupClass, and I think we don't want to do that here | 18:56 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: simplify heat test_limits https://review.openstack.org/82894 | 18:57 |
| *** jlibosva has quit IRC | 19:00 | |
| mtreinish | sdague: probably not | 19:00 |
| lifeless | sdague: mtreinish: hi | 19:01 |
| sdague | lifeless: question on time reporting | 19:01 |
| sdague | do we get the time for setup added to the time for the test? | 19:01 |
| *** rhsu has quit IRC | 19:04 | |
| lifeless | sdague: yes, but not class setup | 19:07 |
| sdague | ok | 19:07 |
| lifeless | class setup is done by the suite that wraps the case, rather than the case itself | 19:07 |
| sdague | yes, that's what I was hoping | 19:07 |
| sdague | that gives me incentive to restructure the heat tests so we do more with setup | 19:08 |
| sdague | because right now it's really hard to evaluate what's slow or not with all the time keeping hidden away | 19:08 |
| mtreinish | clarkb: actually I think setupclass does run even if all the tests are skipped: http://paste.openstack.org/show/74284/ | 19:09 |
| clarkb | mtreinish: huh, I wouldn't expect test runners to import the class at all in that case | 19:09 |
| clarkb | I suppose import may happen if colocated with other classes that are run | 19:10 |
| clarkb | but surely it shouldn't execute the setupClass | 19:10 |
| *** dkranz has quit IRC | 19:10 | |
| mtreinish | clarkb: hmm yeah it's because they're in the same file I think | 19:11 |
| mtreinish | when I added a new file with all skips it doesn't run that one | 19:12 |
| clarkb | interesting | 19:13 |
| clarkb | I bet it is tied to import behavior then | 19:13 |
| *** yfried has quit IRC | 19:14 | |
| *** malini is now known as malini_afk | 19:16 | |
| *** openstackgerrit has quit IRC | 19:18 | |
| *** openstackgerrit has joined #openstack-qa | 19:18 | |
| *** kgriffs_afk is now known as kgriffs | 19:21 | |
| mtreinish | sdague: do you want to push this through?: https://review.openstack.org/#/c/82531/ | 19:22 |
| mtreinish | I think we should merge it before we add anything else to the repo | 19:22 |
| *** dkranz has joined #openstack-qa | 19:22 | |
| openstackgerrit | A change was merged to openstack/qa-specs: Add sphinx support to qa-specs repo https://review.openstack.org/82531 | 19:23 |
| sdague | mtreinish: yep, done | 19:23 |
| *** catohornet__ has quit IRC | 19:26 | |
| *** zul has quit IRC | 19:26 | |
| *** zul has joined #openstack-qa | 19:30 | |
| *** luqas has quit IRC | 19:32 | |
| *** Sukhdev has quit IRC | 19:35 | |
| *** dkorn has quit IRC | 19:51 | |
| *** rhsu has joined #openstack-qa | 19:57 | |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: factor out templates to yaml files https://review.openstack.org/82905 | 20:00 |
| sdague | mtreinish: that's not complete yet, however please take a look when you can | 20:00 |
| mtreinish | sdague: sure looking at it now | 20:01 |
| openstackrecheck | openstack/oslo.messaging change: https://review.openstack.org/81261 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 20:04 |
| mtreinish | sdague: I really like how it cleans up those test files | 20:05 |
| sdague | ok, I'll worth through the template extract as a single patch then | 20:06 |
| sdague | s/worth/work/ | 20:06 |
| sdague | there is another issue with class variables being used everywhere | 20:06 |
| sdague | but I think that's follow on | 20:06 |
| mtreinish | yeah I noticed that too | 20:07 |
| *** torandu has quit IRC | 20:09 | |
| *** torandu has joined #openstack-qa | 20:09 | |
| *** yassine has quit IRC | 20:12 | |
| *** torandu has quit IRC | 20:17 | |
| *** jasondotstar has quit IRC | 20:20 | |
| *** yfried has joined #openstack-qa | 20:23 | |
| *** torandu has joined #openstack-qa | 20:23 | |
| *** mkollaro has quit IRC | 20:23 | |
| *** dkorn has joined #openstack-qa | 20:27 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Official client based auth https://review.openstack.org/82911 | 20:28 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Use auth data to fill credentials https://review.openstack.org/82111 | 20:28 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Define V3 Credentials https://review.openstack.org/81872 | 20:28 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Get credential IDs from Credentials class https://review.openstack.org/80246 | 20:28 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Enforces the use of Credentials (part1) https://review.openstack.org/77594 | 20:28 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Enforces the use of Credentials (part2) https://review.openstack.org/82112 | 20:28 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Access credential fields as attributes https://review.openstack.org/82113 | 20:28 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add config fixture support to unit tests https://review.openstack.org/81896 | 20:28 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Defines a Credentials class https://review.openstack.org/74387 | 20:28 |
| *** rockyg has joined #openstack-qa | 20:28 | |
| jogo | sdague: not yet, I'll mark as WIP | 20:29 |
| andreaf | mtreinish: I rebased my growing pile of patchsets on top of latest master + your unit test change for locks (https://review.openstack.org/#/c/81896/). Also I fixed unit tests so they pass now even with 4 parallel processes | 20:30 |
| mtreinish | andreaf: nice | 20:31 |
| mtreinish | I'll try to take a look soon | 20:31 |
| andreaf | mtreinish: thanks | 20:31 |
| *** mkollaro has joined #openstack-qa | 20:32 | |
| mtreinish | andreaf: what was it you needed to do for the unit tests | 20:32 |
| andreaf | mtreinish: I simplified things a bit I just have a single FakeConfig now | 20:33 |
| andreaf | mtreinish: plus I had some genuine bugs to fix in the unit tests | 20:33 |
| andreaf | mtreinish: one bug at least - you can compare patchset 4 and 5 here https://review.openstack.org/#/c/81872/4..5/tempest/tests/test_credentials.py | 20:34 |
| andreaf | afazekas, maurosr: I had to rebase https://review.openstack.org/#/c/74387/ - can I have your review back? thanks! | 20:37 |
| *** adalbas has quit IRC | 20:38 | |
| *** otherwiseguy has joined #openstack-qa | 20:39 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Adds Ironic test_ironic_basic_ops scenario test https://review.openstack.org/81958 | 20:46 |
| maurosr | andreaf: sure | 20:47 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/82716 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 20:56 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: simplify heat test_limits https://review.openstack.org/82894 | 21:06 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: factor out templates to yaml files https://review.openstack.org/82905 | 21:06 |
| sdague | mtreinish: ok, I think that's a real pass 1 | 21:06 |
| sdague | unwinding this stuff is a bit of a mess | 21:06 |
| sdague | but given the push to get more in here doing it now should save us a ton of time later | 21:06 |
| *** aysyd has quit IRC | 21:07 | |
| *** pafuent has quit IRC | 21:08 | |
| *** mlavalle has quit IRC | 21:09 | |
| *** rfolco has quit IRC | 21:12 | |
| mtreinish | sdague: how about some unit tests for the template loader... | 21:12 |
| mtreinish | that should be pretty straightforward | 21:12 |
| mtreinish | sdague: so I'm curious you're doing the yaml.load and json.load in the template reader | 21:20 |
| mtreinish | but in the tests before it passed the raw string | 21:20 |
| mtreinish | doesn't the load convert it to a python dict | 21:20 |
| mtreinish | so aren't you changing the test | 21:20 |
| sdague | oh... right | 21:20 |
| sdague | yep | 21:20 |
| sdague | well, it would blow up in tests | 21:20 |
| openstackgerrit | Tushar Kalra proposed a change to openstack/tempest: Typo in config.py https://review.openstack.org/82599 | 21:25 |
| dkranz | sdague: I'm seeing a lot of patches for validating json return payloads. Are you thinking reviewers are going to verify that the list is correct. If so, how? | 21:28 |
| *** beisner is now known as beisner-afk | 21:29 | |
| *** beisner-afk is now known as beisner | 21:29 | |
| mtreinish | dkranz: I thought they were just copies of the nova jsonschema | 21:34 |
| mtreinish | but I haven't looked at one in detail yet | 21:34 |
| sdague | dkranz: my take is that was locking down existing behavior (especially for the v2 proxy on v3 for nova) | 21:36 |
| sdague | so execution results and basic sanity checking should be sufficient | 21:36 |
| *** dkranz has quit IRC | 21:36 | |
| sdague | mtreinish: so the issue with the unit test is we'll have to make that function a lot more complicated to test it | 21:39 |
| sdague | because it's a hard coded path for the heat tests | 21:39 |
| openstackgerrit | A change was merged to openstack/tempest: Test current tenant not added to private flavor https://review.openstack.org/81551 | 21:39 |
| sdague | I feel like the fact that it will explode if it does a bad thing is sufficient testing, as it doesn't have a silent fail path | 21:39 |
| sdague | especially in this iteration | 21:40 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: factor out templates to yaml files https://review.openstack.org/82905 | 21:40 |
| mtreinish | sdague: why is hard? just mock the open call | 21:41 |
| sdague | so what exactly are you going to test then... | 21:42 |
| sdague | that string concat works? | 21:42 |
| mtreinish | heh yeah I guess I was thinking the fail path with extensions | 21:42 |
| mtreinish | but if you're not doing loads anymore it's not really needed | 21:42 |
| sdague | yeh | 21:42 |
| *** jeblair_ has joined #openstack-qa | 21:42 | |
| sdague | oh, I need to remove the imports | 21:43 |
| *** jeblair_ is now known as corvus | 21:43 | |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: factor out templates to yaml files https://review.openstack.org/82905 | 21:43 |
| *** jeblair has quit IRC | 21:45 | |
| *** corvus is now known as jeblair | 21:45 | |
| *** StevenK has quit IRC | 21:47 | |
| *** russellb has quit IRC | 21:47 | |
| *** zigo has quit IRC | 21:47 | |
| *** wendar has quit IRC | 21:47 | |
| *** StevenK has joined #openstack-qa | 21:47 | |
| *** russellb has joined #openstack-qa | 21:48 | |
| *** mwagner_lap has quit IRC | 21:49 | |
| *** wendar has joined #openstack-qa | 21:49 | |
| *** zigo has joined #openstack-qa | 21:49 | |
| *** mriedem has quit IRC | 21:50 | |
| *** catohornet__ has joined #openstack-qa | 21:50 | |
| *** malini_afk is now known as malini | 21:52 | |
| *** torandu has quit IRC | 21:55 | |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/70816 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 22:05 |
| *** shwetaap has quit IRC | 22:09 | |
| *** catohornet___ has joined #openstack-qa | 22:11 | |
| *** catohornet__ has quit IRC | 22:15 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/qa-specs: Add spec for bp:config-verification https://review.openstack.org/82933 | 22:17 |
| *** dims_ has quit IRC | 22:18 | |
| *** catohornet____ has joined #openstack-qa | 22:19 | |
| *** catohornet____ is now known as catohornet__ | 22:19 | |
| *** catohornet___ has quit IRC | 22:21 | |
| mtreinish | sdague: ^^^ run tox -ebuild_specs on that, the html version looks pretty good I think | 22:21 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/qa-specs: Add spec for bp:config-verification https://review.openstack.org/82933 | 22:26 |
| adam_g | any pointers on this testr failure? having trouble reproducing locally http://logs.openstack.org/58/81958/2/check/check-tempest-dsvm-full/723ae7e/console.html | 22:29 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/qa-specs: Add a license file https://review.openstack.org/82934 | 22:29 |
| mtreinish | adam_g: well something you add in a base file is causing an exception when the test runner imports the file | 22:30 |
| mtreinish | that's why all the test files are listed after 'import errors' | 22:30 |
| mtreinish | it could be something as simple as a syntax error | 22:31 |
| *** catohornet__ has quit IRC | 22:32 | |
| adam_g | mtreinish, i figured as much, but having trouble replicating that locally. running the equivalent commands here seem to import and run things fine | 22:32 |
| *** dims_ has joined #openstack-qa | 22:34 | |
| * adam_g wonders about dependencies | 22:35 | |
| mtreinish | adam_g: that might be it, when I ran it locally I got a ironicclient import error because I didn't have it installed | 22:35 |
| mtreinish | I'm assuming devstack took care of that though | 22:35 |
| clarkb | if you install testrepository from latest source the message is much better | 22:35 |
| adam_g | mtreinish, yeah--ironicclient was my first thought | 22:36 |
| adam_g | clarkb, oh, cool | 22:36 |
| mtreinish | adam_g: yeah you'll definitely need to add it to requirements.txt there | 22:36 |
| adam_g | mtreinish, :) | 22:37 |
| mtreinish | because I don't think it's there for tempest yet | 22:37 |
| *** catohornet1 has joined #openstack-qa | 22:38 | |
| mtreinish | but there might be something else too | 22:38 |
| mtreinish | I can look at more tomorrow | 22:38 |
| *** catohornet1 has quit IRC | 22:39 | |
| *** catohornet1 has joined #openstack-qa | 22:39 | |
| openstackgerrit | Nachi Ueno proposed a change to openstack-dev/devstack: Change NOVA_VIF_DRIVER for ml2 with NeutronFirewallVIFDriver https://review.openstack.org/82937 | 22:39 |
| *** catohornet1 has quit IRC | 22:39 | |
| *** catohornet1 has joined #openstack-qa | 22:40 | |
| openstackgerrit | Nachi Ueno proposed a change to openstack-dev/devstack: Change NOVA_VIF_DRIVER for ml2 with NeutronFirewallVIFDriver https://review.openstack.org/82937 | 22:41 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Adds Ironic test_ironic_basic_ops scenario test https://review.openstack.org/81958 | 22:41 |
| *** catohornet1 has quit IRC | 22:51 | |
| *** sthaha has joined #openstack-qa | 22:59 | |
| openstackgerrit | A change was merged to openstack/tempest: Verify attributes through Nova "list security groups" API https://review.openstack.org/81452 | 23:02 |
| *** mwagner_lap has joined #openstack-qa | 23:02 | |
| openstackgerrit | A change was merged to openstack/tempest: Add a new exception for invalid structure https://review.openstack.org/80927 | 23:02 |
| *** malini has left #openstack-qa | 23:04 | |
| *** markmcclain has joined #openstack-qa | 23:10 | |
| *** markmcclain1 has joined #openstack-qa | 23:11 | |
| *** otherwiseguy has quit IRC | 23:13 | |
| *** markmcclain has quit IRC | 23:15 | |
| *** rockyg has quit IRC | 23:16 | |
| openstackgerrit | A change was merged to openstack/tempest: simplify heat test_limits https://review.openstack.org/82894 | 23:18 |
| *** andreaf has quit IRC | 23:22 | |
| *** krtaylor has quit IRC | 23:23 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/78950 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 23:24 |
| *** jgrimm has quit IRC | 23:25 | |
| *** giulivo has quit IRC | 23:28 | |
| *** markmcclain1 has quit IRC | 23:30 | |
| *** mkollaro has quit IRC | 23:30 | |
| *** mkollaro has joined #openstack-qa | 23:31 | |
| *** catohornet1 has joined #openstack-qa | 23:45 | |
| openstackgerrit | Chmouel Boudjnah proposed a change to openstack-dev/devstack: Swift3 has been moved to stackforge. https://review.openstack.org/82947 | 23:45 |
| *** mwagner_ has joined #openstack-qa | 23:54 | |
| *** Aimon has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!