Tuesday, 2013-10-15

openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Reduce vm creations in negative tests  https://review.openstack.org/5037400:16
*** dims has quit IRC00:16
*** sthaha has joined #openstack-qa00:18
*** sthaha has joined #openstack-qa00:18
*** dims has joined #openstack-qa00:24
openstackrecheckopenstack/neutron change: https://review.openstack.org/51696 failed tempest because of: https://bugs.launchpad.net/bugs/123548600:32
*** weshay has quit IRC00:32
openstackrecheckopenstack/neutron change: https://review.openstack.org/51738 failed tempest with an unrecognized error00:41
*** matsuhashi has joined #openstack-qa00:48
openstackrecheckopenstack/tempest change: https://review.openstack.org/50374 failed tempest with an unrecognized error01:04
*** nosnos has joined #openstack-qa01:05
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Reduce vm creations in negative tests  https://review.openstack.org/5037401:06
*** wanghao has joined #openstack-qa01:11
openstackrecheckopenstack/nova change: https://review.openstack.org/51753 failed tempest with an unrecognized error01:19
openstackrecheckopenstack/nova change: https://review.openstack.org/50465 failed tempest with an unrecognized error01:21
openstackrecheckopenstack/nova change: https://review.openstack.org/51752 failed tempest with an unrecognized error01:21
*** tianma has joined #openstack-qa01:23
openstackrecheckopenstack/nova change: https://review.openstack.org/50465 failed tempest with an unrecognized error01:23
openstackrecheckopenstack/nova change: https://review.openstack.org/51746 failed tempest because of: https://bugs.launchpad.net/bugs/121839101:23
*** dims has quit IRC01:28
*** dims has joined #openstack-qa01:43
openstackgerritzhangyanzi proposed a change to openstack/tempest: add volume tests  https://review.openstack.org/5135901:52
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: Fix false negative, when HEAT_CREATE_TEST_IMAGE is unset  https://review.openstack.org/5133601:57
*** dkranz has joined #openstack-qa02:17
openstackgerritYang Yu proposed a change to openstack/tempest: Adding tests for image tags API  https://review.openstack.org/5155302:28
*** yfujioka has joined #openstack-qa02:31
openstackrecheckopenstack/neutron change: https://review.openstack.org/51765 failed tempest because of: https://bugs.launchpad.net/bugs/122400102:32
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API update /v1/images/<image_id>  https://review.openstack.org/4857803:00
openstackrecheckopenstack/tempest change: https://review.openstack.org/51553 failed tempest with an unrecognized error03:07
*** dims has quit IRC03:08
openstackgerritYang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> tests  https://review.openstack.org/4798303:08
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API  https://review.openstack.org/4857803:32
*** SergeyLukjanov has joined #openstack-qa03:33
openstackgerritChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension  https://review.openstack.org/5054203:38
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for aggregate  https://review.openstack.org/4915703:52
*** SergeyLukjanov is now known as _SergeyLukjanov03:58
*** _SergeyLukjanov has quit IRC03:58
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some negative tests for attach_interfaces  https://review.openstack.org/4923104:04
*** prekarat has quit IRC04:07
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for host and seperate negative ones  https://review.openstack.org/4977804:08
openstackrecheckopenstack/tempest change: https://review.openstack.org/47983 failed tempest with an unrecognized error04:10
*** sourabh has joined #openstack-qa04:11
*** sourabh has left #openstack-qa04:11
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for hypervisor operation  https://review.openstack.org/4987104:11
*** masayukig has joined #openstack-qa04:16
*** yfried_mob has joined #openstack-qa04:30
*** yfried_mob has quit IRC04:35
*** wanghao has quit IRC04:49
*** yfried has joined #openstack-qa04:54
*** matsuhashi has quit IRC05:11
*** SergeyLukjanov has joined #openstack-qa05:11
*** matsuhashi has joined #openstack-qa05:11
*** matsuhashi has quit IRC05:13
*** matsuhashi has joined #openstack-qa05:14
*** prekarat1 has joined #openstack-qa05:15
*** tianma has quit IRC05:19
*** matsuhashi has quit IRC05:22
*** matsuhashi has joined #openstack-qa05:23
openstackgerritYang Yu proposed a change to openstack/tempest: Adding tests for image tags API  https://review.openstack.org/5155305:26
*** mtreinish has quit IRC05:27
*** yeylon_ has joined #openstack-qa05:27
*** matsuhashi has quit IRC05:27
*** yfried_mob has joined #openstack-qa05:28
__afazekasmkoderer: https://review.openstack.org/#/c/50374/05:28
openstackgerritYang Yu proposed a change to openstack/tempest: Adding tests for image tags API  https://review.openstack.org/5155305:28
*** ivanzhu1 has joined #openstack-qa05:29
*** mtreinish has joined #openstack-qa05:29
*** afazekas has joined #openstack-qa05:29
*** __afazekas has quit IRC05:30
*** matsuhashi has joined #openstack-qa05:34
*** Madan has joined #openstack-qa05:35
*** yfried_mob has quit IRC05:35
*** yfried_mob has joined #openstack-qa05:36
*** afazekas has quit IRC05:43
*** turul_ has joined #openstack-qa05:43
*** turul_ is now known as afazekas05:43
*** mtreinish has quit IRC05:44
*** ken1ohmichi has joined #openstack-qa05:53
*** mtreinish has joined #openstack-qa05:55
openstackrecheckopenstack/tempest change: https://review.openstack.org/51553 failed tempest with an unrecognized error05:57
*** rdekel has joined #openstack-qa06:08
Anjuafazekas:  ping06:11
afazekasAnju: pong06:11
Anjuafazekas:  https://github.com/openstack/tempest/blob/master/tempest/thirdparty/boto/test.py#L54306:14
Anjuwhat is this for06:14
afazekasIt is for ec2 error code checking06:15
*** coolsvap has joined #openstack-qa06:15
Anjuafazekas: so here also the name "Duplicate" should change?06:19
afazekasnot06:19
*** rdekel has quit IRC06:20
afazekasThese codes are part of the ec2 specification,  they should never be anything else06:20
openstackrecheckopenstack/neutron change: https://review.openstack.org/48382 failed tempest because of: https://bugs.launchpad.net/bugs/122400106:21
*** wanghao has joined #openstack-qa06:23
Anjuafazekas:  thanks06:25
*** tianma has joined #openstack-qa06:26
*** jlibosva1 has joined #openstack-qa06:27
*** yfried_mob has joined #openstack-qa06:34
afazekascyeoh, mkoderer, dkranz:  https://review.openstack.org/#/c/50374/6   - please review it - the base.py changes will be used for critical bug fix06:38
*** afazekas has quit IRC06:38
*** rdekel has joined #openstack-qa06:53
*** marun has joined #openstack-qa06:56
openstackgerritA change was merged to openstack/tempest: Rename cli tests from compute to nova  https://review.openstack.org/5041907:03
*** afazekas has joined #openstack-qa07:03
*** rdekel has quit IRC07:08
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Fix to use proper random values  https://review.openstack.org/4688607:10
openstackgerritJoe Hakim Rahme proposed a change to openstack/tempest: Adds tests to cover Swift's crossdomain middleware  https://review.openstack.org/5063707:15
ivanzhu1hi, afazekas could you help to review this : https://review.openstack.org/#/c/50822/ ? how can we enable the vnc on gerrit?07:18
afazekashttps://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L6907:25
afazekasn-vnc,n-cauth07:25
afazekasivanzhu1: you can propose a patch to devstack-gate in order to enable it07:26
ivanzhu1afazekas: got it, thanks:)07:27
afazekasn-novnc,n-auth07:28
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/50923 failed tempest with an unrecognized error07:30
*** yfried_mob has quit IRC07:31
*** yfried_mob has joined #openstack-qa07:32
*** masayukig_ has joined #openstack-qa07:39
*** masayukig has quit IRC07:40
*** Dharmit has joined #openstack-qa07:40
*** mtreinish has quit IRC07:41
openstackgerritIvan-Zhu proposed a change to openstack-infra/devstack-gate: enable novnc and xvpvnc  https://review.openstack.org/5179007:41
*** mtreinish has joined #openstack-qa07:42
*** giulivo has joined #openstack-qa07:43
*** ken1ohmichi has quit IRC07:43
*** SergeyLukjanov has quit IRC07:46
*** jhenner has joined #openstack-qa07:52
*** fbo_away is now known as fbo07:54
Anjuafazekas:  is there  any limit to add volumes in a vm07:56
*** yassine has joined #openstack-qa07:57
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some negative tests for attach_interfaces  https://review.openstack.org/4923107:57
afazekasAnju: the pci bus limits it07:59
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for aggregate  https://review.openstack.org/4915707:59
Anjuafazekas:  can you please clear more07:59
kongAnju, maybe you mean attach-volume07:59
Anjuafazekas:  pci bus limits?08:00
Anjuyes kong08:00
afazekasAs I remember every attached volume will be new PCI device on the guest system08:05
afazekasThe kvm provides a single pci bus with it's usual limitations08:05
afazekashttp://en.wikipedia.org/wiki/PCI_configuration_space08:06
kongAnju, afazekas is right.08:06
kongIt's much like hard disks of your physical computer.08:06
*** ivanzhu has joined #openstack-qa08:07
*** ivanzhu1 has quit IRC08:08
openstackgerritJoe Hakim Rahme proposed a change to openstack/tempest: Adds test to cover Swift healthcheck middleware  https://review.openstack.org/5179508:09
openstackrecheckopenstack/neutron change: https://review.openstack.org/45677 failed tempest with an unrecognized error08:11
raiesafazekas: as per http://docs.openstack.org/grizzly/openstack-network/admin/content/use_cases_tenant_router.html subnet overlappig is possible08:12
masayukig_Hi, does anyone review my patch? This was approved. But a conflict occurred. So I've rebased and resolved it. https://review.openstack.org/#/c/46886/08:12
afazekasAnju: You can have ~20 volumes, but it depends on how many other pci device you have in the vm.08:12
raiesafazekas: can you please let assist me on subnet overlapping ?08:13
raiesI mean which subnets we are talking about ?08:13
*** coolsvap has quit IRC08:14
Anjuafazekas:  is this methioned in any conf file?08:14
*** masayukig_ has left #openstack-qa08:14
Anjus/ methioned/ mentioned08:14
afazekasAFAIK If you have namespace support the  subnets must not overlap within the same network, If you do not have name-space support it must be globally unique08:14
*** matsuhas_ has joined #openstack-qa08:14
*** matsuhashi has quit IRC08:15
*** masayukig has joined #openstack-qa08:15
*** matsuhashi has joined #openstack-qa08:15
afazekasAnju: the libvirt's xml file specifies the devices, but you can add new network and block device, ... at runtime08:15
raiesi.e here we are talking about subnets within same network ?08:16
raiesafazekas: ^^08:17
raiesAs per my understanding, document is talking about subnet overlapping with subnets from different private networks. wht do you say ?08:18
*** matsuhas_ has quit IRC08:19
afazekas'tenant subnets can be overlapping without conflicting, since access to external networks all happens via SNAT or Floating IPs.'  if you have configured this behavior in neutron and your kernel supports the network namespace it is true08:20
afazekasbut the subnets cannot overlap in the same 'net' even in this case.08:20
*** jamielennox is now known as jamielennox|away08:21
raiesafazekas: Yes I wanted to know only this (whether overlapping subnets belong to same network or different network).08:22
afazekashttps://github.com/openstack/neutron/blob/master/etc/neutron.conf#L8608:22
*** yeylon_ has quit IRC08:23
afazekasAFAIK The overlapping just allowed if the overlapping subnet is in a different network08:24
raiesAfazekas: I have studied usecase "Per-tenant router and private network" using OpenvSwitch, there is also network isolation is described using namespaces and subnet overlapping is possible08:25
raiesbut it was not clear more that overlapping subnets are from different network or not08:25
raiesBut as you described overlapping subnets must be from different networks of same tenant08:26
raiesafazekas: May there be overlapping between network subnets belonging to private networks of different tenent ?08:27
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some negative tests for flavor  https://review.openstack.org/4958808:27
afazekaslast time I tried I was able to create two networks with the same subnet cidr in the same tenant08:27
raiestwo networks with same cidr ?? I think you mean  two subnets with same cidr right ?08:28
afazekasyes08:29
*** yfried_mob has quit IRC08:29
*** yfried_mob has joined #openstack-qa08:29
raiesand these subnets belong to same tenant or different one ?08:29
afazekasso the overlapping  is forbidden per 'net', not per tenant08:30
*** safchain has joined #openstack-qa08:30
raiesafazekas: Please a bit elaborate, I didn't get this ?08:31
raiesYou mean that, it does not matter that whether overlapping network may belong to same or different tenant ?08:34
raiesor you mean that overlapping network will be belonging to same tenant ?08:35
raiesafazekas: are you there ?08:36
*** kong has quit IRC08:39
*** BobBall has joined #openstack-qa08:39
afazekasraies: http://www.fpaste.org/46786/18264651/08:41
openstackgerritYa Hong Du proposed a change to openstack-infra/devstack-gate: Enable Neutron FWaaS service  https://review.openstack.org/5179808:41
BobBallafazekas: What's your view on what we should do about the libvirt tempest test for get_server_diagnostics? https://review.openstack.org/#/c/51412/ - There's a half-started discussion on the ML about it, but only the driver maintainers have said something so far, basically that we all agree something should change but no real answers as to what08:43
raiesafazekas: I get that, as you explained above it is correct. But my question was that overlapping subnets may belong to different tenant or same tenant ?08:43
afazekasraies: But I cannot add the two subnet to the same router08:43
raiesIs it functionality ?08:45
*** prekarat has joined #openstack-qa08:46
afazekasBobBall: testing it an incomplete way, is better than not testing it. The test should be able to work in a not libvirt/kvm/qemu environment08:46
raies<tenantA><netA><subnetA><TenantA>---------------------------<tenantB><netB><subnetB><TenantB>08:46
*** prekarat1 has quit IRC08:46
raiesoverlapping in above case08:46
raieswhether possible or not ?08:47
BobBallafazekas: So you think changing the test to find out what driver we're using and only run the test if we're using libvirt?08:49
afazekasif the https://github.com/openstack/neutron/blob/master/etc/neutron.conf#L86 is true + https://github.com/openstack/neutron/blob/master/etc/l3_agent.ini#L23 true + kernel has netns support , your example is allowed08:49
raiesok08:49
BobBallafazekas: perhaps having a new variable in tempest.conf?08:49
raiesafazekas: is the previous fpase example you told that two subnets are being attached to same router ? Is it functionality ?08:50
BobBallor is there a nice way to access the nova variables from tempest?08:50
afazekasBobBall: I am not sure,  I am ok with the submitted patch, and I ok with the new config var proposal as well.08:51
afazekasBobBall: we should not access the nova variables from tempest08:51
BobBallok - that'd need a new var in tempest to say which driver we're using then08:51
BobBallhmmm unless nova can tell us08:51
afazekasBobBall: I would prefer if almost all test case would be able to work remotely without having true admin (root) permission on the target system08:52
BobBallsure08:52
afazekasBobBall: I think we should ask sdague's opinion about this question08:52
raiesafazekas: As per http://docs.openstack.org/grizzly/openstack-network/admin/content/ch_adv_notification_overview.html notification of CRUD can be found using setting in conf file, Can be have notification of overlapping IPs. If yes then How to set the conf for it ??08:53
BobBallhe was going to be my next port of call - but I thought I'd ask yours first since you were already on the review :)08:53
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/51638 failed tempest with an unrecognized error08:58
afazekasraies: probbaly you need to modify the  neutron's code, if it does not send the event what you are looking for09:01
afazekasgiulivo: https://review.openstack.org/#/c/50374/609:05
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/51633 failed tempest with an unrecognized error09:12
openstackrecheckopenstack/tempest change: https://review.openstack.org/49588 failed tempest with an unrecognized error09:12
openstackrecheckopenstack/neutron change: https://review.openstack.org/49478 failed tempest with an unrecognized error09:12
openstackrecheckopenstack/nova change: https://review.openstack.org/51670 failed tempest with an unrecognized error09:12
*** yfried_mob has quit IRC09:26
*** yfried_mob has joined #openstack-qa09:27
*** prekarat has quit IRC09:33
*** prekarat has joined #openstack-qa09:34
raiesafazekas: then any alternative way to get notification for such things ?09:34
*** yfujioka has quit IRC09:36
*** openstackgerrit has quit IRC09:38
*** ivanzhu has quit IRC09:46
afazekasraies: Which event exactly you are looking for ? subnet creation ? subnet creation rejected because of overlapping ? subnet created with a overlapping , without  rejection ?09:47
raiesafazekas: all of them09:48
openstackrecheckopenstack/python-ceilometerclient change: https://review.openstack.org/51783 failed tempest with an unrecognized error09:55
afazekashttp://www.fpaste.org/46799/38183093/ this is what you can have in logs without code modification09:56
raiesafazekas: Basically there are three requirements 1. as shown above(CRUD and overlapping notification) 2.notification related to anymismatch in rabbitmq 3. notification for inconsistency in  database09:56
raiesfor 2. I can see a file in neutron module "rabbit_notifier.py"(rpc_notifier)09:59
raiesbut I donot know how to enable these09:59
openstackrecheckopenstack/neutron change: https://review.openstack.org/48382 failed tempest because of: https://bugs.launchpad.net/bugs/122400109:59
afazekas'notification_driver = neutron.openstack.common.notifier.rpc_notifier'  is in the default devstack config10:00
afazekasyou need to have a program which read those notifications from the "message bus"10:00
raiesmean during my cloud deployment I have write a techniqe to read those notifications from the "message bus" ?10:01
*** wanghao has quit IRC10:02
raiesafazekas: will you a bit help me on this topic reading from message buffer10:05
*** mkollaro has joined #openstack-qa10:05
*** yeylon_ has joined #openstack-qa10:08
raiesafazekas: are you there ?10:11
afazekasnot now10:15
raiesafazekas: one more thing I would like to ask on 3. notification for inconsistency in  database10:17
afazekashttps://wiki.openstack.org/wiki/Oslo/Messaging#oslo.notify10:17
afazekasraies: you would need to change neutron code to generate this event for you10:18
raiesI think that we should use "MySQL Enterprise Monitor"  for monitoring and notification. What is currently used for db notification10:18
*** jlibosva1 has quit IRC10:21
afazekasraies: do you want debug something or are you looking for production system monitoring solution ?10:23
*** yfried_mob has quit IRC10:24
afazekasdo you need statistics or exact happening times ?10:24
*** yfried_mob has joined #openstack-qa10:24
raiesexact happening times10:24
*** nosnos has quit IRC10:28
*** nosnos has joined #openstack-qa10:28
*** prekarat has quit IRC10:32
*** prekarat has joined #openstack-qa10:32
*** nosnos has quit IRC10:33
afazekasraies: why the screen-logs not good enough for you ?10:35
afazekas$ ceilometer sample-list -m subnet.create  ?10:39
raiesafazekas: actually I need to know and use more about notification on these topics10:42
raiesbut no one of above 3 schenarios were resolved till :)10:43
raies:(10:43
raiesI just want if there is something mismatch then there should be a notification raised for above three topics10:44
afazekashttp://exchange.nagios.org/directory/Plugins/Log-Files/Check-File-Content/details10:45
afazekasDo you need a special logger which sends all error/critical/trace level message on AMQP?10:48
*** yfried has quit IRC10:51
raiesNo I do not want to write anything, I just need to configure somehow to give notifications10:52
raiesafazekas: when rpc_notifier is enabled in conf file then the events generated on AMQP (Info, warn, error etc) will be logged ? If yes then where I can see them ?10:56
*** yfried has joined #openstack-qa10:58
raiesafazekas: will you please give some input on this ?11:00
openstackrecheckopenstack/nova change: https://review.openstack.org/51812 failed tempest because of: https://bugs.launchpad.net/bugs/122566411:20
afazekasraies: on debug level they will be logged as any AMQP operation, but if you also configure the 'log' notifier it will be logged on the configured log level11:21
*** yfried_mob has quit IRC11:22
*** yfried_mob has joined #openstack-qa11:22
*** dkranz has quit IRC11:27
*** prekarat1 has joined #openstack-qa11:31
*** prekarat has quit IRC11:31
Anjuafazekas:  500 http code for serverFault11:33
*** prekarat1 has quit IRC11:35
*** jlibosva has joined #openstack-qa11:35
*** dims has joined #openstack-qa11:36
*** jlibosva1 has joined #openstack-qa11:36
*** jlibosva has quit IRC11:39
BobBallsdague: ping11:41
sdaguepong11:41
BobBallhowdy :)11:41
BobBallWe've got a fun question for you11:41
BobBalla recent change to tempest added a libvirt-specific test11:41
BobBallchecking the output of get_compute_diagnostics11:42
BobBallThis fails tempest for xen, powervm and hyper-v (probably the others too)11:42
*** marun has quit IRC11:42
BobBallHow do you want to handle it?  Remove the test? Make it less restrictive?  Add some driver-specific flags to tempest.conf?11:42
BobBallhttps://review.openstack.org/#/c/51412/11:42
sdagueso, actually, how did we get a nova api that is hypervisor specific, but doesn't identify itself?11:43
BobBallthe API isn't hypervisor specific itself11:43
BobBallbut the tempest test is verifying the response11:43
BobBallwhich isn't restricted by any API - it just returns a dict11:43
BobBallthe test looks for the libvirt values in the dict11:43
sdagueBobBall: right, but an API that just returns some random dict without schema isn't really an API11:44
BobBallindeed11:44
sdaguehow do you programatically handle the results?11:44
BobBallyou don't - the current use case is the admin looking at the result from the command line11:44
*** marun has joined #openstack-qa11:44
BobBallnothing is programatically using it yet11:44
BobBallalthough ceilometer apparently wants to use it11:44
sdagueso I think the actual fix here is on the nova side, we need to define out the return data on that API11:45
BobBall(my previous assertion is based on a comment from Matt Reiderman who looked into it - not me)11:45
BobBallOk, I agree that we need a proper API for this11:46
BobBallbut I suspect the discussion and agremenet of the API will take a while11:46
BobBallI'm keen to get a fix into tempest that un-breaks our internal CI in the mean time11:46
sdaguehere's my suggested approach11:47
sdaguefile a bug on Nova for not defining this API in a hypervisor agnostic way11:47
sdagueput in a skip for tempest, based on that bug11:47
BobBallahhh okay - good idea11:48
BobBallthanks11:49
BobBallI'll do that11:49
sdagueI don't think we should run any version of this test unless it's really a defined API, and this API hasn't seem to get there. It would also be good to drive something on the list about defining that return set, because randomly different formatted versions of things aren't good for anyone :)11:49
BobBallWe've got a thread on the ML already11:49
BobBallbut not much response to it :)11:49
BobBallthread is "looking for clarification on the diagnostics API"11:50
*** mriedem has joined #openstack-qa11:50
sdagueyeh, so my suggestion is just propose a json schema there :)11:50
BobBallhttps://bugs.launchpad.net/nova/+bug/1240043 FYI11:53
openstackrecheckopenstack/nova change: https://review.openstack.org/50465 failed tempest because of: https://bugs.launchpad.net/bugs/121839112:00
Anjusdague:  ping ,I need a suggestion12:01
*** safchain has quit IRC12:02
*** yassine has quit IRC12:04
*** jhenner has quit IRC12:05
*** afazekas has quit IRC12:05
*** jhenner has joined #openstack-qa12:05
*** yfried_mob has quit IRC12:06
*** mkollaro has quit IRC12:06
Anjusdague:  can we change the name of exception Duplicate to Conflict12:06
*** mkollaro has joined #openstack-qa12:06
*** afazekas has joined #openstack-qa12:10
*** yassine has joined #openstack-qa12:12
*** marun has quit IRC12:15
raiesafazekas: pvt12:16
sdagueAnju: I guess, I'm not sure what that gets us12:18
Anjusdague:  409 depicts Conflict12:18
sdagueis there a patch in question here?12:19
Anjusdague:  https://review.openstack.org/#/c/51835/12:19
*** jhenner has quit IRC12:19
*** yfried_mob has joined #openstack-qa12:20
*** jhenner has joined #openstack-qa12:20
sdagueyeh, that's probably fine, we'll see what test results look like12:20
Anjusdague:  thanks12:22
sdagueafazekas: https://review.openstack.org/#/c/50267/ needs rebasing12:22
*** adalbas has joined #openstack-qa12:26
cyeohsdague: btw would you mind having another look at https://review.openstack.org/#/c/46941/3 - I've responded to your question12:26
*** Madan has quit IRC12:27
sdaguecyeoh: lgtm12:29
sdaguethanks12:30
Anjusdague:  in Thursday meeting it was decided that all  negative tests will be in a different file .12:35
cyeohsdague: thx!12:37
cyeohAnju: just wondering what the reason for that is? We have a negative tag if we want to include/exclude negative tests.12:39
*** marun has joined #openstack-qa12:40
Anjucyeoh:  yes it was said that , we have to maintain a different file with a negative tag12:41
cyeohAnju: ok12:43
*** weshay has joined #openstack-qa12:45
sdaguecyeoh: the reason it got discussed was that there are differences of opinions about where "negative" tests really belong, and we've been getting a *ton* of them recently12:49
sdaguecloser to the projects might be better, so for now we're taking new tests in separate files to make a possible future separation easier12:49
*** adalbas has quit IRC12:50
*** mkollaro has quit IRC12:58
*** mkollaro1 has joined #openstack-qa12:58
*** afazekas has quit IRC13:00
*** openstackgerrit has joined #openstack-qa13:02
*** jhenner has quit IRC13:03
*** tiantian has joined #openstack-qa13:06
openstackgerritA change was merged to openstack/tempest: Update to latest tox  https://review.openstack.org/4955913:06
*** yfried_mob has quit IRC13:07
*** yfried_mob has joined #openstack-qa13:07
*** yfried|mob has joined #openstack-qa13:07
openstackgerritA change was merged to openstack/tempest: remove tearDownClass in test_quotas.py  https://review.openstack.org/5163213:08
*** tianma has quit IRC13:08
openstackgerritA change was merged to openstack/tempest: Restores passing additional parameters to nosetest  https://review.openstack.org/5081913:09
openstackgerritA change was merged to openstack/tempest: add more test cases to api/network/security_groups  https://review.openstack.org/5112313:11
openstackgerritA change was merged to openstack/tempest: Add a create_router utility function  https://review.openstack.org/4755313:11
openstackgerritBob Ball proposed a change to openstack/tempest: Skip get_server_diagnostics test until bug 1240043 is fixed  https://review.openstack.org/5141213:15
*** yfried|mob has quit IRC13:17
*** yfried|mob has joined #openstack-qa13:17
*** jhenner has joined #openstack-qa13:17
*** adalbas has joined #openstack-qa13:20
*** yassine has quit IRC13:23
*** yassine has joined #openstack-qa13:23
*** afazekas has joined #openstack-qa13:25
*** thomasbiege1 has joined #openstack-qa13:32
afazekassdague: NOTE: If we merge the negative tests to a single test case it also means we can just skip them only together https://review.openstack.org/#/c/46941/313:33
*** safchain has joined #openstack-qa13:33
openstackgerritFlorian Haas proposed a change to openstack-dev/devstack: fixup_stuff.sh: don't break on prettytable perms fixup on RHEL  https://review.openstack.org/5184313:34
openstackgerritRoman Prykhodchenko proposed a change to openstack-dev/devstack: Enable keystone authentication in Ironic  https://review.openstack.org/5184513:42
*** safchain has quit IRC13:47
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Move negative action tests to right place  https://review.openstack.org/4998213:48
*** safchain has joined #openstack-qa13:50
openstackgerritFlavio Percoco proposed a change to openstack-dev/devstack: Add marconi support to devstack  https://review.openstack.org/4799913:51
*** thomasbiege1 has quit IRC13:56
raiesafazekas: where can we enable "namespace" in conf file ?13:57
*** gszasz has joined #openstack-qa14:02
*** rdekel has joined #openstack-qa14:03
*** kong has joined #openstack-qa14:09
kongmriedem, ping14:13
kongmriedem, can you explain more specific?14:13
kongabout https://review.openstack.org/#/c/49231/14:13
mriedemkong: you don't need this change in that patch14:14
mriedemhttps://review.openstack.org/#/c/49231/9/tempest/api/compute/servers/test_attach_interfaces.py14:14
mriedemthat's a cleanup change which doesn't have anything to do with adding more negative tests14:14
openstackgerritRoman Prykhodchenko proposed a change to openstack-dev/devstack: Install Ironic client  https://review.openstack.org/5185314:14
mriedemyou should keep isolated cleanup patches separate from functional changes14:14
*** ivanzhu has joined #openstack-qa14:15
openstackrecheckopenstack/nova change: https://review.openstack.org/43190 failed tempest with an unrecognized error14:15
*** yfried|mob has quit IRC14:15
kongor in which scenaira we should ensure the order of 'packages import'?14:15
*** yfried|mob has joined #openstack-qa14:15
kongs/scenaira/scenario14:15
*** SergeyLukjanov has joined #openstack-qa14:16
mriedemkong: if (1) you're touching the imports for another change or (2) you're doing a specific cleanup patch14:17
mriedemkong: i'm trying to find the wiki page that describes how these should be isolated, sec14:18
kongmriedem, ok, thanks for doing that14:18
mriedemkong: https://wiki.openstack.org/wiki/GitCommitMessages#Things_to_avoid_when_creating_commits14:19
sdaguedtroyer: https://review.openstack.org/#/c/51711/ this is the easy runner for the bash8 bits, if you want to take a look. I'm working on the gate job for this as well, but we'll need that in place before doing it14:20
kongmriedem, you prefer that we should keep them stay where they are, except we really want do some cleanup or happened to touch import?14:20
dtroyersdague: yup, playing with it now…I think we should add *rc too14:21
sdaguedtroyer: good call14:21
mriedemkong: your patch https://review.openstack.org/#/c/49231/ adds new negative tests in a new file, therefore it doesn't even touch test_attach_interfaces.py, except you re-ordered the imports14:21
mriedemthat is an unrelated change14:21
mriedemkong: i'm not opposed to cleaning up the import order in test_attach_interfaces.py, but it should be done in a separate patch14:21
mriedemas the wiki above describes (see the 2nd bullet)14:22
kongmriedem, got it. I should change it in anther patch14:22
mriedemkong: yup14:22
*** rdekel has quit IRC14:22
kongmriedem, I thought we should do things right once for all...14:22
dtroyersdague: I think I want run_tests.sh to eventually execute tests/* too14:22
dtroyersdague: but they need some attention first14:23
sdaguedtroyer: sure14:23
openstackgerritA change was merged to openstack/tempest: Skip get_server_diagnostics test until bug 1240043 is fixed  https://review.openstack.org/5141214:23
sdagueso in that case we should probably give it flags for style checking and unit testing14:23
dtroyerwhen the time comes...14:24
sdagueyep14:24
sdagueUNIT TEST ALL THE THINGS14:24
kongmriedem, thanks!14:25
dtroyerits unit tests all the way down14:25
mriedemkong: fixing it is one thing, but with git you must keep things isolated and small if possible since it's all based on diffs and patches14:25
mriedemkong: so as the wiki explains, if a patch needs to be reverted, you don't want to undo unrelated changes14:25
mriedemtherefore patches should be small and isolated14:25
mtreinishsdague: '-p' to be consistent? (even though it's not pep8)14:26
sdagueyeh, probably14:26
*** rdekel has joined #openstack-qa14:29
*** yfried has quit IRC14:29
*** SergeyLukjanov has quit IRC14:32
*** BobBall has left #openstack-qa14:33
*** safchain has quit IRC14:33
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some negative tests for attach_interfaces  https://review.openstack.org/4923114:33
*** safchain has joined #openstack-qa14:33
*** matsuhashi has quit IRC14:34
*** matsuhashi has joined #openstack-qa14:35
*** SergeyLukjanov has joined #openstack-qa14:38
*** safchain has quit IRC14:38
*** SergeyLukjanov is now known as _SergeyLukjanov14:38
*** matsuhashi has quit IRC14:39
*** _SergeyLukjanov has quit IRC14:39
kongsdague, could you please take a look at https://review.openstack.org/#/c/51341/ and give some suggestions to me, very appreciated for that!14:44
*** adalbas has quit IRC14:45
*** rdekel has quit IRC14:50
raiesafazekas: hi  are you there ?14:51
*** SergeyLukjanov has joined #openstack-qa14:52
raiesafazekas: I want to check rabbitmq robustness in terms of "number of simultaneous request" (for example 15). How can I check it ?14:52
openstackgerritA change was merged to openstack-dev/devstack: add bash8 tool (like pep8, but way hackier)  https://review.openstack.org/5167614:54
openstackgerritSteven Hardy proposed a change to openstack/tempest: keystone : Add coverage of v2 client trust scoping  https://review.openstack.org/5156014:56
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient v3 auth tests  https://review.openstack.org/5155814:56
openstackgerritSteven Hardy proposed a change to openstack/tempest: Add initial keystoneclient trusts tests  https://review.openstack.org/5155914:56
afazekasraies: do you want to connect directly to the rabbit ?14:57
raiesyes14:57
afazekashttp://www.youtube.com/watch?v=1S1fISh-pag if you really need a high load :)14:58
raiesthanks afazekas I will check it now14:59
afazekasthe easy dirty way is just download some example cusnumer and provider code and create shell script wich start 32 process  in the background14:59
raiesok I will try it15:00
afazekasThe message brokers are designed for even higher load15:01
raiesok15:02
afazekasusually creating a new connection has relatively high cost, so you can try to reuse the connection unless you would like to stress the initial connection negation-ion parts15:03
* afazekas bye15:03
*** afazekas has quit IRC15:04
openstackrecheckopenstack/heat change: https://review.openstack.org/51712 failed tempest with an unrecognized error15:10
*** yfried|mob has quit IRC15:12
*** yfried_mob has joined #openstack-qa15:13
openstackrecheckopenstack/cinder change: https://review.openstack.org/43887 failed tempest with an unrecognized error15:15
openstackgerritAnju Tiwari proposed a change to openstack/tempest: Changed the exception name  https://review.openstack.org/5183515:26
*** tiantian has quit IRC15:28
*** tiantian has joined #openstack-qa15:28
openstackrecheckopenstack/nova change: https://review.openstack.org/43190 failed tempest with an unrecognized error15:31
giulivoping sdague15:34
sdaguepong15:34
giulivoso shall I still remove the conflict line from this https://review.openstack.org/#/c/49491/15:34
giulivo?15:34
sdaguegiulivo: no, I'll +215:34
sdaguemy bad15:34
giulivonp, just making sure15:34
sdaguesorry it took so long to circle back around15:35
*** DennyZhang has joined #openstack-qa15:36
*** torandu has left #openstack-qa15:39
*** prekarat has joined #openstack-qa15:41
*** prekarat has left #openstack-qa15:41
*** coolsvap has joined #openstack-qa15:44
*** mlavalle has joined #openstack-qa15:46
*** yassine has quit IRC15:46
*** yfried_mob has quit IRC15:49
*** gszasz has quit IRC15:50
*** ivanzhu has quit IRC15:51
*** kong has quit IRC15:52
*** DennyZhang has quit IRC15:55
*** yeylon_ has quit IRC15:56
*** jlibosva1 has quit IRC16:01
*** marun has quit IRC16:03
openstackgerritA change was merged to openstack-dev/devstack: add a simple run_tests.sh to use bash8  https://review.openstack.org/5171116:05
*** adalbas has joined #openstack-qa16:09
*** sthaha has quit IRC16:10
*** fbo is now known as fbo_away16:18
*** SergeyLukjanov has quit IRC16:20
*** SergeyLukjanov has joined #openstack-qa16:41
*** SergeyLukjanov is now known as _SergeyLukjanov16:42
*** _SergeyLukjanov is now known as SergeyLukjanov16:42
*** SergeyLukjanov is now known as _SergeyLukjanov16:42
*** _SergeyLukjanov is now known as SergeyLukjanov16:42
openstackrecheckopenstack/swift change: https://review.openstack.org/51866 failed tempest with an unrecognized error16:46
*** mriedem has quit IRC16:57
openstackrecheckopenstack/neutron change: https://review.openstack.org/51872 failed tempest with an unrecognized error16:58
*** Dharmit has quit IRC17:02
*** mriedem has joined #openstack-qa17:12
openstackrecheckopenstack/swift change: https://review.openstack.org/51875 failed tempest with an unrecognized error17:13
*** mlavalle has quit IRC17:24
*** DennyZhang has joined #openstack-qa17:25
openstackrecheckopenstack/neutron change: https://review.openstack.org/50389 failed tempest with an unrecognized error17:30
*** openstackgerrit has quit IRC17:37
*** openstackgerrit has joined #openstack-qa17:38
*** rockyg has joined #openstack-qa17:41
*** DennyZhang has quit IRC17:49
*** afazekas has joined #openstack-qa17:52
*** yfried_mob has joined #openstack-qa17:52
*** atiwari has joined #openstack-qa18:00
*** jamielennox|away is now known as jamielennox18:04
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for bug 1239856  https://review.openstack.org/5172818:05
*** yfried_mob has joined #openstack-qa18:05
*** dims has quit IRC18:10
*** salv-orlando_ has joined #openstack-qa18:16
*** salv-orlando has quit IRC18:16
*** salv-orlando_ is now known as salv-orlando18:16
*** yfried_mob has quit IRC18:22
openstackrecheckopenstack/nova change: https://review.openstack.org/51752 failed tempest with an unrecognized error18:25
openstackrecheckopenstack/nova change: https://review.openstack.org/51894 failed tempest with an unrecognized error18:28
openstackrecheckopenstack/nova change: https://review.openstack.org/51893 failed tempest with an unrecognized error18:28
openstackrecheckopenstack/glance change: https://review.openstack.org/37420 failed tempest with an unrecognized error18:30
openstackrecheckopenstack/nova change: https://review.openstack.org/51753 failed tempest with an unrecognized error18:30
*** dkorn has joined #openstack-qa18:35
*** SergeyLukjanov has quit IRC18:35
*** SergeyLukjanov has joined #openstack-qa18:39
*** rdekel has joined #openstack-qa18:51
*** klrmn has left #openstack-qa18:59
*** yfried_mob has joined #openstack-qa19:07
openstackrecheckopenstack/neutron change: https://review.openstack.org/50389 failed tempest because of: https://bugs.launchpad.net/bugs/123548619:12
*** yfried_mob has quit IRC19:19
*** rdekel has quit IRC19:19
*** mlavalle has joined #openstack-qa19:32
*** SergeyLukjanov is now known as _SergeyLukjanov19:46
*** _SergeyLukjanov has quit IRC19:47
*** woodspa has joined #openstack-qa19:49
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add neutron parallel job  https://review.openstack.org/5127519:53
*** SergeyLukjanov has joined #openstack-qa19:58
openstackgerritMatthew Treinish proposed a change to openstack-infra/devstack-gate: Add a neutron parallel job  https://review.openstack.org/5191519:59
openstackrecheckopenstack/nova change: https://review.openstack.org/51893 failed tempest with an unrecognized error20:09
*** SergeyLukjanov has quit IRC20:11
*** dims has joined #openstack-qa20:18
*** yeylon_ has joined #openstack-qa20:44
*** dims has quit IRC20:45
openstackrecheckopenstack/neutron change: https://review.openstack.org/47432 failed tempest because of: https://bugs.launchpad.net/bugs/122400120:46
*** adalbas has quit IRC20:46
*** rockyg has quit IRC20:47
*** rockyg has joined #openstack-qa20:47
*** mkollaro1 is now known as mkollaro20:53
*** yeylon_ has quit IRC21:01
openstackrecheckopenstack/nova change: https://review.openstack.org/43984 failed tempest with an unrecognized error21:03
*** SergeyLukjanov has joined #openstack-qa21:12
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Add stack phases to extras.d handling  https://review.openstack.org/5193921:15
*** malini has joined #openstack-qa21:19
openstackrecheckopenstack/heat change: https://review.openstack.org/50877 failed tempest with an unrecognized error21:27
*** weshay has quit IRC21:43
*** coolsvap has quit IRC21:47
openstackrecheckopenstack/heat change: https://review.openstack.org/51936 failed tempest with an unrecognized error21:53
*** mriedem has quit IRC22:14
*** dkranz has joined #openstack-qa22:15
*** dkranz has quit IRC22:23
*** SergeyLukjanov is now known as _SergeyLukjanov22:34
*** dkranz has joined #openstack-qa22:34
*** _SergeyLukjanov has quit IRC22:34
*** SergeyLukjanov has joined #openstack-qa22:36
*** SergeyLukjanov has quit IRC22:44
dkranzsdague: I thought we were enforcing "negative tests in separate file" now: https://review.openstack.org/#/c/46941/322:53
*** mlavalle has left #openstack-qa22:57
openstackgerritJoe Gordon proposed a change to openstack/tempest: Add nova --os-cache test.  https://review.openstack.org/5107323:06
*** woodspa has quit IRC23:10
*** woodspa has joined #openstack-qa23:10
*** mriedem has joined #openstack-qa23:15
sdaguedkranz: it wasn't new test methods, so I wasn't going to also force it into another file. But I won't complain if someone else wants to -1 it23:16
*** jgriffith has quit IRC23:31
*** mkollaro has quit IRC23:31
openstackgerritA change was merged to openstack/tempest: Split out unit tests as separate tox jobs  https://review.openstack.org/4956223:32
*** mkollaro has joined #openstack-qa23:32
*** jgriffith has joined #openstack-qa23:32
*** mkollaro has quit IRC23:33
*** mkollaro has joined #openstack-qa23:33
*** mkollaro has quit IRC23:33
openstackrecheckopenstack/tempest change: https://review.openstack.org/50266 failed tempest with an unrecognized error23:35
*** atiwari has quit IRC23:36

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