| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-12 | |||
| 09:32:00 | gibi | morning | |
| 09:38:52 | openstackgerrit | Alex Xu proposed openstack/nova-specs master: Report CPU features to placement service by traits API https://review.openstack.org/497733 | |
| 09:48:31 | bauzas | good morning Nova | |
| 09:48:35 | bauzas | just spent 2 hours of investigation because had a severe network outage | |
| 09:48:52 | bauzas | found the root cause https://twitter.com/sylvainbauza/status/973127051074031616 | |
| 09:59:19 | gibi | bauzas: nice start for a Monday | |
| 09:59:37 | bauzas | yeah, it just killed my morning :( | |
| 10:00:15 | bauzas | in general, I always walk the dog in the mornings | |
| 10:00:49 | bauzas | but today, I was wanting to work earlier | |
| 10:00:50 | gibi | bauzas: but you solved a hard problem already so you were productive | |
| 10:01:07 | bauzas | meh | |
| 10:01:26 | bauzas | I haven't understood why the Vera Plus is saturing the switch | |
| 10:03:18 | openstackgerrit | sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | |
| 10:04:33 | openstackgerrit | sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | |
| 10:16:42 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Avoid exploding if guest refuses to detach a volume https://review.openstack.org/551948 | |
| 10:19:13 | Spazmotic | Well that sounds like a gross start to the week. | |
| 10:21:21 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Avoid exploding if guest refuses to detach a volume https://review.openstack.org/551950 | |
| 10:24:01 | Spazmotic | Hmm there's a similar event for that in the XenAPI Drivers that I should add that into | |
| 10:26:50 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501 | |
| 10:31:53 | openstackgerrit | sahid proposed openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605 | |
| 10:43:21 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501 | |
| 11:31:11 | johnthetubaguy | jaypipes: I am wondering about traits on glance images, should the flavor be allowed to deny some / give permission for some (thinking of billing being linked to flavors really) Maybe it goes back to that forbidden traits configuration idea? | |
| 11:34:19 | Spazmotic | Glance Metadata already has restrictions built into its metadata such as minRam, MinDisk. Any reason to duplicate that? | |
| 11:34:50 | johnthetubaguy | well it has protected properties, its just you likely want that per flavor | |
| 11:35:06 | Spazmotic | That's true enough, they are the protected ones | |
| 11:35:08 | johnthetubaguy | well, or rather you want a whitelist not a blacklist I think | |
| 11:36:18 | Spazmotic | I see, I guess I could see the useful ness for that on some image types. Was just curious mostly :) | |
| 11:48:53 | openstackgerrit | Merged openstack/nova-specs master: Return Generation from Resource Provider Creation https://review.openstack.org/548903 | |
| 11:54:40 | stephenfin | gibi, jaypipes: Can we jump on a Hangout sometime later this afternoon when you're both about? I'd like to hash out [1] before I write anything more for it [1] https://review.openstack.org/#/c/541290/ | |
| 11:55:11 | gibi | stephenfin: sure, we can | |
| 11:55:55 | gibi | stephenfin: after the scheduler meeting maybe? | |
| 12:06:51 | johnthetubaguy | stephenfin: just spotted some changes left in limbo, I am tempted to push these forward for you, but should it say rocky now? https://review.openstack.org/#/c/499179/ | |
| 12:15:13 | openstackgerrit | Merged openstack/nova-specs master: Re-propose convert consoles code to use objects framework https://review.openstack.org/543662 | |
| 12:15:47 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add request_id filed to InstanceAction versioned notifications https://review.openstack.org/551982 | |
| 12:18:10 | openstackgerrit | Merged openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/540879 | |
| 12:30:59 | openstackgerrit | Claudiu Belu proposed openstack/nova master: db: Adds live-resize to Migration model migration_type https://review.openstack.org/185961 | |
| 12:35:37 | efried | edleafe: Before I start filtering through a week worth of emails, anything earth-shattering I should know about from last week? | |
| 12:36:25 | Spazmotic | johnthetubaguy, thanks sir :) | |
| 12:36:32 | efried | This is literally the first time I've booted up my computer since the PTG | |
| 12:37:33 | edleafe | efried: glad to hear you know how to do PTO | |
| 12:37:50 | edleafe | efried: but no, nothing other than the usual | |
| 12:37:50 | efried | ~800 unread emails. Sigh. | |
| 12:37:57 | efried | edleafe: Cool, thanks. | |
| 12:40:18 | jaypipes | stephenfin: sure. after scheduler meeting works for me. | |
| 12:40:20 | efried | Hi gibi | |
| 12:42:30 | jaypipes | johnthetubaguy: currently, we have nothing in the os-traits library or in nova that identifies "conflicting" traits. for example, there's nothing preventing an admin from decorating a compute node with both the STORAGE_DISK_HDD and STORAGE_DISK_SSD traits at the same time. We will need some logic somewhere (not in the virt driver I would hope) to process conflicts in requested traits between the flavor and image I would think? | |
| 12:43:51 | openstackgerrit | sahid proposed openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605 | |
| 12:44:00 | cdent | can we call that rope? If you try it, it doesn't work, you investigate, you see why, don't do that again | |
| 12:44:22 | cdent | putting _meaning_ into the traits (rather than just beign symbols) will get messy | |
| 12:44:44 | cdent | jaypipes: ^ | |
| 12:44:58 | efried | jaypipes, cdent: Yeah, I agree we don't want to try to get too clever - *especially* outside of virt driver - about inter-trait semantics like that. | |
| 12:45:27 | efried | For the SSD/HDD example, if I'm not using trees, and I have both kinds of disk on my compute host, that could be a valid configuration. | |
| 12:45:58 | cdent | welcome back efried | |
| 12:46:01 | efried | Even if I am using trees, it's not certain I would split the local disk out from the compute node RP | |
| 12:46:04 | efried | Thanks cdent! | |
| 12:46:24 | efried | oh, you mean in the request | |
| 12:46:44 | efried | So if I make a resource request that has both of those traits, in one request group, it doesn't make sense. | |
| 12:47:57 | efried | ...and would *usually* fail in GET /a_c. Unless you've got the aforementioned host with both kinds of disk on it. | |
| 12:50:49 | jaypipes | efried: for sure. | |
| 12:54:09 | edleafe | I agree that there shouldn't be meaning in traits. If there needs to be logic added, it should be in the layer that is applying the traits, not in the trait API. | |
| 12:55:34 | efried | What we NEED is trait metadata | |
| 12:55:42 | efried | And aggregate metadata. | |
| 12:55:48 | edleafe | somebody shoot that guy | |
| 12:57:36 | jaypipes | heh | |
| 12:58:12 | cdent | actually, I'd prefer trait traits | |
| 13:00:01 | jaypipes | cdent: lol | |
| 13:01:28 | edleafe | that's silly. What we need are aggregates of traits | |
| 13:01:34 | edleafe | with metadata, of course | |
| 13:03:30 | cdent | why have aggregates when we can have trait traits traits? | |
| 13:04:03 | cdent | (I actually do think it would be much more easy to reason about the world if we only had traits and not aggregates) | |
| 13:04:18 | cdent | but I'm a faceted classification kind of guy | |
| 13:04:55 | edleafe | we've been through this. There really is no difference. I prefer traits because it's shorter to type | |
| 13:05:02 | jaypipes | buffalo buffalo buffalo buffalo buffalo buffalo buffalo | |
| 13:06:06 | cdent | parkay | |
| 13:07:03 | cdent | If traits is shorter than aggregates and that's good, then _clearly_ tags are the way to go | |
| 13:07:48 | arvindn05 | continue the discussion on #openstack-meeting-alt ? | |
| 13:07:50 | jaypipes | hehe | |
| 13:08:07 | jaypipes | arvindn05: in 55 minutes, no? | |
| 13:08:17 | edleafe | arvindn05: in 52 minutes, sure | |
| 13:08:20 | edleafe | jinx | |
| 13:08:21 | arvindn05 | ah...DST :) | |
| 13:08:21 | jaypipes | arvindn05: daylight savings... :) | |
| 13:08:24 | jaypipes | yup | |
| 13:08:30 | jaypipes | gets me every time. | |
| 13:09:40 | arvindn05 | would have thought outlook adjusted...oh well...extra hour for me :) | |
| 13:11:01 | arvindn05 | edleafe: cdent: latest patch set on https://review.openstack.org/#/c/541507/ addresses other concerns..can you add your review and original +1's? | |
| 13:11:39 | edleafe | arvindn05: I have that and a bunch of others open for review today. | |
| 13:12:23 | cdent | yup, already have that open in tab | |
| 13:12:43 | arvindn05 | great...thanks. | |
| 13:13:22 | arvindn05 | i think i need a workflow +1...i am guessing that should come from matt? | |
| 13:24:40 | stephenfin | johnthetubaguy: They should. I'll fix them up and ping you then, if that's OK? | |
| 13:25:17 | Spazmotic | Welcome home, BTW, efried | |
| 13:25:58 | efried | Spazmotic: Thanks! | |
| 13:26:12 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove '[conductor] topic' opt https://review.openstack.org/499179 | |
| 13:27:21 | stephenfin | johnthetubaguy: Actually, it was just the commit message so I stripped out that line (there's no rocky equivalent) and sent it through. Fancy hitting https://review.openstack.org/#/c/508487 though? | |
| 13:30:59 | Spazmotic | c'mon tempest-full.. don't play with me | |
| 13:31:21 | Spazmotic | My achey breaky heart and all that. | |
| 13:32:22 | sahid | jaypipes: we have some kind of agrements on this spec: https://review.openstack.org/#/c/539605/ if we can have you to make a first +2 | |
| 13:33:29 | sahid | cfriesen: any chance you have a look at this https://review.openstack.org/#/c/511188/ ? | |
| 13:37:18 | openstackgerrit | Merged openstack/nova master: Raise a proper exception in unit test https://review.openstack.org/550914 | |