| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-15 | |||
| 22:38:06 | efried | melwitt: Okay, I added a note to that effect, with a reference to the doc. | |
| 22:38:52 | melwitt | ok, thanks | |
| 22:39:11 | efried | melwitt: Based on what I'm reading at https://docs.openstack.org/nova/latest/contributor/blueprints.html and https://specs.openstack.org/openstack/nova-specs/readme.html#trivial-specifications I think you're probably right. | |
| 22:41:49 | melwitt | efried: thanks. yeah, was also looking at the git blame to see how other extra specs were added, for example one was from https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/libvirt-emulator-threads-policy.html | |
| 22:42:36 | efried | mm, that one clearly not specless | |
| 22:43:41 | melwitt | yeah. but for this one, it seems small enough that specless is my expectation, but I'll run it by mriedem and dansmith because sometimes a lot of questions arise and if that happens, a spec would be more appropriate | |
| 22:44:10 | melwitt | I can't say I know anyting about KSM myself | |
| 22:44:15 | melwitt | *anything | |
| 22:59:07 | efried | melwitt: Oh, technically I'm relying on sahid and sean-k-mooney to vet it. Procedurally... yeah, Matt and Dan. | |
| 23:00:27 | dansmith | I dunno about the claim that windows crashes when KSM is used.. can't really think of any reason that would be | |
| 23:00:35 | dansmith | normally I think of KSM as being a host decision or not, | |
| 23:00:46 | dansmith | so I'm kinda unsure about why we'd want it to be per-flavor | |
| 23:01:02 | dansmith | especially since it would affect your memory overcommit calculations | |
| 23:01:21 | dansmith | I would think you'd want to keep KSMing and non-KSMing instances separated by aggregate anyway | |
| 23:01:57 | dansmith | so I mean, that all leads me to think that it's probably worth a spec just on that basis | |
| 23:02:12 | dansmith | the other being that we want to make sure that we're not just plumbing a libvirt feature straight out the API | |
| 23:02:31 | melwitt | ack, thanks dansmith | |
| 23:02:35 | dansmith | and we should get input from the other virt drivers on whether they have similar technologies and make sure we can drive them with the same tunable | |
| 23:02:48 | dansmith | like, whether or not they allow it per-VM or only per-host | |
| 23:03:00 | dansmith | and whether it's just on or off, ro, rw, off, etc | |
| 23:04:16 | dansmith | I'd surely not want to add this just because someone thinks windows is destabilized by KSM, or because they have some weird unstable driver and turning KSM off changes some race semantics or something | |
| 23:05:45 | cfriesen | I could see KSM causing brief interruptions to the guest, but I wouldn't expect it to cause actual hangs. | |
| 23:06:54 | dansmith | right, | |
| 23:06:57 | dansmith | it can change timing semantics for sure | |
| 23:07:06 | dansmith | hence the buggy driver example | |
| 23:07:18 | dansmith | but so can swapping on the host to a much larger degree | |
| 23:08:14 | openstackgerrit | Dmitry Sutyagin proposed openstack/nova master: libvirt: add disabling KSM via flavor extra specs https://review.openstack.org/591854 | |
| 23:16:52 | openstackgerrit | Merged openstack/nova stable/pike: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/591762 | |
| 23:24:10 | dansmith | prometheanfire: ^ | |
| 23:26:34 | prometheanfire | :D | |
| #openstack-nova - 2018-08-16 | |||
| 00:44:43 | mriedem | 你好 | |
| 00:47:56 | mriedem | smcginnis: if you're around, we need stable cores for https://review.openstack.org/#/c/591872/ RC2 | |
| 00:48:46 | mriedem | anyone know if jichen is going to clean this up? https://review.openstack.org/#/c/533125/ | |
| 00:48:53 | mriedem | efried: do we need to just update it ourselves? | |
| 00:51:16 | openstackgerrit | Merged openstack/nova master: Fix service list for disabled compute using MC driver https://review.openstack.org/583936 | |
| 00:51:25 | openstackgerrit | Merged openstack/nova master: Revisons on notifications doc https://review.openstack.org/591621 | |
| 00:51:32 | openstackgerrit | Merged openstack/nova master: use static pages for mitaka and newton release notes https://review.openstack.org/592150 | |
| 00:52:56 | mriedem | tonyb: i guess you'd be a more appropriately time-zoned stable core for https://review.openstack.org/#/c/591872/ for RC2 | |
| 00:54:40 | mriedem | Kevin_Zheng: want to start backporting https://review.openstack.org/#/c/583936/ ? it needs to go to rocky, queens and pike | |
| 00:54:47 | mriedem | we'll hold it out until we've released rocky though | |
| 00:55:01 | tonyb | mriedem: on it | |
| 01:04:32 | Kevin_Zheng | sure | |
| 01:05:31 | tonyb | mriedem: I'll ping lyarwood | |
| 01:06:10 | mriedem | tonyb: thanks but he's probably done for the day | |
| 01:07:54 | tonyb | mriedem: I can single core approve it | |
| 01:08:14 | tonyb | mriedem: given you're a stable core etc | |
| 01:09:57 | dansmith | mriedem: tonyb I got it | |
| 01:10:29 | tonyb | dansmith to the rescue! | |
| 01:24:04 | mriedem | cool; i've emailed jichen about the zvm docs since those are the only things remaining on the RC2 todo list | |
| 01:41:54 | mriedem | efried: edmondsw: frankly i care very little if the powervm out of tree driver supports a feature, i don't plan on adding flags to nova to support out of tree features, re: https://review.openstack.org/#/c/591898/ | |
| 02:10:09 | openstackgerrit | fupingxie proposed openstack/nova master: Support list for alias in pci section in nova.conf https://review.openstack.org/592243 | |
| 02:48:25 | openstackgerrit | Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252 | |
| 03:32:20 | pvc | @edmondsw im usign ceph rbd | |
| 03:52:38 | pvc | hi | |
| 03:52:43 | pvc | anyone using ceph | |
| 05:16:08 | openstackgerrit | Merged openstack/nova stable/rocky: placement: ignore policy scope check failures if not enforcing scope https://review.openstack.org/591872 | |
| 05:20:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Explicitly fail if trying to attach SR-IOV port https://review.openstack.org/591898 | |
| 05:39:59 | openstackgerrit | Merged openstack/nova master: Imported Translations from Zanata https://review.openstack.org/591129 | |
| 06:07:46 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976 | |
| 06:37:40 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976 | |
| 06:40:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607 | |
| 06:48:28 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Make monkey patch work in uWSGI mode https://review.openstack.org/592285 | |
| 06:51:37 | openstackgerrit | Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252 | |
| 07:01:40 | openstackgerrit | huanhongda proposed openstack/nova-specs master: Return "deleted" and "estimated_reclaim" time of instance for admin https://review.openstack.org/581638 | |
| 07:03:58 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Support deleting data volume when destroy instance https://review.openstack.org/580336 | |
| 07:04:14 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Add support specify volume type when boot instance https://review.openstack.org/579520 | |
| 08:01:14 | openstackgerrit | Zhenyu Zheng proposed openstack/nova stable/rocky: Fix service list for disabled compute using MC driver https://review.openstack.org/592310 | |
| 08:01:31 | openstackgerrit | Zhenyu Zheng proposed openstack/nova stable/queens: Fix service list for disabled compute using MC driver https://review.openstack.org/592311 | |
| 08:05:19 | mriedem | tssurya: o | |
| 08:05:22 | mriedem | o/ | |
| 08:05:37 | mriedem | i've added a couple of things to the cells meeting agenda if you want to take a peek and start lending brain power | |
| 08:05:38 | tssurya | mriedem: \o/ | |
| 08:05:45 | mriedem | cross cell migration, | |
| 08:05:47 | mriedem | and down cell | |
| 08:06:06 | mriedem | our operators want a config option to not return any VMs from nova list if the tenant has any in a down cell | |
| 08:06:21 | tssurya | mriedem: oh okay | |
| 08:06:24 | mriedem | to avoid confusing users that thought they had like 3 VMs but now only list 2 | |
| 08:06:31 | mriedem | ^ would be pre-microversion | |
| 08:06:44 | mriedem | b/c with the microversion we can list the down cell VMs with the UNKNOWN status | |
| 08:07:06 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Make monkey patch work in uWSGI mode https://review.openstack.org/592285 | |
| 08:07:06 | tssurya | right so we need to change this now nad backport also I guess to Queens ? | |
| 08:07:28 | tssurya | before we start merging the UNKNOWN stuff. | |
| 08:07:35 | mriedem | tssurya: not sure about backports, but maybe if it defaults to current behavior | |
| 08:08:33 | tssurya | mriedem: the previous behaviour was 500 for down cells, do you suggest the config option to return an empty list if the user has stuff in the down cell then ? | |
| 08:08:59 | tssurya | instead of skipping | |
| 08:09:02 | tssurya | that cell | |
| 08:09:09 | mriedem | originally the behavior was 500 right? | |
| 08:09:12 | tssurya | yes | |
| 08:09:13 | mriedem | you fixed that to skip down cells | |
| 08:09:16 | tssurya | yes | |
| 08:09:26 | mriedem | so yes, they want an option to say don't skip, but just return an empty list | |
| 08:09:32 | mriedem | well, | |
| 08:09:37 | mriedem | if the project has any instances in a down cell | |
| 08:09:55 | tssurya | okay, | |
| 08:10:01 | mriedem | i think it would only apply to non-admin instance list; an admin doing nova-list --all-tenants wouldn't apply here | |
| 08:10:09 | openstackgerrit | fupingxie proposed openstack/nova master: Support list for alias in pci section in nova.conf https://review.openstack.org/592243 | |
| 08:10:41 | tssurya | ack, | |
| 08:12:48 | tssurya | mriedem: in the agenda you say "show is OK", well show wouldn't work at all before the new microversion, would just be a 500 | |
| 08:13:01 | tssurya | that's okay right ? | |
| 08:13:04 | mriedem | oh yeah, right | |