| *** ducttape_ has joined #openstack-nova | 00:02 | |
| *** hongbin has quit IRC | 00:02 | |
| *** sdague has quit IRC | 00:03 | |
| *** ayoung has quit IRC | 00:07 | |
| *** yingjun has joined #openstack-nova | 00:09 | |
| *** yingjun has quit IRC | 00:11 | |
| mriedem | melwitt: some replies in https://review.openstack.org/#/c/399710/ most only the cache thing | 00:12 |
|---|---|---|
| mriedem | it seems the cache needs to store a list | 00:12 |
| melwitt | mriedem: replying | 00:12 |
| dansmith | mriedem: read: "explaining why you're wrong" | 00:12 |
| dansmith | it's working pretty well for me with no changes, for what it's worth | 00:13 |
| melwitt | that's the schema cache. we only need one main database schema for an entire test suite run | 00:13 |
| *** yingjun has joined #openstack-nova | 00:13 | |
| mriedem | oh... | 00:13 |
| mriedem | i thought it was cell specific | 00:13 |
| melwitt | we just use the same schema to run migrations on any new database we create | 00:13 |
| mriedem | so blah and wut would have separate cached things | 00:13 |
| dansmith | mriedem: look at the database fixture | 00:13 |
| mriedem | i refuse | 00:13 |
| *** kylek3h has joined #openstack-nova | 00:14 | |
| mriedem | yeah she said it was the same but that fixture also assumes a single db i think | 00:14 |
| melwitt | it's just the data model for the main database. generated and cached once for everyone. plain Database fixture uses the same | 00:14 |
| mriedem | this is multiple, so that's why i thought it was different | 00:14 |
| dansmith | mriedem: no | 00:14 |
| *** lamt has quit IRC | 00:14 | |
| mriedem | right, she's explained now | 00:14 |
| dansmith | mriedem: the point is, it can only run that once ever | 00:14 |
| dansmith | okay | 00:14 |
| mriedem | hey, | 00:14 |
| mriedem | hey! | 00:14 |
| mriedem | at least i didn't put a thing in the main test case that does something 10 lines later | 00:14 |
| mriedem | so there :) | 00:14 |
| dansmith | you may be able to make a chicken pot pie, but leave the cells stuff to us | 00:14 |
| dansmith | mriedem: I have no idea what you're talking about | 00:15 |
| mriedem | will do | 00:15 |
| mriedem | turkey btw | 00:15 |
| melwitt | lol | 00:15 |
| mriedem | needed to do something with the leftovers | 00:15 |
| mriedem | because i'll be damned if i'm going to just microware that dry ass tasteless bird meat | 00:15 |
| mriedem | i would like thanksgiving to henceforth be steak or ribs day or something | 00:16 |
| mriedem | big platter of hamburgers | 00:16 |
| dansmith | speaking of any sort of faux pas related to making things work in parallel mode | 00:16 |
| dansmith | like say, resetting the indirection interface, | 00:16 |
| dansmith | we've definitely got some test interactions that are laying dormant until some of these cells changes | 00:16 |
| dansmith | I've been wrestling with another all day | 00:16 |
| dansmith | where the fping test impacts a few tests in test_compute | 00:16 |
| *** jamielennox is now known as jamielennox|away | 00:17 | |
| dansmith | I'm giving up for the day, but I still haven't fixed them | 00:17 |
| mriedem | fping | 00:17 |
| mriedem | f fping | 00:17 |
| dansmith | ikr | 00:17 |
| mriedem | @testtools.skip('f fping') | 00:17 |
| melwitt | f'ing fping | 00:17 |
| mriedem | problem solved | 00:18 |
| mriedem | anything that remotely touches what test_compute does is subject to problems | 00:18 |
| mriedem | lots of stubs and garbage in there | 00:18 |
| melwitt | yeah, these cells changes have made more apparent how global so much stuff is. even with the CellDatabases fixture I think it's not possible to have more than one compute cell database | 00:19 |
| dansmith | I dunno what is going on exactly, but I stub something out in the fping test, with a fixture that should reset itself, | 00:19 |
| dansmith | and then it's there for compute later | 00:19 |
| melwitt | or multiple compute services with separate configs, rather | 00:19 |
| dansmith | but if you run in a different order, everything is fine | 00:20 |
| *** sreeram_v has joined #openstack-nova | 00:21 | |
| melwitt | :\ | 00:22 |
| dansmith | I wish it was something with my stubbing, but it's a fixture | 00:23 |
| mriedem | dansmith: sdague loves untangling that kind of stuff | 00:23 |
| dansmith | should be pretty sure | 00:23 |
| dansmith | mriedem: yeah I know I'm planning a friday for him | 00:23 |
| mriedem | he usually deletes it all at the end, | 00:23 |
| mriedem | but i think he enjoys the challenge | 00:23 |
| dansmith | mriedem: that's true, he goes nuclear | 00:23 |
| mriedem | "now that i've figured this all out, i'm going to just delete it" | 00:23 |
| mriedem | "let us never speak of this again" | 00:23 |
| *** jamielennox|away is now known as jamielennox | 00:23 | |
| melwitt | heh | 00:24 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method https://review.openstack.org/367557 | 00:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Multicell support for instance listing https://review.openstack.org/396775 | 00:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make RPCFixture support multiple connections https://review.openstack.org/396417 | 00:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 00:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add SingleCellSimple fixture https://review.openstack.org/405684 | 00:25 |
| *** sreeram_v has quit IRC | 00:26 | |
| *** ayoung has joined #openstack-nova | 00:28 | |
| *** duonghq_ is now known as duonghq | 00:29 | |
| *** gyee has joined #openstack-nova | 00:31 | |
| openstackgerrit | ayoung proposed openstack/nova: API Policy enforcement contingent on is_admin_project https://review.openstack.org/384148 | 00:31 |
| *** code-R has joined #openstack-nova | 00:34 | |
| *** takashin has joined #openstack-nova | 00:35 | |
| *** hoangcx has joined #openstack-nova | 00:41 | |
| *** jamielennox is now known as jamielennox|away | 00:46 | |
| *** tovin07 has joined #openstack-nova | 00:47 | |
| *** jamielennox|away is now known as jamielennox | 00:53 | |
| *** unicell1 has quit IRC | 00:53 | |
| *** unicell has joined #openstack-nova | 00:56 | |
| *** unicell has quit IRC | 00:57 | |
| *** unicell has joined #openstack-nova | 00:58 | |
| *** pradk has quit IRC | 01:00 | |
| *** rfolco has joined #openstack-nova | 01:02 | |
| alex_xu | oomichi: re: https://review.openstack.org/#/c/392077/10, i feel we should keep consistent behaviour in the configration file and extra spec | 01:03 |
| *** code-R has quit IRC | 01:03 | |
| *** ducttape_ has quit IRC | 01:04 | |
| alex_xu | pkholkin: thanks for the patch, will take a look at it | 01:04 |
| *** code-R has joined #openstack-nova | 01:04 | |
| *** liusheng has quit IRC | 01:05 | |
| *** rama_y has quit IRC | 01:05 | |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Unexpected API error during server create https://review.openstack.org/404565 | 01:05 |
| *** gyee has quit IRC | 01:06 | |
| *** Swami_ has joined #openstack-nova | 01:07 | |
| *** nic has quit IRC | 01:07 | |
| *** tbachman has joined #openstack-nova | 01:07 | |
| *** fandi__ has joined #openstack-nova | 01:08 | |
| *** zhangjl has joined #openstack-nova | 01:08 | |
| *** Swami has quit IRC | 01:10 | |
| *** fandi has quit IRC | 01:12 | |
| *** zz_dimtruck is now known as dimtruck | 01:12 | |
| *** ducttape_ has joined #openstack-nova | 01:14 | |
| *** int32bit has joined #openstack-nova | 01:16 | |
| *** ijw has joined #openstack-nova | 01:17 | |
| *** ijw has quit IRC | 01:17 | |
| *** mlavalle has quit IRC | 01:17 | |
| *** ijw has joined #openstack-nova | 01:18 | |
| *** Swami_ has quit IRC | 01:19 | |
| *** dave-mccowan has joined #openstack-nova | 01:24 | |
| openstackgerrit | zhangyanxian proposed openstack/nova: Fix import method to follow community guideline https://review.openstack.org/405703 | 01:29 |
| oomichi | alex_xu: yeah, thanks. I can see your point also, it is nice to write some NOTE for us because it would be easy to forget the reason. especially me ;) | 01:29 |
| oomichi | if that is necessary | 01:29 |
| openstackgerrit | Jay Pipes proposed openstack/nova: WIP: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | 01:30 |
| alex_xu | oomichi: yea, a note is great | 01:30 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: minor refactor _allocate_for_instance() https://review.openstack.org/405705 | 01:30 |
| openstackgerrit | zhangyanxian proposed openstack/nova: Fix import method to follow community guideline https://review.openstack.org/405703 | 01:30 |
| *** jaypipes has quit IRC | 01:30 | |
| alex_xu | oomichi: and there is one more https://review.openstack.org/404565, appreciate if you can help :) | 01:31 |
| oomichi | alex_xu: oh, ok. but hey! I also need your eyes on https://review.openstack.org/#/c/385212/ and https://review.openstack.org/#/c/385211 ;) | 01:32 |
| alex_xu | oomichi: yea, I will take a look at those right now! :) | 01:32 |
| oomichi | alex_xu: haha, XieXie | 01:33 |
| alex_xu | oomichi: ありがとう | 01:34 |
| oomichi | alex_xu: perfect japanese word :) | 01:35 |
| alex_xu | heh, by google :) | 01:35 |
| *** yuntongjin has joined #openstack-nova | 01:35 | |
| *** thorst_ has joined #openstack-nova | 01:37 | |
| oomichi | alex_xu: ah, I see. Yeah google translator is much improved with neural machine: http://www.zdnet.com/article/google-announces-neural-machine-translation-to-improve-google-translate/ | 01:38 |
| alex_xu | oomichi: heh | 01:39 |
| *** code-R has quit IRC | 01:39 | |
| oomichi | I am not sure we will still need to use English in this chat in the future | 01:39 |
| *** thorst_ has quit IRC | 01:39 | |
| *** unicell has quit IRC | 01:40 | |
| *** thorst_ has joined #openstack-nova | 01:40 | |
| *** unicell has joined #openstack-nova | 01:40 | |
| *** mtanino has quit IRC | 01:40 | |
| *** yingjun has quit IRC | 01:43 | |
| *** dimtruck is now known as zz_dimtruck | 01:43 | |
| *** zz_dimtruck is now known as dimtruck | 01:47 | |
| openstackgerrit | zhangyanxian proposed openstack/nova: Fix import method to follow community guideline https://review.openstack.org/405703 | 01:48 |
| *** thorst_ has quit IRC | 01:48 | |
| *** yingjun has joined #openstack-nova | 01:49 | |
| *** browne has quit IRC | 01:50 | |
| *** liusheng has joined #openstack-nova | 01:58 | |
| *** Apoorva_ has quit IRC | 01:59 | |
| *** yuntongjin has quit IRC | 02:03 | |
| *** ducttape_ has quit IRC | 02:04 | |
| *** kaisers_ has joined #openstack-nova | 02:05 | |
| openstackgerrit | melanie witt proposed openstack/nova: Raise DeviceNotFound detaching volume from persistent domain https://review.openstack.org/386257 | 02:07 |
| *** kaisers__ has quit IRC | 02:07 | |
| *** ducttape_ has joined #openstack-nova | 02:08 | |
| *** hongbin has joined #openstack-nova | 02:14 | |
| *** fandi__ has quit IRC | 02:15 | |
| *** ijw has quit IRC | 02:18 | |
| *** ijw has joined #openstack-nova | 02:19 | |
| *** dimtruck is now known as zz_dimtruck | 02:19 | |
| *** dave-mccowan has quit IRC | 02:20 | |
| openstackgerrit | 赵凯 proposed openstack/nova: [network] nova with neutron can not inject ip address to vm https://review.openstack.org/354616 | 02:21 |
| *** ducttape_ has quit IRC | 02:21 | |
| *** ijw_ has joined #openstack-nova | 02:21 | |
| *** sreeram_v has joined #openstack-nova | 02:22 | |
| *** ijw has quit IRC | 02:23 | |
| *** claudiub|2 has quit IRC | 02:24 | |
| *** unicell has quit IRC | 02:26 | |
| *** sreeram_v has quit IRC | 02:27 | |
| *** zz_dimtruck is now known as dimtruck | 02:27 | |
| *** unicell has joined #openstack-nova | 02:29 | |
| *** dave-mccowan has joined #openstack-nova | 02:29 | |
| *** dimtruck is now known as zz_dimtruck | 02:31 | |
| *** thorst_ has joined #openstack-nova | 02:32 | |
| *** unicell has quit IRC | 02:33 | |
| *** thorst_ has quit IRC | 02:34 | |
| *** thorst_ has joined #openstack-nova | 02:35 | |
| *** unicell has joined #openstack-nova | 02:35 | |
| *** hfu has joined #openstack-nova | 02:35 | |
| openstackgerrit | zhaolihui proposed openstack/nova: Fix pci_alias that include white spaces https://review.openstack.org/392077 | 02:36 |
| alex_xu | oomichi: https://review.openstack.org/#/c/392077 updated, appreciate you can take a look at whether is good for now. | 02:38 |
| oomichi | alex_xu: oh, my concern is different place.. that should be at line 108 | 02:40 |
| oomichi | because of a new line | 02:40 |
| alex_xu | oomichi: ah, yea | 02:40 |
| alex_xu | oomichi: let me edit it online directly | 02:41 |
| oomichi | alex_xu: it is nice to do that by other guy except you for +2 | 02:41 |
| openstackgerrit | Alex Xu proposed openstack/nova: Fix pci_alias that include white spaces https://review.openstack.org/392077 | 02:42 |
| *** thorst_ has quit IRC | 02:43 | |
| alex_xu | oomichi: just done. emm...yea, sometime I just quick update for small thing for people, avoid another around-trip | 02:43 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port api.openstack.compute.test_disk_config to Python 3 https://review.openstack.org/403536 | 02:44 |
| oomichi | alex_xu: ok, that is nit. you can +2 | 02:44 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port security group related tests to Python 3 https://review.openstack.org/401241 | 02:44 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port compute unit tests to Python 3 https://review.openstack.org/402010 | 02:44 |
| alex_xu | oomichi: :) | 02:44 |
| oomichi | alex_xu: +2 | 02:45 |
| oomichi | thanks | 02:45 |
| alex_xu | oomichi: thanks too | 02:46 |
| *** Jeffrey4l_ has quit IRC | 02:47 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed https://review.openstack.org/403568 | 02:50 |
| *** Jeffrey4l_ has joined #openstack-nova | 02:50 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add block_device_mapping_v2.uuid to api-ref https://review.openstack.org/405751 | 02:50 |
| *** sam_nowitzki has joined #openstack-nova | 02:52 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add block_device_mapping_v2.uuid to api-ref https://review.openstack.org/405751 | 02:53 |
| oomichi | alex_xu: no problem, and can you take a look at ^^^ ? | 02:54 |
| oomichi | that also is a simple one | 02:54 |
| *** namnh has joined #openstack-nova | 02:55 | |
| *** ayogi has joined #openstack-nova | 02:58 | |
| alex_xu | oomichi: yea, let me check | 03:01 |
| oomichi | alex_xu: thanks :) | 03:01 |
| alex_xu | oomichi: np :) | 03:02 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add block_device_mapping_v2.uuid to api-ref https://review.openstack.org/405751 | 03:03 |
| oomichi | alex_xu: sorry, some duplicated words in that. I updated ^^^ | 03:03 |
| *** gjayavelu has quit IRC | 03:03 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Correct the sorting of datetimes for migrations https://review.openstack.org/353123 | 03:06 |
| edleafe | alex_xu: ^^ updated with your suggestion | 03:09 |
| alex_xu | edleafe: thanks | 03:10 |
| *** jamielennox is now known as jamielennox|away | 03:11 | |
| edleafe | alex_xu: took me a while to figure out the o.vo error in the tests | 03:11 |
| alex_xu | oomichi: commented on https://review.openstack.org/#/c/405751/3 | 03:13 |
| oomichi | alex_xu: much better, thanks! | 03:14 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add block_device_mapping_v2.uuid to api-ref https://review.openstack.org/405751 | 03:15 |
| oomichi | alex_xu: ^^^ | 03:15 |
| *** browne has joined #openstack-nova | 03:17 | |
| alex_xu | oomichi: thanks, LGTM | 03:17 |
| oomichi | alex_xu: thanks again :) | 03:18 |
| *** browne has quit IRC | 03:18 | |
| alex_xu | oomichi: :) | 03:20 |
| *** jamielennox|away is now known as jamielennox | 03:20 | |
| *** browne has joined #openstack-nova | 03:22 | |
| *** flaviof has left #openstack-nova | 03:22 | |
| *** masber has joined #openstack-nova | 03:26 | |
| *** masber has quit IRC | 03:26 | |
| *** cheran has quit IRC | 03:27 | |
| *** mtanino has joined #openstack-nova | 03:27 | |
| *** ljxiash has joined #openstack-nova | 03:28 | |
| *** dave-mccowan has quit IRC | 03:28 | |
| *** ayogi has quit IRC | 03:29 | |
| *** unicell has quit IRC | 03:30 | |
| *** jamielennox is now known as jamielennox|away | 03:30 | |
| *** jamielennox|away is now known as jamielennox | 03:37 | |
| *** thorst_ has joined #openstack-nova | 03:37 | |
| *** thorst_ has quit IRC | 03:37 | |
| *** thorst_ has joined #openstack-nova | 03:38 | |
| openstackgerrit | ayoung proposed openstack/nova: Admin API Policy enforcement contingent on is_admin_project https://review.openstack.org/384148 | 03:40 |
| *** code-R has joined #openstack-nova | 03:44 | |
| *** psachin has joined #openstack-nova | 03:46 | |
| *** browne has quit IRC | 03:47 | |
| *** thorst_ has quit IRC | 03:47 | |
| *** jwcroppe has joined #openstack-nova | 03:51 | |
| *** links has joined #openstack-nova | 03:52 | |
| *** nicolasbock has quit IRC | 03:56 | |
| *** diga has joined #openstack-nova | 03:58 | |
| oomichi | alex_xu: https://review.openstack.org/#/c/391744 also is easy one | 04:00 |
| *** fandi has joined #openstack-nova | 04:01 | |
| *** zhangjl has quit IRC | 04:02 | |
| *** baoli has quit IRC | 04:04 | |
| *** sudipto has joined #openstack-nova | 04:04 | |
| *** sudipto_ has joined #openstack-nova | 04:05 | |
| *** rama_y has joined #openstack-nova | 04:05 | |
| *** bhagyashris has quit IRC | 04:07 | |
| *** baoli has joined #openstack-nova | 04:08 | |
| *** code-R has quit IRC | 04:11 | |
| *** code-R has joined #openstack-nova | 04:14 | |
| *** baoli has quit IRC | 04:15 | |
| *** ducttape_ has joined #openstack-nova | 04:18 | |
| *** yamahata has quit IRC | 04:20 | |
| *** int32bit has quit IRC | 04:23 | |
| *** zhangjl has joined #openstack-nova | 04:24 | |
| *** mtanino has quit IRC | 04:26 | |
| *** code-R has quit IRC | 04:28 | |
| gmann | oomichi: alex_xu as you guys are up, can you review this - https://review.openstack.org/#/c/402295/ | 04:28 |
| gmann | oomichi: alex_xu next is to move some of the server tag validation code to schema from python code | 04:29 |
| *** hongbin has quit IRC | 04:31 | |
| *** gongysh has joined #openstack-nova | 04:38 | |
| *** mriedem has quit IRC | 04:39 | |
| *** tojuvone has joined #openstack-nova | 04:39 | |
| *** mdnadeem has joined #openstack-nova | 04:41 | |
| *** tojuvone has quit IRC | 04:42 | |
| *** tojuvone has joined #openstack-nova | 04:42 | |
| *** thorst_ has joined #openstack-nova | 04:44 | |
| *** thorst_ has quit IRC | 04:52 | |
| *** sudipto_ has quit IRC | 04:59 | |
| *** sudipto has quit IRC | 04:59 | |
| *** ayogi has joined #openstack-nova | 05:03 | |
| *** sreeram_v has joined #openstack-nova | 05:06 | |
| *** sreeram_v has joined #openstack-nova | 05:06 | |
| *** yuntongjin has joined #openstack-nova | 05:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 05:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/405895 | 05:13 |
| *** voelzmo has joined #openstack-nova | 05:16 | |
| *** voelzmo has quit IRC | 05:16 | |
| *** voelzmo has joined #openstack-nova | 05:16 | |
| *** fragatina has joined #openstack-nova | 05:21 | |
| *** ijw_ has quit IRC | 05:22 | |
| *** voelzmo has quit IRC | 05:23 | |
| *** ratailor has joined #openstack-nova | 05:23 | |
| *** kaisers_ has quit IRC | 05:25 | |
| *** ducttape_ has quit IRC | 05:26 | |
| *** yamahata has joined #openstack-nova | 05:27 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: conf: Remove deprecated service manager opts https://review.openstack.org/319532 | 05:27 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: conf: Remove config option compute_ manager https://review.openstack.org/405921 | 05:27 |
| *** penick has joined #openstack-nova | 05:27 | |
| *** tbachman has quit IRC | 05:31 | |
| *** fragatina has quit IRC | 05:33 | |
| *** fragatina has joined #openstack-nova | 05:33 | |
| *** fragatina has quit IRC | 05:34 | |
| *** fragatina has joined #openstack-nova | 05:38 | |
| *** fragatina has quit IRC | 05:38 | |
| *** fragatina has joined #openstack-nova | 05:38 | |
| *** fragatina has quit IRC | 05:38 | |
| *** fragatina has joined #openstack-nova | 05:39 | |
| alex_xu | oomichi: yea, will do that after you take a look at https://review.openstack.org/#/c/404565/ :) | 05:39 |
| alex_xu | gmann: got it, put it on my list | 05:39 |
| openstackgerrit | cissy proposed openstack/nova: nova api should not cache az info in process memory https://review.openstack.org/405927 | 05:40 |
| *** fragatina has quit IRC | 05:40 | |
| *** fragatina has joined #openstack-nova | 05:40 | |
| gmann | alex_xu: Thanks | 05:41 |
| *** sudipto has joined #openstack-nova | 05:41 | |
| *** sudipto_ has joined #openstack-nova | 05:41 | |
| gmann | alex_xu: on this - https://review.openstack.org/#/c/404565/ | 05:41 |
| gmann | alex_xu: i was thiking to do on schema side but its all metadata there | 05:41 |
| *** fragatina has quit IRC | 05:42 | |
| *** fragatina has joined #openstack-nova | 05:43 | |
| *** gouthamr has joined #openstack-nova | 05:43 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: VMWare: Move constant power state strings to the constant.py https://review.openstack.org/368383 | 05:45 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Use function is_valid_ipv6_cidr from oslo.utils https://review.openstack.org/389545 | 05:45 |
| *** bnemec has joined #openstack-nova | 05:49 | |
| *** zz_dimtruck is now known as dimtruck | 05:50 | |
| *** rama_y has quit IRC | 05:51 | |
| *** unicell has joined #openstack-nova | 05:59 | |
| *** fandi has quit IRC | 06:05 | |
| *** gouthamr has quit IRC | 06:05 | |
| *** jaosorior has joined #openstack-nova | 06:06 | |
| *** dimtruck is now known as zz_dimtruck | 06:06 | |
| *** bnemec has quit IRC | 06:11 | |
| *** fandi__ has joined #openstack-nova | 06:11 | |
| *** code-R has joined #openstack-nova | 06:11 | |
| *** Jeffrey4l_ has quit IRC | 06:12 | |
| *** Jeffrey4l has joined #openstack-nova | 06:12 | |
| openstackgerrit | Teng Fei proposed openstack/nova: Pass the parameter of extra image properties to the create_snapshot_force function. https://review.openstack.org/404656 | 06:14 |
| openstackgerrit | Teng Fei proposed openstack/nova: Pass the parameter of extra image properties to the create_snapshot_force function. https://review.openstack.org/404656 | 06:16 |
| *** moshele has joined #openstack-nova | 06:17 | |
| *** Kevin_Zheng has quit IRC | 06:19 | |
| openstackgerrit | 赵凯 proposed openstack/nova: [network] nova with neutron can not inject ip address to vm https://review.openstack.org/354616 | 06:19 |
| *** yufei has joined #openstack-nova | 06:20 | |
| *** tlian has quit IRC | 06:20 | |
| *** gouthamr has joined #openstack-nova | 06:23 | |
| *** moshele has quit IRC | 06:24 | |
| *** kaisers_ has joined #openstack-nova | 06:26 | |
| openstackgerrit | Teng Fei proposed openstack/nova: Pass the parameter of extra image properties to create_snapshot_force(). https://review.openstack.org/404656 | 06:29 |
| *** bhagyashris has joined #openstack-nova | 06:31 | |
| *** openstackgerrit has quit IRC | 06:33 | |
| *** penick has quit IRC | 06:39 | |
| *** kaisers_ has quit IRC | 06:41 | |
| *** ratailor_ has joined #openstack-nova | 06:43 | |
| *** ratailor_ has quit IRC | 06:43 | |
| *** ratailor_ has joined #openstack-nova | 06:44 | |
| *** ratailor has quit IRC | 06:44 | |
| *** openstackgerrit has joined #openstack-nova | 06:47 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec https://review.openstack.org/397193 | 06:47 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/cells https://review.openstack.org/323287 | 06:47 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738 | 06:47 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436 | 06:47 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Remove mox in test_block_device_mapping_v1.py https://review.openstack.org/308746 | 06:47 |
| *** Cristina__ has quit IRC | 06:48 | |
| *** josecastroleon has joined #openstack-nova | 06:49 | |
| *** ratailor_ has quit IRC | 06:52 | |
| *** sreeram_v has quit IRC | 06:53 | |
| *** thorst_ has joined #openstack-nova | 06:54 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: add vlan column to virtual_interface table https://review.openstack.org/405948 | 06:55 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: adding vlan filed to virtual_interface https://review.openstack.org/405949 | 06:55 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: adding vlans field to NetworkInterfaceMetadata object https://review.openstack.org/405950 | 06:55 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Adding vlans filed to Device tagging metadata https://review.openstack.org/405951 | 06:55 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: neutron: virtual interfaces should be created for preexisting ports https://review.openstack.org/405952 | 06:55 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: neutron: update vlan in virtual_interface on port update https://review.openstack.org/405953 | 06:55 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: expose virtual interfaces with vlans to metadata https://review.openstack.org/405954 | 06:55 |
| *** takashin has left #openstack-nova | 06:55 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Delete duplicate check when live-migrating https://review.openstack.org/337056 | 07:00 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Improve _is_booted_from_volume implementation https://review.openstack.org/382024 | 07:00 |
| *** thorst_ has quit IRC | 07:01 | |
| *** lpetrut has joined #openstack-nova | 07:03 | |
| *** andreas_s has joined #openstack-nova | 07:07 | |
| *** sridharg has joined #openstack-nova | 07:08 | |
| openstackgerrit | Ankit Agarwal proposed openstack/nova: Adding other instance action logs in Horizon https://review.openstack.org/405960 | 07:08 |
| *** code-R_ has joined #openstack-nova | 07:09 | |
| *** faizy has joined #openstack-nova | 07:10 | |
| *** code-R has quit IRC | 07:10 | |
| *** jaosorior has quit IRC | 07:10 | |
| *** jaosorior has joined #openstack-nova | 07:11 | |
| *** ihrachys has joined #openstack-nova | 07:11 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Change condition of root disk snapshot handling https://review.openstack.org/382030 | 07:13 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Remove backward compatibility checks for disk snapshots https://review.openstack.org/382031 | 07:13 |
| *** vladikr has quit IRC | 07:16 | |
| *** voelzmo has joined #openstack-nova | 07:17 | |
| *** jamielennox is now known as jamielennox|away | 07:18 | |
| *** voelzmo has quit IRC | 07:23 | |
| *** ducttape_ has joined #openstack-nova | 07:27 | |
| openstackgerrit | Ankit Agarwal proposed openstack/nova: Adding other instance action logs in Horizon https://review.openstack.org/405960 | 07:27 |
| openstackgerrit | Ankit Agarwal proposed openstack/nova: Adding other instance action logs in Horizon https://review.openstack.org/405960 | 07:31 |
| *** ducttape_ has quit IRC | 07:32 | |
| *** ratailor has joined #openstack-nova | 07:33 | |
| *** armax has quit IRC | 07:33 | |
| *** voelzmo has joined #openstack-nova | 07:36 | |
| *** GB21 has joined #openstack-nova | 07:36 | |
| *** kaisers_ has joined #openstack-nova | 07:38 | |
| *** ratailor has quit IRC | 07:39 | |
| *** rcernin has joined #openstack-nova | 07:40 | |
| *** nherciu has joined #openstack-nova | 07:43 | |
| *** markus_z has joined #openstack-nova | 07:43 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Simplify internal usage of get_instance_disk_info https://review.openstack.org/334901 | 07:44 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Remove unused disk_info parameter https://review.openstack.org/382025 | 07:44 |
| *** jamielennox|away is now known as jamielennox | 07:44 | |
| *** ratailor has joined #openstack-nova | 07:49 | |
| *** josecastroleon has quit IRC | 07:52 | |
| *** CristinaPauna has joined #openstack-nova | 07:54 | |
| *** GB21 has quit IRC | 07:54 | |
| *** josecastroleon has joined #openstack-nova | 07:55 | |
| *** mkoderer has joined #openstack-nova | 07:56 | |
| *** hfu has quit IRC | 07:56 | |
| *** ralonsoh has joined #openstack-nova | 07:57 | |
| *** gongysh has quit IRC | 07:58 | |
| *** owalsh_ has joined #openstack-nova | 07:58 | |
| *** sahid has joined #openstack-nova | 07:59 | |
| *** thorst_ has joined #openstack-nova | 07:59 | |
| *** owalsh has quit IRC | 08:00 | |
| *** openstackgerrit has quit IRC | 08:03 | |
| *** lpetrut has quit IRC | 08:05 | |
| *** thorst_ has quit IRC | 08:06 | |
| *** GB21 has joined #openstack-nova | 08:07 | |
| *** ihrachys has quit IRC | 08:10 | |
| *** matrohon has joined #openstack-nova | 08:12 | |
| *** ratailor has quit IRC | 08:12 | |
| *** ratailor has joined #openstack-nova | 08:13 | |
| *** lpetrut has joined #openstack-nova | 08:14 | |
| *** amoralej|off is now known as amoralej | 08:15 | |
| *** openstackgerrit has joined #openstack-nova | 08:18 | |
| openstackgerrit | gengchc2 proposed openstack/nova-specs: Fix a typo https://review.openstack.org/405986 | 08:18 |
| *** ratailor_ has joined #openstack-nova | 08:26 | |
| *** lpetrut has quit IRC | 08:26 | |
| *** ratailor has quit IRC | 08:29 | |
| *** sudipto has quit IRC | 08:30 | |
| *** sudipto_ has quit IRC | 08:30 | |
| *** sudipto has joined #openstack-nova | 08:33 | |
| *** unicell1 has joined #openstack-nova | 08:42 | |
| *** unicell has quit IRC | 08:42 | |
| *** ratailor_ has quit IRC | 08:43 | |
| *** ducnc has quit IRC | 08:43 | |
| *** betherly has quit IRC | 08:51 | |
| *** lpetrut has joined #openstack-nova | 08:53 | |
| *** GB21 has quit IRC | 08:56 | |
| *** lpetrut has quit IRC | 08:58 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** aarefiev_afk is now known as aarefiev | 09:00 | |
| *** ratailor_ has joined #openstack-nova | 09:00 | |
| *** code-R has joined #openstack-nova | 09:01 | |
| *** zzzeek has joined #openstack-nova | 09:01 | |
| *** yuntongjin has quit IRC | 09:02 | |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** code-R_ has quit IRC | 09:04 | |
| *** thorst_ has joined #openstack-nova | 09:05 | |
| *** BobBall has quit IRC | 09:08 | |
| *** thorst_ has quit IRC | 09:11 | |
| *** openstackgerrit has joined #openstack-nova | 09:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 09:12 |
| *** matrohon has quit IRC | 09:14 | |
| *** hieulq has joined #openstack-nova | 09:15 | |
| *** zhangjl has left #openstack-nova | 09:15 | |
| *** GB21 has joined #openstack-nova | 09:19 | |
| *** yassine has joined #openstack-nova | 09:23 | |
| *** yassine is now known as Guest51607 | 09:23 | |
| *** ducttape_ has joined #openstack-nova | 09:28 | |
| *** voelzmo has quit IRC | 09:29 | |
| *** aloga has quit IRC | 09:29 | |
| *** aloga has joined #openstack-nova | 09:30 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Change condition of root disk snapshot handling https://review.openstack.org/382030 | 09:30 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Remove backward compatibility checks for disk snapshots https://review.openstack.org/382031 | 09:30 |
| *** tojuvone has quit IRC | 09:30 | |
| sfinucan | mriedem: fine by me :) | 09:31 |
| *** voelzmo has joined #openstack-nova | 09:32 | |
| *** ljxiash has quit IRC | 09:33 | |
| tdurakov | morning nova | 09:33 |
| *** ducttape_ has quit IRC | 09:33 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Remove unused disk_info parameter https://review.openstack.org/382025 | 09:33 |
| *** ljxiash has joined #openstack-nova | 09:34 | |
| *** matrohon has joined #openstack-nova | 09:34 | |
| *** lucas-afk is now known as lucasagomes | 09:35 | |
| *** ljxiash has quit IRC | 09:35 | |
| *** ljxiash has joined #openstack-nova | 09:35 | |
| *** ljxiash has quit IRC | 09:36 | |
| *** ljxiash has joined #openstack-nova | 09:36 | |
| *** DinaBelova has quit IRC | 09:37 | |
| *** rwmjones has quit IRC | 09:38 | |
| *** zeroDivisible has quit IRC | 09:38 | |
| *** rwmjones has joined #openstack-nova | 09:39 | |
| *** DinaBelova has joined #openstack-nova | 09:39 | |
| *** ljxiash has quit IRC | 09:40 | |
| *** tojuvone has joined #openstack-nova | 09:41 | |
| *** zeroDivisible has joined #openstack-nova | 09:41 | |
| *** fandi__ has quit IRC | 09:43 | |
| *** GB21 has quit IRC | 09:45 | |
| kashyap | mdbooth: Was wondering about why Jenkins is -2 here -- https://review.openstack.org/#/c/401366/ | 09:46 |
| *** voelzmo has quit IRC | 09:47 | |
| mdbooth | Doesn't it always -2 if it fails in the gate (rather than check)? | 09:47 |
| kashyap | Yeah | 09:47 |
| kashyap | So I (re-)learn | 09:47 |
| mdbooth | http://logs.openstack.org/66/401366/4/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/3deacc5/console.html | 09:47 |
| mdbooth | no valid host | 09:48 |
| mdbooth | So likely transient | 09:48 |
| kashyap | So a simple 'recheck' should get it over | 09:48 |
| *** rcernin has quit IRC | 09:49 | |
| *** rcernin has joined #openstack-nova | 09:50 | |
| *** code-R has quit IRC | 09:54 | |
| gmann | alex_xu: oomichi johnthetubaguy : do we need spec for this? - https://review.openstack.org/#/c/397034/4 | 09:54 |
| *** code-R has joined #openstack-nova | 09:54 | |
| gmann | alex_xu: oomichi johnthetubaguy even not sure we record lock/unlock action - https://review.openstack.org/#/c/396510/8 | 09:54 |
| gmann | *we should | 09:55 |
| *** tovin07 has quit IRC | 09:58 | |
| *** GB21 has joined #openstack-nova | 09:58 | |
| *** mkoderer has quit IRC | 10:03 | |
| *** markus_z has quit IRC | 10:03 | |
| *** ihrachys has joined #openstack-nova | 10:04 | |
| *** claudiub|2 has joined #openstack-nova | 10:04 | |
| *** yingjun has quit IRC | 10:05 | |
| *** fandi has joined #openstack-nova | 10:09 | |
| *** thorst_ has joined #openstack-nova | 10:09 | |
| *** ljxiash has joined #openstack-nova | 10:15 | |
| *** lpetrut has joined #openstack-nova | 10:15 | |
| *** sam_nowitzki has quit IRC | 10:17 | |
| *** thorst_ has quit IRC | 10:17 | |
| *** markus_z has joined #openstack-nova | 10:17 | |
| *** claudiub has joined #openstack-nova | 10:18 | |
| *** matrohon has quit IRC | 10:18 | |
| *** ljxiash has quit IRC | 10:19 | |
| *** fandi has quit IRC | 10:19 | |
| *** voelzmo has joined #openstack-nova | 10:19 | |
| *** yamahata has quit IRC | 10:19 | |
| *** kaisers_ has quit IRC | 10:20 | |
| *** claudiub|2 has quit IRC | 10:21 | |
| *** code-R has quit IRC | 10:21 | |
| *** claudiub|2 has joined #openstack-nova | 10:22 | |
| *** code-R has joined #openstack-nova | 10:22 | |
| *** mdovgal has left #openstack-nova | 10:24 | |
| *** code-R_ has joined #openstack-nova | 10:25 | |
| *** xiaoyang has quit IRC | 10:25 | |
| *** claudiub has quit IRC | 10:25 | |
| *** sreeram_v has joined #openstack-nova | 10:27 | |
| *** tesseract has joined #openstack-nova | 10:27 | |
| mdbooth | Pop quiz: | 10:27 |
| mdbooth | block_device_mapping = driver.block_device_info_get_mapping( | 10:27 |
| mdbooth | block_device_info) | 10:27 |
| mdbooth | volume_devices = set() | 10:27 |
| mdbooth | for vol in block_device_mapping: | 10:27 |
| mdbooth | disk_dev = vol['mount_device'].rpartition("/")[2] | 10:27 |
| mdbooth | volume_devices.add(disk_dev) | 10:27 |
| mdbooth | What does volume_devices contains? | 10:27 |
| mdbooth | You have 1 minute. | 10:27 |
| *** sudipto has quit IRC | 10:27 | |
| *** code-R has quit IRC | 10:27 | |
| *** tesseract is now known as Guest79986 | 10:27 | |
| * mdbooth checks he didn't write that code | 10:28 | |
| *** slagle has quit IRC | 10:28 | |
| *** jschlueter has quit IRC | 10:28 | |
| *** jschlueter has joined #openstack-nova | 10:29 | |
| *** slagle has joined #openstack-nova | 10:29 | |
| *** sreeram_v has quit IRC | 10:31 | |
| kashyap | mdbooth: Currently mounted devices? (/me is mostly wrong) | 10:31 |
| mdbooth | kashyap: I mean *exactly* what does it contain? | 10:31 |
| mdbooth | I don't know the answer, btw | 10:32 |
| mdbooth | A comment would have been nice... | 10:32 |
| *** code-R_ has quit IRC | 10:32 | |
| * mdbooth grumbles | 10:32 | |
| *** code-R has joined #openstack-nova | 10:32 | |
| * kashyap nods; don't know the answer without examining source, and other tools like `git blame` and co | 10:32 | |
| *** GnomeKing has joined #openstack-nova | 10:36 | |
| *** paul-carlton2 has joined #openstack-nova | 10:39 | |
| *** sudipto has joined #openstack-nova | 10:44 | |
| *** code-R_ has joined #openstack-nova | 10:48 | |
| lyarwood | mdbooth: isn't that the vdx,hdx target device name given by libvirt | 10:49 |
| *** diga has quit IRC | 10:49 | |
| mdbooth | lyarwood: No. | 10:49 |
| mdbooth | That's what it's being compared to, though. | 10:49 |
| mdbooth | I'm not saying it's incorrect, btw. | 10:50 |
| mdbooth | It's just obtuse and uncommented. | 10:50 |
| sfinucan | sahid: You around? | 10:50 |
| *** code-R has quit IRC | 10:51 | |
| *** sridharg has quit IRC | 10:51 | |
| sahid | sfinucan: hello, yes ? | 10:51 |
| *** hoangcx has quit IRC | 10:52 | |
| sfinucan | sahid: Morning :) I've still got concerns about https://review.openstack.org/#/c/385023/5..7/nova/virt/hardware.py | 10:52 |
| *** tbachman has joined #openstack-nova | 10:52 | |
| sahid | sfinucan: yes, let me know ? | 10:52 |
| sfinucan | So I've looked at the other "NFV switches", and they all support both flavor extra specs and image meta data as config options | 10:53 |
| sfinucan | I know it wasn't in the spec but, IMO, that was an omission that we should be correcting now at implementation time | 10:53 |
| *** gszasz has joined #openstack-nova | 10:53 | |
| lyarwood | mdbooth: https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L230 that's it right? | 10:54 |
| sahid | sfinucan: i tried to explain to you, (bunch of time :) | 10:54 |
| sahid | it's not an omission | 10:54 |
| sahid | it just does not make sense to have it | 10:54 |
| mdbooth | lyarwood: Yep. So it comes from wherever that comes from. | 10:54 |
| lyarwood | mdbooth: yeah that's the target dev iirc | 10:54 |
| sfinucan | sahid: But I still don't get why :) | 10:54 |
| * lyarwood checks | 10:54 | |
| mdbooth | Nope | 10:54 |
| lyarwood | hm | 10:55 |
| mdbooth | BDM is created before libvirt domain | 10:55 |
| sfinucan | I mean, image metadata isn't a way to specify something about the image itself | 10:55 |
| sfinucan | i.e. the physical file | 10:55 |
| sahid | sfinucan: emulator threads is not realted to the guest os | 10:55 |
| mdbooth | This is created by compute manager somewhere and passed down to driver | 10:55 |
| sahid | it's related to the host | 10:55 |
| sfinucan | rather, it's a way to specify some property of the application that image provides, be it a simple OS or something more | 10:55 |
| mdbooth | Then driver may pass it to libvirt | 10:56 |
| sfinucan | sahid: Yes, but you could say the same thing about CPU pinning | 10:56 |
| sahid | no... | 10:56 |
| mdbooth | Which then may or may not honour it | 10:56 |
| sahid | cpu pinning is realted to the guest os | 10:56 |
| sahid | since we whant to have some application running on the guest dedicated to an entire pCPU | 10:56 |
| sfinucan | sahid: Aye, but the guest doesn't know that | 10:57 |
| sahid | sfinucan: it knows that | 10:57 |
| sfinucan | The only thing that knows you've got pinned CPUs is libvirt | 10:57 |
| sfinucan | ...and nova, of course :) | 10:57 |
| *** duonghq has quit IRC | 10:57 | |
| sfinucan | sahid: How so? Don't "pinned" CPUs look like any other CPUs in the guest itself | 10:58 |
| sahid | no, the applications running under the guest can be pinned on a vCPU wich is directly realted to a pCPU | 10:58 |
| sahid | that why it would make sense | 10:58 |
| sahid | for emulator threads is not realted to the guest os | 10:58 |
| sfinucan | aye, but you could do the exact same thing in an unpinned guest | 10:58 |
| sfinucan | i.e. run taskset | 10:58 |
| sfinucan | it wouldn't be much good to you, but it would work | 10:58 |
| sahid | sfinucan: we could do several things for sure | 10:58 |
| sahid | we could oslo rewrite nova in go or rush | 10:59 |
| sfinucan | huh? | 10:59 |
| sahid | olso | 10:59 |
| sahid | also | 10:59 |
| sahid | :) | 10:59 |
| *** cdent has joined #openstack-nova | 11:00 | |
| sfinucan | I'm lost :S | 11:00 |
| sahid | i'm trying to explain to you there is no sense to have that property configured from the image, and you say we could, ...so yes we could but that does not mean we should | 11:00 |
| lyarwood | mdbooth: sorry missed your last, so doesn't libvirt just use the bdm.device_name as the target dev name? I was sure it did. | 11:00 |
| mdbooth | It might, but the causal relationship is the other way round | 11:01 |
| sahid | sfinucan: if you want it on the image property, the best you can do is to provide a spec with an exact use case | 11:01 |
| mdbooth | And iirc it doesn't have to honour it? | 11:01 |
| sahid | so i can consider it | 11:01 |
| lyarwood | mdbooth: internally within the instance it doesnt | 11:01 |
| sfinucan | ...and I'm saying there is, but clearly not as well as I should be :/ | 11:01 |
| sahid | so far, and for the use case it does not make sense | 11:01 |
| lyarwood | mdbooth: but it still uses it to lookup the device during blockjobs etc | 11:01 |
| sfinucan | OK, let me make an amendment to your spec and see what you think. Maybe when it's in spec form the point I'm trying to make will make more sense | 11:02 |
| mdbooth | So it's a unique name? | 11:02 |
| sahid | sfinucan: i would prefer you provide a new spec for pike | 11:02 |
| sahid | matt sent a email, it's open | 11:02 |
| mdbooth | Anyway, my point stands. That's way too much required context for no comment. | 11:02 |
| lyarwood | mdbooth: yeah I think it is and yeah I fully agree | 11:03 |
| *** hieulq has quit IRC | 11:04 | |
| sahid | sfinucan: last thing, it would be nice if you can remove your -1 since they do not ask for addressing something, they are just suggestions | 11:05 |
| sahid | and i replied to them :) | 11:05 |
| sfinucan | sahid: They're not suggestions though - they're valid concerns with the spec that I think still stand | 11:05 |
| sahid | sfinucan: which one ? | 11:05 |
| sfinucan | The one we've just be talking about (image metadata), for one | 11:06 |
| sfinucan | My -1 won't block cores from merging the code if I'm wrong | 11:06 |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Use os-xenapi lib for nova https://review.openstack.org/406059 | 11:07 |
| sfinucan | ...but they will indicate that I, as an individual, am not 100% happy with the code as is | 11:08 |
| sahid | sfinucan: ok i understand, it's more than a suggestion, you want it | 11:10 |
| sfinucan | sahid: 'actly :) | 11:11 |
| *** markus_z has quit IRC | 11:12 | |
| *** sdague has joined #openstack-nova | 11:12 | |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: Perform build list of standard classes once https://review.openstack.org/406062 | 11:14 |
| *** namnh has quit IRC | 11:15 | |
| *** thorst_ has joined #openstack-nova | 11:15 | |
| *** sudipto has quit IRC | 11:17 | |
| *** kaisers_ has joined #openstack-nova | 11:17 | |
| *** tbachman has quit IRC | 11:19 | |
| *** matrohon has joined #openstack-nova | 11:20 | |
| *** thorst_ has quit IRC | 11:22 | |
| *** pd_away has joined #openstack-nova | 11:23 | |
| *** pd_away is now known as pordirect | 11:24 | |
| *** yufei has quit IRC | 11:27 | |
| *** siva_krish has joined #openstack-nova | 11:28 | |
| *** tbachman has joined #openstack-nova | 11:30 | |
| *** pordirect has quit IRC | 11:31 | |
| *** harbor has joined #openstack-nova | 11:32 | |
| *** harbor is now known as pordirect | 11:32 | |
| *** pordirect is now known as portdirect_ | 11:32 | |
| gcb | johnthetubaguy, I just rebased https://review.openstack.org/#/c/319532/ | 11:35 |
| *** nicolasbock has joined #openstack-nova | 11:36 | |
| sdague | mdbooth: looks like the reset of the image cache series needs a rebase now because of merge conflicts. If you do that early on today, I'll put it on my queue to review and see how many more of those we can get in | 11:36 |
| mdbooth | sdague: Thanks for that push yesterday! | 11:36 |
| sdague | mdbooth: no prob, thanks for making this code less insane | 11:36 |
| mdbooth | I've been doing some other reviews this morning, just rebasing now. | 11:36 |
| mdbooth | They start getting more interesting in a few patches time, btw | 11:37 |
| sdague | cool, I'm still cafinating, so it will be a couple hours till I get to it. Just wanted to make sure it was on your radar. | 11:37 |
| *** portdirect_ is now known as portdirect | 11:38 | |
| sdague | ok, good to know, I'll ask questions when it gets interesting if I don't get it :) | 11:38 |
| mdbooth | I've agreed with ftersin that the one currently at the head of the queue requires a reno, btw | 11:38 |
| mdbooth | Because it breaks something. | 11:38 |
| *** gcb has quit IRC | 11:41 | |
| sfinucan | johnthetubaguy: I rebased this patch, but lost the +W in the process. Any chance you could re +W it, please? https://review.openstack.org/#/c/385074/7 | 11:42 |
| *** Jeffrey4l has quit IRC | 11:44 | |
| kashyap | Before I waste CI resources, just triple-confirming, for XenServerCI (/me expects Bob Ball to look up), it is this, right?: xenserver: recheck | 11:45 |
| *** rfolco has quit IRC | 11:45 | |
| kashyap | `grep`ing through my IRC logs say so | 11:46 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Live migration of instances with encrypted volumes https://review.openstack.org/389608 | 11:46 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs: Add support for image metadata https://review.openstack.org/406081 | 11:46 |
| *** tbachman has quit IRC | 11:49 | |
| sfinucan | sahid: ^ hopefully that's a little more coherent. Corrected the property name while I was at it | 11:50 |
| *** GB21 has quit IRC | 11:50 | |
| sdague | alex_xu: I just went through - https://review.openstack.org/#/c/393205 in detail, it looks close, but there are a couple of things in there that feel like we should tweak. Let me know what you think. | 11:54 |
| sdague | mdbooth: roger that | 11:54 |
| *** GnomeKing is now known as BobBall | 11:54 | |
| * BobBall looks up to kashyap | 11:54 | |
| kashyap | BobBall: :-) | 11:54 |
| BobBall | kashyap: It should say in the message from the CI | 11:54 |
| BobBall | I believe it is xenserver: recheck yes | 11:54 |
| kashyap | BobBall: Duh, yes. Rapid switching between tasks | 11:55 |
| kashyap | BobBall: Thanks | 11:55 |
| BobBall | I'm looking for a failure to confirm | 11:55 |
| BobBall | yeah - confirmed - xenserver: recheck | 11:56 |
| *** Jeffrey4l has joined #openstack-nova | 11:58 | |
| *** palexster has quit IRC | 11:58 | |
| *** faizy has quit IRC | 11:59 | |
| sdague | kashyap: for the libguestfs bug, I think it's irrelevant though right, as that's only used in libvirt code | 12:00 |
| kashyap | sdague: But they're voting, CI, right? | 12:00 |
| kashyap | sdague: Alright, I'll first let Jenkins give a +2 | 12:01 |
| *** gcb has joined #openstack-nova | 12:01 | |
| *** siva_krish has quit IRC | 12:01 | |
| kashyap | [BTW, if you folks are waiting for unit tests to run or DevStack to finish...what better way to spend time than checking out, it's up & running :-) - http://www.qemu-advent-calendar.org/] | 12:01 |
| *** palexster has joined #openstack-nova | 12:03 | |
| sdague | kashyap: sure, but I'm assuming they are broken for other reasons atm | 12:04 |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova: Handle ImageNotFound exception during instance backup https://review.openstack.org/388519 | 12:04 |
| *** GB21 has joined #openstack-nova | 12:05 | |
| * kashyap nods | 12:05 | |
| *** ratailor_ has quit IRC | 12:07 | |
| *** derekh has joined #openstack-nova | 12:10 | |
| *** portdirect has quit IRC | 12:11 | |
| *** rfolco has joined #openstack-nova | 12:11 | |
| *** rcernin has quit IRC | 12:12 | |
| *** rcernin has joined #openstack-nova | 12:15 | |
| *** bvanhav__ has joined #openstack-nova | 12:16 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384 | 12:16 |
| *** thorst_ has joined #openstack-nova | 12:18 | |
| sfinucan | dansmith: Out of curiousity, would it be possible to remove clauses from obj_make_compatible after a certain number of releases? | 12:20 |
| sfinucan | i.e. if we bumped an object version to 1.3 in release N, could we remove the code to handle pre-1.3 objects in N+2 or so? | 12:21 |
| sfinucan | I'm just curious, as I'm not sure how long those objects would be found "on the wire" or whatever | 12:21 |
| *** portdirect has joined #openstack-nova | 12:22 | |
| *** cdent has quit IRC | 12:24 | |
| *** thorst_ has quit IRC | 12:26 | |
| *** GB21 has quit IRC | 12:29 | |
| *** sreeram_v has joined #openstack-nova | 12:31 | |
| *** fragatina has quit IRC | 12:32 | |
| *** moshele has joined #openstack-nova | 12:32 | |
| *** fragatina has joined #openstack-nova | 12:32 | |
| *** pradk has joined #openstack-nova | 12:33 | |
| openstackgerrit | jichenjc proposed openstack/nova: UnsupportedBDMVolumeAuthMethod is not used https://review.openstack.org/406098 | 12:37 |
| *** nherciu_ has joined #openstack-nova | 12:40 | |
| *** thorst_ has joined #openstack-nova | 12:43 | |
| *** nherciu has quit IRC | 12:44 | |
| *** lucasagomes is now known as lucas-hungry | 12:47 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_simple_tenant_usage to Python 3 https://review.openstack.org/406103 | 12:50 |
| *** tbachman has joined #openstack-nova | 12:54 | |
| *** hamzy has joined #openstack-nova | 12:55 | |
| openstackgerrit | Roman Dobosz proposed openstack/nova: placement: Perform build list of standard classes once https://review.openstack.org/406062 | 12:55 |
| *** psachin has quit IRC | 12:57 | |
| *** tbachman_ has joined #openstack-nova | 12:58 | |
| *** gcb has quit IRC | 12:58 | |
| *** tbachman has quit IRC | 12:59 | |
| *** tbachman_ is now known as tbachman | 12:59 | |
| openstackgerrit | jichenjc proposed openstack/nova: Catch VolumeEncryptionNotSupported during spawn https://review.openstack.org/406109 | 13:00 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 13:01 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Fix backwards compatibility for InstanceNUMACell https://review.openstack.org/396184 | 13:02 |
| *** liverpooler has joined #openstack-nova | 13:03 | |
| *** jaypipes has joined #openstack-nova | 13:04 | |
| *** ihrachys has quit IRC | 13:05 | |
| *** ihrachys has joined #openstack-nova | 13:06 | |
| *** ihrachys has quit IRC | 13:06 | |
| *** sreeram_v has quit IRC | 13:06 | |
| *** rcernin has quit IRC | 13:06 | |
| *** sreeram_v has joined #openstack-nova | 13:07 | |
| *** ducttape_ has joined #openstack-nova | 13:07 | |
| *** ihrachys has joined #openstack-nova | 13:07 | |
| *** rcernin has joined #openstack-nova | 13:08 | |
| jaypipes | mdbooth: morning! hope you woke up today with a smile having seen a good chunk of your refactor series land :) | 13:10 |
| diana_clarke | I was thinking the same thing as I watched all the merge emails come in :) | 13:11 |
| *** sreeram_v has quit IRC | 13:11 | |
| syjulian | dansmith, jaypipes: hey guys, can you review this change on nova ovo? https://review.openstack.org/#/c/391546/ | 13:12 |
| jaypipes | diana_clarke: indeed. it's funny, I have an email filter in thunderbird that marks threads orange when it sees "and it was merged" in the email. it's nice sometimes to come in and see a whole slew of orange threads in the morning :) | 13:13 |
| diana_clarke | :) | 13:13 |
| *** mdnadeem has quit IRC | 13:15 | |
| *** links has quit IRC | 13:17 | |
| *** lamt has joined #openstack-nova | 13:17 | |
| *** dave-mccowan has joined #openstack-nova | 13:21 | |
| jaypipes | syjulian: done. | 13:21 |
| jaypipes | sdague: if you're up, https://review.openstack.org/#/c/391546 is an easy one. | 13:21 |
| jaypipes | lbeliveau: looking at your patch now... | 13:21 |
| *** edleafe is now known as figleaf | 13:22 | |
| syjulian | jaypipes: thanks jay! | 13:22 |
| *** jaypipes is now known as leakypipes | 13:22 | |
| leakypipes | syjulian: any time, np :) | 13:22 |
| *** moshele has quit IRC | 13:23 | |
| *** jaosorior is now known as jaosorior_brb | 13:24 | |
| openstackgerrit | jichenjc proposed openstack/nova: Catch ImageNotAuthorized during boot instance https://review.openstack.org/406118 | 13:28 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 13:29 |
| mdbooth | leakypipes: I did, thanks :) | 13:29 |
| *** cdent has joined #openstack-nova | 13:30 | |
| *** krtaylor has quit IRC | 13:31 | |
| *** dave-mcc_ has joined #openstack-nova | 13:32 | |
| *** andreas_s has quit IRC | 13:33 | |
| *** portdirect is now known as portdirect_away | 13:34 | |
| *** Dinesh_Bhor has quit IRC | 13:34 | |
| *** dave-mccowan has quit IRC | 13:35 | |
| *** portdirect_away has quit IRC | 13:35 | |
| *** eharney has joined #openstack-nova | 13:36 | |
| *** portdirect_away has joined #openstack-nova | 13:37 | |
| *** code-R_ has quit IRC | 13:37 | |
| *** portdirect_away has quit IRC | 13:38 | |
| *** portdirect_away has joined #openstack-nova | 13:39 | |
| *** vladikr has joined #openstack-nova | 13:41 | |
| *** lbeliveau has quit IRC | 13:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 13:44 |
| *** lbeliveau has joined #openstack-nova | 13:46 | |
| *** bhagyashris has quit IRC | 13:50 | |
| *** xyang1 has joined #openstack-nova | 13:51 | |
| *** baoli has joined #openstack-nova | 13:55 | |
| *** bvanhav__ has quit IRC | 13:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 13:57 |
| *** baoli has quit IRC | 13:57 | |
| *** baoli has joined #openstack-nova | 13:57 | |
| *** pradk has quit IRC | 13:58 | |
| *** danpawlik has joined #openstack-nova | 13:59 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Enabling detach root volume for shelved_offloaded https://review.openstack.org/326899 | 14:00 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Attach/detach root volume for shelved_offloaded https://review.openstack.org/317446 | 14:00 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Adding device_present field to bdm table https://review.openstack.org/268053 | 14:00 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/405895 | 14:00 |
| *** lucas-hungry is now known as lucasagomes | 14:01 | |
| *** paw has quit IRC | 14:01 | |
| *** jamesdenton has joined #openstack-nova | 14:02 | |
| *** ljxiash has joined #openstack-nova | 14:03 | |
| *** markus_z has joined #openstack-nova | 14:03 | |
| *** tlian has joined #openstack-nova | 14:04 | |
| *** sreeram_v has joined #openstack-nova | 14:06 | |
| *** code-R has joined #openstack-nova | 14:07 | |
| mdbooth | I remember how this one goes: | 14:07 |
| mdbooth | (Pdb) configdrive.required_by(instance) | 14:08 |
| mdbooth | u'False' | 14:08 |
| *** READ10 has joined #openstack-nova | 14:08 | |
| *** code-R_ has joined #openstack-nova | 14:09 | |
| *** ljxiash has quit IRC | 14:09 | |
| *** bnemec has joined #openstack-nova | 14:09 | |
| gibi | dansmith: hi! I have a question about lazy loaded flavor.projects field and performance in https://review.openstack.org/#/c/398171/9/nova/objects/flavor.py@580 Could you give your opinion if it is a real issue or not? | 14:11 |
| *** code-R has quit IRC | 14:11 | |
| sdague | leakypipes: ok, I'm actually not that familiar with this part of objects. When is get_schema used? | 14:12 |
| sdague | also... it seems wrong that those regex values don't include $ end tokens, because all that re.match does is ensures the strings start with PATERN, but they could have a ☃ at the end, and they'd pass the coerce | 14:13 |
| *** hfu has joined #openstack-nova | 14:14 | |
| *** bvanhav__ has joined #openstack-nova | 14:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 14:15 |
| openstackgerrit | Chris Dent proposed openstack/nova: Some improvement to the process doc https://review.openstack.org/404991 | 14:17 |
| *** gcb has joined #openstack-nova | 14:18 | |
| *** tblakes has joined #openstack-nova | 14:18 | |
| *** liverpooler has quit IRC | 14:19 | |
| *** yingjun has joined #openstack-nova | 14:22 | |
| openstackgerrit | zhaolihui proposed openstack/nova: Fix keypair quota error https://review.openstack.org/403529 | 14:22 |
| *** bkopilov_ has joined #openstack-nova | 14:23 | |
| *** ducttape_ has quit IRC | 14:24 | |
| *** bkopilov has quit IRC | 14:25 | |
| *** syjulian has left #openstack-nova | 14:27 | |
| *** mdrabe has joined #openstack-nova | 14:28 | |
| *** jamesdenton has quit IRC | 14:28 | |
| dansmith | sfinucan: by the rules, it should be that we remove them when we do the major version bumps, like I did for instance | 14:30 |
| dansmith | gibi: yeah, in a sec | 14:30 |
| *** mriedem has joined #openstack-nova | 14:30 | |
| mriedem | howdy | 14:31 |
| *** yingjun has quit IRC | 14:32 | |
| *** yingjun has joined #openstack-nova | 14:32 | |
| *** syjulian has joined #openstack-nova | 14:33 | |
| mriedem | artom: oh wow https://review.openstack.org/#/c/305120/ | 14:33 |
| *** rcernin has quit IRC | 14:33 | |
| *** jheroux has joined #openstack-nova | 14:35 | |
| *** hfu has quit IRC | 14:36 | |
| *** yingjun has quit IRC | 14:36 | |
| *** amoralej is now known as amoralej|lunch | 14:36 | |
| *** yingjun has joined #openstack-nova | 14:36 | |
| *** liverpooler has joined #openstack-nova | 14:38 | |
| *** code-R_ has quit IRC | 14:40 | |
| *** sreeram_v has quit IRC | 14:40 | |
| *** code-R has joined #openstack-nova | 14:40 | |
| artom | mriedem, oh God! | 14:41 |
| artom | Fuck it, it's 9:40 I'm getting piss drunk. | 14:41 |
| *** rcernin has joined #openstack-nova | 14:41 | |
| mriedem | well, it is friday | 14:41 |
| mriedem | i think that's allowed | 14:41 |
| artom | Thanks for pushing it through! | 14:41 |
| mriedem | i had very little to do with it | 14:42 |
| artom | mriedem, you did that last small change. | 14:42 |
| artom | I kept putting it off because I thought I'd have to split the test as well | 14:42 |
| dansmith | wow, that was like 100k changes ago | 14:42 |
| artom | And because I had to rebase my new test for tagged attach on top of it | 14:43 |
| artom | dansmith, I think there are still a few 29xxx changes up, no? | 14:43 |
| dansmith | I'm sure | 14:43 |
| *** jamielennox is now known as jamielennox|away | 14:45 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the error message for failed RC deletion https://review.openstack.org/406149 | 14:46 |
| mriedem | i probably inadvertantly caused the last -1 because i consolidated that volume create/attach code into helpers in the base class, which is why jordan wanted to use that | 14:46 |
| mriedem | so making that change was pretty simple for me | 14:46 |
| mriedem | anyway, high fives all around | 14:46 |
| mriedem | the neutron full ssh job is in nova's experimental queue already too | 14:47 |
| mriedem | so we can run that test on nova changes | 14:47 |
| *** gouthamr_ has joined #openstack-nova | 14:48 | |
| *** edmondsw_ has joined #openstack-nova | 14:49 | |
| *** eharney has quit IRC | 14:49 | |
| *** gouthamr has quit IRC | 14:49 | |
| *** edmondsw_ has quit IRC | 14:50 | |
| dansmith | gibi: see if my response makes sense | 14:53 |
| dansmith | gibi: I'm only 62% of the way through cup #1 | 14:53 |
| *** mtanino has joined #openstack-nova | 14:54 | |
| *** paw has joined #openstack-nova | 14:54 | |
| mriedem | that's an oddly specific percentage superdan | 14:55 |
| *** rcernin has quit IRC | 14:55 | |
| *** ducttape_ has joined #openstack-nova | 14:56 | |
| mriedem | need a +W on this https://review.openstack.org/#/c/391918/ | 14:56 |
| *** yingjun has quit IRC | 14:56 | |
| mriedem | resource class delete | 14:56 |
| *** yingjun has joined #openstack-nova | 14:57 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Create hyperv fake images under proper directory https://review.openstack.org/406163 | 14:57 |
| lbeliveau | ^ I found this "fake/" directory showing up on "git status" to be annoying | 14:57 |
| mriedem | leakypipes: cdent: figleaf: these seem pretty simple to resolve and then i'm +2, someone want to touch those up quick? https://review.openstack.org/#/c/400448/5/nova/objects/resource_provider.py | 14:57 |
| *** tblakes has quit IRC | 14:57 | |
| *** burt has joined #openstack-nova | 14:58 | |
| *** rcernin has joined #openstack-nova | 14:58 | |
| syjulian | sdague: hi sdague, for the get_schema, they're called on the properties of an object when to_json_schema is called. the plan is to use it on notifications so users can see the schema | 14:58 |
| *** dave-mcc_ has quit IRC | 14:58 | |
| mriedem | lbeliveau: can we use tmpdir instead? | 14:58 |
| leakypipes | mriedem: yeah, will fix up shortly. sorry, was working on other patches all day yesterday. | 14:59 |
| lbeliveau | mriedem: let me fix that up | 14:59 |
| *** dansmith is now known as superdan | 14:59 | |
| bauzas | oh man, we're Friday | 14:59 |
| leakypipes | mriedem: currently typing up an ML post about the progress on Ironic and custom resource class integration | 14:59 |
| *** bauzas is now known as bauwser | 14:59 | |
| leakypipes | lbeliveau: go for it. | 14:59 |
| mriedem | leakypipes: he's talking about his change i think | 15:00 |
| mriedem | :) | 15:00 |
| leakypipes | oh! | 15:00 |
| leakypipes | sorry, lbeliveau | 15:00 |
| leakypipes | lbeliveau: and yeah, that fake directory i noticed popping up in the last week or two | 15:00 |
| mriedem | bauwser: this is relatively easy, has a +2 from me, https://review.openstack.org/#/c/391918/17 | 15:00 |
| mriedem | would like to see this series move along as it's mostly there | 15:01 |
| lbeliveau | leakypipes: yeah, found it pretty annoying | 15:01 |
| mdbooth | ftersin: I'm going to (temporarily, at least) drop the config drive change from the refactor series. I don't think it's strictly required. | 15:01 |
| bauwser | mriedem: right, I already looked at it, but forgot to click the button :) | 15:01 |
| figleaf | mriedem: leakypipes: so would you just drop that block that checks for id changes, and let o.vo raise the exception? | 15:01 |
| *** yingjun has quit IRC | 15:01 | |
| mriedem | figleaf: i would yeah | 15:01 |
| mriedem | you can't set a new value on a read-only ovo attr | 15:02 |
| figleaf | mriedem: leakypipes: ok, I can handle that | 15:02 |
| *** krtaylor has joined #openstack-nova | 15:02 | |
| figleaf | mriedem: what, you've never heard of wearing belts and suspenders? | 15:02 |
| mriedem | figleaf: if you're going to update the save patch, | 15:02 |
| *** amoralej|lunch is now known as amoralej | 15:03 | |
| mriedem | there are also some things in the rest api change that could be touched up https://review.openstack.org/#/c/391918/17 | 15:03 |
| *** esberglu has joined #openstack-nova | 15:03 | |
| mriedem | and then i'll review the tests on that one | 15:03 |
| *** rama_y has joined #openstack-nova | 15:03 | |
| *** jhesketh has quit IRC | 15:04 | |
| figleaf | mriedem: Roman was fixing the resouce_class_cache stuff. I already pushed a fix for the name in the RC delete error message | 15:04 |
| openstackgerrit | Thomas Bechtold proposed openstack/nova: Handle deleted instances when refreshing the info_cache https://review.openstack.org/398308 | 15:05 |
| mriedem | figleaf: yeah and i already +2ed that :) | 15:05 |
| *** eharney has joined #openstack-nova | 15:05 | |
| openstackgerrit | Roman Podoliaka proposed openstack/python-novaclient: Add profiling support to novaclient https://review.openstack.org/254699 | 15:05 |
| mriedem | i'm less worried about the cleanups as much as getting the functional pieces in so the RT changes can be using them | 15:05 |
| figleaf | mriedem: I just see a couple in resource_provider.py that I can address | 15:05 |
| *** ljxiash has joined #openstack-nova | 15:05 | |
| *** jaosorior_brb is now known as jaosorior | 15:06 | |
| mriedem | figleaf: which patch are we talking about? | 15:06 |
| mriedem | save()? | 15:06 |
| *** jhesketh has joined #openstack-nova | 15:06 | |
| mriedem | figleaf: 2 changes needed here https://review.openstack.org/#/c/400448/5/nova/objects/resource_provider.py | 15:06 |
| mriedem | 1. remove the 'id' in updates checking | 15:06 |
| mriedem | 2. catch DBDuplicateEntry and raise ResourceClassExists | 15:07 |
| figleaf | mriedem: ah, you had mentioned https://review.openstack.org/#/c/391918 - I'm already working on the save() stuff | 15:07 |
| *** yingjun has joined #openstack-nova | 15:07 | |
| mriedem | i mentioned destroy just for reviews from other cores...but anyway, who's on first btw? | 15:07 |
| mriedem | maybe let leakypipes know you're touching those changes up so you guys aren't racing | 15:08 |
| figleaf | HEY LEAKYPIPES I'M TOUCHING THE CHANGES IN https://review.openstack.org/#/c/400448 | 15:10 |
| mriedem | it's good to shout, you know he just had a birthday | 15:10 |
| figleaf | did we get him a cake or anything? | 15:11 |
| *** dave-mccowan has joined #openstack-nova | 15:11 | |
| sfinucan | dansmith: Ah, gotcha. I wasn't sure, so thanks for the info | 15:11 |
| *** jaosorior has quit IRC | 15:11 | |
| gibi | superdan: thanks for the comment on the lazy load. I still have a question for you | 15:12 |
| sfinucan | or superdan, rather | 15:12 |
| *** jaosorior has joined #openstack-nova | 15:12 | |
| mriedem | figleaf: only mikal gets cakes | 15:12 |
| gibi | superdan: the current patch will emit a flavor notification if the extra_specs field is changed | 15:12 |
| mriedem | and only superdan gets very large beers | 15:12 |
| gibi | superdan: but the notification payload contains the projects fields so it will force a lazy load | 15:13 |
| *** jaosorior has quit IRC | 15:13 | |
| *** jaosorior has joined #openstack-nova | 15:13 | |
| figleaf | mriedem: ok, I'll send him a gift certificate for some sushi. He can share with superdan | 15:13 |
| gibi | superdan: I don't know if this will become a performance issue | 15:13 |
| superdan | gibi: right, that was the original question right? | 15:14 |
| gibi | superdan: yes, more or less | 15:14 |
| superdan | gibi: I'm on a call at the moment, but if you'll be around in 45 minutes, we can chat about it without me being distracted | 15:14 |
| leakypipes | figleaf: ty sir | 15:14 |
| gibi | superdan: sure, let's chat after your meeting | 15:15 |
| figleaf | leakypipes: yw | 15:15 |
| openstackgerrit | Thomas Bechtold proposed openstack/nova: Handle deleted instances when refreshing the info_cache https://review.openstack.org/398308 | 15:20 |
| toabctl | mriedem, hey. could you have another look at https://review.openstack.org/#/c/398308/ ? | 15:21 |
| toabctl | the guy from the other commit (which was first) does not react on my comments (he/she is just reacting on the review comments) | 15:21 |
| toabctl | mriedem, not sure how to continue here. I think my commit is the better fix tbh | 15:22 |
| *** cleong has joined #openstack-nova | 15:22 | |
| mriedem | toabctl: ok i'll take a look, i was wondering where that was all going to end up too | 15:23 |
| mriedem | sahid: ^ did you have opinions on those competing changes? | 15:23 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: adds ResourceClass.save() https://review.openstack.org/400448 | 15:23 |
| figleaf | leakypipes: mriedem: ^^ fixes for save() | 15:23 |
| *** sam_nowitzki has joined #openstack-nova | 15:25 | |
| bauwser | mriedem: you know a bit of oslo lockutils ? | 15:26 |
| toabctl | BobBall, hey. could you have a look at https://review.openstack.org/#/c/402318/ , please? | 15:26 |
| mriedem | bauwser: ummm...i actually thought threading.Lock was fine there in an earlier change, but someone asked jay to change to lockutils | 15:27 |
| mriedem | i think lockutils uses threading.Lock under the covers but i didn't actually dig into it | 15:27 |
| leakypipes | mriedem: johnthetubaguy | 15:27 |
| leakypipes | mriedem: yes, it does | 15:27 |
| bauwser | mriedem: AFAIK, that depends on the external parameter | 15:27 |
| mriedem | cool | 15:27 |
| sahid | mriedem: I think in https://review.openstack.org/#/c/363585/ we avoid hit the database so i guess it's better, but if I can give my opinion I prefer to have it fixed in compute/utils.py like for https://review.openstack.org/#/c/398308/ | 15:27 |
| BobBall | toabctl: TBH that will need input from anthonyper | 15:28 |
| mriedem | sahid: i think i prefer https://review.openstack.org/#/c/398308/ because it handles it globally | 15:28 |
| mriedem | so we don't have to play whack a mole with that type of failure | 15:28 |
| *** tblakes has joined #openstack-nova | 15:28 | |
| sahid | mriedem: yes exactly | 15:29 |
| toabctl | BobBall, ok. then let's see what anthonyper thinks | 15:29 |
| sahid | mriedem: but in that one we hit the database right ? | 15:29 |
| sahid | where we could just add condition like in https://review.openstack.org/#/c/363585/ | 15:30 |
| toabctl | sahid, that is similar to the first patchset I had: https://review.openstack.org/#/c/398308/1/nova/network/neutronv2/api.py | 15:32 |
| *** openstackgerrit has quit IRC | 15:33 | |
| mriedem | toabctl: comment in https://review.openstack.org/#/c/398308/ | 15:33 |
| mriedem | please cleanup the setattr usage and then i'm +2 | 15:33 |
| mriedem | sahid: i think i'd just rather handle the error | 15:33 |
| *** openstackgerrit has joined #openstack-nova | 15:33 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Create hyperv fake images under proper directory https://review.openstack.org/406163 | 15:33 |
| mriedem | in case there is some stale value on instance.deleted or something | 15:34 |
| *** jamesdenton has joined #openstack-nova | 15:34 | |
| lbeliveau | mriedem: leakypipes: ^ easy patch for a cold friday morning | 15:34 |
| mriedem | claudiub|2: lpetrut: want to look at https://review.openstack.org/406163 ? | 15:35 |
| mriedem | lbeliveau: maybe cold in canada, | 15:35 |
| melwitt | does anyone think we have plans anytime soon to run the docs tox target under py3x? 'tox -edocs' got broken again for py3x https://review.openstack.org/#/c/405665/ | 15:36 |
| mriedem | it's a balmy 30 F here in minnesota | 15:36 |
| mriedem | melwitt: +2 | 15:37 |
| lbeliveau | mriedem: whoa, even colder than Montreal (42 F) :) | 15:37 |
| openstackgerrit | Thomas Bechtold proposed openstack/nova: Handle deleted instances when refreshing the info_cache https://review.openstack.org/398308 | 15:37 |
| toabctl | mriedem, updated | 15:37 |
| mriedem | melwitt: i saw andrey's comment, and agree with fixing it, but we'd just break it again as we don't gate docs on py3 | 15:38 |
| mriedem | i've tried to make py3 a top priority for nova but the TC just won't buy in... | 15:38 |
| melwitt | mriedem: thanks, that was my rationale. I was just wondering whether gating docs on py3 is a possibility. I assumed not at this time | 15:39 |
| *** hongbin has joined #openstack-nova | 15:39 | |
| mriedem | i'm sure we could | 15:39 |
| melwitt | hah | 15:39 |
| mriedem | but someone has to push those buttons | 15:39 |
| mriedem | clarkb might know if anyone is working on making docs jobs run under py3 | 15:39 |
| *** zz_dimtruck is now known as dimtruck | 15:40 | |
| mriedem | toabctl: got it | 15:40 |
| sahid | mriedem: yes seems reasonable so | 15:41 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: WIP: [proxy-api] microversion 2.39 deprecates image-metadata proxy API https://review.openstack.org/405461 | 15:41 |
| pkholkin | mriedem: alex_xu: hello again! one more patch set to the spec implementation ^, you are welcome) https://review.openstack.org/#/c/405461/ | 15:42 |
| ihrachys | getting malloc(): memory corruption from libvirtd in gate. is it known? http://logs.openstack.org/33/404933/1/gate/gate-tempest-dsvm-neutron-src-neutron-lib-ubuntu-xenial/6463fc9/logs/syslog.txt.gz#_Dec_02_14_40_20 | 15:42 |
| mriedem | ihrachys: yup | 15:43 |
| *** alexpilotti has quit IRC | 15:43 | |
| mriedem | ihrachys: http://status.openstack.org//elastic-recheck/index.html#1643911 | 15:43 |
| ihrachys | ok, https://img.pandawhale.com/post-64231-this-is-fine-dog-fire-comic-Im-N7mp.png | 15:43 |
| *** tbachman has quit IRC | 15:44 | |
| *** DinaBelova has quit IRC | 15:44 | |
| mriedem | 8 fails in 10 days | 15:44 |
| mriedem | so i wouldn't consider it a fire :) | 15:44 |
| *** DinaBelova has joined #openstack-nova | 15:44 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Integrate OSProfiler and Nova https://review.openstack.org/254703 | 15:44 |
| cardeois | mriedem hello ! I was wondering, would you have time to review this please? https://review.openstack.org/#/c/400785/ | 15:45 |
| cardeois | I would be sooo happy if this gets merged :) | 15:45 |
| ihrachys | mriedem: nah just kidding | 15:46 |
| ihrachys | mriedem: the e-r query captures exact address | 15:46 |
| *** tbachman has joined #openstack-nova | 15:46 | |
| ihrachys | mriedem: I got a different one | 15:46 |
| ihrachys | so no e-r comment for me | 15:46 |
| mriedem | cdent: you've got a CORS thing up for placement right? maybe you'd like to review https://review.openstack.org/#/c/400785/ | 15:46 |
| ihrachys | should I remove it from the match? | 15:46 |
| mriedem | ihrachys: ah yeah probably | 15:46 |
| ihrachys | ok will do | 15:46 |
| leakypipes | bauwser: the "semaphore" comment... in lockutils it's a threading.Lock unless you specify a filepath, which would make it a persistent cross-process file lock. | 15:46 |
| mriedem | ihrachys: well, | 15:46 |
| mriedem | message:"*** Error in `/usr/sbin/libvirtd': malloc(): memory corruption:" AND tags:"syslog" AND voting:1 | 15:46 |
| mriedem | that's pretty generic | 15:47 |
| bauwser | leakypipes: yup, I looked up at the method impl | 15:47 |
| cdent | mriedem: yeah, I was through there once already, its in my queue to go back to | 15:47 |
| bauwser | leakypipes: I just wonder what happens when you don't get the lock | 15:47 |
| ihrachys | mriedem: ehm, sorry, I am probably tired, no idea where I took it from | 15:47 |
| cardeois | cdent cool thanks take you time, as long as this is not forgot, I'm fine with it | 15:47 |
| bauwser | oops | 15:47 |
| mriedem | i'll be honest i do'nt know anything about cors middleware | 15:47 |
| bauwser | I meant "I wondered | 15:47 |
| mriedem | http://docs.openstack.org/developer/oslo.middleware/cors.html | 15:47 |
| * mriedem reads | 15:47 | |
| *** ljxiash has quit IRC | 15:48 | |
| *** eharney has quit IRC | 15:50 | |
| superdan | gibi: okay free now | 15:50 |
| bauwser | mriedem: cardeois: it's a paste change, why should we by default adding that middleware if we don't care about CORS ? | 15:50 |
| leakypipes | bauwser, figleaf, pkoniszewski, tdurakov: please check out this patch from lbeliveau: https://review.openstack.org/#/c/405575. This is a bug (I'm pretty sure) that Ludovic discovered about resource accounting during resizes. I asked him to create a test case that shows the bug (which he has done). Would like your opinion on the questions he asks in the code comments in resource_tracker.py. | 15:51 |
| mriedem | bauwser: well this isn't new | 15:51 |
| cdent | leakypipes, bauwser: I'm having a slow brain, can you guys remind me how/where associated aggregates get involved in http://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/resource-providers-scheduler-db-filters.html ? Do we return a compute noder resource provider when it is associated, via aggregate, with a disk-based resource provider? | 15:51 |
| mriedem | [pipeline:meta] | 15:51 |
| mriedem | pipeline = cors metaapp | 15:51 |
| gibi | superdan: I'm here | 15:52 |
| leakypipes | bauwser: it blocks trying to get the lock. | 15:52 |
| bauwser | mriedem: sure, but here, that's on the oscomputeversions pipeline, right? | 15:52 |
| superdan | gibi: okay, so one thing you were asking in the review, I think, was whether we should send one notification or two, if both projects and extra_specs are changed, right? | 15:52 |
| gibi | superdan: yes, but that is a bit secondary. There I think we can simply go with a single notification | 15:53 |
| bauwser | leakypipes: yup, I know, I meant "I wondered" hence me looking at the implementation, and when I saw a threading lock, I knew that it was blocking | 15:53 |
| superdan | gibi: right, I think one is better | 15:53 |
| bauwser | (thanks my engineering school classes) | 15:53 |
| superdan | gibi: but that leads me to my second point | 15:53 |
| mriedem | cardeois: i'm just +1 on that change | 15:53 |
| mriedem | not comfortable enough with CORS to +2 | 15:53 |
| *** nherciu_ has quit IRC | 15:53 | |
| superdan | gibi: if we are going to send one, then we need to decide at some point whether we're sending a notification or not because of this save, | 15:54 |
| *** Guest79986 has quit IRC | 15:54 | |
| leakypipes | bauwser: ah, gotcha. | 15:54 |
| superdan | gibi: and if we are, we can load projects _and_ extra_specs in a single shot, if needed so that we're not loading them twice right? | 15:54 |
| *** ralonsoh has quit IRC | 15:54 | |
| superdan | gibi: actually, hmm | 15:54 |
| superdan | gibi: I guess for flavor those are the only two mutable things | 15:55 |
| mriedem | sdague: you'd be better than me to review this https://review.openstack.org/#/c/400785/ | 15:55 |
| gibi | superdan: hm, so we can at least spare a load. that is good. | 15:55 |
| bauwser | cardeois: I actually don't see why we need CORS on a pure RESTful interface that is not a webpage | 15:55 |
| leakypipes | cdent: one sec, formulating a response. | 15:55 |
| cdent | leakypipes++ | 15:55 |
| bauwser | cardeois: are you planning to integrate /versions in a richer webpage ? | 15:55 |
| superdan | gibi: sure, lazy loading is fine -- the thing is, if we're doing something where we're always going to need to lazy-load something, we should just pre-query for it | 15:56 |
| gibi | superdan: I was afraid of doing and extra db query for the lazy load but you are right we might even do two extra db query | 15:56 |
| leakypipes | cdent: you might want to respond to bauwser on the CORS stuff :) | 15:56 |
| superdan | gibi: if not, then lazy-load is the optimization | 15:56 |
| bauwser | leakypipes: I know what CORS is, I just wonder what that means in a world where we have clients consuming a REST API | 15:56 |
| gibi | superdan: OK so if we know that there will be notification then we know that we need extra_specs and projects so we load them in the same call | 15:56 |
| superdan | gibi: so if this is actually called in a path where we load flavor with only projects, then modify projects and save, we're *always* going to lazy-load extra_specs, which means we should just load it from the beginning | 15:56 |
| superdan | gibi: right | 15:57 |
| bauwser | leakypipes: that's not a judgment of value, just that I miss the usecase | 15:57 |
| leakypipes | bauwser: I know, and cdent knows a lot about that, which is why I asked him to respond (I personally don't know much about CORS) | 15:57 |
| superdan | gibi: so that would be in the caller of save(), to pre-load both, knowing it is about to modify one or the other | 15:57 |
| bauwser | leakypipes: heh sorry, that's a former life background of myself working in the $web world | 15:57 |
| leakypipes | bauwser: as you know, I'd be happy if SQL was the only interface anyone ever used. screw this HTTP shit. :P | 15:57 |
| cdent | bauwser: CORS is basically required if a javascript application is provided by domain a (say running horizon) that needs to get data from a source on another domain | 15:58 |
| bauwser | cdent: that I know | 15:58 |
| bauwser | cdent: so say a JS client consuming our APIs | 15:58 |
| bauwser | natively | 15:58 |
| gibi | superdan: but that would be a leak of the implementation of the save to the caller of save | 15:58 |
| bauwser | I thought horizon was using novaclient, that's it | 15:58 |
| bauwser | oooh, I see | 15:59 |
| * cdent shakes tiny fist at leakypipes' heresy | 15:59 | |
| superdan | gibi: yeah, and lazy-loading means that they don't *have* to know that detail | 15:59 |
| bauwser | some app developers might want to consume natively Nova | 15:59 |
| superdan | gibi: but it's more efficient if they do | 15:59 |
| *** _ducttape_ has joined #openstack-nova | 15:59 | |
| bauwser | gotcha | 15:59 |
| cdent | bauwser: yes, aggregating the data in the browser, not server side | 15:59 |
| bauwser | maybe not horizon, but anyone wanting to access our REST API using a javascript client | 15:59 |
| bauwser | mmm | 15:59 |
| yingjun | ping gibi | 16:00 |
| superdan | gibi: either way, modifying this attribute of a flavor is a pretty infrequent action, and it's an admin thing, so I don't think it's hugely critical that we squeeze out every optimization | 16:00 |
| bauwser | cdent: okay, so when wanting to do that, a cool JS engine would parse the versions and then call the latest API | 16:00 |
| superdan | gibi: so if you'd rather just let lazy loading cover up the details, that's fine too, IMHO | 16:00 |
| bauwser | and in that case, they'd fail | 16:00 |
| bauwser | I see | 16:00 |
| gibi | superdan: OK. Thank you for your time | 16:00 |
| superdan | gibi: sure, hope that helps :) | 16:01 |
| sdague | cardeois: we probably need a reno for that change | 16:01 |
| gibi | superdan: definitely helps :) | 16:01 |
| superdan | cool | 16:01 |
| sdague | given that management of paste.ini files is really grey area in config / code boundary | 16:01 |
| bauwser | cdent: what is the API-WG consensus on that ? Do we agree clients consuming our APIs that way? | 16:01 |
| cdent | yes | 16:02 |
| cdent | that's sort of the point of having apis | 16:02 |
| bauwser | sdague: I agree with you | 16:02 |
| gibi | yingjun: hi! we just discussed my comments in your review with superdan about the lazy load | 16:02 |
| mriedem | damn! sdague got me on the reno! | 16:02 |
| cdent | otherwise we could just live in leakypipes sql paradise | 16:02 |
| mriedem | all of my stats, obliterated | 16:02 |
| *** lpetrut has quit IRC | 16:02 | |
| yingjun | gibi, great! scrolling | 16:02 |
| *** ducttape_ has quit IRC | 16:02 | |
| bauwser | mriedem: sssht, CodeReview- | 16:03 |
| sdague | cdent: well, we just need to put jay in a time machine to 1995 and he'll be happy then :) | 16:03 |
| gibi | yingjun: basically we agreed that performance is not a concern but we should avoid sending two update notification for the same save() call | 16:03 |
| mriedem | is sql paradise adjacent to gangsta's paradise? | 16:03 |
| superdan | gibi: yingjun: agreed | 16:03 |
| gibi | yingjun: I'm commenting this back to the review for later reference | 16:03 |
| mriedem | sdague: i think 1994 would be preferrable | 16:04 |
| EmilienM | cdent: hi! I'm preparing some work to add nova placement api into puppet-nova and I was wondering if [placement] in nova.conf was the only section/thing to take care of | 16:04 |
| mriedem | that was the last time i think the cleveland browns had a winning season | 16:04 |
| cdent | sdague: a) hey, hi, welcome back b) will jay ever be happy? | 16:04 |
| bauwser | leakypipes: mriedem: https://i.imgflip.com/rtcwz.jpg | 16:04 |
| ftersin | mdbooth: good. the fewer intermediate steps, the faster the goal is reached | 16:04 |
| mriedem | bauwser: exactly | 16:04 |
| cardeois | bauwser we need CORS for root url of nova because we're implementing a javascript sdk to talk to the OpenStack API. This sdk could either be used in node (server side) or in browser directly without having a intermediate API to proxy the calls. In order to do that we need CORS to be enabled | 16:05 |
| bauwser | cardeois: yup, cdent got me thinking about that, thanks | 16:05 |
| mriedem | EmilienM: http://docs.openstack.org/developer/nova/placement.html might help | 16:05 |
| *** rcernin has quit IRC | 16:05 | |
| yingjun | gibi, so just add self._send_notification to save, and remove it from save_extra_specs and save_projects ? | 16:05 |
| *** jamesdenton has quit IRC | 16:06 | |
| EmilienM | mriedem: indeed, thanks! | 16:06 |
| bauwser | cardeois: I was thinking it could be cloud-specific, but if you guys are writing a JS SDK, then /me shrugs | 16:06 |
| cdent | EmilienM: for now, yes, otherwise it uses existing nova.conf settings. lib/placement in devstack is pretty small, if you grep iniset in there it ought to be clear, or point me at the code in progress and I can help | 16:06 |
| sdague | cdent: thanks | 16:06 |
| gibi | yingjun: yes, that is the basic idea | 16:06 |
| cdent | EmilienM: placement intentionally tries to minimize the amount of config it needs | 16:07 |
| *** tbachman has quit IRC | 16:07 | |
| *** voelzmo has quit IRC | 16:07 | |
| gibi | yingjun: I don't know if there is any case when save() is called and neither extra_specs nor projects are changed | 16:07 |
| mriedem | cardeois: http://docs.openstack.org/developer/nova/code-review.html?highlight=reno#release-notes | 16:07 |
| cardeois | thanks mriedem, catching up the discussion right now | 16:07 |
| gibi | yingjun: if such a case exists then we don't need to send anything | 16:08 |
| EmilienM | cdent: ok and iiuc, the api is included in regular nova-api process and you can also run it in wsgi | 16:08 |
| leakypipes | sdague: 1995... hmm, good times. | 16:08 |
| yingjun | gibi, yes | 16:08 |
| EmilienM | cdent: in other words, it's enabled with enabled_apis = osapi_compute | 16:09 |
| leakypipes | bauwser: lol, that pic is awesome. | 16:09 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Create hyperv fake images under proper directory https://review.openstack.org/406163 | 16:09 |
| cdent | EmilienM: no, you can only run it with mod wsgi | 16:09 |
| EmilienM | cdent: oh, ok gotcha | 16:09 |
| *** jamesdenton has joined #openstack-nova | 16:09 | |
| cardeois | mriedem so If I write a reno for this CORS thing, should I create a bug or blueprint? I guess bug in that case? After that I guess there is plenty of examples I would find | 16:10 |
| cdent | EmilienM: or whatever wsgi container you like. pbr generates a script | 16:10 |
| bauwser | leakypipes: I just want to say you teach SQL better than my teacher when I was 21 | 16:10 |
| mriedem | cardeois: neither | 16:10 |
| mriedem | imo | 16:10 |
| bauwser | cardeois: no no | 16:10 |
| bauwser | just a new PS | 16:10 |
| sdague | cardeois: agree with mriedem, just add it into the changeset | 16:10 |
| yingjun | gibi, seems save is the only caller for save_extra_specs and save_projects for now | 16:10 |
| leakypipes | bauwser: aww, shucks, thanks man :) | 16:10 |
| cardeois | bauwser PS? | 16:10 |
| EmilienM | cdent: excellent! thanks. Last question, is there some tempest tests that I can use to test that it actually works? | 16:10 |
| mriedem | cardeois: patch set | 16:11 |
| bauwser | patchset, see other comments from mriedem and sdague | 16:11 |
| cardeois | ok thanks | 16:11 |
| cardeois | will try to do that right now | 16:11 |
| mriedem | our db/sql professor was the CS dept head and he was terrible | 16:11 |
| cardeois | Thanks for your help mriedem, sdague and bauwser ! | 16:11 |
| mriedem | now i can start thinking about BNF again.. | 16:11 |
| bauwser | oh man | 16:11 |
| yingjun | gibi, about the devref comment, did you mean this: http://docs-draft.openstack.org/71/398171/9/check/gate-nova-docs-ubuntu-xenial/95166a1//doc/build/html/notifications.html#existing-versioned-notifications | 16:11 |
| *** ayogi has quit IRC | 16:12 | |
| cdent | EmilienM: not in any concrete fashion not yet. It's one of those "on the list" things (a list which I'm currently updating so I'll make sure it is there). However, if it is configured the resource-tracker will complain in its logs if it is not working | 16:12 |
| *** armax has joined #openstack-nova | 16:13 | |
| gibi | yingjun: ahh I see flavor is there. Most probably because notification.objects.instance now imports notification.objects.flavor | 16:13 |
| yingjun | missing flavor.update there | 16:14 |
| EmilienM | cdent: fair enough, thx for your help! | 16:14 |
| gibi | yingjun: could you please import flavor directly in https://github.com/openstack/nova/blob/master/doc/ext/versioned_notifications.py#L33 justt to future proofness | 16:14 |
| cdent | EmilienM: add me to the review when there's something, and I'll be happy to test it out | 16:14 |
| *** yingjun has quit IRC | 16:14 | |
| *** _ducttape_ has quit IRC | 16:15 | |
| *** yingjun has joined #openstack-nova | 16:15 | |
| cdent | leakypipes: did you manage to crystallize a response on the aggregates thing? If not, that's okay, I'll just write out what I think should happen and then you can correct me in a followup (I'm writing this week's rp update email). | 16:15 |
| EmilienM | cdent: excellent! | 16:15 |
| yingjun | gibi, Okay | 16:15 |
| gibi | yingjun: flavor.update is missing because you missed the decorator at https://review.openstack.org/#/c/398171/9/nova/notifications/objects/flavor.py@L23 | 16:15 |
| cdent | EmilienM: I am forever your most humble and obedient servant | 16:16 |
| EmilienM | cdent: I miss your puppet patches :( | 16:16 |
| * cdent gets back to work on the clone machine | 16:16 | |
| yingjun | gibi, yeah, missing the decorator there | 16:16 |
| *** david-lyle_ has joined #openstack-nova | 16:17 | |
| mriedem | pkholkin: comments inline | 16:17 |
| gibi | yingjun: I have to leave now let's continue in the review later | 16:18 |
| yingjun | gibi, ok | 16:18 |
| diana_clarke | oh man, someone sent my kid that playstation VR thing and it's not working... it's hard to work over all these tears | 16:18 |
| mriedem | cdent: re: https://review.openstack.org/#/c/344448/ | 16:18 |
| * cdent nods | 16:19 | |
| mriedem | cdent: i selfishly don't want to review another one of these for the same thing, | 16:19 |
| mriedem | so if it's all the same, i'm going to go with yours as i've already been through it | 16:19 |
| *** tbachman has joined #openstack-nova | 16:19 | |
| *** david-lyle has quit IRC | 16:19 | |
| cdent | fine with me, I just didn't want to discount the effort they made | 16:19 |
| cdent | (despite only finding it after I started) | 16:19 |
| mriedem | yeah i know | 16:19 |
| cdent | my version is a little more complete | 16:19 |
| mriedem | shit happens | 16:19 |
| mriedem | we can iou him another review on sometihng else | 16:19 |
| mriedem | also, i enjoy working donald trump into as many reviews as possible | 16:20 |
| *** mdrabe has quit IRC | 16:20 | |
| *** gcb has quit IRC | 16:20 | |
| cardeois | sdague sorry again but which type of reno is the best for this CORS issue? features/fixes/other? | 16:21 |
| mriedem | cardeois: upgrade | 16:21 |
| cardeois | ok thanks mriedem | 16:21 |
| mriedem | changing the paste ini is an ugprade issue | 16:21 |
| sdague | cardeois: it's an upgrade issue, because people will need to manually update their paste.ini | 16:21 |
| mriedem | *upgrade even | 16:21 |
| cardeois | Oh ok | 16:21 |
| leakypipes | cdent: still working on it. | 16:21 |
| sdague | as it's not a file that is programically modified by nova | 16:21 |
| leakypipes | cdent: you want the response in English, not SQL, right? | 16:21 |
| cardeois | sdague: thanks | 16:22 |
| leakypipes | figleaf: you saw the pep8 failure in the "improve RC delete failure" thing yes? | 16:22 |
| sdague | cdent: you're fine with the cors add to that pipeline, right? | 16:22 |
| sdague | cdent: I didn't see you on that review yet | 16:22 |
| cdent | sdague: i'm on patchset 1 saying "tests?" | 16:22 |
| cdent | but otherwise the concept I agree with | 16:22 |
| cdent | leakypipes: I do what the response in english, but it could be as simple as "yes" or "no", unless you want to expand. | 16:23 |
| leakypipes | cdent: nes. yo. | 16:24 |
| sneti | :wq | 16:24 |
| leakypipes | sneti: :qa | 16:24 |
| cdent | leakypipes: combined with sneti there that's just hilarious | 16:24 |
| leakypipes | :) | 16:24 |
| cdent | mic drop | 16:24 |
| sneti | leakypipes, sorry..wrong window :) | 16:24 |
| *** mlavalle has joined #openstack-nova | 16:24 | |
| *** Oku_OS is now known as Oku_OS-away | 16:24 | |
| leakypipes | sneti: lol, no worries, I do that quite often myself. | 16:24 |
| *** ducttape_ has joined #openstack-nova | 16:24 | |
| sneti | leakypipes, :) | 16:25 |
| paul-carlton2 | mriedem, could you remove your -2 from https://review.openstack.org/#/c/326899 now we have a tempest test for it please | 16:27 |
| *** mdrabe has joined #openstack-nova | 16:28 | |
| leakypipes | cdent: so, in English, the response to your question is we will only return resource providers to the scheduler that are compute nodes in Ocata. the resource providers that the placement service returns will either have the resources requested or will be associated with aggregates that have providers that match the requested resources. | 16:28 |
| leakypipes | cdent: does that answer your question? | 16:28 |
| cdent | thank you, that's an excellent summary, especially since it agrees with what I thought/hoped? | 16:29 |
| sdague | superdan: I just kicked in the devstack change https://review.openstack.org/#/c/392227/ needs | 16:29 |
| pkholkin | mriedem: thanks for review! tests will be! just an update for today | 16:29 |
| cdent | s/\?$// | 16:29 |
| sdague | superdan: you ready for that to land on the nova side? | 16:29 |
| leakypipes | cdent: once we have a placement service actually doing claims, the returned resource providers for an allocation will be the actual resource providers that were allocated against (which include *both* compute node providers as well as any resource provider of a shared resource that was allocated) | 16:29 |
| sdague | it got approved about a month ago, but was blocked on depends-on | 16:29 |
| *** burt has quit IRC | 16:29 | |
| superdan | sdague: yep | 16:29 |
| cdent | leakypipes: yup, that aligns with my mental model too. (thank goodness) | 16:30 |
| *** bvanhav__ has quit IRC | 16:30 | |
| *** tbachman has quit IRC | 16:30 | |
| sdague | oh... depends-on is smarter now, and actually enqueued it | 16:30 |
| pkholkin | and will fix typos | 16:30 |
| leakypipes | cdent: coolio. | 16:30 |
| * cdent adds these things to his document | 16:30 | |
| *** rama_y has quit IRC | 16:30 | |
| *** ducttape_ has quit IRC | 16:30 | |
| *** ducttape_ has joined #openstack-nova | 16:31 | |
| *** jamesdenton has quit IRC | 16:32 | |
| *** jistr is now known as jistr|biab | 16:33 | |
| *** mlavalle has quit IRC | 16:33 | |
| *** eharney has joined #openstack-nova | 16:34 | |
| *** jamesdenton has joined #openstack-nova | 16:36 | |
| *** bvanhav__ has joined #openstack-nova | 16:37 | |
| *** gouthamr_ has quit IRC | 16:37 | |
| *** gouthamr has joined #openstack-nova | 16:37 | |
| *** jaosorior has quit IRC | 16:38 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Notifications on flavor operations https://review.openstack.org/398171 | 16:39 |
| *** gouthamr has quit IRC | 16:41 | |
| *** gouthamr has joined #openstack-nova | 16:41 | |
| *** Jeffrey4l has quit IRC | 16:42 | |
| openstackgerrit | Corentin Ardeois proposed openstack/nova: Add CORS filter to versions pipeline https://review.openstack.org/400785 | 16:42 |
| cardeois | cdent mriedem sdague ^^ | 16:42 |
| * cdent looks | 16:42 | |
| *** Zer0Byte__ has joined #openstack-nova | 16:43 | |
| openstackgerrit | Corentin Ardeois proposed openstack/nova: Add CORS filter to versions pipeline https://review.openstack.org/400785 | 16:44 |
| cardeois | (updated) | 16:44 |
| openstackgerrit | Francesco Santoro proposed openstack/nova: [WIP] os-vif: add os-vif plugin for vhostuser_fp https://review.openstack.org/385061 | 16:44 |
| *** jlvillal has quit IRC | 16:45 | |
| *** rcernin has joined #openstack-nova | 16:45 | |
| bauwser | sdague: what's the situation when we update paste.ini ? we say to the operators that they have to use the newer ? | 16:46 |
| *** paul-carlton2 has quit IRC | 16:47 | |
| *** josecastroleon has quit IRC | 16:47 | |
| sdague | bauwser: yeh, or make it equivalent | 16:47 |
| sdague | we actually have a lot of behavior that's controlled in that file | 16:47 |
| sdague | which ... sucks from a maintenance perspective | 16:47 |
| bauwser | yeah | 16:47 |
| bauwser | that's the main problem I have | 16:47 |
| sdague | bauwser: legacy | 16:47 |
| lbeliveau | mriedem: easy +2 on this annoying "fake" thing https://review.openstack.org/#/c/406163/ | 16:48 |
| *** matrohon has quit IRC | 16:48 | |
| bauwser | given we're also having functional tests testing that | 16:48 |
| sdague | it's one of the things that was addressed in the placement api, it doesn't have a paste.ini file (though it still uses paste) | 16:48 |
| bauwser | :) | 16:48 |
| markus_z | sahid: Would be nice to have your feedback on my last "virtlogd" change: https://review.openstack.org/#/c/323765/ | 16:48 |
| mriedem | lbeliveau: ok, would have liked the hyperv guys to look at that first, but will check it out | 16:48 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 16:49 |
| *** danpawlik has quit IRC | 16:49 | |
| lbeliveau | mridem: oh sure, no rush | 16:49 |
| cdent | sdague, mriedem: do we consider the reno notes to be final destination information? That is, are they going to be directly read by deployers or are they being used to inform more complete notes? | 16:49 |
| cdent | or to put it another way: how much does grammar and writing style matter | 16:49 |
| markus_z | sahid: FWIW, I reviewed danpb's vencrypt changes and am now somewhat familiar with it and can accompany it over the time. | 16:49 |
| sdague | cdent: they are directly consumed | 16:49 |
| sahid | markus_z: sure, i will do that soon | 16:50 |
| sdague | it's part of why bauwser does an audit at every milestone and does a set of fixes | 16:50 |
| cdent | sdague then I think maybe we can do that one better? is it worht it? | 16:50 |
| sdague | cdent: how about as a follow on patch? | 16:50 |
| markus_z | sahid: awesome, thanks in advance | 16:50 |
| cdent | yeah, good idea. | 16:50 |
| mriedem | cdent: i like them to be good | 16:50 |
| * markus_z calls it a day | 16:50 | |
| mriedem | cdent: they don't always go into other docs | 16:50 |
| mriedem | this one wouldn't probably | 16:50 |
| *** markus_z has quit IRC | 16:51 | |
| sdague | we'll let that go in and unblock cardeois, and we can do word smithing as follow up | 16:51 |
| bauwser | cdent: that's the main way we can provide explanations for operators and deployers, yep | 16:51 |
| *** yingjun has quit IRC | 16:51 | |
| *** yingjun has joined #openstack-nova | 16:52 | |
| * cdent adds to his todo list | 16:52 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: WIP: debugging test_delete_from_select https://review.openstack.org/406225 | 16:53 |
| *** lpetrut has joined #openstack-nova | 16:53 | |
| *** yingjun has quit IRC | 16:56 | |
| *** Swami has joined #openstack-nova | 16:56 | |
| *** Swami_ has joined #openstack-nova | 16:56 | |
| *** jlvillal has joined #openstack-nova | 16:57 | |
| *** tbachman has joined #openstack-nova | 16:58 | |
| *** jlvillal has quit IRC | 16:58 | |
| *** jlvillal has joined #openstack-nova | 16:59 | |
| *** code-R_ has joined #openstack-nova | 16:59 | |
| *** lpetrut has quit IRC | 17:00 | |
| openstackgerrit | Sujitha proposed openstack/nova: Do not merge: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 17:01 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 17:02 |
| *** code-R has quit IRC | 17:02 | |
| *** tbachman has quit IRC | 17:03 | |
| *** mlavalle has joined #openstack-nova | 17:03 | |
| *** penick has joined #openstack-nova | 17:03 | |
| *** mingyu has joined #openstack-nova | 17:04 | |
| *** tbachman has joined #openstack-nova | 17:07 | |
| mriedem | figleaf: pep8 fail https://review.openstack.org/#/c/400448/ | 17:07 |
| *** jlvillal has quit IRC | 17:08 | |
| *** jlvillal has joined #openstack-nova | 17:08 | |
| *** paw has quit IRC | 17:08 | |
| *** ihrachys has quit IRC | 17:09 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: pass the nodename to public methods https://review.openstack.org/398470 | 17:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: minor refactor _allocate_for_instance() https://review.openstack.org/405705 | 17:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: remove fluff from test_resource_tracker https://review.openstack.org/398471 | 17:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: pass nodename to internal methods https://review.openstack.org/398469 | 17:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: WIP: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | 17:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: report client handle InventoryInUse https://review.openstack.org/404990 | 17:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: explicitly pass compute node to _update() https://review.openstack.org/398472 | 17:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: use a single ResourceTracker object instance https://review.openstack.org/398473 | 17:13 |
| *** paw has joined #openstack-nova | 17:14 | |
| superdan | sdague: I'm having a weird test interaction thing | 17:14 |
| superdan | sdague: and I know you love bashing your head against those kinds of things | 17:14 |
| superdan | I've got a new fixture that use a MonkeyPatch, which seems to not be unset for tests that run later | 17:15 |
| superdan | it depends on test order and always runs if in isolation | 17:15 |
| superdan | sdague: what sort of bribes would be required for you to think on that for a bit? | 17:15 |
| *** rama_y has joined #openstack-nova | 17:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/406250 | 17:18 |
| *** nic has joined #openstack-nova | 17:24 | |
| *** paul-carlton2 has joined #openstack-nova | 17:25 | |
| mriedem | lbeliveau: lgtm | 17:25 |
| diana_clarke | superdan: Now I'm curious... is there a patch up I can play with? | 17:26 |
| *** _ducttape_ has joined #openstack-nova | 17:26 | |
| *** ayogi has joined #openstack-nova | 17:26 | |
| superdan | diana_clarke: it's multiple ones, but yeah.. there's a lot of fail noise currently but I can push up again in a bit which will narrow the problem some more | 17:26 |
| *** nic has quit IRC | 17:27 | |
| superdan | I feel like it has to be something really dumb I'm doing with my mocking that I'm just missing due to fatigue | 17:27 |
| *** nic has joined #openstack-nova | 17:27 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method https://review.openstack.org/367557 | 17:27 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Multicell support for instance listing https://review.openstack.org/396775 | 17:27 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 17:27 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add SecurityGroup.identifier to prefer uuid over name https://review.openstack.org/406260 | 17:27 |
| superdan | sdague: diana_clarke: unit tests on 319379 fail because the SingleCellSimple's mocks still seem to be in place, returning something that isn't in our cache for later tests that don't use that fixture | 17:28 |
| *** ducttape_ has quit IRC | 17:29 | |
| mriedem | figleaf: btw, when you change the bottom patch in a series, i think you should pull down the whole series, rebase and change the one patch, then push the whole thing back up, because otherwise the history gets goofy | 17:29 |
| *** sudipto has joined #openstack-nova | 17:29 | |
| *** paul-carlton2 has quit IRC | 17:31 | |
| *** fragatina has quit IRC | 17:32 | |
| mriedem | leakypipes: i'm going to rebase the resource classes rest api series | 17:32 |
| mriedem | and fix the pep8 so i can +2 | 17:32 |
| figleaf | mriedem: ok, I was just going to fix that, but if you got it... | 17:33 |
| mriedem | yup, got it | 17:33 |
| figleaf | kewl | 17:33 |
| *** fragatina has joined #openstack-nova | 17:34 | |
| *** unicell1 has quit IRC | 17:36 | |
| *** bvanhav_ has joined #openstack-nova | 17:37 | |
| *** mlavalle has quit IRC | 17:37 | |
| *** bvanhav__ has quit IRC | 17:37 | |
| *** fragatina has quit IRC | 17:39 | |
| *** sahid has quit IRC | 17:39 | |
| *** gszasz has quit IRC | 17:39 | |
| *** paw has quit IRC | 17:43 | |
| *** unicell has joined #openstack-nova | 17:43 | |
| *** gouthamr has quit IRC | 17:44 | |
| *** gouthamr has joined #openstack-nova | 17:44 | |
| *** rcernin has quit IRC | 17:45 | |
| *** rcernin has joined #openstack-nova | 17:46 | |
| mriedem | i'm not sure what this means | 17:46 |
| mriedem | AssertionError: json path $.links[?rel = "self"].href cannot match None | 17:46 |
| *** lucasagomes is now known as lucas-afk | 17:46 | |
| mriedem | i guess it means either links don't exist in the response, or they do but self isn't in links | 17:46 |
| *** bnemec has quit IRC | 17:47 | |
| mriedem | figleaf: cdent: https://review.openstack.org/#/c/391959/23/nova/api/openstack/placement/handlers/resource_class.py@202 | 17:49 |
| mriedem | should that be a 204 with no content, or 200 with content as written? | 17:49 |
| mriedem | the spec said 204 | 17:49 |
| *** derekh has quit IRC | 17:49 | |
| cdent | mriedem: that json path resolves to None but something else was expected | 17:50 |
| mriedem | ok, that started blowing up when i changed it to 204, i'll just leave it as a 200 | 17:51 |
| mriedem | and we can update the spec | 17:51 |
| *** anish has joined #openstack-nova | 17:51 | |
| figleaf | Yeah, it's returning the json-serialized resource class, so definitely not a 204 | 17:51 |
| cdent | yeah, I think the spec is out of date on some responses. Do you remember sdague , we had some discussions about that. I seem to recall it was your preference for there to be a bodied response? | 17:52 |
| mriedem | here we go ^ | 17:52 |
| *** mriedem is now known as mriedem_lunch | 17:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: placement: REST API for resource classes https://review.openstack.org/391959 | 17:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: placement: adds ResourceClass.save() https://review.openstack.org/400448 | 17:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: placement: Perform build list of standard classes once https://review.openstack.org/406062 | 17:52 |
| mriedem_lunch | addressed the rest api comments of mine as well | 17:52 |
| cdent | I have a todo item to update the grp spec to reflect reality, but haven't done it yet to see if anything else changes | 17:53 |
| *** jistr|biab is now known as jistr | 17:53 | |
| *** Sukhdev has joined #openstack-nova | 17:54 | |
| *** yamahata has joined #openstack-nova | 17:54 | |
| *** Guest51607 has quit IRC | 17:55 | |
| *** lpetrut has joined #openstack-nova | 17:56 | |
| *** mbirru has quit IRC | 17:56 | |
| *** mnestratov has quit IRC | 17:57 | |
| *** mlavalle has joined #openstack-nova | 17:58 | |
| sdague | superdan: I can take a look, I'll pull the patch and see if anything looks odd to me | 18:02 |
| superdan | sdague: thanks, I keep looking at the simple fixture thinking I must be doing something dumb, but it's ... really simple. | 18:03 |
| *** fragatina has joined #openstack-nova | 18:04 | |
| leakypipes | mriedem_lunch: thx for pushing a fix for the customer resource clases REST API pathc. | 18:04 |
| sdague | cdent: my preference was definitely that POST returns some representation if the call is synchronous, as it acts as an ACK | 18:05 |
| *** slaweq_ has joined #openstack-nova | 18:06 | |
| *** mingyu has quit IRC | 18:06 | |
| *** voelzmo has joined #openstack-nova | 18:07 | |
| *** unicell has quit IRC | 18:07 | |
| *** david-lyle_ is now known as david-lyle | 18:08 | |
| *** dimtruck is now known as zz_dimtruck | 18:09 | |
| *** _ducttape_ has quit IRC | 18:13 | |
| *** ducttape_ has joined #openstack-nova | 18:13 | |
| mlavalle | I like leakypipes | 18:14 |
| *** matrohon has joined #openstack-nova | 18:14 | |
| *** gouthamr_ has joined #openstack-nova | 18:16 | |
| *** code-R_ has quit IRC | 18:17 | |
| *** code-R has joined #openstack-nova | 18:17 | |
| *** gouthamr has quit IRC | 18:17 | |
| *** burt has joined #openstack-nova | 18:21 | |
| openstackgerrit | Sujitha proposed openstack/nova: Do not merge: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 18:21 |
| figleaf | mlavalle: wish I was your plumber then - I'd be rich! | 18:23 |
| * cdent breaks for dinner | 18:24 | |
| *** cdent has quit IRC | 18:24 | |
| *** ayogi has quit IRC | 18:24 | |
| mlavalle | figleaf: this wasn't your nick here, was it? | 18:24 |
| mlavalle | figleaf: or is this a 'casual Friday' type of thing? | 18:25 |
| figleaf | It's my casual friday nick | 18:25 |
| mlavalle | figleaf: ahhhh. does that explain leakypipes and superdan? | 18:26 |
| figleaf | yes, yes it does | 18:26 |
| mlavalle | LOL | 18:27 |
| *** fragatina has quit IRC | 18:27 | |
| *** cfriesen_ is now known as cfreezin | 18:27 | |
| cfreezin | in honour of casual Friday | 18:27 |
| *** fragatina has joined #openstack-nova | 18:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/406250 | 18:28 |
| *** ducttape_ has quit IRC | 18:29 | |
| *** stewie925 has joined #openstack-nova | 18:30 | |
| *** ducttape_ has joined #openstack-nova | 18:30 | |
| *** sudipto has quit IRC | 18:30 | |
| *** daemontool has joined #openstack-nova | 18:32 | |
| *** Apoorva has joined #openstack-nova | 18:32 | |
| *** zz_dimtruck is now known as dimtruck | 18:33 | |
| *** ducttape_ has quit IRC | 18:34 | |
| *** ihrachys has joined #openstack-nova | 18:34 | |
| jroll | leakypipes: are we deleting the old-style ironic resources when we make the new-style records? | 18:38 |
| *** daemontool has quit IRC | 18:38 | |
| *** karts has joined #openstack-nova | 18:39 | |
| jroll | I kind of imagined we'd have both for a cycle | 18:39 |
| *** karthiks has quit IRC | 18:39 | |
| jroll | which gives people time to cut their flavors over | 18:39 |
| *** voelzmo has quit IRC | 18:39 | |
| *** voelzmo has joined #openstack-nova | 18:40 | |
| *** unicell has joined #openstack-nova | 18:42 | |
| *** jheroux has quit IRC | 18:44 | |
| *** karts has quit IRC | 18:44 | |
| *** voelzmo has quit IRC | 18:46 | |
| leakypipes | jroll: sec, on phoen. | 18:47 |
| jroll | leakypipes: no rush | 18:48 |
| *** karts has joined #openstack-nova | 18:48 | |
| *** xyang1 has quit IRC | 18:49 | |
| *** diogogmt has joined #openstack-nova | 18:50 | |
| *** READ10 has quit IRC | 18:51 | |
| *** liverpooler is now known as the_dark_lord_mo | 18:52 | |
| *** the_dark_lord_mo is now known as liverpooler | 18:52 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make SessionClient interface similar to HTTPClient https://review.openstack.org/350140 | 18:53 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Clarify meaning of project_id var https://review.openstack.org/350106 | 18:53 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Sort arguments for client's methods https://review.openstack.org/406293 | 18:53 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Restict usage *args for novaclient.client.Client https://review.openstack.org/406294 | 18:53 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Introduce helper for checking args deprecation https://review.openstack.org/406295 | 18:53 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Remove redundant args of _construct_http_client https://review.openstack.org/406296 | 18:53 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename bypass_url to endpoint_override https://review.openstack.org/406297 | 18:53 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename api_key to password https://review.openstack.org/406298 | 18:53 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename interface to endpoint_type https://review.openstack.org/406299 | 18:53 |
| *** ihrachys has quit IRC | 18:54 | |
| andreykurilin | mriedem_lunch: bon appetit! | 18:54 |
| andreykurilin | mriedem_lunch: Finally, I found time to split my patch to several ones ^ :D | 18:55 |
| *** lpetrut has quit IRC | 19:00 | |
| *** mingyu has joined #openstack-nova | 19:01 | |
| *** xyang1 has joined #openstack-nova | 19:02 | |
| *** matrohon has quit IRC | 19:09 | |
| *** stewie925 has quit IRC | 19:10 | |
| *** code-R_ has joined #openstack-nova | 19:10 | |
| *** code-R has quit IRC | 19:13 | |
| *** ijw has joined #openstack-nova | 19:14 | |
| *** ijw has quit IRC | 19:15 | |
| *** ijw has joined #openstack-nova | 19:15 | |
| *** ducttape_ has joined #openstack-nova | 19:19 | |
| *** Sukhdev has quit IRC | 19:21 | |
| *** portdirect_away has quit IRC | 19:23 | |
| *** pradk has joined #openstack-nova | 19:23 | |
| *** pradk has quit IRC | 19:23 | |
| *** ducttape_ has quit IRC | 19:24 | |
| *** bdeschenes has joined #openstack-nova | 19:25 | |
| *** portdirect_away has joined #openstack-nova | 19:26 | |
| *** portdirect_away is now known as portdirect | 19:26 | |
| *** bdeschenes has quit IRC | 19:26 | |
| *** mriedem_lunch is now known as mriedem | 19:27 | |
| *** ducttape_ has joined #openstack-nova | 19:28 | |
| *** slaweq_ has quit IRC | 19:30 | |
| *** slaweq_ has joined #openstack-nova | 19:32 | |
| *** serverascode has quit IRC | 19:34 | |
| *** NobodyCam has quit IRC | 19:34 | |
| *** ameade has quit IRC | 19:34 | |
| *** rpodolyaka has quit IRC | 19:34 | |
| *** tdurakov has quit IRC | 19:34 | |
| *** zhiyan has quit IRC | 19:34 | |
| *** vdrok has quit IRC | 19:34 | |
| *** johnsom has quit IRC | 19:34 | |
| *** scottda has quit IRC | 19:34 | |
| *** morgan has quit IRC | 19:34 | |
| *** sballe_ has quit IRC | 19:34 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Fix bullet formatting in RP custom-resource-classes spec https://review.openstack.org/400459 | 19:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Amend response details for custom resource classes PUT https://review.openstack.org/406322 | 19:37 |
| mriedem | superdan: you wanna just fast approve this https://review.openstack.org/#/c/400459/ ? | 19:37 |
| mriedem | or i can... | 19:38 |
| *** moshele has joined #openstack-nova | 19:38 | |
| superdan | did | 19:38 |
| mriedem | nice | 19:39 |
| *** thorst_ has quit IRC | 19:40 | |
| leakypipes | jroll: sorry about that... | 19:42 |
| *** mbirru has joined #openstack-nova | 19:42 | |
| *** gouthamr_ has quit IRC | 19:42 | |
| leakypipes | jroll: so I'd like to automatically do the migration from old-style to new-style if possible. | 19:43 |
| leakypipes | jroll: however, I'm wondering if folks think it's OK to auto-create those custom resource classes based on the Ironic node's "resource_class" attribute, if set. | 19:44 |
| jroll | leakypipes: don't disagree - but we should leave the old records around for old-style flavors for now, right? | 19:44 |
| jroll | right | 19:44 |
| *** gouthamr has joined #openstack-nova | 19:44 | |
| jroll | I think that's fine, personally, just wanted to clear some things up before emailing | 19:44 |
| *** ljxiash has joined #openstack-nova | 19:45 | |
| *** ijw has quit IRC | 19:48 | |
| *** josecastroleon has joined #openstack-nova | 19:48 | |
| *** avolkov has quit IRC | 19:48 | |
| *** ljxiash has quit IRC | 19:49 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: Perform build list of standard classes once https://review.openstack.org/406062 | 19:49 |
| *** unicell has quit IRC | 19:49 | |
| *** serverascode has joined #openstack-nova | 19:50 | |
| *** kfarr has joined #openstack-nova | 19:50 | |
| *** portdirect has quit IRC | 19:50 | |
| leakypipes | jroll: they will continue to be around in the instance_extra.flavor column and compute_nodes.[memory_mb|vcpus|local_gb] fields until the scheduler stops looking at those fields for scheduling ironic nodes, yes. but I'd like to correct the placement service's representation of Ironic node inventory ASAP. | 19:50 |
| *** ameade has joined #openstack-nova | 19:51 | |
| jroll | leakypipes: right, okay, the first part of that was my question. that sounds good :) | 19:51 |
| *** slaweq_ has quit IRC | 19:52 | |
| *** josecastroleon has quit IRC | 19:52 | |
| *** zhiyan has joined #openstack-nova | 19:52 | |
| *** READ10 has joined #openstack-nova | 19:53 | |
| *** slaweq_ has joined #openstack-nova | 19:54 | |
| *** cleong has quit IRC | 19:54 | |
| leakypipes | jroll: coolio :) your comments on that WIP patch at the end of that patch series would be awesome. basically need someone to validate the logic I'm using in that _cleanup_ironic_legacy_allocations() method. | 19:54 |
| jroll | leakypipes: yeah, I'll do that and reply to your email today or maybe monday | 19:54 |
| *** cleong has joined #openstack-nova | 19:54 | |
| *** sballe_ has joined #openstack-nova | 19:55 | |
| leakypipes | jroll: rock on. | 19:55 |
| *** ijw has joined #openstack-nova | 20:00 | |
| *** esberglu has quit IRC | 20:01 | |
| *** esberglu has joined #openstack-nova | 20:01 | |
| leakypipes | mriedem: you pushing a rev for the custom resource classes REST API patch, yes? | 20:01 |
| mriedem | yup | 20:01 |
| mriedem | nearly there | 20:01 |
| leakypipes | coolio, just checkin. | 20:01 |
| leakypipes | mriedem: thx for hopping on that. | 20:01 |
| *** scottda has joined #openstack-nova | 20:02 | |
| mriedem | np | 20:02 |
| openstackgerrit | jichenjc proposed openstack/nova: InstanceGroupPolicyNotFound not used anymore https://review.openstack.org/406333 | 20:03 |
| leakypipes | bauwser: still around? | 20:04 |
| *** esberglu_ has joined #openstack-nova | 20:05 | |
| *** paw has joined #openstack-nova | 20:05 | |
| mriedem | damn you ddt messing up all of my tox envs | 20:05 |
| *** esberglu has quit IRC | 20:06 | |
| *** ljxiash has joined #openstack-nova | 20:06 | |
| *** esberglu has joined #openstack-nova | 20:07 | |
| *** _ducttape_ has joined #openstack-nova | 20:07 | |
| *** esberglu_ has quit IRC | 20:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: placement: REST API for resource classes https://review.openstack.org/391959 | 20:09 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: placement: Perform build list of standard classes once https://review.openstack.org/406062 | 20:09 |
| *** esberglu_ has joined #openstack-nova | 20:09 | |
| mriedem | i'm pretty sure that's the gd bees knees | 20:10 |
| leakypipes | :) | 20:10 |
| *** ducttape_ has quit IRC | 20:10 | |
| *** ljxiash has quit IRC | 20:11 | |
| *** clenimar has quit IRC | 20:11 | |
| *** esberglu has quit IRC | 20:11 | |
| *** thorst_ has joined #openstack-nova | 20:14 | |
| *** tdurakov has joined #openstack-nova | 20:15 | |
| mriedem | superdan: this is ready to go i think https://review.openstack.org/#/c/400448/ | 20:15 |
| mriedem | for resource class update | 20:15 |
| *** vdrok has joined #openstack-nova | 20:15 | |
| mriedem | i will apologize if i already bugged you about that one today | 20:15 |
| *** rpodolyaka has joined #openstack-nova | 20:16 | |
| *** NobodyCam has joined #openstack-nova | 20:16 | |
| openstackgerrit | jichenjc proposed openstack/nova: remove NetworkDuplicated exception https://review.openstack.org/406343 | 20:17 |
| *** _ducttape_ has quit IRC | 20:17 | |
| *** ducttape_ has joined #openstack-nova | 20:18 | |
| *** code-R_ has quit IRC | 20:19 | |
| *** johnsom has joined #openstack-nova | 20:19 | |
| *** code-R has joined #openstack-nova | 20:20 | |
| *** bvanhav_ has quit IRC | 20:21 | |
| *** morgan has joined #openstack-nova | 20:25 | |
| *** code-R has quit IRC | 20:25 | |
| *** code-R has joined #openstack-nova | 20:25 | |
| openstackgerrit | Sujitha proposed openstack/nova: Do not merge: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 20:28 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix placement API version history 1.1 title https://review.openstack.org/406347 | 20:28 |
| mriedem | leakypipes: easy ^ | 20:28 |
| leakypipes | mriedem: +W'd | 20:29 |
| *** code-R_ has joined #openstack-nova | 20:30 | |
| *** ijw has quit IRC | 20:31 | |
| mriedem | cool, custom resource classes rest api stack has +2s all the way up now | 20:32 |
| *** code-R has quit IRC | 20:33 | |
| leakypipes | mriedem: and turtles all the way down. | 20:36 |
| *** unicell has joined #openstack-nova | 20:38 | |
| superdan | sdague: diana_clarke: I figured it out | 20:44 |
| superdan | the bug was "dansmith being a dumbass", pretty much like I said | 20:44 |
| diana_clarke | superdan: oh man... I tried hard to find it too. WhaT did I miss? | 20:45 |
| superdan | diana_clarke: I'm caching the results of the cell list globally for performance at runtime, and not resetting it, so mock or no mock, you get the cached list of mocked cells once and you're dead after that | 20:46 |
| superdan | so obvious it was hard to spot | 20:46 |
| diana_clarke | superdan: where is the cache? | 20:47 |
| superdan | https://review.openstack.org/#/c/396775/3/nova/compute/api.py | 20:47 |
| superdan | that comment could probably use some self-deprecating humor now | 20:47 |
| leakypipes | :) | 20:48 |
| *** _ducttape_ has joined #openstack-nova | 20:49 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: move console opt to console group https://review.openstack.org/389647 | 20:49 |
| superdan | that has probably been the source of weeks of frustration for me, | 20:49 |
| mriedem | but, | 20:49 |
| superdan | although I've cleaned up a lot of stuff to make it easier to find as a result, so not a total loss | 20:49 |
| mriedem | you feel good now | 20:49 |
| mriedem | like, real good | 20:49 |
| superdan | I feel like kicking myself in the nuts.. but I'll feel better later | 20:49 |
| mriedem | <eyebrows> | 20:49 |
| leakypipes | superdan: naming and cache invalidation and all that.... :) | 20:50 |
| *** lpetrut has joined #openstack-nova | 20:50 | |
| superdan | leakypipes: this is too dumb of a thing to even blame on cache invalidation, but.. sure | 20:50 |
| superdan | when I explain to to people later I'll gloss over some details to make it sound hard | 20:51 |
| mriedem | https://www.youtube.com/watch?v=BI5IA8assfk ? | 20:51 |
| *** amoralej is now known as amoralej|off | 20:51 | |
| *** Apoorva has quit IRC | 20:51 | |
| superdan | heh | 20:51 |
| *** ducttape_ has quit IRC | 20:51 | |
| mriedem | note, | 20:51 |
| mriedem | there are a lot of people in this video for one guy playing on a casio | 20:51 |
| superdan | hmm, almost looks like he's not really singing | 20:52 |
| leakypipes | superdan: http://media.vintagecotton.com/shirt/113/myballs02.jpg | 20:52 |
| mriedem | rip | 20:52 |
| superdan | leakypipes: heh, yeah | 20:52 |
| *** yamahata has quit IRC | 20:52 | |
| mriedem | i don't care if he's not really singing, he makes up for it with the baby blues and ripped up jeans | 20:53 |
| superdan | heh | 20:53 |
| superdan | which is pretty much what the producers were depending on | 20:53 |
| mriedem | welp, time for a palette cleanswer | 20:53 |
| mriedem | *cleanser | 20:53 |
| *** esberglu_ has quit IRC | 20:54 | |
| *** esberglu has joined #openstack-nova | 20:54 | |
| leakypipes | mriedem: might I suggest https://www.youtube.com/watch?v=fsgWUq0fdKk | 20:55 |
| *** esberglu_ has joined #openstack-nova | 20:56 | |
| mriedem | that's actually what it looks like at my house at night | 20:56 |
| leakypipes | lmao | 20:56 |
| mriedem | i've been trying to get laura to understand the reference for 5 years now | 20:57 |
| leakypipes | :) | 20:57 |
| leakypipes | it totally goes with the doberman "painting". | 20:57 |
| *** esbergl__ has joined #openstack-nova | 20:58 | |
| *** esberglu has quit IRC | 20:59 | |
| *** lpetrut has quit IRC | 21:00 | |
| superdan | ah, there's another issue | 21:00 |
| *** esberglu has joined #openstack-nova | 21:00 | |
| *** esberglu_ has quit IRC | 21:00 | |
| diana_clarke | sooo... is it bad that my microversion 2.39 patch merged in python-novaclient, but the microversion 2.39 work hasn't yet merged in nova itself? | 21:00 |
| diana_clarke | I probably should have had a depends on, huh? | 21:00 |
| diana_clarke | (simple tenant usage paging stuff) | 21:01 |
| *** _ducttape_ has quit IRC | 21:02 | |
| mriedem | :/ | 21:02 |
| *** ducttape_ has joined #openstack-nova | 21:02 | |
| mriedem | yeah we'll likely need to revert that | 21:02 |
| superdan | quick ninja revert? | 21:02 |
| superdan | yeah | 21:02 |
| mriedem | was there a functional test? | 21:02 |
| *** esbergl__ has quit IRC | 21:02 | |
| mriedem | b/c there should be if not for any microversion change in the client | 21:03 |
| mriedem | which would require the depends-on | 21:03 |
| diana_clarke | mriedem: no there are no functional tests for usage, but I could add some | 21:04 |
| superdan | diana_clarke: post the revert and we'll fast approve as the first step I think | 21:05 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient: Revert "Microversion 2.39 - Simple tenant usage pagination" https://review.openstack.org/406359 | 21:05 |
| mriedem | ^ | 21:05 |
| superdan | got it | 21:05 |
| *** karts has quit IRC | 21:06 | |
| openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Revert "Microversion 2.39 - Simple tenant usage pagination" https://review.openstack.org/406360 | 21:06 |
| sdague | ah, ok, I had assumed that was already in, my bad for not doulbe checking that | 21:06 |
| superdan | diana_clarke: sorry, mriedem beat you to it | 21:06 |
| diana_clarke | oops | 21:06 |
| mriedem | sdague: you can make up for it by reviewing the server side changes which are probably much hairier :) | 21:07 |
| superdan | diana_clarke: re-propose your change on top of the revert, and then enjoy unlimited review karma from sdague on the server side patch :) | 21:07 |
| *** ihrachys has joined #openstack-nova | 21:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/405829 | 21:08 |
| *** karts has joined #openstack-nova | 21:09 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: correct improper test case inheritance https://review.openstack.org/406363 | 21:09 |
| leakypipes | superdan, mriedem: easy win ^ | 21:09 |
| diana_clarke | superdan, mriedem, sdague: So sorry for not having a depends on there. I'm not sure how to resurrect the client patch now, but I'll circle back Monday (it's baby's b-day today). | 21:10 |
| mriedem | diana_clarke: you basically just revert the revert | 21:10 |
| mriedem | but remove all of the 'revert' language in it | 21:10 |
| superdan | well, don't do that, | 21:10 |
| superdan | just cherry-pick the original on top of the revert | 21:10 |
| superdan | anyway, the revert is the important part | 21:10 |
| mriedem | or that, either way it needs a functional test | 21:10 |
| superdan | time wise | 21:10 |
| *** karts has quit IRC | 21:13 | |
| *** gouthamr has quit IRC | 21:14 | |
| diana_clarke | Okie dokie, thanks. I'll see if I can do that quick before I spring baby from daycare. | 21:15 |
| diana_clarke | oh... but the revert has to merge first | 21:17 |
| *** xinliang has quit IRC | 21:17 | |
| superdan | diana_clarke: you can still stack it on top of mriedem's revert, but.. not a huge rush of course | 21:17 |
| *** thorst_ has quit IRC | 21:18 | |
| openstackgerrit | jichenjc proposed openstack/nova: WIP: avoid set error state when resize fail https://review.openstack.org/406365 | 21:18 |
| *** thorst_ has joined #openstack-nova | 21:18 | |
| *** ihrachys has quit IRC | 21:19 | |
| oomichi | gmann: re: https://review.openstack.org/#/c/402295 At the time, my brain was shutdown. LGTM now | 21:21 |
| *** yamahata has joined #openstack-nova | 21:22 | |
| *** READ10 has quit IRC | 21:24 | |
| *** thorst_ has quit IRC | 21:27 | |
| *** karts has joined #openstack-nova | 21:28 | |
| *** xinliang has joined #openstack-nova | 21:29 | |
| *** mingyu has quit IRC | 21:29 | |
| *** Zer0Byte__ has quit IRC | 21:29 | |
| *** Zer0Byte__ has joined #openstack-nova | 21:31 | |
| *** Zer0Byte__ has quit IRC | 21:31 | |
| *** ijw has joined #openstack-nova | 21:32 | |
| *** Sukhdev has joined #openstack-nova | 21:33 | |
| *** kfarr has quit IRC | 21:34 | |
| *** ijw has quit IRC | 21:38 | |
| *** code-R_ has quit IRC | 21:43 | |
| *** rfolco has quit IRC | 21:48 | |
| *** esberglu has quit IRC | 21:48 | |
| *** esberglu has joined #openstack-nova | 21:49 | |
| *** thorst_ has joined #openstack-nova | 21:51 | |
| *** rfolco has joined #openstack-nova | 21:51 | |
| *** rfolco has quit IRC | 21:51 | |
| *** Apoorva has joined #openstack-nova | 21:52 | |
| *** moshele has quit IRC | 21:53 | |
| *** esberglu has quit IRC | 21:53 | |
| *** ijw has joined #openstack-nova | 21:55 | |
| *** Apoorva has quit IRC | 21:56 | |
| *** daemontool has joined #openstack-nova | 22:04 | |
| *** ljxiash has joined #openstack-nova | 22:07 | |
| *** Apoorva has joined #openstack-nova | 22:08 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Restict usage *args for novaclient.client.Client https://review.openstack.org/406294 | 22:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Introduce helper for checking args deprecation https://review.openstack.org/406295 | 22:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Sort arguments for client's methods https://review.openstack.org/406293 | 22:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make SessionClient interface similar to HTTPClient https://review.openstack.org/350140 | 22:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename api_key to password https://review.openstack.org/406298 | 22:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename interface to endpoint_type https://review.openstack.org/406299 | 22:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Remove redundant args of _construct_http_client https://review.openstack.org/406296 | 22:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Rename bypass_url to endpoint_override https://review.openstack.org/406297 | 22:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Clarify meaning of project_id var https://review.openstack.org/350106 | 22:08 |
| *** ljxiash has quit IRC | 22:12 | |
| *** _ducttape_ has joined #openstack-nova | 22:12 | |
| *** _ducttape_ has quit IRC | 22:12 | |
| *** _ducttape_ has joined #openstack-nova | 22:12 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Deny invalid flavor extraspec for cpu_policy https://review.openstack.org/406377 | 22:12 |
| *** thorst_ has quit IRC | 22:13 | |
| *** thorst_ has joined #openstack-nova | 22:14 | |
| *** Sukhdev has quit IRC | 22:14 | |
| *** ducttape_ has quit IRC | 22:14 | |
| *** ijw has quit IRC | 22:17 | |
| andreykurilin | mriedem: it looks like we have broken server pagination now:( all my patches and your revert to novaclient failed with same error | 22:17 |
| *** cleong has quit IRC | 22:18 | |
| *** thorst_ has quit IRC | 22:19 | |
| oomichi | alex_xu: I saw the similar patch of https://review.openstack.org/#/c/404565 before, I have alternative way as the comment | 22:19 |
| mriedem | andreykurilin: paging where? on the server side or just in the client? | 22:21 |
| andreykurilin | mriedem: at server side | 22:22 |
| *** jamielennox|away is now known as jamielennox | 22:22 | |
| andreykurilin | mriedem: see a failure message here - http://logs.openstack.org/59/406359/1/check/gate-novaclient-dsvm-functional-neutron/d8c0531/testr_results.html.gz | 22:22 |
| andreykurilin | mriedem: novaclient's functional job for mitaka is green | 22:23 |
| andreykurilin | so pagination at novaclient's side is not broken | 22:23 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Do not merge: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 22:24 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22ERROR%20(BadRequest)%3A%20marker%5C%22%20AND%20message%3A%5C%22not%20found%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d | 22:24 |
| *** stewie925 has joined #openstack-nova | 22:25 | |
| mriedem | andreykurilin: looking through the nova changes merged in the last 24 hours it's not jumping out at me | 22:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method https://review.openstack.org/367557 | 22:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Multicell support for instance listing https://review.openstack.org/396775 | 22:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add SecurityGroup.identifier to prefer uuid over name https://review.openstack.org/406260 | 22:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add SingleCellSimple fixture https://review.openstack.org/405684 | 22:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 22:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix up non-cells-aware context managers in test_db_api https://review.openstack.org/406379 | 22:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Setup CellsV2 environment in base test https://review.openstack.org/406380 | 22:26 |
| superdan | getting closer | 22:26 |
| mriedem | andreykurilin: might be something in here https://github.com/openstack/nova/commit/365fcedf63ff37edc91af71e55466fa9c42b7eb7 | 22:28 |
| andreykurilin | mriedem: I can propose a revert to nova's side and make novaclient's change with depends-on | 22:29 |
| *** _ducttape_ has quit IRC | 22:30 | |
| mriedem | andreykurilin: yeah probably a good idea, i'm not seeing it | 22:31 |
| andreykurilin | ok...it is not as easy as I expected:( it has conflicts... | 22:33 |
| *** stewie925 has quit IRC | 22:37 | |
| openstackgerrit | Andrey Kurilin proposed openstack/nova: Revert "conf: Move api options to a group" https://review.openstack.org/406387 | 22:40 |
| mriedem | andreykurilin: i've got to run, but will maybe be on later | 22:41 |
| andreykurilin | mriedem: np. have a nice evening and weekends :) | 22:42 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [DO NOT MERGE] Check nova's revert https://review.openstack.org/406388 | 22:43 |
| *** Sukhdev has joined #openstack-nova | 22:43 | |
| *** tblakes has quit IRC | 22:44 | |
| *** mriedem has quit IRC | 22:44 | |
| *** tbachman has quit IRC | 22:45 | |
| *** alexpilotti has joined #openstack-nova | 22:46 | |
| *** sdague has quit IRC | 22:50 | |
| *** alexpilotti has quit IRC | 22:50 | |
| *** erlon-airlong has quit IRC | 22:52 | |
| *** Swami__ has joined #openstack-nova | 22:53 | |
| *** alexpilotti has joined #openstack-nova | 22:54 | |
| *** dave-mccowan has quit IRC | 22:55 | |
| *** Swami_ has quit IRC | 22:56 | |
| *** Swami has quit IRC | 22:56 | |
| *** mdrabe has quit IRC | 22:57 | |
| *** alexpilotti has quit IRC | 22:58 | |
| *** Swami__ has quit IRC | 22:59 | |
| *** burt has quit IRC | 23:00 | |
| *** ducttape_ has joined #openstack-nova | 23:01 | |
| *** alexpilotti has joined #openstack-nova | 23:02 | |
| *** xyang1 has quit IRC | 23:02 | |
| oomichi | gmann: commented on https://review.openstack.org/#/c/397034 | 23:05 |
| *** daemontool has quit IRC | 23:13 | |
| *** figleaf is now known as edleafe | 23:14 | |
| *** ijw has joined #openstack-nova | 23:15 | |
| *** dave-mccowan has joined #openstack-nova | 23:16 | |
| *** ijw has quit IRC | 23:16 | |
| *** ducttape_ has quit IRC | 23:17 | |
| *** lamt has quit IRC | 23:23 | |
| *** hongbin has quit IRC | 23:26 | |
| *** Sukhdev has quit IRC | 23:28 | |
| *** thorst_ has joined #openstack-nova | 23:31 | |
| *** thorst_ has quit IRC | 23:33 | |
| *** thorst_ has joined #openstack-nova | 23:34 | |
| *** thorst_ has quit IRC | 23:38 | |
| *** rama_y has quit IRC | 23:38 | |
| *** thorst_ has joined #openstack-nova | 23:39 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: [WIP] Placement api: Add json_error_formatter to defaults https://review.openstack.org/395194 | 23:43 |
| *** ducttape_ has joined #openstack-nova | 23:48 | |
| *** ijw has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
