dansmith | oh. snap | 00:05 |
---|---|---|
*** Sukhdev has joined #openstack-nova | 00:06 | |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | 00:07 |
melwitt | dansmith: good news? | 00:08 |
dansmith | melwitt: yes, you'll be crapping on it soon | 00:08 |
melwitt | haha, nice | 00:09 |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | 00:09 |
*** d-bark has joined #openstack-nova | 00:11 | |
*** Sukhdev has quit IRC | 00:11 | |
*** bswartz has quit IRC | 00:12 | |
*** crushil has joined #openstack-nova | 00:15 | |
*** esberglu has joined #openstack-nova | 00:15 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Make start and stop admin actions cells-aware https://review.openstack.org/433362 | 00:16 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make CellDatabases fixture work over RPC https://review.openstack.org/434563 | 00:16 |
dansmith | melwitt: ^ | 00:16 |
*** mtanino has quit IRC | 00:16 | |
*** sdague has quit IRC | 00:18 | |
*** browne has joined #openstack-nova | 00:18 | |
*** thorst_ has joined #openstack-nova | 00:19 | |
*** thorst_ has quit IRC | 00:19 | |
*** Sukhdev has joined #openstack-nova | 00:20 | |
melwitt | dansmith: looks pretty good to me, actually. it's self-contained and makes it clear what's happening | 00:20 |
dansmith | melwitt: it works for my test, but haven't checked to see if it breaks anything else just yet | 00:20 |
*** yingjun has joined #openstack-nova | 00:20 | |
dansmith | \o/ | 00:20 |
*** hfu has joined #openstack-nova | 00:22 | |
*** nic has quit IRC | 00:29 | |
dansmith | heh, so many unit tests pass None for the context so I'll have to be defensive there | 00:29 |
*** adrian_otto has quit IRC | 00:30 | |
*** gongysh has joined #openstack-nova | 00:30 | |
melwitt | dansmith: yeah, I was just noticing that in my change I had "if context and context.db_connection" but it was just because of naughty unit tests probably | 00:30 |
dansmith | yeah | 00:30 |
dansmith | melwitt: did you see this? https://review.openstack.org/#/c/433298 | 00:31 |
dansmith | failed because of the cellsv1 breakage, but should be ready for review at least | 00:31 |
melwitt | maybe there were some tests that were checking for a None context on purpose, or something too | 00:31 |
dansmith | with that, I can boot an instance with "multiple cells" | 00:31 |
*** hongbin has quit IRC | 00:33 | |
*** Swami_ has joined #openstack-nova | 00:33 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Make CellDatabases fixture work over RPC https://review.openstack.org/434563 | 00:33 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make start and stop admin actions cells-aware https://review.openstack.org/433362 | 00:33 |
melwitt | dansmith: yeah, I wondered what you meant by that, meaning, how did you find you can create a true multi cell instance with it? | 00:33 |
dansmith | melwitt: because I have a hack for devstack that makes cell0 the main connection for the api service, which means anything not targeted fails | 00:34 |
dansmith | and it is how I hit these two first patches for multiple cell support .. i.e. it exposes where we're not targeting | 00:34 |
melwitt | oh, cool. so you saw it work in devstack. was just curious | 00:35 |
dansmith | melwitt: yep, with this: https://review.openstack.org/#/c/433318/ | 00:35 |
melwitt | nice | 00:35 |
dansmith | much simpler than actual multi-cell devstack and should let us flush out a ton of things without the complication | 00:36 |
melwitt | pretty cool | 00:36 |
*** mriedem has joined #openstack-nova | 00:37 | |
*** Swami_ has quit IRC | 00:38 | |
*** karimb has quit IRC | 00:44 | |
*** Sukhdev has quit IRC | 00:45 | |
*** jeffrey4l_ircc has joined #openstack-nova | 00:45 | |
*** gcb has quit IRC | 00:46 | |
*** thorst_ has joined #openstack-nova | 00:46 | |
*** thorst_ has quit IRC | 00:46 | |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/390654 | 00:47 |
*** masayukig_ is now known as masayukig | 00:49 | |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/390654 | 00:51 |
cburgess | mriedem do we have a start time for the PTG? 9AM? 10AM? | 00:52 |
*** hoangcx has joined #openstack-nova | 00:54 | |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.rescue notifications https://review.openstack.org/385644 | 00:54 |
dansmith | cburgess: 7am | 00:55 |
dansmith | donuts by 6:30 | 00:55 |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.rescue notifications https://review.openstack.org/385644 | 00:55 |
cburgess | dansmith *glare* | 00:55 |
cburgess | dansmith I wish there was a voodoo in ATL | 00:56 |
cburgess | We do have Krispy Kreme though which is decent. | 00:56 |
mriedem | cburgess: 9am | 00:56 |
*** huanxie has joined #openstack-nova | 00:56 | |
cburgess | I won't have a car though so bringing donuts might be tough. | 00:56 |
dansmith | cburgess: I'm sure you'll work it out | 00:56 |
cburgess | Getting dropped off or taking Marta or Uber most mornings. | 00:56 |
cburgess | mriedem Cool | 00:56 |
cburgess | dansmith Besides, donuts are only for Friday meetup. | 00:56 |
*** dave-mccowan has joined #openstack-nova | 00:57 | |
mriedem | cburgess: just remember that old saying, | 00:57 |
mriedem | cash, grass or ass | 00:57 |
dansmith | cburgess: remember ptg is "all friday all week" | 00:57 |
cburgess | mriedem LOL | 00:57 |
cburgess | dansmith Well that will be a productive time then.. oh wait... | 00:57 |
mriedem | dansmith: i don't think you've yet realized the amount of coordination that is going into the PTG scheduling now across teams | 00:58 |
mriedem | it's like the design summit now, but worse because there aren't actually slots | 00:58 |
mriedem | except those we make ourselves | 00:58 |
dansmith | mriedem: I'm sure :) | 00:58 |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.unrescue notifications https://review.openstack.org/388275 | 00:58 |
dansmith | cinder and nova will meet from 2:13 to 3:07 | 00:58 |
cburgess | This sounds like an award wining idea. | 00:58 |
*** huanxie has quit IRC | 00:59 | |
*** edmondsw has joined #openstack-nova | 01:00 | |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.unrescue notifications https://review.openstack.org/388275 | 01:00 |
*** liangy has joined #openstack-nova | 01:00 | |
mriedem | dansmith: since you're still around https://review.openstack.org/#/c/434552/ | 01:02 |
mriedem | and https://review.openstack.org/#/c/428671/ | 01:02 |
mriedem | ...plus anything else that catches your eye in https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton | 01:02 |
mriedem | tomorrow is the big release party | 01:02 |
*** phuongnh has joined #openstack-nova | 01:03 | |
openstackgerrit | yuhui proposed openstack/nova master: Fix some typo errors https://review.openstack.org/430629 | 01:03 |
mriedem | cburgess: are you in ATL because i thought you lived in CA | 01:04 |
*** edmondsw has quit IRC | 01:04 | |
mriedem | as in caleeforneea, not canada | 01:04 |
melwitt | may I suggest https://review.openstack.org/#/c/425114/ https://review.openstack.org/#/c/416557/ and https://review.openstack.org/#/c/416558/ for dansmith's eye | 01:04 |
dansmith | damn you people | 01:04 |
* melwitt looks for more | 01:05 | |
cburgess | mriedem I live in LA area (Altadena, CA), but I grew up in Atlanta, My folks still live there. | 01:05 |
mriedem | cburgess: orly, well then since you're the one that wanted a group dinner, | 01:06 |
mriedem | may i suggest you take on some delegation duties | 01:06 |
cburgess | mriedem Back away from that slows... | 01:06 |
cburgess | mriedem Damn.. I shouldn't have said anything. To be fair I haven't lived there in almost 20 years. | 01:06 |
mriedem | i was looking at places within walking distance of the sheraton and alma cocina was all i came up with | 01:06 |
cburgess | But ok.. I'll for places in/around downtown this weekend. | 01:06 |
cburgess | Yeah downtown is a big tough. | 01:06 |
dansmith | we should go to that steak place | 01:06 |
dansmith | kevin somethingorother | 01:07 |
mriedem | there is a "rays in the city" that got good reviews | 01:07 |
dansmith | they had a few things for under $100 a plate | 01:07 |
alaski | dansmith: +1 | 01:07 |
dansmith | alaski: what was that place? | 01:07 |
cburgess | mriedem Ray's is pricey though isn't it? | 01:07 |
dansmith | Kevin Goodsteakums or something | 01:07 |
mriedem | cburgess: yes | 01:07 |
alaski | http://www.kevinrathbunsteak.com/menu.html | 01:07 |
cburgess | Becaue the origian Ray's is out by the river and its swanky as hell. | 01:07 |
mriedem | i know a mike rathbun | 01:07 |
dansmith | alaski: ooooghhhhyeahhhummmmmahhhhyeahhh | 01:07 |
mriedem | cburgess: it would only be swankier if it were on a river boat | 01:07 |
melwitt | steak is good | 01:08 |
alaski | dansmith: sooo freaking good | 01:08 |
dansmith | alaski: so good I want to lick the screen right now | 01:09 |
melwitt | they have Blue rare on the menu, that sounds promising | 01:09 |
dansmith | but I won't... now. | 01:09 |
dansmith | melwitt: we shall go one night at least | 01:09 |
melwitt | \o/ | 01:09 |
alaski | You just need to find an uber driver to drop you off close to, but not quite at the place | 01:09 |
cburgess | alaski Why is that? | 01:09 |
dansmith | alaski: hah yeah that was hilarious | 01:10 |
dansmith | alaski: "here um, you're within a half mile g'luck" | 01:10 |
alaski | cburgess: that's just how it worked out last time | 01:10 |
cburgess | Oh lame | 01:10 |
melwitt | hah wat | 01:10 |
cburgess | I hae to admit the worse uber I ever had was last time in ATL at the summit | 01:10 |
dansmith | I watched alaski and comstud share a steak at this place | 01:11 |
dansmith | it was super romantic | 01:11 |
cburgess | mriedem We could just all go to waffle house. :) | 01:11 |
alaski | dansmith: the restaraunt was dimly lit and everything, there may have even been a candle at the table | 01:12 |
mriedem | cburgess: is there seating for 20? | 01:12 |
cburgess | mriedem Depend on the waffle hosue. | 01:12 |
dansmith | alaski: there was definitely inappropriate footsie under the table | 01:12 |
mriedem | dansmith: alaski: table clothes? | 01:12 |
*** iceyao has joined #openstack-nova | 01:13 | |
melwitt | dansmith: lol. I was just noticing all the dry-aged is for 2+, so that explains that. or does it? | 01:13 |
mriedem | was just going to say... | 01:13 |
dansmith | heh | 01:13 |
dansmith | sure, but did they have to hold hands the whole time? | 01:13 |
*** browne has quit IRC | 01:13 | |
melwitt | hah | 01:13 |
mriedem | ATL was the place that i almost got myself, dansmith and russellb locked in a parking gargage | 01:13 |
mriedem | *garage | 01:13 |
alaski | that clearly didn't happen because I couldn't cut my steak with one hand | 01:13 |
mriedem | alaski: comstud cut and fed you probably | 01:14 |
dansmith | mriedem: hmm, not sure I remember that | 01:14 |
dansmith | haha | 01:14 |
dansmith | alaski: busted. | 01:14 |
alaski | I held the fork, he cut... could've worked I think | 01:14 |
dansmith | alaski: actually it was french cut, if you recall, so not sure it needed much cutting | 01:15 |
*** claudiub has quit IRC | 01:15 | |
mriedem | is french cut the same as butterfly cut? | 01:15 |
*** fandi has joined #openstack-nova | 01:15 | |
dansmith | it was cut into slices | 01:15 |
*** tovin07_ has joined #openstack-nova | 01:15 | |
alaski | dansmith: that's true, and it practically melted anyways | 01:15 |
mriedem | oh i see | 01:15 |
dansmith | I thought they said french at the time | 01:15 |
mriedem | looks like brisket | 01:15 |
*** jwcroppe has quit IRC | 01:16 | |
mriedem | can i find some brisket while in GA? | 01:16 |
dansmith | alaski: yeah, you could stare it into smaller pieces if you needed | 01:16 |
cburgess | mriedem http://www.foxbrosbbq.com | 01:16 |
cburgess | Best in/around atlanta | 01:16 |
*** jwcroppe has joined #openstack-nova | 01:16 | |
*** browne has joined #openstack-nova | 01:16 | |
*** yuntongjin has quit IRC | 01:17 | |
mriedem | cburgess: that's only 10 minutes from the hilton | 01:17 |
mriedem | that might work | 01:18 |
cburgess | mriedem Yeah downtown is going to be tough. Its just not really where people go to eat. midtown, buckhead or decatur are really where people go eat and that would mean, car, uber, or subway | 01:18 |
jroll | bauzas: instance.host == compute_node.host, instance.node == compute_node.hypervisor_hostname | 01:19 |
cburgess | mriedem We could try fox bros but they don't do reservations and it can get tough to get into. We would probably need to just sit at smaller tables and have them sit us as they can. | 01:19 |
mriedem | hard to get into on a wed night? | 01:19 |
mriedem | at like 7pm? | 01:20 |
*** zzfoo has joined #openstack-nova | 01:20 | |
zzfoo | Hi, I'm trying to get a multinode devstack setup and stack.sh runs fine. But when i try to boot an instance I'm getting this error: | 01:20 |
*** ZZelle_ has quit IRC | 01:20 | |
cburgess | mriedem Yeah | 01:20 |
cburgess | Probably... | 01:20 |
zzfoo | "message": "Host 'dev-stack-old-2' is not mapped to any cell", "code": 400, "created": "2017-02-16T01:08:47Z" | 01:20 |
*** jwcroppe has quit IRC | 01:20 | |
mriedem | zzfoo: run "nova-manage cell_v2 discover_hosts" | 01:21 |
zzfoo | any ideas why this is occurring and if there is something special i need to do in devstack to get around it? | 01:21 |
cburgess | mriedem There is also http://sweetauburnbbq.com | 01:21 |
cburgess | Thats close too but again.. probably going to be a bit crowded. | 01:21 |
mriedem | zzfoo: run that from the controller node | 01:21 |
dansmith | mriedem: cburgess is kinda proving that he should be the dinner czar, amirite? | 01:21 |
mriedem | dansmith: yes | 01:21 |
dansmith | k, just checking | 01:21 |
mriedem | definitely not me because i can't decide things for groups | 01:22 |
cburgess | Sweet Auburn is at least less touristy | 01:22 |
cburgess | mriedem dansmith LOL | 01:22 |
zzfoo | mriedem: cool thanks, that did the trick. I guess this needs to be run each time a new host is added? | 01:22 |
mriedem | zzfoo: yup | 01:22 |
*** dave-mccowan has quit IRC | 01:23 | |
mriedem | zzfoo: https://docs.openstack.org/developer/nova/cells.html#setup-of-cells-v2 | 01:23 |
mriedem | cburgess: that place is also about 9 min from downtown | 01:23 |
mriedem | so would work | 01:23 |
*** kevinz has joined #openstack-nova | 01:23 | |
mriedem | i'm good with anything really for a large group | 01:23 |
mriedem | if we can't make reservations, | 01:23 |
mriedem | we probably just pick a place and a night and a time | 01:23 |
mriedem | and whoever shows up shows up | 01:24 |
mriedem | that's easier than asking for head count and restrictions and all that crap | 01:24 |
cburgess | mriedem yeah so Thursday night would work for me.. I can't actually do it Wednesday night. But I'm happy to help organize on Thursday night. | 01:25 |
mriedem | cburgess: that works for me | 01:25 |
cburgess | mriedem I can try calling sweet auburn and see if they will take a reso for 20. The issue might be if they want to split checks. | 01:25 |
dansmith | awesome | 01:25 |
cburgess | mriedem I'll contact sweet auburn tomorrow and see what we can do for Thursday. | 01:26 |
mriedem | cburgess: thanks, much appreciated | 01:26 |
mriedem | i was not looking forward to trying to organize a dinner | 01:26 |
*** jwcroppe has joined #openstack-nova | 01:27 | |
*** thorst_ has joined #openstack-nova | 01:29 | |
*** yuntongjin has joined #openstack-nova | 01:30 | |
*** thorst_ has quit IRC | 01:32 | |
*** esberglu has quit IRC | 01:34 | |
*** browne has quit IRC | 01:36 | |
*** wllabs has quit IRC | 01:36 | |
*** wllabs has joined #openstack-nova | 01:38 | |
*** fandi has quit IRC | 01:39 | |
*** sree has quit IRC | 01:42 | |
*** sree has joined #openstack-nova | 01:42 | |
*** liangy has quit IRC | 01:45 | |
*** sree has quit IRC | 01:46 | |
*** salv-orlando has joined #openstack-nova | 01:47 | |
*** jamesdenton has joined #openstack-nova | 01:50 | |
*** Jinxing has joined #openstack-nova | 01:51 | |
*** salv-orlando has quit IRC | 01:52 | |
*** Apoorva has quit IRC | 01:52 | |
*** gcb has joined #openstack-nova | 01:52 | |
*** jamesdenton has quit IRC | 01:54 | |
*** Jinxing has quit IRC | 01:56 | |
*** Jinxing has joined #openstack-nova | 01:56 | |
*** dave-mccowan has joined #openstack-nova | 01:57 | |
*** huanxie has joined #openstack-nova | 01:58 | |
*** jamesdenton has joined #openstack-nova | 01:58 | |
*** crushil has quit IRC | 02:03 | |
*** zerda4 has quit IRC | 02:05 | |
*** zerda3 has joined #openstack-nova | 02:06 | |
*** gongysh has quit IRC | 02:09 | |
*** eliqiao has quit IRC | 02:09 | |
mriedem | jroll: on https://review.openstack.org/#/c/399022/ should we have tooz in test-requirements like how python-ironicclient is in test-requirements? | 02:11 |
*** d-bark has quit IRC | 02:11 | |
*** unicell has quit IRC | 02:12 | |
*** yamahata has quit IRC | 02:16 | |
*** thorst_ has joined #openstack-nova | 02:17 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix doc generation warnings https://review.openstack.org/425549 | 02:18 |
*** thorst_ has quit IRC | 02:18 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608 | 02:19 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 02:20 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 02:22 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 02:23 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 02:24 |
*** yamahata has joined #openstack-nova | 02:25 | |
*** raunak has quit IRC | 02:32 | |
*** sam_nowitzki has joined #openstack-nova | 02:32 | |
*** esberglu has joined #openstack-nova | 02:38 | |
*** thorst_ has joined #openstack-nova | 02:38 | |
*** thorst_ has quit IRC | 02:40 | |
*** esberglu has quit IRC | 02:41 | |
*** zzfoo has quit IRC | 02:42 | |
*** thorst_ has joined #openstack-nova | 02:43 | |
*** thorst_ has quit IRC | 02:43 | |
*** crushil has joined #openstack-nova | 02:43 | |
*** tbachman has quit IRC | 02:43 | |
*** zerda3 has quit IRC | 02:44 | |
*** xyang1 has joined #openstack-nova | 02:44 | |
*** zerda3 has joined #openstack-nova | 02:45 | |
*** thorst_ has joined #openstack-nova | 02:46 | |
*** jeffrey4l_ircc has quit IRC | 02:47 | |
*** thorst_ has quit IRC | 02:51 | |
*** xyang1 has quit IRC | 02:52 | |
*** amotoki has joined #openstack-nova | 02:53 | |
*** ijw has quit IRC | 02:55 | |
mriedem | dansmith: i see how update_cell could not take effect because of host_mapping_cache in conductor also | 02:56 |
*** hongbin has joined #openstack-nova | 02:57 | |
mriedem | it's a map of hosts to hostmapping objects with the cell_mapping set | 02:57 |
mriedem | so if you change a cell mapping transport_url/db_connection you have to restart api and conductor | 02:57 |
*** yamahata has quit IRC | 02:57 | |
*** thorst_ has joined #openstack-nova | 03:01 | |
*** crushil has quit IRC | 03:02 | |
*** thorst_ has quit IRC | 03:03 | |
*** gongysh has joined #openstack-nova | 03:06 | |
*** mdrabe has joined #openstack-nova | 03:08 | |
*** mdrabe has quit IRC | 03:13 | |
*** panda|zZ has quit IRC | 03:18 | |
mriedem | jaypipes: i know you love you some BFV https://review.openstack.org/#/c/335358/ | 03:19 |
mriedem | ...and nightmares | 03:19 |
*** raunak has joined #openstack-nova | 03:21 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Fix hypervisor-stats show invalid info of baremetal resource https://review.openstack.org/391415 | 03:22 |
*** whenry has quit IRC | 03:24 | |
*** ssurana has joined #openstack-nova | 03:24 | |
*** ssurana has quit IRC | 03:24 | |
*** whenry has joined #openstack-nova | 03:40 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove dead code in _bury_in_cell0 https://review.openstack.org/434615 | 03:40 |
*** amotoki has quit IRC | 03:41 | |
*** nicolasbock has quit IRC | 03:42 | |
*** gouthamr has quit IRC | 03:44 | |
*** Hosam has joined #openstack-nova | 03:45 | |
*** Hosam has quit IRC | 03:45 | |
*** Hosam has joined #openstack-nova | 03:45 | |
*** mriedem is now known as mriedem_afk | 03:48 | |
*** salv-orlando has joined #openstack-nova | 03:48 | |
*** yamahata has joined #openstack-nova | 03:52 | |
*** salv-orlando has quit IRC | 03:53 | |
*** sudipto has joined #openstack-nova | 03:57 | |
*** sudipto_ has joined #openstack-nova | 03:57 | |
*** yuntongjin has quit IRC | 03:58 | |
*** armax has quit IRC | 03:59 | |
*** diana_clarke has quit IRC | 04:00 | |
*** ijw has joined #openstack-nova | 04:01 | |
*** links has joined #openstack-nova | 04:02 | |
*** whenry has quit IRC | 04:03 | |
*** nmathew has joined #openstack-nova | 04:07 | |
*** yamamoto has joined #openstack-nova | 04:10 | |
*** ratailor has joined #openstack-nova | 04:11 | |
*** Jinxing has quit IRC | 04:18 | |
*** dave-mccowan has quit IRC | 04:19 | |
*** tyrola has quit IRC | 04:19 | |
*** tyrola has joined #openstack-nova | 04:20 | |
*** adrian_otto has joined #openstack-nova | 04:26 | |
*** esberglu has joined #openstack-nova | 04:29 | |
*** amotoki has joined #openstack-nova | 04:29 | |
*** Hosam has quit IRC | 04:30 | |
*** sree has joined #openstack-nova | 04:32 | |
*** esberglu has quit IRC | 04:34 | |
*** tlian has quit IRC | 04:34 | |
*** mdnadeem has joined #openstack-nova | 04:35 | |
*** sree_ has joined #openstack-nova | 04:36 | |
*** sree_ is now known as Guest58863 | 04:36 | |
*** sree has quit IRC | 04:36 | |
*** hongbin has quit IRC | 04:38 | |
*** amotoki has quit IRC | 04:40 | |
*** jerrygb has quit IRC | 04:41 | |
*** Jinxing has joined #openstack-nova | 04:41 | |
*** sudipto has quit IRC | 04:44 | |
*** sudipto_ has quit IRC | 04:44 | |
*** amotoki has joined #openstack-nova | 04:48 | |
*** Jinxing has quit IRC | 04:49 | |
*** jamesdenton has quit IRC | 04:58 | |
*** Jinxing has joined #openstack-nova | 05:01 | |
*** thorst_ has joined #openstack-nova | 05:04 | |
*** erlon has quit IRC | 05:05 | |
*** sudipto has joined #openstack-nova | 05:07 | |
*** sudipto_ has joined #openstack-nova | 05:07 | |
*** Hosam has joined #openstack-nova | 05:08 | |
*** thorst_ has quit IRC | 05:08 | |
*** dikonoor has joined #openstack-nova | 05:12 | |
*** prateek has joined #openstack-nova | 05:13 | |
*** gongysh has quit IRC | 05:13 | |
*** adrian_otto has quit IRC | 05:15 | |
*** adisky_ has joined #openstack-nova | 05:16 | |
*** gongysh has joined #openstack-nova | 05:19 | |
*** ijw has quit IRC | 05:22 | |
*** dimtruck is now known as zz_dimtruck | 05:24 | |
*** salv-orlando has joined #openstack-nova | 05:28 | |
*** Jinxing has quit IRC | 05:29 | |
*** jaosorior has joined #openstack-nova | 05:31 | |
*** salv-orlando has quit IRC | 05:33 | |
*** unicell has joined #openstack-nova | 05:37 | |
*** dikonoor has quit IRC | 05:39 | |
*** diga has joined #openstack-nova | 05:43 | |
*** Jinxing has joined #openstack-nova | 05:43 | |
*** yingjun has quit IRC | 05:46 | |
*** Jinxing has quit IRC | 05:48 | |
*** Jinxing has joined #openstack-nova | 05:49 | |
*** tojuvone has joined #openstack-nova | 05:52 | |
*** adrian_otto has joined #openstack-nova | 05:52 | |
*** masber has quit IRC | 05:53 | |
*** salv-orlando has joined #openstack-nova | 05:53 | |
*** unicell has quit IRC | 05:53 | |
*** unicell has joined #openstack-nova | 05:54 | |
*** adrian_otto has quit IRC | 05:54 | |
*** dikonoor has joined #openstack-nova | 05:57 | |
*** sandanar has joined #openstack-nova | 05:58 | |
*** salv-orl_ has joined #openstack-nova | 06:04 | |
*** salv-orlando has quit IRC | 06:07 | |
openstackgerrit | Jens Rosenboom proposed openstack/nova master: Fix novncproxy for python3 https://review.openstack.org/434666 | 06:09 |
*** satyar has joined #openstack-nova | 06:17 | |
*** lpetrut has joined #openstack-nova | 06:20 | |
*** tuanluong has joined #openstack-nova | 06:20 | |
*** esberglu has joined #openstack-nova | 06:23 | |
*** claudiub has joined #openstack-nova | 06:25 | |
*** esberglu has quit IRC | 06:28 | |
*** edmondsw has joined #openstack-nova | 06:29 | |
*** edmondsw has quit IRC | 06:34 | |
*** adrian_otto has joined #openstack-nova | 06:41 | |
*** moshele has joined #openstack-nova | 06:43 | |
*** udesale has joined #openstack-nova | 06:43 | |
*** sree has joined #openstack-nova | 06:44 | |
*** diga has quit IRC | 06:45 | |
*** Guest58863 has quit IRC | 06:47 | |
*** Jinxing has quit IRC | 06:51 | |
*** raunak has quit IRC | 06:55 | |
*** hoangcx has quit IRC | 06:55 | |
*** ijw has joined #openstack-nova | 06:57 | |
*** hoangcx has joined #openstack-nova | 06:58 | |
*** Jinxing has joined #openstack-nova | 07:01 | |
*** seongu has joined #openstack-nova | 07:01 | |
*** hfu_ has joined #openstack-nova | 07:03 | |
*** rcernin has joined #openstack-nova | 07:03 | |
*** seongu99 has joined #openstack-nova | 07:03 | |
*** hfu_ has quit IRC | 07:03 | |
seongu99 | . | 07:03 |
*** hfu_ has joined #openstack-nova | 07:04 | |
*** andreas_s has joined #openstack-nova | 07:04 | |
*** sandanar_ has joined #openstack-nova | 07:04 | |
*** thorst_ has joined #openstack-nova | 07:05 | |
*** hfu has quit IRC | 07:05 | |
*** Jinxing has quit IRC | 07:05 | |
*** sandanar has quit IRC | 07:06 | |
*** seongu has quit IRC | 07:07 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Change parent of diagnostics objects to NovaObject class https://review.openstack.org/394480 | 07:07 |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 07:07 |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | 07:07 |
*** Jinxing has joined #openstack-nova | 07:07 | |
*** sandanar__ has joined #openstack-nova | 07:07 | |
*** seongu has joined #openstack-nova | 07:07 | |
*** seongu99 has quit IRC | 07:08 | |
*** thorst_ has quit IRC | 07:10 | |
*** sandanar_ has quit IRC | 07:11 | |
*** seongu has quit IRC | 07:12 | |
*** tesseract has joined #openstack-nova | 07:13 | |
*** lpetrut has quit IRC | 07:16 | |
*** jbryce has quit IRC | 07:18 | |
*** jbryce has joined #openstack-nova | 07:20 | |
*** ekuris has quit IRC | 07:21 | |
*** edand has joined #openstack-nova | 07:22 | |
*** cfriesen has quit IRC | 07:24 | |
*** yamamoto has quit IRC | 07:27 | |
*** sridharg has joined #openstack-nova | 07:27 | |
*** amotoki_ has joined #openstack-nova | 07:29 | |
*** amotoki has quit IRC | 07:32 | |
*** yingjun has joined #openstack-nova | 07:32 | |
*** gus_ is now known as gus | 07:32 | |
*** markus_z has joined #openstack-nova | 07:32 | |
*** ekuris has joined #openstack-nova | 07:35 | |
*** diga has joined #openstack-nova | 07:36 | |
*** karimb has joined #openstack-nova | 07:41 | |
*** willise has joined #openstack-nova | 07:43 | |
*** hoangcx has quit IRC | 07:44 | |
*** yingjun has quit IRC | 07:47 | |
*** yingjun has joined #openstack-nova | 07:47 | |
*** willise has left #openstack-nova | 07:48 | |
*** hfu has joined #openstack-nova | 07:48 | |
*** hfu_ has quit IRC | 07:49 | |
*** amoralej|off is now known as amoralej | 07:50 | |
*** adrian_otto has quit IRC | 07:55 | |
*** hoangcx has joined #openstack-nova | 07:55 | |
*** Jinxing has quit IRC | 07:57 | |
*** yingjun has quit IRC | 07:59 | |
*** vsaienko has joined #openstack-nova | 08:02 | |
*** yamamoto has joined #openstack-nova | 08:03 | |
*** adrian_otto has joined #openstack-nova | 08:08 | |
*** esberglu has joined #openstack-nova | 08:08 | |
*** esberglu has quit IRC | 08:13 | |
*** pcaruana has joined #openstack-nova | 08:14 | |
*** pcaruana has quit IRC | 08:20 | |
*** Jinxing has joined #openstack-nova | 08:21 | |
*** ltomasbo|away is now known as ltomasbo | 08:21 | |
*** adrian_otto has quit IRC | 08:21 | |
*** pcaruana has joined #openstack-nova | 08:22 | |
*** ijw has quit IRC | 08:23 | |
*** thorst_ has joined #openstack-nova | 08:25 | |
*** tyrola has quit IRC | 08:29 | |
*** thorst_ has quit IRC | 08:30 | |
*** tyrola has joined #openstack-nova | 08:30 | |
*** yamamoto has quit IRC | 08:32 | |
*** yamamoto has joined #openstack-nova | 08:33 | |
*** jpena|away is now known as jpena | 08:35 | |
*** yamamoto has quit IRC | 08:37 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Replace diagnostics objects with Nova diagnostics objects https://review.openstack.org/394480 | 08:39 |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 08:39 |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | 08:39 |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Added nova objects for intance diagnostics https://review.openstack.org/434745 | 08:39 |
*** owalsh-afk has quit IRC | 08:44 | |
*** yufei has joined #openstack-nova | 08:47 | |
*** avolkov has joined #openstack-nova | 08:48 | |
*** jpena is now known as jpena|away | 08:48 | |
*** ralonsoh has joined #openstack-nova | 08:49 | |
openstackgerrit | Anh Tran proposed openstack/nova master: Remove mox stubs in api/openstack/compute/test_server_reset_state.py https://review.openstack.org/320826 | 08:52 |
*** fragatin_ has joined #openstack-nova | 08:53 | |
*** jpena|away is now known as jpena | 08:53 | |
openstackgerrit | Huan Xie proposed openstack/nova master: Fix live migrate with XenServer https://review.openstack.org/424428 | 08:55 |
*** fragatina has quit IRC | 08:57 | |
*** fragatin_ has quit IRC | 08:58 | |
*** vsaienko has quit IRC | 08:59 | |
*** vsaienko has joined #openstack-nova | 08:59 | |
*** vsaienko has quit IRC | 08:59 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-nova | 09:00 | |
*** hfu has quit IRC | 09:03 | |
openstackgerrit | Jens Rosenboom proposed openstack/nova master: Fix novncproxy for python3 https://review.openstack.org/434666 | 09:06 |
*** hfu has joined #openstack-nova | 09:06 | |
*** yamahata has quit IRC | 09:06 | |
*** aarefiev_afk is now known as aarefiev | 09:09 | |
*** derekh has joined #openstack-nova | 09:15 | |
*** lpetrut has joined #openstack-nova | 09:15 | |
*** yamamoto has joined #openstack-nova | 09:17 | |
*** yizhongyin has joined #openstack-nova | 09:19 | |
*** yamamoto has quit IRC | 09:19 | |
*** yizhongyin has left #openstack-nova | 09:19 | |
*** yamamoto has joined #openstack-nova | 09:20 | |
*** yamahata has joined #openstack-nova | 09:21 | |
*** yamamoto has quit IRC | 09:25 | |
*** yamahata has quit IRC | 09:30 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Replace diagnostics objects with Nova diagnostics objects https://review.openstack.org/394480 | 09:30 |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 09:30 |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | 09:30 |
*** openstackgerrit has quit IRC | 09:32 | |
*** lucas-afk is now known as lucasagomes | 09:35 | |
*** diga has quit IRC | 09:38 | |
*** diga has joined #openstack-nova | 09:39 | |
*** gcb has quit IRC | 09:44 | |
*** gcb has joined #openstack-nova | 09:44 | |
*** Jinxing has quit IRC | 09:49 | |
*** hfu_ has joined #openstack-nova | 09:51 | |
*** hfu has quit IRC | 09:51 | |
*** claudiub has quit IRC | 09:53 | |
*** Hosam has quit IRC | 09:55 | |
*** hfu_ has quit IRC | 09:55 | |
*** Hosam has joined #openstack-nova | 09:55 | |
*** openstackgerrit has joined #openstack-nova | 09:55 | |
openstackgerrit | jichenjc proposed openstack/python-novaclient master: Avoid limit=-1 description https://review.openstack.org/434781 | 09:55 |
openstackgerrit | jichenjc proposed openstack/python-novaclient master: Avoid limit=-1 description https://review.openstack.org/434781 | 09:56 |
BobBall | mriedem_afk: We've update the patch and provided some comments to your questions. The Bridge driver is nova-net only and can be deprecated. Our Neutron integration only supports the OVS driver. | 09:57 |
*** tovin07_ has quit IRC | 09:58 | |
*** tovin07 has quit IRC | 09:58 | |
*** esberglu has joined #openstack-nova | 09:58 | |
*** liuyulong_ is now known as liuyulong | 09:58 | |
*** Hosam has quit IRC | 10:00 | |
*** amotoki_ has quit IRC | 10:01 | |
*** udesale has quit IRC | 10:03 | |
*** esberglu has quit IRC | 10:03 | |
*** ralonsoh_ has joined #openstack-nova | 10:04 | |
*** ralonsoh has quit IRC | 10:07 | |
*** sambetts|afk is now known as sambetts | 10:09 | |
*** amotoki has joined #openstack-nova | 10:13 | |
*** iceyao has quit IRC | 10:14 | |
*** mvk has quit IRC | 10:17 | |
*** erlon has joined #openstack-nova | 10:18 | |
*** amotoki has quit IRC | 10:20 | |
*** yamamoto has joined #openstack-nova | 10:22 | |
*** ralonsoh_ is now known as ralonsoh | 10:23 | |
*** udesale has joined #openstack-nova | 10:26 | |
*** thorst_ has joined #openstack-nova | 10:26 | |
*** mirrorbox has quit IRC | 10:28 | |
*** satyar has quit IRC | 10:29 | |
*** sam_nowitzki has quit IRC | 10:31 | |
*** thorst_ has quit IRC | 10:31 | |
*** mirrorbox has joined #openstack-nova | 10:31 | |
*** mirrorbox has joined #openstack-nova | 10:31 | |
*** tuanluong has quit IRC | 10:34 | |
*** amotoki has joined #openstack-nova | 10:35 | |
*** hoangcx has quit IRC | 10:36 | |
*** yufei has quit IRC | 10:36 | |
*** kevinz has quit IRC | 10:38 | |
*** yamamoto has quit IRC | 10:40 | |
*** edand has quit IRC | 10:40 | |
*** cdent has joined #openstack-nova | 10:42 | |
*** amotoki has quit IRC | 10:42 | |
*** markus_z has quit IRC | 10:42 | |
*** mirrorbox has quit IRC | 10:44 | |
*** owalsh has joined #openstack-nova | 10:44 | |
*** mvk has joined #openstack-nova | 10:52 | |
*** mirrorbox has joined #openstack-nova | 10:52 | |
*** dikonoor has quit IRC | 10:58 | |
*** ekuris has quit IRC | 11:00 | |
*** takashin has joined #openstack-nova | 11:01 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add polcy-docs spec https://review.openstack.org/433010 | 11:03 |
*** udesale has quit IRC | 11:03 | |
*** sree has quit IRC | 11:04 | |
*** tbachman has joined #openstack-nova | 11:05 | |
*** nicolasbock has joined #openstack-nova | 11:05 | |
*** sdague has joined #openstack-nova | 11:06 | |
*** Hosam has joined #openstack-nova | 11:09 | |
*** phuongnh has quit IRC | 11:11 | |
*** ekuris has joined #openstack-nova | 11:13 | |
*** claudiub has joined #openstack-nova | 11:14 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 11:15 |
*** jaosorior has quit IRC | 11:16 | |
*** jaosorior has joined #openstack-nova | 11:17 | |
*** sridharg has quit IRC | 11:17 | |
*** karthiks has quit IRC | 11:18 | |
*** sandanar__ has quit IRC | 11:25 | |
*** sudipto has quit IRC | 11:25 | |
*** sudipto_ has quit IRC | 11:25 | |
*** lucasagomes is now known as lucas-bbl | 11:26 | |
*** dikonoor has joined #openstack-nova | 11:26 | |
*** abalutoiu has joined #openstack-nova | 11:30 | |
*** rfolco has joined #openstack-nova | 11:30 | |
*** tbachman has quit IRC | 11:31 | |
*** ralonsoh has quit IRC | 11:31 | |
openstackgerrit | John Garbutt proposed openstack/nova master: POC: add descriptions to server tags https://review.openstack.org/434842 | 11:34 |
openstackgerrit | John Garbutt proposed openstack/nova master: POC: add descriptions to server tags https://review.openstack.org/434842 | 11:35 |
openstackgerrit | Roman Podoliaka proposed openstack/nova master: db: remove unused EngineFacadeFixture https://review.openstack.org/434843 | 11:35 |
*** tbachman has joined #openstack-nova | 11:35 | |
*** nmathew has quit IRC | 11:37 | |
*** yamamoto has joined #openstack-nova | 11:38 | |
*** markus_z has joined #openstack-nova | 11:45 | |
*** databus23_ has quit IRC | 11:46 | |
*** sapcc-bot has quit IRC | 11:46 | |
*** carthaca_ has quit IRC | 11:46 | |
*** sapcc-bot1 has joined #openstack-nova | 11:46 | |
*** databus23_ has joined #openstack-nova | 11:46 | |
*** karthiks has joined #openstack-nova | 11:46 | |
*** dgonzalez_ has joined #openstack-nova | 11:46 | |
*** carthaca_ has joined #openstack-nova | 11:46 | |
*** david_1 has joined #openstack-nova | 11:46 | |
*** owalsh has quit IRC | 11:46 | |
*** sridharg has joined #openstack-nova | 11:46 | |
*** yamamoto has quit IRC | 11:47 | |
*** esberglu has joined #openstack-nova | 11:47 | |
*** cdent has quit IRC | 11:48 | |
*** databus23_ has quit IRC | 11:48 | |
*** dgonzalez_ has quit IRC | 11:48 | |
*** david_1 has quit IRC | 11:48 | |
*** carthaca_ has quit IRC | 11:48 | |
*** tbachman has quit IRC | 11:50 | |
*** esberglu has quit IRC | 11:52 | |
*** Hosam has quit IRC | 11:58 | |
*** dikonoor has quit IRC | 12:00 | |
*** sapcc-bot1 has quit IRC | 12:03 | |
*** sapcc-bot has joined #openstack-nova | 12:03 | |
*** david_1 has joined #openstack-nova | 12:03 | |
*** databus23_ has joined #openstack-nova | 12:03 | |
*** carthaca_ has joined #openstack-nova | 12:03 | |
*** dgonzalez_ has joined #openstack-nova | 12:03 | |
*** salv-orlando has joined #openstack-nova | 12:04 | |
*** Hosam has joined #openstack-nova | 12:04 | |
*** fandi has joined #openstack-nova | 12:04 | |
*** jpena is now known as jpena|lunch | 12:05 | |
*** david_1 has quit IRC | 12:05 | |
*** databus23_ has quit IRC | 12:05 | |
*** carthaca_ has quit IRC | 12:05 | |
*** dgonzalez_ has quit IRC | 12:05 | |
*** psachin has joined #openstack-nova | 12:06 | |
*** dave-mccowan has joined #openstack-nova | 12:06 | |
*** diga has quit IRC | 12:06 | |
*** Anticime1 is now known as Anticimex | 12:07 | |
*** salv-orl_ has quit IRC | 12:07 | |
*** fandi has quit IRC | 12:09 | |
*** fandi has joined #openstack-nova | 12:09 | |
*** ratailor has quit IRC | 12:10 | |
*** bvanhav has joined #openstack-nova | 12:12 | |
*** catintheroof has joined #openstack-nova | 12:14 | |
betherly | hi all! having devstack issues with the nova-api not starting. does anyone know why or how i can fix this? | 12:15 |
*** cdent has joined #openstack-nova | 12:16 | |
sdague | betherly: devstack questions probably better for #openstack-qa, we can do the debug over there if you like | 12:19 |
betherly | sdague: many thanks! ill ask again in that channel! | 12:20 |
*** gszasz has joined #openstack-nova | 12:22 | |
*** sudipto_ has joined #openstack-nova | 12:24 | |
*** sudipto has joined #openstack-nova | 12:24 | |
*** yamamoto has joined #openstack-nova | 12:25 | |
openstackgerrit | John Garbutt proposed openstack/nova master: POC: add descriptions to server tags https://review.openstack.org/434842 | 12:26 |
*** yuntongjin has joined #openstack-nova | 12:28 | |
*** takashin has left #openstack-nova | 12:31 | |
*** gongysh has quit IRC | 12:31 | |
*** dikonoor has joined #openstack-nova | 12:36 | |
*** sudipto has quit IRC | 12:39 | |
*** sudipto_ has quit IRC | 12:39 | |
*** smatzek has joined #openstack-nova | 12:41 | |
*** jerrygb has joined #openstack-nova | 12:42 | |
*** jerrygb has quit IRC | 12:42 | |
*** esberglu has joined #openstack-nova | 12:42 | |
*** jerrygb has joined #openstack-nova | 12:43 | |
*** thorst_ has joined #openstack-nova | 12:44 | |
*** smatzek has quit IRC | 12:45 | |
*** smatzek has joined #openstack-nova | 12:45 | |
*** edmondsw has joined #openstack-nova | 12:46 | |
*** esberglu has quit IRC | 12:47 | |
*** tbachman has joined #openstack-nova | 12:52 | |
*** mdnadeem has quit IRC | 12:56 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 12:57 |
*** diga has joined #openstack-nova | 13:00 | |
*** lucas-bbl is now known as lucasagomes | 13:01 | |
*** gcb has quit IRC | 13:01 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 13:02 |
*** jpena|lunch is now known as jpena | 13:04 | |
*** fandi has quit IRC | 13:09 | |
*** fandi has joined #openstack-nova | 13:10 | |
*** tblakes has joined #openstack-nova | 13:11 | |
*** Hosam has quit IRC | 13:11 | |
*** kevinz has joined #openstack-nova | 13:12 | |
*** lucasagomes is now known as lucas-brb | 13:13 | |
*** mtanino has joined #openstack-nova | 13:14 | |
*** gouthamr has joined #openstack-nova | 13:15 | |
*** jwcroppe has quit IRC | 13:15 | |
*** ekuris has quit IRC | 13:18 | |
*** dave-mccowan has quit IRC | 13:19 | |
*** catintheroof has quit IRC | 13:20 | |
*** diga has quit IRC | 13:20 | |
*** kevinz has quit IRC | 13:21 | |
*** kevinz has joined #openstack-nova | 13:21 | |
*** diana_clarke has joined #openstack-nova | 13:25 | |
*** iceyao has joined #openstack-nova | 13:25 | |
*** esberglu has joined #openstack-nova | 13:26 | |
*** bkopilov_ has quit IRC | 13:27 | |
*** bkopilov has quit IRC | 13:27 | |
*** jwcroppe has joined #openstack-nova | 13:29 | |
*** mriedem_afk is now known as mriedem | 13:29 | |
*** Jeffrey4l has quit IRC | 13:30 | |
mriedem | BobBall: ok, but the xen vif bridge driver could be used during a live migration with nova-net in ocata right and then that would need to work (not get that KeyError). granted that's only nova-net with cells v1 in ocata. | 13:30 |
*** Jeffrey4l has joined #openstack-nova | 13:31 | |
*** ekuris has joined #openstack-nova | 13:32 | |
mriedem | BobBall: also, if the vif driver is deprecated we should change the default for the vif_driver config option since that's the default right now - but let's do that for pike | 13:32 |
mriedem | BobBall: also the xenserver ci is failing on https://review.openstack.org/#/c/424428/ | 13:33 |
BobBall | mriedem: Theoretically, sure... but tbh I would be quite surprised if live migration was the only thing broken when using xenapi with nova-net and cells v1 - it's a specialised use case of a specialised use case. If we can get the current change merged then at least we could backport a fix if customers do identify this as an issue. | 13:33 |
mriedem | sure, but a simple in check to avoid the KeyError is pretty simple right? | 13:34 |
BobBall | Indeed. Avoiding the KeyError is trivial - I just can't promise it will work | 13:35 |
BobBall | CI was broken by an update to Cirros version today... but I thought it was resolved now. | 13:35 |
*** salv-orlando has quit IRC | 13:36 | |
*** psachin has quit IRC | 13:39 | |
*** links has quit IRC | 13:39 | |
*** ralonsoh has joined #openstack-nova | 13:40 | |
*** sudipto_ has joined #openstack-nova | 13:41 | |
*** sudipto has joined #openstack-nova | 13:41 | |
*** psachin has joined #openstack-nova | 13:41 | |
*** kevinz has quit IRC | 13:42 | |
mriedem | Details: {u'message': u"Cannot 'createImage' instance e0197e39-f124-4f50-980a-d60c0cfef623 while it is in task_state image_uploading", u'code': 409} | 13:42 |
mriedem | yeah looks unrelated | 13:42 |
*** gouthamr has quit IRC | 13:42 | |
*** kevinz has joined #openstack-nova | 13:42 | |
*** yamamoto has quit IRC | 13:43 | |
*** gouthamr has joined #openstack-nova | 13:43 | |
*** fandi has quit IRC | 13:45 | |
mriedem | as you say maybe we just handle the KeyError in a follow up patch and backport it, let me see the diff from my review | 13:45 |
*** belmoreira has joined #openstack-nova | 13:46 | |
BobBall | I've just fixed the key error (inline) can can push a patch - just running pep8 to make sure I haven't done something stupid | 13:47 |
BobBall | As we don't have a +2 from John and the CI will need a recheck anyway I'm quite happy to push this and replace the existing change? | 13:47 |
BobBall | Sorry for not spotting the CI had returned -1: The change is owned by a team member from PRC and I'm not getting CI vote reports | 13:48 |
BobBall | Happy to go with whatever route you recommend on this one ofc | 13:48 |
*** yamamoto has joined #openstack-nova | 13:49 | |
*** lucas-brb is now known as lucasagomes | 13:49 | |
*** xyang1 has joined #openstack-nova | 13:50 | |
*** mtanino has quit IRC | 13:51 | |
mriedem | BobBall: +2 on it now | 13:52 |
mriedem | i left comments but those can be dealt with in a follow up | 13:52 |
BobBall | I'll put the KeyError fix in a follow up. | 13:52 |
mriedem | johnthetubaguy: are you still happy with https://review.openstack.org/#/c/424428/ ? | 13:52 |
mriedem | BobBall: i'm good if you want to push that change up now too | 13:53 |
*** xyang1 has quit IRC | 13:53 | |
*** yamamoto has quit IRC | 13:53 | |
*** bvanhav_ has joined #openstack-nova | 13:54 | |
*** burt has joined #openstack-nova | 13:54 | |
*** bkopilov_ has joined #openstack-nova | 13:54 | |
*** bkopilov has joined #openstack-nova | 13:54 | |
BobBall | mriedem: Do you want me to change the default driver as well in this follow on change? (i.e. is the follow-on destined for Pike, or do you want to try and approve for Ocata?) | 13:54 |
*** gszasz has quit IRC | 13:54 | |
mriedem | BobBall: the default vif_driver change won't be backported | 13:54 |
*** bvanhav has quit IRC | 13:55 | |
mriedem | the keyerror thing could be for ocata, so i'd leave those separate | 13:55 |
BobBall | Will do. | 13:55 |
*** gongysh has joined #openstack-nova | 13:55 | |
mriedem | bauzas: lyarwood: i'm going to request a newton release, it contains an online data migration (not a schema migration) to fix a bug when upgrading from mitaka. i was just going to version this as 14.0.4 but wanted to ask first since last time we had a db migration there was a frackus at red hat about it not being a minor version update | 13:56 |
*** mdrabe has joined #openstack-nova | 13:56 | |
mriedem | *fracas | 13:56 |
johnthetubaguy | mriedem: I duno, I dug into that whole 'nicira-iface-id' a few times, annoyed I didn't see the whole bridge thing, that makes more sense though, make the doubt whether I am happy with it | 13:57 |
mriedem | johnthetubaguy: sounds like BobBall has a fix locally for the keyerror thing which could go into that patch now if we want | 13:57 |
BobBall | BTW - we can't continue the loop if the network doesn't have a mapping - this must be fatal; we can return the exception No mapping from source network though | 13:57 |
mriedem | doesn't matter to me, might as well if it's ready | 13:57 |
johnthetubaguy | mriedem: yeah, duno, just a .get() would fix it | 13:58 |
*** tlian has joined #openstack-nova | 13:58 | |
*** yuntongjin has quit IRC | 13:58 | |
mriedem | johnthetubaguy: that's true, it would result in the exception | 13:58 |
*** zz_dimtruck is now known as dimtruck | 13:58 | |
BobBall | I've added a .get() and then an "if neutron_id is None or neutron_id not in ..." as my fix, and a UT to make sure it raises. | 13:58 |
bauzas | p!spy | 13:58 |
johnthetubaguy | yeah | 13:58 |
mriedem | BobBall: the existing "if neutron_id not in vif_uuid_map.keys():" will handle the None case | 13:59 |
johnthetubaguy | +1 mriedem | 13:59 |
mriedem | i think the get() is all we need | 13:59 |
lyarwood | mriedem: hey sorry, yeah I think that was more because someone missed that the migration had already landed before we GA'd | 13:59 |
*** sudipto has quit IRC | 13:59 | |
*** sudipto_ has quit IRC | 13:59 | |
BobBall | Well a dictionary *could* have None in the keys... but I agree that's very unlikely. | 13:59 |
BobBall | and would be indicative of a totally different bug. | 13:59 |
mriedem | i don't think it would happen in this case because of https://review.openstack.org/#/c/424428/11/nova/virt/xenapi/vmops.py@2390 | 14:00 |
BobBall | I just like to be more verbose and be sure that we know that None is an error rather than two errors potentially resulting in a weird pass | 14:00 |
mriedem | i'm fine if you want to be cautious | 14:00 |
*** liangy has joined #openstack-nova | 14:01 | |
*** eharney has joined #openstack-nova | 14:01 | |
BobBall | So shall I post a follow-up or a replacement? | 14:01 |
lyarwood | mriedem: https://review.openstack.org/#/c/408725/ right? | 14:01 |
mriedem | lyarwood: yeah | 14:02 |
mriedem | BobBall: might as well do a replacement | 14:02 |
mriedem | new patch set on the existing change | 14:02 |
mriedem | i'll be around to shephard it into ocata | 14:02 |
bauzas | mriedem: well, not sure if it would be a .z version | 14:03 |
*** burgerk has joined #openstack-nova | 14:03 | |
BobBall | Will do. Just running another couple of tests to make sure I didn't do something really stupid. | 14:04 |
mriedem | it's not a new feature | 14:04 |
bauzas | mriedem: in case we need more than just upgrading the packages | 14:04 |
bauzas | mriedem: yeah, I know | 14:04 |
mriedem | bauzas: it's basically there to catch any upgrade issues from mitaka to newton | 14:04 |
bauzas | mriedem: but I wonder how we could signal that just upgrading is not enough | 14:04 |
mriedem | not from upgrading from 14.0.3 to 14.0.4 | 14:04 |
mriedem | because if you hit this bug on upgrade to newton you're already broken | 14:04 |
lyarwood | bauzas: we can document this in our own releasenotes tbh | 14:04 |
*** gszasz has joined #openstack-nova | 14:04 | |
bauzas | mmmm | 14:04 |
mriedem | bauzas: upgrading is enough, this is the 13.0.0->14.0.4 case | 14:05 |
mriedem | not 14.0.3->14.0.4 | 14:05 |
bauzas | I see | 14:05 |
bauzas | well | 14:05 |
lyarwood | yeah I'm fine with this being in a 14.0.4 release tbh | 14:05 |
bauzas | mriedem: then okay | 14:05 |
openstackgerrit | Thomas Bechtold proposed openstack/nova master: Log exception when hypervisor version check fails https://review.openstack.org/434927 | 14:05 |
bauzas | mriedem: if people just do 14.0.3 to .4, they don't need to call the online data migration, yup ? If so, +1 with me | 14:05 |
*** markus_z has quit IRC | 14:06 | |
*** kberger has quit IRC | 14:06 | |
bauzas | sorry, I misunderstood the impact then | 14:06 |
*** kberger has joined #openstack-nova | 14:06 | |
mriedem | that's fine, this was a confusing one | 14:08 |
*** rmart04 has joined #openstack-nova | 14:09 | |
*** sree has joined #openstack-nova | 14:09 | |
*** Jinxing has joined #openstack-nova | 14:11 | |
mriedem | BobBall: johnthetubaguy: i'm going to be away for awhile, would be good if john can be happy with the change once it's posted, then i'll handle the +W and backport stuff when i'm in the office later | 14:13 |
*** mriedem has quit IRC | 14:13 | |
BobBall | Thanks mriedem. Should be posted in ~15 minutes I guess (just waiting for stack.sh to complete then I can test) | 14:13 |
*** yingjun has joined #openstack-nova | 14:14 | |
*** baoli has joined #openstack-nova | 14:15 | |
*** kevinz has quit IRC | 14:16 | |
*** kevinz has joined #openstack-nova | 14:16 | |
*** owalsh has joined #openstack-nova | 14:18 | |
*** mlavalle has joined #openstack-nova | 14:19 | |
*** nicodemus_ has joined #openstack-nova | 14:20 | |
nicodemus_ | hello | 14:21 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 14:21 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 14:21 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 14:21 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 14:21 |
*** adrian_otto has joined #openstack-nova | 14:21 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 14:21 |
nicodemus_ | I have a question regarding https://bugs.launchpad.net/nova/+bug/1646181 | 14:21 |
openstack | Launchpad bug 1646181 in OpenStack Compute (nova) newton "NFS: Fail to boot VM out of large snapshots (30GB+)" [Medium,Fix committed] - Assigned to Kashyap Chamarthy (kashyapc) | 14:21 |
*** markus_z has joined #openstack-nova | 14:21 | |
nicodemus_ | Is it possible/makes sense to increase cpu_time to a higher value than 8, in the case or really large images (80GB+)? | 14:22 |
*** kevinz has quit IRC | 14:23 | |
*** tbachman has quit IRC | 14:23 | |
*** liverpooler has joined #openstack-nova | 14:23 | |
*** adrian_otto has quit IRC | 14:24 | |
*** gcb has joined #openstack-nova | 14:26 | |
bhagyashris | alex_xu: Hi, | 14:27 |
*** felipemonteiro_ has joined #openstack-nova | 14:27 | |
*** tbachman has joined #openstack-nova | 14:29 | |
*** sudipto_ has joined #openstack-nova | 14:29 | |
*** sudipto has joined #openstack-nova | 14:29 | |
bhagyashris | alex_xu: I want to discuss regarding the backup api. have free time? | 14:30 |
*** READ10 has joined #openstack-nova | 14:35 | |
*** sree has quit IRC | 14:35 | |
*** smatzek has quit IRC | 14:36 | |
kashyap | nicodemus_: If you have a test env. mind trying it and see? | 14:36 |
kashyap | nicodemus_: The current default isn't working for you? | 14:36 |
nicodemus_ | kashyap, it's not a test env, it's productive... but could give it a try | 14:38 |
nicodemus_ | I've applied the fix when the customer reported the issue, but the issue seems to still be present with very large images | 14:39 |
nicodemus_ | (it's a Mitaka deployment) | 14:39 |
*** gszasz has quit IRC | 14:39 | |
kashyap | nicodemus_: Yeah, in that case, I'm curious what you'll see if you increase the timeout. | 14:41 |
*** esberglu has quit IRC | 14:43 | |
*** esberglu has joined #openstack-nova | 14:44 | |
nicodemus_ | kashyap, I'll give it a try and see what happens. Though it's failing on volume-backed instances, so maybe this is not the root cause. | 14:44 |
kashyap | nicodemus_: Can you please make a comment with full details in the bug, once you complete your test? | 14:45 |
nicodemus_ | kashyap, sure | 14:45 |
*** burgerk has quit IRC | 14:45 | |
*** mkoderer has joined #openstack-nova | 14:45 | |
*** hongbin has joined #openstack-nova | 14:48 | |
*** esberglu has quit IRC | 14:48 | |
*** yamamoto has joined #openstack-nova | 14:50 | |
*** kaisers_ has joined #openstack-nova | 14:51 | |
*** dave-mccowan has joined #openstack-nova | 14:54 | |
*** cfriesen has joined #openstack-nova | 14:54 | |
*** iceyao has quit IRC | 14:56 | |
*** yamamoto has quit IRC | 14:57 | |
*** shardy has joined #openstack-nova | 14:58 | |
*** esberglu has joined #openstack-nova | 14:58 | |
*** xyang1 has joined #openstack-nova | 14:58 | |
openstackgerrit | Bob Ball proposed openstack/nova master: Fix live migrate with XenServer https://review.openstack.org/424428 | 14:59 |
*** mriedem has joined #openstack-nova | 14:59 | |
BobBall | Sorry that took so long johnthetubaguy ^^ my live migrate env got broke by upgrading devstack and the slave not being a placement-client... :) | 14:59 |
mriedem | johnthetubaguy: BobBall: wild thought, but we could take this opportunity to deprecate the xen vif_driver option in ocata | 14:59 |
*** jose-phillips has joined #openstack-nova | 15:00 | |
mriedem | BobBall: johnthetubaguy: my thinking is, deprecate in ocata, remove in pike, and pick the vif_driver based on the use_neutron boolean | 15:00 |
mriedem | we should probably at least change the default to the xen ovs vif driver in ocata because we changed the default value of use_neutron=True in ocata | 15:00 |
*** markmcclain has quit IRC | 15:00 | |
*** smatzek has joined #openstack-nova | 15:00 | |
mriedem | so the xen vif driver config default is broken (by default) | 15:00 |
*** annegentle has joined #openstack-nova | 15:00 | |
*** markmcclain has joined #openstack-nova | 15:00 | |
johnthetubaguy | I would have through bridge and ovs work with both, I guess thats not true? | 15:01 |
johnthetubaguy | thought | 15:01 |
*** jose-phillips has quit IRC | 15:01 | |
mriedem | like neutron + linuxbridge? | 15:02 |
mriedem | i'm told the xen vif bridge driver is deprecated and only for nova-net | 15:02 |
*** nicodemus_ has quit IRC | 15:03 | |
*** gszasz has joined #openstack-nova | 15:03 | |
mdbooth_ | lyarwood: I'm also just looking at that DiskFilter change, btw. The if requested > 0 seems kinda arbitrary, as presumably the real problem is that the total is wrong? | 15:04 |
*** mdbooth_ is now known as mdbooth | 15:04 | |
*** annegentle has quit IRC | 15:05 | |
mdbooth | lyarwood: So IIUC you'd be able to create a BFV instance, but not an instance with ephemeral root disk smaller than the actual available space. | 15:06 |
owalsh | mriedem: FYI https://bugs.launchpad.net/nova/+bug/1665370 - the newton & ocata api db migrations conflict | 15:06 |
openstack | Launchpad bug 1665370 in tripleo "api db_sync fails on upgrade due to duplicate migration files" [Critical,Triaged] - Assigned to Steven Hardy (shardy) | 15:06 |
mdbooth | In either case you should be able to create, but the latter case would still be prevented. | 15:06 |
mdbooth | lyarwood: Unless I missed something, which is possible. Still looking. | 15:06 |
*** annegentle has joined #openstack-nova | 15:06 | |
*** markus_z has quit IRC | 15:06 | |
mriedem | owalsh: guh, pretty soon i'm in meetings for the next 2 hours | 15:07 |
*** mtanino has joined #openstack-nova | 15:07 | |
mriedem | dansmith: melwitt: can you take a look at ^? | 15:07 |
mdbooth | The code which populates this from the driver gets it directly from the storage, so my current working assumption is an incorrect interim calculation in resourcemanager | 15:07 |
* mdbooth is looking for the code which does that | 15:07 | |
owalsh | mriedem: should be a simple fix, I can push a patch | 15:08 |
mriedem | owalsh: i'm not sure we can just start renaming db migrations | 15:08 |
dansmith | owalsh: yeah, hang on a sec | 15:08 |
dansmith | not sure how this could have happened | 15:09 |
mriedem | right, how would grenade not explode? | 15:09 |
mriedem | owalsh: it's 21 in stable/newton https://github.com/openstack/nova/tree/stable/newton/nova/db/sqlalchemy/api_migrations/migrate_repo/versions | 15:09 |
mriedem | and 28 in ocata https://github.com/openstack/nova/tree/stable/ocata/nova/db/sqlalchemy/api_migrations/migrate_repo/versions | 15:10 |
dansmith | mriedem: because grenade uses old/ and new/ | 15:10 |
dansmith | mriedem: it's because you didn't put it in the placeholder in master too and then backport *that* | 15:10 |
dansmith | I should have caught that | 15:11 |
*** rmart04 has quit IRC | 15:11 | |
*** ltomasbo is now known as ltomasbo|away | 15:11 | |
mriedem | oh i see | 15:11 |
mriedem | 21 in master is the issue | 15:11 |
dansmith | owalsh: so you were going to copy the migration the same place in ocata and remove that placeholder right? | 15:11 |
mriedem | ok bug is marked for ocata rc2 | 15:12 |
mriedem | i'm entering the roundtable danger zone, bbiab | 15:12 |
owalsh | dansmith: yea, assuming it's not too late to rename ocata migrations | 15:13 |
mriedem | we can rename it and backport it | 15:13 |
*** ltomasbo|away is now known as ltomasbo | 15:13 | |
dansmith | owalsh: so, tbh, I'm not sure we've ever actually done this | 15:13 |
dansmith | mriedem: that won't really fix people in the field, actually | 15:14 |
dansmith | mriedem: if they have ocata now, and then update to this, they'll get two migrations for #21 | 15:14 |
owalsh | other option is to remove the 021 placeholder in ocata and turn 028 into a no-op | 15:14 |
dansmith | I guess if they use a proper package manager then it should remove the old one and install the new one | 15:14 |
dansmith | owalsh: I don't think that helps | 15:15 |
*** rmart04 has joined #openstack-nova | 15:15 | |
*** lucasxu has joined #openstack-nova | 15:15 | |
dansmith | owalsh: and we definitely don't want to do that in case someone was at #27 right now | 15:15 |
lyarwood | mdbooth: sorry just reading up, so iirc negative totals are valid right? BFV instances using flavors with local disk space etc? That's why I asked that the author add the examples where we can hit this into the commit message. | 15:15 |
dansmith | owalsh: so I think you have to do the rename in ocata, add a release note about it, and we will have to depend on packaging to properly clean up the two #21s | 15:15 |
lyarwood | mdbooth: ah but that's going to be > 0, I see your point | 15:16 |
mdbooth | lyarwood: That's only a working assumption, btw. Plenty of scope to be entirely wrong :) | 15:17 |
*** lucasxu has quit IRC | 15:18 | |
*** burgerk has joined #openstack-nova | 15:18 | |
*** Jinxing has quit IRC | 15:20 | |
cfriesen | what's the process for documenting a known issue in stable/newton? (It'll be affected by bug 1644248 but we're not fixing it.) | 15:20 |
openstack | bug 1644248 in OpenStack Compute (nova) ocata "Nova incorrectly tracks live migration progress" [High,In progress] https://launchpad.net/bugs/1644248 - Assigned to Matt Riedemann (mriedem) | 15:20 |
*** marst has quit IRC | 15:20 | |
*** tbachman has quit IRC | 15:21 | |
owalsh | dansmith: hmm, anyone on #27 will need #28 so it needs to remain, but it also has to tolerate the case where the migration was already performed in newton #21 | 15:22 |
dansmith | owalsh: right, it has to be in both places | 15:22 |
dansmith | owalsh: but it should be idempotent so that's fine right? otherwise a backport of it doesn't work at all | 15:23 |
owalsh | dansmith: ok, then the only Q is where packaging does the right thing here. I'll take a look | 15:23 |
owalsh | s/where/whether/ | 15:24 |
*** tbachman has joined #openstack-nova | 15:24 | |
dansmith | owalsh: yeah, so I'm pretty sure we've done this at least once in the far past, but I'm not positive.. so if that did happen, packaging must have done the right thing as I don't remember ever having to consider this rename issue before | 15:24 |
mriedem | dansmith: can't we just have the same migration in ocata at 21 and 28? | 15:28 |
mriedem | it's idempotent | 15:28 |
dansmith | mriedem: yes | 15:28 |
mriedem | ok | 15:28 |
*** sridharg has quit IRC | 15:29 | |
dansmith | mriedem: but anyone with ocata installed would get two different #21s if the packaging didn't remove the 21_placeholder and replace with with 21_do_stuff | 15:29 |
dansmith | that's he concern | 15:29 |
mriedem | sure | 15:29 |
mriedem | fwiw i don't expect much of that to be the case right now, but i know it's a concern | 15:29 |
dansmith | sure, but have to account for it | 15:30 |
*** yingjun has quit IRC | 15:30 | |
*** marst has joined #openstack-nova | 15:30 | |
mriedem | so grenade didn't fail b/c it installed newton up to 21, and then when syncing ocata on the new side it saw it was already past 21 so didn't care? | 15:30 |
dansmith | no | 15:31 |
dansmith | I think grenade didn't catch it because it installs a new nova into /opt/stack/new and doesn't have the migrate_repo overlay with the old one | 15:31 |
dansmith | even if it did, it would use git as its package manager and it would remove the old one during an update I think | 15:31 |
mriedem | gah | 15:32 |
*** sree has joined #openstack-nova | 15:32 | |
mriedem | should fix that gap at some point but i don't have an easy solution off the top of my head | 15:32 |
*** sudipto has quit IRC | 15:32 | |
*** sudipto_ has quit IRC | 15:32 | |
dansmith | yeah, it's actually weird that the same thing didn't happen with their packages already, actually | 15:32 |
*** markus_z has joined #openstack-nova | 15:33 | |
sdague | dansmith: did we ever actually have a backported migration? | 15:33 |
sdague | I know the facility has existed for a while, but it's not regularly used | 15:33 |
dansmith | sdague: well, that's what I was trying to think ... I feel like we must've had one at least, but I'm not positive | 15:33 |
dansmith | sdague: yeah, very infrequently if ever | 15:33 |
sdague | so, I'm trying to understand the bug - https://bugs.launchpad.net/nova/+bug/1665370 | 15:34 |
openstack | Launchpad bug 1665370 in tripleo "api db_sync fails on upgrade due to duplicate migration files" [Critical,Triaged] - Assigned to Steven Hardy (shardy) | 15:34 |
dansmith | sdague: if you use python setup.py install, | 15:34 |
dansmith | sdague: you'll end up with two 21_ migrations, you get that right? | 15:34 |
*** lucasxu has joined #openstack-nova | 15:34 | |
mdbooth | bauzas: You around? | 15:35 |
sdague | dansmith: i'm not sure I do | 15:35 |
sdague | because won't the new rpm file remove the old one? | 15:35 |
mriedem | sdague: different names | 15:36 |
shardy | hey folks, thanks for looking into this - I'm around if you need any more info from my environment | 15:36 |
shardy | and/or to test any fixes | 15:36 |
mriedem | 021_placeholder in ocata, 021_build_something_fix in newton | 15:36 |
dansmith | sdague: not rpm, I said setup.py | 15:36 |
sdague | dansmith: ok, how do you get the filesystem | 15:36 |
sdague | maybe that's where I'm trying to understand | 15:36 |
dansmith | sdague: the confusing thing is that I think the rpm should be removing the old one even in their case | 15:36 |
sdague | I don't understand how you create a filesystem where they both are there | 15:37 |
dansmith | sdague: I'm just saying, take a system, setup.py install newton, then ocata, you'll end up with two 21_ files because you don't have a package manager in the mix | 15:37 |
mdbooth | bauzas: I'm trying to join some dots and I'm missing something. When ComputeManager calls update_instance_info(instance), I can see that the scheduler squirrels that instance away in host_state. However, I can't see anywhere that the information is actually used. I'm assuming that something somewhere uses it to keep resource usage up to date in between complete syncs, but I'm being blind. | 15:37 |
mriedem | we don't pip install nova during grenade? | 15:37 |
mriedem | i.e. pip install . | 15:37 |
sdague | mriedem: we do | 15:37 |
dansmith | mriedem: we "develop" install it don't we? | 15:37 |
dansmith | which is an egg link | 15:37 |
mriedem | to put the files in /var/lib/python27/site-packages/nova/.. | 15:38 |
sdague | mriedem: oh, we do do dev install | 15:38 |
bauzas | mriedem: I think we pip develop | 15:38 |
dansmith | yeah so dev install won't catch this | 15:38 |
sdague | that might be isolating | 15:38 |
dansmith | right | 15:38 |
mriedem | shart | 15:38 |
dansmith | the thing is, | 15:38 |
bauzas | devstack too | 15:38 |
dansmith | they install from rpms, | 15:38 |
bauzas | mdbooth: not sure I get your problem | 15:38 |
dansmith | so they should be removing the old 21 and installing the new one, AFAIK | 15:38 |
sdague | dansmith: right | 15:39 |
sdague | dansmith: the thing I could imagine is stale pyc files | 15:39 |
dansmith | sdague: yeah, could be that | 15:39 |
dansmith | shardy: owalsh ^ | 15:39 |
owalsh | dansmith: well, rpm should be clearing out .pyc/pyo files too | 15:39 |
mdbooth | bauzas: I'm assuming that when ComputeManager calls update_instance_info(instance), it somehow updates the host_state data which is ultimately used by DiskFilter | 15:39 |
bauzas | mdbooth: oh gotcha | 15:39 |
mdbooth | bauzas: Is that correct? | 15:39 |
BobBall | mriedem johnthetubaguy: I'd be quite happy to deprecate the vif driver config option - BUT - is it used by Rackspace with Quark? | 15:39 |
dansmith | owalsh: so maybe before we go tweaking stuff in tree we should try to figure out why the cleaning isn't happening? | 15:40 |
bauzas | mdbooth: yeah, some filters | 15:40 |
owalsh | dansmith: grabbing the bits to create a local rpm now... | 15:40 |
mdbooth | I'm just trying to follow the whole code path, because I suspect there's a bug somewhere in it | 15:40 |
shardy | dansmith, owalsh: actually this could be a packaging issue: | 15:40 |
shardy | http://paste.openstack.org/show/599261/ | 15:40 |
bauzas | mdbooth: not sure about the DiskFilter, but some others | 15:40 |
mdbooth | But I can't even find the code path :) | 15:40 |
bauzas | mdbooth: sec then | 15:40 |
shardy | the new python-nova doesn't seem to have obsoleted the old one | 15:40 |
sdague | dansmith: yeh, because I can't imagine there is another way to fix this, those placement backports are important | 15:40 |
BobBall | mriedem: johnthetubaguy: Also very happy to change the default to the OVS driver | 15:40 |
dansmith | shardy: maybe building the rpm again in an unclean space or something? | 15:40 |
sdague | I guess the only thing you could do is not change 21_ to be the new name | 15:40 |
sdague | it just stays placehold | 15:40 |
owalsh | shardy: you mean both ocata & newton are installed? | 15:41 |
bauzas | mdbooth: found one at least | 15:41 |
dansmith | sdague: yeah, although I think when we made up this procedure we were supposed to put it in both places in master, which didn't happen here, but agree, it's probably not the root cause | 15:41 |
shardy | dansmith: not sure, these are dlrn packages, I can try locally rebuilding with dlrn to check tho | 15:41 |
shardy | owalsh: rpm -qa seems to indicate that, which is a surprise | 15:41 |
sdague | dansmith: ok, I didn't realize the double master policy | 15:41 |
bauzas | mdbooth: eg. https://github.com/openstack/nova/blob/master/nova/scheduler/filters/affinity_filter.py#L89 | 15:42 |
dansmith | sdague: we never do this though, so not surprising we didn't do it right | 15:42 |
sdague | shardy: yeh, that looks like something unclean about the build | 15:42 |
sdague | dansmith: yeh, agreed | 15:42 |
shardy | sdague: yeah, I'll go and take a closer look at that, sorry for not spotting it sooner | 15:42 |
sdague | black swans are what they are | 15:42 |
mdbooth | bauzas: Ah... I was expecting it to use the instance info to update usage totals | 15:42 |
shardy | I'll leave the bug open for now until I figure it out | 15:42 |
mdbooth | Does it not do that? | 15:42 |
*** gongysh has quit IRC | 15:43 | |
mdbooth | Also, instances is accessed there as a property, so I may have been grepping the wrong thing | 15:43 |
mriedem | BobBall: i'd like johnthetubaguy's opinion on that too | 15:44 |
*** fragatina has joined #openstack-nova | 15:44 | |
mriedem | but it seems the default vif driver should be the ovs one if use_neutron=True by default in ocata | 15:44 |
*** faizy has joined #openstack-nova | 15:45 | |
*** fragatina has quit IRC | 15:45 | |
*** fragatina has joined #openstack-nova | 15:46 | |
*** jsho has joined #openstack-nova | 15:47 | |
mriedem | sdague: is there a reason we can't pip install rather than pip develop in a grenade run? | 15:48 |
mdbooth | bauzas: The code in question is here: https://review.openstack.org/#/c/430091/4/nova/scheduler/filters/disk_filter.py | 15:48 |
mdbooth | It's examining host_state.total_usable_disk_gb | 15:48 |
dansmith | mriedem: so I'm not sure the dev install is all that unusual, | 15:48 |
*** jsho has quit IRC | 15:48 | |
dansmith | mriedem: because the people I know that deploy from git do so in venvs, which is kinda how this behaves | 15:48 |
mriedem | yeah i thought about venvs | 15:49 |
*** jsho has joined #openstack-nova | 15:49 | |
mriedem | that's what bluebox does | 15:49 |
mdbooth | AFAICT that value is only ever set by a periodic task, and not if ComputeManager is sending updates? | 15:49 |
sdague | mriedem: the whole reason for the dev install is so that you can actually hack on and fix things | 15:49 |
mdbooth | bauzas: TBH I'm pretty confused | 15:49 |
sdague | it's kind of the purpose of devstack | 15:49 |
mriedem | sdague: yeah but we wouldn't do that in a CI run | 15:49 |
dansmith | mriedem: then it's different from local | 15:49 |
mriedem | seems d-g could tell us to actually install | 15:49 |
sdague | mriedem: sure, but then you end up with another difference between CI and local | 15:49 |
sdague | and people get all mad why reproduce is not a thing | 15:50 |
dansmith | yeah, which isn't great, especially if what we have is testing a legit mode | 15:50 |
dansmith | which I think it is because of venvs | 15:50 |
mriedem | fine fine | 15:50 |
mriedem | just spit balling | 15:50 |
sdague | mriedem: yeh, sure | 15:50 |
mriedem | i'll go back to trashing things in this roundtable | 15:50 |
bauzas | mdbooth: wait, which value ? | 15:50 |
bauzas | mdbooth: you're showing me something not related to the instance | 15:51 |
sdague | this is just one of those places where we have to remember that testing is a model of production, and as a model, we've made certain approximations | 15:51 |
mdbooth | bauzas: Right. I was trying to find an association between those 2 things, and I can't find one. | 15:51 |
mriedem | at the least i guess we need something doc'ed for this since it was missed | 15:51 |
mriedem | so in 2 years from now when we do another one of these we can consult the guide | 15:51 |
mdbooth | I was assuming it exists, because otherwise I don't see how that value is updated exception by a heavyweight periodic task. | 15:51 |
bauzas | mdbooth: so, the HostState is generated/updated at every request in the HostManager | 15:52 |
mdbooth | However, that's just assumption on my part. | 15:52 |
*** unicell1 has joined #openstack-nova | 15:52 | |
bauzas | and then we pass it to the filter for each compute | 15:52 |
*** gcb has quit IRC | 15:53 | |
*** unicell has quit IRC | 15:54 | |
mdbooth | bauzas: We're not calling back to the compute host for each request though, right? | 15:54 |
mdbooth | bauzas: The model is occasionally we get real values by inspecting the compute host, and in between we do a best effort ? | 15:54 |
jaypipes | I see that mordred has stepped in a pile of mascot poo this morning. | 15:55 |
* mriedem hasn't forwarded the nova mascot yet | 15:56 | |
*** jwcroppe has quit IRC | 15:57 | |
bauzas | mdbooth: I need to go find my daughter | 15:58 |
bauzas | mdbooth: but let's discuss in 1 hour if I'm there | 15:58 |
*** prateek has quit IRC | 15:59 | |
*** jwcroppe has joined #openstack-nova | 15:59 | |
*** salv-orlando has joined #openstack-nova | 16:00 | |
mriedem | jaypipes: link me | 16:02 |
*** crushil has joined #openstack-nova | 16:02 | |
*** ekuris has quit IRC | 16:03 | |
*** mvk has quit IRC | 16:03 | |
*** rcernin has quit IRC | 16:03 | |
*** moshele has quit IRC | 16:03 | |
*** xyang1 has quit IRC | 16:04 | |
frickler | BobBall: could you take a look at https://review.openstack.org/434666 please? second (different) error after xenserver recheck, both look unrelated to my patch | 16:06 |
*** mriedem has quit IRC | 16:07 | |
*** unicell1 has quit IRC | 16:07 | |
*** annegentle has quit IRC | 16:07 | |
*** mriedem has joined #openstack-nova | 16:08 | |
BobBall | frickler: We believe the ssh timeout issue is one we have a fix for (found the probable fix yesterday) - but we're currently doing more testing on it before merging to the CI. I've not seen that subnet failure before though. | 16:08 |
BobBall | There was a cirros change which made the CI very sad this morning, so I don't have enough statistics after that was fixed to say whether there is another wider issue or bad luck. | 16:09 |
BobBall | frickler: Hit recheck again - we are monitoring things and when we have a few hours more worth of data hopefully we can say whether there is a new issue or not | 16:09 |
*** des_consolado has joined #openstack-nova | 16:10 | |
*** pcaruana has quit IRC | 16:11 | |
*** xyang1 has joined #openstack-nova | 16:11 | |
*** derekh has quit IRC | 16:12 | |
*** tesseract has quit IRC | 16:14 | |
*** faizy has quit IRC | 16:15 | |
*** derekh has joined #openstack-nova | 16:15 | |
mordred | jaypipes: could soemeone please revoke my email access? | 16:16 |
mriedem | bauzas: https://review.openstack.org/#/c/434552/ | 16:16 |
mriedem | for rc2 | 16:16 |
mdbooth | mordred jaypipes: Did I miss the juicy bits? Seemed civil to me. | 16:17 |
mriedem | nova got lumped in there i saw | 16:18 |
mriedem | nova didn't fight the mascot thing that much, we just said we didn't care (or i didn't care and thought it was silly) | 16:19 |
*** ssurana has joined #openstack-nova | 16:19 | |
mriedem | but we don't have a homegrown mascot | 16:19 |
*** dimtruck is now known as zz_dimtruck | 16:19 | |
*** tyrola has quit IRC | 16:20 | |
*** annegentle has joined #openstack-nova | 16:21 | |
*** Oku_OS is now known as Oku_OS-away | 16:21 | |
*** psachin has quit IRC | 16:21 | |
*** tyrola has joined #openstack-nova | 16:21 | |
*** Apoorva has joined #openstack-nova | 16:22 | |
*** ssurana has quit IRC | 16:23 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Make CellDatabases fixture work over RPC https://review.openstack.org/434563 | 16:26 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make start and stop admin actions cells-aware https://review.openstack.org/433362 | 16:26 |
*** yamahata has joined #openstack-nova | 16:26 | |
*** ssurana has joined #openstack-nova | 16:27 | |
*** kaisers_ has quit IRC | 16:30 | |
*** yamahata has quit IRC | 16:31 | |
mriedem | sdague: johnthetubaguy: can one of you put this through? https://review.openstack.org/#/c/434552/ | 16:32 |
mriedem | needed for rc2 | 16:32 |
mriedem | dansmith: did you want to get this into docs for rc2 or don't care? https://review.openstack.org/#/c/432482/ | 16:33 |
frickler | BobBall: will do, thanks for checking | 16:33 |
dansmith | mriedem: I was a little worried with that one that it might be implying that you could upgrade from cellsv1 to cellsv2, but I never articulated that | 16:34 |
dansmith | mriedem: but yeah, I'm fine with it going into ocata if you want | 16:34 |
sdague | mriedem: bam! | 16:34 |
mriedem | sweet | 16:35 |
*** jwcroppe has quit IRC | 16:35 | |
sdague | mriedem: so I need to tweak that spec for keystone validation of projects, as the 403 case wasn't really specified well enough | 16:35 |
mriedem | so anyone else want to help sway me either way on whether or not we should quickly change the default on the xen vif_driver config option to be the ovs driver since neutron is the default now in ocata? | 16:35 |
mriedem | otherwise the xen vif driver is not correct by default out of the box in a fresh ocata install for xen | 16:35 |
mriedem | sdague: ok | 16:35 |
BobBall | I'm perfectly happy for the default to change - I wasn't sure if we could deprecate it (not sure if rax are using for quark) | 16:36 |
mriedem | BobBall: johnthetubaguy: i'm +2 on https://review.openstack.org/#/c/424428/ | 16:37 |
mriedem | any changes to the default vif_driver config can be a separate change | 16:37 |
mriedem | BobBall: do you know if rax is even using an in-tree vif driver for xen? | 16:37 |
dansmith | mriedem: I thought the logo was going to have some hands on it | 16:37 |
mriedem | i noticed the vif_driver option is a classloading option so you can load out of tree vif drivers for xen | 16:37 |
*** zz_dimtruck is now known as dimtruck | 16:37 | |
mriedem | dansmith: as in the gaping butthole? | 16:37 |
mriedem | that's only on twitter | 16:37 |
dansmith | mriedem: yeah, I must have missed where that was downvoted | 16:37 |
dansmith | ah, maybe so | 16:37 |
*** sree has quit IRC | 16:38 | |
BobBall | mriedem: I know RAX use an out-of-tree VIF driver but I'm not certain if they use vif_driver or have custom patches | 16:38 |
mriedem | ok, because i'd like to close the out of tree vif driver support in here via the vif_driver option | 16:38 |
mriedem | and just make the vif driver decision based on use_neutron | 16:38 |
mriedem | we've been closing that window on lots of other configs | 16:39 |
mriedem | that would actually allow us to kill the vif_driver option and not use it | 16:39 |
mriedem | BobBall: you're sure that bridge driver won't work with neutron? | 16:39 |
mriedem | i won't hold rc2 up for it, anyone using xen i'm sure should already know about this | 16:40 |
BobBall | mriedem: No. But I'm sure we don't test it with Neutron and I would be quite shocked (or should I say pleasantly surprised?) if it does work. I always considered it to be nova-net only. | 16:40 |
mriedem | well we know it won't work with live migration and xenserver 7.0 i guess? | 16:40 |
BobBall | Right. | 16:41 |
mriedem | i'd kind of like to start the deprecation timer on that option in ocata if possible | 16:41 |
mriedem | so we don't lose 6 months | 16:42 |
mriedem | BobBall: who from rax might know what they use? | 16:42 |
BobBall | Totally agreed - what should we do? Log message if anyone uses the function? Update help on the vif_driver option? | 16:42 |
*** annegentle has quit IRC | 16:42 | |
BobBall | johnthetubaguy: will know I'm sure - he's just been AWOL for a few hours ;) | 16:42 |
mriedem | let me post my idea | 16:42 |
johnthetubaguy | he is here | 16:43 |
johnthetubaguy | rackspace is neutron, with the quark plugin | 16:43 |
BobBall | Does you set vif_driver to the quark plugin? | 16:43 |
johnthetubaguy | uses ovs | 16:43 |
BobBall | or can we deprecate the vif_driver option in nova? | 16:43 |
openstackgerrit | Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228 | 16:44 |
*** annegentle has joined #openstack-nova | 16:44 | |
*** andreas_s has quit IRC | 16:44 | |
johnthetubaguy | BobBall: seems fine to me | 16:44 |
mriedem | heh the config option says this: | 16:45 |
mriedem | "NOTE: The XenAPIBridgeDriver should be used for running OVS or Bridge in XenServer." | 16:45 |
mriedem | i guess that's totally wrong? | 16:45 |
BobBall | heh... Indeed. | 16:45 |
*** belmoreira has quit IRC | 16:46 | |
BobBall | frickler: There was a new tempest test added which made use of floating IPs. I've excluded it now. Please re-check. | 16:47 |
*** felipemonteiro_ has quit IRC | 16:49 | |
BobBall | mriedem: sorry for the sadness in the XenServer CI - as mentioned to frickler above, we have identified the probable cause for the ssh timeouts (we had two neutron agents with the same hostname - which meant sometimes the wrong agent was picked for something or other...). A fix was made today, but further investigation is being done to prove it doesn't break secure groups (there was some suggestion yesterday it might, but today we believe it won't). | 16:50 |
mriedem | BobBall: that's fine, i know it's not failing on this patch | 16:50 |
BobBall | Indeed. There are many false negatives today - but rechecks should fix them now. Hopefully we'll have the proper fix in the CI early next week and we will re-enable these tests. | 16:51 |
openstackgerrit | Sean Dague proposed openstack/nova master: WIP: verify project_id when quotas are checked https://review.openstack.org/435010 | 16:53 |
sdague | mriedem: that kind of works, no tests, I'll update the spec with the differences | 16:54 |
*** moshele has joined #openstack-nova | 16:54 | |
*** mdrabe has quit IRC | 16:55 | |
*** markus_z has quit IRC | 16:56 | |
mriedem | BobBall: changing the default vif driver causes all sorts of unit test failure of course | 16:57 |
*** jwcroppe has joined #openstack-nova | 16:58 | |
BobBall | mriedem: would it be enough to add help text in the vif driver, a release note and a log message when loading the deprecated one stating that it is deprecated? Then I can change the default, fix the UT and remove the old one in Pike? | 16:59 |
mriedem | yeah i think i'm going to leave the default and mark the option deprecated for pike - we could also just hack the tests to use the bridge driver to pass for now | 16:59 |
mriedem | i'm trying to find a spot to do that | 16:59 |
openstackgerrit | Sean Dague proposed openstack/nova-specs master: Updates for validation spec based on implementation https://review.openstack.org/435016 | 16:59 |
sdague | mriedem: ^^^ | 17:00 |
*** mdrabe has joined #openstack-nova | 17:01 | |
*** pbandark has joined #openstack-nova | 17:01 | |
*** baoli has quit IRC | 17:01 | |
*** derekh has quit IRC | 17:01 | |
*** liusheng has quit IRC | 17:02 | |
cfriesen | for nova microversion 2.42 is there going to be any work required in novaclient? | 17:03 |
mriedem | cfriesen: it's a bug fix, andreykurilin had a patch | 17:04 |
mriedem | https://review.openstack.org/#/c/424745/ | 17:04 |
*** derekh has joined #openstack-nova | 17:05 | |
*** thorst_ is now known as thorst_afk | 17:05 | |
andreykurilin | mriedem: also there is a patch for 2.42 - https://review.openstack.org/#/c/429512/ . I suppose we need to merge 424745 change with 429512 | 17:06 |
mdbooth | bauzas: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L1049-L1053 | 17:06 |
mdbooth | bauzas: That's the info that eventually ends up in the scheduler, right? | 17:06 |
mriedem | BobBall: got a workaround for the tests, pushing the change up shortly | 17:06 |
BobBall | Ah awesome - I was just looking into that. | 17:07 |
mdbooth | If so, I think the bug is that it should be looking at bdms, otherwise it doesn't know the actual sizes (or existence) of the root and ephemeral disks | 17:07 |
mdbooth | Also, it takes no account of swap | 17:07 |
mdbooth | Or kernel/ramdisk | 17:07 |
*** rmart04 has quit IRC | 17:07 | |
mdbooth | bauzas: Can you sanity check^^^? | 17:07 |
*** bvanhav_ has quit IRC | 17:08 | |
*** moshele has quit IRC | 17:08 | |
*** moshele has joined #openstack-nova | 17:08 | |
cfriesen | mriedem: andreykurilin: thanks, was hoping someone had done it since I'm proposing a change for 2.43. :) | 17:09 |
*** dimtruck is now known as zz_dimtruck | 17:09 | |
andreykurilin | cfriesen: unfortunately, I'll not have an ability to finish my change till March :( | 17:10 |
*** baoli has joined #openstack-nova | 17:10 | |
cfriesen | andreykurilin: okay. I see your change handles the fact that tags don't work for certain versions...what about handling the fact that they should work again as of 2.42? | 17:11 |
andreykurilin | cfriesen: it should be handled here - https://review.openstack.org/#/c/429512/ | 17:12 |
*** bvanhav_ has joined #openstack-nova | 17:12 | |
cfriesen | andreykurilin: right, so now you accept 2.42. but doesn't the change at https://review.openstack.org/#/c/424745 mean that we won't use it? | 17:15 |
*** lpetrut has quit IRC | 17:15 | |
cfriesen | won't use tags, I mean, even though they're supported on the server side | 17:15 |
andreykurilin | cfriesen: change 424745 should clarify situation for microversions <2.42 . The change 429512 should return support of tags with 2.42 | 17:16 |
cfriesen | 429512 would work standalone because it's assuming "there are no version-wrapped shell method changes for this". However, change 424745 introduces a version-wrapped change. | 17:18 |
*** jwcroppe has quit IRC | 17:18 | |
*** jwcroppe has joined #openstack-nova | 17:18 | |
cfriesen | so if one goes in first the other one has to be reworked to account for it otherwise tags won't work for version 2.42. (I think, anyway. I could be wrong.) | 17:18 |
*** jwcroppe_ has joined #openstack-nova | 17:20 | |
*** jwcroppe has quit IRC | 17:23 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate xenserver.vif_driver config option and change default https://review.openstack.org/435037 | 17:24 |
mriedem | BobBall: johnthetubaguy: ^ | 17:24 |
mriedem | dansmith: sdague: ^ let me know if you think that's wacky | 17:24 |
mriedem | but i kind of want to get that into ocata to set the timer | 17:24 |
*** moshele has quit IRC | 17:24 | |
sdague | mriedem: I'd be fine with it. I think that given the lower level of xenserver usage, things like that can move a bit more nimbly | 17:27 |
dansmith | mriedem: why is it whacky? | 17:27 |
cfriesen | andreykurilin: mriedem: would it make sense to push https://review.openstack.org/#/c/429512 for Ocata so that the client/server match up on their max version support? Then we can go back and fix up the intermediate versions where tags were broken. | 17:27 |
dansmith | er, wacky.. heh | 17:27 |
mriedem | dansmith: just b/c i want to get it into ocata | 17:27 |
mriedem | not really wacky | 17:27 |
mriedem | cfriesen: huh? | 17:28 |
dansmith | seems decidedly unwacky to me | 17:28 |
mriedem | cfriesen: novaclient is frozen for ocata | 17:28 |
mriedem | except for backporting fixes | 17:28 |
*** Apoorva_ has joined #openstack-nova | 17:28 | |
*** derekh has quit IRC | 17:28 | |
sdague | BobBall: mriedem: why is this failing xenserver ci? - https://review.openstack.org/#/c/424428/12 | 17:28 |
cfriesen | mriedem: so we have nova with max version 2.42 but novaclient with 2.41? | 17:29 |
mriedem | sdague: everything is | 17:29 |
cfriesen | mriedem: and no way to make use of the fact that 2.42 fixes tags? | 17:29 |
mriedem | known issue | 17:29 |
mriedem | cfriesen: novaclient will work with 2.42 | 17:29 |
*** Apoorva has quit IRC | 17:29 | |
mriedem | oh i guess we didn't set the max api thing in novaclient | 17:30 |
cfriesen | API_MAX_VERSION = api_versions.APIVersion("2.41") | 17:30 |
cfriesen | that's what https://review.openstack.org/#/c/429512 does | 17:30 |
mriedem | yeah, well, we can't release novaclient for ocata right now | 17:30 |
mriedem | cfriesen: i think we'll just need to backport that as a bug fix later | 17:30 |
mriedem | to stable/ocata | 17:30 |
sdague | mriedem: ok, so are we waiting for it to work? | 17:31 |
sdague | or is it safe to push it in | 17:31 |
mriedem | sdague: it's safe | 17:31 |
mriedem | the xenserver ci uses the ovs vif driver already | 17:32 |
mriedem | i checked that earlier | 17:32 |
mriedem | b/c the xenserver ci on master is neutron, and has to use the ovs driver | 17:32 |
sdague | mriedem: eta on xenserver ci coming back to life? | 17:34 |
mriedem | sdague: (10:50:14 AM) BobBall: mriedem: sorry for the sadness in the XenServer CI - as mentioned to frickler above, we have identified the probable cause for the ssh timeouts (we had two neutron agents with the same hostname - which meant sometimes the wrong agent was picked for something or other...). A fix was made today, but further investigation is being done to prove it doesn't break secure groups (there was some suggestion yesterd | 17:34 |
mriedem | (10:52:02 AM) BobBall: Indeed. There are many false negatives today - but rechecks should fix them now. Hopefully we'll have the proper fix in the CI early next week and we will re-enable these tests. | 17:34 |
openstackgerrit | Roman Podoliaka proposed openstack/nova-specs master: Strict isolation of group of hosts for image and flavor https://review.openstack.org/381912 | 17:35 |
*** jamesdenton has joined #openstack-nova | 17:35 | |
mriedem | sdague: so maybe results within the hour or two? | 17:36 |
mriedem | sdague: keeping in mind i need to backport this to ocata and get it merged there before we can cut rc2 today | 17:36 |
mriedem | sdague: oh you got the bottom change too, thanks | 17:37 |
mriedem | fwiw john was +2 on that yesterday | 17:37 |
mriedem | i just found some corner cases that needed updating, so the bottom change is safe | 17:38 |
sdague | mriedem: yeh | 17:38 |
*** shardy is now known as shardy_afk | 17:40 | |
*** jamesdenton has quit IRC | 17:40 | |
*** karthiks has quit IRC | 17:40 | |
*** owalsh has quit IRC | 17:41 | |
*** jschluet is now known as jschlueter | 17:45 | |
*** owalsh has joined #openstack-nova | 17:48 | |
*** rmart04 has joined #openstack-nova | 17:50 | |
*** zz_dimtruck is now known as dimtruck | 17:50 | |
mriedem | backports are up for those xen patches | 17:51 |
mriedem | going to lunch | 17:51 |
openstackgerrit | Ben Nemec proposed openstack/nova-specs master: Configurable PXE Boot for Instances https://review.openstack.org/435052 | 17:51 |
*** jpena is now known as jpena|away | 17:52 | |
*** mriedem1 has joined #openstack-nova | 17:53 | |
*** lucasxu has quit IRC | 17:53 | |
*** cdent has quit IRC | 17:54 | |
*** tyrola has quit IRC | 17:54 | |
*** lucasxu has joined #openstack-nova | 17:55 | |
*** mriedem has quit IRC | 17:55 | |
*** tyrola has joined #openstack-nova | 17:55 | |
*** chyka has joined #openstack-nova | 17:57 | |
*** sudipto_ has joined #openstack-nova | 17:59 | |
*** sudipto has joined #openstack-nova | 17:59 | |
*** Apoorva_ has quit IRC | 17:59 | |
*** felipemonteiro_ has joined #openstack-nova | 17:59 | |
*** lpetrut has joined #openstack-nova | 18:00 | |
*** mvk has joined #openstack-nova | 18:02 | |
*** armax has joined #openstack-nova | 18:02 | |
*** lpetrut has quit IRC | 18:03 | |
*** salv-orl_ has joined #openstack-nova | 18:04 | |
*** salv-orlando has quit IRC | 18:07 | |
openstackgerrit | Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298 | 18:12 |
openstackgerrit | MarkMielke proposed openstack/nova master: DiskFilter should only apply when storage is requested https://review.openstack.org/430091 | 18:12 |
*** lucasagomes is now known as lucas-afk | 18:14 | |
*** rmart04 has quit IRC | 18:14 | |
*** rmart04 has joined #openstack-nova | 18:15 | |
*** jamesdenton has joined #openstack-nova | 18:16 | |
*** lpetrut has joined #openstack-nova | 18:17 | |
*** sambetts is now known as sambetts|afk | 18:18 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: get inventory shared with provider https://review.openstack.org/433978 | 18:18 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: rename can_host to shared https://review.openstack.org/433977 | 18:18 |
*** slaweq has quit IRC | 18:19 | |
*** jamesdenton has quit IRC | 18:21 | |
*** slaweq has joined #openstack-nova | 18:21 | |
jaypipes | mriedem1: on https://bugs.launchpad.net/nova/+bug/1581230, walt asks "Nova shouldn't care about the internal state management of Cinder Volumes. What happens if Cinder adds a new state that Nova doesn't know about?" | 18:25 |
openstack | Launchpad bug 1581230 in OpenStack Compute (nova) "Nova should not inspect cinder volume state during attach" [Medium,In progress] - Assigned to Ildiko Vancsa (ildiko-vancsa) | 18:25 |
*** mriedem1 is now known as mriedem | 18:25 | |
jaypipes | mriedem1: I guess I'm wondering why internal state is being exposed out of Cinder's API, then? | 18:25 |
mriedem | jaypipes: well it's true that nova doesn't need or want to check the attached state of a volume if we're just going to reserve it, because the reserve action will do that atomically for us on the cinder side | 18:26 |
*** Swami_ has joined #openstack-nova | 18:26 | |
mriedem | the resulting error from the nova side is the same either way | 18:26 |
jaypipes | mriedem: k. | 18:26 |
mriedem | unreserve on the cinder side is idempotent | 18:26 |
mriedem | i think all of that old attach status checking is probably from nova-volume | 18:27 |
jaypipes | mriedem: cool (as it should be) | 18:27 |
jaypipes | the idempotency, not the nova-volume :) | 18:27 |
*** unicell has joined #openstack-nova | 18:27 | |
mriedem | right | 18:28 |
jaypipes | mriedem: in any case, I'm reviewing the patch now... | 18:28 |
mriedem | cool, it's going to be discussed next week too, and i've sort of started thinking a spec for this might be nice | 18:29 |
mriedem | since it's pretty invasive about how much of the machinery it's touching | 18:29 |
mriedem | api, conductur, upgrades, bfv, etc | 18:29 |
*** satyar has joined #openstack-nova | 18:30 | |
*** kaisers_ has joined #openstack-nova | 18:31 | |
mriedem | shardy_afk: owalsh: any updates on https://bugs.launchpad.net/nova/+bug/1665370 ? | 18:31 |
openstack | Launchpad bug 1665370 in tripleo "api db_sync fails on upgrade due to duplicate migration files" [Critical,Triaged] - Assigned to Steven Hardy (shardy) | 18:31 |
*** ltomasbo is now known as ltomasbo|away | 18:31 | |
*** crushil has quit IRC | 18:31 | |
mriedem | i'm unclear on if we're making a change in nova for that | 18:31 |
dansmith | mriedem: we think not | 18:32 |
*** cdent has joined #openstack-nova | 18:32 | |
*** claudiub has quit IRC | 18:33 | |
owalsh | mrodden: I think it's a pkg issue in this case, shardy_afk is trying to reproduce it | 18:33 |
mriedem | uh oh | 18:33 |
mriedem | someone woke the mrodden troll | 18:33 |
owalsh | oop ^^ | 18:33 |
*** jaosorior has quit IRC | 18:34 | |
*** kaisers_ has quit IRC | 18:35 | |
owalsh | mriedem (too many m... nicks here), dansmith do we need to worry about it for pip or setup.py? | 18:36 |
*** Swami has joined #openstack-nova | 18:36 | |
dansmith | owalsh: pip will do the right thing I think | 18:37 |
dansmith | owalsh: setup.py has this kind of issue in other places, so I think we're probably okay there | 18:37 |
*** Swami_ has quit IRC | 18:38 | |
*** yassine_ has quit IRC | 18:42 | |
*** bvanhav_ has quit IRC | 18:42 | |
mordred | mriedem, jaypipes: just reading a little scrollback - may or may not be relevant - but we're seeing not-infrequent flaky test issues in shades functional tests these days inside the flow "delete server ; delete volumes that were attached to server" | 18:43 |
*** Swami has quit IRC | 18:43 | |
mordred | mriedem, jaypipes: specifically because the volume throws an error that you can't delete an attached volume | 18:44 |
*** dharinic is now known as dharinic|lunch | 18:44 | |
mordred | I know that's a cinder not a nova thing - but since you're tlaking about the attach flow I figured I'd mention it | 18:44 |
mriedem | mordred: you're racing | 18:45 |
smcginnis | Yeah, that sounds like the attach hasn't finished before trying to delete. | 18:45 |
melwitt | does anyone know if there's a way to see all of the code module docs on https://docs.openstack.org/developer/nova ? | 18:45 |
smcginnis | *dettach | 18:45 |
mriedem | smcginnis: the detach | 18:45 |
mriedem | yeah | 18:45 |
smcginnis | Keyboard... | 18:45 |
mriedem | mordred: tempest flow is usually create volume, create server, attach volume to server, detach volume, delete volume, delete server, | 18:46 |
mriedem | sometihng like that | 18:46 |
sdague | melwitt: we pulled that part out of the docs a while ago because so little was actually docs | 18:46 |
sdague | docced | 18:46 |
mriedem | nova will detach the volume, but you have to wait for the volume to be 'available' before you can delete it | 18:46 |
melwitt | k, thanks sdague | 18:46 |
*** adisky_ has quit IRC | 18:49 | |
*** cdent has quit IRC | 18:50 | |
*** cdent has joined #openstack-nova | 18:50 | |
*** yassine_ has joined #openstack-nova | 18:50 | |
mordred | mriedem: yah - that's the basic flow we're doing now (I just recently added the wait-for-volume-available) - tempest is apparently also seeing some flakiness in the same general flow | 18:50 |
mordred | mriedem: jordanP is looking in to it some | 18:50 |
*** ralonsoh has quit IRC | 18:51 | |
mriedem | guess it depends on the specific low, | 18:51 |
mriedem | *flow | 18:51 |
mriedem | if you do boot from volume with delete_on_termination=True, then nova deletes the volume | 18:51 |
mriedem | when you delete the server | 18:51 |
*** Apoorva has joined #openstack-nova | 18:52 | |
*** smatzek has quit IRC | 18:52 | |
*** pbandark has quit IRC | 18:52 | |
*** jsho has quit IRC | 18:55 | |
mordred | yah. we have different tests for that - it was "boot server ; create volume ; attach volume ; delete server ; delete volume" that's the issue - but I'll keep working to figure out more places I can poll :) | 18:55 |
*** gszasz has quit IRC | 18:57 | |
*** READ10 has quit IRC | 18:58 | |
*** jsho has joined #openstack-nova | 18:58 | |
*** sudipto has quit IRC | 18:59 | |
*** sudipto_ has quit IRC | 18:59 | |
*** bvanhav_ has joined #openstack-nova | 19:00 | |
*** crushil has joined #openstack-nova | 19:03 | |
*** kaisers_ has joined #openstack-nova | 19:05 | |
*** kaisers_ has quit IRC | 19:06 | |
*** kaisers_ has joined #openstack-nova | 19:06 | |
*** ZZelle_ has joined #openstack-nova | 19:10 | |
*** jamesdenton has joined #openstack-nova | 19:10 | |
*** kaisers_ has quit IRC | 19:12 | |
dansmith | mriedem: melwitt: do you have any opinion on this? https://review.openstack.org/#/c/433362/3/nova/api/openstack/compute/servers.py@1159 | 19:12 |
dansmith | like, whether we should target from the REST API side, or modify every compute/api method to take the mapping, | 19:12 |
dansmith | or stash the mapping legally or illegally inside the instance somewhere? | 19:12 |
frickler | mriedem: not sure if you have seen this, there seems to be another nonvc issue, unrelated to py3, but in the multinode case. I'm thinking https://review.openstack.org/434996 might help but that surely needs some more eyes on it | 19:14 |
melwitt | dansmith: I think I remember we discussed this a bit before with alaski where an idea came up about stashing the targeted context in instance._context to avoid having to target everywhere | 19:15 |
dansmith | hmm | 19:16 |
*** owalsh is now known as owalsh-afk | 19:16 | |
dansmith | melwitt: the thing that kinda concerns me about that is that it becomes super obscure such that someone will come along and use that context to try to do something elsewhere and not know it's targeted | 19:16 |
melwitt | there's pros and cons to each way, so I think there wasn't like an obvious choice. I don't think I like the idea of modifying every compute/api method to take a mapping, at least | 19:16 |
dansmith | I mean, I guess that's the problem with targeting in the REST side anyway | 19:17 |
melwitt | yep. that's why I was thinking I'm wary of making it too obscure | 19:17 |
dansmith | melwitt: yeah, well, glad we agree on that, I'd like to rule that out right here and now :) | 19:17 |
dansmith | the api modification route I mean | 19:17 |
melwitt | yeah | 19:17 |
dansmith | melwitt: right now target_cell modifies the context you pass it I think, | 19:18 |
dansmith | which means we'd need to change that or make another thing to say "give me a targeted context" | 19:18 |
*** Apoorva has quit IRC | 19:19 | |
*** Apoorva has joined #openstack-nova | 19:19 | |
alaski | I can't recall the various pros/cons I worked through in my head, but I do like the idea of stashing the target somewhere. that could be the targeted context on instance, or update the context to be targeted when used to pull from a cell | 19:19 |
melwitt | yeah. if you do with...as then I think you could set it on the instance._context and let it go on its way from then on. or am I wrong on that | 19:19 |
dansmith | melwitt: you can't I don't think | 19:20 |
*** shaner has quit IRC | 19:20 | |
melwitt | because it would set it back. okay | 19:20 |
dansmith | melwitt: because returning from the with will untarget the context you set on instance | 19:20 |
dansmith | yeah | 19:20 |
dansmith | alaski: yeah, I like that because it is less work, I just worry about it being super confusing later | 19:20 |
*** shaner has joined #openstack-nova | 19:20 | |
*** shaner has quit IRC | 19:20 | |
melwitt | yeah. originally I had two methods for targeted context, one that returns one and one that is the context manager. got rid of one because at the time because there wasn't a strong case for needing it yet | 19:21 |
dansmith | the event patch needs to be more prescriptive because it has to collate multiple events, | 19:21 |
dansmith | but I can make the top patch do the magic thing and see how much has to change all at once I guess | 19:22 |
alaski | dansmith: yeah. I'm slowly remembering that I liked storing the targeted context on object._context and just treating an object as attached to the cell it came from, since nothing else should pull object._context to try to use it for other purposes | 19:22 |
dansmith | alaski: meaning it's a copy and targeted, whereas the context we pass to methods remains clean? | 19:23 |
dansmith | that is probably less obscure than how I was thinking about it | 19:23 |
alaski | yeah | 19:24 |
alaski | there's still some risk, but it seemed the least magic | 19:24 |
dansmith | melwitt: if you agree, I'll go do that thing and we can blame alaski for any fallout sometime when he's not looking | 19:24 |
melwitt | I guess the thing is, will that do the job. because the target_cell modifies the external context, not the internal one. so will things look at instance._context to switch | 19:25 |
melwitt | dansmith: yeah, I'm not too opinionated about it. alaski's thinking makes sense to me, plus we can blame him | 19:26 |
dansmith | melwitt: well, we would still target the external context if and when we need to, | 19:26 |
dansmith | melwitt: but instance.save() would just work (tm) | 19:26 |
melwitt | I'm just not sure how it's going to work out because of context vs instance._context, but yeah. yeah, true | 19:26 |
openstackgerrit | Sean Dague proposed openstack/nova master: WIP: verify project_id when quotas are checked https://review.openstack.org/435010 | 19:27 |
dansmith | melwitt: well, given the fun I just had with the test fixture, | 19:27 |
dansmith | it'll be interesting to see what doesn't fall into that realm with just the instance context set | 19:27 |
dansmith | like, looking up BDMs for an instance in the middle of an operation or something | 19:27 |
* alaski will be fine to take the blame for his bad decisions | 19:27 | |
dansmith | melwitt: like, we'll make the rpc call with an untargeted context, which means my serializer hack won't work | 19:28 |
dansmith | which will suck | 19:28 |
dansmith | a lot. | 19:28 |
dansmith | unless the rpc layer auto-targets the context before it makes the call | 19:28 |
melwitt | hm, yeah that's what I was just thinking | 19:28 |
dansmith | that makes me sad | 19:28 |
melwitt | we have the lookup things in there | 19:28 |
dansmith | right, which I thought we wanted to maybe replace by going back to targeting... | 19:29 |
dansmith | we added transport back into the context, right? | 19:29 |
dansmith | ugh, I dunno | 19:29 |
melwitt | really? I didn't know we wanted to :P | 19:29 |
melwitt | not yet. I have a patch up that does that but you crapped on it | 19:29 |
dansmith | didn't we discuss that? I thought we added it back in | 19:29 |
dansmith | hah, well | 19:30 |
dansmith | link? | 19:30 |
*** lpetrut has quit IRC | 19:30 | |
melwitt | we don't technically need it, it would just save lookups if we did | 19:30 |
melwitt | https://review.openstack.org/411543 | 19:30 |
dansmith | right, I thought that was the plan.. to not have to do that lookup again in the rpc layer and thus go with the targeting keeping things pointed in the right place | 19:30 |
dansmith | melwitt: oh please.. that's like a small dingleberry | 19:31 |
*** lpetrut has joined #openstack-nova | 19:31 | |
dansmith | I didn't even -1 | 19:31 |
melwitt | I did ^ weirdly because we have tried to not pass cell mapping around so user/pass don't get dumped out in some exception somewhere, potentially | 19:31 |
melwitt | haha, I know | 19:31 |
melwitt | okay, well, the original plan was to target everything explicitly but then we thought it would be easier if we just looked it up in there | 19:32 |
dansmith | so I know I raised it as an issue, but I wonder what kind of things might come up from just having a targeted external context all the time | 19:33 |
melwitt | another thing is belliott had said the construction of the rpc transport objects is expensive, so there was value in caching them that way instead of creating them on-the-fly each target | 19:33 |
dansmith | the biggest thing would be going back to the api database again, but that shouldn't be affected I think right? | 19:33 |
*** smatzek has joined #openstack-nova | 19:33 | |
dansmith | melwitt: we can still cache them even if we target them early | 19:33 |
melwitt | yeah. we'd have to find a place for that, maybe nova.context is not the best place is another thing belliott didn't like about it | 19:34 |
dansmith | yeah, I don't really get those concerns, but.. | 19:34 |
melwitt | dansmith: yeah going back to api database isn't affected. those can use the global api db context managers | 19:34 |
dansmith | yeah, so, maybe I'm just being overly concerned about doing the target hard and early | 19:35 |
*** prateek has joined #openstack-nova | 19:36 | |
*** shaner has joined #openstack-nova | 19:36 | |
melwitt | yeah, I don't think we've thought a lot about targeting a context and just letting it carry the db/mq connections around | 19:37 |
melwitt | it might work fine | 19:37 |
*** fragatina has quit IRC | 19:37 | |
dansmith | let me just do the early full target thing and see what explodes | 19:38 |
melwitt | sure | 19:38 |
*** fragatina has joined #openstack-nova | 19:38 | |
*** jsho has quit IRC | 19:39 | |
*** fragatina has quit IRC | 19:40 | |
*** fragatina has joined #openstack-nova | 19:40 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 19:43 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | 19:43 |
*** dharinic|lunch is now known as dharinic | 19:44 | |
mriedem | dansmith: on https://review.openstack.org/#/c/433362/4/nova/api/openstack/compute/servers.py i wonder how much of that pattern is going to be everywhere | 19:46 |
melwitt | dansmith: I'll play around with making my mq connection patch cache early in the target_cell later | 19:46 |
dansmith | mriedem: lots of places | 19:46 |
mriedem | we could put some common code into nova/api/openstack/common, but that doesn't help with all of the target_cell stuff | 19:46 |
dansmith | mriedem: well, it does if it just targets the context permanently, which is what I'm trying now | 19:47 |
mriedem | why do this in the REST API handlers rather than the compute API class methods themselves? | 19:47 |
*** lpetrut has quit IRC | 19:47 | |
*** jsho has joined #openstack-nova | 19:47 | |
*** fragatina has quit IRC | 19:48 | |
dansmith | mriedem: they have to look up the instance | 19:48 |
dansmith | mriedem: and we don't want the rest api looking up the mapping, then making compute/api do it again | 19:48 |
mriedem | oh yeah | 19:48 |
*** rmart04 has quit IRC | 19:49 | |
mriedem | dansmith: before i read scrollback i was thinking we could pass the mapping to every compute API method, like the context, and then we could put a decorator on the compute API methods to do the target cell thing for us | 19:51 |
mriedem | but i see melwitt doesn't like that | 19:51 |
dansmith | mriedem: _I_ don't like that | 19:51 |
mriedem | oh | 19:51 |
melwitt | we both don't like it | 19:51 |
dansmith | I asked so someone could say "fuck no" | 19:51 |
melwitt | heh | 19:51 |
mriedem | was that "(1:17:41 PM) dansmith: melwitt: yeah, well, glad we agree on that, I'd like to rule that out right here and now :)" ? | 19:52 |
dansmith | yeah | 19:52 |
mriedem | because you were also talking about stashing the mapping in the context or the instance at the same time | 19:52 |
*** dikonoor has quit IRC | 19:52 | |
dansmith | yeah, but I meant the param thing | 19:52 |
jaypipes | mriedem, dansmith, cdent, edleafe, johnthetubaguy, bauzas, melwitt: https://etherpad.openstack.org/p/decision-finding-shared-inventory | 19:53 |
jaypipes | mriedem, dansmith, cdent, edleafe, johnthetubaguy, bauzas, melwitt: feel free to vote and leave comments... adding that to the scheduler/placement session discussions at PTG | 19:54 |
cdent | jaypipes: thank you for writing that up. will digest properly tomorrow morning, unless I'm suddenly overcome with desire to think about it now (which is somewhat likely) | 19:54 |
mriedem | that bile yellow color hurts my eyes | 19:55 |
jaypipes | mriedem: then lucky for you you didn't see the shirt I was wearing this morning. cdent is still recovering. | 19:56 |
cdent | jaypipes: for completeness could you add a bit more info on why that particular question needs to be asked? | 19:56 |
jaypipes | cdent: yuppers. | 19:56 |
cdent | yeah, I'm going to have nightmares tonight. the combo is... | 19:56 |
jaypipes | cdent: will do. | 19:57 |
mriedem | were the nipples cut out? | 19:57 |
mriedem | because if so, sign me up | 19:57 |
jaypipes | mriedem: lol | 19:57 |
*** mkoderer has quit IRC | 19:57 | |
mriedem | dansmith: so it sounds like stashing the targeted context is the plan? | 19:57 |
dansmith | mriedem: potentially, until I find some reason to hate that more | 19:58 |
cdent | mriedem: great, thanks. I'm totally broken now. | 19:58 |
*** satyar has quit IRC | 19:59 | |
*** krtaylor has quit IRC | 19:59 | |
mriedem | i'm trying to think of an operatoin where we could have multiple instances in different cells | 20:00 |
mriedem | but i don't think that's possible right now | 20:00 |
dansmith | mriedem: it's not now for sure, | 20:00 |
dansmith | mriedem: and I was thinking of migrations of some sort, but that'll clearly require some special handling anyway | 20:00 |
mriedem | what apis take multiple instances anyway? | 20:00 |
dansmith | well, the events api, which is why I said what I did, but that one is already handles | 20:00 |
dansmith | *handled | 20:00 |
mriedem | ok | 20:01 |
*** Hosam has joined #openstack-nova | 20:01 | |
mriedem | maybe we'll revive that spec to expose quiesce in the api for multiple instances, which could all be in different cells | 20:01 |
mriedem | :) | 20:01 |
* dansmith starts stabbing the air | 20:01 | |
mriedem | haha | 20:01 |
*** liangy has quit IRC | 20:05 | |
openstackgerrit | Sean Dague proposed openstack/nova master: Verify project_id when quotas are checked https://review.openstack.org/435010 | 20:06 |
*** kaisers_ has joined #openstack-nova | 20:07 | |
*** slaweq has quit IRC | 20:08 | |
mriedem | sdague: your spec update https://review.openstack.org/#/c/435016/ looks good to me | 20:11 |
openstackgerrit | Yuming Ma proposed openstack/nova-specs master: Nova BFV by reserving volume https://review.openstack.org/417541 | 20:12 |
*** bvanhav_ is now known as bvanhav | 20:12 | |
mriedem | it does clarify the logic | 20:12 |
*** liverpooler has quit IRC | 20:12 | |
sdague | it seems to be working on the happy paths, I'll need to put some other real testing in place to test the rest of the cases | 20:13 |
*** Sukhdev has joined #openstack-nova | 20:14 | |
*** prateek has quit IRC | 20:15 | |
*** salv-orl_ has quit IRC | 20:16 | |
*** amoralej is now known as amoralej|off | 20:16 | |
openstackgerrit | Yuming Ma proposed openstack/nova-specs master: Nova BFV by reserving volume https://review.openstack.org/417541 | 20:17 |
*** alexpilotti has joined #openstack-nova | 20:18 | |
*** schlueter has joined #openstack-nova | 20:18 | |
*** lucasxu has quit IRC | 20:19 | |
*** alexpilo_ has quit IRC | 20:19 | |
*** lucasxu has joined #openstack-nova | 20:19 | |
*** salv-orlando has joined #openstack-nova | 20:23 | |
*** gouthamr has quit IRC | 20:24 | |
*** jsho has quit IRC | 20:24 | |
*** Hosam has quit IRC | 20:24 | |
*** lucasxu has quit IRC | 20:26 | |
*** lucasxu has joined #openstack-nova | 20:26 | |
*** rfolco has quit IRC | 20:27 | |
*** rfolco has joined #openstack-nova | 20:27 | |
*** salv-orlando has quit IRC | 20:28 | |
*** jwcroppe_ has quit IRC | 20:28 | |
*** rfolco has quit IRC | 20:28 | |
*** jwcroppe has joined #openstack-nova | 20:28 | |
*** baoli has quit IRC | 20:29 | |
*** kaisers_ has quit IRC | 20:31 | |
*** dtp has joined #openstack-nova | 20:32 | |
*** moshele has joined #openstack-nova | 20:35 | |
*** bvanhav has quit IRC | 20:38 | |
*** kberger has quit IRC | 20:39 | |
openstackgerrit | Yuming Ma proposed openstack/nova-specs master: Nova BFV by reserving volume https://review.openstack.org/417541 | 20:39 |
*** baoli has joined #openstack-nova | 20:40 | |
mriedem | anyone aware of a way to basically squash a series of changes into a single patch using git format-patch? | 20:40 |
mriedem | without first doing a git rebase to squash them into a single commit | 20:40 |
*** gouthamr has joined #openstack-nova | 20:40 | |
*** armax has quit IRC | 20:41 | |
*** armax has joined #openstack-nova | 20:41 | |
mriedem | nvm thanks google http://stackoverflow.com/questions/616556/how-do-you-squash-commits-into-one-patch-with-git-format-patch | 20:42 |
*** liangy has joined #openstack-nova | 20:42 | |
*** armax has quit IRC | 20:44 | |
cdent | edleafe: you the light blue on jay's etherpad? I think we're running into similar mental blocks :( | 20:46 |
edleafe | dat's me | 20:46 |
*** lpetrut has joined #openstack-nova | 20:48 | |
*** satyar has joined #openstack-nova | 20:48 | |
*** ducttape_ has quit IRC | 20:51 | |
*** moshele has quit IRC | 20:51 | |
*** ducttape_ has joined #openstack-nova | 20:51 | |
mriedem | nova meeting in 9 minutes | 20:51 |
*** takashin has joined #openstack-nova | 20:54 | |
*** lpetrut has quit IRC | 20:54 | |
*** aleph1 is now known as agarner_away | 20:54 | |
*** READ10 has joined #openstack-nova | 20:57 | |
*** slaweq has joined #openstack-nova | 20:58 | |
openstackgerrit | Maciej Szankin proposed openstack/nova master: Tests: remove .testrepository/times.dbm in tox.ini https://review.openstack.org/435111 | 20:59 |
mriedem | it's time | 21:00 |
*** fragatina has joined #openstack-nova | 21:02 | |
*** krtaylor has joined #openstack-nova | 21:04 | |
*** moshele has joined #openstack-nova | 21:04 | |
*** READ10 has quit IRC | 21:05 | |
*** slaweq has quit IRC | 21:06 | |
*** slaweq has joined #openstack-nova | 21:08 | |
*** moshele has quit IRC | 21:13 | |
*** lucasxu has quit IRC | 21:14 | |
*** lucasxu has joined #openstack-nova | 21:14 | |
*** dimtruck is now known as zz_dimtruck | 21:14 | |
*** cdent has quit IRC | 21:16 | |
*** yamamoto has joined #openstack-nova | 21:17 | |
*** yamamoto has quit IRC | 21:19 | |
*** dave-mccowan has quit IRC | 21:21 | |
*** karimb has quit IRC | 21:21 | |
*** karimb has joined #openstack-nova | 21:21 | |
*** armax has joined #openstack-nova | 21:22 | |
*** salv-orlando has joined #openstack-nova | 21:24 | |
*** eharney has quit IRC | 21:25 | |
*** kaisers_ has joined #openstack-nova | 21:27 | |
*** tblakes has quit IRC | 21:27 | |
*** salv-orlando has quit IRC | 21:29 | |
*** armax has quit IRC | 21:30 | |
*** lucasxu has quit IRC | 21:33 | |
*** lucasxu has joined #openstack-nova | 21:33 | |
*** tyrola has quit IRC | 21:34 | |
*** Jeffrey4l has quit IRC | 21:35 | |
*** tyrola has joined #openstack-nova | 21:35 | |
*** takashin has left #openstack-nova | 21:35 | |
cfriesen | mriedem: you had proposed that I send in a stable/newton release note for bug 1644248....what's the process for that given that there won't be an equivalent commit in master or stable/ocata? | 21:35 |
openstack | bug 1644248 in OpenStack Compute (nova) ocata "Nova incorrectly tracks live migration progress" [High,In progress] https://launchpad.net/bugs/1644248 - Assigned to Matt Riedemann (mriedem) | 21:35 |
*** Jeffrey4l has joined #openstack-nova | 21:36 | |
mriedem | cfriesen: just commit a release note to stable/newton with the known issue listed | 21:36 |
*** karimb has quit IRC | 21:37 | |
cfriesen | mriedem: okay, can do | 21:37 |
*** crushil has quit IRC | 21:38 | |
*** salv-orlando has joined #openstack-nova | 21:38 | |
*** crushil has joined #openstack-nova | 21:39 | |
*** zz_dimtruck is now known as dimtruck | 21:39 | |
*** takashin has joined #openstack-nova | 21:39 | |
*** jerrygb has quit IRC | 21:41 | |
openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy https://review.openstack.org/391288 | 21:42 |
*** ducttape_ has quit IRC | 21:42 | |
*** Sukhdev has quit IRC | 21:45 | |
*** crushil has quit IRC | 21:48 | |
*** ducttape_ has joined #openstack-nova | 21:50 | |
*** edmondsw has quit IRC | 21:52 | |
*** edmondsw has joined #openstack-nova | 21:53 | |
*** edmondsw has quit IRC | 21:57 | |
openstackgerrit | melanie witt proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521 | 21:59 |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512 | 22:00 |
*** satyar has quit IRC | 22:00 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43: List/Show all server migration types https://review.openstack.org/430839 | 22:02 |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43 - Enable cold migration with target host https://review.openstack.org/406707 | 22:02 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make servers API use cell-targeted context https://review.openstack.org/433362 | 22:02 |
*** tbachman has quit IRC | 22:02 | |
dansmith | melwitt: surprisingly little drama with unit and functional ^ | 22:02 |
*** smatzek has quit IRC | 22:02 | |
dansmith | we'll see how devstack likes it | 22:02 |
*** ducttape_ has quit IRC | 22:03 | |
melwitt | dansmith: yeah, that's almost no change to the unit and functional. huh. | 22:04 |
openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | 22:05 |
dansmith | well, it kinda makes sense since I'm just making the set call a no-op for the simple cell fixture | 22:05 |
dansmith | although I'm surprised a bunch of functional tests didn't fail | 22:05 |
dansmith | but we'll see | 22:05 |
melwitt | yeah | 22:06 |
mriedem | bauzas: got it for you https://review.openstack.org/#/c/433855/ | 22:06 |
bauzas | mriedem: heh | 22:08 |
bauzas | anyway, I promised a comment | 22:09 |
*** ducttape_ has joined #openstack-nova | 22:10 | |
openstackgerrit | Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298 | 22:12 |
*** smatzek has joined #openstack-nova | 22:12 | |
cfriesen | dansmith: regarding https://review.openstack.org/#/c/265282/ if we remove the "name" and "id" from the flavor details, do we just display all of the flavor info when doing "nova list --fields flavor"? Currently it shows just the flavor ID. | 22:12 |
*** smatzek has quit IRC | 22:13 | |
*** smatzek_ has joined #openstack-nova | 22:13 | |
*** dave-mccowan has joined #openstack-nova | 22:14 | |
*** viks has joined #openstack-nova | 22:15 | |
*** viks has left #openstack-nova | 22:15 | |
*** gouthamr has quit IRC | 22:15 | |
*** viks has joined #openstack-nova | 22:16 | |
*** zzfoo has joined #openstack-nova | 22:20 | |
zzfoo | Hi, I'm trying to get live migration working. I have shared store setup and working with NFS though when i do a live migrate I get the following error in the nova-compute logs: error: | 22:21 |
zzfoo | Unable to acquire lock on `/opt/stack/data/nova/instances/locks/nova-fd2836e4eef36d5483a37a5000a42a72fb95ea7c` due to [Errno 37] No locks available | 22:21 |
cfriesen | does anyone know why I'm not getting an auto-generated Change-Id when creating a python-novaclient commit? Is this expected or did I mess something up? | 22:22 |
zzfoo | on the shared storage though that file has been created. | 22:22 |
melwitt | cfriesen: not expected. did you 'git review -s' first after you cloned python-novaclient? | 22:22 |
melwitt | you have to do it anytime you clone a new rep, afaik | 22:23 |
melwitt | *repo | 22:23 |
cfriesen | melwitt: it's entirely possible that I didn't | 22:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: localdisk https://review.openstack.org/409401 | 22:24 |
melwitt | cfriesen: you can try 'git review -s' in the repo dir, then 'git commit --amend' and then you should get a change-id | 22:24 |
cfriesen | melwitt: that seems to have done it. thanks | 22:25 |
*** armax has joined #openstack-nova | 22:27 | |
*** avolkov has quit IRC | 22:28 | |
EmilienM | mriedem: I took the freedom to write some thoughts on https://etherpad.openstack.org/p/nova-ocata-retrospective - I'll probably have more later | 22:32 |
openstackgerrit | Chris Friesen proposed openstack/python-novaclient master: Add support for showing flavor details (v2.43) https://review.openstack.org/435141 | 22:32 |
mriedem | EmilienM: ok, i expected it :) | 22:33 |
*** yamamoto has joined #openstack-nova | 22:35 | |
*** ijw has joined #openstack-nova | 22:36 | |
*** annegentle has quit IRC | 22:36 | |
*** smatzek_ has quit IRC | 22:38 | |
*** rfolco has joined #openstack-nova | 22:39 | |
*** thorst_afk has quit IRC | 22:39 | |
*** rfolco has quit IRC | 22:39 | |
*** kaisers_ has quit IRC | 22:40 | |
mriedem | hmm, what runs qemu-img info in a periodic? | 22:40 |
mriedem | get_available_resources? | 22:40 |
*** burgerk has quit IRC | 22:41 | |
cfriesen | seems likely | 22:42 |
mriedem | so, i've seen this a few times lately, | 22:42 |
mriedem | the live migration job fails because we timeout waiting for an instance to spawn | 22:42 |
mriedem | and we get here http://logs.openstack.org/50/435050/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/49adcde/logs/subnode-2/screen-n-cpu.txt.gz#_2017-02-16_22_22_11_143 | 22:42 |
mriedem | 2017-02-16 22:22:11.143 14954 DEBUG nova.virt.libvirt.driver [req-025bf4d2-e5ba-4236-a334-f9eb98105ada tempest-LiveBlockMigrationTestJSON-793129345 tempest-LiveBlockMigrationTestJSON-793129345] [instance: 0ee93807-d206-4ddf-878c-efd1dd2eab3c] Instance is running spawn /opt/stack/new/nova/nova/virt/libvirt/driver.py:2689 | 22:42 |
mriedem | which calls spawn() on the libvirt driver | 22:43 |
mriedem | and after that, it's radio silence on that instance and request except for periodic tasks | 22:43 |
*** Sukhdev has joined #openstack-nova | 22:43 | |
mriedem | i have a feeling we have some kind of deadlock or something in the libvirt driver's spawn flow | 22:43 |
mriedem | i've seen it since newton | 22:43 |
cfriesen | so you're thinking self._domain.createWithFlags() is triggering something wonky? | 22:47 |
cfriesen | or is there another async call before that? | 22:48 |
*** acabot has quit IRC | 22:48 | |
*** liangy has quit IRC | 22:50 | |
openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402 | 22:50 |
*** efried has quit IRC | 22:51 | |
mriedem | cfriesen: i don't know what it is | 22:51 |
mriedem | last time i looked into this i was going to add some debug logging in the image create code | 22:51 |
mriedem | because that's gone through a lot of churn since newton | 22:51 |
cfriesen | mriedem: we're getting back to the debug log though, which means that we've done the image create, no? | 22:53 |
*** efried has joined #openstack-nova | 22:54 | |
*** marst has quit IRC | 22:54 | |
mriedem | this is also weird http://logs.openstack.org/50/435050/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/49adcde/logs/subnode-2/screen-n-cpu.txt.gz#_2017-02-16_22_22_06_286 | 22:54 |
mriedem | Instance network_info: |[]| | 22:55 |
mriedem | so i wonder if, | 22:57 |
*** felipemonteiro_ has quit IRC | 22:57 | |
mriedem | the compute manager is just waiting for the async network_info stuff to get done | 22:57 |
mriedem | but it doesn't complete because it's empty - but i don't think that's how it works anyway, we allow booting a server with no networkin | 22:57 |
mriedem | and test it in tempest with the 2.37 microversion | 22:58 |
cfriesen | whatever it is, it's not blocking the self._create_domain_and_network() call because we're seeing the log right after it. So it has to be something async that we're not waiting for. | 22:58 |
*** acabot has joined #openstack-nova | 23:01 | |
*** jamielennox is now known as jamielennox|away | 23:02 | |
*** catintheroof has joined #openstack-nova | 23:02 | |
mriedem | cfriesen: yeah right after that we see timer = loopingcall.FixedIntervalLoopingCall(_wait_for_boot) | 23:02 |
mriedem | which is waiting for the power state to be RUNNING | 23:02 |
mriedem | i guess we just never finish there | 23:03 |
mriedem | because i never see "Instance spawned successfully." | 23:03 |
*** thorst_afk has joined #openstack-nova | 23:03 | |
mriedem | even though i see http://logs.openstack.org/50/435050/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/49adcde/logs/subnode-2/screen-n-cpu.txt.gz#_2017-02-16_22_22_11_214 | 23:03 |
mriedem | [instance: 0ee93807-d206-4ddf-878c-efd1dd2eab3c] VM Started (Lifecycle Event) | 23:03 |
mriedem | and then right after that [instance: 0ee93807-d206-4ddf-878c-efd1dd2eab3c] VM Paused (Lifecycle Event) | 23:04 |
*** cdent has joined #openstack-nova | 23:04 | |
*** mdrabe has quit IRC | 23:04 | |
mriedem | Synchronizing instance power state after lifecycle event "Paused"; current vm_state: building, current task_state: spawning, current DB power_state: 0, VM power_state: 3 | 23:04 |
mriedem | 3 = paused | 23:04 |
mriedem | i don't see "Instance spawned successfully." anywhere in http://logs.openstack.org/50/435050/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/49adcde/logs/subnode-2/screen-n-cpu.txt.gz | 23:05 |
*** dave-mccowan has quit IRC | 23:06 | |
mriedem | so the instance goes from started to paused, | 23:06 |
mriedem | and then is never unpaused | 23:06 |
mriedem | which is why the spawn() method never considers it done i guess | 23:07 |
*** thorst_afk has quit IRC | 23:07 | |
*** esberglu has quit IRC | 23:07 | |
*** dimtruck is now known as zz_dimtruck | 23:07 | |
*** esberglu has joined #openstack-nova | 23:07 | |
mriedem | http://logs.openstack.org/50/435050/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/49adcde/logs/subnode-2/libvirt/qemu/instance-00000001.txt.gz | 23:08 |
mriedem | what does this mean? "KVM: entry failed, hardware error 0x0" | 23:08 |
*** esberglu has quit IRC | 23:12 | |
openstackgerrit | Sujitha proposed openstack/nova master: This patch is followup for https://review.openstack.org/#/c/415298/. https://review.openstack.org/435146 | 23:13 |
*** jamielennox|away is now known as jamielennox | 23:14 | |
*** lucasxu has quit IRC | 23:15 | |
cdent | mriedem: casual googling suggests problems with nested virtualization | 23:15 |
*** marst has joined #openstack-nova | 23:16 | |
mriedem | i'm crawling through the libvirtd logs | 23:16 |
openstackgerrit | Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298 | 23:18 |
*** baoli has quit IRC | 23:18 | |
openstackgerrit | Sujitha proposed openstack/nova master: This patch is followup for https://review.openstack.org/#/c/415298/. https://review.openstack.org/435146 | 23:19 |
mriedem | i see this start to go crazy | 23:19 |
mriedem | debug : qemuMonitorJSONIOProcess:255 : Total used 0 bytes out of 1023 available in buffer | 23:19 |
mriedem | debug : qemuMonitorJSONIOProcess:255 : Total used 0 bytes out of 2047 available in buffer | 23:19 |
mriedem | debug : qemuMonitorJSONIOProcess:255 : Total used 0 bytes out of 3071 available in buffer | 23:19 |
mriedem | debug : qemuMonitorJSONIOProcess:255 : Total used 0 bytes out of 4095 available in buffer | 23:19 |
mriedem | ... | 23:20 |
mriedem | debug : qemuMonitorJSONIOProcess:255 : Total used 6689 bytes out of 6689 available in buffer | 23:20 |
mriedem | maybe that's normal | 23:20 |
*** esberglu has joined #openstack-nova | 23:21 | |
mriedem | 2017-02-16 22:22:11.139+0000: 8968: debug : qemuProcessHandleResume:765 : Transitioned guest instance-00000001 out of paused into resumed state | 23:21 |
mriedem | 2017-02-16 22:22:11.140+0000: 8968: debug : qemuProcessHandleStop:716 : Transitioned guest instance-00000001 to paused state | 23:22 |
openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404 | 23:22 |
openstackgerrit | Sujitha proposed openstack/nova master: [WIP] Reduce calls to load_tags() to 0 https://review.openstack.org/435146 | 23:23 |
*** jamesdenton has quit IRC | 23:23 | |
mriedem | no errors in the libvirt logs | 23:23 |
*** salv-orlando has quit IRC | 23:24 | |
*** esberglu has quit IRC | 23:24 | |
*** esberglu has joined #openstack-nova | 23:24 | |
mriedem | aha | 23:25 |
mriedem | right around that time in the syslog | 23:25 |
mriedem | http://logs.openstack.org/50/435050/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/49adcde/logs/subnode-2/syslog.txt.gz#_Feb_16_22_22_11 | 23:25 |
mriedem | End of file while reading data: Input/output error | 23:26 |
mriedem | we shouldn't even be using virtlogd | 23:27 |
mriedem | dansmith: you ever seen anything like this? ^ | 23:27 |
mriedem | channel your inner berrange | 23:27 |
mriedem | maybe the i/o error with virtlogd is a result of the guest hw error? | 23:28 |
*** takashin has left #openstack-nova | 23:29 | |
mriedem | i guess that's really normal | 23:29 |
mriedem | it's all over syslog in our ci runs | 23:29 |
cdent | that's an unfortunate way to spell normal | 23:30 |
*** xyang1 has quit IRC | 23:33 | |
mriedem | virt_type = kvm in nova.conf | 23:34 |
mriedem | that should be qemu | 23:34 |
mriedem | we don't support that in the gate with nested virt, how the hell was this even running | 23:34 |
mriedem | so yeah, that's the issue | 23:35 |
cdent | mriedem: how/where/why? | 23:36 |
cdent | as in wat? | 23:37 |
cdent | and wtf | 23:37 |
mriedem | it's qemu on the primary node, but kvm on the subnode | 23:37 |
mriedem | that's the bug | 23:37 |
cfriesen | anyone got a suggestion on the best way to deal with this? http://paste.openstack.org/show/599318/ I'm adding a check against cs.api_version in v2.shell.do_list() and it's failing a bunch of tests with that error | 23:38 |
cdent | mriedem: ah, and somehow this only showed up as a problem now? | 23:38 |
*** Hosam has joined #openstack-nova | 23:38 | |
mriedem | cdent: i've been seeing it off and on | 23:39 |
mriedem | i just haven't taken the 2 hours to investigate :) | 23:39 |
cdent | ow | 23:39 |
cdent | do you ever sleep or eat or recreate in some fashion (please don't tell me this is your recreation)? | 23:40 |
mriedem | when my wife forces me, yes | 23:41 |
mriedem | i only care about this since it kicked the last change for ocata ga out of the gagte | 23:41 |
mriedem | *Gate | 23:41 |
mriedem | she gets up at 4am and goes to bed at 7pm though so i will start getting texts here pretty quick to get my ass home and take over parenting duties | 23:42 |
*** esberglu has quit IRC | 23:46 | |
cdent | g'night | 23:47 |
*** cdent has quit IRC | 23:47 | |
openstackgerrit | Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 23:47 |
*** edmondsw has joined #openstack-nova | 23:49 | |
mriedem | https://review.openstack.org/#/c/435154/1 | 23:51 |
mriedem | YES! | 23:51 |
mriedem | sdague: ^^^^ | 23:51 |
mriedem | jogo: even you ^ | 23:51 |
mriedem | mtreinish: ^ | 23:51 |
mtreinish | mriedem: hah, nice | 23:52 |
*** edmondsw has quit IRC | 23:53 | |
mriedem | ok i'm tapping out, back online later | 23:54 |
jogo | mriedem: done | 23:55 |
mriedem | ha | 23:55 |
mriedem | o/ | 23:55 |
*** mriedem has quit IRC | 23:56 | |
*** jaypipes has quit IRC | 23:56 | |
*** kylek3h has joined #openstack-nova | 23:57 | |
*** armax has quit IRC | 23:58 | |
*** kylek3h_ has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!