| *** hogepodge has joined #openstack-nova | 00:00 | |
| *** sacharya has joined #openstack-nova | 00:04 | |
| *** hogepodge has quit IRC | 00:06 | |
| *** hogepodge has joined #openstack-nova | 00:06 | |
| mriedem | jaypipes: huzzah i've got it all working | 00:08 |
|---|---|---|
| mriedem | pushing up shortly | 00:08 |
| *** hogepodge has quit IRC | 00:08 | |
| *** hogepodge has joined #openstack-nova | 00:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Return uuid attribute for aggregates https://review.openstack.org/415031 | 00:09 |
| *** sacharya has quit IRC | 00:09 | |
| *** markvoelker has quit IRC | 00:11 | |
| *** hogepodge has quit IRC | 00:11 | |
| *** jose-phillips has quit IRC | 00:12 | |
| *** hogepodge has joined #openstack-nova | 00:12 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Fix formatting errors and warnings https://review.openstack.org/407364 | 00:13 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Abort Cold Migration https://review.openstack.org/334732 | 00:13 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: List/show all server migration types https://review.openstack.org/407237 | 00:13 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec https://review.openstack.org/397193 | 00:13 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876 | 00:13 |
| *** jcook has quit IRC | 00:22 | |
| *** jcook has joined #openstack-nova | 00:22 | |
| *** baoli has quit IRC | 00:29 | |
| *** baoli has joined #openstack-nova | 00:29 | |
| *** mlavalle has quit IRC | 00:35 | |
| *** aignatov has quit IRC | 00:43 | |
| *** tonyb has quit IRC | 00:43 | |
| *** aignatov has joined #openstack-nova | 00:43 | |
| *** aarefiev has quit IRC | 00:44 | |
| *** tonyb has joined #openstack-nova | 00:44 | |
| *** aarefiev has joined #openstack-nova | 00:44 | |
| oomichi | mriedem: jaypipes: can I get your reviews on https://review.openstack.org/#/c/413453 ? | 00:45 |
| oomichi | https://review.openstack.org/#/c/413453/ is a tempest patch for reproducing that, and I'd like to backport it to stable branches also | 00:46 |
| *** hoangcx has joined #openstack-nova | 00:47 | |
| *** stvnoyes has quit IRC | 00:50 | |
| *** stvnoyes has joined #openstack-nova | 00:51 | |
| *** baoli has quit IRC | 00:51 | |
| *** jamespage has quit IRC | 00:55 | |
| *** jamespage has joined #openstack-nova | 00:56 | |
| *** jamespage has joined #openstack-nova | 00:56 | |
| *** phuongnh has joined #openstack-nova | 01:00 | |
| *** ducttape_ has quit IRC | 01:01 | |
| *** amotoki has joined #openstack-nova | 01:02 | |
| *** sacharya has joined #openstack-nova | 01:05 | |
| *** baoli has joined #openstack-nova | 01:09 | |
| *** sacharya has quit IRC | 01:10 | |
| *** yamahata has quit IRC | 01:11 | |
| *** markvoelker has joined #openstack-nova | 01:11 | |
| *** markvoelker has quit IRC | 01:16 | |
| *** hongbin has quit IRC | 01:17 | |
| *** yamahata has joined #openstack-nova | 01:23 | |
| *** sacharya has joined #openstack-nova | 01:33 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Amend tag-instance-when-boot spec https://review.openstack.org/415315 | 01:38 |
| *** hongbin has joined #openstack-nova | 01:40 | |
| *** rfolco has joined #openstack-nova | 01:42 | |
| *** rfolco has quit IRC | 01:42 | |
| *** amotoki has quit IRC | 01:44 | |
| *** dave-mccowan has joined #openstack-nova | 01:48 | |
| *** baoli has quit IRC | 01:54 | |
| *** xinliang has joined #openstack-nova | 01:56 | |
| *** sam_nowitzki has joined #openstack-nova | 01:57 | |
| *** yufei has joined #openstack-nova | 01:58 | |
| *** sacharya has quit IRC | 02:04 | |
| *** amotoki has joined #openstack-nova | 02:08 | |
| *** ducttape_ has joined #openstack-nova | 02:08 | |
| *** zsli has joined #openstack-nova | 02:12 | |
| *** ducttape_ has quit IRC | 02:13 | |
| *** zsli_ has joined #openstack-nova | 02:18 | |
| *** zsli has quit IRC | 02:21 | |
| *** amotoki has quit IRC | 02:28 | |
| *** ababich has quit IRC | 02:29 | |
| *** ababich has joined #openstack-nova | 02:29 | |
| *** baoli has joined #openstack-nova | 02:29 | |
| *** mdnadeem has joined #openstack-nova | 02:32 | |
| *** baoli has quit IRC | 02:34 | |
| *** dimtruck is now known as zz_dimtruck | 02:36 | |
| *** gongysh has joined #openstack-nova | 02:36 | |
| *** dave-mccowan has quit IRC | 02:38 | |
| *** zz_dimtruck is now known as dimtruck | 02:47 | |
| *** amotoki has joined #openstack-nova | 02:53 | |
| *** diga has joined #openstack-nova | 02:57 | |
| *** shaohe_feng has quit IRC | 02:57 | |
| *** shaohe_feng has joined #openstack-nova | 02:58 | |
| *** hamzy has quit IRC | 02:58 | |
| *** shaohe_feng is now known as shaohe_fe | 02:59 | |
| *** Yingxin has quit IRC | 03:00 | |
| *** Yingxin has joined #openstack-nova | 03:00 | |
| mriedem | oomichi: i guess you mean this tempest patch https://review.openstack.org/#/c/415043/ | 03:02 |
| *** Jeffrey4l has joined #openstack-nova | 03:04 | |
| *** hongbin_ has joined #openstack-nova | 03:05 | |
| oomichi | mriedem: ah, yeah you are right. | 03:06 |
| mriedem | oomichi: is this also wrong then? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/server_groups.py#L30 | 03:07 |
| mriedem | since items is a list rather than a dict? | 03:07 |
| *** Jeffrey4l has quit IRC | 03:07 | |
| *** Jeffrey4l has joined #openstack-nova | 03:07 | |
| *** shaohe_fe has quit IRC | 03:07 | |
| *** hongbin has quit IRC | 03:07 | |
| oomichi | mriedem: humm, maybe we need to investigate it more deeply | 03:08 |
| *** amotoki has quit IRC | 03:08 | |
| *** Yingxin has quit IRC | 03:09 | |
| mriedem | ok i think https://spacetelescope.github.io/understanding-json-schema/reference/array.html helps me though | 03:10 |
| mriedem | 'items' is a dict if all items in the array are validated against the same schema | 03:10 |
| mriedem | like for bdms | 03:10 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 03:10 |
| mriedem | it's a list if the items in the array can have different schemas | 03:10 |
| mriedem | i'm not entirely sure why jsonschema only validates the first item in the bdm list | 03:12 |
| mriedem | when 'items' is a list and not a dict | 03:12 |
| mriedem | "To do this, we set the items keyword to an array, where each item is a schema that corresponds to each index of the document’s array. That is, an array where the first element validates the first element of the input array, the second element validates the second element of the input array, etc." | 03:12 |
| mriedem | so if we have the list form (the bug), then only the first entry in the list is validated, the rest aren't | 03:13 |
| *** yamahata has quit IRC | 03:14 | |
| *** amotoki has joined #openstack-nova | 03:15 | |
| mriedem | oomichi: Kevin_Zheng: alex_xu: ok +W on https://review.openstack.org/#/c/413453/ | 03:16 |
| mriedem | that's really tricky | 03:16 |
| alex_xu | mriedem: cool | 03:17 |
| alex_xu | oomichi: mriedem another one about schema https://review.openstack.org/415103 | 03:17 |
| oomichi | mriedem: yeah, I have the same feeling as you are saying at the above. The first dict in the list works only for the first input. | 03:17 |
| *** liusheng has quit IRC | 03:17 | |
| oomichi | I will add some test to confirm this JSON-Schema expectation later | 03:18 |
| oomichi | mriedem: alex_xu: thanks anyways | 03:18 |
| alex_xu | oomichi: np | 03:18 |
| *** liusheng has joined #openstack-nova | 03:18 | |
| *** bkopilov has quit IRC | 03:18 | |
| *** hongbin_ has quit IRC | 03:18 | |
| *** amotoki has quit IRC | 03:19 | |
| *** hfu has quit IRC | 03:22 | |
| *** ducttape_ has joined #openstack-nova | 03:23 | |
| *** hfu has joined #openstack-nova | 03:23 | |
| *** ducttape_ has quit IRC | 03:28 | |
| *** hongbin has joined #openstack-nova | 03:29 | |
| *** amotoki has joined #openstack-nova | 03:29 | |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add Traits API to placement service https://review.openstack.org/376200 | 03:31 |
| openstackgerrit | Alex Xu proposed openstack/nova: [PoC][placement] Integrate with os-traits library https://review.openstack.org/377382 | 03:31 |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199 | 03:31 |
| *** diga has quit IRC | 03:35 | |
| *** amotoki has quit IRC | 03:46 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP]Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 03:57 |
| *** nandal has joined #openstack-nova | 03:58 | |
| *** zsli_ has quit IRC | 04:02 | |
| *** sacharya has joined #openstack-nova | 04:04 | |
| *** mdnadeem has quit IRC | 04:05 | |
| *** links has joined #openstack-nova | 04:08 | |
| *** gouthamr has quit IRC | 04:09 | |
| *** sacharya has quit IRC | 04:09 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add unit tests for JSON-Schema' items behavior https://review.openstack.org/415333 | 04:10 |
| oomichi | mriedem: alex_xu: ^^^ unit tests for a little unclear json-schema behavior about the list of "items" | 04:11 |
| *** baoli has joined #openstack-nova | 04:11 | |
| oomichi | it would be easy to misunderstand the behavior.. | 04:14 |
| *** psachin has joined #openstack-nova | 04:15 | |
| *** baoli has quit IRC | 04:16 | |
| *** kfarr has quit IRC | 04:21 | |
| *** ducttape_ has joined #openstack-nova | 04:24 | |
| *** ducttape_ has quit IRC | 04:29 | |
| *** nandal has quit IRC | 04:30 | |
| *** yamahata has joined #openstack-nova | 04:31 | |
| *** baoli has joined #openstack-nova | 04:33 | |
| *** baoli has quit IRC | 04:33 | |
| *** bkopilov has joined #openstack-nova | 04:37 | |
| Kevin_Zheng | oomichi | 04:46 |
| Kevin_Zheng | ping oomichi | 04:46 |
| Kevin_Zheng | I was trying to fix the server_group schema | 04:46 |
| Kevin_Zheng | I realized that it might be wrong, could one server group have multiple policies? seems it should not be an array in schema | 04:47 |
| Kevin_Zheng | one group cannot be both affinity and anti-affinity I suppose | 04:49 |
| *** nandal has joined #openstack-nova | 04:49 | |
| *** ayogi has joined #openstack-nova | 04:49 | |
| *** pbandark has joined #openstack-nova | 04:53 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: server_group json-schema would not work correctly https://review.openstack.org/415340 | 04:56 |
| *** hongbin has quit IRC | 05:00 | |
| *** ratailor has joined #openstack-nova | 05:04 | |
| *** sacharya has joined #openstack-nova | 05:06 | |
| *** sacharya has quit IRC | 05:10 | |
| *** unicell has joined #openstack-nova | 05:10 | |
| *** unicell has quit IRC | 05:10 | |
| *** unicell has joined #openstack-nova | 05:10 | |
| *** hfu has quit IRC | 05:10 | |
| *** hfu has joined #openstack-nova | 05:13 | |
| *** baoli has joined #openstack-nova | 05:16 | |
| *** baoli_ has joined #openstack-nova | 05:18 | |
| *** baoli has quit IRC | 05:18 | |
| *** prateek has joined #openstack-nova | 05:22 | |
| *** baoli_ has quit IRC | 05:23 | |
| *** lamt has joined #openstack-nova | 05:24 | |
| *** hfu has quit IRC | 05:25 | |
| *** ducttape_ has joined #openstack-nova | 05:25 | |
| *** lamt has quit IRC | 05:29 | |
| *** ducttape_ has quit IRC | 05:30 | |
| *** adisky_ has joined #openstack-nova | 05:35 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port libvirt.test_firewall to Python 3 https://review.openstack.org/403423 | 05:38 |
| *** yuntongjin has joined #openstack-nova | 05:48 | |
| *** ekuris has joined #openstack-nova | 05:49 | |
| *** amotoki has joined #openstack-nova | 05:53 | |
| openstackgerrit | Swami Reddy proposed openstack/nova: Add volume status check when deleting instances https://review.openstack.org/260339 | 05:55 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Remove unused validation code from block_device https://review.openstack.org/410006 | 05:58 |
| *** hfu has joined #openstack-nova | 05:59 | |
| *** kristian__ has joined #openstack-nova | 06:00 | |
| *** kristian__ has quit IRC | 06:00 | |
| *** kristian__ has joined #openstack-nova | 06:00 | |
| *** hfu has quit IRC | 06:01 | |
| *** kristian__ has quit IRC | 06:04 | |
| *** moshele has joined #openstack-nova | 06:06 | |
| *** hfu has joined #openstack-nova | 06:08 | |
| *** sridharg has joined #openstack-nova | 06:12 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port libvirt.test_firewall to Python 3 https://review.openstack.org/403423 | 06:13 |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: fix memory usage miscalculation https://review.openstack.org/394857 | 06:17 |
| *** baoli has joined #openstack-nova | 06:19 | |
| *** baoli has quit IRC | 06:24 | |
| *** sahid has joined #openstack-nova | 06:28 | |
| *** baoli has joined #openstack-nova | 06:29 | |
| *** baoli has quit IRC | 06:34 | |
| *** zhs__ has joined #openstack-nova | 06:36 | |
| *** hfu has quit IRC | 06:39 | |
| *** zhs_ has quit IRC | 06:40 | |
| *** hfu has joined #openstack-nova | 06:40 | |
| *** hfu has quit IRC | 06:40 | |
| *** hfu has joined #openstack-nova | 06:41 | |
| *** david-lyle has joined #openstack-nova | 06:50 | |
| *** rcernin has joined #openstack-nova | 06:54 | |
| *** david-lyle has quit IRC | 06:55 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP]Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 06:56 |
| *** mjura has joined #openstack-nova | 06:57 | |
| *** tesseract has joined #openstack-nova | 07:01 | |
| *** diga has joined #openstack-nova | 07:01 | |
| *** mjura has quit IRC | 07:02 | |
| *** mjura has joined #openstack-nova | 07:03 | |
| *** cfriesen has quit IRC | 07:20 | |
| *** Satya_ has joined #openstack-nova | 07:24 | |
| *** ducttape_ has joined #openstack-nova | 07:27 | |
| *** takashin has left #openstack-nova | 07:28 | |
| *** baoli has joined #openstack-nova | 07:29 | |
| *** ducttape_ has quit IRC | 07:31 | |
| *** Satya_ is now known as satyar | 07:31 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-server-groups.inc https://review.openstack.org/415377 | 07:32 |
| *** Satya_ has joined #openstack-nova | 07:33 | |
| *** satyar has quit IRC | 07:33 | |
| *** satyar has joined #openstack-nova | 07:34 | |
| *** baoli has quit IRC | 07:34 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port virt.libvirt.test_imagebackend to Python 3 https://review.openstack.org/403431 | 07:44 |
| *** xinliang has quit IRC | 07:45 | |
| *** CristinaPauna has quit IRC | 07:47 | |
| *** diga has quit IRC | 07:48 | |
| *** xinliang has joined #openstack-nova | 07:48 | |
| *** aarefiev is now known as aarefiev_afk | 08:01 | |
| *** jed56 has joined #openstack-nova | 08:04 | |
| *** vtech has joined #openstack-nova | 08:06 | |
| *** gouthamr has joined #openstack-nova | 08:08 | |
| *** baoli has joined #openstack-nova | 08:15 | |
| *** baoli has quit IRC | 08:20 | |
| *** sapcc-bot has joined #openstack-nova | 08:26 | |
| *** carthaca_ has joined #openstack-nova | 08:27 | |
| *** dgonzalez_ has joined #openstack-nova | 08:27 | |
| *** ducttape_ has joined #openstack-nova | 08:27 | |
| *** dgonzalez_ has quit IRC | 08:28 | |
| *** carthaca_ has quit IRC | 08:28 | |
| *** ekuris has quit IRC | 08:29 | |
| *** frickler_ is now known as frickler | 08:29 | |
| *** ducttape_ has quit IRC | 08:32 | |
| *** kristian__ has joined #openstack-nova | 08:39 | |
| *** chohoor has joined #openstack-nova | 08:39 | |
| *** zsli has joined #openstack-nova | 08:42 | |
| *** amotoki has quit IRC | 08:43 | |
| *** ekuris has joined #openstack-nova | 08:44 | |
| *** gcb has quit IRC | 08:45 | |
| *** chohoor has left #openstack-nova | 08:48 | |
| *** david-lyle has joined #openstack-nova | 08:52 | |
| *** slaweq has joined #openstack-nova | 08:52 | |
| *** david-lyle has quit IRC | 08:56 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:00 | |
| *** oanson has quit IRC | 09:01 | |
| *** oanson has joined #openstack-nova | 09:02 | |
| *** slaweq has quit IRC | 09:03 | |
| *** shaohe_feng has joined #openstack-nova | 09:05 | |
| *** sacharya has joined #openstack-nova | 09:07 | |
| *** oanson has quit IRC | 09:11 | |
| *** sacharya has quit IRC | 09:11 | |
| *** oanson has joined #openstack-nova | 09:12 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 09:14 |
| *** mdnadeem has joined #openstack-nova | 09:17 | |
| *** markvoelker has joined #openstack-nova | 09:18 | |
| *** markvoelker has quit IRC | 09:22 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: DNM: just test https://review.openstack.org/415413 | 09:26 |
| *** gouthamr has quit IRC | 09:26 | |
| *** gouthamr has joined #openstack-nova | 09:27 | |
| *** gouthamr has quit IRC | 09:27 | |
| *** ducttape_ has joined #openstack-nova | 09:29 | |
| *** satyar has quit IRC | 09:30 | |
| *** satyar has joined #openstack-nova | 09:31 | |
| *** ducttape_ has quit IRC | 09:33 | |
| *** pcaruana has joined #openstack-nova | 09:42 | |
| openstackgerrit | sahid proposed openstack/nova: objects: add new field cpu_emulator_threads_policy https://review.openstack.org/385022 | 09:43 |
| openstackgerrit | sahid proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023 | 09:43 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 09:43 |
| openstackgerrit | sahid proposed openstack/nova: objects: add new field cpus_overhead in NUMACell https://review.openstack.org/400874 | 09:43 |
| openstackgerrit | sahid proposed openstack/nova: numa: fit instance toplogy with cpu overhead onto cores https://review.openstack.org/400875 | 09:43 |
| openstackgerrit | sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 09:43 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 09:43 |
| openstackgerrit | sahid proposed openstack/nova: numa: take into account cpus overhead https://review.openstack.org/400876 | 09:43 |
| openstackgerrit | sahid proposed openstack/nova: numa: update numa usage incording the cpu overhead https://review.openstack.org/400877 | 09:43 |
| *** yuntongjin has quit IRC | 09:47 | |
| *** david-lyle has joined #openstack-nova | 09:52 | |
| *** zsli has quit IRC | 09:54 | |
| *** hfu has quit IRC | 09:56 | |
| *** david-lyle has quit IRC | 09:57 | |
| *** gongysh has quit IRC | 10:01 | |
| *** zsli has joined #openstack-nova | 10:02 | |
| *** baoli_ has joined #openstack-nova | 10:04 | |
| *** matrohon has joined #openstack-nova | 10:05 | |
| *** amotoki has joined #openstack-nova | 10:05 | |
| *** pcaruana has quit IRC | 10:06 | |
| *** yufei has quit IRC | 10:06 | |
| *** amotoki has quit IRC | 10:07 | |
| *** yufei has joined #openstack-nova | 10:07 | |
| *** phuongnh has quit IRC | 10:07 | |
| *** pooja_j has quit IRC | 10:08 | |
| *** baoli_ has quit IRC | 10:09 | |
| *** slaweq has joined #openstack-nova | 10:09 | |
| *** zsli has quit IRC | 10:10 | |
| *** yufei has quit IRC | 10:12 | |
| *** zsli has joined #openstack-nova | 10:16 | |
| *** markvoelker has joined #openstack-nova | 10:19 | |
| *** vtech has quit IRC | 10:19 | |
| *** slaweq has quit IRC | 10:22 | |
| *** markvoelker has quit IRC | 10:23 | |
| *** vtech has joined #openstack-nova | 10:24 | |
| *** zsli has quit IRC | 10:24 | |
| *** amotoki has joined #openstack-nova | 10:28 | |
| *** satyar has quit IRC | 10:31 | |
| *** amotoki has quit IRC | 10:34 | |
| *** avolkov has joined #openstack-nova | 10:38 | |
| *** hoangcx has quit IRC | 10:43 | |
| *** matrohon has quit IRC | 10:46 | |
| *** raorn_ is now known as raorn | 10:53 | |
| *** gcb has joined #openstack-nova | 10:57 | |
| *** sacharya has joined #openstack-nova | 11:08 | |
| *** amotoki has joined #openstack-nova | 11:09 | |
| *** sacharya has quit IRC | 11:12 | |
| *** amotoki_ has joined #openstack-nova | 11:13 | |
| *** amotoki has quit IRC | 11:15 | |
| *** yamahata has quit IRC | 11:18 | |
| *** markvoelker has joined #openstack-nova | 11:19 | |
| *** amotoki_ has quit IRC | 11:20 | |
| *** xinliang has quit IRC | 11:20 | |
| *** kristian__ has quit IRC | 11:20 | |
| *** markvoelker has quit IRC | 11:24 | |
| *** kristian__ has joined #openstack-nova | 11:27 | |
| *** bkopilov has quit IRC | 11:29 | |
| *** xinliang has joined #openstack-nova | 11:32 | |
| *** longlq_ has joined #openstack-nova | 11:34 | |
| *** longlq_ has quit IRC | 11:35 | |
| *** baoli has joined #openstack-nova | 11:35 | |
| *** kristian__ has quit IRC | 11:36 | |
| *** sam_nowitzki has quit IRC | 11:36 | |
| *** baoli has quit IRC | 11:39 | |
| *** nandal has quit IRC | 11:41 | |
| *** bkopilov has joined #openstack-nova | 11:41 | |
| *** kristian__ has joined #openstack-nova | 11:43 | |
| *** prateek has quit IRC | 11:44 | |
| *** rfolco has joined #openstack-nova | 12:04 | |
| *** rfolco has quit IRC | 12:04 | |
| *** rfolco has joined #openstack-nova | 12:05 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 12:08 |
| *** markvoelker has joined #openstack-nova | 12:20 | |
| *** diga has joined #openstack-nova | 12:24 | |
| *** markvoelker has quit IRC | 12:25 | |
| *** neophy has joined #openstack-nova | 12:27 | |
| *** diga_ has joined #openstack-nova | 12:28 | |
| *** diga has quit IRC | 12:29 | |
| *** bkopilov has quit IRC | 12:34 | |
| *** portdirect___ is now known as m5v9 | 12:43 | |
| *** catinthe_ has joined #openstack-nova | 12:52 | |
| *** ratailor has quit IRC | 12:53 | |
| *** catintheroof has quit IRC | 12:55 | |
| *** david-lyle has joined #openstack-nova | 12:55 | |
| *** ducttape_ has joined #openstack-nova | 12:57 | |
| *** david-lyle has quit IRC | 12:59 | |
| *** avolkov has quit IRC | 13:03 | |
| *** sacharya has joined #openstack-nova | 13:08 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: update the active migrations ds to support migration result https://review.openstack.org/409260 | 13:11 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: pass to live migration monitor the API result https://review.openstack.org/409261 | 13:11 |
| *** mdnadeem has quit IRC | 13:12 | |
| *** sacharya has quit IRC | 13:13 | |
| *** markvoelker has joined #openstack-nova | 13:21 | |
| *** kristian__ has quit IRC | 13:21 | |
| *** ducttape_ has quit IRC | 13:22 | |
| *** markvoelker has quit IRC | 13:25 | |
| *** jeblair has quit IRC | 13:35 | |
| *** jeblair has joined #openstack-nova | 13:35 | |
| *** hfu has joined #openstack-nova | 13:36 | |
| *** liverpooler has quit IRC | 13:37 | |
| *** liverpooler has joined #openstack-nova | 13:38 | |
| *** gcb has quit IRC | 13:39 | |
| *** gcb has joined #openstack-nova | 13:39 | |
| *** hfu has quit IRC | 13:41 | |
| *** catintheroof has joined #openstack-nova | 13:44 | |
| *** kristian__ has joined #openstack-nova | 13:46 | |
| *** jed56 has quit IRC | 13:47 | |
| *** catinthe_ has quit IRC | 13:48 | |
| *** david-lyle has joined #openstack-nova | 13:56 | |
| *** brault_ has quit IRC | 13:57 | |
| *** david-lyle has quit IRC | 14:00 | |
| *** nklenke has quit IRC | 14:01 | |
| mriedem | Kevin_Zheng: i think the enum just means it can be affinity or anti-affinity | 14:03 |
| mriedem | Kevin_Zheng: i might be wrong based on the description here http://developer.openstack.org/api-ref/compute/?expanded=create-server-group-detail#id339 | 14:04 |
| catintheroof | Guys, on nova's block device mapping, can i choose cinder's volume type to boot from a volume ? what happens if i have 10 volume types and i want to choose a specific one ? | 14:04 |
| Kevin_Zheng | mriedm：yes I know, I mean should it be "array" in schemes? | 14:04 |
| Kevin_Zheng | Schema | 14:05 |
| Kevin_Zheng | Maybe | 14:05 |
| Kevin_Zheng | Maybe just "string" | 14:05 |
| mriedem | it can be an array, | 14:05 |
| mriedem | but i don't think 'items' should be in a [] | 14:05 |
| mriedem | because then jsonschema treats it like a tuple and validates the indexes | 14:05 |
| Kevin_Zheng | Yeah | 14:06 |
| Kevin_Zheng | I know that part | 14:06 |
| Kevin_Zheng | But why array | 14:06 |
| mriedem | Kevin_Zheng: so you could try something like creating a server group with policies=['affinity', 'invalid'] and see what happens | 14:06 |
| Kevin_Zheng | Could one sg have multiple policies? | 14:06 |
| mriedem | according to the schema and the docs yes | 14:07 |
| mriedem | what the use case is i'm not sure | 14:07 |
| mriedem | bauzas might know | 14:07 |
| Kevin_Zheng | Yeah, I'm just wondering | 14:07 |
| Kevin_Zheng | We also have a test, it tests one sg cannot have affinity and anti at the same time | 14:08 |
| Kevin_Zheng | I using the phone now, so very sorry that I cannot show you which one it is | 14:08 |
| Kevin_Zheng | But seems useless to have multiple policies since we only have affinity soft affinity. Anti and soft anti | 14:09 |
| *** ekuris has quit IRC | 14:10 | |
| Kevin_Zheng | Can't see any useful combination | 14:10 |
| mriedem | ildikov might also know the use cases for a server group with multiple policies | 14:10 |
| Kevin_Zheng | Yeah I will ask them | 14:11 |
| Kevin_Zheng | Maybe I can send out a mail list tomorrow | 14:12 |
| Kevin_Zheng | Because of the time zones | 14:12 |
| Kevin_Zheng | :) | 14:12 |
| *** mlavalle has joined #openstack-nova | 14:13 | |
| mriedem | sure np | 14:13 |
| *** mlavalle has quit IRC | 14:13 | |
| ildikov | Kevin_Zheng: I would assume it's more for future use cases it we will have policies that makes sense together | 14:14 |
| Kevin_Zheng | Ah ok | 14:14 |
| smcginnis | catintheroof: Not at instance creation time. You can by first creating the volume in Cinder, then using that to boot an instance. | 14:14 |
| ildikov | Kevin_Zheng: having affinity and anti-affinity for the same group would make it impossible to schedule any servers in that group | 14:14 |
| Kevin_Zheng | Yeah that's what I thought | 14:15 |
| mriedem | ildikov: if it's really a list for policies that can work together then that must have been written that way solely for out of tree scheduler filters, | 14:15 |
| mriedem | because none of the acceptable policies in tree would allow that | 14:16 |
| ildikov | mriedem: I would need to check the code to figure out why we might have that array as type there | 14:18 |
| mriedem | looks like this was added way back to validate that you don't specify both affinity and anti-affinity https://github.com/openstack/nova/commit/26bf8c0dd0c8b61381e35c278b30cc6739885373 | 14:18 |
| catintheroof | smcginnis: if i can give my feedback, is a real pain from a user point of view, basically because the user can't use the instance launch wizard, because it cant "create" a volume from there, since it cant "use" it | 14:18 |
| catintheroof | smcginnis: when there are lots of volume types | 14:19 |
| *** baoli has joined #openstack-nova | 14:19 | |
| *** kristian__ has quit IRC | 14:22 | |
| *** ducttape_ has joined #openstack-nova | 14:22 | |
| Kevin_Zheng | Yeah I removed the [] and that test failed | 14:23 |
| mriedem | here is the original design wiki for server groups https://wiki.openstack.org/wiki/InstanceGroupApiExtension | 14:24 |
| mriedem | sounds like there was some future proofing in mind there | 14:24 |
| ildikov | mriedem: yep and in that commit there's a test_create_server_group_not_supported test, which looks like an example that we could have more policies, but we don't atm | 14:24 |
| *** baoli has quit IRC | 14:24 | |
| mriedem | ildikov: yeah, so that started enforcing the restriction | 14:25 |
| *** ekuris has joined #openstack-nova | 14:25 | |
| mriedem | so this is probably all very old legacy behavior where the API wasn't strict about what you could do, so people took advantage with out of tree scheduler filters | 14:25 |
| *** ducttape_ has quit IRC | 14:27 | |
| *** lamt has joined #openstack-nova | 14:28 | |
| *** brault has joined #openstack-nova | 14:29 | |
| ildikov | in the sense of that SUPPORTED_POLICIES list it needs some hacking for any out of tree filters | 14:29 |
| mriedem | before that it would allow anything though | 14:30 |
| *** lamt has quit IRC | 14:30 | |
| mriedem | so my guess is the multiple policy thing came before that validation was put in place | 14:30 |
| *** palexster has quit IRC | 14:30 | |
| ildikov | I don't think it's an issue to have the code ready for non-mutually-exclusive policies | 14:30 |
| *** palexster has joined #openstack-nova | 14:31 | |
| ildikov | it seems to me too that it originates back before the time of enforcing anything and I guess it looked harmless enough not to remove | 14:32 |
| *** lamt has joined #openstack-nova | 14:32 | |
| ildikov | I wonder whether it would make sense to make it configurable if we think out of tree filters are using it | 14:33 |
| ildikov | or we can just stamp it as 'future proof thinking' for now | 14:33 |
| *** brault has quit IRC | 14:33 | |
| ildikov | when we added the soft-... policies we didn't change that part | 14:34 |
| mriedem | -1 on configurable | 14:35 |
| mriedem | we don't want config impacting the API behavior | 14:35 |
| *** satyar has joined #openstack-nova | 14:35 | |
| mriedem | i'm happy to just forget it exists this way, or bury my head in the sand | 14:35 |
| ildikov | ah right, it's API facing... | 14:36 |
| *** ekuris has quit IRC | 14:36 | |
| mriedem | what scares me is i don't see any validation that keeps you from creating a server group with both affinity and anti-affinity policies | 14:36 |
| ildikov | then we can pick the second option and bury our heads in the sand :) | 14:36 |
| *** palexster has quit IRC | 14:36 | |
| ildikov | doesn't the json schema validation ensure that? | 14:37 |
| ildikov | I think it does | 14:37 |
| ildikov | I remember writing tests for it :) | 14:37 |
| mriedem | ah you're right | 14:38 |
| mriedem | 'policies': { | 14:38 |
| mriedem | 'type': 'array', | 14:38 |
| mriedem | 'items': [{'enum': ['anti-affinity', 'affinity']}], | 14:38 |
| mriedem | 'uniqueItems': True, | 14:38 |
| mriedem | 'additionalItems': False, | 14:38 |
| mriedem | } | 14:38 |
| ildikov | I hope I don't remember only the future proof thinking regarding those tests :D | 14:38 |
| ildikov | yeap, that's the one! | 14:38 |
| ildikov | I played with the validation a bit, so it should work :) | 14:38 |
| *** ayogi has quit IRC | 14:39 | |
| mriedem | so i guess additionalItems=False is what enforces that | 14:40 |
| mriedem | yeah | 14:40 |
| mriedem | stack@filters:~$ nova server-group-create conflicting-policy-server-group affinity anti-affinity | 14:40 |
| mriedem | ERROR (BadRequest): Invalid input for field/attribute policies. Value: [u'affinity', u'anti-affinity']. Additional items are not allowed (u'anti-affinity' was unexpected) (HTTP 400) (Request-ID: req-3b2c8b4b-0080-4c26-83dd-a3c5db748beb) | 14:40 |
| mriedem | so the only bug we have is in our api reference docs | 14:41 |
| *** armax has joined #openstack-nova | 14:41 | |
| *** bkopilov has joined #openstack-nova | 14:41 | |
| *** sridharg has quit IRC | 14:42 | |
| ildikov | from format perspective it's correct, you just cannot specify multiple from the currently supported policies... | 14:43 |
| mriedem | yeah | 14:45 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1652943 | 14:45 |
| openstack | Launchpad bug 1652943 in OpenStack Compute (nova) "api-ref: os-server-groups POST description is wrong for policies parameter" [Medium,Triaged] | 14:45 |
| mriedem | easy morning bug fixin' | 14:46 |
| Kevin_Zheng | haha so the schema is still array then？ | 14:47 |
| ildikov | this year is too short now for any complex ones anyway :) | 14:47 |
| mriedem | Kevin_Zheng: yeah the array part is odd | 14:47 |
| ildikov | Kevin_Zheng: we call that future-proof thinking | 14:47 |
| mriedem | Kevin_Zheng: probably legacy, and we'd have to change it with a microversion to make it a string | 14:48 |
| ildikov | mriedem: I'm not sure that would worth the effort TBH... | 14:48 |
| mriedem | not right now no | 14:48 |
| Kevin_Zheng | ok | 14:48 |
| openstackgerrit | Jay Pipes proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023 | 14:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 14:49 |
| mriedem | ildikov: unless you're saying it would break the out of tree code you already have :) | 14:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova: numa: fit instance toplogy with cpu overhead onto cores https://review.openstack.org/400875 | 14:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 14:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 14:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova: numa: take into account cpus overhead https://review.openstack.org/400876 | 14:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova: numa: update numa usage incording the cpu overhead https://review.openstack.org/400877 | 14:49 |
| jaypipes | sahid: fixed the pep8 issue in the above stack (again) ^^ | 14:49 |
| ildikov | mriedem: I'm just lazy :) | 14:49 |
| *** mlavalle has joined #openstack-nova | 14:50 | |
| *** mlavalle has quit IRC | 14:50 | |
| *** mlavalle has joined #openstack-nova | 14:50 | |
| Kevin_Zheng | so, who fix apiref? | 14:51 |
| *** Jinxing has joined #openstack-nova | 14:52 | |
| Kevin_Zheng | And [] in schema should still be removed? | 14:52 |
| mriedem | Kevin_Zheng: the [] is fine | 14:53 |
| mriedem | since additionalItems is False | 14:53 |
| mriedem | Kevin_Zheng: i'm fixing the api-ref | 14:53 |
| mriedem | should have a patch up shortly | 14:53 |
| Kevin_Zheng | Ah ok | 14:53 |
| ildikov | Kevin_Zheng: the only thing that's missing from the docs at the moment that the currently supported items are all mutually exclusive so you can specify only one | 14:54 |
| sahid | hum.. thanks jaypipes | 14:54 |
| Kevin_Zheng | https://review.openstack.org/#/c/415340 so this one can be abandoned? | 14:56 |
| jaypipes | sahid: it was my fault. I had accidentally git add'ed some bogus files in doc/api_samples/ when pushing a revision to one of your patches yesterday. | 14:56 |
| jaypipes | sahid: https://twitter.com/jaypipes/status/813744689551732736 :) | 14:56 |
| *** armax has quit IRC | 14:57 | |
| ildikov | jaypipes: nice Xmas gift :) | 14:59 |
| jaypipes | ildikov: :) | 14:59 |
| alex_xu | mriedem: ildikov, maybe the confuse is the error message from json-schema, see the original error message at https://review.openstack.org/#/c/115532/1/nova/api/openstack/compute/plugins/v3/server_groups.py@80, if we have such error message I guess there won't have such long discussion today :) | 15:03 |
| ildikov | alex_xu: the long discussion part was about why it can be a list if we cannot specify more than one item anyway | 15:04 |
| mriedem | Kevin_Zheng: yes you can abandon https://review.openstack.org/#/c/415340/1 | 15:04 |
| mriedem | alex_xu: when i update the api-ref i'm also going to leave a comment in the schema itself | 15:05 |
| Kevin_Zheng | Thanks | 15:05 |
| mriedem | to avoid future confusion | 15:05 |
| alex_xu | mriedem: +1 | 15:05 |
| ildikov | mriedem: +1 | 15:05 |
| ildikov | jsonschema messages can be pretty confusing... | 15:05 |
| alex_xu | ildikov: yea :) | 15:06 |
| mriedem | this isn't too bad | 15:06 |
| mriedem | Invalid input for field/attribute policies. Value: [u'affinity', u'anti-affinity']. Additional items are not allowed (u'anti-affinity' was unexpected) | 15:06 |
| mriedem | at least now that i know what i'm doing :) | 15:06 |
| mriedem | it would be confusing with the current state of our api ref | 15:06 |
| ildikov | ok, if someone knows what those policies mean then it shouldn't be confusing after one or at most two minutes of thinking, but it's still better not to torture users/operators with it :) | 15:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: api-ref: cleanup os-server-groups 'policies' parameter description https://review.openstack.org/415482 | 15:09 |
| mriedem | ildikov: that's the other thing my api-ref patch does is adds descriptions to those policies ^ | 15:09 |
| *** sacharya has joined #openstack-nova | 15:09 | |
| *** sacharya has quit IRC | 15:14 | |
| *** baoli has joined #openstack-nova | 15:19 | |
| *** kristian__ has joined #openstack-nova | 15:22 | |
| *** markvoelker has joined #openstack-nova | 15:22 | |
| *** ducttape_ has joined #openstack-nova | 15:23 | |
| *** baoli has quit IRC | 15:24 | |
| *** kristian__ has quit IRC | 15:27 | |
| *** markvoelker has quit IRC | 15:27 | |
| *** ducttape_ has quit IRC | 15:28 | |
| *** kfarr has joined #openstack-nova | 15:29 | |
| *** pbandark has quit IRC | 15:30 | |
| *** hongbin has joined #openstack-nova | 15:38 | |
| *** yufei has joined #openstack-nova | 15:41 | |
| *** psachin has quit IRC | 15:42 | |
| *** nklenke has joined #openstack-nova | 15:43 | |
| *** mjura has quit IRC | 15:46 | |
| *** kristian__ has joined #openstack-nova | 15:52 | |
| sahid | jaypipes: ahaha, i see... :) (nice gif) | 15:54 |
| *** links has quit IRC | 15:57 | |
| *** david-lyle has joined #openstack-nova | 15:57 | |
| *** david-lyle has quit IRC | 16:01 | |
| *** nklenke has quit IRC | 16:02 | |
| mlavalle | mriedem: hi. so what is next with https://bugs.launchpad.net/nova/+bug/1652642? Is it going to be addressed in this cycle? | 16:05 |
| openstack | Launchpad bug 1652642 in OpenStack Compute (nova) "Nova API doesn't return aggregate's uuid, which is needed when using the placement API" [Critical,In progress] - Assigned to Matt Riedemann (mriedem) | 16:05 |
| mriedem | mlavalle: it's an api change so it needs a spec | 16:05 |
| mlavalle | mriedem: does that mean it is not going to be fixed this cycle? | 16:05 |
| mriedem | mlavalle: i think an exception could be made based on it's proximity to a priority effort | 16:06 |
| mlavalle | mriedem: when will that decision be made? | 16:06 |
| mriedem | mlavalle: well the first step either way is a spec | 16:06 |
| mriedem | which should be pretty simple | 16:06 |
| mriedem | return uuid from os-aggregates | 16:07 |
| mriedem | and explain the use case for why that's needed | 16:07 |
| mriedem | unless, | 16:07 |
| mriedem | we just bend the rules and say this is part of generic-resource-pools-ocata | 16:07 |
| mriedem | but i don't really think it is, or it's in addition to that | 16:08 |
| mriedem | jaypipes: just checked that aggregates are in the API DB now - i was losing some sleep over that last night thinking even if you had the aggregate uuid, we wouldn't know which cells v2 cell to look it up in, | 16:09 |
| mriedem | but alas that's not a problem since it's in the API DB | 16:09 |
| jaypipes | ya | 16:09 |
| *** baoli has joined #openstack-nova | 16:09 | |
| mlavalle | mriedem: yeah, it is in the API DB | 16:09 |
| mriedem | so do you guys want me to write the spec for this since i'm pushing on it? | 16:10 |
| mlavalle | mriedem: would you please? | 16:10 |
| mriedem | noting that it won't be merged regardless until next week when sean/dan/john are back | 16:10 |
| mlavalle | mriedem: I don't need it merged this week. I have enough to have finished my testing yesterday | 16:11 |
| mlavalle | mriedem: the litlle piece missing in my Neutron side code ia couple of calls to associate aggergates to a resource provider | 16:11 |
| mlavalle | and get the uuid of an aggregate associated to a resource provider | 16:12 |
| *** brault has joined #openstack-nova | 16:12 | |
| *** zhs__ has quit IRC | 16:13 | |
| *** zhs__ has joined #openstack-nova | 16:13 | |
| *** baoli has quit IRC | 16:14 | |
| *** nklenke has joined #openstack-nova | 16:15 | |
| mriedem | mlavalle: jaypipes: ok i'll work on the spec this morning, it should be pretty straight-forward | 16:16 |
| mlavalle | mriedem, jaypipes: you guys rock! Thanks for the quick response to this issue :-) | 16:16 |
| *** yufei has quit IRC | 16:16 | |
| jaypipes | mriedem: I'm still working on unit tests for the 0-record inventory bug/patch | 16:19 |
| jaypipes | mriedem: should have it done in 30 minutes. | 16:19 |
| mriedem | k | 16:20 |
| jaypipes | mriedem: it's this damn PTL who keeps requesting changes... :P | 16:21 |
| mriedem | psh | 16:21 |
| mriedem | yeah who needs docs | 16:21 |
| mriedem | oh wait | 16:21 |
| mriedem | i'm confusing things that i'm blocking :) | 16:21 |
| *** links has joined #openstack-nova | 16:22 | |
| jaypipes | mriedem: :) | 16:22 |
| *** markvoelker has joined #openstack-nova | 16:23 | |
| *** moshele has quit IRC | 16:24 | |
| *** ducttape_ has joined #openstack-nova | 16:24 | |
| *** ducttape_ has quit IRC | 16:25 | |
| *** ducttape_ has joined #openstack-nova | 16:25 | |
| *** links has quit IRC | 16:26 | |
| *** markvoelker has quit IRC | 16:28 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: WIP(2): Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | 16:28 |
| *** Jinxing has quit IRC | 16:31 | |
| *** lpetrut has joined #openstack-nova | 16:31 | |
| *** yamahata has joined #openstack-nova | 16:33 | |
| *** kristian__ has quit IRC | 16:33 | |
| *** dtp has joined #openstack-nova | 16:37 | |
| *** sahid has quit IRC | 16:40 | |
| *** lbeliveau has quit IRC | 16:42 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: Do not save 0-valued inventory https://review.openstack.org/414214 | 16:45 |
| jaypipes | vsaienk0, vdrok, mriedem, TheJulia: ^^ | 16:45 |
| mriedem | gibi: i suspect at some point we should include the aggregate uuid in versioned notifications | 16:50 |
| *** mdnadeem has joined #openstack-nova | 16:50 | |
| *** lpetrut has quit IRC | 16:52 | |
| *** kristian__ has joined #openstack-nova | 16:52 | |
| mriedem | well, before that i guess we need to emit versioned notifications for aggregates | 17:02 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Return uuid from os-aggregates API (spec) https://review.openstack.org/415511 | 17:04 |
| mriedem | mlavalle: jaypipes: ^ | 17:04 |
| mlavalle | mriedem: thanks. Taking a look | 17:06 |
| *** krtaylor has quit IRC | 17:08 | |
| *** sacharya has joined #openstack-nova | 17:10 | |
| *** unicell has quit IRC | 17:10 | |
| jaypipes | mriedem: cheers man, appreciate you doing that | 17:11 |
| *** sacharya has quit IRC | 17:15 | |
| *** jose-phillips has joined #openstack-nova | 17:16 | |
| mriedem | jaypipes: so on your ironic patch for 0 inventory, i suppose you are going with a single PUT for all 3 resource classes instead of 3 DELETE inventory calls just for efficiency? | 17:16 |
| jaypipes | mriedem: no, I'm doing that because there's no DELETE /r-p/{uuid}/inventories endpoint :) | 17:17 |
| jaypipes | mriedem: as I found out when implementing that :) | 17:17 |
| jaypipes | mriedem: and yes, I didn't want to do 3 calls.. | 17:17 |
| mriedem | jaypipes: then what's this? https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/inventory.py#L241 | 17:18 |
| jaypipes | mriedem: single resource class/record | 17:18 |
| *** gouthamr has joined #openstack-nova | 17:18 | |
| mriedem | oh delete ALL vs single | 17:18 |
| jaypipes | ya | 17:18 |
| mriedem | yeah i figured that's what you were going to do | 17:18 |
| *** brault has quit IRC | 17:18 | |
| jaypipes | mriedem: would be cleaner to have a DELETE all, but we can do that in future. | 17:18 |
| *** _fortis has quit IRC | 17:19 | |
| *** krtaylor has joined #openstack-nova | 17:21 | |
| *** diga_ has quit IRC | 17:23 | |
| *** markvoelker has joined #openstack-nova | 17:24 | |
| mriedem | jaypipes: ok comment inline | 17:28 |
| *** markvoelker has quit IRC | 17:29 | |
| *** jose-phillips has quit IRC | 17:29 | |
| *** ducttape_ has quit IRC | 17:31 | |
| *** moshele has joined #openstack-nova | 17:35 | |
| *** jose-phillips has joined #openstack-nova | 17:36 | |
| *** mdnadeem has quit IRC | 17:38 | |
| *** gouthamr has quit IRC | 17:39 | |
| sneti | gibi, ping | 17:41 |
| *** tesseract has quit IRC | 17:44 | |
| dtp | tdurakov are there any vm states or task states associated with live migration that are not in master? | 17:45 |
| mriedem | jaypipes: what is the "end" of the custom resource classes blueprint to mark it complete? this? https://review.openstack.org/#/c/404472/ | 17:48 |
| *** haplo37_ has quit IRC | 17:48 | |
| jaypipes | mriedem: yes. | 17:49 |
| *** kristian__ has quit IRC | 17:49 | |
| *** moshele has quit IRC | 17:49 | |
| mriedem | ok, i was a bit confused about https://review.openstack.org/#/c/405705/ | 17:50 |
| mriedem | and where it's falling into the series | 17:50 |
| jaypipes | the reason for your confusion is because I cherry-picked out the cleanup of the scheduler reporting client unit tests. | 17:51 |
| jaypipes | mriedem: ^ | 17:51 |
| jaypipes | mriedem: which used to be in between that and the 404472. | 17:52 |
| *** vtech has quit IRC | 17:52 | |
| mriedem | ok | 17:53 |
| openstackgerrit | Sujitha proposed openstack/nova: Adding tags field to InstancePayload https://review.openstack.org/407228 | 17:54 |
| *** hongbin has quit IRC | 17:54 | |
| jaypipes | mriedem: remember, that was the patch you'd asked me to cherry-pick out. | 17:56 |
| jaypipes | mriedem: it refactored the unit test classes so that a bunch of tests weren't re-executed over and over again unnecessarily. | 17:56 |
| mriedem | jaypipes: i remember it | 17:57 |
| jaypipes | coo. | 17:57 |
| mriedem | it wasn't necessary to pull out, but i see it's merged now so that's fine | 17:57 |
| *** baoli has joined #openstack-nova | 17:58 | |
| mriedem | what's up with this blueprint? https://blueprints.launchpad.net/nova/+spec/inventory-allocation-ocata | 17:58 |
| *** haplo37_ has joined #openstack-nova | 17:58 | |
| mriedem | i guess that's just all pci device inventory, was that folded into the nested resource providers work? | 17:58 |
| *** dtp_ has joined #openstack-nova | 18:02 | |
| mriedem | edleafe: ^? | 18:02 |
| *** baoli has quit IRC | 18:02 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Tagged attach compute manager and net API support https://review.openstack.org/391941 | 18:03 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Tagged attach API support https://review.openstack.org/376021 | 18:03 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged attach https://review.openstack.org/415520 | 18:03 |
| *** dtp__ has joined #openstack-nova | 18:04 | |
| *** dtp has quit IRC | 18:06 | |
| *** dtp__ has quit IRC | 18:07 | |
| *** dtp has joined #openstack-nova | 18:07 | |
| *** dtp_ has quit IRC | 18:08 | |
| *** baoli has joined #openstack-nova | 18:12 | |
| jaypipes | mriedem: yeah. we can obselete that blueprint. | 18:14 |
| jaypipes | mriedem: moshele and others in the PCI subteam said there was no need for tracking "generic" PCI devices. | 18:15 |
| *** sacharya has joined #openstack-nova | 18:15 | |
| *** baoli has quit IRC | 18:16 | |
| *** neophy has quit IRC | 18:21 | |
| *** adisky_ has quit IRC | 18:29 | |
| *** ducttape_ has joined #openstack-nova | 18:32 | |
| mriedem | ok i'll supersede it with the nested resource providers one | 18:35 |
| *** ducttape_ has quit IRC | 18:36 | |
| *** kristian__ has joined #openstack-nova | 18:39 | |
| mriedem | hmm, ironic jobs must have been changed v | 18:42 |
| mriedem | http://logs.openstack.org/14/414214/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/4413603/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-12-28_17_03_09_238 | 18:42 |
| *** ejat_ has quit IRC | 18:42 | |
| *** ejat_ has joined #openstack-nova | 18:42 | |
| *** ejat_ is now known as ejat | 18:42 | |
| mriedem | yup https://github.com/openstack/ironic/commit/be6f34c3388365f93c84671f07af7a1cffb6efd8 | 18:43 |
| *** kristian__ has quit IRC | 18:43 | |
| jaypipes | mriedem: they disabled placement api in the ironic gate... | 18:43 |
| mriedem | yeah | 18:44 |
| mriedem | i just put up a revert of that change which depends on your fix | 18:44 |
| mriedem | https://review.openstack.org/#/c/415523/ | 18:44 |
| jaypipes | cool, thanks mriedem | 18:44 |
| *** vtech has joined #openstack-nova | 18:50 | |
| openstackgerrit | Sujitha proposed openstack/nova: Document testing process for zero downtime upgrade https://review.openstack.org/395269 | 18:54 |
| *** david-lyle has joined #openstack-nova | 18:59 | |
| *** david-lyle has quit IRC | 19:03 | |
| *** yamahata has quit IRC | 19:03 | |
| *** jose-phillips has quit IRC | 19:15 | |
| *** jose-phillips has joined #openstack-nova | 19:17 | |
| *** slaweq has joined #openstack-nova | 19:18 | |
| *** brault has joined #openstack-nova | 19:19 | |
| *** jose-phillips has quit IRC | 19:21 | |
| *** tbachman has joined #openstack-nova | 19:22 | |
| *** brault has quit IRC | 19:23 | |
| *** rcernin has quit IRC | 19:24 | |
| *** markvoelker has joined #openstack-nova | 19:25 | |
| *** brault has joined #openstack-nova | 19:26 | |
| *** brault has quit IRC | 19:28 | |
| *** moshele has joined #openstack-nova | 19:28 | |
| *** markvoelker has quit IRC | 19:30 | |
| *** kristian__ has joined #openstack-nova | 19:31 | |
| *** pcaruana has joined #openstack-nova | 19:33 | |
| *** slaweq has quit IRC | 19:35 | |
| *** kristian__ has quit IRC | 19:36 | |
| *** slaweq has joined #openstack-nova | 19:37 | |
| *** slaweq has quit IRC | 19:42 | |
| *** rfolco has quit IRC | 19:51 | |
| *** kristian__ has joined #openstack-nova | 19:58 | |
| *** romainh has joined #openstack-nova | 19:58 | |
| *** pcaruana has quit IRC | 20:01 | |
| *** kristian__ has quit IRC | 20:03 | |
| *** kristian__ has joined #openstack-nova | 20:04 | |
| mriedem | mdbooth: i'm a bit confused about how this series is lined up https://review.openstack.org/#/c/408543/ - there are a bunch of changes, but only a few in the same topic, but they seem to be rebased against each other somehow...and lots are in merge conflict | 20:09 |
| mriedem | and i can't track any of it from the blueprint in lp | 20:09 |
| mriedem | jaypipes: can we consider this complete? https://blueprints.launchpad.net/nova/+spec/placement-api-policy-authz - we have a single admin-only policy check for the placement API and it doesn't look like any finer-grained policy checking is going to be built into that, at least near-term in ocata. | 20:11 |
| jaypipes | mriedem: cool by me. | 20:12 |
| *** hongbin has joined #openstack-nova | 20:15 | |
| *** kfarr has quit IRC | 20:16 | |
| jaypipes | ok wtf... | 20:16 |
| jaypipes | Different rcs for rp b665cdb4-4b95-4660-ba10-943dd39c1946! set(['b665cdb4-4b95-4660-ba10-943dd39c1946']) != set(['b665cdb4-4b95-4660-ba10-943dd39c1946']) | 20:16 |
| jaypipes | oops, wrong buffer.. | 20:17 |
| *** catinthe_ has joined #openstack-nova | 20:27 | |
| *** catintheroof has quit IRC | 20:28 | |
| *** ducttape_ has joined #openstack-nova | 20:34 | |
| *** matrohon has joined #openstack-nova | 20:35 | |
| mriedem | heh duh a != a | 20:38 |
| *** ducttape_ has quit IRC | 20:39 | |
| *** baoli has joined #openstack-nova | 20:39 | |
| jaypipes | mriedem: indeed! | 20:40 |
| jaypipes | I believe it's almost beer thirty.. | 20:41 |
| *** slaweq has joined #openstack-nova | 20:41 | |
| mriedem | don't leave me | 20:41 |
| *** baoli has quit IRC | 20:43 | |
| jaypipes | mriedem: heh, I'm not :) | 20:44 |
| *** jose-phillips has joined #openstack-nova | 20:47 | |
| *** ducttape_ has joined #openstack-nova | 20:47 | |
| *** jose-phillips has quit IRC | 20:47 | |
| *** vtech has quit IRC | 20:48 | |
| *** satyar has quit IRC | 20:50 | |
| *** sacharya has quit IRC | 20:50 | |
| *** nicolasbock has joined #openstack-nova | 20:51 | |
| *** jose-phillips has joined #openstack-nova | 20:51 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add zero down time upgrade steps https://review.openstack.org/393931 | 20:54 |
| *** david-lyle has joined #openstack-nova | 21:00 | |
| *** david-lyle has quit IRC | 21:05 | |
| *** ducttape_ has quit IRC | 21:06 | |
| *** moshele has quit IRC | 21:07 | |
| *** kristian__ has quit IRC | 21:14 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | 21:19 |
| jaypipes | mriedem: phew. ^ | 21:19 |
| mriedem | gibi: i don't suppose it's possible to do something with the notification samples such that every time we change one field we don't have to adjust ALL samples, because that's going to get annoying over time, e.g. https://review.openstack.org/#/c/407128/ | 21:24 |
| mriedem | i don't suppose we want to actually pin the versions in the notification samples tests though.... | 21:24 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | 21:26 |
| *** nicolasbock has quit IRC | 21:27 | |
| *** Jeffrey4l has quit IRC | 21:34 | |
| *** nklenke has quit IRC | 21:35 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Return uuid attribute for aggregates https://review.openstack.org/415031 | 21:38 |
| jaypipes | mriedem: ^^ | 21:38 |
| jaypipes | mriedem: I'm logging off for the day now. catch you tomorrow. | 21:38 |
| jaypipes | oih, shit... the 0-valued inventory one. | 21:39 |
| jaypipes | lemme fix that up for you first. | 21:39 |
| *** baoli has joined #openstack-nova | 21:39 | |
| *** baoli has quit IRC | 21:44 | |
| *** romainh1 has joined #openstack-nova | 21:45 | |
| *** romainh has quit IRC | 21:46 | |
| *** lamt has quit IRC | 21:46 | |
| *** Jeffrey4l has joined #openstack-nova | 21:47 | |
| *** sacharya has joined #openstack-nova | 21:51 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: Do not save 0-valued inventory https://review.openstack.org/414214 | 21:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Return uuid attribute for aggregates https://review.openstack.org/415031 | 21:52 |
| jaypipes | mriedem: ok, really am logging off now :) | 21:53 |
| jaypipes | hasta manana | 21:53 |
| *** jaypipes has quit IRC | 21:54 | |
| *** sacharya has quit IRC | 21:56 | |
| *** david-lyle has joined #openstack-nova | 22:01 | |
| *** david-lyle has quit IRC | 22:06 | |
| *** ducttape_ has joined #openstack-nova | 22:07 | |
| *** ducttape_ has quit IRC | 22:12 | |
| openstackgerrit | Sujitha proposed openstack/nova: Adding tags field to InstancePayload https://review.openstack.org/407228 | 22:16 |
| *** baoli has joined #openstack-nova | 22:21 | |
| mriedem | raj_singh: https://review.openstack.org/#/c/411501/ - needed a space between Depends-On and the change id on that line :) | 22:25 |
| mriedem | it wasn't picking up the devstack dependency | 22:25 |
| *** markvoelker has joined #openstack-nova | 22:27 | |
| *** romainh1 has quit IRC | 22:28 | |
| *** markvoelker has quit IRC | 22:33 | |
| *** markvoelker has joined #openstack-nova | 22:35 | |
| *** slaweq has quit IRC | 22:50 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add more field's in InstancePayload https://review.openstack.org/407128 | 22:50 |
| *** markvoelker has quit IRC | 22:50 | |
| *** ducttape_ has joined #openstack-nova | 22:53 | |
| *** markvoelker has joined #openstack-nova | 22:53 | |
| *** baoli has quit IRC | 22:59 | |
| *** baoli has joined #openstack-nova | 22:59 | |
| *** slaweq has joined #openstack-nova | 23:02 | |
| *** ducttape_ has quit IRC | 23:05 | |
| *** ducttape_ has joined #openstack-nova | 23:07 | |
| mriedem | jroll: vdrok: gonna need to get ironicclient released with https://review.openstack.org/#/c/335964 and bump the minimum required version in global-requirements so nova can use it in https://review.openstack.org/#/c/388756/ | 23:18 |
| dtp | what's a scenario that would result it getting to nova/compute/api.py/_delete_instance with self.cell_type != 'api'? https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1815 | 23:24 |
| *** dimtruck is now known as zz_dimtruck | 23:24 | |
| dtp | _delete, not _delete_instance | 23:25 |
| *** lamt has joined #openstack-nova | 23:30 | |
| *** diga has joined #openstack-nova | 23:35 | |
| *** ducttape_ has quit IRC | 23:37 | |
| dtp | i'm heading out, but i'll check the log later if anyone responds | 23:38 |
| *** dtp has quit IRC | 23:38 | |
| *** yamahata has joined #openstack-nova | 23:46 | |
| *** catinthe_ has quit IRC | 23:48 | |
| *** ducttape_ has joined #openstack-nova | 23:54 | |
| *** slaweq has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!