Thursday, 2017-02-16

dansmithoh. snap00:05
*** Sukhdev has joined #openstack-nova00:06
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.resize_revert notification  https://review.openstack.org/39681100:07
melwittdansmith: good news?00:08
dansmithmelwitt: yes, you'll be crapping on it soon00:08
melwitthaha, nice00:09
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.resize_revert notification  https://review.openstack.org/39681100:09
*** d-bark has joined #openstack-nova00:11
*** Sukhdev has quit IRC00:11
*** bswartz has quit IRC00:12
*** crushil has joined #openstack-nova00:15
*** esberglu has joined #openstack-nova00:15
openstackgerritDan Smith proposed openstack/nova master: Make start and stop admin actions cells-aware  https://review.openstack.org/43336200:16
openstackgerritDan Smith proposed openstack/nova master: Make CellDatabases fixture work over RPC  https://review.openstack.org/43456300:16
dansmithmelwitt: ^00:16
*** mtanino has quit IRC00:16
*** sdague has quit IRC00:18
*** browne has joined #openstack-nova00:18
*** thorst_ has joined #openstack-nova00:19
*** thorst_ has quit IRC00:19
*** Sukhdev has joined #openstack-nova00:20
melwittdansmith: looks pretty good to me, actually. it's self-contained and makes it clear what's happening00:20
dansmithmelwitt: it works for my test, but haven't checked to see if it breaks anything else just yet00:20
*** yingjun has joined #openstack-nova00:20
dansmith\o/00:20
*** hfu has joined #openstack-nova00:22
*** nic has quit IRC00:29
dansmithheh, so many unit tests pass None for the context so I'll have to be defensive there00:29
*** adrian_otto has quit IRC00:30
*** gongysh has joined #openstack-nova00:30
melwittdansmith: 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 probably00:30
dansmithyeah00:30
dansmithmelwitt: did you see this? https://review.openstack.org/#/c/43329800:31
dansmithfailed because of the cellsv1 breakage, but should be ready for review at least00:31
melwittmaybe there were some tests that were checking for a None context on purpose, or something too00:31
dansmithwith that, I can boot an instance with "multiple cells"00:31
*** hongbin has quit IRC00:33
*** Swami_ has joined #openstack-nova00:33
openstackgerritDan Smith proposed openstack/nova master: Make CellDatabases fixture work over RPC  https://review.openstack.org/43456300:33
openstackgerritDan Smith proposed openstack/nova master: Make start and stop admin actions cells-aware  https://review.openstack.org/43336200:33
melwittdansmith: 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
dansmithmelwitt: because I have a hack for devstack that makes cell0 the main connection for the api service, which means anything not targeted fails00:34
dansmithand it is how I hit these two first patches for multiple cell support .. i.e. it exposes where we're not targeting00:34
melwittoh, cool. so you saw it work in devstack. was just curious00:35
dansmithmelwitt: yep, with this: https://review.openstack.org/#/c/433318/00:35
melwittnice00:35
dansmithmuch simpler than actual multi-cell devstack and should let us flush out a ton of things without the complication00:36
melwittpretty cool00:36
*** mriedem has joined #openstack-nova00:37
*** Swami_ has quit IRC00:38
*** karimb has quit IRC00:44
*** Sukhdev has quit IRC00:45
*** jeffrey4l_ircc has joined #openstack-nova00:45
*** gcb has quit IRC00:46
*** thorst_ has joined #openstack-nova00:46
*** thorst_ has quit IRC00:46
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.resize_confirm notification  https://review.openstack.org/39065400:47
*** masayukig_ is now known as masayukig00:49
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.resize_confirm notification  https://review.openstack.org/39065400:51
cburgessmriedem do we have a start time for the PTG? 9AM? 10AM?00:52
*** hoangcx has joined #openstack-nova00:54
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.rescue notifications  https://review.openstack.org/38564400:54
dansmithcburgess: 7am00:55
dansmithdonuts by 6:3000:55
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.rescue notifications  https://review.openstack.org/38564400:55
cburgessdansmith *glare*00:55
cburgessdansmith I wish there was a voodoo in ATL00:56
cburgessWe do have Krispy Kreme though which is decent.00:56
mriedemcburgess: 9am00:56
*** huanxie has joined #openstack-nova00:56
cburgessI won't have a car though so bringing donuts might be tough.00:56
dansmithcburgess: I'm sure you'll work it out00:56
cburgessGetting dropped off or taking Marta or Uber most mornings.00:56
cburgessmriedem Cool00:56
cburgessdansmith Besides, donuts are only for Friday meetup.00:56
*** dave-mccowan has joined #openstack-nova00:57
mriedemcburgess: just remember that old saying,00:57
mriedemcash, grass or ass00:57
dansmithcburgess: remember ptg is "all friday all week"00:57
cburgessmriedem LOL00:57
cburgessdansmith Well that will be a productive time then.. oh wait...00:57
mriedemdansmith: i don't think you've yet realized the amount of coordination that is going into the PTG scheduling now across teams00:58
mriedemit's like the design summit now, but worse because there aren't actually slots00:58
mriedemexcept those we make ourselves00:58
dansmithmriedem: I'm sure :)00:58
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.unrescue notifications  https://review.openstack.org/38827500:58
dansmithcinder and nova will meet from 2:13 to 3:0700:58
cburgessThis sounds like an award wining idea.00:58
*** huanxie has quit IRC00:59
*** edmondsw has joined #openstack-nova01:00
openstackgerritSTEW TY proposed openstack/nova master: Transform instance.unrescue notifications  https://review.openstack.org/38827501:00
*** liangy has joined #openstack-nova01:00
mriedemdansmith: since you're still around https://review.openstack.org/#/c/434552/01:02
mriedemand 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/newton01:02
mriedemtomorrow is the big release party01:02
*** phuongnh has joined #openstack-nova01:03
openstackgerrityuhui proposed openstack/nova master: Fix some typo errors  https://review.openstack.org/43062901:03
mriedemcburgess: are you in ATL because i thought you lived in CA01:04
*** edmondsw has quit IRC01:04
mriedemas in caleeforneea, not canada01:04
melwittmay I suggest https://review.openstack.org/#/c/425114/ https://review.openstack.org/#/c/416557/ and https://review.openstack.org/#/c/416558/ for dansmith's eye01:04
dansmithdamn you people01:04
* melwitt looks for more01:05
cburgessmriedem I live in LA area (Altadena, CA), but I grew up in Atlanta, My folks still live there.01:05
mriedemcburgess: orly, well then since you're the one that wanted a group dinner,01:06
mriedemmay i suggest you take on some delegation duties01:06
cburgessmriedem Back away from that slows...01:06
cburgessmriedem Damn.. I shouldn't have said anything. To be fair I haven't lived there in almost 20 years.01:06
mriedemi was looking at places within walking distance of the sheraton and alma cocina was all i came up with01:06
cburgessBut ok.. I'll for places in/around downtown this weekend.01:06
cburgessYeah downtown is a big tough.01:06
dansmithwe should go to that steak place01:06
dansmithkevin somethingorother01:07
mriedemthere is a "rays in the city" that got good reviews01:07
dansmiththey had a few things for under $100 a plate01:07
alaskidansmith: +101:07
dansmithalaski: what was that place?01:07
cburgessmriedem Ray's is pricey though isn't it?01:07
dansmithKevin Goodsteakums or something01:07
mriedemcburgess: yes01:07
alaskihttp://www.kevinrathbunsteak.com/menu.html01:07
cburgessBecaue the origian Ray's is out by the river and its swanky as hell.01:07
mriedemi know a mike rathbun01:07
dansmithalaski: ooooghhhhyeahhhummmmmahhhhyeahhh01:07
mriedemcburgess: it would only be swankier if it were on a river boat01:07
melwittsteak is good01:08
alaskidansmith: sooo freaking good01:08
dansmithalaski: so good I want to lick the screen right now01:09
melwittthey have Blue rare on the menu, that sounds promising01:09
dansmithbut I won't... now.01:09
dansmithmelwitt: we shall go one night at least01:09
melwitt\o/01:09
alaskiYou just need to find an uber driver to drop you off close to, but not quite at the place01:09
cburgessalaski Why is that?01:09
dansmithalaski: hah yeah that was hilarious01:10
dansmithalaski: "here um, you're within a half mile g'luck"01:10
alaskicburgess: that's just how it worked out last time01:10
cburgessOh lame01:10
melwitthah wat01:10
cburgessI hae to admit the worse uber I ever had was last time in ATL at the summit01:10
dansmithI watched alaski and comstud share a steak at this place01:11
dansmithit was super romantic01:11
cburgessmriedem We could just all go to waffle house. :)01:11
alaskidansmith: the restaraunt was dimly lit and everything, there may have even been a candle at the table01:12
mriedemcburgess: is there seating for 20?01:12
cburgessmriedem Depend on the waffle hosue.01:12
dansmithalaski: there was definitely inappropriate footsie under the table01:12
mriedemdansmith: alaski: table clothes?01:12
*** iceyao has joined #openstack-nova01:13
melwittdansmith: lol. I was just noticing all the dry-aged is for 2+, so that explains that. or does it?01:13
mriedemwas just going to say...01:13
dansmithheh01:13
dansmithsure, but did they have to hold hands the whole time?01:13
*** browne has quit IRC01:13
melwitthah01:13
mriedemATL was the place that i almost got myself, dansmith and russellb locked in a parking gargage01:13
mriedem*garage01:13
alaskithat clearly didn't happen because I couldn't cut my steak with one hand01:13
mriedemalaski: comstud cut and fed you probably01:14
dansmithmriedem: hmm, not sure I remember that01:14
dansmithhaha01:14
dansmithalaski: busted.01:14
alaskiI held the fork, he cut... could've worked I think01:14
dansmithalaski: actually it was french cut, if you recall, so not sure it needed much cutting01:15
*** claudiub has quit IRC01:15
mriedemis french cut the same as butterfly cut?01:15
*** fandi has joined #openstack-nova01:15
dansmithit was cut into slices01:15
*** tovin07_ has joined #openstack-nova01:15
alaskidansmith: that's true, and it practically melted anyways01:15
mriedemoh i see01:15
dansmithI thought they said french at the time01:15
mriedemlooks like brisket01:15
*** jwcroppe has quit IRC01:16
mriedemcan i find some brisket while in GA?01:16
dansmithalaski: yeah, you could stare it into smaller pieces if you needed01:16
cburgessmriedem http://www.foxbrosbbq.com01:16
cburgessBest in/around atlanta01:16
*** jwcroppe has joined #openstack-nova01:16
*** browne has joined #openstack-nova01:16
*** yuntongjin has quit IRC01:17
mriedemcburgess: that's only 10 minutes from the hilton01:17
mriedemthat might work01:18
cburgessmriedem 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 subway01:18
jrollbauzas: instance.host == compute_node.host, instance.node == compute_node.hypervisor_hostname01:19
cburgessmriedem 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
mriedemhard to get into on a wed night?01:19
mriedemat like 7pm?01:20
*** zzfoo has joined #openstack-nova01:20
zzfooHi, 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 IRC01:20
cburgessmriedem Yeah01:20
cburgessProbably...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 IRC01:20
mriedemzzfoo: run "nova-manage cell_v2 discover_hosts"01:21
zzfooany ideas why this is occurring and if there is something special i need to do in devstack to get around it?01:21
cburgessmriedem There is also http://sweetauburnbbq.com01:21
cburgessThats close too but again.. probably going to be a bit crowded.01:21
mriedemzzfoo: run that from the controller node01:21
dansmithmriedem: cburgess is kinda proving that he should be the dinner czar, amirite?01:21
mriedemdansmith: yes01:21
dansmithk, just checking01:21
mriedemdefinitely not me because i can't decide things for groups01:22
cburgessSweet Auburn is at least less touristy01:22
cburgessmriedem dansmith LOL01:22
zzfoomriedem:  cool thanks, that did the trick. I guess this needs to be run each time a new host is added?01:22
mriedemzzfoo: yup01:22
*** dave-mccowan has quit IRC01:23
mriedemzzfoo: https://docs.openstack.org/developer/nova/cells.html#setup-of-cells-v201:23
mriedemcburgess: that place is also about 9 min from downtown01:23
mriedemso would work01:23
*** kevinz has joined #openstack-nova01:23
mriedemi'm good with anything really for a large group01:23
mriedemif we can't make reservations,01:23
mriedemwe probably just pick a place and a night and a time01:23
mriedemand whoever shows up shows up01:24
mriedemthat's easier than asking for head count and restrictions and all that crap01:24
cburgessmriedem 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
mriedemcburgess: that works for me01:25
cburgessmriedem 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
dansmithawesome01:25
cburgessmriedem I'll contact sweet auburn tomorrow and see what we can do for Thursday.01:26
mriedemcburgess: thanks, much appreciated01:26
mriedemi was not looking forward to trying to organize a dinner01:26
*** jwcroppe has joined #openstack-nova01:27
*** thorst_ has joined #openstack-nova01:29
*** yuntongjin has joined #openstack-nova01:30
*** thorst_ has quit IRC01:32
*** esberglu has quit IRC01:34
*** browne has quit IRC01:36
*** wllabs has quit IRC01:36
*** wllabs has joined #openstack-nova01:38
*** fandi has quit IRC01:39
*** sree has quit IRC01:42
*** sree has joined #openstack-nova01:42
*** liangy has quit IRC01:45
*** sree has quit IRC01:46
*** salv-orlando has joined #openstack-nova01:47
*** jamesdenton has joined #openstack-nova01:50
*** Jinxing has joined #openstack-nova01:51
*** salv-orlando has quit IRC01:52
*** Apoorva has quit IRC01:52
*** gcb has joined #openstack-nova01:52
*** jamesdenton has quit IRC01:54
*** Jinxing has quit IRC01:56
*** Jinxing has joined #openstack-nova01:56
*** dave-mccowan has joined #openstack-nova01:57
*** huanxie has joined #openstack-nova01:58
*** jamesdenton has joined #openstack-nova01:58
*** crushil has quit IRC02:03
*** zerda4 has quit IRC02:05
*** zerda3 has joined #openstack-nova02:06
*** gongysh has quit IRC02:09
*** eliqiao has quit IRC02:09
mriedemjroll: 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 IRC02:11
*** unicell has quit IRC02:12
*** yamahata has quit IRC02:16
*** thorst_ has joined #openstack-nova02:17
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix doc generation warnings  https://review.openstack.org/42554902:18
*** thorst_ has quit IRC02:18
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types  https://review.openstack.org/43060802:19
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895502:20
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896402:22
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492602:23
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004202:24
*** yamahata has joined #openstack-nova02:25
*** raunak has quit IRC02:32
*** sam_nowitzki has joined #openstack-nova02:32
*** esberglu has joined #openstack-nova02:38
*** thorst_ has joined #openstack-nova02:38
*** thorst_ has quit IRC02:40
*** esberglu has quit IRC02:41
*** zzfoo has quit IRC02:42
*** thorst_ has joined #openstack-nova02:43
*** thorst_ has quit IRC02:43
*** crushil has joined #openstack-nova02:43
*** tbachman has quit IRC02:43
*** zerda3 has quit IRC02:44
*** xyang1 has joined #openstack-nova02:44
*** zerda3 has joined #openstack-nova02:45
*** thorst_ has joined #openstack-nova02:46
*** jeffrey4l_ircc has quit IRC02:47
*** thorst_ has quit IRC02:51
*** xyang1 has quit IRC02:52
*** amotoki has joined #openstack-nova02:53
*** ijw has quit IRC02:55
mriedemdansmith: i see how update_cell could not take effect because of host_mapping_cache in conductor also02:56
*** hongbin has joined #openstack-nova02:57
mriedemit's a map of hosts to hostmapping objects with the cell_mapping set02:57
mriedemso if you change a cell mapping transport_url/db_connection you have to restart api and conductor02:57
*** yamahata has quit IRC02:57
*** thorst_ has joined #openstack-nova03:01
*** crushil has quit IRC03:02
*** thorst_ has quit IRC03:03
*** gongysh has joined #openstack-nova03:06
*** mdrabe has joined #openstack-nova03:08
*** mdrabe has quit IRC03:13
*** panda|zZ has quit IRC03:18
mriedemjaypipes: i know you love you some BFV https://review.openstack.org/#/c/335358/03:19
mriedem...and nightmares03:19
*** raunak has joined #openstack-nova03:21
openstackgerritTuan Luong-Anh proposed openstack/nova master: Fix hypervisor-stats show invalid info of baremetal resource  https://review.openstack.org/39141503:22
*** whenry has quit IRC03:24
*** ssurana has joined #openstack-nova03:24
*** ssurana has quit IRC03:24
*** whenry has joined #openstack-nova03:40
openstackgerritMatt Riedemann proposed openstack/nova master: Remove dead code in _bury_in_cell0  https://review.openstack.org/43461503:40
*** amotoki has quit IRC03:41
*** nicolasbock has quit IRC03:42
*** gouthamr has quit IRC03:44
*** Hosam has joined #openstack-nova03:45
*** Hosam has quit IRC03:45
*** Hosam has joined #openstack-nova03:45
*** mriedem is now known as mriedem_afk03:48
*** salv-orlando has joined #openstack-nova03:48
*** yamahata has joined #openstack-nova03:52
*** salv-orlando has quit IRC03:53
*** sudipto has joined #openstack-nova03:57
*** sudipto_ has joined #openstack-nova03:57
*** yuntongjin has quit IRC03:58
*** armax has quit IRC03:59
*** diana_clarke has quit IRC04:00
*** ijw has joined #openstack-nova04:01
*** links has joined #openstack-nova04:02
*** whenry has quit IRC04:03
*** nmathew has joined #openstack-nova04:07
*** yamamoto has joined #openstack-nova04:10
*** ratailor has joined #openstack-nova04:11
*** Jinxing has quit IRC04:18
*** dave-mccowan has quit IRC04:19
*** tyrola has quit IRC04:19
*** tyrola has joined #openstack-nova04:20
*** adrian_otto has joined #openstack-nova04:26
*** esberglu has joined #openstack-nova04:29
*** amotoki has joined #openstack-nova04:29
*** Hosam has quit IRC04:30
*** sree has joined #openstack-nova04:32
*** esberglu has quit IRC04:34
*** tlian has quit IRC04:34
*** mdnadeem has joined #openstack-nova04:35
*** sree_ has joined #openstack-nova04:36
*** sree_ is now known as Guest5886304:36
*** sree has quit IRC04:36
*** hongbin has quit IRC04:38
*** amotoki has quit IRC04:40
*** jerrygb has quit IRC04:41
*** Jinxing has joined #openstack-nova04:41
*** sudipto has quit IRC04:44
*** sudipto_ has quit IRC04:44
*** amotoki has joined #openstack-nova04:48
*** Jinxing has quit IRC04:49
*** jamesdenton has quit IRC04:58
*** Jinxing has joined #openstack-nova05:01
*** thorst_ has joined #openstack-nova05:04
*** erlon has quit IRC05:05
*** sudipto has joined #openstack-nova05:07
*** sudipto_ has joined #openstack-nova05:07
*** Hosam has joined #openstack-nova05:08
*** thorst_ has quit IRC05:08
*** dikonoor has joined #openstack-nova05:12
*** prateek has joined #openstack-nova05:13
*** gongysh has quit IRC05:13
*** adrian_otto has quit IRC05:15
*** adisky_ has joined #openstack-nova05:16
*** gongysh has joined #openstack-nova05:19
*** ijw has quit IRC05:22
*** dimtruck is now known as zz_dimtruck05:24
*** salv-orlando has joined #openstack-nova05:28
*** Jinxing has quit IRC05:29
*** jaosorior has joined #openstack-nova05:31
*** salv-orlando has quit IRC05:33
*** unicell has joined #openstack-nova05:37
*** dikonoor has quit IRC05:39
*** diga has joined #openstack-nova05:43
*** Jinxing has joined #openstack-nova05:43
*** yingjun has quit IRC05:46
*** Jinxing has quit IRC05:48
*** Jinxing has joined #openstack-nova05:49
*** tojuvone has joined #openstack-nova05:52
*** adrian_otto has joined #openstack-nova05:52
*** masber has quit IRC05:53
*** salv-orlando has joined #openstack-nova05:53
*** unicell has quit IRC05:53
*** unicell has joined #openstack-nova05:54
*** adrian_otto has quit IRC05:54
*** dikonoor has joined #openstack-nova05:57
*** sandanar has joined #openstack-nova05:58
*** salv-orl_ has joined #openstack-nova06:04
*** salv-orlando has quit IRC06:07
openstackgerritJens Rosenboom proposed openstack/nova master: Fix novncproxy for python3  https://review.openstack.org/43466606:09
*** satyar has joined #openstack-nova06:17
*** lpetrut has joined #openstack-nova06:20
*** tuanluong has joined #openstack-nova06:20
*** esberglu has joined #openstack-nova06:23
*** claudiub has joined #openstack-nova06:25
*** esberglu has quit IRC06:28
*** edmondsw has joined #openstack-nova06:29
*** edmondsw has quit IRC06:34
*** adrian_otto has joined #openstack-nova06:41
*** moshele has joined #openstack-nova06:43
*** udesale has joined #openstack-nova06:43
*** sree has joined #openstack-nova06:44
*** diga has quit IRC06:45
*** Guest58863 has quit IRC06:47
*** Jinxing has quit IRC06:51
*** raunak has quit IRC06:55
*** hoangcx has quit IRC06:55
*** ijw has joined #openstack-nova06:57
*** hoangcx has joined #openstack-nova06:58
*** Jinxing has joined #openstack-nova07:01
*** seongu has joined #openstack-nova07:01
*** hfu_ has joined #openstack-nova07:03
*** rcernin has joined #openstack-nova07:03
*** seongu99 has joined #openstack-nova07:03
*** hfu_ has quit IRC07:03
seongu99.07:03
*** hfu_ has joined #openstack-nova07:04
*** andreas_s has joined #openstack-nova07:04
*** sandanar_ has joined #openstack-nova07:04
*** thorst_ has joined #openstack-nova07:05
*** hfu has quit IRC07:05
*** Jinxing has quit IRC07:05
*** sandanar has quit IRC07:06
*** seongu has quit IRC07:07
openstackgerritSergey Nikitin proposed openstack/nova master: Change parent of diagnostics objects to NovaObject class  https://review.openstack.org/39448007:07
openstackgerritSergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API.  https://review.openstack.org/35554007:07
openstackgerritSergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen  https://review.openstack.org/39961307:07
*** Jinxing has joined #openstack-nova07:07
*** sandanar__ has joined #openstack-nova07:07
*** seongu has joined #openstack-nova07:07
*** seongu99 has quit IRC07:08
*** thorst_ has quit IRC07:10
*** sandanar_ has quit IRC07:11
*** seongu has quit IRC07:12
*** tesseract has joined #openstack-nova07:13
*** lpetrut has quit IRC07:16
*** jbryce has quit IRC07:18
*** jbryce has joined #openstack-nova07:20
*** ekuris has quit IRC07:21
*** edand has joined #openstack-nova07:22
*** cfriesen has quit IRC07:24
*** yamamoto has quit IRC07:27
*** sridharg has joined #openstack-nova07:27
*** amotoki_ has joined #openstack-nova07:29
*** amotoki has quit IRC07:32
*** yingjun has joined #openstack-nova07:32
*** gus_ is now known as gus07:32
*** markus_z has joined #openstack-nova07:32
*** ekuris has joined #openstack-nova07:35
*** diga has joined #openstack-nova07:36
*** karimb has joined #openstack-nova07:41
*** willise has joined #openstack-nova07:43
*** hoangcx has quit IRC07:44
*** yingjun has quit IRC07:47
*** yingjun has joined #openstack-nova07:47
*** willise has left #openstack-nova07:48
*** hfu has joined #openstack-nova07:48
*** hfu_ has quit IRC07:49
*** amoralej|off is now known as amoralej07:50
*** adrian_otto has quit IRC07:55
*** hoangcx has joined #openstack-nova07:55
*** Jinxing has quit IRC07:57
*** yingjun has quit IRC07:59
*** vsaienko has joined #openstack-nova08:02
*** yamamoto has joined #openstack-nova08:03
*** adrian_otto has joined #openstack-nova08:08
*** esberglu has joined #openstack-nova08:08
*** esberglu has quit IRC08:13
*** pcaruana has joined #openstack-nova08:14
*** pcaruana has quit IRC08:20
*** Jinxing has joined #openstack-nova08:21
*** ltomasbo|away is now known as ltomasbo08:21
*** adrian_otto has quit IRC08:21
*** pcaruana has joined #openstack-nova08:22
*** ijw has quit IRC08:23
*** thorst_ has joined #openstack-nova08:25
*** tyrola has quit IRC08:29
*** thorst_ has quit IRC08:30
*** tyrola has joined #openstack-nova08:30
*** yamamoto has quit IRC08:32
*** yamamoto has joined #openstack-nova08:33
*** jpena|away is now known as jpena08:35
*** yamamoto has quit IRC08:37
openstackgerritSergey Nikitin proposed openstack/nova master: Replace diagnostics objects with Nova diagnostics objects  https://review.openstack.org/39448008:39
openstackgerritSergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API.  https://review.openstack.org/35554008:39
openstackgerritSergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen  https://review.openstack.org/39961308:39
openstackgerritSergey Nikitin proposed openstack/nova master: Added nova objects for intance diagnostics  https://review.openstack.org/43474508:39
*** owalsh-afk has quit IRC08:44
*** yufei has joined #openstack-nova08:47
*** avolkov has joined #openstack-nova08:48
*** jpena is now known as jpena|away08:48
*** ralonsoh has joined #openstack-nova08:49
openstackgerritAnh Tran proposed openstack/nova master: Remove mox stubs in api/openstack/compute/test_server_reset_state.py  https://review.openstack.org/32082608:52
*** fragatin_ has joined #openstack-nova08:53
*** jpena|away is now known as jpena08:53
openstackgerritHuan Xie proposed openstack/nova master: Fix live migrate with XenServer  https://review.openstack.org/42442808:55
*** fragatina has quit IRC08:57
*** fragatin_ has quit IRC08:58
*** vsaienko has quit IRC08:59
*** vsaienko has joined #openstack-nova08:59
*** vsaienko has quit IRC08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:00
*** hfu has quit IRC09:03
openstackgerritJens Rosenboom proposed openstack/nova master: Fix novncproxy for python3  https://review.openstack.org/43466609:06
*** hfu has joined #openstack-nova09:06
*** yamahata has quit IRC09:06
*** aarefiev_afk is now known as aarefiev09:09
*** derekh has joined #openstack-nova09:15
*** lpetrut has joined #openstack-nova09:15
*** yamamoto has joined #openstack-nova09:17
*** yizhongyin has joined #openstack-nova09:19
*** yamamoto has quit IRC09:19
*** yizhongyin has left #openstack-nova09:19
*** yamamoto has joined #openstack-nova09:20
*** yamahata has joined #openstack-nova09:21
*** yamamoto has quit IRC09:25
*** yamahata has quit IRC09:30
openstackgerritSergey Nikitin proposed openstack/nova master: Replace diagnostics objects with Nova diagnostics objects  https://review.openstack.org/39448009:30
openstackgerritSergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API.  https://review.openstack.org/35554009:30
openstackgerritSergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen  https://review.openstack.org/39961309:30
*** openstackgerrit has quit IRC09:32
*** lucas-afk is now known as lucasagomes09:35
*** diga has quit IRC09:38
*** diga has joined #openstack-nova09:39
*** gcb has quit IRC09:44
*** gcb has joined #openstack-nova09:44
*** Jinxing has quit IRC09:49
*** hfu_ has joined #openstack-nova09:51
*** hfu has quit IRC09:51
*** claudiub has quit IRC09:53
*** Hosam has quit IRC09:55
*** hfu_ has quit IRC09:55
*** Hosam has joined #openstack-nova09:55
*** openstackgerrit has joined #openstack-nova09:55
openstackgerritjichenjc proposed openstack/python-novaclient master: Avoid limit=-1 description  https://review.openstack.org/43478109:55
openstackgerritjichenjc proposed openstack/python-novaclient master: Avoid limit=-1 description  https://review.openstack.org/43478109:56
BobBallmriedem_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 IRC09:58
*** tovin07 has quit IRC09:58
*** esberglu has joined #openstack-nova09:58
*** liuyulong_ is now known as liuyulong09:58
*** Hosam has quit IRC10:00
*** amotoki_ has quit IRC10:01
*** udesale has quit IRC10:03
*** esberglu has quit IRC10:03
*** ralonsoh_ has joined #openstack-nova10:04
*** ralonsoh has quit IRC10:07
*** sambetts|afk is now known as sambetts10:09
*** amotoki has joined #openstack-nova10:13
*** iceyao has quit IRC10:14
*** mvk has quit IRC10:17
*** erlon has joined #openstack-nova10:18
*** amotoki has quit IRC10:20
*** yamamoto has joined #openstack-nova10:22
*** ralonsoh_ is now known as ralonsoh10:23
*** udesale has joined #openstack-nova10:26
*** thorst_ has joined #openstack-nova10:26
*** mirrorbox has quit IRC10:28
*** satyar has quit IRC10:29
*** sam_nowitzki has quit IRC10:31
*** thorst_ has quit IRC10:31
*** mirrorbox has joined #openstack-nova10:31
*** mirrorbox has joined #openstack-nova10:31
*** tuanluong has quit IRC10:34
*** amotoki has joined #openstack-nova10:35
*** hoangcx has quit IRC10:36
*** yufei has quit IRC10:36
*** kevinz has quit IRC10:38
*** yamamoto has quit IRC10:40
*** edand has quit IRC10:40
*** cdent has joined #openstack-nova10:42
*** amotoki has quit IRC10:42
*** markus_z has quit IRC10:42
*** mirrorbox has quit IRC10:44
*** owalsh has joined #openstack-nova10:44
*** mvk has joined #openstack-nova10:52
*** mirrorbox has joined #openstack-nova10:52
*** dikonoor has quit IRC10:58
*** ekuris has quit IRC11:00
*** takashin has joined #openstack-nova11:01
openstackgerritJohn Garbutt proposed openstack/nova-specs master: Add polcy-docs spec  https://review.openstack.org/43301011:03
*** udesale has quit IRC11:03
*** sree has quit IRC11:04
*** tbachman has joined #openstack-nova11:05
*** nicolasbock has joined #openstack-nova11:05
*** sdague has joined #openstack-nova11:06
*** Hosam has joined #openstack-nova11:09
*** phuongnh has quit IRC11:11
*** ekuris has joined #openstack-nova11:13
*** claudiub has joined #openstack-nova11:14
openstackgerritSergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API.  https://review.openstack.org/35554011:15
*** jaosorior has quit IRC11:16
*** jaosorior has joined #openstack-nova11:17
*** sridharg has quit IRC11:17
*** karthiks has quit IRC11:18
*** sandanar__ has quit IRC11:25
*** sudipto has quit IRC11:25
*** sudipto_ has quit IRC11:25
*** lucasagomes is now known as lucas-bbl11:26
*** dikonoor has joined #openstack-nova11:26
*** abalutoiu has joined #openstack-nova11:30
*** rfolco has joined #openstack-nova11:30
*** tbachman has quit IRC11:31
*** ralonsoh has quit IRC11:31
openstackgerritJohn Garbutt proposed openstack/nova master: POC: add descriptions to server tags  https://review.openstack.org/43484211:34
openstackgerritJohn Garbutt proposed openstack/nova master: POC: add descriptions to server tags  https://review.openstack.org/43484211:35
openstackgerritRoman Podoliaka proposed openstack/nova master: db: remove unused EngineFacadeFixture  https://review.openstack.org/43484311:35
*** tbachman has joined #openstack-nova11:35
*** nmathew has quit IRC11:37
*** yamamoto has joined #openstack-nova11:38
*** markus_z has joined #openstack-nova11:45
*** databus23_ has quit IRC11:46
*** sapcc-bot has quit IRC11:46
*** carthaca_ has quit IRC11:46
*** sapcc-bot1 has joined #openstack-nova11:46
*** databus23_ has joined #openstack-nova11:46
*** karthiks has joined #openstack-nova11:46
*** dgonzalez_ has joined #openstack-nova11:46
*** carthaca_ has joined #openstack-nova11:46
*** david_1 has joined #openstack-nova11:46
*** owalsh has quit IRC11:46
*** sridharg has joined #openstack-nova11:46
*** yamamoto has quit IRC11:47
*** esberglu has joined #openstack-nova11:47
*** cdent has quit IRC11:48
*** databus23_ has quit IRC11:48
*** dgonzalez_ has quit IRC11:48
*** david_1 has quit IRC11:48
*** carthaca_ has quit IRC11:48
*** tbachman has quit IRC11:50
*** esberglu has quit IRC11:52
*** Hosam has quit IRC11:58
*** dikonoor has quit IRC12:00
*** sapcc-bot1 has quit IRC12:03
*** sapcc-bot has joined #openstack-nova12:03
*** david_1 has joined #openstack-nova12:03
*** databus23_ has joined #openstack-nova12:03
*** carthaca_ has joined #openstack-nova12:03
*** dgonzalez_ has joined #openstack-nova12:03
*** salv-orlando has joined #openstack-nova12:04
*** Hosam has joined #openstack-nova12:04
*** fandi has joined #openstack-nova12:04
*** jpena is now known as jpena|lunch12:05
*** david_1 has quit IRC12:05
*** databus23_ has quit IRC12:05
*** carthaca_ has quit IRC12:05
*** dgonzalez_ has quit IRC12:05
*** psachin has joined #openstack-nova12:06
*** dave-mccowan has joined #openstack-nova12:06
*** diga has quit IRC12:06
*** Anticime1 is now known as Anticimex12:07
*** salv-orl_ has quit IRC12:07
*** fandi has quit IRC12:09
*** fandi has joined #openstack-nova12:09
*** ratailor has quit IRC12:10
*** bvanhav has joined #openstack-nova12:12
*** catintheroof has joined #openstack-nova12:14
betherlyhi 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-nova12:16
sdaguebetherly: devstack questions probably better for #openstack-qa, we can do the debug over there if you like12:19
betherlysdague: many thanks! ill ask again in that channel!12:20
*** gszasz has joined #openstack-nova12:22
*** sudipto_ has joined #openstack-nova12:24
*** sudipto has joined #openstack-nova12:24
*** yamamoto has joined #openstack-nova12:25
openstackgerritJohn Garbutt proposed openstack/nova master: POC: add descriptions to server tags  https://review.openstack.org/43484212:26
*** yuntongjin has joined #openstack-nova12:28
*** takashin has left #openstack-nova12:31
*** gongysh has quit IRC12:31
*** dikonoor has joined #openstack-nova12:36
*** sudipto has quit IRC12:39
*** sudipto_ has quit IRC12:39
*** smatzek has joined #openstack-nova12:41
*** jerrygb has joined #openstack-nova12:42
*** jerrygb has quit IRC12:42
*** esberglu has joined #openstack-nova12:42
*** jerrygb has joined #openstack-nova12:43
*** thorst_ has joined #openstack-nova12:44
*** smatzek has quit IRC12:45
*** smatzek has joined #openstack-nova12:45
*** edmondsw has joined #openstack-nova12:46
*** esberglu has quit IRC12:47
*** tbachman has joined #openstack-nova12:52
*** mdnadeem has quit IRC12:56
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova master: Port binding based on events during live migration  https://review.openstack.org/43487012:57
*** diga has joined #openstack-nova13:00
*** lucas-bbl is now known as lucasagomes13:01
*** gcb has quit IRC13:01
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova master: Port binding based on events during live migration  https://review.openstack.org/43487013:02
*** jpena|lunch is now known as jpena13:04
*** fandi has quit IRC13:09
*** fandi has joined #openstack-nova13:10
*** tblakes has joined #openstack-nova13:11
*** Hosam has quit IRC13:11
*** kevinz has joined #openstack-nova13:12
*** lucasagomes is now known as lucas-brb13:13
*** mtanino has joined #openstack-nova13:14
*** gouthamr has joined #openstack-nova13:15
*** jwcroppe has quit IRC13:15
*** ekuris has quit IRC13:18
*** dave-mccowan has quit IRC13:19
*** catintheroof has quit IRC13:20
*** diga has quit IRC13:20
*** kevinz has quit IRC13:21
*** kevinz has joined #openstack-nova13:21
*** diana_clarke has joined #openstack-nova13:25
*** iceyao has joined #openstack-nova13:25
*** esberglu has joined #openstack-nova13:26
*** bkopilov_ has quit IRC13:27
*** bkopilov has quit IRC13:27
*** jwcroppe has joined #openstack-nova13:29
*** mriedem_afk is now known as mriedem13:29
*** Jeffrey4l has quit IRC13:30
mriedemBobBall: 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-nova13:31
*** ekuris has joined #openstack-nova13:32
mriedemBobBall: 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 pike13:32
mriedemBobBall: also the xenserver ci is failing on https://review.openstack.org/#/c/424428/13:33
BobBallmriedem: 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
mriedemsure, but a simple in check to avoid the KeyError is pretty simple right?13:34
BobBallIndeed.  Avoiding the KeyError is trivial - I just can't promise it will work13:35
BobBallCI was broken by an update to Cirros version today... but I thought it was resolved now.13:35
*** salv-orlando has quit IRC13:36
*** psachin has quit IRC13:39
*** links has quit IRC13:39
*** ralonsoh has joined #openstack-nova13:40
*** sudipto_ has joined #openstack-nova13:41
*** sudipto has joined #openstack-nova13:41
*** psachin has joined #openstack-nova13:41
*** kevinz has quit IRC13:42
mriedemDetails: {u'message': u"Cannot 'createImage' instance e0197e39-f124-4f50-980a-d60c0cfef623 while it is in task_state image_uploading", u'code': 409}13:42
mriedemyeah looks unrelated13:42
*** gouthamr has quit IRC13:42
*** kevinz has joined #openstack-nova13:42
*** yamamoto has quit IRC13:43
*** gouthamr has joined #openstack-nova13:43
*** fandi has quit IRC13:45
mriedemas you say maybe we just handle the KeyError in a follow up patch and backport it, let me see the diff from my review13:45
*** belmoreira has joined #openstack-nova13:46
BobBallI've just fixed the key error (inline) can can push a patch - just running pep8 to make sure I haven't done something stupid13:47
BobBallAs 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
BobBallSorry 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 reports13:48
BobBallHappy to go with whatever route you recommend on this one ofc13:48
*** yamamoto has joined #openstack-nova13:49
*** lucas-brb is now known as lucasagomes13:49
*** xyang1 has joined #openstack-nova13:50
*** mtanino has quit IRC13:51
mriedemBobBall: +2 on it now13:52
mriedemi left comments but those can be dealt with in a follow up13:52
BobBallI'll put the KeyError fix in a follow up.13:52
mriedemjohnthetubaguy: are you still happy with https://review.openstack.org/#/c/424428/ ?13:52
mriedemBobBall: i'm good if you want to push that change up now too13:53
*** xyang1 has quit IRC13:53
*** yamamoto has quit IRC13:53
*** bvanhav_ has joined #openstack-nova13:54
*** burt has joined #openstack-nova13:54
*** bkopilov_ has joined #openstack-nova13:54
*** bkopilov has joined #openstack-nova13:54
BobBallmriedem: 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 IRC13:54
mriedemBobBall: the default vif_driver change won't be backported13:54
*** bvanhav has quit IRC13:55
mriedemthe keyerror thing could be for ocata, so i'd leave those separate13:55
BobBallWill do.13:55
*** gongysh has joined #openstack-nova13:55
mriedembauzas: 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 update13:56
*** mdrabe has joined #openstack-nova13:56
mriedem*fracas13:56
johnthetubaguymriedem: 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 it13:57
mriedemjohnthetubaguy: sounds like BobBall has a fix locally for the keyerror thing which could go into that patch now if we want13:57
BobBallBTW - 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 though13:57
mriedemdoesn't matter to me, might as well if it's ready13:57
johnthetubaguymriedem: yeah, duno, just a .get() would fix it13:58
*** tlian has joined #openstack-nova13:58
*** yuntongjin has quit IRC13:58
mriedemjohnthetubaguy: that's true, it would result in the exception13:58
*** zz_dimtruck is now known as dimtruck13:58
BobBallI'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
bauzasp!spy13:58
johnthetubaguyyeah13:58
mriedemBobBall: the existing "if neutron_id not in vif_uuid_map.keys():" will handle the None case13:59
johnthetubaguy+1 mriedem13:59
mriedemi think the get() is all we need13:59
lyarwoodmriedem: hey sorry, yeah I think that was more because someone missed that the migration had already landed before we GA'd13:59
*** sudipto has quit IRC13:59
*** sudipto_ has quit IRC13:59
BobBallWell a dictionary *could* have None in the keys... but I agree that's very unlikely.13:59
BobBalland would be indicative of a totally different bug.13:59
mriedemi don't think it would happen in this case because of https://review.openstack.org/#/c/424428/11/nova/virt/xenapi/vmops.py@239014:00
BobBallI 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 pass14:00
mriedemi'm fine if you want to be cautious14:00
*** liangy has joined #openstack-nova14:01
*** eharney has joined #openstack-nova14:01
BobBallSo shall I post a follow-up or a replacement?14:01
lyarwoodmriedem: https://review.openstack.org/#/c/408725/ right?14:01
mriedemlyarwood: yeah14:02
mriedemBobBall: might as well do a replacement14:02
mriedemnew patch set on the existing change14:02
mriedemi'll be around to shephard it into ocata14:02
bauzasmriedem: well, not sure if it would be a .z version14:03
*** burgerk has joined #openstack-nova14:03
BobBallWill do.  Just running another couple of tests to make sure I didn't do something really stupid.14:04
mriedemit's not a new feature14:04
bauzasmriedem: in case we need more than just upgrading the packages14:04
bauzasmriedem: yeah, I know14:04
mriedembauzas: it's basically there to catch any upgrade issues from mitaka to newton14:04
bauzasmriedem: but I wonder how we could signal that just upgrading is not enough14:04
mriedemnot from upgrading from 14.0.3 to 14.0.414:04
mriedembecause if you hit this bug on upgrade to newton you're already broken14:04
lyarwoodbauzas: we can document this in our own releasenotes tbh14:04
*** gszasz has joined #openstack-nova14:04
bauzasmmmm14:04
mriedembauzas: upgrading is enough, this is the 13.0.0->14.0.4 case14:05
mriedemnot 14.0.3->14.0.414:05
bauzasI see14:05
bauzaswell14:05
lyarwoodyeah I'm fine with this being in a 14.0.4 release tbh14:05
bauzasmriedem: then okay14:05
openstackgerritThomas Bechtold proposed openstack/nova master: Log exception when hypervisor version check fails  https://review.openstack.org/43492714:05
bauzasmriedem: if people just do 14.0.3 to .4, they don't need to call the online data migration, yup ? If so, +1 with me14:05
*** markus_z has quit IRC14:06
*** kberger has quit IRC14:06
bauzassorry, I misunderstood the impact then14:06
*** kberger has joined #openstack-nova14:06
mriedemthat's fine, this was a confusing one14:08
*** rmart04 has joined #openstack-nova14:09
*** sree has joined #openstack-nova14:09
*** Jinxing has joined #openstack-nova14:11
mriedemBobBall: 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 later14:13
*** mriedem has quit IRC14:13
BobBallThanks 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-nova14:14
*** baoli has joined #openstack-nova14:15
*** kevinz has quit IRC14:16
*** kevinz has joined #openstack-nova14:16
*** owalsh has joined #openstack-nova14:18
*** mlavalle has joined #openstack-nova14:19
*** nicodemus_ has joined #openstack-nova14:20
nicodemus_hello14:21
openstackgerritArtom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment  https://review.openstack.org/41562214:21
openstackgerritArtom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment  https://review.openstack.org/41810014:21
openstackgerritArtom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment  https://review.openstack.org/39194114:21
openstackgerritArtom Lifshitz proposed openstack/nova master: API support for tagged device attachment  https://review.openstack.org/37602114:21
*** adrian_otto has joined #openstack-nova14:21
openstackgerritArtom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment  https://review.openstack.org/41552014:21
nicodemus_I have a question regarding https://bugs.launchpad.net/nova/+bug/164618114:21
openstackLaunchpad 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-nova14: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 IRC14:23
*** tbachman has quit IRC14:23
*** liverpooler has joined #openstack-nova14:23
*** adrian_otto has quit IRC14:24
*** gcb has joined #openstack-nova14:26
bhagyashrisalex_xu: Hi,14:27
*** felipemonteiro_ has joined #openstack-nova14:27
*** tbachman has joined #openstack-nova14:29
*** sudipto_ has joined #openstack-nova14:29
*** sudipto has joined #openstack-nova14:29
bhagyashrisalex_xu:  I want to discuss regarding the backup api.  have free time?14:30
*** READ10 has joined #openstack-nova14:35
*** sree has quit IRC14:35
*** smatzek has quit IRC14:36
kashyapnicodemus_: If you have a test env. mind trying it and see?14:36
kashyapnicodemus_: 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 try14:38
nicodemus_I've applied the fix when the customer reported the issue, but the issue seems to still be present with very large images14:39
nicodemus_(it's a Mitaka deployment)14:39
*** gszasz has quit IRC14:39
kashyapnicodemus_: Yeah, in that case, I'm curious what you'll see if you increase the timeout.14:41
*** esberglu has quit IRC14:43
*** esberglu has joined #openstack-nova14: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
kashyapnicodemus_: Can you please make a comment with full details in the bug, once you complete your test?14:45
nicodemus_kashyap, sure14:45
*** burgerk has quit IRC14:45
*** mkoderer has joined #openstack-nova14:45
*** hongbin has joined #openstack-nova14:48
*** esberglu has quit IRC14:48
*** yamamoto has joined #openstack-nova14:50
*** kaisers_ has joined #openstack-nova14:51
*** dave-mccowan has joined #openstack-nova14:54
*** cfriesen has joined #openstack-nova14:54
*** iceyao has quit IRC14:56
*** yamamoto has quit IRC14:57
*** shardy has joined #openstack-nova14:58
*** esberglu has joined #openstack-nova14:58
*** xyang1 has joined #openstack-nova14:58
openstackgerritBob Ball proposed openstack/nova master: Fix live migrate with XenServer  https://review.openstack.org/42442814:59
*** mriedem has joined #openstack-nova14:59
BobBallSorry that took so long johnthetubaguy ^^ my live migrate env got broke by upgrading devstack and the slave not being a placement-client... :)14:59
mriedemjohnthetubaguy: BobBall: wild thought, but we could take this opportunity to deprecate the xen vif_driver option in ocata14:59
*** jose-phillips has joined #openstack-nova15:00
mriedemBobBall: johnthetubaguy: my thinking is, deprecate in ocata, remove in pike, and pick the vif_driver based on the use_neutron boolean15:00
mriedemwe 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 ocata15:00
*** markmcclain has quit IRC15:00
*** smatzek has joined #openstack-nova15:00
mriedemso the xen vif driver config default is broken (by default)15:00
*** annegentle has joined #openstack-nova15:00
*** markmcclain has joined #openstack-nova15:00
johnthetubaguyI would have through bridge and ovs work with both, I guess thats not true?15:01
johnthetubaguythought15:01
*** jose-phillips has quit IRC15:01
mriedemlike neutron + linuxbridge?15:02
mriedemi'm told the xen vif bridge driver is deprecated and only for nova-net15:02
*** nicodemus_ has quit IRC15:03
*** gszasz has joined #openstack-nova15: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 mdbooth15:04
*** annegentle has quit IRC15:05
mdboothlyarwood: 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
owalshmriedem: FYI https://bugs.launchpad.net/nova/+bug/1665370 - the newton & ocata api db migrations conflict15:06
openstackLaunchpad bug 1665370 in tripleo "api db_sync fails on upgrade due to duplicate migration files" [Critical,Triaged] - Assigned to Steven Hardy (shardy)15:06
mdboothIn either case you should be able to create, but the latter case would still be prevented.15:06
mdboothlyarwood: Unless I missed something, which is possible. Still looking.15:06
*** annegentle has joined #openstack-nova15:06
*** markus_z has quit IRC15:06
mriedemowalsh: guh, pretty soon i'm in meetings for the next 2 hours15:07
*** mtanino has joined #openstack-nova15:07
mriedemdansmith: melwitt: can you take a look at ^?15:07
mdboothThe code which populates this from the driver gets it directly from the storage, so my current working assumption is an incorrect interim calculation in resourcemanager15:07
* mdbooth is looking for the code which does that15:07
owalshmriedem: should be a simple fix, I can push a patch15:08
mriedemowalsh: i'm not sure we can just start renaming db migrations15:08
dansmithowalsh: yeah, hang on a sec15:08
dansmithnot sure how this could have happened15:09
mriedemright, how would grenade not explode?15:09
mriedemowalsh: it's 21 in stable/newton https://github.com/openstack/nova/tree/stable/newton/nova/db/sqlalchemy/api_migrations/migrate_repo/versions15:09
mriedemand 28 in ocata https://github.com/openstack/nova/tree/stable/ocata/nova/db/sqlalchemy/api_migrations/migrate_repo/versions15:10
dansmithmriedem: because grenade uses old/ and new/15:10
dansmithmriedem: it's because you didn't put it in the placeholder in master too and then backport *that*15:10
dansmithI should have caught that15:11
*** rmart04 has quit IRC15:11
*** ltomasbo is now known as ltomasbo|away15:11
mriedemoh i see15:11
mriedem21 in master is the issue15:11
dansmithowalsh: so you were going to copy the migration the same place in ocata and remove that placeholder right?15:11
mriedemok bug is marked for ocata rc215:12
mriedemi'm entering the roundtable danger zone, bbiab15:12
owalshdansmith: yea, assuming it's not too late to rename ocata migrations15:13
mriedemwe can rename it and backport it15:13
*** ltomasbo|away is now known as ltomasbo15:13
dansmithowalsh: so, tbh, I'm not sure we've ever actually done this15:13
dansmithmriedem: that won't really fix people in the field, actually15:14
dansmithmriedem: if they have ocata now, and then update to this, they'll get two migrations for #2115:14
owalshother option is to remove the 021 placeholder in ocata and turn 028 into a no-op15:14
dansmithI guess if they use a proper package manager then it should remove the old one and install the new one15:14
dansmithowalsh: I don't think that helps15:15
*** rmart04 has joined #openstack-nova15:15
*** lucasxu has joined #openstack-nova15:15
dansmithowalsh: and we definitely don't want to do that in case someone was at #27 right now15:15
lyarwoodmdbooth: 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
dansmithowalsh: 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 #21s15:15
lyarwoodmdbooth: ah but that's going to be > 0, I see your point15:16
mdboothlyarwood: That's only a working assumption, btw. Plenty of scope to be entirely wrong :)15:17
*** lucasxu has quit IRC15:18
*** burgerk has joined #openstack-nova15:18
*** Jinxing has quit IRC15:20
cfriesenwhat'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
openstackbug 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 IRC15:20
*** tbachman has quit IRC15:21
owalshdansmith: 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 #2115:22
dansmithowalsh: right, it has to be in both places15:22
dansmithowalsh: but it should be idempotent so that's fine right? otherwise a backport of it doesn't work at all15:23
owalshdansmith: ok, then the only Q is where packaging does the right thing here. I'll take a look15:23
owalshs/where/whether/15:24
*** tbachman has joined #openstack-nova15:24
dansmithowalsh: 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 before15:24
mriedemdansmith: can't we just have the same migration in ocata at 21 and 28?15:28
mriedemit's idempotent15:28
dansmithmriedem: yes15:28
mriedemok15:28
*** sridharg has quit IRC15:29
dansmithmriedem: 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_stuff15:29
dansmiththat's he concern15:29
mriedemsure15:29
mriedemfwiw i don't expect much of that to be the case right now, but i know it's a concern15:29
dansmithsure,  but have to account for it15:30
*** yingjun has quit IRC15:30
*** marst has joined #openstack-nova15:30
mriedemso 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
dansmithno15:31
dansmithI 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 one15:31
dansmitheven if it did, it would use git as its package manager and it would remove the old one during an update I think15:31
mriedemgah15:32
*** sree has joined #openstack-nova15:32
mriedemshould fix that gap at some point but i don't have an easy solution off the top of my head15:32
*** sudipto has quit IRC15:32
*** sudipto_ has quit IRC15:32
dansmithyeah, it's actually weird that the same thing didn't happen with their packages already, actually15:32
*** markus_z has joined #openstack-nova15:33
sdaguedansmith: did we ever actually have a backported migration?15:33
sdagueI know the facility has existed for a while, but it's not regularly used15:33
dansmithsdague: well, that's what I was trying to think ... I feel like we must've had one at least, but I'm not positive15:33
dansmithsdague: yeah, very infrequently if ever15:33
sdagueso, I'm trying to understand the bug - https://bugs.launchpad.net/nova/+bug/166537015:34
openstackLaunchpad bug 1665370 in tripleo "api db_sync fails on upgrade due to duplicate migration files" [Critical,Triaged] - Assigned to Steven Hardy (shardy)15:34
dansmithsdague: if you use python setup.py install,15:34
dansmithsdague: you'll end up with two 21_ migrations, you get that right?15:34
*** lucasxu has joined #openstack-nova15:34
mdboothbauzas: You around?15:35
sdaguedansmith: i'm not sure I do15:35
sdaguebecause won't the new rpm file remove the old one?15:35
mriedemsdague: different names15:36
shardyhey folks, thanks for looking into this - I'm around if you need any more info from my environment15:36
shardyand/or to test any fixes15:36
mriedem021_placeholder in ocata, 021_build_something_fix in newton15:36
dansmithsdague: not rpm, I said setup.py15:36
sdaguedansmith: ok, how do you get the filesystem15:36
sdaguemaybe that's where I'm trying to understand15:36
dansmithsdague: the confusing thing is that I think the rpm should be removing the old one even in their case15:36
sdagueI don't understand how you create a filesystem where they both are there15:37
dansmithsdague: 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 mix15:37
mdboothbauzas: 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
mriedemwe don't pip install nova during grenade?15:37
mriedemi.e. pip install .15:37
sdaguemriedem: we do15:37
dansmithmriedem: we "develop" install it don't we?15:37
dansmithwhich is an egg link15:37
mriedemto put the files in /var/lib/python27/site-packages/nova/..15:38
sdaguemriedem: oh, we do do dev install15:38
bauzasmriedem: I think we pip develop15:38
dansmithyeah so dev install won't catch this15:38
sdaguethat might be isolating15:38
dansmithright15:38
mriedemshart15:38
dansmiththe thing is,15:38
bauzasdevstack too15:38
dansmiththey install from rpms,15:38
bauzasmdbooth: not sure I get your problem15:38
dansmithso they should be removing the old 21 and installing the new one, AFAIK15:38
sdaguedansmith: right15:39
sdaguedansmith: the thing I could imagine is stale pyc files15:39
dansmithsdague: yeah, could be that15:39
dansmithshardy: owalsh ^15:39
owalshdansmith: well, rpm should be clearing out .pyc/pyo files too15:39
mdboothbauzas: I'm assuming that when ComputeManager calls update_instance_info(instance), it somehow updates the host_state data which is ultimately used by DiskFilter15:39
bauzasmdbooth: oh gotcha15:39
mdboothbauzas: Is that correct?15:39
BobBallmriedem johnthetubaguy: I'd be quite happy to deprecate the vif driver config option - BUT - is it used by Rackspace with Quark?15:39
dansmithowalsh: so maybe before we go tweaking stuff in tree we should try to figure out why the cleaning isn't happening?15:40
bauzasmdbooth: yeah, some filters15:40
owalshdansmith: grabbing the bits to create a local rpm now...15:40
mdboothI'm just trying to follow the whole code path, because I suspect there's a bug somewhere in it15:40
shardydansmith, owalsh: actually this could be a packaging issue:15:40
shardyhttp://paste.openstack.org/show/599261/15:40
bauzasmdbooth: not sure about the DiskFilter, but some others15:40
mdboothBut I can't even find the code path :)15:40
bauzasmdbooth: sec then15:40
shardythe new python-nova doesn't seem to have obsoleted the old one15:40
sdaguedansmith: yeh, because I can't imagine there is another way to fix this, those placement backports are important15:40
BobBallmriedem: johnthetubaguy: Also very happy to change the default to the OVS driver15:40
dansmithshardy: maybe building the rpm again in an unclean space or something?15:40
sdagueI guess the only thing you could do is not change 21_ to be the new name15:40
sdagueit just stays placehold15:40
owalshshardy: you mean both ocata & newton are installed?15:41
bauzasmdbooth: found one at least15:41
dansmithsdague: 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 cause15:41
shardydansmith: not sure, these are dlrn packages, I can try locally rebuilding with dlrn to check tho15:41
shardyowalsh: rpm -qa seems to indicate that, which is a surprise15:41
sdaguedansmith: ok, I didn't realize the double master policy15:41
bauzasmdbooth: eg. https://github.com/openstack/nova/blob/master/nova/scheduler/filters/affinity_filter.py#L8915:42
dansmithsdague: we never do this though, so not surprising we didn't do it right15:42
sdagueshardy: yeh, that looks like something unclean about the build15:42
sdaguedansmith: yeh, agreed15:42
shardysdague: yeah, I'll go and take a closer look at that, sorry for not spotting it sooner15:42
sdagueblack swans are what they are15:42
mdboothbauzas: Ah... I was expecting it to use the instance info to update usage totals15:42
shardyI'll leave the bug open for now until I figure it out15:42
mdboothDoes it not do that?15:42
*** gongysh has quit IRC15:43
mdboothAlso, instances is accessed there as a property, so I may have been grepping the wrong thing15:43
mriedemBobBall: i'd like johnthetubaguy's opinion on that too15:44
*** fragatina has joined #openstack-nova15:44
mriedembut it seems the default vif driver should be the ovs one if use_neutron=True by default in ocata15:44
*** faizy has joined #openstack-nova15:45
*** fragatina has quit IRC15:45
*** fragatina has joined #openstack-nova15:46
*** jsho has joined #openstack-nova15:47
mriedemsdague: is there a reason we can't pip install rather than pip develop in a grenade run?15:48
mdboothbauzas: The code in question is here: https://review.openstack.org/#/c/430091/4/nova/scheduler/filters/disk_filter.py15:48
mdboothIt's examining host_state.total_usable_disk_gb15:48
dansmithmriedem: so I'm not sure the dev install is all that unusual,15:48
*** jsho has quit IRC15:48
dansmithmriedem: because the people I know that deploy from git do so in venvs, which is kinda how this behaves15:48
mriedemyeah i thought about venvs15:49
*** jsho has joined #openstack-nova15:49
mriedemthat's what bluebox does15:49
mdboothAFAICT that value is only ever set by a periodic task, and not if ComputeManager is sending updates?15:49
sdaguemriedem: the whole reason for the dev install is so that you can actually hack on and fix things15:49
mdboothbauzas: TBH I'm pretty confused15:49
sdagueit's kind of the purpose of devstack15:49
mriedemsdague: yeah but we wouldn't do that in a CI run15:49
dansmithmriedem: then it's different from local15:49
mriedemseems d-g could tell us to actually install15:49
sdaguemriedem: sure, but then you end up with another difference between CI and local15:49
sdagueand people get all mad why reproduce is not a thing15:50
dansmithyeah, which isn't great, especially if what we have is testing a legit mode15:50
dansmithwhich I think it is because of venvs15:50
mriedemfine fine15:50
mriedemjust spit balling15:50
sdaguemriedem: yeh, sure15:50
mriedemi'll go back to trashing things in this roundtable15:50
bauzasmdbooth: wait, which value ?15:50
bauzasmdbooth: you're showing me something not related to the instance15:51
sdaguethis 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 approximations15:51
mdboothbauzas: Right. I was trying to find an association between those 2 things, and I can't find one.15:51
mriedemat the least i guess we need something doc'ed for this since it was missed15:51
mriedemso in 2 years from now when we do another one of these we can consult the guide15:51
mdboothI was assuming it exists, because otherwise I don't see how that value is updated exception by a heavyweight periodic task.15:51
bauzasmdbooth: so, the HostState is generated/updated at every request in the HostManager15:52
mdboothHowever, that's just assumption on my part.15:52
*** unicell1 has joined #openstack-nova15:52
bauzasand then we pass it to the filter for each compute15:52
*** gcb has quit IRC15:53
*** unicell has quit IRC15:54
mdboothbauzas: We're not calling back to the compute host for each request though, right?15:54
mdboothbauzas: The model is occasionally we get real values by inspecting the compute host, and in between we do a best effort ?15:54
jaypipesI see that mordred has stepped in a pile of mascot poo this morning.15:55
* mriedem hasn't forwarded the nova mascot yet15:56
*** jwcroppe has quit IRC15:57
bauzasmdbooth: I need to go find my daughter15:58
bauzasmdbooth: but let's discuss in 1 hour if I'm there15:58
*** prateek has quit IRC15:59
*** jwcroppe has joined #openstack-nova15:59
*** salv-orlando has joined #openstack-nova16:00
mriedemjaypipes: link me16:02
*** crushil has joined #openstack-nova16:02
*** ekuris has quit IRC16:03
*** mvk has quit IRC16:03
*** rcernin has quit IRC16:03
*** moshele has quit IRC16:03
*** xyang1 has quit IRC16:04
fricklerBobBall: could you take a look at https://review.openstack.org/434666 please? second (different) error after xenserver recheck, both look unrelated to my patch16:06
*** mriedem has quit IRC16:07
*** unicell1 has quit IRC16:07
*** annegentle has quit IRC16:07
*** mriedem has joined #openstack-nova16:08
BobBallfrickler: 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
BobBallThere 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
BobBallfrickler: 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 not16:09
*** des_consolado has joined #openstack-nova16:10
*** pcaruana has quit IRC16:11
*** xyang1 has joined #openstack-nova16:11
*** derekh has quit IRC16:12
*** tesseract has quit IRC16:14
*** faizy has quit IRC16:15
*** derekh has joined #openstack-nova16:15
mordredjaypipes: could soemeone please revoke my email access?16:16
mriedembauzas: https://review.openstack.org/#/c/434552/16:16
mriedemfor rc216:16
mdboothmordred jaypipes: Did I miss the juicy bits? Seemed civil to me.16:17
mriedemnova got lumped in there i saw16:18
mriedemnova 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-nova16:19
mriedembut we don't have a homegrown mascot16:19
*** dimtruck is now known as zz_dimtruck16:19
*** tyrola has quit IRC16:20
*** annegentle has joined #openstack-nova16:21
*** Oku_OS is now known as Oku_OS-away16:21
*** psachin has quit IRC16:21
*** tyrola has joined #openstack-nova16:21
*** Apoorva has joined #openstack-nova16:22
*** ssurana has quit IRC16:23
openstackgerritDan Smith proposed openstack/nova master: Make CellDatabases fixture work over RPC  https://review.openstack.org/43456316:26
openstackgerritDan Smith proposed openstack/nova master: Make start and stop admin actions cells-aware  https://review.openstack.org/43336216:26
*** yamahata has joined #openstack-nova16:26
*** ssurana has joined #openstack-nova16:27
*** kaisers_ has quit IRC16:30
*** yamahata has quit IRC16:31
mriedemsdague: johnthetubaguy: can one of you put this through? https://review.openstack.org/#/c/434552/16:32
mriedemneeded for rc216:32
mriedemdansmith: did you want to get this into docs for rc2 or don't care? https://review.openstack.org/#/c/432482/16:33
fricklerBobBall: will do, thanks for checking16:33
dansmithmriedem: 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 that16:34
dansmithmriedem: but yeah, I'm fine with it going into ocata if you want16:34
sdaguemriedem: bam!16:34
mriedemsweet16:35
*** jwcroppe has quit IRC16:35
sdaguemriedem: so I need to tweak that spec for keystone validation of projects, as the 403 case wasn't really specified well enough16:35
mriedemso 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
mriedemotherwise the xen vif driver is not correct by default out of the box in a fresh ocata install for xen16:35
mriedemsdague: ok16:35
BobBallI'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
mriedemBobBall: johnthetubaguy: i'm +2 on https://review.openstack.org/#/c/424428/16:37
mriedemany changes to the default vif_driver config can be a separate change16:37
mriedemBobBall: do you know if rax is even using an in-tree vif driver for xen?16:37
dansmithmriedem: I thought the logo was going to have some hands on it16:37
mriedemi noticed the vif_driver option is a classloading option so you can load out of tree vif drivers for xen16:37
*** zz_dimtruck is now known as dimtruck16:37
mriedemdansmith: as in the gaping butthole?16:37
mriedemthat's only on twitter16:37
dansmithmriedem: yeah, I must have missed where that was downvoted16:37
dansmithah, maybe so16:37
*** sree has quit IRC16:38
BobBallmriedem: I know RAX use an out-of-tree VIF driver but I'm not certain if they use vif_driver or have custom patches16:38
mriedemok, because i'd like to close the out of tree vif driver support in here via the vif_driver option16:38
mriedemand just make the vif driver decision based on use_neutron16:38
mriedemwe've been closing that window on lots of other configs16:39
mriedemthat would actually allow us to kill the vif_driver option and not use it16:39
mriedemBobBall: you're sure that bridge driver won't work with neutron?16:39
mriedemi won't hold rc2 up for it, anyone using xen i'm sure should already know about this16:40
BobBallmriedem: 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
mriedemwell we know it won't work with live migration and xenserver 7.0 i guess?16:40
BobBallRight.16:41
mriedemi'd kind of like to start the deprecation timer on that option in ocata if possible16:41
mriedemso we don't lose 6 months16:42
mriedemBobBall: who from rax might know what they use?16:42
BobBallTotally agreed - what should we do?  Log message if anyone uses the function?  Update help on the vif_driver option?16:42
*** annegentle has quit IRC16:42
BobBalljohnthetubaguy: will know I'm sure - he's just been AWOL for a few hours ;)16:42
mriedemlet me post my idea16:42
johnthetubaguyhe is here16:43
johnthetubaguyrackspace is neutron, with the quark plugin16:43
BobBallDoes you set vif_driver to the quark plugin?16:43
johnthetubaguyuses ovs16:43
BobBallor can we deprecate the vif_driver option in nova?16:43
openstackgerritSujitha proposed openstack/nova master: Adding tags field to InstancePayload  https://review.openstack.org/40722816:44
*** annegentle has joined #openstack-nova16:44
*** andreas_s has quit IRC16:44
johnthetubaguyBobBall: seems fine to me16:44
mriedemheh the config option says this:16:45
mriedem"NOTE: The XenAPIBridgeDriver should be used for running OVS or Bridge in XenServer."16:45
mriedemi guess that's totally wrong?16:45
BobBallheh... Indeed.16:45
*** belmoreira has quit IRC16:46
BobBallfrickler: 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 IRC16:49
BobBallmriedem: 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
mriedemBobBall: that's fine, i know it's not failing on this patch16:50
BobBallIndeed.  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
openstackgerritSean Dague proposed openstack/nova master: WIP: verify project_id when quotas are checked  https://review.openstack.org/43501016:53
sdaguemriedem: that kind of works, no tests, I'll update the spec with the differences16:54
*** moshele has joined #openstack-nova16:54
*** mdrabe has quit IRC16:55
*** markus_z has quit IRC16:56
mriedemBobBall: changing the default vif driver causes all sorts of unit test failure of course16:57
*** jwcroppe has joined #openstack-nova16:58
BobBallmriedem: 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
mriedemyeah 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 now16:59
mriedemi'm trying to find a spot to do that16:59
openstackgerritSean Dague proposed openstack/nova-specs master: Updates for validation spec based on implementation  https://review.openstack.org/43501616:59
sdaguemriedem: ^^^17:00
*** mdrabe has joined #openstack-nova17:01
*** pbandark has joined #openstack-nova17:01
*** baoli has quit IRC17:01
*** derekh has quit IRC17:01
*** liusheng has quit IRC17:02
cfriesenfor nova microversion 2.42 is there going to be any work required in novaclient?17:03
mriedemcfriesen: it's a bug fix, andreykurilin had a patch17:04
mriedemhttps://review.openstack.org/#/c/424745/17:04
*** derekh has joined #openstack-nova17:05
*** thorst_ is now known as thorst_afk17:05
andreykurilinmriedem: also there is a patch for 2.42 - https://review.openstack.org/#/c/429512/ . I suppose we need to merge 424745 change with 42951217:06
mdboothbauzas: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L1049-L105317:06
mdboothbauzas: That's the info that eventually ends up in the scheduler, right?17:06
mriedemBobBall: got a workaround for the tests, pushing the change up shortly17:06
BobBallAh awesome - I was just looking into that.17:07
mdboothIf 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 disks17:07
mdboothAlso, it takes no account of swap17:07
mdboothOr kernel/ramdisk17:07
*** rmart04 has quit IRC17:07
mdboothbauzas: Can you sanity check^^^?17:07
*** bvanhav_ has quit IRC17:08
*** moshele has quit IRC17:08
*** moshele has joined #openstack-nova17:08
cfriesenmriedem: 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_dimtruck17:09
andreykurilincfriesen: unfortunately, I'll not have an ability to finish my change till March :(17:10
*** baoli has joined #openstack-nova17:10
cfriesenandreykurilin: 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
andreykurilincfriesen: it should be handled here - https://review.openstack.org/#/c/429512/17:12
*** bvanhav_ has joined #openstack-nova17:12
cfriesenandreykurilin: 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 IRC17:15
cfriesenwon't use tags, I mean, even though they're supported on the server side17:15
andreykurilincfriesen: change 424745 should clarify situation for microversions <2.42 . The change 429512 should return support of tags with 2.4217:16
cfriesen429512 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 IRC17:18
*** jwcroppe has joined #openstack-nova17:18
cfriesenso 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-nova17:20
*** jwcroppe has quit IRC17:23
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate xenserver.vif_driver config option and change default  https://review.openstack.org/43503717:24
mriedemBobBall: johnthetubaguy: ^17:24
mriedemdansmith: sdague: ^ let me know if you think that's wacky17:24
mriedembut i kind of want to get that into ocata to set the timer17:24
*** moshele has quit IRC17:24
sdaguemriedem: I'd be fine with it. I think that given the lower level of xenserver usage, things like that can move a bit more nimbly17:27
dansmithmriedem: why is it whacky?17:27
cfriesenandreykurilin: 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
dansmither, wacky.. heh17:27
mriedemdansmith: just b/c i want to get it into ocata17:27
mriedemnot really wacky17:27
mriedemcfriesen: huh?17:28
dansmithseems decidedly unwacky to me17:28
mriedemcfriesen: novaclient is frozen for ocata17:28
mriedemexcept for backporting fixes17:28
*** Apoorva_ has joined #openstack-nova17:28
*** derekh has quit IRC17:28
sdagueBobBall: mriedem: why is this failing xenserver ci? - https://review.openstack.org/#/c/424428/1217:28
cfriesenmriedem: so we have nova with max version 2.42 but novaclient with 2.41?17:29
mriedemsdague: everything is17:29
cfriesenmriedem: and no way to make use of the fact that 2.42 fixes tags?17:29
mriedemknown issue17:29
mriedemcfriesen: novaclient will work with 2.4217:29
*** Apoorva has quit IRC17:29
mriedemoh i guess we didn't set the max api thing in novaclient17:30
cfriesenAPI_MAX_VERSION = api_versions.APIVersion("2.41")17:30
cfriesenthat's what https://review.openstack.org/#/c/429512  does17:30
mriedemyeah, well, we can't release novaclient for ocata right now17:30
mriedemcfriesen: i think we'll just need to backport that as a bug fix later17:30
mriedemto stable/ocata17:30
sdaguemriedem: ok, so are we waiting for it to work?17:31
sdagueor is it safe to push it in17:31
mriedemsdague: it's safe17:31
mriedemthe xenserver ci uses the ovs vif driver already17:32
mriedemi checked that earlier17:32
mriedemb/c the xenserver ci on master is neutron, and has to use the ovs driver17:32
sdaguemriedem: eta on xenserver ci coming back to life?17:34
mriedemsdague: (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 yesterd17: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
openstackgerritRoman Podoliaka proposed openstack/nova-specs master: Strict isolation of group of hosts for image and flavor  https://review.openstack.org/38191217:35
*** jamesdenton has joined #openstack-nova17:35
mriedemsdague: so maybe results within the hour or two?17:36
mriedemsdague: keeping in mind i need to backport this to ocata and get it merged there before we can cut rc2 today17:36
mriedemsdague: oh you got the bottom change too, thanks17:37
mriedemfwiw john was +2 on that yesterday17:37
mriedemi just found some corner cases that needed updating, so the bottom change is safe17:38
sdaguemriedem: yeh17:38
*** shardy is now known as shardy_afk17:40
*** jamesdenton has quit IRC17:40
*** karthiks has quit IRC17:40
*** owalsh has quit IRC17:41
*** jschluet is now known as jschlueter17:45
*** owalsh has joined #openstack-nova17:48
*** rmart04 has joined #openstack-nova17:50
*** zz_dimtruck is now known as dimtruck17:50
mriedembackports are up for those xen patches17:51
mriedemgoing to lunch17:51
openstackgerritBen Nemec proposed openstack/nova-specs master: Configurable PXE Boot for Instances  https://review.openstack.org/43505217:51
*** jpena is now known as jpena|away17:52
*** mriedem1 has joined #openstack-nova17:53
*** lucasxu has quit IRC17:53
*** cdent has quit IRC17:54
*** tyrola has quit IRC17:54
*** lucasxu has joined #openstack-nova17:55
*** mriedem has quit IRC17:55
*** tyrola has joined #openstack-nova17:55
*** chyka has joined #openstack-nova17:57
*** sudipto_ has joined #openstack-nova17:59
*** sudipto has joined #openstack-nova17:59
*** Apoorva_ has quit IRC17:59
*** felipemonteiro_ has joined #openstack-nova17:59
*** lpetrut has joined #openstack-nova18:00
*** mvk has joined #openstack-nova18:02
*** armax has joined #openstack-nova18:02
*** lpetrut has quit IRC18:03
*** salv-orl_ has joined #openstack-nova18:04
*** salv-orlando has quit IRC18:07
openstackgerritSujitha proposed openstack/nova master: Change tags to default field in Instance object.  https://review.openstack.org/41529818:12
openstackgerritMarkMielke proposed openstack/nova master: DiskFilter should only apply when storage is requested  https://review.openstack.org/43009118:12
*** lucasagomes is now known as lucas-afk18:14
*** rmart04 has quit IRC18:14
*** rmart04 has joined #openstack-nova18:15
*** jamesdenton has joined #openstack-nova18:16
*** lpetrut has joined #openstack-nova18:17
*** sambetts is now known as sambetts|afk18:18
openstackgerritJay Pipes proposed openstack/nova master: placement: get inventory shared with provider  https://review.openstack.org/43397818:18
openstackgerritJay Pipes proposed openstack/nova master: placement: rename can_host to shared  https://review.openstack.org/43397718:18
*** slaweq has quit IRC18:19
*** jamesdenton has quit IRC18:21
*** slaweq has joined #openstack-nova18:21
jaypipesmriedem1: 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
openstackLaunchpad 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 mriedem18:25
jaypipesmriedem1: I guess I'm wondering why internal state is being exposed out of Cinder's API, then?18:25
mriedemjaypipes: 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 side18:26
*** Swami_ has joined #openstack-nova18:26
mriedemthe resulting error from the nova side is the same either way18:26
jaypipesmriedem: k.18:26
mriedemunreserve on the cinder side is idempotent18:26
mriedemi think all of that old attach status checking is probably from nova-volume18:27
jaypipesmriedem: cool (as it should be)18:27
jaypipesthe idempotency, not the nova-volume :)18:27
*** unicell has joined #openstack-nova18:27
mriedemright18:28
jaypipesmriedem: in any case, I'm reviewing the patch now...18:28
mriedemcool, it's going to be discussed next week too, and i've sort of started thinking a spec for this might be nice18:29
mriedemsince it's pretty invasive about how much of the machinery it's touching18:29
mriedemapi, conductur, upgrades, bfv, etc18:29
*** satyar has joined #openstack-nova18:30
*** kaisers_ has joined #openstack-nova18:31
mriedemshardy_afk: owalsh: any updates on https://bugs.launchpad.net/nova/+bug/1665370 ?18:31
openstackLaunchpad 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|away18:31
*** crushil has quit IRC18:31
mriedemi'm unclear on if we're making a change in nova for that18:31
dansmithmriedem: we think not18:32
*** cdent has joined #openstack-nova18:32
*** claudiub has quit IRC18:33
owalshmrodden: I think it's a pkg issue in this case, shardy_afk is trying to reproduce it18:33
mriedemuh oh18:33
mriedemsomeone woke the mrodden troll18:33
owalshoop ^^18:33
*** jaosorior has quit IRC18:34
*** kaisers_ has quit IRC18:35
owalshmriedem (too many m... nicks here), dansmith do we need to worry about it for pip or setup.py?18:36
*** Swami has joined #openstack-nova18:36
dansmithowalsh: pip will do the right thing I think18:37
dansmithowalsh: setup.py has this kind of issue in other places, so I think we're probably okay there18:37
*** Swami_ has quit IRC18:38
*** yassine_ has quit IRC18:42
*** bvanhav_ has quit IRC18:42
mordredmriedem, 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 IRC18:43
mordredmriedem, jaypipes: specifically because the volume throws an error that you can't delete an attached volume18:44
*** dharinic is now known as dharinic|lunch18:44
mordredI know that's a cinder not a nova thing - but since you're tlaking about the attach flow I figured I'd mention it18:44
mriedemmordred: you're racing18:45
smcginnisYeah, that sounds like the attach hasn't finished before trying to delete.18:45
melwittdoes 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*dettach18:45
mriedemsmcginnis: the detach18:45
mriedemyeah18:45
smcginnisKeyboard...18:45
mriedemmordred: tempest flow is usually create volume, create server, attach volume to server, detach volume, delete volume, delete server,18:46
mriedemsometihng like that18:46
sdaguemelwitt: we pulled that part out of the docs a while ago because so little was actually docs18:46
sdaguedocced18:46
mriedemnova will detach the volume, but you have to wait for the volume to be 'available' before you can delete it18:46
melwittk, thanks sdague18:46
*** adisky_ has quit IRC18:49
*** cdent has quit IRC18:50
*** cdent has joined #openstack-nova18:50
*** yassine_ has joined #openstack-nova18:50
mordredmriedem: 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 flow18:50
mordredmriedem: jordanP is looking in to it some18:50
*** ralonsoh has quit IRC18:51
mriedemguess it depends on the specific low,18:51
mriedem*flow18:51
mriedemif you do boot from volume with delete_on_termination=True, then nova deletes the volume18:51
mriedemwhen you delete the server18:51
*** Apoorva has joined #openstack-nova18:52
*** smatzek has quit IRC18:52
*** pbandark has quit IRC18:52
*** jsho has quit IRC18:55
mordredyah. 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 IRC18:57
*** READ10 has quit IRC18:58
*** jsho has joined #openstack-nova18:58
*** sudipto has quit IRC18:59
*** sudipto_ has quit IRC18:59
*** bvanhav_ has joined #openstack-nova19:00
*** crushil has joined #openstack-nova19:03
*** kaisers_ has joined #openstack-nova19:05
*** kaisers_ has quit IRC19:06
*** kaisers_ has joined #openstack-nova19:06
*** ZZelle_ has joined #openstack-nova19:10
*** jamesdenton has joined #openstack-nova19:10
*** kaisers_ has quit IRC19:12
dansmithmriedem: melwitt: do you have any opinion on this? https://review.openstack.org/#/c/433362/3/nova/api/openstack/compute/servers.py@115919:12
dansmithlike, whether we should target from the REST API side, or modify every compute/api method to take the mapping,19:12
dansmithor stash the mapping legally or illegally inside the instance somewhere?19:12
fricklermriedem: 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 it19:14
melwittdansmith: 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 everywhere19:15
dansmithhmm19:16
*** owalsh is now known as owalsh-afk19:16
dansmithmelwitt: 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 targeted19:16
melwittthere'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 least19:16
dansmithI mean, I guess that's the problem with targeting in the REST side anyway19:17
melwittyep. that's why I was thinking I'm wary of making it too obscure19:17
dansmithmelwitt: yeah, well, glad we agree on that, I'd like to rule that out right here and now :)19:17
dansmiththe api modification route I mean19:17
melwittyeah19:17
dansmithmelwitt: right now target_cell modifies the context you pass it I think,19:18
dansmithwhich means we'd need to change that or make another thing to say "give me a targeted context"19:18
*** Apoorva has quit IRC19:19
*** Apoorva has joined #openstack-nova19:19
alaskiI 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 cell19:19
melwittyeah. 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 that19:19
dansmithmelwitt: you can't I don't think19:20
*** shaner has quit IRC19:20
melwittbecause it would set it back. okay19:20
dansmithmelwitt: because returning from the with will untarget the context you set on instance19:20
dansmithyeah19:20
dansmithalaski: yeah, I like that because it is less work, I just worry about it being super confusing later19:20
*** shaner has joined #openstack-nova19:20
*** shaner has quit IRC19:20
melwittyeah. 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 yet19:21
dansmiththe event patch needs to be more prescriptive because it has to collate multiple events,19:21
dansmithbut I can make the top patch do the magic thing and see how much has to change all at once I guess19:22
alaskidansmith: 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 purposes19:22
dansmithalaski: meaning it's a copy and targeted, whereas the context we pass to methods remains clean?19:23
dansmiththat is probably less obscure than how I was thinking about it19:23
alaskiyeah19:24
alaskithere's still some risk, but it seemed the least magic19:24
dansmithmelwitt: if you agree, I'll go do that thing and we can blame alaski for any fallout sometime when he's not looking19:24
melwittI 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 switch19:25
melwittdansmith: yeah, I'm not too opinionated about it. alaski's thinking makes sense to me, plus we can blame him19:26
dansmithmelwitt: well, we would still target the external context if and when we need to,19:26
dansmithmelwitt: but instance.save() would just work (tm)19:26
melwittI'm just not sure how it's going to work out because of context vs instance._context, but yeah. yeah, true19:26
openstackgerritSean Dague proposed openstack/nova master: WIP: verify project_id when quotas are checked  https://review.openstack.org/43501019:27
dansmithmelwitt: well, given the fun I just had with the test fixture,19:27
dansmithit'll be interesting to see what doesn't fall into that realm with just the instance context set19:27
dansmithlike, looking up BDMs for an instance in the middle of an operation or something19:27
* alaski will be fine to take the blame for his bad decisions19:27
dansmithmelwitt: like, we'll make the rpc call with an untargeted context, which means my serializer hack won't work19:28
dansmithwhich will suck19:28
dansmitha lot.19:28
dansmithunless the rpc layer auto-targets the context before it makes the call19:28
melwitthm, yeah that's what I was just thinking19:28
dansmiththat makes me sad19:28
melwittwe have the lookup things in there19:28
dansmithright, which I thought we wanted to maybe replace by going back to targeting...19:29
dansmithwe added transport back into the context, right?19:29
dansmithugh, I dunno19:29
melwittreally? I didn't know we wanted to :P19:29
melwittnot yet. I have a patch up that does that but you crapped on it19:29
dansmithdidn't we discuss that? I thought we added it back in19:29
dansmithhah, well19:30
dansmithlink?19:30
*** lpetrut has quit IRC19:30
melwittwe don't technically need it, it would just save lookups if we did19:30
melwitthttps://review.openstack.org/41154319:30
dansmithright, 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 place19:30
dansmithmelwitt: oh please.. that's like a small dingleberry19:31
*** lpetrut has joined #openstack-nova19:31
dansmithI didn't even -119:31
melwittI did ^ weirdly because we have tried to not pass cell mapping around so user/pass don't get dumped out in some exception somewhere, potentially19:31
melwitthaha, I know19:31
melwittokay, well, the original plan was to target everything explicitly but then we thought it would be easier if we just looked it up in there19:32
dansmithso 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 time19:33
melwittanother 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 target19:33
dansmiththe 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-nova19:33
dansmithmelwitt: we can still cache them even if we target them early19:33
melwittyeah. 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 it19:34
dansmithyeah, I don't really get those concerns, but..19:34
melwittdansmith: yeah going back to api database isn't affected. those can use the global api db context managers19:34
dansmithyeah, so, maybe I'm just being overly concerned about doing the target hard and early19:35
*** prateek has joined #openstack-nova19:36
*** shaner has joined #openstack-nova19:36
melwittyeah, I don't think we've thought a lot about targeting a context and just letting it carry the db/mq connections around19:37
melwittit might work fine19:37
*** fragatina has quit IRC19:37
dansmithlet me just do the early full target thing and see what explodes19:38
melwittsure19:38
*** fragatina has joined #openstack-nova19:38
*** jsho has quit IRC19:39
*** fragatina has quit IRC19:40
*** fragatina has joined #openstack-nova19:40
openstackgerritArtom Lifshitz proposed openstack/nova master: API support for tagged device attachment  https://review.openstack.org/37602119:43
openstackgerritArtom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume  https://review.openstack.org/43508219:43
*** dharinic|lunch is now known as dharinic19:44
mriedemdansmith: 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 everywhere19:46
melwittdansmith: I'll play around with making my mq connection patch cache early in the target_cell later19:46
dansmithmriedem: lots of places19:46
mriedemwe could put some common code into nova/api/openstack/common, but that doesn't help with all of the target_cell stuff19:46
dansmithmriedem: well, it does if it just targets the context permanently, which is what I'm trying now19:47
mriedemwhy do this in the REST API handlers rather than the compute API class methods themselves?19:47
*** lpetrut has quit IRC19:47
*** jsho has joined #openstack-nova19:47
*** fragatina has quit IRC19:48
dansmithmriedem: they have to look up the instance19:48
dansmithmriedem: and we don't want the rest api looking up the mapping, then making compute/api do it again19:48
mriedemoh yeah19:48
*** rmart04 has quit IRC19:49
mriedemdansmith: 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 us19:51
mriedembut i see melwitt doesn't like that19:51
dansmithmriedem: _I_ don't like that19:51
mriedemoh19:51
melwittwe both don't like it19:51
dansmithI asked so someone could say "fuck no"19:51
melwittheh19:51
mriedemwas 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
dansmithyeah19:52
mriedembecause you were also talking about stashing the mapping in the context or the instance at the same time19:52
*** dikonoor has quit IRC19:52
dansmithyeah, but I meant the param thing19:52
jaypipesmriedem, dansmith, cdent, edleafe, johnthetubaguy, bauzas, melwitt: https://etherpad.openstack.org/p/decision-finding-shared-inventory19:53
jaypipesmriedem, dansmith, cdent, edleafe, johnthetubaguy, bauzas, melwitt: feel free to vote and leave comments... adding that to the scheduler/placement session discussions at PTG19:54
cdentjaypipes: 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
mriedemthat bile yellow color hurts my eyes19:55
jaypipesmriedem: then lucky for you you didn't see the shirt I was wearing this morning. cdent is still recovering.19:56
cdentjaypipes: for completeness could you add a bit more info on why that particular question needs to be asked?19:56
jaypipescdent: yuppers.19:56
cdentyeah, I'm going to have nightmares tonight. the combo is...19:56
jaypipescdent: will do.19:57
mriedemwere the nipples cut out?19:57
mriedembecause if so, sign me up19:57
jaypipesmriedem: lol19:57
*** mkoderer has quit IRC19:57
mriedemdansmith: so it sounds like stashing the targeted context is the plan?19:57
dansmithmriedem: potentially, until I find some reason to hate that more19:58
cdentmriedem: great, thanks. I'm totally broken now.19:58
*** satyar has quit IRC19:59
*** krtaylor has quit IRC19:59
mriedemi'm trying to think of an operatoin where we could have multiple instances in different cells20:00
mriedembut i don't think that's possible right now20:00
dansmithmriedem: it's not now for sure,20:00
dansmithmriedem: and I was thinking of migrations of some sort, but that'll clearly require some special handling anyway20:00
mriedemwhat apis take multiple instances anyway?20:00
dansmithwell, the events api, which is why I said what I did, but that one is already handles20:00
dansmith*handled20:00
mriedemok20:01
*** Hosam has joined #openstack-nova20:01
mriedemmaybe we'll revive that spec to expose quiesce in the api for multiple instances, which could all be in different cells20:01
mriedem:)20:01
* dansmith starts stabbing the air20:01
mriedemhaha20:01
*** liangy has quit IRC20:05
openstackgerritSean Dague proposed openstack/nova master: Verify project_id when quotas are checked  https://review.openstack.org/43501020:06
*** kaisers_ has joined #openstack-nova20:07
*** slaweq has quit IRC20:08
mriedemsdague: your spec update https://review.openstack.org/#/c/435016/ looks good to me20:11
openstackgerritYuming Ma proposed openstack/nova-specs master: Nova BFV by reserving volume  https://review.openstack.org/41754120:12
*** bvanhav_ is now known as bvanhav20:12
mriedemit does clarify the logic20:12
*** liverpooler has quit IRC20:12
sdagueit 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 cases20:13
*** Sukhdev has joined #openstack-nova20:14
*** prateek has quit IRC20:15
*** salv-orl_ has quit IRC20:16
*** amoralej is now known as amoralej|off20:16
openstackgerritYuming Ma proposed openstack/nova-specs master: Nova BFV by reserving volume  https://review.openstack.org/41754120:17
*** alexpilotti has joined #openstack-nova20:18
*** schlueter has joined #openstack-nova20:18
*** lucasxu has quit IRC20:19
*** alexpilo_ has quit IRC20:19
*** lucasxu has joined #openstack-nova20:19
*** salv-orlando has joined #openstack-nova20:23
*** gouthamr has quit IRC20:24
*** jsho has quit IRC20:24
*** Hosam has quit IRC20:24
*** lucasxu has quit IRC20:26
*** lucasxu has joined #openstack-nova20:26
*** rfolco has quit IRC20:27
*** rfolco has joined #openstack-nova20:27
*** salv-orlando has quit IRC20:28
*** jwcroppe_ has quit IRC20:28
*** rfolco has quit IRC20:28
*** jwcroppe has joined #openstack-nova20:28
*** baoli has quit IRC20:29
*** kaisers_ has quit IRC20:31
*** dtp has joined #openstack-nova20:32
*** moshele has joined #openstack-nova20:35
*** bvanhav has quit IRC20:38
*** kberger has quit IRC20:39
openstackgerritYuming Ma proposed openstack/nova-specs master: Nova BFV by reserving volume  https://review.openstack.org/41754120:39
*** baoli has joined #openstack-nova20:40
mriedemanyone aware of a way to basically squash a series of changes into a single patch using git format-patch?20:40
mriedemwithout first doing a git rebase to squash them into a single commit20:40
*** gouthamr has joined #openstack-nova20:40
*** armax has quit IRC20:41
*** armax has joined #openstack-nova20:41
mriedemnvm thanks google http://stackoverflow.com/questions/616556/how-do-you-squash-commits-into-one-patch-with-git-format-patch20:42
*** liangy has joined #openstack-nova20:42
*** armax has quit IRC20:44
cdentedleafe: you the light blue on jay's etherpad? I think we're running into similar mental blocks :(20:46
edleafedat's me20:46
*** lpetrut has joined #openstack-nova20:48
*** satyar has joined #openstack-nova20:48
*** ducttape_ has quit IRC20:51
*** moshele has quit IRC20:51
*** ducttape_ has joined #openstack-nova20:51
mriedemnova meeting in 9 minutes20:51
*** takashin has joined #openstack-nova20:54
*** lpetrut has quit IRC20:54
*** aleph1 is now known as agarner_away20:54
*** READ10 has joined #openstack-nova20:57
*** slaweq has joined #openstack-nova20:58
openstackgerritMaciej Szankin proposed openstack/nova master: Tests: remove .testrepository/times.dbm in tox.ini  https://review.openstack.org/43511120:59
mriedemit's time21:00
*** fragatina has joined #openstack-nova21:02
*** krtaylor has joined #openstack-nova21:04
*** moshele has joined #openstack-nova21:04
*** READ10 has quit IRC21:05
*** slaweq has quit IRC21:06
*** slaweq has joined #openstack-nova21:08
*** moshele has quit IRC21:13
*** lucasxu has quit IRC21:14
*** lucasxu has joined #openstack-nova21:14
*** dimtruck is now known as zz_dimtruck21:14
*** cdent has quit IRC21:16
*** yamamoto has joined #openstack-nova21:17
*** yamamoto has quit IRC21:19
*** dave-mccowan has quit IRC21:21
*** karimb has quit IRC21:21
*** karimb has joined #openstack-nova21:21
*** armax has joined #openstack-nova21:22
*** salv-orlando has joined #openstack-nova21:24
*** eharney has quit IRC21:25
*** kaisers_ has joined #openstack-nova21:27
*** tblakes has quit IRC21:27
*** salv-orlando has quit IRC21:29
*** armax has quit IRC21:30
*** lucasxu has quit IRC21:33
*** lucasxu has joined #openstack-nova21:33
*** tyrola has quit IRC21:34
*** Jeffrey4l has quit IRC21:35
*** tyrola has joined #openstack-nova21:35
*** takashin has left #openstack-nova21:35
cfriesenmriedem: 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
openstackbug 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-nova21:36
mriedemcfriesen: just commit a release note to stable/newton with the known issue listed21:36
*** karimb has quit IRC21:37
cfriesenmriedem: okay, can do21:37
*** crushil has quit IRC21:38
*** salv-orlando has joined #openstack-nova21:38
*** crushil has joined #openstack-nova21:39
*** zz_dimtruck is now known as dimtruck21:39
*** takashin has joined #openstack-nova21:39
*** jerrygb has quit IRC21:41
openstackgerritEric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy  https://review.openstack.org/39128821:42
*** ducttape_ has quit IRC21:42
*** Sukhdev has quit IRC21:45
*** crushil has quit IRC21:48
*** ducttape_ has joined #openstack-nova21:50
*** edmondsw has quit IRC21:52
*** edmondsw has joined #openstack-nova21:53
*** edmondsw has quit IRC21:57
openstackgerritmelanie witt proposed openstack/nova master: Count resources to check quota for cells  https://review.openstack.org/41652121:59
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing  https://review.openstack.org/42951222:00
*** satyar has quit IRC22:00
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43: List/Show all server migration types  https://review.openstack.org/43083922:02
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43 - Enable cold migration with target host  https://review.openstack.org/40670722:02
openstackgerritDan Smith proposed openstack/nova master: Make servers API use cell-targeted context  https://review.openstack.org/43336222:02
*** tbachman has quit IRC22:02
dansmithmelwitt: surprisingly little drama with unit and functional ^22:02
*** smatzek has quit IRC22:02
dansmithwe'll see how devstack likes it22:02
*** ducttape_ has quit IRC22:03
melwittdansmith: yeah, that's almost no change to the unit and functional. huh.22:04
openstackgerritEric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot  https://review.openstack.org/42738022:05
dansmithwell, it kinda makes sense since I'm just making the set call a no-op for the simple cell fixture22:05
dansmithalthough I'm surprised a bunch of functional tests didn't fail22:05
dansmithbut we'll see22:05
melwittyeah22:06
mriedembauzas: got it for you https://review.openstack.org/#/c/433855/22:06
bauzasmriedem: heh22:08
bauzasanyway, I promised a comment22:09
*** ducttape_ has joined #openstack-nova22:10
openstackgerritSujitha proposed openstack/nova master: Change tags to default field in Instance object.  https://review.openstack.org/41529822:12
*** smatzek has joined #openstack-nova22:12
cfriesendansmith: 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 IRC22:13
*** smatzek_ has joined #openstack-nova22:13
*** dave-mccowan has joined #openstack-nova22:14
*** viks has joined #openstack-nova22:15
*** viks has left #openstack-nova22:15
*** gouthamr has quit IRC22:15
*** viks has joined #openstack-nova22:16
*** zzfoo has joined #openstack-nova22:20
zzfooHi, 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 available22:21
cfriesendoes 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
zzfooon the shared storage though that file has been created.22:22
melwittcfriesen: not expected. did you 'git review -s' first after you cloned python-novaclient?22:22
melwittyou have to do it anytime you clone a new rep, afaik22:23
melwitt*repo22:23
cfriesenmelwitt: it's entirely possible that I didn't22:23
openstackgerritEric Fried proposed openstack/nova master: PowerVM Driver: localdisk  https://review.openstack.org/40940122:24
melwittcfriesen: you can try 'git review -s' in the repo dir, then 'git commit --amend' and then you should get a change-id22:24
cfriesenmelwitt: that seems to have done it.  thanks22:25
*** armax has joined #openstack-nova22:27
*** avolkov has quit IRC22:28
EmilienMmriedem: I took the freedom to write some thoughts on https://etherpad.openstack.org/p/nova-ocata-retrospective - I'll probably have more later22:32
openstackgerritChris Friesen proposed openstack/python-novaclient master: Add support for showing flavor details (v2.43)  https://review.openstack.org/43514122:32
mriedemEmilienM: ok, i expected it :)22:33
*** yamamoto has joined #openstack-nova22:35
*** ijw has joined #openstack-nova22:36
*** annegentle has quit IRC22:36
*** smatzek_ has quit IRC22:38
*** rfolco has joined #openstack-nova22:39
*** thorst_afk has quit IRC22:39
*** rfolco has quit IRC22:39
*** kaisers_ has quit IRC22:40
mriedemhmm, what runs qemu-img info in a periodic?22:40
mriedemget_available_resources?22:40
*** burgerk has quit IRC22:41
cfriesenseems likely22:42
mriedemso, i've seen this a few times lately,22:42
mriedemthe live migration job fails because we timeout waiting for an instance to spawn22:42
mriedemand 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_14322:42
mriedem2017-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:268922:42
mriedemwhich calls spawn() on the libvirt driver22:43
mriedemand after that, it's radio silence on that instance and request except for periodic tasks22:43
*** Sukhdev has joined #openstack-nova22:43
mriedemi have a feeling we have some kind of deadlock or something in the libvirt driver's spawn flow22:43
mriedemi've seen it since newton22:43
cfriesenso you're thinking self._domain.createWithFlags() is triggering something wonky?22:47
cfriesenor is there another async call before that?22:48
*** acabot has quit IRC22:48
*** liangy has quit IRC22:50
openstackgerritEric Fried proposed openstack/nova master: PowerVM Driver: console  https://review.openstack.org/40940222:50
*** efried has quit IRC22:51
mriedemcfriesen: i don't know what it is22:51
mriedemlast time i looked into this i was going to add some debug logging in the image create code22:51
mriedembecause that's gone through a lot of churn since newton22:51
cfriesenmriedem: 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-nova22:54
*** marst has quit IRC22:54
mriedemthis 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_28622:54
mriedemInstance network_info: |[]|22:55
mriedemso i wonder if,22:57
*** felipemonteiro_ has quit IRC22:57
mriedemthe compute manager is just waiting for the async network_info stuff to get done22:57
mriedembut 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 networkin22:57
mriedemand test it in tempest with the 2.37 microversion22:58
cfriesenwhatever 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-nova23:01
*** jamielennox is now known as jamielennox|away23:02
*** catintheroof has joined #openstack-nova23:02
mriedemcfriesen: yeah right after that we see timer = loopingcall.FixedIntervalLoopingCall(_wait_for_boot)23:02
mriedemwhich is waiting for the power state to be RUNNING23:02
mriedemi guess we just never finish there23:03
mriedembecause i never see "Instance spawned successfully."23:03
*** thorst_afk has joined #openstack-nova23:03
mriedemeven 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_21423:03
mriedem[instance: 0ee93807-d206-4ddf-878c-efd1dd2eab3c] VM Started (Lifecycle Event)23:03
mriedemand then right after that [instance: 0ee93807-d206-4ddf-878c-efd1dd2eab3c] VM Paused (Lifecycle Event)23:04
*** cdent has joined #openstack-nova23:04
*** mdrabe has quit IRC23:04
mriedemSynchronizing instance power state after lifecycle event "Paused"; current vm_state: building, current task_state: spawning, current DB power_state: 0, VM power_state: 323:04
mriedem3 = paused23:04
mriedemi 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.gz23:05
*** dave-mccowan has quit IRC23:06
mriedemso the instance goes from started to paused,23:06
mriedemand then is never unpaused23:06
mriedemwhich is why the spawn() method never considers it done i guess23:07
*** thorst_afk has quit IRC23:07
*** esberglu has quit IRC23:07
*** dimtruck is now known as zz_dimtruck23:07
*** esberglu has joined #openstack-nova23:07
mriedemhttp://logs.openstack.org/50/435050/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/49adcde/logs/subnode-2/libvirt/qemu/instance-00000001.txt.gz23:08
mriedemwhat does this mean? "KVM: entry failed, hardware error 0x0"23:08
*** esberglu has quit IRC23:12
openstackgerritSujitha proposed openstack/nova master: This patch is followup for https://review.openstack.org/#/c/415298/.  https://review.openstack.org/43514623:13
*** jamielennox|away is now known as jamielennox23:14
*** lucasxu has quit IRC23:15
cdentmriedem: casual googling suggests problems with nested virtualization23:15
*** marst has joined #openstack-nova23:16
mriedemi'm crawling through the libvirtd logs23:16
openstackgerritSujitha proposed openstack/nova master: Change tags to default field in Instance object.  https://review.openstack.org/41529823:18
*** baoli has quit IRC23:18
openstackgerritSujitha proposed openstack/nova master: This patch is followup for https://review.openstack.org/#/c/415298/.  https://review.openstack.org/43514623:19
mriedemi see this start to go crazy23:19
mriedemdebug : qemuMonitorJSONIOProcess:255 : Total used 0 bytes out of 1023 available in buffer23:19
mriedemdebug : qemuMonitorJSONIOProcess:255 : Total used 0 bytes out of 2047 available in buffer23:19
mriedemdebug : qemuMonitorJSONIOProcess:255 : Total used 0 bytes out of 3071 available in buffer23:19
mriedemdebug : qemuMonitorJSONIOProcess:255 : Total used 0 bytes out of 4095 available in buffer23:19
mriedem...23:20
mriedemdebug : qemuMonitorJSONIOProcess:255 : Total used 6689 bytes out of 6689 available in buffer23:20
mriedemmaybe that's normal23:20
*** esberglu has joined #openstack-nova23:21
mriedem2017-02-16 22:22:11.139+0000: 8968: debug : qemuProcessHandleResume:765 : Transitioned guest instance-00000001 out of paused into resumed state23:21
mriedem2017-02-16 22:22:11.140+0000: 8968: debug : qemuProcessHandleStop:716 : Transitioned guest instance-00000001 to paused state23:22
openstackgerritEric Fried proposed openstack/nova master: PowerVM Driver: config drive  https://review.openstack.org/40940423:22
openstackgerritSujitha proposed openstack/nova master: [WIP] Reduce calls to load_tags() to 0  https://review.openstack.org/43514623:23
*** jamesdenton has quit IRC23:23
mriedemno errors in the libvirt logs23:23
*** salv-orlando has quit IRC23:24
*** esberglu has quit IRC23:24
*** esberglu has joined #openstack-nova23:24
mriedemaha23:25
mriedemright around that time in the syslog23:25
mriedemhttp://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_1123:25
mriedemEnd of file while reading data: Input/output error23:26
mriedemwe shouldn't even be using virtlogd23:27
mriedemdansmith: you ever seen anything like this? ^23:27
mriedemchannel your inner berrange23:27
mriedemmaybe the i/o error with virtlogd is a result of the guest hw error?23:28
*** takashin has left #openstack-nova23:29
mriedemi guess that's really normal23:29
mriedemit's all over syslog in our ci runs23:29
cdentthat's an unfortunate way to spell normal23:30
*** xyang1 has quit IRC23:33
mriedemvirt_type = kvm in nova.conf23:34
mriedemthat should be qemu23:34
mriedemwe don't support that in the gate with nested virt, how the hell was this even running23:34
mriedemso yeah, that's the issue23:35
cdentmriedem: how/where/why?23:36
cdentas in wat?23:37
cdentand wtf23:37
mriedemit's qemu on the primary node, but kvm on the subnode23:37
mriedemthat's the bug23:37
cfriesenanyone 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 error23:38
cdentmriedem: ah, and somehow this only showed up as a problem now?23:38
*** Hosam has joined #openstack-nova23:38
mriedemcdent: i've been seeing it off and on23:39
mriedemi just haven't taken the 2 hours to investigate :)23:39
cdentow23:39
cdentdo you ever sleep or eat or recreate in some fashion (please don't tell me this is your recreation)?23:40
mriedemwhen my wife forces me, yes23:41
mriedemi only care about this since it kicked the last change for ocata ga out of the gagte23:41
mriedem*Gate23:41
mriedemshe 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 duties23:42
*** esberglu has quit IRC23:46
cdentg'night23:47
*** cdent has quit IRC23:47
openstackgerritSujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload  https://review.openstack.org/41918523:47
*** edmondsw has joined #openstack-nova23:49
mriedemhttps://review.openstack.org/#/c/435154/123:51
mriedemYES!23:51
mriedemsdague: ^^^^23:51
mriedemjogo: even you ^23:51
mriedemmtreinish: ^23:51
mtreinishmriedem: hah, nice23:52
*** edmondsw has quit IRC23:53
mriedemok i'm tapping out, back online later23:54
jogomriedem: done23:55
mriedemha23:55
mriedemo/23:55
*** mriedem has quit IRC23:56
*** jaypipes has quit IRC23:56
*** kylek3h has joined #openstack-nova23:57
*** armax has quit IRC23:58
*** kylek3h_ has quit IRC23:58

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