| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-18 | |||
| 02:20:56 | openstackgerrit | lei zhang proposed openstack/nova master: Fix openstackdocstheme options for api-ref https://review.openstack.org/534999 | |
| 02:21:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | |
| 02:35:12 | openstackgerrit | Merged openstack/nova master: Fix fake libvirt XML generation for disks https://review.openstack.org/531165 | |
| 02:35:19 | openstackgerrit | Merged openstack/nova master: Account for deprecation of personality files https://review.openstack.org/534849 | |
| 02:52:25 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506 | |
| 03:12:01 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/535030 | |
| 03:22:00 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994 | |
| 03:22:11 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918 | |
| 03:30:09 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/535121 | |
| 04:06:52 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [WIP]Fix update server's wrong meta when update concurrently https://review.openstack.org/534682 | |
| 04:36:41 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823 | |
| 05:13:55 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258 | |
| 05:47:36 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266 | |
| 06:09:06 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280 | |
| 06:16:19 | diga | jaypipes: hi | |
| 06:59:02 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299 | |
| 07:05:24 | openstackgerrit | Kevin Zhao proposed openstack/os-traits master: Add CPU features for AArch64 https://review.openstack.org/523665 | |
| 07:11:46 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | |
| 07:21:15 | openstackgerrit | sunjiazz proposed openstack/python-novaclient master: Update documentation links https://review.openstack.org/535196 | |
| 07:42:22 | pmannidi | Hi all, i've a question, what happens if we have a aggregate with a AZ having 1 host where you spawned a instance and try to resize the same? | |
| 07:45:18 | openstackgerrit | sunjiazz proposed openstack/python-novaclient master: Update documentation links https://review.openstack.org/535196 | |
| 08:13:59 | jc_ | Hello everybody. Is it here that I can ask questions regarding Nova installation ? | |
| 08:25:47 | ttx | jc_: you can try #openstack too. This channel is not very busy until the US people get up | |
| 08:26:19 | jc_ | ttx OK I'll try this, thanks ! | |
| 08:38:47 | gmann | alex_xu: can you check these (stuck for long time) - https://review.openstack.org/#/c/522918/ and https://review.openstack.org/#/c/527042/ | |
| 08:47:35 | alex_xu | gmann: got it | |
| 08:48:53 | gmann | alex_xu: thanks. | |
| 08:50:17 | alex_xu | gmann: np | |
| 09:10:56 | openstackgerrit | Merged openstack/nova master: add _has_provider_trees() utility function https://review.openstack.org/531474 | |
| 09:15:52 | openstackgerrit | Gao Fei proposed openstack/nova master: Modify broken link https://review.openstack.org/535244 | |
| 09:21:54 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [WIP]Fix update server's wrong meta when update concurrently https://review.openstack.org/534682 | |
| 09:39:34 | mdbooth | lyarwood: Comment in https://review.openstack.org/#/c/531524/ about stable rescue. Did you get an email? | |
| 09:40:38 | kashyap | ttx: Hey, some people are in the Eu too :P | |
| 09:40:55 | kashyap | (Who are active on the channel, I mean) | |
| 09:42:08 | lyarwood | mdbooth: yeah sorry, lost in all of the FFU review mail I'm getting at the moment | |
| 09:42:19 | lyarwood | mdbooth: I'll get to it today | |
| 09:42:44 | Spazmotic | and don't forget third shift :p | |
| 09:42:48 | mdbooth | lyarwood: Ok. Have you had a chance to fix up the luks patch? If you're swamped I'm happy to find and fix the test errors. | |
| 09:43:01 | lyarwood | mdbooth: just working on it now | |
| 09:43:23 | lyarwood | mdbooth: had to sort out c-vol last night https://review.openstack.org/#/c/534811 | |
| 09:43:26 | mdbooth | lyarwood: Ok, cool. Ping me when you want a review. | |
| 09:43:33 | lyarwood | mdbooth: will do, thanks | |
| 09:43:44 | kashyap | Can someone take a look at this one, for a quick feedback on the core libvirt XML wrangling? -- https://review.openstack.org/#/c/534384/ | |
| 09:44:16 | kashyap | (I still need to fix 9 or so unit tests.) | |
| 09:44:23 | mdbooth | kashyap: Looking at that, too | |
| 09:46:12 | mdbooth | kashyap: Whoa, just cranking up my brain to process the commit message :) | |
| 09:46:13 | kashyap | mdbooth: Thank you. I am not sure if I got all the LibvirtConfigGuestCPUFeature() object generation stuff right | |
| 09:46:23 | kashyap | mdbooth: Hehe, that's why I gave the tl;dr :P | |
| 09:46:30 | kashyap | Is it obscenely verbose? | |
| 09:47:03 | mdbooth | No, it's probably just right. | |
| 09:48:23 | kashyap | Gerrit lacks "cover letter" concept. That actually should go into that (although it won't be in the commit message, though) | |
| 09:49:05 | kashyap | mdbooth: The most practical bits are tl;dr and the "Other use cases" -- as in, what else I can use this for | |
| 09:49:36 | kashyap | (s/I/one/) | |
| 09:50:15 | mdbooth | kashyap: Personally I think this is appropriate detail for a commit message. A cover letter is more for navigating multiple commits imho. | |
| 09:50:50 | mdbooth | You can't find a cover letter when git spelunking, so you shouldn't leave critical information there. | |
| 09:51:47 | Spazmotic | I hope john takes mercy on my poor xenapi code today.. I will build an alter of sorts to his xenapis knowledge to assist.+ | |
| 09:53:01 | kashyap | mdbooth: Yeah, you're right. I mixed it up | |
| 09:53:37 | kashyap | mdbooth: Still a bit shaken from my bike ride, where the 140 KM/hr wind gust almost pushed me into the tram! | |
| 09:54:26 | mdbooth | Sounds like an excellent excuse not to be on your bike! | |
| 09:54:28 | kashyap | But point for navigating multiple messages via a cover letter is still a missing feature of Gerrit | |
| 09:55:31 | kashyap | mdbooth: Yeah. I couldn't believe for a moment that the wind was simply "pushing" me aside, like a leaf, and I had to really resist it | |
| 09:55:57 | mdbooth | kashyap: This is where I ride my bike on windy days: https://photos.app.goo.gl/OCo5xBNqWi0yNs5y1 | |
| 09:56:00 | Spazmotic | You need more spoilers and wings on that thing | |
| 09:57:14 | kashyap | mdbooth: Hehe | |
| 09:57:19 | kashyap | I love the bike | |
| 09:57:21 | mdbooth | Cold is fine, wet is... sometimes ok. Ice and wind are never ok. | |
| 09:57:27 | kashyap | Aside: http://knmi.nl/waarschuwingen (About the weather alarm; use translate) | |
| 10:11:19 | mdbooth | kashyap: So I just finished reading and digesting your commit message, and the linked article | |
| 10:12:34 | mdbooth | kashyap: FWIW, assuming libvirt/qemu aren't planning on handling this at that layer, we should probably switch this on *by default* in Nova, with a tuning knob to switch it back off if somebody has a weird reason to need that. | |
| 10:14:21 | kashyap | mdbooth: So you raise a good point | |
| 10:15:04 | kashyap | mdbooth: About being handled at QEMU level. I was involved on the qemu-devel list thread where there was a debate about adding a specific CPU model | |
| 10:15:04 | mdbooth | kashyap: That wouldn't be a something for this change, though | |
| 10:15:16 | kashyap | That will include the PCID feature by *default* | |
| 10:15:34 | kashyap | However. It's soon getting ridiculous, because of this exhibit: | |
| 10:16:03 | kashyap | E.g. one CPU model name will be: 'Haswell-noTSX-IBRS' | |
| 10:16:08 | kashyap | See anything odd? | |
| 10:16:23 | kashyap | How about this: 'Haswell-noTSX-IBRS-nowThisBug-andAlsoThusBug-AndMoreBugs' | |
| 10:16:33 | mdbooth | Yeah, that's pretty silly | |
| 10:17:14 | kashyap | So that was the thread, please read DanPB's response on context of Nova: https://lists.nongnu.org/archive/html/qemu-devel/2018-01/msg03689.html | |
| 10:17:24 | kashyap | mdbooth: Thanks for taking time to read this! I appreciate the concentration | |
| 10:17:52 | kashyap | 'this' as in: the Nova change and all the related info | |
| 10:18:18 | kashyap | mdbooth: But the thing is...about switching on by _default_, it's complicated. | |
| 10:18:23 | mdbooth | From reading that post, it sounds like this is something (?:pretty much)? all our users will want. | |
| 10:18:47 | kashyap | As this perf impact will only occur when someone chooses one of the named CPU models that doesn't have the PCID feature in libvirt's notion of the model. | |
| 10:18:52 | kashyap | (But, it exists in the hardware) | |
| 10:18:59 | kashyap | mdbooth: It won't affect users | |
| 10:19:06 | kashyap | ...*if* they chose 'host-model' | |
| 10:19:12 | mdbooth | users==operators | |
| 10:19:16 | kashyap | In that case, libvirt will do the grump work for you | |
| 10:19:23 | kashyap | mdbooth: (Yeah, I translated as much) | |
| 10:19:29 | kashyap | s/grump/grunt/ | |
| 10:20:27 | kashyap | mdbooth: One thing is for *sure*. We (or I) have to do some clear write-up about what is the operator impact in all three cases: | |
| 10:20:31 | kashyap | (1) 'host-model' | |
| 10:20:35 | kashyap | (2) 'host-passthrough' | |
| 10:20:51 | kashyap | (3) Custom CPU model <-- this is where my change comes in | |
| 10:20:51 | mdbooth | kashyap: Is that something we can detect at startup? Host-model = X, real cpu is Y, so we should change host-model to X + PCID | |
| 10:22:30 | mdbooth | A reason *not* to do this would be the upgrade impact of not being able to live-migrate across the boundary? | |
| 10:22:45 | mdbooth | i.e. X != X+PCID, so you can't live migrate | |
| 10:23:04 | kashyap | mdbooth: Yes, live migration is a strict requirement | |
| 10:23:04 | Spazmotic | You mean for the CPU mismatch? | |