Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-15
22:50:54 mriedem to not use the deprecated option
22:51:10 dansmith mriedem: every time we do that it's upgrade pain and it just seems like we have too much of that already
22:51:25 cfriesen dansmith: not necessarily, nested vmx for example is buggy in older systems but may be stable by the time rocky is released
22:51:26 melwitt mriedem: but wouldn't you also have to add "pcid" to your rocky config elsewhere then?
22:51:27 dansmith I get the precedent thing
22:51:42 mriedem melwitt: yes, you'd do it on the new option
22:51:44 dansmith cfriesen: that's a super weak reason :)
22:52:02 dansmith cfriesen: because (a) I don't think it will be and (b) people can already do crazy stuff with lots of options
22:52:24 dansmith cfriesen: you could work around this by just defining your own cpu models in the libvirt system xml, and you can do way crazier stuff to your systems that way
22:52:55 cfriesen dansmith: true enough. for what it's worth, adding pdpe1gb seems to be generally fine.
22:53:06 dansmith heh
22:53:07 cfriesen for the subset of intel hosts I've dealt with anyway
22:54:27 melwitt we can just slap "experimental" on it on the backport, use at your own risk
22:54:45 dansmith it's not experimental but sure :)
22:55:32 dansmith melwitt: I need to be done for the day anyway,
22:55:44 melwitt well, what I mean is if you use it and it breaks something in queens/pike/ocata then we're probably not going to fix things to accommodate the various flags they enable
22:55:47 dansmith so you make the call, my input is all above
22:55:59 dansmith melwitt: it won't be any less experimental later,
22:56:03 dansmith it just lets you choose cpu flags
22:56:15 dansmith it will be just as dangerous in five years if you use it wrong
22:56:42 melwitt okay
23:20:06 openstackgerrit Eric Fried proposed openstack/nova master: Stop assuming initial provider generation is 0 https://review.openstack.org/548975
23:21:04 mriedem yikun: Kevin_Zheng_: replied with my opinion on the limited affinity spec options for how to model the limit in the API https://review.openstack.org/#/c/546925/2/specs/rocky/approved/allow-specifying-limit-for-affrinity-group.rst@49
23:21:45 efried jaypipes, alex_xu_: Trivial pep/tox fixes, ready for re+A --^
23:48:06 alex_xu_ efried: done
#openstack-nova - 2018-03-16
00:43:51 artom_ efried, if you're still around, I didn't want to pollute the mailing list with that's essentially just a "+1", but that update_provider_tree design update that you sent was really nice
00:44:17 artom_ For what it's worth, my opinion is that the precedent that this sets should be valued and repeated
00:47:50 Spaz-Home Morning folks
01:24:34 yikun mriedem, OK, thanks. : )
02:06:35 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734
02:06:35 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805
02:25:38 mriedem 享受工作，朋友。
02:25:39 mriedem o/
02:28:29 Spazmotic Annnnnd morning from the Office.
02:30:49 jianghuaw_ Good morning.
02:31:47 jianghuaw_ Spazmotic, I saw your message on live-migration.
02:32:38 jianghuaw_ that's expected behavior to get the exception when block_migration is False but it's not shared storage.
02:32:59 jianghuaw_ Look this: https://developer.openstack.org/api-ref/compute/
02:33:07 Spazmotic Will do, will take a look soon
02:33:09 jianghuaw_ Set to False means the request will fail when the source and destination hosts are not on the shared storage.
02:33:44 jianghuaw_ Searching "Live-Migrate Server" there:-)
02:34:23 jianghuaw_ the above statement is included in the table of the detail for "Live-Migrate Server (os-migrateLive Action)"
02:37:20 Spazmotic I see... Interesting
02:37:47 Spazmotic I'll keep reading through it more completely so I can understand what it's doing differently then sir
02:37:53 Spazmotic thanks
02:38:34 jianghuaw_ np. you're always welcome:-)
02:52:27 Spazmotic Co workers had me doubting the migrate_data field again and how it gets passed around
02:52:31 Spazmotic Live migrate you too confusing for your own good
04:00:13 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/553288
06:00:05 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
06:15:58 openstackgerrit Merged openstack/nova master: placement: generation in provider aggregate APIs https://review.openstack.org/548249
06:19:49 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
06:54:48 eandersson Maybe a silly idea... but would it be possible to hook into metadata changes (at least user initiated)?
06:56:51 eandersson We use the metadata from vm create to perform specific actions using notifications (e.g. compute.instance.create.end), but would be nice to be able to retroactively do the same on existing VMs using metadata.
07:13:30 openstackgerrit Julia Kreger proposed openstack/nova master: WIP: Update Ironic UTs to enable REST migration https://review.openstack.org/553699
07:33:13 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
07:34:11 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/553211
07:42:50 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
07:45:15 Spazmotic I wish I understand the entirity of that patch XD
08:14:16 openstackgerrit Merged openstack/nova master: Scheduler multiple workers support https://review.openstack.org/159382
08:18:27 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791
08:19:33 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925
08:23:17 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/553288
09:06:34 openstackgerrit Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
09:06:34 openstackgerrit Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862
09:06:35 openstackgerrit Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435
09:15:55 openstackgerrit Chris Dent proposed openstack/nova master: DNM: Demo code for microversion parse extraction https://review.openstack.org/550265
09:23:36 openstackgerrit Chris Dent proposed openstack/nova master: Provide framework for setting placement error codes https://review.openstack.org/546177
09:26:21 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791
09:40:20 openstackgerrit Surya Seetharaman proposed openstack/nova stable/queens: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553496
09:40:51 openstackgerrit Lee Yarwood proposed openstack/nova master: docs: Disable smartquotes https://review.openstack.org/553730
09:41:07 lyarwood stephenfin: ^ FWIW Doug suggested this in https://bugs.launchpad.net/nova/+bug/1755783
09:41:09 openstack Launchpad bug 1755783 in OpenStack Compute (nova) "PCI configuration documentation renders with smart quotes" [Undecided,In progress] - Assigned to Lee Yarwood (lyarwood)
09:41:28 lyarwood stephenfin: I think that's correct, ./me wait for the docs to build
09:42:05 stephenfin lyarwood: That's kind of working around a oslo.config bug though
09:43:08 stephenfin That being said, we're stuck with older versions of oslo.config for older versions of nova, so that could be the only backportable solution
09:43:50 stephenfin I've got no machine to work on NUMA-aware vSwitches, so I can work on that oslo.config fix today
09:44:13 lyarwood stephenfin: indeed, so the bug is that even with back-ticks we still end up with smartquotes right?
09:48:48 openstackgerrit Surya Seetharaman proposed openstack/nova stable/queens: Update the nova-manage db archive_deleted_rows description https://review.openstack.org/553733
09:49:18 openstackgerrit jichenjc proposed openstack/nova master: Move test_allocation_candidates.py https://review.openstack.org/553149
09:49:19 openstackgerrit jichenjc proposed openstack/nova master: Move test_resource_class_cache.py https://review.openstack.org/553153
09:49:19 openstackgerrit jichenjc proposed openstack/nova master: Move test_resource_provider.py https://review.openstack.org/553150
09:50:43 stephenfin lyarwood: Yeah, we're not parsing the config option help text properly so only some of the formatting is being picked up
09:51:42 stephenfin Biggest issue I see to resolving this is all the broken config option descriptions that likely exist at this point. Guess I'll have to try handle any warnings that might be raised
10:01:01 lyarwood stephenfin: urgh, so that doesn't work, still seeing smartquotes
10:10:06 openstackgerrit Surya Seetharaman proposed openstack/nova stable/queens: Make archive_deleted_rows handle a missing CONF.api_database.connection https://review.openstack.org/553742
10:12:24 stephenfin lyarwood: You need to do that in 'docutils.conf'
10:12:31 stephenfin lyarwood: See https://github.com/rtfd/readthedocs.org/issues/2940
10:17:55 lyarwood stephenfin: huh
10:18:18 lyarwood stephenfin: FWIW I was using http://www.sphinx-doc.org/en/stable/config.html#confval-smartquotes
10:18:54 stephenfin lyarwood: "New in version 1.6.6"
10:19:34 stephenfin https://github.com/openstack/requirements/blob/master/global-requirements.txt#L421
10:19:52 lyarwood *facepalm*
10:19:54 stephenfin I wasn't aware of that option but it seems we can't use it anyway
10:19:58 stephenfin :/
10:20:14 lyarwood I might just leave it in the change anyway for when we do get above 1.6.6
10:20:57 stephenfin lyarwood: That...could be a while. Some projects are seeing issues with it and I can't figure out why my "fix" won't pass CI https://review.openstack.org/#/c/552100/
10:21:33 stephenfin lyarwood: If you're backporting, maybe use the older 'html' prefixed option, assuming we're using <1.6 on those branches?
10:23:00 lyarwood stephenfin: yup good point
10:24:39 lyarwood stephenfin: `while setting up extension ['?', 'sphinx.ext.todo']: node class 'todolist' is already registered, its visitors will be overridden`

Earlier   Later