*** thorst has joined #openstack-nova | 00:00 | |
*** salv-orl_ has joined #openstack-nova | 00:01 | |
*** heyongli has quit IRC | 00:02 | |
*** heyongli has joined #openstack-nova | 00:02 | |
*** salv-orlando has quit IRC | 00:04 | |
*** salv-orl_ has quit IRC | 00:04 | |
*** pushkaru has joined #openstack-nova | 00:06 | |
*** heyongli has quit IRC | 00:12 | |
*** Swami has quit IRC | 00:12 | |
*** heyongli has joined #openstack-nova | 00:13 | |
*** ljxiash has joined #openstack-nova | 00:15 | |
*** zenoway has joined #openstack-nova | 00:17 | |
*** ljxiash has quit IRC | 00:20 | |
*** zenoway has quit IRC | 00:22 | |
*** heyongli has quit IRC | 00:22 | |
*** gongysh_ has joined #openstack-nova | 00:22 | |
*** heyongli has joined #openstack-nova | 00:23 | |
*** fawadkhaliq has quit IRC | 00:23 | |
*** ivase_ has joined #openstack-nova | 00:23 | |
*** ducttape_ has quit IRC | 00:24 | |
*** ivase has quit IRC | 00:26 | |
*** daemontool_ has joined #openstack-nova | 00:26 | |
*** q3k has quit IRC | 00:26 | |
*** BobBall_ has joined #openstack-nova | 00:27 | |
*** pushkaru has quit IRC | 00:27 | |
*** daemontool has quit IRC | 00:27 | |
*** BobBall_ is now known as Guest28565 | 00:27 | |
*** q3k has joined #openstack-nova | 00:28 | |
*** tbachman has quit IRC | 00:30 | |
*** BobBall has quit IRC | 00:30 | |
*** angdraug has quit IRC | 00:31 | |
*** Guest28565 has quit IRC | 00:31 | |
*** BobBall has joined #openstack-nova | 00:32 | |
*** BobBall is now known as Guest74619 | 00:32 | |
*** heyongli has quit IRC | 00:33 | |
*** heyongli has joined #openstack-nova | 00:33 | |
*** aginwala has joined #openstack-nova | 00:35 | |
*** ssurana has quit IRC | 00:37 | |
*** MVenesio has joined #openstack-nova | 00:38 | |
mriedem | mdbooth: i backported that recent fix to liberty, required backporting 2 things to keep the conflicts to a minimum https://review.openstack.org/#/c/295583/ | 00:40 |
---|---|---|
*** eliqiao has quit IRC | 00:42 | |
*** ducttape_ has joined #openstack-nova | 00:43 | |
*** heyongli has quit IRC | 00:43 | |
*** MVenesio has quit IRC | 00:43 | |
*** eliqiao has joined #openstack-nova | 00:43 | |
*** heyongli has joined #openstack-nova | 00:43 | |
*** zz_dimtruck is now known as dimtruck | 00:44 | |
openstackgerrit | Wenzhi Yu proposed openstack/nova: Set 'libvirt.sysinfo_serial' to 'none' in RealTimeServersTest https://review.openstack.org/295118 | 00:45 |
*** fawadkhaliq has joined #openstack-nova | 00:47 | |
oomichi | alex_xu: morning | 00:50 |
*** gongysh_ has quit IRC | 00:50 | |
oomichi | gmann: morning | 00:51 |
gmann | oomichi: morning | 00:52 |
oomichi | gmann: do you know sdague's PoC detail about api doc | 00:53 |
oomichi | gmann: ? | 00:53 |
*** zenoway has joined #openstack-nova | 00:53 | |
*** heyongli has quit IRC | 00:53 | |
oomichi | gmann: https://review.openstack.org/#/q/topic:wip_api_docs | 00:53 |
*** heyongli has joined #openstack-nova | 00:53 | |
oomichi | gmann: I am trying to understand it and want some hints if you know | 00:54 |
oomichi | sdague is off now | 00:54 |
gmann | oomichi: oh, actually i did not look into those yet. | 00:54 |
*** hoangcx has joined #openstack-nova | 00:54 | |
oomichi | gmann: ok, thanks for your response. I am trying to do that :) | 00:55 |
gmann | oomichi: ok, I will try to understand those today. | 00:55 |
gmann | and in tomorrow meeting we can get more info from Sean | 00:56 |
*** slaweq_ has quit IRC | 00:57 | |
*** dims has joined #openstack-nova | 00:57 | |
*** slaweq_ has joined #openstack-nova | 00:58 | |
*** zenoway has quit IRC | 00:58 | |
*** heyongli has quit IRC | 01:03 | |
*** heyongli has joined #openstack-nova | 01:04 | |
*** fawadkhaliq has quit IRC | 01:05 | |
*** fawadkhaliq has joined #openstack-nova | 01:06 | |
*** nic has quit IRC | 01:08 | |
openstackgerrit | Wenzhi Yu proposed openstack/nova-specs: Modify Nova API to reject invalid operations on physical servers https://review.openstack.org/283904 | 01:08 |
*** ducttape_ has quit IRC | 01:09 | |
*** Sukhdev has quit IRC | 01:09 | |
*** mtanino has quit IRC | 01:10 | |
*** salv-orlando has joined #openstack-nova | 01:10 | |
*** Jeffrey4l__ has quit IRC | 01:11 | |
*** browne has quit IRC | 01:12 | |
*** ivase_ has quit IRC | 01:13 | |
*** heyongli has quit IRC | 01:14 | |
*** ivase has joined #openstack-nova | 01:14 | |
*** heyongli has joined #openstack-nova | 01:14 | |
*** lucas_ has quit IRC | 01:14 | |
*** mriedem has quit IRC | 01:17 | |
*** ivase_ has joined #openstack-nova | 01:18 | |
*** ivase has quit IRC | 01:18 | |
*** apoorvad has quit IRC | 01:20 | |
*** ivase_ has quit IRC | 01:22 | |
*** heyongli has quit IRC | 01:24 | |
*** heyongli has joined #openstack-nova | 01:24 | |
*** salv-orlando has quit IRC | 01:26 | |
*** gongysh_ has joined #openstack-nova | 01:27 | |
*** ducttape_ has joined #openstack-nova | 01:27 | |
*** salv-orlando has joined #openstack-nova | 01:28 | |
*** EinstCrazy has joined #openstack-nova | 01:28 | |
*** ducttape_ has quit IRC | 01:29 | |
alex_xu | oomichi: morning | 01:29 |
*** lucas_ has joined #openstack-nova | 01:30 | |
oomichi | alex_xu: did you see sdague PoC related to API doc? | 01:31 |
*** zenoway has joined #openstack-nova | 01:32 | |
oomichi | alex_xu: I'd like to know the overview of that | 01:32 |
oomichi | alex_xu: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:wip_api_docs | 01:32 |
alex_xu | oomichi: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:wip_api_docs | 01:32 |
oomichi | alex_xu: yeah, that is | 01:32 |
alex_xu | oomichi: I didn't see all the part yet | 01:32 |
oomichi | alex_xu: Is that trying to get doc from the code? | 01:33 |
alex_xu | oomichi: I guess no, it will reuse the wadl doc | 01:33 |
alex_xu | probably convert wadl to rst | 01:33 |
oomichi | alex_xu: ah, WADL files come from api-site which is out of nova tree, right? | 01:34 |
*** lucas_ has quit IRC | 01:34 | |
alex_xu | oomichi: ye | 01:34 |
alex_xu | oomichi: yes | 01:34 |
oomichi | alex_xu: thanks, I got a little that | 01:34 |
*** heyongli has quit IRC | 01:34 | |
alex_xu | oomichi: np | 01:34 |
*** heyongli has joined #openstack-nova | 01:34 | |
alex_xu | oomichi: I take a look at swagger again yesterday, try to find out a way to support nova action, but failed again :( | 01:35 |
*** aginwala has quit IRC | 01:35 | |
oomichi | alex_xu: but I cannot get the point yet. api-site is wrong as sdague said on irc meeting. but the patches are tyring to convert from these files. | 01:35 |
*** aginwala has joined #openstack-nova | 01:35 | |
*** zenoway has quit IRC | 01:36 | |
alex_xu | oomichi: I think that is the initial doc, copy from wadl, then fix them | 01:36 |
*** lucas_ has joined #openstack-nova | 01:36 | |
alex_xu | oomichi: I didn't remember detail, what kind of wrong sdague pointed? | 01:37 |
*** ducttape_ has joined #openstack-nova | 01:38 | |
oomichi | alex_xu: http://eavesdrop.openstack.org/meetings/nova_api/2016/nova_api.2016-03-16-13.00.log.html | 01:38 |
*** Daisy has joined #openstack-nova | 01:38 | |
*** Daisy has quit IRC | 01:38 | |
*** Daisy has joined #openstack-nova | 01:39 | |
alex_xu | oomichi: I guess he point to the toolchain | 01:39 |
oomichi | alex_xu: "1) the api-site is very wrong and very confusing, and being in wadl with a toolchain no one understands, is very hard to fix" | 01:39 |
alex_xu | oomichi: the content we already have in the wadl doc can be re-used | 01:40 |
*** lucas_ has quit IRC | 01:41 | |
*** EinstCra_ has joined #openstack-nova | 01:41 | |
oomichi | alex_xu: yeah, I see the point now. maybe we need to migrate the content of api-site into nova tree, and we can fix them as your understanding, right? | 01:41 |
alex_xu | oomichi: yes | 01:41 |
oomichi | alex_xu: sdague's patches are doing that | 01:42 |
oomichi | as base part | 01:42 |
alex_xu | oomichi: yes, i guess so | 01:42 |
*** EinstCr__ has joined #openstack-nova | 01:42 | |
*** EinstCrazy has quit IRC | 01:42 | |
*** salv-orlando has quit IRC | 01:43 | |
oomichi | and we need to fix them with... how-to | 01:43 |
oomichi | by-hands, by-swagger, or our original way | 01:43 |
*** yamahata has quit IRC | 01:43 | |
*** Daisy has quit IRC | 01:43 | |
alex_xu | oomichi: i guess Sean's point is we already familiar with rst | 01:44 |
alex_xu | like we already have api-concept doc write as rst | 01:44 |
*** heyongli has quit IRC | 01:44 | |
*** heyongli has joined #openstack-nova | 01:45 | |
alex_xu | oomichi: so another point for swagger, if we generate the swagger from code, how can we migrate existed wadl doc into the code? | 01:45 |
*** lucas_ has joined #openstack-nova | 01:45 | |
alex_xu | if the only way is by-hand, that probably is another huge works | 01:45 |
oomichi | alex_xu: yeah, I know the point from the doc viewpoint. | 01:46 |
*** EinstCra_ has quit IRC | 01:46 | |
*** lucas___ has joined #openstack-nova | 01:46 | |
oomichi | alex_xu: but from client viewpoint, there is another merit on swagger | 01:46 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Support "locked" filter when list servers https://review.openstack.org/292749 | 01:47 |
oomichi | alex_xu: if client gets swagger definition data from API endpoint, clients can avoid hard-code parameters on their own implementations. | 01:47 |
*** lucas_ has quit IRC | 01:48 | |
*** lucas___ has quit IRC | 01:48 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for driver attach_interfaces capability https://review.openstack.org/272471 | 01:48 |
*** lucas_ has joined #openstack-nova | 01:48 | |
alex_xu | oomichi: emm...I didn't get that, how client avoid hard-code parameters? | 01:48 |
oomichi | alex_xu: for example, magnum-ui(dashboard part of magnum) needs to implement available parameters on client side based on server side code. | 01:49 |
oomichi | alex_xu: like enum | 01:49 |
alex_xu | oomichi: ah, i see now | 01:49 |
oomichi | alex_xu: yeah, nova also contains available parameters on enum with json-schema as you know | 01:50 |
*** vishwanathj has quit IRC | 01:50 | |
alex_xu | oomichi: if we have more expectation more than documentation on swagger, I guess we will get more support | 01:50 |
oomichi | alex_xu: on current situation, we need to implement the same code on client and server sides | 01:50 |
oomichi | alex_xu: in addition, if server code is updated by adding available parameters, clients also need to be updated | 01:51 |
oomichi | alex_xu: but if server can provide swagger definition data to client, client can avoid doing that | 01:51 |
alex_xu | oomichi: yeah, that is good point | 01:51 |
oomichi | alex_xu: yeah, that is a reason why our session consists of magnum-core also :-) | 01:52 |
*** Daisy has joined #openstack-nova | 01:52 | |
oomichi | alex_xu: this idea came from him | 01:52 |
alex_xu | oomichi: another big problem is support 'nova action', as Sean's point if we extend swagger, then we will drop the ecosystem of swagger | 01:52 |
oomichi | alex_xu: yeah, action APIs always become a problem | 01:53 |
*** lucas_ has quit IRC | 01:53 | |
*** Daisy_ has joined #openstack-nova | 01:53 | |
oomichi | alex_xu: I am guessing future microversions will change these APIs for fitting them to standard APIs which fit to swagger also | 01:54 |
alex_xu | oomichi: yes, maybe, but for now, it is a problem | 01:54 |
*** lucas-dinner has quit IRC | 01:55 | |
*** heyongli has quit IRC | 01:55 | |
oomichi | alex_xu: yeah, I have the same feeling. but I'd like to try finding a way on current APIs this week :-) | 01:55 |
*** heyongli has joined #openstack-nova | 01:55 | |
alex_xu | oomichi: cool :) | 01:55 |
*** Marga_ has quit IRC | 01:55 | |
*** zenoway has joined #openstack-nova | 01:56 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Pass BDM in swap_volume method of LibvirtDriver https://review.openstack.org/257135 | 01:56 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Add validations for volume_size and destination_type https://review.openstack.org/258788 | 01:56 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Replace assert statements in iptables and virt tests https://review.openstack.org/282992 | 01:56 |
oomichi | alex_xu: anyway, thanks for your advice, I can understand current situation based on yours ;) | 01:56 |
*** Marga_ has joined #openstack-nova | 01:57 | |
*** woodster_ has quit IRC | 01:57 | |
alex_xu | oomichi: np, happy to help on find out a good path for nova api doc | 01:57 |
*** Daisy has quit IRC | 01:57 | |
oomichi | alex_xu: yeah, interesting to do that for me :-) | 02:00 |
*** zenoway has quit IRC | 02:00 | |
*** lucasagomes has joined #openstack-nova | 02:00 | |
oomichi | alex_xu: btw can you take a look at https://review.openstack.org/#/c/295419/ ? that is a simple patch for doc | 02:00 |
alex_xu | oomichi: yeah, checking now | 02:01 |
*** Marga_ has quit IRC | 02:01 | |
*** aginwala has quit IRC | 02:02 | |
*** ircuser-1 has joined #openstack-nova | 02:02 | |
*** baoli has quit IRC | 02:03 | |
*** baoli has joined #openstack-nova | 02:04 | |
*** browne has joined #openstack-nova | 02:04 | |
*** heyongli has quit IRC | 02:05 | |
*** heyongli has joined #openstack-nova | 02:05 | |
*** vishwanathj has joined #openstack-nova | 02:05 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Expose quiesce/unquiesce API https://review.openstack.org/295595 | 02:05 |
*** baoli has quit IRC | 02:06 | |
*** ljxiash has joined #openstack-nova | 02:09 | |
*** ljxiash has quit IRC | 02:09 | |
*** ljxiash has joined #openstack-nova | 02:09 | |
*** thorst has quit IRC | 02:10 | |
oomichi | alex_xu: thanks for quick reviewing :-) | 02:11 |
alex_xu | oomichi: np :) | 02:11 |
*** thorst has joined #openstack-nova | 02:11 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Expose quiesce/unquiesce API https://review.openstack.org/295595 | 02:13 |
*** ducttape_ has quit IRC | 02:13 | |
*** knikolla has quit IRC | 02:15 | |
*** heyongli has quit IRC | 02:15 | |
*** ducttape_ has joined #openstack-nova | 02:15 | |
*** aginwala has joined #openstack-nova | 02:15 | |
*** heyongli has joined #openstack-nova | 02:16 | |
*** ducttape_ has joined #openstack-nova | 02:16 | |
*** baoli has joined #openstack-nova | 02:17 | |
*** Daisy_ has quit IRC | 02:18 | |
*** Daisy has joined #openstack-nova | 02:18 | |
*** thorst has quit IRC | 02:19 | |
*** unicell has quit IRC | 02:20 | |
*** fawadkhaliq has quit IRC | 02:24 | |
*** houming has joined #openstack-nova | 02:24 | |
*** heyongli has quit IRC | 02:25 | |
*** heyongli has joined #openstack-nova | 02:26 | |
*** ducttape_ has quit IRC | 02:27 | |
*** Daisy has quit IRC | 02:28 | |
*** Daisy has joined #openstack-nova | 02:28 | |
*** dims has quit IRC | 02:28 | |
*** Marga_ has joined #openstack-nova | 02:28 | |
*** tbachman has joined #openstack-nova | 02:32 | |
*** Marga_ has quit IRC | 02:33 | |
*** baoli_ has joined #openstack-nova | 02:33 | |
*** heyongli has quit IRC | 02:36 | |
*** heyongli has joined #openstack-nova | 02:36 | |
*** baoli has quit IRC | 02:37 | |
*** LZ has joined #openstack-nova | 02:38 | |
*** salv-orlando has joined #openstack-nova | 02:39 | |
*** MVenesio has joined #openstack-nova | 02:39 | |
*** Daisy has quit IRC | 02:42 | |
*** Daisy has joined #openstack-nova | 02:42 | |
*** Fdaisuke has joined #openstack-nova | 02:43 | |
*** mylu has quit IRC | 02:43 | |
*** aginwala has quit IRC | 02:44 | |
*** MVenesio has quit IRC | 02:44 | |
*** unicell has joined #openstack-nova | 02:45 | |
*** mylu has joined #openstack-nova | 02:46 | |
*** coreywright has quit IRC | 02:46 | |
*** heyongli has quit IRC | 02:46 | |
*** heyongli has joined #openstack-nova | 02:46 | |
*** vishwana_ has joined #openstack-nova | 02:47 | |
*** vishwana_ is now known as vishwanathj__ | 02:48 | |
*** mylu has quit IRC | 02:48 | |
*** mylu has joined #openstack-nova | 02:48 | |
*** salv-orlando has quit IRC | 02:50 | |
*** vishwanathj has quit IRC | 02:50 | |
*** vishwanathj__ is now known as vishwanathj | 02:53 | |
*** ducttape_ has joined #openstack-nova | 02:53 | |
*** artom has quit IRC | 02:53 | |
*** artom has joined #openstack-nova | 02:54 | |
*** rfolco has quit IRC | 02:55 | |
*** heyongli has quit IRC | 02:56 | |
*** heyongli has joined #openstack-nova | 02:57 | |
*** ducttape_ has quit IRC | 02:57 | |
*** hoangcx has quit IRC | 02:59 | |
*** hoangcx has joined #openstack-nova | 02:59 | |
*** fawadkhaliq has joined #openstack-nova | 03:00 | |
*** lucas_ has joined #openstack-nova | 03:01 | |
*** mspreitz has joined #openstack-nova | 03:02 | |
*** coreywright has joined #openstack-nova | 03:05 | |
*** baoli_ has quit IRC | 03:05 | |
*** lucas___ has joined #openstack-nova | 03:05 | |
*** lucas_ has quit IRC | 03:06 | |
*** baoli has joined #openstack-nova | 03:06 | |
*** heyongli has quit IRC | 03:06 | |
*** salv-orlando has joined #openstack-nova | 03:07 | |
*** heyongli has joined #openstack-nova | 03:07 | |
*** Marga_ has joined #openstack-nova | 03:10 | |
*** lucas___ has quit IRC | 03:10 | |
*** vishwanathj is now known as vishwanathj_zzz | 03:10 | |
*** ducttape_ has joined #openstack-nova | 03:11 | |
*** lucas_ has joined #openstack-nova | 03:12 | |
*** Marga_ has quit IRC | 03:14 | |
*** lucas_ has quit IRC | 03:16 | |
*** heyongli has quit IRC | 03:17 | |
*** baoli has quit IRC | 03:17 | |
*** thorst has joined #openstack-nova | 03:17 | |
*** heyongli has joined #openstack-nova | 03:17 | |
*** baoli has joined #openstack-nova | 03:17 | |
*** 20WAALIJF has joined #openstack-nova | 03:17 | |
*** unicell has quit IRC | 03:18 | |
*** unicell1 has joined #openstack-nova | 03:18 | |
*** aginwala has joined #openstack-nova | 03:19 | |
*** mylu has quit IRC | 03:19 | |
*** links has joined #openstack-nova | 03:19 | |
*** phuongnh has joined #openstack-nova | 03:21 | |
*** baoli has quit IRC | 03:22 | |
*** aginwala has quit IRC | 03:23 | |
*** baoli has joined #openstack-nova | 03:23 | |
*** salv-orlando has quit IRC | 03:24 | |
*** thorst has quit IRC | 03:24 | |
*** lucas_ has joined #openstack-nova | 03:25 | |
*** heyongli has quit IRC | 03:27 | |
*** heyongli has joined #openstack-nova | 03:27 | |
*** 20WAALIJF has quit IRC | 03:28 | |
*** ayoung has quit IRC | 03:29 | |
*** ljxiash has quit IRC | 03:30 | |
*** lucas_ has quit IRC | 03:30 | |
*** haomaiwang has joined #openstack-nova | 03:30 | |
*** Marga_ has joined #openstack-nova | 03:31 | |
*** ljxiash has joined #openstack-nova | 03:31 | |
*** Daisy has quit IRC | 03:34 | |
*** Daisy has joined #openstack-nova | 03:34 | |
*** ducttape_ has quit IRC | 03:35 | |
*** Marga_ has quit IRC | 03:35 | |
*** ljxiash has quit IRC | 03:35 | |
*** heyongli has quit IRC | 03:37 | |
*** heyongli has joined #openstack-nova | 03:37 | |
*** Daisy has quit IRC | 03:39 | |
*** tbachman has quit IRC | 03:41 | |
*** achanda has quit IRC | 03:43 | |
*** yamahata has joined #openstack-nova | 03:43 | |
*** dimtruck is now known as zz_dimtruck | 03:44 | |
*** mspreitz has quit IRC | 03:44 | |
*** pwnall has quit IRC | 03:45 | |
*** tbachman has joined #openstack-nova | 03:45 | |
*** Sukhdev has joined #openstack-nova | 03:47 | |
*** SamYaple is now known as contributed_this | 03:47 | |
*** heyongli has quit IRC | 03:47 | |
*** contributed_this is now known as SamYaple | 03:48 | |
*** heyongli has joined #openstack-nova | 03:48 | |
*** lucas_ has joined #openstack-nova | 03:49 | |
*** Marga_ has joined #openstack-nova | 03:52 | |
*** zenoway has joined #openstack-nova | 03:54 | |
*** tbachman_ has joined #openstack-nova | 03:57 | |
*** heyongli has quit IRC | 03:58 | |
*** heyongli has joined #openstack-nova | 03:58 | |
*** zenoway has quit IRC | 03:58 | |
*** tbachman has quit IRC | 03:59 | |
*** tbachman_ is now known as tbachman | 03:59 | |
*** haomaiwang has quit IRC | 04:01 | |
*** Marga_ has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-nova | 04:01 | |
*** tbachman has quit IRC | 04:01 | |
*** Marga_ has joined #openstack-nova | 04:01 | |
*** aswadr_ has joined #openstack-nova | 04:01 | |
*** adisky has joined #openstack-nova | 04:02 | |
*** baoli has quit IRC | 04:03 | |
*** heyongli has quit IRC | 04:08 | |
*** heyongli has joined #openstack-nova | 04:08 | |
*** jmhale has quit IRC | 04:11 | |
*** lucas_ has quit IRC | 04:15 | |
*** jmhale has joined #openstack-nova | 04:16 | |
*** heyongli has quit IRC | 04:18 | |
*** heyongli has joined #openstack-nova | 04:18 | |
*** gongysh_ has quit IRC | 04:20 | |
*** Sukhdev has quit IRC | 04:21 | |
*** Sukhdev has joined #openstack-nova | 04:21 | |
*** fawadkhaliq has quit IRC | 04:22 | |
*** fawadkhaliq has joined #openstack-nova | 04:22 | |
*** salv-orlando has joined #openstack-nova | 04:23 | |
*** thorst has joined #openstack-nova | 04:24 | |
*** heyongli has quit IRC | 04:28 | |
*** heyongli has joined #openstack-nova | 04:29 | |
*** thorst has quit IRC | 04:29 | |
*** salv-orlando has quit IRC | 04:30 | |
*** salv-orlando has joined #openstack-nova | 04:32 | |
*** achanda has joined #openstack-nova | 04:32 | |
*** heyongli has quit IRC | 04:39 | |
*** heyongli has joined #openstack-nova | 04:39 | |
*** MVenesio has joined #openstack-nova | 04:40 | |
*** MVenesio has quit IRC | 04:44 | |
*** heyongli has quit IRC | 04:49 | |
*** heyongli has joined #openstack-nova | 04:49 | |
*** klkumar has joined #openstack-nova | 04:51 | |
*** salv-orlando has quit IRC | 04:52 | |
*** daemontool_ has quit IRC | 04:53 | |
*** daemontool__ has joined #openstack-nova | 04:53 | |
*** sridharg has joined #openstack-nova | 04:55 | |
*** takashin has joined #openstack-nova | 04:57 | |
*** salv-orlando has joined #openstack-nova | 04:59 | |
*** heyongli has quit IRC | 04:59 | |
*** heyongli has joined #openstack-nova | 05:00 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-nova | 05:01 | |
*** Daisy has joined #openstack-nova | 05:01 | |
*** dave-mccowan has quit IRC | 05:03 | |
*** salv-orl_ has joined #openstack-nova | 05:04 | |
*** zz_dimtruck is now known as dimtruck | 05:05 | |
*** salv-orl_ has quit IRC | 05:06 | |
*** jwcroppe has quit IRC | 05:06 | |
*** salv-orlando has quit IRC | 05:09 | |
*** heyongli has quit IRC | 05:09 | |
*** heyongli has joined #openstack-nova | 05:10 | |
*** houming has quit IRC | 05:12 | |
openstackgerrit | Abhishek Talwar proposed openstack/python-novaclient: flavor-id should be only integer or UUID https://review.openstack.org/287702 | 05:14 |
*** klkumar has quit IRC | 05:17 | |
*** ljxiash has joined #openstack-nova | 05:18 | |
*** ljxiash has quit IRC | 05:19 | |
*** heyongli has quit IRC | 05:20 | |
*** heyongli has joined #openstack-nova | 05:20 | |
*** ljxiash has joined #openstack-nova | 05:22 | |
*** ljxiash has quit IRC | 05:23 | |
*** ljxiash has joined #openstack-nova | 05:23 | |
*** gongysh_ has joined #openstack-nova | 05:23 | |
*** dimtruck is now known as zz_dimtruck | 05:25 | |
*** thorst has joined #openstack-nova | 05:26 | |
*** houming has joined #openstack-nova | 05:27 | |
*** heyongli has quit IRC | 05:30 | |
*** heyongli has joined #openstack-nova | 05:30 | |
*** klkumar has joined #openstack-nova | 05:34 | |
*** thorst has quit IRC | 05:34 | |
*** houming has quit IRC | 05:36 | |
*** heyongli has quit IRC | 05:40 | |
*** heyongli has joined #openstack-nova | 05:41 | |
*** houming has joined #openstack-nova | 05:42 | |
*** heyongli has quit IRC | 05:50 | |
*** heyongli has joined #openstack-nova | 05:51 | |
*** fawadkhaliq has quit IRC | 05:52 | |
*** harlowja_at_home has quit IRC | 05:52 | |
*** zenoway has joined #openstack-nova | 05:53 | |
*** achanda has quit IRC | 05:55 | |
*** zenoway has quit IRC | 05:57 | |
*** haomaiwang has quit IRC | 06:01 | |
*** heyongli has quit IRC | 06:01 | |
*** lintan has quit IRC | 06:01 | |
*** heyongli has joined #openstack-nova | 06:01 | |
*** haomaiwang has joined #openstack-nova | 06:01 | |
*** gongysh_ has quit IRC | 06:02 | |
*** aginwala has joined #openstack-nova | 06:03 | |
*** salv-orlando has joined #openstack-nova | 06:07 | |
*** Daisy has quit IRC | 06:07 | |
*** Daisy has joined #openstack-nova | 06:08 | |
*** heyongli has quit IRC | 06:11 | |
*** heyongli has joined #openstack-nova | 06:11 | |
*** hoangcx has quit IRC | 06:15 | |
*** hoangcx has joined #openstack-nova | 06:16 | |
*** zz_dimtruck is now known as dimtruck | 06:17 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 06:21 |
*** heyongli has quit IRC | 06:21 | |
*** heyongli has joined #openstack-nova | 06:21 | |
*** salv-orlando has quit IRC | 06:24 | |
*** Daisy_ has joined #openstack-nova | 06:24 | |
*** Daisy has quit IRC | 06:27 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/293921 | 06:28 |
*** nkrinner has joined #openstack-nova | 06:29 | |
*** thorst has joined #openstack-nova | 06:31 | |
*** heyongli has quit IRC | 06:31 | |
*** heyongli has joined #openstack-nova | 06:32 | |
*** Daisy_ has quit IRC | 06:34 | |
*** Daisy has joined #openstack-nova | 06:34 | |
*** salv-orlando has joined #openstack-nova | 06:35 | |
*** thorst has quit IRC | 06:38 | |
*** MVenesio has joined #openstack-nova | 06:41 | |
*** salv-orlando has quit IRC | 06:41 | |
*** heyongli has quit IRC | 06:42 | |
*** heyongli has joined #openstack-nova | 06:42 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/nova: Fix multipath iSCSI encrypted volume attach failure https://review.openstack.org/196482 | 06:45 |
*** MVenesio has quit IRC | 06:45 | |
*** Daisy_ has joined #openstack-nova | 06:48 | |
*** dimtruck is now known as zz_dimtruck | 06:48 | |
*** Daisy has quit IRC | 06:50 | |
*** heyongli has quit IRC | 06:52 | |
openstackgerrit | garyk proposed openstack/nova: VMware: ensure volume is accessible by instance https://review.openstack.org/218639 | 06:52 |
*** heyongli has joined #openstack-nova | 06:52 | |
*** Fdaisuke_ has joined #openstack-nova | 06:54 | |
*** klkumar has quit IRC | 06:55 | |
*** Fdaisuke has quit IRC | 06:55 | |
openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: set service status based on vc connection https://review.openstack.org/290315 | 06:55 |
*** hoangcx has left #openstack-nova | 06:55 | |
*** hoangcx has quit IRC | 06:56 | |
*** hoangcx has joined #openstack-nova | 06:56 | |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-nova | 07:01 | |
*** davidlenwell has quit IRC | 07:01 | |
*** heyongli has quit IRC | 07:02 | |
*** heyongli has joined #openstack-nova | 07:02 | |
*** jwcroppe has joined #openstack-nova | 07:04 | |
*** Daisy_ has quit IRC | 07:07 | |
*** Daisy has joined #openstack-nova | 07:07 | |
*** aginwala has quit IRC | 07:07 | |
*** sridharg has quit IRC | 07:08 | |
*** sridharg has joined #openstack-nova | 07:08 | |
*** davidlenwell has joined #openstack-nova | 07:09 | |
*** zz_dimtruck is now known as dimtruck | 07:09 | |
*** Sukhdev has quit IRC | 07:11 | |
*** heyongli has quit IRC | 07:12 | |
*** heyongli has joined #openstack-nova | 07:13 | |
*** liverpooler has joined #openstack-nova | 07:22 | |
*** heyongli has quit IRC | 07:23 | |
*** heyongli has joined #openstack-nova | 07:23 | |
*** atuvenie has joined #openstack-nova | 07:23 | |
*** tesseract has joined #openstack-nova | 07:28 | |
*** tesseract is now known as Guest91251 | 07:28 | |
*** hoangcx_ has joined #openstack-nova | 07:30 | |
*** hoangcx has quit IRC | 07:31 | |
*** paul-carlton1 has joined #openstack-nova | 07:31 | |
*** hoangcx_ is now known as hoangcx | 07:32 | |
*** heyongli has quit IRC | 07:33 | |
*** klkumar has joined #openstack-nova | 07:33 | |
*** heyongli has joined #openstack-nova | 07:34 | |
*** ekuris_ has joined #openstack-nova | 07:35 | |
*** thorst has joined #openstack-nova | 07:36 | |
*** Daisy has quit IRC | 07:37 | |
*** Daisy has joined #openstack-nova | 07:38 | |
*** Daisy has quit IRC | 07:39 | |
*** rcernin has joined #openstack-nova | 07:39 | |
*** Daisy has joined #openstack-nova | 07:39 | |
*** heyongli has quit IRC | 07:43 | |
*** heyongli has joined #openstack-nova | 07:43 | |
*** thorst has quit IRC | 07:44 | |
*** jichen has joined #openstack-nova | 07:44 | |
*** Daisy_ has joined #openstack-nova | 07:45 | |
*** dimtruck is now known as zz_dimtruck | 07:46 | |
*** salv-orlando has joined #openstack-nova | 07:46 | |
*** salv-orlando has quit IRC | 07:48 | |
*** Daisy has quit IRC | 07:48 | |
*** moshele has joined #openstack-nova | 07:49 | |
*** ihrachys has joined #openstack-nova | 07:49 | |
*** daemontool__ has quit IRC | 07:50 | |
*** zenoway has joined #openstack-nova | 07:51 | |
*** mpavone has joined #openstack-nova | 07:52 | |
*** Kennan has quit IRC | 07:53 | |
*** salv-orlando has joined #openstack-nova | 07:53 | |
*** Kennan has joined #openstack-nova | 07:53 | |
*** heyongli has quit IRC | 07:53 | |
*** heyongli has joined #openstack-nova | 07:54 | |
*** belmoreira has joined #openstack-nova | 07:56 | |
*** liusheng has quit IRC | 07:56 | |
*** liusheng has joined #openstack-nova | 07:57 | |
*** browne has quit IRC | 07:58 | |
*** salv-orlando has quit IRC | 07:59 | |
*** scheuran has joined #openstack-nova | 08:00 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-nova | 08:01 | |
*** salv-orlando has joined #openstack-nova | 08:03 | |
*** heyongli has quit IRC | 08:04 | |
*** heyongli has joined #openstack-nova | 08:04 | |
*** drankis has joined #openstack-nova | 08:06 | |
*** haukebruno has joined #openstack-nova | 08:07 | |
*** esracelik has joined #openstack-nova | 08:07 | |
*** Daisy_ has quit IRC | 08:08 | |
*** Daisy has joined #openstack-nova | 08:09 | |
*** ljxiash has quit IRC | 08:10 | |
*** paul-carlton1 has quit IRC | 08:10 | |
*** ljxiash has joined #openstack-nova | 08:11 | |
*** Daisy has quit IRC | 08:13 | |
*** matrohon has joined #openstack-nova | 08:14 | |
*** salv-orlando has quit IRC | 08:14 | |
*** heyongli has quit IRC | 08:14 | |
*** heyongli has joined #openstack-nova | 08:14 | |
*** ljxiash has quit IRC | 08:16 | |
*** salv-orlando has joined #openstack-nova | 08:16 | |
*** markvoelker has quit IRC | 08:16 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 08:17 |
*** takashin has left #openstack-nova | 08:24 | |
*** takashin has joined #openstack-nova | 08:24 | |
*** takashin has left #openstack-nova | 08:24 | |
*** heyongli has quit IRC | 08:24 | |
*** heyongli has joined #openstack-nova | 08:25 | |
*** salv-orlando has quit IRC | 08:25 | |
*** paul-carlton1 has joined #openstack-nova | 08:27 | |
*** permalac has quit IRC | 08:33 | |
*** openstackgerrit has quit IRC | 08:33 | |
*** openstackgerrit has joined #openstack-nova | 08:34 | |
*** daemontool has joined #openstack-nova | 08:34 | |
*** heyongli has quit IRC | 08:34 | |
*** heyongli has joined #openstack-nova | 08:35 | |
*** jlanoux has joined #openstack-nova | 08:37 | |
*** zz_dimtruck is now known as dimtruck | 08:38 | |
*** Fdaisuke_ has quit IRC | 08:40 | |
*** pcaruana has joined #openstack-nova | 08:40 | |
*** Daisy has joined #openstack-nova | 08:40 | |
*** p01nt3r75 has joined #openstack-nova | 08:41 | |
*** MVenesio has joined #openstack-nova | 08:41 | |
*** thorst has joined #openstack-nova | 08:42 | |
*** Daisy has quit IRC | 08:45 | |
*** heyongli has quit IRC | 08:45 | |
*** heyongli has joined #openstack-nova | 08:45 | |
*** MVenesio has quit IRC | 08:46 | |
*** Daisy has joined #openstack-nova | 08:48 | |
*** thorst has quit IRC | 08:49 | |
*** rk4n has joined #openstack-nova | 08:51 | |
openstackgerrit | lyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked https://review.openstack.org/207373 | 08:53 |
*** heyongli has quit IRC | 08:55 | |
*** heyongli has joined #openstack-nova | 08:55 | |
*** paul-carlton1 has quit IRC | 08:57 | |
bauzas | good morning Nova | 08:57 |
* kashyap waves | 08:57 | |
*** daemontool has quit IRC | 08:57 | |
*** haomaiwang has quit IRC | 09:01 | |
*** RuiChen has joined #openstack-nova | 09:01 | |
*** 92AAALWH6 has joined #openstack-nova | 09:01 | |
*** achanda has joined #openstack-nova | 09:02 | |
*** achanda has quit IRC | 09:02 | |
*** mgoddard has joined #openstack-nova | 09:03 | |
*** doug-fish has quit IRC | 09:04 | |
*** heyongli has quit IRC | 09:05 | |
*** heyongli has joined #openstack-nova | 09:06 | |
*** ihrachys has quit IRC | 09:06 | |
*** rdopiera has joined #openstack-nova | 09:06 | |
openstackgerrit | Abhishek Talwar proposed openstack/python-novaclient: flavor postional argument in nova boot help https://review.openstack.org/295681 | 09:06 |
*** hoangcx has quit IRC | 09:13 | |
*** ihrachys has joined #openstack-nova | 09:15 | |
*** heyongli has quit IRC | 09:15 | |
*** heyongli has joined #openstack-nova | 09:16 | |
*** lpetrut has joined #openstack-nova | 09:16 | |
*** derekh has joined #openstack-nova | 09:17 | |
*** markvoelker has joined #openstack-nova | 09:17 | |
*** hoangcx has joined #openstack-nova | 09:18 | |
*** ndipanov has quit IRC | 09:18 | |
*** ndipanov has joined #openstack-nova | 09:18 | |
*** ndipanov has quit IRC | 09:18 | |
*** ndipanov has joined #openstack-nova | 09:19 | |
*** lpetrut has quit IRC | 09:21 | |
*** salv-orlando has joined #openstack-nova | 09:22 | |
*** e0ne has joined #openstack-nova | 09:22 | |
*** jaosorior has joined #openstack-nova | 09:23 | |
*** klkumar has quit IRC | 09:24 | |
*** sfinucan has joined #openstack-nova | 09:24 | |
*** rk4n has quit IRC | 09:25 | |
*** dimtruck is now known as zz_dimtruck | 09:26 | |
*** heyongli has quit IRC | 09:26 | |
*** heyongli has joined #openstack-nova | 09:26 | |
*** klkumar has joined #openstack-nova | 09:26 | |
*** rk4n has joined #openstack-nova | 09:26 | |
ndipanov | bauzas, I think you were looking into this at some point | 09:28 |
ndipanov | what do we use as a requirements file for gate devstack runs? | 09:29 |
*** zenoway has quit IRC | 09:30 | |
johnthetubaguy | ndipanov: we use the constraints file stuff now, its in tox.ini for the unit tests at least | 09:31 |
*** dedery has joined #openstack-nova | 09:31 | |
johnthetubaguy | or are you thinking the wider dvsm job? | 09:31 |
ndipanov | johnthetubaguy, tempest | 09:32 |
ndipanov | so unit tests don't spin up devstack I hope :) | 09:32 |
bauzas | ndipanov: I didn't get your question | 09:32 |
ndipanov | if they do I think I have an idea how to speed up our test runs :) | 09:32 |
johnthetubaguy | https://github.com/openstack/requirements/blob/master/global-requirements.txt | 09:32 |
bauzas | ndipanov: for unittests, we use constraints now | 09:32 |
ndipanov | bauzas, right | 09:32 |
ndipanov | but for devstack I understand we use upper-constraints.txt | 09:33 |
ndipanov | from requirements | 09:33 |
johnthetubaguy | we use those for unit tests as well | 09:33 |
ndipanov | so I was hoping someone knows exactly where this happens | 09:33 |
johnthetubaguy | I thought devstack was using global-requirements | 09:33 |
bauzas | https://github.com/openstack/nova/blob/master/tox.ini#L12 | 09:33 |
ndipanov | johnthetubaguy, I think dansmith told me that changed not too long ago | 09:33 |
bauzas | ndipanov: johnthetubaguy: ^ | 09:33 |
johnthetubaguy | right, that was a few weeks back, that tox.ini change was added | 09:34 |
johnthetubaguy | and we use constraints on top of our local requirements files | 09:34 |
bauzas | https://github.com/openstack/nova/commit/1f0dec3a0347462f554b8f8ea04e8dff50d831d4 | 09:34 |
johnthetubaguy | but for tempest tests, I thought we still did global-requriements, and also with the constraints | 09:34 |
bauzas | which means both the dsvm and unittests jobs consistent | 09:34 |
bauzas | which makes* | 09:34 |
bauzas | johnthetubaguy: see tox.ini, we fetch the global reqs | 09:35 |
ndipanov | bauzas, ok so that's unit tests right | 09:35 |
bauzas | ndipanov: same goes for devstack afaik | 09:35 |
ndipanov | they use upper-constraints which is cool | 09:35 |
ndipanov | bauzas, can you point me to the code maybe | 09:35 |
ndipanov | it would make sense yes | 09:35 |
bauzas | ndipanov: I'm just digging that out :) | 09:35 |
ndipanov | bauzas, thanks | 09:36 |
bauzas | that's probably a job def | 09:36 |
*** heyongli has quit IRC | 09:36 | |
*** heyongli has joined #openstack-nova | 09:36 | |
*** yassine has joined #openstack-nova | 09:37 | |
*** rpodolyaka has quit IRC | 09:37 | |
*** Jeffrey4l has joined #openstack-nova | 09:37 | |
ndipanov | so I guess my next question is - why do we keep our local requirements file around then since it can be different to what is specified on test runs | 09:37 |
*** yassine is now known as Guest82492 | 09:37 | |
bauzas | ndipanov: that's the only leak proposal I found https://github.com/openstack-dev/devstack/blob/c615993fe16902bfc1ac91cad7da34084c77ed99/inc/python#L138 | 09:38 |
ndipanov | bauzas, yeah makes sense | 09:38 |
*** rpodolyaka has joined #openstack-nova | 09:38 | |
johnthetubaguy | ndipanov: I thought it was used for the unit test runs, I think there was talk of perproject venvs, but thats a whole heap of extra work | 09:38 |
bauzas | ndipanov: good question, I'd defer that to sdague | 09:38 |
bauzas | I'm not super skilled on that | 09:39 |
*** salv-orlando has quit IRC | 09:39 | |
ndipanov | johnthetubaguy, the reason I am asking is because distro guys may have hit a bug with an upper constraint | 09:39 |
* bauzas has to leave for an appointment, bbiab | 09:39 | |
ndipanov | and it's not obvious to them why the gate doesn't know about it | 09:39 |
*** rk4n has quit IRC | 09:39 | |
ndipanov | looking at requirements.txt in Nova | 09:39 |
*** rk4n has joined #openstack-nova | 09:39 | |
johnthetubaguy | ndipanov: the ops folks did mention something about per project requirements files being fine for unit tests, but not so good for running the full project, now you mention it | 09:40 |
*** zenoway has joined #openstack-nova | 09:41 | |
*** rk4n has quit IRC | 09:41 | |
ndipanov | well iiuc - gate is pegged to the version in upper-constraints which is the pip latest version when the change was made to the requirements repo | 09:42 |
ndipanov | but unless you have the exact same == requirement constraint in project repos | 09:42 |
ndipanov | you may be installing different things | 09:43 |
ndipanov | at some point | 09:43 |
ndipanov | which should be fine as that means | 09:43 |
ndipanov | if it fails we want to reflect that in the global requirements | 09:43 |
ndipanov | it's just not obvious right... | 09:43 |
*** claudiub has joined #openstack-nova | 09:43 | |
*** sdague has joined #openstack-nova | 09:44 | |
claudiub | johnthetubaguy: hi. here's the bp we talked about yesterday: https://blueprints.launchpad.net/nova/+spec/nova-python3-newton also, you could remove your -2s on the following patches: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/nova-python3-newton | 09:45 |
*** heyongli has quit IRC | 09:46 | |
*** heyongli has joined #openstack-nova | 09:47 | |
*** thorst has joined #openstack-nova | 09:47 | |
johnthetubaguy | claudiub: need to get it approved first, let me do that. | 09:47 |
*** claudiub|2 has joined #openstack-nova | 09:49 | |
johnthetubaguy | claudiub: thanks for creating that bp, does that all look good now? | 09:49 |
*** dims has joined #openstack-nova | 09:51 | |
*** danpb has joined #openstack-nova | 09:51 | |
*** claudiub has quit IRC | 09:52 | |
*** markvoelker has quit IRC | 09:52 | |
claudiub|2 | johnthetubaguy: no problem, yep all good now. ty. :) | 09:53 |
claudiub|2 | johnthetubaguy: you commented on the patches that the bp is approved for mitaka, but it doesn't matter. :) | 09:53 |
johnthetubaguy | claudiub|2: heh, I did, oops | 09:54 |
*** thorst has quit IRC | 09:54 | |
* johnthetubaguy kicks time machine keyboard | 09:54 | |
*** danpb has quit IRC | 09:55 | |
*** heyongli has quit IRC | 09:56 | |
*** heyongli has joined #openstack-nova | 09:57 | |
*** EinstCr__ has quit IRC | 10:00 | |
*** 92AAALWH6 has quit IRC | 10:01 | |
*** 14WAAJIS3 has joined #openstack-nova | 10:01 | |
*** dims has quit IRC | 10:04 | |
*** heyongli has quit IRC | 10:07 | |
*** heyongli has joined #openstack-nova | 10:07 | |
*** danpb has joined #openstack-nova | 10:09 | |
*** Daisy has quit IRC | 10:10 | |
*** Daisy has joined #openstack-nova | 10:10 | |
*** dims has joined #openstack-nova | 10:12 | |
*** danpb has quit IRC | 10:14 | |
*** jaosorior has quit IRC | 10:15 | |
*** jaosorior has joined #openstack-nova | 10:15 | |
*** lpetrut has joined #openstack-nova | 10:16 | |
*** lpetrut has quit IRC | 10:17 | |
*** heyongli has quit IRC | 10:17 | |
*** lpetrut has joined #openstack-nova | 10:17 | |
*** heyongli has joined #openstack-nova | 10:17 | |
*** zz_dimtruck is now known as dimtruck | 10:18 | |
*** zenoway has quit IRC | 10:21 | |
openstackgerrit | jichenjc proposed openstack/nova: Add functional test for v2.7 https://review.openstack.org/292881 | 10:22 |
*** danpb has joined #openstack-nova | 10:26 | |
*** claudiub has joined #openstack-nova | 10:27 | |
*** heyongli has quit IRC | 10:27 | |
*** heyongli has joined #openstack-nova | 10:28 | |
*** cdent has joined #openstack-nova | 10:28 | |
*** claudiub|2 has quit IRC | 10:30 | |
*** achanda has joined #openstack-nova | 10:31 | |
*** achanda has quit IRC | 10:31 | |
*** danpb has quit IRC | 10:31 | |
*** salv-orlando has joined #openstack-nova | 10:33 | |
*** zenoway has joined #openstack-nova | 10:34 | |
*** 14WAAJIS3 has quit IRC | 10:36 | |
*** 7F1AAKFVO has joined #openstack-nova | 10:36 | |
*** 7F1AAKFVO has quit IRC | 10:36 | |
*** haomaiwang has joined #openstack-nova | 10:36 | |
*** haomaiwang has quit IRC | 10:37 | |
*** heyongli has quit IRC | 10:37 | |
*** heyongli has joined #openstack-nova | 10:38 | |
*** atuvenie has quit IRC | 10:39 | |
*** rk4n has joined #openstack-nova | 10:41 | |
*** MVenesio has joined #openstack-nova | 10:42 | |
openstackgerrit | Lenny Verkhovsky proposed openstack/nova: Added missing "=" in debug message https://review.openstack.org/295713 | 10:43 |
*** ihrachys has quit IRC | 10:43 | |
*** danpb has joined #openstack-nova | 10:46 | |
*** MVenesio has quit IRC | 10:47 | |
*** heyongli has quit IRC | 10:48 | |
*** smatzek has joined #openstack-nova | 10:48 | |
*** heyongli has joined #openstack-nova | 10:48 | |
*** ihrachys has joined #openstack-nova | 10:51 | |
*** thorst has joined #openstack-nova | 10:52 | |
*** dave-mcnally has quit IRC | 10:53 | |
*** hoangcx has quit IRC | 10:54 | |
*** dave-mcnally has joined #openstack-nova | 10:54 | |
*** klkumar has quit IRC | 10:55 | |
*** danpb has quit IRC | 10:56 | |
*** heyongli has quit IRC | 10:58 | |
*** heyongli has joined #openstack-nova | 10:58 | |
*** thorst has quit IRC | 10:59 | |
*** Daisy has quit IRC | 11:00 | |
*** Daisy has joined #openstack-nova | 11:01 | |
*** dimtruck is now known as zz_dimtruck | 11:05 | |
*** Daisy has quit IRC | 11:05 | |
*** atuvenie has joined #openstack-nova | 11:06 | |
*** heyongli has quit IRC | 11:08 | |
*** heyongli has joined #openstack-nova | 11:08 | |
*** danpb has joined #openstack-nova | 11:09 | |
*** phuongnh has quit IRC | 11:10 | |
*** mvk_ has joined #openstack-nova | 11:12 | |
*** daemontool has joined #openstack-nova | 11:13 | |
*** danpb has quit IRC | 11:13 | |
*** mvk has quit IRC | 11:15 | |
*** heyongli has quit IRC | 11:18 | |
*** heyongli has joined #openstack-nova | 11:19 | |
*** hamzy has quit IRC | 11:19 | |
*** jlanoux_ has joined #openstack-nova | 11:24 | |
*** danpb has joined #openstack-nova | 11:26 | |
*** jlanoux has quit IRC | 11:26 | |
openstackgerrit | Wenzhi Yu proposed openstack/nova: Rewrite 'test_filter_schedule_skipping' method using Mock https://review.openstack.org/285146 | 11:27 |
*** jlanoux_ has quit IRC | 11:28 | |
*** jlanoux has joined #openstack-nova | 11:29 | |
*** heyongli has quit IRC | 11:29 | |
*** heyongli has joined #openstack-nova | 11:29 | |
*** cdent has quit IRC | 11:31 | |
*** danpb has quit IRC | 11:33 | |
*** gcb has joined #openstack-nova | 11:36 | |
*** rtheis has joined #openstack-nova | 11:38 | |
*** heyongli has quit IRC | 11:39 | |
*** heyongli has joined #openstack-nova | 11:39 | |
*** jwcroppe has quit IRC | 11:40 | |
*** doug-fish has joined #openstack-nova | 11:42 | |
*** jlanoux_ has joined #openstack-nova | 11:42 | |
*** libu has joined #openstack-nova | 11:42 | |
*** houming has quit IRC | 11:44 | |
*** jlanoux has quit IRC | 11:45 | |
*** ihrachys has quit IRC | 11:46 | |
*** danpb has joined #openstack-nova | 11:47 | |
*** thorst has joined #openstack-nova | 11:47 | |
*** jwcroppe_ has joined #openstack-nova | 11:48 | |
*** markvoelker has joined #openstack-nova | 11:48 | |
*** heyongli has quit IRC | 11:49 | |
*** doug-fis_ has joined #openstack-nova | 11:50 | |
*** heyongli has joined #openstack-nova | 11:50 | |
*** vladikr has joined #openstack-nova | 11:51 | |
*** danpb has quit IRC | 11:51 | |
*** doug-fish has quit IRC | 11:52 | |
*** cdelatte has quit IRC | 11:52 | |
bauzas | markus_z: morning, could you please remind me what was the outcome of having wishlist bugs N | 11:54 |
bauzas | ? | 11:54 |
bauzas | markus_z: that one seems really a nit https://bugs.launchpad.net/nova/+bug/1560437 | 11:54 |
openstack | Launchpad bug 1560437 in OpenStack Compute (nova) "Logic of nova-cells test case 'test_cells_scheduler.test_filter_schedule_skipping' is odd" [Undecided,In progress] - Assigned to Wenzhi Yu (yuywz) | 11:54 |
bauzas | markus_z: so I'm tempted to either categorize it as wishlist or make it invalid | 11:54 |
*** jlanoux_ has quit IRC | 11:55 | |
*** jlanoux has joined #openstack-nova | 11:55 | |
*** zz_dimtruck is now known as dimtruck | 11:57 | |
*** cdent has joined #openstack-nova | 11:58 | |
pkholkin | johnthetubaguy: hello John! are you around? | 11:59 |
*** heyongli has quit IRC | 11:59 | |
*** heyongli has joined #openstack-nova | 12:00 | |
*** libu has quit IRC | 12:01 | |
*** salv-orl_ has joined #openstack-nova | 12:01 | |
*** ljjjustin has joined #openstack-nova | 12:02 | |
*** EinstCrazy has joined #openstack-nova | 12:03 | |
*** salv-orlando has quit IRC | 12:05 | |
*** danpb has joined #openstack-nova | 12:05 | |
*** moshele has quit IRC | 12:06 | |
*** aysyd has joined #openstack-nova | 12:06 | |
*** moshele has joined #openstack-nova | 12:06 | |
*** ihrachys has joined #openstack-nova | 12:06 | |
*** mvk_ has quit IRC | 12:07 | |
*** baoli has joined #openstack-nova | 12:10 | |
*** heyongli has quit IRC | 12:10 | |
*** heyongli has joined #openstack-nova | 12:10 | |
*** ducttape_ has joined #openstack-nova | 12:11 | |
*** danpb has quit IRC | 12:11 | |
*** cdelatte has joined #openstack-nova | 12:13 | |
*** purplerbot has quit IRC | 12:15 | |
*** purplerbot has joined #openstack-nova | 12:15 | |
*** purplerbot has quit IRC | 12:15 | |
*** purplerbot has joined #openstack-nova | 12:16 | |
dulek | Hi! In Cinder we're considering removal of <service>_topic options (like scheduler_topic). I know these are present in Nova. Is there a serious use case or reason to keep them? | 12:17 |
*** klkumar has joined #openstack-nova | 12:17 | |
*** delattec has joined #openstack-nova | 12:18 | |
*** heyongli has quit IRC | 12:20 | |
*** MVenesio has joined #openstack-nova | 12:20 | |
*** heyongli has joined #openstack-nova | 12:20 | |
*** delattec has quit IRC | 12:21 | |
*** cdelatte has quit IRC | 12:21 | |
*** markvoelker has quit IRC | 12:21 | |
*** pradk_ has joined #openstack-nova | 12:21 | |
*** hamzy has joined #openstack-nova | 12:21 | |
*** dave-mccowan has joined #openstack-nova | 12:22 | |
*** jwcroppe_ has quit IRC | 12:22 | |
*** ducttape_ has quit IRC | 12:23 | |
*** ljjjustin has quit IRC | 12:24 | |
*** danpb has joined #openstack-nova | 12:24 | |
*** ljjjustin has joined #openstack-nova | 12:28 | |
*** danpb has quit IRC | 12:29 | |
*** rfolco has joined #openstack-nova | 12:29 | |
markus_z | bauzas: hi, that specific bug report you linked is not valid imo. They should just push a patch for it. No need to go through bug triaging. | 12:29 |
markus_z | bauzas: The outcome of the general "wishlist" discussion is, if you see actual RFEs, mark it as wishlist. | 12:30 |
*** raildo-afk is now known as raildo | 12:30 | |
*** mtanino has joined #openstack-nova | 12:30 | |
markus_z | bauzas: The ops ML has a second discussion about how they wish to do it | 12:30 |
*** heyongli has quit IRC | 12:30 | |
*** heyongli has joined #openstack-nova | 12:30 | |
markus_z | bauzas: I'm not yet ready to completely close the "wishlist" channel without providing an accepted alternative. | 12:32 |
*** pradk_ is now known as pradk | 12:34 | |
*** mmm has joined #openstack-nova | 12:35 | |
*** ljxiash has joined #openstack-nova | 12:39 | |
openstackgerrit | Wenzhi Yu proposed openstack/nova: Set 'libvirt.sysinfo_serial' to 'none' in RealTimeServersTest https://review.openstack.org/295118 | 12:39 |
*** heyongli has quit IRC | 12:40 | |
*** heyongli has joined #openstack-nova | 12:41 | |
*** danpb has joined #openstack-nova | 12:42 | |
*** ljxiash has quit IRC | 12:43 | |
*** dimtruck is now known as zz_dimtruck | 12:44 | |
*** danpb has quit IRC | 12:47 | |
*** rlrossit has joined #openstack-nova | 12:47 | |
*** markvoelker has joined #openstack-nova | 12:47 | |
*** efried has joined #openstack-nova | 12:48 | |
*** moshele has quit IRC | 12:49 | |
*** moshele has joined #openstack-nova | 12:50 | |
*** paul-carlton1 has joined #openstack-nova | 12:50 | |
*** yuywz has quit IRC | 12:51 | |
*** heyongli has quit IRC | 12:51 | |
*** heyongli has joined #openstack-nova | 12:51 | |
*** klkumar has quit IRC | 12:51 | |
*** paul-carlton2 has quit IRC | 12:52 | |
*** paul-carlton has quit IRC | 12:52 | |
*** dave-mcnally has quit IRC | 12:52 | |
*** dave-mcnally has joined #openstack-nova | 12:52 | |
*** danpb has joined #openstack-nova | 12:52 | |
*** paul-carlton has joined #openstack-nova | 12:53 | |
*** klkumar has joined #openstack-nova | 12:53 | |
*** efried has quit IRC | 12:54 | |
*** paul-carlton2 has joined #openstack-nova | 12:54 | |
*** paul-carlton1 has quit IRC | 12:56 | |
*** pwnall has joined #openstack-nova | 12:56 | |
*** danpb has quit IRC | 12:57 | |
*** mriedem has joined #openstack-nova | 12:59 | |
*** ociuhandu has quit IRC | 12:59 | |
markus_z | dulek: We are also thinking to remove those config options. | 13:00 |
*** drankis has quit IRC | 13:00 | |
*** moshele has quit IRC | 13:00 | |
markus_z | dulek: We're currently improving the docs of our options and these *_topic" options were among those options which look unnecessary: https://etherpad.openstack.org/p/config-options | 13:00 |
*** klkumar has quit IRC | 13:01 | |
*** heyongli has quit IRC | 13:01 | |
*** links has quit IRC | 13:01 | |
*** heyongli has joined #openstack-nova | 13:01 | |
dulek | markus_z: Thanks! So it seems like we can remove them as well. In Cinder changing them from the default is causing RPC layer failures anyway. | 13:02 |
*** dims_ has joined #openstack-nova | 13:02 | |
*** dims has quit IRC | 13:02 | |
dulek | Unrelated question - do I need a new microversion when adding PUT method support to already existing resource URL? Docs doesn't mention that case explicitly. | 13:03 |
*** jed56 has quit IRC | 13:03 | |
markus_z | dulek: I didn't analyze them into deep. I *assume* they got introduced prematurely without a real need. | 13:03 |
markus_z | dulek: Good to know Cinder has the same issue there. Thanks for this info. | 13:03 |
mriedem | dulek: adding a PUT method would require a microversion | 13:06 |
dulek | mriedem: Okay, thanks! | 13:07 |
*** lucasagomes is now known as lucas-hungry | 13:07 | |
*** daemontool has quit IRC | 13:07 | |
*** bswartz has quit IRC | 13:08 | |
*** efried has joined #openstack-nova | 13:08 | |
*** daemontool has joined #openstack-nova | 13:09 | |
*** mdrabe has joined #openstack-nova | 13:10 | |
*** ayoung has joined #openstack-nova | 13:11 | |
*** ctrath has joined #openstack-nova | 13:11 | |
*** heyongli has quit IRC | 13:11 | |
*** heyongli has joined #openstack-nova | 13:12 | |
*** ducttape_ has joined #openstack-nova | 13:13 | |
*** yuywz has joined #openstack-nova | 13:13 | |
*** bswartz has joined #openstack-nova | 13:14 | |
*** sridharg has quit IRC | 13:15 | |
*** ivase has joined #openstack-nova | 13:15 | |
*** baoli has quit IRC | 13:15 | |
*** ducttape_ has quit IRC | 13:16 | |
*** baoli has joined #openstack-nova | 13:16 | |
*** moshele has joined #openstack-nova | 13:17 | |
*** jed56 has joined #openstack-nova | 13:17 | |
*** mtanino has quit IRC | 13:17 | |
*** PaulMurray has joined #openstack-nova | 13:18 | |
*** ivase has quit IRC | 13:20 | |
johnthetubaguy | pkholkin: sorry, I missed your ping, hows things? | 13:21 |
*** heyongli has quit IRC | 13:21 | |
*** knikolla has joined #openstack-nova | 13:22 | |
*** heyongli has joined #openstack-nova | 13:22 | |
openstackgerrit | Wenzhi Yu proposed openstack/nova: Rewrite 'test_filter_schedule_skipping' method using Mock https://review.openstack.org/285146 | 13:22 |
openstackgerrit | Chris Dent proposed openstack/nova: WIP Add ResourcePool and ResourcePoolList objects https://review.openstack.org/284963 | 13:23 |
openstackgerrit | Wenzhi Yu proposed openstack/nova: Rewrite 'test_filter_schedule_skipping' method using Mock https://review.openstack.org/285146 | 13:24 |
*** Jeffrey4l has quit IRC | 13:25 | |
*** tbachman has joined #openstack-nova | 13:25 | |
*** ccarmack has joined #openstack-nova | 13:25 | |
*** takedakn has joined #openstack-nova | 13:26 | |
openstackgerrit | Wenzhi Yu proposed openstack/nova: Rewrite 'test_filter_schedule_skipping' method using Mock https://review.openstack.org/285146 | 13:26 |
*** ccarmack has quit IRC | 13:27 | |
*** mvk_ has joined #openstack-nova | 13:27 | |
*** READ10 has joined #openstack-nova | 13:27 | |
openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 13:29 |
scottda | ildikov: Ping | 13:29 |
*** takedakn has quit IRC | 13:31 | |
*** heyongli has quit IRC | 13:32 | |
*** heyongli has joined #openstack-nova | 13:32 | |
pkholkin | johnthetubaguy: no problem, I wanted to ask your opinion about my spec https://review.openstack.org/#/c/239663/ | 13:33 |
pkholkin | I also created POC to it | 13:33 |
pkholkin | the code is similar to other nova objects | 13:33 |
*** mylu has joined #openstack-nova | 13:33 | |
pkholkin | but there is dependant spec whic is not merged ( | 13:33 |
*** klkumar has joined #openstack-nova | 13:33 | |
*** jwcroppe has joined #openstack-nova | 13:34 | |
*** mylu has quit IRC | 13:34 | |
*** markvoelker_ has joined #openstack-nova | 13:34 | |
*** erlon has joined #openstack-nova | 13:34 | |
*** markvoelker has quit IRC | 13:36 | |
*** zz_dimtruck is now known as dimtruck | 13:37 | |
*** markvoelker has joined #openstack-nova | 13:38 | |
*** baoli has quit IRC | 13:40 | |
*** markvoelker_ has quit IRC | 13:40 | |
*** eharney has joined #openstack-nova | 13:41 | |
*** RuiChen has quit IRC | 13:41 | |
*** baoli has joined #openstack-nova | 13:41 | |
*** heyongli has quit IRC | 13:42 | |
*** heyongli has joined #openstack-nova | 13:42 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Fix alignment of wsgi options https://review.openstack.org/278024 | 13:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Move wsgi options into a group https://review.openstack.org/263812 | 13:42 |
openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Remove 'wsgi_' prefix from opts https://review.openstack.org/278023 | 13:42 |
*** efried has quit IRC | 13:43 | |
*** RuiChen has joined #openstack-nova | 13:43 | |
*** mylu has joined #openstack-nova | 13:43 | |
mriedem | sfinucan: maybe we need to start using hungarian notation :) | 13:43 |
*** ametts has joined #openstack-nova | 13:44 | |
*** pradk has quit IRC | 13:44 | |
*** Guest74619 is now known as BobBall | 13:44 | |
*** efried has joined #openstack-nova | 13:44 | |
*** gszasz has joined #openstack-nova | 13:45 | |
sfinucan | mriedem: Boo-urns | 13:45 |
*** ljxiash has joined #openstack-nova | 13:45 | |
*** mylu has quit IRC | 13:46 | |
*** burt has joined #openstack-nova | 13:48 | |
*** daemontool has quit IRC | 13:48 | |
*** liverpooler has quit IRC | 13:52 | |
*** ociuhandu has joined #openstack-nova | 13:52 | |
*** heyongli has quit IRC | 13:52 | |
*** heyongli has joined #openstack-nova | 13:52 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 13:53 |
*** daemontool has joined #openstack-nova | 13:54 | |
*** rhagarty has joined #openstack-nova | 13:54 | |
*** sc68cal has joined #openstack-nova | 13:54 | |
johnthetubaguy | pkholkin: that pagination one looks good, I do wonder if we could get the API WG stuff agreed though: http://specs.openstack.org/openstack/api-wg/guidelines/pagination_filter_sort.html | 13:55 |
PaulMurray | live migration meeting in 5 mins on #openstack-meeting-3 | 13:57 |
PaulMurray | PaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski jlanoux yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov | 13:57 |
*** mylu has joined #openstack-nova | 13:57 | |
*** yassine__ has joined #openstack-nova | 13:58 | |
*** mylu has quit IRC | 13:58 | |
*** mgoddard_ has joined #openstack-nova | 13:59 | |
*** rk4n has quit IRC | 13:59 | |
*** rk4n has joined #openstack-nova | 14:00 | |
*** edtubill has joined #openstack-nova | 14:00 | |
*** mylu has joined #openstack-nova | 14:01 | |
*** Guest82492 has quit IRC | 14:01 | |
*** mtanino has joined #openstack-nova | 14:01 | |
*** heyongli has quit IRC | 14:02 | |
*** mgoddard has quit IRC | 14:03 | |
*** heyongli has joined #openstack-nova | 14:03 | |
*** jichen has quit IRC | 14:03 | |
*** links has joined #openstack-nova | 14:05 | |
*** jwcroppe has quit IRC | 14:08 | |
*** aysyd has quit IRC | 14:08 | |
*** aysyd has joined #openstack-nova | 14:09 | |
*** achanda has joined #openstack-nova | 14:09 | |
*** achanda has quit IRC | 14:10 | |
*** doug-fis_ is now known as doug-fish | 14:11 | |
*** haukebruno has quit IRC | 14:11 | |
*** baoli has quit IRC | 14:12 | |
*** heyongli has quit IRC | 14:13 | |
*** heyongli has joined #openstack-nova | 14:13 | |
*** baoli has joined #openstack-nova | 14:13 | |
*** pushkaru has joined #openstack-nova | 14:14 | |
ndipanov | jaypipes, any chance you would look at this https://review.openstack.org/262583 | 14:15 |
jaypipes | ndipanov: heh, I'm already halfway through a review on that in gertty. | 14:16 |
ndipanov | jaypipes, it's as if I could sense it | 14:16 |
jaypipes | :) | 14:16 |
*** READ10 has quit IRC | 14:18 | |
*** READ10 has joined #openstack-nova | 14:19 | |
*** dims_ has quit IRC | 14:19 | |
*** lucas-hungry is now known as lucasagomes | 14:21 | |
ndipanov | jaypipes, I am fixing your concerns on the live migration thing | 14:22 |
jaypipes | coolio, thanks man | 14:22 |
ndipanov | should post it soon | 14:22 |
*** achanda has joined #openstack-nova | 14:23 | |
*** heyongli has quit IRC | 14:23 | |
*** mpavone has quit IRC | 14:23 | |
*** heyongli has joined #openstack-nova | 14:23 | |
*** mpavone has joined #openstack-nova | 14:24 | |
*** _ducttape_ has joined #openstack-nova | 14:24 | |
*** moshele has quit IRC | 14:24 | |
*** dimtruck is now known as zz_dimtruck | 14:24 | |
openstackgerrit | Wenzhi Yu proposed openstack/nova: Rewrite 'test_filter_schedule_skipping' method using Mock https://review.openstack.org/285146 | 14:28 |
sarafraj | Good Morning! | 14:28 |
sdague | mriedem: did you ever figure out why multinode is fail? | 14:28 |
mriedem | sdague: nope | 14:28 |
sdague | did you bring it to infra? | 14:28 |
mriedem | sdague: nope | 14:29 |
markus_z | sdague: Is this related to the "make tests use real UUIDs" topic you proposed in the summit etherpad: https://review.openstack.org/#/c/293840/1 ? | 14:29 |
markus_z | sdague: I didn't find a blueprint or ML post about the effort to clean that up. | 14:30 |
*** efried has quit IRC | 14:30 | |
sdague | markus_z: yes | 14:30 |
sdague | although that had been ongoing in mitaka as well | 14:30 |
openstackgerrit | Stephen Finucane proposed openstack/nova: nova-manage: Use consistent prints for errors https://review.openstack.org/289309 | 14:31 |
markus_z | sdague: Was that mapped to a bp or or bug report? | 14:31 |
sdague | markus_z: I don't know | 14:31 |
*** yuywz has quit IRC | 14:32 | |
markus_z | sdague: ok. I dig a little more. Maybe there are more people who want to help with that. | 14:32 |
ndipanov | jaypipes, see my comment here and if it makes sense https://review.openstack.org/262583 | 14:33 |
*** heyongli has quit IRC | 14:33 | |
*** openstackgerrit has quit IRC | 14:33 | |
*** heyongli has joined #openstack-nova | 14:33 | |
*** openstackgerrit has joined #openstack-nova | 14:34 | |
jaypipes | ndipanov: yeah, I'm looking at the next patch. I'll be honest, I've never been a fan of passing the physical network names like this :( | 14:34 |
openstackgerrit | Saverio Proto proposed openstack/nova: Allow backup operation in paused and suspend state when Server booted from Volume https://review.openstack.org/295865 | 14:34 |
ndipanov | jaypipes, it's horrible | 14:35 |
*** rk4n has quit IRC | 14:36 | |
*** ekuris_ has quit IRC | 14:37 | |
*** drankis has joined #openstack-nova | 14:37 | |
*** ivase has joined #openstack-nova | 14:39 | |
markus_z | sdague: FYI: Thats the ML thread http://openstack.markmail.org/thread/twtpl37r4zgfxoq5 | 14:41 |
openstackgerrit | sahid proposed openstack/nova: network: make nova to handle port_security_enabled=False https://review.openstack.org/284095 | 14:41 |
*** mylu has quit IRC | 14:41 | |
*** e0ne has quit IRC | 14:41 | |
*** ivase has quit IRC | 14:41 | |
*** baoli has quit IRC | 14:42 | |
*** ivase has joined #openstack-nova | 14:42 | |
*** baoli has joined #openstack-nova | 14:42 | |
*** mylu has joined #openstack-nova | 14:42 | |
openstackgerrit | Gage Hugo proposed openstack/nova: Fixed multiple issues with server_concepts page https://review.openstack.org/294284 | 14:42 |
*** efried has joined #openstack-nova | 14:42 | |
*** zioproto has joined #openstack-nova | 14:42 | |
*** links has quit IRC | 14:42 | |
mriedem | markus_z: there is probably an easier way to track that work, just see what was converted to using this https://github.com/openstack/nova/blob/master/nova/tests/uuidsentinel.py | 14:43 |
*** achanda has quit IRC | 14:43 | |
*** heyongli has quit IRC | 14:43 | |
*** heyongli has joined #openstack-nova | 14:44 | |
markus_z | mriedem: uh, fancy. Lemme check. | 14:44 |
mriedem | or ask dansmith, i can't remember who was doing that work | 14:44 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: neutron: pci_request logic considers 'direct-physical' vnic type https://review.openstack.org/262584 | 14:44 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: pci: Add utility method for getting the MAC addr https://review.openstack.org/262585 | 14:44 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: neutron: Update the port with a MAC address for PFs https://review.openstack.org/262586 | 14:44 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: Add a vnic type for PF passthrough and a new libvirt vif driver https://review.openstack.org/262583 | 14:44 |
dansmith | mriedem: migi ? | 14:44 |
ndipanov | jaypipes, ^ updated | 14:44 |
mriedem | dansmith: yeah | 14:45 |
mriedem | e.g. https://review.openstack.org/#/c/264685/ | 14:45 |
mriedem | markus_z: there was no bp or bug for that | 14:45 |
migi | dansmith, mriedem: UUID thing, right? | 14:46 |
mriedem | https://review.openstack.org/#/q/owner:mpryc%2540redhat.com+status:merged | 14:46 |
dansmith | mriedem: FYI, functional tests for the marker stuff found a couple buglets, unsurprisingly.. fixing that and pushing up in a few | 14:46 |
mriedem | yeah | 14:46 |
mriedem | dansmith: \o/ | 14:46 |
markus_z | mriedem: we could use https://bugs.launchpad.net/nova/+bug/1557378 to track it | 14:46 |
openstack | Launchpad bug 1557378 in OpenStack Compute (nova) "tox gives invalid uuid warnings in unit test" [Medium,In progress] - Assigned to Srushti Gadadare (srushti-gadadare) | 14:46 |
dansmith | mriedem: the uuid work was just started as cleanup because we needed it for o.vo | 14:46 |
mriedem | markus_z: yeah, sounds like a good idea | 14:46 |
markus_z | it's written pretty generically | 14:47 |
*** moshele has joined #openstack-nova | 14:47 | |
jaypipes | ndipanov: can the string just be 'hostdev'? | 14:47 |
ndipanov | well yes but we already have a very similar thing | 14:48 |
ndipanov | ib_hostdev | 14:48 |
ndipanov | so this kinda gives a hint that this is reserved for a PF | 14:48 |
jaypipes | ndipanov: why not just call it sriov_pf then? | 14:49 |
*** zz_dimtruck is now known as dimtruck | 14:49 | |
jaypipes | johnthetubaguy: can you remove your -2 block on cdent's resource providers stuff? patch starting with 281945 | 14:50 |
*** coreywright has quit IRC | 14:50 | |
migi | dansmith: mriedem: actually I gave -1 for https://review.openstack.org/#/c/293840/1 | 14:50 |
ndipanov | jaypipes, let me check this might already be in neutron | 14:50 |
dansmith | migi: cool :) | 14:50 |
cdent | jaypipes: I asked the same thing yesterday and was told to re-propose the related specs. Did that happen | 14:51 |
ndipanov | jaypipes, https://review.openstack.org/#/c/262604/6/neutron/plugins/ml2/drivers/mech_sriov/mech_driver/mech_driver.py :) | 14:52 |
ndipanov | er :( | 14:52 |
mriedem | sdague: is there any way in graphite.o.o (and thus failgraph.py) to filter on branches? | 14:52 |
mriedem | like adding a ZUUL_BRANCH=stable/kilo filter? | 14:53 |
jaypipes | cdent: you mean the blueprint amendment patch that was approved for mitakea? re-propossedx that amendment? | 14:53 |
sdague | mriedem: I have no idea | 14:53 |
jaypipes | cdent: can you take care of that? | 14:53 |
migi | markus_z: the original review which will require all tests to be fixed is this: https://review.openstack.org/#/c/250493/11 | 14:53 |
*** heyongli has quit IRC | 14:54 | |
*** heyongli has joined #openstack-nova | 14:54 | |
migi | markus_z: but yes your bug is good to track all changes and I'll use it | 14:54 |
*** mgoddard_ has quit IRC | 14:55 | |
cdent | jaypipes: I'm not entirely clear on the process but can figure it out and clean up | 14:55 |
*** mgoddard has joined #openstack-nova | 14:55 | |
markus_z | migi: cool, good to know, thanks. | 14:55 |
pkholkin | johnthetubaguy: yes, the problem is that mentioned spec isn't merged | 14:55 |
pkholkin | there are -1s ( | 14:56 |
pkholkin | and I'm not sure whether it will be merged | 14:56 |
pkholkin | is it blocker for us in Nova? | 14:56 |
jaypipes | cdent: ty | 14:56 |
pkholkin | my concern is that my implementation is almost the same as other pagination mechanisms in the project | 14:57 |
openstackgerrit | Dan Smith proposed openstack/nova: Make FlavorList.get_all() return results from the API and main DBs https://review.openstack.org/294252 | 14:58 |
openstackgerrit | Dan Smith proposed openstack/nova: Make flavor extra_specs operations work against the API DB https://review.openstack.org/294710 | 14:58 |
openstackgerrit | Dan Smith proposed openstack/nova: Block flavor creation until main database is empty https://review.openstack.org/295310 | 14:58 |
openstackgerrit | Dan Smith proposed openstack/nova: Add flavor migration routine https://review.openstack.org/294771 | 14:58 |
openstackgerrit | Dan Smith proposed openstack/nova: Make Flavor access routines work against API database https://review.openstack.org/294313 | 14:58 |
*** Swanson has quit IRC | 14:58 | |
openstackgerrit | Dan Smith proposed openstack/nova: Make Flavor create() and destroy() work against API DB https://review.openstack.org/294758 | 14:58 |
pkholkin | cdent: good day Chris! | 14:58 |
pkholkin | could you please share your opinion about my pagination spec https://review.openstack.org/#/c/239663/ | 14:58 |
dansmith | mriedem: alaski ^ with matt's comment addressed on the base patch and functional tests added on the create patch | 14:58 |
cdent | pkholkin: yup wil stick it on the queue | 14:58 |
pkholkin | cdent: I'm asking you because you reviewed another spec https://review.openstack.org/#/c/190743/ | 14:59 |
pkholkin | my spec is related or even dependant on it | 15:00 |
* cdent nods | 15:00 | |
*** atuvenie has quit IRC | 15:01 | |
pkholkin | cdent: do you think maybe that spec will be merged soon, I need your opinion :) | 15:02 |
*** angdraug has joined #openstack-nova | 15:02 | |
cdent | pkholkin: it needs some minor cleanups but I think it will merge mostly in the state it is in | 15:03 |
*** heyongli has quit IRC | 15:04 | |
*** zioproto has quit IRC | 15:04 | |
*** heyongli has joined #openstack-nova | 15:04 | |
*** jwcroppe has joined #openstack-nova | 15:05 | |
pkholkin | cdent: it sounds good, I will look at it too | 15:07 |
*** coreywright has joined #openstack-nova | 15:09 | |
*** rk4n has joined #openstack-nova | 15:10 | |
openstackgerrit | Dan Smith proposed openstack/nova: Make flavor extra_specs operations work against the API DB https://review.openstack.org/294710 | 15:10 |
openstackgerrit | Dan Smith proposed openstack/nova: Block flavor creation until main database is empty https://review.openstack.org/295310 | 15:10 |
openstackgerrit | Dan Smith proposed openstack/nova: Add flavor migration routine https://review.openstack.org/294771 | 15:10 |
openstackgerrit | Dan Smith proposed openstack/nova: Make Flavor create() and destroy() work against API DB https://review.openstack.org/294758 | 15:10 |
*** haukebruno has joined #openstack-nova | 15:11 | |
*** rk4n has quit IRC | 15:12 | |
*** mdrabe has quit IRC | 15:12 | |
dansmith | sdague: ah thanks for merging that grenade change | 15:12 |
sdague | dansmith: yep, no worries, was starting to look at branching today | 15:13 |
dansmith | sdague: awesome | 15:13 |
openstackgerrit | Chris Dent proposed openstack/nova-specs: Re-propose resource-providers for Newton https://review.openstack.org/295890 | 15:13 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Cleanup NovaObjectDictCompat from BandwidthUsage https://review.openstack.org/263638 | 15:13 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Cleanup NovaObjectDictCompat from NumaTopology https://review.openstack.org/260016 | 15:13 |
dansmith | that grenade change lets me see that flavors get migrated properly in the nova-manage command, | 15:13 |
dansmith | which was broken when we did this for kilo | 15:13 |
dansmith | so... progress! | 15:13 |
sdague | yep | 15:13 |
johnthetubaguy | pkholkin: that shouldn't be a blocker I guess, its more nice to have, to save any re-work later really, but this seems consistent with the rest of the Nova PI. | 15:14 |
johnthetubaguy | API^ | 15:14 |
*** heyongli has quit IRC | 15:14 | |
*** rmart04 has joined #openstack-nova | 15:14 | |
*** heyongli has joined #openstack-nova | 15:14 | |
openstackgerrit | sahid proposed openstack/nova: libvirt: check graphic/serial consoles at pre_live_migration step https://review.openstack.org/254709 | 15:15 |
*** browne has joined #openstack-nova | 15:15 | |
*** achanda has joined #openstack-nova | 15:17 | |
dansmith | so nova-pep8 job is failing | 15:18 |
dansmith | failing to connect to git.openstack.org | 15:18 |
* dansmith goes to infra | 15:18 | |
*** dedery has quit IRC | 15:19 | |
*** harlowja_at_home has joined #openstack-nova | 15:20 | |
*** heyongli has quit IRC | 15:24 | |
*** heyongli has joined #openstack-nova | 15:25 | |
*** ducttape_ has joined #openstack-nova | 15:26 | |
*** mdrabe has joined #openstack-nova | 15:26 | |
*** rodrigods has quit IRC | 15:27 | |
*** rodrigods has joined #openstack-nova | 15:27 | |
*** _ducttape_ has quit IRC | 15:29 | |
* bauzas waves back after some time playing with his local infra for pxe booting things | 15:29 | |
*** adisky has quit IRC | 15:30 | |
openstackgerrit | Chris Dent proposed openstack/nova-specs: Re-propose resource-providers for Newton https://review.openstack.org/295890 | 15:30 |
bauzas | johnthetubaguy: I had made some bug triage this morning, not seeing a regression yet | 15:30 |
openstackgerrit | Sujitha proposed openstack/nova: Config options: Centralize console options https://review.openstack.org/294261 | 15:30 |
bauzas | johnthetubaguy: do we plan to cut another RC2 on Thur and land a RC3 later (for people wanting to check https://review.openstack.org/#/c/295229/ | 15:31 |
bauzas | johnthetubaguy: or should we wait another week ? | 15:31 |
johnthetubaguy | bauzas: I said on the ML we would cut a new one | 15:31 |
johnthetubaguy | its probably worth it already. | 15:31 |
*** pc_m has quit IRC | 15:31 | |
bauzas | oooh right | 15:32 |
*** vishwanathj_zzz is now known as vishwanathj | 15:32 | |
bauzas | johnthetubaguy: okay, let's discuss that on tomorrow afternoon then | 15:32 |
bauzas | johnthetubaguy: because a new regression would mean to wait for a master change to be merged before backporting it | 15:32 |
bauzas | hence Wed afternoon for us I think | 15:33 |
johnthetubaguy | bauzas: oh, are you thinking cut sooner than thursday? | 15:33 |
bauzas | no | 15:33 |
bauzas | I mean, we should see if another regression is provided on Wed afternoon | 15:34 |
johnthetubaguy | ah, go ya | 15:34 |
johnthetubaguy | got ya | 15:34 |
bauzas | because if a new one is coming like Wed night, we couldn't have it merged by RC2 | 15:34 |
* bauzas goes to spec reviewing | 15:34 | |
johnthetubaguy | bauzas: +1 lets see what is found, sfinucan was mentioning something about CPU pinning, but not sure how bad that is, sounded like a slightly broken new feature, rather than a regression, but I am not sure yet | 15:34 |
*** heyongli has quit IRC | 15:35 | |
*** heyongli has joined #openstack-nova | 15:35 | |
sfinucan | johnthetubaguy: Yes, bug in new feature. Not a regression. | 15:35 |
johnthetubaguy | sfinucan: cool, that feels like a nice to have, not a blocker for RC. Did you add that tag? | 15:36 |
johnthetubaguy | mitaka-rc-potential | 15:36 |
bauzas | mitaka-stable-potential rather, no ? | 15:37 |
dansmith | mitaka-backport-potential I think | 15:37 |
bauzas | yeah that | 15:37 |
bauzas | s/stable/backport | 15:37 |
*** pc_m has joined #openstack-nova | 15:38 | |
*** yamahata has quit IRC | 15:38 | |
*** jwcroppe has quit IRC | 15:39 | |
johnthetubaguy | dansmith: bauzas: yeah, sorry, thats a better choice for that one | 15:39 |
openstackgerrit | Sujitha proposed openstack/nova: Config options: Centralize ldapdns options https://review.openstack.org/294179 | 15:39 |
*** mmm has quit IRC | 15:40 | |
*** eil397 has joined #openstack-nova | 15:40 | |
openstackgerrit | Thomas Maddox proposed openstack/nova: [WIP - DO NOT MERGE] Testing LXC Gate https://review.openstack.org/274792 | 15:41 |
*** eil397 has quit IRC | 15:41 | |
*** eil397 has joined #openstack-nova | 15:41 | |
*** nkrinner has quit IRC | 15:42 | |
*** ducttape_ has quit IRC | 15:42 | |
*** ducttape_ has joined #openstack-nova | 15:42 | |
*** heyongli has quit IRC | 15:45 | |
*** heyongli has joined #openstack-nova | 15:45 | |
*** palexster has quit IRC | 15:45 | |
*** dims has joined #openstack-nova | 15:45 | |
sfinucan | johnthetubaguy: Yup. Done now. | 15:48 |
*** Sukhdev has joined #openstack-nova | 15:48 | |
*** jlanoux has quit IRC | 15:49 | |
mriedem | johnthetubaguy: bauzas: i did add the rc potential tag to the vhostuser mtu thing | 15:49 |
*** rk4n has joined #openstack-nova | 15:49 | |
mriedem | https://review.openstack.org/#/c/294649/ | 15:50 |
mriedem | if we want that in rc2 then it needs to be cleaned up, but i could do that today | 15:50 |
mriedem | it might not be a bad idea though given neutron changed the default mtu value in mitaka so people upgrading to neutron in mitaka might be broken if they don't also have this when they upgrade nova to mitaka | 15:50 |
johnthetubaguy | mriedem: ah, thats a good idea | 15:50 |
*** woodster_ has joined #openstack-nova | 15:51 | |
mriedem | ok, i'll clean that backport up | 15:51 |
johnthetubaguy | mriedem: awesome, thank you | 15:52 |
*** mylu has quit IRC | 15:53 | |
*** mylu has joined #openstack-nova | 15:53 | |
*** EinstCrazy has quit IRC | 15:53 | |
mriedem | done https://review.openstack.org/#/c/294649/ | 15:54 |
mriedem | sean-k-m1: ^ | 15:54 |
*** dims_ has joined #openstack-nova | 15:55 | |
*** dims has quit IRC | 15:55 | |
*** heyongli has quit IRC | 15:55 | |
*** heyongli has joined #openstack-nova | 15:55 | |
*** ljxiash has quit IRC | 15:56 | |
bauzas | mriedem: okay, feels like a reg, +1 | 15:56 |
*** ljxiash has joined #openstack-nova | 15:57 | |
*** browne has quit IRC | 15:57 | |
bauzas | mriedem: remember me the Kilo stable backport policy for https://review.openstack.org/#/c/289374/2 | 15:58 |
pkholkin | johnthetubaguy: ok, thanks for review ! | 15:58 |
bauzas | mriedem: amirite with my comment ? | 15:58 |
pkholkin | I will monitor this question ) | 15:58 |
bauzas | mriedem: speaking of http://docs.openstack.org/project-team-guide/stable-branches.html#support-phases | 15:59 |
mriedem | bauzas: http://docs.openstack.org/project-team-guide/stable-branches.html#support-phases | 15:59 |
mriedem | yeah | 15:59 |
bauzas | mriedem: I asked you because you added the Kilo tag for the bug report | 16:00 |
mriedem | bauzas: well, sean-k-m1 had already backported everything all the way back | 16:00 |
mriedem | before master was approved | 16:00 |
bauzas | mriedem: yup, I saw your -2 | 16:00 |
bauzas | was talking of added: kilo-backport-potential liberty-backport-potential | 16:00 |
*** ducttape_ has quit IRC | 16:00 | |
mriedem | yeah, i'm a bit on the fence on this one | 16:00 |
mriedem | it's pretty isolated, so i'm not really worried about regression, there is a clear user impact | 16:01 |
mriedem | i.e. you can't boot instances if plugging vhostuser vifs and mtu is set to anything but 0 | 16:01 |
*** ljxiash has quit IRC | 16:01 | |
andrearosa | diana_clarke: are you around? | 16:01 |
diana_clarke | yup! | 16:02 |
diana_clarke | andrearosa: ^^ | 16:02 |
andrearosa | diana_clarke: FYI: https://bugs.launchpad.net/python-glanceclient/+bug/1560093 | 16:02 |
openstack | Launchpad bug 1560093 in python-glanceclient "let the list images fails fast if the glance API is not responding" [Undecided,New] | 16:02 |
mriedem | bauzas: given mtu defaults to 0 in kilo though, and there is a workaround, it's not critical for stable/kilo | 16:02 |
bauzas | mriedem: okay, I can remove the Kilo tag, then ? | 16:02 |
mriedem | bauzas: yeah | 16:02 |
* bauzas axes | 16:02 | |
* bauzas tries to speak stable policy as a 3rd language | 16:03 | |
bauzas | soy tries to comprendre | 16:04 |
diana_clarke | andrearosa: Thanks! I'll subscribe to the issue to stay in the loop. | 16:04 |
*** heyongli has quit IRC | 16:05 | |
*** heyongli has joined #openstack-nova | 16:06 | |
*** aunnam has joined #openstack-nova | 16:06 | |
*** aginwala has joined #openstack-nova | 16:07 | |
*** phuongnh has joined #openstack-nova | 16:07 | |
*** apoorvad has joined #openstack-nova | 16:09 | |
openstackgerrit | Carl Baldwin proposed openstack/nova-specs: Neutron Routed Networks https://review.openstack.org/263898 | 16:09 |
*** aunnam has quit IRC | 16:10 | |
openstackgerrit | sahid proposed openstack/nova: libvirt: check graphic/serial consoles at pre_live_migration step https://review.openstack.org/254709 | 16:11 |
openstackgerrit | sahid proposed openstack/nova: libvirt: "migrate" to Guest's object https://review.openstack.org/250365 | 16:11 |
openstackgerrit | sahid proposed openstack/nova: libvirt: live_migration_flags/block_migration_flags default to 0 https://review.openstack.org/295933 | 16:11 |
*** yamahata has joined #openstack-nova | 16:12 | |
*** aginwala has quit IRC | 16:13 | |
openstackgerrit | Mark Doffman proposed openstack/nova-specs: Move aggregates tables to the API database https://review.openstack.org/288084 | 16:15 |
*** heyongli has quit IRC | 16:16 | |
*** mylu has quit IRC | 16:16 | |
*** heyongli has joined #openstack-nova | 16:16 | |
*** ducttape_ has joined #openstack-nova | 16:18 | |
*** jaosorior has quit IRC | 16:18 | |
*** aginwala has joined #openstack-nova | 16:18 | |
*** mylu has joined #openstack-nova | 16:19 | |
*** mylu has quit IRC | 16:19 | |
*** mylu has joined #openstack-nova | 16:19 | |
*** jlanoux has joined #openstack-nova | 16:21 | |
*** cdent has quit IRC | 16:21 | |
*** e0ne has joined #openstack-nova | 16:21 | |
*** bswartz has quit IRC | 16:21 | |
openstackgerrit | Vasyl Saienko proposed openstack/nova: Allow to update resource per single node. https://review.openstack.org/287485 | 16:22 |
*** belmoreira has quit IRC | 16:25 | |
*** heyongli has quit IRC | 16:26 | |
*** mylu has quit IRC | 16:26 | |
*** Swami has joined #openstack-nova | 16:26 | |
openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "volume" https://review.openstack.org/253747 | 16:26 |
*** heyongli has joined #openstack-nova | 16:26 | |
openstackgerrit | Sujitha proposed openstack/nova: Config options: Centralize compute options https://review.openstack.org/295938 | 16:27 |
*** atuvenie has joined #openstack-nova | 16:27 | |
*** efried has quit IRC | 16:27 | |
openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "volume" https://review.openstack.org/253747 | 16:28 |
pumaranikar | hi mriedem , I am working on a bug which stats about incorrect error logs on post live migration. https://bugs.launchpad.net/nova/+bug/1555320 | 16:28 |
openstack | Launchpad bug 1555320 in OpenStack Compute (nova) ""Migration for instance 0763227e-e192-4e0b-a49d-0ea0b181fca6 refers to another host's instance!" should not be an error" [Medium,Confirmed] - Assigned to Pushkar Umaranikar (pushkar-umaranikar) | 16:28 |
pumaranikar | mriedem, I am unable to reproduce it. Can you please suggest steps to reproduce ? | 16:28 |
*** sc68cal has quit IRC | 16:29 | |
*** aginwala has quit IRC | 16:29 | |
*** Jeffrey4l has joined #openstack-nova | 16:30 | |
*** moshele has quit IRC | 16:31 | |
mriedem | pumaranikar: when i opened that bug i believe the point was that we shouldn't be logging that as an error since it's not an error condition | 16:31 |
*** aginwala has joined #openstack-nova | 16:31 | |
*** mylu has joined #openstack-nova | 16:32 | |
mriedem | pumaranikar: it's happening in a periodic task during our multinode gate runs which test live migration | 16:32 |
*** zenoway has quit IRC | 16:32 | |
*** zenoway has joined #openstack-nova | 16:32 | |
*** unicell1 has quit IRC | 16:33 | |
mriedem | pumaranikar: happens here http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/resource_tracker.py#n522 | 16:33 |
mriedem | it probably assumes it's an error because the list of migrations are grabbed from the database by host and nodename right before that http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/resource_tracker.py#n568 | 16:35 |
mriedem | and that happens in a lock, but that lock is only going to be per compute node | 16:35 |
*** mylu has quit IRC | 16:35 | |
mriedem | but we also get the instances by host and nodename http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/resource_tracker.py#n558 | 16:36 |
*** jwcroppe has joined #openstack-nova | 16:36 | |
*** heyongli has quit IRC | 16:36 | |
*** heyongli has joined #openstack-nova | 16:36 | |
*** matrohon has quit IRC | 16:37 | |
*** zenoway has quit IRC | 16:37 | |
*** drankis has joined #openstack-nova | 16:38 | |
pumaranikar | mriedem, thanks. I will look into that. | 16:39 |
mriedem | pumaranikar: it might be worth putting up a debug patch that checks which host the migration.instance_uuid is on when we hit that KeyError | 16:39 |
*** earlephilhower has joined #openstack-nova | 16:39 | |
mriedem | you'd have to get the instance from the database and compare the instance.host to CONF.host | 16:39 |
pumaranikar | mriedem, yes. Is the lazy loading from db might be a problem ? | 16:40 |
*** sc68cal has joined #openstack-nova | 16:40 | |
mriedem | pumaranikar: i don't think so | 16:41 |
mriedem | i'm not really familiar with that code though, it has dansmith's name on it | 16:41 |
*** efried has joined #openstack-nova | 16:41 | |
pkoniszewski | bauzas: hi, there? | 16:43 |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: WIP: Cold migrate using the RequestSpec object https://review.openstack.org/284974 | 16:43 |
bauzas | yup, 'sup ? | 16:43 |
bauzas | oh | 16:43 |
pumaranikar | mriedem, ok. As you suggested, first will check by adding debug pointer. It will help to know the cause. | 16:43 |
pkoniszewski | bauzas: still WIP, have a one question | 16:43 |
bauzas | pkoniszewski: yep ? | 16:43 |
pumaranikar | mriedem, thank you for the help. | 16:44 |
mriedem | pumaranikar: if you're looking for something to do, there is this uuid field cleanup stuff we need to do, and replacing mox usage with stub_out | 16:44 |
mriedem | pumaranikar: see L197 in https://etherpad.openstack.org/p/newton-nova-summit-ideas | 16:45 |
pkoniszewski | i haven't fixed functional tests yet, im ending up with datetime objects inside request spec dict | 16:45 |
*** akshai has joined #openstack-nova | 16:45 | |
pumaranikar | mriedem, cool. thanks for sharing. | 16:46 |
pkoniszewski | i mean, after conversion from an object to legacy dict there is 'image' nested and it has attributes, e.g., created_at that is datetime which is not serializable | 16:46 |
*** eil397 has quit IRC | 16:46 | |
pkoniszewski | bauzas: ^ | 16:46 |
*** heyongli has quit IRC | 16:46 | |
*** dims_ has quit IRC | 16:46 | |
pkoniszewski | therefore RPC call crashes | 16:47 |
*** eil397 has joined #openstack-nova | 16:47 | |
*** heyongli has joined #openstack-nova | 16:47 | |
bauzas | pkoniszewski: not sure I get you, do you have a stack to show me ? | 16:47 |
pkoniszewski | yup | 16:47 |
mriedem | johnthetubaguy: alaski: want to trivial approve this? https://blueprints.launchpad.net/nova/+spec/remove-mox-newton | 16:47 |
*** cdent has joined #openstack-nova | 16:48 | |
alaski | mriedem: sure. It's just a bp, no spec? | 16:48 |
mriedem | alaski: yeah, continuation of cleanup work from mitaka | 16:49 |
pumaranikar | mriedem, I am currently working on config options. Will also go through proposals you just shared. | 16:49 |
*** haomaiwang has joined #openstack-nova | 16:50 | |
*** MCoLo__ has joined #openstack-nova | 16:50 | |
*** pcaruana has quit IRC | 16:52 | |
*** akshai has quit IRC | 16:52 | |
dansmith | alaski: you have any ideas about this? http://pastebin.com/ZsY6mUGx | 16:52 |
*** MCoLo has quit IRC | 16:52 | |
*** akshai has joined #openstack-nova | 16:52 | |
dansmith | alaski: I've been battling it for a couple hours and I have no idea where it's coming from | 16:52 |
pkoniszewski | bauzas: http://paste.openstack.org/show/491478/ | 16:52 |
dansmith | alaski: I know what line causes it, but it's in Flavor.create() and makes no sense to me at all | 16:53 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/nova: Implement setup_networks_on_host for Neutron networks https://review.openstack.org/275073 | 16:54 |
bauzas | dansmith: meh, I remember having already seen that in the past, lemme try to find where | 16:54 |
*** akshai_ has joined #openstack-nova | 16:54 | |
dansmith | bauzas: oh good, because I'm at table-flip stage | 16:54 |
bauzas | pkoniszewski: oh that | 16:54 |
*** haomaiwang has quit IRC | 16:55 | |
alaski | dansmith: my first guess would be a race on session.configure(), since there's no locking around checking the global SESSION_CONFIGURED | 16:55 |
dansmith | um | 16:55 |
dansmith | a race with what? | 16:55 |
dansmith | you think we're getting swapped out whilst we setup those fixtures? | 16:56 |
alaski | the tests are threaded right? | 16:56 |
dansmith | the thing is, I'm not touching any of this crap | 16:56 |
dansmith | alaski: well, greenthreaded, this is global to one process | 16:56 |
dansmith | so I guess if we're getting swapped out in the fixture setup, maybe | 16:56 |
dansmith | le-sigh | 16:56 |
*** heyongli has quit IRC | 16:57 | |
alaski | just a guess. the enginefacade changed things there a bit ago, and it's possible there's an edge case | 16:57 |
*** heyongli has joined #openstack-nova | 16:57 | |
*** mpavone has quit IRC | 16:58 | |
pkoniszewski | bauzas: yeah, image is converted 1:1 from an object to dict, i have a feeling like updated_at and created_at shouldn't be there at all | 16:58 |
bauzas | alaski: yeah, that was due to the enginefacade rework, but I don't remember how I made this to be fixed in a past change | 16:58 |
pkoniszewski | bauzas: but wanted to ask about your opinion | 16:58 |
*** akshai has quit IRC | 16:58 | |
mriedem | dansmith: you could add the REQUIRES_LOCKING=True attribute to the test and see if that helps | 16:58 |
*** kfarr has joined #openstack-nova | 16:58 | |
bauzas | pkoniszewski: sec, I had this problem a lot of times in the past, I fixed that easilu | 16:58 |
dansmith | mriedem: it's a thousand tests | 16:59 |
dansmith | mriedem: needs to be in the fixture if anything | 16:59 |
bauzas | pkoniszewski: that's for functional tests, right? | 16:59 |
pkoniszewski | bauzas: that's right | 16:59 |
bauzas | pkoniszewski: IIRC, that's a known problem with the fake driver | 17:01 |
bauzas | pkoniszewski: not really a bug | 17:01 |
dansmith | mriedem: so lockutils works without REQUIRES_LOCKING, just not for external locks, right? | 17:02 |
*** lucasagomes is now known as lucas-afk | 17:02 | |
alaski | dansmith: I can't think of anything else based on the error atm. | 17:02 |
bauzas | pkoniszewski: I hacked that in the past, trying to figure out the thign | 17:02 |
dansmith | alaski: locking seems not to help | 17:02 |
*** Guest91251 has quit IRC | 17:02 | |
*** dims has joined #openstack-nova | 17:02 | |
alaski | hmm, it must be getting configured outside of the fixture somewhere | 17:02 |
dansmith | I don't see "session.configure" anywhere else | 17:03 |
dansmith | is there some automatic config the first time it's used or something? | 17:03 |
*** klkumar has quit IRC | 17:04 | |
*** nic has joined #openstack-nova | 17:04 | |
alaski | there's a get_context_manager method that can configure it | 17:05 |
alaski | sorry, create_context_manager | 17:05 |
bauzas | pkoniszewski: I don't recall how I fixed it, but I think I was stubbing out the fake method | 17:05 |
*** klkumar has joined #openstack-nova | 17:06 | |
melwitt | alaski: I'm looking at it too and that can happen if the connection kwarg is not None | 17:06 |
bauzas | pkoniszewski: yeah that https://bugs.launchpad.net/oslo.messaging/+bug/1529084 | 17:06 |
openstack | Launchpad bug 1529084 in oslo.messaging "RPC fake driver should accept datetime items for data" [Undecided,New] | 17:06 |
pkoniszewski | bauzas: you mean oslomessaging one? | 17:06 |
bauzas | pkoniszewski: see my bug report | 17:07 |
*** heyongli has quit IRC | 17:07 | |
*** heyongli has joined #openstack-nova | 17:07 | |
melwitt | alaski: the only other place I find is in nova/config.py where global config happens | 17:08 |
dansmith | alaski: melwitt: so it's L471 here: https://review.openstack.org/#/c/295310/8/nova/objects/flavor.py | 17:08 |
pkoniszewski | bauzas: thanks, i will try to stub it out | 17:08 |
dansmith | that call to _ensure_migrated() is what does it | 17:08 |
dansmith | but that is only called from create, which shouldn't actually be called all that often | 17:08 |
bauzas | pkoniszewski: I don't recall the exact fix I made, but you see the problem | 17:08 |
dansmith | it's not the raise or anything, it's that call (AFAICT) | 17:08 |
pkoniszewski | bauzas: yup | 17:08 |
*** phuongnh has quit IRC | 17:08 | |
bauzas | what's surprising is that grepping the bug number in the nova tree doesn't find comments | 17:09 |
*** jwcroppe has quit IRC | 17:09 | |
*** palexster has joined #openstack-nova | 17:10 | |
*** unicell has joined #openstack-nova | 17:10 | |
bauzas | I'd have added that bug number in some comment if I would have had workaround this | 17:10 |
alaski | melwitt: yeah. looks like configure_db is False when that's called from unit tests though | 17:10 |
*** baoli has quit IRC | 17:10 | |
melwitt | alaski: yeah, I figured that global config doesn't run but didn't know where/how | 17:10 |
pkoniszewski | bauzas: i'll make sure to add one because it is a pain | 17:11 |
markus_z | nova-bugs team meeting in 50 minutes in #openstack-meeting-4. New contributors are welcome as well. https://wiki.openstack.org/wiki/Meetings/Nova/BugsTeam | 17:11 |
*** haplo37 has joined #openstack-nova | 17:11 | |
*** baoli has joined #openstack-nova | 17:11 | |
alaski | melwitt: nova/tests/unit/conf_fixture.py is the only place I see it used | 17:11 |
dansmith | melwitt: alaski: FWIW, I don't seem to hit it if I run serially | 17:11 |
alaski | ugh | 17:11 |
dansmith | it's just really odd that this suddenly goes from never hitting it to KABLAMO | 17:11 |
*** phuongnh has joined #openstack-nova | 17:11 | |
*** dims has quit IRC | 17:13 | |
melwitt | yeah. I guess it must be running at the same time as a db connection switch unit test. really reliably | 17:13 |
alaski | dansmith: is it being hit from the migrate_flavors in https://review.openstack.org/#/c/295310/8/nova/test.py | 17:13 |
dansmith | alaski: if I comment that out it still happens | 17:14 |
bauzas | pkoniszewski: found it | 17:14 |
bauzas | https://review.openstack.org/#/c/203078/37 was having a functional test issue | 17:14 |
bauzas | which was fixed by PS38 | 17:14 |
*** scheuran has quit IRC | 17:15 | |
dansmith | alaski: how do you determine that it's related | 17:15 |
dansmith | alaski: it was my first thought, and the first thing I tried removing | 17:15 |
alaski | dansmith: can you log when session.configure is called in the fixture and see if it's actually getting called twice there? | 17:15 |
dansmith | okay | 17:15 |
bauzas | pkoniszewski: https://review.openstack.org/#/c/203078/37..38/nova/scheduler/client/query.py | 17:15 |
alaski | as far as I know it should only be configured once and then used forever more | 17:15 |
pkoniszewski | bauzas: cool, thanks, i will dig into it tomorrow as i need to drop in few minutes | 17:16 |
pkoniszewski | bauzas: well, that's a super ugly bit :D | 17:16 |
mriedem | quoth the laski? | 17:16 |
dansmith | heh | 17:16 |
dansmith | that's nevermore | 17:16 |
mriedem | close enough that you got it | 17:17 |
alaski | +1 | 17:17 |
dansmith | alaski: don't reward him | 17:17 |
*** heyongli has quit IRC | 17:17 | |
alaski | heh | 17:17 |
*** aginwala has quit IRC | 17:17 | |
*** heyongli has joined #openstack-nova | 17:17 | |
*** baoli has quit IRC | 17:18 | |
dansmith | alaski: only once | 17:18 |
*** aginwala has joined #openstack-nova | 17:18 | |
alaski | dansmith: I haven't dug into the enginefacade stuff enough to know what sets enough state to trigger your error. I'm wondering if using the decorator before session.configure would trigger it, but that's a wild guess | 17:19 |
dansmith | alaski: http://pastebin.com/XUKx3zzZ | 17:19 |
dansmith | alaski: everything would hit that | 17:19 |
*** palexster has quit IRC | 17:19 | |
*** palexster has joined #openstack-nova | 17:20 | |
*** bswartz has joined #openstack-nova | 17:20 | |
alaski | I'm still tracing the issue from yesterday a bit and you had to find a new one | 17:20 |
dansmith | I'm past that | 17:20 |
*** aginwala has quit IRC | 17:20 | |
dansmith | my patches are passing the py3 tests now | 17:20 |
melwitt | dansmith: are you running the test_db_api tests at the same time as this? | 17:21 |
dansmith | melwitt: I'm running anything that matches test_flavor | 17:21 |
melwitt | hm, okay | 17:21 |
*** baoli has joined #openstack-nova | 17:21 | |
alaski | dansmith: yeah, but I'm stubborn and want to figure it out | 17:22 |
dansmith | so this happens in multiple modules | 17:22 |
alaski | it looks like get_legacy_facade could set the state to trigger this exception | 17:22 |
dansmith | and all seem fine if I run them alone and serially | 17:22 |
alaski | and I've seen warnings about that being in use | 17:22 |
*** daemontool has quit IRC | 17:22 | |
dansmith | I also don't see this reported in logstash at all, | 17:23 |
dansmith | which I'd think we'd see some evidence of, if we had it occurring very infrequently due to race | 17:23 |
alaski | get_engine or get_api_engine used before the fixture could trigger it | 17:24 |
alaski | using the enginefacade decorator before session.configure is called looks like it could trigger it | 17:27 |
*** heyongli has quit IRC | 17:27 | |
*** krtaylor has quit IRC | 17:27 | |
dansmith | which one? | 17:28 |
*** heyongli has joined #openstack-nova | 17:28 | |
dansmith | and you mean executing something that is wrapped, or just wrapping? | 17:28 |
dansmith | so I ran unit.test_flavors and didn't hit it | 17:28 |
*** mriedem1 has joined #openstack-nova | 17:28 | |
alaski | executing something that is wrapped | 17:29 |
dansmith | so I wonder if it's something really unrelated that I'm bringing in | 17:29 |
dansmith | alaski: where did you get the idea that it was the migration in the test base? because I see it without that | 17:29 |
dansmith | but wonder if I'm missing something | 17:29 |
alaski | just that you said it was related to create and the migration calls create(), and that's called early in the test setup process | 17:30 |
*** klkumar has quit IRC | 17:30 | |
dansmith | it doesn't actually, it calls the low-level method | 17:30 |
dansmith | which bypasses the thing that seems to cause the problem | 17:30 |
*** mriedem has quit IRC | 17:31 | |
*** Sukhdev has quit IRC | 17:31 | |
alaski | ahh | 17:31 |
alaski | so the root of the problem seems to be that http://git.openstack.org/cgit/openstack/oslo.db/tree/oslo_db/sqlalchemy/enginefacade.py#n169 is set to True before session.configure() is called | 17:31 |
*** mriedem has joined #openstack-nova | 17:31 | |
alaski | maybe make that a property and see where it gets set? | 17:32 |
dansmith | lemme log there I guess | 17:32 |
*** phuongnh has quit IRC | 17:32 | |
*** kfarr has quit IRC | 17:32 | |
*** earlephilhower has quit IRC | 17:32 | |
*** mriedem has quit IRC | 17:32 | |
*** mriedem has joined #openstack-nova | 17:33 | |
*** mriedem1 has quit IRC | 17:33 | |
*** moshele has joined #openstack-nova | 17:34 | |
*** rlrossit has quit IRC | 17:34 | |
*** lucas-afk is now known as lucasagomes | 17:34 | |
dansmith | STARTED | 17:36 |
dansmith | STARTED | 17:36 |
*** heyongli has quit IRC | 17:38 | |
*** ducttape_ has quit IRC | 17:38 | |
*** heyongli has joined #openstack-nova | 17:38 | |
dansmith | alaski: so we should actually get two started sets because of api and main right? | 17:38 |
*** rlrossi__ has joined #openstack-nova | 17:39 | |
*** rmart04 has quit IRC | 17:40 | |
alaski | dansmith: yeah | 17:40 |
dansmith | so those are the only two I see | 17:40 |
*** angdraug has quit IRC | 17:40 | |
dansmith | although I only hit the one session.configure() in the the fixture | 17:41 |
alaski | is that one each or two for one? | 17:41 |
*** krtaylor has joined #openstack-nova | 17:41 | |
alaski | session.configure() calls the enginefacade configure for each context manager | 17:41 |
dansmith | I get one session.configure() call, and then two hits at the TransactionFactory started property | 17:41 |
dansmith | lemme paste | 17:41 |
*** daemontool has joined #openstack-nova | 17:42 | |
dansmith | http://pastebin.com/nuptA9ba | 17:43 |
dansmith | I think I was mixing runs | 17:43 |
dansmith | that one looks like we never even hit the bottom | 17:43 |
*** krotscheck has quit IRC | 17:43 | |
*** pcaruana has joined #openstack-nova | 17:44 | |
*** EinstCrazy has joined #openstack-nova | 17:44 | |
dansmith | alaski: I hit the two started properties in other runs | 17:44 |
dansmith | but not those | 17:44 |
dansmith | so maybe we're failing to clean something up in another test? | 17:45 |
*** derekh has quit IRC | 17:45 | |
*** jlanoux has quit IRC | 17:45 | |
alaski | seems likely | 17:45 |
alaski | I don't think there's anything that cleans up the enginefacade initialization, it's expected to be done once | 17:45 |
dansmith | what is the critical cleanup step? engine.dispose? | 17:46 |
*** EinstCrazy has quit IRC | 17:46 | |
dansmith | ah, so I should see us hit the bottom once per worker I guess? | 17:46 |
*** EinstCrazy has joined #openstack-nova | 17:46 | |
dansmith | so check this: http://pastebin.com/AhqJssB2 | 17:47 |
dansmith | that's a different configure | 17:47 |
*** haukebruno has quit IRC | 17:47 | |
dansmith | er, no, nevermind | 17:47 |
*** EinstCrazy has quit IRC | 17:47 | |
*** jwcroppe has joined #openstack-nova | 17:48 | |
alaski | looks like dispose is a sqla thing | 17:48 |
*** heyongli has quit IRC | 17:48 | |
*** EinstCrazy has joined #openstack-nova | 17:48 | |
alaski | dansmith: yeah, I would expect to see it once per worker | 17:48 |
*** heyongli has joined #openstack-nova | 17:48 | |
*** lpetrut has quit IRC | 17:49 | |
dansmith | so who all wrote this? looks like you, melwitt, sdague, snikitin_ | 17:49 |
*** EinstCrazy has quit IRC | 17:50 | |
alaski | the enginefacade stuff, which is what's causing issues, was integrated by snikitin and I think sdague worked on getting the fixture updated | 17:50 |
*** EinstCrazy has joined #openstack-nova | 17:50 | |
dansmith | just seems like we're flailing here | 17:50 |
dansmith | ah, you were just the api extension to it I gues | 17:50 |
openstackgerrit | Gage Hugo proposed openstack/nova: Fixed multiple issues with server_concepts page https://review.openstack.org/294284 | 17:51 |
alaski | what tox command are you using? | 17:51 |
sdague | that work around was brought in by snikitin because the engine fascade was not letting us do db testing outside of the oslo.db migration fixtures | 17:51 |
dansmith | tox -epy27 test_flavor | 17:51 |
alaski | and which review is this | 17:51 |
dansmith | alaski: https://review.openstack.org/#/c/295310/ | 17:51 |
sdague | if it's an issue, it can be reverted | 17:51 |
dansmith | sdague: I don't think we can, this is pretty core now | 17:52 |
sdague | I was told that was the work around, and that oslo.db would eventually fix it | 17:52 |
dansmith | sdague: which workaround? | 17:52 |
dansmith | the schema caching stuff? | 17:52 |
sdague | the anti schema caching | 17:52 |
alaski | dansmith: I'm going to break something if this doesn't reproduce locally | 17:52 |
dansmith | okay | 17:52 |
cdent | jaypipes, johnthetubaguy, dansmith : re-proposal of resource-providers spec, so we can get some of the patches out from under their -2 | 17:53 |
sdague | dansmith: maybe I need more context | 17:53 |
*** paul-carlton1 has joined #openstack-nova | 17:53 | |
*** READ10 has quit IRC | 17:53 | |
dansmith | alaski: hmm, maybe it's not in the logs in the gate | 17:53 |
dansmith | alaski: maybe I need to tox -r? | 17:53 |
*** EinstCrazy has quit IRC | 17:53 | |
alaski | dansmith: it fails for me | 17:53 |
sdague | if you want a non sqlite db in tests, you need to be really really careful about it | 17:53 |
dansmith | alaski: okay, but not in the gate.. wtf | 17:53 |
*** EinstCrazy has joined #openstack-nova | 17:53 | |
sdague | dansmith: where is your patch that's failing? | 17:54 |
dansmith | alaski: it shouldn't pass tests, but you see the transaction already started thing? | 17:54 |
alaski | yeah | 17:54 |
dansmith | sdague: I don't have logs to point you at | 17:54 |
dansmith | sdague: I have this: http://pastebin.com/AhqJssB2 | 17:54 |
*** EinstCrazy has quit IRC | 17:54 | |
dansmith | okay, I still get it on tox -r | 17:55 |
dansmith | alaski: but that sounds more and more like a race | 17:55 |
*** EinstCrazy has joined #openstack-nova | 17:55 | |
alaski | yeah | 17:55 |
*** browne has joined #openstack-nova | 17:55 | |
dansmith | all I want is to debug the actual fails, but I get so many of these it's near impossible | 17:55 |
sdague | dansmith: there is something all super magical about when engine fascade gets started | 17:56 |
*** gcb has quit IRC | 17:56 | |
sdague | this looks kind of like the fail I ran into | 17:56 |
sdague | let me go find that fix | 17:56 |
*** lucasagomes is now known as lucas-dinner | 17:57 | |
*** gcb has joined #openstack-nova | 17:57 | |
*** p01nt3r75 has quit IRC | 17:58 | |
*** angdraug has joined #openstack-nova | 17:58 | |
*** heyongli has quit IRC | 17:58 | |
*** rlrossi__ has quit IRC | 17:58 | |
*** heyongli has joined #openstack-nova | 17:59 | |
markus_z | nova-bugs team meeting in #openstack-meeting-4. New contributors are welcome as well. https://wiki.openstack.org/wiki/Meetings/Nova/BugsTeam | 17:59 |
*** baoli has quit IRC | 17:59 | |
sdague | dansmith: https://github.com/openstack/nova/blob/859ff4893f699b680fec4db7dedd3bec8c8d0a1c/nova/tests/fixtures.py#L551 | 18:00 |
*** baoli has joined #openstack-nova | 18:00 | |
dansmith | sdague: okay but I get this on an asston of tests | 18:01 |
*** angdraug has quit IRC | 18:01 | |
dansmith | is that in the base test now? | 18:01 |
sdague | nope | 18:01 |
sdague | but, if something screwed it up, then you cascade | 18:01 |
sdague | dansmith: do you have a review where this is a thing? | 18:01 |
dansmith | sdague: no seems not to fail in jenkins | 18:02 |
dansmith | sdague: should I use that after the db tests? | 18:02 |
dansmith | er, | 18:02 |
*** salv-orlando has joined #openstack-nova | 18:02 | |
dansmith | use that after the db fixtures in test.py | 18:02 |
sdague | do you have a review where this is a thing locally? | 18:02 |
sdague | dansmith: honestly, we probably need oslo.db folks engaged on this | 18:02 |
*** rlrossit_ has joined #openstack-nova | 18:03 | |
dansmith | sdague: I don't even know how to use that actually | 18:03 |
sdague | because there is so much magic in engine fascade, and using it in the testing model was definitely not thought through | 18:03 |
dansmith | sdague: https://review.openstack.org/#/c/295310/ | 18:03 |
sdague | https://review.openstack.org/#/c/283364/ is how you would use that in a test to work with mysql | 18:03 |
*** dimtruck is now known as zz_dimtruck | 18:04 | |
*** zz_dimtruck is now known as dimtruck | 18:04 | |
dansmith | I really don't understand why we need to do the caching thing | 18:05 |
*** pushkaru has quit IRC | 18:05 | |
dansmith | and why this is not simple setup/teardown | 18:05 |
alaski | dansmith: I think I found something | 18:05 |
sdague | dansmith: which caching thing? | 18:05 |
*** salv-orl_ has quit IRC | 18:05 | |
dansmith | sdague: the schema caching thing and the global session_configured thing | 18:05 |
sdague | because it saves 1s per test | 18:06 |
sdague | and we have 14000 tests | 18:06 |
dansmith | the caching thing saves that or the session thing? | 18:06 |
alaski | dansmith: so http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/unit/api/openstack/compute/test_flavor_manage.py#n187 calls flavor.create() eventually which configures the session | 18:06 |
sdague | the schema caching | 18:06 |
dansmith | sdague: okay, well, that's a good reason of course | 18:06 |
sdague | global session, i don't know | 18:07 |
sdague | but that's all oslo.db | 18:07 |
alaski | dansmith: http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/unit/api/openstack/compute/test_flavor_manage.py#n378 uses test.TestCase which sets up the database fixture calling session.configure which fails because it was started with the earlier flavor.create() | 18:07 |
*** rk4n has quit IRC | 18:07 | |
dansmith | how can those run out of order? | 18:08 |
alaski | they're different test classes | 18:08 |
alaski | what I think the issue is is that we have a NoDBTestCase that runs first, but which calls through an enginefacade context manager | 18:08 |
alaski | so the session gets configured | 18:08 |
dansmith | ah | 18:08 |
*** heyongli has quit IRC | 18:08 | |
alaski | then the TestCase comes along next and BOOM | 18:09 |
dansmith | I bet we have plenty of that | 18:09 |
*** heyongli has joined #openstack-nova | 18:09 | |
sdague | right, that is probably true | 18:09 |
alaski | yeah | 18:09 |
dansmith | so, | 18:09 |
dansmith | I wonder if we are legit to just try to detect that this is the case and ignore in the base class? the problem is, it raises a friggin TypeError | 18:09 |
*** EinstCrazy has quit IRC | 18:10 | |
dansmith | I guess this means we have a ticking time bomb | 18:10 |
dansmith | I just happened to shake it loose locally, but this *will* explode in the gate at some point | 18:10 |
alaski | we need an 'if session not configured' type check | 18:10 |
dansmith | and be really hard to fix | 18:10 |
sdague | we probably need a poison fixture for the db which we setup on the NoDBTestCase | 18:10 |
*** EinstCrazy has joined #openstack-nova | 18:10 | |
sdague | so that if you touch any of that from a NoDBTestCase, then you die | 18:10 |
alaski | on the context_manager stuff, yeah | 18:11 |
dansmith | sdague: the problem is, we have some tests that are NoDB, but then do their own DB setup | 18:11 |
dansmith | like alaski's test for cell_pammping | 18:11 |
dansmith | mapping | 18:11 |
dansmith | heh | 18:11 |
alaski | hah | 18:11 |
sdague | dansmith: sure, they can unpoison themselves | 18:11 |
sdague | self.db_poison.cleanup() | 18:11 |
sdague | as the first part of setup | 18:11 |
dansmith | so, I wonder if we could do this in two steps: | 18:11 |
alaski | dansmith: I'd actually like to split into NoDB, TestCase, APIDB, CellDB | 18:11 |
*** aginwala has joined #openstack-nova | 18:11 | |
alaski | but that may be a bit much | 18:11 |
dansmith | 1. Mock out the configure and warn if it's called | 18:12 |
dansmith | 2. Actually poison it and fix the tests that need fixing | 18:12 |
dansmith | 1 would just not actually do the configure | 18:12 |
dansmith | which should be a small patch to land and disarm the bomb | 18:12 |
dansmith | without depending on a bunch of test changer | 18:12 |
*** ctrath1 has joined #openstack-nova | 18:12 | |
*** ctrath has quit IRC | 18:13 | |
*** phuongnh has joined #openstack-nova | 18:13 | |
*** aginwala has quit IRC | 18:15 | |
*** EinstCrazy has quit IRC | 18:15 | |
*** EinstCrazy has joined #openstack-nova | 18:16 | |
alaski | on Flavor.create() it's not calling configure directly. the setup happens somewhere in the long chain of magic enginefacade things that happen | 18:17 |
*** aginwala has joined #openstack-nova | 18:18 | |
*** heyongli has quit IRC | 18:19 | |
*** heyongli has joined #openstack-nova | 18:19 | |
*** EinstCrazy has quit IRC | 18:19 | |
*** boris-42 has joined #openstack-nova | 18:21 | |
dansmith | I haven't found a case where it hit the poison yet | 18:22 |
*** pushkaru has joined #openstack-nova | 18:22 | |
melwitt | alaski: I noticed many things can "start" things without configure. or at least get_legacy_facade as you mentioned | 18:22 |
alaski | dansmith: here's the traceback of the Flavor.create() updating _start inappropriately http://pastebin.com/TReJ64kq | 18:23 |
*** ducttape_ has joined #openstack-nova | 18:24 | |
alaski | melwitt: yeah, it's a tangled web that just sort of assumes it should be done at almost any point | 18:24 |
*** daemontool has quit IRC | 18:25 | |
dansmith | alaski: okay yeah | 18:26 |
markus_z | cfriesen__: Could you already have a look at https://bugs.launchpad.net/nova/+bug/1296967 ? It's pretty old in the mean time. | 18:26 |
openstack | Launchpad bug 1296967 in OpenStack Compute (nova) "instances stuck with task_state of REBOOTING after controller switchover" [High,Incomplete] | 18:26 |
*** PaulMurray has quit IRC | 18:28 | |
melwitt | alaski: nice detective work :) | 18:29 |
*** heyongli has quit IRC | 18:29 | |
dansmith | well, for sure, | 18:29 |
dansmith | but I still haven't gotten anything here to hit that poison | 18:29 |
*** heyongli has joined #openstack-nova | 18:29 | |
alaski | melwitt: thanks :) | 18:31 |
alaski | dansmith: what are you poisoning? | 18:33 |
dansmith | alaski: the neighbor's dog | 18:33 |
dansmith | oh | 18:33 |
dansmith | nova.db.sqlalchemy.api.configure | 18:33 |
*** jed56 has quit IRC | 18:33 | |
alaski | heh | 18:33 |
alaski | I think it needs to be db_api.main_context_manager and api.main_context_manager | 18:33 |
alaski | it's the use of those as wrappers that's implicitly configuring | 18:34 |
*** paul-carlton1 has quit IRC | 18:34 | |
dansmith | hmm | 18:34 |
dansmith | I thought it still calls configure | 18:34 |
alaski | err, I guess main_context_manager and api_context_manager | 18:34 |
*** vilobhmm11 has joined #openstack-nova | 18:35 | |
dansmith | so I have to poison them with legit deorators? | 18:35 |
dansmith | not sure that works since I'm running later than they're running | 18:35 |
alaski | no, it goes into enginefacade land and half configures itself there | 18:35 |
melwitt | alaski: those decorators are on everything in the db api, I was wondering how it doesn't happen a lot more | 18:35 |
alaski | poison enginefacade.transaction_context maybe? | 18:36 |
dansmith | cripes this is ugly | 18:36 |
*** Marga_ has quit IRC | 18:36 | |
markus_z | ndipanov: Could you help me to understand this one please? https://bugs.launchpad.net/nova/+bug/1441903 | 18:36 |
openstack | Launchpad bug 1441903 in OpenStack Compute (nova) "rootwrap.d ln doesn't work for non iSCSI volumes" [Undecided,Incomplete] - Assigned to lvmxh (shaohef) | 18:36 |
*** fawadkhaliq has joined #openstack-nova | 18:36 | |
alaski | melwitt: I think it's a combo of two things, the decorator being run in a NoDBTestCase class, and the decorated method not being mocked out there | 18:37 |
*** tongli has joined #openstack-nova | 18:37 | |
* melwitt ponders | 18:37 | |
alaski | actually the decorated method being mocked may not matter | 18:38 |
dansmith | alaski: so you're thinking: 'oslo_db.sqlalchemy.enginefacade._TransactionFactory.configure', | 18:38 |
dansmith | ? | 18:38 |
*** heyongli has quit IRC | 18:39 | |
alaski | I don't think it ever goes through configure in this case | 18:39 |
*** heyongli has joined #openstack-nova | 18:39 | |
alaski | oslo_db.sqlalchemy.enginefacade._TransactionFactory._create_session is the culprit I believe | 18:40 |
dansmith | I don't see anywhere else I can get hold of it | 18:40 |
dansmith | oh | 18:40 |
dansmith | yeah, I might've meant that | 18:40 |
*** angdraug has joined #openstack-nova | 18:40 | |
dansmith | okay I got hits | 18:42 |
alaski | you could do it on _transaction_scope() | 18:42 |
dansmith | hey what do you know, eight hits | 18:42 |
alaski | okay | 18:42 |
* dansmith takes a big poop on this | 18:42 | |
*** aginwala has quit IRC | 18:43 | |
*** ihrachys has quit IRC | 18:43 | |
alaski | I'm narrowing in on the staticmethod issue as well. it's a problem in fixtures, I'm fairly certain | 18:43 |
dansmith | alaski: you know it was the self.stub_out() case that was causing it right? | 18:44 |
alaski | yeah, I can reliably reproduce it with two test classes | 18:44 |
alaski | stub_out uses fixtures.MonkeyPatch which I'm fairly sure is the culprit here | 18:45 |
dansmith | so you're going to fix it so I have to move all this shit back before we merge eh? | 18:45 |
alaski | I aim to make your life difficult | 18:45 |
dansmith | bah | 18:46 |
dansmith | okay I think I got this | 18:46 |
*** lpetrut has joined #openstack-nova | 18:46 | |
*** heyongli has quit IRC | 18:49 | |
*** haomaiwang has joined #openstack-nova | 18:49 | |
diana_clarke | :( | 18:49 |
*** heyongli has joined #openstack-nova | 18:50 | |
dansmith | sdague: how do you want this fixture tested? it's mocking out something deep in private oslo_db internals | 18:51 |
dansmith | sdague: which is an instance method.. so I can reach deep in there and make sure it returns the magicmock I expect | 18:51 |
sdague | dansmith: honestly, I don't even know | 18:51 |
*** dkliban has joined #openstack-nova | 18:51 | |
dansmith | but it's fairly synthetic | 18:51 |
sdague | yeh | 18:51 |
dansmith | once the poison really poisons, then I can test that it explodes | 18:51 |
dansmith | right now, this just disarms | 18:51 |
dansmith | without changing all the tests | 18:51 |
sdague | sure | 18:51 |
sdague | honestly, for something like this, I'm not sure there is any non synthetic testing for it | 18:52 |
dansmith | cool | 18:52 |
sdague | do realize, we are mocking private parts of oslo.db, which means, that might randomly change | 18:52 |
sdague | we do need a real bug up on oslo.db to give us a well defined way to do this thing | 18:53 |
sdague | the other thing I was wondering, the reason that this works at all is that oslo.db has some default sqlite connect string, right? | 18:53 |
sdague | what if we made that default to garbage | 18:54 |
dansmith | um | 18:54 |
dansmith | not sure about that | 18:54 |
sdague | and only set it in our fixture for the db | 18:54 |
dansmith | but agreed, some oslo_db bug to let us disable auto magic conf would be good | 18:54 |
*** haomaiwang has quit IRC | 18:54 | |
*** aginwala has joined #openstack-nova | 18:55 | |
*** zenoway has joined #openstack-nova | 18:55 | |
alaski | sdague: it's not even getting as far as configuring/using a db. the wrappers set some state in the TransactionContextFactory that causes the later session.configure() to blow up. All this happens above actually looking at a db | 18:55 |
sdague | alaski: gah | 18:56 |
dansmith | oh, right | 18:56 |
dansmith | I don't even know what to write in the bug actually | 18:56 |
dansmith | although I could probably work something out assuming launchpad supports unicode | 18:56 |
alaski | heh | 18:56 |
alaski | it's arguably a bug in our usage for using the wrappers before configuring things | 18:57 |
*** aginwala has quit IRC | 18:57 | |
dansmith | well, | 18:57 |
dansmith | setting state when we're not configured seems bad to me | 18:57 |
dansmith | obviously we're hitting this because of accidental stuff in our tests | 18:57 |
alaski | agreed | 18:57 |
*** aginwala has joined #openstack-nova | 18:58 | |
sdague | right, but the library needs to be durable | 18:58 |
dansmith | but without a wedge or something it's hard to catch | 18:58 |
alaski | I would expect it to yell at us for doing it wrong | 18:58 |
dansmith | alaski: does it seem reasonable to not just explode if we do this before configuring | 18:58 |
dansmith | yeah | 18:58 |
sdague | good interfaces aren't just easy to use, they are hard to misuse | 18:58 |
*** mylu has joined #openstack-nova | 18:58 | |
sdague | and this is way too easy to misuse and not know it | 18:58 |
sdague | dansmith: I think exploding before configuring sounds right to me | 18:58 |
*** Sukhdev has joined #openstack-nova | 18:59 | |
dansmith | yeah | 18:59 |
dansmith | it's what I'm trying to do subversively | 18:59 |
sdague | right, but that should be the default behavior of the library | 18:59 |
dansmith | yeah | 18:59 |
*** heyongli has quit IRC | 19:00 | |
*** jichen has joined #openstack-nova | 19:00 | |
*** heyongli has joined #openstack-nova | 19:00 | |
*** Marga_ has joined #openstack-nova | 19:02 | |
*** ircuser-1 has quit IRC | 19:02 | |
*** mtanino has quit IRC | 19:04 | |
*** palexster has quit IRC | 19:05 | |
*** gszasz has quit IRC | 19:06 | |
*** mylu has quit IRC | 19:07 | |
*** achanda has quit IRC | 19:07 | |
*** mylu has joined #openstack-nova | 19:07 | |
dansmith | gah the resource pool functional tests fail with this on | 19:10 |
*** aswadr_ has quit IRC | 19:10 | |
*** heyongli has quit IRC | 19:10 | |
dansmith | and the cell mapping ones | 19:10 |
*** heyongli has joined #openstack-nova | 19:10 | |
*** ihrachys has joined #openstack-nova | 19:12 | |
*** ihrachys has quit IRC | 19:13 | |
*** rmevans has joined #openstack-nova | 19:13 | |
openstackgerrit | Mark Doffman proposed openstack/nova: Add aggregates tables to the API db. https://review.openstack.org/294806 | 19:15 |
openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate.get operation favor the API db https://review.openstack.org/296014 | 19:15 |
*** ihrachys has joined #openstack-nova | 19:15 | |
*** vilobhmm111 has joined #openstack-nova | 19:16 | |
*** achanda has joined #openstack-nova | 19:16 | |
dansmith | doffm: that looks familiar | 19:16 |
*** ociuhandu has quit IRC | 19:16 | |
dansmith | alaski: what was the point of being a nodb test and then using the fixture? | 19:17 |
dansmith | because that's getting in my way here | 19:17 |
alaski | it's only setting up one db rather than both | 19:17 |
*** ametts has quit IRC | 19:17 | |
*** ihrachys has quit IRC | 19:17 | |
*** yassine__ has quit IRC | 19:17 | |
dansmith | okay | 19:17 |
* dansmith steams | 19:17 | |
alaski | we can set them both up for now and then work around it later if necessary | 19:18 |
dansmith | well, I ahve to do it for several tests if I do | 19:18 |
*** vilobhmm11 has quit IRC | 19:18 | |
*** heyongli has quit IRC | 19:20 | |
*** heyongli has joined #openstack-nova | 19:21 | |
*** rmevans has quit IRC | 19:25 | |
mriedem | bauzas: should this be abandoned? https://review.openstack.org/#/c/221384/ | 19:26 |
bauzas | mriedem: well, the problem is still there but I had zero time to work on it | 19:27 |
*** mylu has quit IRC | 19:27 | |
mriedem | bauzas: ok, np | 19:27 |
*** rmevans has joined #openstack-nova | 19:27 | |
*** ihrachys has joined #openstack-nova | 19:27 | |
mriedem | edleafe: should this be abandoned? https://review.openstack.org/#/c/242636/ | 19:28 |
bauzas | mriedem: the main discussion I had with johnthetubaguy was about having some way to see if deading the cache wouldn't hit the perf | 19:28 |
*** fawadkhaliq has quit IRC | 19:30 | |
edleafe | mriedem: Yeah, there doesn't seem to be support for it. I'll abandon it. | 19:30 |
*** mylu has joined #openstack-nova | 19:30 | |
*** fawadkhaliq has joined #openstack-nova | 19:30 | |
*** heyongli has quit IRC | 19:30 | |
*** heyongli has joined #openstack-nova | 19:31 | |
*** mylu has quit IRC | 19:34 | |
openstackgerrit | Dan Smith proposed openstack/nova: Add a DatabasePoisonFixture https://review.openstack.org/296023 | 19:35 |
dansmith | alaski: sdague ^ | 19:35 |
alaski | cool | 19:35 |
dansmith | alaski: no not cool | 19:35 |
alaski | dansmith: https://github.com/testing-cabal/fixtures/pull/23 fixes your other issue | 19:35 |
dansmith | alaski: I'm going to hell for that | 19:35 |
*** ociuhandu has joined #openstack-nova | 19:35 | |
alaski | heh | 19:35 |
dansmith | alaski: I'm not sure if I'm happy or not | 19:36 |
*** fhermeni has joined #openstack-nova | 19:36 | |
fhermeni | hi | 19:36 |
alaski | dansmith: I continued on it because I was curious, but it may also fix some of our other py34 failures | 19:36 |
dansmith | alaski: yeah, that's cool | 19:37 |
dansmith | kinda nasty issue | 19:37 |
dansmith | I'm sure I've hit that before and never noticed | 19:37 |
dansmith | or known why it was that way | 19:37 |
alaski | yeah | 19:37 |
*** harlowja_at_home has quit IRC | 19:37 | |
alaski | what took forever to find it is that setattr() appears to work fine until you instantiate the class | 19:38 |
dansmith | so I'm checking my functional tests here and then I'll try rebasing my flavor crap on top of that fix and see if I can get the blocker at the end to be happy | 19:38 |
dansmith | heh, yeah | 19:38 |
*** heyongli has quit IRC | 19:41 | |
*** heyongli has joined #openstack-nova | 19:41 | |
*** mylu has joined #openstack-nova | 19:42 | |
*** mylu has quit IRC | 19:44 | |
openstackgerrit | Dan Smith proposed openstack/nova: Add a DatabasePoisonFixture https://review.openstack.org/296023 | 19:44 |
*** dkliban has left #openstack-nova | 19:45 | |
*** mylu has joined #openstack-nova | 19:47 | |
*** heyongli has quit IRC | 19:51 | |
*** heyongli has joined #openstack-nova | 19:51 | |
*** ircuser-1 has joined #openstack-nova | 19:51 | |
*** ccarmack has joined #openstack-nova | 19:52 | |
*** mylu has quit IRC | 19:52 | |
*** jwcroppe has quit IRC | 19:54 | |
*** david-lyle_ has joined #openstack-nova | 19:56 | |
*** kfarr has joined #openstack-nova | 19:57 | |
*** david-lyle has quit IRC | 19:57 | |
*** gongysh has quit IRC | 19:58 | |
*** ekarlso- has quit IRC | 19:58 | |
*** jeblair has quit IRC | 19:58 | |
*** jwcroppe has joined #openstack-nova | 19:58 | |
*** salv-orlando has quit IRC | 19:58 | |
*** jeblair has joined #openstack-nova | 19:58 | |
*** jlvillal has quit IRC | 19:58 | |
*** tanee has quit IRC | 19:58 | |
dansmith | alaski: my set on top of that fixture seems to work well | 20:00 |
dansmith | I mean, I can get at the fails I really need to get at | 20:00 |
alaski | great | 20:01 |
*** heyongli has quit IRC | 20:01 | |
*** mylu has joined #openstack-nova | 20:01 | |
*** heyongli has joined #openstack-nova | 20:01 | |
*** david-lyle_ is now known as david-lyle | 20:02 | |
*** baoli has quit IRC | 20:02 | |
*** jlvillal has joined #openstack-nova | 20:03 | |
*** tanee has joined #openstack-nova | 20:04 | |
*** yassine has joined #openstack-nova | 20:04 | |
*** Sukhdev has quit IRC | 20:05 | |
*** yassine is now known as Guest68794 | 20:05 | |
*** sc68cal has quit IRC | 20:09 | |
*** ekarlso- has joined #openstack-nova | 20:11 | |
*** heyongli has quit IRC | 20:11 | |
*** heyongli has joined #openstack-nova | 20:12 | |
*** aginwala has quit IRC | 20:13 | |
*** gongysh has joined #openstack-nova | 20:14 | |
*** atuvenie has quit IRC | 20:16 | |
*** sc68cal has joined #openstack-nova | 20:17 | |
*** aginwala has joined #openstack-nova | 20:18 | |
*** mvk_ has quit IRC | 20:18 | |
*** heyongli has quit IRC | 20:22 | |
*** heyongli has joined #openstack-nova | 20:22 | |
*** atuvenie has joined #openstack-nova | 20:22 | |
jaypipes | dansmith: on 284895, do we want to encourage any more additions to nova.db.api? I thought we were from now on just doing the SQLAlchmey calls directly in the object facade. is that correct? | 20:22 |
dansmith | jaypipes: yeah, although we've mostly just been doing that in api-db objects | 20:23 |
dansmith | but we can start spreading it | 20:24 |
jaypipes | dansmith: right, but aggregates are intended to move to API db, right? | 20:24 |
*** drankis has quit IRC | 20:24 | |
dansmith | true story | 20:24 |
*** mylu has quit IRC | 20:25 | |
alaski | cdent: https://github.com/testing-cabal/fixtures/pull/23 is the postmortem from the issue yesterday | 20:25 |
cdent | thanks alaski | 20:26 |
dansmith | meta question: why do I always get stuck working on flavor stuff? | 20:26 |
dansmith | I don't even like flavors | 20:26 |
*** rdopiera has quit IRC | 20:26 | |
jaypipes | lol | 20:26 |
jaypipes | dansmith: sucker for pain? | 20:27 |
*** lpetrut has quit IRC | 20:27 | |
dansmith | jaypipes: apparently I'm just really really stupid | 20:27 |
alaski | would it help if we called them instance_types? | 20:27 |
dansmith | no. | 20:27 |
jaypipes | heh | 20:27 |
dansmith | it might help if we called them "jaybirds" | 20:27 |
dansmith | cause I know jaypipes loooooves them | 20:27 |
jaypipes | heh | 20:27 |
*** mylu has joined #openstack-nova | 20:30 | |
*** fawadkhaliq has quit IRC | 20:30 | |
*** fawadkhaliq has joined #openstack-nova | 20:30 | |
*** sdague has quit IRC | 20:31 | |
*** toure_ is now known as toure | 20:32 | |
*** heyongli has quit IRC | 20:32 | |
*** heyongli has joined #openstack-nova | 20:32 | |
*** achanda has quit IRC | 20:34 | |
*** moshele has quit IRC | 20:37 | |
*** moshele has joined #openstack-nova | 20:38 | |
*** aginwala has quit IRC | 20:38 | |
*** fawadkhaliq has quit IRC | 20:40 | |
*** mylu has quit IRC | 20:40 | |
*** atuvenie has quit IRC | 20:40 | |
*** fawadkhaliq has joined #openstack-nova | 20:40 | |
*** aginwala has joined #openstack-nova | 20:41 | |
*** hamzy has quit IRC | 20:41 | |
*** heyongli has quit IRC | 20:42 | |
*** heyongli has joined #openstack-nova | 20:43 | |
*** mylu has joined #openstack-nova | 20:46 | |
*** efoley has joined #openstack-nova | 20:47 | |
mriedem | dansmith: i had some questions in https://review.openstack.org/#/c/294252/ but didn't want to score, and didn't want to interrupt earlier | 20:49 |
*** jwcroppe has quit IRC | 20:49 | |
*** fawadkhaliq has quit IRC | 20:50 | |
dansmith | mriedem: I shall fix the alignment when I resubmit on top of the db fix | 20:51 |
mriedem | that was the least of my worries :) | 20:51 |
dansmith | heh | 20:51 |
*** fawadkhaliq has joined #openstack-nova | 20:51 | |
dansmith | the original patch was full of copy-and-paste alignment stuff | 20:51 |
dansmith | it was bad | 20:51 |
*** jaypipes has quit IRC | 20:52 | |
dansmith | man, the xenapi driver is just always the one that is reaching way too far into the DB internals for being a virt driver | 20:52 |
*** heyongli has quit IRC | 20:52 | |
*** e0ne has quit IRC | 20:53 | |
*** jwcroppe has joined #openstack-nova | 20:53 | |
*** heyongli has joined #openstack-nova | 20:53 | |
dansmith | I'm always like " this should be done" and I do a full run to find the xenapi driver mucking with aggregates or flavors or something | 20:53 |
*** wanghao has quit IRC | 20:53 | |
dansmith | 46 flavor fails in xenapi, zero in any other driver | 20:54 |
*** wanghao has joined #openstack-nova | 20:54 | |
*** salv-orlando has joined #openstack-nova | 20:56 | |
*** moshele has quit IRC | 20:57 | |
*** geedo has joined #openstack-nova | 20:57 | |
*** rtheis has quit IRC | 20:57 | |
*** jwcroppe has quit IRC | 20:58 | |
mriedem | dansmith: how about this? https://review.openstack.org/#/c/294252/6/nova/tests/unit/objects/test_flavor.py@418 | 20:58 |
*** raildo is now known as raildo-afk | 20:58 | |
dansmith | oh yeah probably | 20:59 |
*** Sukhdev has joined #openstack-nova | 20:59 | |
mriedem | i also had a follow on question about that marker scenario | 20:59 |
mriedem | i'm probably overthinking that one | 21:00 |
*** thorst has quit IRC | 21:00 | |
*** thorst has joined #openstack-nova | 21:00 | |
dansmith | did you check the functional tests? | 21:02 |
dansmith | I think we could go either way, | 21:02 |
dansmith | of returning a short page or not | 21:02 |
dansmith | but I think I'm doing what I would expect from the API | 21:02 |
*** heyongli has quit IRC | 21:03 | |
*** heyongli has joined #openstack-nova | 21:03 | |
*** thorst has quit IRC | 21:05 | |
*** mylu has quit IRC | 21:05 | |
mriedem | dansmith: doffm and i just worked out the scenario on the whiteboard here, i think what was messing me up was thinking about the marker and the primary key resetting in the api/cell db but he pointed out that the id is going to be carried over in the migration to the api db | 21:05 |
dansmith | yeah | 21:05 |
dansmith | but either way, | 21:05 |
dansmith | this is flavorid | 21:05 |
dansmith | which makes it even silly-er | 21:06 |
dansmith | it's not very guaranteed deterministic | 21:06 |
*** mylu has joined #openstack-nova | 21:06 | |
dansmith | even though it's sorted, you could create another flavor in the middle and mess it up | 21:06 |
mriedem | yeah..... | 21:06 |
mriedem | because the default sort key is flavor_id | 21:06 |
dansmith | eyah | 21:06 |
mriedem | and by default flavorid is a uuid | 21:06 |
mriedem | so good luck asshole! | 21:06 |
dansmith | hah, yeah | 21:07 |
dansmith | that like HTTP 442 I think | 21:07 |
mriedem | is that the kick you in the balls code? | 21:07 |
dansmith | similar | 21:07 |
*** haomaiwa_ has joined #openstack-nova | 21:08 | |
*** dims has joined #openstack-nova | 21:11 | |
*** haomaiwa_ has quit IRC | 21:12 | |
openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: set service status based on vc connection https://review.openstack.org/290315 | 21:12 |
*** heyongli has quit IRC | 21:13 | |
openstackgerrit | Mark Doffman proposed openstack/nova: Make AggregateList.get_ return API & cell db items https://review.openstack.org/296062 | 21:13 |
*** heyongli has joined #openstack-nova | 21:13 | |
*** haplo37 has quit IRC | 21:13 | |
*** efried has quit IRC | 21:16 | |
*** efried has joined #openstack-nova | 21:18 | |
*** rk4n has joined #openstack-nova | 21:18 | |
*** Sukhdev has quit IRC | 21:19 | |
*** thorst has joined #openstack-nova | 21:22 | |
*** heyongli has quit IRC | 21:23 | |
*** rk4n has quit IRC | 21:23 | |
*** heyongli has joined #openstack-nova | 21:23 | |
*** thorst has quit IRC | 21:26 | |
*** cdent has quit IRC | 21:27 | |
dansmith | alaski: I hold you personally responsible for the hate I'm currently experiencing from the xenapi driver | 21:32 |
*** mylu has quit IRC | 21:32 | |
*** MVenesio has quit IRC | 21:33 | |
mriedem | so i thought we didn't like cluster drivers? https://review.openstack.org/#/c/279043/ | 21:33 |
*** heyongli has quit IRC | 21:33 | |
*** heyongli has joined #openstack-nova | 21:34 | |
*** achanda has joined #openstack-nova | 21:34 | |
dansmith | are you asking about a thing you approved? | 21:34 |
mriedem | dansmith: heh, well, trivial re-approval | 21:36 |
mriedem | and then i started reading it | 21:36 |
*** rk4n has joined #openstack-nova | 21:36 | |
mriedem | it just reminds me of the periodic task for syncing power states that garyk complains about every few months because it shuts down vm's in the vcenter cluster that is doing it's own HA | 21:37 |
*** ssurana has joined #openstack-nova | 21:39 | |
openstackgerrit | Christopher J Schaefer proposed openstack/python-novaclient: Adding tox support for bandit https://review.openstack.org/296079 | 21:39 |
*** achanda has quit IRC | 21:40 | |
*** daemontool has joined #openstack-nova | 21:41 | |
*** atuvenie has joined #openstack-nova | 21:41 | |
dims | mriedem : lol. yes, we said got the vcenter folks to change the way they do stuff | 21:41 |
*** aginwala has quit IRC | 21:42 | |
*** tongli has quit IRC | 21:42 | |
mriedem | dims: yeah, i think in that hyper-v spec there are the 2 options, and option A was what we nixed for vmware in liberty | 21:42 |
dims | dansmith mriedem : i had someone ask me about deploying to mesos via nova driver :) | 21:42 |
dims | ok | 21:42 |
*** ctrath1 has quit IRC | 21:42 | |
*** moshele has joined #openstack-nova | 21:43 | |
*** sc68cal has quit IRC | 21:44 | |
*** heyongli has quit IRC | 21:44 | |
*** kfarr has quit IRC | 21:44 | |
*** heyongli has joined #openstack-nova | 21:44 | |
openstackgerrit | Diana Clarke proposed openstack/nova: Clean up the TestGlanceClientWrapper retry tests https://review.openstack.org/296081 | 21:44 |
*** pushkaru has quit IRC | 21:45 | |
*** openstackgerrit has quit IRC | 21:48 | |
*** openstackgerrit has joined #openstack-nova | 21:49 | |
dims | dansmith : ayoung : honestly if i can get one cycle where oslo.messaging + rabbitmq is stable out of the box under scale, i'd be very very happy | 21:49 |
dansmith | dims: heh, me too. | 21:49 |
ayoung | dims, since we are dreaming, I want a new house | 21:50 |
dims | ayoung : dansmith : LOL | 21:50 |
*** ccarmack has quit IRC | 21:50 | |
ayoung | one with at least one bath room per family memeber | 21:50 |
*** ctrath has joined #openstack-nova | 21:50 | |
*** e0ne has joined #openstack-nova | 21:50 | |
dims | ayoung : dansmith : am trying to sell qpid/proton + pyngus as alternate in Fuel | 21:50 |
*** _ducttape_ has joined #openstack-nova | 21:50 | |
*** edtubill has quit IRC | 21:51 | |
dims | don't know what kinds of worms that will unearth | 21:51 |
ayoung | dims, might I interest you in a FreeIPA instance to go along with that? | 21:51 |
dims | ayoung : always up for free beer :) | 21:51 |
*** ctrath has quit IRC | 21:51 | |
ayoung | dims, I don't know if Rabbit is the issue, or if it really is that we overload it without giving it proper archictecture | 21:51 |
dims | ayoung : right | 21:52 |
ayoung | dims, we did a Proof of concept using RH IdM (FreeIPA) and Proton in Vancouver | 21:52 |
dims | ayoung : i've been meaning to dump from/to/via for all messages somehow for a while now | 21:52 |
ayoung | kerberos for encryption and authentication on the Queues | 21:52 |
dims | ayoung : nice | 21:52 |
ayoung | dims, that would be awesome....getting that mapped out is the first step | 21:52 |
*** mylu has joined #openstack-nova | 21:53 | |
dims | ayoung : y, haven't had a chance to dig into that | 21:53 |
ayoung | dims, http://adam.younglogic.com/2015/12/rippowam/ | 21:53 |
ayoung | dims, most especially https://github.com/admiyo/rippowam/blob/master/roles/packstack/tasks/proton.yml | 21:53 |
*** ducttape_ has quit IRC | 21:54 | |
*** heyongli has quit IRC | 21:54 | |
ayoung | dims, and, I just got a Tripleo deploy with IdM running in a container on the undercloud. I think I can make is such that IDM is just something you can assume will be there. | 21:54 |
*** Sukhdev has joined #openstack-nova | 21:54 | |
dims | ayoung : thanks, will read those in a bit. | 21:54 |
*** heyongli has joined #openstack-nova | 21:54 | |
*** jwcroppe has joined #openstack-nova | 21:54 | |
ayoung | dims, I know that proton was looking in to point to point queues, too | 21:54 |
ayoung | that should be the norm, I would think. Use the broker to ...well.. broker the setup of the processes | 21:55 |
ayoung | but then let the consumer and producer talk directly | 21:55 |
*** rlrossit_ has quit IRC | 21:55 | |
ayoung | I can understand why people wanted 0mq for that. | 21:55 |
dims | ayoung : zmq by default at the moment is point to point, that chews up sockets :( so large scale its a problem | 21:55 |
*** ctrath has joined #openstack-nova | 21:56 | |
ayoung | dims, point to point makes sense from compute node to compute node, only, and they should be short term queues | 21:56 |
ayoung | from conductor to compute node, broker | 21:56 |
*** _ducttape_ has quit IRC | 21:57 | |
*** mriedem has quit IRC | 21:58 | |
*** burt has quit IRC | 21:59 | |
*** pwnall has quit IRC | 21:59 | |
*** ihrachys has quit IRC | 21:59 | |
*** aginwala has joined #openstack-nova | 22:00 | |
*** fhermeni has quit IRC | 22:00 | |
*** jwcroppe has quit IRC | 22:02 | |
*** dimtruck is now known as zz_dimtruck | 22:03 | |
*** ccarmack has joined #openstack-nova | 22:04 | |
*** heyongli has quit IRC | 22:04 | |
*** heyongli has joined #openstack-nova | 22:05 | |
*** eharney has quit IRC | 22:06 | |
*** baoli has joined #openstack-nova | 22:07 | |
*** harlowja_at_home has joined #openstack-nova | 22:08 | |
*** gcb has quit IRC | 22:09 | |
*** mylu has quit IRC | 22:09 | |
*** baoli has quit IRC | 22:11 | |
*** hogepodge has quit IRC | 22:12 | |
*** mylu has joined #openstack-nova | 22:14 | |
*** darrenc is now known as darrenc_afk | 22:14 | |
*** heyongli has quit IRC | 22:14 | |
*** heyongli has joined #openstack-nova | 22:15 | |
*** aysyd has quit IRC | 22:17 | |
*** mdrabe has quit IRC | 22:17 | |
*** rk4n has quit IRC | 22:20 | |
*** e0ne has quit IRC | 22:21 | |
*** achanda has joined #openstack-nova | 22:21 | |
*** zenoway has quit IRC | 22:21 | |
*** rk4n has joined #openstack-nova | 22:22 | |
alaski | dansmith: I'm only responsible for the good things in the xenapi driver | 22:22 |
dansmith | orly | 22:22 |
* dansmith finds none | 22:22 | |
*** rk4n has quit IRC | 22:22 | |
*** gcb has joined #openstack-nova | 22:22 | |
*** moshele has quit IRC | 22:22 | |
*** zenoway has joined #openstack-nova | 22:22 | |
alaski | hah | 22:23 |
*** gcb has quit IRC | 22:23 | |
alaski | touché | 22:23 |
*** mylu has quit IRC | 22:23 | |
alaski | there are some very weird things in there, and unfortunately there hasn't been much incentive to change it because it mostly just works | 22:24 |
*** heyongli has quit IRC | 22:25 | |
*** heyongli has joined #openstack-nova | 22:25 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/nova: Implement setup_networks_on_host for Neutron networks https://review.openstack.org/275073 | 22:26 |
*** zenoway has quit IRC | 22:26 | |
*** pwnall has joined #openstack-nova | 22:27 | |
openstackgerrit | Ronald Bradford proposed openstack/python-novaclient: Removed unused Oslo Incubator code https://review.openstack.org/296092 | 22:31 |
*** Sukhdev has quit IRC | 22:31 | |
*** baoli has joined #openstack-nova | 22:32 | |
*** mylu has joined #openstack-nova | 22:33 | |
*** Sukhdev has joined #openstack-nova | 22:33 | |
*** baoli_ has joined #openstack-nova | 22:33 | |
dansmith | alaski: yeah, just so much legacy | 22:35 |
dansmith | alaski: just works because people keep kicking the can along when they update it for you :) | 22:35 |
*** heyongli has quit IRC | 22:35 | |
*** darrenc_afk is now known as darrenc | 22:35 | |
*** heyongli has joined #openstack-nova | 22:35 | |
*** jichen has quit IRC | 22:36 | |
alaski | :) | 22:36 |
*** baoli has quit IRC | 22:37 | |
*** Sukhdev has quit IRC | 22:37 | |
*** haomaiwang has joined #openstack-nova | 22:39 | |
*** vilobhmm111 has quit IRC | 22:39 | |
*** hogepodge has joined #openstack-nova | 22:39 | |
dansmith | omg I'm going to scream | 22:40 |
dansmith | I swear some of these tests are just waiting to fail until I've addressed everything else | 22:40 |
*** jichen has joined #openstack-nova | 22:40 | |
*** ociuhandu has quit IRC | 22:40 | |
*** achanda has quit IRC | 22:43 | |
*** haomaiwang has quit IRC | 22:43 | |
*** ayoung has quit IRC | 22:43 | |
*** Sukhdev has joined #openstack-nova | 22:44 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Remove unused argument v3mode https://review.openstack.org/296096 | 22:44 |
*** heyongli has quit IRC | 22:45 | |
*** heyongli has joined #openstack-nova | 22:45 | |
*** jichen has quit IRC | 22:48 | |
*** aginwala has quit IRC | 22:48 | |
*** knikolla has quit IRC | 22:49 | |
*** ivase has quit IRC | 22:50 | |
*** rmevans has quit IRC | 22:50 | |
* auggy sympathizes | 22:51 | |
*** pwnall has quit IRC | 22:55 | |
*** heyongli has quit IRC | 22:55 | |
*** heyongli has joined #openstack-nova | 22:56 | |
*** gcb has joined #openstack-nova | 22:56 | |
*** rmevans has joined #openstack-nova | 22:56 | |
*** unicell has quit IRC | 22:58 | |
*** ociuhandu has joined #openstack-nova | 22:59 | |
*** baoli_ has quit IRC | 23:00 | |
*** apoorvad has quit IRC | 23:01 | |
*** baoli has joined #openstack-nova | 23:01 | |
*** aginwala has joined #openstack-nova | 23:01 | |
*** zz_dimtruck is now known as dimtruck | 23:02 | |
*** rmevans has quit IRC | 23:05 | |
*** heyongli has quit IRC | 23:06 | |
*** heyongli has joined #openstack-nova | 23:06 | |
*** apoorvad has joined #openstack-nova | 23:06 | |
*** ctrath has quit IRC | 23:08 | |
*** smatzek has quit IRC | 23:08 | |
*** Guest68794 has quit IRC | 23:09 | |
*** dims_ has joined #openstack-nova | 23:12 | |
*** dims has quit IRC | 23:15 | |
*** dims has joined #openstack-nova | 23:15 | |
*** heyongli has quit IRC | 23:16 | |
*** zenoway has joined #openstack-nova | 23:16 | |
*** baoli has quit IRC | 23:16 | |
*** heyongli has joined #openstack-nova | 23:16 | |
*** dims_ has quit IRC | 23:18 | |
*** dims_ has joined #openstack-nova | 23:19 | |
*** zenoway has quit IRC | 23:20 | |
*** ivase has joined #openstack-nova | 23:21 | |
*** achanda has joined #openstack-nova | 23:22 | |
*** dims has quit IRC | 23:22 | |
*** Jeffrey4l has quit IRC | 23:23 | |
*** dims has joined #openstack-nova | 23:23 | |
dansmith | hope someone is still around to see this: | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Make FlavorList.get_all() return results from the API and main DBs https://review.openstack.org/294252 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Make flavor extra_specs operations work against the API DB https://review.openstack.org/294710 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Block flavor creation until main database is empty https://review.openstack.org/295310 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Add flavor migration routine https://review.openstack.org/294771 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Make Flavor access routines work against API database https://review.openstack.org/294313 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Make Flavor create() and destroy() work against API DB https://review.openstack.org/294758 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Objectify test_instance_type_extra_specs https://review.openstack.org/296103 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Objectify some tests in test_compute and test_flavors https://review.openstack.org/296104 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Objectify test_flavors and test_flavors_extra_specs https://review.openstack.org/296105 | 23:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Further hack up the nova.tests.unit.db.fakes module of horribleness https://review.openstack.org/296106 | 23:24 |
*** mylu has quit IRC | 23:24 | |
*** dims_ has quit IRC | 23:25 | |
*** ivase_ has joined #openstack-nova | 23:25 | |
*** ivase has quit IRC | 23:26 | |
*** mylu has joined #openstack-nova | 23:26 | |
*** heyongli has quit IRC | 23:26 | |
*** heyongli has joined #openstack-nova | 23:27 | |
*** mylu has quit IRC | 23:28 | |
*** ivase has joined #openstack-nova | 23:31 | |
*** ivase_ has quit IRC | 23:32 | |
*** MVenesio has joined #openstack-nova | 23:33 | |
*** mylu has joined #openstack-nova | 23:35 | |
melwitt | dansmith: it has been seen | 23:36 |
dansmith | lol | 23:36 |
*** Sukhdev has quit IRC | 23:36 | |
*** heyongli has quit IRC | 23:36 | |
melwitt | :D hehe | 23:37 |
*** heyongli has joined #openstack-nova | 23:37 | |
dansmith | I can't imagine a less-excited response | 23:37 |
*** ivase has quit IRC | 23:37 | |
melwitt | I was trying to be funny | 23:37 |
dansmith | "your electrons were processed by my eyeballs and I am unimpressed" | 23:37 |
melwitt | aside from that I was going to say, you're a real trooper doing all those patches | 23:38 |
*** MVenesio has quit IRC | 23:38 | |
*** dimtruck is now known as zz_dimtruck | 23:38 | |
dansmith | ugh, what a pain | 23:38 |
*** knikolla has joined #openstack-nova | 23:39 | |
*** mylu has quit IRC | 23:40 | |
*** geedo1 has joined #openstack-nova | 23:41 | |
*** mylu has joined #openstack-nova | 23:41 | |
melwitt | dansmith: thanks for doing all of that, you're awesome. and hopefully not too traumatized | 23:43 |
dansmith | I will reserve judgment on the traumatization until it's done | 23:43 |
dansmith | but thanks :) | 23:43 |
melwitt | heh | 23:43 |
*** markvoelker has quit IRC | 23:44 | |
*** heyongli has quit IRC | 23:47 | |
*** markvoelker has joined #openstack-nova | 23:47 | |
*** heyongli has joined #openstack-nova | 23:47 | |
*** fawadkhaliq has quit IRC | 23:48 | |
*** fawadkhaliq has joined #openstack-nova | 23:49 | |
*** zenoway has joined #openstack-nova | 23:52 | |
*** baoli has joined #openstack-nova | 23:54 | |
*** rfolco has quit IRC | 23:55 | |
*** ducttape_ has joined #openstack-nova | 23:56 | |
*** zenoway has quit IRC | 23:56 | |
*** heyongli has quit IRC | 23:57 | |
*** heyongli has joined #openstack-nova | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!