Earlier  
Posted Nick Remark
#openstack-nova - 2017-03-22
16:19:03 sdague it's all testable on the functional side
16:19:07 mriedem yes i know
16:19:25 mriedem can't you just let me worry
16:22:32 openstackgerrit Sujitha proposed openstack/nova master: Add description to policies in certificates.py https://review.openstack.org/448267
16:24:01 mriedem replied in https://review.openstack.org/#/c/447149/1/specs/pike/approved/service-hyper-uuid-in-api.rst@148 and left examples of what the single PUT would look like
16:24:11 mriedem i can detail how a single PUT replaces the existing 4 PUT actions
16:25:50 openstackgerrit Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677
16:30:11 cfriesen maybe an odd question....when we download a qcow2-formatted image we convert it to raw before storing it under "_base". Is there a way to leave it as-is? I'm pretty sure qemu understands using qcow2 images as backing files.
16:30:19 cfriesen Or does that cause a performance hit at runtime?
16:53:24 johnthetubaguy cfriesen: I think there is a spec about that, looking at similar things with ploop
16:56:30 openstackgerrit Sujitha proposed openstack/nova master: Add description to policies in deferred_delete.py https://review.openstack.org/448681
16:56:48 openstackgerrit Sujitha proposed openstack/nova master: Add description to policies in create_backup.py https://review.openstack.org/448682
16:57:00 openstackgerrit Sujitha proposed openstack/nova master: Add description to policies in consoles.py https://review.openstack.org/448683
16:57:19 openstackgerrit Sujitha proposed openstack/nova master: Add description to policies in console_output.py https://review.openstack.org/448684
17:06:56 openstackgerrit Thomas Stewart proposed openstack/nova master: Add lan9118 as valid nic for hw_vif_model property for qemu https://review.openstack.org/393489
17:37:42 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Use uuids in services and os-hypervisors APIs (spec) https://review.openstack.org/447149
17:37:54 mriedem johnthetubaguy: sdague: cdent: dansmith: alex_xu: ^ has the updates we talked about earlire
17:37:56 mriedem *earlier
17:39:19 cdent ✔
18:07:07 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Reserve NUMA nodes with PCI devices attached https://review.openstack.org/364468
18:08:53 mriedem johnthetubaguy: you can drop the -2 on https://review.openstack.org/#/c/379524/ now, the bp is approved
18:21:38 mriedem dansmith: i've got a pickle, i'm removing the aggregate.uuid migrate on load in the object https://review.openstack.org/#/c/448579/
18:21:49 mriedem that's failing because there is a test for the online data migration which uses the object,
18:22:04 mriedem and because i've removed that code, the online data migration blows up since it can't use the object anymore
18:22:16 dansmith ...right
18:22:29 dansmith we can remove the migration now
18:22:31 mriedem i can't remove the online data migration thing from nova-manage can i?
18:22:33 mriedem oh
18:22:45 mriedem so i guess if you jump from what, mitaka to pike, well, sorry
18:22:46 mriedem derp
18:22:56 dansmith ...for this and many other reasons
18:23:18 mriedem ok i retract the pickle
18:23:32 dansmith you .. retract... the pickle ....
18:31:40 johnthetubaguy mriedem: ack
18:33:07 johnthetubaguy mdbooth: I should catch up with your -1 on this patch, I don't understand it: https://review.openstack.org/#/c/437597/11
20:29:18 oomichi mriedem: about http://lists.openstack.org/pipermail/openstack-dev/2017-March/114487.html
20:30:08 oomichi mriedem: I am not sure the difference happens between os-hosts and os-services if a single ironic *compute* service manages multiple nodes
20:30:46 oomichi mriedem: I don't have such envs TBH, but it is just a concern
20:31:02 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove aggregate uuid generation on load from DB https://review.openstack.org/448579
20:32:02 oomichi Can't we see multipe ironic nodes on os-hosts and single ironic compute service on os-services?
20:33:10 mriedem oomichi: looking at the index() methods in both os-services and os-hosts, they both use HostAPI.service_get_all
20:33:16 mriedem and both filter out API services from the response
20:33:39 openstackgerrit Sujitha proposed openstack/nova master: [WIP] Add BDM to InstancePayload https://review.openstack.org/448779
20:34:10 mriedem oomichi: the nodes are actually shown in the os-hypervisors API
20:35:52 oomichi mriedem: oh, nice investigation. that should be the same.
20:35:58 mriedem as far as i can tell on listing,
20:36:05 mriedem the only difference is that in os-hosts you can filter by zone
20:36:15 mriedem but in os-services you can only filter by host or binary
20:36:48 mriedem so if we deprecate os-hosts, we need to allow filtering services by zone in os-services too
20:37:27 oomichi that is a nice direction. +1 on deprecation and adding zone filter to os-services
20:38:35 bauzas mriedem: FWIW, AZs are for services, not nodes...
20:38:50 bauzas ie. n-cpu
20:39:08 mriedem bauzas: i know
20:39:12 bauzas so for example, 2 nodes for the same n-cpu service are always on the same zone
20:39:47 bauzas okay, good to me then, agreed w/ your direction then
20:41:01 snecklifter mriedem: https://review.openstack.org/#/c/393489/ <-better?
20:42:39 mriedem snecklifter: yes, +2, thanks
20:42:59 snecklifter mriedem: great, thanks for reviewing, much appreciated
20:46:18 mriedem dansmith: cells meeting in 14 minutes?
20:46:22 dansmith yeah
20:58:05 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Fix the bullet list in user-id-based-policy-enforcement https://review.openstack.org/448788
21:22:20 openstackgerrit Chris Dent proposed openstack/nova master: WIP: [placement] Idempotent PUT /resource_classes/{name} https://review.openstack.org/448791
21:24:21 openstackgerrit Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/357151
21:24:28 cdent mriedem: you want a real spec on ^^?
21:24:49 mriedem spec on or spec off?
21:29:39 cdent this is not wax
21:30:32 openstack Launchpad bug 1675143 in OpenStack Compute (nova) "placement microversion comparisons are sometimes floats" [High,Triaged] - Assigned to Chris Dent (cdent)
21:30:32 cdent mriedem: I also found an ossum bug: https://bugs.launchpad.net/nova/+bug/1675143
21:30:45 cdent that's ^ in the server
21:31:33 cdent will attend to it tomorrow or thereabouts
21:33:16 mriedem k
21:47:55 mriedem dansmith: bauzas: you guys are probably going to have to go through this https://review.openstack.org/#/c/433037/ - it's clear it's way over my head
21:47:59 mriedem sdague: you too ^
21:48:29 mriedem either i'm way overthinking it, or there is just some fundamental piece of the policy pie that I'm not grasping
21:48:37 sdague mriedem: starred, will look tomorrow
21:55:34 bauzas mriedem: /me looking to the spec
21:55:49 bauzas ah that
22:09:41 openstackgerrit Sujitha proposed openstack/nova master: [WIP] Add BDM to InstancePayload https://review.openstack.org/448779
22:28:07 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189
22:43:50 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402
23:50:38 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Specify keymap on server boot https://review.openstack.org/448830
23:53:41 openstackgerrit Dan Peschman proposed openstack/nova master: Option to always create network metadata information as static https://review.openstack.org/336293
#openstack-nova - 2017-03-23
00:06:27 sbezverk hello, do I need both cell0 and cell1 in ocata or just cell0 is sufficient?
00:55:26 openstackgerrit Bing Li proposed openstack/python-novaclient master: Remove log translations https://review.openstack.org/448336
01:01:48 lifeless efried: is this you rbug? https://github.com/testing-cabal/mock/issues/398
01:27:31 openstackgerrit Duan Jiong proposed openstack/nova master: Fix some reST field lists in docstrings https://review.openstack.org/448841
01:52:32 openstackgerrit Huan Xie proposed openstack/nova master: Create ovs port with other params together https://review.openstack.org/444230
02:25:32 openstackgerrit Danfly proposed openstack/nova master: remove mox from unit/virt/xenapi/image/test_bittorrent.py https://review.openstack.org/448855
02:26:00 openstackgerrit Alex Xu proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625
02:26:01 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200
02:26:01 openstackgerrit Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201
02:26:02 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
02:32:31 openstackgerrit Alex Xu proposed openstack/nova master: api-ref: Complete all the verifications of remote consoles https://review.openstack.org/338051
02:44:32 openstackgerrit Danfly proposed openstack/nova master: remove mox from tests/unit/compute/test_compute_utils.py https://review.openstack.org/448862
02:47:37 openstackgerrit Alex Xu proposed openstack/nova master: Add description to policies in admin_actions https://review.openstack.org/440829
03:13:52 Danfly who has time to review my patch? tks a lot. https://review.openstack.org/#/c/445504/
04:16:34 openstackgerrit Maciej Szankin proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936
04:22:21 openstackgerrit Maciej Szankin proposed openstack/nova-specs master: Add deprecate-nova-cert spec https://review.openstack.org/448882
05:09:56 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
05:12:05 openstackgerrit Maciej Szankin proposed openstack/nova-specs master: Add deprecate-nova-cert spec https://review.openstack.org/448882

Earlier   Later