*** achanda has quit IRC | 00:04 | |
*** mriedem_away has quit IRC | 00:06 | |
*** mriedem has joined #openstack-qa | 00:08 | |
*** mriedem1 has joined #openstack-qa | 00:09 | |
*** mriedem has quit IRC | 00:12 | |
*** mwagner_lap has joined #openstack-qa | 00:17 | |
*** zz_dimtruck is now known as dimtruck | 00:17 | |
*** achanda has joined #openstack-qa | 00:26 | |
oomichi | mtreinish: hi, are you still there? | 00:31 |
---|---|---|
*** jhenner has quit IRC | 00:31 | |
oomichi | mtreinish: I'd like to release a new tempest-lib now. | 00:31 |
*** yamamoto has joined #openstack-qa | 00:32 | |
*** yamamoto has quit IRC | 00:37 | |
*** ranger81 has joined #openstack-qa | 00:38 | |
openstackgerrit | Jamie Lennox proposed openstack/tempest: Give access to CredClient in tests https://review.openstack.org/214987 | 00:41 |
*** jasonsb has joined #openstack-qa | 00:41 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Make create_server use **kwargs https://review.openstack.org/217468 | 00:41 |
*** Sukhdev has quit IRC | 00:43 | |
*** dims__ has quit IRC | 00:46 | |
oomichi | masayukig: I am glad if you can take time for https://review.openstack.org/#/c/221216/ | 00:47 |
oomichi | masayukig: and https://review.openstack.org/#/c/220391/ | 00:47 |
*** dims__ has joined #openstack-qa | 00:47 | |
*** swamireddy has quit IRC | 00:48 | |
*** hockeynut_afk has quit IRC | 00:48 | |
*** lbragstad has quit IRC | 00:48 | |
*** BadCub has quit IRC | 00:48 | |
*** eglute_s has quit IRC | 00:48 | |
*** dimtruck has quit IRC | 00:48 | |
*** wendar has quit IRC | 00:48 | |
*** Nakato has quit IRC | 00:48 | |
*** mattoliverau has quit IRC | 00:48 | |
*** wendar has joined #openstack-qa | 00:49 | |
*** vaidy_ril has quit IRC | 00:49 | |
*** otherwiseguy has quit IRC | 00:49 | |
*** jamielennox has quit IRC | 00:49 | |
*** flwang has quit IRC | 00:49 | |
*** jaypipes has quit IRC | 00:50 | |
*** StevenK has quit IRC | 00:50 | |
*** pvaneck has quit IRC | 00:50 | |
*** comstud has quit IRC | 00:50 | |
*** jroll has quit IRC | 00:50 | |
*** ttx has quit IRC | 00:50 | |
*** Nakato has joined #openstack-qa | 00:50 | |
*** mgagne has quit IRC | 00:50 | |
*** mattoliverau has joined #openstack-qa | 00:51 | |
*** BadCub has joined #openstack-qa | 00:51 | |
*** StevenK has joined #openstack-qa | 00:51 | |
*** dtroyer has quit IRC | 00:51 | |
openstackgerrit | Takashi NATSUME proposed openstack/tempest: Add container and disk format parameters https://review.openstack.org/211065 | 00:52 |
*** ttx has joined #openstack-qa | 00:53 | |
*** dimtruck has joined #openstack-qa | 00:53 | |
*** otherwiseguy has joined #openstack-qa | 00:53 | |
*** ranger81 has quit IRC | 00:53 | |
*** jroll has joined #openstack-qa | 00:53 | |
*** eglute has joined #openstack-qa | 00:53 | |
*** comstud has joined #openstack-qa | 00:53 | |
*** hockeynut has joined #openstack-qa | 00:53 | |
*** mgagne has joined #openstack-qa | 00:54 | |
*** jamielennox has joined #openstack-qa | 00:54 | |
*** lbragstad has joined #openstack-qa | 00:54 | |
*** dtroyer has joined #openstack-qa | 00:54 | |
*** swamireddy has joined #openstack-qa | 00:54 | |
*** flwang has joined #openstack-qa | 00:56 | |
*** zhangjn has joined #openstack-qa | 00:56 | |
*** vaidy_ril has joined #openstack-qa | 00:56 | |
*** jaypipes has joined #openstack-qa | 00:56 | |
*** ChanServ sets mode: +o jaypipes | 00:56 | |
*** zhangjn has quit IRC | 00:58 | |
*** tpeoples has quit IRC | 00:59 | |
*** cody-somerville has quit IRC | 01:00 | |
*** zhangjn has joined #openstack-qa | 01:06 | |
*** rossella_s has quit IRC | 01:06 | |
*** rossella_s has joined #openstack-qa | 01:06 | |
*** mlavalle has quit IRC | 01:09 | |
*** zhangjn_ has joined #openstack-qa | 01:11 | |
*** zhangjn has quit IRC | 01:14 | |
*** mriedem1 is now known as mriedem_away | 01:15 | |
openstackrecheck | Console logs not available after 13:22s for gate-tempest-dsvm-full-test-accounts 165233,22,ba1d951 | 01:21 |
*** dimtruck is now known as zz_dimtruck | 01:24 | |
*** salv-orlando has joined #openstack-qa | 01:29 | |
*** cody-somerville has joined #openstack-qa | 01:33 | |
*** cody-somerville has joined #openstack-qa | 01:33 | |
*** salv-orlando has quit IRC | 01:36 | |
*** salv-orlando has joined #openstack-qa | 01:43 | |
*** salv-orlando has quit IRC | 01:47 | |
*** armax has joined #openstack-qa | 01:59 | |
mtreinish | oomichi: ok, one sec I'll push it out right now | 02:01 |
*** sabeen has joined #openstack-qa | 02:05 | |
oomichi | mtreinish: cool, thanks so much | 02:05 |
*** zz_dimtruck is now known as dimtruck | 02:05 | |
*** sabeen has quit IRC | 02:06 | |
mtreinish | oomichi: ok, the tag is pushed it's working its way through the gate now | 02:06 |
mtreinish | oomichi: fwiw, you can just push the tag yourself :) | 02:06 |
oomichi | mtreinish: yeah, I can but I am the second ;) | 02:07 |
oomichi | mtreinish: tempest-lib affects a lot of probject gates | 02:07 |
oomichi | mtreinish: can we check the effect of new tempest-lib release before releasing? | 02:08 |
oomichi | mtreinish: I'd like to check the effect on Tempest gate which contains a lot of gate jobs. | 02:09 |
oomichi | mtreinish: before releasing | 02:09 |
oomichi | mtreinish: if possible | 02:09 |
mtreinish | oomichi: oh, well constraints will protect us from breaking the world now on master | 02:09 |
mtreinish | and stable has tempest-lib capped (except for tempest which is venv isolated) | 02:10 |
mtreinish | and we know tempest still works because we gate a neutron job on tempest master | 02:10 |
mtreinish | s/tempest master/tempest-lib changes | 02:10 |
mtreinish | I think we might have a gap if we forgot to flip the venv only switch on stable/kilo, but that's simple enough to fix | 02:11 |
mtreinish | and mriedem_away will catch it when he's back :) | 02:11 |
mtreinish | oomichi: I mean if you want to push an infra job to add experimental jobs to tempest-lib for a bunch of different gate jobs that use tempest-lib feel free | 02:12 |
mtreinish | that way we can test a patch that we're going to use as release before we push a tag | 02:12 |
mtreinish | oomichi: https://pypi.python.org/pypi/tempest-lib/0.8.0 | 02:13 |
*** yamamoto has joined #openstack-qa | 02:15 | |
oomichi | mtreinish: yeah, we can always test the latest tempest-lib on a neutron job. my concern is we forgot effects to the other jobs. | 02:16 |
oomichi | mtreinish: but maybe a neutron job can cover most cases. | 02:16 |
*** tpeoples has joined #openstack-qa | 02:16 | |
mtreinish | oomichi: sure, I mean I'm ok with that risk, but if we wanted it wouldn't be too hard to add experimental jobs to tempest-lib to run other job types | 02:16 |
oomichi | mtreinish: I wanted to know if you have some magic for knowing the impact before releasing ;) | 02:16 |
mtreinish | oomichi: heh, I just push the tag and cross my fingers | 02:17 |
mtreinish | I've only broken things once, maybe twice :) | 02:17 |
oomichi | mtreinish: ok, I got it. and I can release a new version after now | 02:17 |
oomichi | mtreinish: :) | 02:18 |
oomichi | mtreinish: so I will release it morning for recovering some bad release quickly | 02:18 |
gmann | mtreinish: In free time, can you please have a look - https://review.openstack.org/#/c/219169/ | 02:18 |
oomichi | mtreinish: thanks for a new version, anyway :) | 02:19 |
oomichi | mtreinish: it is nice idea to add experimental jobs to tempest-lib | 02:19 |
*** yamahata has quit IRC | 02:19 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/tempest: Add a simple test for port security vs security group https://review.openstack.org/221220 | 02:20 |
mtreinish | oomichi: fwiw, to a certain extent I'm not even sure how many jobs are using tempest-lib. That's part of the fun of having a library | 02:21 |
mtreinish | people use it and you have no idea :) | 02:21 |
mtreinish | gmann: so, I'm looking at: https://review.openstack.org/#/c/200486/10/tempest/clients.py and from what I can tell the old config opt didn't do anything | 02:21 |
mtreinish | (that's the patch which split up the options) | 02:21 |
*** yamamoto has quit IRC | 02:22 | |
gmann | mtreinish: actually we had default value of old config option 'endpoint_type' as publicURL | 02:23 |
gmann | mtreinish: and our new option were made as depreciated opt for that old one | 02:23 |
oomichi | mtreinish: yeah, the fun and terrifying part as a library | 02:24 |
mtreinish | gmann: oh, I see what you're saying. So if someone had set that to something else assuming it was publicurl even though it didn't do anything when the run new tempest with old config it breaks | 02:24 |
mtreinish | gmann: ok, I'm fine with the patch then | 02:24 |
gmann | mtreinish: so if old users have uncommented the old option 'endpoint_type = publicURL' in old version of config then, that will fail with new config | 02:24 |
gmann | mtreinish: yea :) | 02:25 |
openstackgerrit | Reedip proposed openstack/tempest: Added Test Case for InterfacesClient https://review.openstack.org/218133 | 02:25 |
gmann | mtreinish: Thanks | 02:25 |
mtreinish | oomichi: just fyi, constraints will block the new tempest-lib release from showing up in the gate job, you need to bump it in g-r first | 02:29 |
oomichi | mtreinish: oops, got it. and forgot it when recheck | 02:30 |
mtreinish | gmann: I'll trade you: https://review.openstack.org/221330 and https://review.openstack.org/216438 | 02:32 |
gmann | mtreinish: sure, ll do. | 02:33 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Only add roles for enabled services https://review.openstack.org/216522 | 02:36 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Fix test_port_list_filter_by_ip with test_accounts https://review.openstack.org/216887 | 02:36 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Deprecate credential config options https://review.openstack.org/211601 | 02:36 |
mtreinish | gmann: cool, thanks they're both simple doc changes | 02:37 |
oomichi | mtreinish: g-r patch is https://review.openstack.org/#/c/221576 | 02:37 |
mtreinish | gmann: if you get more adventurous ^^^ those 3 are to start the deprecation timer for the credential config options | 02:38 |
gmann | mtreinish: yup | 02:38 |
*** amotoki has joined #openstack-qa | 02:38 | |
*** woodster_ has quit IRC | 02:39 | |
mtreinish | oomichi: also upper-constraints.txt will need to be bumped. That's actually what's holding it at the lower version | 02:39 |
mtreinish | oomichi: I think there is a bot which updates that periodically, but I think you can do it manually too | 02:39 |
* mtreinish hasn't had to deal with it yet | 02:40 | |
mtreinish | oomichi: the other thing is I think we might be in requirements freeze now because we're post L-3 | 02:41 |
* mtreinish checks | 02:41 | |
oomichi | mtreinish: thanks for quick review, done | 02:41 |
mtreinish | oomichi: err, I don't know where to look for that anymore | 02:42 |
mtreinish | heh, I'm sure the g-r reviewers will let you know if we're in freeze :) | 02:42 |
oomichi | mtreinish: I think tempest-lib should be an exception from requirements freeze because tempest wll help improving quality | 02:43 |
oomichi | mtreinish: during -rc time | 02:43 |
mtreinish | oomichi: it wasn't last year. We were blocked doing using the lib at all until the juno rc period on adding tempest-lib to requirements | 02:44 |
mtreinish | oomichi: but I agree | 02:44 |
*** zhangjn has joined #openstack-qa | 02:46 | |
*** zhangjn_ has quit IRC | 02:49 | |
oomichi | mtreinish: ok, I will write the NOTE on the commit message to try getting an exception ;-) | 02:49 |
openstackgerrit | Takashi NATSUME proposed openstack/tempest: Use container/disk format settings in glance v1 test https://review.openstack.org/218775 | 02:51 |
*** dims_ has joined #openstack-qa | 02:56 | |
*** dims__ has quit IRC | 02:58 | |
*** salv-orlando has joined #openstack-qa | 03:03 | |
*** ranger81 has joined #openstack-qa | 03:05 | |
*** armax has quit IRC | 03:05 | |
*** dimtruck is now known as zz_dimtruck | 03:08 | |
*** yamamoto has joined #openstack-qa | 03:08 | |
*** zhangjn_ has joined #openstack-qa | 03:09 | |
*** ranger81 has quit IRC | 03:09 | |
*** salv-orlando has quit IRC | 03:10 | |
*** mylu_ has quit IRC | 03:12 | |
*** zhangjn has quit IRC | 03:13 | |
*** mylu has joined #openstack-qa | 03:15 | |
*** zhangjn has joined #openstack-qa | 03:16 | |
*** zhangjn_ has quit IRC | 03:19 | |
*** armax has joined #openstack-qa | 03:22 | |
*** zz_dimtruck has quit IRC | 03:29 | |
*** ranger81 has joined #openstack-qa | 03:31 | |
piyush_ | i have added the functionality of namespaces api https://review.openstack.org/#/c/217555/ please have a look.. | 03:31 |
*** armax has quit IRC | 03:33 | |
*** zhangjn_ has joined #openstack-qa | 03:36 | |
*** zz_dimtruck has joined #openstack-qa | 03:38 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/219169 failed gate-tempest-python34, gate-tempest-python27 in the gate queue with an unrecognized error | 03:38 |
*** zz_dimtruck is now known as dimtruck | 03:38 | |
*** zhangjn has quit IRC | 03:40 | |
mtreinish | oomichi, gmann: so I think the tempest-lib release broke the tempest unit tests: http://logs.openstack.org/69/219169/1/gate/gate-tempest-python27/ddc6955/console.html#_2015-09-09_02_30_44_704 | 03:41 |
mtreinish | I don't have time to debug it right now (I shouldn't really be online right now) | 03:42 |
oomichi | mtreinish: ok, will see it soon | 03:42 |
*** fedexo has joined #openstack-qa | 03:42 | |
mtreinish | but if you guys could take a look, I'll take over anything in motion when I'm online in the morning | 03:42 |
mtreinish | oomichi: cool, thanks | 03:42 |
oomichi | mtreinish: no problem, thanks for working hardly in night | 03:43 |
*** ajmiller has joined #openstack-qa | 03:43 | |
gmann | mtreinish: Sure, have gn | 03:44 |
oomichi | oops, that is gmann's patch | 03:44 |
*** rbak has joined #openstack-qa | 03:46 | |
*** rbak has quit IRC | 03:48 | |
*** rbak has joined #openstack-qa | 03:49 | |
*** dave-mccowan has quit IRC | 03:51 | |
*** achanda has quit IRC | 03:53 | |
*** ajmiller has quit IRC | 03:54 | |
*** fedexo has quit IRC | 03:56 | |
*** gamado has joined #openstack-qa | 03:58 | |
*** salv-orlando has joined #openstack-qa | 03:59 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/220391 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1355573, gate-tempest-python27: unrecognized error, gate-tempest-python34: unrecognized error | 03:59 |
openstack | Launchpad bug 1355573 in grenade "grenade volume scenarios periodically fail /w SSHTimeout" [Undecided,New] | 03:59 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default https://review.openstack.org/221300 | 03:59 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/tempest: Add a simple test for port security vs security group https://review.openstack.org/221220 | 04:00 |
*** salv-orlando has quit IRC | 04:04 | |
*** alex_xu has quit IRC | 04:04 | |
*** alex_xu has joined #openstack-qa | 04:05 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Switch using token_client to the latest for gate https://review.openstack.org/221588 | 04:13 |
oomichi | mtreinish: gmann: https://review.openstack.org/221588 is for fixing them. | 04:13 |
*** dims_ has quit IRC | 04:13 | |
oomichi | but to be honest, I cannot understand why stubs doesn't work for inheritated classes | 04:14 |
oomichi | anyway, I'd like to wait for the gate job because I didn't run all unit test locally. just some tests which failed | 04:15 |
gmann | oomichi: yea me too, stub should work for that too | 04:15 |
*** armax has joined #openstack-qa | 04:16 | |
oomichi | gmann: yeah, | 04:17 |
gmann | oomichi: checking on oslotests for stub | 04:17 |
oomichi | gmann: mtreinish: anyway, the gate job passed on https://review.openstack.org/#/c/221588/ | 04:17 |
gmann | oomichi: but if someone else try to stub old client class? | 04:18 |
oomichi | gmann: yeah, nice point. if someone else also is using tempest-lib on the same way, they face the same problem | 04:19 |
gmann | oomichi: yea | 04:19 |
gmann | oomichi: m +2 on that to unblock gate and meanwhile i will try to find why stub did not work | 04:21 |
oomichi | gmann: thanks:-) | 04:21 |
*** achanda has joined #openstack-qa | 04:28 | |
*** Sukhdev has joined #openstack-qa | 04:28 | |
*** SridharG has joined #openstack-qa | 04:29 | |
openstackgerrit | Anusha Ramineni proposed openstack/tempest: Full response for missed IdentityV2Client methods https://review.openstack.org/216109 | 04:30 |
*** jswarren has quit IRC | 04:30 | |
*** ranger81 has quit IRC | 04:32 | |
oomichi | gmann: oslotest seems to use mox for stubs instead of mock internally | 04:33 |
gmann | oomichi: yea, they use mox3 | 04:33 |
oomichi | gmann: if we use mock instead of stub on unit tests, can we avoid that? | 04:34 |
gmann | oomichi: may be but i think that should work for mox3 also | 04:34 |
oomichi | gmann: I didn't try it yet | 04:34 |
gmann | oomichi: i can try quickly | 04:34 |
oomichi | gmann: thanks again | 04:35 |
*** Poornima has joined #openstack-qa | 04:37 | |
openstackgerrit | Anusha Ramineni proposed openstack/tempest: Full response for missed IdentityV2Client methods https://review.openstack.org/216109 | 04:38 |
*** ranger81 has joined #openstack-qa | 04:38 | |
oomichi | mkoderer: hello | 04:39 |
*** yfried_ has quit IRC | 04:41 | |
openstackgerrit | garyk proposed openstack-dev/devstack: VMware: remove configuration integration_bridge from nova https://review.openstack.org/221592 | 04:41 |
*** mylu has quit IRC | 04:43 | |
*** fedexo has joined #openstack-qa | 04:43 | |
*** salv-orlando has joined #openstack-qa | 04:55 | |
*** tpeoples has quit IRC | 04:57 | |
*** salv-orlando has quit IRC | 04:58 | |
*** viktor-tikkanen has joined #openstack-qa | 05:05 | |
*** rossella_s has quit IRC | 05:06 | |
*** rossella_s has joined #openstack-qa | 05:06 | |
*** BharatK has joined #openstack-qa | 05:08 | |
*** zhangjn_ has quit IRC | 05:11 | |
gmann | oomichi: mtreinish : got the issue | 05:15 |
gmann | oomichi: actually there is no issue with inherit class stub (raw_request was anyways rest_client.py method) | 05:16 |
gmann | oomichi: actually we changed the auth provider class to use new class name - https://review.openstack.org/#/c/198930/11/tempest_lib/auth.py | 05:16 |
gmann | oomichi: and due to that request start going through TokenClient not TokenClientJSON and thats why stub did not work | 05:17 |
gmann | stub was done on TokenClientJSON class | 05:17 |
gmann | oomichi: i think we should keep using old class in auth.py and change to new once we remove the old class ? | 05:18 |
gmann | what you say? | 05:18 |
*** gamado has quit IRC | 05:25 | |
*** dimtruck is now known as zz_dimtruck | 05:26 | |
*** ranger81 has quit IRC | 05:29 | |
*** avico has quit IRC | 05:29 | |
*** avico has joined #openstack-qa | 05:32 | |
*** BharatK has quit IRC | 05:34 | |
*** yfried_ has joined #openstack-qa | 05:35 | |
*** ekarlso- has joined #openstack-qa | 05:37 | |
*** salv-orlando has joined #openstack-qa | 05:38 | |
openstackgerrit | Hirofumi Ichihara proposed openstack-dev/devstack: Provide hook for neutron plugin config https://review.openstack.org/221146 | 05:38 |
*** yfried__ has joined #openstack-qa | 05:38 | |
*** fedexo has quit IRC | 05:39 | |
*** yfried_ has quit IRC | 05:39 | |
oomichi | gmann: thanks for your investitaion! | 05:40 |
oomichi | gmann: that is great input for me | 05:40 |
oomichi | gmann: I feel current unit test way seems bad | 05:41 |
*** BharatK has joined #openstack-qa | 05:41 | |
oomichi | gmann: because current unit test way depends on internal implementation of tempest-lib | 05:41 |
gmann | oomichi: yea | 05:41 |
openstackgerrit | Jamie Lennox proposed openstack/tempest: Give access to CredClient in tests https://review.openstack.org/214987 | 05:41 |
oomichi | gmann: we need to mock the surface of using library interface directly without depending on internal implementation | 05:42 |
gmann | oomichi: and I agree to break users if anything doing this kind of stub. (as earlier mock release broke many of project :)) | 05:42 |
gmann | oomichi: +2 | 05:42 |
*** yfried__ is now known as yfried|afk | 05:42 | |
*** ranger81 has joined #openstack-qa | 05:42 | |
*** mylu has joined #openstack-qa | 05:44 | |
oomichi | andreaf: hello | 05:46 |
oomichi | mkoderer: andreaf: can you take a look at https://review.openstack.org/#/c/221588/ ? | 05:47 |
*** ramineni_ has joined #openstack-qa | 05:49 | |
*** mylu has quit IRC | 05:49 | |
*** ekarlso- has quit IRC | 05:50 | |
*** yfried|afk is now known as yfried__ | 05:51 | |
*** itzikb has joined #openstack-qa | 05:53 | |
*** vgridnev has joined #openstack-qa | 05:58 | |
openstackgerrit | Wei Jiangang proposed openstack-dev/devstack: Fix repeatedly enter the same string (fake-service) https://review.openstack.org/221598 | 06:00 |
mkoderer | oomichi: good morning | 06:06 |
oomichi | mkoderer: yeah, Guten Morgen :) | 06:07 |
mkoderer | oomichi: oh... sry have no idea how to say good morning in japanese :) | 06:08 |
oomichi | mkoderer: no problem :) "O Ha Yo" works for japanese. it is useful on Tokyo Summit also | 06:09 |
mkoderer | oomichi: cool.. unfortunately I can't make it to the summit this time | 06:09 |
oomichi | mkoderer: oops, that is bad news :( | 06:10 |
oomichi | mkoderer: maybe you are busy for many data centers, I guess | 06:11 |
mkoderer | oomichi: +A | 06:11 |
oomichi | mkoderer: Danke! | 06:12 |
mkoderer | oomichi: no acutally good news for me... my child will be born in the next weeks :) | 06:12 |
*** neeti has joined #openstack-qa | 06:12 | |
oomichi | mkoderer: oh!! super congress in advance! | 06:12 |
oomichi | mkoderer: that will be great news :) | 06:12 |
mkoderer | oomichi: thanks :) | 06:13 |
oomichi | mkoderer: is she/he your first baby ? | 06:13 |
mkoderer | oomichi: yes :) | 06:14 |
oomichi | mkoderer: you will be very busy after the birthday and it is hard to get enough time to sleep and review | 06:14 |
oomichi | mkoderer: anyway, I am looking forward to your good news | 06:15 |
*** SridharG has left #openstack-qa | 06:16 | |
mkoderer | oomichi: yeah I guess I won't be available much for reviews during that time | 06:16 |
oomichi | mkoderer: no problem, I also could not take enough time when 3rd baby born 5 months ago. | 06:17 |
mkoderer | oomichi: 3rd baby.. cool.. I will contact you if I need an advice :) | 06:18 |
oomichi | mkoderer: nice to take care for your family at the time :) | 06:18 |
oomichi | mkoderer: ^^^ is really important for us(husband) | 06:18 |
openstackgerrit | Merged openstack/tempest: Switch using token_client to the latest for gate https://review.openstack.org/221588 | 06:20 |
*** yfried__ is now known as yfried|afk | 06:21 | |
*** yog_ has joined #openstack-qa | 06:22 | |
mkoderer | oomichi: if you have a sec for a review: https://review.openstack.org/#/c/217675/ | 06:26 |
mkoderer | I would like to have have some unit testing for the tempest plugins | 06:27 |
*** Sukhdev has quit IRC | 06:28 | |
oomichi | mkoderer: sure, that is necessary. will see it | 06:28 |
*** mpaolino has joined #openstack-qa | 06:31 | |
*** sshnaidm has joined #openstack-qa | 06:38 | |
*** yfried|afk is now known as yfried__ | 06:41 | |
*** ttx has quit IRC | 06:48 | |
*** ttx has joined #openstack-qa | 06:48 | |
*** Poornima has quit IRC | 06:51 | |
*** shausy has joined #openstack-qa | 06:55 | |
*** shardy_ has joined #openstack-qa | 06:56 | |
*** shardy has quit IRC | 06:57 | |
*** b10n1k__ has quit IRC | 06:59 | |
*** jaypipes has quit IRC | 07:00 | |
*** shardy_ has quit IRC | 07:01 | |
*** shardy has joined #openstack-qa | 07:02 | |
*** armax has quit IRC | 07:02 | |
*** Poornima has joined #openstack-qa | 07:09 | |
openstackgerrit | Merged openstack/tempest: Add docs section on config options to reviewers guide https://review.openstack.org/221330 | 07:12 |
*** coolsvap is now known as coolsvap|brb | 07:14 | |
*** armax has joined #openstack-qa | 07:18 | |
*** gamado has joined #openstack-qa | 07:19 | |
*** safchain has joined #openstack-qa | 07:20 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Return complete resp from volumes_extensions_client https://review.openstack.org/216164 | 07:23 |
*** SridharG has joined #openstack-qa | 07:25 | |
*** anshul has joined #openstack-qa | 07:29 | |
openstackgerrit | Wei Jiangang proposed openstack-dev/devstack: Fix repeatedly enter the same string (fake-service) https://review.openstack.org/221598 | 07:32 |
*** achanda has quit IRC | 07:32 | |
*** SridharG has quit IRC | 07:36 | |
*** achanda has joined #openstack-qa | 07:36 | |
*** jlanoux has joined #openstack-qa | 07:40 | |
*** yfried__ is now known as yfried|afk | 07:44 | |
*** gszasz has joined #openstack-qa | 07:44 | |
*** gfidente has joined #openstack-qa | 07:45 | |
*** shardy_ has joined #openstack-qa | 07:47 | |
*** shardy has quit IRC | 07:49 | |
*** salv-orlando has quit IRC | 07:51 | |
*** SridharG has joined #openstack-qa | 07:51 | |
*** shardy_ has quit IRC | 07:53 | |
*** tkammer|bot has joined #openstack-qa | 07:53 | |
*** jordanP has joined #openstack-qa | 07:53 | |
*** shardy has joined #openstack-qa | 07:53 | |
*** achanda has quit IRC | 08:00 | |
*** vgridnev has quit IRC | 08:01 | |
*** k4n0 has joined #openstack-qa | 08:02 | |
*** yfried|afk is now known as yfried__ | 08:02 | |
*** jordanP has quit IRC | 08:02 | |
*** jordanP has joined #openstack-qa | 08:02 | |
*** achanda has joined #openstack-qa | 08:02 | |
*** cazino has joined #openstack-qa | 08:03 | |
*** cazino has left #openstack-qa | 08:03 | |
*** cdent has joined #openstack-qa | 08:05 | |
*** achanda has quit IRC | 08:07 | |
openstackgerrit | MD NADEEM proposed openstack/tempest: Add unit tests for floating_ips_bulk_client https://review.openstack.org/219166 | 08:09 |
*** tkammer|bot has quit IRC | 08:11 | |
openstackgerrit | Merged openstack/tempest: Add unit tests for security_groups_client https://review.openstack.org/219645 | 08:15 |
openstackgerrit | Merged openstack/tempest: Add unit test for floating_ip_pools_client https://review.openstack.org/214073 | 08:15 |
openstackgerrit | Merged openstack/tempest: Added Test Case for BaremetalNodesClient https://review.openstack.org/215038 | 08:16 |
*** jordanP has quit IRC | 08:17 | |
*** yfried__ is now known as yfried|afk | 08:29 | |
*** hichihara has quit IRC | 08:32 | |
openstackgerrit | Chris Dent proposed openstack-dev/devstack: Update lib/ceilometer to reflect script renames https://review.openstack.org/221634 | 08:36 |
*** yfried|afk is now known as yfried__ | 08:36 | |
*** ranger81 has quit IRC | 08:42 | |
*** jordanP has joined #openstack-qa | 08:43 | |
*** mylu has joined #openstack-qa | 08:47 | |
*** mylu has quit IRC | 08:52 | |
*** e0ne has joined #openstack-qa | 08:54 | |
*** salv-orlando has joined #openstack-qa | 08:56 | |
openstackgerrit | Chris Dent proposed openstack-dev/devstack: Update lib/ceilometer to reflect script renames https://review.openstack.org/221634 | 08:58 |
*** oomichi has quit IRC | 09:01 | |
*** e0ne has quit IRC | 09:04 | |
*** SridharG has quit IRC | 09:04 | |
*** rossella_s has quit IRC | 09:06 | |
*** rossella_s has joined #openstack-qa | 09:06 | |
*** e0ne has joined #openstack-qa | 09:07 | |
openstackgerrit | Itzik Brown proposed openstack/tempest: qos basic scenario https://review.openstack.org/217177 | 09:12 |
openstackgerrit | Anusha Ramineni proposed openstack/tempest: Add unit tests for volumes_extensions_client https://review.openstack.org/216618 | 09:13 |
*** Kennan_Vacation2 has quit IRC | 09:17 | |
openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Return Rules not sec grp from list_security_group_rules https://review.openstack.org/221660 | 09:22 |
*** Kennan_Vacation has joined #openstack-qa | 09:25 | |
openstackgerrit | Takashi NATSUME proposed openstack/tempest: Fix typos in scenario tests, etc. https://review.openstack.org/221139 | 09:26 |
*** yfried__ is now known as yfried|afk | 09:29 | |
cdent | jordanP: have you got devstack approval powers? | 09:30 |
jordanP | cdent, nope | 09:33 |
cdent | no worries, just hoping :) | 09:33 |
jordanP | yeah I know you cant to unblock ceilo :) | 09:33 |
cdent | people will wake up soon enough | 09:33 |
jordanP | *you want | 09:34 |
*** vgridnev has joined #openstack-qa | 09:34 | |
jordanP | chmouel, should be in Europe | 09:35 |
jordanP | but 11h35 is a bit early for him :) | 09:35 |
openstackgerrit | Merged openstack/tempest: Fix identity new endpoint_type options for old users https://review.openstack.org/219169 | 09:35 |
openstackgerrit | Merged openstack/tempest: Cleanup API version schema https://review.openstack.org/220391 | 09:35 |
*** dtantsur|afk is now known as dtantsur | 09:36 | |
*** tosky has joined #openstack-qa | 09:41 | |
*** coolsvap|brb is now known as coolsvap | 09:42 | |
cdent | jordanP: any time before noon is too early for me | 09:44 |
*** nadya has joined #openstack-qa | 09:44 | |
*** amotoki has quit IRC | 09:46 | |
jordanP | cdent, haha :) | 09:48 |
*** yfried|afk is now known as yfried__ | 09:49 | |
*** jffische1 has joined #openstack-qa | 09:50 | |
*** jffische has quit IRC | 09:51 | |
*** SridharG has joined #openstack-qa | 09:53 | |
*** SridharG has left #openstack-qa | 09:53 | |
*** SridharG has joined #openstack-qa | 09:54 | |
*** yamamoto has quit IRC | 09:55 | |
*** yfried__ is now known as yfried|afk | 09:59 | |
*** nadya has quit IRC | 10:02 | |
*** dims__ has joined #openstack-qa | 10:02 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default https://review.openstack.org/221300 | 10:06 |
*** apevec has joined #openstack-qa | 10:24 | |
*** apevec has quit IRC | 10:24 | |
*** apevec has joined #openstack-qa | 10:24 | |
*** yfried|afk is now known as yfried__ | 10:25 | |
*** mylu has joined #openstack-qa | 10:32 | |
*** mylu has quit IRC | 10:36 | |
*** SridharG1 has joined #openstack-qa | 10:40 | |
*** nadya has joined #openstack-qa | 10:41 | |
*** SridharG1 has left #openstack-qa | 10:41 | |
openstackgerrit | Vladyslav Drok proposed openstack/tempest: [WIP] Skip unnecessary tests when using Ironic virt driver https://review.openstack.org/220223 | 10:41 |
*** pc_m has joined #openstack-qa | 10:42 | |
*** SridharG has quit IRC | 10:42 | |
*** SridharG has joined #openstack-qa | 10:43 | |
*** mpaolino has quit IRC | 10:47 | |
*** ranger81 has joined #openstack-qa | 10:53 | |
*** asselin_ has quit IRC | 10:59 | |
*** sayali has quit IRC | 11:00 | |
*** ranger81 has quit IRC | 11:00 | |
*** mwagner_lap has quit IRC | 11:02 | |
*** sayali has joined #openstack-qa | 11:03 | |
*** vgridnev has quit IRC | 11:04 | |
*** aix has quit IRC | 11:06 | |
*** vgridnev has joined #openstack-qa | 11:07 | |
*** jswarren has joined #openstack-qa | 11:12 | |
openstackgerrit | nayna-patel proposed openstack/tempest: Adds 'list-cells' and 'cell-capacities' tests https://review.openstack.org/220932 | 11:13 |
piyush_ | https://review.openstack.org/217555/ if possible please review the same | 11:14 |
*** e0ne has quit IRC | 11:16 | |
*** yfried__ is now known as yfried|afk | 11:22 | |
*** ramineni_ has left #openstack-qa | 11:24 | |
*** vgridnev has quit IRC | 11:24 | |
*** yfried|afk is now known as yfried__ | 11:24 | |
*** jhenner has joined #openstack-qa | 11:25 | |
*** vgridnev has joined #openstack-qa | 11:25 | |
*** jffische1 has quit IRC | 11:26 | |
*** zz_dimtruck is now known as dimtruck | 11:28 | |
*** nadya has quit IRC | 11:29 | |
openstackgerrit | Merged openstack-dev/devstack: Update lib/ceilometer to reflect script renames https://review.openstack.org/221634 | 11:35 |
*** jhenner has quit IRC | 11:41 | |
*** dave-mccowan has joined #openstack-qa | 11:44 | |
*** jhenner has joined #openstack-qa | 11:45 | |
*** dtantsur is now known as dtantsur|brb | 11:49 | |
openstackgerrit | Itzik Brown proposed openstack/tempest: qos basic scenario https://review.openstack.org/217177 | 11:50 |
*** nadya has joined #openstack-qa | 11:50 | |
*** tochi has quit IRC | 11:51 | |
*** viktor-tikkanen has quit IRC | 11:52 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default https://review.openstack.org/221300 | 11:53 |
*** dkranz has joined #openstack-qa | 11:53 | |
*** SridharG has left #openstack-qa | 11:57 | |
*** e0ne has joined #openstack-qa | 12:00 | |
*** aysyd has joined #openstack-qa | 12:03 | |
*** woodster_ has joined #openstack-qa | 12:06 | |
*** rfolco has joined #openstack-qa | 12:06 | |
*** mpaolino has joined #openstack-qa | 12:10 | |
*** dimtruck is now known as zz_dimtruck | 12:12 | |
*** krtaylor has joined #openstack-qa | 12:14 | |
*** e0ne has quit IRC | 12:15 | |
openstackgerrit | Merged openstack-dev/devstack: Add aodh and gnocchi to the plugin registry https://review.openstack.org/217277 | 12:22 |
*** zz_dimtruck is now known as dimtruck | 12:23 | |
*** dimtruck is now known as zz_dimtruck | 12:23 | |
*** coolsvap has quit IRC | 12:27 | |
*** armax has quit IRC | 12:29 | |
*** e0ne has joined #openstack-qa | 12:30 | |
*** coolsvap has joined #openstack-qa | 12:30 | |
*** nadya has quit IRC | 12:31 | |
*** dkranz has quit IRC | 12:32 | |
*** shausy has quit IRC | 12:33 | |
*** shausy has joined #openstack-qa | 12:33 | |
*** edmondsw has joined #openstack-qa | 12:33 | |
*** tpeoples has joined #openstack-qa | 12:35 | |
*** mwagner_lap has joined #openstack-qa | 12:36 | |
*** shuquan has joined #openstack-qa | 12:36 | |
*** amotoki has joined #openstack-qa | 12:37 | |
*** BharatK has quit IRC | 12:39 | |
*** gamado has quit IRC | 12:44 | |
*** regXboi has joined #openstack-qa | 12:47 | |
*** shardy_ has joined #openstack-qa | 12:50 | |
*** sabeen1 has joined #openstack-qa | 12:50 | |
*** aix has joined #openstack-qa | 12:51 | |
*** weshay_xchat has joined #openstack-qa | 12:51 | |
*** shardy has quit IRC | 12:51 | |
*** jffische has joined #openstack-qa | 12:52 | |
*** weshay_xchat has quit IRC | 12:52 | |
*** frayedknot has joined #openstack-qa | 12:52 | |
*** dims__ is now known as dims | 12:52 | |
*** Poornima has quit IRC | 12:55 | |
*** shardy_ has quit IRC | 12:55 | |
mtreinish | gmann: ah ok, yeah that's a bad way to stub it. It's honestly probably a hold over from when things lived in tempest before the lib migration | 12:55 |
mtreinish | gmann: you're we should fix that to only do it on the surface of the interface | 12:56 |
*** shardy has joined #openstack-qa | 12:56 | |
*** ctrath has joined #openstack-qa | 12:57 | |
*** shuquan has quit IRC | 12:58 | |
*** bknudson has joined #openstack-qa | 12:59 | |
*** VanL has joined #openstack-qa | 12:59 | |
*** jswarren has quit IRC | 13:03 | |
*** ctrath has quit IRC | 13:04 | |
*** yfried__ has quit IRC | 13:04 | |
*** dustins has joined #openstack-qa | 13:07 | |
*** armax has joined #openstack-qa | 13:10 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add DB API methods to deal with test metadata https://review.openstack.org/221478 | 13:11 |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add option to specify global test attr prefix in subunit2sql https://review.openstack.org/221479 | 13:12 |
*** jecarey has joined #openstack-qa | 13:13 | |
*** lexholden has joined #openstack-qa | 13:16 | |
mtreinish | andreaf: ^^^ that's my idea for dealing with the idempotent ids from tempest | 13:17 |
mtreinish | hogepodge: ^^^ you might be interested in that too | 13:17 |
*** jswarren has joined #openstack-qa | 13:18 | |
*** zz_dimtruck is now known as dimtruck | 13:19 | |
*** tosky has quit IRC | 13:21 | |
*** lexholden has quit IRC | 13:23 | |
mkoderer | mtreinish: did you used "tox -ecover" latley? it seems it produces nothing useful... | 13:24 |
*** nadya has joined #openstack-qa | 13:24 | |
mtreinish | mkoderer: on tempest? it's been a while since I tried it there | 13:25 |
* mtreinish tries | 13:26 | |
*** hop has joined #openstack-qa | 13:26 | |
*** mriedem_away is now known as mriedem | 13:29 | |
mtreinish | mkoderer: it seems to generate a report fine | 13:31 |
mtreinish | heh, but the unit test coverage reports for tempest were always of questionable value | 13:31 |
mtreinish | since most of tempest is self validating because it's test code | 13:31 |
mkoderer | mtreinish: yeah I mean I don't get any coverage for the clients although we have unit test for them | 13:35 |
mtreinish | mkoderer: hmm, I see coverage reported for the service clients that have tests | 13:37 |
mkoderer | mtreinish: e.g. the coverage of agents_client is only 30% in my env... | 13:37 |
mtreinish | oh, I actually just found a unit test bug because of that | 13:37 |
mkoderer | mtreinish: hehe :) | 13:37 |
hop | mtreinish: hi, could you please review https://review.openstack.org/#/c/189687/ and https://review.openstack.org/#/c/189795/ ? | 13:38 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Use proper method for nova disable service unit test https://review.openstack.org/221757 | 13:39 |
mtreinish | mkoderer: ^^^ | 13:39 |
mtreinish | mkoderer: oh, that's weird my coverage report shows 93% on tempest/services/compute/json/agents_client | 13:39 |
*** lexholden has joined #openstack-qa | 13:40 | |
*** dtantsur|brb is now known as dtantsur | 13:40 | |
mkoderer | mtreinish: mh seems it runs for you env | 13:40 |
mtreinish | the only line not executed in the report is L31 because we never run it with params | 13:40 |
mtreinish | mkoderer: I had to wipe all the old coverage stuff | 13:40 |
mtreinish | and ran tox with -r | 13:40 |
mtreinish | I had weird results before I did that | 13:40 |
mkoderer | mtreinish: let me try this one | 13:41 |
*** ctrath has joined #openstack-qa | 13:43 | |
openstackgerrit | John Warren proposed openstack/tempest: Add unit test for floating_ips_client https://review.openstack.org/217289 | 13:44 |
mkoderer | mtreinish: thanks for fixing that - I felt like a code monkey after fixing all that code duplication | 13:44 |
mtreinish | mkoderer: heh, sure np | 13:52 |
mtreinish | I'm more surprised that I got value from that coverage report and found the issue :) | 13:52 |
mtreinish | mkoderer: oh actually even after my patch it's still not right | 13:54 |
mkoderer | mtreinish: yep, you need to change self.client.enable_service to self.client.disable_service in _test_disable_service too | 13:55 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Use proper method for nova disable service unit test https://review.openstack.org/221757 | 13:55 |
mtreinish | mkoderer: ^^^ yep | 13:55 |
*** dkranz has joined #openstack-qa | 13:56 | |
mkoderer | mtreinish: ok seems coverage works for me now... "-r" fixed the issue.... really strange | 13:57 |
*** bknudson has quit IRC | 13:57 | |
mtreinish | mkoderer: stale venvs always cause issues :) | 13:58 |
*** lexholden has quit IRC | 14:03 | |
*** lexholden has joined #openstack-qa | 14:07 | |
*** salv-orlando has quit IRC | 14:07 | |
*** mmedvede has quit IRC | 14:09 | |
vgridnev | hey folks! please review the following change: https://review.openstack.org/#/c/214194/ . I already have a bunch of +1's | 14:12 |
*** sabeen1 has quit IRC | 14:12 | |
*** hop has quit IRC | 14:16 | |
*** jaypipes has joined #openstack-qa | 14:16 | |
*** ChanServ sets mode: +o jaypipes | 14:16 | |
*** gszasz has quit IRC | 14:20 | |
*** pglass has joined #openstack-qa | 14:21 | |
*** neeti has quit IRC | 14:22 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default https://review.openstack.org/221300 | 14:23 |
openstackgerrit | Jordan Pittier proposed openstack-dev/devstack: Tempest: add a Cinder extend_with_snapshot feature flag https://review.openstack.org/213638 | 14:24 |
*** malini has joined #openstack-qa | 14:27 | |
*** yamahata has joined #openstack-qa | 14:31 | |
*** VanL_ has joined #openstack-qa | 14:31 | |
*** VanL has quit IRC | 14:35 | |
*** yamahata has quit IRC | 14:35 | |
*** yamahata has joined #openstack-qa | 14:36 | |
*** ajmiller has joined #openstack-qa | 14:36 | |
*** bknudson has joined #openstack-qa | 14:39 | |
*** mmedvede has joined #openstack-qa | 14:39 | |
*** k4n0 has quit IRC | 14:40 | |
*** dwalleck has joined #openstack-qa | 14:40 | |
openstackgerrit | Daryl Walleck proposed openstack/tempest-lib: Provide alternative ssh exec_command for non-linux environments https://review.openstack.org/212691 | 14:46 |
*** malini1 has joined #openstack-qa | 14:48 | |
*** malini has quit IRC | 14:52 | |
hogepodge | mtreinish: thanks | 14:52 |
*** jsuchome has joined #openstack-qa | 14:53 | |
*** armax has quit IRC | 14:54 | |
sdague | dtroyer: are you about? Because I'd really like to get the permissions on the grenade launchpad project sorted, it's in a weird state | 14:56 |
jordanP | mhorban, hi | 14:56 |
jordanP | mhorban, a moment for https://review.openstack.org/#/c/153005 ? | 14:57 |
dtroyer | sdague: yup, lemme pull it up | 14:57 |
*** dwalleck has quit IRC | 14:57 | |
*** nadya has quit IRC | 14:58 | |
*** dwalleck has joined #openstack-qa | 14:59 | |
dtroyer | looks like the standard is a -drivers team? we only have a grenade-core ATM, could just use that | 15:02 |
*** dave-mccowan has quit IRC | 15:02 | |
*** tpeoples has quit IRC | 15:03 | |
sdague | dtroyer: yeh, and I think you are supposed to make it owned by openstack-admins | 15:05 |
*** dwalleck has quit IRC | 15:05 | |
sdague | so that dhellmann and ttx can fix things if they go wrong | 15:06 |
dtroyer | I see OpenStack release team as members in other projects. | 15:06 |
dtroyer | If I can find where I'll create Grenade Drivers so it's consistent | 15:06 |
dtroyer | ok, seeded it with you me and mtreinish, do you want administrator in the drivers group? | 15:09 |
*** dwalleck has joined #openstack-qa | 15:10 | |
dtroyer | sdague: ok, I think I've got it, take a look | 15:12 |
*** tpeoples has joined #openstack-qa | 15:12 | |
jsuchome | hi, I wanted to disable one test from running (scenario/test_network_basic_ops/test_subnet_details) by setting empty value for dhcp_client in /etc/tempest/tempest.conf), but when I do it | 15:12 |
jsuchome | tempest fails with "Non-zero exit code (2) from test listing" | 15:12 |
*** erlon has joined #openstack-qa | 15:13 | |
jsuchome | what's wrong? only difference is that value change in tempest.conf | 15:13 |
mtreinish | jsuchome: looking at the code the dhcp_client config option has only 2 allowed values "udhcpc" or "dhclient" (its the dhcp client installed in the guest image) | 15:14 |
mtreinish | by setting it to none that'll cause an exception to be raised since it's not an allowed value | 15:14 |
mtreinish | you should have a traceback somewhere showing that | 15:15 |
*** dave-mccowan has joined #openstack-qa | 15:16 | |
jsuchome | mtreinish: no I think the test should be skipped based on https://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_network_basic_ops.py#n511 | 15:16 |
mtreinish | jsuchome: that's not relevent because during import the config loading will raise an exception based on the option definition: https://github.com/openstack/tempest/blob/master/tempest/config.py#L1037 | 15:17 |
jsuchome | and the problem is that after that config file change, no test is actually executed, it ends early with http://pastebin.com/LWxXbGd5 | 15:18 |
mtreinish | you asked why things were failing during the test list, that's why | 15:18 |
mtreinish | if you update your testtools, subunit, and testrepository python packages it'll show you a traceback as to why test discovery failed | 15:18 |
mtreinish | and you'll see a traceback from oslo config saying that the option didn't have a valid choice set | 15:19 |
*** sshnaidm has quit IRC | 15:19 | |
jsuchome | mtreinish: but the smae config.py says "'If left empty, update operation will be skipped. ' | 15:19 |
mtreinish | jsuchome: it's wrong | 15:20 |
*** vgridnev has quit IRC | 15:20 | |
mtreinish | which is clear based on what you're seeing | 15:20 |
mtreinish | i's a bug that needs to be fixed | 15:20 |
mtreinish | jsuchome: file one to update that help message | 15:20 |
jsuchome | so updating value of 'choices' could help? | 15:20 |
*** salv-orlando has joined #openstack-qa | 15:21 | |
jsuchome | mtreinish: thanks, I'll file a bug (or prepare pull request), but is there currently a correct way to skip that test? | 15:22 |
mtreinish | jsuchome: just use the test runner to not run it. With testr you have to use a negative lookahead regex (there are examples in the tempest tox.ini) | 15:23 |
mtreinish | jsuchome: just curious what specifically is your issue with the test? | 15:24 |
mtreinish | is it the guest env doesn't have a supported dhcp client? | 15:24 |
mtreinish | or you just don't want to be running it | 15:24 |
*** ccneill has joined #openstack-qa | 15:25 | |
*** gszasz has joined #openstack-qa | 15:25 | |
jsuchome | mtreinish: I have docker based instance and that doesn't seem to have supported dhcp client | 15:25 |
*** gszasz has quit IRC | 15:26 | |
mtreinish | jsuchome: yeah, that's why was opposed to that test when it was proposed | 15:27 |
*** tosky has joined #openstack-qa | 15:27 | |
mtreinish | it's very env specific which makes it less useful | 15:27 |
openstackgerrit | Vladyslav Drok proposed openstack/tempest: [WIP] Skip unnecessary tests when using Ironic virt driver https://review.openstack.org/220223 | 15:27 |
*** gszasz has joined #openstack-qa | 15:27 | |
*** hemnafk is now known as hemna | 15:28 | |
*** vgridnev has joined #openstack-qa | 15:28 | |
*** vgridnev has quit IRC | 15:30 | |
*** armax has joined #openstack-qa | 15:33 | |
*** dkranz has quit IRC | 15:33 | |
*** vgridnev has joined #openstack-qa | 15:35 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default https://review.openstack.org/221300 | 15:36 |
*** Swami has joined #openstack-qa | 15:36 | |
*** salv-orl_ has joined #openstack-qa | 15:40 | |
*** VanL_ has quit IRC | 15:41 | |
*** vgridnev has quit IRC | 15:44 | |
*** itzikb has quit IRC | 15:44 | |
*** salv-orlando has quit IRC | 15:44 | |
*** asselin_ has joined #openstack-qa | 15:44 | |
*** VanL has joined #openstack-qa | 15:45 | |
*** mlavalle has joined #openstack-qa | 15:45 | |
*** dkranz has joined #openstack-qa | 15:46 | |
*** dspano has joined #openstack-qa | 15:47 | |
*** prnk28 has joined #openstack-qa | 15:48 | |
*** maurosr has quit IRC | 15:50 | |
*** ranger81 has joined #openstack-qa | 15:51 | |
*** anshul has quit IRC | 15:52 | |
*** Sukhdev has joined #openstack-qa | 15:55 | |
*** salv-orl_ has quit IRC | 15:57 | |
*** salv-orlando has joined #openstack-qa | 15:58 | |
*** nadya has joined #openstack-qa | 15:58 | |
*** yog_ has quit IRC | 16:00 | |
*** yunpengli has joined #openstack-qa | 16:01 | |
*** nadya has quit IRC | 16:03 | |
*** dwalleck has quit IRC | 16:04 | |
*** safchain has quit IRC | 16:06 | |
*** jlanoux has quit IRC | 16:07 | |
*** jsuchome has left #openstack-qa | 16:08 | |
*** dims has quit IRC | 16:10 | |
*** salv-orl_ has joined #openstack-qa | 16:10 | |
*** dims has joined #openstack-qa | 16:10 | |
*** salv-orlando has quit IRC | 16:14 | |
jordanP | mkoderer, ping ? | 16:20 |
*** nic has joined #openstack-qa | 16:21 | |
*** nadya has joined #openstack-qa | 16:25 | |
*** dtantsur is now known as dtantsur|afk | 16:25 | |
*** salv-orl_ has quit IRC | 16:33 | |
*** jordanP has quit IRC | 16:34 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Add possibility to skip test_subnet_details if DHCP client not available https://review.openstack.org/221874 | 16:36 |
*** malini1 has quit IRC | 16:37 | |
*** vgridnev has joined #openstack-qa | 16:40 | |
*** malini has joined #openstack-qa | 16:40 | |
*** amotoki has quit IRC | 16:43 | |
openstackgerrit | Chris Dent proposed openstack-dev/devstack: Remove ceilometer in favor of plugin https://review.openstack.org/196383 | 16:44 |
*** VanL_ has joined #openstack-qa | 16:47 | |
*** shausy has quit IRC | 16:49 | |
*** VanL has quit IRC | 16:51 | |
*** cdent has quit IRC | 16:53 | |
*** achanda has joined #openstack-qa | 16:53 | |
openstackgerrit | Arkadiy Kraminsky proposed openstack/tempest-lib: Add in a more verbose exception message https://review.openstack.org/221449 | 16:56 |
openstackgerrit | Arkadiy Kraminsky proposed openstack/tempest-lib: Add in a more verbose exception message https://review.openstack.org/221449 | 16:57 |
*** dwalleck has joined #openstack-qa | 16:59 | |
*** malini has quit IRC | 17:00 | |
*** dzamboni has joined #openstack-qa | 17:02 | |
*** lexholden has quit IRC | 17:05 | |
*** rossella_s has quit IRC | 17:06 | |
*** rossella_s has joined #openstack-qa | 17:06 | |
*** dkranz has quit IRC | 17:06 | |
*** e0ne has quit IRC | 17:08 | |
*** tosky has quit IRC | 17:08 | |
mtreinish | andreaf: qq, does the __str__ method on credentials object print the password? | 17:10 |
andreaf | mtreinish: yes, I think it will print it | 17:11 |
andreaf | mtreinish: I guess it could be made optional? | 17:11 |
andreaf | mtreinish: security concerns about pwd in logs? | 17:12 |
mtreinish | andreaf: yeah | 17:12 |
mtreinish | that's something we shouldn't normally be printing | 17:12 |
mtreinish | andreaf: I was thinking about it for this patch: https://review.openstack.org/#/c/221449/7 | 17:14 |
mtreinish | we're hitting that exception on some of the experimental/periodic jobs right now and I can't figure out why | 17:14 |
mtreinish | I expect it's a configuration issue, but I'm not sure what | 17:15 |
*** yamahata has quit IRC | 17:16 | |
*** dwalleck has quit IRC | 17:16 | |
*** malini has joined #openstack-qa | 17:18 | |
*** yunpengli has quit IRC | 17:19 | |
*** salv-orlando has joined #openstack-qa | 17:19 | |
*** salv-orl_ has joined #openstack-qa | 17:21 | |
*** dwalleck has joined #openstack-qa | 17:21 | |
*** dkranz has joined #openstack-qa | 17:22 | |
*** VanL_ has quit IRC | 17:22 | |
*** salv-orlando has quit IRC | 17:23 | |
*** Sukhdev has quit IRC | 17:30 | |
*** ranger81 has quit IRC | 17:31 | |
*** ranger81 has joined #openstack-qa | 17:32 | |
*** achanda has quit IRC | 17:34 | |
*** salv-orlando has joined #openstack-qa | 17:35 | |
*** ccneill has quit IRC | 17:38 | |
*** salv-orl_ has quit IRC | 17:38 | |
*** shardy_ has joined #openstack-qa | 17:38 | |
*** achanda has joined #openstack-qa | 17:38 | |
*** shardy has quit IRC | 17:40 | |
*** dwalleck has quit IRC | 17:42 | |
*** VanL has joined #openstack-qa | 17:43 | |
*** shardy_ has quit IRC | 17:43 | |
*** shardy has joined #openstack-qa | 17:44 | |
*** VanL has quit IRC | 17:47 | |
*** apevec has quit IRC | 17:49 | |
*** ranger81 has quit IRC | 17:49 | |
*** ranger81 has joined #openstack-qa | 17:50 | |
*** pvaneck has joined #openstack-qa | 17:50 | |
*** mpaolino has quit IRC | 17:51 | |
*** tosky has joined #openstack-qa | 17:52 | |
*** yfried__ has joined #openstack-qa | 17:53 | |
*** armax has quit IRC | 17:58 | |
*** mylu has joined #openstack-qa | 17:58 | |
*** rbak has quit IRC | 17:59 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default https://review.openstack.org/221300 | 18:00 |
*** apevec has joined #openstack-qa | 18:03 | |
*** vgridnev has quit IRC | 18:03 | |
*** dkranz has quit IRC | 18:04 | |
openstackgerrit | Roxana Gherle proposed openstack/tempest-lib: Introduce scope in the auth API https://review.openstack.org/202705 | 18:06 |
*** dkranz has joined #openstack-qa | 18:07 | |
*** krtaylor has quit IRC | 18:13 | |
*** rbak has joined #openstack-qa | 18:15 | |
andreaf | mtreinish: I think using print(str()) would be the right approach, we can change __str__ to not include the pwd in clear | 18:16 |
*** vgridnev has joined #openstack-qa | 18:22 | |
*** e0ne has joined #openstack-qa | 18:23 | |
*** nadya has quit IRC | 18:28 | |
*** mylu has quit IRC | 18:29 | |
*** aix has quit IRC | 18:29 | |
*** mylu has joined #openstack-qa | 18:30 | |
mtreinish | andreaf: ok cool, can you leave a review comment about that on the review | 18:30 |
*** mylu has quit IRC | 18:31 | |
*** mylu has joined #openstack-qa | 18:31 | |
*** rbak has quit IRC | 18:32 | |
*** rbak has joined #openstack-qa | 18:32 | |
*** jhesketh has quit IRC | 18:32 | |
*** rbak has quit IRC | 18:32 | |
*** rbak has joined #openstack-qa | 18:34 | |
*** jhesketh has joined #openstack-qa | 18:34 | |
*** nadya has joined #openstack-qa | 18:35 | |
*** hoangcx has joined #openstack-qa | 18:37 | |
*** VanL has joined #openstack-qa | 18:37 | |
*** DinaBelova has quit IRC | 18:40 | |
*** VanL has quit IRC | 18:42 | |
*** vponomaryov has quit IRC | 18:42 | |
*** dwalleck has joined #openstack-qa | 18:43 | |
*** DinaBelova has joined #openstack-qa | 18:43 | |
*** vponomaryov has joined #openstack-qa | 18:43 | |
openstackgerrit | Mauro Rodrigues proposed openstack-dev/devstack: Enables the user to set arch for the baremetal flavor and ironic node https://review.openstack.org/221909 | 18:44 |
*** apevec has quit IRC | 18:44 | |
*** nadya has quit IRC | 18:46 | |
*** e0ne has quit IRC | 18:47 | |
*** dwalleck has quit IRC | 18:47 | |
*** krtaylor has joined #openstack-qa | 18:48 | |
openstackgerrit | Mauro Rodrigues proposed openstack-dev/devstack: Enables the user to set arch for the baremetal flavor and ironic node https://review.openstack.org/221909 | 18:48 |
*** maurosr has joined #openstack-qa | 18:48 | |
*** ccneill has joined #openstack-qa | 18:48 | |
*** tosky has quit IRC | 18:51 | |
*** tosky has joined #openstack-qa | 18:51 | |
openstackgerrit | Mauro Rodrigues proposed openstack-dev/devstack: Enables the user to set arch for the baremetal flavor and ironic node https://review.openstack.org/221912 | 18:51 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default https://review.openstack.org/221300 | 18:52 |
*** Sukhdev has joined #openstack-qa | 18:55 | |
*** yamahata has joined #openstack-qa | 18:57 | |
openstackgerrit | Mauro Rodrigues proposed openstack-dev/devstack: Enables the user to set arch for the baremetal flavor and ironic node https://review.openstack.org/221915 | 18:58 |
*** krtaylor has quit IRC | 19:00 | |
*** ccneill has quit IRC | 19:00 | |
*** krtaylor has joined #openstack-qa | 19:00 | |
*** pglbutt has joined #openstack-qa | 19:04 | |
*** jasonsb has quit IRC | 19:04 | |
*** jasonsb has joined #openstack-qa | 19:05 | |
*** jasonsb has quit IRC | 19:05 | |
*** jasonsb has joined #openstack-qa | 19:06 | |
*** pglass has quit IRC | 19:07 | |
*** e0ne has joined #openstack-qa | 19:07 | |
*** nic has quit IRC | 19:08 | |
*** malini has quit IRC | 19:08 | |
*** boris-42 has quit IRC | 19:10 | |
*** jasonsb has quit IRC | 19:10 | |
*** nadya has joined #openstack-qa | 19:10 | |
mtreinish | mkoderer: just curious did we ever get anywhere with the tempest-plugin-cookiecutter stuff? | 19:11 |
*** achanda has quit IRC | 19:13 | |
*** malini has joined #openstack-qa | 19:13 | |
openstackgerrit | Roxana Gherle proposed openstack/tempest-lib: Introduce scope in the auth API https://review.openstack.org/202705 | 19:16 |
*** nic has joined #openstack-qa | 19:17 | |
mtreinish | dkranz, andreaf: if you get a sec: https://review.openstack.org/#/c/216522/ it'll be good to unblock the periodic test-accounts jobs | 19:17 |
mtreinish | I really want to start the timer on the creds option deprecation before the release | 19:17 |
*** kiran-r has joined #openstack-qa | 19:17 | |
dkranz | mtreinish: ok, it passed pep8 | 19:24 |
*** edmondsw has quit IRC | 19:25 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default https://review.openstack.org/221300 | 19:25 |
mtreinish | dkranz: yeah, it was off by 1 space. It took me a sec to see it | 19:25 |
*** hoangcx has quit IRC | 19:27 | |
*** apevec has joined #openstack-qa | 19:29 | |
*** kiran-r has quit IRC | 19:29 | |
*** grafuls_ has joined #openstack-qa | 19:31 | |
*** e0ne has quit IRC | 19:32 | |
*** jhenner has quit IRC | 19:33 | |
*** aix has joined #openstack-qa | 19:33 | |
*** jhenner has joined #openstack-qa | 19:33 | |
*** grafuls has quit IRC | 19:35 | |
*** dtantsur|afk has quit IRC | 19:35 | |
*** jhenner has quit IRC | 19:38 | |
*** jhenner has joined #openstack-qa | 19:39 | |
*** grafuls_ has quit IRC | 19:39 | |
*** dtantsur has joined #openstack-qa | 19:40 | |
*** jhenner1 has joined #openstack-qa | 19:42 | |
*** jhenner has quit IRC | 19:43 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements https://review.openstack.org/217222 | 19:43 |
*** mylu has quit IRC | 19:45 | |
*** mylu has joined #openstack-qa | 19:46 | |
*** pglass has joined #openstack-qa | 19:46 | |
*** pglass has quit IRC | 19:46 | |
*** pglbutt has quit IRC | 19:49 | |
*** mylu has quit IRC | 19:50 | |
*** nadya has quit IRC | 19:50 | |
*** grafuls_ has joined #openstack-qa | 19:50 | |
*** malini has quit IRC | 19:50 | |
*** tosky has quit IRC | 19:51 | |
*** tosky has joined #openstack-qa | 19:51 | |
*** jhenner has joined #openstack-qa | 19:51 | |
*** jesusaurus has joined #openstack-qa | 19:53 | |
*** jhenner1 has quit IRC | 19:53 | |
*** gszasz has quit IRC | 19:55 | |
akraminsky_ | mtreinish, I looked at your comments and went ahead and changed __str__. just want your opinion before I push it up, this is my change http://paste.openstack.org/show/453131/ | 19:58 |
*** pglass has joined #openstack-qa | 19:59 | |
akraminsky_ | mtreinish, oops switched that not around too | 19:59 |
*** boris-42 has joined #openstack-qa | 20:00 | |
openstackgerrit | Merged openstack/tempest: Use proper method for nova disable service unit test https://review.openstack.org/221757 | 20:00 |
mtreinish | akraminsky_: yep, that looks like the basic idea | 20:02 |
mtreinish | akraminsky_: it probably make senses to add unit tests to that too, just to verify the logic | 20:03 |
mtreinish | we also should probably split the changes into 2 patches, so make the first do the str fix | 20:03 |
mtreinish | and then on top of that change the exception msg fix | 20:03 |
*** jasonsb has joined #openstack-qa | 20:05 | |
*** dkranz has quit IRC | 20:10 | |
*** e0ne has joined #openstack-qa | 20:11 | |
*** achanda has joined #openstack-qa | 20:12 | |
*** b10n1k__ has joined #openstack-qa | 20:16 | |
*** achanda has quit IRC | 20:18 | |
*** b10n1k__ has quit IRC | 20:21 | |
*** tosky has quit IRC | 20:21 | |
*** tosky has joined #openstack-qa | 20:21 | |
*** salv-orlando has quit IRC | 20:22 | |
*** salv-orlando has joined #openstack-qa | 20:22 | |
*** ccneill has joined #openstack-qa | 20:22 | |
*** dkranz has joined #openstack-qa | 20:23 | |
*** dkranz has quit IRC | 20:23 | |
*** achanda has joined #openstack-qa | 20:28 | |
*** b10n1k__ has joined #openstack-qa | 20:33 | |
openstackgerrit | John F. Fischer proposed openstack/tempest: Add unit test for hypervisor_client https://review.openstack.org/217925 | 20:35 |
*** salv-orlando has quit IRC | 20:40 | |
*** salv-orlando has joined #openstack-qa | 20:41 | |
openstackgerrit | Merged openstack-infra/subunit2sql: Add option to subunit2sql for attr regex https://review.openstack.org/221477 | 20:41 |
*** salv-orl_ has joined #openstack-qa | 20:42 | |
mtreinish | sdague, gmann: didn't we remove the nova v21 job definition as part of that nova catalog refactor in devstack? | 20:42 |
*** salv-or__ has joined #openstack-qa | 20:44 | |
*** salv-orl_ has quit IRC | 20:44 | |
*** prnk28 has quit IRC | 20:44 | |
*** salv-orlando has quit IRC | 20:46 | |
sdague | mtreinish: it's in the stack of cleanups | 20:46 |
sdague | it may not be landed yet | 20:46 |
*** ianw has quit IRC | 20:47 | |
mtreinish | sdague: ok, I just had a patch bounce off the job and I was surprised it was still a thing | 20:47 |
*** ianw has joined #openstack-qa | 20:48 | |
*** achanda has quit IRC | 20:49 | |
openstackgerrit | Vitaly Gridnev proposed openstack-dev/devstack: Support BlockDeviceDriver as as cinder backend https://review.openstack.org/214194 | 20:50 |
*** dave-mccowan has quit IRC | 20:51 | |
*** jasonsb has quit IRC | 20:51 | |
*** nikita_09 has quit IRC | 20:51 | |
*** jasonsb has joined #openstack-qa | 20:52 | |
mtreinish | andreaf: so I think I the ci job for test accounts might have found a smallish issue: http://logs.openstack.org/22/216522/5/experimental/gate-tempest-dsvm-full-test-accounts/d800694//console.html#_2015-09-09_20_15_47_757 | 20:53 |
mtreinish | I think thats caused by test accounts pulling an account with a role that has additional permissions so the quota operation being performed there is allowed | 20:53 |
mtreinish | like it's getting swift operator because test accounts treats all roles except for admin as nothing extra (unless it's asked for) and it's allowed to modify quotas | 20:54 |
mtreinish | but the test isn't expecting that | 20:54 |
*** Sukhdev has quit IRC | 20:55 | |
mtreinish | andreaf: err, maybe not that test should be always be using the operator role | 20:57 |
mtreinish | oh nm, I think it's because the users generated for the accounts.yaml always have both operator and reseller_admin assigned together | 20:57 |
mtreinish | instead of being mutually exclusive | 20:58 |
mtreinish | and the test isn't expecting a reseller_admin user to run the quota op | 20:58 |
*** achanda has joined #openstack-qa | 20:59 | |
*** frayedknot has quit IRC | 21:02 | |
*** jhesketh has quit IRC | 21:03 | |
*** bknudson has quit IRC | 21:04 | |
*** krtaylor has quit IRC | 21:05 | |
*** rossella_s has quit IRC | 21:06 | |
*** rossella_s has joined #openstack-qa | 21:06 | |
*** jhesketh has joined #openstack-qa | 21:07 | |
*** dustins has quit IRC | 21:08 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Fix test_port_list_filter_by_ip with test_accounts https://review.openstack.org/216887 | 21:08 |
sdague | mtreinish: https://review.openstack.org/#/c/219553/ - should be fixed when that merges | 21:09 |
mtreinish | sdague: ok, thanks | 21:09 |
*** edmondsw has joined #openstack-qa | 21:10 | |
sdague | mtreinish: so we're running those v2.0 jobs on nova check queue non-voting | 21:10 |
sdague | do you want those in tempest experimental, or tempest check | 21:10 |
mtreinish | sdague: I'd probably prefer experimental/periodic unless you think the difference between v2.1 and v2.0 is significant enough to warrant running it on check | 21:12 |
sdague | mtreinish: well it shouldn't be | 21:12 |
sdague | that's the point | 21:12 |
sdague | oh... before I forget, why are there now 4 emails a day on the qa list? | 21:13 |
*** sabeen1 has joined #openstack-qa | 21:14 | |
mtreinish | sdague: someone added 3 neutron pg jobs to the qa periodic job list, I'm not sure who | 21:14 |
mtreinish | I haven't gotten around to removing them | 21:14 |
mtreinish | because there isn't a dashboard or any other reporting mechanism there really isn't anywhere else to dump the periodic test results to | 21:14 |
*** salv-orlando has joined #openstack-qa | 21:15 | |
mtreinish | sdague: give me a min, I'll push up the patch to remove the jobs | 21:15 |
mtreinish | they've never worked from what I can tell | 21:15 |
lifeless | mtreinish: polleyz put up a dashboard thingy | 21:15 |
lifeless | mtreinish: poc at least | 21:16 |
*** salv-or__ has quit IRC | 21:17 | |
mtreinish | lifeless: I haven't heard about that, do you have a link? | 21:17 |
mtreinish | lifeless: but if he has something he should come to the qa sprint in ft collins on mon. :) | 21:17 |
mtreinish | because I want to get that off the todo list once and for all | 21:18 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Split the operator and reseller users in the account generator https://review.openstack.org/221966 | 21:19 |
mtreinish | tchaypo: ^^^ | 21:21 |
tchaypo | polleyz :) | 21:22 |
tchaypo | I think that would be https://review.openstack.org/#/c/192253/ | 21:23 |
mtreinish | heh, it took me a sec to figure that out | 21:23 |
tchaypo | no wait | 21:23 |
tchaypo | it’d be the other thing | 21:23 |
*** pc_m has quit IRC | 21:23 | |
mtreinish | tchaypo: that's the spec about what I want to get done at the sprint next week | 21:23 |
tchaypo | http://bruce.jamezpolley.com/reports/ | 21:23 |
tchaypo | which is just an implementation of a slight rework of a script that derek originally wrote for tripleo | 21:24 |
mtreinish | tchaypo: ah ok, cool | 21:25 |
mtreinish | tchaypo: fwiw, I put some of my initial thoughts up on: https://etherpad.openstack.org/p/PuDfHylBgI | 21:25 |
mtreinish | of how I thought it would look | 21:26 |
mtreinish | of course I can't draw at all so it's just words poorly describing the pictures in my head | 21:26 |
*** pvaneck_ has joined #openstack-qa | 21:28 | |
*** pvaneck has quit IRC | 21:28 | |
*** sabeen1 has quit IRC | 21:31 | |
*** jasonsb has quit IRC | 21:31 | |
*** dave-mccowan has joined #openstack-qa | 21:32 | |
*** jasonsb has joined #openstack-qa | 21:32 | |
*** dspano has quit IRC | 21:35 | |
*** e0ne has quit IRC | 21:39 | |
*** jffische has quit IRC | 21:40 | |
*** bknudson has joined #openstack-qa | 21:40 | |
*** mestery has quit IRC | 21:41 | |
*** mestery has joined #openstack-qa | 21:42 | |
*** dave-mccowan has quit IRC | 21:42 | |
*** tsekiyama has joined #openstack-qa | 21:42 | |
*** frayedknot has joined #openstack-qa | 21:46 | |
openstackrecheck | openstack/ceilometer change: https://review.openstack.org/221545 failed because of: gate-grenade-dsvm-ceilometer: https://bugs.launchpad.net/bugs/1463631 | 21:49 |
openstack | Launchpad bug 1463631 in grenade "60_nova/resources.sh:106:ping_check_public fails intermittently" [Undecided,Confirmed] | 21:49 |
tchaypo | you mentioned a spring next week | 21:51 |
*** b10n1k__ has quit IRC | 21:53 | |
openstackgerrit | Arkadiy Kraminsky proposed openstack/tempest-lib: Add in a more verbose exception message https://review.openstack.org/221449 | 21:56 |
*** Sukhdev has joined #openstack-qa | 21:57 | |
*** krtaylor has joined #openstack-qa | 22:04 | |
*** vgridnev has quit IRC | 22:07 | |
akraminsky_ | mtreinish, would the unit test for the str method go in test_credentials.py or test_auth.py? | 22:08 |
*** b10n1k__ has joined #openstack-qa | 22:10 | |
*** regXboi has quit IRC | 22:12 | |
*** ctrath has quit IRC | 22:12 | |
*** dzamboni has quit IRC | 22:12 | |
andreaf | mtreinish, jamielennox: I think https://review.openstack.org/#/c/202705 looks ok now, if you have time to review - I added a WIP tempest patch (https://review.openstack.org/#/c/221020/) which depends on this - on the tempest-lib src job you can see this code actually used | 22:16 |
*** pglass has quit IRC | 22:19 | |
jamielennox | andreaf: it looks good to me | 22:22 |
andreaf | jamielennox: thanks | 22:22 |
*** masayukig has quit IRC | 22:24 | |
*** masayukig has joined #openstack-qa | 22:26 | |
*** mwagner_lap has quit IRC | 22:26 | |
ianw | jamielennox: this is correct? -> https://review.openstack.org/#/c/220458/ i'm pretty sure | 22:27 |
jamielennox | ianw: sigh, swift again | 22:27 |
jamielennox | yea, it needs it. i thought the env variables were still versioned but apparently not if this is failing | 22:28 |
*** jecarey has quit IRC | 22:36 | |
*** aysyd has quit IRC | 22:50 | |
openstackgerrit | Merged openstack-dev/devstack: Tempest: add a Cinder extend_with_snapshot feature flag https://review.openstack.org/213638 | 22:50 |
*** ccneill has quit IRC | 22:53 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/200108 failed gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error | 22:55 |
*** hemna is now known as hemnafk | 22:55 | |
*** rbak has quit IRC | 22:55 | |
*** mlavalle has quit IRC | 22:56 | |
mtreinish | tchaypo: yeah: https://wiki.openstack.org/wiki/QA/CodeSprintLibertyFortCollins | 23:05 |
mtreinish | andreaf: ok, I'll take a look in a bit | 23:05 |
mtreinish | more realistically tomorrow | 23:05 |
jamielennox | mtreinish: do you have any idea what i did to screw up: https://review.openstack.org/#/c/214987/9 | 23:05 |
*** bknudson has quit IRC | 23:06 | |
mtreinish | the unit test failures are unrelated, we pushed a new tempest-lib release last night (well your morning) and it broke the tempest unit tests in an unexpected way | 23:06 |
mtreinish | but that's been fixed | 23:06 |
jamielennox | ok, so i'll try a rechec | 23:06 |
jamielennox | k | 23:06 |
mtreinish | and the postgres failure looks like a quota bug, but I haven't seen that one in a while | 23:07 |
*** achanda has quit IRC | 23:08 | |
jamielennox | yea, i've had a few gate issues so far but the 27,34 test failures threw me | 23:08 |
*** erlon has quit IRC | 23:08 | |
jamielennox | it happens locally as well but i couldn't see where i would have changed anything | 23:08 |
mtreinish | jamielennox: yeah it was tempest-lib 0.8.0 we deprecated/renamed a class and made it just a subclass that logged a warning on init | 23:09 |
mtreinish | but doing that some how broke a mox stub causing those tests to actually try and go out on the network | 23:10 |
jamielennox | ergh, mox | 23:10 |
jamielennox | oh, because you use httplib2 and not requests | 23:10 |
mtreinish | yeah, we need to stop using mox there | 23:10 |
mtreinish | also it was stubbing the wrong class it was relying on internals of the tempest-lib rest client | 23:11 |
*** david-lyle has quit IRC | 23:11 | |
jamielennox | i was going to pimp requests-mock mut it's not much good if you're not using requests | 23:11 |
mtreinish | heh | 23:11 |
*** david-lyle has joined #openstack-qa | 23:12 | |
*** achanda has joined #openstack-qa | 23:20 | |
*** david-lyle has quit IRC | 23:22 | |
*** Swami has quit IRC | 23:23 | |
*** edmondsw has quit IRC | 23:23 | |
*** david-lyle has joined #openstack-qa | 23:23 | |
*** b10n1k__ has quit IRC | 23:24 | |
gmann | mtreinish: just merged - for v21 job cleanup - https://review.openstack.org/#/c/219553/ | 23:29 |
mtreinish | gmann: cool, thanks | 23:31 |
openstackgerrit | Arkadiy Kraminsky proposed openstack/tempest: Add volume backed live migration test https://review.openstack.org/165233 | 23:31 |
gmann | mtreinish: on devstack side cleanup - https://review.openstack.org/#/c/219555/ | 23:31 |
gmann | mtreinish: and please look into related one for v20 as experimental on Tempest - https://review.openstack.org/#/c/221608/ | 23:32 |
*** tosky has quit IRC | 23:33 | |
mtreinish | gmann: ok, I'll take a look at those tomorrow | 23:35 |
gmann | mtreinish: sure. Thanks. | 23:35 |
mtreinish | gmann: I'll trade you for: https://review.openstack.org/221966 , https://review.openstack.org/216887 , and https://review.openstack.org/211601 | 23:36 |
mtreinish | which are to fix bugs with test-accounts and start the deprecation timer on the creds config options | 23:36 |
gmann | mtreinish: sure i will look today | 23:37 |
*** bknudson has joined #openstack-qa | 23:37 | |
gmann | mtreinish: can you quickly +A on https://review.openstack.org/#/c/217222/ | 23:37 |
gmann | mtreinish: updating oslo.config version which needed for your patch | 23:37 |
*** apevec has quit IRC | 23:38 | |
*** ranger81 has quit IRC | 23:38 | |
mtreinish | gmann: done | 23:38 |
gmann | mtreinish: Thanks | 23:38 |
*** ranger81 has joined #openstack-qa | 23:38 | |
*** tochi has joined #openstack-qa | 23:39 | |
*** jasonsb has quit IRC | 23:40 | |
mtreinish | akraminsky_: probably test_credentials.py, since you'll be adding unit tests for __str__ on the credentials class | 23:42 |
*** mriedem1 has joined #openstack-qa | 23:44 | |
*** Sukhdev has quit IRC | 23:45 | |
*** Sukhdev has joined #openstack-qa | 23:45 | |
openstackgerrit | Merged openstack-dev/devstack: Fix typos in devstack/stack.sh https://review.openstack.org/221232 | 23:46 |
*** mriedem has quit IRC | 23:47 | |
*** dimtruck is now known as zz_dimtruck | 23:48 | |
*** jasonsb has joined #openstack-qa | 23:48 | |
*** jasonsb has quit IRC | 23:49 | |
*** nic has quit IRC | 23:51 | |
*** Swami has joined #openstack-qa | 23:51 | |
openstackgerrit | Merged openstack/tempest: Only add roles for enabled services https://review.openstack.org/216522 | 23:53 |
*** mmedvede has quit IRC | 23:55 | |
*** ajmiller has quit IRC | 23:56 | |
*** bknudson has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!