*** openstack has joined #openstack-qa | 12:41 | |
-sendak.freenode.net- [freenode-info] channel trolls and no channel staff around to help? please check with freenode support: http://freenode.net/faq.shtml#gettinghelp | 12:41 | |
*** yamahata has quit IRC | 12:43 | |
*** vaidy has quit IRC | 12:47 | |
*** tosky has quit IRC | 12:48 | |
*** mestery has joined #openstack-qa | 12:50 | |
*** beagles has joined #openstack-qa | 12:53 | |
*** beagles is now known as Guest87163 | 12:53 | |
*** mestery has quit IRC | 12:53 | |
*** Guest87163 is now known as b3nt_pin | 13:00 | |
*** dkranz has quit IRC | 13:04 | |
*** mmedvede has joined #openstack-qa | 13:05 | |
*** dustins has joined #openstack-qa | 13:10 | |
*** jecarey has quit IRC | 13:14 | |
*** dkranz has joined #openstack-qa | 13:15 | |
openstackgerrit | A change was merged to openstack/tempest: Drop OfficialClientTest and NetworkTest https://review.openstack.org/124013 | 13:15 |
---|---|---|
*** sponomarenko has quit IRC | 13:17 | |
*** toure has joined #openstack-qa | 13:18 | |
andreaf | mtreinish: good morning - https://review.openstack.org/#/c/124014 has +2 x 2 and no +A, would you mind? | 13:19 |
sdague | andreaf: +A | 13:20 |
andreaf | sdague, thanks | 13:20 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds test for deleting external network with floatingIPs https://review.openstack.org/124722 | 13:23 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds "list external networks" test to non admin user https://review.openstack.org/124625 | 13:24 |
*** andreaf is now known as andreaf_ | 13:26 | |
EmilienM | sdague: if you have time, there is a javelin2 patch to enable volume resources, already +2: https://review.openstack.org/#/c/121648/ | 13:28 |
sdague | EmilienM: +A, thanks | 13:29 |
EmilienM | sdague: cool | 13:29 |
*** mriedem has joined #openstack-qa | 13:29 | |
*** tosky has joined #openstack-qa | 13:29 | |
*** nmagnezi_ has joined #openstack-qa | 13:31 | |
jordanP | guys, any openstack-infra/project-config pro here ? Jenkins -1ed my patch here : https://review.openstack.org/#/c/124729/ but I don't understand why. Any help would be appreciated :) | 13:35 |
*** bknudson has joined #openstack-qa | 13:35 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Use shared values for test_security_group_rules https://review.openstack.org/124002 | 13:38 |
*** gordc has joined #openstack-qa | 13:40 | |
yfried | gmann: here? | 13:41 |
*** sabeen has joined #openstack-qa | 13:42 | |
*** omrim_ has joined #openstack-qa | 13:45 | |
*** Dafna has quit IRC | 13:46 | |
omrim_ | Hi, I would like to create an agent that support SRIOV by adjust the nova + ml2 conf files | 13:47 |
omrim_ | I would like to do it with Q_AGENT | 13:47 |
omrim_ | Is there any procedure to create such an agent? | 13:47 |
*** beekneemech is now known as bnemec | 13:53 | |
*** Dafna has joined #openstack-qa | 13:54 | |
openstackgerrit | Sergey Shnaidman proposed a change to openstack/tempest: Create subnet without gateway and explicit IP ver https://review.openstack.org/118424 | 13:55 |
*** dmorita has joined #openstack-qa | 13:58 | |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds network to large scale scenario https://review.openstack.org/120070 | 13:58 |
*** swamireddy has quit IRC | 13:59 | |
yfried | andreaf_: were can I push updates to the gates? what's the repo? | 14:01 |
*** yamahata has joined #openstack-qa | 14:02 | |
*** lordd has quit IRC | 14:05 | |
*** markmcclain has joined #openstack-qa | 14:06 | |
*** lbragstad has joined #openstack-qa | 14:06 | |
*** jecarey_ has joined #openstack-qa | 14:06 | |
*** rbak has joined #openstack-qa | 14:08 | |
*** markmcclain has quit IRC | 14:12 | |
*** markmcclain has joined #openstack-qa | 14:13 | |
*** rbak has quit IRC | 14:14 | |
*** yamahata has quit IRC | 14:17 | |
*** yamahata has joined #openstack-qa | 14:17 | |
yfried | gmann: ping | 14:19 |
*** atiwari has joined #openstack-qa | 14:20 | |
*** rbak has joined #openstack-qa | 14:22 | |
*** zz_dimtruck is now known as dimtruck | 14:23 | |
openstackgerrit | A change was merged to openstack/tempest: Rename NeutronScenarioTest to NetworkScenarioTest https://review.openstack.org/124014 | 14:23 |
*** jordanP has quit IRC | 14:24 | |
yfried | anyone knows how to change tempest_conf values for a gate in openstack-project repo? | 14:26 |
*** sergsh has joined #openstack-qa | 14:27 | |
*** sergsh is now known as nuclearworm | 14:28 | |
*** jordanP has joined #openstack-qa | 14:30 | |
*** nuclearworm has left #openstack-qa | 14:31 | |
*** sergsh has joined #openstack-qa | 14:31 | |
*** aix has quit IRC | 14:31 | |
*** rwsu has joined #openstack-qa | 14:33 | |
*** naggappan has quit IRC | 14:33 | |
*** k4n0 has quit IRC | 14:34 | |
*** zz_jgrimm is now known as jgrimm | 14:34 | |
*** afazekas has quit IRC | 14:37 | |
dkranz | yfried: You need to modify lib/tempest in https://github.com/openstack-dev/devstack.git | 14:39 |
dkranz | yfried: Just remember that tempest will be running against a system deployed with the icehouse version of devstack as well. | 14:40 |
yfried | dkranz: I'm looking to extend tenant_network_cidr in large_ops gates only. not on all devstacks | 14:40 |
dkranz | yfried: ok | 14:41 |
yfried | dkranz: so how do I modify tempest.conf for a specific gate job? | 14:41 |
dkranz | yfried: that I am not sure about | 14:42 |
dkranz | yfried: probably need to ask on #openstack-infra | 14:42 |
mtreinish | yfried: you have to make the option exposed via an env variable in devstack | 14:42 |
mtreinish | then add support to devstack gate to set the option | 14:42 |
yfried | mtreinish: do you have an example for step #1? | 14:43 |
yfried | mtreinish: tnx, btw | 14:43 |
mtreinish | which you'll add to the jjb job definition in the project-config repo to set it on for the job | 14:43 |
mtreinish | yfried: just look at the devstack-gate repo, it's pretty obvious in devstack-gate.sh and devstack-gate-wrap.sh | 14:43 |
*** SridharG has left #openstack-qa | 14:48 | |
*** pcrews has joined #openstack-qa | 14:48 | |
sdague | mtreinish: got a moment to look at the javascript ER stack? I'd like to get that in today so that the ES backlog becomes obvious to people, because it still creates a lot of confusion at this point | 14:49 |
sdague | also - once we get to - https://review.openstack.org/#/c/124590/ - it becomes a lot easier to review | 14:49 |
sdague | as I put real templating in there | 14:49 |
mtreinish | sdague: sure, although I was curious about 124472, those time fields aren't in the output yet so does that cause things to explode? (I don't do much js) | 14:51 |
*** andreaf has joined #openstack-qa | 14:52 | |
sdague | no, because update_critical_dates is nested inside the if block | 14:55 |
sdague | so it only runs if the json includes the 'buglist' param | 14:55 |
sdague | which means it's the new format, which has the other fields | 14:55 |
sdague | so that change is safe to land, we land the next change which creates the new format, then the new fields take | 14:56 |
sdague | then some time in the future we can remove the if guard | 14:56 |
sdague | mtreinish: let me know once you are through the stack, I'll swing by and do the fixes | 14:58 |
mtreinish | ok, yeah I see that now, I forgot that buglist was moved into a field | 15:00 |
mtreinish | sdague: sure | 15:00 |
*** andreaf has quit IRC | 15:01 | |
*** dustins has quit IRC | 15:02 | |
mtreinish | sdague: looks like it was just that 1 comment, everything else looked fine to me | 15:03 |
*** omrim_ has quit IRC | 15:03 | |
*** dustins has joined #openstack-qa | 15:03 | |
mriedem | mtreinish: seen these? http://logs.openstack.org/44/112944/4/check/check-tempest-dsvm-postgres-full-icehouse/3dac88f/logs/tempest.txt.gz?level=TRACE | 15:04 |
mriedem | AttributeError: type object 'TestDrivers' has no attribute 'created_objects' | 15:04 |
yfried | mtreinish: tnx. any idea how to change neutron and nova conf quotas on a job? | 15:05 |
mtreinish | mriedem: that does look familiar | 15:05 |
mtreinish | adam_g: ^^^ | 15:05 |
mriedem | mtreinish: i could get a bug open and push a fix later, should be simple enough - there are others besides the baremetal fails | 15:05 |
mriedem | sloppy teardown code | 15:05 |
mtreinish | mriedem: oh, yeah that's probably being exposed as part of https://blueprints.launchpad.net/tempest/+spec/resource-cleanup | 15:06 |
mtreinish | mriedem: so you can just push the fixes up as part of the bp | 15:06 |
mtreinish | yfried: well, the quotas will be set as part of each projects setup, so you'll have to add an option if there isn't already one to lib/nova and lib/neutron | 15:08 |
mtreinish | then work your way up the stack adding the option to devstack-gate and then using it in the job definition | 15:08 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: bring the sanity: convert to handlebars.js https://review.openstack.org/124590 | 15:08 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: collapse graph area if there are no fails https://review.openstack.org/124591 | 15:08 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: update web pages to support additional date info https://review.openstack.org/124472 | 15:08 |
yfried | mtreinish: large-ops already changes some of the quotas. I just can't find where it's doing that | 15:08 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: add support for computing relevant dates https://review.openstack.org/124473 | 15:08 |
sdague | mtreinish: fixed | 15:09 |
openstackgerrit | Yair Fried proposed a change to openstack-dev/devstack: Adds tenant_network_conf to tempest.conf https://review.openstack.org/124766 | 15:09 |
*** sergsh has left #openstack-qa | 15:10 | |
openstackgerrit | Yair Fried proposed a change to openstack-dev/devstack: Adds tenant_network_conf to tempest.conf https://review.openstack.org/124766 | 15:12 |
mtreinish | yfried: it looks like it sets it here: http://git.openstack.org/cgit/openstack-dev/devstack/tree/lib/nova_plugins/hypervisor-fake#n36 | 15:13 |
*** jvrbanac has joined #openstack-qa | 15:13 | |
openstackgerrit | Yair Fried proposed a change to openstack-infra/devstack-gate: Adds tenant_network_conf to tempest.conf https://review.openstack.org/124768 | 15:14 |
*** andreaf has joined #openstack-qa | 15:15 | |
*** Poornima has joined #openstack-qa | 15:17 | |
*** comstud has joined #openstack-qa | 15:20 | |
*** Longgeek has quit IRC | 15:20 | |
*** lordd has joined #openstack-qa | 15:20 | |
*** yfried has quit IRC | 15:20 | |
*** nmagnezi_ has quit IRC | 15:21 | |
openstackgerrit | David Kranz proposed a change to openstack/tempest: Remove skips for tests that are closed in Launchpad https://review.openstack.org/124774 | 15:25 |
openstackgerrit | John Griffith proposed a change to openstack-dev/devstack: Relocate SERVICE_TIMEOUT to stackrc https://review.openstack.org/122783 | 15:27 |
mtreinish | dkranz: ^^^ I did a round of that a week or 2 ago, there were 4 unskips that didn't work | 15:28 |
dkranz | mtreinish: Then shouldn't the bug be reopened? Or were they icehouse issues? | 15:28 |
*** Guest73730 is now known as mgagne | 15:29 | |
*** mgagne has quit IRC | 15:29 | |
*** mgagne has joined #openstack-qa | 15:29 | |
mtreinish | dkranz: heh, that was the next step, but I got sidetracked :) | 15:29 |
dkranz | mtreinish: ok, I'll look at the current results and ensure world peace between tempest code and launchpad | 15:30 |
mtreinish | dkranz: I've still got the 4 broken unskips up as WIP | 15:31 |
*** lordd has quit IRC | 15:31 | |
*** nikhil_k|afk is now known as nikhil_k | 15:32 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Separate security group rule test for each args https://review.openstack.org/124003 | 15:35 |
*** hockeynut has quit IRC | 15:39 | |
*** hockeynut has joined #openstack-qa | 15:40 | |
jgriffith | sdague: mriedem FWIW http://goo.gl/N0ve6E | 15:41 |
jgriffith | sdague: mriedem wondering if the kernel update hit our images Friday | 15:42 |
sdague | doubtful, ES is just way backed up | 15:42 |
jgriffith | sdague: darn you | 15:42 |
jgriffith | sdague: bursting my bubble | 15:42 |
sdague | you know, I think I'm just going to land those ER changes so I don't have to keep explaining that to everyone :) | 15:43 |
jgriffith | sdague: LOL, sorry to add to your pain on a Monday :) | 15:43 |
sdague | no worries | 15:43 |
sdague | it's confusing | 15:43 |
sdague | jgriffith: is there an ER query for that yet? | 15:45 |
jgriffith | sdague: no | 15:46 |
jgriffith | sdague: I'm happy to muttle through adding one if you like though | 15:46 |
sdague | yeh, that would be cool | 15:46 |
jgriffith | sdague: k, I'll work on that now | 15:46 |
sdague | https://github.com/openstack-infra/elastic-recheck/blob/master/README.rst | 15:46 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: update web pages to support additional date info https://review.openstack.org/124472 | 15:47 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: add support for computing relevant dates https://review.openstack.org/124473 | 15:47 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: bring the sanity: convert to handlebars.js https://review.openstack.org/124590 | 15:47 |
*** lordd has joined #openstack-qa | 15:56 | |
openstackgerrit | John Griffith proposed a change to openstack-infra/elastic-recheck: Add e-r query for bug 1373513 https://review.openstack.org/124787 | 15:57 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate scenario utils to tempest client https://review.openstack.org/124015 | 15:59 |
*** aimon has quit IRC | 16:00 | |
*** dmorita has quit IRC | 16:02 | |
*** lordd has quit IRC | 16:04 | |
*** mlavalle has joined #openstack-qa | 16:04 | |
*** lordd has joined #openstack-qa | 16:05 | |
*** sc68cal has joined #openstack-qa | 16:05 | |
*** markus_z has quit IRC | 16:07 | |
*** jlanoux has quit IRC | 16:08 | |
*** lordd_ has joined #openstack-qa | 16:08 | |
*** lordd has quit IRC | 16:09 | |
sdague | thanks jgriffith | 16:09 |
*** marun has joined #openstack-qa | 16:13 | |
*** marun has quit IRC | 16:13 | |
jgriffith | sdague: absolutely | 16:14 |
*** marun has joined #openstack-qa | 16:15 | |
*** jlibosva has quit IRC | 16:20 | |
*** jordanP has quit IRC | 16:21 | |
*** evgenyf has quit IRC | 16:22 | |
*** hockeynut has quit IRC | 16:22 | |
*** hockeynut has joined #openstack-qa | 16:23 | |
*** dimsum_ is now known as dims | 16:23 | |
*** safchain has quit IRC | 16:27 | |
*** aimon has joined #openstack-qa | 16:27 | |
*** amotoki has quit IRC | 16:27 | |
*** eglynn-office is now known as eglynn-officeafk | 16:31 | |
*** rushiagr is now known as rushiagr_away | 16:31 | |
*** comstud has quit IRC | 16:35 | |
openstackgerrit | A change was merged to openstack/tempest: Add bash to tox whitelist_externals https://review.openstack.org/123896 | 16:38 |
*** comstud has joined #openstack-qa | 16:41 | |
cdent | I'll chase that process shutdown (with USE_SCREEN=False) stuff tomorrow if that's useful. Anything you've already discounted? | 16:41 |
cdent | sdague: ^ | 16:41 |
openstackgerrit | A change was merged to openstack/tempest: Work toward Python 3.4 support and testing https://review.openstack.org/118814 | 16:42 |
openstackgerrit | A change was merged to openstack/tempest: enable volumes v2 snapshot tests by sharing codes https://review.openstack.org/106664 | 16:42 |
sdague | cdent: I haven't dug into it, I expect the issue is something's still not right with the pid we capture, so the service stop isn't happening correctly | 16:42 |
sdague | cdent: thanks for diving in on that! | 16:42 |
* cdent nods | 16:42 | |
*** tosky has quit IRC | 16:43 | |
cdent | some things are getting killed but some not, so that may be a clue, but...tomorrow | 16:44 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add e-r query for bug 1373513 https://review.openstack.org/124787 | 16:45 |
*** suneelb has joined #openstack-qa | 16:52 | |
*** yfried has joined #openstack-qa | 16:52 | |
*** gszasz has quit IRC | 16:54 | |
openstackgerrit | A change was merged to openstack/tempest: Stop using intersphinx https://review.openstack.org/123963 | 16:58 |
openstackgerrit | A change was merged to openstack-dev/devstack: Re-order stack.sh 3: logging and error traps https://review.openstack.org/111734 | 16:59 |
*** lordd_ has quit IRC | 17:01 | |
*** lordd has joined #openstack-qa | 17:01 | |
*** e0ne has quit IRC | 17:02 | |
*** lordd has quit IRC | 17:05 | |
*** tkammer has quit IRC | 17:09 | |
*** rushiagr_away is now known as rushiagr | 17:09 | |
*** mpaolino has quit IRC | 17:10 | |
*** qba73 has quit IRC | 17:10 | |
*** harlowja has joined #openstack-qa | 17:16 | |
*** dmorita has joined #openstack-qa | 17:19 | |
*** Dafna has quit IRC | 17:21 | |
*** harlowja has quit IRC | 17:22 | |
*** cdent has quit IRC | 17:23 | |
*** harlowja has joined #openstack-qa | 17:24 | |
openstackgerrit | Baodong (Robert) Li proposed a change to openstack-dev/devstack: Preserve quotes and allow multi-line config items in meta-section of local.conf https://review.openstack.org/124502 | 17:28 |
*** lordd has joined #openstack-qa | 17:35 | |
*** lordd has quit IRC | 17:35 | |
openstackgerrit | Baodong (Robert) Li proposed a change to openstack-dev/devstack: Preserve quotes and allow multi-line config items in meta-section of local.conf https://review.openstack.org/124502 | 17:37 |
*** e0ne has joined #openstack-qa | 17:44 | |
*** johnthetubaguy is now known as zz_johnthetubagu | 17:46 | |
*** atiwari has quit IRC | 17:47 | |
*** catohornet has joined #openstack-qa | 17:48 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: collapse graph area if there are no fails https://review.openstack.org/124591 | 17:50 |
*** sweston has joined #openstack-qa | 17:52 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for heat slow icehouse timeout bug 1374175 https://review.openstack.org/124627 | 17:55 |
*** yamahata has quit IRC | 18:05 | |
openstackgerrit | A change was merged to openstack/tempest: Javelin: enable volume resources https://review.openstack.org/121648 | 18:06 |
*** sabeen has quit IRC | 18:08 | |
*** sabeen has joined #openstack-qa | 18:10 | |
*** mdbooth has quit IRC | 18:21 | |
*** edwa66431 has joined #openstack-qa | 18:25 | |
*** edwa6643 has quit IRC | 18:25 | |
*** mdbooth has joined #openstack-qa | 18:27 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Clean up tox.ini and make tempest and unit test distinct https://review.openstack.org/123589 | 18:28 |
*** Poornima has quit IRC | 18:30 | |
*** dmorita has quit IRC | 18:34 | |
*** e0ne has quit IRC | 18:35 | |
openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Fix MismatchError for LB scenario test https://review.openstack.org/120533 | 18:37 |
*** rushiagr is now known as rushiagr_away | 18:42 | |
mtreinish | andreaf: so I'm looking at the sample configs for subunit2sql, I was curious what you thought the best way to split it was | 18:50 |
mtreinish | because there are 3 commands all of which share the same base option set (from oslo.db) and then 2 have a couple of extra options which would normally be set from the cli | 18:50 |
mtreinish | so would would having 1 sample for each command with the command specific options be better, just one sample file with just the oslo.db options, or something else? | 18:51 |
*** dmellado has quit IRC | 18:54 | |
openstackgerrit | Yair Fried proposed a change to openstack-dev/devstack: Adds tenant_network_conf to tempest.conf https://review.openstack.org/124766 | 18:56 |
*** Sukhdev has joined #openstack-qa | 18:58 | |
*** sdmonov has joined #openstack-qa | 19:00 | |
openstackgerrit | Mikhail S Medvedev proposed a change to openstack/tempest: Skip orchestration test if Neutron is not available https://review.openstack.org/118454 | 19:00 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add sample config files for commands https://review.openstack.org/124832 | 19:01 |
*** rockyg has joined #openstack-qa | 19:01 | |
*** mdbooth has quit IRC | 19:02 | |
*** andreaf has quit IRC | 19:03 | |
*** andreaf has joined #openstack-qa | 19:04 | |
*** mdbooth has joined #openstack-qa | 19:07 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: expose what kernel is running https://review.openstack.org/124458 | 19:12 |
openstackgerrit | Emilien Macchi proposed a change to openstack-dev/grenade: grenade.sh: destroy javelin resources by the end https://review.openstack.org/120295 | 19:15 |
*** sabeen has quit IRC | 19:25 | |
*** sabeen has joined #openstack-qa | 19:25 | |
*** marun has quit IRC | 19:25 | |
*** nmagnezi_ has joined #openstack-qa | 19:37 | |
*** e0ne has joined #openstack-qa | 19:38 | |
mtreinish | jogo: btw, I figured out what was wrong with my patch using the * for the tox whitelist. I had it in the wrong section of the tox.ini... | 19:39 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add sample config files for commands https://review.openstack.org/124832 | 19:43 |
*** toure has quit IRC | 19:43 | |
*** zz_johnthetubagu has quit IRC | 19:45 | |
*** cnesa15 has quit IRC | 19:49 | |
*** harlowja has quit IRC | 19:50 | |
*** atiwari has joined #openstack-qa | 19:52 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add session objects for get_all* db api calls https://review.openstack.org/118419 | 19:55 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to run table https://review.openstack.org/117861 | 19:55 |
*** harlowja has joined #openstack-qa | 19:55 | |
*** melwitt has joined #openstack-qa | 19:57 | |
*** dmorita has joined #openstack-qa | 20:00 | |
*** marun has joined #openstack-qa | 20:01 | |
*** b3nt_pin is now known as beageles | 20:03 | |
*** beageles is now known as beagles | 20:03 | |
*** dimtruck is now known as zz_dimtruck | 20:05 | |
*** zz_dimtruck is now known as dimtruck | 20:06 | |
*** sdmonov has quit IRC | 20:11 | |
*** boden has quit IRC | 20:16 | |
*** boden has joined #openstack-qa | 20:17 | |
*** mdbooth has quit IRC | 20:18 | |
*** boden has quit IRC | 20:18 | |
*** nmagnezi_ has quit IRC | 20:18 | |
*** boden has joined #openstack-qa | 20:18 | |
*** boden has quit IRC | 20:18 | |
*** nmagnezi has joined #openstack-qa | 20:19 | |
*** boden has joined #openstack-qa | 20:19 | |
*** boden has quit IRC | 20:19 | |
*** melwitt1 has joined #openstack-qa | 20:20 | |
*** boden has joined #openstack-qa | 20:21 | |
*** boden has quit IRC | 20:21 | |
*** boden has joined #openstack-qa | 20:22 | |
*** boden has quit IRC | 20:23 | |
*** melwitt has quit IRC | 20:23 | |
*** mdbooth has joined #openstack-qa | 20:23 | |
*** boden has joined #openstack-qa | 20:23 | |
*** boden has quit IRC | 20:24 | |
*** boden has joined #openstack-qa | 20:24 | |
*** boden has quit IRC | 20:24 | |
*** sdmonov has joined #openstack-qa | 20:25 | |
mriedem | mtreinish: andreaf: regarding https://review.openstack.org/#/c/121542/7/tempest/api/baremetal/admin/base.py - in what case would resource_cleanup be called before resource_setup was done - assuming if resource_setup failed? | 20:26 |
*** boden has joined #openstack-qa | 20:26 | |
mriedem | like with the skipException? | 20:26 |
*** stevebaker has joined #openstack-qa | 20:27 | |
mriedem | nvm, answered my own question | 20:27 |
openstackgerrit | Martine Silbermann proposed a change to openstack-dev/devstack: Permit non-loopback guest access to Rabbit https://review.openstack.org/124861 | 20:28 |
*** boden has quit IRC | 20:28 | |
mtreinish | mriedem: yeah any exception that gets raised during resource_setup will trigger resource_cleanup | 20:29 |
jogo | mtreinish: heh | 20:29 |
mtreinish | mriedem: the next step on that bp is to do it in stages to make it clearer what cleanup should be called for certain failures | 20:30 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Init created_objects for baremetal API tests before skipException https://review.openstack.org/124863 | 20:31 |
mriedem | there you go | 20:31 |
mtreinish | mriedem: that will just move the exception. It'll raise a keyerror now | 20:32 |
mriedem | oh gdi | 20:33 |
mriedem | WIP | 20:33 |
mtreinish | just do an if cls.created_objects before the loop | 20:33 |
mriedem | hasattr you mean? | 20:33 |
mtreinish | mriedem: or that. I was thinking 'if {} == False' but whatever works | 20:34 |
mriedem | oh nvm | 20:34 |
mriedem | i was thinking you meant don't touch resource_setup | 20:34 |
mriedem | could just do cls.created_objects.get(resource) | 20:35 |
*** comstud has quit IRC | 20:38 | |
*** mmedvede has quit IRC | 20:38 | |
*** comstud has joined #openstack-qa | 20:38 | |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Chekc for created_objects in resource_cleanup https://review.openstack.org/124863 | 20:40 |
mtreinish | mriedem: ^^^ s/Chekc/Check | 20:40 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Check for created_objects in resource_cleanup https://review.openstack.org/124863 | 20:40 |
mriedem | go go commit message edit button | 20:40 |
mriedem | when i write tempest patches i work fast and loose :) | 20:41 |
mtreinish | mriedem: heh, I do the same. It's like the wild west | 20:41 |
mriedem | ever watch deadwood on hbo? | 20:42 |
mriedem | you should run tempest like the guy with the stache | 20:42 |
mtreinish | no, I haven't. I guess I'll have to watch it now to get that reference | 20:42 |
mriedem | this guy http://images6.fanpop.com/image/photos/36600000/Deadwood-image-deadwood-36688010-1600-1200.jpg | 20:43 |
*** rockyg has quit IRC | 20:46 | |
EmilienM | sdague: jogo: I realize I need to improve destoy_volumes (http://git.io/IWPBEQ) because we need to detach the volume first and then delete it. Though I don't understand the logic since we have deleted the servers before : http://git.io/2y98NQ | 20:48 |
mtreinish | mriedem: heh, I can imagine. I guess I'll have to grow a mustache like that for paris | 20:48 |
EmilienM | sdague: jogo: I'm investigating https://review.openstack.org/#/c/120295/ failure - http://logs.openstack.org/95/120295/3/check/check-grenade-dsvm/06ae252/logs/javelin.txt.gz | 20:48 |
*** andreaf has quit IRC | 20:49 | |
*** rfolco has quit IRC | 20:49 | |
*** rfolco has joined #openstack-qa | 20:50 | |
*** andreaf has joined #openstack-qa | 20:50 | |
jogo | EmilienM: cool thanks for digging in | 20:50 |
*** cnesa15 has joined #openstack-qa | 20:51 | |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Check for class attributes on resource_cleanup in case of skipException https://review.openstack.org/124870 | 20:51 |
*** e0ne has quit IRC | 20:52 | |
*** nmagnezi has quit IRC | 20:52 | |
andreaf | sdague, dkranz: https://review.openstack.org/#/c/121547/ for resource cleanup is waiting for a +A | 20:53 |
*** jgrimm is now known as zz_jgrimm | 20:54 | |
mriedem | andreaf: comment inline | 20:55 |
*** aix has joined #openstack-qa | 20:56 | |
*** e0ne has joined #openstack-qa | 20:56 | |
andreaf | mriedem: yes I know it's not ideal, but I really wanted to get the first pass of migration though quickly - easy to review - a then go back and fix those errors | 20:57 |
*** Sukhdev has quit IRC | 20:58 | |
andreaf | mriedem: in fact the only difference is additional errors logged in the logs, so I thought that was acceptable | 20:58 |
mriedem | andreaf: up to you guys, from my point of view i go through the tempest logs on jenkins failures and try to find stack traces for fingerprints when i don't want to use a generic error message from the console log (for elastic-recheck), so the clutter in the tempest log is ugly | 20:59 |
*** zz_jgrimm is now known as jgrimm | 21:00 | |
andreaf | mriedem, mtreinish, sdague: would it be an acceptable solution to mute AttributeError exceptions from tearDown? | 21:00 |
mriedem | andreaf: i wouldn't think so, seems leaky | 21:01 |
andreaf | or to log them as info | 21:01 |
mriedem | i agree that cleaning these up is low priority | 21:01 |
mtreinish | andreaf: no, muting them would not be good. If you want to play with log levels that might be ok, but the cleaning them up isn't too hard. | 21:03 |
mtreinish | they'll also go away once everything is properly staged | 21:03 |
mtreinish | andreaf: in the short term recruit mriedem to cleanup the fails as you introduce them... | 21:03 |
mriedem | already going through the logs :) | 21:03 |
andreaf | mtreinish, mriedem: that would be fine with me :) | 21:04 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 21:05 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: compute: check for class attributes on resource_cleanup https://review.openstack.org/124870 | 21:08 |
*** Sukhdev has joined #openstack-qa | 21:09 | |
*** mmedvede has joined #openstack-qa | 21:11 | |
*** cnesa15 has quit IRC | 21:12 | |
openstackgerrit | John Griffith proposed a change to openstack-dev/devstack: Relocate SERVICE_TIMEOUT to stackrc https://review.openstack.org/122783 | 21:14 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: data_processing: check for attributes in resource_cleanup https://review.openstack.org/124887 | 21:14 |
openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: Detach the volume before destruction https://review.openstack.org/121647 | 21:17 |
*** edwa6643 has joined #openstack-qa | 21:17 | |
*** Guest78856 has joined #openstack-qa | 21:18 | |
*** Guest78856 is now known as annegentle | 21:19 | |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: image: make sure created_images attribute exists on resource_cleanup https://review.openstack.org/124891 | 21:19 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: image: make sure created_images attribute exists on resource_cleanup https://review.openstack.org/124891 | 21:19 |
openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: Detach the volume before destruction https://review.openstack.org/121647 | 21:20 |
*** edwa66431 has quit IRC | 21:21 | |
dkranz | andreaf: So I guess you don't want +A anymore? | 21:23 |
mriedem | dkranz: i can just clean it up on top | 21:24 |
mriedem | with a dep patch | 21:24 |
dkranz | mriedem: So you will remove -1 now? | 21:24 |
mriedem | done | 21:24 |
andreaf | dkranz, mriedem: that works for me | 21:24 |
dkranz | mriedem: ok, cool | 21:24 |
dkranz | andreaf: I'll look now. | 21:25 |
andreaf | thanks | 21:25 |
andreaf | dkranz: there are more like this but they are going through recheck... | 21:25 |
*** sdmonov has quit IRC | 21:29 | |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: orchestration: check for attributes in resource_cleanup https://review.openstack.org/124896 | 21:32 |
*** krtaylor has quit IRC | 21:34 | |
*** krtaylor has joined #openstack-qa | 21:36 | |
*** sdmonov has joined #openstack-qa | 21:38 | |
*** gfidente has quit IRC | 21:40 | |
*** dims_ has joined #openstack-qa | 21:44 | |
*** dims_ has quit IRC | 21:47 | |
*** dims has quit IRC | 21:47 | |
*** dims has joined #openstack-qa | 21:47 | |
*** dkranz has quit IRC | 21:49 | |
*** dustins has quit IRC | 21:49 | |
*** cnesa15 has joined #openstack-qa | 21:50 | |
*** dmorita has quit IRC | 21:52 | |
*** mriedem has quit IRC | 21:53 | |
*** krtaylor has quit IRC | 21:53 | |
*** aix_ has joined #openstack-qa | 21:57 | |
openstackgerrit | A change was merged to openstack/tempest: Make SwiftScenarioTest methods public https://review.openstack.org/120385 | 21:58 |
*** aix has quit IRC | 22:00 | |
*** krtaylor has joined #openstack-qa | 22:00 | |
jogo | mtreinish: something must be wrong with my patch: https://review.openstack.org/#/c/119081/ | 22:04 |
jogo | it keeps failing but I don't see how its related to my patch | 22:04 |
*** timello has quit IRC | 22:06 | |
*** ekhugen has quit IRC | 22:06 | |
*** maurosr has quit IRC | 22:06 | |
*** jgrimm has quit IRC | 22:06 | |
*** ekhugen has joined #openstack-qa | 22:07 | |
*** jgrimm has joined #openstack-qa | 22:07 | |
*** jgrimm is now known as zz_jgrimm | 22:07 | |
sdague | jogo: no, neutron is just very sad | 22:08 |
*** maurosr has joined #openstack-qa | 22:08 | |
jogo | sdague: icehouse neutron is that bad wow ... | 22:08 |
sdague | it failed on master neutron too | 22:08 |
*** zz_jgrimm is now known as jgrimm | 22:08 | |
jogo | sdague: I must have rechecked that patch 7 times | 22:08 |
*** timello has joined #openstack-qa | 22:08 | |
openstackgerrit | Joe Gordon proposed a change to openstack-dev/grenade: Run dstat during the upgrade process https://review.openstack.org/119591 | 22:12 |
*** marun has quit IRC | 22:18 | |
mtreinish | jogo: are all the failures the same? | 22:23 |
jogo | mtreinish: they appear to be mostly the same | 22:24 |
jogo | different jobs (all stable neutron) | 22:25 |
*** sdmonov has quit IRC | 22:28 | |
*** e0ne has quit IRC | 22:29 | |
*** JayJ has joined #openstack-qa | 22:30 | |
openstackgerrit | Adam Gandelman proposed a change to openstack-infra/elastic-recheck: Add query for 1375488 https://review.openstack.org/124913 | 22:31 |
mtreinish | jogo: hmm, I guess you could be stressing things in just the right way to trigger an unknown thing on icehouse | 22:33 |
*** e0ne has joined #openstack-qa | 22:33 | |
*** e0ne has quit IRC | 22:33 | |
* mtreinish wonders if that's vague enough | 22:33 | |
jogo | mtreinish: heh | 22:34 |
*** sdmonov has joined #openstack-qa | 22:35 | |
EmilienM | jogo: sdague: javelin/fix cinder destruction https://review.openstack.org/#/c/121647/ | 22:35 |
*** cnesa15 has quit IRC | 22:35 | |
*** ekhugen has quit IRC | 22:36 | |
*** jgrimm has quit IRC | 22:36 | |
*** timello has quit IRC | 22:36 | |
*** maurosr has quit IRC | 22:36 | |
*** ekhugen has joined #openstack-qa | 22:37 | |
*** zz_jgrimm has joined #openstack-qa | 22:37 | |
*** maurosr has joined #openstack-qa | 22:38 | |
*** timello has joined #openstack-qa | 22:38 | |
*** bknudson has quit IRC | 22:40 | |
*** gordc has quit IRC | 22:40 | |
*** dimtruck is now known as zz_dimtruck | 22:43 | |
*** andreaf has quit IRC | 22:44 | |
*** andreaf has joined #openstack-qa | 22:45 | |
*** andreaf has quit IRC | 22:47 | |
*** andreaf has joined #openstack-qa | 22:47 | |
*** JayJ has quit IRC | 23:01 | |
*** rbak has quit IRC | 23:03 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: add links to specific bugs https://review.openstack.org/124921 | 23:05 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: add links to specific bugs https://review.openstack.org/124921 | 23:06 |
*** mlavalle has quit IRC | 23:06 | |
*** weshay has quit IRC | 23:08 | |
*** Sukhdev has quit IRC | 23:08 | |
*** dims has quit IRC | 23:12 | |
*** dims has joined #openstack-qa | 23:13 | |
*** dims has quit IRC | 23:17 | |
*** lordd has joined #openstack-qa | 23:17 | |
*** yamahata has joined #openstack-qa | 23:18 | |
openstackgerrit | Kevin Benton proposed a change to openstack-dev/devstack: Use service role for neutron instead of admin https://review.openstack.org/108167 | 23:22 |
*** jecarey_ has quit IRC | 23:29 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/elastic-recheck: [WIP] Added javascript toolchain https://review.openstack.org/124927 | 23:29 |
*** lordd has quit IRC | 23:30 | |
*** sdmonov has quit IRC | 23:34 | |
*** Sukhdev has joined #openstack-qa | 23:38 | |
*** atiwari has quit IRC | 23:40 | |
*** aimon has quit IRC | 23:52 | |
*** andreaf has quit IRC | 23:54 | |
*** andreaf has joined #openstack-qa | 23:55 | |
*** harlowja is now known as harlowja_away | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!