Tuesday, 2013-08-06

*** hemna is now known as hemnafk00:03
*** rhsu has joined #openstack-qa00:07
openstackgerritA change was merged to openstack-infra/devstack-gate: Use Jenkins credentials store if specified  https://review.openstack.org/4031000:25
*** rhsu has quit IRC00:45
*** sthaha has joined #openstack-qa00:54
*** sthaha has joined #openstack-qa00:54
*** prekarat has quit IRC02:07
openstackgerritA change was merged to openstack-dev/devstack: Update projects to global requirements before setup  https://review.openstack.org/3570502:15
*** lifeless has quit IRC02:16
*** adalbas has quit IRC02:47
*** jhenner has quit IRC02:48
*** adalbas has joined #openstack-qa02:57
*** prekarat has joined #openstack-qa03:00
*** lifeless has joined #openstack-qa03:00
*** prekarat has quit IRC03:04
*** prekarat has joined #openstack-qa03:06
openstackgerritPrem Karat proposed a change to openstack/tempest: Add nova v3 tests for test_servers  https://review.openstack.org/4014703:45
*** afazekas has quit IRC03:47
*** SergeyLukjanov has joined #openstack-qa04:03
openstackgerritanju Tiwari proposed a change to openstack/tempest: Added a test for stop and start a server  https://review.openstack.org/4019104:05
*** SergeyLukjanov has quit IRC04:30
*** SergeyLukjanov has joined #openstack-qa04:31
openstackgerritSteve Baker proposed a change to openstack-dev/devstack: Generate heat images for tempest tests  https://review.openstack.org/4034105:04
openstackgerritSteve Baker proposed a change to openstack-dev/devstack: Colorize heat engine log  https://review.openstack.org/4034205:04
openstackgerritSteve Baker proposed a change to openstack-dev/devstack: Baremetal uses packaged diskimage-builder  https://review.openstack.org/4034305:04
*** prekarat has quit IRC05:17
*** SergeyLukjanov has quit IRC05:24
*** prekarat has joined #openstack-qa05:31
openstackgerritanju Tiwari proposed a change to openstack/tempest: Added a test for stop and start a server  https://review.openstack.org/4019105:33
openstackgerritafazekas proposed a change to openstack-dev/devstack: .conf suffix for the horizon config on Fedora  https://review.openstack.org/4035206:03
openstackgerritafazekas proposed a change to openstack-dev/devstack: .conf suffix for the horizon config on Fedora  https://review.openstack.org/4035206:11
openstackgerritPrem Karat proposed a change to openstack/tempest: Add nova-v3-api tests for flavors  https://review.openstack.org/4016606:26
*** sthaha has quit IRC06:36
*** prekarat has quit IRC06:56
*** jhenner has joined #openstack-qa07:01
*** prekarat has joined #openstack-qa07:11
*** jhenner has quit IRC07:23
*** tkammer has joined #openstack-qa07:28
*** fbo_away is now known as fbo07:39
openstackgerritanju Tiwari proposed a change to openstack/tempest: A volume upload test added for xml  https://review.openstack.org/3966407:41
openstackgerritanju Tiwari proposed a change to openstack/tempest: A volume upload test added for xml  https://review.openstack.org/3966407:44
*** dosaboy_alt has joined #openstack-qa08:08
*** SergeyLukjanov has joined #openstack-qa08:08
*** jhenner has joined #openstack-qa08:22
*** jhenner has quit IRC08:28
*** jhenner has joined #openstack-qa08:37
*** jhenner has quit IRC08:42
*** woodspa has joined #openstack-qa09:24
*** woodspa has quit IRC09:24
openstackgerritSalvatore Orlando proposed a change to openstack-dev/devstack: Run Neutron migrations after creating schema  https://review.openstack.org/4024309:40
*** giulivo has joined #openstack-qa09:41
openstackgerritAditi Raveesh proposed a change to openstack/tempest: Adding extra_specs to flavor format  https://review.openstack.org/4038410:12
openstackgerritGiulio Fidente proposed a change to openstack/tempest: scenario test involving glance, cinder and nova functionalities  https://review.openstack.org/3894310:19
openstackgerritNejc Saje proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms  https://review.openstack.org/3923710:51
giulivoping sdague10:52
*** sthaha has joined #openstack-qa11:01
*** prekarat has quit IRC11:07
*** mkollaro has joined #openstack-qa11:07
*** key has quit IRC11:14
*** woodspa has joined #openstack-qa11:27
*** prekarat has joined #openstack-qa11:28
mkoderergiulivo: pong (from yesterday)11:32
*** raies has quit IRC11:46
*** jasondotstar has quit IRC11:47
*** weshay has joined #openstack-qa11:52
openstackgerritNejc Saje proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms  https://review.openstack.org/3923711:56
*** dosaboy_alt has quit IRC12:00
*** dosaboy_alt has joined #openstack-qa12:01
openstackgerritZhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client  https://review.openstack.org/4039612:08
*** dkranz has joined #openstack-qa12:09
giulivohi mkoderer I think I forgot what it was about :(12:16
openstackgerritZhi Kun Liu proposed a change to openstack/tempest: test_servers_whitebox - using official client  https://review.openstack.org/4039612:16
mkoderergiulivo: np ;)12:17
*** prekarat has quit IRC12:18
giulivomkoderer, I don't have such a long scrollback , was it before or after we discussed the decorator?12:18
mkodererafter12:18
mkodereryou should try irccloud.com12:19
giulivomkoderer, only thing I can think about is this https://review.openstack.org/#/c/40019/12:26
mkoderergiulivo: ok what goes on there?12:27
giulivojust wanted to ask if you faced issues around the dependencies installation12:28
giulivoand if you had a "preferred" way to do it12:28
mkoderergiulivo: dependency issues in devstack?12:30
giulivomkoderer, with devstack yes12:31
mkoderergiulivo: never had.. but I didn't update devstack for a while now12:32
*** prekarat has joined #openstack-qa12:33
*** sthaha has quit IRC12:34
*** jasondotstar has joined #openstack-qa12:35
openstackgerritnayna-patel proposed a change to openstack/tempest: Added 3 Routers related testcases for Neutron API  https://review.openstack.org/3662912:40
*** psedlak has joined #openstack-qa12:42
*** psedlak has quit IRC12:44
*** whayutin_ has joined #openstack-qa12:57
*** weshay has quit IRC12:57
*** anteaya has joined #openstack-qa12:59
*** fbo is now known as fbo_away13:11
*** woodspa_ has joined #openstack-qa13:13
*** psedlak has joined #openstack-qa13:14
*** woodspa has quit IRC13:16
*** sthaha has joined #openstack-qa13:17
*** sthaha has joined #openstack-qa13:17
*** sthaha has quit IRC13:17
rahmuhello. I have submitted a patch last week here (https://review.openstack.org/#/c/39659/) and it still hasn't been reviewed. Am I missing something? What's the correct procedure to get the reviewing started?13:19
*** fbo_away is now known as fbo13:20
*** mriedem has joined #openstack-qa13:22
openstackgerritBob Ball proposed a change to openstack-dev/devstack: Add NOVA_DEFAULT_LOG_LEVELS setting  https://review.openstack.org/4041113:27
*** sthaha has joined #openstack-qa13:28
openstackgerritafazekas proposed a change to openstack-dev/devstack: Show ip address before associating address  https://review.openstack.org/4041213:34
*** krtaylor has quit IRC13:36
openstackgerritFabien Boucher proposed a change to openstack/tempest: Add tests for swift container listing filters  https://review.openstack.org/4041613:39
openstackgerritGiulio Fidente proposed a change to openstack/tempest: scenario test involving glance, cinder and nova functionalities  https://review.openstack.org/3894313:41
openstackgerritMonty Taylor proposed a change to openstack-dev/devstack: Re-enble pip -e in setup_develop  https://review.openstack.org/4041713:42
openstackgerritMonty Taylor proposed a change to openstack-dev/devstack: Protect local dev changes from the update logic  https://review.openstack.org/4041813:42
openstackgerritBob Ball proposed a change to openstack-dev/devstack: Some bash versions don't support negative lengths  https://review.openstack.org/3923613:42
*** prekarat has quit IRC13:42
*** woodspa__ has joined #openstack-qa13:49
openstackgerritA change was merged to openstack/tempest: Sync up the default value for the network_for_ssh option  https://review.openstack.org/3957713:51
*** woodspa_ has quit IRC13:52
mkodererrahmu: it's already started13:55
mkodererrahmu: usually you have to wait until 2 core reviewers have the time13:56
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Make test_admin_timeout CLI test configurable  https://review.openstack.org/4041913:57
openstackgerritA change was merged to openstack-dev/devstack: Don't add bulk middleware.  https://review.openstack.org/3982613:59
openstackgerritA change was merged to openstack-dev/devstack: Ensures cross compatibility for zsh and bash.  https://review.openstack.org/3997113:59
*** whayutin_ has quit IRC14:05
*** dkranz has quit IRC14:05
openstackgerritFabien Boucher proposed a change to openstack/tempest: Add tests for swift container listing filters  https://review.openstack.org/4041614:06
*** weshay has joined #openstack-qa14:10
*** sthaha has joined #openstack-qa14:11
rahmumkoderer: thanks for your answer, I just got worried when I didn't see any activity after a few days14:12
*** psedlak has quit IRC14:15
*** sthaha has joined #openstack-qa14:16
*** sthaha has joined #openstack-qa14:16
*** sthaha has quit IRC14:16
*** krtaylor has joined #openstack-qa14:17
*** sthaha has joined #openstack-qa14:19
*** sthaha has joined #openstack-qa14:19
*** sthaha has quit IRC14:19
*** sthaha has joined #openstack-qa14:19
*** sthaha has joined #openstack-qa14:19
*** sthaha has quit IRC14:19
*** sthaha has joined #openstack-qa14:21
*** sthaha has joined #openstack-qa14:21
*** burt has joined #openstack-qa14:24
mriedemmkoderer: ping14:27
mkoderermriedem: hey matt14:27
mriedemmkoderer: hey, wanted to discuss a global cli timeout for https://review.openstack.org/#/c/40419/14:28
mkoderermriedem: ok14:28
mriedemin case anyone else had an opinion14:28
mriedemmkoderer: so the fact that nova's timeout is 2 seconds and keystones is 15, if i made a global option i'd have to default to 15 for keystone,14:28
mriedembut bumping the nova timeout to 15 seconds from 2 seems odd14:28
mkoderermriedem: yes thats also true14:29
mkoderermriedem: but having a parameter just for one test case...14:29
mriedemmkoderer: yeah, i know14:29
mriedemmkoderer: i don't have a great answer either way, so bringing to IRC14:29
mkodererand glance does have the timeout parameter as well14:29
mkodererbut it isn't tested14:30
mkoderermriedem: ^^14:30
mkoderermriedem: I mean we could have a global parameter cli.timeout and cli.compute_timeout or cli.keystone_timeout14:32
mkodererso if one of those specific parameters isn't set it uses the global one14:32
mkoderermriedem: but  I think this is a bit too much just for this simple problem14:33
mriedemmkoderer: yeah, agree that's a bit much14:34
mriedemsdague: any opinion on this? https://review.openstack.org/#/c/40419/14:37
mriedemmtreinish: ^14:37
mkoderermriedem: but what happens if you switch it to 15 secs on nova14:37
*** Anju_ has joined #openstack-qa14:37
mkodererjust in case something is broken you need to wait 15 secs14:38
mriedemmkoderer: nothing, 15 seconds would be fine for nova - in fact 3 seconds worked for me but 2 seconds didn't14:38
mkoderermriedem: I see :)14:38
mriedemmkoderer: basically i wasn't sure if someone thought nova list should be super fast or what,14:38
mriedemi'm inclined to think that if we want to avoid a project-specific timeout for CLI, i'm ok with a global that defaults to 15 to cover keystone14:39
mriedemif nova list times out at 15 seconds, it's probably busted anyway :)14:39
mriedemmkoderer: as long as we can get rid of the hard-coding in the test, i'm happy14:39
mkoderermriedem: is there any reason why keystone is set to 15?14:42
mriedemmkoderer: no idea14:42
mriedemmkoderer: i guess someone thought it was super slow?14:42
mkoderermriedem: I have the feeling we can put the default to 514:42
mkoderermriedem: you have such a slow system? maybe just test it with 5 ;)14:42
sdaguemtreinish: this is the patch - https://review.openstack.org/#/c/40417/14:42
mriedemmkoderer: let me find the commit that added the keystone test to see if there is any justification for the big timeout14:43
mkoderermriedem: thats a plan14:43
mriedemlooks like jog0 added those14:44
mriedemand i think he's here14:44
mriedemhttps://review.openstack.org/#/c/31541/14:44
mriedemmkoderer: so it was defaulted to 2 seconds, afazekas asked him to change to 5 seconds, and he changed to 15 seconds to be safe14:45
openstackgerritMonty Taylor proposed a change to openstack-dev/devstack: Don't uninstall packages when on xenserver  https://review.openstack.org/4043114:46
mriedemmkoderer: given that, i think it's probably safe to use the same logic for the nova timeout and make a global cli timeout option that defaults to 1514:46
mriedemmkoderer: sound good?14:46
mkoderermriedem: great - thanks for clarification14:47
*** mlavalle has joined #openstack-qa14:49
Anju_mkoderer:https://review.openstack.org/#/c/40191/ please have a look14:52
openstackgerritFabien Boucher proposed a change to openstack/tempest: Add test for swift ACLs  https://review.openstack.org/3882314:54
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Make CLI timeout tests configurable  https://review.openstack.org/4041915:00
Anju_mkoderer:means there should bei only one line15:02
mkodererAnju_: yes but in general you could improve the commit message15:04
mriedemAnju_: there were no previous tests for stopping and starting an instance?15:04
Anju_noo15:04
Anju_i check that15:04
mkoderermriedem: yes I was surprised about that15:05
mkodererdidn't find any ..15:06
mriedemAnju_: mkoderer: me neither, i think i was thinking of test_create_image_from_stopped_server which is an images test15:06
mkodererbut anyway strange since this is such a basic and important test15:07
Anju_yes it is .but for server like pause,unpause,15:07
mriedempause and unpause are non-core APIs though15:07
Anju_is available15:07
mriedemso i'm not really surprised by that15:07
mriedemstop/start are core APIS15:07
mkoderermriedem: ok I see15:07
Anju_mriedem:so do u think that it should so important15:09
Anju_i mean that it is not imp15:09
mriedemAnju_: yes, stop/start coverage for nova is important :)15:09
openstackgerritMike Perez proposed a change to openstack-dev/devstack: Start using cinder rest api v2 by default  https://review.openstack.org/2248915:09
*** ianw has quit IRC15:10
*** sthaha has quit IRC15:11
Anju_mriedem :thanks for reviewing :)15:14
mriedemAnju_: no problem, a pretty simple patch15:14
mkodererAnju_: simple but important ;)15:15
Anju_mkoderer::)15:15
mriedemtesting core APIs important? no way.15:16
openstackgerritFabien Boucher proposed a change to openstack/tempest: Add tests for swift container listing filters  https://review.openstack.org/4041615:16
Anju_i started from the Api document :mkoderer: u  have a good idea about patches15:16
mriedemthe problem i've had with nova is knowing what's a core api15:16
mtreinishsdague, mordred: with https://review.openstack.org/#/c/40417/ I'm hitting on my local environment: http://paste.openstack.org/show/43320/ and nova-api doesn't start15:17
*** dkranz has joined #openstack-qa15:17
clarkbmtreinish: I think you want 4001915:18
clarkbmtreinish: that at least gives more background on the problem and a potential fix15:19
mtreinishclarkb: ok, yeah that seems like it would at least change things. I'll give it a try15:19
mordredmtreinish: I think before we land 40019, we might want to pull a few specific known problem children out15:20
*** psedlak has joined #openstack-qa15:34
*** prekarat has joined #openstack-qa15:35
*** rhsu has joined #openstack-qa15:49
*** prekarat has quit IRC15:51
*** prekarat has joined #openstack-qa15:51
sdagueactually 40019 is a sledge hammer15:53
sdaguemtreinish: that paramiko thing should be fixed by the pip install15:53
sdagueif not, we need to figure out a way to make it so15:54
openstackgerritA change was merged to openstack-dev/devstack: Re-enble pip -e in setup_develop  https://review.openstack.org/4041715:56
*** dkranz has quit IRC16:22
*** tkammer has quit IRC16:24
*** dkranz has joined #openstack-qa16:35
*** fbo is now known as fbo_away16:43
*** krtaylor has quit IRC16:53
openstackgerritA change was merged to openstack-dev/devstack: Update tempest test sample in devstack/README.md  https://review.openstack.org/3982017:00
*** dkranz has quit IRC17:00
*** dosaboy_alt has quit IRC17:07
openstackgerritA change was merged to openstack-dev/devstack: Run Neutron migrations after creating schema  https://review.openstack.org/4024317:14
*** mlavalle has quit IRC17:15
*** dkranz has joined #openstack-qa17:16
*** afazekas has joined #openstack-qa17:20
*** krtaylor has joined #openstack-qa17:31
*** SergeyLukjanov has quit IRC17:32
*** torandu_ has quit IRC17:33
*** SergeyLukjanov has joined #openstack-qa17:33
*** torandu has joined #openstack-qa17:33
*** SergeyLukjanov has quit IRC17:34
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Add tools/install_pip.sh  https://review.openstack.org/3982717:48
*** dkranz has quit IRC17:57
*** mlavalle has joined #openstack-qa17:58
*** Anju_ has quit IRC18:01
*** afazekas has quit IRC18:02
*** dkranz has joined #openstack-qa18:10
*** jhenner has joined #openstack-qa18:16
*** SergeyLukjanov has joined #openstack-qa18:16
*** jhenner has quit IRC18:25
*** jhenner has joined #openstack-qa18:25
*** jhenner has quit IRC18:32
*** harlowja_ has quit IRC18:32
*** jhenner has joined #openstack-qa18:34
*** harlowja has joined #openstack-qa18:35
*** jhenner has quit IRC18:38
jog0mtreinish: ping18:52
*** krtaylor has quit IRC19:01
jog0mtreinish: I wanted to follow up an the aggregates  bug19:02
mtreinishjog0: sorry was out getting coffee19:14
mtreinishjog0: which bug? There have been a few19:15
jog0mtreinish: the one related to https://review.openstack.org/#/c/36651/1919:17
mtreinishjog0: I pushed a fix for those bugs: https://review.openstack.org/#/c/40424/ I've got to add tests for it though. It's on my list for today19:18
*** anteaya has quit IRC19:19
jog0cool19:19
jog0have to step out myself but thanks19:19
mtreinishjog0: sure np19:19
*** rhsu has quit IRC19:22
*** krtaylor has joined #openstack-qa19:27
*** anteaya has joined #openstack-qa19:34
openstackgerritA change was merged to openstack-dev/devstack: Some bash versions don't support negative lengths  https://review.openstack.org/3923619:36
*** krtaylor has quit IRC20:10
openstackgerritA change was merged to openstack-dev/devstack: Don't uninstall packages when on xenserver  https://review.openstack.org/4043120:11
*** psedlak has quit IRC20:16
*** dosaboy_ has joined #openstack-qa20:21
*** dosaboy has quit IRC20:24
*** krtaylor has joined #openstack-qa21:01
*** SergeyLukjanov has quit IRC21:08
*** dosaboy has joined #openstack-qa21:27
*** dosaboy_ has quit IRC21:28
*** jasondotstar has quit IRC21:30
*** woodspa__ has quit IRC21:31
*** mriedem has quit IRC21:37
*** dosaboy has quit IRC21:53
*** mkollaro has quit IRC21:58
*** dosaboy has joined #openstack-qa22:01
*** giulivo has quit IRC22:03
*** burt has quit IRC22:10
*** weshay has quit IRC22:22
openstackgerritA change was merged to openstack/tempest: Added a test for stop and start a server  https://review.openstack.org/4019122:26
openstackgerritA change was merged to openstack/tempest: Make CLI timeout tests configurable  https://review.openstack.org/4041922:31
*** ianw has joined #openstack-qa22:47
stevebakerhey, are there any reviews yet for the slow-heat gate job?22:55
*** mlavalle has quit IRC23:18

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!