| *** haomaiwa_ has quit IRC | 00:01 | |
| *** zenoway has joined #openstack-nova | 00:02 | |
| *** dave-mccowan has joined #openstack-nova | 00:03 | |
| mriedem | tjones: hyper-v isn't gating | 00:04 |
|---|---|---|
| mriedem | tjones: so ignore | 00:04 |
| mriedem | vmware api test changes don't care about hyper-v ci | 00:05 |
| tjones | oh ok - can you see why it is not merging? thought it was hyper-v | 00:05 |
| mriedem | ociuhandu: ^ known hyperv ci issues? | 00:05 |
| mriedem | tjones: the gate queue is backed up over 30 hours | 00:05 |
| mriedem | you can monitor it here if you want http://status.openstack.org/zuul/ | 00:06 |
| mriedem | but at over 30 hours, it'll probably be friday by the time it's merged, assuming it's not reset by a test failure | 00:06 |
| melwitt | hyperv ci has been failing for a couple of weeks it seemed to me | 00:06 |
| *** zenoway has quit IRC | 00:06 | |
| tjones | mriedem: ok thanks. i'll stop fretting. thought i needed to do something :-) | 00:07 |
| *** ZZelle_ has quit IRC | 00:08 | |
| *** hblixt has joined #openstack-nova | 00:08 | |
| *** diogogmt has joined #openstack-nova | 00:10 | |
| *** haomaiw__ has joined #openstack-nova | 00:10 | |
| *** lyarwood has joined #openstack-nova | 00:10 | |
| *** haomaiw__ has quit IRC | 00:10 | |
| *** lyarwood has quit IRC | 00:10 | |
| *** haomaiw__ has joined #openstack-nova | 00:10 | |
| *** lyarwood has joined #openstack-nova | 00:10 | |
| *** diogogmt_ has joined #openstack-nova | 00:13 | |
| Thelo | mriedem: when you said that the requirements should match this exactly "os-brick>=0.8.0 # Apache-2.0", what is the difference with the current version | 00:15 |
| alex_xu | oomichi: morning, I will address the comments | 00:15 |
| openstackgerrit | Tracy Jones proposed openstack/nova: remove mox from vmwareapi/test_driver_api.py https://review.openstack.org/270303 | 00:15 |
| *** diogogmt has quit IRC | 00:15 | |
| *** diogogmt_ is now known as diogogmt | 00:15 | |
| *** Fdaisuke has quit IRC | 00:17 | |
| mriedem | Thelo: there were spaces in the g-r version between the version and the apache part | 00:17 |
| *** ctrath has quit IRC | 00:17 | |
| *** PsionTheory has quit IRC | 00:17 | |
| mriedem | Thelo: like it looks like you have to literally copy/paste from the g-r change | 00:17 |
| mriedem | Thelo: see the linked job failure | 00:17 |
| *** tjones has quit IRC | 00:20 | |
| *** su_zhang has quit IRC | 00:20 | |
| Thelo | mriedem: the only difference I see is the "u" in front of each string | 00:21 |
| Thelo | mriedem: you mean it should be a tab rather than a space? | 00:23 |
| *** nic has quit IRC | 00:25 | |
| oomichi | alex_xu: cool :) | 00:27 |
| mriedem | Thelo: idk, but your change has this: | 00:27 |
| mriedem | os-brick>=0.8.0 # Apache-2.0 | 00:27 |
| mriedem | the g-r change has this: | 00:28 |
| mriedem | os-brick>=0.8.0 # Apache-2.0 | 00:28 |
| *** jinxing has joined #openstack-nova | 00:28 | |
| mriedem | so i assume the job is puking b/c those are not exactly the same | 00:28 |
| mriedem | so just copy/paste what's in https://review.openstack.org/#/c/267725/2/global-requirements.txt into https://review.openstack.org/#/c/253353/3/requirements.txt | 00:28 |
| *** mriedem is now known as mriedem_dinner | 00:28 | |
| *** dims has quit IRC | 00:29 | |
| oomichi | mriedem: have a good dinner :) | 00:29 |
| openstackgerrit | David Edery proposed openstack/nova: Fix create snapshot failure on VMs with SRIOV https://review.openstack.org/262341 | 00:32 |
| *** otter768 has joined #openstack-nova | 00:36 | |
| *** houming has joined #openstack-nova | 00:37 | |
| *** markvoelker_ has quit IRC | 00:41 | |
| *** otter768 has quit IRC | 00:41 | |
| openstackgerrit | Thelo Gaultier proposed openstack/nova: Add ITRI DISCO os-brick connector for libvirt https://review.openstack.org/253353 | 00:42 |
| *** su_zhang has joined #openstack-nova | 00:46 | |
| *** RichardRaseley has quit IRC | 00:51 | |
| *** slDabbler has quit IRC | 00:59 | |
| openstackgerrit | Wen Zhi Yu proposed openstack/nova: Make sure destination will rollback if live migration fails https://review.openstack.org/270093 | 00:59 |
| *** changbl has quit IRC | 00:59 | |
| *** haomaiw__ has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-nova | 01:01 | |
| *** su_zhang has quit IRC | 01:02 | |
| *** baoli_ has quit IRC | 01:03 | |
| *** mriedem_dinner is now known as mriedem | 01:05 | |
| *** jacky-zhang has joined #openstack-nova | 01:05 | |
| *** salv-orlando has quit IRC | 01:09 | |
| mriedem | cburgess: now it's on you https://review.openstack.org/#/c/205282/ | 01:10 |
| mriedem | to whip the votes | 01:10 |
| openstackgerrit | Thelo Gaultier proposed openstack/nova: Add ITRI DISCO os-brick connector for libvirt https://review.openstack.org/253353 | 01:10 |
| *** haomaiwang has quit IRC | 01:13 | |
| *** zhangjn has joined #openstack-nova | 01:16 | |
| *** diana_clarke has joined #openstack-nova | 01:16 | |
| mriedem | Thelo: can you please use a consistent copyright in your change | 01:17 |
| *** kaisers1 has joined #openstack-nova | 01:18 | |
| mriedem | Thelo: also, the depends-on might not be working b/c you have all that whitespace before it in the commit message | 01:18 |
| mriedem | it's probably messing up the parser | 01:18 |
| *** kaisers has quit IRC | 01:20 | |
| *** ssurana has quit IRC | 01:23 | |
| Thelo | mriedem: Ok I check that right now | 01:23 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: WIP: define NUMA memory bw monitor https://review.openstack.org/270344 | 01:24 |
| openstackgerrit | Thelo Gaultier proposed openstack/nova: Add ITRI DISCO os-brick connector for libvirt https://review.openstack.org/253353 | 01:28 |
| openstackgerrit | melanie witt proposed openstack/nova: Add a cell and host mapping utility to nova-manage https://review.openstack.org/270565 | 01:29 |
| *** penick has quit IRC | 01:33 | |
| *** gjayavelu has joined #openstack-nova | 01:35 | |
| Thelo | mreidem: I changed the commit message it seems better now, also I updated the os-brick CI to run it against DISCO | 01:37 |
| *** thorst has joined #openstack-nova | 01:39 | |
| *** Sukhdev has quit IRC | 01:40 | |
| mriedem | \o/ | 01:40 |
| auggy | so are you saying it's a DISCO-brick? | 01:41 |
| Thelo | auggy: what do you mean? | 01:41 |
| mriedem | Thelo: you didn't fix those copyrights :( | 01:41 |
| *** ljxiash has joined #openstack-nova | 01:41 | |
| auggy | Thelo: sorry just making a bad joke | 01:42 |
| Thelo | mriedem: you want all of them to be like in disco.py? | 01:42 |
| mriedem | Thelo: i just want the 2 new files to have consistent copyrights | 01:42 |
| mriedem | b/c that makes sense | 01:42 |
| *** jschlueter has joined #openstack-nova | 01:42 | |
| mriedem | fix that and i'm +2 | 01:43 |
| smcginnis | Thelo: Thanks for getting the CI updated. | 01:43 |
| Thelo | mriedem: should I remove the # All Rights Reserved.? | 01:43 |
| Thelo | the date should be 2016 although disco.py is 2015? | 01:44 |
| mriedem | Thelo: probably | 01:44 |
| mriedem | Thelo: does your company have specific rules about what the copyright line should be for upstream contributions? | 01:44 |
| Thelo | mriedem: I don't think so | 01:44 |
| *** zhangjn has quit IRC | 01:45 | |
| mriedem | Thelo: well in that case, i'd probably use what you have here: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/disco/disco.py#L1 | 01:45 |
| *** changbl has joined #openstack-nova | 01:45 | |
| mriedem | and leave it as 2015 since that's when you propose the change | 01:45 |
| * mriedem is not earning a law degree | 01:46 | |
| *** gjayavelu has quit IRC | 01:47 | |
| openstackgerrit | Thelo Gaultier proposed openstack/nova: Add ITRI DISCO os-brick connector for libvirt https://review.openstack.org/253353 | 01:49 |
| *** zhangjn has joined #openstack-nova | 01:51 | |
| *** dims has joined #openstack-nova | 01:54 | |
| mriedem | thanks, +2 | 01:55 |
| mriedem | oomichi: alex_xu: easy new volume driver add https://review.openstack.org/#/c/253353/ | 01:56 |
| *** erlon has quit IRC | 01:56 | |
| *** haomaiwang has joined #openstack-nova | 01:58 | |
| *** zhangjn has quit IRC | 01:58 | |
| oomichi | mriedem: ok, checking it. more easy one: https://review.openstack.org/#/c/270536/ | 01:59 |
| *** vivekd has joined #openstack-nova | 02:00 | |
| *** haomaiwang has quit IRC | 02:01 | |
| *** 17WABLQSD has joined #openstack-nova | 02:01 | |
| eliqiao | hi there, does any one know if there is any assert method to compare two object in unit test case frame work? | 02:01 |
| Thelo | mriedem: Many thanks, is there any other nova core here that could take a look at this : https://review.openstack.org/#/c/253353, I would very much appreciate | 02:02 |
| oomichi | mriedem: copyright year is a little old(2015), is it ok? | 02:02 |
| mriedem | oomichi: the code was pushed up in 2015 so i figure that's ok | 02:02 |
| *** paul-carlton2 has quit IRC | 02:04 | |
| oomichi | mriedem: thanks, +W | 02:11 |
| oomichi | Thelo: ^^^ | 02:11 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 02:13 |
| Thelo | oomichi: Thanks | 02:14 |
| *** diana_clarke has quit IRC | 02:15 | |
| *** zhangjn has joined #openstack-nova | 02:16 | |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: add version discovery guideline in api concept doc https://review.openstack.org/264044 | 02:18 |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: add client interactive guideline for microversions https://review.openstack.org/264045 | 02:18 |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: completes microversion use-cases in api concept doc https://review.openstack.org/263220 | 02:18 |
| alex_xu | oomichi: ^^ updated | 02:19 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Calculate block_migration in virt drivers if it's None https://review.openstack.org/254101 | 02:19 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Return host and block_migration in task_api._execute() https://review.openstack.org/254102 | 02:19 |
| oomichi | alex_xu: ok, checking... there is a trade, super easy one: https://review.openstack.org/#/c/270536/ ;) | 02:20 |
| alex_xu | oomichi: btw, there is another api patch looks very close https://review.openstack.org/#/c/254950/, if you can take look at that will be great :) | 02:20 |
| *** thorst has quit IRC | 02:20 | |
| alex_xu | oomichi: ok :) | 02:20 |
| alex_xu | oomichi: ^^ another trade :) | 02:21 |
| *** thorst has joined #openstack-nova | 02:21 | |
| gmann | alex_xu: oomichi that might still need updated version number due to https://review.openstack.org/#/c/233076/27 | 02:21 |
| oomichi | alex_xu: yours are 4, mine is 1 btw. that is fine anyways ;) | 02:22 |
| alex_xu | gmann: yea, I guess so, and there is another in the gate pipeline... | 02:23 |
| eliqiao | have a question why https://review.openstack.org/233076 and https://review.openstack.org/202617 are on gate for a whole night ? | 02:24 |
| alex_xu | oomichi: ;) | 02:24 |
| eliqiao | they are still not get merged. | 02:24 |
| eliqiao | alex_xu: oomichi any idea? | 02:24 |
| oomichi | eliqiao: because the gate is super busy now | 02:25 |
| oomichi | eliqiao: http://status.openstack.org/zuul/ | 02:25 |
| eliqiao | oomichi: okay, hehe.. | 02:25 |
| mriedem | 30+ hours | 02:25 |
| *** thorst has quit IRC | 02:25 | |
| alex_xu | or just say due to we are close to freeze~ | 02:26 |
| oomichi | alex_xu: right :) | 02:26 |
| eliqiao | in my mind, I saw there are 4 patch with microversion 2.17(not include mine)... lots of rebases. | 02:26 |
| eliqiao | this is bad that for developers. | 02:27 |
| mriedem | alex_xu: it's actually b/c of a lot of gate issues - job failures | 02:27 |
| mriedem | pip 8 | 02:27 |
| mriedem | multinode failing | 02:27 |
| mriedem | etc | 02:27 |
| oomichi | eliqiao: yeah, but that is just rebased. A single core can approve patches in this case without another +1 | 02:27 |
| eliqiao | I am thinking if there is any automaticly way to do the micro-version setting stuff. | 02:27 |
| gmann | oomichi: eliqiao yea | 02:28 |
| *** baoli has joined #openstack-nova | 02:28 | |
| eliqiao | So maybe adding some helper script to do that? | 02:28 |
| eliqiao | Okay, find, I am just brain storming.. | 02:29 |
| alex_xu | mriedem: ah, got it | 02:29 |
| eliqiao | gotcha! b/c == because ? | 02:30 |
| alex_xu | eliqiao: yeah, I learned that last time also :) | 02:31 |
| eliqiao | alex_xu: great, good to learn lots of words :) | 02:31 |
| *** Jeffrey4l has joined #openstack-nova | 02:31 | |
| *** zhangjn has quit IRC | 02:32 | |
| *** baoli has quit IRC | 02:33 | |
| eliqiao | just pull master,there isn't any patch get merged for nova since last night(12 hours) :( | 02:37 |
| *** otter768 has joined #openstack-nova | 02:37 | |
| *** jeffrey4l_|away is now known as emperor | 02:38 | |
| *** vladikr has joined #openstack-nova | 02:40 | |
| *** otter768 has quit IRC | 02:42 | |
| oomichi | alex_xu: done on your doc patches, just one comment about the last one. | 02:43 |
| alex_xu | oomichi: thanks, checking now | 02:44 |
| *** hblixt has quit IRC | 02:46 | |
| *** hblixt has joined #openstack-nova | 02:46 | |
| *** hblixt has quit IRC | 02:46 | |
| *** hblixt has joined #openstack-nova | 02:47 | |
| alex_xu | oomichi: should we use "microversions" with "s" | 02:47 |
| vilobhmm111 | alex_xu : ping | 02:47 |
| alex_xu | vilobhmm111: pong | 02:47 |
| *** ducttape_ has joined #openstack-nova | 02:48 | |
| oomichi | alex_xu: on https://review.openstack.org/#/c/263220/9/api-guide/source/microversions.rst , i just am saying s/Micro/micro/ | 02:48 |
| oomichi | alex_xu: because you are doing that at the other places with this pathc | 02:48 |
| alex_xu | oomichi: oops, miss that | 02:49 |
| *** 17WABLQSD has quit IRC | 02:49 | |
| oomichi | alex_xu: the title is fine as "Microversions" | 02:49 |
| *** 77CAAADFV has joined #openstack-nova | 02:49 | |
| oomichi | alex_xu: at line 15 | 02:49 |
| alex_xu | oomichi: ok, got it | 02:49 |
| oomichi | alex_xu: cool :) | 02:50 |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: add version discovery guideline in api concept doc https://review.openstack.org/264044 | 02:50 |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: add client interactive guideline for microversions https://review.openstack.org/264045 | 02:50 |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: completes microversion use-cases in api concept doc https://review.openstack.org/263220 | 02:50 |
| vilobhmm111 | alex_xu : lets say this https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3318 get replace with an attribute service.is_up() i.e. rather than calling servicegroup method we now invoke service.is_up ….here https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute_api.py#L3073 instance list is passed which includes service object…if i want the value of is_up (service.is_up()) to be differe | 02:51 |
| *** mgoddard_ has quit IRC | 02:53 | |
| vilobhmm111 | alex_xu : ^^ | 02:54 |
| alex_xu | vilobhmm111: sorry, I didn't get you what you point to | 02:55 |
| alex_xu | vilobhmm111: your patch already merged? | 02:55 |
| *** apoorvad has quit IRC | 02:55 | |
| alex_xu | vilobhmm111: that patch already merged, so you get a problem? | 02:56 |
| vilobhmm111 | alex_xu : nope working on it…unit test is breaking so wanted to check | 02:56 |
| *** chenli has joined #openstack-nova | 02:56 | |
| vilobhmm111 | alex_xu : here https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute_api.py#L3074 list of instances will be passed | 02:57 |
| vilobhmm111 | which will invoke https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3308 | 02:57 |
| *** mdnadeem has joined #openstack-nova | 02:58 | |
| alex_xu | vilobhmm111: yes | 02:58 |
| vilobhmm111 | since the list is passed how do you associate different value for service.foo() https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3318 lets says servicegroup call gets replace by service.foo | 02:59 |
| vilobhmm111 | alex_xu : ^^ | 02:59 |
| *** chenli_ has joined #openstack-nova | 02:59 | |
| chenli_ | hello, I have a devstack environment, everything works fine. | 03:00 |
| *** yamahata has quit IRC | 03:00 | |
| chenli_ | Today I installed another host with n-compute & ovn-controller enabled only. | 03:00 |
| chenli_ | But the log for n-cpu looks strange, | 03:00 |
| chenli_ | http://paste.openstack.org/show/484490/ | 03:00 |
| chenli_ | Anyone know what's the log means ?? | 03:00 |
| *** chenli has quit IRC | 03:01 | |
| *** chenli_ is now known as chenli | 03:01 | |
| alex_xu | vilobhmm111: so it should depend on the server obj which defined in test https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute_api.py#L3040 ? | 03:02 |
| *** zenoway has joined #openstack-nova | 03:02 | |
| *** EinstCrazy has joined #openstack-nova | 03:03 | |
| alex_xu | vilobhmm111: and the servicegroup driver used in the test, I guess that is db driver | 03:03 |
| vilobhmm111 | alex_xu : sure but how do you associate an instance in this case the service object (instance.service) to work on a method (foo) and return the result | 03:03 |
| *** dgonzalez has quit IRC | 03:03 | |
| alex_xu | vilobhmm111: "a method (foo)" point to what? | 03:04 |
| vilobhmm111 | what we are passing here simple atributes of service obj https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute_api.py#L3040 | 03:04 |
| eliqiao | chenli: have you try to upgrade your data base schema? | 03:04 |
| vilobhmm111 | alex_xu : method will look like https://review.openstack.org/#/c/235637/8/nova/objects/service.py #208 | 03:04 |
| eliqiao | chenli: try nova-manage db sync on you controller node | 03:05 |
| vilobhmm111 | chenli : yes there seems to be problem with db sync or db upgrade as eliqiao mentioned | 03:05 |
| *** lyanchih has joined #openstack-nova | 03:05 | |
| vilobhmm111 | eliqiao : +1 | 03:05 |
| eliqiao | vilobhmm111: thx | 03:05 |
| *** zenoway has quit IRC | 03:06 | |
| alex_xu | vilobhmm111: just from the reading, it should work with that is_up method | 03:07 |
| gmann | mriedem: alex_xu oomichi nova spec might need to update pbr version | 03:08 |
| gmann | error - http://logs.openstack.org/64/269664/3/check/gate-nova-specs-docs/e69f81b/console.html#_2016-01-20_22_24_10_372 | 03:08 |
| gmann | pip 8 gets installed and gives same distutils error | 03:08 |
| *** ducttape_ has quit IRC | 03:09 | |
| gmann | but m not sure same thing is passed on another patch where correct pip==7.1.2 and pass | 03:10 |
| gmann | http://logs.openstack.org/23/256323/6/check/gate-nova-specs-pep8/2cd1800/console.html#_2015-12-15_15_39_35_935 | 03:10 |
| vilobhmm111 | alex_xu : but don't you think i will need to mock is_up ? | 03:10 |
| *** baoli has joined #openstack-nova | 03:10 | |
| vilobhmm111 | because it internaly has call to sg api;s | 03:10 |
| gmann | any idea why its taking randomly with same requirements.txt | 03:10 |
| *** otter768 has joined #openstack-nova | 03:10 | |
| alex_xu | vilobhmm111: I think not, is_up method just access some obj's attribute, and there is value for those attributes, and sg api should works as current code. | 03:11 |
| *** agireud has quit IRC | 03:11 | |
| alex_xu | vilobhmm111: or you got error when you didn't mock it? | 03:11 |
| gmann | mriedem: alex_xu oomichi I see, passed one is not latest run. | 03:12 |
| alex_xu | gmann: ok | 03:12 |
| gmann | alex_xu: i will update pbr version in spec requirement. not sure why it was < 1 | 03:13 |
| eliqiao | hi gmann can I ask a tempest question here(I don't join to temp channel yet) ? | 03:13 |
| gmann | eliqiao: sure | 03:13 |
| *** dgonzalez has joined #openstack-nova | 03:13 | |
| *** shuquan has joined #openstack-nova | 03:14 | |
| eliqiao | gmann: could you please point me out some wiki/doc for how to setup tempset with source code ? | 03:14 |
| eliqiao | gmann: I got this http://docs.openstack.org/developer/tempest/overview.html#legacy-run-method but seems it's not very detail. | 03:15 |
| *** agireud has joined #openstack-nova | 03:15 | |
| chenli | eliqiao: vilobhmm111 : did you mean I have old db on controller node ? | 03:16 |
| eliqiao | chenli: maybe | 03:16 |
| chenli | eliqiao: vilobhmm111 : yes, my controller node's code is older.... but isn't n-cpu communicate with n-api with versioned-object ? | 03:16 |
| *** doug-fish has joined #openstack-nova | 03:16 | |
| eliqiao | chenli: not nova-api, it should be nova-conductor. | 03:17 |
| eliqiao | chenli: I don't know the detail of your infra, but please do a db sync first. | 03:17 |
| Thelo | mriedem: https://review.openstack.org/#/c/253353/, this one needs a recheck, do you mind reaccepting it when its finished? | 03:17 |
| chenli | eliqiao: I did, it didn't work.... So, should I update the code in controller ? | 03:18 |
| gmann | eliqiao: yea, legacy way is not recommended | 03:18 |
| vilobhmm111 | alex_xu : it worked but needed to populate https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute_api.py#L3037 with created_at, updated_at, last_seen_up …thanks for your help | 03:18 |
| gmann | eliqiao: check this- http://docs.openstack.org/developer/tempest/configuration.html#dynamic-credentials | 03:18 |
| gmann | eliqiao: you can use with dynamic cred, which will easy. it create cred at runtime and clean those automatically | 03:19 |
| eliqiao | gmann: Thank you, I will take a try. | 03:19 |
| eliqiao | gmann: that's great, appreciate for your help. | 03:19 |
| alex_xu | vilobhmm111: ok | 03:19 |
| gmann | eliqiao: np. please let me know if any issue you face | 03:19 |
| eliqiao | gmann: sure, thank you. | 03:20 |
| eliqiao | chenli: looking your trace in compute node. compute manager sue oslo_messaging to call nova-conductor , and nova-conductor(in controller node) try to access db, but failed. | 03:21 |
| *** doug-fish has quit IRC | 03:21 | |
| *** oomichi is now known as oomichi_away | 03:23 | |
| chenli | eliqiao: ....yep.... I forget that we should always keep nova-conductor to be the newest, then others can work with other versions..... Updated controller code, restart nova-conductor... and it works now... | 03:23 |
| chenli | eliqiao: thanks for the kindly help ! | 03:23 |
| eliqiao | chenli: I guess the new nova-cpu with older conductor are not compatible, the upgrade sequence should be nova-conductor then nova-compute | 03:23 |
| eliqiao | chenli: yes, you are right. np, feel free to ask question in irc. lots of people will help. :) | 03:24 |
| chenli | eliqiao: :) | 03:24 |
| *** browne has quit IRC | 03:25 | |
| *** thorst has joined #openstack-nova | 03:25 | |
| *** agireud has quit IRC | 03:27 | |
| openstackgerrit | Alex Xu proposed openstack/nova: REST API changes for user settable server description https://review.openstack.org/254950 | 03:28 |
| *** thorst has quit IRC | 03:30 | |
| *** shuquan has quit IRC | 03:30 | |
| mriedem | Thelo: yeah | 03:31 |
| mriedem | i rechecked it | 03:31 |
| *** shuquan has joined #openstack-nova | 03:31 | |
| *** ljxiash has quit IRC | 03:32 | |
| *** ljxiash has joined #openstack-nova | 03:32 | |
| *** ljxiash has quit IRC | 03:33 | |
| *** ljxiash has joined #openstack-nova | 03:33 | |
| Thelo | mriedem: Yes I saw that, thanks | 03:33 |
| *** zhangjn has joined #openstack-nova | 03:34 | |
| *** zenoway has joined #openstack-nova | 03:38 | |
| *** shuquan has quit IRC | 03:38 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 03:38 |
| *** dgonzalez has quit IRC | 03:39 | |
| *** dgonzalez has joined #openstack-nova | 03:40 | |
| *** dims has quit IRC | 03:42 | |
| *** zenoway has quit IRC | 03:42 | |
| *** armax has quit IRC | 03:42 | |
| *** agireud has joined #openstack-nova | 03:52 | |
| *** su_zhang has joined #openstack-nova | 03:53 | |
| *** baoli has quit IRC | 03:54 | |
| *** 77CAAADFV has quit IRC | 03:59 | |
| *** haomaiwa_ has joined #openstack-nova | 03:59 | |
| *** haomaiwa_ has quit IRC | 04:01 | |
| *** haomaiwa_ has joined #openstack-nova | 04:01 | |
| *** takashin has joined #openstack-nova | 04:05 | |
| *** ljxiash has quit IRC | 04:08 | |
| *** ljxiash has joined #openstack-nova | 04:08 | |
| openstackgerrit | melanie witt proposed openstack/nova: Add a cell and host mapping utility to nova-manage https://review.openstack.org/270565 | 04:09 |
| *** Fdaisuke has joined #openstack-nova | 04:10 | |
| *** lyanchih has quit IRC | 04:10 | |
| *** mriedem has quit IRC | 04:12 | |
| gmann | mriedem: you there? about failure on nova spec gate - https://review.openstack.org/#/c/269664/ | 04:12 |
| *** ljxiash has quit IRC | 04:13 | |
| gmann | pbr > 1.0 seems fixing that pip version | 04:13 |
| *** zenoway has joined #openstack-nova | 04:13 | |
| *** shyama has joined #openstack-nova | 04:18 | |
| *** zenoway has quit IRC | 04:18 | |
| *** browne has joined #openstack-nova | 04:18 | |
| *** armax has joined #openstack-nova | 04:20 | |
| *** claudiub has quit IRC | 04:21 | |
| *** diana_clarke has joined #openstack-nova | 04:21 | |
| *** aghadge has joined #openstack-nova | 04:26 | |
| *** thorst has joined #openstack-nova | 04:27 | |
| *** browne has quit IRC | 04:28 | |
| *** browne has joined #openstack-nova | 04:29 | |
| *** otter768 has quit IRC | 04:31 | |
| *** thorst has quit IRC | 04:34 | |
| *** zhangjn has quit IRC | 04:36 | |
| *** shyama has quit IRC | 04:36 | |
| *** browne has quit IRC | 04:40 | |
| *** tjones has joined #openstack-nova | 04:40 | |
| *** Thelo_ has joined #openstack-nova | 04:41 | |
| *** Thelo_ has quit IRC | 04:46 | |
| tojuvone | oomichi: thanks | 04:48 |
| *** coolsvap|away is now known as coolsvap | 04:49 | |
| *** aghadge has quit IRC | 04:49 | |
| *** nelsnels_ has quit IRC | 04:53 | |
| *** Thelo_ has joined #openstack-nova | 04:55 | |
| *** nelsnelson has joined #openstack-nova | 04:56 | |
| *** tjones has quit IRC | 04:58 | |
| *** haomaiwa_ has quit IRC | 05:01 | |
| *** shyama has joined #openstack-nova | 05:01 | |
| *** ljxiash has joined #openstack-nova | 05:01 | |
| *** ljxiash has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** ljxiash has joined #openstack-nova | 05:01 | |
| *** david-lyle has quit IRC | 05:02 | |
| *** lyanchih has joined #openstack-nova | 05:03 | |
| *** ljxiash has quit IRC | 05:03 | |
| *** shyama has quit IRC | 05:03 | |
| *** ljxiash has joined #openstack-nova | 05:03 | |
| *** david-lyle has joined #openstack-nova | 05:03 | |
| *** zhangjn has joined #openstack-nova | 05:05 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/nova: Replace deprecated LOG.warn with LOG.warning https://review.openstack.org/264133 | 05:05 |
| *** markvoelker has joined #openstack-nova | 05:07 | |
| *** Marga_ has quit IRC | 05:07 | |
| *** chhavi has joined #openstack-nova | 05:07 | |
| *** Marga_ has joined #openstack-nova | 05:08 | |
| *** Thelo_ has quit IRC | 05:10 | |
| *** markvoelker has quit IRC | 05:12 | |
| *** SridharG has joined #openstack-nova | 05:12 | |
| *** markvoelker has joined #openstack-nova | 05:12 | |
| *** su_zhang has quit IRC | 05:13 | |
| *** gjayavelu has joined #openstack-nova | 05:18 | |
| openstackgerrit | Rushi Agrawal proposed openstack/nova: Remove almost all mox from tests/unit/objects/test_instance.py https://review.openstack.org/264292 | 05:23 |
| *** SridharG has quit IRC | 05:28 | |
| *** lyanchih has quit IRC | 05:29 | |
| *** doug-fish has joined #openstack-nova | 05:30 | |
| *** vishwanathj has quit IRC | 05:33 | |
| *** thorst has joined #openstack-nova | 05:33 | |
| *** jinxing has quit IRC | 05:33 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Workaround for pip 8 distutils issue https://review.openstack.org/270606 | 05:33 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Fix outdated parameter network_info description in virt/driver https://review.openstack.org/263166 | 05:34 |
| *** doug-fish has quit IRC | 05:35 | |
| *** cfriesen has quit IRC | 05:36 | |
| *** jinxing has joined #openstack-nova | 05:38 | |
| openstackgerrit | yuntongjin proposed openstack/nova: _can_fallocate should throw a warning instead of error https://review.openstack.org/257638 | 05:39 |
| *** SridharG has joined #openstack-nova | 05:40 | |
| *** thorst has quit IRC | 05:40 | |
| *** vivekd has quit IRC | 05:44 | |
| *** doug-fish has joined #openstack-nova | 05:44 | |
| *** jaosorior has joined #openstack-nova | 05:47 | |
| openstackgerrit | LisaLi proposed openstack/nova: Attach/detach encryptors during swap_volume https://review.openstack.org/252809 | 05:47 |
| *** lyanchih has joined #openstack-nova | 05:48 | |
| *** doug-fish has quit IRC | 05:49 | |
| gmann | alex_xu: spec gate pass with this (pbr > 1.0)- https://review.openstack.org/#/c/270606/1 | 05:49 |
| gmann | oomichi_away: ^^ | 05:49 |
| *** apoorv has joined #openstack-nova | 05:53 | |
| *** SridharG has quit IRC | 05:54 | |
| *** dave-mccowan has quit IRC | 05:57 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwa_ has joined #openstack-nova | 06:01 | |
| *** lyanchih has quit IRC | 06:01 | |
| *** markvoelker_ has joined #openstack-nova | 06:01 | |
| *** lyanchih has joined #openstack-nova | 06:02 | |
| *** markvoelker has quit IRC | 06:05 | |
| *** markvoelker has joined #openstack-nova | 06:05 | |
| *** markvoelker_ has quit IRC | 06:06 | |
| *** oomichi_away is now known as oomichi | 06:11 | |
| oomichi | gmann: g-r patch is already merged? | 06:12 |
| oomichi | on requirements side | 06:12 |
| gmann | oomichi: actually that is the issue. g-r revuirement does not update any of the specs repo requirements | 06:13 |
| gmann | oomichi: checking to update requirements for all spec repo also | 06:13 |
| oomichi | gmann: pbr>=1.6 on g-r side | 06:14 |
| gmann | oomichi: yea but in spec repo it is old one | 06:15 |
| oomichi | gmann: I'm not sure why your patch can pass on the gate | 06:15 |
| oomichi | gmann: anyways, it is nice to change your patch to 1.6 for fitting to g-r | 06:16 |
| oomichi | I am guessing | 06:16 |
| gmann | oomichi: yea making >1.6 but that will not fail as specs repo are not sync with g-r | 06:16 |
| oomichi | gmann: can you update your patch soon? | 06:17 |
| gmann | oomichi: sure | 06:17 |
| oomichi | gmann: I cannot approve nova-spec patch anyways and maybe there are not any nova-spec cores in this timezone now | 06:18 |
| gmann | oomichi: humm | 06:18 |
| *** Jeffrey4l has quit IRC | 06:18 | |
| *** Jeffrey4l has joined #openstack-nova | 06:19 | |
| *** liverpooler has quit IRC | 06:19 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI: Do not rely on XenAPI to get device names https://review.openstack.org/213112 | 06:19 |
| *** zhangjn has quit IRC | 06:20 | |
| *** zhangjn has joined #openstack-nova | 06:23 | |
| *** vishwanathj has joined #openstack-nova | 06:23 | |
| Thelo | oomichi: Regarding this one https://review.openstack.org/#/c/253353/ , the depends-on failed in the gate for what seems to be a non-related issue (https://review.openstack.org/#/c/267725/) can you do anything? | 06:26 |
| *** nkrinner has joined #openstack-nova | 06:27 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: pbr > 1.6 to avoid pip 8 distutils issue https://review.openstack.org/270606 | 06:27 |
| gmann | oomichi: done ^^ | 06:28 |
| oomichi | gmann: thanks, but = is not necessary | 06:29 |
| *** SridharG has joined #openstack-nova | 06:29 | |
| gmann | oomichi: this is same as g-r | 06:30 |
| gmann | https://github.com/openstack/requirements/blob/master/global-requirements.txt#L125 | 06:30 |
| *** derekh has joined #openstack-nova | 06:31 | |
| *** [1]Thelo has joined #openstack-nova | 06:32 | |
| oomichi | gmann: oops, sorry about that. I lost my eyes.. | 06:32 |
| gmann | oomichi: np | 06:32 |
| *** otter768 has joined #openstack-nova | 06:32 | |
| oomichi | gmann: +1 | 06:33 |
| gmann | oomichi: Thanks | 06:33 |
| *** unicell has joined #openstack-nova | 06:33 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/265601 | 06:33 |
| *** Thelo has quit IRC | 06:34 | |
| *** [1]Thelo is now known as Thelo | 06:34 | |
| oomichi | Thelo: type "recheck" on https://review.openstack.org/#/c/267725/ like me if you are sure that is unrelated gate issue. | 06:34 |
| *** gcb has quit IRC | 06:35 | |
| *** gcb has joined #openstack-nova | 06:35 | |
| *** ljxiash has quit IRC | 06:36 | |
| *** szhsong has joined #openstack-nova | 06:36 | |
| *** Marga_ has quit IRC | 06:37 | |
| *** otter768 has quit IRC | 06:37 | |
| openstackgerrit | Tobias Urdin proposed openstack/nova: Move is_volume_backed_instance to compute.utils https://review.openstack.org/270482 | 06:37 |
| *** shyama has joined #openstack-nova | 06:37 | |
| *** thorst has joined #openstack-nova | 06:38 | |
| *** zenoway has joined #openstack-nova | 06:39 | |
| *** yamahata has joined #openstack-nova | 06:41 | |
| *** ljxiash has joined #openstack-nova | 06:42 | |
| *** zenoway has quit IRC | 06:43 | |
| *** zhangjn has quit IRC | 06:44 | |
| *** thorst has quit IRC | 06:45 | |
| *** diana_clarke has quit IRC | 06:47 | |
| Thelo | oomichi: This will re-run both check and gate? | 06:52 |
| oomichi | Thelo: yeah, right. you can check the patch situation on http://status.openstack.org/zuul/ | 06:53 |
| oomichi | Thelo: you can find 267725 on "check" of http://status.openstack.org/zuul/ now | 06:53 |
| oomichi | Thelo: after passing "check", 267725 is queued on "gate" of http://status.openstack.org/zuul/ | 06:54 |
| Thelo | oomichi: since 253353 depends on 267725 will it go to gate too? | 06:57 |
| *** zhangjn has joined #openstack-nova | 06:58 | |
| *** zhangjn has quit IRC | 06:58 | |
| *** zhangjn has joined #openstack-nova | 06:59 | |
| oomichi | Thelo: 253353 is not queued now, I'm not sure the reason. nice to type "recheck" again on 253353 | 07:00 |
| *** haomaiwa_ has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| *** lpetrut has joined #openstack-nova | 07:02 | |
| Thelo | oomichi: ok ok thanks | 07:02 |
| *** vilobhmm111 has quit IRC | 07:03 | |
| *** agireud has quit IRC | 07:04 | |
| *** achanda has quit IRC | 07:06 | |
| *** mjura has joined #openstack-nova | 07:06 | |
| *** ssurana has joined #openstack-nova | 07:10 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 07:11 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add some validations for BDM https://review.openstack.org/258788 | 07:12 |
| *** agireud has joined #openstack-nova | 07:16 | |
| *** rcernin has joined #openstack-nova | 07:20 | |
| *** su_zhang has joined #openstack-nova | 07:31 | |
| *** davidhadas_ has joined #openstack-nova | 07:38 | |
| *** thorst has joined #openstack-nova | 07:43 | |
| *** migi_ is now known as migi | 07:43 | |
| openstackgerrit | Tobias Urdin proposed openstack/nova: Move is_volume_backed_instance to compute.utils and add dict support https://review.openstack.org/270482 | 07:48 |
| *** unicell has quit IRC | 07:49 | |
| *** unicell has joined #openstack-nova | 07:49 | |
| *** andymaier has joined #openstack-nova | 07:50 | |
| *** thorst has quit IRC | 07:50 | |
| *** jyuso1 has joined #openstack-nova | 07:51 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova: Validate translations https://review.openstack.org/270190 | 07:52 |
| *** jaosorior has quit IRC | 07:53 | |
| *** liverpooler has joined #openstack-nova | 07:54 | |
| *** liverpooler has quit IRC | 07:55 | |
| *** liverpooler has joined #openstack-nova | 07:55 | |
| *** otter768 has joined #openstack-nova | 08:00 | |
| *** takashin has left #openstack-nova | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** ihrachys has joined #openstack-nova | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** otter768_ has joined #openstack-nova | 08:02 | |
| *** mkoderer_ is now known as mkoderer | 08:03 | |
| *** davidhadas_ has quit IRC | 08:03 | |
| *** lpetrut has quit IRC | 08:03 | |
| *** otter768_ has quit IRC | 08:04 | |
| *** otter76__ has joined #openstack-nova | 08:04 | |
| *** otter768 has quit IRC | 08:04 | |
| *** otter76__ has quit IRC | 08:05 | |
| *** jaosorior has joined #openstack-nova | 08:06 | |
| *** otter768 has joined #openstack-nova | 08:06 | |
| *** achanda has joined #openstack-nova | 08:06 | |
| Thelo | not sure it is the right place to ask but where to take a look at then the gate-grenade-dsvm-multinode job fails (https://jenkins02.openstack.org/job/gate-grenade-dsvm-multinode/1988/console) from the console I dont see the root cause | 08:06 |
| *** mpavone has joined #openstack-nova | 08:07 | |
| *** otter768 has quit IRC | 08:07 | |
| *** otter768 has joined #openstack-nova | 08:07 | |
| *** zhangjn has quit IRC | 08:09 | |
| *** zhangjn_ has joined #openstack-nova | 08:09 | |
| *** otter768_ has joined #openstack-nova | 08:09 | |
| *** otter768 has quit IRC | 08:09 | |
| *** otter768 has joined #openstack-nova | 08:11 | |
| *** zhangjn has joined #openstack-nova | 08:11 | |
| *** otter768_ has quit IRC | 08:11 | |
| *** otter768 has quit IRC | 08:11 | |
| *** lpetrut has joined #openstack-nova | 08:12 | |
| *** achanda has quit IRC | 08:13 | |
| *** Jeffrey4l has quit IRC | 08:13 | |
| *** markus_z has joined #openstack-nova | 08:14 | |
| *** zhangjn_ has quit IRC | 08:14 | |
| *** ildikov has quit IRC | 08:15 | |
| *** dedery has joined #openstack-nova | 08:15 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 08:18 |
| *** su_zhang has quit IRC | 08:18 | |
| *** scheuran has joined #openstack-nova | 08:18 | |
| *** irenab has joined #openstack-nova | 08:18 | |
| *** tdurakov has joined #openstack-nova | 08:19 | |
| *** ajo_ is now known as ajo | 08:24 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Pass req to all API plugins https://review.openstack.org/270658 | 08:24 |
| *** jdurgin has quit IRC | 08:24 | |
| *** jdurgin has joined #openstack-nova | 08:25 | |
| *** Jeffrey4l has joined #openstack-nova | 08:26 | |
| AJaeger | nova cores, mriedem, sdague, tonyb: As discussed yesterday the new import of translations passes the validation check. Could you import translations for master: https://review.openstack.org/#/c/265601/12 and also +A again (I rebased) the validation https://review.openstack.org/#/c/270190/4 , please? | 08:27 |
| *** liusheng has quit IRC | 08:28 | |
| *** liusheng has joined #openstack-nova | 08:29 | |
| oomichi | tangchen: hi | 08:30 |
| *** lpetrut has quit IRC | 08:31 | |
| *** zenoway has joined #openstack-nova | 08:33 | |
| *** zakora has joined #openstack-nova | 08:33 | |
| *** lpetrut has joined #openstack-nova | 08:33 | |
| oomichi | gmann: is https://review.openstack.org/#/c/202431/ still necessary? | 08:34 |
| *** agireud has quit IRC | 08:35 | |
| gmann | oomichi: yea, i need to see how we should do that in more deeper way in nova | 08:35 |
| gmann | oomichi: with top, bottom and changed | 08:35 |
| oomichi | gmann: ok, nice to keep it now | 08:35 |
| gmann | oomichi: yea, i will check after all extension things | 08:35 |
| Thelo | oomichi: are you familiar with the gate-grenade-dsvm-multinode job? | 08:36 |
| *** armax has quit IRC | 08:36 | |
| oomichi | Thelo: not so, what happens? | 08:36 |
| Thelo | oomichi: a patch keeps failing on this one, and I cannot find much details in the log https://review.openstack.org/#/c/267725/ | 08:38 |
| oomichi | Thelo: I am checking the log... | 08:40 |
| Thelo | oomichi: thanks | 08:42 |
| *** andrearo1 is now known as andrearosa | 08:44 | |
| *** tdurakov has quit IRC | 08:45 | |
| *** agireud has joined #openstack-nova | 08:47 | |
| *** jlanoux has joined #openstack-nova | 08:47 | |
| *** sahid has joined #openstack-nova | 08:47 | |
| oomichi | Thelo: the ssh timeout happened again on stable/liberty side: http://logs.openstack.org/25/267725/2/check/gate-grenade-dsvm-multinode/829c6ac/logs/old/testr_results.html.gz | 08:47 |
| *** thorst has joined #openstack-nova | 08:48 | |
| oomichi | Thelo: that didn't happen on grenade without multinode | 08:49 |
| *** ijw_ has joined #openstack-nova | 08:49 | |
| oomichi | gmann: ^^^ do you know the above situation? | 08:49 |
| gmann | oomichi: not exactly but i also see lot of failure of ssh timeout | 08:50 |
| oomichi | gmann: on grenade with multinode? | 08:50 |
| *** gjayavelu has quit IRC | 08:50 | |
| *** tdurakov has joined #openstack-nova | 08:50 | |
| gmann | oomichi: no, in normal jobs. need to check about multinode | 08:51 |
| oomichi | gmann: ok, thanks for your info :) | 08:51 |
| *** ijw has quit IRC | 08:52 | |
| *** derekh has quit IRC | 08:52 | |
| *** thorst has quit IRC | 08:55 | |
| dedery | hi, I'm about to cherry-pick a few patch-sets from master to liberty (agreed with mriedem). Never did it before so I want to validate that the procedure is the one located at the bottom of http://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes | 08:55 |
| dedery | (git checkout origin/stable/liberty; git cherry-pick -x <commit #1>; git cherry-pick -x <commit #2>; git review stable/liberty) | 08:56 |
| *** rdopiera has joined #openstack-nova | 08:56 | |
| *** ijw_ has quit IRC | 08:58 | |
| oomichi | dedery: did you try "Cherry Pick" botton on the gerrit? it is the easist way for creating a backporting patch | 08:59 |
| *** haomaiwang has quit IRC | 09:01 | |
| dedery | oomichi: thanks, i didn't try anything yet. wanted to ask before messing around stuff. i though about the "cherry pick" button but i don't know if it fits 2 commits. | 09:01 |
| Thelo | oomichi: is this fixable with a recheck? | 09:01 |
| *** rdopiera has quit IRC | 09:01 | |
| *** rdopiera has joined #openstack-nova | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| oomichi | dedery: to be honest, I'm not familar with the command line way. but the button way succeeded to create a patch. and the patch has been merged by mriedem :) | 09:04 |
| oomichi | dedery: Click "Cherry Pick" button and select stable/liberty, that's all | 09:04 |
| *** ildikov has joined #openstack-nova | 09:04 | |
| *** agireud has quit IRC | 09:05 | |
| oomichi | Thelo: yeah, I feel so. | 09:05 |
| *** eglynn_ has joined #openstack-nova | 09:05 | |
| *** agireud has joined #openstack-nova | 09:09 | |
| *** ndipanov has joined #openstack-nova | 09:10 | |
| *** markvoelker has quit IRC | 09:12 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 09:12 |
| *** ndipanov has quit IRC | 09:13 | |
| *** jistr has joined #openstack-nova | 09:16 | |
| oomichi | AJaeger: hi, one question about https://review.openstack.org/#/c/265601 | 09:19 |
| oomichi | AJaeger: is the patch created with the same way of all OpenStack projects? | 09:20 |
| oomichi | AJaeger: like g-r bumping patches | 09:20 |
| *** daemontool has joined #openstack-nova | 09:20 | |
| *** ndipanov has joined #openstack-nova | 09:21 | |
| AJaeger | oomichi: yes, this is an automatic import - see the link with explanations | 09:21 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 09:22 |
| AJaeger | oomichi: we run it daily: Check whether there are translations in our translation server that are not in the project and propose them to the project | 09:22 |
| dedery | oomichi: 10x! will give it a try | 09:22 |
| *** sfinucan has joined #openstack-nova | 09:22 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: API: Remove unexpected from errors get_console_output https://review.openstack.org/270685 | 09:22 |
| *** mgoddard has joined #openstack-nova | 09:22 | |
| *** Guest45232 is now known as BobBall | 09:22 | |
| oomichi | AJaeger: thanks, +2 | 09:23 |
| *** paul-carlton2 has joined #openstack-nova | 09:23 | |
| AJaeger | oomichi: see also https://review.openstack.org/#/q/status:open+topic:zanata/translations,n,z for all open reviews | 09:23 |
| AJaeger | thanks, oomichi | 09:23 |
| *** derekh has joined #openstack-nova | 09:23 | |
| *** yassine has joined #openstack-nova | 09:23 | |
| oomichi | AJaeger: btw, I am curious why we need to run *-tempest-dvms-* for this kind of patches | 09:23 |
| tobasco | can I please get some feedback on https://review.openstack.org/#/c/270482 and some reviews when all CI is done | 09:24 |
| tobasco | johnthetubaguy: the above is related to the bug I was talking to you about earlier, like last week | 09:24 |
| oomichi | AJaeger: if avoiding these tests, we don't need to run recheck. | 09:24 |
| *** PaulMurray has joined #openstack-nova | 09:25 | |
| johnthetubaguy | tobasco: ack, running around trying to get mitaka-2 out the door right now, I am afraid | 09:26 |
| ildikov | johnthetubaguy: hi | 09:30 |
| gmann | johnthetubaguy: if you can have look to unblock nova-specs gate- https://review.openstack.org/#/c/270606/2 | 09:30 |
| *** shyama has quit IRC | 09:30 | |
| johnthetubaguy | gmann: oh, is that the patch folks are using elsewhere? I figured we would just apply something globally? | 09:31 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 09:31 |
| ildikov | johnthetubaguy: quick question, can we have this puppy a push now: https://review.openstack.org/#/c/193134/ or should I prep an exception request for the whole chain? | 09:32 |
| johnthetubaguy | ildikov: there is no exception process right now, I haven't invented it yet, folks will try get to that today now its got a +2 on it | 09:32 |
| gmann | johnthetubaguy: for devstack it is applied but for docs and py27 its needed and this is bacause spec repo requirements does not get sync with g-r | 09:33 |
| ildikov | johnthetubaguy: ok, so this one then should be fine I guess, the next one with the +2 is a bug fix anyway | 09:33 |
| johnthetubaguy | gmann: ah, that was the bit I was trying to check, why we didn't get the sync, OK | 09:33 |
| gmann | johnthetubaguy: i am trying to add spec repo in sync queue - https://review.openstack.org/#/c/270671/ | 09:34 |
| *** mgoddard has quit IRC | 09:34 | |
| ndipanov | ildikov, I'll +2 it and add my comments from yesterday - but I'll dodge the responsibility of +w ing it and make johnthetubaguy work for his massive PTL salary (oh wait...) | 09:34 |
| gmann | johnthetubaguy: but before that check requirement job needs to be added for those | 09:34 |
| *** mgoddard has joined #openstack-nova | 09:34 | |
| gmann | johnthetubaguy: i will do that in parallel to avoid any such future issues | 09:34 |
| johnthetubaguy | gmann: cool, nice | 09:34 |
| ildikov | johnthetubaguy: as for the FFE as discussed yesterday with mriedem and ndipanov we can finalize what's needed in Mitaka during the mid-cycle and then I fix whatever is missing in the patches and let them land to finish prep | 09:35 |
| johnthetubaguy | ndipanov: heh, yeah, I hope to hit that one today | 09:35 |
| ildikov | ndipanov: lol :), thanks | 09:35 |
| gmann | johnthetubaguy: Thanks | 09:35 |
| ildikov | ndipanov: this patch is needed anyway and regarding disconnect_volume I need to think how to fix that | 09:36 |
| ildikov | ndipanov: or if you have any hints, just add them as comments | 09:36 |
| ndipanov | will do | 09:36 |
| ildikov | ndipanov: appreciate it pretty much, I don't know that part so deeply yet | 09:36 |
| *** haomaiwang has quit IRC | 09:37 | |
| *** haomaiwa_ has joined #openstack-nova | 09:37 | |
| dedery | ndipanov: hi, https://review.openstack.org/#/c/262341 is ready for review (changed according to your comments and modified the structure a little bit to look better) | 09:39 |
| *** jyuso1 has quit IRC | 09:40 | |
| *** shyama has joined #openstack-nova | 09:41 | |
| *** femnad has joined #openstack-nova | 09:42 | |
| *** markvoelker has joined #openstack-nova | 09:42 | |
| *** jacky-zhang has quit IRC | 09:42 | |
| ndipanov | dedery, yeah it's on my list but today is FF day so we mostly try to focus on features, while we can land bugs any time up to the release | 09:43 |
| ndipanov | well | 09:43 |
| ndipanov | bug fixes | 09:43 |
| ndipanov | we can land bugs as well but it's not something I personally focus on :) | 09:43 |
| ndipanov | though some people might argue with that... | 09:43 |
| johnthetubaguy | landing release breaking bugs is for the RC period, thats weeks away | 09:44 |
| ndipanov | bonus points if they also break the gate | 09:45 |
| johnthetubaguy | good point | 09:45 |
| *** acabot has quit IRC | 09:45 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 09:47 |
| *** markvoelker has quit IRC | 09:48 | |
| *** zhangjn has quit IRC | 09:48 | |
| markus_z | sahid: do you have a minute? | 09:50 |
| *** PaulMurray has quit IRC | 09:51 | |
| johnthetubaguy | ildikov: that patch reads really nicely now, the attachment_id stuff, top work | 09:51 |
| tdurakov | hi everyone | 09:51 |
| *** rotbeard has joined #openstack-nova | 09:52 | |
| *** ociuhandu has quit IRC | 09:52 | |
| tdurakov | johnthetubaguy, do you have time to review https://review.openstack.org/#/c/252283 ? | 09:52 |
| ildikov | johnthetubaguy: I owe many people many beers for all the help with it | 09:52 |
| *** zhangjn has joined #openstack-nova | 09:52 | |
| *** thorst has joined #openstack-nova | 09:53 | |
| johnthetubaguy | tdurakov: I will try, let us know if you get the live-migrate test passing on there later on | 09:53 |
| johnthetubaguy | ildikov: the folks that need volume multi-attach owe you more beer! | 09:53 |
| ildikov | johnthetubaguy: I had a hard time with tests in the sense of many relies on old mocking style that I didn't know you're deprecating, etc. so I needed a few rounds with it to get cleaned :) | 09:53 |
| *** zhangjn has quit IRC | 09:53 | |
| ildikov | johnthetubaguy: so thanks :) | 09:53 |
| ildikov | johnthetubaguy: that's true, I cannot argue with that statement ;) | 09:54 |
| *** tdurakov has quit IRC | 09:54 | |
| *** tdurakov has joined #openstack-nova | 09:54 | |
| dedery | ndipanov: lol! had my share of breaking the gate with neutron ipv6 fix. mriedem pointed me to the Vancouver talk he gave about it one month before saying something like "let's say someone pushes a fix for ipv6 and it breaks the gate". the guy is a prophet ;) | 09:54 |
| tdurakov | johnthetubaguy, ok, I've triggered experimental check for latest patch set, ping you as it's done | 09:55 |
| johnthetubaguy | tdurakov: awesome, thank you | 09:55 |
| *** kbyrne has joined #openstack-nova | 09:55 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: API: Catch NovaException when get virt driver's console_output https://review.openstack.org/270695 | 09:56 |
| *** lpetrut has quit IRC | 09:58 | |
| *** acabot has joined #openstack-nova | 09:58 | |
| *** ssurana has quit IRC | 09:59 | |
| ndipanov | ildikov, johnthetubaguy https://review.openstack.org/#/c/256273/71 | 10:00 |
| *** thorst has quit IRC | 10:00 | |
| ndipanov | I left the comment as expected | 10:00 |
| ildikov | ndipanov: I'm just reading it, thanks | 10:00 |
| ndipanov | as promised* (not sure if it was expected :)) | 10:01 |
| *** haomaiwa_ has quit IRC | 10:01 | |
| ildikov | ndipanov: with mriedem we were thinking about blocking live migration for multiattach volumes until we don't figure out something | 10:01 |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| ildikov | ndipanov: for all the other cases Cinder should be able to handle the situation by getting the host_name with the attach request | 10:01 |
| *** ssurana has joined #openstack-nova | 10:01 | |
| ndipanov | ildikov, live migration I haven't even looked at :) | 10:02 |
| ildikov | ndipanov: I expected you keep your promise :) | 10:02 |
| *** openstackgerrit has quit IRC | 10:02 | |
| johnthetubaguy | ndipanov: oh, I think I get what you mean, is that driver specific? | 10:02 |
| ildikov | ndipanov: we will be busy next week for the joint Cinder slot on the mid-cycle :) | 10:02 |
| *** openstackgerrit has joined #openstack-nova | 10:02 | |
| ildikov | johnthetubaguy: 3PAR exports targets per attachment | 10:03 |
| ildikov | johnthetubaguy: lvm is problematic as it does not, therefore without host info Cinder has no clue when not to detach | 10:03 |
| ndipanov | johnthetubaguy, os-brick volume driver yes | 10:03 |
| ndipanov | I'd imagine iscsi is also problematic | 10:03 |
| ildikov | johnthetubaguy: and also there's disconnect_volume which is not handled on Nova side | 10:03 |
| ildikov | johnthetubaguy: I missed that one so far, it will need an extra patch | 10:03 |
| *** PaulMurray has joined #openstack-nova | 10:04 | |
| *** yamahata has quit IRC | 10:04 | |
| johnthetubaguy | OK, yeah, disconnect_volume is very live-migrate related | 10:04 |
| ndipanov | so it may be something we want to make os-brick aware of as in part of it's API - but we can't reasonably ask ildikov to fix that first :) | 10:04 |
| johnthetubaguy | ndipanov: +1 on both counts | 10:04 |
| ildikov | ndipanov: thanks, that's good news for me :) so +2 ;) | 10:05 |
| *** zhangjn has joined #openstack-nova | 10:05 | |
| ndipanov | johnthetubaguy, https://review.openstack.org/#/c/262938/4 if you have a sec - if we decide it's a bad idea during m3 - it's easy to take out | 10:05 |
| ildikov | ndipanov: I mean I would fix it if I could, but I would rather ask hemna to look at it, he's my expert I usually ask on that front | 10:06 |
| ndipanov | what I meant was it would not be fair to block feature work on that | 10:06 |
| ndipanov | not that you personally have to do it :0 | 10:06 |
| ndipanov | :) | 10:06 |
| *** ssurana has quit IRC | 10:06 | |
| ildikov | yeah, I got that part too :) | 10:06 |
| *** inara` has quit IRC | 10:07 | |
| *** daemontool_ has joined #openstack-nova | 10:07 | |
| *** daemontool has quit IRC | 10:07 | |
| *** jaosorior has quit IRC | 10:09 | |
| *** inara has joined #openstack-nova | 10:09 | |
| *** jaosorior has joined #openstack-nova | 10:10 | |
| *** jaosorior has quit IRC | 10:10 | |
| *** achanda has joined #openstack-nova | 10:11 | |
| *** jaosorior has joined #openstack-nova | 10:11 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Issue an info log msg when port quota is exceeded https://review.openstack.org/269135 | 10:12 |
| johnthetubaguy | ndipanov: I am close to +2 on that scheduler one, it seems a nice step to share all that claim code | 10:12 |
| ildikov | johnthetubaguy: thanks! | 10:13 |
| *** daemontool_ is now known as daemontool | 10:14 | |
| ildikov | johnthetubaguy: I will fix mriedem's comment on the third patch and remove the WIP tag and we can discuss it further next week from there | 10:14 |
| ildikov | johnthetubaguy: I assume we can finalize this amendtment next week too to the spec: https://review.openstack.org/#/c/264965/ | 10:15 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Remove catching of ComputeHostNotFound exception https://review.openstack.org/259470 | 10:15 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'service' https://review.openstack.org/241178 | 10:16 |
| *** achanda has quit IRC | 10:17 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'compute_node' https://review.openstack.org/241189 | 10:17 |
| ndipanov | johnthetubaguy, yeah and it doesn't rip everything out but it's a nice incremental step | 10:17 |
| mikal | johnthetubaguy: I wont be able to come to the nova meeting today btw, as I have a clashing thing | 10:19 |
| mikal | johnthetubaguy: unless you're coming to this kickoff thign as well... | 10:19 |
| johnthetubaguy | mikal: I think its the late one this week | 10:20 |
| johnthetubaguy | mikal: so yeah, I will not be there either (concert band today) | 10:21 |
| *** doug-fish has joined #openstack-nova | 10:22 | |
| PaulMurray | johnthetubaguy, ndipanov the patch series to allow attach/detach volumes for shelved instances is complete and has been checked by scottda - its non-priority so looking to get it in. I think you were both interested in this one, any chance of a reiew? | 10:23 |
| *** aix has joined #openstack-nova | 10:24 | |
| *** lpetrut has joined #openstack-nova | 10:25 | |
| johnthetubaguy | PaulMurray: possibly in a bit, need to chase for a git hash | 10:26 |
| johnthetubaguy | that came out wrong... but hey | 10:26 |
| bauzas | johnthetubaguy: I'm really afk today because of my conference, but I think we could cut when you want | 10:26 |
| *** doug-fish has quit IRC | 10:27 | |
| ndipanov | PaulMurray, I'll try | 10:27 |
| bauzas | johnthetubaguy: the reno changes have been merged | 10:27 |
| bauzas | johnthetubaguy: we just have a few notes to be merged, but waiting for them is IMHO not really needed | 10:28 |
| PaulMurray | ndipanov, cool, thanks - this is the last patch in the series: https://review.openstack.org/#/c/265898 | 10:28 |
| PaulMurray | three in total - the last in the series is moving on to boot volumes which is a different bp | 10:29 |
| *** StevenK has quit IRC | 10:29 | |
| *** ssurana has joined #openstack-nova | 10:30 | |
| PaulMurray | The one that enables the API is the last part of the bp - never mind, you'll understand..... | 10:31 |
| sahid | markus_z: hello, yes? | 10:32 |
| *** zhangjn has quit IRC | 10:33 | |
| markus_z | sahid: Hey, I was wondering if you could me give a hint here: https://review.openstack.org/#/c/247396/3/nova/virt/libvirt/imagebackend.py | 10:33 |
| markus_z | sahid: Matt pointed out that there is a test missing and I was unsure where the test should live. | 10:34 |
| *** ssurana has quit IRC | 10:34 | |
| markus_z | sahid: "virt/libvirt/test_imagebackend.py" has multiple test cases and I cannot differentiate their purpose. | 10:34 |
| johnthetubaguy | bauzas: ack, I think the only nice to have now is this: https://review.openstack.org/#/c/265601/12 | 10:35 |
| bauzas | johnthetubaguy: yeah I saw | 10:35 |
| markus_z | sahid: Do you think the generic "BackendTestCase" is the right one? | 10:36 |
| johnthetubaguy | bauzas: not sure we can wait for that one though | 10:36 |
| *** gszasz has joined #openstack-nova | 10:36 | |
| johnthetubaguy | bauzas: so gate queue is 34hours, so lets just ready a release patch | 10:37 |
| johnthetubaguy | bauzas: are you OK to pick the latest commit, and upload the paper work? I can do that if not | 10:37 |
| bauzas | johnthetubaguy: ack, I can do it | 10:37 |
| johnthetubaguy | bauzas: thanks, we have the bug fix I wanted in merged this morning, so I think we just go for it | 10:37 |
| sahid | markus_z: yes, i'm not completly aware of this part of code but BackendTestCase seems to be a good fit for such tests | 10:38 |
| markus_z | sahid: ok, cool, thanks! Just wanted to double-check upfront before pushing the patch. | 10:38 |
| Thelo | johnthetubaguy: any chance to include this one https://review.openstack.org/#/c/253353/? | 10:39 |
| *** markvoelker has joined #openstack-nova | 10:39 | |
| johnthetubaguy | Thelo: not in mitaka-2, the gate queue is too big | 10:39 |
| Thelo | johnthetubagay: how about mitaka-3? | 10:40 |
| *** amotoki has quit IRC | 10:40 | |
| johnthetubaguy | Thelo: its +W already, so yes, it should get into m-3 | 10:40 |
| tdurakov | johnthetubaguy, ready | 10:40 |
| Thelo | johnthetubaguy: Fantastic, thanks | 10:40 |
| tdurakov | https://review.openstack.org/#/c/252283/ | 10:40 |
| bauzas | johnthetubaguy: since we need a merge commit, which one should be cut ? | 10:41 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: use native AIO mode for image backends https://review.openstack.org/247396 | 10:41 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: use native AIO mode for cinder volumes https://review.openstack.org/251829 | 10:41 |
| *** shyama has quit IRC | 10:41 | |
| *** markvoelker has quit IRC | 10:44 | |
| johnthetubaguy | tdurakov: I still find that really hard to read, for some reason | 10:45 |
| tdurakov | johnthetubaguy, well, how to make it clearer? | 10:46 |
| *** Jeffrey4l has quit IRC | 10:46 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 10:47 |
| johnthetubaguy | tdurakov: trying to put some suggestions in there, its basically what cdent was also saying, basically need a set of functions called from run_tests that ideally are all at the same level of abstraction | 10:48 |
| *** mfedosin has joined #openstack-nova | 10:48 | |
| tdurakov | johnthetubaguy, leave a comment please | 10:50 |
| *** gjayavelu has joined #openstack-nova | 10:51 | |
| *** cdent has joined #openstack-nova | 10:51 | |
| tdurakov | johnthetubaguy, I could wrap all ceph preparation in another function, it could make run_tests hook file clearer | 10:51 |
| johnthetubaguy | tdurakov: writing up comments | 10:52 |
| bauzas | johnthetubaguy: cutting at http://git.openstack.org/cgit/openstack/nova/commit/?id=278b16ed24a3e114445d80e49153edc080064113 you okay ? | 10:53 |
| *** StevenK has joined #openstack-nova | 10:53 | |
| bauzas | for the translations, they will be in the next milestone | 10:53 |
| *** danpb has joined #openstack-nova | 10:54 | |
| *** gjayavelu has quit IRC | 10:55 | |
| *** thorst has joined #openstack-nova | 10:58 | |
| *** e0ne has joined #openstack-nova | 10:58 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwa_ has joined #openstack-nova | 11:01 | |
| johnthetubaguy | bauzas: yeah, I think thats the best we can do at this point | 11:02 |
| *** shyama has joined #openstack-nova | 11:02 | |
| johnthetubaguy | bauzas: have you got the release change up for that? | 11:02 |
| *** dims has joined #openstack-nova | 11:04 | |
| *** thorst has quit IRC | 11:04 | |
| johnthetubaguy | tdurakov: I added some ideas about what we could do | 11:05 |
| tdurakov | johnthetubaguy, ok, reading | 11:05 |
| tdurakov | thank you | 11:05 |
| bauzas | johnthetubaguy: https://review.openstack.org/270732 | 11:06 |
| bauzas | well, gerrit is bouncing | 11:07 |
| *** aix has quit IRC | 11:07 | |
| *** aysyd has joined #openstack-nova | 11:07 | |
| *** irenab has quit IRC | 11:07 | |
| *** PaulMurray has quit IRC | 11:08 | |
| *** shyama has quit IRC | 11:09 | |
| openstackgerrit | Dina Belova proposed openstack/nova: Integrate OSprofiler and Nova https://review.openstack.org/254703 | 11:12 |
| *** houming has quit IRC | 11:13 | |
| tdurakov | johnthetubaguy, read your comments, do we really need to wrap tempest config in for nfs in function as it one line of code? Also not sure that it's good idea to check nfs setup in tempest function. It looks like side-effect | 11:14 |
| mikal | johnthetubaguy: do we have a process at the moment for tracking things we thing are bonkers and should be removed later? | 11:15 |
| mikal | johnthetubaguy: i.e., while reading new help text for the scheduler it has become clear to me that isolated_images needs to die | 11:15 |
| mikal | johnthetubaguy: but how do I track that? A backlog blueprint seems like overkill | 11:16 |
| tonyb | mikal: git rm? | 11:16 |
| *** PaulMurray has joined #openstack-nova | 11:16 | |
| mikal | tonyb: you're hired! | 11:17 |
| tonyb | \0/ | 11:17 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Moved tags filtering tests to TestInstanceTagsFiltering test case https://review.openstack.org/258515 | 11:21 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 11:22 |
| johnthetubaguy | tdurakov: I think its worth wrapping, because its then clear what its doing, the check I saw as confirming the setup worked, so maybe its part of the setup_nfs stuff? | 11:22 |
| *** ajayaa has joined #openstack-nova | 11:22 | |
| johnthetubaguy | mikal: maybe lets start a text file in the nova tree, or a devref? not sure | 11:23 |
| *** lyanchih has quit IRC | 11:23 | |
| mikal | johnthetubaguy: yeah, I don't mind what it is, as long as we track things somehow | 11:23 |
| mikal | johnthetubaguy: how about a todo above the help text? | 11:23 |
| tdurakov | johnthetubaguy, ok, I'll wrap that and try to align all of these as much as possible, will ping you later | 11:23 |
| mikal | johnthetubaguy: at least then its with the code, not that we ever do todos | 11:24 |
| mikal | RANT(mikal): how is this not a host aggregate?!? | 11:24 |
| tonyb | then we can git grep 'CRAZYPANTS' to find them all | 11:24 |
| *** ajo has quit IRC | 11:24 | |
| *** ociuhandu has joined #openstack-nova | 11:25 | |
| *** ajo has joined #openstack-nova | 11:25 | |
| *** lpetrut has quit IRC | 11:26 | |
| * mikal wanders off to a thing involving being kicked | 11:27 | |
| * tonyb copies mikal because he's a leader ..... | 11:29 | |
| *** sahid has quit IRC | 11:29 | |
| johnthetubaguy | tdurakov: its more just so you can read the top level quickly, and know where to find the details, etc | 11:30 |
| johnthetubaguy | mikal: yeah, something like that, FIXME seems fine for that mind | 11:30 |
| johnthetubaguy | mikal: tonyb: enjoy the kicking | 11:30 |
| *** rfolco has joined #openstack-nova | 11:32 | |
| *** szhsong has quit IRC | 11:32 | |
| *** lpetrut has joined #openstack-nova | 11:33 | |
| johnthetubaguy | bauzas: thanks, that looks good | 11:33 |
| *** szhsong has joined #openstack-nova | 11:34 | |
| *** markvoelker has joined #openstack-nova | 11:34 | |
| *** ccarmack has joined #openstack-nova | 11:36 | |
| *** tdurakov has quit IRC | 11:38 | |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: remove configurable action_peek https://review.openstack.org/270747 | 11:38 |
| *** markvoelker has quit IRC | 11:39 | |
| sdague | mikal: or a bug category | 11:39 |
| sdague | or just push the deprecation patch | 11:40 |
| *** gcb has quit IRC | 11:40 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova-specs: REST API changes for user settable server description https://review.openstack.org/269664 | 11:40 |
| *** szhsong has quit IRC | 11:41 | |
| markus_z | mikal: wrt to "isolated_images" stuff, how about an addtion to the http://docs.openstack.org/developer/nova/scheduler_evolution.html ? | 11:42 |
| ccarmack | alex_xu, gmann, johnthetuba: I rebased the spec and need it reviewed again ^ | 11:43 |
| gmann | ccarmack: did you rebase on https://review.openstack.org/#/c/270606/ | 11:44 |
| gmann | ccarmack: or it will still fail | 11:44 |
| ccarmack | gmann: sorry, I didn't rebase on this | 11:46 |
| gmann | ccarmack: from UI only you can do | 11:46 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Pass req to all API plugins https://review.openstack.org/270658 | 11:46 |
| ccarmack | ok, let me do it again | 11:46 |
| ccarmack | oomichi: could you review https://review.openstack.org/#/c/254950/.. I need a 2nd core to review it | 11:48 |
| oomichi | ccarmack: sure, looking... | 11:48 |
| ccarmack | thanks... | 11:49 |
| *** tdurakov has joined #openstack-nova | 11:50 | |
| *** gcb has joined #openstack-nova | 11:53 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova-specs: REST API changes for user settable server description https://review.openstack.org/269664 | 11:53 |
| oomichi | ccarmack: the current nova-spec doesn't contain rebuild thing | 11:56 |
| oomichi | ccarmack: create/update server only | 11:56 |
| oomichi | ccarmack: for the consistency, I prefer current implementation patch. | 11:57 |
| oomichi | ccarmack: ah, the above patch is fixing it. | 11:58 |
| *** tdurakov has quit IRC | 11:58 | |
| ccarmack | oomichi: I added sections to the nova spec for rebuild: https://review.openstack.org/#/c/269664/5/specs/mitaka/approved/user-settable-server-description.rst | 11:58 |
| oomichi | ccarmack: yeah, I am looking it now | 11:58 |
| *** smatzek_ has joined #openstack-nova | 11:59 | |
| ccarmack | gmann: ok, the specs build worked... thanks a lot for the help | 12:00 |
| *** haomaiwa_ has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-nova | 12:01 | |
| *** thorst has joined #openstack-nova | 12:02 | |
| *** aix has joined #openstack-nova | 12:03 | |
| *** boris-42 has quit IRC | 12:03 | |
| gmann | ccarmack: np!, in https://review.openstack.org/#/c/254950/ looks like you are running tests twice | 12:03 |
| gmann | ccarmack: checking.. | 12:03 |
| *** cdent has quit IRC | 12:05 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Moved tags filtering tests to TestInstanceTagsFiltering test case https://review.openstack.org/258515 | 12:06 |
| *** cdent has joined #openstack-nova | 12:06 | |
| gmann | ccarmack: sorry thats fine. just commented . looks fine | 12:06 |
| *** apoorv has quit IRC | 12:07 | |
| *** claudiub has joined #openstack-nova | 12:07 | |
| *** coolsvap is now known as coolsvap|away | 12:07 | |
| ccarmack | gmann: excellent! | 12:08 |
| *** thorst has quit IRC | 12:10 | |
| *** achanda has joined #openstack-nova | 12:14 | |
| *** dave-mccowan has joined #openstack-nova | 12:17 | |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: remove custom serializer support https://review.openstack.org/270763 | 12:18 |
| *** lpetrut has quit IRC | 12:18 | |
| *** haomaiwang has quit IRC | 12:18 | |
| *** raildo-afk is now known as raildo | 12:20 | |
| *** achanda has quit IRC | 12:20 | |
| lyarwood | eglynn_: any objections to me creating and taking ownership of a 2015.1.3 rebase bug for OSP 7? I just noticed that it's due soon, not sure if we normally wait for the release before creating a rebase BZ etc. | 12:21 |
| lyarwood | eglynn_: whops ignore please ;) | 12:22 |
| oomichi | ccarmack: +A with nit comment | 12:22 |
| oomichi | nice work anyways :) | 12:23 |
| *** erlon has joined #openstack-nova | 12:24 | |
| ccarmack | oomichi: yes I see it...oh well | 12:24 |
| *** tdurakov_ has joined #openstack-nova | 12:24 | |
| *** lpetrut has joined #openstack-nova | 12:24 | |
| *** jinxing has quit IRC | 12:24 | |
| *** szhsong has joined #openstack-nova | 12:24 | |
| *** tdurakov_ has quit IRC | 12:25 | |
| *** sahid has joined #openstack-nova | 12:25 | |
| oomichi | ccarmack: it is fine to change it with *another* patch later | 12:25 |
| *** doug-fish has joined #openstack-nova | 12:25 | |
| *** oomichi is now known as oomichi_away | 12:26 | |
| ccarmack | oomichi: yes, and I have another similar comment from gmann | 12:26 |
| oomichi_away | ccarmack: cool. need to go back home now. see you :) | 12:26 |
| ccarmack | thanks for the quick review :) | 12:26 |
| *** lpetrut has quit IRC | 12:29 | |
| ccarmack | alex_xu: I need a +W on https://review.openstack.org/#/c/269664/ and https://review.openstack.org/#/c/254950/ ... the errors on the 2nd one are those ssh timeouts | 12:29 |
| *** markvoelker has joined #openstack-nova | 12:29 | |
| *** otter768 has joined #openstack-nova | 12:30 | |
| *** oomichi_away has quit IRC | 12:31 | |
| alex_xu | ccarmack: oomichi already ave you + w | 12:32 |
| alex_xu | s/ave/gave | 12:33 |
| ccarmack | oh sorry, didn't see that | 12:33 |
| ccarmack | alex_xu: thanks a lot for the help | 12:33 |
| alex_xu | ccarmack: np :) | 12:34 |
| *** nelsnelson has quit IRC | 12:34 | |
| *** markvoelker has quit IRC | 12:35 | |
| sdague | cdent: good catch, thanks | 12:35 |
| *** otter768 has quit IRC | 12:35 | |
| *** thorst has joined #openstack-nova | 12:43 | |
| *** vladikr has quit IRC | 12:44 | |
| *** vladikr has joined #openstack-nova | 12:44 | |
| *** ccarmack has left #openstack-nova | 12:45 | |
| *** chenli has quit IRC | 12:46 | |
| *** chenli has joined #openstack-nova | 12:47 | |
| *** esracelik has quit IRC | 13:00 | |
| *** diogogmt_ has joined #openstack-nova | 13:00 | |
| *** diogogmt has quit IRC | 13:01 | |
| *** diogogmt_ is now known as diogogmt | 13:01 | |
| *** haomaiwang has joined #openstack-nova | 13:01 | |
| *** baoli has joined #openstack-nova | 13:02 | |
| *** ihrachys has quit IRC | 13:04 | |
| *** pradipta has joined #openstack-nova | 13:05 | |
| *** ihrachys has joined #openstack-nova | 13:05 | |
| *** lyanchih has joined #openstack-nova | 13:06 | |
| *** jamielennox is now known as jamielennox|away | 13:07 | |
| *** tdurakov_ has joined #openstack-nova | 13:07 | |
| *** efoley has joined #openstack-nova | 13:07 | |
| *** tdurakov_ has quit IRC | 13:08 | |
| *** tdurakov_ has joined #openstack-nova | 13:09 | |
| *** chenli has quit IRC | 13:09 | |
| *** tdurakov_ has quit IRC | 13:11 | |
| *** Guest7791 is now known as bradjones | 13:12 | |
| *** bradjones has joined #openstack-nova | 13:12 | |
| *** ducttape_ has joined #openstack-nova | 13:12 | |
| *** amotoki has joined #openstack-nova | 13:12 | |
| openstackgerrit | Sven Anderson proposed openstack/nova: Add a new setup command "compile_all_catalogs" https://review.openstack.org/270789 | 13:15 |
| ansiwen | AJaeger: I think I found a way, see change above | 13:18 |
| *** daemontool has quit IRC | 13:19 | |
| *** baoli has quit IRC | 13:21 | |
| *** baoli has joined #openstack-nova | 13:22 | |
| *** jed56 has quit IRC | 13:23 | |
| *** ducttape_ has quit IRC | 13:24 | |
| *** markvoelker has joined #openstack-nova | 13:24 | |
| *** alejandrito has joined #openstack-nova | 13:26 | |
| *** markvoelker has quit IRC | 13:29 | |
| *** ivase has joined #openstack-nova | 13:30 | |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: remove custom serializer support https://review.openstack.org/270763 | 13:31 |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: add comments about the pre/post extension processing https://review.openstack.org/270792 | 13:31 |
| *** cdelatte has joined #openstack-nova | 13:32 | |
| sdague | ansiwen: I wonder if that should be a pbr add | 13:33 |
| sdague | mordred: ^^ ? | 13:34 |
| mordred | sdague: morning! | 13:35 |
| *** irenab has joined #openstack-nova | 13:35 | |
| AJaeger | ansiwen: thanks, will look at it later | 13:35 |
| sdague | mordred: it turns out, compile_catalog does not | 13:36 |
| sdague | https://review.openstack.org/#/c/270789/1/setup.cfg | 13:36 |
| sdague | a bunch of translation files are lost in the process | 13:36 |
| mordred | sdague: yes - I think that looks like an excellent thing to add to pbr | 13:36 |
| sdague | mordred, can you work with ansiwen to get that change in ? | 13:37 |
| mordred | sure! | 13:37 |
| sdague | ansiwen: you good with solving this generically in pbr? Then nova will just get fixed on the next pbr release, as will all the rest of the projects | 13:38 |
| mordred | ansiwen: biggest difference between your patch and a pbr version of the patch is that you'll want to wrap it in a try/except on babel - since there are pbr consumers who will not have babel and we don't want those people to all of a sudden break | 13:38 |
| johnthetubaguy | Kevin_Zheng: looking at Active + Quesing, should that not be Unknown for older requests, I don't think I see that logic? | 13:38 |
| mordred | ansiwen: we do that pattern already with the sphinx-related commands though - so there's plenty to cargo-cult from :) | 13:39 |
| *** pradipta has quit IRC | 13:39 | |
| Kevin_Zheng | johnthetubaguy: you mean status or vm_state for this one | 13:40 |
| johnthetubaguy | Kevin_Zheng: neither, the task_state = quecsing while state=Active | 13:41 |
| *** tiagogomes has quit IRC | 13:41 | |
| Kevin_Zheng | johnthetubaguy: yes I know, I added STATUS = QUIESCE for this one in v2.17 | 13:42 |
| Kevin_Zheng | johnthetubaguy: so I think the STATUS should be unknown in < 2.17 | 13:42 |
| johnthetubaguy | Kevin_Zheng: hmm... not sure that works, I will keep reading | 13:43 |
| Kevin_Zheng | johnthetubaguy: okay, if the overall is good and only this one is not good, I can change it | 13:44 |
| sdague | cdent: you'll also probably appreciate this patch - https://review.openstack.org/#/c/270792/1/nova/api/openstack/wsgi.py - which is all comments | 13:44 |
| sdague | but at least trying to pay it forward a bit | 13:45 |
| *** tdurakov_ has joined #openstack-nova | 13:45 | |
| *** dguitarbite has quit IRC | 13:45 | |
| *** tdurakov_ has quit IRC | 13:46 | |
| *** EinstCrazy has quit IRC | 13:47 | |
| *** dguitarbite has joined #openstack-nova | 13:47 | |
| johnthetubaguy | Kevin_Zheng: I am struggling to understand the changes in here: https://review.openstack.org/#/c/248989/80/nova/api/openstack/wsgi.py | 13:48 |
| ansiwen | sdague: sure, I don't mind where it's gonna be fixed, I just want it to be fixed ;-) | 13:49 |
| johnthetubaguy | Kevin_Zheng: surely one request can only ever support one request version, so there is no need to worry about changing things when they go in the cache? | 13:49 |
| Kevin_Zheng | johnthetubaguy: actually this is the hardest part | 13:49 |
| Kevin_Zheng | johnthetubaguy: when call list and show | 13:49 |
| Kevin_Zheng | johnthetubaguy: the vm_states and task_state are actually get from here | 13:50 |
| *** markvoelker has joined #openstack-nova | 13:50 | |
| johnthetubaguy | Kevin_Zheng: thats fine, but surely we should just but the correct values in the cache for the given request? | 13:51 |
| ansiwen | mordred: ok, but this means it will be available for all components, right? So, I have to be sure that my assumtions of that code are true in general | 13:51 |
| Kevin_Zheng | johnthetubaguy: what are you suggesting? | 13:51 |
| *** cdelatte has quit IRC | 13:52 | |
| *** markvoelker has quit IRC | 13:53 | |
| *** tdurakov_ has joined #openstack-nova | 13:53 | |
| *** markvoelker has joined #openstack-nova | 13:53 | |
| johnthetubaguy | Kevin_Zheng: it feels like the wrong place to be manipulating the server | 13:53 |
| *** szhsong has quit IRC | 13:53 | |
| mordred | ansiwen: yes - although I believe your assumptions are true in general - but it's definitely good to doublecheck | 13:53 |
| mordred | ansiwen: also, as a warning, lifeless will want tests | 13:53 |
| *** dslevin has quit IRC | 13:54 | |
| Kevin_Zheng | johnthetubaguy: but those states are get here directly from DB | 13:54 |
| Kevin_Zheng | johnthetubaguy: No more place to deal with them | 13:54 |
| ansiwen | mordred: sure, this is more like a draft anyways, no documentation so far... | 13:55 |
| mordred | ansiwen: :) | 13:55 |
| mordred | ansiwen: well, I like it - it's exactly the sort of thing we wrote pbr for - when we encounter problems like this, they really are things that solving for all of openstack is a good idea | 13:56 |
| *** szhsong has joined #openstack-nova | 13:56 | |
| ansiwen | mordred: ok, cool. never looked at pbr before, really... | 13:57 |
| *** Jeffrey4l has joined #openstack-nova | 13:57 | |
| mordred | ansiwen: there are parts of it that are very scary | 13:57 |
| Kevin_Zheng | johnthetubaguy: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/servers.py#L278~L299 | 13:57 |
| mordred | ansiwen: but lifeless has been doing a good job of making them less scary | 13:57 |
| *** markvoelker has quit IRC | 13:57 | |
| Kevin_Zheng | johnthetubaguy: only these are returned for display, others are get directly from DB | 13:58 |
| *** ljxiash has quit IRC | 13:58 | |
| ansiwen | mordred: ok, I'll be AFK for lunch... will ping you later when I had a look at it | 13:58 |
| *** markvoelker has joined #openstack-nova | 13:59 | |
| mordred | cool! | 13:59 |
| *** ansiwen is now known as ansiwen|afk | 13:59 | |
| johnthetubaguy | Kevin_Zheng: we already do quite a bit of alias work, I just expected this to be all grouped together, I will add comments on the patch | 13:59 |
| Kevin_Zheng | johnthetubaguy: OK, I will check it | 14:00 |
| *** ljxiash has joined #openstack-nova | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| *** Jeffrey4l has quit IRC | 14:02 | |
| *** shyama has joined #openstack-nova | 14:02 | |
| *** burgerk has joined #openstack-nova | 14:03 | |
| *** efoley has quit IRC | 14:04 | |
| *** annegentle has joined #openstack-nova | 14:04 | |
| *** efoley has joined #openstack-nova | 14:04 | |
| *** daemontool has joined #openstack-nova | 14:05 | |
| *** ljxiash has quit IRC | 14:05 | |
| *** mdrabe has joined #openstack-nova | 14:06 | |
| *** irenab has quit IRC | 14:07 | |
| *** irenab has joined #openstack-nova | 14:08 | |
| *** edmondsw has joined #openstack-nova | 14:08 | |
| openstackgerrit | jichenjc proposed openstack/nova: prevent non-admin user search soft-delete server https://review.openstack.org/258472 | 14:08 |
| *** lyanchih has quit IRC | 14:09 | |
| openstackgerrit | jichenjc proposed openstack/nova: Return empty object list instead [] https://review.openstack.org/258962 | 14:09 |
| *** ctrath has joined #openstack-nova | 14:10 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 14:11 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow volume multiattach https://review.openstack.org/267587 | 14:11 |
| *** rlrossit has joined #openstack-nova | 14:11 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 14:11 |
| *** tdurakov_ has quit IRC | 14:14 | |
| *** flip214 has quit IRC | 14:14 | |
| *** tdurakov_ has joined #openstack-nova | 14:14 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'security_group' https://review.openstack.org/246302 | 14:15 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'security_group' https://review.openstack.org/246302 | 14:17 |
| *** jacky-zhang has joined #openstack-nova | 14:17 | |
| *** rlrossit has quit IRC | 14:17 | |
| *** flip214 has joined #openstack-nova | 14:17 | |
| *** achanda has joined #openstack-nova | 14:17 | |
| *** tdurakov_ has quit IRC | 14:18 | |
| *** rvasilets has joined #openstack-nova | 14:18 | |
| *** jlanoux has quit IRC | 14:19 | |
| *** tdurakov_ has joined #openstack-nova | 14:21 | |
| *** achanda has quit IRC | 14:23 | |
| *** lixiaoy1 has quit IRC | 14:24 | |
| *** raildo is now known as raildo-afk | 14:24 | |
| *** tdurakov_ has quit IRC | 14:24 | |
| *** alejandrito has quit IRC | 14:26 | |
| rvasilets | Hi guys. I took minimal configuration of devstack from here https://github.com/openstack-dev/devstack/blob/master/doc/source/configuration.rst And tried to boot server. It was booted but there is no internet connection on the instance. Does some one faced with such an issue? I have no idea how fix this. The image was taken from here http://cloud-images.ubuntu.com/trusty/current/ Logs from the instance http://paste.openstack.org/show/484574/ | 14:26 |
| *** mriedem has joined #openstack-nova | 14:28 | |
| *** szhsong has quit IRC | 14:28 | |
| *** ljxiash has joined #openstack-nova | 14:28 | |
| *** ihrachys has quit IRC | 14:28 | |
| rvasilets | devstack is the newest | 14:29 |
| *** rlrossit_ has joined #openstack-nova | 14:30 | |
| *** tdurakov_ has joined #openstack-nova | 14:30 | |
| *** otter768 has joined #openstack-nova | 14:31 | |
| *** tdurakov_ has quit IRC | 14:32 | |
| *** tdurakov_ has joined #openstack-nova | 14:33 | |
| mriedem | hmm, looks like this should maybe not be an error: http://logs.openstack.org/67/269867/4/check/gate-tempest-dsvm-neutron-src-os-brick/7617a9f/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-01-21_05_38_48_667 | 14:34 |
| mriedem | shows up in a lot of job runs | 14:34 |
| *** tdurakov_ has quit IRC | 14:35 | |
| *** otter768 has quit IRC | 14:36 | |
| *** tdurakov_ has joined #openstack-nova | 14:37 | |
| *** ihrachys has joined #openstack-nova | 14:39 | |
| *** lpetrut has joined #openstack-nova | 14:39 | |
| *** dslev has joined #openstack-nova | 14:39 | |
| mriedem | sahid: ^ looks like that is coming from the guest.detach_device code, seems we should handle device not found on detach as a warning? | 14:40 |
| *** cfriesen has joined #openstack-nova | 14:42 | |
| *** fawadkhaliq has joined #openstack-nova | 14:42 | |
| *** jacky-zhang has quit IRC | 14:44 | |
| *** mrkz has joined #openstack-nova | 14:44 | |
| sahid | mriedem: yeah... we only handle domain not gound | 14:45 |
| sahid | s/gound/found | 14:45 |
| mriedem | https://bugzilla.redhat.com/show_bug.cgi?id=1056902 looked related | 14:45 |
| openstack | bugzilla.redhat.com bug 1056902 in libvirt "virsh attach-interface/detach-interface mishandles inactive configuration on device hot(un)plug commands" [Medium,Closed: errata] - Assigned to pkrempa | 14:45 |
| *** paul-carlton2 has quit IRC | 14:46 | |
| *** psargent has quit IRC | 14:46 | |
| *** tdurakov_ has quit IRC | 14:46 | |
| *** busterswt has joined #openstack-nova | 14:46 | |
| *** lpetrut has quit IRC | 14:46 | |
| mriedem | so we must be hitting this http://libvirt.org/git/?p=libvirt.git;a=blob;f=src/qemu/qemu_driver.c;h=e04a32841807851fd1386ab8a7fd91672f114dce;hb=e8684eb541f01df9b45e87e0a8ce446c7bc90a17#l6764 | 14:47 |
| mriedem | there is no specific error code for that :( | 14:47 |
| mriedem | so you'd have to parse the error message, which is translatable | 14:47 |
| sahid | mriedem: yes that seems related to config passed to detach device and so resolved start to libvirt 1.2.7 | 14:49 |
| *** thangp has joined #openstack-nova | 14:49 | |
| mriedem | sahid: on guest.detach_device(conf, persistent=True, live=live) | 14:50 |
| mriedem | what does persistent=True mean? | 14:50 |
| mriedem | changes the domain xml? | 14:50 |
| *** liverpooler has quit IRC | 14:50 | |
| sahid | i want to say yes, so we update persistent domain xml of guest | 14:51 |
| mriedem | ok, i just wish we could handle that as a warning or something, because | 14:52 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22detach_interface%5C%22%20AND%20message%3A%5C%22libvirtError%3A%20operation%20failed%3A%20no%20matching%20network%20device%20was%20found%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d | 14:52 |
| *** ildikov has quit IRC | 14:52 | |
| mriedem | i'm guessing this is racing with deleting the domain | 14:52 |
| *** nelsnelson has joined #openstack-nova | 14:53 | |
| *** mtanino has joined #openstack-nova | 14:54 | |
| *** nelsnels_ has joined #openstack-nova | 14:55 | |
| *** nelsnelson has quit IRC | 14:55 | |
| mriedem | looks like in this case it's coming in from a neutron event http://logs.openstack.org/67/269867/4/check/gate-tempest-dsvm-neutron-src-os-brick/7617a9f/logs/screen-n-cpu.txt.gz#_2016-01-21_05_38_48_361 | 14:56 |
| *** ildikov has joined #openstack-nova | 14:58 | |
| mriedem | is there another way to look up the device before trying to detach it? | 14:59 |
| mriedem | to verify it exists before the detach? | 14:59 |
| *** jlanoux has joined #openstack-nova | 14:59 | |
| alaski | are fixes making their way through the gate yet, or should we hold off on approving things atm? | 15:00 |
| *** haomaiwang has quit IRC | 15:01 | |
| mriedem | danpb: fyi in case you have ideas https://bugs.launchpad.net/nova/+bug/1536671 | 15:01 |
| openstack | Launchpad bug 1536671 in OpenStack Compute (nova) "libvirt detach_interface logs errors for network device not found after neutron network-vif-deleted event" [Undecided,New] | 15:01 |
| mriedem | sdague: any idea what the gate fail rate is right now? | 15:01 |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| sdague | all of it | 15:01 |
| mriedem | mtreinish: looks like https://review.openstack.org/#/c/270528/ didn't fare so well | 15:02 |
| mriedem | sdague: did you see ^ ? | 15:02 |
| alaski | okay, so adding things to the gate would not be wise at this point | 15:02 |
| *** tdurakov_ has joined #openstack-nova | 15:03 | |
| danpb | mriedem: is that saying that we're trying to do detach_interface() twice for the same nic ? | 15:03 |
| *** fawadkhaliq has quit IRC | 15:03 | |
| mriedem | danpb: the request starts here http://logs.openstack.org/67/269867/4/check/gate-tempest-dsvm-neutron-src-os-brick/7617a9f/logs/screen-n-cpu.txt.gz#_2016-01-21_05_38_48_361 | 15:03 |
| *** burgerk has quit IRC | 15:04 | |
| mriedem | danpb: aha http://logs.openstack.org/67/269867/4/check/gate-tempest-dsvm-neutron-src-os-brick/7617a9f/logs/screen-n-cpu.txt.gz#_2016-01-21_05_38_44_455 | 15:04 |
| mriedem | yeah it's already gone 4 seconds earlier | 15:04 |
| danpb | so something higher up probably ought to avoid calling into the virt driver in that case | 15:05 |
| mtreinish | mriedem: yeah, that wasn't gonna work the more I thought about it | 15:05 |
| *** kodokuu has joined #openstack-nova | 15:05 | |
| mtreinish | and the gate confirmed it | 15:05 |
| danpb | though you could silence it in the libvirt driver by querying XML to see if the NIC still exists before unplugging | 15:05 |
| *** pushkaru has joined #openstack-nova | 15:05 | |
| danpb | feels nicer to avoid it higher up though if possible | 15:05 |
| mriedem | danpb: that's what i was wondering | 15:05 |
| mtreinish | mriedem: I think I'm just gonna try reverting sdague's use fixed network patch and recheck loop until we get an internap or bluebox node for the multinode grenade job | 15:06 |
| *** lyanchih has joined #openstack-nova | 15:06 | |
| mtreinish | if it works there then we at least have a short term workaround | 15:06 |
| mriedem | danpb: well, it's 2 things, | 15:06 |
| johnthetubaguy | bauzas: pkholkin: just looking at this: https://review.openstack.org/#/c/259470/4 I think it was merged in kilo, not liberty | 15:06 |
| mriedem | danpb: i think we're deleting the instance, and the port is being deleted in neutron, which sends the vif-deleted event to nova, which then tries to detach the interface concurrently | 15:07 |
| danpb | yeah seems plausible | 15:07 |
| *** boris-42 has joined #openstack-nova | 15:07 | |
| mriedem | alaski: i think if you were looking at non-priority bp stuff and were going to +W, I'd do that regardless b/c of the FF deadline | 15:07 |
| kodokuu | Hi, Anyone know how we can choice a specific folder for a new instance with Vcenter driver ? | 15:07 |
| johnthetubaguy | mriedem: alaski: I am OK counting two +2s as making the FF deadline, if that helps the gate | 15:08 |
| mriedem | johnthetubaguy: ah ok | 15:08 |
| alaski | that works for me | 15:09 |
| mriedem | kodokuu: https://review.openstack.org/#/c/165060/ | 15:09 |
| mriedem | ^ is only in mitaka | 15:09 |
| *** ajayaa has quit IRC | 15:10 | |
| mriedem | danpb: so we get the config here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1347 | 15:11 |
| mriedem | danpb: is that not the same as "querying XML to see if the NIC still exists before unplugging" | 15:11 |
| mriedem | i guess that's not actually getting it from the domain right? | 15:11 |
| mriedem | more like building a request i think? | 15:11 |
| kodokuu | mriedem ok great | 15:12 |
| kodokuu | commit in mitaka ? | 15:12 |
| *** bdrich has joined #openstack-nova | 15:12 | |
| mriedem | kodokuu: yes | 15:12 |
| openstackgerrit | John Garbutt proposed openstack/nova: Remove catching of ComputeHostNotFound exception https://review.openstack.org/259470 | 15:12 |
| kodokuu | mriedem DO you think I can manual commit in kilo ? | 15:12 |
| mriedem | kodokuu: i don't understand the question | 15:13 |
| mriedem | kodokuu: you need a mitaka installation or you have to patch that into your deployment to use folder support with vcenter | 15:13 |
| danpb | mriedem: yeah that's not querying libvirt - its just building fresh xml | 15:14 |
| kodokuu | mriedem Ok I can patch | 15:14 |
| kodokuu | thx | 15:14 |
| *** rmart04 has joined #openstack-nova | 15:14 | |
| mriedem | danpb: could i call https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L172 to get the interfaces and then filter, or add a specific get_interface(vif) to the guest as a call in the driver to see if the vif is already gone? not really sure how else to check for the not found case | 15:16 |
| mriedem | and i don't know of a better way to handle the asynchronous instance delete + neutron vif-deleted event race | 15:16 |
| danpb | mriedem: i'm not sure if you have access to the 'dev' attribute in the caller context you have | 15:17 |
| mriedem | we have the vif | 15:17 |
| mriedem | we have the guest | 15:17 |
| danpb | yeah but interfaces.append(target.get('dev')) | 15:18 |
| sdague | mtreinish: what are you trying to fix here? | 15:18 |
| danpb | is recording the name of the dynamically named tap device | 15:18 |
| mtreinish | sdague: the grenade multinode job | 15:18 |
| danpb | i'm not sure we have that available in nova in all cases | 15:18 |
| danpb | probably need to match on mac address instead | 15:18 |
| mtreinish | sdague: it's fail rate is so high because it 100% fails on bluebox and internap | 15:18 |
| mtreinish | sdague: look at the infra backscroll from last night | 15:19 |
| danpb | mriedem: probably better to call guest.get_all_devices() | 15:19 |
| mriedem | danpb: we seem to have the device right here in the config http://logs.openstack.org/67/269867/4/check/gate-tempest-dsvm-neutron-src-os-brick/7617a9f/logs/screen-n-cpu.txt.gz#_2016-01-21_05_38_48_664 | 15:19 |
| sdague | mtreinish: I don't go back and read overnight backscroll. | 15:19 |
| mriedem | <target dev="tapac3ca8e7-c2"/>\n</interface> | 15:19 |
| danpb | mriedem: for /that/ kind of VIF type | 15:19 |
| sdague | it would be good if someone sent an email about discoveries like that | 15:19 |
| danpb | mriedem: you're not going to have that available for all vif types i think | 15:20 |
| *** achanda has joined #openstack-nova | 15:20 | |
| danpb | eg a assigned PCI device won't have a tap device at all | 15:20 |
| danpb | but i think it should still have a MAC addr available | 15:20 |
| *** achanda has quit IRC | 15:20 | |
| *** ccarmack has joined #openstack-nova | 15:20 | |
| *** achanda has joined #openstack-nova | 15:20 | |
| *** jaypipes has joined #openstack-nova | 15:21 | |
| *** markmc` is now known as markmc | 15:22 | |
| mtreinish | sdague: well, it'd be nice if there were other people looking at these things too. There are only so many hours in the day | 15:22 |
| *** burgerk has joined #openstack-nova | 15:22 | |
| *** tonytan4ever has joined #openstack-nova | 15:22 | |
| *** fawadkhaliq has joined #openstack-nova | 15:25 | |
| *** lpetrut has joined #openstack-nova | 15:25 | |
| *** mjura has quit IRC | 15:26 | |
| *** zz_dimtruck is now known as dimtruck | 15:26 | |
| *** Daisy has joined #openstack-nova | 15:27 | |
| *** mragupat has joined #openstack-nova | 15:28 | |
| *** migi has quit IRC | 15:29 | |
| *** mdbooth has quit IRC | 15:29 | |
| *** ansiwen|afk has quit IRC | 15:30 | |
| *** rlrossit_ has quit IRC | 15:31 | |
| *** burgerk has quit IRC | 15:32 | |
| *** achanda has quit IRC | 15:33 | |
| *** efoley_ has joined #openstack-nova | 15:34 | |
| *** efoley has quit IRC | 15:34 | |
| *** mdrabe has quit IRC | 15:35 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: no longer convert image meta from dict to object https://review.openstack.org/270867 | 15:36 |
| *** markvoelker has quit IRC | 15:38 | |
| cfriesen | danpb: I'm seeing some odd behaviour from libvirt. It's giving me the " Failed to terminate process 40787 with SIGKILL: Device or resource busy" message, but then I see "virProcessGetScheduler:637 : cannot get CPU scheduling policy of process 40790: No such process" these messages (and other similar) keep coming out over at least 6 minutes. Any ideas? | 15:38 |
| markus_z | markmc: do you have a minute? | 15:39 |
| *** annegent_ has joined #openstack-nova | 15:39 | |
| markus_z | markmc: I try to implement Kevins feedback from this review: https://review.openstack.org/#/c/242078/8/nova/hacking/checks.py | 15:40 |
| markus_z | markmc: But I fail to understand how tox and hacking work together | 15:40 |
| *** andymaier has quit IRC | 15:40 | |
| danpb | cfriesen: usuallyindicates QEMU is stuck in an uninterruptable storage I/Owait in kernel space | 15:41 |
| *** su_zhang has joined #openstack-nova | 15:41 | |
| danpb | of course if the storage is merely slow, it might recover and finally die, at which could the scheduling poliucy message is plausible | 15:41 |
| *** irenab has quit IRC | 15:41 | |
| cfriesen | danpb: yeah, I saw that in my digging. but if it's stuck then why the second message about no such process? And I see them alternating for minutes. | 15:42 |
| *** annegentle has quit IRC | 15:42 | |
| danpb | those are different pids in the two mmessages there | 15:42 |
| *** fawadkhaliq has quit IRC | 15:42 | |
| *** rlrossit has joined #openstack-nova | 15:43 | |
| cfriesen | danpb: doh. right. :) | 15:43 |
| *** tdurakov_ has quit IRC | 15:43 | |
| *** amotoki has quit IRC | 15:44 | |
| *** mragupat has quit IRC | 15:44 | |
| cfriesen | danpb: so why would libvirt keep trying to get the scheduler policy of a nonexistant process? I see dozens of each message. | 15:44 |
| *** ndipanov has quit IRC | 15:44 | |
| *** Daisy has quit IRC | 15:45 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove docs about format extensions https://review.openstack.org/270873 | 15:45 |
| *** mragupat has joined #openstack-nova | 15:45 | |
| *** EinstCrazy has joined #openstack-nova | 15:45 | |
| danpb | cfriesen: something in nova requesting this info i guess | 15:46 |
| *** ljxiash has quit IRC | 15:47 | |
| cfriesen | danpb: actually, new data point. both of those pids are coming from the same qemu instance | 15:47 |
| *** ljxiash has joined #openstack-nova | 15:47 | |
| cfriesen | 40787 is the main qemu pid, and 40790 is a vcpu pid | 15:47 |
| danpb | cfriesen: sounds like the kernel has reaped the vcpu pids | 15:48 |
| *** slDabbler has joined #openstack-nova | 15:48 | |
| danpb | but the main pid still exists and stuck in unkillable state | 15:48 |
| danpb | so from libvirt POV the VM is still alive (but a branedead zombie) | 15:49 |
| danpb | and thus also from nova's POV it is still alive, so probably conitinuing to query it | 15:49 |
| flip214 | I'd like to ask for reviews of https://review.openstack.org/#/c/256292. It's already got four +1. | 15:49 |
| cfriesen | danpb: right, then when we ask about cpu info libvirt happily tries to query it, but the vcpu threads don't exist anymore | 15:49 |
| danpb | so best thing is to investigate why it is stuck - ie what part ofo storage stack has failed | 15:51 |
| *** tjones has joined #openstack-nova | 15:51 | |
| danpb | a kernel space stack trace might give you a clue | 15:51 |
| danpb | (eg echo t > /proc/sysrq-trigger) | 15:51 |
| *** ljxiash has quit IRC | 15:52 | |
| *** hblixt has quit IRC | 15:53 | |
| mriedem | has anyone here ever thought it was a pain in the ass to parse http://logs.openstack.org/67/269867/4/check/gate-tempest-dsvm-neutron-src-os-brick/7617a9f/logs/screen-n-cpu.txt.gz#_2016-01-21_05_38_48_524 with your eyes? | 15:54 |
| mriedem | i found https://jsonformatter.curiousconcept.com/ | 15:54 |
| cfriesen | danpb: unfortunately I'm looking at logs right now. a bunch of retries was not sufficient to reproduce the problem. | 15:54 |
| kashyap | danpb: Shouldn't it be echo c > /proc/sysrq-trigger, which forces a crash? | 15:54 |
| * kashyap reads sysrq-trigger doc meanwhile | 15:54 | |
| danpb | kashyap: no | 15:55 |
| danpb | that crashes your whole system | 15:55 |
| andrearosa | kashyap: let me test it ...oh shit | 15:55 |
| andrearosa | :) | 15:55 |
| cfriesen | :) | 15:55 |
| danpb | 't' prints kernel stack traces to dmesg | 15:55 |
| *** mdrabe has joined #openstack-nova | 15:55 | |
| kashyap | 't' - Will dump a list of current tasks and their information to your console. | 15:55 |
| kashyap | Yeah, sorry, ignore me like the plague | 15:55 |
| cfriesen | Also, /proc/<pid>/stack is useful | 15:55 |
| *** achanda has joined #openstack-nova | 15:55 | |
| danpb | cfriesen: oh nice, that's even better | 15:56 |
| * danpb wonders when they added that file | 15:56 | |
| cfriesen | Somone pointed it out to me and I was like "crap, why wasn't this more publicized?" | 15:57 |
| kashyap | andrearosa: I mentioned 'c' because, as it'd force a host crash, and thus giving an insight into Kernel storage state. But, in this case 't' seems applicable | 15:57 |
| *** angdraug has joined #openstack-nova | 15:57 | |
| kashyap | cfriesen: That's nice, I didn't know of it too | 15:58 |
| *** ndipanov has joined #openstack-nova | 15:59 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** markvoelker has joined #openstack-nova | 16:01 | |
| *** haomaiwa_ has joined #openstack-nova | 16:01 | |
| *** achanda has quit IRC | 16:02 | |
| *** ducttape_ has joined #openstack-nova | 16:02 | |
| andrearosa | kashyap: thanks | 16:03 |
| *** scheuran has quit IRC | 16:04 | |
| *** liusheng has quit IRC | 16:04 | |
| kashyap | mriedem: That JSON data formatter looks useful, thanks! | 16:05 |
| *** nkrinner has quit IRC | 16:05 | |
| pkholkin | johnthetubaguy, bauzas: sorry, wasn't near the computer. yes, I think you are right | 16:06 |
| mriedem | kashyap: yeah, it's nice - now i don't have to try and follow the braces with my eyes | 16:06 |
| pkholkin | johnthetubaguy: thanks for updating commit message | 16:06 |
| pkholkin | and for reviews | 16:06 |
| * pkholkin looking | 16:07 | |
| *** dhellmann has joined #openstack-nova | 16:07 | |
| *** tdurakov_ has joined #openstack-nova | 16:07 | |
| *** SridharG has quit IRC | 16:07 | |
| *** slDabbler has quit IRC | 16:08 | |
| *** rcernin has quit IRC | 16:09 | |
| johnthetubaguy | pkholkin: no worries, thanks | 16:10 |
| *** yamahata has joined #openstack-nova | 16:10 | |
| *** amotoki has joined #openstack-nova | 16:13 | |
| *** lyanchih has quit IRC | 16:13 | |
| *** slagle has quit IRC | 16:14 | |
| gibi | rlrossit, cdent: As I promised I created a pad about versioned notification TODOs https://etherpad.openstack.org/p/nova-versioned-notifications Any comments are welcome. | 16:15 |
| cdent | thanks gibi will have a look in my copious free time ;) | 16:16 |
| pkholkin | johnthetubaguy: John, have one question related to this patch https://review.openstack.org/#/c/240253/ | 16:17 |
| *** fawadkhaliq has joined #openstack-nova | 16:17 | |
| gibi | cdent: no worries we can go through it next week on the midcycle | 16:17 |
| cdent | It looks like an excellent list. It's really nice having things written out in such detail | 16:17 |
| *** pratikmallya has joined #openstack-nova | 16:17 | |
| *** busterswt has quit IRC | 16:18 | |
| *** busterswt has joined #openstack-nova | 16:18 | |
| *** bnemec has quit IRC | 16:18 | |
| pkholkin | johnthetubaguy: Sean asked about @require_context. This decorator wrap not every method and it maybe confusing for developers of new methods. And the code if this decorator seems strange to me when I looked at it some time ago | 16:19 |
| mriedem | danpb: needs tests but how does this look? https://review.openstack.org/#/c/270891/ | 16:19 |
| dims | mriedem : my fav is jsonlint http://jsonlint.com/ | 16:19 |
| pkholkin | johnthetubaguy: maybe we should remove it after? I think it doesn't relate to switching to enginefacade, the question is more general) | 16:20 |
| mriedem | dims: not bad | 16:20 |
| *** bnemec has joined #openstack-nova | 16:20 | |
| danpb | mriedem: seems reasonable to me | 16:20 |
| rvasilets | Guys? | 16:20 |
| johnthetubaguy | pkholkin: yeah, sdague has a good point, that just seems to get in the way, everything needs context anyways. seems like a good follow up patch | 16:21 |
| jaypipes | rvasilets: yes? | 16:21 |
| mriedem | danpb: ok, cool, i'll add some tests | 16:21 |
| *** busterswt has quit IRC | 16:21 | |
| rvasilets | I took minimal configuration of devstack from here https://github.com/openstack-dev/devstack/blob/master/doc/source/configuration.rst And tried to boot server. It was booted but there is no internet connection on the instance. Does some one faced with such an issue? I have no idea how fix this. The image was taken from here http://cloud-images.ubuntu.com/trusty/current/ Logs from the instance http://paste.openstack.org/show/484574/ | 16:21 |
| rvasilets | jaypipes, ^ | 16:22 |
| *** pratikma_ has joined #openstack-nova | 16:22 | |
| jaypipes | rvasilets: please see /topic :) That question is best asked on #openstack channel or (better) on the openstack@ mailing list. | 16:22 |
| pkholkin | johnthetubaguy: do you think we should add it in all required methods? (and not remove) | 16:22 |
| *** rmart04 has quit IRC | 16:23 | |
| *** krtaylor has quit IRC | 16:23 | |
| johnthetubaguy | pkholkin: its seems really pointless to keep it | 16:24 |
| *** zenoway has quit IRC | 16:24 | |
| *** raildo-afk is now known as raildo | 16:24 | |
| *** pratikmallya has quit IRC | 16:25 | |
| pkholkin | johnthetubaguy: without context nothing will work for sure) | 16:25 |
| johnthetubaguy | pkholkin: yeah, exactly | 16:25 |
| pkholkin | because we use the session from it | 16:25 |
| *** ahelkhou has joined #openstack-nova | 16:25 | |
| *** fawadkhaliq has quit IRC | 16:25 | |
| pkholkin | that's why maybe it would be better to remove it | 16:25 |
| pkholkin | after our changes | 16:25 |
| *** asdasd has joined #openstack-nova | 16:25 | |
| *** asdasd has quit IRC | 16:26 | |
| *** aix has quit IRC | 16:26 | |
| pkholkin | johnthetubaguy: can you write some comments there to Sean? | 16:29 |
| pkholkin | I can too :) | 16:29 |
| *** browne has joined #openstack-nova | 16:29 | |
| rlrossit | gibi: the pad also looks good to me | 16:30 |
| *** matrohon has joined #openstack-nova | 16:31 | |
| hemna | ildikov, mornin | 16:32 |
| *** otter768 has joined #openstack-nova | 16:32 | |
| ildikov | hemna: mornin :) | 16:32 |
| *** efoley_ is now known as efoley | 16:32 | |
| *** kodokuu has quit IRC | 16:33 | |
| *** [1]Thelo has joined #openstack-nova | 16:33 | |
| *** busterswt has joined #openstack-nova | 16:33 | |
| hemna | so, what's up | 16:33 |
| ildikov | hemna: a few things are on the gate thanks to johnthetubaguy, mriedem, ndipanov and many others around | 16:33 |
| *** sabari has quit IRC | 16:34 | |
| hemna | :) | 16:34 |
| ildikov | hemna: we need to prep to the mid-cycle to get everything in for Mitaka | 16:34 |
| hemna | anything I can do to help ? | 16:34 |
| ildikov | hemna: ndipanov pointed out that we have issues with disconnect_volume that I totally missed | 16:35 |
| *** Thelo has quit IRC | 16:35 | |
| *** [1]Thelo is now known as Thelo | 16:35 | |
| hemna | ? | 16:35 |
| *** krtaylor has joined #openstack-nova | 16:35 | |
| *** sabari has joined #openstack-nova | 16:35 | |
| ildikov | hemna: I think it's the same two attachments on the same host issue | 16:35 |
| *** mpavone has quit IRC | 16:36 | |
| hemna | ok | 16:36 |
| jaypipes | sdague: lol "by trending up we mean 75%" | 16:36 |
| * jaypipes goes to review ildikov's patch. | 16:36 | |
| hemna | so, basically not calling disconnect_volume if there is only 1 volume for the 2 instances ? | 16:36 |
| *** otter768 has quit IRC | 16:37 | |
| pkholkin | johnthetubaguy: posted | 16:37 |
| ildikov | jaypipes: there're two patches in the chain we will need in for Mitaka | 16:37 |
| *** shyama has quit IRC | 16:37 | |
| jaypipes | ildikov: right. | 16:37 |
| ildikov | jaypipes: I already know I missed to fix the mocks for the whole chain | 16:37 |
| jaypipes | ildikov: so should I wait for a bit? | 16:37 |
| ildikov | jaypipes: but if you could check those two from feature perspective, then I would fix all the comments in one rebase | 16:38 |
| jaypipes | ildikov: k, gimme fix. | 16:38 |
| jaypipes | gimme five :) | 16:38 |
| jaypipes | sorry | 16:38 |
| pkholkin | jaypipes: hello Jay! I restored this dead patch :) https://review.openstack.org/#/c/225326/3 | 16:38 |
| jaypipes | pkholkin: oh, nice. I will put that in my queue for the day. | 16:39 |
| ildikov | jaypipes: :) | 16:39 |
| pkholkin | now I'm waiting for comments novaclient guys) | 16:39 |
| pkholkin | jaypipes: no problem) just a notification :) | 16:39 |
| ildikov | hemna: full comment is here from ndipanov: https://review.openstack.org/#/c/256273/71/nova/virt/block_device.py | 16:39 |
| ndipanov | hemna, ping me if it's unclear - wrote it after only one coffee this morning :) | 16:40 |
| hemna | checking.... | 16:40 |
| hemna | I'm still ingesting my coffee | 16:40 |
| hemna | :P | 16:40 |
| ildikov | ndipanov: I know the feeling I almost needed a coffee this morning to be able to drink my morning coffee... | 16:40 |
| jaypipes | lol | 16:41 |
| hemna | ildikov, :) | 16:41 |
| ndipanov | haha | 16:42 |
| *** tdurakov_ has quit IRC | 16:42 | |
| ildikov | can I throw a grenate to grenade? | 16:42 |
| ildikov | I mean the gate job | 16:42 |
| openstackgerrit | Béla Vancsics proposed openstack/nova: Reduce code duplication https://review.openstack.org/270911 | 16:43 |
| *** tdurakov_ has joined #openstack-nova | 16:43 | |
| ildikov | jaypipes: I'm not sure you have the link to chain, I meant these two patches: https://review.openstack.org/#/c/267587/ , https://review.openstack.org/#/c/193135 | 16:44 |
| *** fawadkhaliq has joined #openstack-nova | 16:44 | |
| *** tdurakov_ has quit IRC | 16:45 | |
| ildikov | jaypipes: I need to move the reno to the last WIP patch too, so I'm already aware of some of my sins :) | 16:45 |
| *** tjones has quit IRC | 16:45 | |
| jaypipes | ildikov: k, thx | 16:46 |
| *** burgerk has joined #openstack-nova | 16:46 | |
| *** mgoddard_ has joined #openstack-nova | 16:46 | |
| *** whenry has quit IRC | 16:46 | |
| ildikov | jaypipes: me thanks | 16:46 |
| *** bdrich has quit IRC | 16:47 | |
| *** mgoddard has quit IRC | 16:49 | |
| *** rcernin has joined #openstack-nova | 16:49 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: no longer convert image meta from dict to object https://review.openstack.org/270867 | 16:50 |
| *** bnemec has quit IRC | 16:52 | |
| *** Marga_ has joined #openstack-nova | 16:52 | |
| *** Daisy has joined #openstack-nova | 16:53 | |
| *** zakora has quit IRC | 16:54 | |
| mriedem | sahid: i'm adding an interface entry to the xml here https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/libvirt/test_guest.py#L316 | 16:54 |
| mriedem | sahid: but when it parses it's not getting the interfaces | 16:54 |
| mriedem | i saw https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/libvirt/test_guest.py#L376 | 16:54 |
| *** fawadkhaliq has quit IRC | 16:54 | |
| mriedem | sahid: do you know what's actually filtering out everything but disks and hostdev? ^ | 16:54 |
| *** fawadkhaliq has joined #openstack-nova | 16:54 | |
| mriedem | oh i see | 16:55 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/config.py#L1995-L2004 | 16:56 |
| danpb | mriedem: opps, yes, i promised ponies which don't exist yet | 16:57 |
| openstackgerrit | Rushi Agrawal proposed openstack/nova: Remove last instance of mox from unit/objects/test_instance.py https://review.openstack.org/270921 | 16:57 |
| mriedem | danpb: easy update to start parsing interface | 16:57 |
| *** paul-carlton2 has joined #openstack-nova | 16:57 | |
| danpb | yeah, just wire up a parse_dom method on the Interface class | 16:57 |
| *** Daisy has quit IRC | 16:58 | |
| danpb | ultimately we should have parse_dom + format_dom for all classes, but we're lazy and only add them on an as-needed basis | 16:58 |
| mriedem | works for me | 16:58 |
| mriedem | i was just confused by that test not working | 16:58 |
| mriedem | good thing there was a comment | 16:58 |
| *** davidlenwell has quit IRC | 16:58 | |
| *** PaulMurray has quit IRC | 16:58 | |
| *** mfedosin has quit IRC | 16:59 | |
| *** baoli has quit IRC | 16:59 | |
| *** davidlenwell has joined #openstack-nova | 16:59 | |
| *** rlrossit has quit IRC | 17:00 | |
| *** haomaiwa_ has quit IRC | 17:01 | |
| mriedem | just wish there weren't so many gd fields on that config object | 17:01 |
| *** mgoddard_ has quit IRC | 17:01 | |
| *** haomaiwa_ has joined #openstack-nova | 17:01 | |
| *** mgoddard has joined #openstack-nova | 17:01 | |
| *** bnemec has joined #openstack-nova | 17:02 | |
| *** baoli has joined #openstack-nova | 17:05 | |
| *** knikolla has joined #openstack-nova | 17:05 | |
| *** jdurgin1 has joined #openstack-nova | 17:06 | |
| *** burt has joined #openstack-nova | 17:06 | |
| *** edtubill has joined #openstack-nova | 17:06 | |
| *** Marga_ has quit IRC | 17:07 | |
| *** coolsvap|away has quit IRC | 17:07 | |
| *** apoorvad has joined #openstack-nova | 17:07 | |
| *** Marga_ has joined #openstack-nova | 17:08 | |
| cdent | bauzas: I reckon I've addressed your comments on https://review.openstack.org/#/c/262561/ could you give it another pass (at your leisure)? Thanks | 17:08 |
| *** salv-orlando has joined #openstack-nova | 17:10 | |
| ildikov | jaypipes: I actually found more things to move in the first patch, I added comments and I'll also updte the commit message | 17:10 |
| ildikov | jaypipes: sorry for the mess, we're running in circles with this feature sometimes, so I cannot follow even myself... | 17:11 |
| ildikov | jaypipes: you know, the coffee story from earlier... :) | 17:11 |
| *** Kevin_Zheng has quit IRC | 17:12 | |
| *** eharney has quit IRC | 17:12 | |
| *** rlrossit has joined #openstack-nova | 17:12 | |
| jaypipes | ildikov: ok, no worries. just ping me when I should proceed. | 17:13 |
| *** ccarmack has quit IRC | 17:14 | |
| jaypipes | I will continue to look at the patches. | 17:14 |
| ildikov | jaypipes: cool, thanks | 17:14 |
| *** ccarmack has joined #openstack-nova | 17:14 | |
| *** unicell has quit IRC | 17:21 | |
| *** jogo has quit IRC | 17:22 | |
| *** mragupat_ has joined #openstack-nova | 17:23 | |
| *** jistr has quit IRC | 17:24 | |
| *** jogo has joined #openstack-nova | 17:25 | |
| *** mragupat has quit IRC | 17:26 | |
| *** markstur has quit IRC | 17:26 | |
| *** ansiwen has joined #openstack-nova | 17:28 | |
| *** armax has joined #openstack-nova | 17:28 | |
| *** gjayavelu has joined #openstack-nova | 17:29 | |
| *** sahid has quit IRC | 17:29 | |
| *** _ducttape_ has joined #openstack-nova | 17:30 | |
| *** ducttape_ has quit IRC | 17:32 | |
| *** rotbeard has quit IRC | 17:33 | |
| *** thinrichs has joined #openstack-nova | 17:33 | |
| *** tdurakov_ has joined #openstack-nova | 17:34 | |
| *** nic has joined #openstack-nova | 17:35 | |
| mriedem | danpb: the libvirt interface config has a vlan flag which looks like it's used as a vlan tag id, but according to the libvirt domain xml docs, the vlan element can have multiple tag sub-elements with their own tags | 17:35 |
| mriedem | i also don't see the vlan tag id set anywhere in the driver... | 17:35 |
| mriedem | so maybe it's wrong but just not used? | 17:36 |
| *** rdopiera has quit IRC | 17:36 | |
| *** paul-carlton2 has quit IRC | 17:37 | |
| *** markus_z has quit IRC | 17:37 | |
| *** e0ne has quit IRC | 17:37 | |
| *** _ducttape_ has quit IRC | 17:37 | |
| *** hemna has quit IRC | 17:38 | |
| *** ducttape_ has joined #openstack-nova | 17:38 | |
| *** whenry has joined #openstack-nova | 17:41 | |
| johnthetubaguy | mriedem: its not the old VLAN manager thing for nova-network I guess? | 17:41 |
| mriedem | no idea | 17:41 |
| mriedem | shouldn't matter | 17:41 |
| *** tdurakov_ has quit IRC | 17:41 | |
| mriedem | this is just parsing the libvirt domain xml - and formatting it | 17:41 |
| mriedem | but according to the docs, the vlan element has multiple tag sub-elements, but we're only tracking one of them | 17:41 |
| mriedem | i don't actually see the vlan thing set anywhere in the libvirt driver though | 17:41 |
| mriedem | so i'm guessing it's just unused | 17:41 |
| *** tdurakov_ has joined #openstack-nova | 17:42 | |
| * mriedem is not having tons of fun parsing the libvirt domain xml interface tag | 17:42 | |
| *** sfinucan has quit IRC | 17:42 | |
| *** ducttape_ has quit IRC | 17:42 | |
| johnthetubaguy | mriedem: oh, gotcha | 17:44 |
| *** tdurakov_ has quit IRC | 17:44 | |
| *** gjayavelu has quit IRC | 17:45 | |
| *** tdurakov_ has joined #openstack-nova | 17:45 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/266160 | 17:45 |
| *** femnad has quit IRC | 17:46 | |
| *** matrohon has quit IRC | 17:46 | |
| *** ccarmack has quit IRC | 17:47 | |
| *** tdurakov_ has quit IRC | 17:48 | |
| *** paul-carlton2 has joined #openstack-nova | 17:49 | |
| *** tdurakov_ has joined #openstack-nova | 17:49 | |
| *** davidlenwell has quit IRC | 17:50 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: Introduce VNC Security Proxy Framework https://review.openstack.org/115483 | 17:50 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: Add VeNCrypt (TLS/x509) Security Proxy Driver https://review.openstack.org/115484 | 17:50 |
| *** su_zhang has quit IRC | 17:50 | |
| *** yassine has quit IRC | 17:51 | |
| danpb | mriedem: i can have many but nova doesn't need that, so ust assuming 1 is fine | 17:51 |
| mriedem | ok, i'm just taking the first and then breaking the loop (if there were multiple) | 17:52 |
| johnthetubaguy | danpb: mriedem: If you have a sec, I think we need this to unblock the specs gate, as I understand it: https://review.openstack.org/#/c/270606/2 | 17:52 |
| *** ducttape_ has joined #openstack-nova | 17:52 | |
| *** unicell has joined #openstack-nova | 17:53 | |
| *** danpb has quit IRC | 17:53 | |
| *** gszasz has quit IRC | 17:54 | |
| *** sahid has joined #openstack-nova | 17:55 | |
| *** ndipanov has quit IRC | 17:55 | |
| *** davidlenwell has joined #openstack-nova | 17:55 | |
| *** fesp has joined #openstack-nova | 17:56 | |
| *** ndipanov has joined #openstack-nova | 17:56 | |
| *** garyk has joined #openstack-nova | 17:56 | |
| *** ducttape_ has quit IRC | 17:56 | |
| garyk | are there any cores around? the vmware driver is broke. can someone please look at https://review.openstack.org/#/c/270867/ | 17:56 |
| *** ducttape_ has joined #openstack-nova | 17:57 | |
| mriedem | yup http://208.91.1.172/logs/nova-vc6-dsvm-tempest/189685/38/1350/n-cpu.log.gz#_2016-01-18_20_21_30_611 | 17:58 |
| mriedem | vmware nsx ci had caught it too | 17:58 |
| dims | garyk : danpb sent a heads up a couple of days ago http://markmail.org/message/7x6cw6h5ay4hlh3k | 17:59 |
| mriedem | dims: the vmware driver is in tree :) | 17:59 |
| dims | mriedem : right :) | 17:59 |
| mriedem | garyk: +2 | 17:59 |
| garyk | mriedem: gracias! | 17:59 |
| mriedem | de nada | 18:00 |
| *** ihrachys has quit IRC | 18:00 | |
| *** jlanoux has quit IRC | 18:00 | |
| *** haomaiwa_ has quit IRC | 18:01 | |
| *** aix has joined #openstack-nova | 18:01 | |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| *** Marga_ has quit IRC | 18:03 | |
| *** derekh has quit IRC | 18:03 | |
| *** fawadk has joined #openstack-nova | 18:04 | |
| *** fawadkhaliq has quit IRC | 18:04 | |
| *** kfarr has joined #openstack-nova | 18:05 | |
| *** efoley has quit IRC | 18:06 | |
| *** mragupat_ has quit IRC | 18:09 | |
| *** mragupat has joined #openstack-nova | 18:09 | |
| *** jaosorior has quit IRC | 18:09 | |
| *** jaosorior has joined #openstack-nova | 18:10 | |
| lifeless | mordred: hai, pbr stuff? | 18:10 |
| mordred | lifeless: o hai | 18:11 |
| mordred | lifeless: tl;dr nova patch to add setuptools command to compile all not just some of the translations - sdague suggested adding it to pbr instead | 18:11 |
| *** rmart04 has joined #openstack-nova | 18:11 | |
| sdague | mordred: right, because it turns out the current compile_catalog only gets the nova.po files, not the translation domain files | 18:12 |
| mordred | yup | 18:12 |
| lifeless | if its being done at build time, then yes, it should be pbr | 18:12 |
| *** ndipanov has quit IRC | 18:12 | |
| *** slagle has joined #openstack-nova | 18:13 | |
| *** mragupat has quit IRC | 18:13 | |
| ansiwen | mordred: sorry, was offline 'til now... can we do that tomorrow? | 18:15 |
| *** lpetrut has quit IRC | 18:15 | |
| ansiwen | mordred: which timezone you are? | 18:15 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the linuxnet options (1) https://review.openstack.org/270959 | 18:15 |
| mordred | ansiwen: totally - I'm US East - altohugh lifeless is New Zealand if that works better for you | 18:17 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 18:17 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow volume multiattach https://review.openstack.org/267587 | 18:17 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 18:17 |
| *** ccarmack has joined #openstack-nova | 18:18 | |
| sdague | mriedem: to be fair, vmware nsx ci showed up 3 days late | 18:19 |
| mriedem | oh, heh | 18:19 |
| mriedem | yeah that's not helpful then | 18:19 |
| ansiwen | mordred: ok, I'm UTC+1 | 18:19 |
| sdague | 3.5 days late | 18:19 |
| sdague | and after it had been approved, which was 3 days after danpb pushed the patch | 18:19 |
| *** rmart04 has left #openstack-nova | 18:20 | |
| mriedem | yeah | 18:20 |
| mriedem | i see that now | 18:20 |
| *** ccarmack has quit IRC | 18:22 | |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: add comments about the pre/post extension processing https://review.openstack.org/270792 | 18:24 |
| openstackgerrit | Sean Dague proposed openstack/nova: cleanup: remove custom serializer support https://review.openstack.org/270763 | 18:24 |
| sdague | mriedem: if you like deleting code ^^^ | 18:25 |
| *** ctrath has quit IRC | 18:27 | |
| ildikov | jaypipes: I uploaded the next version | 18:27 |
| *** ccarmack has joined #openstack-nova | 18:27 | |
| jaypipes | ildikov: cool. will examine them now. | 18:27 |
| ildikov | jaypipes: the first patch became pretty small, I will need to add a test there I think | 18:27 |
| jaypipes | k | 18:28 |
| ildikov | jaypipes: the patches are organized conceptually as compute changes and virt driver + libvirt changes | 18:28 |
| jaypipes | got it. | 18:28 |
| *** knikolla has quit IRC | 18:29 | |
| jaypipes | ildikov: you want me to start with 267587 right? | 18:30 |
| *** fesp has quit IRC | 18:32 | |
| *** otter768 has joined #openstack-nova | 18:33 | |
| *** daemontool has quit IRC | 18:35 | |
| *** ccarmack has quit IRC | 18:36 | |
| *** zenoway has joined #openstack-nova | 18:38 | |
| *** otter768 has quit IRC | 18:38 | |
| *** penick has joined #openstack-nova | 18:39 | |
| *** ccarmack has joined #openstack-nova | 18:40 | |
| *** edmondsw has quit IRC | 18:41 | |
| ildikov | jaypipes: sorry, yes, that's right | 18:41 |
| jaypipes | k | 18:41 |
| ildikov | jaypipes: the two you checked before are on the gate by now, this one is the next in the chain | 18:42 |
| *** zenoway has quit IRC | 18:42 | |
| *** fawadk has quit IRC | 18:43 | |
| *** su_zhang has joined #openstack-nova | 18:44 | |
| *** Marga_ has joined #openstack-nova | 18:44 | |
| *** eglynn_ has quit IRC | 18:44 | |
| *** ctrath has joined #openstack-nova | 18:46 | |
| garyk | johnthetubaguy: mriedem: this has the +1 from the vmware ci - https://review.openstack.org/#/c/270867/ | 18:48 |
| vladikr | jaypipes, could you please also look at the libosinfo stuff? | 18:48 |
| vladikr | https://review.openstack.org/#/c/149625 | 18:48 |
| *** mriedem has quit IRC | 18:48 | |
| *** nic has quit IRC | 18:48 | |
| *** AJaeger has left #openstack-nova | 18:48 | |
| *** tonytan4ever has quit IRC | 18:49 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: remove mox from vmwareapi/test_driver_api.py https://review.openstack.org/270303 | 18:50 |
| *** mriedem has joined #openstack-nova | 18:51 | |
| *** lpetrut has joined #openstack-nova | 18:52 | |
| *** ssurana has joined #openstack-nova | 18:52 | |
| *** vilobhmm11 has joined #openstack-nova | 18:54 | |
| vilobhmm11 | mriedem : ping | 18:55 |
| *** hemna has joined #openstack-nova | 18:55 | |
| *** e0ne has joined #openstack-nova | 18:57 | |
| *** ansiwen is now known as ansiwen|afk | 18:57 | |
| *** angdraug has quit IRC | 18:57 | |
| *** ansiwen|afk is now known as ansiwen | 18:57 | |
| *** zenoway has joined #openstack-nova | 18:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: check for interface when detach_interface fails https://review.openstack.org/270891 | 18:59 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: implement LibvirtConfigGuestInterface.parse_dom https://review.openstack.org/270981 | 18:59 |
| vilobhmm11 | mriedem : wanted to understand what features come under non-priority for whom deadline is jan 22 http://docs.openstack.org/releases/schedules/mitaka.html#m-nova-npff | 19:00 |
| *** tdurakov_ has quit IRC | 19:00 | |
| *** haomaiwang has quit IRC | 19:01 | |
| mriedem | vilobhmm11: anything not in this list https://specs.openstack.org/openstack/nova-specs/priorities/mitaka-priorities.html | 19:01 |
| *** rlrossit has quit IRC | 19:01 | |
| *** haomaiwang has joined #openstack-nova | 19:01 | |
| mriedem | vilobhmm11: which is exactly what the doc that you linked me to says | 19:01 |
| *** tdurakov_ has joined #openstack-nova | 19:02 | |
| *** ctrath has quit IRC | 19:02 | |
| *** ducttape_ has quit IRC | 19:02 | |
| vilobhmm11 | mriedem : yes just that it doesn't list any spec that fall under that category wanted to check if there is a list of specs | 19:02 |
| *** lpetrut has quit IRC | 19:02 | |
| vilobhmm11 | but i think whatever specs try to address the problems mentioned will be priority ones | 19:03 |
| *** mragupat has joined #openstack-nova | 19:03 | |
| vilobhmm11 | mriedem : thanks! | 19:03 |
| mriedem | vilobhmm11: https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking | 19:04 |
| *** zenoway has quit IRC | 19:04 | |
| *** zenoway has joined #openstack-nova | 19:04 | |
| *** rlrossit_ has joined #openstack-nova | 19:05 | |
| *** chhavi has quit IRC | 19:05 | |
| *** ctrath has joined #openstack-nova | 19:05 | |
| *** ctrath1 has joined #openstack-nova | 19:06 | |
| *** ijw has joined #openstack-nova | 19:07 | |
| *** Sukhdev has joined #openstack-nova | 19:07 | |
| *** zenoway has quit IRC | 19:08 | |
| openstackgerrit | melanie witt proposed openstack/nova: Add a cell and host mapping utility to nova-manage https://review.openstack.org/270565 | 19:09 |
| *** ctrath has quit IRC | 19:10 | |
| *** Marga_ has quit IRC | 19:11 | |
| *** eharney has joined #openstack-nova | 19:11 | |
| *** yamahata has quit IRC | 19:12 | |
| *** fesp has joined #openstack-nova | 19:13 | |
| vilobhmm11 | mriedem : thanks! | 19:14 |
| *** sahid has quit IRC | 19:15 | |
| *** Marga_ has joined #openstack-nova | 19:16 | |
| *** ociuhandu has quit IRC | 19:16 | |
| *** ijw_ has joined #openstack-nova | 19:17 | |
| *** zenoway has joined #openstack-nova | 19:19 | |
| *** aix has quit IRC | 19:19 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "wsgi" https://review.openstack.org/252687 | 19:19 |
| *** ijw has quit IRC | 19:20 | |
| *** ducttape_ has joined #openstack-nova | 19:22 | |
| *** cdent has quit IRC | 19:22 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 19:23 |
| *** tjones has joined #openstack-nova | 19:23 | |
| *** jdurgin1 has quit IRC | 19:24 | |
| *** chhavi has joined #openstack-nova | 19:24 | |
| *** tonytan4ever has joined #openstack-nova | 19:26 | |
| *** pradk has quit IRC | 19:29 | |
| *** edmondsw has joined #openstack-nova | 19:29 | |
| *** markstur has joined #openstack-nova | 19:32 | |
| ildikov | mriedem: hi. are you around? | 19:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Update import_file to provide a general-purpose interface https://review.openstack.org/267662 | 19:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove fake_imagebackend.Raw and cleanup dependent tests https://review.openstack.org/267661 | 19:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove deprecated option libvirt.remove_unused_kernels https://review.openstack.org/265886 | 19:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Only attempt to inject files if the injection disk exists https://review.openstack.org/250872 | 19:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove _create_local function https://review.openstack.org/270997 | 19:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Rename Image.check_image_exists to Image.exists() https://review.openstack.org/270998 | 19:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Trivial logic cleanup in libvirt pre_live_migration https://review.openstack.org/270999 | 19:35 |
| *** zhenq has joined #openstack-nova | 19:35 | |
| mriedem | ildikov: sort of | 19:36 |
| ildikov | mriedem: well, not urgent, I'm just cleaning up the patches for multiattach that should go into Mitaka and I still get confused at one part, which is this: https://review.openstack.org/#/c/267587/16/nova/compute/manager.py | 19:36 |
| ildikov | mriedem: maybe not now, but are you the right person to discuss what that function is exactly for and how it fits into the whole attach path? | 19:37 |
| mriedem | ildikov: you want to know what _await_block_device_map_created is used for? | 19:38 |
| mriedem | that's the wait_func passed to the nova.virt.block_device attach method, | 19:38 |
| mriedem | in the case that source_type is blank/image/snapshot, | 19:39 |
| mriedem | nova creates the volume, | 19:39 |
| mriedem | and then uses _await_block_device_map_created to wait for it to be available | 19:39 |
| *** Sukhdev has quit IRC | 19:40 | |
| mriedem | ildikov: i commented in the change with details | 19:41 |
| mriedem | basically i think you can remove that | 19:41 |
| *** yamahata has joined #openstack-nova | 19:41 | |
| *** tjones has quit IRC | 19:42 | |
| mriedem | which i guess effectively nulls out the changes for that patch, except the tests....which it's weird that we'd just have a test-only patch | 19:42 |
| *** knikolla has joined #openstack-nova | 19:42 | |
| *** daemontool has joined #openstack-nova | 19:42 | |
| *** salv-orlando has quit IRC | 19:42 | |
| ildikov | mriedem: a-ha, ok, I wasn't sure it's used only in case of volume creation | 19:42 |
| mriedem | double check but i'm pretty sure that's the only place that function is used | 19:43 |
| ildikov | mriedem: can that happen that we only need changes in manager in the virt layer? | 19:43 |
| mriedem | idk | 19:43 |
| ildikov | this part of the code is not too easy to figure out | 19:44 |
| mriedem | but see it's passed here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1746 | 19:44 |
| mriedem | yeah, i know - it's taken me years :) | 19:44 |
| ildikov | yeah, patience is a completely missing skill of mine :) | 19:45 |
| mriedem | but look for where _call_wait_func is used here: https://github.com/openstack/nova/blob/master/nova/virt/block_device.py | 19:45 |
| *** rlrossit_ has quit IRC | 19:45 | |
| mriedem | it's only the blank/image/snapshot cases | 19:45 |
| mriedem | where nova creates the volume and then waits for it to be available so it can attach it | 19:45 |
| ildikov | I found the prep_block_device, but the name and the comment does not say when to create that block device | 19:45 |
| *** Sukhdev has joined #openstack-nova | 19:46 | |
| mriedem | _await_block_device_map_created could use documentation yes | 19:46 |
| ildikov | ok, don't listen to me | 19:46 |
| mriedem | i was going to do that after the first time i figured this all out | 19:46 |
| mriedem | b/c it's odd that the nova.virt.block_device relies on this wait function defined in the compute manager to be passed down, you could argue that _await_block_device_map_created should just be in nova.virt.block_device | 19:46 |
| ildikov | so prepare_block_device obviously does not equal to prepare block device mapping | 19:46 |
| mriedem | shrug? | 19:47 |
| mriedem | you'd have to ask ndipanov about that | 19:47 |
| mriedem | i think this is his baby | 19:47 |
| ildikov | ok, will do tomorrow for sure :) | 19:47 |
| mriedem | i just babysit for a few dollars a week | 19:47 |
| mriedem | + whatever is in the fridge | 19:47 |
| ildikov | but what you wrote that's one thing that confuses me | 19:47 |
| ildikov | lol :) | 19:47 |
| ildikov | I mean having that wait function defined in the compute manager, where we don't even use it | 19:48 |
| *** e0ne has quit IRC | 19:49 | |
| ildikov | ok, I will keep the patch as is until tomorrow and I will ask ndipanov about this, but I'm kind of convinced that no changes are needed in the manager and then I will just move the tests to the virt driver patch | 19:49 |
| *** ccarmack has quit IRC | 19:50 | |
| *** nic has joined #openstack-nova | 19:51 | |
| ildikov | mriedem: thanks! I think ndipanov commented on this once, but I didn't get his question (Why is it needed?) at that time, I thought the function can be used for non volume creation cases too :( | 19:52 |
| mriedem | ildikov: nope | 19:53 |
| mriedem | it's just weird | 19:53 |
| mriedem | i think the nova.virt.block_device stuff was written to probably handle more future cases ,but it was written back in havana i think | 19:53 |
| mriedem | and those future cases never came | 19:53 |
| mriedem | so it's probably a bit over engineerd which makes it confusing | 19:53 |
| mriedem | like if you had a different wait function for a different type of block device | 19:53 |
| *** mfedosin has joined #openstack-nova | 19:55 | |
| ildikov | ah, ok got it | 19:55 |
| *** paul-carlton2 has quit IRC | 19:57 | |
| ildikov | I also can understand now, why people are defensive with the prep code too | 19:57 |
| *** ctrath1 has quit IRC | 19:57 | |
| *** ccarmack has joined #openstack-nova | 19:58 | |
| *** ctrath has joined #openstack-nova | 19:58 | |
| *** amotoki has quit IRC | 19:58 | |
| *** amotoki has joined #openstack-nova | 19:58 | |
| openstackgerrit | jichenjc proposed openstack/nova: Check host status before scheduler https://review.openstack.org/216759 | 20:00 |
| *** _ducttape_ has joined #openstack-nova | 20:00 | |
| *** haomaiwang has quit IRC | 20:01 | |
| *** daemontool has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-nova | 20:01 | |
| *** rlrossit has joined #openstack-nova | 20:02 | |
| *** daemontool has joined #openstack-nova | 20:02 | |
| *** ijw has joined #openstack-nova | 20:03 | |
| *** ducttape_ has quit IRC | 20:04 | |
| *** fesp has quit IRC | 20:04 | |
| *** smatzek_ has quit IRC | 20:07 | |
| *** markstur has left #openstack-nova | 20:07 | |
| *** ijw_ has quit IRC | 20:07 | |
| *** tdurakov_ has quit IRC | 20:07 | |
| *** tdurakov_ has joined #openstack-nova | 20:08 | |
| *** aysyd has quit IRC | 20:09 | |
| *** lpetrut has joined #openstack-nova | 20:10 | |
| *** markvoelker has quit IRC | 20:11 | |
| *** rfolco has quit IRC | 20:11 | |
| *** thinrichs has quit IRC | 20:13 | |
| *** _ducttape_ has quit IRC | 20:14 | |
| *** ducttape_ has joined #openstack-nova | 20:14 | |
| *** daemontool_ has joined #openstack-nova | 20:15 | |
| *** mfedosin has quit IRC | 20:15 | |
| dims | mriedem : seeing a DBDeadlock in a gate job - http://logs.openstack.org/76/269976/1/gate/gate-tempest-dsvm-full/d537680//logs/screen-n-api.txt.gz?level=ERROR#_2016-01-21_19_23_54_340 | 20:15 |
| *** lpetrut has quit IRC | 20:15 | |
| *** jbernard has quit IRC | 20:15 | |
| mriedem | dims: known issue | 20:16 |
| mriedem | dims: http://status.openstack.org/elastic-recheck/index.html#1480305 | 20:16 |
| mriedem | btw, we need to get oslo.db released to get these out of the logs http://logs.openstack.org/76/269976/1/gate/gate-tempest-dsvm-full/d537680//logs/screen-n-api.txt.gz?level=ERROR#_2016-01-21_19_24_00_191 | 20:16 |
| *** jbernard has joined #openstack-nova | 20:16 | |
| mriedem | i was going to put up an oslo.db release request patch this morning but got sidetracked | 20:16 |
| mriedem | and now it's late in the week | 20:16 |
| dims | mriedem : didn't want to release it during m2 week for fear of breaking things | 20:17 |
| *** daemontool has quit IRC | 20:17 | |
| mriedem | we can't get much worse :) | 20:17 |
| dims | mriedem : ya, but i didn't know that before :) | 20:17 |
| mriedem | dims: btw, are you going to be at any midcycles next week? | 20:17 |
| *** su_zhang has quit IRC | 20:18 | |
| dims | mriedem : jay etc will be at the nova one, Alexander is over at keystone one. i won't make it to either one | 20:18 |
| mriedem | jay etc | 20:18 |
| mriedem | jay + entourage | 20:19 |
| dims | yep :) | 20:19 |
| mriedem | could be pugs, could be russian mobsters, no one can tell | 20:19 |
| dims | lol | 20:19 |
| ildikov | lol :) | 20:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: check for interface when detach_interface fails https://review.openstack.org/270891 | 20:20 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: implement LibvirtConfigGuestInterface.parse_dom https://review.openstack.org/270981 | 20:20 |
| jaypipes | mriedem: me, cdent, tdurakov_ and alaski. | 20:20 |
| jaypipes | mriedem: and some pugs on my phone. | 20:21 |
| mriedem | cdent is mirantis now too? | 20:21 |
| jaypipes | mriedem: has been for over a month :) | 20:21 |
| mriedem | sheesh | 20:21 |
| *** amotoki has quit IRC | 20:21 | |
| mriedem | i was wondering why he was working on nova | 20:21 |
| jaypipes | lol | 20:21 |
| *** Marga_ has quit IRC | 20:22 | |
| tdurakov_ | jaypipes, hi, anything for me? | 20:22 |
| *** Marga_ has joined #openstack-nova | 20:22 | |
| jaypipes | tdurakov_: no, just telling mriedem that you'll be in Bristol :) | 20:22 |
| *** mdnadeem has quit IRC | 20:22 | |
| tdurakov_ | jaypipes, ah, ok | 20:23 |
| *** enriquetaso has joined #openstack-nova | 20:23 | |
| *** dimtruck is now known as zz_dimtruck | 20:24 | |
| mriedem | oh no | 20:27 |
| mriedem | NFC Championship: Carolina Panthers vs. Arizona Cardinals, 6:40 p.m., FOX | 20:27 |
| * mriedem leaves for bristol around 8pm | 20:27 | |
| *** tjones has joined #openstack-nova | 20:28 | |
| enriquetaso | jaja | 20:28 |
| openstackgerrit | Tracy Jones proposed openstack/nova: Stop using mox stubs in test_configdrive https://review.openstack.org/265476 | 20:28 |
| *** rlrossit has quit IRC | 20:30 | |
| *** ctrath has quit IRC | 20:30 | |
| *** krtaylor has quit IRC | 20:31 | |
| *** markvoelker has joined #openstack-nova | 20:31 | |
| *** ctrath has joined #openstack-nova | 20:31 | |
| *** otter768 has joined #openstack-nova | 20:34 | |
| *** thinrichs has joined #openstack-nova | 20:34 | |
| *** zz_dimtruck is now known as dimtruck | 20:35 | |
| *** zenoway has quit IRC | 20:36 | |
| *** zenoway has joined #openstack-nova | 20:36 | |
| *** otter768 has quit IRC | 20:39 | |
| openstackgerrit | jichenjc proposed openstack/nova: Make ImageMeta:disk_format use DiskFormatField https://review.openstack.org/231460 | 20:39 |
| mriedem | ildikov: can we hold off on adding the microversion to the multiattach rest api change for now (in case you haven't already done that)? i'd like to get the tempest test working first w/o having to deal with microversoins | 20:42 |
| *** ZZelle_ has joined #openstack-nova | 20:44 | |
| *** matrohon has joined #openstack-nova | 20:45 | |
| *** ijw_ has joined #openstack-nova | 20:45 | |
| *** yarkot has joined #openstack-nova | 20:45 | |
| *** Sukhdev has quit IRC | 20:46 | |
| *** cdent has joined #openstack-nova | 20:47 | |
| *** jamielennox|away is now known as jamielennox | 20:47 | |
| *** baoli has quit IRC | 20:47 | |
| *** mragupat has quit IRC | 20:47 | |
| *** Daisy has joined #openstack-nova | 20:47 | |
| *** mragupat has joined #openstack-nova | 20:47 | |
| *** ijw has quit IRC | 20:48 | |
| *** baoli has joined #openstack-nova | 20:49 | |
| mriedem | ildikov: i guess the rest api change isn't passing dsvm anyway http://logs.openstack.org/38/153038/104/check/gate-tempest-dsvm-full/c97c74c/console.html#_2016-01-21_20_33_04_584 | 20:50 |
| *** Sukhdev has joined #openstack-nova | 20:50 | |
| *** Daisy has quit IRC | 20:51 | |
| mriedem | found the problem https://review.openstack.org/#/c/153038/104/nova/api/openstack/compute/volumes.py | 20:52 |
| *** yamahata has quit IRC | 20:52 | |
| *** daemontool_ has quit IRC | 20:52 | |
| *** daemontool_ has joined #openstack-nova | 20:52 | |
| *** MVenesio has joined #openstack-nova | 20:53 | |
| *** ccarmack has quit IRC | 20:54 | |
| mriedem | oh cripes nova meeting in 6 minutes | 20:54 |
| mriedem | johnthetubaguy: are you running that? | 20:54 |
| ildikov | mriedem: no, I didn't and if I can be honest I didn't want to hurry that one | 20:54 |
| MVenesio | Hi guys, i'm running a Windows instance with openstack KVM and it get freeze randomly, do you have some idea about this issue ? | 20:55 |
| ildikov | mriedem: the tests are sad because the multiattach flag is added to the volume info in the response | 20:55 |
| mriedem | ildikov: yup | 20:55 |
| MVenesio | its windows 2012 R2 image from cloudbase | 20:55 |
| mriedem | ildikov: i think i can get the tempest test stuf going without the rest api change | 20:55 |
| ildikov | mriedem: I can remove that and the change to the translation function for now just to have it stable | 20:55 |
| ildikov | mriedem: I moved the check_attach change to that patch | 20:56 |
| mriedem | no rush | 20:56 |
| mriedem | i just wanted to work on getting the tempest test working this afternoon | 20:56 |
| *** rlrossit_ has joined #openstack-nova | 20:56 | |
| mriedem | i don't need that change though | 20:56 |
| mriedem | the tempest change is pointing at the libvirt change | 20:56 |
| mriedem | oh, but i'll need the check_attach thing... | 20:56 |
| mriedem | so yeah, would be nice if you could update that :) | 20:57 |
| ildikov | yeah, sorry, I reorganized already :( | 20:57 |
| ildikov | ok | 20:57 |
| ildikov | lemme check quickly | 20:57 |
| *** gyee has joined #openstack-nova | 20:57 | |
| *** mgoddard_ has joined #openstack-nova | 20:58 | |
| ildikov | there's something with the functional tests there too, I will never figure out that template stuff there... | 20:58 |
| *** ccarmack has joined #openstack-nova | 20:58 | |
| *** gtt116__ has joined #openstack-nova | 20:59 | |
| *** ljxiash has joined #openstack-nova | 20:59 | |
| *** dslev has quit IRC | 21:00 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** markus_z has joined #openstack-nova | 21:01 | |
| *** haomaiwang has joined #openstack-nova | 21:01 | |
| *** markus_z has quit IRC | 21:02 | |
| *** markus_z has joined #openstack-nova | 21:02 | |
| *** gjayavelu has joined #openstack-nova | 21:02 | |
| *** gtt116_ has quit IRC | 21:03 | |
| *** Sukhdev has quit IRC | 21:03 | |
| *** ccarmack has quit IRC | 21:03 | |
| mriedem | ...no meeting today? | 21:03 |
| mriedem | mikal: johnthetubaguy: ? | 21:03 |
| * edleafe was wondering the same thing | 21:03 | |
| *** mgoddard_ has left #openstack-nova | 21:03 | |
| Vek | should be, according to my calendar... | 21:03 |
| *** ljxiash has quit IRC | 21:04 | |
| *** MVenesio has quit IRC | 21:04 | |
| *** markus_z has quit IRC | 21:04 | |
| *** mgoddard__ has joined #openstack-nova | 21:04 | |
| *** lpetrut has joined #openstack-nova | 21:04 | |
| *** ccarmack has joined #openstack-nova | 21:05 | |
| mriedem | ping mikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes garyk edleafe dimsmoshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov kashyap aloga rgeragnov bauzas xyang tpatil med_ nic scottda nagyz belliottdguitarbite sdague jroll diana_clarke _diana_ raildo jichen gja | 21:05 |
| *** yamahata has joined #openstack-nova | 21:06 | |
| *** erlon has quit IRC | 21:06 | |
| *** diana_clarke has joined #openstack-nova | 21:07 | |
| *** mgoddard has quit IRC | 21:08 | |
| *** mgoddard has joined #openstack-nova | 21:09 | |
| *** dimtruck is now known as zz_dimtruck | 21:11 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: no longer convert image meta from dict to object https://review.openstack.org/270867 | 21:11 |
| garyk | mriedem: johnthetubaguy: ^ | 21:11 |
| garyk | sorry i had a dumbass pep8 error. sorry | 21:11 |
| *** salv-orlando has joined #openstack-nova | 21:11 | |
| mriedem | +W | 21:12 |
| *** mgoddard__ has quit IRC | 21:12 | |
| *** matrohon has quit IRC | 21:13 | |
| *** nelsnels_ has quit IRC | 21:14 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Ceph for live-migration job https://review.openstack.org/252283 | 21:15 |
| *** lpetrut has quit IRC | 21:15 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add description for server query https://review.openstack.org/259565 | 21:15 |
| *** matrohon has joined #openstack-nova | 21:16 | |
| *** thangp has left #openstack-nova | 21:17 | |
| *** yarkot has quit IRC | 21:18 | |
| edleafe | garyk: is there any other kind? :) | 21:18 |
| *** dslev has joined #openstack-nova | 21:19 | |
| *** nelsnelson has joined #openstack-nova | 21:22 | |
| *** matrohon has quit IRC | 21:23 | |
| garyk | edleafe: :) | 21:26 |
| *** baoli has quit IRC | 21:27 | |
| *** baoli has joined #openstack-nova | 21:29 | |
| *** zz_dimtruck is now known as dimtruck | 21:29 | |
| *** mdnadeem has joined #openstack-nova | 21:30 | |
| *** ctrath1 has joined #openstack-nova | 21:32 | |
| *** baoli has quit IRC | 21:35 | |
| *** su_zhang has joined #openstack-nova | 21:35 | |
| *** ctrath has quit IRC | 21:35 | |
| *** baoli has joined #openstack-nova | 21:36 | |
| *** tjones has quit IRC | 21:36 | |
| *** penick_ has joined #openstack-nova | 21:37 | |
| *** penick has quit IRC | 21:37 | |
| *** penick_ is now known as penick | 21:37 | |
| *** baoli has quit IRC | 21:38 | |
| *** baoli has joined #openstack-nova | 21:38 | |
| *** mdnadeem has quit IRC | 21:38 | |
| *** markvoelker has quit IRC | 21:39 | |
| *** markvoelker has joined #openstack-nova | 21:41 | |
| *** cdent has quit IRC | 21:42 | |
| *** tjones has joined #openstack-nova | 21:44 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: Allow multi-attach in compute api https://review.openstack.org/153038 | 21:44 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: Modify API response to reflect on multiattach changes https://review.openstack.org/271047 | 21:44 |
| thomasem | mriedem: does this note suffice? https://wiki.openstack.org/wiki/HypervisorSupportMatrix#Hypervisor_feature_support_matrix | 21:45 |
| ildikov | mriedem: that should serve the purpose ^^ | 21:46 |
| ildikov | mriedem: the compute api patch should work with tempest | 21:46 |
| *** ijw_ has quit IRC | 21:46 | |
| *** nelsnelson has quit IRC | 21:46 | |
| *** cfriesen has quit IRC | 21:49 | |
| *** thorst has quit IRC | 21:49 | |
| *** thorst has joined #openstack-nova | 21:50 | |
| *** gjayavelu has quit IRC | 21:50 | |
| *** knikolla has quit IRC | 21:51 | |
| *** ccarmack has quit IRC | 21:53 | |
| *** krtaylor has joined #openstack-nova | 21:53 | |
| openstackgerrit | Dimitry Ushakov proposed openstack/nova: (lxc) Updated regex to ignore failing tests https://review.openstack.org/267724 | 21:55 |
| *** ccarmack has joined #openstack-nova | 21:55 | |
| *** dims_ has joined #openstack-nova | 21:56 | |
| *** burgerk has quit IRC | 21:57 | |
| *** dims has quit IRC | 21:57 | |
| mriedem | thomasem: the support matrix is actually an ini file in the in-tree docs | 21:58 |
| *** kfarr has quit IRC | 21:59 | |
| thomasem | Ah | 21:59 |
| thomasem | I see | 21:59 |
| mriedem | thomasem: https://github.com/openstack/nova/blob/master/doc/source/support-matrix.ini | 21:59 |
| *** dslev has quit IRC | 21:59 | |
| mriedem | thomasem: so something like this | 21:59 |
| mriedem | https://github.com/openstack/nova/blob/master/doc/source/support-matrix.ini#L425 | 21:59 |
| mriedem | for stop + libvirt/lxc | 22:00 |
| thomasem | mriedem: perfect, okay thank you | 22:00 |
| thomasem | I see now | 22:00 |
| tjones | edleafe: fixed my import issue in https://review.openstack.org/#/c/252687/ | 22:00 |
| mriedem | ildikov: ok, cool, thanks | 22:00 |
| *** haomaiwang has quit IRC | 22:01 | |
| ildikov | mriedem: do you need anything else? | 22:01 |
| mriedem | ildikov: probably not today, it's getting late for me | 22:01 |
| mriedem | like, you know, 4pm | 22:01 |
| *** haomaiwang has joined #openstack-nova | 22:01 | |
| ildikov | yeah, I know, it's 11pm here :) | 22:01 |
| edleafe | tjones: +1 | 22:02 |
| ildikov | allright, I go and get some sleep then :) | 22:02 |
| tjones | thanks | 22:02 |
| *** penick has quit IRC | 22:02 | |
| *** gjayavelu has joined #openstack-nova | 22:04 | |
| *** markvoelker has quit IRC | 22:04 | |
| *** darrenc is now known as darrenc_afk | 22:05 | |
| *** Sukhdev has joined #openstack-nova | 22:06 | |
| *** ahelkhou has quit IRC | 22:06 | |
| *** huats_ has quit IRC | 22:07 | |
| *** pratikma_ has quit IRC | 22:11 | |
| thomasem | mriedem: this is sort of perplexing... so operation.stop doesn't exist in that matrix. Do you know if that's roughly equivalent to something else. There's shutting down an instance, which is called operation.terminate, which I think is 'delete'? | 22:11 |
| thomasem | I'm confused. | 22:11 |
| mriedem | thomasem: is shutdown in there? | 22:12 |
| mriedem | thomasem: https://github.com/openstack/nova/blob/master/doc/source/support-matrix.ini#L522 | 22:12 |
| thomasem | https://github.com/openstack/nova/blob/master/doc/source/support-matrix.ini#L521-L523 | 22:12 |
| thomasem | Okay, couldn't tell by the wording there, since it uses the word 'terminate' | 22:12 |
| thomasem | And that still utilizes disk space, if that's the case. | 22:13 |
| mriedem | yeah terminate is confusing, | 22:13 |
| thomasem | Okay, I'll add it there. | 22:13 |
| mriedem | could be b/c danpb wrote that and libvirt stop destorys the domain right? | 22:13 |
| mriedem | *destroys | 22:13 |
| *** pradk has joined #openstack-nova | 22:13 | |
| thomasem | Nova is calling virsh shutdown on nova stop | 22:14 |
| *** dslev has joined #openstack-nova | 22:14 | |
| thomasem | That's what's failing | 22:14 |
| thomasem | If it called destroy, then we wouldn't even be having this chat. :P | 22:14 |
| mriedem | oh ok | 22:14 |
| thomasem | Possibly another, though. 'shutdown' is supported in libvirt/lxc now, though | 22:14 |
| *** huats_ has joined #openstack-nova | 22:17 | |
| *** huats_ has quit IRC | 22:17 | |
| *** huats_ has joined #openstack-nova | 22:17 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Dump dhcp options before dnsmasq HUP https://review.openstack.org/271056 | 22:17 |
| ccarmack | mriedem: can I get a +W on https://review.openstack.org/#/c/269664/ It had +W, then I had to rebase it | 22:17 |
| thomasem | Cool, thanks mriedem!! patch incoming shortly | 22:17 |
| mriedem | ccarmack: done | 22:18 |
| ccarmack | thanks | 22:18 |
| *** penick has joined #openstack-nova | 22:18 | |
| mriedem | claudiub: a unit test would be helpful in demonstrating what you're trying to do here https://review.openstack.org/#/c/265239/ | 22:18 |
| mriedem | edleafe: alaski: ^ maybe your python-foo can parse that | 22:18 |
| *** thorst has quit IRC | 22:21 | |
| *** thorst has joined #openstack-nova | 22:21 | |
| *** darrenc_afk is now known as darrenc | 22:22 | |
| *** tdurakov_ has quit IRC | 22:22 | |
| claudiub | mriedem: eh, it's going to be a bit troublesome to prove in a unit test that the original exception is maintained. | 22:22 |
| mriedem | can you fake out a stacktrace? | 22:23 |
| *** markvoelker has joined #openstack-nova | 22:24 | |
| claudiub | i'll see what i can do, but no promises. :) | 22:24 |
| *** gjayavelu has quit IRC | 22:24 | |
| alaski | the traceback part of that makes sense, but I have no clue what raised_exception is here | 22:25 |
| *** thorst has quit IRC | 22:26 | |
| *** prometheanfire has joined #openstack-nova | 22:26 | |
| claudiub | alaski: raised_exception is a nova exception. All os_win exceptions are caught and reraised as nova exceptions. | 22:26 |
| openstackgerrit | Thomas Maddox proposed openstack/nova: Note in HypervisorSupportMatrix for Libvirt/LXC shutdown kernel bug https://review.openstack.org/271063 | 22:27 |
| edleafe | yeah, I was just trying to figure that out too | 22:29 |
| *** thorst has joined #openstack-nova | 22:29 | |
| *** baoli has quit IRC | 22:30 | |
| alaski | there is a test for exceptions being converted and that's still passing | 22:31 |
| claudiub | yep, there is. | 22:31 |
| *** busterswt has quit IRC | 22:32 | |
| *** thorst has quit IRC | 22:34 | |
| *** dslev has quit IRC | 22:34 | |
| *** otter768 has joined #openstack-nova | 22:35 | |
| alaski | if you address mriedems comments I'm +2 | 22:35 |
| claudiub | k, thanks! will do it in the morning though.. | 22:36 |
| *** ahelkhou has joined #openstack-nova | 22:38 | |
| *** otter768 has quit IRC | 22:39 | |
| *** ahelkhou has quit IRC | 22:39 | |
| *** ahelkhou has joined #openstack-nova | 22:40 | |
| mriedem | mmm, with multiattach volumes, | 22:41 |
| mriedem | oh nvm | 22:41 |
| *** claudiub has quit IRC | 22:42 | |
| *** rlrossit_ has quit IRC | 22:43 | |
| *** ahelkhou has quit IRC | 22:44 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: WIP - remove mox from vmwareapi/test_vif.py https://review.openstack.org/271069 | 22:44 |
| *** thinrichs has quit IRC | 22:47 | |
| *** thinrichs has joined #openstack-nova | 22:47 | |
| *** diana_clarke has quit IRC | 22:48 | |
| *** ccarmack has quit IRC | 22:51 | |
| *** ctrath1 has quit IRC | 22:53 | |
| *** mriedem has quit IRC | 22:53 | |
| *** tonytan4ever has quit IRC | 22:53 | |
| *** harlowja has quit IRC | 22:56 | |
| *** cfriesen has joined #openstack-nova | 22:56 | |
| *** harlowja has joined #openstack-nova | 22:56 | |
| *** mragupat has quit IRC | 22:56 | |
| kashyap | mriedem: Heya, still around by any chance? | 22:57 |
| *** hblixt has joined #openstack-nova | 22:57 | |
| kashyap | When you are - | 22:57 |
| *** lpetrut has joined #openstack-nova | 22:58 | |
| *** diana_clarke has joined #openstack-nova | 22:58 | |
| *** ctrath has joined #openstack-nova | 22:59 | |
| *** annegent_ has quit IRC | 23:00 | |
| *** pushkaru has quit IRC | 23:00 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-nova | 23:01 | |
| kashyap | mriedem: It's something you're familiar with, wonder you know why I see this unit test failure (code & its test invoc): http://paste.openstack.org/show/484625/ | 23:01 |
| * kashyap will check back in here in the morning, late here in CET. Don't bother if you don't have time. | 23:02 | |
| *** lpetrut has quit IRC | 23:02 | |
| *** dimtruck is now known as zz_dimtruck | 23:06 | |
| *** Sukhdev has quit IRC | 23:06 | |
| *** unicell1 has joined #openstack-nova | 23:07 | |
| *** mdrabe has quit IRC | 23:08 | |
| *** unicell has quit IRC | 23:08 | |
| *** ivase has quit IRC | 23:10 | |
| *** hblixt_ has joined #openstack-nova | 23:10 | |
| *** ivase has joined #openstack-nova | 23:11 | |
| *** Sukhdev has joined #openstack-nova | 23:12 | |
| *** hblixt has quit IRC | 23:13 | |
| *** Daisy has joined #openstack-nova | 23:14 | |
| *** ivase has quit IRC | 23:15 | |
| *** ccarmack has joined #openstack-nova | 23:17 | |
| *** Daisy has quit IRC | 23:18 | |
| *** dedery_ has joined #openstack-nova | 23:21 | |
| *** diana_clarke has quit IRC | 23:22 | |
| *** dedery has quit IRC | 23:25 | |
| *** dedery_ has quit IRC | 23:26 | |
| *** burt has quit IRC | 23:27 | |
| *** bradjones_ has joined #openstack-nova | 23:31 | |
| *** bradjones_ has quit IRC | 23:31 | |
| *** bradjones_ has joined #openstack-nova | 23:31 | |
| nic | I'd like to get a spec together to convert Nova to TrumpScript: http://samshadwell.github.io/TrumpScript/ | 23:34 |
| *** mrkz has quit IRC | 23:42 | |
| *** mrkz has joined #openstack-nova | 23:44 | |
| *** ccarmack has quit IRC | 23:44 | |
| *** _hblixt_ has joined #openstack-nova | 23:45 | |
| *** ctrath has quit IRC | 23:46 | |
| *** mrkz has quit IRC | 23:47 | |
| *** rcernin has quit IRC | 23:47 | |
| *** bradjones_ has quit IRC | 23:47 | |
| *** bradjones_ has joined #openstack-nova | 23:47 | |
| *** bradjones_ has quit IRC | 23:47 | |
| *** bradjones_ has joined #openstack-nova | 23:47 | |
| *** hblixt_ has quit IRC | 23:48 | |
| *** isq_ has quit IRC | 23:48 | |
| *** edtubill has quit IRC | 23:49 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: remove mox from vmwareapi/test_vif.py https://review.openstack.org/271069 | 23:51 |
| *** darrenc is now known as darrenc_afk | 23:51 | |
| *** edmondsw has quit IRC | 23:55 | |
| *** isq has joined #openstack-nova | 23:56 | |
| *** nic has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!