| dansmith | now, he made it sound like they _do_ have a block device even in the containery thing | 00:00 |
|---|---|---|
| mikal | We talked through a user space fat implementation they could use instead | 00:00 |
| mikal | And they shook their fists at us for being big meanies | 00:00 |
| *** aslezil has quit IRC | 00:00 | |
| *** dimtruck has quit IRC | 00:00 | |
| mikal | I have no comment on if the block thing is still true | 00:00 |
| mikal | Simply don't know | 00:00 |
| *** scottda has quit IRC | 00:00 | |
| *** betherly has quit IRC | 00:00 | |
| *** spotz has quit IRC | 00:00 | |
| *** McNinja has quit IRC | 00:00 | |
| *** leifz has quit IRC | 00:00 | |
| dansmith | mikal: can you make sure that all gets reflected in the current patches? I'll circle back and admit I didn't have that context while we were discussing it and I was not coming up with any reasons to block it | 00:01 |
| dansmith | (which is why we wanted to re-get your context post-ptg) | 00:01 |
| *** geekinutah has quit IRC | 00:01 | |
| *** comstud has quit IRC | 00:01 | |
| mikal | Sure. Its not going to happen today though. I'll try and get it done by the end of the week. | 00:01 |
| *** patrickeast has quit IRC | 00:02 | |
| *** fnordahl has quit IRC | 00:02 | |
| *** melwitt has quit IRC | 00:02 | |
| mikal | I'm currently trying to remember why a directory on the host OS was better than a ext4 filesystem | 00:02 |
| mikal | And its not coming to me instantly | 00:02 |
| *** nikhil|afk has quit IRC | 00:02 | |
| *** toan has quit IRC | 00:02 | |
| *** Hazelesque has quit IRC | 00:02 | |
| dansmith | well, it is the same behavior for lxc and parallels, and doesn't require needing to know what filesystem is there | 00:02 |
| mikal | I do recall someone else wanting directories, it might have been LXC | 00:03 |
| *** nikhil|afk has joined #openstack-nova | 00:03 | |
| mikal | I'm a bit fuzzy on the details | 00:03 |
| mikal | That said, it didn't merge | 00:03 |
| tonyb | dansmith: I don't see any open reviews, just the old 2015 blueprint | 00:03 |
| mikal | So how does LXC do config drives at the moment? | 00:03 |
| dansmith | tonyb: they had a patch I think | 00:03 |
| dansmith | mikal: it doesn't AFAIK | 00:03 |
| mikal | Oh, ok | 00:03 |
| *** jwcroppe has quit IRC | 00:04 | |
| dansmith | actually, | 00:04 |
| *** owalsh_ is now known as owalsh | 00:04 | |
| dansmith | tonyb: maybe I'm thinking of the other libvirt image type thing | 00:04 |
| *** Hazelesque has joined #openstack-nova | 00:04 | |
| *** owalsh is now known as owalsh-afk | 00:04 | |
| dansmith | I think we're to expect them to have a spec though, IIRC | 00:04 |
| tonyb | dansmith: Yeah could be ... I can't see an configdrive patch and we discussed them at the same time late on Friday soe memory==fuzzy | 00:05 |
| dansmith | yeah | 00:05 |
| *** scottda has joined #openstack-nova | 00:05 | |
| *** nic has quit IRC | 00:05 | |
| *** hongbin has quit IRC | 00:05 | |
| dansmith | tonyb: he should be around during your day today I think, so maybe ping him | 00:05 |
| mikal | Was there any discussion of vendordata? I kind of consider it complete at this point, but I don't know if people showed up with pitchforks with further complaints. | 00:05 |
| *** geekinutah has joined #openstack-nova | 00:06 | |
| *** patrickeast has joined #openstack-nova | 00:06 | |
| tonyb | mikal: not near me | 00:06 |
| dansmith | mikal: we declared it a success given the lack of any pitchforks (or other representation) | 00:06 |
| *** yingjun has joined #openstack-nova | 00:06 | |
| mikal | dansmith: good. I shall pretend it never happened then. | 00:07 |
| *** aslezil_ has joined #openstack-nova | 00:07 | |
| dansmith | well you might miss out on eventual praise if you do | 00:07 |
| mikal | dansmith: oh no, I shall tell random people on the street of my exploits | 00:07 |
| *** toan has joined #openstack-nova | 00:07 | |
| dansmith | heh | 00:07 |
| tonyb | dansmith: he'll have a minion pass on the praise in mime | 00:08 |
| mikal | dansmith: and it will go on my application form for Fellow. "I invented the shit out of config drive v2, and the vendordata re-write!" | 00:08 |
| dansmith | heh | 00:08 |
| cburgess | lol | 00:08 |
| mikal | Both fundamental internet technologies | 00:08 |
| *** dimtruck has joined #openstack-nova | 00:08 | |
| mikal | Actually, a tshirt that said "I invented the shit out of config drive v2" would probably be a thing I'd wear | 00:08 |
| *** McNinja has joined #openstack-nova | 00:09 | |
| *** melwitt has joined #openstack-nova | 00:09 | |
| *** leifz has joined #openstack-nova | 00:09 | |
| *** comstud has joined #openstack-nova | 00:09 | |
| *** spotz has joined #openstack-nova | 00:09 | |
| dansmith | and most people would not know it's not that impressive, so, sounds like a good idea | 00:09 |
| *** melwitt is now known as Guest68076 | 00:09 | |
| mikal | I would like to note we intended to have deleted config drive by now | 00:10 |
| mikal | But yet, here we are | 00:10 |
| mikal | Its the gift that keeps on giving | 00:10 |
| *** diga has joined #openstack-nova | 00:10 | |
| *** betherly has joined #openstack-nova | 00:11 | |
| *** fnordahl has joined #openstack-nova | 00:11 | |
| *** jamesdenton has joined #openstack-nova | 00:11 | |
| tonyb | mikal: we shoudl just fix the metadata service so configdrive isn't needed | 00:12 |
| *** iceyao has joined #openstack-nova | 00:12 | |
| cburgess | Yeah I don't see how we can ever get rid of it at this poing given we can't do metadata over IPv6 | 00:12 |
| tonyb | mikal: how hard can it be ;P | 00:12 |
| cburgess | and yeah metadata service is... umm... horrible | 00:12 |
| tonyb | cburgess: it's only code from being less horrible | 00:12 |
| *** Guest68076 is now known as melwitt | 00:12 | |
| mikal | cburgess: that's probably what I'd do next if I had any time | 00:12 |
| openstackgerrit | John Griffith proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750 | 00:12 |
| mikal | But instead, I write project plans | 00:13 |
| cburgess | tonyb mikal Yeah I wish I had the time to fix metadata as well. Its caused us a lot of scaling problems | 00:13 |
| *** kaisers has quit IRC | 00:13 | |
| mikal | cburgess: you just need better users... | 00:13 |
| cburgess | mikal Don't get me started | 00:13 |
| cburgess | Somewhere.. someone on the internet created a chef plugin of some type that spams the hell out of metatata with request from every conciviable cloud format every seconds. | 00:14 |
| cburgess | I have users making 350+ metadata requests a second. | 00:14 |
| cburgess | Its... insane | 00:14 |
| dansmith | cburgess: yeah, it's super good for that | 00:15 |
| dansmith | cburgess: users love hitting it hard | 00:15 |
| mikal | cburgess: you're really going to hate vendordata then | 00:15 |
| mikal | Cause we fan out that request to N other services | 00:15 |
| *** darrenc has quit IRC | 00:16 | |
| *** iceyao has quit IRC | 00:16 | |
| cburgess | dansmith Is that supposed to be in the sarcasm font? | 00:16 |
| dansmith | cburgess: no | 00:16 |
| dansmith | cburgess: I have 'em too | 00:16 |
| cburgess | mikal I still have to configure that though right? | 00:16 |
| dansmith | cburgess: run puppet every minute and slam the metadata server | 00:16 |
| mikal | cburgess: yes, its opt in | 00:16 |
| *** jamielennox|away is now known as jamielennox | 00:16 | |
| mikal | cburgess: but if any of those users want windows active directory registration... | 00:16 |
| cburgess | dansmith Ahh ok yeah, its insane to me, and they seem to not understand how its a problem | 00:16 |
| dansmith | yeah | 00:16 |
| dansmith | but alas | 00:16 |
| cburgess | mikal Hmmm I might need that | 00:17 |
| mikal | cburgess: good news! | 00:17 |
| mikal | cburgess: and by good I mean bad | 00:17 |
| *** darrenc has joined #openstack-nova | 00:17 | |
| cburgess | dansmith Yeah so... like I said.. if I had the time I would propose a re-write of metadata because its horrible. | 00:17 |
| mikal | I could write a project plan for that | 00:17 |
| mikal | I'm getting quite good at them | 00:17 |
| mikal | Not good at getting things resourced | 00:17 |
| mikal | But I have nice Microsoft Word documents | 00:17 |
| cburgess | LOL | 00:17 |
| openstackgerrit | Li Yingjun proposed openstack/nova master: Error message should not include SQL command https://review.openstack.org/436773 | 00:19 |
| *** sdague has quit IRC | 00:20 | |
| *** ZZelle has quit IRC | 00:23 | |
| *** sapcc-bot has quit IRC | 00:23 | |
| *** sapcc-bot has joined #openstack-nova | 00:23 | |
| *** anthonyper has quit IRC | 00:28 | |
| *** ijw has quit IRC | 00:29 | |
| *** mtanino_ has joined #openstack-nova | 00:29 | |
| *** ijw has joined #openstack-nova | 00:30 | |
| *** ijw has quit IRC | 00:30 | |
| *** mtanino has quit IRC | 00:30 | |
| *** gfhellma_ has quit IRC | 00:31 | |
| *** gfhellma has joined #openstack-nova | 00:32 | |
| *** anthonyper has joined #openstack-nova | 00:32 | |
| *** edmondsw has quit IRC | 00:33 | |
| *** iceyao has joined #openstack-nova | 00:33 | |
| *** edmondsw has joined #openstack-nova | 00:33 | |
| *** hoangcx has joined #openstack-nova | 00:36 | |
| *** jamesdenton has quit IRC | 00:37 | |
| *** iceyao has quit IRC | 00:38 | |
| *** edmondsw has quit IRC | 00:38 | |
| *** gfhellma_ has joined #openstack-nova | 00:39 | |
| *** gfhellma has quit IRC | 00:40 | |
| *** iceyao has joined #openstack-nova | 00:47 | |
| *** yingjun has quit IRC | 00:49 | |
| *** iceyao has quit IRC | 00:51 | |
| *** edmondsw has joined #openstack-nova | 00:51 | |
| *** iceyao has joined #openstack-nova | 00:54 | |
| *** dave-mccowan has joined #openstack-nova | 00:55 | |
| *** gfhellma_ has quit IRC | 00:56 | |
| *** iceyao has quit IRC | 00:59 | |
| *** fragatina has quit IRC | 01:01 | |
| *** nic has joined #openstack-nova | 01:01 | |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| *** sambetts|afk has quit IRC | 01:04 | |
| *** sambetts_ has joined #openstack-nova | 01:07 | |
| *** edmondsw has quit IRC | 01:08 | |
| *** kaisers has joined #openstack-nova | 01:09 | |
| *** kaisers has quit IRC | 01:14 | |
| *** Swami_ has quit IRC | 01:16 | |
| *** raunak has quit IRC | 01:16 | |
| *** bastafidli has joined #openstack-nova | 01:16 | |
| *** thorst has joined #openstack-nova | 01:26 | |
| *** baoli has quit IRC | 01:28 | |
| *** Apoorva_ has joined #openstack-nova | 01:30 | |
| *** dave-mccowan has quit IRC | 01:30 | |
| *** edmondsw has joined #openstack-nova | 01:31 | |
| *** Apoorva has quit IRC | 01:33 | |
| *** Apoorva_ has quit IRC | 01:34 | |
| *** ljxiash has joined #openstack-nova | 01:36 | |
| *** fragatina has joined #openstack-nova | 01:36 | |
| *** dave-mccowan has joined #openstack-nova | 01:37 | |
| *** iceyao has joined #openstack-nova | 01:38 | |
| *** kevinz has joined #openstack-nova | 01:38 | |
| *** namnh has joined #openstack-nova | 01:39 | |
| *** vladikr has quit IRC | 01:46 | |
| *** sam_nowitzki has joined #openstack-nova | 01:47 | |
| *** adrian_otto has quit IRC | 01:51 | |
| *** yuntongjin has joined #openstack-nova | 01:57 | |
| *** dave-mccowan has quit IRC | 01:59 | |
| *** edmondsw has quit IRC | 02:00 | |
| *** baoli has joined #openstack-nova | 02:02 | |
| *** McNinja has quit IRC | 02:05 | |
| *** gcb has joined #openstack-nova | 02:06 | |
| *** eliqiao has quit IRC | 02:07 | |
| *** kaisers has joined #openstack-nova | 02:10 | |
| *** McNinja has joined #openstack-nova | 02:11 | |
| *** unicell has quit IRC | 02:14 | |
| *** mtanino_ has quit IRC | 02:17 | |
| *** eliqiao has joined #openstack-nova | 02:19 | |
| *** bkopilov has quit IRC | 02:27 | |
| *** kaisers has quit IRC | 02:28 | |
| *** baoli has quit IRC | 02:32 | |
| *** diga has quit IRC | 02:37 | |
| *** takashin has joined #openstack-nova | 02:42 | |
| *** thorst has quit IRC | 02:44 | |
| *** thorst has joined #openstack-nova | 02:44 | |
| *** nic has quit IRC | 02:45 | |
| *** thorst has quit IRC | 02:49 | |
| *** gongysh has joined #openstack-nova | 02:56 | |
| *** hongbin has joined #openstack-nova | 02:59 | |
| *** hongbin has quit IRC | 03:00 | |
| *** hongbin has joined #openstack-nova | 03:01 | |
| *** ljxiash has quit IRC | 03:11 | |
| *** dimtruck is now known as zz_dimtruck | 03:12 | |
| *** ljxiash has joined #openstack-nova | 03:12 | |
| *** zz_dimtruck is now known as dimtruck | 03:13 | |
| *** ljxiash has quit IRC | 03:16 | |
| *** ljxiash has joined #openstack-nova | 03:19 | |
| *** kaisers has joined #openstack-nova | 03:30 | |
| openstackgerrit | Li Yingjun proposed openstack/nova-specs master: Spec for support set vgamem for video https://review.openstack.org/438787 | 03:31 |
| *** raunak has joined #openstack-nova | 03:34 | |
| *** zhenguo has joined #openstack-nova | 03:37 | |
| *** thorst has joined #openstack-nova | 03:45 | |
| *** mdnadeem has joined #openstack-nova | 03:46 | |
| *** bkopilov has joined #openstack-nova | 03:48 | |
| *** thorst has quit IRC | 03:50 | |
| openstackgerrit | Li Yingjun proposed openstack/nova master: libvirt: Enable custom video vgamem setting https://review.openstack.org/411191 | 03:51 |
| *** baoli has joined #openstack-nova | 03:51 | |
| *** lucasxu has joined #openstack-nova | 03:53 | |
| *** gouthamr has quit IRC | 03:54 | |
| *** nmathew has joined #openstack-nova | 03:56 | |
| *** yamahata_ has quit IRC | 03:57 | |
| *** Dinesh_Bhor has joined #openstack-nova | 03:59 | |
| *** fragatin_ has joined #openstack-nova | 03:59 | |
| *** fragati__ has joined #openstack-nova | 04:01 | |
| *** yuntongjin has quit IRC | 04:02 | |
| *** fragatina has quit IRC | 04:03 | |
| *** fragatin_ has quit IRC | 04:04 | |
| *** tojuvone has quit IRC | 04:04 | |
| *** liusheng has quit IRC | 04:05 | |
| *** fragati__ has quit IRC | 04:05 | |
| *** liusheng has joined #openstack-nova | 04:07 | |
| *** gongysh has quit IRC | 04:12 | |
| openstackgerrit | Huan Xie proposed openstack/nova master: Make xenserver driver compatible with assert_can_migrate https://review.openstack.org/438792 | 04:12 |
| *** kaisers has quit IRC | 04:12 | |
| *** hongbin has quit IRC | 04:17 | |
| *** kaisers has joined #openstack-nova | 04:20 | |
| *** kaisers has quit IRC | 04:22 | |
| *** gcb has quit IRC | 04:22 | |
| *** crushil has quit IRC | 04:22 | |
| *** psachin has joined #openstack-nova | 04:23 | |
| *** tlian has quit IRC | 04:29 | |
| *** gcb has joined #openstack-nova | 04:34 | |
| *** adisky_ has joined #openstack-nova | 04:37 | |
| *** tiendc has joined #openstack-nova | 04:38 | |
| *** sudipto has joined #openstack-nova | 04:40 | |
| *** sudipto_ has joined #openstack-nova | 04:40 | |
| *** sudipto_ has quit IRC | 04:41 | |
| *** sudipto has quit IRC | 04:41 | |
| *** sudipto has joined #openstack-nova | 04:43 | |
| *** sudipto_ has joined #openstack-nova | 04:43 | |
| *** Jeffrey4l__ has quit IRC | 04:49 | |
| *** trinaths has joined #openstack-nova | 04:50 | |
| *** Jeffrey4l__ has joined #openstack-nova | 04:52 | |
| *** kaisers has joined #openstack-nova | 04:52 | |
| *** baoli has quit IRC | 04:54 | |
| *** crushil has joined #openstack-nova | 04:54 | |
| *** sreeram_v has joined #openstack-nova | 04:54 | |
| *** lucasxu has quit IRC | 04:54 | |
| *** trinaths1 has joined #openstack-nova | 04:57 | |
| *** trinaths has quit IRC | 04:57 | |
| *** unicell has joined #openstack-nova | 04:58 | |
| *** sudipto_ has quit IRC | 04:58 | |
| *** sudipto has quit IRC | 04:59 | |
| *** sridharg has joined #openstack-nova | 05:00 | |
| *** ayogi has joined #openstack-nova | 05:06 | |
| *** udesale has joined #openstack-nova | 05:19 | |
| *** karthiks has joined #openstack-nova | 05:20 | |
| *** fragatina has joined #openstack-nova | 05:20 | |
| *** fragatina has quit IRC | 05:21 | |
| *** fragatina has joined #openstack-nova | 05:21 | |
| *** yamamoto has quit IRC | 05:24 | |
| *** trinaths1 has quit IRC | 05:26 | |
| *** trinaths has joined #openstack-nova | 05:26 | |
| *** diga has joined #openstack-nova | 05:29 | |
| *** udesale__ has joined #openstack-nova | 05:33 | |
| *** prateek has joined #openstack-nova | 05:34 | |
| *** udesale has quit IRC | 05:35 | |
| *** armax has quit IRC | 05:36 | |
| *** yuntongjin has joined #openstack-nova | 05:39 | |
| *** jaosorior has joined #openstack-nova | 05:41 | |
| *** larainema has quit IRC | 05:42 | |
| *** kaisers has quit IRC | 05:48 | |
| *** hshiina has joined #openstack-nova | 05:52 | |
| *** unicell1 has joined #openstack-nova | 05:56 | |
| *** unicell has quit IRC | 05:56 | |
| *** phuongnh has quit IRC | 05:59 | |
| *** phuongnh has joined #openstack-nova | 05:59 | |
| *** salv-orlando has joined #openstack-nova | 06:03 | |
| *** yamamoto has joined #openstack-nova | 06:04 | |
| *** phuongnh has quit IRC | 06:10 | |
| *** Jeffrey4l__ is now known as Jeffrey4l | 06:10 | |
| *** yamamoto has quit IRC | 06:10 | |
| *** phuongnh has joined #openstack-nova | 06:11 | |
| *** yamamoto has joined #openstack-nova | 06:11 | |
| *** fragatina has quit IRC | 06:12 | |
| *** tuanluong has joined #openstack-nova | 06:13 | |
| *** sacharya has quit IRC | 06:13 | |
| *** tojuvone has joined #openstack-nova | 06:14 | |
| *** sacharya has joined #openstack-nova | 06:14 | |
| *** ratailor has joined #openstack-nova | 06:17 | |
| *** salv-orl_ has joined #openstack-nova | 06:18 | |
| *** sacharya has quit IRC | 06:19 | |
| *** salv-orl_ has quit IRC | 06:20 | |
| *** salv-orl_ has joined #openstack-nova | 06:20 | |
| *** yamamoto has quit IRC | 06:21 | |
| *** salv-orlando has quit IRC | 06:21 | |
| *** nkrinner_afk is now known as nkrinner | 06:24 | |
| *** hieulq_ has joined #openstack-nova | 06:26 | |
| *** gk-1wm-su has joined #openstack-nova | 06:26 | |
| *** gk-1wm-su has left #openstack-nova | 06:26 | |
| *** tovin07_ has joined #openstack-nova | 06:27 | |
| *** masuberu has quit IRC | 06:27 | |
| *** artom has quit IRC | 06:28 | |
| *** yamamoto has joined #openstack-nova | 06:29 | |
| *** abhishekk has joined #openstack-nova | 06:29 | |
| *** satyar has joined #openstack-nova | 06:30 | |
| *** yamamoto has quit IRC | 06:30 | |
| *** ratailor has quit IRC | 06:35 | |
| *** udesale__ has quit IRC | 06:35 | |
| *** yamahata_ has joined #openstack-nova | 06:35 | |
| *** kaisers has joined #openstack-nova | 06:41 | |
| *** artom has joined #openstack-nova | 06:42 | |
| *** raunak has quit IRC | 06:45 | |
| *** mjura has joined #openstack-nova | 06:46 | |
| *** ratailor has joined #openstack-nova | 06:46 | |
| *** thorst has joined #openstack-nova | 06:46 | |
| *** yamamoto has joined #openstack-nova | 06:47 | |
| *** yamamoto has quit IRC | 06:47 | |
| *** sudipto_ has joined #openstack-nova | 06:48 | |
| *** sudipto has joined #openstack-nova | 06:48 | |
| *** thorst has quit IRC | 06:51 | |
| *** tojuvone has quit IRC | 06:56 | |
| *** tojuvone has joined #openstack-nova | 06:56 | |
| *** ljxiash has quit IRC | 07:00 | |
| *** ljxiash has joined #openstack-nova | 07:00 | |
| *** ygl has joined #openstack-nova | 07:03 | |
| *** ygl has left #openstack-nova | 07:03 | |
| *** ljxiash has quit IRC | 07:05 | |
| *** karthiks has quit IRC | 07:05 | |
| *** ljxiash has joined #openstack-nova | 07:06 | |
| *** ljxiash has quit IRC | 07:07 | |
| *** ljxiash has joined #openstack-nova | 07:07 | |
| *** ijw has joined #openstack-nova | 07:07 | |
| *** Oku_OS-away is now known as Oku_OS | 07:08 | |
| *** artom has quit IRC | 07:09 | |
| *** yamamoto has joined #openstack-nova | 07:10 | |
| *** rcernin has joined #openstack-nova | 07:11 | |
| *** ijw has quit IRC | 07:12 | |
| *** zsli has joined #openstack-nova | 07:13 | |
| *** yamamoto has quit IRC | 07:13 | |
| *** andreas_s has joined #openstack-nova | 07:13 | |
| *** Kevin_Zheng_ has joined #openstack-nova | 07:14 | |
| *** sacharya has joined #openstack-nova | 07:15 | |
| *** Kevin_Zheng_ is now known as Kevin_Zheng | 07:17 | |
| *** karthiks has joined #openstack-nova | 07:18 | |
| *** hieulq_ has quit IRC | 07:23 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_xenapi[1] https://review.openstack.org/438825 | 07:26 |
| *** lpetrut has joined #openstack-nova | 07:28 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Remove mox from nova/tests/unit/virt/xenapi/test_xenapi.py[1] https://review.openstack.org/438825 | 07:28 |
| *** tesseract has joined #openstack-nova | 07:34 | |
| *** markus_z has joined #openstack-nova | 07:34 | |
| *** gmann has quit IRC | 07:35 | |
| *** crushil has quit IRC | 07:36 | |
| *** lpetrut has quit IRC | 07:36 | |
| *** gmann has joined #openstack-nova | 07:37 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 07:38 |
| *** jpena|away is now known as jpena | 07:41 | |
| *** kaisers_ has joined #openstack-nova | 07:45 | |
| *** carthaca_ has joined #openstack-nova | 07:46 | |
| *** jpena is now known as jpena|away | 07:54 | |
| *** leifz has quit IRC | 07:54 | |
| *** yuntongjin has quit IRC | 07:55 | |
| *** leifz has joined #openstack-nova | 07:56 | |
| *** tovin07_ has quit IRC | 07:57 | |
| Dinesh_Bhor | Hi all, I have fresh OpenStack installed on Ubuntu 14.04 but I am unable to boot vm. Logs in n-cpu says that: "InternalError: Nova requires libvirt version 1.2.9 or greater." | 07:59 |
| *** jpena|away is now known as jpena | 08:01 | |
| Dinesh_Bhor | I tried to update libvirt version but it only installs 1.2.2. I have ubuntu 14.04 LTS. | 08:01 |
| Dinesh_Bhor | I found some bug here: https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1532460 | 08:02 |
| openstack | Launchpad bug 1532460 in libvirt (Ubuntu) "Please upgrade to allow virsh blockcommit in Ubuntu 14. libvirt-1.2.9 (and above)" [Undecided,Fix released] | 08:02 |
| *** acabot has joined #openstack-nova | 08:03 | |
| *** kaisers_ has quit IRC | 08:07 | |
| *** ljxiash has quit IRC | 08:08 | |
| *** gongysh has joined #openstack-nova | 08:08 | |
| *** yuntongjin has joined #openstack-nova | 08:08 | |
| *** ljxiash has joined #openstack-nova | 08:08 | |
| *** ljxiash has quit IRC | 08:12 | |
| *** yamamoto has joined #openstack-nova | 08:13 | |
| *** pcaruana has joined #openstack-nova | 08:17 | |
| *** markus_z has quit IRC | 08:17 | |
| *** yamamoto has quit IRC | 08:19 | |
| *** kaisers_ has joined #openstack-nova | 08:19 | |
| *** sreeram__ has joined #openstack-nova | 08:21 | |
| *** ltomasbo|away is now known as ltomasbo | 08:21 | |
| *** sreera___ has joined #openstack-nova | 08:22 | |
| *** romainh has joined #openstack-nova | 08:22 | |
| *** masber has joined #openstack-nova | 08:24 | |
| *** sreeram_v has quit IRC | 08:25 | |
| *** sreeram__ has quit IRC | 08:26 | |
| *** moshele has joined #openstack-nova | 08:29 | |
| *** diga has quit IRC | 08:29 | |
| *** sreera___ has quit IRC | 08:31 | |
| *** yingjun has joined #openstack-nova | 08:33 | |
| *** romainh1 has joined #openstack-nova | 08:33 | |
| *** romainh1 has quit IRC | 08:34 | |
| *** romainh1 has joined #openstack-nova | 08:34 | |
| *** markus_z has joined #openstack-nova | 08:34 | |
| *** romainh has quit IRC | 08:35 | |
| *** sreeram_v has joined #openstack-nova | 08:36 | |
| *** diga has joined #openstack-nova | 08:41 | |
| gmann | alex_xu: mriedem: this looks good but new error code bug me - https://review.openstack.org/#/c/429476/4 | 08:43 |
| *** kornica has quit IRC | 08:46 | |
| *** zsli has quit IRC | 08:47 | |
| *** karimb has joined #openstack-nova | 08:48 | |
| *** zsli has joined #openstack-nova | 08:48 | |
| *** ljxiash has joined #openstack-nova | 08:48 | |
| *** owalsh-afk is now known as owalsh | 08:57 | |
| *** vsaienk0 has joined #openstack-nova | 09:00 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** Jeffrey4l has quit IRC | 09:00 | |
| *** Jeffrey4l has joined #openstack-nova | 09:00 | |
| vsaienk0 | Hello nova team I'm wondered why multinode tempest test is skipped on grenade multinode in base tests? http://logs.openstack.org/76/429476/4/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/f6613ff/logs/grenade.sh.txt.gz#_2017-02-26_23_56_30_146 | 09:00 |
| *** aarefiev_afk is now known as aarefiev | 09:02 | |
| vsaienk0 | I meant is it expected behavior? | 09:05 |
| frickler | can someone explain the difference between default_availability_zone and default_schedule_zone? both descriptions look roughly the same to me here: https://docs.openstack.org/draft/config-reference/compute/config-options.html#id3 | 09:08 |
| *** zzzeek has joined #openstack-nova | 09:12 | |
| *** yamamoto has joined #openstack-nova | 09:15 | |
| *** yamamoto has quit IRC | 09:16 | |
| *** yamamoto has joined #openstack-nova | 09:16 | |
| *** udesale has joined #openstack-nova | 09:17 | |
| *** trinaths has left #openstack-nova | 09:20 | |
| *** fandi has joined #openstack-nova | 09:20 | |
| *** zioproto has joined #openstack-nova | 09:23 | |
| *** tiendc has quit IRC | 09:25 | |
| openstackgerrit | Gao Yuan proposed openstack/nova master: Stop using mox stubs in nova.tests.unit.api.openstack.compute.test_serversV21 https://review.openstack.org/438873 | 09:25 |
| *** kaisers_ has quit IRC | 09:25 | |
| *** dimtruck is now known as zz_dimtruck | 09:28 | |
| *** zsli has quit IRC | 09:36 | |
| *** derekh has joined #openstack-nova | 09:37 | |
| johnthetubaguy | frickler: I did look into all that AZ stuff when we tidied up the descriptions, but you are right, its not very clear from that | 09:38 |
| frickler | johnthetubaguy: I just opened https://bugs.launchpad.net/openstack-manuals/+bug/1668542 because looking at the code I became even more confused | 09:39 |
| openstack | Launchpad bug 1668542 in openstack-manuals "nova.conf - az configuration options in Configuration Reference" [Undecided,New] | 09:39 |
| *** jaosorior has quit IRC | 09:39 | |
| johnthetubaguy | frickler: that probably needs to be a nova bug, the text lives in nova | 09:39 |
| frickler | johnthetubaguy: added. probably the "open bug" link on the bug page could need amending then, too | 09:40 |
| *** jaosorior has joined #openstack-nova | 09:40 | |
| frickler | s/bug page/doc page/ | 09:40 |
| johnthetubaguy | frickler: yeah, it probably does, not sure if you can change that per page, probably worth a manuals bug for that! | 09:42 |
| johnthetubaguy | frickler: ah, so one is about where the host lives by default, and one is about where instances go, I think... | 09:44 |
| *** yamahata_ has quit IRC | 09:45 | |
| *** gongysh has quit IRC | 09:46 | |
| frickler | johnthetubaguy: yes, that is my interpretation, but would be nice to get it confirmed, then I could make a patch for the docs | 09:50 |
| *** lucas-afk is now known as lucasagomes | 09:50 | |
| frickler | johnthetubaguy: hmm, the source for the page seems to be ./doc/config-reference/source/compute/config-options.rst in openstack-manuals, though | 09:52 |
| johnthetubaguy | frickler: that thing is auto-generated by a script, so its a two part ifx | 09:57 |
| johnthetubaguy | fix | 09:57 |
| *** yuntongjin has quit IRC | 09:57 | |
| openstackgerrit | John Garbutt proposed openstack/nova master: Fix default_availability_zone docs https://review.openstack.org/438887 | 09:58 |
| johnthetubaguy | frickler: does that patch look like it makes it cleaer? | 09:59 |
| johnthetubaguy | hmm, not sure my wording is brilliant, but suggestions welcome | 10:00 |
| *** gcb has quit IRC | 10:00 | |
| *** artom has joined #openstack-nova | 10:01 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Backlog spec on preemptible instances https://review.openstack.org/438640 | 10:01 |
| *** artom has quit IRC | 10:01 | |
| *** artom has joined #openstack-nova | 10:02 | |
| *** GnomeKing has joined #openstack-nova | 10:02 | |
| *** sapcc-bot has quit IRC | 10:03 | |
| frickler | johnthetubaguy: ah, nice, I was just about to do a patch for doc/config-reference/source/tables/nova-availabilityzones.rst , will that get synced from the data in nova? or should I do my patch in parallel anyway? | 10:04 |
| *** sapcc-bot has joined #openstack-nova | 10:04 | |
| johnthetubaguy | frickler: I think the sync is manual, there is a readme on how to do that in that repo somewhere | 10:04 |
| johnthetubaguy | frickler: but I think you could checkout my patch on your system, inside the docs build system, to pull in the change | 10:04 |
| johnthetubaguy | frickler: maybe its easier just to patch it directly for now! yeah, do it in parallel, and have a depends-on header pointing to my patch I think | 10:05 |
| * johnthetubaguy feels like this should have come up before, but I don't remember it happening before | 10:05 | |
| frickler | johnthetubaguy: will do after you replied to my small comments ;) | 10:06 |
| * johnthetubaguy looks... | 10:07 | |
| bauzas | morning folks | 10:08 |
| *** jed56 has joined #openstack-nova | 10:09 | |
| bauzas | johnthetubaguy: frickler: context around AZs ? | 10:09 |
| *** hoangcx has quit IRC | 10:10 | |
| bauzas | johnthetubaguy: frickler: just in case you need more knowledge, there are kludges with those opts https://docs.openstack.org/developer/nova/aggregates.html#availability-zones-azs | 10:10 |
| bauzas | see the warning section ^ | 10:10 |
| johnthetubaguy | yeah, its the config doc that is confusing | 10:10 |
| openstackgerrit | John Garbutt proposed openstack/nova master: Fix default_availability_zone docs https://review.openstack.org/438887 | 10:10 |
| johnthetubaguy | bauzas: its very tempting to just deprecate that one | 10:11 |
| * bauzas looks at the change | 10:11 | |
| bauzas | ah, gotcha | 10:11 |
| johnthetubaguy | frickler: bauzas makes a good point though, the AZ stuff was never really "finished" its all a bit broken honestly. Now placement API work should mean we can look at fixing some of that all soo | 10:11 |
| johnthetubaguy | n | 10:11 |
| bauzas | so, lemme clarify some tribal knowledge | 10:12 |
| johnthetubaguy | bauzas: sure | 10:12 |
| bauzas | first, we have a weird concept of a "default" AZ | 10:13 |
| bauzas | ie. controlplane services have AZs too | 10:13 |
| bauzas | while we can't actually schedule them as a target :) | 10:13 |
| *** fandi has quit IRC | 10:13 | |
| bauzas | secondly, we differenciate hosts vs. instances when they have no AZs | 10:14 |
| bauzas | instead of saying "I don't care at all about AZs so the AZ field should be blank" | 10:15 |
| bauzas | we then have 2 opts that are different: | 10:15 |
| bauzas | if you don't care about AZs for your instance, then you need to set it blank | 10:15 |
| bauzas | but if you don't care about AZs for your hosts, then you need to set it as "nova" | 10:15 |
| bauzas | so much fun around | 10:15 |
| bauzas | anyway, given that, fixing AZs a long way towards, and I don't want us to diverge from the placement effort | 10:16 |
| bauzas | somehow, that has to be a trait | 10:16 |
| johnthetubaguy | bauzas: yeah, in summary, its nuts | 10:17 |
| *** sambetts_ is now known as sambetts | 10:17 | |
| bauzas | yeah, I'm personnally in favor of deprecating the whole concept of AZs and just leave it at the API level | 10:17 |
| bauzas | instead, computes have location traits | 10:17 |
| johnthetubaguy | so... once we get the cinder "AZ" and routed networks sorted, it gets interesting again | 10:17 |
| bauzas | and instances could request affinity based on those location traits | 10:17 |
| johnthetubaguy | ah, generic location traits are my preference, but AZ is just one of those | 10:18 |
| bauzas | long-term anyway | 10:18 |
| johnthetubaguy | there was a great spec on that from randall nortman, somewhere | 10:18 |
| bauzas | so in the meantime, what we can fix is docs | 10:18 |
| bauzas | but there could be room for some slight improvement tho | 10:18 |
| johnthetubaguy | changing up server groups to deal with generic location specifiers, and multiple of them | 10:18 |
| bauzas | yeah I remember that one | 10:18 |
| johnthetubaguy | thats the one our users really want | 10:19 |
| bauzas | I know | 10:19 |
| johnthetubaguy | cool, just making that the explicit goal I guess | 10:19 |
| johnthetubaguy | we might want to make that a backlog spec actually | 10:19 |
| bauzas | tbh, affinity based on location specifiers is an old old story I have in my pipe | 10:20 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: objects: Add attachment_id to BlockDeviceMapping https://review.openstack.org/437665 | 10:20 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: db: Add attachment_id to block_device_mapping https://review.openstack.org/437597 | 10:20 |
| johnthetubaguy | lyarwood: is that the first steps of the Cinder thing? | 10:20 |
| bauzas | anyway, let's move on | 10:20 |
| lyarwood | johnthetubaguy: yeah I guess, I had the bdm uuid changes still in my backlog so thought I'd just do the same again for attachment_id | 10:21 |
| johnthetubaguy | lyarwood: ah, good idea | 10:21 |
| bauzas | wow, Gerrit is in a bad shape | 10:21 |
| *** kaisers_ has joined #openstack-nova | 10:21 | |
| johnthetubaguy | lyarwood: I have this one that could do with some eyes: https://review.openstack.org/#/c/373203/ | 10:21 |
| johnthetubaguy | I mean you could just poke your eyes yourself, but I have a lovely spec to do that for you | 10:21 |
| johnthetubaguy | bauzas: so I thought my internet was just slow, but you might be right | 10:22 |
| bauzas | HTTP502 killed me | 10:22 |
| bauzas | so, no, this is not internet :p | 10:22 |
| johnthetubaguy | oh, its tipped over into death land then | 10:22 |
| lyarwood | johnthetubaguy: haha yeah, I'll go over the spec again this morning | 10:22 |
| johnthetubaguy | lyarwood: awesome, thank you | 10:22 |
| johnthetubaguy | I have a feeling I forgot some details from the discussion | 10:22 |
| bauzas | johnthetubaguy: I actually wonder if we couldn't just try to fix default_availability_zone as wezll | 10:23 |
| johnthetubaguy | bauzas: ? | 10:23 |
| bauzas | johnthetubaguy: having it set to none by default would just fix a looooooot of problems | 10:23 |
| bauzas | s/problems/misunderstandings | 10:23 |
| bauzas | johnthetubaguy: and we could then deprecate default_schedule_zon | 10:24 |
| johnthetubaguy | at this point most folks are used to seeing the Nova AZ default I guess, I could go either way | 10:24 |
| johnthetubaguy | hmm, I duno, some folks want to send all builds into a certain zone at times, I think thats why HP public cloud added that | 10:24 |
| bauzas | johnthetubaguy: lemme rephrase, if we could have default_availability_zone having None as default and having default_schedule_zone be deprecated, that would be a nice improvement | 10:24 |
| johnthetubaguy | ah, I just mixed them up | 10:25 |
| bauzas | johnthetubaguy: you mean, HPE wanted to default add the AZ without asking the user to do this ? | 10:25 |
| johnthetubaguy | yeah, maybe, duno if that breaks things, like existing deployments | 10:25 |
| bauzas | that's probably then a good reason for *not | 10:25 |
| bauzas | *not* deprecating it | 10:25 |
| *** sandanar has joined #openstack-nova | 10:26 | |
| *** kevinz has quit IRC | 10:26 | |
| johnthetubaguy | bauzas: its just a scheduling decision, if the user didn't specify it, it probably should be a scheduler filter switch really | 10:26 |
| johnthetubaguy | its all a bit of a hack right now | 10:26 |
| bauzas | johnthetubaguy: that conf opt is more than that | 10:26 |
| *** kaisers_ has quit IRC | 10:26 | |
| bauzas | johnthetubaguy: if you set default_schedule_zone to something else | 10:26 |
| johnthetubaguy | if it is, then the docs are wrong | 10:26 |
| bauzas | johnthetubaguy: then your instance will be constrainted forever to that AZ | 10:27 |
| bauzas | johnthetubaguy: yeah, I'm about to review that patch | 10:27 |
| johnthetubaguy | yep, anywhere you land its meant to stay there forever, I thought | 10:27 |
| frickler | so let me just add our deployment scenario for reference: we have three AZs of compute nodes with local storage and do maintainance on them in turn for a single AZ at a time. | 10:27 |
| bauzas | johnthetubaguy: yup, because instance.az should never be modified | 10:27 |
| johnthetubaguy | right, so its just a scheduling decision | 10:28 |
| frickler | customers that want HA for their service will need to deploy three instances, one per AZ | 10:28 |
| johnthetubaguy | frickler: totally, that makes sense | 10:28 |
| *** ljxiash has quit IRC | 10:28 | |
| bauzas | johnthetubaguy: in terms of placement, you're right | 10:28 |
| openstackgerrit | Gao Yuan proposed openstack/nova master: Stop using mox stubs in nova.tests.unit.api.openstack.compute.test_serversV21 https://review.openstack.org/438873 | 10:28 |
| *** yingjun has quit IRC | 10:28 | |
| *** sacharya has quit IRC | 10:29 | |
| bauzas | johnthetubaguy: that conf opt default instances to be placed on a specific AZ, that's what it means | 10:29 |
| johnthetubaguy | frickler: the server groups stuff, once its fixed up, is how you can do that in a better way, but we got loads of placement work before we get there | 10:29 |
| johnthetubaguy | bauzas: yep, I thats what I was trying to say, anyways | 10:29 |
| bauzas | (but there are ways to migrate your server out of that AZ :D) | 10:29 |
| * johnthetubaguy nods | 10:29 | |
| frickler | now we still would like to have "don't care" instances scheduled to any of the AZs. which seems difficult, at least in the dashboard | 10:29 |
| johnthetubaguy | frickler: ah, that should be the default, the dashboard may have broken that | 10:30 |
| frickler | any my idea was to set default_schedule_zone differently on three API servers that are accessed round robin | 10:30 |
| johnthetubaguy | frickler: like "nova boot myserver image_7" should do that | 10:30 |
| bauzas | frickler: that's a separate usecase | 10:30 |
| *** jsheeren has joined #openstack-nova | 10:31 | |
| johnthetubaguy | frickler: so I have a feeling we spoke about this a few months back, and I got confused then too | 10:31 |
| bauzas | frickler: setting default_schedule_zone to None (well, to that ugly empty string rather) means "I don't care about AZs" | 10:31 |
| jsheeren | hi all, i looking at curl calls and filtering results on listing the servers | 10:31 |
| johnthetubaguy | frickler: I thought default_schedule_zone = None does what you want | 10:31 |
| bauzas | but it doesn't force the instance to stick with that AZ | 10:31 |
| johnthetubaguy | bauzas: it should do I thought | 10:32 |
| bauzas | frickler: for your usecase, you could just use a scheduling filter tho | 10:32 |
| jsheeren | with a curl to "https://<api>:8774/v2.1/<tenant>/servers/detail" i can get all the servers for that tenant | 10:32 |
| jsheeren | if i use "https://<api>:8774/v2.1/<tenant>/servers/detail?name=blah" i can filter to show only the blah instance | 10:32 |
| bauzas | johnthetubaguy: nope, if you wanna RR between AZs, you need to set default_schedule_zone differently between your API workers | 10:33 |
| jsheeren | but when i try to filter on metadata, it just returns all the servers instead of filtering on the metadata | 10:33 |
| johnthetubaguy | jsheeren: loads of details on here: https://developer.openstack.org/api-ref/compute/ | 10:33 |
| johnthetubaguy | jsheeren: I don't think you can filter by everything (its a performance thing), and in some cases its a policy thing | 10:33 |
| johnthetubaguy | bauzas: whaaaat? | 10:34 |
| bauzas | frickler: see what I said above ^ | 10:34 |
| jsheeren | filter on metadata using: "https://<api>:8774/v2.1/<tenant>/servers/detail?metadata=%7b%22foo%22:%22bar%22%7d" | 10:34 |
| *** gszasz has joined #openstack-nova | 10:34 | |
| bauzas | johnthetubaguy: yup | 10:34 |
| bauzas | johnthetubaguy: lemme clarify | 10:34 |
| *** sreeram_v has quit IRC | 10:35 | |
| jsheeren | johnthetubaguy: aha,gonna check the policy first; i read through the developer api-ref doc a few times now :) | 10:35 |
| *** avolkov has joined #openstack-nova | 10:35 | |
| bauzas | johnthetubaguy: again, default_schedule_zone is just a terrible thing for saying what should be the default if the user hasn't specific --az in their boot command | 10:35 |
| *** yamamoto has quit IRC | 10:35 | |
| johnthetubaguy | jsheeren: all the available filters are listed in the docs now (thats a recent change): https://developer.openstack.org/api-ref/compute/?expanded=list-servers-detail#list-servers | 10:36 |
| jsheeren | compute policy: "compute:get_instance_metadata": "rule:admin_or_owner" i would assume i can see the metadata, which i can, cause it's in the full detail | 10:36 |
| bauzas | the semantics of the empty string meaning the exact same thing as if the user hasn't specify --az : ie. the instance won't be AZ-constrained | 10:36 |
| johnthetubaguy | jsheeren: you can see it, but not query based on it | 10:36 |
| *** satyar has quit IRC | 10:36 | |
| johnthetubaguy | jsheeren: not 100% sure, you can query based on server tags though | 10:36 |
| frickler | bauzas: I will look into writing a custom filter, thx | 10:37 |
| johnthetubaguy | bauzas: right, but None means just pick any, which appears to be the request? | 10:37 |
| johnthetubaguy | bauzas: I am clearly missing something | 10:37 |
| bauzas | johnthetubaguy: None means "leave that field unset" | 10:37 |
| johnthetubaguy | bauzas: right, but doesn't it get set when it lands on a compute node | 10:38 |
| bauzas | johnthetubaguy: I mean, None means "leave the Instance object having the AZ field unset" | 10:38 |
| bauzas | exactly | 10:38 |
| bauzas | if the Instance object doesn't have an AZ information, it will never stick to the AZ the host where it was placed belongs to | 10:38 |
| jsheeren | johnthetubaguy: https://github.com/openstack/nova/blob/stable/mitaka/nova/compute/api.py#L2112 | 10:38 |
| jsheeren | metadata is listed in the filter_mapping | 10:39 |
| bauzas | there are 2 colocality informations : one for the instance and one for the host | 10:39 |
| bauzas | an host can belong to an AZ | 10:40 |
| bauzas | and one instance can belong to an host | 10:40 |
| *** laci has joined #openstack-nova | 10:40 | |
| laci | hi! | 10:40 |
| bauzas | so, this is transitive | 10:40 |
| *** laci has left #openstack-nova | 10:41 | |
| bauzas | one instance can be in one AZ by the transitivity of the relationship AZ -> host -> instance | 10:41 |
| *** markus_z has quit IRC | 10:41 | |
| *** phuongnh has quit IRC | 10:41 | |
| bauzas | that's the default | 10:41 |
| bauzas | but you can give a specific colocality information on one instance if you specify which AZ it should belong to | 10:42 |
| *** mvk has quit IRC | 10:42 | |
| bauzas | in that case, we would force to honor the transitivity by making sure we pick hosts only belonging to the same AZ | 10:42 |
| johnthetubaguy | bauzas: was looking in the code, I was expecting AZ to get set to the host value when we set the host on the instance, somewhere near here: https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L579 | 10:43 |
| johnthetubaguy | bauzas: I can't find the right place though right now | 10:43 |
| bauzas | johnthetubaguy: I would violently -1 it if I was seeing that code | 10:43 |
| *** udesale has quit IRC | 10:43 | |
| *** namnh has quit IRC | 10:43 | |
| johnthetubaguy | bauzas: I guess I don't get why | 10:43 |
| bauzas | johnthetubaguy: because the AZ information attached to the instance is what the user expressed | 10:43 |
| johnthetubaguy | thats in the build request | 10:43 |
| bauzas | not the result of the placement decision we made | 10:44 |
| bauzas | well | 10:44 |
| johnthetubaguy | yeah, that sounds wrong to me | 10:44 |
| bauzas | first, build request is only temporary | 10:44 |
| johnthetubaguy | yeah, I was meaning the other one | 10:44 |
| bauzas | secondly, I agree, it's rather RequestSpec | 10:44 |
| johnthetubaguy | yeah, request spec, my bad | 10:44 |
| bauzas | but I wrote that object later | 10:44 |
| bauzas | so | 10:44 |
| bauzas | a move could be to totally remove the instance AZfield | 10:45 |
| bauzas | AZ field | 10:45 |
| bauzas | I mean, deprecate it at the object level, and wait for a major version bump to remove it | 10:45 |
| johnthetubaguy | instance.host tells you the current host, instance.az feels like it should be similar | 10:45 |
| johnthetubaguy | but thats just me | 10:45 |
| johnthetubaguy | anyways, I think I get enough for the docs patch now | 10:46 |
| jsheeren | johnthetubaguy: tags look like an option, but we're still on mitaka .. :( | 10:46 |
| bauzas | johnthetubaguy: no, that's then duplicate information | 10:46 |
| bauzas | johnthetubaguy: I by far prefer to only keep instance.host and leave the transitivity to determine an instance AZ | 10:46 |
| johnthetubaguy | bauzas: it is duplicating host, but its kinda useful, its the AZ cached at the time the instance landed on the host | 10:46 |
| johnthetubaguy | just like flavor | 10:46 |
| bauzas | there are a full list of bugs related to the fact we badly manage that relationship | 10:47 |
| johnthetubaguy | but anyways, I clearly see this differently, needs a spec to sort that out | 10:47 |
| *** ijw has joined #openstack-nova | 10:47 | |
| *** ijw has quit IRC | 10:47 | |
| bauzas | well, you're kinda busy this cycle, so I could give an hand | 10:47 |
| johnthetubaguy | we should record, where we first landed, and what the user requested, I believe | 10:47 |
| johnthetubaguy | bauzas: I have no time to sort that out this cycle, if you have cycles feel free | 10:48 |
| bauzas | but honestly, touching AZs is like trying to unfuse a bomb | 10:48 |
| * johnthetubaguy nods | 10:48 | |
| bauzas | certainly not a thing for people willing to contribute at first hand, unfortunately :( | 10:48 |
| johnthetubaguy | bauzas: maybe a backlog spec to capture the thoughts? | 10:48 |
| *** thorst has joined #openstack-nova | 10:48 | |
| bauzas | sorting all that out took me like 5 cycles | 10:48 |
| bauzas | I could do that | 10:49 |
| *** yamamoto has joined #openstack-nova | 10:49 | |
| bauzas | I'm pretty free this cycle since I really wanted to make reviews | 10:49 |
| johnthetubaguy | bauzas: include the use cases frickler talks about, its easier than the harder server groups one, and its what folks want | 10:49 |
| * bauzas noting it | 10:49 | |
| bauzas | the fact that one would want to RR between AZs ? | 10:49 |
| johnthetubaguy | yeah, just want to capture these requests, I know my product org asked me some questions on that | 10:49 |
| johnthetubaguy | RR? | 10:49 |
| johnthetubaguy | round robin | 10:50 |
| bauzas | round-robin, my bad | 10:50 |
| johnthetubaguy | yeah, not sure its round robin you want | 10:50 |
| johnthetubaguy | you want to balance load between them all | 10:50 |
| bauzas | well, it's a placement question for sure | 10:50 |
| *** sreeram_v has joined #openstack-nova | 10:50 | |
| bauzas | because RR is one way to spread your load | 10:50 |
| johnthetubaguy | like if AZ_3 fills up, send more builds to the other two, if folks don't have a preference | 10:50 |
| bauzas | yeah | 10:51 |
| johnthetubaguy | thats what people *really* want | 10:51 |
| johnthetubaguy | frickler | 10:51 |
| johnthetubaguy | oops | 10:51 |
| johnthetubaguy | frickler please do correct me if I am wrong ^ | 10:51 |
| bauzas | to me, the default_schedule_zone flag would rather be "stick_with_the_AZ_I'm_scheduled_for_all_my_lifecycle" and a boolean as value | 10:52 |
| frickler | well that is one use case. but thinking about it, the other one would be: create three instances in three different AZs | 10:52 |
| bauzas | that's a scheduling thing | 10:52 |
| johnthetubaguy | yeah, thats the server groups one | 10:52 |
| bauzas | you could have different placement strategies | 10:52 |
| bauzas | using weighters or other things | 10:52 |
| johnthetubaguy | if you just create one instance, and you don't care were it goes, try get an even load between all AZs so the folks targeting to a specific AZ are able to get there | 10:53 |
| *** esberglu has joined #openstack-nova | 10:53 | |
| johnthetubaguy | bauzas: there is a clear 80% default here I think | 10:53 |
| bauzas | so, my thoughts are | 10:53 |
| *** thorst has quit IRC | 10:53 | |
| johnthetubaguy | bauzas: lets get them in the spec, I need to get on with other things | 10:53 |
| bauzas | first boot schedule is a placement decision | 10:53 |
| *** sam_nowitzki has quit IRC | 10:53 | |
| bauzas | but a move operation is depending on that first placement decision or not | 10:53 |
| bauzas | yeah, lemme try to capture all the best I can | 10:53 |
| johnthetubaguy | ping me for a review, I am interested it getting that all down on paper | 10:54 |
| *** sreeram_v has quit IRC | 10:55 | |
| *** yamamoto has quit IRC | 10:55 | |
| sfinucan | johnthetubaguy: I think I have all comments on https://review.openstack.org/#/c/364468/ addressed now | 10:56 |
| bauzas | johnthetubaguy: sure thing | 10:56 |
| *** sreeram_v has joined #openstack-nova | 10:57 | |
| jsheeren | johnthetubaguy: small update on filtering on metadata. if you are an admin user, you can use the filter detail?metadata=<urlencoded-metadata-json> but not as a regular user | 10:57 |
| jsheeren | so, this is a policy issue then? | 10:57 |
| *** esberglu has quit IRC | 10:57 | |
| *** salv-orl_ has quit IRC | 10:57 | |
| johnthetubaguy | jsheeren: possibly, its a really nasty DB query, so you probably don't want to allow that | 10:58 |
| jsheeren | johnthetubaguy: in the policy.json everything with metadata in the string has admin_or_owner so i'm confused | 11:00 |
| jsheeren | going to look at the code again .. :-) | 11:00 |
| akijak | hi, I've found a bug in nova concerning rebuilding and 'hw_disk_bus' image property (https://bugs.launchpad.net/nova/+bug/1667667), I'm looking for someone who could give me some directions how to fix that :) can you help? | 11:01 |
| openstack | Launchpad bug 1667667 in OpenStack Compute (nova) "Rebuilding instance ignores image's property hw_disk_bus" [Low,Confirmed] | 11:01 |
| *** iceyao has quit IRC | 11:03 | |
| *** sreeram_v has quit IRC | 11:03 | |
| jsheeren | https://github.com/openstack/nova/blob/stable/mitaka/nova/compute/api.py#L3304 check_policy(context, 'get_all_instance_%s' % metadata_type,instance) .. but in the policy: "compute:get_all_instance_metadata": "rule:admin_or_owner" | 11:03 |
| *** sreeram_v has joined #openstack-nova | 11:04 | |
| *** salv-orlando has joined #openstack-nova | 11:05 | |
| *** sdague has joined #openstack-nova | 11:06 | |
| *** prateek_ has joined #openstack-nova | 11:07 | |
| *** sreeram_v has quit IRC | 11:08 | |
| *** nicolasbock has joined #openstack-nova | 11:09 | |
| *** mvk has joined #openstack-nova | 11:14 | |
| *** prateek has quit IRC | 11:14 | |
| *** sreeram_v has joined #openstack-nova | 11:15 | |
| *** prateek has joined #openstack-nova | 11:15 | |
| *** sreeram_v has quit IRC | 11:19 | |
| *** kaisers_ has joined #openstack-nova | 11:22 | |
| *** ratailor has quit IRC | 11:25 | |
| *** sudipto has quit IRC | 11:32 | |
| *** sudipto_ has quit IRC | 11:32 | |
| johnthetubaguy | bauzas: can you take over that patch and fix up the comments how you think makes sense: https://review.openstack.org/#/c/438887 | 11:36 |
| openstackgerrit | jichenjc proposed openstack/nova master: api-ref: Body verification for os-hypervisors.inc https://review.openstack.org/383691 | 11:36 |
| *** yamamoto has joined #openstack-nova | 11:38 | |
| *** sreeram_v has joined #openstack-nova | 11:38 | |
| *** sridharg has quit IRC | 11:39 | |
| *** kaisers_ has quit IRC | 11:41 | |
| *** nmathew has quit IRC | 11:41 | |
| *** tojuvone has quit IRC | 11:41 | |
| *** tojuvone has joined #openstack-nova | 11:41 | |
| *** karthiks has quit IRC | 11:47 | |
| *** yamamoto has quit IRC | 11:49 | |
| *** sean-k-m1 is now known as sean-k-mooney | 11:49 | |
| *** tbachman has quit IRC | 11:51 | |
| *** yassine has quit IRC | 11:52 | |
| andymccr | are the [placement_database] specifics configured in nova.conf working? I'm noticing that although everything is working and no errors/issues are caused the database specified is empty - wondering if thats a configuration issue my side | 11:52 |
| *** yassine has joined #openstack-nova | 11:53 | |
| *** betherly has quit IRC | 11:53 | |
| johnthetubaguy | andymccr: hmm, thats a bit surprizing for sure | 11:54 |
| *** diga has quit IRC | 11:54 | |
| *** markus_z has joined #openstack-nova | 11:54 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Prevent delete cell0 in nova-manage command https://review.openstack.org/433476 | 11:54 |
| *** betherly has joined #openstack-nova | 11:55 | |
| andymccr | johnthetubaguy: there is a probably high chance its a config issue my side :) but then i almost expect errors somewhere - so thought it'd be worth checking before i release | 11:55 |
| *** jsheeren has left #openstack-nova | 11:57 | |
| *** jsheeren has quit IRC | 11:57 | |
| frickler | andymccr: https://review.openstack.org/362766 does still seem pending. did you check the api-db? | 11:57 |
| *** artom has quit IRC | 11:58 | |
| *** artom has joined #openstack-nova | 11:59 | |
| *** salv-orlando has quit IRC | 12:01 | |
| andymccr | frickler: ahh ok thanks! so the separation of the database hasn't merged yet - i figured it must be using the nova_api one since it was working and not creating errors. | 12:01 |
| *** ildikov has quit IRC | 12:01 | |
| andymccr | but good to know i havent misconfigured it (yet) :) | 12:02 |
| *** tojuvone has quit IRC | 12:02 | |
| *** tojuvone has joined #openstack-nova | 12:03 | |
| *** ildikov has joined #openstack-nova | 12:03 | |
| *** smatzek has joined #openstack-nova | 12:04 | |
| sean-k-mooney | sfinucan: free to pop into the neutron channel for a sec | 12:04 |
| *** karthiks has joined #openstack-nova | 12:04 | |
| *** johnthetubaguy has quit IRC | 12:04 | |
| *** smatzek_ has joined #openstack-nova | 12:05 | |
| *** smatzek has quit IRC | 12:05 | |
| sean-k-mooney | sfinucan: kevinbenton: has a patch for port filtering for os-vif that relates to the security behavior we were disscussing last week | 12:05 |
| sean-k-mooney | https://review.openstack.org/#/c/438272/2 | 12:05 |
| frickler | andymccr: still the current situation seems pretty broken, https://review.openstack.org/#/c/349021/13/releasenotes/notes/placement-database-5aa94ece29eab820.yaml is in place and if people follow that advice, their setup is broken. guess you could open a bug for that | 12:07 |
| *** bkopilov has quit IRC | 12:07 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936 | 12:08 |
| openstackgerrit | Kevin Benton proposed openstack/os-vif master: Don't install iptables rules if neutron is filtering https://review.openstack.org/438272 | 12:09 |
| *** fungi has quit IRC | 12:11 | |
| *** tuanluong has quit IRC | 12:12 | |
| *** bvanhav_ has joined #openstack-nova | 12:13 | |
| *** aysyd has joined #openstack-nova | 12:14 | |
| *** johnthetubaguy has joined #openstack-nova | 12:14 | |
| andymccr | frickler: hmm that placement sync bit has been removed | 12:14 |
| * johnthetubaguy hmm, got disconnected there | 12:16 | |
| johnthetubaguy | andymccr: frickler was right, we send stuff to the API db | 12:16 |
| johnthetubaguy | andymccr: if you configure the compute nodes to have the api-db point to the placement API, you might get what you want | 12:16 |
| johnthetubaguy | its not officially supported right now, but thats probably what you want | 12:16 |
| johnthetubaguy | oops | 12:16 |
| johnthetubaguy | I mean placement-api conf has api-db point to placement-db | 12:16 |
| johnthetubaguy | andymccr: ^ | 12:16 |
| *** bvanhav_ is now known as bvanhav | 12:16 | |
| *** iceyao has joined #openstack-nova | 12:16 | |
| johnthetubaguy | thats assuming all the tables are created in there now | 12:16 |
| johnthetubaguy | but doing api-db sync on a node that has api pointing to placement | 12:16 |
| johnthetubaguy | api-db pointing, that is | 12:17 |
| *** fungi has joined #openstack-nova | 12:17 | |
| *** bvanhav has quit IRC | 12:19 | |
| *** bvanhav_ has joined #openstack-nova | 12:20 | |
| andymccr | johnthetubaguy: not really looking to work around it until its ready i guess :) hoping there is a decent migration path though so its unobtrusive - just wanted to make sure i wasnt doing something bogus since the configuration is still in the sample conf with an explanation and recommendation that it is used. | 12:20 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936 | 12:20 |
| johnthetubaguy | andymccr: its worth avoiding the migration path | 12:20 |
| *** iceyao has quit IRC | 12:21 | |
| johnthetubaguy | andymccr: but we sure will have to write one | 12:21 |
| johnthetubaguy | andymccr: BTW do you like the idea in this spec: https://review.openstack.org/438936 Basically give you less services you need to install and run | 12:21 |
| * johnthetubaguy goes for some lunch | 12:22 | |
| andymccr | johnthetubaguy: reading - from our perspective fewer containers == better, if they aren't required it definitely makes sense to me | 12:23 |
| johnthetubaguy | andymccr: cool, yeah, placement has changed the need for the separate process | 12:24 |
| *** sreeram_v has quit IRC | 12:26 | |
| *** irenab_ has joined #openstack-nova | 12:26 | |
| *** ayogi has quit IRC | 12:30 | |
| *** ijw has joined #openstack-nova | 12:31 | |
| *** yamamoto has joined #openstack-nova | 12:34 | |
| *** yamamoto has quit IRC | 12:35 | |
| *** yamamoto has joined #openstack-nova | 12:35 | |
| *** larainema has joined #openstack-nova | 12:35 | |
| *** amotoki has joined #openstack-nova | 12:35 | |
| *** ijw has quit IRC | 12:36 | |
| *** kaisers_ has joined #openstack-nova | 12:37 | |
| *** Dinesh_Bhor has quit IRC | 12:39 | |
| *** lpetrut has joined #openstack-nova | 12:42 | |
| *** salv-orlando has joined #openstack-nova | 12:42 | |
| *** thorst has joined #openstack-nova | 12:47 | |
| *** iceyao has joined #openstack-nova | 12:47 | |
| *** ljxiash has joined #openstack-nova | 12:48 | |
| *** breton has joined #openstack-nova | 12:51 | |
| breton | hi | 12:51 |
| breton | is it a known problem that having role admin in any project i can edit flavors for all users? | 12:52 |
| *** kaisers_ has quit IRC | 12:52 | |
| *** kaisers_ has joined #openstack-nova | 12:53 | |
| *** markmcclain has quit IRC | 12:53 | |
| *** yamamoto has quit IRC | 12:54 | |
| *** romainh1 has quit IRC | 12:55 | |
| *** romainh has joined #openstack-nova | 12:56 | |
| frickler | breton: admin anywhere is admin everywhere, yes | 12:56 |
| breton | ok, 968696 then | 12:57 |
| frickler | yes, was about to paste that bug link ;) | 12:58 |
| *** Hosam has joined #openstack-nova | 12:59 | |
| alex_xu | johnthetubaguy: sdague, just checked, there isn't any API work in the priority tasks, but i guess there still have some work can be tracked by the API team? so probably we can talk about that in the tomorrow api meeting | 12:59 |
| *** takashin has left #openstack-nova | 13:00 | |
| *** markmcclain has joined #openstack-nova | 13:00 | |
| *** jwcroppe has joined #openstack-nova | 13:02 | |
| *** tbachman has joined #openstack-nova | 13:02 | |
| *** yingjun has joined #openstack-nova | 13:02 | |
| *** kaisers_ has quit IRC | 13:03 | |
| *** tblakes has joined #openstack-nova | 13:04 | |
| *** kaisers_ has joined #openstack-nova | 13:04 | |
| *** satyar has joined #openstack-nova | 13:04 | |
| *** _samdeep has joined #openstack-nova | 13:05 | |
| *** edmondsw has joined #openstack-nova | 13:05 | |
| _samdeep | Hi All | 13:07 |
| -openstackstatus- NOTICE: restarting gerrit to address performance problems | 13:07 | |
| *** ChanServ changes topic to "restarting gerrit to address performance problems" | 13:07 | |
| _samdeep | I want to fetch instance properties inside a filter | 13:07 |
| _samdeep | but I am unable to get it using | 13:09 |
| _samdeep | objects.Instance.get_by_uuid(spec_obj.obj_context, spec_obj.instance_uuid) | 13:09 |
| _samdeep | any idea how to get these details? | 13:09 |
| *** jpena is now known as jpena|lunch | 13:09 | |
| *** Hosam_ has joined #openstack-nova | 13:10 | |
| johnthetubaguy | alex_xu: wasn't the policy docs included? | 13:11 |
| *** edand has joined #openstack-nova | 13:11 | |
| johnthetubaguy | alex_xu: but yeah, we can keep meeting to keep the other stuff moving anywys | 13:11 |
| *** yamamoto has joined #openstack-nova | 13:11 | |
| *** kylek3h has joined #openstack-nova | 13:12 | |
| *** jaypipes has joined #openstack-nova | 13:13 | |
| *** Hosam has quit IRC | 13:14 | |
| *** iceyao has quit IRC | 13:15 | |
| *** hshiina has quit IRC | 13:15 | |
| alex_xu | johnthetubaguy: yeah, anyway if you and sdague have any plan or something you want me help on, let us talk that in the meeting | 13:16 |
| johnthetubaguy | alex_xu: there was the API concepts guide that needed finishing, and a few other loose ends | 13:16 |
| johnthetubaguy | alex_xu: we should go through that as a team at the next meeting I guess | 13:17 |
| *** yamamoto has quit IRC | 13:17 | |
| alex_xu | johnthetubaguy: yea | 13:17 |
| *** sudipto_ has joined #openstack-nova | 13:17 | |
| *** sudipto has joined #openstack-nova | 13:17 | |
| sdague | alex_xu: yeh, I think the thing is that when we got there most of the mindset had been around policy and the limits setting api going into keystone | 13:17 |
| sdague | I honestly think that limits in keystone would be something that we should keep api team focus on this cycle | 13:18 |
| alex_xu | sdague: what is limits in keystone, i probably didn't follow that in the ptg | 13:18 |
| openstackgerrit | John Garbutt proposed openstack/nova master: Add device_id when creating ports https://review.openstack.org/431545 | 13:19 |
| sdague | the new idea on how hierarchical quotas is going to work is that we're going to store limits in keystone | 13:19 |
| openstackgerrit | John Garbutt proposed openstack/nova master: Stop leaking ports on port_create timeout https://review.openstack.org/347183 | 13:19 |
| *** irenab_ has quit IRC | 13:19 | |
| sdague | so the entire hierarchy is efficiently mapped there | 13:19 |
| sdague | brb | 13:19 |
| alex_xu | ah, i probably have some memory about that | 13:20 |
| alex_xu | I remember we said keystone need to tell nova about the hierarchy | 13:20 |
| *** lucasagomes is now known as lucas-hungry | 13:21 | |
| alex_xu | anyway i will grab the etherpad to get more context before meeting | 13:21 |
| *** irenab_ has joined #openstack-nova | 13:21 | |
| *** iceyao has joined #openstack-nova | 13:23 | |
| *** dave-mccowan has joined #openstack-nova | 13:25 | |
| *** Hosam_ has quit IRC | 13:26 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add prep-for-network-aware-scheduling-pike spec https://review.openstack.org/432489 | 13:26 |
| *** jaypipes has quit IRC | 13:28 | |
| *** bvanhav__ has joined #openstack-nova | 13:31 | |
| *** bvanhav_ has quit IRC | 13:32 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop using mox in compute/test_hypervisors.py https://review.openstack.org/326814 | 13:32 |
| *** bastafidli has quit IRC | 13:32 | |
| *** amoralej is now known as amoralej|lunch | 13:32 | |
| *** Hosam has joined #openstack-nova | 13:33 | |
| -openstackstatus- NOTICE: ok gerrit is back to normal | 13:36 | |
| *** ChanServ changes topic to "ok gerrit is back to normal" | 13:36 | |
| *** sudipto_ has quit IRC | 13:36 | |
| *** sudipto has quit IRC | 13:36 | |
| *** Hosam has quit IRC | 13:38 | |
| *** mdrabe has joined #openstack-nova | 13:38 | |
| zioproto | johnthetubaguy, I am about to build ubuntu packages on top of commit 2e05c82b807624b8420df4c94d68b75563b230d2 . Is that a good candidate for new stable/mitaka tag ? | 13:39 |
| *** jaypipes has joined #openstack-nova | 13:40 | |
| johnthetubaguy | zioproto: unsure, I would ask lyarwood about that | 13:40 |
| *** tojuvone has quit IRC | 13:41 | |
| *** salv-orlando has quit IRC | 13:42 | |
| *** edmondsw_ has joined #openstack-nova | 13:42 | |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Ocata_Release_Schedule" | 13:43 | |
| zioproto | lyarwood, ping me when you are here :) | 13:43 |
| -openstackstatus- NOTICE: gerrit is back to normal and I don't know how to use the openstackstaus bot | 13:43 | |
| lyarwood | zioproto: https://review.openstack.org/#/c/438570/ but I'd wait until the tag lands | 13:43 |
| *** edmonds__ has joined #openstack-nova | 13:43 | |
| sfinucan | macsz: I wonder if it's time to look at switching back to nose, now that it supports parallel testing? :P https://review.openstack.org/#/c/435111 | 13:44 |
| zioproto | great that is the correct commit | 13:44 |
| *** tojuvone has joined #openstack-nova | 13:45 | |
| lyarwood | zioproto: right but I'd wait until the tag lands in the repo to build, we've already respun that to include an additional commit in the last 24 hours | 13:45 |
| lyarwood | zioproto: it shouldn't be long until that lands tbh | 13:45 |
| *** edmondsw has quit IRC | 13:46 | |
| *** hoangcx has joined #openstack-nova | 13:46 | |
| *** kaisers_ has quit IRC | 13:47 | |
| *** edmondsw_ has quit IRC | 13:47 | |
| zioproto | thanks | 13:48 |
| *** yassine has quit IRC | 13:49 | |
| *** gouthamr has joined #openstack-nova | 13:50 | |
| *** zhurong has joined #openstack-nova | 13:52 | |
| *** yassine has joined #openstack-nova | 13:52 | |
| *** baoli has joined #openstack-nova | 13:52 | |
| *** diana_clarke has left #openstack-nova | 13:52 | |
| *** xyang1 has joined #openstack-nova | 13:54 | |
| *** baoli_ has joined #openstack-nova | 13:54 | |
| *** esberglu has joined #openstack-nova | 13:54 | |
| bauzas | johnthetubaguy: I do wonder, do we need to change some modules if we're just changing where a conf opt is set ? | 13:55 |
| bauzas | johnthetubaguy: like the one you made about moving default_schedule_az | 13:55 |
| johnthetubaguy | bauzas: its still looked up by doing CONF.xxx_xxx | 13:55 |
| bauzas | yeah I know | 13:55 |
| johnthetubaguy | bauzas: if the tests pass we should be fine | 13:55 |
| bauzas | but I do wonder if we weren't explicitely saying "this or that"' | 13:56 |
| bauzas | oh wait | 13:56 |
| bauzas | that's a global opt | 13:56 |
| johnthetubaguy | its not moving group, right | 13:56 |
| johnthetubaguy | its just in the default group | 13:56 |
| bauzas | we're only do that for groups | 13:56 |
| bauzas | yeah | 13:56 |
| bauzas | nevermind then | 13:56 |
| *** Oku_OS has quit IRC | 13:56 | |
| *** mlavalle has joined #openstack-nova | 13:56 | |
| bauzas | johnthetubaguy: FWIW, saw your spec about local scheduler, will review it soon | 13:57 |
| lyarwood | hmmm is gerrit still messed up for anyone? | 13:57 |
| * lyarwood f5's a few times | 13:57 | |
| johnthetubaguy | mine seems OK now, but its worth a shift-F5 maybe? | 13:58 |
| johnthetubaguy | bauzas: that the one we discussed at the PTG | 13:58 |
| *** baoli has quit IRC | 13:58 | |
| *** liangy has joined #openstack-nova | 13:58 | |
| bauzas | johnthetubaguy: yeah I remember | 13:58 |
| johnthetubaguy | although I don't remember quite when we covered it | 13:58 |
| lyarwood | ah there we go | 13:58 |
| bauzas | tbh, sometimes my French tricks me during summits | 13:59 |
| bauzas | but I was pretty following that convo | 13:59 |
| *** edmondsw has joined #openstack-nova | 13:59 | |
| bauzas | s/summits/f2f | 13:59 |
| *** edmonds__ has quit IRC | 13:59 | |
| *** mlavalle has quit IRC | 14:00 | |
| *** mlavalle has joined #openstack-nova | 14:00 | |
| *** jaypipes has quit IRC | 14:01 | |
| *** openstackgerrit has quit IRC | 14:03 | |
| *** bkopilov has joined #openstack-nova | 14:03 | |
| *** mriedem has joined #openstack-nova | 14:07 | |
| *** bastafidli has joined #openstack-nova | 14:07 | |
| *** openstackgerrit has joined #openstack-nova | 14:08 | |
| *** vladikr has joined #openstack-nova | 14:08 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859 | 14:08 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Pass instance to connect_volume and disconnect_volume https://review.openstack.org/438970 | 14:08 |
| *** Oku_OS has joined #openstack-nova | 14:09 | |
| *** fyxim has quit IRC | 14:09 | |
| *** vladikr has quit IRC | 14:09 | |
| *** vladikr has joined #openstack-nova | 14:10 | |
| *** jpena|lunch is now known as jpena | 14:10 | |
| *** fyxim has joined #openstack-nova | 14:12 | |
| *** yamamoto has joined #openstack-nova | 14:13 | |
| *** kfarr has joined #openstack-nova | 14:13 | |
| *** iceyao has quit IRC | 14:14 | |
| *** lucas-hungry is now known as lucasagomes | 14:15 | |
| *** jaypipes has joined #openstack-nova | 14:16 | |
| *** crushil has joined #openstack-nova | 14:18 | |
| *** yamamoto has quit IRC | 14:18 | |
| *** lpetrut has quit IRC | 14:19 | |
| *** ljxiash has quit IRC | 14:20 | |
| mriedem | mdbooth: so you plumbed an instance through for mount/unmount but i don't see where you are using the instance counting to tell when it's OK to unmount the volume in the case of multi-attach? | 14:20 |
| mdbooth | mriedem: In the next patch. I explain in the commit message. | 14:21 |
| mdbooth | mriedem: I could squish them together if you like, but as I explained in the commit message I think it's clearer this way. | 14:21 |
| mriedem | next patch being https://review.openstack.org/#/c/383859/ ? | 14:22 |
| mdbooth | Just makes it a bit easier to read imho, because this is a mechanical change which touches a bunch of files. | 14:22 |
| mriedem | because as i commented in that change, i don't see where you're looking at the set of attachments for the same volume to see when it's fully gone | 14:22 |
| mriedem | also, | 14:22 |
| mriedem | this is jumping the gun a bit | 14:22 |
| *** sudipto_ has joined #openstack-nova | 14:22 | |
| *** sudipto has joined #openstack-nova | 14:22 | |
| mdbooth | mriedem: Yeah, that one | 14:22 |
| mdbooth | mriedem: Originally I thought so, which is why I removed it promptly when it turned out it was problematic. | 14:23 |
| mriedem | i think the plan was, | 14:23 |
| mdbooth | However, I keep hearing multi-attach noises, and this change allows it to work with multi-attach | 14:23 |
| mdbooth | I don't want my code to be a barrier to that | 14:23 |
| mriedem | we don't even call disconnect_volume on the volume driver if cinder is reporting more than one attachment for the same volume/host | 14:23 |
| lyarwood | yeah that's the plan in the cinderv3 spec | 14:24 |
| mriedem | that's in johnthetubaguy's spec somewhere | 14:24 |
| * mriedem digs | 14:24 | |
| *** mdnadeem has quit IRC | 14:24 | |
| mdbooth | Surely that's wrong, because it depends entirely on the volume driver implementation? | 14:24 |
| mdbooth | If it's a direct attachment to the instance rather than eg a host mount, that's not going to work correctly | 14:25 |
| lyarwood | https://review.openstack.org/#/c/373203/15/specs/pike/approved/cinder-new-attach-apis.rst@196 | 14:25 |
| *** jamesdenton has joined #openstack-nova | 14:25 | |
| *** zz_dimtruck is now known as dimtruck | 14:25 | |
| mdbooth | Yeah, I don't think you can do that at the api level. It's a driver thing. | 14:26 |
| johnthetubaguy | I am curious, reading the scroll back | 14:26 |
| *** sacharya has joined #openstack-nova | 14:26 | |
| lyarwood | mdbooth: IIRC cinderv3 should tell us if the connection is shared or not directly from the volume driver | 14:26 |
| johnthetubaguy | so its the cinder driver that tells the cinder API what should happen | 14:26 |
| *** tlian has joined #openstack-nova | 14:26 | |
| johnthetubaguy | we pass the cinder driver details from os-brick in the connector | 14:26 |
| mdbooth | lyarwood: But it can't tell you if the driver needs to perform a global action or not | 14:26 |
| johnthetubaguy | cinder folks seemed to think that could be specific enough for our needs | 14:26 |
| mdbooth | e.g. connecting a gluster volume directly to a domain via a monitor call | 14:26 |
| mdbooth | vs mounting a gluster volume on the host and passing a file to the domain | 14:27 |
| mdbooth | The gluster driver supports both of those things currently, and only it knows which of them it's doing | 14:27 |
| johnthetubaguy | so the cinder backend needs to tell cinder API what happens, if that needs to know which one we are doing, it needs to go into the connector, I assume? | 14:28 |
| mdbooth | So no higher level can decide whether or not to call disconnect | 14:28 |
| lyarwood | ah it's not configured locally on the host is it? | 14:28 |
| johnthetubaguy | there is no higher level deciding here | 14:28 |
| johnthetubaguy | its the driver | 14:28 |
| lyarwood | connection_info isn't telling it which one to do? | 14:28 |
| mdbooth | lyarwood: No | 14:28 |
| johnthetubaguy | so it needs to add that now | 14:28 |
| lyarwood | yeah | 14:28 |
| johnthetubaguy | and default to a global lock if that might be possibe | 14:28 |
| mdbooth | It literally can't | 14:28 |
| johnthetubaguy | and it doesn't know which | 14:28 |
| mdbooth | And it's none of its business anyway | 14:29 |
| johnthetubaguy | it is now | 14:29 |
| mdbooth | In the gluster case it's based on the qemu and libvirt versions on the host | 14:29 |
| mdbooth | cinder doesn't know that | 14:29 |
| johnthetubaguy | so what we are saying, I think, is the cinder driver, on the cinder side, needs to tell us if we need the lock or not | 14:29 |
| mriedem | we can tell cinder via the connector dict | 14:29 |
| smcginnis | There is no gluster driver, but anyway... | 14:29 |
| mdbooth | As it happens we can safely remove it, but it's still an example of how cinder cannot know this | 14:29 |
| johnthetubaguy | to make that possible cinder always stores the connector I think | 14:30 |
| mdbooth | It's a host-side driver implementation detail | 14:30 |
| mdbooth | All cinder knows is that there will be a connection from the host | 14:30 |
| smcginnis | I'm trying to read the scrollback, but not sure what the issue is here. | 14:30 |
| johnthetubaguy | my preference is to always call os-brick, and tell it what we know | 14:30 |
| *** sacharya has quit IRC | 14:30 | |
| mdbooth | It's not cinder's business where that comes from | 14:30 |
| johnthetubaguy | would that make things happier? | 14:30 |
| mdbooth | johnthetubaguy: Yup. That's also entirely consistent with my patch. | 14:31 |
| *** smatzek_ has quit IRC | 14:31 | |
| johnthetubaguy | mdbooth: I am totally fine with that approach, but I guess we will have to version that somehow, but thats fine | 14:31 |
| mdbooth | johnthetubaguy: version what? | 14:31 |
| johnthetubaguy | deciding if we always call detach | 14:32 |
| *** jdandrea has joined #openstack-nova | 14:32 | |
| mriedem | mdbooth: the nfs volume driver in nova today doesn't call os-brick right | 14:32 |
| mriedem | ? | 14:32 |
| johnthetubaguy | although maybe thats what we always do today anyways? | 14:32 |
| mriedem | johnthetubaguy: we always call disconnect_volume on the volume driver today | 14:32 |
| mdbooth | mriedem: No, it doesn't. None of the fs backed drivers do, I think. | 14:32 |
| lyarwood | yeah correct | 14:32 |
| mriedem | johnthetubaguy: then the nfs vol driver just handles a "device is busy" error and ignores it on umount | 14:32 |
| johnthetubaguy | mriedem: ah, fair enough | 14:33 |
| mdbooth | mriedem: ... which is broken for a few reasons. | 14:33 |
| mriedem | mdbooth's patch https://review.openstack.org/#/c/383859/ is adding ref counting | 14:33 |
| mriedem | so we don't blindly do that | 14:33 |
| johnthetubaguy | so all this shared connection stuff is the very last bit of the cinder API changes | 14:33 |
| mdbooth | Yeah | 14:33 |
| mdbooth | Well "ref counting" | 14:33 |
| johnthetubaguy | if we don't need it, thats cool | 14:33 |
| mdbooth | Because our connect and disconnect calls don't actually match up in practise | 14:34 |
| *** bastafidli has quit IRC | 14:34 | |
| jdandrea | Can someone please clarify? From Placement API doc: "The placement service tracks the inventory and usage of each provider." Does the Placement service *perform* placement, or is it a means to that end, or is it something else? | 14:34 |
| mriedem | "ref counting" is how i'm summarizing with wavy high-level hands a 1200 LOC patch :) | 14:34 |
| *** bastafidli has joined #openstack-nova | 14:34 | |
| mdbooth | It's not that big, is it? | 14:34 |
| * mdbooth checks | 14:34 | |
| mriedem | mdbooth: massive | 14:34 |
| mdbooth | Eurgh, it bloody is | 14:34 |
| mriedem | jdandrea: nova-scheduler performs the actual placement, | 14:35 |
| mriedem | the placement service is the means to the end | 14:35 |
| mdbooth | Anyway, have a look at how big glusterfs is using it :) | 14:35 |
| jdandrea | mriedem Thank you! My brain was running in circles, confused as to why it isn't called an Inventory API, f'rinstance. | 14:35 |
| mdbooth | It's all about error handling, and it can be reused without modification by all the other filesystem drivers | 14:35 |
| mriedem | jdandrea: the placement service holds the global view on resources in the deployment, and nova-scheduler asks it which resource providers it knows about can fit a given instance build/move request | 14:35 |
| jdandrea | mriedem *nodnod* got it | 14:36 |
| mdbooth | mriedem: Look at this: https://review.openstack.org/#/c/423402/ | 14:36 |
| jdandrea | Appreciatd! | 14:36 |
| jdandrea | (and also appreciated) | 14:36 |
| mdbooth | I can do that to ever other filesystem driver | 14:36 |
| jdandrea | :) | 14:36 |
| mriedem | mdbooth: still https://review.openstack.org/#/c/383859/19/nova/virt/libvirt/volume/mount.py@192 | 14:37 |
| mriedem | what am i missing there? | 14:37 |
| mdbooth | mriedem: It's aset | 14:37 |
| mdbooth | a set | 14:37 |
| mdbooth | So you can add (vol_a, instance_a) and (vol_a, instance_b) | 14:37 |
| mdbooth | and they're different things | 14:38 |
| *** amoralej|lunch is now known as amoralej | 14:38 | |
| mdbooth | When you remove (vol_a, instance_b), the mountpoint is still in use | 14:38 |
| mdbooth | There's a test for it | 14:38 |
| mriedem | ah because self.attachments is still len == 1 | 14:38 |
| mdbooth | Yeah | 14:38 |
| mdbooth | mriedem: Anyway, don't look at this as a 1200 line nfs driver patch. It's a 1200 line driver for every filesystem volume driver with really good error handling and really good tests. | 14:40 |
| *** tlian has quit IRC | 14:40 | |
| mriedem | yeah i know - you also told me how really good it was right before rc1 and i found an issue in there, so the size and complexity does matter :) | 14:40 |
| mriedem | not trying to knock it, but you know | 14:40 |
| mriedem | i was just trying to sort out why the multiattach stuff was playing into it since the new bottom patch in the series is new | 14:41 |
| johnthetubaguy | mdbooth: is there are reason not to use the usual oslo.concurrency lock thing instead of using threading directly? | 14:41 |
| mdbooth | johnthetubaguy: Probably. Do you have a specific? | 14:41 |
| johnthetubaguy | mdbooth: well I was just expecting to see us generate a string, and use that for the lock, in your locking patch | 14:42 |
| *** Matias has quit IRC | 14:42 | |
| *** kaisers_ has joined #openstack-nova | 14:43 | |
| mdbooth | johnthetubaguy: I think my solution is much simpler. I don't think oslo.concurrency locks were really designed for this. | 14:43 |
| *** yingjun has quit IRC | 14:43 | |
| johnthetubaguy | mdbooth: https://review.openstack.org/#/c/383859/19/nova/virt/libvirt/volume/mount.py@183 and https://review.openstack.org/#/c/383859/19/nova/virt/libvirt/volume/mount.py@295 and https://review.openstack.org/#/c/383859/19/nova/virt/libvirt/volume/mount.py@62 | 14:44 |
| johnthetubaguy | mdbooth: its just not following the pattern I was expecting that we know is eventlet friendly | 14:44 |
| mdbooth | johnthetubaguy: I don't think you can do conditions with oslo.concurrency for eg, can you? | 14:44 |
| *** READ10 has joined #openstack-nova | 14:44 | |
| *** Matias has joined #openstack-nova | 14:44 | |
| mdbooth | johnthetubaguy: This is all eventletised, as I have discovered | 14:44 |
| mdbooth | It's monkey patched | 14:44 |
| johnthetubaguy | yeah, I figured it had to be | 14:45 |
| mdbooth | I was going to use evently primitives explicitly, in fact I think I had one version where I did that | 14:45 |
| mdbooth | But iirc the api is a bit crippled compared to the python threading api | 14:45 |
| *** sridharg has joined #openstack-nova | 14:45 | |
| johnthetubaguy | just I know getting this stuff "correct" is surpizingly hard, I was kinda expecting us to use some more higher level construct, but I am probably missing something | 14:45 |
| *** thorst is now known as thorst_afk | 14:46 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove mox in nova/tests/unit/compute/test_shelve.py (3) https://review.openstack.org/342648 | 14:47 |
| johnthetubaguy | mdbooth: its probably the need for notify_all that rings alarm bells, I remember that being tricky to get correct | 14:47 |
| mriedem | dansmith: melwitt: sdague: btw, i enjoyed some wonderful high fiber whole grain cereal this morning | 14:47 |
| johnthetubaguy | mdbooth: we could just use a regular lock to protect the counts, and if we did, I am not sure what we would loose | 14:47 |
| mdbooth | johnthetubaguy: Not really, if you ensure all your waits are guarded. | 14:47 |
| dansmith | mriedem: sounds like my morning 18 minutes from now | 14:48 |
| *** kaisers_ has quit IRC | 14:48 | |
| mdbooth | johnthetubaguy: You'd lose condition.wait(), which is a really useful construct | 14:49 |
| *** lpetrut has joined #openstack-nova | 14:49 | |
| johnthetubaguy | mdbooth: but why not guard the count with a regular lock? with eventlet I can't see there being much of a gain between the two, and one is more obviously "OK ish" | 14:49 |
| johnthetubaguy | I am probably missing something big | 14:49 |
| *** yingjun has joined #openstack-nova | 14:49 | |
| mdbooth | johnthetubaguy: Yeah, so I want to wait until the in-use count reaches zero. | 14:50 |
| mdbooth | I take the lock | 14:50 |
| mdbooth | It's not zero | 14:50 |
| mdbooth | What do I do? | 14:50 |
| mdbooth | This is exactly what the Condition api is for | 14:51 |
| johnthetubaguy | mdbooth: ah, so its because you want to wait rather than just sleep or something dumb like that, thats what I was missing | 14:51 |
| *** burt has joined #openstack-nova | 14:51 | |
| mdbooth | Yeah, otherwise you have to drop the lock and poll | 14:51 |
| *** felipemonteiro_ has joined #openstack-nova | 14:52 | |
| *** felipemonteiro has joined #openstack-nova | 14:52 | |
| mdbooth | That uses more resources, and it's more complicated | 14:52 |
| mdbooth | And there's an existing API which designed for exactly this | 14:52 |
| johnthetubaguy | mdbooth: we could just do callbacks on triggers I guess? | 14:52 |
| mdbooth | That's not sounding simpler :) | 14:52 |
| mdbooth | Condition gives us: wait until X is true | 14:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: ironic inventory reporting https://review.openstack.org/437602 | 14:53 |
| mdbooth | johnthetubaguy: I also wrote a slightly funky test library for this, btw | 14:54 |
| sdague | mriedem: thank you for sharing | 14:54 |
| mdbooth | It's in that patch, but it's entirely generic | 14:55 |
| mdbooth | It allows you to insert arbitrary wait points in test code so you can force race conditions | 14:55 |
| *** smatzek_ has joined #openstack-nova | 14:57 | |
| *** hongbin has joined #openstack-nova | 15:02 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix default_availability_zone docs https://review.openstack.org/438887 | 15:03 |
| sfinucan | mdbooth: The fact that your Red Hat email isn't also mdbooth annoys me | 15:03 |
| sfinucan | mriedem: That run_tests patch you requested is here https://review.openstack.org/#/c/437947 | 15:04 |
| johnthetubaguy | mdbooth: maybe the bit I am not getting is why you have to reset the state when you loose connection to libvirt? | 15:05 |
| *** andreas_s has quit IRC | 15:07 | |
| *** hemna is now known as guest_666 | 15:08 | |
| *** guest_666 is now known as hemna | 15:08 | |
| *** zhurong has quit IRC | 15:10 | |
| *** hoangcx has quit IRC | 15:13 | |
| mdbooth | johnthetubaguy: That was originally danpb's suggestion. It's a robustness measure. | 15:14 |
| mdbooth | johnthetubaguy: We need to tie initialisation to host_up anyway, because initialisation uses the libvirt connection. | 15:14 |
| *** yamamoto has joined #openstack-nova | 15:15 | |
| johnthetubaguy | yeah, host up makes sense | 15:15 |
| mdbooth | So re-initialising it after host_down is no great complication. | 15:15 |
| johnthetubaguy | just seems odd to drop it | 15:15 |
| *** lucasxu has joined #openstack-nova | 15:15 | |
| mdbooth | johnthetubaguy: Well, if libvirt went down we don't really know what happened. | 15:15 |
| mdbooth | That doesn't really complicate the code all that much, anyway, as it's separate in the manager | 15:16 |
| bauzas | jaypipes: around ? | 15:17 |
| bauzas | jaypipes: I have a question for you before +Wign https://review.openstack.org/#/c/376198/11 | 15:17 |
| mdbooth | The code is especially paranoid as it's reference counting. We want to do our best to be correct, even if operators are doing unexpected things underneath us, or things are crashing. | 15:18 |
| *** larainema has quit IRC | 15:18 | |
| bauzas | jaypipes: ie. we set the collation to latin1 encoding, I guess it's for memory reasons, right? | 15:18 |
| bauzas | I just wonder if that's really worth it in case we would need to add another column needing utf8 encoding, like a description TEXT field | 15:19 |
| *** larainema has joined #openstack-nova | 15:19 | |
| *** yamamoto has quit IRC | 15:19 | |
| jaypipes | bauzas: gimme a bit.. back to back meetings | 15:20 |
| bauzas | heh, no worries | 15:20 |
| *** liangy has quit IRC | 15:20 | |
| mriedem | bauzas: before you go approving that code, | 15:21 |
| mriedem | https://blueprints.launchpad.net/openstack/?searchtext=resource-provider-traits | 15:21 |
| mriedem | can we get the bp actually approved? | 15:21 |
| bauzas | oh very good point | 15:21 |
| *** liangy has joined #openstack-nova | 15:21 | |
| bauzas | was there a spec reproposed ? | 15:21 |
| mriedem | https://review.openstack.org/#/c/345138/ | 15:22 |
| bauzas | there it is https://review.openstack.org/#/c/345138/ | 15:22 |
| bauzas | yeah | 15:22 |
| bauzas | here is the thing, given it would take more time for that as we haven't previously approved the spec | 15:23 |
| bauzas | mriedem: I would then wonder if it would be worth decoupling that spec into 2 pieces | 15:23 |
| mriedem | i haven't read the spec, | 15:23 |
| spotz | Hey nova, anyone about who could review the doc we're working on for the placement API and cells and give feedback? We can't build instances so we're missing something https://review.openstack.org/#/c/438328/ | 15:23 |
| bauzas | mriedem: one would be the the basement piece we agreed as the top prio | 15:23 |
| mriedem | if it's the full hog of traits, then yeah probably | 15:23 |
| bauzas | mriedem: the other would be exposing it thru the API | 15:23 |
| mriedem | since we just need the piece to fix can_host | 15:23 |
| mriedem | spotz: have you seen the docs in the nova devref? | 15:24 |
| mriedem | https://docs.openstack.org/developer/nova/cells.html | 15:24 |
| mriedem | https://docs.openstack.org/developer/nova/placement.html | 15:24 |
| *** tovin07_ has joined #openstack-nova | 15:24 | |
| dansmith | spotz: also, kinda need to know the error you're hitting to know what you're missing | 15:24 |
| *** marst has joined #openstack-nova | 15:25 | |
| spotz | dansmith: I'm getting the very unhelpful No valid hosts, I'm not sure what Brian got last night though | 15:27 |
| *** sacharya has joined #openstack-nova | 15:27 | |
| dansmith | spotz: got host_mappings in the apidb? | 15:27 |
| *** sacharya has quit IRC | 15:27 | |
| *** sacharya has joined #openstack-nova | 15:28 | |
| spotz | dansmith. Let me bring up the VM and double check | 15:28 |
| spotz | dansmith: Nice call, table is empty | 15:30 |
| dansmith | spotz: nova-manage cell_v2 discover_hosts | 15:30 |
| dansmith | -v | 15:30 |
| *** karthiks has quit IRC | 15:31 | |
| *** sneti_ has joined #openstack-nova | 15:31 | |
| *** sneti_ has quit IRC | 15:31 | |
| spotz | dansmith: Nada, not even with debug nova-manage --debug cell_v2 discover_hosts | 15:32 |
| dansmith | spotz: got hosts actually registering with their cell db? | 15:32 |
| dansmith | spotz: compute_nodes and services tables in the cell1 db | 15:32 |
| *** owalsh has quit IRC | 15:33 | |
| *** owalsh_ has joined #openstack-nova | 15:33 | |
| *** mtanino has joined #openstack-nova | 15:33 | |
| *** satyar has quit IRC | 15:35 | |
| spotz | dansmith: I thought so, but let me go back to the DB vs output from list_cells | 15:35 |
| dansmith | spotz: -v should show you something at least I thought | 15:36 |
| dansmith | spotz: like "checking cell foo" | 15:36 |
| *** sandanar has quit IRC | 15:36 | |
| dansmith | yeah, confirmed | 15:36 |
| spotz | dansmith: I have a nova_cell0 DB no cell1 DB | 15:37 |
| dansmith | spotz: well that's a real problem :P | 15:37 |
| spotz | dansmith: So our create_cell didn't work? | 15:37 |
| dansmith | spotz: create_cell just tells nova where the db is for a given cell | 15:38 |
| dansmith | spotz: you should have an apidb, a cell0 db, and a cell1 db | 15:38 |
| spotz | dansmith: Ok so we're definitely missing steps, I do get this for list_cells though cell1 | 143df967-eb22-4d2c-baff-bfc2538816fd | | 15:38 |
| dansmith | yeah, but that doesn't mean anything if you don't have a db to back it up | 15:38 |
| dansmith | spotz: have you looked through our devref docs mriedem pointed to? | 15:38 |
| *** jgriffith has quit IRC | 15:39 | |
| *** jgriffith has joined #openstack-nova | 15:39 | |
| spotz | dansmith: yeah that's where the steps came from | 15:39 |
| dansmith | spotz: okay I see, you're doing both of those mappings with the same config, which means they probably point to the same place? | 15:40 |
| dansmith | spotz: an unfortunate side-effect of the config file method for getting to this point :/ | 15:40 |
| macsz | sfinucan: lol what about no :D | 15:41 |
| dansmith | spotz: can you pastebin your cel_mappings table? | 15:41 |
| spotz | dansmith: sure hang on | 15:41 |
| macsz | sfinucan: and also from nose's docs: "nose itself supports python 3, but many 3rd-party plugins do not!" | 15:41 |
| *** psachin has quit IRC | 15:41 | |
| *** yamahata_ has joined #openstack-nova | 15:41 | |
| sfinucan | macsz: Who says we need plugins, eh? ;) | 15:41 |
| * sfinucan is pretty sure a few OpenStack projects use nose. Swift, perhaps? | 15:42 | |
| * sfinucan is also not 100% serious. That's a lot of work :P | 15:42 | |
| *** adrian_otto has joined #openstack-nova | 15:42 | |
| spotz | dansmith: https://gist.github.com/spotz/42b9d15f016073458742268735644578 | 15:42 |
| dansmith | spotz: oh, no you do have a cell1 db | 15:43 |
| dansmith | spotz: in that db (nova) do you have services and compute_nodes? | 15:43 |
| *** ediardo has quit IRC | 15:44 | |
| *** karthiks has joined #openstack-nova | 15:44 | |
| macsz | sfinucan: I have had enough of beers last week, so i can invest my friday's evenings to convert tox to nose, if you want it :P | 15:45 |
| sfinucan | macsz: *testrepository to nose | 15:45 |
| sfinucan | tox would stay | 15:45 |
| spotz | dansmith: added as a comment on the gist | 15:45 |
| macsz | sfinucan: ah, sorry, was focused on typing "beer" | 15:45 |
| *** liangy has quit IRC | 15:46 | |
| sfinucan | Ima take a look and see if it would even be possible. The testrepository library annoys me in many ways | 15:46 |
| sfinucan | macsz: :D | 15:46 |
| *** ediardo has joined #openstack-nova | 15:46 | |
| dansmith | spotz: cool, so can you show me the exact result of "nova-manage cell_v2 discover_hosts -v" ? | 15:46 |
| spotz | dansmith: it didn't like -v as a switch when I tried earlier | 15:46 |
| spotz | --debug gave no results to screen | 15:47 |
| spotz | moved it but no help | 15:47 |
| *** armax has joined #openstack-nova | 15:47 | |
| spotz | root@controller:~# nova-manage -v cell_v2 discover_hosts | 15:47 |
| spotz | Option "verbose" from group "DEFAULT" is deprecated for removal. Its value may be silently ignored in the future. | 15:47 |
| *** kaisers_ has joined #openstack-nova | 15:48 | |
| dansmith | spotz: -v goes after discover_hosts | 15:48 |
| spotz | dansmith: that's no joy nova-manage: error: unrecognized arguments: -v | 15:48 |
| dansmith | spotz: sorry it's --verbose | 15:49 |
| spotz | that worked adding as a comment | 15:49 |
| dansmith | spotz: right so I think it worked before when you did --debug | 15:50 |
| macsz | sfinucan: yeah... and I hate my change with "rm" as well, but I am so fed up with doing it manually, and I could not see the use for having this file around either way... | 15:50 |
| dansmith | spotz: it just didn't say anything because you didn't have --verbose | 15:50 |
| dansmith | spotz: that "checking" means the mapping is there now | 15:50 |
| dansmith | spotz: so you should have that host_mapping in the api db and maybe even be able to create instances | 15:50 |
| spotz | dansmith: 2017-02-28 15:31:57 | NULL | 1 | 2 | compute | | 15:51 |
| dansmith | spotz: I need more context, can you paste the full thing in the gist? | 15:51 |
| spotz | dansmith: I wonder why it didn't work before though, I've run the command before | 15:51 |
| *** pkholkin has quit IRC | 15:51 | |
| dansmith | spotz: it needs to run after the host shows up in the cell db, so if you run it before you start the service it won't find it, of course | 15:52 |
| spotz | dansmith: It was up but I'll make sure we make a note for that | 15:52 |
| *** ijw has joined #openstack-nova | 15:54 | |
| spotz | dansmith: no joy on the instance, checking logs to see if it's something new | 15:54 |
| *** unicell has joined #openstack-nova | 15:54 | |
| spotz | dansmith: still no valid hosts in the nova-conductor.log though I do think we made progress:) | 15:55 |
| *** unicell1 has quit IRC | 15:55 | |
| dansmith | spotz: well paste some logs | 15:56 |
| *** unicell has quit IRC | 15:56 | |
| *** owalsh_ is now known as owalsh | 15:57 | |
| spotz | dansmith: api, scheduler and conductor logs up | 15:59 |
| *** ociuhandu has joined #openstack-nova | 15:59 | |
| mriedem | spotz: where? | 16:00 |
| spotz | mriedem: https://gist.github.com/spotz/42b9d15f016073458742268735644578 | 16:00 |
| spotz | looks like placement-api not found from compute side, gonna peek at apache | 16:01 |
| dansmith | spotz: would be a lot easier to read those as a new gist in fixed-width font, fwiw | 16:01 |
| dansmith | damn docs people and their variable-width nonsense :P | 16:01 |
| spotz | dansmith: Ha I'm not really a docs person:) | 16:01 |
| *** salv-orlando has joined #openstack-nova | 16:01 | |
| dansmith | heh | 16:01 |
| mriedem | is the nova-compute service reporting as up? | 16:02 |
| mriedem | nova service-list | 16:02 |
| spotz | mriedem: Yep reporting as up | 16:02 |
| mriedem | hmm, but yeah | 16:02 |
| mriedem | 2017-02-28 09:58:07.852 1621 WARNING nova.scheduler.client.report [req-62fa3550-e8f7-4111-a297-6b24b20a7e1f - - - - -] The placement API endpoint not found. Placement is optional in Newton, but required in Ocata. Please enable the placement service before upgrading. | 16:02 |
| mriedem | is the placement endpoint in "openstack endpoint list"? | 16:02 |
| spotz | mriedem: Brian noted an apache issue last night but he's doing rhel install. Public, internal and admin present | 16:03 |
| mriedem | my guess is, the compute node isn't reporting into placement, so when the scheduler asks placement for somewhere to build an instance, it says it doesn't know about anything and we get NoValidHost | 16:03 |
| *** jaosorior has quit IRC | 16:03 | |
| mriedem | are the credentials correct in the [placement] section of nova.conf on the compute node? | 16:04 |
| spotz | mriedem: Yep | 16:04 |
| spotz | I can paste them if you want | 16:04 |
| *** ociuhandu has quit IRC | 16:05 | |
| mriedem | did you configure the compute before or after placement was running? | 16:07 |
| mriedem | i.e. do you need to restart the nova-compute service? | 16:07 |
| spotz | I did just notice os_region was there twice, once as the default openstack and the one I set to RegionOne | 16:07 |
| spotz | mriedem: I've restarted compute several times since config. New error Quota exceeded for instances: Requested 1, but already used 10 of 10 instances (HTTP 403) (Request-ID: req-6fedaf70-89b9-4bc4-a978-9ab04090767f) | 16:08 |
| *** rcernin has quit IRC | 16:08 | |
| mriedem | so you've already got 10 dead instances in ERROR state right? | 16:08 |
| mriedem | nova list would show 10 ERROR instances | 16:08 |
| mriedem | delete those | 16:09 |
| spotz | mriedem: I've been clearing them out | 16:09 |
| akijak | hi, I've found a bug in nova concerning rebuilding and 'hw_disk_bus' image property (https://bugs.launchpad.net/nova/+bug/1667667), I'm looking for someone who could give me some directions how to fix that :) can you help? | 16:09 |
| openstack | Launchpad bug 1667667 in OpenStack Compute (nova) "Rebuilding instance ignores image's property hw_disk_bus" [Low,Confirmed] | 16:09 |
| spotz | yeah just 2 in error currently | 16:09 |
| *** markus_z has quit IRC | 16:10 | |
| efried | Howdy folks, is there a Pike equivalent of https://etherpad.openstack.org/p/ocata-nova-priorities-tracking ? I.e. somewhere I can put up-to-date status & review priorities for the PowerVM driver "subteam"? | 16:11 |
| lyarwood | akijak: hey, have you looked at https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2679 and how we are passing image_meta around? | 16:11 |
| mriedem | efried: i can create one | 16:11 |
| efried | mriedem Thanks. And wipe the agenda for Thursday's meeting? | 16:11 |
| mriedem | efried: it's only tuesday! | 16:11 |
| mriedem | i get to the thurs meeting agenda by about wed night | 16:12 |
| efried | Heh. Didn't know how much lead time we were supposed to have ;-) | 16:12 |
| mriedem | efried: if you need to change something in the meeting agenda, you can do that now | 16:12 |
| *** _samdeep has quit IRC | 16:12 | |
| efried | rgr | 16:12 |
| *** kaisers_ has quit IRC | 16:14 | |
| *** yingjun has quit IRC | 16:14 | |
| spotz | mriedem: 0 instances in Error, same result | 16:14 |
| *** lpetrut has quit IRC | 16:14 | |
| dansmith | mriedem: so I noticed some weird quota behavior when I was trying to get my multicell patch working | 16:15 |
| dansmith | spotz: what is the [database]/connection set to in the nova conf that you're using for your apis? | 16:15 |
| dansmith | and conductors | 16:15 |
| dansmith | maybe we're reserving in cell0 | 16:16 |
| *** yamamoto has joined #openstack-nova | 16:16 | |
| mriedem | i assume it's mysql+pymysql://nova:linuxacademy123@controller/nova | 16:17 |
| mriedem | from https://gist.github.com/spotz/42b9d15f016073458742268735644578 | 16:17 |
| *** Oku_OS is now known as Oku_OS-away | 16:17 | |
| dansmith | actually, it has to be or other things won't work because ocata | 16:18 |
| spotz | mriedem: Yep | 16:18 |
| spotz | I know the basic config docs that are there minus placement-api and cells work as I also built a newton cluster and it's happy | 16:19 |
| spotz | We're missing something, whether it's a step or something out of order | 16:19 |
| *** liangy has joined #openstack-nova | 16:19 | |
| *** yamamoto has quit IRC | 16:21 | |
| efried | mriedem: I can start populating https://etherpad.openstack.org/p/pike-nova-priorities-tracking with the structure of its ocata predecessor if you like. | 16:23 |
| efried | You have more important (or at least more urgent) things to do ;-) | 16:23 |
| mriedem | efried: yes please | 16:24 |
| *** mdrabe_ has joined #openstack-nova | 16:24 | |
| mriedem | i'm doing 3 things at once | 16:24 |
| *** Matias has quit IRC | 16:25 | |
| *** nkrinner is now known as nkrinner_afk | 16:26 | |
| mriedem | spotz: can you make a curl request to the placement endpoint using the same creds as are in the nova.conf on the compute? | 16:26 |
| spotz | mriedem: Yep hang on | 16:27 |
| *** mdrabe has quit IRC | 16:27 | |
| mriedem | to /resource_providers would be enough | 16:27 |
| *** bvanhav_ has joined #openstack-nova | 16:27 | |
| *** Matias has joined #openstack-nova | 16:27 | |
| *** bvanhav_ is now known as bvanhav | 16:28 | |
| *** tovin07_ has left #openstack-nova | 16:28 | |
| *** baoli_ has quit IRC | 16:28 | |
| *** bvanhav__ has quit IRC | 16:28 | |
| *** baoli has joined #openstack-nova | 16:29 | |
| *** sridharg has quit IRC | 16:31 | |
| *** salv-orlando has quit IRC | 16:32 | |
| *** prateek has quit IRC | 16:32 | |
| *** tbachman has quit IRC | 16:33 | |
| *** tbachman has joined #openstack-nova | 16:34 | |
| *** artom has quit IRC | 16:34 | |
| *** pcaruana has quit IRC | 16:34 | |
| *** lpetrut has joined #openstack-nova | 16:34 | |
| *** edand has quit IRC | 16:35 | |
| spotz | mriedem: Bad Request but it could have been my curl | 16:36 |
| *** artom has joined #openstack-nova | 16:36 | |
| *** baoli has quit IRC | 16:37 | |
| *** artom has quit IRC | 16:37 | |
| *** prateek_ has quit IRC | 16:38 | |
| *** jaypipes has quit IRC | 16:40 | |
| *** Jeffrey4l has quit IRC | 16:41 | |
| *** Jeffrey4l has joined #openstack-nova | 16:42 | |
| *** salv-orlando has joined #openstack-nova | 16:44 | |
| spotz | mriedem: Yeah everytime I try to curl I'm getting the 400, I'm assuming you wanted me to just get like tokens? | 16:49 |
| mriedem | spotz: just get the /resource_providers | 16:50 |
| mriedem | you'll need a token | 16:50 |
| mdbooth | mriedem: https://review.openstack.org/#/c/395709/ | 16:51 |
| mriedem | mdbooth: you don't need that | 16:52 |
| mdbooth | Nothing unexpected in the logs, either | 16:52 |
| mriedem | we run the nfs job in the nova experimental queue | 16:52 |
| mdbooth | mriedem: You mean the check experimental job? | 16:52 |
| mdbooth | Yeah, we run a bunch of other stuff in there too, though | 16:52 |
| mriedem | "check experimental" runs the set of experimental jobs on your nova change | 16:52 |
| mriedem | directly | 16:52 |
| mdbooth | That one only runs the nfs job, so it's kinda nice | 16:52 |
| *** nic has joined #openstack-nova | 16:52 | |
| *** igordcar1 has quit IRC | 16:54 | |
| *** ijw_ has joined #openstack-nova | 16:54 | |
| *** igordcard has joined #openstack-nova | 16:54 | |
| mdbooth | mriedem: check experimental is a bit of a nose bleed. How much of it is intended to be red vs green? | 16:54 |
| mriedem | i need to remove some of it | 16:55 |
| mriedem | ala https://review.openstack.org/#/c/415033/ | 16:55 |
| mriedem | but time is an issue | 16:55 |
| mdbooth | mriedem: The nfs job it ran passed | 16:56 |
| mdbooth | mriedem: Incidentally, would it be practical to do 'check | 16:57 |
| mdbooth | gate-tempest-dsvm-full-devstack-plugin-nfs-nv', and have it run just that job? | 16:57 |
| mriedem | spotz: http://paste.openstack.org/show/600806/ | 16:57 |
| mriedem | ^ is from my devstack, with curl to placement | 16:57 |
| *** ijw has quit IRC | 16:57 | |
| mriedem | mdbooth: that's not a thing | 16:57 |
| mdbooth | Or have a much wider array of 'interest areas' maybe, like 'check nfs', or 'check storage' | 16:57 |
| mdbooth | mriedem: Right, but is there any reason it's not a thing? | 16:58 |
| mriedem | someone would have to do it | 16:58 |
| mdbooth | Sure. Would it be a useful thing? | 16:58 |
| mriedem | idk | 16:58 |
| mdbooth | For eg, in this case we've run gate-tempest-dsvm-lxc-ubuntu-xenial, and it failed, but I don't really care. | 16:59 |
| mriedem | what would have been useful was if the hilton didn't f up charging me for last week so my internal expense reporting tool took over an hour to trick to make up for a separate .05 charge | 16:59 |
| mriedem | recheck hilton | 16:59 |
| mdbooth | lol | 16:59 |
| mriedem | spotz: i get the token via: OS_TOKEN=$(openstack token issue -c id -f value) | 17:00 |
| mdbooth | I'm pretty sure that the scope of expense reporting tools vastly exceeds anything we have to deal with :) | 17:00 |
| *** zioproto has quit IRC | 17:00 | |
| *** gfhellma has joined #openstack-nova | 17:01 | |
| spotz | mriedem: Still getting no json object | 17:01 |
| mriedem | spotz: but a 200 or an error? | 17:02 |
| *** gfhellma has quit IRC | 17:02 | |
| *** mvk has quit IRC | 17:02 | |
| *** gfhellma has joined #openstack-nova | 17:02 | |
| spotz | mriedem: 200, I think it's me I suck at curl | 17:03 |
| mriedem | and the creds you're using for that token are the same as what's in nova.conf on the compute? | 17:03 |
| spotz | mriedem: Yeah I set up an rc and did an openstack token issue | 17:04 |
| *** nicolasbock has quit IRC | 17:04 | |
| spotz | Or I could be doing things ass backwards | 17:05 |
| mriedem | are you still seeing warnings in the nova-compute logs about not being able to connect to placement? | 17:05 |
| *** crushil has quit IRC | 17:05 | |
| *** ijw has joined #openstack-nova | 17:06 | |
| spotz | mriedem: Nope | 17:06 |
| *** gcb has joined #openstack-nova | 17:06 | |
| *** browne has joined #openstack-nova | 17:07 | |
| *** irenab_ has quit IRC | 17:07 | |
| *** tbachman has quit IRC | 17:08 | |
| *** nicolasbock has joined #openstack-nova | 17:08 | |
| *** moshele has quit IRC | 17:09 | |
| *** tesseract has quit IRC | 17:09 | |
| mriedem | but there are no resource providers coming back... | 17:09 |
| *** ijw_ has quit IRC | 17:09 | |
| mriedem | something is janked there then | 17:10 |
| mriedem | because the compute is running a periodic task every minute that will update it's info in the placement api | 17:10 |
| *** Swami has joined #openstack-nova | 17:10 | |
| mriedem | if it can't, you should be seeing a warning every 10 minutes or so in the compute logs | 17:10 |
| *** kaisers_ has joined #openstack-nova | 17:10 | |
| *** raunak has joined #openstack-nova | 17:11 | |
| spotz | nova-compute.log is now lines of 017-02-28 11:05:39.099 2618 INFO nova.compute.resource_tracker [req-8025f2c1-a990-4383-ab04-46e929ffce39 - - - - -] Final resource view: name=compute phys_ram=7983MB used_ram=512MB phys_disk=5GB used_disk=0GB total_vcpus=1 used_vcpus=0 pci_stats=[] | 17:11 |
| mriedem | that's not it | 17:11 |
| cfriesen | with the change to make live migration checks async in Newton, did horizon get updated to properly report any errors in the checks? | 17:11 |
| mriedem | cfriesen: it's a microversion so horizon would have to be opting in | 17:11 |
| mriedem | and if so, | 17:11 |
| mriedem | they have to adjust for the behavior change | 17:11 |
| openstackgerrit | John Griffith proposed openstack/nova master: Add attachment_uuid to BDM table https://review.openstack.org/438744 | 17:12 |
| openstackgerrit | John Griffith proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750 | 17:13 |
| openstackgerrit | STEW TY proposed openstack/nova master: Complete verification of servers-action-fixed-ip.inc https://review.openstack.org/338606 | 17:14 |
| mriedem | sfinucan: mdbooth: you were asking why lyarwood was adding an attachment_id to the bdm table | 17:14 |
| mriedem | https://review.openstack.org/#/c/373203/ | 17:14 |
| mriedem | is why | 17:14 |
| mriedem | you should read it since we need more eyes on it | 17:14 |
| mriedem | dansmith: what do you think, should we shit or get off this pot? https://review.openstack.org/#/c/242602/ | 17:15 |
| dansmith | mriedem: lol | 17:15 |
| mriedem | we have no immediate need for it right now that i'm aware of | 17:16 |
| mdbooth | mriedem: Coincidentally just reading that | 17:16 |
| mriedem | but it's always something that comes up as "if we only had a uuid on the bdm" | 17:16 |
| dansmith | mriedem: I don't care.. seems like a good thing, and people seem to occasionally need it | 17:16 |
| *** jaypipes has joined #openstack-nova | 17:16 | |
| dansmith | but yeah if no need then whatever | 17:16 |
| mdbooth | lyarwood: You still need a commit message btw :P | 17:16 |
| dansmith | I can abandon if it's bothering you | 17:16 |
| mriedem | it's not bothering me | 17:16 |
| * lyarwood will add more context to that later | 17:16 | |
| mriedem | but lyarwood is building on it | 17:16 |
| lyarwood | mdbooth: meh, can't you read my mind? | 17:16 |
| mdbooth | lyarwood: Not until you push it to gerrit :) | 17:17 |
| mriedem | lyarwood: consequently, what state is https://review.openstack.org/#/c/290793/ in now? | 17:17 |
| *** yamamoto has joined #openstack-nova | 17:18 | |
| *** nic has left #openstack-nova | 17:18 | |
| *** nic has joined #openstack-nova | 17:19 | |
| lyarwood | mriedem: rotting on the floor, I had an open question around how to actually use the new UUIDs to avoid this iirc | 17:19 |
| jgriffith | omg that change is everywhere :) | 17:19 |
| mriedem | spotz: looking for the errors from here https://github.com/openstack/nova/blob/45cb83829e5b16dbacb7c71b7a49060c75cad986/nova/scheduler/client/report.py#L51 | 17:19 |
| mriedem | that will log the warning every 10 tries, so 10 minutes https://github.com/openstack/nova/blob/45cb83829e5b16dbacb7c71b7a49060c75cad986/nova/scheduler/client/report.py#L43 | 17:20 |
| mriedem | earlier you said you hit this https://github.com/openstack/nova/blob/45cb83829e5b16dbacb7c71b7a49060c75cad986/nova/scheduler/client/report.py#L59 | 17:20 |
| *** dtp has joined #openstack-nova | 17:20 | |
| mriedem | spotz: can you share the nova.conf from your compute node? | 17:20 |
| cfriesen | question about aborting live migration...where are users supposed to find the appropriate migration id? I don't see it returned in the response to the live-migrate request or instance-action. Are they supposed to parse the output of migration-list? | 17:21 |
| mriedem | nova migration-list or something | 17:21 |
| *** armax has quit IRC | 17:21 | |
| spotz | mriedem: Yep, its just the one built through the install guide, nothing prorietary let me get it for you | 17:21 |
| mriedem | sdague: i still owe you that nickel, i totally forgot to pay up in person | 17:23 |
| mriedem | cfriesen: https://docs.openstack.org/cli-reference/nova.html#nova-migration-list | 17:23 |
| *** yamahata_ has quit IRC | 17:23 | |
| mriedem | that api currently filters so that only live migration records are returned | 17:23 |
| sfinucan | tonyb: Just FYI, https://review.openstack.org/#/c/438510/ merged so the next pbr release should be coming shortly (I hope) | 17:23 |
| *** yamamoto has quit IRC | 17:23 | |
| mriedem | cfriesen: see https://review.openstack.org/#/c/407237/ | 17:24 |
| spotz | mriedem: It's gthe latest comment but it decided it needed a BIG font for some reason | 17:25 |
| *** lpetrut has quit IRC | 17:25 | |
| *** lpetrut has joined #openstack-nova | 17:26 | |
| mriedem | spotz: ? | 17:26 |
| mriedem | comment where? | 17:26 |
| spotz | mriedem: here new gist https://gist.github.com/spotz/4550ffcd9568d608bf6f483ef5feebab | 17:26 |
| mriedem | efried: https://etherpad.openstack.org/p/pike-nova-priorities-tracking looks grand, thanks | 17:26 |
| *** thorst_afk is now known as thorst | 17:27 | |
| efried | mriedem Thanks, still working on it. | 17:27 |
| mriedem | spotz: and a gist for the rc you used to make the curl request to the placement endpoint? | 17:28 |
| *** ijw has quit IRC | 17:28 | |
| *** jdandrea has quit IRC | 17:28 | |
| *** karimb has quit IRC | 17:28 | |
| openstackgerrit | STEW TY proposed openstack/nova master: Complete verification of servers-action-fixed-ip.inc https://review.openstack.org/338606 | 17:29 |
| *** ltomasbo is now known as ltomasbo|away | 17:29 | |
| spotz | mriedem: https://gist.github.com/spotz/779bd1045c0422b6dd1cc6640dd11787 | 17:30 |
| *** gcb has quit IRC | 17:30 | |
| *** ZZelle has joined #openstack-nova | 17:31 | |
| mriedem | that all looks fine | 17:31 |
| spotz | mriedem: It's gotta be something stupid just not sure what | 17:34 |
| spotz | I do appreciate all the help today mriedem and dansmith! | 17:34 |
| mriedem | yeah i don't really understand it, if the creds are ok for you to make a curl request, then nova-compute should be reporting into the placement api | 17:35 |
| mriedem | and the curl request you made should return something | 17:35 |
| mriedem | and there are no errors or warnings in the nova-compute logs... | 17:35 |
| *** claudiub has quit IRC | 17:36 | |
| *** ijw has joined #openstack-nova | 17:36 | |
| spotz | mriedem: yeah it's not making much sense and the more I tinker I worry I'm losing track of what I changed. Between Brian and I testing each version of the doc we'll get it just not as quickly as I was hoping | 17:36 |
| mriedem | do you see this debug message in the n-cpu logs? https://github.com/openstack/nova/blob/45cb83829e5b16dbacb7c71b7a49060c75cad986/nova/scheduler/client/report.py#L582 | 17:37 |
| mriedem | e.g. http://logs.openstack.org/59/383859/19/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/4377f7d/logs/screen-n-cpu.txt.gz#_2017-02-28_14_35_05_210 | 17:37 |
| spotz | mriedem: Nope, just 2017-02-28 11:37:29.369 2204 INFO nova.scheduler.host_manager [req-b549cd44-392e-4992-a88e-31f7180a3451 - - - - -] Successfully synced instances from host 'compute'. | 17:38 |
| *** antwash has joined #openstack-nova | 17:38 | |
| spotz | And one retry filter a while ago | 17:39 |
| *** jpena is now known as jpena|away | 17:39 | |
| cfriesen | mriedem: thanks, I knew about that command. The inconvenience is that there's no way to list the currently active migration for a server, so the user has to list all the migrations, then find the currently active one for the server of interest. | 17:39 |
| *** sudipto_ has quit IRC | 17:39 | |
| *** sudipto has quit IRC | 17:39 | |
| cfriesen | mriedem: why doesn't the live migration request return the migration ID? (or the "nova instance-action" command?) | 17:40 |
| antwash | johnthetubaguy, sdague : hey there! I have something for you to look at https://review.openstack.org/#/c/439070/ -- marked as WIP until feedback is received. | 17:40 |
| antwash | ^^ when yo have time | 17:40 |
| *** ijw has quit IRC | 17:41 | |
| *** edand has joined #openstack-nova | 17:41 | |
| spotz | mriedem: Thanks again for the help, need to afk | 17:42 |
| mriedem | cfriesen: i don't think we have a migration id before we return from the live migration request | 17:44 |
| mriedem | cfriesen: would have to look to confirm | 17:44 |
| mriedem | the instance-action is not specific to live migration | 17:44 |
| mriedem | so it wouldn't have a migration id in it | 17:44 |
| *** derekh has quit IRC | 17:44 | |
| mriedem | you'd have to extend instance actions if you wanted to store extra meta per action | 17:44 |
| *** Apoorva has joined #openstack-nova | 17:45 | |
| efried | mriedem Think I'm done; added announcement & link on meeting agenda. | 17:47 |
| mriedem | cfriesen: so we should be pulling migrations per server /servers/{uuid}/migrations | 17:47 |
| mriedem | cfriesen: would have to look at what nova migration-list does, because i thought it would take a server uuid | 17:48 |
| *** annegentle has joined #openstack-nova | 17:48 | |
| mriedem | ah | 17:48 |
| mriedem | https://docs.openstack.org/cli-reference/nova.html#nova-server-migration-list | 17:48 |
| mriedem | cfriesen: ^ | 17:48 |
| mriedem | that will return the list of live migrations for the given server | 17:49 |
| mriedem | if you have a high enough api to provide that microversion | 17:49 |
| *** pkholkin has joined #openstack-nova | 17:49 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in conductor cold migrate https://review.openstack.org/438022 | 17:52 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | 17:52 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | 17:52 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 17:52 |
| dansmith | no whammies no whammies no whammies | 17:53 |
| *** crushil has joined #openstack-nova | 17:54 | |
| *** slaweq_ has quit IRC | 17:55 | |
| *** fragatina has joined #openstack-nova | 17:55 | |
| *** slaweq_ has joined #openstack-nova | 17:56 | |
| *** fragatina has quit IRC | 17:57 | |
| *** fragatina has joined #openstack-nova | 17:58 | |
| *** kaisers_ has quit IRC | 17:58 | |
| *** kfarr has quit IRC | 17:59 | |
| cfriesen | mriedem: okay, server-migration-list is useful. still requires parsing though. would it maybe make sense to have a "--active" flag on that to return just the in-progress migration for that server? | 18:00 |
| cfriesen | or do we just make client code sort the response by ID and pick the latest to try to get the active one? | 18:02 |
| *** moshele has joined #openstack-nova | 18:02 | |
| mriedem | cfriesen: it might be nice to add a status filter query parameter to the /servers/id/migrations REST API | 18:03 |
| mriedem | we could validate that with json schema now | 18:03 |
| mriedem | cfriesen: but it does look like you could filter client side | 18:04 |
| mriedem | we print out the migration status | 18:04 |
| *** lucasagomes is now known as lucas-afk | 18:05 | |
| cfriesen | mriedem: hmm...in newton at least it appears that "server-migration-list" doesn't show completed migrations | 18:07 |
| *** bastafidli has quit IRC | 18:08 | |
| *** bastafidli has joined #openstack-nova | 18:08 | |
| *** romainh1 has joined #openstack-nova | 18:09 | |
| *** romainh1 has quit IRC | 18:09 | |
| mriedem | cfriesen: that's because it's calling objects.MigrationList.get_in_progress_by_instance | 18:10 |
| mriedem | where in-progress is one of | 18:11 |
| mriedem | 'queued', 'preparing', | 18:11 |
| mriedem | 'running', | 18:11 |
| mriedem | 'post-migrating'] | 18:11 |
| mriedem | why we only return in progress migrations, idk | 18:11 |
| mriedem | i think because that api was added for being able to cancel a live migration | 18:11 |
| mriedem | which you can only do if it's running | 18:11 |
| mriedem | tdurakov might remember more | 18:11 |
| *** romainh has quit IRC | 18:12 | |
| mriedem | cfriesen: so before you ask "why only in progress live migrations?" i think that's the original idea of why, but "can we return all, not just in progress?" - i'd think sure | 18:12 |
| mriedem | i believe the os-migrations API returns all migrations | 18:13 |
| mriedem | imo it's a pretty clunky gd API and is tailor made just for canceling a live migration | 18:13 |
| mriedem | which seemed to be too short sighted | 18:13 |
| cfriesen | mriedem: hmm...the API docs do specify that it's in-progress live migrations, but the CLI docs don't specify that it's only for in-progress migrations. | 18:14 |
| *** bvanhav_ has joined #openstack-nova | 18:15 | |
| tdurakov | It was done to return adress live-mig abort functionality | 18:15 |
| *** bvanhav has quit IRC | 18:16 | |
| cfriesen | tdurakov: okay, thanks. I agree it's a bit clunky, but it does at least let you get the info needed to abort. | 18:17 |
| *** salv-orl_ has joined #openstack-nova | 18:17 | |
| cfriesen | mriedem: tdurakov: is it just me that thinks "nova server-migration-list" is a wierd command for something that should only ever have a single result? | 18:18 |
| *** yamamoto has joined #openstack-nova | 18:19 | |
| *** salv-orlando has quit IRC | 18:20 | |
| *** dtp has quit IRC | 18:20 | |
| mriedem | cfriesen: i believe i wrote the api ref docs because they didn't exist | 18:21 |
| mriedem | and i was sorting this all out about 5 months ago | 18:21 |
| tdurakov | Migration list is awful... | 18:21 |
| mriedem | cfriesen: you could push a patch to update the cli help | 18:23 |
| mriedem | to make it more clear | 18:23 |
| *** yamamoto has quit IRC | 18:25 | |
| *** unicell has joined #openstack-nova | 18:25 | |
| *** tbachman has joined #openstack-nova | 18:26 | |
| *** dtp has joined #openstack-nova | 18:28 | |
| *** bvanhav_ has quit IRC | 18:29 | |
| *** bvanhav_ has joined #openstack-nova | 18:29 | |
| *** dharinic is now known as dharinic|lunch | 18:31 | |
| *** krtaylor has quit IRC | 18:32 | |
| *** gfhellma_ has joined #openstack-nova | 18:33 | |
| *** sambetts is now known as sambetts|afk | 18:33 | |
| *** lucasxu has quit IRC | 18:34 | |
| *** lpetrut has quit IRC | 18:35 | |
| *** gfhellma has quit IRC | 18:35 | |
| *** karimb has joined #openstack-nova | 18:35 | |
| *** mlavalle has quit IRC | 18:35 | |
| *** karimb has quit IRC | 18:35 | |
| *** bvanhav_ is now known as bvanhav | 18:38 | |
| *** artom has joined #openstack-nova | 18:39 | |
| *** thumpba has quit IRC | 18:40 | |
| *** jed56 has quit IRC | 18:40 | |
| *** baoli has joined #openstack-nova | 18:41 | |
| *** liangy has quit IRC | 18:43 | |
| *** salv-orl_ has quit IRC | 18:45 | |
| *** tbachman has quit IRC | 18:46 | |
| *** tbachman has joined #openstack-nova | 18:48 | |
| *** moshele has quit IRC | 18:48 | |
| *** bkopilov has quit IRC | 18:49 | |
| *** yamahata has joined #openstack-nova | 18:49 | |
| *** ijw has joined #openstack-nova | 18:51 | |
| *** gfhellma__ has joined #openstack-nova | 18:54 | |
| *** kaisers_ has joined #openstack-nova | 18:54 | |
| *** irenab_ has joined #openstack-nova | 18:55 | |
| *** bastafidli has quit IRC | 18:57 | |
| *** ijw has quit IRC | 18:57 | |
| *** gfhellma_ has quit IRC | 18:58 | |
| *** adisky_ has quit IRC | 18:59 | |
| *** gfhellma__ has quit IRC | 19:00 | |
| *** baoli has quit IRC | 19:00 | |
| *** baoli has joined #openstack-nova | 19:01 | |
| *** baoli has quit IRC | 19:01 | |
| *** bastafidli has joined #openstack-nova | 19:03 | |
| *** dtp has quit IRC | 19:04 | |
| *** bkopilov has joined #openstack-nova | 19:05 | |
| *** lucasxu has joined #openstack-nova | 19:06 | |
| *** gfhellma has joined #openstack-nova | 19:07 | |
| *** lucasxu has quit IRC | 19:08 | |
| *** thumpba has joined #openstack-nova | 19:10 | |
| *** thumpba has quit IRC | 19:10 | |
| *** baoli has joined #openstack-nova | 19:11 | |
| *** salv-orlando has joined #openstack-nova | 19:12 | |
| *** amoralej is now known as amoralej|off | 19:13 | |
| *** dharinic|lunch is now known as dharinic | 19:14 | |
| *** krtaylor has joined #openstack-nova | 19:15 | |
| *** edand has quit IRC | 19:15 | |
| *** gszasz has quit IRC | 19:16 | |
| *** fragatina has quit IRC | 19:17 | |
| *** kaisers_ has quit IRC | 19:18 | |
| *** krtaylor has quit IRC | 19:18 | |
| openstackgerrit | Monty Taylor proposed openstack/nova master: WIP Clean up and consolidate client creation https://review.openstack.org/438512 | 19:19 |
| openstackgerrit | Monty Taylor proposed openstack/nova master: Be a little more explicit about checking ok status https://review.openstack.org/439121 | 19:19 |
| *** krtaylor has joined #openstack-nova | 19:19 | |
| *** lpetrut has joined #openstack-nova | 19:19 | |
| *** kaisers_ has joined #openstack-nova | 19:21 | |
| *** yamamoto has joined #openstack-nova | 19:21 | |
| *** armax has joined #openstack-nova | 19:23 | |
| *** krtaylor has quit IRC | 19:23 | |
| *** Apoorva has quit IRC | 19:24 | |
| *** Apoorva has joined #openstack-nova | 19:25 | |
| *** liangy has joined #openstack-nova | 19:26 | |
| *** yamamoto has quit IRC | 19:27 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Remove mox from nova/tests/unit/db/test_db_api.py https://review.openstack.org/434557 | 19:32 |
| *** krtaylor has joined #openstack-nova | 19:36 | |
| *** krtaylor has quit IRC | 19:39 | |
| *** READ10 has quit IRC | 19:39 | |
| *** krtaylor has joined #openstack-nova | 19:39 | |
| *** krtaylor has quit IRC | 19:42 | |
| *** fragatina has joined #openstack-nova | 19:42 | |
| *** baoli has quit IRC | 19:42 | |
| *** baoli has joined #openstack-nova | 19:43 | |
| *** krtaylor has joined #openstack-nova | 19:44 | |
| *** lucasxu has joined #openstack-nova | 19:45 | |
| *** tbachman_ has joined #openstack-nova | 19:45 | |
| tdurakov | mriedem: I wonder what if we just merge these 2 comands from the client side, I mean ,nova migration-list and nove server-migration-list | 19:45 |
| *** tbachman has quit IRC | 19:46 | |
| *** tbachman_ is now known as tbachman | 19:46 | |
| *** krtaylor has quit IRC | 19:46 | |
| mriedem | tdurakov: as in like nova migration-list --server <optional server id> which toggles the behavior, i.e. which REST API to call? | 19:47 |
| *** fragatina has quit IRC | 19:47 | |
| tdurakov | afair there is no --server param in nova migration list | 19:48 |
| mriedem | there isn't | 19:48 |
| mriedem | but that's how i'd unify the CLIs | 19:48 |
| tdurakov | yeah | 19:48 |
| *** krtaylor has joined #openstack-nova | 19:48 | |
| tdurakov | sounds good! | 19:48 |
| tdurakov | once takashi implement the spec, it will be more useful | 19:49 |
| *** krtaylor has quit IRC | 19:50 | |
| *** fragatina has joined #openstack-nova | 19:50 | |
| *** fragatina has quit IRC | 19:50 | |
| mriedem | it is a bit odd, the os-migrations API behaves differently depending on if you're using >=2.23 | 19:51 |
| *** fragatina has joined #openstack-nova | 19:51 | |
| mriedem | 2.23+ will add links to servers for in-progress live migrations | 19:52 |
| mriedem | 2.23+ will always return the migration_type | 19:52 |
| mriedem | and os-migrations doesn't filter on just live migrations | 19:52 |
| mriedem | so it can also return cold migrations and evacuations | 19:52 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 19:53 |
| mriedem | /servers/uuid/migrations will only return in-progress live migrations | 19:53 |
| mriedem | we are definitely missing some behavioral details in the api-ref for os-migrations https://developer.openstack.org/api-ref/compute/?expanded=list-migrations-detail#list-migrations | 19:53 |
| *** tbachman has quit IRC | 19:54 | |
| *** ijw has joined #openstack-nova | 19:55 | |
| tdurakov | right, but the problem with the link, that is passed after 2.23 is already there | 19:55 |
| *** baoli has quit IRC | 19:56 | |
| mriedem | i'm not sure i understand your point | 19:56 |
| mriedem | os-migrations will return the links for 2.23+, and we also have /servers/id/migrations at 2.23 | 19:56 |
| mriedem | so yes it's redundant | 19:56 |
| mriedem | i believe that's why os-migrations is marked as "frozen" | 19:56 |
| mriedem | although the APIs do behave differently | 19:57 |
| mriedem | so it's kind of odd that os-migrations is frozen | 19:57 |
| mriedem | it seems to be frozen for a very specific case, which is the in-progress live migration case so you can cancel those things | 19:57 |
| *** tbachman has joined #openstack-nova | 19:57 | |
| tdurakov | my current understanding of the problem: I'd like to see /servers/uuid/migrations become more useful(not only live-migrations) and maybe not only active, based on that I think it's worth to have a mechanism to filter migrations not only by instnace, but host and cell as well | 19:57 |
| mriedem | but if i'm an admin and i just want to see any migration, including completed ones, i can't use /servers/id/migrations | 19:57 |
| mriedem | i don't even see where os-migrations filters by cell | 19:58 |
| tdurakov | mriedem: http://paste.openstack.org/show/600833/ | 19:59 |
| mriedem | tdurakov: in general yes, i think if /servers/id/migrations replaces os-migrations, then the former needs to be built up to serve the needs that the latter already provideds | 19:59 |
| mriedem | *provided | 19:59 |
| *** ijw has quit IRC | 19:59 | |
| tdurakov | is doc help doc misleading? | 19:59 |
| mriedem | i think so | 19:59 |
| mriedem | i don't see anything in the db api code about filtering migrations by cell name | 19:59 |
| * mriedem looks at the client code | 20:00 | |
| tdurakov | at the same point it would be weird to filter out migration by host at the uri: /servers/id/migrations because it's about servers, righht? | 20:01 |
| mriedem | yeah cell_name is passed as a query parameter | 20:01 |
| mriedem | correct | 20:01 |
| mriedem | so the GET query parameters are passed way down to migration_get_all_by_filters in the db api | 20:01 |
| mriedem | and cell_name isn't a thing that gets filtered on | 20:01 |
| mriedem | so the CLI code is lying to us | 20:01 |
| mriedem | :) | 20:01 |
| mriedem | cell_name is not even in the migrations table | 20:02 |
| mriedem | so i don't know who thought that up | 20:02 |
| *** baoli has joined #openstack-nova | 20:03 | |
| *** bastafidli has quit IRC | 20:04 | |
| mriedem | https://github.com/openstack/python-novaclient/commit/3c2a817efcb6954f3c8246c32ce997b9e3a4d000 | 20:07 |
| mriedem | cell_name was just always there | 20:07 |
| mriedem | crazy ass rax | 20:07 |
| mriedem | i'll remove that from novaclient | 20:07 |
| *** belmoreira has joined #openstack-nova | 20:08 | |
| *** mlavalle has joined #openstack-nova | 20:09 | |
| mriedem | this will allow me to procrastinate on recapping the resource provider talk from the PTG, which i didn't follow most of i don't think | 20:11 |
| mriedem | something something traits ironic inventory success glory fame | 20:11 |
| mriedem | jaypipes: ^ is that the summary from the PTG? | 20:11 |
| *** mrhillsman has quit IRC | 20:13 | |
| mriedem | sdague: i suppose to be a nice guy, i should deprecate a completely worthless and unused CLI arg/python API binding option in novaclient for a release before removing it right? | 20:13 |
| mriedem | even though it's never ever worked | 20:13 |
| *** baoli has quit IRC | 20:15 | |
| tdurakov | mriedem: patch is in progress | 20:15 |
| *** baoli has joined #openstack-nova | 20:15 | |
| mriedem | tdurakov: are you deprecating or removing? | 20:15 |
| mriedem | also https://bugs.launchpad.net/python-novaclient/+bug/1668743 | 20:15 |
| openstack | Launchpad bug 1668743 in python-novaclient "nova migration-list --cell_name does not actually do anything" [Undecided,New] - Assigned to Matt Riedemann (mriedem) | 20:15 |
| tdurakov | no reason procrastinating on resource provider | 20:15 |
| *** harlowja has quit IRC | 20:15 | |
| *** tbachman has quit IRC | 20:15 | |
| *** harlowja has joined #openstack-nova | 20:16 | |
| *** adrian_otto has quit IRC | 20:16 | |
| *** tbachman has joined #openstack-nova | 20:18 | |
| *** READ10 has joined #openstack-nova | 20:18 | |
| *** nic has quit IRC | 20:19 | |
| *** mrhillsman has joined #openstack-nova | 20:20 | |
| *** yamamoto has joined #openstack-nova | 20:23 | |
| *** yamamoto has quit IRC | 20:23 | |
| *** baoli has quit IRC | 20:25 | |
| *** baoli has joined #openstack-nova | 20:25 | |
| jaypipes | mriedem: kind of, though I'm actually writing an email summarizing resource providers/placement stuff at the PTG. do you want me to continue that? | 20:27 |
| mriedem | jaypipes: that would be tops | 20:28 |
| jaypipes | mriedem: so, continue this email then? :) | 20:29 |
| openstackgerrit | Timofey Durakov proposed openstack/python-novaclient master: remove cell_name parameter from nova migration-list https://review.openstack.org/439141 | 20:29 |
| tdurakov | mriedem: if you still want to procrastinate^ | 20:29 |
| mriedem | jaypipes: yes please | 20:30 |
| *** baoli has quit IRC | 20:30 | |
| mriedem | contrary to popular belief i don't love doing the recap emails | 20:30 |
| *** kaisers_ has quit IRC | 20:32 | |
| *** salv-orlando has quit IRC | 20:32 | |
| *** liverpooler has joined #openstack-nova | 20:34 | |
| *** bastafidli has joined #openstack-nova | 20:34 | |
| tdurakov | mriedem: what's the proper way deprecating params in nova-client? | 20:35 |
| mriedem | tdurakov: there is a DeprecatedAction class | 20:35 |
| mriedem | it's used elsewhere | 20:35 |
| tdurakov | mriedem: ok, will submit new deprecation one | 20:36 |
| tdurakov | but will remove that in a next release:-p | 20:36 |
| *** mvk has joined #openstack-nova | 20:37 | |
| *** kfarr has joined #openstack-nova | 20:41 | |
| *** bvanhav has quit IRC | 20:42 | |
| *** tbachman has quit IRC | 20:43 | |
| *** baoli has joined #openstack-nova | 20:43 | |
| *** liverpooler has quit IRC | 20:43 | |
| *** liverpooler has joined #openstack-nova | 20:44 | |
| openstackgerrit | Timofey Durakov proposed openstack/python-novaclient master: Deperecate cell_name cli arg for migration-list https://review.openstack.org/439147 | 20:50 |
| tdurakov | mriedem: ^ | 20:50 |
| *** mrda has left #openstack-nova | 20:51 | |
| *** tbachman has joined #openstack-nova | 20:51 | |
| mriedem | sfinucan: you've been deprecating nova-network related options, why not also firewall_driver? | 20:53 |
| *** aslezil_ has quit IRC | 20:53 | |
| mriedem | once nova-network is gone, there is only one impl for firewall_driver which is the noop one | 20:53 |
| *** aslezil_ has joined #openstack-nova | 20:53 | |
| *** yamamoto has joined #openstack-nova | 20:54 | |
| *** aslezil_ has quit IRC | 20:55 | |
| *** aslezil_ has joined #openstack-nova | 20:55 | |
| *** ijw has joined #openstack-nova | 20:56 | |
| *** lpetrut has quit IRC | 20:56 | |
| *** aslezil_ has quit IRC | 20:56 | |
| *** aslezil_ has joined #openstack-nova | 20:57 | |
| *** baoli has quit IRC | 20:58 | |
| *** yamamoto has quit IRC | 20:59 | |
| *** ijw has quit IRC | 21:01 | |
| *** Apoorva_ has joined #openstack-nova | 21:01 | |
| *** kaisers_ has joined #openstack-nova | 21:02 | |
| *** baoli has joined #openstack-nova | 21:03 | |
| *** sdague has quit IRC | 21:03 | |
| *** moshele has joined #openstack-nova | 21:05 | |
| *** Apoorva has quit IRC | 21:05 | |
| *** lpetrut has joined #openstack-nova | 21:05 | |
| tonyb | sfinucan: \o/ | 21:05 |
| *** abhishek_k has joined #openstack-nova | 21:08 | |
| *** abhishekk has quit IRC | 21:11 | |
| *** armax has quit IRC | 21:12 | |
| *** adrian_otto has joined #openstack-nova | 21:13 | |
| *** dave-mccowan has quit IRC | 21:13 | |
| *** READ10 has quit IRC | 21:15 | |
| *** READ10 has joined #openstack-nova | 21:17 | |
| *** yamamoto has joined #openstack-nova | 21:17 | |
| *** belmoreira has quit IRC | 21:18 | |
| *** belmoreira has joined #openstack-nova | 21:20 | |
| *** yamamoto has quit IRC | 21:21 | |
| *** crushil has quit IRC | 21:24 | |
| *** moshele has quit IRC | 21:25 | |
| *** mjura has quit IRC | 21:25 | |
| *** aysyd has quit IRC | 21:25 | |
| *** artom has quit IRC | 21:27 | |
| *** kfarr has quit IRC | 21:27 | |
| *** belmoreira has quit IRC | 21:27 | |
| *** jwcroppe has quit IRC | 21:28 | |
| *** smatzek_ has quit IRC | 21:28 | |
| *** jwcroppe has joined #openstack-nova | 21:28 | |
| *** ociuhandu has joined #openstack-nova | 21:31 | |
| *** artom has joined #openstack-nova | 21:32 | |
| *** artom has quit IRC | 21:32 | |
| *** jwcroppe_ has joined #openstack-nova | 21:32 | |
| *** artom has joined #openstack-nova | 21:33 | |
| *** jwcroppe has quit IRC | 21:33 | |
| dansmith | mriedem: so on the traits thing, | 21:33 |
| *** acabot has quit IRC | 21:33 | |
| dansmith | mriedem: I would hate to see us delay merging the mechanical stuff because we're obsessing over the stuff in the spec which is a little higher level than what we need right away | 21:34 |
| dansmith | obviously I don't want to have two specs, but I wonder if we could move the -2 up a couple patches without risking too much? | 21:34 |
| *** slaweq__ has joined #openstack-nova | 21:34 | |
| dansmith | I haven't looked over the comments on the spec, I just see that there are some | 21:35 |
| *** slaweq_ has quit IRC | 21:35 | |
| *** Jeffrey4l has quit IRC | 21:35 | |
| *** yassine has quit IRC | 21:36 | |
| mriedem | do we have any api db blocker migrations we need to do first or anything? | 21:38 |
| dansmith | I don't think so | 21:38 |
| *** crushil has joined #openstack-nova | 21:38 | |
| dansmith | that said, I'm looking at the set of patches and I'm not sure where I'd re-place the -2 without just delaying at a different spot | 21:38 |
| dansmith | so maybe it's not worth it | 21:39 |
| *** ijw has joined #openstack-nova | 21:39 | |
| *** salv-orlando has joined #openstack-nova | 21:39 | |
| mriedem | unicode traits huh | 21:40 |
| mriedem | pile of poo can be a trait | 21:40 |
| *** thorst has quit IRC | 21:41 | |
| *** thorst has joined #openstack-nova | 21:42 | |
| *** gouthamr has quit IRC | 21:44 | |
| *** yassine has joined #openstack-nova | 21:45 | |
| mriedem | do we have a resource tracker patch ready to set the trait that depends on this data model change? | 21:46 |
| *** thorst has quit IRC | 21:46 | |
| mriedem | i don't see anything in RT building on this yet | 21:46 |
| *** acabot has joined #openstack-nova | 21:46 | |
| *** lucasxu has quit IRC | 21:46 | |
| *** Jeffrey4l has joined #openstack-nova | 21:48 | |
| *** unicell has quit IRC | 21:49 | |
| dansmith | mriedem: the only RT intersection is reporting the compute-node trait on startup/RP creation or something | 21:49 |
| *** lucasxu has joined #openstack-nova | 21:49 | |
| *** Apoorva_ has quit IRC | 21:50 | |
| mriedem | yeah, and then the scheduler passing that into the resource provider GET query with the filter param right? | 21:50 |
| mriedem | and something something aggregates fits in there too | 21:50 |
| *** Apoorva has joined #openstack-nova | 21:50 | |
| mriedem | so, | 21:50 |
| mriedem | the immediate need for this is for scheduling with shared storage pools and aggregate resource providers right? | 21:51 |
| mriedem | we have a separate bp for that - we could just tie this into that work and decouple the data model change from the REST API/scheduling fallout that will probably come from the traits spec | 21:51 |
| mriedem | if that makes sense | 21:51 |
| *** ijw has quit IRC | 21:52 | |
| dansmith | scheduler asks for that trait to placement, but yeah | 21:52 |
| *** ijw has joined #openstack-nova | 21:52 | |
| dansmith | the immediate need is that we can't start reporting any shared pool resources (regardless of if we use them) until we can distinguish them from compute nodes | 21:52 |
| mriedem | yeah, so let's do the traits data model change under https://blueprints.launchpad.net/nova/+spec/shared-resources-pike | 21:52 |
| mriedem | is that a fair compromise? | 21:53 |
| dansmith | is that a paperwork hack? | 21:53 |
| dansmith | I don't care how you justify it :P | 21:53 |
| *** unicell has joined #openstack-nova | 21:53 | |
| *** annegentle has quit IRC | 21:54 | |
| mriedem | it's a papercut | 21:54 |
| dansmith | *rimshot* | 21:55 |
| *** yamahata has quit IRC | 21:55 | |
| mriedem | jaypipes: oh sql wizard of yore, can you answer my question in https://review.openstack.org/#/c/376198/11/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/041_resource_provider_traits.py@57 ? | 21:55 |
| *** ijw has quit IRC | 21:56 | |
| mriedem | cburgess: L33 here https://etherpad.openstack.org/p/nova-ptg-pike-cinder | 21:57 |
| cburgess | mriedem Yes what? | 21:57 |
| mriedem | cburgess: what was the outcome of that? wasn't it "roll this into johnthetubaguy's spec"? | 21:57 |
| cburgess | Yes it was | 21:57 |
| *** tblakes has quit IRC | 21:57 | |
| mriedem | hot | 21:57 |
| mriedem | damn | 21:57 |
| cburgess | mriedem See my comments on https://review.openstack.org/#/c/373203/14 | 21:58 |
| cburgess | I need to re-review that though now. | 21:58 |
| mriedem | thus concludes my nova/cinder recap email, yay | 21:59 |
| cburgess | mriedem I'll review rev 15 of that spec today and update with any comments I have as it related to our spec that needs to be merged into this one. | 22:00 |
| *** dtp has joined #openstack-nova | 22:00 | |
| mriedem | super | 22:00 |
| *** Apoorva_ has joined #openstack-nova | 22:05 | |
| *** armax has joined #openstack-nova | 22:06 | |
| *** Apoorva has quit IRC | 22:08 | |
| *** krtaylor has joined #openstack-nova | 22:08 | |
| cburgess | mriedem Comments added. We still need to account for the changes that will impact BFV in the spec | 22:09 |
| *** ducttape_ has joined #openstack-nova | 22:10 | |
| *** claudiub has joined #openstack-nova | 22:12 | |
| *** yamamoto has joined #openstack-nova | 22:12 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Default firewall_driver to nova.virt.firewall.NoopFirewallDriver https://review.openstack.org/439187 | 22:12 |
| *** tbachman has quit IRC | 22:13 | |
| *** baoli has quit IRC | 22:13 | |
| *** baoli has joined #openstack-nova | 22:13 | |
| *** baoli has quit IRC | 22:14 | |
| *** edmondsw has quit IRC | 22:17 | |
| *** bastafidli has quit IRC | 22:22 | |
| *** baoli has joined #openstack-nova | 22:24 | |
| *** gouthamr has joined #openstack-nova | 22:30 | |
| *** mnestratov|2 has joined #openstack-nova | 22:33 | |
| *** annegentle has joined #openstack-nova | 22:34 | |
| jaypipes | mriedem: answered. sorry for delay. | 22:36 |
| *** raunak has quit IRC | 22:36 | |
| *** mnestratov has quit IRC | 22:36 | |
| jaypipes | mriedem: I blame cburgess. | 22:36 |
| cburgess | jaypipes Wait what did I do this time? | 22:36 |
| mriedem | jaypipes: i'm not sure i understand your response | 22:37 |
| mriedem | resource_provider_traits doesn't have a unique constraint defined, but it has 2 primary keys | 22:37 |
| *** raunak has joined #openstack-nova | 22:37 | |
| *** amotoki has quit IRC | 22:37 | |
| mriedem | do we not need a unique constraint because of those 2 primary keys? | 22:37 |
| *** mnestratov|2 has quit IRC | 22:38 | |
| cburgess | A primary key is already unique. Why would you need to unique them again? | 22:38 |
| jaypipes | cburgess: everything. you did everything. | 22:38 |
| mriedem | it's the combo, but i think i'm just over thinking it | 22:38 |
| jaypipes | mriedem: the primary key is unique and already contains trait and provider id, yes. | 22:38 |
| *** antwash has left #openstack-nova | 22:38 | |
| cburgess | jaypipes I'm glad someone has finally realized that. | 22:39 |
| jaypipes | mriedem: and the uniqueness isn't dependent on the order. In other words, the constraint of uniqueness over (trait, provider) applies implicitly to (provider, trait) | 22:39 |
| *** amotoki has joined #openstack-nova | 22:42 | |
| mriedem | and we're doing a mapping table b/c you can have M:M | 22:42 |
| mriedem | unlike inventories/resource_classes | 22:42 |
| mriedem | a single inventory has a single resource class | 22:43 |
| mriedem | but a resource provider can have multiple traits | 22:43 |
| mriedem | and a trait can be used by multiple resource providers | 22:43 |
| mriedem | used/exposed whatever | 22:43 |
| mriedem | alright | 22:43 |
| *** baoli has quit IRC | 22:43 | |
| *** lpetrut has quit IRC | 22:44 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: [placement] Add Traits related table to the api database https://review.openstack.org/376198 | 22:44 |
| mriedem | jaypipes: dansmith: ok +2 on https://review.openstack.org/#/c/376198/ now | 22:45 |
| *** tjones has joined #openstack-nova | 22:45 | |
| *** tjones has left #openstack-nova | 22:46 | |
| *** amotoki has quit IRC | 22:46 | |
| *** crushil has quit IRC | 22:48 | |
| *** burt has quit IRC | 22:49 | |
| jaypipes | mriedem: yup, zactly correct. | 22:49 |
| *** adrian_otto has quit IRC | 22:50 | |
| jaypipes | mriedem: and thanks for adding that blueprint to get this going. | 22:50 |
| *** salv-orlando has quit IRC | 22:50 | |
| dansmith | jaypipes: left the +W for you | 22:50 |
| *** salv-orlando has joined #openstack-nova | 22:50 | |
| mriedem | ok hosers i'll talk to you tomorrow | 22:51 |
| *** mriedem is now known as mriedem_away | 22:51 | |
| dansmith | melwitt: *gasp* https://review.openstack.org/#/c/438025/ | 22:52 |
| dansmith | looks like the gate is too busy to run my devstack patch against that, but at least it's passing the regular setup | 22:52 |
| *** ijw has joined #openstack-nova | 22:53 | |
| jaypipes | Wallaby'd. | 22:53 |
| *** edmondsw has joined #openstack-nova | 22:58 | |
| *** ijw has quit IRC | 22:58 | |
| masber | hi | 23:00 |
| *** esberglu has quit IRC | 23:00 | |
| dansmith | infra says nodepool is jammed up, fyi to anyone waiting | 23:00 |
| *** edmondsw has quit IRC | 23:01 | |
| masber | anybody knows why when I do "nova image-create --poll ..." the image size is 0? | 23:01 |
| *** edmondsw has joined #openstack-nova | 23:01 | |
| masber | I am following this document --> https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/5/html/End_User_Guide/cli_migrate_instances.html#create_snapshot | 23:01 |
| masber | and this one --> https://docs.openstack.org/user-guide/cli-use-snapshots-to-migrate-instances.html | 23:02 |
| dansmith | masber: that first document is about icehouse, which is ancient. nova image routines are deprecated, and this is not a support channel (please see topic) | 23:02 |
| *** esberglu has joined #openstack-nova | 23:02 | |
| *** jwcroppe_ has quit IRC | 23:03 | |
| *** felipemonteiro_ has quit IRC | 23:03 | |
| *** felipemonteiro has quit IRC | 23:03 | |
| *** armax has quit IRC | 23:03 | |
| *** esberglu has quit IRC | 23:07 | |
| *** baoli has joined #openstack-nova | 23:07 | |
| *** jamesdenton has quit IRC | 23:08 | |
| masber | dansmith, sorry, I couldn't get help on #openstack channel | 23:11 |
| masber | so i was wondering if I could get some ideas from here as the people is specialised on nova... | 23:11 |
| *** tbachman has joined #openstack-nova | 23:11 | |
| masber | apologise again | 23:12 |
| *** lucasxu has quit IRC | 23:15 | |
| *** Apoorva_ has quit IRC | 23:17 | |
| *** Apoorva has joined #openstack-nova | 23:17 | |
| *** edmondsw has quit IRC | 23:18 | |
| *** edmondsw has joined #openstack-nova | 23:19 | |
| *** crushil has joined #openstack-nova | 23:20 | |
| *** baoli has quit IRC | 23:22 | |
| *** ducttape_ has quit IRC | 23:23 | |
| *** jwcroppe has joined #openstack-nova | 23:25 | |
| *** tbachman_ has joined #openstack-nova | 23:27 | |
| *** tbachman has quit IRC | 23:30 | |
| *** tbachman_ is now known as tbachman | 23:30 | |
| *** Fdaisuke has joined #openstack-nova | 23:33 | |
| *** tbachman has quit IRC | 23:35 | |
| *** tbachman has joined #openstack-nova | 23:41 | |
| *** marst has quit IRC | 23:42 | |
| *** liangy has quit IRC | 23:45 | |
| *** edmondsw has quit IRC | 23:45 | |
| *** edmondsw has joined #openstack-nova | 23:46 | |
| melwitt | dansmith: woot \o/ | 23:46 |
| *** yamahata has joined #openstack-nova | 23:47 | |
| *** sacharya has quit IRC | 23:47 | |
| *** tbachman has quit IRC | 23:48 | |
| *** edmondsw has quit IRC | 23:50 | |
| *** amotoki has joined #openstack-nova | 23:53 | |
| *** jwcroppe has quit IRC | 23:53 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: ironic inventory reporting https://review.openstack.org/437602 | 23:53 |
| *** crushil has quit IRC | 23:53 | |
| jaypipes | dansmith: ok dokey :) ^^ hopefully I got it right this time. | 23:54 |
| *** ijw has joined #openstack-nova | 23:54 | |
| jaypipes | heading back to hotel from office... biab. | 23:54 |
| *** annegentle has quit IRC | 23:55 | |
| *** dave-mccowan has joined #openstack-nova | 23:57 | |
| cburgess | jaypipes Yick another week of travel right after PTG? | 23:58 |
| *** ijw has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!