| *** ljxiash has joined #openstack-nova | 00:00 | |
| *** haomaiwa_ has quit IRC | 00:01 | |
| *** akshai has joined #openstack-nova | 00:01 | |
| *** haomaiwang has joined #openstack-nova | 00:01 | |
| *** ljxiash_ has joined #openstack-nova | 00:01 | |
| *** mylu has quit IRC | 00:02 | |
| *** stevemar has quit IRC | 00:02 | |
| *** mvk_ has quit IRC | 00:03 | |
| *** ljxiash has quit IRC | 00:05 | |
| *** Sukhdev has quit IRC | 00:06 | |
| *** fawadkhaliq has quit IRC | 00:06 | |
| *** ljxiash_ has quit IRC | 00:06 | |
| *** mylu has joined #openstack-nova | 00:07 | |
| *** fawadkhaliq has joined #openstack-nova | 00:07 | |
| *** p01nt3r75 has quit IRC | 00:08 | |
| *** dimtruck is now known as zz_dimtruck | 00:08 | |
| *** salv-orl_ has quit IRC | 00:09 | |
| *** akshai_ has joined #openstack-nova | 00:09 | |
| *** zehua has joined #openstack-nova | 00:10 | |
| *** zehua has left #openstack-nova | 00:10 | |
| *** akshai has quit IRC | 00:13 | |
| *** fawadkhaliq has quit IRC | 00:13 | |
| *** ZZelle has quit IRC | 00:14 | |
| *** rk4n has quit IRC | 00:14 | |
| *** markvoelker has joined #openstack-nova | 00:15 | |
| *** fawadkhaliq has joined #openstack-nova | 00:16 | |
| *** fawadkhaliq has quit IRC | 00:21 | |
| *** markvoelker_ has joined #openstack-nova | 00:22 | |
| *** diogogmt has joined #openstack-nova | 00:22 | |
| *** mriedem has joined #openstack-nova | 00:23 | |
| *** davideag_ has quit IRC | 00:23 | |
| *** fawadkhaliq has joined #openstack-nova | 00:23 | |
| *** markvoelker has quit IRC | 00:25 | |
| *** mtanino has quit IRC | 00:29 | |
| *** Swami has quit IRC | 00:33 | |
| *** Sukhdev has joined #openstack-nova | 00:33 | |
| *** Sukhdev has quit IRC | 00:33 | |
| *** sdake has quit IRC | 00:34 | |
| *** sdake has joined #openstack-nova | 00:36 | |
| *** mvk_ has joined #openstack-nova | 00:37 | |
| *** Jeffrey4l_ has quit IRC | 00:38 | |
| *** ccarmack has quit IRC | 00:39 | |
| *** ducttape_ has joined #openstack-nova | 00:39 | |
| *** mylu has quit IRC | 00:39 | |
| *** mkrcmari__ has quit IRC | 00:40 | |
| *** achanda has quit IRC | 00:41 | |
| *** efoley_ has quit IRC | 00:42 | |
| *** mylu has joined #openstack-nova | 00:45 | |
| *** Marga_ has quit IRC | 00:48 | |
| *** ducttape_ has quit IRC | 00:50 | |
| *** mnestratov|2 has quit IRC | 00:50 | |
| *** vladikr has quit IRC | 00:52 | |
| *** dave-mccowan has quit IRC | 00:53 | |
| *** stevemar has joined #openstack-nova | 00:58 | |
| *** dave-mccowan has joined #openstack-nova | 00:58 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-nova | 01:01 | |
| *** jamielennox is now known as jamielennox|away | 01:01 | |
| *** nic has quit IRC | 01:02 | |
| *** stevemar has quit IRC | 01:03 | |
| *** ducttape_ has joined #openstack-nova | 01:04 | |
| *** mylu has quit IRC | 01:07 | |
| *** mylu has joined #openstack-nova | 01:08 | |
| *** aysyd has quit IRC | 01:08 | |
| *** ivase has quit IRC | 01:09 | |
| *** vilobhmm11 has quit IRC | 01:09 | |
| *** ivase has joined #openstack-nova | 01:09 | |
| *** ducttape_ has quit IRC | 01:09 | |
| *** vilobhmm11 has joined #openstack-nova | 01:10 | |
| *** yuywz has joined #openstack-nova | 01:12 | |
| *** Goneri has quit IRC | 01:12 | |
| *** apoorvad has quit IRC | 01:13 | |
| *** ivase_ has joined #openstack-nova | 01:13 | |
| *** fawadkhaliq has quit IRC | 01:14 | |
| *** fawadkhaliq has joined #openstack-nova | 01:14 | |
| *** ivase has quit IRC | 01:17 | |
| *** yuywz has quit IRC | 01:20 | |
| *** fawadkhaliq has quit IRC | 01:20 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: remove glance deprecated config https://review.openstack.org/299535 | 01:20 |
|---|---|---|
| *** houming has joined #openstack-nova | 01:22 | |
| *** fawadkhaliq has joined #openstack-nova | 01:23 | |
| *** vilobhmm11 has quit IRC | 01:23 | |
| *** fawadkhaliq has quit IRC | 01:24 | |
| *** fawadkhaliq has joined #openstack-nova | 01:25 | |
| *** fawadkhaliq has quit IRC | 01:27 | |
| *** vishwanathj has joined #openstack-nova | 01:30 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: centralize section: "glance" https://review.openstack.org/257153 | 01:32 |
| *** browne has quit IRC | 01:34 | |
| *** baoli has joined #openstack-nova | 01:37 | |
| *** baoli_ has joined #openstack-nova | 01:39 | |
| *** stevemar has joined #openstack-nova | 01:39 | |
| *** achanda has joined #openstack-nova | 01:41 | |
| *** baoli_ has quit IRC | 01:42 | |
| *** baoli has quit IRC | 01:42 | |
| *** Daisy has joined #openstack-nova | 01:43 | |
| mriedem | dansmith: can we please just go back to the good old days where you can add new APIs to cinder and nova can assume they are just available and the policy settings are the same as the defaults forever https://review.openstack.org/#/c/41298/ | 01:45 |
| *** efried has quit IRC | 01:45 | |
| *** achanda has quit IRC | 01:46 | |
| yonglihe | mriedem: ping | 01:47 |
| openstackgerrit | Mohammed Naser proposed openstack/nova: Convert MonitorMetric to use FloatField for values https://review.openstack.org/299708 | 01:48 |
| openstackgerrit | Mohammed Naser proposed openstack/nova: Convert MonitorMetric to use FloatField for values https://review.openstack.org/299708 | 01:49 |
| yonglihe | shaohe is on vacation, and the Intel SRIOV CI comments to Neutron, vnic_type is macvtap. | 01:49 |
| *** sdake_ has joined #openstack-nova | 01:49 | |
| *** ducttape_ has joined #openstack-nova | 01:49 | |
| *** Marga_ has joined #openstack-nova | 01:49 | |
| *** sdake has quit IRC | 01:50 | |
| *** Daisy has quit IRC | 01:51 | |
| *** Daisy has joined #openstack-nova | 01:51 | |
| *** ljxiash has joined #openstack-nova | 01:52 | |
| *** gjayavelu has joined #openstack-nova | 01:52 | |
| *** Marga_ has quit IRC | 01:54 | |
| *** Daisy has quit IRC | 01:55 | |
| *** Daisy has joined #openstack-nova | 01:55 | |
| *** achanda has joined #openstack-nova | 01:56 | |
| *** pushkaru has joined #openstack-nova | 01:57 | |
| mriedem | yonglihe: hi | 01:58 |
| mriedem | yonglihe: ok, can you update the third party CI wiki please with that information? | 01:58 |
| mriedem | yonglihe: are there any plans to run that against nova? | 01:58 |
| mriedem | yonglihe: because the wiki says it runs against nova today | 01:59 |
| mriedem | OpenStack Programs: Neutron,Nova | 01:59 |
| *** haomaiwang has quit IRC | 01:59 | |
| yonglihe | mriedem: ok, i will update the wiki. | 01:59 |
| *** Daisy has quit IRC | 02:00 | |
| *** mtanino has joined #openstack-nova | 02:00 | |
| *** Daisy has joined #openstack-nova | 02:00 | |
| *** unicell1 has quit IRC | 02:02 | |
| *** ducttape_ has quit IRC | 02:02 | |
| yonglihe | for running against nova: SRIOV need neutron been starting up, it's almost a new CI for Intel PCI/SRIOV CI. do you think we better to do that? we have a SRIOV test run against the nova. | 02:03 |
| *** Daisy has quit IRC | 02:03 | |
| *** Daisy has joined #openstack-nova | 02:03 | |
| *** ducttape_ has joined #openstack-nova | 02:03 | |
| mriedem | yonglihe: i'm confused, | 02:04 |
| *** zz_dimtruck is now known as dimtruck | 02:04 | |
| mriedem | yonglihe: there is intel nfv ci, intel sriov ci, and intel networking ci | 02:04 |
| mriedem | you said intel sriov ci uses macvtap and only runs on neutron, correct? | 02:04 |
| mriedem | what do the other 2 test? | 02:04 |
| yonglihe | yeah, the other 2 test is not operated by me, we have escalate your questing internally. so someone else will update you about this. | 02:05 |
| yonglihe | questing->question | 02:06 |
| *** liunian_ has joined #openstack-nova | 02:06 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config options: Centralize netconf options https://review.openstack.org/294312 | 02:07 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config options: Centralize resource tracker options https://review.openstack.org/294211 | 02:07 |
| *** ducttape_ has quit IRC | 02:08 | |
| *** gjayavelu has quit IRC | 02:09 | |
| *** Marga_ has joined #openstack-nova | 02:10 | |
| yonglihe | mriedem: SRIOV CI wiki updated. sriov ci runs on neutron. according to wiki: Intel-Networking-CI runs on Neutron, Intel-NFV-CI runs on Nova. | 02:11 |
| *** mylu has quit IRC | 02:11 | |
| *** jianghuaw has joined #openstack-nova | 02:12 | |
| *** thorst has quit IRC | 02:13 | |
| *** thorst has joined #openstack-nova | 02:13 | |
| mriedem | yonglihe: ok, thanks, i look forward to the details. | 02:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix swap-volume for non-admin users https://review.openstack.org/299715 | 02:14 |
| *** Goneri has joined #openstack-nova | 02:14 | |
| *** Marga_ has quit IRC | 02:14 | |
| gmann | mriedem: just replied in ML, so in https://review.openstack.org/299715 | 02:15 |
| gmann | mriedem: if nova allow swap volume for non-admin then with your patch it should do attach/detach explicitly right? | 02:15 |
| *** Jeffrey4l has joined #openstack-nova | 02:17 | |
| mriedem | gmann: well, if non-admin it just shouldn't call cinder's migrate_volume_completion API | 02:17 |
| mriedem | my fix isn't checking admin vs non-admin | 02:17 |
| mriedem | its' checking to see if the volume migration_status field is set | 02:18 |
| mriedem | if it is, we're an admin and it is a call initiated by cinder | 02:18 |
| gmann | mriedem: yea | 02:18 |
| gmann | and in case non-admin user call initiated on nova | 02:18 |
| mriedem | i might work on a tempest test for the non-admin swap-volume case tomorrow | 02:18 |
| mriedem | unless you want to :) | 02:19 |
| jianghuaw | hi, all. If I want to avoid running python34 testing on all unit tests which located under a specific directory, can I just add this directory into nova/tests-py3.txt? or I have to add each test case in it? Thanks. | 02:19 |
| gmann | in that case volume migration_status filed would not be present right | 02:19 |
| mriedem | gmann: right | 02:19 |
| mriedem | jianghuaw: nova/tests-py3.txt is a blacklist file | 02:20 |
| gmann | mriedem: so in that case, new volume attach and old detach is not happened which was done in migrate_volume_completion on cinder side | 02:20 |
| mriedem | so add tests that you don't want to run to that file for py34 | 02:20 |
| mriedem | gmann: i'm not following the question | 02:20 |
| gmann | mriedem: actually i was thinking the case where swap volume call is initiated by non-admin on nova side | 02:21 |
| gmann | in that case migrate_volume_completion will not be called as volume migration_status will be unset | 02:21 |
| *** salv-orlando has joined #openstack-nova | 02:21 | |
| gmann | so attach new volume and detach old volume on server would not happen (which is done in cinder->migrate_volume_completion) | 02:22 |
| *** thorst has quit IRC | 02:22 | |
| gmann | I think long back we moved that part from nova to cinder side | 02:22 |
| jianghuaw | mriedem: thanks. Can I just add a single line for the directory? e.g. I have several test cases under nova/tests/unit/virt/xenapi/plugins, can I just add a line as: nova.tests.unit.virt.xenapi.plugins to exclude all of those tests under that path? | 02:23 |
| mriedem | gmann: are you sure that doesn't happen in the virt driver's swap_volume method? | 02:23 |
| *** Goneri has quit IRC | 02:23 | |
| gmann | mriedem: humm, let me check that. from manager it was removed | 02:24 |
| mriedem | jianghuaw: nova.tests.unit.virt.xenapi.plugins is already in there | 02:24 |
| gmann | but i will check and see if that happens | 02:24 |
| mriedem | gmann: ok, i want to write a tempest test before this change goes in anyway and depend on that to see it passing, | 02:24 |
| mriedem | but it's getting late for me so i'm going to sign off | 02:24 |
| gmann | mriedem: yea that is nice idea. I will add tempest one and once you are in tomorrow can see the result | 02:25 |
| mriedem | gmann: cool, thanks | 02:25 |
| gmann | mriedem: Thanks, have a GN | 02:25 |
| *** mriedem has quit IRC | 02:25 | |
| jianghuaw | mriedem: indeed. But py34 seems not exclude the new test when I added a new one in the same path today. | 02:26 |
| *** xyang1 has quit IRC | 02:26 | |
| *** ljxiash has quit IRC | 02:26 | |
| jianghuaw | that's why I'm thinking if I have to add a line for each test. | 02:26 |
| *** ljxiash has joined #openstack-nova | 02:27 | |
| *** browne has joined #openstack-nova | 02:28 | |
| *** ljjjustin_ has joined #openstack-nova | 02:30 | |
| *** markvoelker_ has quit IRC | 02:31 | |
| *** zehua_ has joined #openstack-nova | 02:31 | |
| *** zehua_ has quit IRC | 02:31 | |
| *** hoangcx has joined #openstack-nova | 02:31 | |
| *** ljxiash has quit IRC | 02:32 | |
| *** salv-orlando has quit IRC | 02:32 | |
| *** markvoelker_ has joined #openstack-nova | 02:32 | |
| jianghuaw | mriedem: oh. I misunderstood the failure. the py34 testing failed due to "urllib2 is not available with py3x". So actually it fails at syntax parsing. I think I've fixed that. So indeed the path way works at here. | 02:34 |
| jianghuaw | Thanks mriedem. | 02:34 |
| *** vilobhmm11 has joined #openstack-nova | 02:34 | |
| *** markvoel_ has joined #openstack-nova | 02:36 | |
| *** liunian_ has quit IRC | 02:36 | |
| *** mylu has joined #openstack-nova | 02:36 | |
| *** salv-orlando has joined #openstack-nova | 02:36 | |
| *** yamahata has quit IRC | 02:38 | |
| *** haomaiwang has joined #openstack-nova | 02:38 | |
| *** markvoelker_ has quit IRC | 02:40 | |
| *** liunian_ has joined #openstack-nova | 02:41 | |
| *** liunian_ has quit IRC | 02:43 | |
| *** ljjjustin_ is now known as zehua | 02:44 | |
| *** mylu has quit IRC | 02:44 | |
| *** mylu has joined #openstack-nova | 02:44 | |
| *** liunian_ has joined #openstack-nova | 02:46 | |
| *** mylu_ has joined #openstack-nova | 02:48 | |
| *** Marga_ has joined #openstack-nova | 02:48 | |
| *** mylu has quit IRC | 02:49 | |
| *** salv-orlando has quit IRC | 02:52 | |
| *** Marga_ has quit IRC | 02:53 | |
| *** liunian_ has quit IRC | 02:54 | |
| *** liunian_ has joined #openstack-nova | 02:58 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** markvoelker has joined #openstack-nova | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** hoangcx has quit IRC | 03:02 | |
| *** jianghuaw has quit IRC | 03:02 | |
| *** achanda has quit IRC | 03:03 | |
| *** markvoelker_ has joined #openstack-nova | 03:04 | |
| *** markvoel_ has quit IRC | 03:04 | |
| openstackgerrit | Jiajun Liu proposed openstack/nova: config options: centralize cinder options https://review.openstack.org/298191 | 03:04 |
| *** sdake has joined #openstack-nova | 03:07 | |
| *** Marga_ has joined #openstack-nova | 03:07 | |
| *** Marga_ has quit IRC | 03:07 | |
| *** fawadkhaliq has joined #openstack-nova | 03:08 | |
| *** Marga_ has joined #openstack-nova | 03:08 | |
| *** markvoelker has quit IRC | 03:08 | |
| *** Marga_ has quit IRC | 03:08 | |
| *** hoangcx has joined #openstack-nova | 03:08 | |
| *** Marga_ has joined #openstack-nova | 03:09 | |
| *** sdake_ has quit IRC | 03:10 | |
| *** pumarani__ has joined #openstack-nova | 03:10 | |
| *** apoorvad has joined #openstack-nova | 03:11 | |
| *** pushkaru has quit IRC | 03:14 | |
| *** apoorvad has quit IRC | 03:19 | |
| *** thorst has joined #openstack-nova | 03:19 | |
| *** tangchen has joined #openstack-nova | 03:21 | |
| *** pkoniszewski has quit IRC | 03:24 | |
| *** coreywright has quit IRC | 03:24 | |
| *** garyk1 has joined #openstack-nova | 03:24 | |
| *** jplopezgu_ has joined #openstack-nova | 03:25 | |
| *** fawadkhaliq has quit IRC | 03:26 | |
| *** fawadkhaliq has joined #openstack-nova | 03:26 | |
| *** ajo_ has joined #openstack-nova | 03:27 | |
| *** jeblair_ has joined #openstack-nova | 03:27 | |
| *** thorst has quit IRC | 03:27 | |
| *** haomaiw__ has joined #openstack-nova | 03:28 | |
| *** rhagarty__ has joined #openstack-nova | 03:28 | |
| *** keekz_ has joined #openstack-nova | 03:30 | |
| *** krotscheck has joined #openstack-nova | 03:30 | |
| *** hamzy_ has joined #openstack-nova | 03:31 | |
| *** dmellado_ has joined #openstack-nova | 03:31 | |
| *** scottda_ has joined #openstack-nova | 03:31 | |
| *** pkoniszewski has joined #openstack-nova | 03:31 | |
| *** fawadkhaliq has quit IRC | 03:32 | |
| *** rpodolyaka_ has joined #openstack-nova | 03:32 | |
| *** fawadkhaliq has joined #openstack-nova | 03:32 | |
| *** jlopezgu has quit IRC | 03:32 | |
| *** jplopezgu_ is now known as jlopezgu | 03:32 | |
| *** keekz has quit IRC | 03:32 | |
| *** dougwig has quit IRC | 03:32 | |
| *** jeblair has quit IRC | 03:32 | |
| *** ajo has quit IRC | 03:32 | |
| *** inara` has quit IRC | 03:32 | |
| *** haomaiwang has quit IRC | 03:32 | |
| *** rhagarty_ has quit IRC | 03:32 | |
| *** hamzy has quit IRC | 03:32 | |
| *** krotscheck_dcm has quit IRC | 03:32 | |
| *** scottda has quit IRC | 03:32 | |
| *** dmellado has quit IRC | 03:32 | |
| *** inara has joined #openstack-nova | 03:32 | |
| *** aorourke_ has joined #openstack-nova | 03:33 | |
| *** fmccrthy has quit IRC | 03:33 | |
| *** beagles has quit IRC | 03:33 | |
| *** qeelee has quit IRC | 03:33 | |
| *** rpodolyaka has quit IRC | 03:33 | |
| *** andrewbogott has quit IRC | 03:33 | |
| *** serverascode has quit IRC | 03:33 | |
| *** tpatzig has quit IRC | 03:33 | |
| *** BlackDex has quit IRC | 03:33 | |
| *** aorourke has quit IRC | 03:33 | |
| *** adreznec has quit IRC | 03:33 | |
| *** tpatzig has joined #openstack-nova | 03:33 | |
| *** rpodolyaka_ is now known as rpodolyaka | 03:34 | |
| *** b3nt_pin has joined #openstack-nova | 03:34 | |
| *** adreznec has joined #openstack-nova | 03:34 | |
| *** fmccrthy has joined #openstack-nova | 03:34 | |
| *** b3nt_pin is now known as Guest40396 | 03:34 | |
| *** Jeffrey4l_ has joined #openstack-nova | 03:34 | |
| *** scottda_ is now known as scottda | 03:34 | |
| *** Jeffrey4l has quit IRC | 03:37 | |
| *** BlackDex has joined #openstack-nova | 03:38 | |
| *** andrewbogott has joined #openstack-nova | 03:38 | |
| *** dougwig has joined #openstack-nova | 03:38 | |
| *** serverascode has joined #openstack-nova | 03:39 | |
| *** pumarani__ has quit IRC | 03:41 | |
| *** liunian_ has quit IRC | 03:42 | |
| *** coreywright has joined #openstack-nova | 03:42 | |
| *** links has joined #openstack-nova | 03:44 | |
| *** liunian_ has joined #openstack-nova | 03:47 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: add unit test for the plugin - glance https://review.openstack.org/299533 | 03:47 |
| *** yarkot_ has joined #openstack-nova | 03:51 | |
| *** liunian_ has quit IRC | 03:51 | |
| *** salv-orlando has joined #openstack-nova | 03:52 | |
| *** yarkot_ has quit IRC | 03:53 | |
| *** salv-orlando has quit IRC | 03:55 | |
| *** liunian_ has joined #openstack-nova | 03:56 | |
| *** ljxiash has joined #openstack-nova | 03:57 | |
| *** pushkaru has joined #openstack-nova | 03:57 | |
| *** mylu_ has quit IRC | 03:58 | |
| *** fawadkhaliq has quit IRC | 03:58 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add tag column to vifs and bdm https://review.openstack.org/264014 | 03:59 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add metadata objects for device tagging https://review.openstack.org/264015 | 03:59 |
| *** fawadkhaliq has joined #openstack-nova | 03:59 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging user visible changes https://review.openstack.org/264018 | 03:59 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add compute manager device tagging support https://review.openstack.org/264017 | 03:59 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 03:59 |
| *** fawadkhaliq has quit IRC | 04:00 | |
| *** fawadkhaliq has joined #openstack-nova | 04:01 | |
| *** haomaiw__ has quit IRC | 04:01 | |
| *** ivase has joined #openstack-nova | 04:01 | |
| *** liunian_ has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:01 | |
| *** ljxiash has quit IRC | 04:01 | |
| *** liunian_ has joined #openstack-nova | 04:01 | |
| *** irenab_ has joined #openstack-nova | 04:02 | |
| *** fawadkhaliq has quit IRC | 04:02 | |
| *** fawadkhaliq has joined #openstack-nova | 04:02 | |
| *** ivase_ has quit IRC | 04:03 | |
| *** achanda has joined #openstack-nova | 04:04 | |
| *** tangchen has quit IRC | 04:04 | |
| *** irenab has quit IRC | 04:04 | |
| *** irenab_ is now known as irenab | 04:04 | |
| *** anush_ has joined #openstack-nova | 04:04 | |
| *** liunian_ has quit IRC | 04:05 | |
| *** liunian_ has joined #openstack-nova | 04:08 | |
| *** achanda has quit IRC | 04:09 | |
| *** achanda has joined #openstack-nova | 04:09 | |
| *** baoli has joined #openstack-nova | 04:10 | |
| *** liunian_ has quit IRC | 04:13 | |
| *** baoli has quit IRC | 04:14 | |
| *** salv-orlando has joined #openstack-nova | 04:16 | |
| *** mylu has joined #openstack-nova | 04:17 | |
| *** liunian_ has joined #openstack-nova | 04:17 | |
| *** pushkaru has quit IRC | 04:21 | |
| *** markvoelker_ has quit IRC | 04:21 | |
| *** salv-orlando has quit IRC | 04:23 | |
| *** liunian_ has quit IRC | 04:27 | |
| *** sdake_ has joined #openstack-nova | 04:28 | |
| *** fawadkhaliq has quit IRC | 04:29 | |
| *** liunian_ has joined #openstack-nova | 04:29 | |
| *** fawadkhaliq has joined #openstack-nova | 04:30 | |
| *** sdake has quit IRC | 04:30 | |
| *** baoli has joined #openstack-nova | 04:31 | |
| *** dimtruck is now known as zz_dimtruck | 04:32 | |
| *** EinstCrazy has joined #openstack-nova | 04:33 | |
| *** efoley has joined #openstack-nova | 04:34 | |
| *** mylu has quit IRC | 04:34 | |
| *** Jeffrey4l_ has quit IRC | 04:34 | |
| *** rk4n has joined #openstack-nova | 04:35 | |
| *** rk4n has quit IRC | 04:35 | |
| *** baoli has quit IRC | 04:36 | |
| *** vilobhmm11 has quit IRC | 04:41 | |
| *** mylu has joined #openstack-nova | 04:42 | |
| *** moshele has joined #openstack-nova | 04:45 | |
| *** vilobhmm11 has joined #openstack-nova | 04:47 | |
| *** yamahata has joined #openstack-nova | 04:47 | |
| *** liunian_ has quit IRC | 04:49 | |
| *** efoley has quit IRC | 04:50 | |
| *** liunian_ has joined #openstack-nova | 04:51 | |
| *** liunian_ has quit IRC | 04:52 | |
| *** vishwanathj has quit IRC | 04:53 | |
| *** EinstCra_ has joined #openstack-nova | 04:57 | |
| *** haukebruno has joined #openstack-nova | 04:58 | |
| *** rcernin has joined #openstack-nova | 04:59 | |
| *** gjayavelu has joined #openstack-nova | 05:00 | |
| *** liunian_ has joined #openstack-nova | 05:00 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** EinstCrazy has quit IRC | 05:01 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: add unit test for the plugin - glance https://review.openstack.org/299533 | 05:01 |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** liunian_ has quit IRC | 05:05 | |
| *** liunian_ has joined #openstack-nova | 05:11 | |
| *** liunian_ has quit IRC | 05:11 | |
| *** nikhil has quit IRC | 05:13 | |
| *** dave-mccowan has quit IRC | 05:13 | |
| *** nikhil has joined #openstack-nova | 05:14 | |
| *** sdake_ is now known as sdake | 05:14 | |
| *** jaosorior has joined #openstack-nova | 05:14 | |
| *** fawadkhaliq has quit IRC | 05:15 | |
| *** fawadkhaliq has joined #openstack-nova | 05:16 | |
| *** liunian_ has joined #openstack-nova | 05:21 | |
| *** unicell has joined #openstack-nova | 05:23 | |
| *** salv-orlando has joined #openstack-nova | 05:23 | |
| *** liunian_ has quit IRC | 05:25 | |
| *** Sukhdev has joined #openstack-nova | 05:26 | |
| *** salv-orlando has quit IRC | 05:28 | |
| *** mtanino has quit IRC | 05:31 | |
| *** baoli has joined #openstack-nova | 05:32 | |
| *** thorst has joined #openstack-nova | 05:32 | |
| *** gjayavelu has quit IRC | 05:36 | |
| *** lpetrut has joined #openstack-nova | 05:36 | |
| *** baoli has quit IRC | 05:37 | |
| *** thorst has quit IRC | 05:37 | |
| *** fawadkhaliq has quit IRC | 05:37 | |
| *** sridharg has joined #openstack-nova | 05:38 | |
| *** Jeffrey4l_ has joined #openstack-nova | 05:39 | |
| *** mylu has quit IRC | 05:39 | |
| *** fawadkhaliq has joined #openstack-nova | 05:39 | |
| *** browne has quit IRC | 05:40 | |
| *** liunian_ has joined #openstack-nova | 05:42 | |
| *** pcaruana has quit IRC | 05:49 | |
| *** mtanino has joined #openstack-nova | 05:50 | |
| *** Jeffrey4l_ has quit IRC | 05:54 | |
| *** e0ne has joined #openstack-nova | 05:56 | |
| *** gjayavelu has joined #openstack-nova | 05:57 | |
| *** ihrachys has joined #openstack-nova | 05:57 | |
| *** ivase_ has joined #openstack-nova | 05:57 | |
| *** Sukhdev has quit IRC | 05:57 | |
| *** Swami has joined #openstack-nova | 05:59 | |
| *** fawadkhaliq has quit IRC | 05:59 | |
| *** fawadkhaliq has joined #openstack-nova | 06:00 | |
| *** ivase has quit IRC | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** gjayavelu has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-nova | 06:01 | |
| *** mtanino has quit IRC | 06:03 | |
| *** fawadkhaliq has quit IRC | 06:04 | |
| *** fawadkhaliq has joined #openstack-nova | 06:08 | |
| *** mnestratov|2 has joined #openstack-nova | 06:11 | |
| *** ihrachys has quit IRC | 06:11 | |
| *** nkrinner has joined #openstack-nova | 06:12 | |
| *** rk4n has joined #openstack-nova | 06:12 | |
| *** mpavone has joined #openstack-nova | 06:15 | |
| *** mjura has joined #openstack-nova | 06:16 | |
| *** unicell has quit IRC | 06:16 | |
| *** unicell has joined #openstack-nova | 06:17 | |
| *** lpetrut has quit IRC | 06:17 | |
| *** woodster_ has joined #openstack-nova | 06:18 | |
| *** fawadkhaliq has quit IRC | 06:19 | |
| *** fawadkhaliq has joined #openstack-nova | 06:19 | |
| *** esracelik has joined #openstack-nova | 06:20 | |
| *** daemontool has joined #openstack-nova | 06:22 | |
| *** anush_ has quit IRC | 06:22 | |
| *** yamahata has quit IRC | 06:23 | |
| *** e0ne has quit IRC | 06:24 | |
| *** Swami has quit IRC | 06:24 | |
| *** liunian__ has joined #openstack-nova | 06:29 | |
| *** liunian_ has quit IRC | 06:29 | |
| *** thorst has joined #openstack-nova | 06:34 | |
| *** sdake_ has joined #openstack-nova | 06:37 | |
| *** sdake has quit IRC | 06:40 | |
| *** garyk1 has quit IRC | 06:40 | |
| *** thorst has quit IRC | 06:42 | |
| *** sdake has joined #openstack-nova | 06:45 | |
| *** rk4n has quit IRC | 06:46 | |
| *** hoangcx has quit IRC | 06:47 | |
| *** hoangcx has joined #openstack-nova | 06:48 | |
| *** salv-orlando has joined #openstack-nova | 06:48 | |
| *** sdake_ has quit IRC | 06:48 | |
| *** fawadkhaliq has quit IRC | 06:48 | |
| *** fawadkhaliq has joined #openstack-nova | 06:49 | |
| *** tesseract has joined #openstack-nova | 06:49 | |
| *** tesseract is now known as Guest11037 | 06:49 | |
| *** baoli has joined #openstack-nova | 06:50 | |
| *** salv-orlando has quit IRC | 06:51 | |
| *** rk4n has joined #openstack-nova | 06:52 | |
| *** baoli has quit IRC | 06:55 | |
| *** rk4n has quit IRC | 06:56 | |
| *** ihrachys has joined #openstack-nova | 06:58 | |
| *** moshele has quit IRC | 07:00 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| *** moshele has joined #openstack-nova | 07:02 | |
| *** daemontool has quit IRC | 07:04 | |
| *** rk4n has joined #openstack-nova | 07:05 | |
| *** gongysh has quit IRC | 07:06 | |
| *** liverpooler has joined #openstack-nova | 07:06 | |
| *** Daisy_ has joined #openstack-nova | 07:07 | |
| *** ivase has joined #openstack-nova | 07:07 | |
| *** Daisy has quit IRC | 07:10 | |
| *** ivase_ has quit IRC | 07:10 | |
| *** Daisy has joined #openstack-nova | 07:11 | |
| *** Daisy__ has joined #openstack-nova | 07:12 | |
| *** Daisy___ has joined #openstack-nova | 07:12 | |
| *** rk4n has quit IRC | 07:14 | |
| *** Daisy_ has quit IRC | 07:14 | |
| *** e0ne has joined #openstack-nova | 07:15 | |
| *** Daisy has quit IRC | 07:16 | |
| *** rdopiera has joined #openstack-nova | 07:16 | |
| *** Daisy___ has quit IRC | 07:16 | |
| *** Daisy__ has quit IRC | 07:16 | |
| *** Daisy has joined #openstack-nova | 07:16 | |
| *** pushkaru has joined #openstack-nova | 07:17 | |
| *** Daisy_ has joined #openstack-nova | 07:18 | |
| *** e0ne has quit IRC | 07:18 | |
| *** Daisy has quit IRC | 07:18 | |
| *** Daisy has joined #openstack-nova | 07:20 | |
| *** atuvenie has joined #openstack-nova | 07:21 | |
| *** Daisy_ has quit IRC | 07:23 | |
| *** gongysh has joined #openstack-nova | 07:23 | |
| *** Daisy has quit IRC | 07:24 | |
| *** takashin has joined #openstack-nova | 07:26 | |
| *** jlanoux has joined #openstack-nova | 07:26 | |
| *** jlanoux_ has joined #openstack-nova | 07:28 | |
| *** jlanoux has quit IRC | 07:28 | |
| *** p01nt3r75 has joined #openstack-nova | 07:29 | |
| *** zenoway has joined #openstack-nova | 07:30 | |
| *** stevemar has quit IRC | 07:33 | |
| *** omnipresent has joined #openstack-nova | 07:33 | |
| *** zenoway has quit IRC | 07:35 | |
| *** pushkaru has quit IRC | 07:38 | |
| *** wuhg has joined #openstack-nova | 07:38 | |
| *** zenoway has joined #openstack-nova | 07:39 | |
| *** omnipresent has quit IRC | 07:40 | |
| *** liverpooler has quit IRC | 07:40 | |
| *** liverpooler has joined #openstack-nova | 07:41 | |
| *** thorst has joined #openstack-nova | 07:41 | |
| *** pcaruana has joined #openstack-nova | 07:41 | |
| *** coreywright has quit IRC | 07:47 | |
| *** thorst has quit IRC | 07:47 | |
| *** permalac has joined #openstack-nova | 07:48 | |
| *** moshele has quit IRC | 07:48 | |
| *** armax_ has joined #openstack-nova | 07:49 | |
| *** moshele has joined #openstack-nova | 07:49 | |
| *** rk4n has joined #openstack-nova | 07:49 | |
| *** mgoddard has joined #openstack-nova | 07:50 | |
| *** armax has quit IRC | 07:52 | |
| *** armax_ is now known as armax | 07:52 | |
| *** ihrachys has quit IRC | 07:53 | |
| *** jistr has joined #openstack-nova | 07:53 | |
| *** matrohon has joined #openstack-nova | 07:54 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: centralize section: "crypto" https://review.openstack.org/273916 | 07:56 |
| *** fawadkhaliq has quit IRC | 07:59 | |
| *** cristicalin has joined #openstack-nova | 07:59 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** fawadkhaliq has joined #openstack-nova | 08:01 | |
| Kevin_Zheng | mikal: Hi, can you remove your -2 on https://review.openstack.org/#/c/272949/ , since Newon is open and I have a dependent patch on this one. | 08:03 |
| *** fawadkhaliq has quit IRC | 08:03 | |
| *** coreywright has joined #openstack-nova | 08:03 | |
| *** fawadkhaliq has joined #openstack-nova | 08:04 | |
| *** permalac has quit IRC | 08:06 | |
| *** salv-orlando has joined #openstack-nova | 08:07 | |
| *** salv-orlando has quit IRC | 08:09 | |
| *** Daisy has joined #openstack-nova | 08:14 | |
| *** PaulMurray has joined #openstack-nova | 08:15 | |
| *** Daisy has quit IRC | 08:15 | |
| PaulMurray | good morning everyone | 08:15 |
| *** Daisy has joined #openstack-nova | 08:15 | |
| *** sdake_ has joined #openstack-nova | 08:17 | |
| *** salv-orlando has joined #openstack-nova | 08:18 | |
| *** sdake has quit IRC | 08:19 | |
| *** ndipanov has joined #openstack-nova | 08:20 | |
| *** sdake has joined #openstack-nova | 08:22 | |
| *** salv-orlando has quit IRC | 08:22 | |
| *** sdake_ has quit IRC | 08:24 | |
| *** permalac has joined #openstack-nova | 08:25 | |
| *** woodster_ has quit IRC | 08:27 | |
| *** Daisy_ has joined #openstack-nova | 08:27 | |
| *** e0ne has joined #openstack-nova | 08:28 | |
| *** e0ne has quit IRC | 08:29 | |
| *** ihrachys has joined #openstack-nova | 08:30 | |
| *** ndipanov has quit IRC | 08:30 | |
| *** Daisy has quit IRC | 08:30 | |
| *** achanda has quit IRC | 08:30 | |
| *** Daisy_ has quit IRC | 08:31 | |
| *** sfinucan has joined #openstack-nova | 08:33 | |
| *** stevemar has joined #openstack-nova | 08:33 | |
| *** achanda has joined #openstack-nova | 08:34 | |
| *** e0ne has joined #openstack-nova | 08:34 | |
| *** e0ne has quit IRC | 08:35 | |
| *** akshai_ has quit IRC | 08:37 | |
| *** fawadkhaliq has quit IRC | 08:37 | |
| *** yassine has joined #openstack-nova | 08:37 | |
| openstackgerrit | Li Yingjun proposed openstack/nova-specs: Hypervisor notifications spec https://review.openstack.org/299807 | 08:37 |
| *** yassine is now known as Guest41459 | 08:38 | |
| *** Guest41459 is now known as yassou | 08:38 | |
| *** stevemar has quit IRC | 08:39 | |
| *** fawadkhaliq has joined #openstack-nova | 08:40 | |
| *** daemontool has joined #openstack-nova | 08:41 | |
| *** ivase_ has joined #openstack-nova | 08:41 | |
| *** ivase has quit IRC | 08:44 | |
| *** thorst has joined #openstack-nova | 08:45 | |
| *** ljxiash has joined #openstack-nova | 08:46 | |
| *** tbachman has quit IRC | 08:46 | |
| *** cristicalin has quit IRC | 08:47 | |
| *** hoangcx has quit IRC | 08:47 | |
| *** tbachman has joined #openstack-nova | 08:47 | |
| *** daemontool has quit IRC | 08:48 | |
| *** achanda has quit IRC | 08:48 | |
| *** achanda has joined #openstack-nova | 08:49 | |
| *** ljxiash has quit IRC | 08:50 | |
| *** baoli has joined #openstack-nova | 08:51 | |
| *** josecastroleon has joined #openstack-nova | 08:51 | |
| *** garyk2 has joined #openstack-nova | 08:52 | |
| *** thorst has quit IRC | 08:52 | |
| *** achanda has quit IRC | 08:52 | |
| *** zioproto has joined #openstack-nova | 08:53 | |
| garyk2 | markus_z: please see https://review.openstack.org/218639 - that should address the bug that we spoke about yesterday | 08:53 |
| zioproto | hello there. I tried to submit a change into Openstack https://review.openstack.org/#/c/295865/ . So far I had positive review. Should I worry about the -1 from Xen Project CI ? | 08:55 |
| *** garyk2 has quit IRC | 08:55 | |
| *** garyk1 has joined #openstack-nova | 08:55 | |
| *** baoli has quit IRC | 08:55 | |
| *** hoangcx has joined #openstack-nova | 08:56 | |
| *** pece has joined #openstack-nova | 08:56 | |
| *** daemontool has joined #openstack-nova | 08:58 | |
| *** achanda has joined #openstack-nova | 08:59 | |
| *** rdopiera has quit IRC | 09:00 | |
| *** Daisy has joined #openstack-nova | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| *** Daisy_ has joined #openstack-nova | 09:01 | |
| *** rdopiera has joined #openstack-nova | 09:01 | |
| * sheel looking for review of https://review.openstack.org/#/c/280371/ | 09:02 | |
| gibi | zioproto: as far as I see the failure is not related to your change set so you should retrigger the Xen run to see if it succeeds. You can do that by adding a comment 'xen: recheck' to your review | 09:02 |
| *** vilobhmm11 has quit IRC | 09:03 | |
| *** Daisy_ has quit IRC | 09:03 | |
| *** fawadkhaliq has quit IRC | 09:05 | |
| *** Daisy has quit IRC | 09:06 | |
| markus_z | garyk1: thanks! I'll take a look. I'm not yet sure if we released RC3, if yes, I'll propose it for a backport. | 09:09 |
| *** fawadkhaliq has joined #openstack-nova | 09:09 | |
| *** Daisy has joined #openstack-nova | 09:10 | |
| *** ihrachys has quit IRC | 09:11 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: centralize section "workarounds" https://review.openstack.org/298083 | 09:11 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'utils' options https://review.openstack.org/298775 | 09:11 |
| *** Daisy has quit IRC | 09:11 | |
| zioproto | gibi: so just click on 'Reply ...' on top with text 'xen: recheck' and leave Code-Review and Workflow with 0 0 ? | 09:11 |
| zioproto | gibi: I hope I got it right https://review.openstack.org/#/c/295865/5 | 09:15 |
| gibi | zioproto: you did it right. :) Xen will redo the verification of your patchset | 09:17 |
| *** fhorvath has joined #openstack-nova | 09:18 | |
| *** derekh has joined #openstack-nova | 09:19 | |
| *** eliqiao has quit IRC | 09:19 | |
| *** danpb has joined #openstack-nova | 09:19 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'monkeypatch' options https://review.openstack.org/298776 | 09:20 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'monkey_patch' conf options documentation https://review.openstack.org/298796 | 09:20 |
| markus_z | garyk1: Could you add the "closes-bug 1563684" to that patch please? Or is Vipin double-checking it? | 09:23 |
| openstack | bug 1563684 in OpenStack Compute (nova) "VMware: rebuild an instance failed" [Medium,New] https://launchpad.net/bugs/1563684 - Assigned to Vipin Balachandran (vbala) | 09:23 |
| *** salv-orlando has joined #openstack-nova | 09:23 | |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Allow to update resource per single node. https://review.openstack.org/287485 | 09:24 |
| *** fawadkhaliq has quit IRC | 09:25 | |
| *** lpetrut has joined #openstack-nova | 09:25 | |
| *** achanda has quit IRC | 09:26 | |
| *** Jeffrey4l_ has joined #openstack-nova | 09:26 | |
| *** fawadkhaliq has joined #openstack-nova | 09:26 | |
| *** Daisy has joined #openstack-nova | 09:29 | |
| *** e0ne has joined #openstack-nova | 09:30 | |
| *** lpetrut has quit IRC | 09:31 | |
| *** lpetrut has joined #openstack-nova | 09:31 | |
| *** sdake_ has joined #openstack-nova | 09:32 | |
| *** sdake has quit IRC | 09:32 | |
| *** sdake has joined #openstack-nova | 09:33 | |
| *** gszasz has joined #openstack-nova | 09:35 | |
| *** bogdano has joined #openstack-nova | 09:35 | |
| *** sdake_ has quit IRC | 09:36 | |
| *** e0ne has quit IRC | 09:37 | |
| *** takashin has left #openstack-nova | 09:39 | |
| *** salv-orlando has quit IRC | 09:39 | |
| *** Daisy has quit IRC | 09:40 | |
| *** Daisy has joined #openstack-nova | 09:40 | |
| *** fawadkhaliq has quit IRC | 09:41 | |
| *** atuvenie has quit IRC | 09:42 | |
| bauzas | good morning nova | 09:42 |
| *** e0ne has joined #openstack-nova | 09:42 | |
| *** fawadkhaliq has joined #openstack-nova | 09:42 | |
| *** daemontool has quit IRC | 09:43 | |
| *** mkrcmari__ has joined #openstack-nova | 09:44 | |
| *** daemontool has joined #openstack-nova | 09:44 | |
| *** fawadkhaliq has quit IRC | 09:44 | |
| *** Daisy has quit IRC | 09:44 | |
| *** ndipanov has joined #openstack-nova | 09:45 | |
| *** liunian__ has quit IRC | 09:46 | |
| *** mvk_ has quit IRC | 09:47 | |
| gmann | mriedem: Added tempest tests https://review.openstack.org/#/c/299830/ | 09:48 |
| gmann | I will add Depends-On(or you add) once we have results with current code | 09:49 |
| *** liunian_ has joined #openstack-nova | 09:49 | |
| *** thorst has joined #openstack-nova | 09:50 | |
| *** fawadkhaliq has joined #openstack-nova | 09:50 | |
| *** mvk_ has joined #openstack-nova | 09:52 | |
| *** mkrcmari__ has quit IRC | 09:55 | |
| *** yassou has quit IRC | 09:55 | |
| *** thorst has quit IRC | 09:56 | |
| *** rmart04 has joined #openstack-nova | 09:59 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| *** cdent has joined #openstack-nova | 10:01 | |
| *** eliqiao has joined #openstack-nova | 10:01 | |
| *** sdague has joined #openstack-nova | 10:03 | |
| *** sdake has quit IRC | 10:03 | |
| *** rook-lappy has quit IRC | 10:05 | |
| *** fawadkhaliq has quit IRC | 10:09 | |
| *** ihrachys has joined #openstack-nova | 10:10 | |
| *** atuvenie has joined #openstack-nova | 10:10 | |
| *** fawadkhaliq has joined #openstack-nova | 10:10 | |
| *** mkrcmari__ has joined #openstack-nova | 10:13 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Handle pre-migration flavor creation failures in the crusty old API https://review.openstack.org/299625 | 10:13 |
| *** e0ne has quit IRC | 10:15 | |
| *** atuvenie has quit IRC | 10:17 | |
| *** mvk_ has quit IRC | 10:17 | |
| *** e0ne has joined #openstack-nova | 10:17 | |
| *** hoangcx has quit IRC | 10:18 | |
| *** fawadkhaliq has quit IRC | 10:18 | |
| *** fawadkhaliq has joined #openstack-nova | 10:19 | |
| *** e0ne has quit IRC | 10:20 | |
| *** EinstCra_ has quit IRC | 10:21 | |
| *** e0ne has joined #openstack-nova | 10:21 | |
| *** e0ne has quit IRC | 10:22 | |
| *** e0ne has joined #openstack-nova | 10:23 | |
| *** atuvenie has joined #openstack-nova | 10:23 | |
| *** yassine has joined #openstack-nova | 10:23 | |
| *** e0ne has quit IRC | 10:24 | |
| *** yassine is now known as Guest23687 | 10:24 | |
| *** Guest23687 is now known as yassou | 10:25 | |
| *** fawadkhaliq has quit IRC | 10:25 | |
| *** mvk_ has joined #openstack-nova | 10:28 | |
| *** ociuhandu has quit IRC | 10:30 | |
| *** smatzek has joined #openstack-nova | 10:32 | |
| *** mkrcmari__ has quit IRC | 10:32 | |
| *** atuvenie has quit IRC | 10:34 | |
| *** stevemar has joined #openstack-nova | 10:35 | |
| *** fawadkhaliq has joined #openstack-nova | 10:39 | |
| *** stevemar has quit IRC | 10:39 | |
| *** ivase_ has quit IRC | 10:41 | |
| *** permalac has quit IRC | 10:44 | |
| liunian_ | I'm really puzzled ,why not nova provide snapshot function？ | 10:46 |
| *** fawadkhaliq has quit IRC | 10:49 | |
| ndipanov | liunian_, it does provide several ways to do it | 10:49 |
| ndipanov | liunian_, see nova image-create cli command | 10:50 |
| liunian_ | yes， but i think ,it is too complicated。 | 10:51 |
| *** salv-orlando has joined #openstack-nova | 10:51 | |
| liunian_ | https://blueprints.launchpad.net/nova/+spec/nova-snapshot it is my bp。 | 10:51 |
| *** doug-fish has joined #openstack-nova | 10:51 | |
| *** baoli has joined #openstack-nova | 10:52 | |
| *** dmellado_ is now known as dmellado | 10:52 | |
| *** thorst has joined #openstack-nova | 10:54 | |
| *** baoli has quit IRC | 10:57 | |
| markus_z | tonyb: regarding libvirt test job: I pushed https://review.openstack.org/#/c/299840/ Could you have a look please? | 10:59 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: pci: Add utility method for getting the MAC addr https://review.openstack.org/262585 | 10:59 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: neutron: Update the port with a MAC address for PFs https://review.openstack.org/262586 | 10:59 |
| markus_z | tonyb: The test job is merged to the "project.config" but a test run fails: http://logs.openstack.org/20/299320/2/experimental/gate-tempest-dsvm-nova-libvirt-kvm-apr/ee9c4ef/logs/devstacklog.txt.gz | 11:00 |
| *** haomaiwang has quit IRC | 11:01 | |
| *** ociuhandu has joined #openstack-nova | 11:01 | |
| *** thorst has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-nova | 11:01 | |
| *** moshele has quit IRC | 11:03 | |
| *** moshele has joined #openstack-nova | 11:03 | |
| ndipanov | moshele, when you get a sec: https://review.openstack.org/#/c/262585/ | 11:05 |
| *** liunian_ has quit IRC | 11:09 | |
| *** cdent has quit IRC | 11:11 | |
| *** Kevin_Zheng has quit IRC | 11:12 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add "get_job_info" to Guest's object https://review.openstack.org/250740 | 11:12 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add method to configure max downtime when migrating https://review.openstack.org/250739 | 11:12 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add "abort_job" to Guest's object https://review.openstack.org/250738 | 11:12 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add method "migrate" to Guest's object https://review.openstack.org/250365 | 11:12 |
| *** fhorvath has quit IRC | 11:12 | |
| openstackgerrit | YaoZheng proposed openstack/nova: After migrate in-use volume the BDM information lost https://review.openstack.org/288433 | 11:13 |
| *** cdent has joined #openstack-nova | 11:14 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [extensions] cache discovered extensions https://review.openstack.org/287131 | 11:16 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add support of caching https://review.openstack.org/299860 | 11:16 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Discover extensions from contrib dir with pkgutil https://review.openstack.org/299861 | 11:16 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Cache discover_version method https://review.openstack.org/299862 | 11:16 |
| *** e0ne has joined #openstack-nova | 11:17 | |
| *** e0ne has quit IRC | 11:18 | |
| *** unicell has quit IRC | 11:22 | |
| *** jlvillal_ has joined #openstack-nova | 11:22 | |
| *** adreznec has quit IRC | 11:22 | |
| *** tbachman has quit IRC | 11:22 | |
| *** allen_gao has quit IRC | 11:22 | |
| *** jlvillal has quit IRC | 11:22 | |
| *** dhellmann has quit IRC | 11:22 | |
| *** allen_ga- has joined #openstack-nova | 11:22 | |
| *** dhellmann_ has joined #openstack-nova | 11:22 | |
| *** unicell has joined #openstack-nova | 11:23 | |
| *** adreznec has joined #openstack-nova | 11:23 | |
| *** e0ne has joined #openstack-nova | 11:23 | |
| *** jlvillal_ is now known as jlvillal | 11:23 | |
| *** jlvillal is now known as Guest87578 | 11:24 | |
| *** claudiub has joined #openstack-nova | 11:25 | |
| *** Guest40396 is now known as b3nt_pin | 11:26 | |
| *** e0ne has quit IRC | 11:26 | |
| *** dhellmann_ is now known as dhellmann | 11:26 | |
| *** b3nt_pin is now known as beagles | 11:27 | |
| *** tbachman has joined #openstack-nova | 11:30 | |
| *** vladikr has joined #openstack-nova | 11:32 | |
| *** rtheis has joined #openstack-nova | 11:33 | |
| *** e0ne has joined #openstack-nova | 11:35 | |
| *** vladikr has quit IRC | 11:39 | |
| *** thorst has joined #openstack-nova | 11:41 | |
| *** e0ne has quit IRC | 11:41 | |
| *** haukebruno has quit IRC | 11:42 | |
| cdent | sdague: if you've not already started I'll go ahead and see about intergrating microversion-parse into nova | 11:43 |
| *** zenoway has quit IRC | 11:44 | |
| *** zenoway has joined #openstack-nova | 11:45 | |
| *** zenoway has quit IRC | 11:49 | |
| *** vladikr has joined #openstack-nova | 11:52 | |
| *** rfolco has joined #openstack-nova | 11:53 | |
| openstackgerrit | sahid proposed openstack/nova: virt: reserved number of mempages on compute host https://review.openstack.org/292499 | 11:53 |
| *** sahid has joined #openstack-nova | 11:57 | |
| *** e0ne has joined #openstack-nova | 11:59 | |
| *** zenoway has joined #openstack-nova | 11:59 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [extensions] cache discovered extensions https://review.openstack.org/287131 | 11:59 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Cache discover_version method https://review.openstack.org/299862 | 11:59 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add support of caching https://review.openstack.org/299860 | 11:59 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Discover extensions from contrib dir with pkgutil https://review.openstack.org/299861 | 12:00 |
| *** e0ne has quit IRC | 12:00 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** moshele has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-nova | 12:01 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [extensions] cache discovered extensions https://review.openstack.org/287131 | 12:03 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Cache discover_version method https://review.openstack.org/299862 | 12:03 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add support of caching https://review.openstack.org/299860 | 12:03 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Discover extensions from contrib dir with pkgutil https://review.openstack.org/299861 | 12:03 |
| *** vladikr has quit IRC | 12:05 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: centralize section "cloudpipe" https://review.openstack.org/272949 | 12:09 |
| *** aysyd has joined #openstack-nova | 12:09 | |
| *** ayoung has quit IRC | 12:10 | |
| *** sdague_ has joined #openstack-nova | 12:11 | |
| *** e0ne has joined #openstack-nova | 12:11 | |
| *** aysyd has quit IRC | 12:11 | |
| *** sfinucan has quit IRC | 12:11 | |
| *** efried has joined #openstack-nova | 12:12 | |
| *** e0ne has quit IRC | 12:13 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: centralize section: "crypto" https://review.openstack.org/273916 | 12:14 |
| *** aysyd has joined #openstack-nova | 12:16 | |
| *** ducttape_ has joined #openstack-nova | 12:16 | |
| *** pradk has joined #openstack-nova | 12:16 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove glance deprecated config https://review.openstack.org/299535 | 12:17 |
| openstackgerrit | sahid proposed openstack/os-vif: test: use real UUID in all UUID fields https://review.openstack.org/299404 | 12:17 |
| *** sfinucan has joined #openstack-nova | 12:18 | |
| *** vladikr has joined #openstack-nova | 12:18 | |
| *** EinstCrazy has joined #openstack-nova | 12:19 | |
| *** tbachman has quit IRC | 12:19 | |
| *** moshele has joined #openstack-nova | 12:19 | |
| *** dave-mccowan has joined #openstack-nova | 12:20 | |
| *** rodrigods has quit IRC | 12:21 | |
| *** rodrigods has joined #openstack-nova | 12:22 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove the ability to disable v2.1 https://review.openstack.org/299893 | 12:26 |
| *** baoli has joined #openstack-nova | 12:27 | |
| *** baoli has quit IRC | 12:27 | |
| *** baoli has joined #openstack-nova | 12:27 | |
| moshele | ndipanov: done | 12:28 |
| openstackgerrit | Sean Dague proposed openstack/nova: WIP: warn if using the legacy v2 api stack https://review.openstack.org/299895 | 12:31 |
| *** ivase has joined #openstack-nova | 12:31 | |
| *** haomaiwang has quit IRC | 12:32 | |
| *** ducttape_ has quit IRC | 12:33 | |
| *** baoli_ has joined #openstack-nova | 12:34 | |
| *** stevemar has joined #openstack-nova | 12:36 | |
| ndipanov | moshele, thanks | 12:36 |
| *** ivase_ has joined #openstack-nova | 12:36 | |
| *** ivase has quit IRC | 12:36 | |
| *** raildo-afk is now known as raildo | 12:36 | |
| *** p01nt3r75 has quit IRC | 12:37 | |
| *** baoli has quit IRC | 12:37 | |
| *** ivase has joined #openstack-nova | 12:38 | |
| *** daemontool_ has joined #openstack-nova | 12:39 | |
| *** tbachman has joined #openstack-nova | 12:39 | |
| *** burgerk has joined #openstack-nova | 12:39 | |
| *** ivase_ has quit IRC | 12:40 | |
| *** delatte has quit IRC | 12:40 | |
| *** stevemar has quit IRC | 12:40 | |
| johnthetubaguy | alaski: are you around to talk about policy things? | 12:41 |
| alaski | johnthetubaguy: yeah | 12:41 |
| johnthetubaguy | alaski: I was thinking more generally about the fine grained policy stuff, like: os_compute_api:servers:create:attach_volume | 12:42 |
| *** daemontool has quit IRC | 12:42 | |
| johnthetubaguy | alaski: I wonder if we want to transition to be more opinionated, so there is something like os_compute_api:supports_volumes that turns on or off a whole load of things | 12:42 |
| johnthetubaguy | alaski: I am really thinking about interoperability | 12:43 |
| johnthetubaguy | its just that corse grained thing, might we want we want to expose to users, a bit like you are proposing | 12:43 |
| sdague_ | johnthetubaguy: honestly, I think we should get it loaded to code first | 12:44 |
| sdague_ | then build a fascility for aliases and groups in policy, so that we can start renaming things in sensible ways | 12:44 |
| alaski | agreed | 12:44 |
| sdague_ | but until that's code annotated, it's going to be a mess to manage | 12:44 |
| johnthetubaguy | yeah, that makes sense | 12:44 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: [WIP] Add console log offset https://review.openstack.org/292473 | 12:45 |
| johnthetubaguy | make upgrade work, so we can then actually go fix things | 12:45 |
| alaski | johnthetubaguy: I like the idea, but it breaks backwards compat because currently deployers can configure things that wouldn't work at a coarse grained level | 12:45 |
| sdague_ | right, exactly | 12:45 |
| *** ZZelle has joined #openstack-nova | 12:45 | |
| johnthetubaguy | alaski: yeah, it would totally need a deprecation cycle and alias stuff, etc | 12:45 |
| sdague_ | it gets made a ton easier if people are only specifying overrides in policy.json | 12:46 |
| sdague_ | and a lot of people are running with empty policy files | 12:46 |
| alaski | johnthetubaguy: yeah. I originally has some of that in my proposal and had to back it out when claudiub pointed out that current policies weren't necessarily expressable like that | 12:46 |
| alaski | sdague_: yep | 12:46 |
| sdague_ | I also think scope wise, policies into code with no change on names, might actually fit in a cycle | 12:47 |
| johnthetubaguy | sdague_: +1 getting most folks onto empty policy files | 12:47 |
| sdague_ | the rest definitely won't | 12:47 |
| *** houming has quit IRC | 12:47 | |
| johnthetubaguy | sdague: yeah, thats sounds about right | 12:47 |
| sdague_ | johnthetubaguy: so, you have a chance to look at the api-ref patches yet? | 12:47 |
| johnthetubaguy | so I totally by just getting the defaults into code, as a step | 12:47 |
| johnthetubaguy | sdague: crud, I did just after the API meeting, then got distracted... arg | 12:48 |
| sdague_ | I think we all have a similar vision of where to get to eventually, we just need to keep the scope narrow enough to make real progress | 12:48 |
| alaski | +1 | 12:48 |
| johnthetubaguy | OK, I wasn't so sure we all have a similar end vision | 12:49 |
| johnthetubaguy | but sounds like I am wrong, sounds like we are shooting in very similar directions | 12:49 |
| *** daemontool__ has joined #openstack-nova | 12:49 | |
| johnthetubaguy | totally agreed getting the defaults into code makes sense as a first step, so we can do the alias thing | 12:49 |
| *** e0ne has joined #openstack-nova | 12:50 | |
| *** dmellado is now known as dmellado|lunch | 12:50 | |
| johnthetubaguy | alaski: is it worth reducing the scope of that spec a little, so it just looks at moving the defaults into code, running gate on an empty policy file? | 12:50 |
| alaski | I think we all agree that we need a better way to express capabilities/endpoints/whatever. we may not yet agree on exactly the form, but first we need a way to even transition | 12:50 |
| sdague_ | alaski: ++ | 12:51 |
| *** e0ne has quit IRC | 12:51 | |
| sdague_ | and nothing about this precludes anything I've heard from folks as expressed end states | 12:51 |
| sdague_ | ok, got to drop for a bit. johnthetubaguy if you could take a look at - https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:wip_api_docs2 would be great | 12:52 |
| johnthetubaguy | I mean it stops the canonical defaults living in a keystone database, but thats no bad thing | 12:52 |
| *** daemontool_ has quit IRC | 12:52 | |
| edleafe | johnthetubaguy: can you remove your -2 from https://review.openstack.org/#/c/282419/ ? | 12:52 |
| sdague_ | johnthetubaguy: right, we always thought that was a bad idea though | 12:52 |
| johnthetubaguy | sdague_: within the Nova discussions certainly | 12:53 |
| *** liusheng has quit IRC | 12:53 | |
| johnthetubaguy | sdague: is this commented out code here on purpose? https://review.openstack.org/#/c/292420/10/api-ref/ext/rest_parameters.py@78 | 12:53 |
| alaski | johnthetubaguy: I can reduce the scope, sure. we lose the ability to build capability exposure on top so I think we want a second spec to add something for that | 12:53 |
| johnthetubaguy | alaski: yeah, I think we want to talk about that separately really | 12:54 |
| *** liusheng has joined #openstack-nova | 12:54 | |
| johnthetubaguy | alaski: once we expose that, its a micro versioned API that lives for ever, I guess, hence me thinking about very course grained things that are consistent across all deployments | 12:54 |
| *** daemontool__ has quit IRC | 12:55 | |
| tbachman | garyk1: you around? | 12:55 |
| alaski | johnthetubaguy: sure. I've very intentionally avoided discussing the API. My spec only deals with what we can glean internally from policy rules | 12:55 |
| johnthetubaguy | sdague_: maybe we ignore https://review.openstack.org/#/c/292420/10/api-ref/ext/rest_parameters.py@78 and fix it up in a follow on patch, given we have a +2 on there | 12:55 |
| tbachman | garyk1: was wondering if you could give your blessings to the latest iteration of: https://review.openstack.org/#/c/288076 | 12:56 |
| *** pushkaru has joined #openstack-nova | 12:56 | |
| alaski | johnthetubaguy: I think to your concern, https://review.openstack.org/#/c/289405/ is already a separate spec for the API side of this | 12:57 |
| *** sdague_ has quit IRC | 12:57 | |
| *** penick has quit IRC | 12:57 | |
| *** edmondsw has joined #openstack-nova | 12:57 | |
| johnthetubaguy | alaski: OK, so I should re-read to see how that other bit helps us | 12:57 |
| *** cdelatte has joined #openstack-nova | 12:57 | |
| *** rlrossit has joined #openstack-nova | 12:58 | |
| *** links has quit IRC | 12:58 | |
| alaski | johnthetubaguy: what I'm proposing is the mechanism that allows us to link policies into a coarse grained definition, but it doesn't get into what definitions we use | 12:58 |
| *** daemontool__ has joined #openstack-nova | 12:59 | |
| *** daemontool__ is now known as daemontool | 13:00 | |
| *** achanda has joined #openstack-nova | 13:00 | |
| *** Goneri has joined #openstack-nova | 13:01 | |
| *** Daisy has joined #openstack-nova | 13:01 | |
| johnthetubaguy | alaski: my gut says get the defaults in code, and work out some alias system for those as a first step, on its own, I am just not sure about doing the coarse grained thing until we have a clear user of it | 13:01 |
| johnthetubaguy | sdague: there is commented out code in the plugin, maybe tidy that up in a follow on patch? | 13:01 |
| *** mdrabe has joined #openstack-nova | 13:04 | |
| *** jaosorior has quit IRC | 13:05 | |
| *** achanda has quit IRC | 13:06 | |
| *** mvk_ has quit IRC | 13:06 | |
| *** jaosorior has joined #openstack-nova | 13:06 | |
| *** Daisy has quit IRC | 13:06 | |
| *** zul has quit IRC | 13:08 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 unit tests https://review.openstack.org/278810 | 13:08 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Fix wrong watermark state during live migration https://review.openstack.org/279616 | 13:08 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Fix wrong watermark state during live migration https://review.openstack.org/279616 | 13:10 |
| *** akshai has joined #openstack-nova | 13:10 | |
| *** rfolco has quit IRC | 13:11 | |
| * alex_xu try to review api-ref patch also, but looks like he need more knowledge about sphinx | 13:12 | |
| *** penick has joined #openstack-nova | 13:13 | |
| *** ctrath has joined #openstack-nova | 13:13 | |
| *** yamahata has joined #openstack-nova | 13:13 | |
| *** liverpooler has quit IRC | 13:15 | |
| *** baoli_ has quit IRC | 13:16 | |
| tbachman | garyk1: thanks! | 13:17 |
| *** salv-orlando has quit IRC | 13:18 | |
| *** rfolco has joined #openstack-nova | 13:19 | |
| *** salv-orlando has joined #openstack-nova | 13:19 | |
| *** mtanino has joined #openstack-nova | 13:20 | |
| *** pwnall has joined #openstack-nova | 13:21 | |
| *** Goneri has quit IRC | 13:22 | |
| *** anush_ has joined #openstack-nova | 13:22 | |
| *** cleong has joined #openstack-nova | 13:22 | |
| *** garyk1 has quit IRC | 13:23 | |
| *** salv-orl_ has joined #openstack-nova | 13:24 | |
| sdague | johnthetubaguy: yeh, at lot of it is logging | 13:25 |
| *** salv-orlando has quit IRC | 13:26 | |
| sdague | johnthetubaguy: there isn't really a great way to conditionally enable that stuff, so right now leaving it there for people to know where the important parts are is probably useful | 13:26 |
| johnthetubaguy | sdague: oh, I see what you mean, I figured there would be a purpose to that | 13:26 |
| *** ccarmack has joined #openstack-nova | 13:27 | |
| *** zul has joined #openstack-nova | 13:27 | |
| sdague | for instance, you can't print debug sphinx, that's all swallowed | 13:29 |
| sdague | you have to use it's logger | 13:29 |
| *** akshai has quit IRC | 13:29 | |
| sdague | and where that logger is in the object model changes depending on context you are in | 13:29 |
| *** tritro has joined #openstack-nova | 13:30 | |
| *** diogogmt has quit IRC | 13:30 | |
| *** akshai has joined #openstack-nova | 13:30 | |
| *** knikolla has joined #openstack-nova | 13:32 | |
| johnthetubaguy | sdague: I wish I knew about that thing when I was looking at the feature classification stuff, seems worth leaving in | 13:32 |
| *** e0ne has joined #openstack-nova | 13:32 | |
| sdague | johnthetubaguy: yeh... there are some hard fought discoveries in this code | 13:33 |
| sdague | I read a lot of sphinx / docutils code, plus asked dhellmann many questions | 13:33 |
| mnaser | whos the best person to bother regarding the objects code in nova | 13:33 |
| mnaser | i've resolved a bug but tests are not failing because the object change so it wants a version update, however, the old version was broken to start with from the first place | 13:34 |
| mnaser | so i'm not sure how to go about doing this | 13:34 |
| rlrossit | mnaser: either dansmith or myself. If I can't answer the question, I can just toss it over to him | 13:34 |
| mnaser | this is regarding patch: https://review.openstack.org/#/c/299708/ | 13:34 |
| rlrossit | mnaser: I'll take a look | 13:34 |
| mnaser | thank you :) | 13:34 |
| *** diogogmt has joined #openstack-nova | 13:35 | |
| *** stevemar has joined #openstack-nova | 13:36 | |
| alex_xu | johnthetubaguy: sdague are we safe to remove extension options https://github.com/openstack/nova/blob/master/nova/api/openstack/__init__.py#L49 ? if yes, I will cook a patch for those. | 13:36 |
| *** Goneri has joined #openstack-nova | 13:37 | |
| dansmith | mnaser: yeah, you can't do that.. also, jaypipes is the person to look at that | 13:37 |
| sdague | alex_xu: I don't know on that one honestly | 13:37 |
| dansmith | mnaser: this floatness came up at some point and I thought he had a solution for it (which might be to just integerify it) | 13:37 |
| sdague | I feel like that probably needs to be an austin conversation | 13:37 |
| johnthetubaguy | alex_xu: hmm, the v2.1 ones I would say yes, not so sure about v2.0 ones, I would rather they died when the v2.0 code died | 13:37 |
| *** mriedem has joined #openstack-nova | 13:37 | |
| johnthetubaguy | sdague: you are right though, its a summit conversation I think | 13:37 |
| sdague | I still think the extensions thing is going to catch a lot of folks by surprise | 13:38 |
| mnaser | dansmith: I see... I will try to get in touch with him to see what would be the ideal solution in that case | 13:38 |
| alex_xu | johnthetubaguy: they are separaed option between v2 and v2.1 I think | 13:38 |
| johnthetubaguy | sdague: so if we have policy being only overrides, it would be an easier call, for sure | 13:38 |
| mnaser | Because all values were returning 0, and now that we have it working with that patch internally, CPU usage is *much* better distributed | 13:38 |
| dansmith | mnaser: I just added him to the review | 13:38 |
| mnaser | Cool, thank you so much :) | 13:38 |
| *** salv-orl_ has quit IRC | 13:38 | |
| sdague | johnthetubaguy: yeh | 13:38 |
| johnthetubaguy | alex_xu: yep, its two different set of variables | 13:38 |
| dansmith | mnaser: which hypervisor? | 13:38 |
| mnaser | dansmith: kvm | 13:38 |
| dansmith | mnaser: okay, well, anyway, punt to jay | 13:39 |
| mnaser | yep, i'll follow up once he replies and hopefully we can get this resolved (and justified to be backported to liberty) | 13:39 |
| mnaser | but that's another discussion :d | 13:39 |
| alex_xu | johnthetubaguy: ok, got it, let's wait after summit | 13:39 |
| *** ametts has joined #openstack-nova | 13:39 | |
| *** mylu has joined #openstack-nova | 13:40 | |
| sdague | dansmith: I updated - https://review.openstack.org/#/c/299625/ with a test | 13:40 |
| dansmith | sdague: I saw, thanks | 13:40 |
| *** stevemar has quit IRC | 13:41 | |
| *** lixiaoy1 has quit IRC | 13:41 | |
| *** salv-orlando has joined #openstack-nova | 13:43 | |
| rlrossit | mnaser: dansmith: I went hunting down the changes from when we had this discussion last time. The commits of interest are https://github.com/openstack/nova/commit/ddb3c4e26f3a908de7836a8fac48360afa6aa1d7 and https://github.com/openstack/nova/commit/87e1d8fab2b5888e711f07001025600fe709aaa7 | 13:43 |
| *** burgerk has quit IRC | 13:44 | |
| *** pushkaru has quit IRC | 13:44 | |
| *** haomaiwang has joined #openstack-nova | 13:44 | |
| dansmith | rlrossit: ah, yeah, thanks.. that's what I thought we did.. integer and *= 100 | 13:44 |
| mnaser | rlrossit you know what, i think the https://github.com/openstack/nova/commit/ddb3c4e26f3a908de7836a8fac48360afa6aa1d7 patch might be missing from liberty | 13:45 |
| dansmith | mnaser: maybe this fixes it for you on an older verson but isn't related? | 13:45 |
| mnaser | (or at least the ubuntu release) | 13:45 |
| *** permalac has joined #openstack-nova | 13:45 | |
| mnaser | because when i was troubleshooting, that code was not there | 13:45 |
| rlrossit | oh yeah, I suppose that wouldn't be in liberty | 13:45 |
| *** cdent has quit IRC | 13:45 | |
| mnaser | dansmith: yep, not a thing in liberty - https://github.com/openstack/nova/blob/stable/liberty/nova/objects/monitor_metric.py | 13:46 |
| rlrossit | mnaser: you want to backport that? | 13:46 |
| *** cdent has joined #openstack-nova | 13:46 | |
| *** klkumar has joined #openstack-nova | 13:46 | |
| mnaser | i will do that later today, i will drop this review and do that | 13:46 |
| openstackgerrit | Cédric LECOMTE proposed openstack/nova: Archive instance_actions and instance_actions_event https://review.openstack.org/299474 | 13:46 |
| dansmith | mnaser: you'll need both if they're not there I think right? | 13:46 |
| mnaser | the first one is there (from august) | 13:47 |
| rlrossit | I think the second was much earlier | 13:47 |
| mnaser | the second one from november was not | 13:47 |
| dansmith | okay | 13:47 |
| mnaser | so just this one -- https://github.com/openstack/nova/commit/ddb3c4e26f3a908de7836a8fac48360afa6aa1d7 | 13:47 |
| rlrossit | mnaser: I'll also rearrange the bug you opened to point to the old bug to make the bugs people happier | 13:48 |
| mnaser | please :) | 13:48 |
| rlrossit | mnaser: should be. you can add me as a reviewer too when you get that backported | 13:48 |
| mnaser | will do, i will abandon this patch | 13:48 |
| rlrossit | (If you didn't know, gerrit can really help you out by just clicking the "Cherry Pick" button on the commit you want to backport) | 13:48 |
| mnaser | is there a wiki page or something that can help with that? | 13:49 |
| mnaser | rlrossit : nevermind, got it - http://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes | 13:50 |
| *** ametts has quit IRC | 13:50 | |
| rlrossit | mnaser: I just got there myself :) | 13:50 |
| *** ayoung has joined #openstack-nova | 13:50 | |
| rlrossit | you beat me to it | 13:50 |
| *** baoli has joined #openstack-nova | 13:50 | |
| mnaser | alright, ill check this out later and take care of it | 13:51 |
| *** xyang1 has joined #openstack-nova | 13:52 | |
| *** esracelik has quit IRC | 13:52 | |
| *** e0ne has quit IRC | 13:52 | |
| *** salv-orlando has quit IRC | 13:52 | |
| *** mtanino has quit IRC | 13:52 | |
| *** Daisy has joined #openstack-nova | 13:53 | |
| *** permalac has quit IRC | 13:53 | |
| *** pushkaru has joined #openstack-nova | 13:53 | |
| *** diogogmt has quit IRC | 13:54 | |
| *** e0ne has joined #openstack-nova | 13:54 | |
| *** pushkaru has quit IRC | 13:54 | |
| *** dmellado|lunch is now known as dmellado | 13:54 | |
| *** ametts has joined #openstack-nova | 13:54 | |
| rlrossit | mriedem: how do you mark a bug as duplicate in LP? | 13:56 |
| *** e0ne has quit IRC | 13:56 | |
| *** mriedem has left #openstack-nova | 13:56 | |
| *** mriedem has joined #openstack-nova | 13:57 | |
| *** ducttape_ has joined #openstack-nova | 13:57 | |
| mriedem | rlrossit: there is a link in the top right | 13:57 |
| rlrossit | ah found it. thanks | 13:57 |
| rlrossit | mriedem: and then if we will be backporting a bug that is in fix released, do we need to say it's still a bug in a different branch, or do we not have to care about that? | 13:59 |
| mriedem | rlrossit: you can nominate it for another series | 13:59 |
| *** anush_ has quit IRC | 13:59 | |
| mriedem | and then someone on the bug team can accept/reject that | 14:00 |
| mriedem | you could also just tag with *-backport-potential | 14:00 |
| rlrossit | I'll do the latter | 14:00 |
| *** mgoddard_ has joined #openstack-nova | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| mriedem | looks like we need to either blacklist more xenapi unit tests from py34 or find some patches that replace mox usage with stub_out | 14:01 |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| *** eharney has joined #openstack-nova | 14:02 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: remove live migrate workaround for an unsupported ver https://review.openstack.org/299489 | 14:02 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduces object to handle domain xml migration https://review.openstack.org/299490 | 14:02 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: move graphic/serial consoles check to pre_live_migration https://review.openstack.org/254709 | 14:02 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add "get_job_info" to Guest's object https://review.openstack.org/250740 | 14:02 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add method to configure max downtime when migrating https://review.openstack.org/250739 | 14:02 |
| sdague | honestly, blacklisting xenapi seem like the better option | 14:02 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add "abort_job" to Guest's object https://review.openstack.org/250738 | 14:02 |
| *** salv-orlando has joined #openstack-nova | 14:02 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add method "migrate" to Guest's object https://review.openstack.org/250365 | 14:02 |
| *** anush_ has joined #openstack-nova | 14:02 | |
| *** achanda has joined #openstack-nova | 14:02 | |
| *** cdent_ has joined #openstack-nova | 14:03 | |
| mriedem | sdague: because it's easier or because we know stub_out has it's own issues with staticmethods? | 14:03 |
| *** mgoddard has quit IRC | 14:03 | |
| *** cdent has quit IRC | 14:03 | |
| *** cdent_ has quit IRC | 14:04 | |
| dansmith | mriedem: are you seeing the problem in xen tests, or you know that xen tests are the problem? | 14:04 |
| sdague | mriedem: because it's easier, and the xenapi tests use mox in non trivial ways | 14:04 |
| dansmith | because the stub out staticmethod thing manifests somewhere other than the test that does the problematic behavior | 14:04 |
| mriedem | http://logs.openstack.org/25/299625/2/check/gate-nova-python34-db/75276a4/console.html#_2016-03-31_10_32_46_710 | 14:04 |
| mriedem | nova.tests.unit.virt.xenapi.test_vmops.VMOpsTestCase.test_get_vm_opaque_ref_raises_instance_not_found | 14:04 |
| *** cdent has joined #openstack-nova | 14:04 | |
| mriedem | dansmith: failed on your top of series flavor migration chnage | 14:04 |
| mriedem | *change | 14:04 |
| * dansmith waits an eternity for it to load | 14:05 | |
| *** rook-lappy has joined #openstack-nova | 14:05 | |
| dansmith | ah, lovely | 14:06 |
| *** eharney has quit IRC | 14:06 | |
| dansmith | that one is not hard to replace with mock, but yeah | 14:06 |
| *** keekz_ is now known as keekz | 14:07 | |
| *** penick has quit IRC | 14:07 | |
| dansmith | I feel like just increasing the size of the blacklist isn't really helping the problem, because later if we need to really start working on py3 compat, we'll just have a higher mountain to climb | 14:07 |
| dansmith | but I also don't want to work on it muself | 14:07 |
| mriedem | i have a separate question | 14:08 |
| *** sdague has quit IRC | 14:08 | |
| *** achanda has quit IRC | 14:08 | |
| mriedem | did anyone know that swap-volume's api has been broken since 2013? | 14:08 |
| dansmith | like, I wonder what the py3 test is getting us if we're just adding to the blacklist any time something fails | 14:08 |
| *** sdague has joined #openstack-nova | 14:08 | |
| mriedem | swap-volume is the API that's been broken at least twice :( | 14:09 |
| *** cdent has quit IRC | 14:09 | |
| *** akshai has quit IRC | 14:10 | |
| * dansmith doesn't know anything about it | 14:10 | |
| mriedem | https://review.openstack.org/#/c/299715/ | 14:10 |
| mriedem | basically, it's been hacked up so badly to fit cinder's volume migration scenario, that it's really an admin-only API for nova now | 14:10 |
| mriedem | because it totally sharts itself for non-admins | 14:10 |
| *** akshai has joined #openstack-nova | 14:11 | |
| dansmith | wow, those are some huge except handlers you have there | 14:12 |
| *** rfolco has quit IRC | 14:12 | |
| *** coreywright has quit IRC | 14:12 | |
| mriedem | latent, let those mellow | 14:12 |
| dansmith | heh | 14:13 |
| bauzas | p!spy | 14:14 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove the ability to disable v2.1 https://review.openstack.org/299893 | 14:16 |
| openstackgerrit | Sean Dague proposed openstack/nova: WIP: warn if using the legacy v2 api stack https://review.openstack.org/299895 | 14:16 |
| tbachman | p!spy | 14:17 |
| *** burgerk has joined #openstack-nova | 14:18 | |
| edleafe | ok, can someone point me to a link that explains the 'p!spy' usage? | 14:19 |
| mriedem | so vish added swap-volume way back in the day | 14:19 |
| danpb | mriedem: dansmith the length of the py34 blacklist has reduced from 453 to 216 tests, since it was first enabled, so we seem to be making decent progress despite occassionally adding more tests to the blacklist | 14:19 |
| *** cdent has joined #openstack-nova | 14:19 | |
| mriedem | then cinder started calling it as part of their volume migration scenario, | 14:19 |
| mriedem | and added a callback to cinder from swap-volume to tell cinder that the operation was done | 14:19 |
| mriedem | which is admin-only | 14:19 |
| *** cdent has quit IRC | 14:20 | |
| mriedem | so that broke swap-volume for non-admin useres | 14:20 |
| dansmith | danpb: okay | 14:20 |
| mriedem | then later, they also removed swap-volume doing explicit detach/attach calls to cinder, so it only works via the admin-only api call to cinder | 14:20 |
| mriedem | that's api break the second | 14:20 |
| mriedem | effectively making swap-volume only useful for the admin-initiated volume migration from the cinder side | 14:20 |
| openstackgerrit | Chuck Carmack proposed openstack/nova: DB API changes for the nova-manage quota_usage_refresh command https://review.openstack.org/299441 | 14:21 |
| mriedem | and is not useful at all when initiated from nova, as admin or user | 14:21 |
| *** hogepodge has joined #openstack-nova | 14:21 | |
| *** eharney has joined #openstack-nova | 14:21 | |
| mriedem | DuncanT: so nova should probably not only make swap-volume an admin-only API, it should probably fail in the compute API if the old volume's migration_status is not 'migrating', right? | 14:21 |
| *** rook-lappy has quit IRC | 14:22 | |
| mriedem | DuncanT: because that's what cinder is validating when nova calls migrate_volume_completion | 14:22 |
| johnthetubaguy | mriedem: dansmith: I have a feeling sfinucan might fancy helping create a extra specs object, a bit like image_meta and props, do we want to hold that kind of thing till after the summit? | 14:22 |
| johnthetubaguy | s/extra/flavor extra/ | 14:23 |
| mriedem | johnthetubaguy: is it a new spec? | 14:23 |
| sahid | markus_z: can you remind me why you want to reuse same ports instead of just release ports when we destroy an instance here: https://review.openstack.org/#/c/275800/ ? | 14:23 |
| mriedem | or specless bp with just lots of conversion work | 14:23 |
| johnthetubaguy | mriedem: well image_meta snook in as part of mitaka-objects, but it feels like a separate specless bp to me | 14:23 |
| johnthetubaguy | yeah, the later | 14:24 |
| dansmith | mriedem: definitely something needing discussion, IMHO | 14:24 |
| dansmith | I do not think it's just a specless bit of mindless conversion | 14:24 |
| *** cdent has joined #openstack-nova | 14:24 | |
| mriedem | if it needs discussion then it needs a spec | 14:24 |
| mriedem | and it waits til after the summit | 14:24 |
| dansmith | image properties is a lot more straightforward I tthink | 14:24 |
| johnthetubaguy | dansmith: mriedem: OK, that fair enough, I don't really know the true horrors living in there | 14:25 |
| johnthetubaguy | I was assuming its a me too thing like image_meta | 14:25 |
| *** gongysh has quit IRC | 14:25 | |
| *** coreywright has joined #openstack-nova | 14:25 | |
| dansmith | johnthetubaguy: it's also a thing that would have to be serialized in the db, inside another object which we already serialize, etc | 14:26 |
| sahid | dansmith: why do you think it's lot more complicated ? | 14:26 |
| *** rfolco has joined #openstack-nova | 14:26 | |
| dansmith | johnthetubaguy: much more work than just the imagemeta object which is used for parsing metadata from glance and a few things from sysmeta | 14:26 |
| dansmith | sahid: because ^ | 14:27 |
| *** haukebruno has joined #openstack-nova | 14:27 | |
| sahid | hum that is because we will have to serialize that object inside the instance object ? | 14:28 |
| dansmith | no, it's serialized inside of flavor | 14:28 |
| mriedem | flavor extra specs would be serialized in flavor which is serialized in instance | 14:28 |
| *** links has joined #openstack-nova | 14:28 | |
| johnthetubaguy | dansmith: hmm, and the API sets stuff, etc, there are some details we should straighten out there | 14:28 |
| sahid | i see | 14:28 |
| dansmith | mriedem: flavor is not serialized inside flavor in the db | 14:29 |
| *** e0ne has joined #openstack-nova | 14:29 | |
| dansmith | mriedem: it is over the wire though | 14:29 |
| mriedem | dansmith: flavor extra specs you mean? | 14:29 |
| *** e0ne has quit IRC | 14:29 | |
| dansmith | mriedem: sorry, flavor is not serialized inside instance in the database | 14:29 |
| mriedem | right, it's in instance_extra | 14:30 |
| mriedem | but instance.flavor | 14:30 |
| johnthetubaguy | yeah, old and new, etc | 14:30 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove glance deprecated config https://review.openstack.org/299535 | 14:30 |
| johnthetubaguy | sfinucan: did you get all that ^ | 14:31 |
| openstackgerrit | Dan Smith proposed openstack/nova: Migrate compute node resource information to Inventory objects https://review.openstack.org/279313 | 14:31 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add Service.get_minimum_version_multi() for multiple binaries https://review.openstack.org/279312 | 14:31 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make compute_node_statistics() use new schema https://review.openstack.org/286931 | 14:31 |
| openstackgerrit | Dan Smith proposed openstack/nova: Move resource provider staticmethods to proxies https://review.openstack.org/299526 | 14:31 |
| rlrossit | dansmith: just a thought as I'm working through this frustration on the FixedIP dict compat stuff... do we want to make a plan to (at some point) migrate over the network api to use the existing versioned objects? | 14:32 |
| sfinucan | johnthetubaguy: Reading through it now | 14:32 |
| rlrossit | these divided dicts/objects are very frustrating to work with | 14:32 |
| dansmith | rlrossit: I'm not sure what you mean | 14:32 |
| mriedem | rlrossit: the nova.network.model stuff you mean? | 14:33 |
| rlrossit | mriedem: yeah | 14:33 |
| mriedem | beagles had a thing up for that awhile back | 14:33 |
| rlrossit | and converting over the dicting in https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py | 14:33 |
| *** anush_ has quit IRC | 14:34 | |
| markus_z | sahid: Let me refresh the context in my head | 14:34 |
| *** Jeffrey4l_ has quit IRC | 14:35 | |
| *** Jeffrey4l_ has joined #openstack-nova | 14:35 | |
| *** nkrinner has quit IRC | 14:35 | |
| mriedem | rlrossit: dansmith was doing some of that a couple of releases ago, it gets weird though | 14:35 |
| dansmith | yeah, it's really ugly | 14:36 |
| mriedem | rlrossit: because the nova.objects.* for networks are modeled on nova-network, | 14:36 |
| dansmith | I did the networkrequest work which makes it cleaner over the wire | 14:36 |
| mriedem | and don't fit well for neutron | 14:36 |
| rlrossit | well... so is dictcompat stuff | 14:36 |
| mriedem | rlrossit: i don't tihnk you understand | 14:36 |
| mriedem | the level of fugly | 14:36 |
| rlrossit | so I'm guessing these are two conflicting formats that will never be able to be settled until n-net dies? | 14:37 |
| markus_z | sahid: IIRC it's because a multiple rebuild of the same instance would otherwise cause a port exhaustion | 14:37 |
| mriedem | the versioned network objects we have today are more for nova-network based on our nova db tables for nova-network | 14:37 |
| mriedem | anything we use them for for the neutron api is just shoe-horning that in | 14:37 |
| dansmith | things like FloatingIP and FixedIP are yeah | 14:37 |
| rlrossit | I actually don't think we use them in the neutron api stuff at all | 14:38 |
| rlrossit | unless I'm forgetting someting | 14:38 |
| markus_z | sahid: The "_create_serial_console_devices()" method also gets called multiple times in a live-migration which would result in a wrong xml. | 14:39 |
| mriedem | neutronv2.api really just uses network and network_request | 14:39 |
| *** READ10 has joined #openstack-nova | 14:39 | |
| markus_z | sahid: That's the things I remember. Would another approach make more sense? | 14:39 |
| rlrossit | mriedem: so this appears to be a can of worms? I probably don't want to look at it then until I finish the dict compat stuff | 14:40 |
| rlrossit | one level of frustration at a time | 14:40 |
| *** jpomeroy has joined #openstack-nova | 14:41 | |
| sahid | markus_z: yes, about the rebuild thing, so for libvirt we use something like destroy/spawn which is basically same process for hard_reboot | 14:41 |
| mriedem | rlrossit: yeah i'd probably avoid that can of worms for now | 14:41 |
| sahid | i think we should just to release ports when we destroy | 14:41 |
| dansmith | rlrossit: it's like a ticking can of worms | 14:42 |
| sahid | in _create_serial_console_devices what is exactly the problem ? can't we just look t the xml and see if serial consoles are already defined ? | 14:42 |
| *** mvk_ has joined #openstack-nova | 14:42 | |
| *** gongysh has joined #openstack-nova | 14:42 | |
| rlrossit | dansmith: ticking? does that mean if I pick it up it will blow up and cover me in worm shrapnel? | 14:43 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova-specs: Add offset to console log https://review.openstack.org/300006 | 14:43 |
| sahid | i'm not sure it's the good solution to add all of this code just to keep using same ports whereas we can just release them and acquire them when we need | 14:44 |
| dansmith | rlrossit: it might yes, and worm shrapnel is gross | 14:44 |
| markus_z | sahid: I think we can. We have to, to be honest. The xml must be updated to avoid a port conflict on the target host. | 14:44 |
| sahid | then your patch can't be backported, and if you just change where we release ports we can probably backport it | 14:44 |
| *** e0ne has joined #openstack-nova | 14:45 | |
| markus_z | sahid: yeah, I inherited it and the original author is not available anymore. | 14:45 |
| rlrossit | dansmith: so now that (I think) I've found my problems with these actually dicts vs. objects things, it looks like the changes in https://review.openstack.org/#/c/267768/ won't be necessary | 14:45 |
| *** jpomeroy has left #openstack-nova | 14:45 | |
| sahid | markus_z: i see, so do you want to continue to work on it or do you want me to jump on ? | 14:45 |
| bauzas | dansmith: have you seen my comment on the havana migration creating old-style flavors ? | 14:45 |
| rlrossit | dansmith: but... do we still want me to at least keep that patch around for the unit tests it adds? they're good for coverage at least I think | 14:46 |
| *** jlanoux_ has quit IRC | 14:46 | |
| dansmith | rlrossit: yeah | 14:46 |
| dansmith | bauzas: yeah, I replied right? | 14:46 |
| markus_z | sahid: I wouldn't say no if you want to take over :) | 14:46 |
| bauzas | heh | 14:46 |
| bauzas | dansmith: I'm lagging today | 14:46 |
| dansmith | bauzas: I'm not sure what you're really saying there, other than that you like FIXME better than NOTE and that you agree the havana migration is ugly | 14:47 |
| sahid | markus_z: ok no problem i take it | 14:47 |
| *** andrewbogott has quit IRC | 14:47 | |
| *** andrewbogott has joined #openstack-nova | 14:47 | |
| markus_z | sahid: That's really nice, thank you! I set a workflow -1 and when you patches are ready I abandon mine. | 14:48 |
| markus_z | *your | 14:48 |
| sdague | on the realm of things to delete - https://review.openstack.org/#/c/299535 | 14:48 |
| sdague | cleans up glance a bit before we start making new changes | 14:48 |
| bauzas | dansmith: sorry, my point was that maybe we should see how to remove the havana migration script rather than doing that kind of online migration in test.py | 14:48 |
| sahid | markus_z: cool | 14:48 |
| dansmith | bauzas: right, we definitely have to do that, but it's a bigger effort | 14:48 |
| *** links has quit IRC | 14:48 | |
| bauzas | dansmith: that's a big thing ? I maybe misunderstood that | 14:49 |
| *** wuhg has quit IRC | 14:49 | |
| *** links has joined #openstack-nova | 14:49 | |
| dansmith | bauzas: yeah, the implications of removing that from our first migration have wide-reaching impacts | 14:49 |
| *** lyarwood has quit IRC | 14:49 | |
| dansmith | bauzas: all of our docs assume those default flavors are there, for example, as does devstack, etc | 14:49 |
| bauzas | dansmith: I see, the main problem is that devstack/tempest users wait for a flavor #42 for m1.tiny as an example ? | 14:51 |
| *** jaosorior_ has joined #openstack-nova | 14:52 | |
| sdague | the gate will be fine | 14:52 |
| sdague | but the docs are a huge issue | 14:52 |
| *** jaosorior has quit IRC | 14:53 | |
| bauzas | I see | 14:53 |
| sdague | because nova has been this way for litterally ever | 14:53 |
| dansmith | bauzas: right, tempest actually creates its own flavors | 14:53 |
| dansmith | sdague: although, grenade will fail right? it uses the default flavors no? | 14:53 |
| sdague | dansmith: yeh, it does today I think | 14:53 |
| dansmith | for the test instances | 14:53 |
| dansmith | yeah | 14:53 |
| bauzas | okay, this is the problem right ? https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/216_havana.py#L88 | 14:54 |
| *** jwcroppe has quit IRC | 14:54 | |
| sdague | though, the reality is we're going to create a few base flavors in devstack anyway | 14:54 |
| *** jwcroppe has joined #openstack-nova | 14:54 | |
| dansmith | sdague: yep, and if we do that then grenade should be okay | 14:55 |
| sdague | if they aren't in nova. The only reason we don't also download an image, is the time is too long | 14:55 |
| dansmith | I gotta run out for a bit, back in about an hour | 14:55 |
| *** gongysh has quit IRC | 14:56 | |
| *** gjayavelu has joined #openstack-nova | 14:57 | |
| *** sdake has joined #openstack-nova | 14:57 | |
| *** mgoddard_ has quit IRC | 14:57 | |
| *** mgoddard has joined #openstack-nova | 14:58 | |
| *** ljxiash has joined #openstack-nova | 14:58 | |
| *** jwcroppe has quit IRC | 14:59 | |
| *** mtanino has joined #openstack-nova | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** pece has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| *** e0ne has quit IRC | 15:01 | |
| mriedem | sdague: on https://review.openstack.org/#/c/299535/ why not just set glance.api_servers in the ConfFixture so we don't have to copy that all over the place? | 15:02 |
| *** ametts has quit IRC | 15:02 | |
| *** rdopiera has quit IRC | 15:04 | |
| *** stevemar has joined #openstack-nova | 15:06 | |
| *** jistr has quit IRC | 15:07 | |
| *** ametts has joined #openstack-nova | 15:07 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from FixedIP https://review.openstack.org/265951 | 15:07 |
| *** mdrabe has quit IRC | 15:07 | |
| *** mtanino has quit IRC | 15:09 | |
| *** jistr has joined #openstack-nova | 15:09 | |
| *** mtanino has joined #openstack-nova | 15:11 | |
| *** salv-orlando has quit IRC | 15:11 | |
| *** andymaier has joined #openstack-nova | 15:12 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Config options: Centralize consoleauth options https://review.openstack.org/293118 | 15:13 |
| ndipanov | mriedem, https://review.openstack.org/#/c/262585/ <-- this one should be ready now | 15:14 |
| mriedem | looking | 15:14 |
| sdague | mriedem: because it would be good to know when we need it | 15:15 |
| sdague | instead of just massively papering over it | 15:15 |
| mriedem | sdague: i figured you'd say something like that | 15:15 |
| doffm | So the configdrive contains instance metadata. If we modify instance metadata, and then reboot or perform a cold migration... do we expect the config drive to get rebuilt? | 15:16 |
| mriedem | ndipanov: that series is getting hammered with job timeouts on vexxhost | 15:16 |
| doffm | Or is config drive a one time... first time you launch... thing. | 15:16 |
| mriedem | ndipanov: you must be jinxed | 15:16 |
| *** zz_dimtruck is now known as dimtruck | 15:16 | |
| *** jlanoux has joined #openstack-nova | 15:17 | |
| mriedem | sdague: if you remove the api_insecure=False from self.flags i'll +2 it | 15:17 |
| *** klkumar has quit IRC | 15:17 | |
| *** cdent has left #openstack-nova | 15:18 | |
| *** cdent has joined #openstack-nova | 15:18 | |
| *** links has quit IRC | 15:19 | |
| *** rk4n has quit IRC | 15:19 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove glance deprecated config https://review.openstack.org/299535 | 15:20 |
| sdague | mriedem: done | 15:21 |
| *** rk4n has joined #openstack-nova | 15:21 | |
| *** mdrabe has joined #openstack-nova | 15:21 | |
| *** armax_ has joined #openstack-nova | 15:22 | |
| alaski | doffm: I would only expect it to get rebuilt for 'rebuild'. But I don't know what the intent was | 15:23 |
| *** jaosorior_ has quit IRC | 15:24 | |
| *** vks has joined #openstack-nova | 15:24 | |
| *** ayoung has quit IRC | 15:24 | |
| doffm | alaski: I can't find anywhere where it would be re-built on reboot. Currently we attempt to rebuild it when doing a migration or resize. (libvirt) | 15:25 |
| doffm | I'm guessing that wasn't intended though. | 15:25 |
| *** armax has quit IRC | 15:25 | |
| *** armax_ is now known as armax | 15:25 | |
| *** ayoung has joined #openstack-nova | 15:25 | |
| alaski | yeah. there may be a reason it needs to be rebuilt, but I doubt the intention was to change what's in there at all | 15:26 |
| *** mjura has quit IRC | 15:27 | |
| alaski | I don't know who would have knowledge around that | 15:27 |
| *** jistr has quit IRC | 15:28 | |
| *** jistr has joined #openstack-nova | 15:28 | |
| *** jdurgin1 has joined #openstack-nova | 15:31 | |
| mriedem | ndipanov: for sriov-v pf, is it required that the port is created in neutron first and then provided to nova when booting the server? | 15:31 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [extensions] cache discovered extensions https://review.openstack.org/287131 | 15:32 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Cache discover_version method https://review.openstack.org/299862 | 15:32 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add support of caching https://review.openstack.org/299860 | 15:32 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Discover extensions from contrib dir with pkgutil https://review.openstack.org/299861 | 15:32 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [tests] initialize client object inside setUp https://review.openstack.org/300032 | 15:32 |
| *** dimtruck is now known as zz_dimtruck | 15:34 | |
| ndipanov | mriedem, I think that's true for any sriov port | 15:35 |
| ndipanov | mriedem, you can't pass vnic-type via --nic | 15:35 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: remove glance deprecated config https://review.openstack.org/299535 | 15:35 |
| mriedem | ndipanov: ok, that's what i thought | 15:35 |
| mriedem | ndipanov: left comments on the top patch, i think we should only call that populate mac addr method in the request.port_id case | 15:36 |
| vks | why openstack agents can't access DB directly , instead agents do a RPC to plugin to get any additional data ? | 15:36 |
| mriedem | just to tighten it up a bit | 15:36 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Config options: centralize common console options https://review.openstack.org/222113 | 15:36 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Config options: make console a group https://review.openstack.org/300036 | 15:36 |
| *** markvoelker has joined #openstack-nova | 15:37 | |
| *** e0ne has joined #openstack-nova | 15:37 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: [WIP] Add console log offset https://review.openstack.org/292473 | 15:38 |
| *** tritro has quit IRC | 15:39 | |
| *** e0ne has quit IRC | 15:40 | |
| *** markvoelker has quit IRC | 15:40 | |
| *** markvoelker_ has joined #openstack-nova | 15:41 | |
| *** tobe has joined #openstack-nova | 15:43 | |
| *** efoley has joined #openstack-nova | 15:44 | |
| *** e0ne has joined #openstack-nova | 15:44 | |
| *** Swami has joined #openstack-nova | 15:44 | |
| *** e0ne has quit IRC | 15:46 | |
| *** EinstCrazy has quit IRC | 15:47 | |
| *** e0ne has joined #openstack-nova | 15:47 | |
| doffm | gibi: So i'm currently looking at: https://review.openstack.org/#/c/299045/ and hanrong has probably correctly pointed out that, when migrating, instances shouldn't have affinity with themselves. | 15:48 |
| doffm | gibi: I can't find anywhere in the soft-affinity code to take this in to account. Do you know if its done somewhere? | 15:48 |
| *** tobe has quit IRC | 15:48 | |
| *** markvoelker_ has quit IRC | 15:50 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Make compute_node_statistics() use new schema https://review.openstack.org/286931 | 15:51 |
| *** markvoelker has joined #openstack-nova | 15:51 | |
| gibi | doffm: I don't know top of my head, I have to look into a bit. However if the problem is reproducible then we can be pretty sure that we don't have such a code in the code patch | 15:52 |
| gibi | s/patch/path/ | 15:52 |
| migi | sdague: hi | 15:52 |
| *** cdelatte has quit IRC | 15:52 | |
| migi | sdague: I have question around your change: https://review.openstack.org/#/c/283261/ | 15:52 |
| *** klkumar has joined #openstack-nova | 15:53 | |
| *** gszasz has quit IRC | 15:53 | |
| *** eharney has quit IRC | 15:53 | |
| *** zz_dimtruck is now known as dimtruck | 15:54 | |
| *** Guest87578 is now known as jlvillal | 15:55 | |
| doffm | gibi: I'll investigate some more, and try writing a test to check the affinity weigher. | 15:55 |
| gibi | doffm: hm, as a second thought. if we talk about soft-affinity then we have less problems as even if a VM is soft-affin to itself it stil can be migrated as soft-affinity allows breaking it | 15:55 |
| doffm | Its true Its not such a big problem as hard affinity. | 15:55 |
| *** markvoelker has quit IRC | 15:55 | |
| doffm | But the weights will be incorrect, especially for small server groups. | 15:55 |
| *** jaypipes has joined #openstack-nova | 15:55 | |
| gibi | doffm: probably you are right and the soft-affinity is broken this way as well | 15:56 |
| *** markvoelker has joined #openstack-nova | 15:56 | |
| doffm | Ok, well I'll check. And file a bug if neccessary. | 15:56 |
| *** baoli_ has joined #openstack-nova | 15:56 | |
| *** rmart04 has quit IRC | 15:56 | |
| *** markvoelker_ has joined #openstack-nova | 15:57 | |
| *** baoli has quit IRC | 15:57 | |
| gibi | doffm: thanks | 15:57 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add support of caching https://review.openstack.org/299860 | 15:57 |
| *** e0ne has quit IRC | 15:58 | |
| mriedem | alaski: i suspect that compute:swap_volume in the policy.json is not actually used anywhere, is there an easier way to tell than just checking the authorizer calls in the volume attachment update REST API code? | 15:59 |
| *** mgoddard_ has joined #openstack-nova | 15:59 | |
| mriedem | or just removing it i guess | 15:59 |
| gibi | doffm: basically here https://github.com/openstack/nova/blob/master/nova/scheduler/weights/affinity.py#L49 you have to take into account if the instance being scheduled is on the host and remove that from the calculation | 15:59 |
| *** shangxdy has joined #openstack-nova | 15:59 | |
| *** shangxdy_ has joined #openstack-nova | 15:59 | |
| *** jlanoux_ has joined #openstack-nova | 16:00 | |
| *** eharney has joined #openstack-nova | 16:00 | |
| *** angdraug has joined #openstack-nova | 16:00 | |
| andreykurilin | mriedem sdague: hi! Can you look at my update for spec(first revision was published long time ago in augest) about microversion support in novaclient? please... it is very simple change | 16:00 |
| *** jlanoux has quit IRC | 16:00 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** markvoelker has quit IRC | 16:01 | |
| mriedem | andreykurilin: link? | 16:01 |
| andreykurilin | mriedem: https://review.openstack.org/#/c/211206 | 16:01 |
| *** sdake has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| *** mgoddard has quit IRC | 16:02 | |
| mriedem | andreykurilin: seems we should also move that to specs/liberty/implemented/ right? | 16:02 |
| mriedem | the bp was marked as implemented in liberty https://blueprints.launchpad.net/python-novaclient/+spec/api-microversion-support | 16:02 |
| andreykurilin | mriedem: yes. there is a separate patch | 16:03 |
| andreykurilin | with +2 from you:) | 16:03 |
| mriedem | oh on top of it | 16:03 |
| mriedem | i see | 16:03 |
| mriedem | touche | 16:03 |
| *** Daisy has quit IRC | 16:04 | |
| *** Daisy has joined #openstack-nova | 16:04 | |
| *** Jeffrey4l_ has quit IRC | 16:04 | |
| andreykurilin | mridem: also, you use this patch as an example of moving specs to implemented in your latest notification mail:) http://lists.openstack.org/pipermail/openstack-dev/2016-March/089570.html | 16:04 |
| *** irenab_ has joined #openstack-nova | 16:05 | |
| *** irenab has quit IRC | 16:05 | |
| *** irenab_ is now known as irenab | 16:05 | |
| *** Jeffrey4l_ has joined #openstack-nova | 16:05 | |
| mriedem | that was a doppleganger | 16:05 |
| andreykurilin | lol | 16:05 |
| dansmith | sahid: how does this break before the next patch? https://review.openstack.org/#/c/292499/12 | 16:05 |
| *** achanda has joined #openstack-nova | 16:05 | |
| dansmith | sahid: does it just mean reserved calculation is wrong, or instances fail to boot or what? | 16:05 |
| dansmith | oh, you already squashed | 16:06 |
| *** rlrossit has quit IRC | 16:07 | |
| sahid | dansmith: yes yes, i changed my mind, instead of overiding the __init__'s method which you said is not good i prefered to make libvirt driver to set that new field | 16:08 |
| *** salv-orlando has joined #openstack-nova | 16:08 | |
| *** thorst is now known as thorst_afk | 16:08 | |
| *** pcaruana has quit IRC | 16:08 | |
| *** Daisy has quit IRC | 16:09 | |
| dansmith | sahid: how did it fail though? | 16:10 |
| *** e0ne has joined #openstack-nova | 16:11 | |
| *** achanda has quit IRC | 16:11 | |
| dansmith | sahid: I ask because I would have thought that was okay, so I want to make sure we're not missing something | 16:11 |
| sahid | dansmith: from my point of view... | 16:12 |
| sahid | libvirt constructs the new version of PageTopology without to set that 'reserved' field, this object is stored in database | 16:12 |
| *** Jeffrey4l_ has quit IRC | 16:12 | |
| *** rfolco has quit IRC | 16:12 | |
| sahid | then it get deserialized and explode | 16:12 |
| sahid | makes sense? | 16:12 |
| dansmith | no, because deserializing an older object without the field should be fine | 16:13 |
| dansmith | sahid: do you have some logs or something you can point me to? | 16:13 |
| sahid | dansmith: you can see some in PS10 | 16:14 |
| dansmith | oh, his comment, okay | 16:14 |
| *** unicell1 has joined #openstack-nova | 16:15 | |
| mriedem | andreykurilin: i commented on ps6 | 16:15 |
| sahid | yes i was not able to reproduce the case (i did not try too much actually) | 16:15 |
| *** unicell has quit IRC | 16:15 | |
| mriedem | andreykurilin: seems like most of this should just go in the novaclient devref | 16:15 |
| dansmith | sahid: I think that you're covering up this issue by squashing | 16:15 |
| dansmith | sahid: when really, you need to tolerate that field being missing | 16:16 |
| mriedem | since it's talking about things that are not even close to happening today, like nova raising the minimum microversion past v2.1 | 16:16 |
| alaski | mriedem: yeah, there's no good way to track down policy usage | 16:16 |
| dansmith | let me go look at the notification code | 16:16 |
| *** jschlueter has quit IRC | 16:17 | |
| mriedem | alaski: oh sneaky | 16:17 |
| mriedem | test_volume_ops_invalid_task_state | 16:17 |
| mriedem | oops | 16:17 |
| mriedem | _action = '%s:%s' % (scope, action) | 16:17 |
| mriedem | nova.compute.api.API.swap_volume has a decorator that checks it | 16:18 |
| alaski | mriedem: I started working on a hacking check for it briefly, but it has the same problem as notifications. many ways to call it and no assurance that all branches get hit | 16:18 |
| mriedem | so not only do we have the policy checks in the REST API using different values, we also have the compute API policy check | 16:18 |
| alaski | yeah | 16:18 |
| *** fawadkhaliq has joined #openstack-nova | 16:19 | |
| sahid | dansmith: yes, so if my thinking was not the root cause i just have covered the issue | 16:19 |
| dansmith | sahid: yeah, I think you should split these back up because it's going to be easier to reproduce the issue.. what did you try to reproduce? | 16:19 |
| dansmith | sahid: just apply the first patch and boot an instance with the proper config in place? | 16:20 |
| *** ljxiash has quit IRC | 16:20 | |
| dansmith | sahid: the exception on the first line of his log output is getting hidden so we can't tell the true origin I think | 16:20 |
| sahid | dansmith: yes, the i have restarted all processes | 16:20 |
| dansmith | and that's what we need to figure out | 16:20 |
| *** manjeets has joined #openstack-nova | 16:21 | |
| openstackgerrit | Sujitha proposed openstack/nova: Config options: Centralize compute options https://review.openstack.org/295938 | 16:21 |
| sahid | dansmith: i see, so i will split these back up and do futher investigations | 16:22 |
| *** cdent has quit IRC | 16:22 | |
| openstackgerrit | Cale Rath proposed openstack/nova: Make generate_password more secure https://review.openstack.org/210092 | 16:22 |
| *** cdent has joined #openstack-nova | 16:23 | |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP initial use of microversion_parse https://review.openstack.org/300076 | 16:23 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP Initial support for both microversion headers https://review.openstack.org/300077 | 16:23 |
| jaypipes | dansmith, edleafe, cdent: would be great to get https://review.openstack.org/#/c/286931/ merged today. | 16:23 |
| dansmith | sahid: yeah, and I think that if you can reproduce it I have a path forward for you, but I need to look at some o.vo code and see the actual failure you get first | 16:23 |
| andreykurilin__ | mriedem: agreed, as soon as I have final version of this update, I'll copy a part of it to novaclient's doc | 16:23 |
| cdent | sdague: those ^ are just wips for now but might be of interest and I could do with some guidance on where to take things | 16:24 |
| dansmith | jaypipes: edleafe: looks like edleafe pushed up a change on top of my push this morning.. what else changed? | 16:24 |
| *** gagehugo has joined #openstack-nova | 16:24 | |
| sahid | dansmith: ack | 16:24 |
| dansmith | oh, the coalesce thing | 16:24 |
| *** sdake has joined #openstack-nova | 16:24 | |
| bauzas | I don't want to be nitpicking, but is the spec merged ? | 16:25 |
| *** burt has joined #openstack-nova | 16:26 | |
| *** cdelatte has joined #openstack-nova | 16:26 | |
| edleafe | dansmith: yeah, just the coalesce change in my patch | 16:26 |
| cdent | jaypipes: yar +1 now, with edleafe's adjustments | 16:26 |
| mriedem | bauzas: it is https://blueprints.launchpad.net/nova/+spec/compute-node-inventory-newton | 16:27 |
| mriedem | well, the bp is approved | 16:27 |
| dansmith | I dont' really feel like I can approve that code, both because of my close-ness and lack of understanding any of it | 16:27 |
| bauzas | okay, will review it tonight | 16:27 |
| dansmith | so I propose other cores who do, or who have less integrity :P | 16:27 |
| *** browne has joined #openstack-nova | 16:28 | |
| * edleafe bites his lip | 16:28 | |
| *** sdake has quit IRC | 16:28 | |
| dansmith | edleafe: :( | 16:29 |
| *** sdake has joined #openstack-nova | 16:29 | |
| *** apoorvad has joined #openstack-nova | 16:29 | |
| manjeets | is there any limit on nova list ? in my case I created more than 1200 instances but see only 1000 when i do nova list | 16:29 |
| openstackgerrit | Cale Rath proposed openstack/nova: Make generate_password more secure https://review.openstack.org/210092 | 16:29 |
| *** yamahata has quit IRC | 16:30 | |
| *** sdake has quit IRC | 16:30 | |
| *** davideagnello has joined #openstack-nova | 16:30 | |
| jaypipes | mriedem: the blueprint is approved, but the spec still isn't. https://review.openstack.org/#/c/260048/ | 16:31 |
| *** e0ne has quit IRC | 16:32 | |
| cdent | jaypipes: how/where did we leave the placement of the db queries in https://review.openstack.org/#/c/284895/ (Add a get_by_uuid for aggregates) | 16:32 |
| jaypipes | cdent: I said "cool if you do the nova.db.sqlalchemy.api approach to keep it consistent with what's there now" and let doffm refactor it all later. | 16:33 |
| jaypipes | cdent: lemme do a re-review of that... | 16:33 |
| cdent | thanks | 16:34 |
| *** penick has joined #openstack-nova | 16:34 | |
| *** whenry has joined #openstack-nova | 16:34 | |
| * jaypipes opens gerrty | 16:34 | |
| jaypipes | or even gertty. | 16:34 |
| *** mylu has quit IRC | 16:34 | |
| * edleafe would love to one day get gertty to actually work | 16:35 | |
| cdent | edleafe: Yeah, I keep trying it and it keeps failing in weird ways | 16:36 |
| cdent | It's also too hard to read commentary on earlier patchsets | 16:36 |
| *** fawadkhaliq has quit IRC | 16:36 | |
| *** fawadkhaliq has joined #openstack-nova | 16:37 | |
| dansmith | yeah, the visual interface to gertty is just not usable IMHO | 16:38 |
| dansmith | really hard to tell where the cursor thing is, really strange rules for moving between fields, etc | 16:38 |
| dansmith | I customized the colors to make it a little better, but.. it's just not for me I think | 16:38 |
| *** shangxdy_ has quit IRC | 16:38 | |
| *** mattymo has left #openstack-nova | 16:39 | |
| edleafe | It must be like emacs - once you figure out how to use it, everything else sucks | 16:39 |
| *** shangxdy_ has joined #openstack-nova | 16:39 | |
| *** sfinucan has quit IRC | 16:39 | |
| *** jdurgin1 has quit IRC | 16:40 | |
| *** cdent has quit IRC | 16:41 | |
| *** fawadkhaliq has quit IRC | 16:41 | |
| *** mpavone has quit IRC | 16:41 | |
| *** fawadkhaliq has joined #openstack-nova | 16:41 | |
| *** matrohon has quit IRC | 16:42 | |
| *** cdent has joined #openstack-nova | 16:43 | |
| *** cdelatte has quit IRC | 16:45 | |
| *** sahid has quit IRC | 16:47 | |
| *** e0ne has joined #openstack-nova | 16:47 | |
| *** sdake has joined #openstack-nova | 16:48 | |
| *** e0ne has quit IRC | 16:50 | |
| manjeets | is there any limit on nova list ? in my case I created more than 1200 instances but see only 1000 when i do nova list | 16:52 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate metadata functions work with API db https://review.openstack.org/297350 | 16:52 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add aggregates tables to the API db. https://review.openstack.org/294806 | 16:52 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate host operations work against API db https://review.openstack.org/296597 | 16:52 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make AggregateList.get_ return API & cell db items https://review.openstack.org/296062 | 16:52 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate.get operation favor the API db https://review.openstack.org/296014 | 16:52 |
| mriedem | manjeets: see the osapi_max_limit config option | 16:53 |
| mriedem | it defaults to 1000 | 16:53 |
| *** moshele has quit IRC | 16:53 | |
| mriedem | danpb: ndipanov: have you ever heard of a case where non-admins find swap-volume useful? | 16:54 |
| manjeets | thanks mriedem which file ? | 16:54 |
| mriedem | manjeets: http://docs.openstack.org/developer/nova/sample_config.html | 16:54 |
| Prophidys | mriedem, if you can recheck this one : https://review.openstack.org/#/c/299474/4 I followed the ML decision | 16:55 |
| manjeets | mriedem thanks :) | 16:55 |
| *** zenoway has quit IRC | 16:55 | |
| danpb | mriedem: i'm not particularly familiar with it but i bet it would be people running pets who care | 16:55 |
| dansmith | yeah, I know people that care about it, | 16:55 |
| dansmith | not sure if they care about non-admins though | 16:55 |
| mriedem | i was going to say, people, like admins? | 16:56 |
| dansmith | can non-admins migrate their storage on the cinder side? | 16:56 |
| mriedem | b/c it's been busted for non-admins for a looong time | 16:56 |
| mriedem | no | 16:56 |
| dansmith | it's only really useful in nova for that case, right/ | 16:56 |
| dansmith | so what's the point of having it be available to non-admins? | 16:56 |
| mriedem | i guess? vish added it in havana with little detail on the use case | 16:56 |
| mriedem | then later cinder started using it for their volume migration scenario | 16:56 |
| dansmith | I think it's only really for volume migration | 16:57 |
| dansmith | and I think it was intended to be not only admins, but really just for cinder | 16:57 |
| *** thorst_afk is now known as thorst | 16:57 | |
| dansmith | which, as you know, makes for the best APIs around | 16:57 |
| *** rpodolyaka is now known as rpodolyaka_afk | 16:57 | |
| mriedem | ok, i'm just trying to figure out if nova should restrict to admin or cinder should loosen up their migrate_volume_completion API to admin-or-owner, | 16:57 |
| *** gjayavelu has quit IRC | 16:57 | |
| mriedem | since it was originally not admin-only, | 16:58 |
| mriedem | but has been changed twice to make it now require admin | 16:58 |
| mriedem | we never had tempest testing for this so no one noticed the api changes | 16:58 |
| dansmith | heh | 16:58 |
| dansmith | also good makins' for an API | 16:58 |
| mriedem | yeah, | 16:58 |
| mriedem | so given that, | 16:59 |
| *** shangxdy_ has quit IRC | 16:59 | |
| mriedem | i'm inclined to just say eff it and make swap-volume admin-only in nova | 16:59 |
| *** shangxdy has quit IRC | 16:59 | |
| *** pwnall has quit IRC | 16:59 | |
| mriedem | 'because it's been this way for years, if you didn't know, you probably already forked it' | 16:59 |
| dansmith | well, I definitely agree with the "eff it" part | 16:59 |
| *** mylu has joined #openstack-nova | 16:59 | |
| *** cdelatte has joined #openstack-nova | 16:59 | |
| mriedem | btw, we should probably change the channel topic since i think someone is auto-pinging me every time i join | 17:00 |
| dansmith | I'm fine with that | 17:00 |
| mriedem | we do'nt have a newton schedule yet, so i'm thinking a link to the review dashboard | 17:00 |
| *** jistr has quit IRC | 17:00 | |
| *** ChanServ sets mode: +o dansmith | 17:00 | |
| *** haomaiwang has quit IRC | 17:01 | |
| mriedem | just add "Reviews: https://goo.gl/1vTS0Z" | 17:01 |
| Prophidys | mriedem, thank you | 17:01 |
| *** mgoddard has joined #openstack-nova | 17:01 | |
| *** diogogmt has joined #openstack-nova | 17:01 | |
| *** dansmith changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack - Reviews: https://goo.gl/1vTS0Z"- PTL says: if it's latent let it mellow, if it's 500 flush it down" | 17:01 | |
| *** haomaiwa_ has joined #openstack-nova | 17:01 | |
| alaski | hah | 17:01 |
| mriedem | heh, i see what you did there | 17:01 |
| *** dansmith changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack - Reviews: https://goo.gl/1vTS0Z - PTL says: if it's latent let it mellow, if it's 500 flush it down" | 17:01 | |
| *** mgoddard_ has quit IRC | 17:02 | |
| *** ChanServ sets mode: -o dansmith | 17:02 | |
| *** dedery has joined #openstack-nova | 17:03 | |
| *** pwnall has joined #openstack-nova | 17:04 | |
| mriedem | bauzas: for a release note that fixes a bug by changing a default policy value, for a thing that didn't work without it, do you think that goes in upgrades, fixes or other? | 17:04 |
| *** jlanoux_ has quit IRC | 17:04 | |
| *** Guest11037 has quit IRC | 17:05 | |
| *** yamahata has joined #openstack-nova | 17:05 | |
| mriedem | i'm thinking other | 17:05 |
| *** ZZelle has quit IRC | 17:05 | |
| *** ptm_away has joined #openstack-nova | 17:05 | |
| *** ZZelle has joined #openstack-nova | 17:06 | |
| *** rcernin has quit IRC | 17:06 | |
| *** derekh has quit IRC | 17:06 | |
| *** sridharg has quit IRC | 17:07 | |
| *** haukebruno has quit IRC | 17:08 | |
| *** ihrachys has quit IRC | 17:08 | |
| *** dedery has quit IRC | 17:09 | |
| *** PaulMurray has quit IRC | 17:09 | |
| *** angdraug has quit IRC | 17:10 | |
| *** haukebruno has joined #openstack-nova | 17:13 | |
| *** ptm_away has quit IRC | 17:13 | |
| *** woodster_ has joined #openstack-nova | 17:14 | |
| *** achanda has joined #openstack-nova | 17:15 | |
| mriedem | one thing i think we can all agree on is chocolate covered marshmallows in egg shapes are good | 17:19 |
| *** penick has quit IRC | 17:21 | |
| cdent | mriedem: wrong | 17:22 |
| *** haukebruno has quit IRC | 17:23 | |
| *** zenoway has joined #openstack-nova | 17:24 | |
| mriedem | cdent: don't let the easter bunny hear you say that | 17:25 |
| *** haukebruno has joined #openstack-nova | 17:25 | |
| *** daemontool has quit IRC | 17:25 | |
| cdent | The easter bunny and I disagree a fair bit. | 17:25 |
| jaypipes | cdelatte: done. | 17:26 |
| jaypipes | grrr.... cdent, done. | 17:26 |
| cdent | need me to change my name? | 17:26 |
| jaypipes | cdent: though it doesn't really matter since the dependent change has the black mark of 02 death. | 17:26 |
| jaypipes | -2 death. | 17:27 |
| jaypipes | ffs my typing sucks today. | 17:27 |
| *** penick has joined #openstack-nova | 17:27 | |
| cdent | jaypipes: Yeah, I'm totally confused on what is or is important and is or is not going to get some attention and whether I can reorder things to make them legit. | 17:27 |
| *** fawadkhaliq has quit IRC | 17:28 | |
| *** fawadkhaliq has joined #openstack-nova | 17:30 | |
| jaypipes | mriedem, johnthetubaguy: I am not able to set the URL for this specification: https://blueprints.launchpad.net/nova/+spec/compute-node-inventory-newton. Can you set it to https://review.openstack.org/#/c/260048/ please? | 17:31 |
| jaypipes | mriedem, johnthetubaguy: and would it be possible to approve https://review.openstack.org/#/c/260048/? :) | 17:31 |
| mriedem | jaypipes: you can't | 17:31 |
| mriedem | because the mitaka bp has that link for it's spec already | 17:31 |
| mriedem | jaypipes: i think i already tried that once | 17:31 |
| *** yonglihe has quit IRC | 17:32 | |
| jaypipes | mriedem: it says I don't have permission to set the link... | 17:32 |
| mriedem | this guy https://blueprints.launchpad.net/nova/+spec/compute-node-inventory | 17:32 |
| mriedem | i'm not really sure why https://blueprints.launchpad.net/nova/+spec/compute-node-inventory was marked complete honestly | 17:32 |
| jaypipes | mriedem: I don't know either. I certainly didn't do that. | 17:33 |
| mriedem | i suspect john did | 17:33 |
| jaypipes | mriedem: what do I need to do? Abandon 279313 and resubmit a whole new spec? | 17:33 |
| mriedem | i do'nt think so... | 17:34 |
| mriedem | sec | 17:34 |
| jaypipes | mriedem: I can certainly do so for all remaining specs in the resource-providers series. and in doing that I can make them depend on each other appropriately in git. | 17:34 |
| *** anush_ has joined #openstack-nova | 17:35 | |
| mriedem | jaypipes: do you mean drop compute-node-inventory-newton in lp and we just re-open compute-node-inventory ? | 17:35 |
| jaypipes | mriedem: no, I mean the specs in Gerrit... | 17:36 |
| jaypipes | mriedem: I can leave you and john to do what you want with the blueprints in LP. | 17:36 |
| *** sdague has quit IRC | 17:36 | |
| mriedem | i was just going to remove the 260048 from the compute-node-inventory bp in lp | 17:37 |
| jaypipes | mriedem: tell you what, don't do that... | 17:37 |
| *** ctrath has quit IRC | 17:37 | |
| *** jwcroppe has joined #openstack-nova | 17:38 | |
| *** dimtruck is now known as zz_dimtruck | 17:38 | |
| *** sdague has joined #openstack-nova | 17:38 | |
| *** rlrossit has joined #openstack-nova | 17:38 | |
| jaypipes | mriedem: lemme push a whole new set of specs, with proper dependencies, with -newton at the end of each blueprint link. we can then obselete the original blueprints in LP and get a new set of blueprints associated with the correct Gerrit patches, all with proper dependencies in git. | 17:39 |
| *** anush_ has quit IRC | 17:39 | |
| *** ducttape_ has quit IRC | 17:39 | |
| *** dims_ has quit IRC | 17:39 | |
| mriedem | sounds fine to me...gonna be quite a bit of churn | 17:40 |
| *** gjayavelu has joined #openstack-nova | 17:40 | |
| *** dims has joined #openstack-nova | 17:40 | |
| jaypipes | mriedem: yeah, but you and dansmith have been asking me to fix up the dependency chain for a while now and I've been putting that off... might as well just get it done :) | 17:40 |
| *** ctrath has joined #openstack-nova | 17:40 | |
| *** zz_dimtruck is now known as dimtruck | 17:41 | |
| mriedem | sounds like cdent wants that also | 17:41 |
| mriedem | i can't speak for that dansmith guy | 17:41 |
| dansmith | there is really no hope of pleasing me, so I'd go with what mriedem wants | 17:41 |
| cdent | mriedem: that's not quite what I wanted, I was more sort of flailing in a kind of "what's going on? where's my hat? how come this is over there and I'm upside down?" | 17:42 |
| *** mylu has quit IRC | 17:42 | |
| cdent | A newly refined stack may help that. | 17:43 |
| mriedem | you know what i want? | 17:44 |
| mriedem | https://review.openstack.org/#/c/295118/ | 17:44 |
| mriedem | looks like i have to fix it | 17:44 |
| *** READ10 has quit IRC | 17:45 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Set 'libvirt.sysinfo_serial' to 'none' in RealTimeServersTest https://review.openstack.org/295118 | 17:45 |
| *** ccarmack has quit IRC | 17:45 | |
| mriedem | someone want to +W that ^? | 17:45 |
| *** fawadkhaliq has quit IRC | 17:46 | |
| *** fawadkhaliq has joined #openstack-nova | 17:46 | |
| *** fawadkhaliq has quit IRC | 17:47 | |
| sdague | mriedem: done | 17:47 |
| *** fawadkhaliq has joined #openstack-nova | 17:47 | |
| *** fawadkhaliq has quit IRC | 17:47 | |
| *** fawadkhaliq has joined #openstack-nova | 17:48 | |
| *** fawadkhaliq has quit IRC | 17:48 | |
| dansmith | mriedem: so, we need one more ack on the top flavor patch, and then a full set on the very bottom one | 17:48 |
| -openstackstatus- NOTICE: Job failures for missing traceroute packages are in the process of being fixed now, ETA 30 minutes to effectiveness for new jobs | 17:48 | |
| dansmith | mriedem: we're pretty close and I think we can/should merge it this week | 17:49 |
| jaypipes | dansmith: link? I'll do it. | 17:49 |
| *** fawadkhaliq has joined #openstack-nova | 17:49 | |
| dansmith | this is the top one: https://review.openstack.org/#/c/295310/28 | 17:49 |
| *** jaosorior has joined #openstack-nova | 17:49 | |
| dansmith | and the bottom one in that stack | 17:49 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP Initial support for both microversion headers https://review.openstack.org/300077 | 17:49 |
| openstackgerrit | Chris Dent proposed openstack/nova: Initial use of microversion_parse https://review.openstack.org/300076 | 17:50 |
| *** fawadkhaliq has quit IRC | 17:50 | |
| dansmith | I also think we need to get jaypipes' stats thing merged because three of us have been stepping on each other with it | 17:50 |
| *** mriedem has quit IRC | 17:50 | |
| dansmith | but I'm also kinda chicken | 17:50 |
| *** ducttape_ has joined #openstack-nova | 17:50 | |
| dansmith | I guess I should just do it because I wrote the code on top of it that wouldn't work if it was broken | 17:50 |
| *** fawadkhaliq has joined #openstack-nova | 17:50 | |
| *** mriedem has joined #openstack-nova | 17:51 | |
| dansmith | eff it | 17:53 |
| *** gyee has joined #openstack-nova | 17:53 | |
| *** unicell has joined #openstack-nova | 17:54 | |
| *** klkumar has quit IRC | 17:54 | |
| *** vks has quit IRC | 17:55 | |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP Initial support for both microversion headers https://review.openstack.org/300077 | 17:55 |
| openstackgerrit | Chris Dent proposed openstack/nova: Initial use of microversion_parse https://review.openstack.org/300076 | 17:55 |
| * cdent is touched | 17:55 | |
| *** boden has joined #openstack-nova | 17:55 | |
| *** unicell1 has quit IRC | 17:56 | |
| dansmith | is docimpact still a thing? | 17:57 |
| *** boden has left #openstack-nova | 17:58 | |
| *** ccarmack has joined #openstack-nova | 17:58 | |
| jaypipes | dansmith: 295310 wallaby'd | 17:58 |
| dansmith | thanks | 17:58 |
| dansmith | I will have everyone note that I just ran this command: | 17:58 |
| dansmith | git checkout master -b sins-of-the-father | 17:59 |
| sdague | dansmith: DocImpact creates a bug in the nova tracker now instead of the openstack-manuals tracker | 18:00 |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove flavor seeding from the base migration https://review.openstack.org/300127 | 18:00 |
| dansmith | sdague: okay that was the thing I was hearing and wondering about I guess | 18:00 |
| *** haomaiwa_ has quit IRC | 18:01 | |
| dansmith | sdague: so for the flavor base migration thing (^) should I do a docimpact? | 18:01 |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| *** jaosorior has quit IRC | 18:01 | |
| sdague | dansmith: or just push the bug directly to openstack-manuals explaining what needs to change in the manuals | 18:01 |
| sdague | I expect you'll need to follow up on the bug with the details regardless | 18:02 |
| *** vilobhmm11 has joined #openstack-nova | 18:02 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Make swap-volume an admin-only API by default https://review.openstack.org/299715 | 18:02 |
| mriedem | "" | 18:03 |
| mriedem | "In a time long ago and a land far far away" | 18:03 |
| mriedem | s/land/galaxy/? | 18:03 |
| mriedem | or don't want to get sued | 18:03 |
| dansmith | sdague: meaning let it auto-create and then add openstack-manuals to the bug? | 18:04 |
| dansmith | mriedem: I'm just not that familiar with star wars.. figure this was close enough | 18:04 |
| *** danpb has quit IRC | 18:04 | |
| *** haukebruno has quit IRC | 18:05 | |
| mriedem | dansmith: you could do that | 18:05 |
| mriedem | rlrossit: policy file changes, your favorite https://review.openstack.org/299715 | 18:05 |
| rlrossit | oh right, I was going to look at that spec | 18:05 |
| *** anush_ has joined #openstack-nova | 18:06 | |
| *** fawadk has joined #openstack-nova | 18:06 | |
| *** fawadkhaliq has quit IRC | 18:07 | |
| *** rtheis has quit IRC | 18:08 | |
| *** rtheis has joined #openstack-nova | 18:09 | |
| *** fawadk has quit IRC | 18:09 | |
| *** gyee has quit IRC | 18:10 | |
| mriedem | dansmith: can you post something to the operators ML about this https://review.openstack.org/#/c/300127/ ? | 18:10 |
| *** jwcroppe_ has joined #openstack-nova | 18:11 | |
| bauzas | mriedem: dansmith: oh, the consensus is now to remove that? | 18:11 |
| dansmith | mriedem: I either want to wait until I have a look at all the failures, or not include that link and just meta-discuss it, so which do you want? | 18:11 |
| *** zenoway has quit IRC | 18:11 | |
| bauzas | <3 | 18:11 |
| dansmith | bauzas: was there any question/ | 18:11 |
| mriedem | dansmith: meta is fine | 18:11 |
| mriedem | or both | 18:11 |
| dansmith | mriedem: damn. | 18:11 |
| bauzas | dansmith: about https://review.openstack.org/#/c/300127/ | 18:11 |
| *** Sukhdev has joined #openstack-nova | 18:12 | |
| *** anush_ has quit IRC | 18:12 | |
| * bauzas was a bit off today because of the French general strike needing me to help my daughters :( | 18:12 | |
| *** jwcroppe has quit IRC | 18:12 | |
| mriedem | bauzas: if we don't remove them from the initial migration, you have to run the online data migrations thing right away with new deployments | 18:12 |
| dansmith | bauzas: I said here: https://review.openstack.org/#/c/295310/28/nova/test.py | 18:12 |
| dansmith | bauzas: that we clearly needed to do that, I just don't want to wrap it all up in this work | 18:12 |
| sdague | dansmith: that's the normal process anyway, but you have to explain what should change. | 18:13 |
| *** lpetrut has quit IRC | 18:13 | |
| bauzas | dansmith: okay, it seems I maybe misunderstood your thoughts | 18:13 |
| dansmith | sdague: what is normal process, the bug thing? | 18:13 |
| sdague | dansmith: normal process == | 18:13 |
| sdague | DocImpact -> triggers creation of bug in the nova tracker | 18:13 |
| *** devananda has quit IRC | 18:14 | |
| sdague | someone from nova team updates bug with details that are sufficient for the openstack-manuals team to make the fixes we are suggesting | 18:14 |
| dansmith | sdague: right, okay got it | 18:14 |
| sdague | add openstack-manuals to the bug | 18:14 |
| *** gyee has joined #openstack-nova | 18:14 | |
| sdague | mark nova part as invalid | 18:14 |
| dansmith | okay | 18:15 |
| sdague | dansmith: shouldn't https://review.openstack.org/#/c/300127/ have the reno explaining that you must now manually create flavors before nova works ? | 18:15 |
| *** Sukhdev has quit IRC | 18:15 | |
| *** tbachman has quit IRC | 18:15 | |
| dansmith | sdague: that patch is going to probably need lots more, I just wanted a run to see if anything else is broken :) | 18:16 |
| dansmith | sdague: I figure that's blocked on probable devstack changes, the manual stuff being ready, etc | 18:16 |
| dansmith | I put it in the commit message | 18:16 |
| dansmith | I'll do the docimpact in a sec, re-push, and throw a -W on it | 18:16 |
| *** diogogmt_ has joined #openstack-nova | 18:16 | |
| *** diogogmt has quit IRC | 18:17 | |
| *** diogogmt_ is now known as diogogmt | 18:17 | |
| *** baoli_ has quit IRC | 18:17 | |
| mriedem | rlrossit: https://review.openstack.org/#/c/115749/ | 18:17 |
| mriedem | rlrossit: resurrect that whydontcha | 18:17 |
| *** zengine has joined #openstack-nova | 18:18 | |
| rlrossit | I bet dansmith is gonna love me for this one | 18:18 |
| ildikov | jaypipes: hmm, I think thanks for abandoning that patch :) | 18:19 |
| *** zul has quit IRC | 18:20 | |
| ildikov | jaypipes: I thought to reuse it, but in the current stage I'm not sure how much of will be reusable anyway | 18:20 |
| *** zul has joined #openstack-nova | 18:22 | |
| *** cdelatte has quit IRC | 18:22 | |
| *** zenoway has joined #openstack-nova | 18:24 | |
| *** dims_ has joined #openstack-nova | 18:24 | |
| *** dims has quit IRC | 18:24 | |
| rlrossit | "and have 1 and 8215 different commits each, respectively." | 18:25 |
| rlrossit | boy this is going to be fun | 18:25 |
| *** rfolco has joined #openstack-nova | 18:25 | |
| *** salv-orlando has quit IRC | 18:26 | |
| *** devananda has joined #openstack-nova | 18:27 | |
| *** penick has quit IRC | 18:27 | |
| *** zenoway has quit IRC | 18:28 | |
| *** fawadkhaliq has joined #openstack-nova | 18:29 | |
| jaypipes | ildikov: yeah, that was super old :) | 18:31 |
| *** Marga_ has quit IRC | 18:32 | |
| *** zul has quit IRC | 18:32 | |
| *** zul has joined #openstack-nova | 18:32 | |
| *** cdelatte has joined #openstack-nova | 18:33 | |
| doffm | gibi: https://bugs.launchpad.net/nova/+bug/1564559 | 18:33 |
| openstack | Launchpad bug 1564559 in OpenStack Compute (nova) "Instances should not have soft-affinity with themselves." [Low,New] - Assigned to Mark Doffman (mjdoffma) | 18:33 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Make git clean actually remove covhtml https://review.openstack.org/300140 | 18:33 |
| ildikov | jaypipes: yeah, I did not really maintained that last patch as it wasn't decided at that point how the change should be reflected on the Nova API | 18:33 |
| openstackgerrit | Jeremy Stanley proposed openstack/nova: List system dependencies for running common tests https://review.openstack.org/200672 | 18:33 |
| doffm | jdurgin, dansmith: Could you take another look at https://review.openstack.org/#/c/241777/ Its a bug fix that has been sitting for a while. | 18:35 |
| *** delattec has joined #openstack-nova | 18:36 | |
| *** mvk_ has quit IRC | 18:36 | |
| *** cdent has quit IRC | 18:38 | |
| *** gjayavelu has quit IRC | 18:38 | |
| *** cdelatte has quit IRC | 18:39 | |
| *** dave-mccowan has quit IRC | 18:41 | |
| *** delatte has joined #openstack-nova | 18:41 | |
| *** delatte has quit IRC | 18:42 | |
| *** delattec has quit IRC | 18:44 | |
| *** efoley has quit IRC | 18:46 | |
| dansmith | sdague: so I see 69 instances of catching exception to avoid state corruption in compute manager alone | 18:46 |
| dansmith | sdague: and several places use a common decorator to do it, which means we end up with that case many more than that in just one file | 18:46 |
| dansmith | sdague: given the ramifications of us corrupting our own state so that we disallow any new operations after a failure, what else would you suggest? | 18:47 |
| dansmith | short of "do all the task things" of course :) | 18:47 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add support of caching https://review.openstack.org/299860 | 18:47 |
| lifeless | sdague: did you get that constraints question answered? I did follow up but I think you'd EOD'd... | 18:51 |
| sdague | dansmith: in reference to - https://review.openstack.org/#/c/298997/ ? | 18:51 |
| dansmith | sdague: yeah | 18:51 |
| sdague | lifeless: the question was mostly if there was a clever way to hack requirements.txt so that you can install a library from a specific git commit for spot testing purposes only | 18:52 |
| dansmith | holy necrotic thread batman! | 18:53 |
| sdague | dansmith: so this moves from error on shutdown to random leaks of things, right? | 18:53 |
| dansmith | sdague: it actually closes leaks.. | 18:53 |
| dansmith | if we don't continue here, we leave things unterminated across us and cinder | 18:53 |
| dansmith | which normal people can't resolve | 18:53 |
| dansmith | and we don't emit the notification for instance shutdown | 18:54 |
| sdague | dansmith: ok, well it still seems like we should be logging the bdm in question as well so it's clear what didn't get cleaned up on the otherside, because if cinder is down, then that just gets orphaned in an odd state, right? | 18:55 |
| openstackgerrit | Jeremy Stanley proposed openstack/nova: List system dependencies for running common tests https://review.openstack.org/200672 | 18:55 |
| dansmith | sdague: what gets orphaned? | 18:55 |
| dansmith | sdague: the volume or the bdm/ | 18:55 |
| sdague | the volume on the cinder side will be in the wrong state, right? | 18:56 |
| *** zenoway has joined #openstack-nova | 18:56 | |
| openstackgerrit | Clenimar Filemon proposed openstack/nova: Use Identity v3 params when connecting to Ironic https://review.openstack.org/300154 | 18:56 |
| dansmith | sdague: depends on the failure, but any of them after this one fails are guaranteed to be bad right? | 18:56 |
| dansmith | sdague: so if we're talking to one backend that is broken but the rest aren't we don't clean up the others | 18:57 |
| lifeless | sdague: yes, grep for edit-constraints in devstack | 18:57 |
| dansmith | which is the reason we already exclude a bunch of them by name | 18:57 |
| dansmith | sdague: I believe the reason cinder allows things other than nova to call force-detach is also to allow this sort of cleanup | 18:57 |
| dansmith | sdague: but if we die here, then it requires nova intervention as well | 18:58 |
| sdague | ok, so I now get that, though it still sucks that the best we have is Exception | 18:58 |
| dansmith | sdague: well, we exclude a bunch by name, but given how bad it is if we die here, I think we should be careful | 18:59 |
| dansmith | like we try to be lots of other places | 18:59 |
| dansmith | the other bad case here, | 18:59 |
| dansmith | is if you're trying to delete an instance to create another and attach your volumes elsewhere, | 18:59 |
| sdague | however it still seems like we should log something more specific about the bdm in question. This is going to be a WARN with a giant stack trace right? | 18:59 |
| dansmith | all your volumes are completely hosed and stuck to this until you get support intervention | 18:59 |
| dansmith | instead of just the bad one | 18:59 |
| lifeless | if you need to emulate try:finally for some reason, BaseException is your friend | 19:00 |
| lifeless | but really think hard before doing that | 19:00 |
| dansmith | sdague: no stack trace just a warning | 19:00 |
| *** zenoway has quit IRC | 19:00 | |
| sdague | dansmith: ok so shutdown instance | 19:00 |
| sdague | WARNING Ignoring unknown exception | 19:00 |
| sdague | what does that even mean | 19:00 |
| dansmith | sdague: it includes the exception name | 19:00 |
| dansmith | or are you saying you want a trace? | 19:00 |
| *** haomaiwang has quit IRC | 19:01 | |
| sdague | sure, but it doesn't include the bdm | 19:01 |
| openstackgerrit | Jeremy Stanley proposed openstack/nova: List system dependencies for running common tests https://review.openstack.org/200672 | 19:01 |
| *** haomaiwang has joined #openstack-nova | 19:01 | |
| sdague | I want enough left in the logs so that the devices that never got talked to on the cinder side are there | 19:01 |
| sdague | a warn should have enough info to be actionable | 19:01 |
| sdague | so something that an admin could realize ... oh there is a think potentially screwing on these vols now | 19:02 |
| dansmith | sdague: okay, so volume_id then? | 19:02 |
| sdague | dansmith: sure, I don't know everything in that structure that might be useful | 19:02 |
| sdague | but it seems like breadcrumbs would help | 19:02 |
| mriedem | volume_id might not be set if boot from volume failed... | 19:02 |
| dansmith | sdague: we log the instance already, so I'm not sure.. | 19:03 |
| dansmith | yeah | 19:03 |
| *** lpetrut has joined #openstack-nova | 19:03 | |
| dansmith | instance should be enough | 19:03 |
| dansmith | because you can just go see what is still attached to that instance in cinder | 19:03 |
| sdague | we log the instance, but this is a fail inside a bdm loop | 19:03 |
| mriedem | bdm.device_path should be set though | 19:03 |
| sdague | we should actually say which bdm is the cause | 19:03 |
| dansmith | mriedem: device_path isn't really useful is it? | 19:03 |
| *** openstackgerrit has quit IRC | 19:03 | |
| dansmith | sdague: we don't have uuids on BDMs so there's really not much identifying information, other than volume id if it's set | 19:04 |
| mriedem | dansmith: well device_path is part of the bdm unique constraint (if that existed) | 19:04 |
| *** dave-mccowan has joined #openstack-nova | 19:04 | |
| *** openstackgerrit has joined #openstack-nova | 19:04 | |
| mriedem | the uniqueness constraints for bdm today are volume_id and device_path | 19:04 |
| mriedem | *device_name | 19:04 |
| sdague | well all those calls are bdm.volume_id | 19:04 |
| sdague | so I kind of assume that's a thing | 19:04 |
| dansmith | yeah, I guess so given they don't check first | 19:05 |
| sdague | unless we're using Except to catch attr error | 19:05 |
| mriedem | yar | 19:05 |
| mriedem | vol_bdms = [bdm for bdm in bdms if bdm.is_volume] | 19:05 |
| mriedem | they do check i think | 19:05 |
| dansmith | ah | 19:05 |
| mriedem | i remember looking at this before | 19:05 |
| dansmith | right, but this is volumes only, | 19:05 |
| mriedem | b/c of the gate race failure here | 19:05 |
| dansmith | but are you saying BFV might not have a volume_id here even though it is_volume? | 19:05 |
| dansmith | otherwise I don't know how it could be making this call to cinder | 19:05 |
| mriedem | dansmith: i'd have to look but i think that's possible | 19:06 |
| mriedem | if boot from volume where source_type is image/snapshot/blank fails | 19:06 |
| mriedem | b/c that's what gives us the volume_id from cinder and we set it on the bdm | 19:06 |
| mriedem | down in nova.virt.block_device | 19:06 |
| sdague | in which case the Exception catch is actually masking that fact, because we'll end up with a typeerror or something | 19:06 |
| dansmith | mriedem: so we might be calling terminate here, but without expecting it to work for that case or something? | 19:06 |
| *** penick has joined #openstack-nova | 19:06 | |
| mriedem | which should be in the prep_block_devices lock that is shared with the build_resources clock | 19:06 |
| mriedem | *lock | 19:06 |
| sdague | which is actually part of the concern in my mind | 19:06 |
| dansmith | sdague: not if the connector is how we find it | 19:07 |
| dansmith | sdague: even if we did, we need to terminate the rest of the volumes | 19:07 |
| sdague | dansmith: which is fine | 19:07 |
| dansmith | okay, so regardless, bdm.volume_id will not crash | 19:07 |
| *** penick has quit IRC | 19:07 | |
| sdague | however if we're going to steamroll grind to success, we should at least say which bodies we left behidn | 19:07 |
| dansmith | it might be none or something, | 19:07 |
| dansmith | but it's not going to kill us here, | 19:07 |
| dansmith | _diana_: so I think the outcome of that is, please add bdm.volume_id to that log message | 19:08 |
| mriedem | dansmith: sdague: hold the phone | 19:09 |
| *** Marga_ has joined #openstack-nova | 19:09 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova-specs: Add project id validation with keystone https://review.openstack.org/294337 | 19:09 |
| dansmith | sdague: I would say if we're going to do that we should add it to all the ignores above right? | 19:09 |
| sdague | dansmith: yeh, I think that would be worthwhile as well | 19:09 |
| _diana_ | dansmith: will do, thanks | 19:09 |
| *** cdent has joined #openstack-nova | 19:10 | |
| *** Sukhdev has joined #openstack-nova | 19:10 | |
| sdague | it will be helpful to track down when things went wrong in people's environments, and what resources might be in inconsistent states | 19:10 |
| dansmith | mriedem: volume_id is nullable, which I hope means it's always set, but might be none | 19:10 |
| mriedem | dansmith: commented, look at the comment from me in terminate_instance | 19:10 |
| dansmith | mriedem: it'll be set or None right? | 19:11 |
| dansmith | yep, it will | 19:12 |
| *** cdelatte has joined #openstack-nova | 19:12 | |
| dansmith | _diana_: so I would, at the top, try to find the best identification you can for the bdm | 19:12 |
| mriedem | block_device_obj[key] = db_block_device[key] | 19:12 |
| mriedem | dansmith: yeah | 19:12 |
| dansmith | _diana_: volume_id if it's non-None, and then the device_name, and if nothing else set it to sdague's email address | 19:13 |
| _diana_ | lol | 19:13 |
| sdague | dansmith: :P | 19:13 |
| dansmith | I'm not joking, who is joking? | 19:14 |
| dansmith | okay, I'm joking | 19:14 |
| dansmith | but not really | 19:14 |
| dansmith | no, I am | 19:14 |
| * dansmith channels his inner izzard | 19:14 | |
| *** penick has joined #openstack-nova | 19:14 | |
| *** ociuhandu has quit IRC | 19:15 | |
| mriedem | to see how it works with the bfv case, you'd have to add a sleep or something in the flow that gives you time to take down the cinder-api when nova tries to create the volume | 19:15 |
| *** penick has quit IRC | 19:16 | |
| mriedem | if nova creates the volume and we fail (or timeout) while waiting for it to be available, then we clean that up ourselves | 19:16 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova-specs: Add project id validation with keystone https://review.openstack.org/294337 | 19:16 |
| mriedem | i remember in one of my changes that i tried changing "vol_bdms = [bdm for bdm in bdms if bdm.is_volume]" to check bdm.volume_id but ndipanov didn't like that | 19:17 |
| *** zenoway has joined #openstack-nova | 19:18 | |
| mriedem | _diana_: fyi https://review.openstack.org/#/c/258118/ | 19:19 |
| _diana_ | mriedem: Thanks! I'll review the logs here and that review, and check back in a bit with any questions. | 19:21 |
| mriedem | if bdm.volume_id is None, we don't get to Exception | 19:26 |
| mriedem | we get VolumeNotFound | 19:26 |
| mriedem | https://github.com/openstack/cinder/blob/master/cinder/api/contrib/volume_actions.py#L233-L236 | 19:26 |
| mriedem | dansmith: _diana_: sdague: ^ | 19:26 |
| mriedem | so it should be safe to use bdm.volume_id in the generic Exception block | 19:26 |
| dansmith | mriedem: well, it sounds like we'll grab an id first and be able to log it for all the cases | 19:26 |
| sdague | mriedem: cool | 19:26 |
| dansmith | but yeah, cool | 19:26 |
| *** jamielennox|away is now known as jamielennox | 19:26 | |
| mriedem | btw, this brings back some bad memories of that gate bug | 19:27 |
| mriedem | sdague: ever seen failures like this? http://logs.openstack.org/40/300140/1/check/gate-tempest-dsvm-full/0a6f567/console.html | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs: resource-providers: Migrate compute node inventory https://review.openstack.org/300175 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs: resource-providers: generic resource pools https://review.openstack.org/300176 | 19:28 |
| mriedem | jenkins hated my little .gitignore patch https://review.openstack.org/#/c/300140/ | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs: resource-providers: Move allocation fields https://review.openstack.org/300177 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs: resource-providers: Scheduler filters in DB https://review.openstack.org/300178 | 19:28 |
| mriedem | sdague: nvm, i see you asked the same in infra | 19:28 |
| jaypipes | mriedem, dansmith, johnthetubaguy: k ^^ is four specs in the resource-providers blueprint series, properly ordered and dependency linked. | 19:29 |
| *** sileht has quit IRC | 19:30 | |
| *** apoorvad has quit IRC | 19:30 | |
| jaypipes | mriedem: if you wouldn't mind setting the spec URL of https://blueprints.launchpad.net/nova/+spec/compute-node-inventory-newton to https://review.openstack.org/#/c/300175/ that would be appreciated. | 19:31 |
| mriedem | jaypipes: done | 19:32 |
| doffm | alaski: Do you think it would be worthwhile to add the insertion of cell0 to the mappings to one of the test fixtures? | 19:32 |
| alaski | doffm: I think so. for the most part we really just want to assume it's there | 19:34 |
| doffm | alaski: Thanks. I guess I'll add to Database fixture. | 19:36 |
| *** ociuhandu has joined #openstack-nova | 19:37 | |
| dansmith | doffm: alaski: you think it should be integrated at that level and not a fixture of its own or something? | 19:37 |
| dansmith | perhaps the flavors-exist-since-dinosaurs thing has me a little overly sensitive | 19:37 |
| doffm | dansmith: I really don't know. I will basically accept whatever you suggest here. | 19:38 |
| alaski | I don't have a strong stance on existing vs new fixture. just that a fixture would be good | 19:38 |
| dansmith | how many of our tests will have to have that mapping in place? | 19:39 |
| dansmith | only scheduling tests | 19:39 |
| doffm | Anything that touches the scheduling. | 19:39 |
| dansmith | or tests that hit the create api | 19:39 |
| doffm | Or building of instances really. | 19:39 |
| mriedem | so 90% of the functional tests | 19:39 |
| alaski | or list/show instances | 19:39 |
| dansmith | I dunno, I think it'd be nice to know what code requires that to be in place by being rather explicit | 19:39 |
| dansmith | mriedem: sure the functional test base should have it | 19:40 |
| *** vilobhmm11 has quit IRC | 19:40 | |
| *** krtaylor has quit IRC | 19:40 | |
| dansmith | alaski: list show through the api | 19:40 |
| dansmith | right? | 19:40 |
| *** vilobhmm11 has joined #openstack-nova | 19:40 | |
| alaski | right | 19:40 |
| dansmith | so of the non-mocked cases of that, I think it's almost all in fuctional tests, an then tests specifically testing this stuff | 19:40 |
| dansmith | er, unit tests specifically covering this | 19:41 |
| *** mgoddard has quit IRC | 19:41 | |
| dansmith | give how easy it is to just use a fixture if you need it | 19:41 |
| dansmith | I kinda think, separate fixture until it's clear that's too painful | 19:41 |
| dansmith | just MHO | 19:41 |
| rlrossit | dansmith: I hope your blast from the past works | 19:42 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Make neutronapi get_floating*() methods return objects https://review.openstack.org/115749 | 19:42 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Give the FloatingIP object an external_id field, because neutron https://review.openstack.org/115801 | 19:42 |
| alaski | dansmith: sure, seems reasonable to me | 19:42 |
| doffm | Ok. | 19:42 |
| *** ociuhandu has quit IRC | 19:43 | |
| *** sdake_ has joined #openstack-nova | 19:43 | |
| dansmith | rlrossit: I wish you luck | 19:43 |
| sdague | mriedem: https://review.openstack.org/#/c/299893/ what you asked for yesterday | 19:43 |
| sdague | and alex_xu pointed out that we actually do have a warning for the legacy v2.0 stack already | 19:44 |
| *** sdake has quit IRC | 19:44 | |
| sdague | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/__init__.py#L58 | 19:44 |
| *** ociuhandu has joined #openstack-nova | 19:44 | |
| mriedem | sdague: ahh, i was looking for something like that in https://github.com/openstack/nova/blob/master/nova/api/openstack/__init__.py yesterday | 19:45 |
| sdague | mriedem: yeh, me too, he caught in on the review that I pushed that added it another place | 19:46 |
| mriedem | oh sdague https://review.openstack.org/#/c/299893/2/nova/api/openstack/compute/versions.py | 19:47 |
| mriedem | you know i have to -2 b/c of that whitespace change right | 19:47 |
| mriedem | sdague: touch up the reno and i'll +W quick | 19:49 |
| *** jwcroppe has joined #openstack-nova | 19:49 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove the ability to disable v2.1 https://review.openstack.org/299893 | 19:50 |
| sdague | done, and cleaned up the whitespace | 19:51 |
| *** jwcroppe_ has quit IRC | 19:51 | |
| sdague | that's what happens when I delete a method, then have to add part of it back in | 19:51 |
| mriedem | +W | 19:51 |
| mriedem | thanks | 19:51 |
| mriedem | sdague: i guess you can just abandon the top patch | 19:54 |
| sdague | mriedem: yeh, I parked it for now mostly as a reminder | 19:54 |
| *** ihrachys has joined #openstack-nova | 19:54 | |
| -openstackstatus- NOTICE: Any jobs which erroneously failed on missing traceroute packages should be safe to recheck now | 19:55 | |
| *** sdake_ has quit IRC | 19:56 | |
| *** jaugustine has joined #openstack-nova | 19:57 | |
| *** mylu has joined #openstack-nova | 19:58 | |
| openstackgerrit | Saverio Proto proposed openstack/nova: Allow snapshot_volume_backed in pause/suspended https://review.openstack.org/295865 | 19:58 |
| sdague | mriedem: I'm looking at the summit planning etherpad | 19:59 |
| sdague | and I'm a bit concerned that "API: Policy/Capability Discoverbility and Microversion Docs" is 17 lbs of dirt in a 5 lb bag. There are like 20 bullets in that item | 19:59 |
| *** haomaiwang has quit IRC | 20:01 | |
| *** haomaiwa_ has joined #openstack-nova | 20:01 | |
| mriedem | sdague: that reminds me, i was going to look that over again before the meeting in an hour | 20:01 |
| mriedem | sdague: are we going to be talking about the api docs in the nova time-boxed sessions, or just friday meetup? are you going to be talking about any of that in the xp tuesday? | 20:02 |
| sdague | mriedem: I have a xp session proposed for that | 20:03 |
| sdague | I'd be fine with that and a small slice on friday | 20:03 |
| sdague | and encourage nova folks to come to the xp session | 20:03 |
| mriedem | ok, and the first session on thursday i penciled in ironic, which might also be a tuesday xp talk | 20:03 |
| sdague | I think we should dedicate a whole block to "discoverable policy in code" | 20:04 |
| sdague | that's going to be important to make sure we actually give it time | 20:04 |
| *** rlrossit has quit IRC | 20:05 | |
| sdague | I also proposed another bit on what's the actual path to nova network deprecation | 20:06 |
| mriedem | maybe we move johnthetubaguy's feature classification thing to thursday | 20:07 |
| mriedem | that would free up the last 2 slots on wed for api talks | 20:07 |
| sdague | yeh, that might be best. We definitely want to earmark one block entirely about policy in code | 20:07 |
| sdague | probably the first one | 20:07 |
| *** salv-orlando has joined #openstack-nova | 20:08 | |
| sdague | is os-brick mostly about privsep? | 20:08 |
| sdague | because privsep and libraries seems like a thing we should propose into xp day | 20:09 |
| mriedem | yeah, which i'm not sure there is actually anything to go over there | 20:09 |
| sdague | as we seem to continue to be confused here | 20:09 |
| mriedem | the cinder session is really going to be all multi-attach i think | 20:09 |
| sdague | about how anything lands | 20:09 |
| sdague | mriedem: yeh, I'd just call it multi-attach then | 20:09 |
| *** Sukhdev has quit IRC | 20:09 | |
| sdague | mriedem: ok, who has the todo to get privsep / os-brick onto the xp discussion list? | 20:10 |
| openstackgerrit | Saverio Proto proposed openstack/nova: Allow snapshot_volume_backed in pause/suspended https://review.openstack.org/295865 | 20:10 |
| mriedem | sdague: you want it? | 20:13 |
| mriedem | else i can add it | 20:14 |
| sdague | not really, I was actually just poking dims_ | 20:14 |
| *** jwcroppe has quit IRC | 20:14 | |
| *** baoli has joined #openstack-nova | 20:15 | |
| *** _diana_ is now known as diana_clarke | 20:15 | |
| *** efoley has joined #openstack-nova | 20:15 | |
| *** sileht has joined #openstack-nova | 20:15 | |
| *** smatzek has quit IRC | 20:15 | |
| *** baoli_ has joined #openstack-nova | 20:15 | |
| *** browne has quit IRC | 20:19 | |
| *** baoli has quit IRC | 20:19 | |
| *** ctrath has quit IRC | 20:20 | |
| dansmith | alaski: doffm: so, on this cell0 command | 20:21 |
| *** browne has joined #openstack-nova | 20:21 | |
| dansmith | alaski: doffm: I feel like that's mechanical enough that we can/should tie it into something else | 20:21 |
| dansmith | people will only ever run that once and on an empty database | 20:22 |
| dansmith | can't we, like, just do it at the end of api_db sync if we notice it's missing or something? | 20:22 |
| sdague | dansmith: ++ yes please :) | 20:22 |
| alaski | dansmith: how would you specify which db to use for cell0? | 20:22 |
| dansmith | alaski: well, we'd need something, but I just feel like we're talking about an explosion of new commands here | 20:23 |
| dansmith | and everyone will have to do this, | 20:23 |
| dansmith | not just multi-cell people | 20:23 |
| sdague | alaski: the connect strings have to be in the nova.conf already, right? | 20:23 |
| dansmith | well, | 20:23 |
| alaski | sdague: this can be a new database if desired | 20:23 |
| dansmith | yeah | 20:23 |
| alaski | dansmith: right, there is a little more to bootstrap nova with cells | 20:23 |
| dansmith | alaski: so we talked about a single command to move both dbs forward if you're a single-cell deploy | 20:24 |
| alaski | dansmith: if we can solve how to specify the db rolling it into something else seems great | 20:24 |
| dansmith | how about we make that command and do it there? | 20:24 |
| dansmith | I don't really mind if new godaddy type deploys have to run a command to make sure they have theirs split, | 20:24 |
| dansmith | but most people are going to want to run "get me a database" | 20:25 |
| dansmith | and do both sets of migrations and any other setup that they need to be able to go create their flavors (see what I did there?) and images | 20:25 |
| alaski | sure. in my head these have always been low level commands that we could layer a high level command on top of | 20:25 |
| *** penick has joined #openstack-nova | 20:25 | |
| sdague | yeh, I think it's important that we get that high level command before cell0 shows up | 20:26 |
| dansmith | alaski: we could also have a conf variable called "cell-o-one" that short circuits a lot of setup steps like this | 20:26 |
| *** fawadkhaliq has quit IRC | 20:27 | |
| dansmith | I dunno, I just ... feel like we're adding a ton of steps and we need to have a straightforward path for the 90% case | 20:27 |
| dansmith | and | 20:27 |
| dansmith | since this is't going to be done in newton, | 20:27 |
| dansmith | if we have a bunch of new shit people have to do just to upgrade to newton regardless | 20:27 |
| dansmith | that just seems uncool | 20:27 |
| *** jlopezgu has left #openstack-nova | 20:27 | |
| sdague | I have no concern having the ability to do more complicated things as long as there still is an easy path for the small scale case | 20:27 |
| *** fawadkhaliq has joined #openstack-nova | 20:27 | |
| sdague | dansmith: ++ | 20:27 |
| alaski | well, cell0 should be used in newton | 20:27 |
| dansmith | alaski: right, I know | 20:28 |
| dansmith | that means they'll have to do it | 20:28 |
| dansmith | even though no more than one cell in newton | 20:28 |
| *** krtaylor has joined #openstack-nova | 20:28 | |
| dansmith | and before someone else brings it up, the difference I see between this and default flavors is that this is not a thing that varies based on your size | 20:28 |
| dansmith | everyone needs this record | 20:28 |
| *** ihrachys has quit IRC | 20:28 | |
| *** apoorvad has joined #openstack-nova | 20:28 | |
| mriedem | dansmith: you can drop the -2 on this now https://review.openstack.org/#/c/248780/ | 20:29 |
| dansmith | mriedem: don | 20:29 |
| dansmith | e | 20:29 |
| alaski | dansmith: sdague so the options we're looking at are have the proposed nova-manage command with support for explicit db info, add a config for that which people can set and roll the setup into something else, all of the above and provide deployer choice. right? | 20:30 |
| sdague | mriedem: I'm going to have to drop before the meeting, anything you need out of me in advance of it? | 20:30 |
| alaski | dansmith: the reason I haven't been overly concerned about this is setting up cells is something that will require some form of interaction from deployers | 20:31 |
| alaski | we can ease the initial upgrade path, but there will always be commands for this | 20:31 |
| dansmith | alaski: right, but I don't want newton non-cell people with 10-node deploys to have to run a bunch of commands for cells when (a) they can't have two and (b) they don't need it | 20:31 |
| mriedem | sdague: don't think so, i was just going through the agenda, | 20:31 |
| mriedem | sdague: dansmith: alaski: jaypipes: johnthetubaguy: i'm hoping to meet (maybe a hangout) next week to go over the design summit schedule | 20:32 |
| sdague | right, the promiss of cells v2 was it was a seamless upgrade for non cells people | 20:32 |
| mriedem | sdague: when does the tuesday xp get voted on? | 20:32 |
| sdague | mriedem: next week | 20:32 |
| mriedem | tuesday's TC meeting? | 20:32 |
| alaski | mriedem: sounds good | 20:32 |
| sdague | probably later than that | 20:32 |
| dansmith | alaski: there is no way to re-create or change the record with this command, which means to me that its only ever run once, and I just feel like we could absorb multiple things into a "setup my cell" command or something | 20:32 |
| dansmith | alaski: which does the host mapping thing, and this, and whatever else | 20:33 |
| dansmith | I dunno | 20:33 |
| mriedem | sdague: hmm, kind of have to wait on the nova sessions being scheduled until after the tuesday stuff is in place | 20:33 |
| *** stevemar has quit IRC | 20:33 | |
| sdague | dansmith / alaski could we etherpad a set of pseudo commands that we'd expect people would have to run in a few different scenarios, and work backwards from that? | 20:33 |
| *** stevemar has joined #openstack-nova | 20:33 | |
| alaski | sdague: bauzas is working on that | 20:34 |
| *** cleong has quit IRC | 20:34 | |
| mriedem | i think the project sessions have to be in the schedule ~4/14, so i guess we could sort it out the week after next | 20:34 |
| alaski | dansmith: that's reasonable. I think where we differ is that I'm expecting a third command that wraps the cell0/host_mapping thing. which still allows for doing it all manually. | 20:34 |
| sdague | mriedem: right, I was intending to get this nailed next week, we should have in/out by wed. | 20:35 |
| sdague | alaski: ok, it seems like this conversation about how many commands, and of what type gets easier if we actually see them on paper somewhere | 20:35 |
| dansmith | alaski: that would be the fourth.. that's the concern I have, this is now three complicated commands that take db urls that everyone would have to run | 20:35 |
| *** cdelatte has quit IRC | 20:35 | |
| dansmith | alaski: for most people with only an api and single cell db, we should be able to automate all of that in a single "nova-manage cells setup --one-cell" or somethng | 20:36 |
| dansmith | and in reality, | 20:36 |
| dansmith | since cellsv1 people can't convert yet, | 20:36 |
| dansmith | *everyone* will run the same command I think | 20:36 |
| dansmith | so until they can be separate, it just seems like work for the sake of it | 20:36 |
| *** ccarmack has quit IRC | 20:36 | |
| *** vilobhmm11 has quit IRC | 20:37 | |
| dansmith | but, I agree we should lay out the steps as they are now and figure out what we can consolidate | 20:37 |
| *** vilobhmm11 has joined #openstack-nova | 20:37 | |
| *** vilobhmm11 has quit IRC | 20:37 | |
| alaski | yeah | 20:37 |
| alaski | I'm not opposing a single command to do it all. I just want to allow for power users to customize | 20:37 |
| sdague | alaski: right, I don't think anyone's said that wouldn't be an option | 20:38 |
| alaski | yeah, just seems like top-down vs bottom-up right now | 20:38 |
| dansmith | alaski: it's mostly about not wanting to risk releasing newton with no unified approach | 20:39 |
| dansmith | so I'd rather build with that from the start | 20:39 |
| dansmith | and anything we add right now, has to be done with an understanding of how the simple case goes | 20:39 |
| sdague | right, because retrofiting easy user experience late in the game never works, we need that use case in the mix as all the commands get designed | 20:40 |
| alaski | okay. my experience is very much the heavily customized deployment, so it's good to get this pushback | 20:41 |
| dansmith | yeah, I know, RAX _wants_ a bucket of parts :) | 20:41 |
| *** tbachman has joined #openstack-nova | 20:41 | |
| dansmith | none of my customers want that :) | 20:41 |
| *** unicell has quit IRC | 20:42 | |
| alaski | it seems easy enough to me, sync two dbs, setup a cell0, setup a first cell, then add cells as necessary. pretty much all scriptable | 20:42 |
| *** fawadkhaliq has quit IRC | 20:42 | |
| alaski | but I see your points | 20:42 |
| dansmith | heh | 20:42 |
| dansmith | alaski: you left out two steps you know | 20:43 |
| *** ayoung has quit IRC | 20:43 | |
| sdague | right, if only every web shop with a 10 node deploy had rax experienced ops team :) | 20:43 |
| alaski | sdague: only one way to get that experience :) | 20:43 |
| alaski | doffm: melwitt are y'all following along? | 20:44 |
| dansmith | use hard-to-use software? | 20:44 |
| alaski | I'm kidding, but sometimes you just do it and learn | 20:45 |
| *** fawadkhaliq has joined #openstack-nova | 20:45 | |
| *** artom has quit IRC | 20:45 | |
| mriedem | doffm is at buffalo wild wings | 20:46 |
| melwitt | alaski: sort of :) | 20:46 |
| alaski | the gist of it for me is that we should roll the logic from your nova-manage command and doffms cell0 command into one easy to use button | 20:47 |
| *** Bjoern_ has joined #openstack-nova | 20:47 | |
| *** Bjoern_ is now known as BjoernT | 20:48 | |
| melwitt | ah, okay | 20:48 |
| alaski | basically roll with all defaults at the start, and add customization controls later | 20:48 |
| alaski | though transport_url is still tricky here | 20:48 |
| dansmith | alaski: you can add the custom command alongside the thing that makes the unified command do the same thing with defaults | 20:49 |
| dansmith | I just want to see them built along the way | 20:49 |
| dansmith | not in n3 us sit back and say "how do we automate the mess we made?" | 20:49 |
| alaski | okay | 20:50 |
| *** ccarmack has joined #openstack-nova | 20:50 | |
| *** dave-mccowan has quit IRC | 20:51 | |
| *** browne has quit IRC | 20:51 | |
| *** tbachman has quit IRC | 20:53 | |
| *** browne has joined #openstack-nova | 20:54 | |
| *** rfolco has quit IRC | 20:56 | |
| *** stevemar_ has joined #openstack-nova | 20:56 | |
| *** knikolla has quit IRC | 20:56 | |
| *** stevemar has quit IRC | 20:56 | |
| mriedem | adrian_otto akuriata alevine alexpilotti aloga andreykurilin anteaya artom auggy | 20:58 |
| mriedem | bauzas belliott belmoreira bobball cburgess claudiub danpb dguitarbite _diana_ | 20:58 |
| mriedem | diana_clarke dims duncant edleafe efried flip214 funzo garyk gcb gjayavelu | 20:58 |
| mriedem | irina_pov jaypipes jcookekhugen jgrimm jichen jlvillal jroll kashyap klindgren | 20:58 |
| mriedem | krtaylor lbeliveau lxsli macsz markus_z mdorman med_ mikal mjturek mnestratov | 20:58 |
| mriedem | moshele mrda nagyz ndipanov neiljerram nic Nisha PaulMurray raildo rgeragnov | 20:58 |
| mriedem | sc68cal scottda sdague sileht sorrison swamireddy thomasem thorst tjones tonyb | 20:58 |
| mriedem | tpatil tpatzig xyang rdopiera sarafraj woodster | 20:58 |
| mriedem | nova meeting in 2 minutes | 20:58 |
| *** tbachman has joined #openstack-nova | 20:59 | |
| *** edmondsw has quit IRC | 21:00 | |
| *** mylu has quit IRC | 21:00 | |
| *** haomaiwa_ has quit IRC | 21:01 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Change BuildRequest to contain a serialized instance https://review.openstack.org/298455 | 21:01 |
| alaski | dansmith: I need to see what breaks, but ^ | 21:01 |
| dansmith | okay | 21:01 |
| *** haomaiwang has joined #openstack-nova | 21:01 | |
| *** aysyd has quit IRC | 21:02 | |
| alaski | it cleans things up quite a bit, but a lot had to change to get there | 21:03 |
| *** Goneri has quit IRC | 21:04 | |
| *** unicell has joined #openstack-nova | 21:05 | |
| *** zengine has quit IRC | 21:06 | |
| *** thorst has quit IRC | 21:07 | |
| *** thorst has joined #openstack-nova | 21:08 | |
| *** cdent has quit IRC | 21:08 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: config options: Centralize xenapi driver options https://review.openstack.org/300217 | 21:09 |
| tonyb | dansmith: So because I didn't know who Warren Haynes was I listened to a couple of albums the other day | 21:11 |
| *** mylu has joined #openstack-nova | 21:12 | |
| dansmith | tonyb: good man | 21:12 |
| tonyb | dansmith: I don't think going to a concert would be my cup'o'tea but I'd listen to the albums again | 21:12 |
| dansmith | tonyb: heh, okay | 21:12 |
| dansmith | tonyb: it's mostly continuous jamming for two hours | 21:12 |
| dansmith | so it's pretty awesome show material | 21:12 |
| *** thorst has quit IRC | 21:12 | |
| tonyb | dansmith: :) | 21:13 |
| dansmith | but kudos for at least exploring :) | 21:13 |
| * tonyb will listen to most things at least once | 21:13 | |
| *** thorst has joined #openstack-nova | 21:14 | |
| auggy | tonyb: dansmith iggy pop was pretty awesome | 21:14 |
| dansmith | well, I didn't see you show up at the venue with the real music, so I assumed it was at least tolerable | 21:15 |
| auggy | he had so much energy, i can only imagine what he must have been like when he was younger haha | 21:15 |
| dansmith | cocaine is quite the wonder drug | 21:15 |
| auggy | ha | 21:15 |
| tonyb | :) | 21:16 |
| auggy | i mean besides that, he was just an energetic guy i'm sure | 21:16 |
| dansmith | I figure it would really ruin the illusion if we ever saw each other on this side of the planet, much less in this town | 21:16 |
| *** wuhg has joined #openstack-nova | 21:16 | |
| *** baoli_ has quit IRC | 21:17 | |
| auggy | haha well in this case i even ducked out right at the encore | 21:17 |
| auggy | i strained my back a bit ago and i needed to head home :( | 21:17 |
| dansmith | so dd we, because "being trampled" is not part of the experience for me | 21:17 |
| auggy | luckily the iggy pop concert was a few blocks from my apartment | 21:17 |
| *** Sukhdev has joined #openstack-nova | 21:18 | |
| auggy | yeah i was not even gonna try to get out of keller in that crowd | 21:18 |
| dansmith | there are things you can yell to the crowd to help speed it up | 21:18 |
| dansmith | but you have to be ready for it | 21:18 |
| *** eharney has quit IRC | 21:18 | |
| dansmith | like "free flannel in the parking lot!" | 21:18 |
| *** thorst has quit IRC | 21:19 | |
| auggy | maybe that would work at your concert ;) | 21:19 |
| auggy | i think "free cocaine in the parking lot" would have worked better for mine hahaha | 21:19 |
| *** rmevans has joined #openstack-nova | 21:19 | |
| dansmith | heh, no, my concert was all old people | 21:19 |
| auggy | what, old people in portland don't wear flannel?? | 21:19 |
| dansmith | I just mean portland in general, but good call.. know your audience | 21:19 |
| dansmith | not these old people | 21:20 |
| auggy | oh right, these were all the yuppies from hillboro | 21:20 |
| auggy | actually i was surprised at the age range of the audience at iggy pop | 21:21 |
| auggy | like a good 1/3 were probably into the stooges back in the day | 21:21 |
| auggy | 1/3 maybe their grandparents were into the stooges | 21:21 |
| *** rtheis has quit IRC | 21:21 | |
| *** efried has quit IRC | 21:21 | |
| auggy | and then 1/3 genx'ers like me ;) | 21:21 |
| openstackgerrit | Gage Hugo proposed openstack/nova: Verify pci_passthrough_whitelist fields https://review.openstack.org/300219 | 21:22 |
| *** pmathews has joined #openstack-nova | 21:22 | |
| *** achanda has quit IRC | 21:22 | |
| mriedem | tonyb: you should have announced this in the nova meeting https://review.openstack.org/#/c/289255/ | 21:23 |
| *** mylu has quit IRC | 21:23 | |
| tonyb | mriedem: but we were going for a record! | 21:23 |
| mriedem | i know | 21:24 |
| tonyb | mriedem: you're right I shoudl have. | 21:24 |
| auggy | yeah i joined the meeting late and you all had left already :( | 21:24 |
| *** baoli has joined #openstack-nova | 21:24 | |
| * auggy missed the party :((( | 21:24 | |
| *** mylu has joined #openstack-nova | 21:27 | |
| *** vilobhmm11 has joined #openstack-nova | 21:28 | |
| *** knikolla has joined #openstack-nova | 21:28 | |
| *** haplo37 has joined #openstack-nova | 21:29 | |
| mriedem | has anyone figured out if there is easy public transit from the austin airport to the hotel areas? | 21:29 |
| auggy | when i was in austin a couple years ago for YAPC::NA the public transit was, erm, lacking | 21:29 |
| *** sdake has joined #openstack-nova | 21:30 | |
| alaski | I was provided this link http://www.capmetro.org/airport/, but I haven't looked into it yet | 21:30 |
| mriedem | maybe i'll just pay mtreinish to pick me up in his rental ferrari | 21:30 |
| mriedem | ha | 21:31 |
| mriedem | $1.75 Premium fare | 21:31 |
| mriedem | if premium is $1.75, what's standard? | 21:31 |
| auggy | https://goo.gl/maps/FgWEv4nnm4n | 21:31 |
| sarafraj | mriedem: Austin lacks in public transport especially from airport. I suggest using ride hailing apps. | 21:31 |
| mtreinish | I have a rental ferrari? | 21:31 |
| mriedem | sarafraj: even that capmetro thing isn't good? | 21:32 |
| mriedem | i thought austin was overrun with hippy liberals, this is like a crazy paradox | 21:32 |
| auggy | it looks like ther's "100-Airport Flyer-OB" | 21:32 |
| auggy | that's the bus that came up in the google maps i plugged in | 21:33 |
| sarafraj | mriedem: I am from Austin and I prefer uber or lyft over all others | 21:33 |
| manjeets | mriedem this time I have osapi_max_limit = 4000 in nova.conf i checked conf after re deployment but still see 1000 returned by nova list | 21:33 |
| mriedem | manjeets: did you restart the nova-api service? | 21:33 |
| manjeets | i re deployed openstack | 21:33 |
| manjeets | i did nova_api restart first it did not work then i just re deployed | 21:34 |
| auggy | wow that capmetro thing looks pretty sweet | 21:34 |
| auggy | that's what gmaps had suggested, i just didn't know what 100 Airport Flyer was | 21:34 |
| mriedem | manjeets: ok, well, not sure then. but it's end of working hours for me, so i'm signing off. | 21:35 |
| *** mriedem has quit IRC | 21:35 | |
| manjeets | mriedem no worries | 21:35 |
| *** fawadkhaliq has quit IRC | 21:38 | |
| *** ivase has quit IRC | 21:39 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Add Service.get_minimum_version_multi() for multiple binaries https://review.openstack.org/279312 | 21:39 |
| *** fawadkhaliq has joined #openstack-nova | 21:39 | |
| jaypipes | dansmith: ^^ just rebasing... | 21:40 |
| *** fawadkhaliq has quit IRC | 21:40 | |
| *** fawadkhaliq has joined #openstack-nova | 21:41 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Move resource provider staticmethods to proxies https://review.openstack.org/299526 | 21:41 |
| *** mdrabe has quit IRC | 21:41 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Migrate compute node resource information to Inventory objects https://review.openstack.org/279313 | 21:41 |
| *** rushiagr has quit IRC | 21:42 | |
| *** haplo37 has quit IRC | 21:42 | |
| *** cdelatte has joined #openstack-nova | 21:43 | |
| *** fawadkhaliq has quit IRC | 21:44 | |
| *** fawadkhaliq has joined #openstack-nova | 21:44 | |
| *** burt has quit IRC | 21:44 | |
| *** zenoway has quit IRC | 21:45 | |
| *** gyee has quit IRC | 21:46 | |
| *** rushiagr has joined #openstack-nova | 21:47 | |
| *** tojuvone_ has joined #openstack-nova | 21:47 | |
| *** angdraug has joined #openstack-nova | 21:48 | |
| *** gyee has joined #openstack-nova | 21:48 | |
| *** burgerk has quit IRC | 21:49 | |
| *** tojuvone has quit IRC | 21:51 | |
| *** gagehugo has quit IRC | 21:52 | |
| *** jaugustine has quit IRC | 21:55 | |
| *** mylu has quit IRC | 21:56 | |
| *** krtaylor has quit IRC | 21:57 | |
| *** zenoway has joined #openstack-nova | 21:57 | |
| *** artom has joined #openstack-nova | 21:59 | |
| artom | dansmith, still around? Can we quickly discuss https://review.openstack.org/#/c/264015/20/nova/objects/metadata.py@76 ? | 21:59 |
| *** jschlueter has joined #openstack-nova | 22:00 | |
| dansmith | artom: which part? | 22:00 |
| dansmith | I haven't looked at the comments since my last reply | 22:00 |
| *** haomaiwang has quit IRC | 22:01 | |
| *** xyang1 has quit IRC | 22:01 | |
| artom | dansmith, the to_dict method generating guest-visible stuff, thus making it an API | 22:01 |
| *** haomaiwang has joined #openstack-nova | 22:01 | |
| *** amotoki has joined #openstack-nova | 22:01 | |
| dansmith | okay | 22:01 |
| *** zenoway has quit IRC | 22:02 | |
| artom | dansmith, so, I definitely agree that having API stuff buried in objects is dangerous. | 22:03 |
| dansmith | yeah | 22:04 |
| *** BjoernT has quit IRC | 22:04 | |
| dansmith | I think it probably needs to be closer to the metadata code we have now | 22:04 |
| *** unicell has quit IRC | 22:04 | |
| dansmith | a consumer of the objects and not part of them | 22:04 |
| *** ametts has quit IRC | 22:09 | |
| artom | dansmith, right, so in https://review.openstack.org/#/c/264018/19/nova/api/metadata/base.py@346 instead of just calling to_dict, get the fields one by one and stick the values in the outputted dict... | 22:10 |
| *** Goneri has joined #openstack-nova | 22:10 | |
| artom | (My initial reasoning was having the object be responsible for their own attributes, but this is clearly a case where that doesn't apply) | 22:10 |
| dansmith | artom: right, well, the organization is nice of course, I just think it's setting us up to fail | 22:10 |
| artom | And then add some tests in functional/api_sample_tests... | 22:11 |
| dansmith | yeah, something to make sure the format sticks | 22:11 |
| artom | dansmith, oh yeah. I read your comments and though "Damn, dude's got a point." :) | 22:11 |
| dansmith | I also think you probably want to be more structural with your handling of them and not just loop over the fields, | 22:12 |
| dansmith | because that means we add something and it just goes in, or we change the type or something and it's reflected there | 22:12 |
| *** Bjoern_ has joined #openstack-nova | 22:12 | |
| dansmith | but I say move them over there and then we can argue about the method | 22:12 |
| artom | Yeah, I wasn't planning to... Essentially, we can't "trust" the object in the metadata handler | 22:12 |
| dansmith | right | 22:12 |
| dansmith | it's model, not view | 22:12 |
| artom | 'tis settled then. | 22:14 |
| artom | vladikr, if you're seeing this ^^^ | 22:14 |
| dansmith | I wil try to figure out what vladikr's deal is with the base class too | 22:15 |
| dansmith | tomorrowish | 22:15 |
| *** thorst has joined #openstack-nova | 22:16 | |
| artom | The registered stuff? He was saying the problem is that other example of non registered base classes aren't used as fields in other objects. | 22:16 |
| artom | So it looks like this field-ness is what's causing problems here | 22:16 |
| dansmith | ah | 22:16 |
| dansmith | well, then we have to fix the test | 22:16 |
| *** erlon has quit IRC | 22:17 | |
| *** vladikr has quit IRC | 22:18 | |
| *** cdelatte has quit IRC | 22:19 | |
| *** andymaier has quit IRC | 22:25 | |
| *** vilobhmm11 has quit IRC | 22:26 | |
| *** vladikr has joined #openstack-nova | 22:27 | |
| *** vilobhmm11 has joined #openstack-nova | 22:27 | |
| *** ducttape_ has quit IRC | 22:30 | |
| *** dimtruck is now known as zz_dimtruck | 22:31 | |
| edleafe | auggy: (reading scrollback) I've seen Iggy many times. The last was in 1980! | 22:34 |
| claudiub | hah, interesting news... :D http://fossbytes.com/microsoft-buys-canonical-kills-ubuntu-linux-forever/?fb_action_ids=1102810479758847&fb_action_types=og.comments | 22:35 |
| claudiub | *empire music starts playing* | 22:36 |
| edleafe | claudiub: A little early on this side of the world... | 22:36 |
| tbachman | lol | 22:37 |
| tbachman | let the fun begin :) | 22:37 |
| tbachman | I know I’ll fall prey to one of these | 22:37 |
| *** thorst has quit IRC | 22:37 | |
| *** baoli has quit IRC | 22:38 | |
| *** unicell has joined #openstack-nova | 22:38 | |
| *** dave-mccowan has joined #openstack-nova | 22:40 | |
| *** vladikr_ has joined #openstack-nova | 22:43 | |
| *** vladikr has quit IRC | 22:44 | |
| *** ayoung has joined #openstack-nova | 22:45 | |
| *** lpetrut has quit IRC | 22:46 | |
| *** rmevans has quit IRC | 22:49 | |
| *** dave-mccowan has quit IRC | 22:50 | |
| *** sdague has quit IRC | 22:50 | |
| *** akshai has quit IRC | 22:51 | |
| *** browne has quit IRC | 22:55 | |
| *** sdake has quit IRC | 22:57 | |
| *** penick has quit IRC | 22:58 | |
| *** browne has joined #openstack-nova | 22:59 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** akshai has joined #openstack-nova | 23:02 | |
| *** haomaiwa_ has joined #openstack-nova | 23:02 | |
| *** rwmjones has quit IRC | 23:03 | |
| *** akshai_ has joined #openstack-nova | 23:03 | |
| *** akshai has quit IRC | 23:06 | |
| *** rwmjones has joined #openstack-nova | 23:09 | |
| *** sdake has joined #openstack-nova | 23:09 | |
| *** Bjoern_ has quit IRC | 23:10 | |
| *** knikolla has quit IRC | 23:10 | |
| *** salv-orlando has quit IRC | 23:11 | |
| *** sdake_ has joined #openstack-nova | 23:12 | |
| *** sdake has quit IRC | 23:13 | |
| *** amotoki has quit IRC | 23:13 | |
| *** pwnall has quit IRC | 23:15 | |
| *** tjones_zzzz has left #openstack-nova | 23:17 | |
| *** READ10 has joined #openstack-nova | 23:24 | |
| *** ayoung has quit IRC | 23:26 | |
| *** pradk has quit IRC | 23:26 | |
| *** efried has joined #openstack-nova | 23:34 | |
| *** mriedem has joined #openstack-nova | 23:36 | |
| * tbachman wonders if his april fools joke will be getting +2’s on his nova patch :-) | 23:38 | |
| *** thorst has joined #openstack-nova | 23:38 | |
| tbachman | I guess it would be funnier to get -2s | 23:38 |
| *** rk4n has quit IRC | 23:39 | |
| tbachman | which is worse — a false -2, or a false +2? | 23:39 |
| * tbachman goes back to his lurking corner | 23:39 | |
| jroll | false -2 is worse, a false +2 means you only have to bribe one more core | 23:40 |
| jroll | :P | 23:40 |
| tbachman | lol | 23:41 |
| *** sdake_ has quit IRC | 23:43 | |
| *** ivase has joined #openstack-nova | 23:46 | |
| *** thorst has quit IRC | 23:46 | |
| *** ivase has quit IRC | 23:50 | |
| *** knikolla has joined #openstack-nova | 23:53 | |
| *** cdelatte has joined #openstack-nova | 23:53 | |
| *** unicell has quit IRC | 23:57 | |
| *** Bofu2MBP has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!