Earlier  
Posted Nick Remark
#openstack-nova - 2017-06-15
19:20:32 melwitt though I will say the test isn't doing what it thinks it's doing. it's supposed to be booting two instances on the same host and then expecting the third one to fail. but it's really booting one server on each host and then failing the third
19:23:15 openstackgerrit Erik Olof Gunnar Andersson proposed openstack/nova master: Re-worked ram filter and weights to support overcommit https://review.openstack.org/390984
19:24:23 eandersson Will need additional review on that component, because I thought I understood how that part worked. ^
19:26:08 eandersson Actually the above is very confusing in general, because it looks like free_mb can go negative, and the assumption was that it shouldn't be able to go negative
19:28:01 melwitt eandersson: I noticed the old code comment said it can go negative, but it didn't explain why. and I can't remember why it can or was supposed to be able to
19:28:21 eandersson The reason overcommit broke was because they changed it so that it couldn't go negative
19:29:25 melwitt I see
19:29:48 eandersson but maybe in some circumstances it's still possible
19:30:47 eandersson https://github.com/openstack/nova/commit/016b810f675b20e8ce78f4c82dc9c679c0162b7a
19:46:49 openstackgerrit Merged openstack/nova master: Use CONF.host for powervm nodename https://review.openstack.org/473975
20:07:45 rickd Hi, Anyone aware of a bug within libvirtd v2.5.0 and nested virtulization. Having issues running Ocata on a VMware guest : (
20:08:41 rickd Booting from hardisk ... GRUB...
20:09:52 clarkb rickd: openstack infra's experience is that it is quite buggy. We disable it by default as a result
20:09:55 clarkb it being nested virt
20:11:17 rickd clarkb I didnt release the infras had issues as well (?)
20:11:26 rickd Thought it would just be the computes...
20:11:43 clarkb rickd: sorry the infra team, eg the group that runs all the CI testing systems
20:11:49 rickd When you say you disabled it. Do you mean within Libvirt
20:11:58 clarkb rickd: we test with nova and qemu not nova and kvm
20:12:39 rickd ie you test nova and qemu for nested?
20:12:50 clarkb also when I dug into it I found the linux kernel does not enable nested virt by default on Intel CPUs because it is considered too buggy. It is enabled by default with amd though
20:13:16 clarkb rickd: yes, beacuse the testing itself runs on virtual machines we have the nested nova compute use qemu not kvm otherwise things fail in weird ways
20:14:04 rickd ok but for prod, its Nova + KVM?
20:14:25 clarkb yes the cloud we run tests on use kvm. I'm just saying we don't do nested virtualization
20:14:33 clarkb (cloud we run runs on baremetal so no nesting)
20:15:07 rickd ah ok, just trying to understand if there is a way around this.
20:15:31 clarkb rickd: according to the linux kernel use amd cpus... (I haven't been able to test reliability on amd though)
20:15:31 rickd But it looks like libvirt has pulled support for nested anyway
20:16:05 clarkb I don't think its a libvirt issue, it is a kvm thing
20:17:04 rickd from what I can see its was buggy before libvirt 2.5.0. Then nested support was fully pulled.
20:17:11 rickd I could be wrong though..
20:18:03 clarkb where do you see that? my understanding is its purely a kvm option and libvirt will just do whatever with kvm being configured the way it is
20:19:26 rickd ah yer see your point - https://bugs.launchpad.net/devstack/+bug/1689370
20:19:26 openstack Launchpad bug 1689370 in devstack "using ubuntu cloud archive gets a version of libvirtd which doesn't do nested virtualization" [Undecided,Fix released] - Assigned to Amrith Kumar (amrith)
20:22:30 rickd I wonder if I will be ok with a physical compute and virtual controller nodes. Think I would be fine.
20:41:41 eandersson melwitt, so yea looks like it was an issue with the code
20:41:51 eandersson odd though that the tests sometimes passes even when the scheduler fails completelt
20:46:42 mriedem melwitt: small things within https://review.openstack.org/#/c/446245/21
20:56:53 mriedem jaypipes: i've got an hour to make a difference in your life
20:57:01 mriedem jaypipes: what placement change do you want me to rubber stamp?
20:59:21 openstackgerrit Merged openstack/nova master: Add PowerVM to nova support matrix https://review.openstack.org/470999
21:05:28 jaypipes mriedem: I don't want you to rubberstamp anything :)
21:07:42 mriedem just tell me what to approve
21:08:03 mriedem i need to redeem this lost afternoon of mailing list reading
21:08:45 clarkb mriedem: are you saying the thing you are looking at for me doesn't have redeeming qualities? :)
21:11:23 melwitt eandersson: yeah, it seems like something is weird there
21:11:40 melwitt mriedem: cool, thanks
21:12:26 mriedem clarkb: heh, your thing just got more complicated
21:38:32 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/474672
#openstack-nova - 2017-06-16
00:08:01 mikal .
00:09:06 mriedem .~O~.
00:09:26 mikal You need an ascii hat and then you can be the over the fence guy from home improvement
00:09:49 mriedem they locked that guy up for being too nosey
00:09:54 mikal Heh
00:12:49 openstackgerrit Erik Olof Gunnar Andersson proposed openstack/nova master: Re-worked ram filter and weights to support overcommit https://review.openstack.org/390984
00:23:21 mriedem1 alex_xu: the current pagination_parameters parameter type for limit and marker uses multi_params but i'm assuming that's for backward compat? for a new microversion would we want to use single_param for validation?
00:23:29 mriedem1 so you can't send like marker=A&marker=B
00:53:29 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Fix title for cinder v3 API spec https://review.openstack.org/474819
01:01:20 openstackgerrit Merged openstack/nova-specs master: Fix title for cinder v3 API spec https://review.openstack.org/474819
01:22:56 alex_xu mriedem: yes, that is a vision. I'm thinking maybe worth a email to get wide agreement on that kind of strict validation?
01:25:38 alex_xu mriedem: I pointed the same thing out at https://review.openstack.org/#/c/459483/13/nova/api/openstack/compute/schemas/server_migrations.py also
01:34:09 mriedem yeah i basically have the same thing in my os-hypervisors 2.49 change now
01:36:32 openstackgerrit melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246
01:36:32 openstackgerrit melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247
01:36:33 openstackgerrit melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244
01:36:33 openstackgerrit melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245
01:36:35 openstackgerrit melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
01:36:37 openstackgerrit melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
01:36:38 openstackgerrit melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521
01:36:40 openstackgerrit melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860
01:36:46 openstackgerrit melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
01:36:49 openstackgerrit melanie witt proposed openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861
01:36:52 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
01:36:54 openstackgerrit melanie witt proposed openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862
01:44:31 openstackgerrit Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023
01:45:08 alex_xu mriedem: just send mail to double check that agreement http://lists.openstack.org/pipermail/openstack-dev/2017-June/118464.html
01:48:47 mriedem thanks
02:35:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532
02:35:04 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API https://review.openstack.org/471899
02:35:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Use uuid for id in os-services API https://review.openstack.org/464280
03:28:23 ratailor jaypipes, you around ?
04:32:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add X-OpenStack-Request-Id description https://review.openstack.org/474847
04:35:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add X-OpenStack-Request-Id description https://review.openstack.org/474847
06:19:25 felix001 anyone know what broke between Kilo and Ocata that means you cant use KVM for nested virt? I see the libvir versions are quite a bit diff
06:27:24 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform keypair.import notification https://review.openstack.org/467514
06:47:14 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-tenant-networks instead of stevedore https://review.openstack.org/474475
06:47:14 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore https://review.openstack.org/474463
06:47:15 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-cloudpipe endpoint instead of stevedore https://review.openstack.org/474116
06:47:16 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore https://review.openstack.org/461641
07:03:04 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-cells endpoint instead of stevedore https://review.openstack.org/474869
07:25:40 openstackgerrit Chris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details https://review.openstack.org/435141
07:28:47 moshele sfinucan: hi, kindly reminder to remove your -2 from https://review.openstack.org/#/c/460278/
07:31:43 openstackgerrit Chris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details https://review.openstack.org/435141
07:43:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Verify parameters in os-migrations.inc https://review.openstack.org/474879
07:59:55 openstackgerrit sahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller https://review.openstack.org/459741
07:59:56 openstackgerrit sahid proposed openstack/nova master: libvirt: update logic to configure device for scsi controller https://review.openstack.org/459740
08:05:59 openstackgerrit sahid proposed openstack/nova master: libvirt: refresh info_cache before creating devices metadata https://review.openstack.org/468758
08:06:36 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
08:21:49 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects https://review.openstack.org/474892
08:30:58 bauzas p!spy

Earlier   Later