Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-11-01
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
11:50:09 openstackgerrit Jose Castro Leon proposed openstack/nova master: Fixes bug on attaching volume scsi on an instance boot from iso https://review.openstack.org/615141
12:11:31 openstackgerrit Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363
12:13:41 openstackgerrit Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328
12:14:12 openstackgerrit huanhongda proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/611833
12:23:42 openstackgerrit Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329
12:32:21 openstackgerrit Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786
12:34:52 openstackgerrit Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787
12:39:41 johnthetubaguy jaypipes: melwitt: suddenly noticed we can do per-project instance count checks from the CellMapping table. I worked this out before and keep forgetting about it.
12:46:08 openstackgerrit John Garbutt proposed openstack/nova master: WIP: Unified limits ideas https://review.openstack.org/615180
12:49:55 openstackgerrit Mohammed Naser proposed openstack/nova master: Default embedded instance.flavor.is_public attribute https://review.openstack.org/615134
12:50:07 mnaser thanks for the input jroll ! %
12:50:15 jroll :)
12:56:06 jaypipes johnthetubaguy: yeah, that's what melwitt added the queue_for_delete column I think in the API DB's instance_mappings table, right?
13:06:21 openstackgerrit John Garbutt proposed openstack/nova master: WIP: Unified limits ideas https://review.openstack.org/615180
13:34:01 sri_ Hello folks, quick quastion Does Ceph monitor IPs are hardcoded in the database <block_device_mapping> ? did anybody tried replacing ceph monitors(IP) recently ?
13:39:22 johnthetubaguy leakypipes: yeah, mind like a sieve
13:40:04 mriedem sri_: you might be looking for https://review.openstack.org/#/c/579004/

Earlier   Later