Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-11-01
21:20:12 sean-k-mooney mriedem: yes it is
21:21:45 sean-k-mooney mriedem: when i was orignally try to standardise tpm trais i have multiple version traits https://review.openstack.org/#/c/514712/3/os_traits/hw/platform/security.py
21:22:02 sean-k-mooney but honelst 1.2 and 2.0 are all that matter
21:22:38 sean-k-mooney as far a i know very few deplopyment of tpm 1.0 or 1.1 were ever a thing
21:24:41 cfriesen on a totally different topic, I'd like to draw your attention to https://review.openstack.org/#/c/473973/
21:26:29 cfriesen originally we used these for the nova/neutron update where we were being blasted with a bunch of neutron updates. now with the changes to get fewer neutron updates it's probably not as big a deal, but we might want to consider using the fair locks in a few places.
21:27:38 sean-k-mooney cfriesen: so these are basically the opisite of pirority locks hehe
21:27:53 cfriesen they're like ticket spinlocks
21:29:09 sean-k-mooney cfriesen: just looking at the implementaiton
21:29:11 cfriesen the original problem we hit was that the nova-compute thread handling "real work" (like a migration or something) was being starved by tons of incoming neutron events that always got the lock first
21:29:55 cfriesen sean-k-mooney: for simplicity it uses the fact that fasteners writer locks are queued
21:30:13 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix NoneType error in _notify_volume_usage_detach https://review.openstack.org/614872
21:31:09 sean-k-mooney cfriesen: how does this interact with and without eventlets monkypatching
21:33:48 cfriesen should just work. the underlying stuff is threading.Condition
21:38:41 sean-k-mooney cfriesen: once comment if you re spin the patch but ya it neat
21:39:12 sean-k-mooney that said if you did not need a named lock you could jsut use the readerwriter lock directly
21:45:47 jackding mriedem: I forgot to push my change, Thank you for doing that.
22:06:06 sean-k-mooney cfriesen: so for real time guests do you care that we cant disable the perfomance moniting unit in the libvir xml in nova
22:10:08 cfriesen sean-k-mooney: I don't think it's come up. Do they default to on?
22:10:17 sean-k-mooney cfriesen: yep
22:10:41 sean-k-mooney i have no idea what the impact of that is
22:10:45 sean-k-mooney i assume low
22:11:12 sean-k-mooney but i have an internal email asking about turning realtime instance and that was the only itme that is not already supported upstream
22:11:51 sean-k-mooney i could write a patch to allow disableing it in like an hour just not sure its worth my time and or if people would accpet the patch if i did
22:12:40 cfriesen sean-k-mooney: I don't see a "perf" section if I do "virsh dumpxml"
22:13:05 cfriesen maybe we default it to off or something in libvirt
22:13:53 sean-k-mooney its in this section https://libvirt.org/formatdomain.html#elementsFeatures
22:14:06 sean-k-mooney and its defalted to on in libvirt
22:16:02 cfriesen "virsh domstats --perf <domain>" gives me nothing
22:16:56 sean-k-mooney cfriesen: virsh dumpxml | grep pmu ?
22:17:24 cfriesen nothing
22:17:43 sean-k-mooney what version of libvirt are you running
22:17:55 sean-k-mooney the docs could be wrong
22:18:00 cfriesen 3.5.0
22:18:18 sean-k-mooney and qemu
22:18:44 cfriesen qemu-kvm-ev-2.10.0
22:19:29 sean-k-mooney ok it said since 1.2.12 ill assume the docs are wrong until they show me a vm xml with this from an openstack instance
22:20:44 cfriesen I have a specific CPU model though, not host-passhtrough, if that matters
22:21:19 sean-k-mooney it may in this case it was using host-passtrogh
22:21:38 sean-k-mooney that said i pmu is not a cpu flag so it should not
22:23:49 sean-k-mooney actully maybe when the default is on it just does not include it in the xml
22:24:18 sean-k-mooney ill get them to verify its actully on before spending any more time on it. thanks cfriesen :)
22:24:55 cfriesen how do we handle long URLs in specs?
22:25:16 sean-k-mooney i belive flake8 ignore them
22:25:56 sean-k-mooney at least it appeared to in the onse i was writing so i just put them in the refernece section and use [0]_ to refer to them
22:26:39 sean-k-mooney i dont belive there is an openstack url shortenaer so just use google or something else if you need too
22:32:06 cfriesen hmm..just had a thought. is there a way to schedule based on libvirt version?
22:33:13 sean-k-mooney cfriesen: nope but you could have trait
22:34:00 cfriesen heh. actually, I think I'm okay. I have a trait for TPM 2.0, and that requires libvirt 4.5 which will also support CRB
22:35:01 sean-k-mooney ya i think realistically we dont want to expose software versions to schdule on things and use feautre instead
22:36:20 sean-k-mooney TPM 2.0 is defernt at that is refering to an iso standard and well they take a bit more time to have revisions and get implementd in hardware
22:38:10 jaypipes sean-k-mooney: you're now officially on the naughty list.
22:38:57 sean-k-mooney hehe i did say lets not use traits for this :) also was i ever not?
22:48:53 jaypipes :)
23:06:33 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Trust the report client cache more https://review.openstack.org/614886
23:07:04 efried mriedem, sean-k-mooney, jaypipes, cfriesen, belmoreira: ^^
23:07:25 efried I should link today's IRC discussion in there. But I gotta run riiight now.
#openstack-nova - 2018-11-02
00:16:12 openstackgerrit Vladyslav Drok proposed openstack/nova master: Do not remove baremetal ports in deallocate_for_instance https://review.openstack.org/473873
00:38:24 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
00:38:38 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
00:39:24 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
00:39:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629
00:40:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: Use oslo_db.sqlalchemy.test_fixtures https://review.openstack.org/609352
01:14:54 openstackgerrit gaobin proposed openstack/nova master: Improve the properties of the api https://review.openstack.org/614730
01:43:35 openstackgerrit Nguyen Hai Truong proposed openstack/nova master: [Trivial Fix] Correct spelling error of "should" https://review.openstack.org/614899
02:03:49 gmann artom: johnthetubaguy yeah, we keep tempest config as close as with latest defaults on service side. if that cause to fail on stable branch testing then we can modify them via devstack which is branched
02:32:06 openstackgerrit Nguyen Hai Truong proposed openstack/nova master: [Trivial Fix] Correct spelling error of "should" and "resource" https://review.openstack.org/614899
02:38:32 openstackgerrit Merged openstack/nova master: Modify get_by_cell_and_project() to get_not_deleted_by_cell_and_project() https://review.openstack.org/607663
02:38:41 openstackgerrit Merged openstack/nova master: Minimal construct plumbing for nova list when a cell is down https://review.openstack.org/567785
03:06:05 openstackgerrit Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
03:06:47 openstackgerrit huanhongda proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/611833
03:41:55 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP support attach/detach root volume 2 https://review.openstack.org/614750
05:55:31 openstackgerrit Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
06:06:40 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/614757
07:00:24 openstackgerrit Vieri proposed openstack/os-vif master: Update min tox version to 2.0 https://review.openstack.org/615046
07:08:33 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Bump compute service to indicate attach/detach root volume is supported https://review.openstack.org/614750
07:58:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for vol-backed snapshots https://review.openstack.org/615084
08:08:32 openstackgerrit Silvan Kaiser proposed openstack/nova master: [WIP] Added Qemu libquobyte Support to the Quobyte Driver https://review.openstack.org/546500
08:23:42 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support delete_on_termination in volume attach api https://review.openstack.org/612949
08:52:46 bauwser good Friday nova
09:26:21 openstackgerrit Jie Li proposed openstack/nova-specs master: Support volume-backed server rebuild https://review.openstack.org/532407
09:38:14 sri_ Hello, folks!, Does Ceph monitor IPs are still hardcoded in the database ? did anybody tried replacing ceph monitors(IP) recently ?
09:40:15 openstackgerrit Yikun Jiang proposed openstack/nova master: Use new ``initial_xxx_allocation_ratio`` CONF https://review.openstack.org/602804
09:40:15 openstackgerrit Yikun Jiang proposed openstack/nova master: Change the default values of XXX_allocation_ratio https://review.openstack.org/602803
09:40:16 openstackgerrit Yikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration script https://review.openstack.org/609995
09:40:16 openstackgerrit Yikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499
09:53:22 openstackgerrit Nguyen Hai Truong proposed openstack/nova master: Add tox python3 https://review.openstack.org/615118
10:09:36 openstackgerrit gaobin proposed openstack/nova master: Improve the properties of the api https://review.openstack.org/614730
10:20:44 openstackgerrit Jie Li proposed openstack/nova-specs master: Support volume-backed server rebuild https://review.openstack.org/532407
10:25:23 openstack Launchpad bug 1789423 in OpenStack Compute (nova) "Server operations fail to complete with versioned notifications if payload contains unset is_public field" [Medium,Triaged] - Assigned to Mohammed Naser (mnaser)
10:25:23 mnaser hey -- trying to fix https://bugs.launchpad.net/nova/+bug/1789423 locally, got it working but i need to get my pep8 to pass..
10:25:27 mnaser http://paste.openstack.org/show/733952/
10:25:36 mnaser any ideas how to ideally clean that up to fit 80 chars?
11:23:12 openstackgerrit Mohammed Naser proposed openstack/nova master: Default embedded instance.flavor.is_public attribute https://review.openstack.org/615134
11:24:59 mnaser ^ if someone can help with that briefly
11:42:43 jroll mnaser: I'd do Flavor(\n and indent the rest by whatever looks most readable
11:43:39 jroll mnaser: probably this if it fits, I didn't count: http://paste.openstack.org/show/734030/
11:44:18 jroll or maybe http://paste.openstack.org/show/734031/
11:44:50 jroll of course you could always assign those to a variable above where you define fake_flavor, too
11:45:59 openstackgerrit huanhongda proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/611833

Earlier   Later