| melwitt | dansmith: how did the fields get undirtied before that point? I see it's using create_instance=False to skip the create earlier | 00:02 |
|---|---|---|
| melwitt | the BuildRequest create I guess. this is confusing | 00:03 |
| dansmith | melwitt: correct | 00:04 |
| dansmith | same reason BDMs were getting reset | 00:04 |
| dansmith | I had encountered some of this a couple weeks ago, | 00:05 |
| melwitt | and a clone wouldn't work because it's not dirty | 00:05 |
| dansmith | and was just focusing on the end result since lots of this goes away shortly | 00:05 |
| dansmith | but alas. | 00:05 |
| openstackgerrit | Dan Smith proposed openstack/nova: Return build_requests instead of instances https://review.openstack.org/374267 | 00:05 |
| openstackgerrit | Dan Smith proposed openstack/nova: Refactor two nearly useless secgroup tests https://review.openstack.org/393438 | 00:05 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make RPCFixture support multiple connections https://review.openstack.org/396417 | 00:05 |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix context on loaded BDMs from build_request https://review.openstack.org/401007 | 00:05 |
| melwitt | then you got a bunch of us that didn't know any of this stuff. yay :P | 00:05 |
| * dansmith ragequits for the evening | 00:06 | |
| melwitt | aww, sorry dansmith | 00:06 |
| *** victorhe has quit IRC | 00:10 | |
| *** victorhe has joined #openstack-nova | 00:11 | |
| *** Sukhdev has quit IRC | 00:16 | |
| *** browne has joined #openstack-nova | 00:17 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Reset instance status on invalid host https://review.openstack.org/401009 | 00:24 |
| openstackgerrit | zhangyanxian proposed openstack/nova: Fix typo in image_meta.py & checks.py & flavor.py https://review.openstack.org/393664 | 00:35 |
| *** stewie925 has quit IRC | 00:37 | |
| *** TravT has joined #openstack-nova | 00:38 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Reset instance status on invalid host https://review.openstack.org/401009 | 00:39 |
| *** siva_krish has quit IRC | 00:39 | |
| *** tuanluong has joined #openstack-nova | 00:43 | |
| *** TravT has quit IRC | 00:44 | |
| *** ijw has quit IRC | 00:45 | |
| *** browne has quit IRC | 00:51 | |
| *** anush has quit IRC | 00:52 | |
| *** huanxie has joined #openstack-nova | 00:53 | |
| *** rama_y has joined #openstack-nova | 00:54 | |
| *** mriedem has joined #openstack-nova | 00:57 | |
| *** mdrabe has quit IRC | 00:57 | |
| *** tovin07 has joined #openstack-nova | 00:59 | |
| *** tovin07 has quit IRC | 00:59 | |
| *** tovin07 has joined #openstack-nova | 01:00 | |
| *** ijw has joined #openstack-nova | 01:00 | |
| *** tovin07 has quit IRC | 01:01 | |
| *** tovin07 has joined #openstack-nova | 01:01 | |
| *** tovin07 has quit IRC | 01:02 | |
| *** tovin07 has joined #openstack-nova | 01:02 | |
| *** mlavalle has quit IRC | 01:02 | |
| *** siva_krish has joined #openstack-nova | 01:11 | |
| *** ducnc has joined #openstack-nova | 01:12 | |
| *** penick has joined #openstack-nova | 01:14 | |
| *** penick_ has joined #openstack-nova | 01:15 | |
| *** siva_krish has quit IRC | 01:19 | |
| *** penick has quit IRC | 01:19 | |
| *** penick_ is now known as penick | 01:19 | |
| *** yuntongjin has joined #openstack-nova | 01:28 | |
| *** Alagar has joined #openstack-nova | 01:31 | |
| mriedem | hmm, volume attach returns a 200 code but it's an rpc cast to the compute to attach the volume, not sure if we can change the 200 to a 202 without a microversion | 01:32 |
| mriedem | oomichi: gmann: alex_xu: ^? | 01:32 |
| *** nic has quit IRC | 01:32 | |
| * mriedem consults the magic 8 ball for when a microversion is required | 01:32 | |
| oomichi | mriedem: maybe tempest gets angry if changing it without microversion bump | 01:33 |
| mriedem | oomichi: i think tempest only cares about 400+ errors | 01:33 |
| *** victorhe has quit IRC | 01:34 | |
| mriedem | tempest tests for volume attach are checking the volume status and waiting for it to go to 'in-use' | 01:34 |
| mriedem | i mean, i could test it out and see | 01:34 |
| *** dave-mccowan has joined #openstack-nova | 01:35 | |
| oomichi | mriedem: https://github.com/openstack/tempest/blob/master/tempest/lib/api_schema/response/compute/v2_1/servers.py#L289 | 01:35 |
| mriedem | oh no | 01:35 |
| mriedem | that's a list, so it could be modified to accept 202 too? | 01:35 |
| oomichi | mriedem: yeah, it is possible technically, but please don't do that | 01:36 |
| *** ljxiash_ has quit IRC | 01:37 | |
| mriedem | oh come on :) | 01:37 |
| mriedem | ok so i guess that's a microversion bump | 01:37 |
| mriedem | gross | 01:37 |
| oomichi | mriedem: I can see your point, the requirement is valid. | 01:38 |
| oomichi | mriedem: And you can see the similar invalid status code if running '$ grep TODO * | grep 20' under nova/api/openstack/compute | 01:39 |
| oomichi | yeah, gross I know :-( | 01:39 |
| mriedem | yeah i see http://paste.openstack.org/show/590154/ | 01:41 |
| *** anush has joined #openstack-nova | 01:41 | |
| mriedem | i thought we had a spec to clean these all up at some point, maybe alex or jichen had that | 01:41 |
| mriedem | might have been a bug, i don't remember | 01:41 |
| oomichi | mriedem: yeah, that is. I don't want to change each status code on each microversion bump, that seems more gross | 01:42 |
| oomichi | yeah, maybe that is right | 01:42 |
| oomichi | now alex_xu's morning. lets wait for him :) | 01:43 |
| mriedem | i'll just add a todo here so we don't forget this one too | 01:43 |
| oomichi | mriedem: ++ | 01:43 |
| *** mtanino has joined #openstack-nova | 01:43 | |
| *** kaisers has quit IRC | 01:43 | |
| *** ducttape_ has joined #openstack-nova | 01:44 | |
| *** kaisers has joined #openstack-nova | 01:44 | |
| *** swamireddy has joined #openstack-nova | 01:45 | |
| *** diogogmt has quit IRC | 01:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add TODO for returning a 202 from the volume attach API https://review.openstack.org/401031 | 01:47 |
| *** gcb has joined #openstack-nova | 01:51 | |
| *** yufei has joined #openstack-nova | 01:52 | |
| *** ljxiash has joined #openstack-nova | 01:56 | |
| *** ljxiash has quit IRC | 01:57 | |
| *** ljxiash has joined #openstack-nova | 01:57 | |
| alex_xu | mriedem: we can't change that without Microversion | 01:57 |
| mriedem | alex_xu: oomichi told me that was never possible in a thousand years | 01:58 |
| alex_xu | heh | 01:58 |
| oomichi | mriedem: alex_xu: how about giving up accurate success code? All success codes should be 200 only, that is very easy and consistent ;) | 01:59 |
| *** huanxie has quit IRC | 02:01 | |
| mriedem | accuracy is overrated | 02:01 |
| *** zhangjl has joined #openstack-nova | 02:08 | |
| *** mirrorbo1 has quit IRC | 02:08 | |
| *** mirrorbo1 has joined #openstack-nova | 02:08 | |
| *** anush has quit IRC | 02:09 | |
| *** dave-mcc_ has joined #openstack-nova | 02:11 | |
| *** dave-mccowan has quit IRC | 02:12 | |
| *** dave-mccowan has joined #openstack-nova | 02:13 | |
| *** anush has joined #openstack-nova | 02:13 | |
| openstackgerrit | zhaolihui proposed openstack/nova: Fix failed to boot vm with pci_alias's name including space https://review.openstack.org/392077 | 02:13 |
| *** dave-mcc_ has quit IRC | 02:16 | |
| mriedem | there are a couple of easy changes with +2s at the bottom of this series https://review.openstack.org/#/c/386844/17 | 02:16 |
| mriedem | for custom resource classes | 02:16 |
| *** hieulq has joined #openstack-nova | 02:18 | |
| *** dave-mccowan has quit IRC | 02:18 | |
| *** diga has quit IRC | 02:21 | |
| *** hieulq has quit IRC | 02:21 | |
| *** hieulq has joined #openstack-nova | 02:23 | |
| *** gongysh has joined #openstack-nova | 02:24 | |
| *** ducttape_ has quit IRC | 02:25 | |
| alex_xu | oomichi: that means we need change all the 202/201 back to 200 ;) | 02:26 |
| *** yuntongjin has quit IRC | 02:26 | |
| oomichi | alex_xu: yeah, I did wonder proposing it to api-wg, but I avoided. If you are interested, I don't stop ;) | 02:27 |
| *** ducttape_ has joined #openstack-nova | 02:28 | |
| *** yuntongjin has joined #openstack-nova | 02:29 | |
| *** jamielennox is now known as jamielennox|away | 02:30 | |
| *** thorst has joined #openstack-nova | 02:32 | |
| *** thorst has quit IRC | 02:32 | |
| *** thorst has joined #openstack-nova | 02:33 | |
| *** shuquan has joined #openstack-nova | 02:34 | |
| mriedem | oomichi: btw, claudiu got a patch up to make the hyperv driver wait for vif plugged events from neutron https://review.openstack.org/#/c/399001/ - that was related to that tempest patch you asked me about a couple of weeks ago | 02:36 |
| mriedem | i think it makes the tempest patch less necessary, at least for hyperv on master | 02:36 |
| mriedem | which i see you've -2ed https://review.openstack.org/#/c/383049/ | 02:36 |
| oomichi | mriedem: cool, thanks for picking it up. I am checking it before going back home today | 02:37 |
| *** dave-mccowan has joined #openstack-nova | 02:39 | |
| *** anush has quit IRC | 02:39 | |
| *** shuquan_ has joined #openstack-nova | 02:39 | |
| *** shuquan has quit IRC | 02:40 | |
| *** thorst has quit IRC | 02:40 | |
| *** thorst has joined #openstack-nova | 02:41 | |
| *** thorst has quit IRC | 02:45 | |
| *** kaisers__ has joined #openstack-nova | 02:46 | |
| *** mriedem has quit IRC | 02:46 | |
| *** dave-mccowan has quit IRC | 02:48 | |
| *** kaisers_ has quit IRC | 02:49 | |
| *** gjayavelu has quit IRC | 02:52 | |
| *** Alagar has quit IRC | 02:52 | |
| *** ducttape_ has quit IRC | 02:52 | |
| *** shuquan_ has quit IRC | 02:52 | |
| *** Alagar has joined #openstack-nova | 02:53 | |
| *** fragatina has quit IRC | 02:53 | |
| *** shuquan has joined #openstack-nova | 02:54 | |
| *** Alagar has quit IRC | 02:55 | |
| *** diogogmt has joined #openstack-nova | 02:56 | |
| *** yufei has quit IRC | 02:57 | |
| *** alex_xu__ has joined #openstack-nova | 02:58 | |
| *** yufei has joined #openstack-nova | 02:58 | |
| *** Alagar has joined #openstack-nova | 02:58 | |
| *** jamielennox|away is now known as jamielennox | 02:59 | |
| *** shuquan_ has joined #openstack-nova | 02:59 | |
| *** shuquan has quit IRC | 02:59 | |
| *** shuquan_ has quit IRC | 03:04 | |
| *** brault has quit IRC | 03:04 | |
| *** shuquan has joined #openstack-nova | 03:04 | |
| *** ducttape_ has joined #openstack-nova | 03:05 | |
| *** thorst has joined #openstack-nova | 03:09 | |
| *** thorst has quit IRC | 03:10 | |
| *** bkopilov has quit IRC | 03:15 | |
| *** jwcroppe has joined #openstack-nova | 03:23 | |
| *** jwcroppe has quit IRC | 03:24 | |
| *** kylek3h_ has quit IRC | 03:24 | |
| *** jwcroppe has joined #openstack-nova | 03:24 | |
| *** jwcroppe has quit IRC | 03:24 | |
| *** anush has joined #openstack-nova | 03:24 | |
| *** kylek3h has joined #openstack-nova | 03:25 | |
| *** jwcroppe has joined #openstack-nova | 03:25 | |
| *** catintheroof has joined #openstack-nova | 03:27 | |
| *** kylek3h has quit IRC | 03:29 | |
| *** yamahata has quit IRC | 03:29 | |
| *** liusheng has quit IRC | 03:33 | |
| Alagar | good morning all | 03:33 |
| Alagar | iam using devstack script for setup openstack environment | 03:34 |
| Alagar | when ever i could restart the openstack virtual machine i could see install_os_domU.sh automaticly running and seting up the script again | 03:34 |
| *** unicell has quit IRC | 03:34 | |
| *** ducttape_ has quit IRC | 03:34 | |
| *** thorst has joined #openstack-nova | 03:36 | |
| *** thorst has quit IRC | 03:37 | |
| *** ducttape_ has joined #openstack-nova | 03:38 | |
| *** catintheroof has quit IRC | 03:40 | |
| *** ducttape_ has quit IRC | 03:40 | |
| *** links has joined #openstack-nova | 03:42 | |
| *** armax has quit IRC | 03:46 | |
| *** ljxiash has quit IRC | 03:50 | |
| *** yufei has quit IRC | 03:51 | |
| *** fragatina has joined #openstack-nova | 03:51 | |
| *** psachin has joined #openstack-nova | 03:52 | |
| *** fragatina has quit IRC | 03:52 | |
| *** fragatina has joined #openstack-nova | 03:52 | |
| *** yufei has joined #openstack-nova | 03:53 | |
| *** shuquan has quit IRC | 03:53 | |
| *** kevinz has joined #openstack-nova | 03:54 | |
| *** Guest83036 has joined #openstack-nova | 03:56 | |
| *** sudipto_ has joined #openstack-nova | 03:56 | |
| *** yuntongjin has quit IRC | 03:56 | |
| *** ijw has quit IRC | 03:56 | |
| *** shuquan has joined #openstack-nova | 04:01 | |
| *** tbachman has quit IRC | 04:01 | |
| *** markvoelker has quit IRC | 04:02 | |
| *** kevinz has quit IRC | 04:04 | |
| *** huanxie has joined #openstack-nova | 04:06 | |
| *** mdnadeem has joined #openstack-nova | 04:08 | |
| *** shuquan_ has joined #openstack-nova | 04:09 | |
| *** prashkre has joined #openstack-nova | 04:10 | |
| *** jgriffith is now known as jgriffith_away | 04:10 | |
| *** shuquan has quit IRC | 04:10 | |
| *** ljxiash has joined #openstack-nova | 04:11 | |
| *** siva_krish has joined #openstack-nova | 04:15 | |
| *** ljxiash has quit IRC | 04:15 | |
| *** rama_y has quit IRC | 04:16 | |
| *** GB21 has joined #openstack-nova | 04:17 | |
| *** anush has quit IRC | 04:18 | |
| *** yufei has quit IRC | 04:19 | |
| *** GB21 has quit IRC | 04:22 | |
| *** Alagar has quit IRC | 04:24 | |
| *** ijw has joined #openstack-nova | 04:24 | |
| *** shuquan_ has quit IRC | 04:24 | |
| *** kylek3h has joined #openstack-nova | 04:25 | |
| *** ijw has quit IRC | 04:29 | |
| *** yamahata has joined #openstack-nova | 04:29 | |
| *** kylek3h has quit IRC | 04:30 | |
| *** GB21 has joined #openstack-nova | 04:34 | |
| *** bkopilov has joined #openstack-nova | 04:37 | |
| *** Guest83036 has quit IRC | 04:41 | |
| *** sudipto_ has quit IRC | 04:41 | |
| *** yamahata has quit IRC | 04:43 | |
| *** Alagar has joined #openstack-nova | 04:44 | |
| *** tlian has quit IRC | 04:44 | |
| *** moshele has joined #openstack-nova | 04:47 | |
| *** ratailor has joined #openstack-nova | 04:48 | |
| *** lpetrut has joined #openstack-nova | 04:54 | |
| *** prashkre has quit IRC | 04:55 | |
| *** gjayavelu has joined #openstack-nova | 04:56 | |
| *** josecastroleon has joined #openstack-nova | 04:56 | |
| *** jogo has joined #openstack-nova | 05:02 | |
| *** jogo has quit IRC | 05:02 | |
| *** jogo has joined #openstack-nova | 05:02 | |
| *** ayogi has joined #openstack-nova | 05:02 | |
| *** ljxiash has joined #openstack-nova | 05:03 | |
| *** gjayavelu has quit IRC | 05:04 | |
| *** sree has joined #openstack-nova | 05:04 | |
| *** yamahata has joined #openstack-nova | 05:05 | |
| *** victorhe has joined #openstack-nova | 05:05 | |
| *** gjayavelu has joined #openstack-nova | 05:06 | |
| *** Jeffrey4l has quit IRC | 05:06 | |
| *** victorhe has quit IRC | 05:06 | |
| *** victorhe has joined #openstack-nova | 05:07 | |
| *** shuquan has joined #openstack-nova | 05:09 | |
| *** cfriesen has quit IRC | 05:11 | |
| *** klkumar has joined #openstack-nova | 05:12 | |
| *** moshele has quit IRC | 05:12 | |
| *** unicell has joined #openstack-nova | 05:14 | |
| *** yuntongjin has joined #openstack-nova | 05:16 | |
| *** victorhe has quit IRC | 05:16 | |
| *** shuquan has quit IRC | 05:16 | |
| *** Alagar has quit IRC | 05:16 | |
| *** shuquan has joined #openstack-nova | 05:18 | |
| *** prateek has joined #openstack-nova | 05:18 | |
| *** Alagar has joined #openstack-nova | 05:20 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port xenapi test_xenapi to Python 3 https://review.openstack.org/401065 | 05:20 |
| *** jogo has quit IRC | 05:22 | |
| *** unicell has quit IRC | 05:26 | |
| *** kylek3h has joined #openstack-nova | 05:26 | |
| *** unicell has joined #openstack-nova | 05:26 | |
| *** shuquan_ has joined #openstack-nova | 05:29 | |
| *** shuquan has quit IRC | 05:30 | |
| *** sandanar has joined #openstack-nova | 05:30 | |
| *** kylek3h has quit IRC | 05:31 | |
| *** mfeoktistov has joined #openstack-nova | 05:34 | |
| *** sandanar_ has joined #openstack-nova | 05:34 | |
| *** victorhe has joined #openstack-nova | 05:37 | |
| *** sandanar has quit IRC | 05:38 | |
| *** victorhe has quit IRC | 05:42 | |
| *** lpetrut has quit IRC | 05:48 | |
| *** gongysh has quit IRC | 05:49 | |
| *** sudipto_ has joined #openstack-nova | 05:50 | |
| *** Guest83036 has joined #openstack-nova | 05:50 | |
| *** Alagar has quit IRC | 05:52 | |
| *** alex_xu__ has quit IRC | 05:56 | |
| *** moshele has joined #openstack-nova | 05:56 | |
| *** huanxie has quit IRC | 05:59 | |
| *** markvoelker has joined #openstack-nova | 06:02 | |
| *** liusheng has joined #openstack-nova | 06:07 | |
| *** GnomeKing has joined #openstack-nova | 06:07 | |
| *** markvoelker has quit IRC | 06:08 | |
| *** GnomeKing is now known as Bobba | 06:10 | |
| *** diana_clarke has quit IRC | 06:10 | |
| *** Alagar has joined #openstack-nova | 06:12 | |
| *** yufei has joined #openstack-nova | 06:13 | |
| *** shuquan_ has quit IRC | 06:13 | |
| *** shuquan has joined #openstack-nova | 06:14 | |
| *** siva_krish has quit IRC | 06:18 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:19 | |
| *** jogo has joined #openstack-nova | 06:20 | |
| *** jogo has quit IRC | 06:20 | |
| *** jogo has joined #openstack-nova | 06:20 | |
| *** kylek3h has joined #openstack-nova | 06:27 | |
| *** kylek3h has quit IRC | 06:31 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port virt vmwareapi tests to Python 3 https://review.openstack.org/401080 | 06:32 |
| *** tojuvone has quit IRC | 06:34 | |
| *** sridharg has joined #openstack-nova | 06:35 | |
| *** karthiks has quit IRC | 06:42 | |
| *** oanson has joined #openstack-nova | 06:43 | |
| *** msno has joined #openstack-nova | 06:46 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Notifications on flavor operations https://review.openstack.org/398171 | 06:47 |
| msno | Hi Guyz.. | 06:47 |
| msno | i have a HP Helion Openstack setup which is running in a opensaf cluster setup | 06:47 |
| msno | a strange issue of VMs getting deleted is happening .. in the setup | 06:48 |
| msno | not able to trace how its been triggered.. any pointers. | 06:48 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Transform instance.resize_prep notification https://review.openstack.org/396988 | 06:48 |
| *** markus_z has joined #openstack-nova | 06:50 | |
| *** karthiks has joined #openstack-nova | 06:56 | |
| *** ekuris has joined #openstack-nova | 06:57 | |
| *** jdurgin1 has joined #openstack-nova | 06:57 | |
| *** shuquan_ has joined #openstack-nova | 07:00 | |
| *** shuquan has quit IRC | 07:01 | |
| *** prashkre has joined #openstack-nova | 07:03 | |
| *** kwathore_ has quit IRC | 07:03 | |
| *** mjura has joined #openstack-nova | 07:03 | |
| *** kwathore has joined #openstack-nova | 07:04 | |
| *** mjura has quit IRC | 07:08 | |
| *** qwertyco has joined #openstack-nova | 07:08 | |
| *** mjura has joined #openstack-nova | 07:08 | |
| *** tjones has quit IRC | 07:09 | |
| *** tesseract has joined #openstack-nova | 07:12 | |
| *** Alagar has quit IRC | 07:12 | |
| *** Alagar has joined #openstack-nova | 07:12 | |
| *** tesseract is now known as Guest59528 | 07:13 | |
| *** Guest2_ has joined #openstack-nova | 07:13 | |
| *** shuquan_ has quit IRC | 07:16 | |
| *** nkrinner_afk is now known as nkrinner | 07:17 | |
| *** nkrinner has quit IRC | 07:19 | |
| *** paul-carlton has joined #openstack-nova | 07:20 | |
| *** GB21 has quit IRC | 07:21 | |
| *** nkrinner has joined #openstack-nova | 07:24 | |
| ratailor | is it a good candidate to backport to newton ? https://review.openstack.org/#/c/399672/ | 07:25 |
| *** mfeoktistov has quit IRC | 07:26 | |
| openstackgerrit | liaonanhai proposed openstack/nova: Use uuidutils instead of uuid.uuid4() https://review.openstack.org/394775 | 07:27 |
| *** kylek3h has joined #openstack-nova | 07:28 | |
| *** nkrinner has quit IRC | 07:29 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: live-migration test hook for serial console https://review.openstack.org/347471 | 07:29 |
| *** nkrinner has joined #openstack-nova | 07:29 | |
| *** gouthamr has joined #openstack-nova | 07:31 | |
| *** kylek3h has quit IRC | 07:32 | |
| *** brault has joined #openstack-nova | 07:39 | |
| *** yamahata has quit IRC | 07:39 | |
| *** zenoway has joined #openstack-nova | 07:39 | |
| markus_z | mikal: Do you maybe have a chance to look at https://review.openstack.org/#/c/385367/ please? | 07:40 |
| openstackgerrit | bailin.zhang proposed openstack/nova: Fix the file permissions of test_compute_mgr.py https://review.openstack.org/401109 | 07:45 |
| *** lpetrut has joined #openstack-nova | 07:46 | |
| *** GB21 has joined #openstack-nova | 07:53 | |
| openstackgerrit | chenyingnan proposed openstack/nova: Remove mox from tests/unit/api/openstack/compute/test_extended_volumes.py https://review.openstack.org/401112 | 07:54 |
| *** matrohon has joined #openstack-nova | 07:56 | |
| *** josecastroleon has quit IRC | 07:57 | |
| *** victorhe has joined #openstack-nova | 07:58 | |
| *** nherciu has joined #openstack-nova | 08:01 | |
| *** victorhe has quit IRC | 08:02 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_metadata to Python 3 https://review.openstack.org/398750 | 08:02 |
| openstackgerrit | Moshe Levi proposed openstack/nova: Enhance PCI passthrough whitelist to support regex https://review.openstack.org/199488 | 08:05 |
| *** dimtruck is now known as zz_dimtruck | 08:11 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_keypairs to Python 3 https://review.openstack.org/398909 | 08:11 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port libvirt test_driver to Python 3 https://review.openstack.org/400697 | 08:11 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port virt vmwareapi tests to Python 3 https://review.openstack.org/401080 | 08:11 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port xenapi test_xenapi to Python 3 https://review.openstack.org/401065 | 08:11 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port ironic unit tests to Python 3 https://review.openstack.org/398948 | 08:12 |
| *** liverpooler has quit IRC | 08:13 | |
| *** Guest2_ has quit IRC | 08:14 | |
| *** ihrachys has joined #openstack-nova | 08:14 | |
| *** yuntongjin has quit IRC | 08:16 | |
| *** diga has joined #openstack-nova | 08:18 | |
| *** ralonsoh has joined #openstack-nova | 08:20 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 08:21 |
| mlakat | johnthetubaguy ping | 08:25 |
| *** ekuris has quit IRC | 08:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Transform instance.resize_prep notification https://review.openstack.org/396988 | 08:27 |
| *** kylek3h has joined #openstack-nova | 08:28 | |
| *** jaosorior has joined #openstack-nova | 08:29 | |
| *** kylek3h has quit IRC | 08:33 | |
| *** victorhe has joined #openstack-nova | 08:34 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Optimize creating security_group https://review.openstack.org/315311 | 08:34 |
| *** victorhe has quit IRC | 08:38 | |
| *** ekuris has joined #openstack-nova | 08:41 | |
| *** amoralej|off is now known as amoralej | 08:43 | |
| *** ihrachys has quit IRC | 08:45 | |
| *** josecastroleon has joined #openstack-nova | 08:46 | |
| *** Alagar has quit IRC | 08:48 | |
| *** kubix has quit IRC | 08:53 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** sree has quit IRC | 09:01 | |
| *** gongysh has joined #openstack-nova | 09:01 | |
| *** sree has joined #openstack-nova | 09:01 | |
| *** diga has quit IRC | 09:02 | |
| *** zzzeek has joined #openstack-nova | 09:02 | |
| *** willise has joined #openstack-nova | 09:03 | |
| *** sree is now known as sreeram_v | 09:04 | |
| *** gouthamr has quit IRC | 09:04 | |
| *** sreeram_v is now known as sree | 09:05 | |
| *** rmart04 has joined #openstack-nova | 09:07 | |
| *** paul-carlton has quit IRC | 09:07 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: raise exception if create virtuozzo container with swap disk https://review.openstack.org/397829 | 09:07 |
| *** sree is now known as sreeram_v | 09:10 | |
| *** moshele has quit IRC | 09:13 | |
| *** ekuris has quit IRC | 09:14 | |
| *** ihrachys has joined #openstack-nova | 09:20 | |
| johnthetubaguy | mlakat: hey hows things? | 09:20 |
| *** ihrachys has quit IRC | 09:20 | |
| *** fragatina has quit IRC | 09:20 | |
| *** ihrachys has joined #openstack-nova | 09:20 | |
| *** fragatina has joined #openstack-nova | 09:20 | |
| *** liverpooler has joined #openstack-nova | 09:23 | |
| mlakat | fine, thanks. wanted to ask if you could look at this bug description: https://bugs.launchpad.net/horizon/+bug/1638581 mainly was interested if any changes are about to be made in nova around the messy root disk size and boot from volume world. | 09:24 |
| openstack | Launchpad bug 1638581 in OpenStack Dashboard (Horizon) "Flavor with zero disk cannot be selected while booting from volume" [Undecided,In progress] - Assigned to Mateusz Kowalski (makowals) | 09:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 09:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Remove 'vendordata_driver' opt https://review.openstack.org/397835 | 09:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Trivial fix of indentation in 'api' https://review.openstack.org/397834 | 09:27 |
| *** kylek3h has joined #openstack-nova | 09:29 | |
| *** ekuris has joined #openstack-nova | 09:30 | |
| *** sreeram_v is now known as sree | 09:30 | |
| *** lucas-afk is now known as lucasagomes | 09:31 | |
| *** sree is now known as sreeram_v | 09:32 | |
| *** markvoelker has joined #openstack-nova | 09:32 | |
| *** kylek3h has quit IRC | 09:34 | |
| johnthetubaguy | mlakat: to follow up on the public channel, its all about this patch series: https://review.openstack.org/#/q/topic:bp/flavor-root-disk-none I would talk to tpatzig_1 about helping refresh that series | 09:35 |
| *** gjayavelu has quit IRC | 09:35 | |
| mlakat | johnthetubaguy, thanks, will do! | 09:36 |
| *** markvoelker has quit IRC | 09:37 | |
| *** derekh has joined #openstack-nova | 09:41 | |
| *** yassine has joined #openstack-nova | 09:46 | |
| *** yassine is now known as Guest58074 | 09:46 | |
| *** pilgrimstack has joined #openstack-nova | 09:47 | |
| gibi | johnthetubaguy: hi! There is test only bug fix with already a +2 , could you check it and +W it if looks good to you? https://review.openstack.org/#/c/344334/ | 09:50 |
| *** pilgrimstack1 has joined #openstack-nova | 09:51 | |
| *** pilgrimstack has quit IRC | 09:51 | |
| *** victorhe has joined #openstack-nova | 09:54 | |
| johnthetubaguy | gibi: trying to go through all the patches with +2s already this morning | 09:55 |
| *** liverpooler has quit IRC | 09:56 | |
| *** gjayavelu has joined #openstack-nova | 09:56 | |
| tpatzig_1 | mlakat: the whole feature is more or less ready. only a missing tempest test. and i need to rebase the stuff | 09:57 |
| gibi | johnthetubaguy: thanks | 09:58 |
| *** preethipy has joined #openstack-nova | 10:00 | |
| *** paul-carlton has joined #openstack-nova | 10:01 | |
| *** gjayavelu has quit IRC | 10:01 | |
| mlakat | tpatzig_1, thanks, I'll look at the changes. | 10:01 |
| *** victorhe has quit IRC | 10:03 | |
| johnthetubaguy | +1 what tpatzig_1, looks mostly like a rebase + a tempest test, and its done | 10:05 |
| *** ftersin has joined #openstack-nova | 10:07 | |
| *** zhangjl has quit IRC | 10:11 | |
| *** gjayavelu has joined #openstack-nova | 10:12 | |
| *** gongysh has quit IRC | 10:15 | |
| *** gjayavelu has quit IRC | 10:16 | |
| *** liuyulong has joined #openstack-nova | 10:20 | |
| lyarwood | mdbooth: would you mind taking a look at https://review.openstack.org/#/c/399735/ this morning if you have time? | 10:20 |
| mdbooth | lyarwood: Looking now | 10:20 |
| lyarwood | kashyap: ^ could you also take a look, again if you have time | 10:20 |
| * lyarwood is looking at the func test failure | 10:21 | |
| *** jamielennox is now known as jamielennox|away | 10:21 | |
| kashyap | lyarwood: In a moment. Was being pinged by QEMU migration people for some OpenStack question | 10:21 |
| lyarwood | ah ha https://review.openstack.org/#/c/394585/ | 10:21 |
| lyarwood | kashyap: kk np | 10:21 |
| *** liverpooler has joined #openstack-nova | 10:24 | |
| *** sambetts|afk is now known as sambetts | 10:24 | |
| mdbooth | lyarwood: How long has qemu supported this flag, btw? | 10:24 |
| lyarwood | mdbooth: I'm not sure when QEMU introduced it, libvirt has support since 1.2.7 listed in the XML docs | 10:25 |
| lyarwood | mdbooth: but this is all optional anyway, if it isn't present we just use the cur and end markers from blockjobinfo | 10:26 |
| lyarwood | urgh I didn't update the comment to use element instead of flag | 10:27 |
| *** ihrachys has quit IRC | 10:28 | |
| *** sahid has joined #openstack-nova | 10:28 | |
| *** rfolco has joined #openstack-nova | 10:28 | |
| *** claudiub|2 has joined #openstack-nova | 10:29 | |
| mdbooth | lyarwood: Hmm, docs aren't 100% clear: | 10:30 |
| mdbooth | The attribute ready, if present, tracks progress of the job: yes if the disk is known to be ready to pivot, or, since 1.2.7, abort or pivot if the job is in the process of completing. | 10:30 |
| mdbooth | That might mean that abort is since 1.2.7, but mirror itself is older | 10:30 |
| *** tojuvone has joined #openstack-nova | 10:30 | |
| mdbooth | Sorry, ready is older | 10:31 |
| mdbooth | Meh, anyway as you say it doesn't matter. | 10:31 |
| lyarwood | ah, good point, assumed it was 1.2.7 | 10:32 |
| *** gjayavelu has joined #openstack-nova | 10:32 | |
| *** pkoniszewski has quit IRC | 10:35 | |
| *** isq has quit IRC | 10:35 | |
| *** gjayavelu has quit IRC | 10:37 | |
| gcb | johnthetubaguy, alex_xu : please check https://etherpad.openstack.org/p/nova_py35 when you're free :-) | 10:38 |
| *** zhangjn has joined #openstack-nova | 10:40 | |
| *** qwertyco has quit IRC | 10:40 | |
| *** zhangjn has quit IRC | 10:40 | |
| *** tuanluong has quit IRC | 10:40 | |
| mdbooth | You make your first gerrit comment of the day. You've diligently typed a 53 page essay in the comment box and you click save. At this point gerrit helpfully tells you what it should have told you when you opened the comment box, which is that you're no longer logged in. The text in the comment box is no longer selectable, clicking either of the 2 presented options will lose your 53 page essay. Cue wailing and gnashing of teeth. | 10:41 |
| *** zhangjn has joined #openstack-nova | 10:41 | |
| mdbooth | TOP TIP: hit escape! | 10:41 |
| *** zhangjn has quit IRC | 10:41 | |
| * lyarwood awaits his 53 page review comment | 10:41 | |
| mdbooth | This will dismiss the annoying intrusion. You can copy your text elsewhere, log in, then paste it back again. | 10:41 |
| *** zhangjn has joined #openstack-nova | 10:41 | |
| *** yufei has quit IRC | 10:41 | |
| *** zhangjn has quit IRC | 10:41 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: remove deprecated nova quota options https://review.openstack.org/395706 | 10:42 |
| *** zhangjn has joined #openstack-nova | 10:43 | |
| *** zhangjn has quit IRC | 10:43 | |
| *** zhangjn has joined #openstack-nova | 10:44 | |
| *** zhangjn has quit IRC | 10:44 | |
| *** pkoniszewski has joined #openstack-nova | 10:44 | |
| *** zhangjn has joined #openstack-nova | 10:44 | |
| *** zhangjn has quit IRC | 10:44 | |
| tobasco | critical bug in new ubuntu UCA stable/liberty release, cannot spawn instances anymore | 10:45 |
| tobasco | https://bugs.launchpad.net/nova/+bug/1644187 | 10:45 |
| openstack | Launchpad bug 1644187 in nova (Ubuntu) "ValueError on creating new nova instance" [Undecided,New] | 10:45 |
| kashyap | mdbooth: I've been super wary lately that wherever I tend to write beyond a paragraph, I just turn to a local text file, then copy / paste in Bugzilla / whatever. | 10:46 |
| mdbooth | tobasco: Are UCA bugs reported here? Don't they have their own support? | 10:46 |
| *** ihrachys has joined #openstack-nova | 10:46 | |
| kashyap | Yeah, was wondering that, too. They have their own IRC channel, too, IIRC | 10:46 |
| tobasco | mdbooth: sorry, just pushing it through everywhere, didn't think... | 10:47 |
| mdbooth | tobasco: This may be a bug in Nova, but if it's not vanilla you should probably start with your vendor. | 10:47 |
| lyarwood | stable/kilo is EOL btw so you really need to start with your vendor. | 10:47 |
| tobasco | lyarwood: it's liberty tho | 10:48 |
| *** cdent has joined #openstack-nova | 10:48 | |
| lyarwood | urgh, sorry | 10:48 |
| mdbooth | ValueError: invalid literal for int() with base 10: '' | 10:49 |
| mdbooth | I hate python | 10:49 |
| lyarwood | tobasco: liberty is about to be | 10:49 |
| lyarwood | tobasco: apologies for the confusion. | 10:49 |
| cdent | put python loves you mdbooth | 10:49 |
| cdent | oops, that would be "but" | 10:50 |
| cdent | I hate typing | 10:50 |
| mdbooth | hehe | 10:50 |
| tobasco | lyarwood: yea think this was the last update also, going mitaka after that | 10:50 |
| *** gjayavelu has joined #openstack-nova | 10:53 | |
| *** ihrachys has quit IRC | 10:53 | |
| *** kevinz has joined #openstack-nova | 10:55 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Detach volumes from a domain before detaching any encryptors https://review.openstack.org/398479 | 10:56 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: encryptors: Introduce encryption provider constants https://review.openstack.org/393901 | 10:57 |
| *** gjayavelu has quit IRC | 10:57 | |
| *** emil1 has joined #openstack-nova | 10:59 | |
| cdent | has the vmware ci been declared dead in the water yet? | 11:00 |
| openstackgerrit | liaonanhai proposed openstack/nova: Use uuidutils instead of uuid.uuid4() https://review.openstack.org/394775 | 11:02 |
| *** avolkov has quit IRC | 11:02 | |
| lyarwood | I don't think so, Intel NFV is also pretty broken atm | 11:02 |
| *** liverpooler has quit IRC | 11:03 | |
| *** avolkov has joined #openstack-nova | 11:03 | |
| *** ducnc has quit IRC | 11:05 | |
| * cdent nods | 11:05 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 11:11 |
| openstackgerrit | sahid proposed openstack/nova: numa: fit instance toplogy with cpu overhead onto cores https://review.openstack.org/400875 | 11:12 |
| openstackgerrit | sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 11:12 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 11:12 |
| openstackgerrit | sahid proposed openstack/nova: numa: take into account cpus overhead https://review.openstack.org/400876 | 11:12 |
| openstackgerrit | sahid proposed openstack/nova: numa: update numa usage incording the cpu overhead https://review.openstack.org/400877 | 11:12 |
| wznoinsk | lyarwood, cdent we've retriggered failed jobs for Intel NFV CI | 11:13 |
| *** gjayavelu has joined #openstack-nova | 11:13 | |
| lyarwood | wznoinsk: thanks :) | 11:13 |
| *** GB21 has quit IRC | 11:14 | |
| *** kevinz has quit IRC | 11:14 | |
| *** Jeffrey4l has joined #openstack-nova | 11:14 | |
| *** igordcard has quit IRC | 11:14 | |
| lyarwood | mdbooth: did gerrit eat your review? np if you haven't posted yet, just wanted to make sure it hasn't gone AWOL. | 11:15 |
| *** kevinz has joined #openstack-nova | 11:15 | |
| *** igordcard has joined #openstack-nova | 11:15 | |
| cdent | thanks wznoinsk. This CI business is hard enough to be a bit worrying :) | 11:15 |
| *** gjayavelu has quit IRC | 11:18 | |
| *** sreeram_v has quit IRC | 11:18 | |
| *** sreeram_v has joined #openstack-nova | 11:19 | |
| wznoinsk | cdent, true, especially when you leaven -1 Verified slops :) | 11:20 |
| *** sreeram_v has quit IRC | 11:23 | |
| kashyap | lyarwood: How's Gertty treating you lately? | 11:23 |
| kashyap | Been using it consistently/ | 11:23 |
| kashyap | s/\//? | 11:24 |
| lyarwood | kashyap: yeah, I find myself restarting it each morning but it's normally fine for the rest of the day then. Added a few things to help with stable reviews (https://review.openstack.org/#/q/project:openstack/gertty+owner:%22Lee+Yarwood+%253Clyarwood%2540redhat.com%253E%22) and overall really enjoying it. | 11:26 |
| * lyarwood should really rebase and update his install | 11:26 | |
| *** Guest83036 has quit IRC | 11:26 | |
| *** sudipto_ has quit IRC | 11:26 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_serversV21 to Python 3 https://review.openstack.org/401207 | 11:27 |
| kashyap | Ah, nice | 11:27 |
| openstackgerrit | Abhishek Kekane proposed openstack/nova: Use is_valid_cidr and is_valid_ipv6_cidr from oslo_utils https://review.openstack.org/320375 | 11:28 |
| *** GB21 has joined #openstack-nova | 11:30 | |
| *** kevinz has quit IRC | 11:33 | |
| cdent | kashyap: I really wanted to like gertty, forced myself to use it for a while, but could never adapt properly, and it frequently did inscrutable things that I couldn't get to repeat, so I sadly gave up :( | 11:34 |
| *** kevinz has joined #openstack-nova | 11:34 | |
| *** kylek3h has joined #openstack-nova | 11:34 | |
| *** ratailor has quit IRC | 11:35 | |
| *** sridharg has quit IRC | 11:37 | |
| *** prateek has quit IRC | 11:37 | |
| kashyap | cdent: Yeah, I hear you. I even package it for Fedora, but like Lee said, most times I have to re-create the SQLite DB | 11:38 |
| *** kylek3h has quit IRC | 11:39 | |
| *** josecastroleon has quit IRC | 11:40 | |
| *** takedakn has joined #openstack-nova | 11:40 | |
| *** sridharg has joined #openstack-nova | 11:40 | |
| *** Guest58074 has quit IRC | 11:41 | |
| *** lpetrut has quit IRC | 11:42 | |
| *** lpetrut has joined #openstack-nova | 11:42 | |
| *** alexpilotti has joined #openstack-nova | 11:43 | |
| *** alexpilo_ has quit IRC | 11:43 | |
| *** victorhe has joined #openstack-nova | 11:49 | |
| *** mvk has quit IRC | 11:49 | |
| *** liverpooler has joined #openstack-nova | 11:51 | |
| *** claudiub has joined #openstack-nova | 11:52 | |
| *** sridharg has quit IRC | 11:53 | |
| *** victorhe has quit IRC | 11:53 | |
| *** nicolasbock has joined #openstack-nova | 11:54 | |
| *** claudiub|2 has quit IRC | 11:54 | |
| mdbooth | lyarwood: Reviewed :( | 11:58 |
| *** bkopilov has quit IRC | 11:58 | |
| *** gszasz has joined #openstack-nova | 11:59 | |
| mdbooth | lyarwood: See narrative in my pseudish code comment | 11:59 |
| *** mkoderer has joined #openstack-nova | 11:59 | |
| mdbooth | I'm not sure this works at all, but that may just be my stomach complaining | 11:59 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port libvirt test_imagecache to Python 3 https://review.openstack.org/401223 | 12:00 |
| *** emil1 has quit IRC | 12:03 | |
| *** openstackgerrit has quit IRC | 12:03 | |
| *** openstackgerrit has joined #openstack-nova | 12:03 | |
| *** liverpooler has quit IRC | 12:04 | |
| *** raildo has joined #openstack-nova | 12:05 | |
| *** GB21 has quit IRC | 12:05 | |
| *** prateek has joined #openstack-nova | 12:05 | |
| lyarwood | mdbooth: thanks and it does already work but I get your point regarding fetching the domain XML each time | 12:06 |
| *** jaypipes has quit IRC | 12:06 | |
| lyarwood | mdbooth: mirror is going to be not None even if ready isn't present | 12:07 |
| *** prateek has quit IRC | 12:07 | |
| lyarwood | mdbooth: that's why I added the config class instead of just adding mirror_ready to the disk object | 12:07 |
| *** prateek has joined #openstack-nova | 12:08 | |
| lyarwood | mdbooth: I'll have to look into the ready == abort case, I assumed that would only be set if we aborted the job for some reason | 12:08 |
| *** yassine has joined #openstack-nova | 12:08 | |
| * lyarwood adds the above to the review | 12:08 | |
| *** catintheroof has joined #openstack-nova | 12:09 | |
| *** yassine is now known as Guest22022 | 12:09 | |
| *** prateek has quit IRC | 12:10 | |
| *** GB21 has joined #openstack-nova | 12:12 | |
| *** alexpilotti has quit IRC | 12:12 | |
| *** alexpilotti has joined #openstack-nova | 12:12 | |
| *** sudipto_ has joined #openstack-nova | 12:13 | |
| *** Guest83036 has joined #openstack-nova | 12:13 | |
| mdbooth | lyarwood: Does this handle the race condition | 12:13 |
| mdbooth | ? | 12:13 |
| mdbooth | The issue we were getting in the race was that cur == end, but the mirror was *not* ready | 12:14 |
| mdbooth | So we were ending pre-emptively | 12:14 |
| kashyap | Exactly | 12:14 |
| mdbooth | The problem is that we can't differentiate between ready just not having been set yet | 12:14 |
| mdbooth | And ready not being supported by this version of libvirt | 12:14 |
| *** catintheroof has quit IRC | 12:15 | |
| kashyap | Merely deciding on the singular check of cur == end has what burnt us | 12:15 |
| *** gongysh has joined #openstack-nova | 12:15 | |
| lyarwood | mdbooth: yes it does | 12:15 |
| mdbooth | So I'm not sure this fixes the race, unless we can say that we will *always* wait for ready to be set | 12:15 |
| kashyap | mdbooth: You raise difficult questions | 12:15 |
| mdbooth | But that would break on older libvirt | 12:15 |
| lyarwood | mdbooth: if mirror is present then we can wait for ready to show up | 12:16 |
| *** prashkre has quit IRC | 12:16 | |
| lyarwood | mdbooth: that's the current logic | 12:16 |
| lyarwood | mdbooth: and it's working in the gate and locally for me thus far | 12:16 |
| *** prateek has joined #openstack-nova | 12:17 | |
| *** gongysh has quit IRC | 12:17 | |
| lyarwood | mdbooth: that's also backwards compat as both mirror and ready were added at the same time AFAIK | 12:17 |
| *** GB21 has quit IRC | 12:17 | |
| mdbooth | lyarwood: The issue we're trying to fix is a race, so N passes in the gate doesn't mean much unfortunately | 12:17 |
| *** prateek has quit IRC | 12:17 | |
| mdbooth | In that case, I think we need more historical context than we currently have | 12:18 |
| *** prateek has joined #openstack-nova | 12:18 | |
| lyarwood | sure I can add that into the commit and comment above this block | 12:18 |
| lyarwood | I'll respin anyway to reduce the amount of XML fetching | 12:18 |
| mdbooth | We need to know exactly what we can assume about libvirt 1.2.1, which is the current min version | 12:18 |
| *** mvk has joined #openstack-nova | 12:19 | |
| mdbooth | We need to ensure that the presence of mirror assures us of the eventual presence of ready | 12:19 |
| *** sridharg has joined #openstack-nova | 12:19 | |
| lyarwood | mdbooth: I'm 99% sure it does but I'll confirm | 12:19 |
| kashyap | Need to excavate the libvirt sources to find out exactly from which version libvirt supports the 'ready' element supported, no? | 12:19 |
| kashyap | s/supported// | 12:20 |
| mdbooth | Is there a race where we might fetch this info before mirror has been added to the domain xml? | 12:20 |
| *** catintheroof has joined #openstack-nova | 12:20 | |
| mdbooth | In which case we might erroneously conclude that it is not supported | 12:20 |
| lyarwood | I think ae6aa8c3965e9aaa245b8e669c6324d44312ac1b introduced it all, /me checks again | 12:20 |
| mdbooth | lyarwood: Sorry to be a pita, but as always with this crap the devil is in the detail. | 12:21 |
| *** zenoway has quit IRC | 12:21 | |
| lyarwood | mdbooth: np I'd rather get this correct first time ;) | 12:21 |
| *** zenoway has joined #openstack-nova | 12:21 | |
| *** lucasagomes is now known as lucas-hungry | 12:21 | |
| kashyap | lyarwood: Yep, you're right | 12:21 |
| kashyap | (On the Git commit ID) | 12:22 |
| kashyap | So, it's been there since v0.10.0 | 12:22 |
| lyarwood | lovely | 12:22 |
| lyarwood | thanks kashyap | 12:22 |
| kashyap | lyarwood: Np. Just did a quick `git tag --contains ae6aa8c` | 12:23 |
| mdbooth | kashyap: That's both mirror and ready? | 12:23 |
| *** josecastroleon has joined #openstack-nova | 12:24 | |
| *** prateek has quit IRC | 12:24 | |
| mdbooth | lyarwood: If we are assured that mirror and ready are both supported, we should ditch the fallback. | 12:24 |
| mdbooth | It can only cause problems. | 12:24 |
| *** prateek has joined #openstack-nova | 12:25 | |
| kashyap | mdbooth: Yes, it is. "This is done with a new <mirror> sub-element to <disk>, as in:" | 12:25 |
| mdbooth | johnthetubaguy: Prod re https://review.openstack.org/#/c/383859/ | 12:25 |
| kashyap | (Quoting from the commit message) | 12:25 |
| mdbooth | johnthetubaguy: Hopefully pre-thanksgiving prod :) | 12:25 |
| *** zenoway has quit IRC | 12:25 | |
| kashyap | mdbooth: Full commit in its glory - http://libvirt.org/git/?p=libvirt.git;a=commitdiff;h=ae6aa8c | 12:26 |
| *** yuntongjin has joined #openstack-nova | 12:26 | |
| *** rtheis has joined #openstack-nova | 12:26 | |
| *** rtheis has quit IRC | 12:26 | |
| mdbooth | kashyap: Excellent, so ready was always present, and the feature is supported in the min version. | 12:27 |
| mdbooth | lyarwood: Can you link to that in the commit message, btw? | 12:27 |
| lyarwood | mdbooth: yup, I'll sort that out now and drop the other workarounds. | 12:28 |
| kashyap | Yes, linking to that commit would be useful - years later when some poor sod looks at that, they'll appreciate it | 12:29 |
| *** gouthamr has joined #openstack-nova | 12:32 | |
| *** prashkre has joined #openstack-nova | 12:32 | |
| *** mdnadeem has quit IRC | 12:33 | |
| *** kylek3h has joined #openstack-nova | 12:36 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.add_fixed_ip notification https://review.openstack.org/332876 | 12:36 |
| *** bvanhav has joined #openstack-nova | 12:37 | |
| *** prashkre has quit IRC | 12:39 | |
| *** victorhe has joined #openstack-nova | 12:39 | |
| *** ihrachys has joined #openstack-nova | 12:39 | |
| *** kylek3h has quit IRC | 12:40 | |
| *** takedakn has quit IRC | 12:42 | |
| *** GB21 has joined #openstack-nova | 12:43 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port security related tests to Python 3 https://review.openstack.org/401241 | 12:43 |
| *** victorhe has quit IRC | 12:43 | |
| *** catintheroof has quit IRC | 12:44 | |
| *** tbachman has joined #openstack-nova | 12:50 | |
| alex_xu | nova-api sub-team meeting is in 10 mins at #openstack-meeting-4 | 12:50 |
| *** kylek3h has joined #openstack-nova | 12:52 | |
| *** oanson has quit IRC | 12:53 | |
| *** jaypipes has joined #openstack-nova | 12:53 | |
| *** links has quit IRC | 12:54 | |
| *** sreeram_v has joined #openstack-nova | 12:57 | |
| alex_xu | johnthetubaguy: are you around for api meeting? | 13:00 |
| *** sreeram_v has quit IRC | 13:02 | |
| *** yuntongjin has quit IRC | 13:02 | |
| *** ljxiash has quit IRC | 13:02 | |
| *** willise has quit IRC | 13:04 | |
| *** nherciu_ has joined #openstack-nova | 13:04 | |
| *** victorhe has joined #openstack-nova | 13:05 | |
| *** yufei has joined #openstack-nova | 13:06 | |
| *** GB21 has quit IRC | 13:06 | |
| *** zenoway has joined #openstack-nova | 13:08 | |
| *** nherciu has quit IRC | 13:08 | |
| *** anush has joined #openstack-nova | 13:09 | |
| jaypipes | gibi: ping | 13:09 |
| *** mtanino has quit IRC | 13:09 | |
| jaypipes | gibi: how we doing with notifications reviews? | 13:09 |
| jaypipes | gibi: which patch is most critical at the moment? | 13:09 |
| gibi | jaypipes: hi! give me 10 minutes, I will ping you | 13:10 |
| jaypipes | np | 13:10 |
| *** victorhe has quit IRC | 13:10 | |
| *** preethipy has quit IRC | 13:14 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 13:15 |
| *** rmstar has quit IRC | 13:16 | |
| *** nandal has joined #openstack-nova | 13:16 | |
| *** ihrachys has quit IRC | 13:17 | |
| *** Guest59528 has quit IRC | 13:19 | |
| *** amoralej is now known as amoralej|lunch | 13:20 | |
| gibi | jaypipes: ping, I'm here now | 13:20 |
| gibi | jaypipes: thanks for the reviews | 13:21 |
| gibi | jaypipes: the https://etherpad.openstack.org/p/ocata-nova-priorities-tracking has a list of notification patches that is ready for review | 13:21 |
| jaypipes | gibi: yup, I have been going through those. | 13:21 |
| gibi | jaypipes: but there is no real priority order | 13:21 |
| jaypipes | gibi: ok | 13:21 |
| gibi | jaypipes: they are pretty independent patches | 13:22 |
| jaypipes | got it. will review the remainder today. | 13:22 |
| gibi | jaypipes: there is one patch that might be in limbo | 13:22 |
| gibi | jaypipes: https://review.openstack.org/#/c/390950/ this has +2 and +W but I don't see it on the gate | 13:23 |
| gibi | jaypipes: Is it safe to just recheck it? | 13:23 |
| gibi | jaypipes: or it needs a second +2? | 13:23 |
| *** lucas-hungry is now known as lucasagomes | 13:23 | |
| *** anush has quit IRC | 13:26 | |
| *** prateek_ has joined #openstack-nova | 13:26 | |
| jaypipes | gibi: safe to just recheck. I did so. | 13:27 |
| *** ljxiash has joined #openstack-nova | 13:28 | |
| *** markvoelker has joined #openstack-nova | 13:28 | |
| jaypipes | gibi: one question I had for you and julian... | 13:29 |
| jaypipes | gibi: https://review.openstack.org/#/c/396225/4/nova/tests/functional/notification_sample_tests/notification_sample_base.py | 13:29 |
| jaypipes | gibi: is there a better way other than waiting in a sleep loop to grab that notification? | 13:29 |
| jaypipes | gibi: perhaps a fixture that short-circuits actual message sending? | 13:29 |
| *** prateek has quit IRC | 13:30 | |
| gibi | jaypipes: you mean patching the notifiction part of nova.rpc and getting a callback in the test instead of polling? | 13:31 |
| *** ljxiash has quit IRC | 13:31 | |
| *** ljxiash has joined #openstack-nova | 13:32 | |
| *** Guest83036 has quit IRC | 13:32 | |
| *** sudipto_ has quit IRC | 13:32 | |
| *** victorhe has joined #openstack-nova | 13:33 | |
| jaypipes | gibi: ya | 13:33 |
| gibi | jaypipes: anyhow as the API response and the notification sending is async we have to wait for the notification even if we have a callback we have to wait for that callback to happen as it happens some time after the API response | 13:34 |
| gibi | jaypipes: I don't see now how we can avoid waiting, but we can avoid polling with a callback | 13:35 |
| gibi | jaypipes: is that OK for you? | 13:35 |
| jaypipes | gibi: so, I'm more thinking about just monkeypatching the notification sending to use a sync call instead of async, just for the purposes of testing. | 13:36 |
| jaypipes | just thinking out louds... | 13:36 |
| *** xyang1 has joined #openstack-nova | 13:36 | |
| gibi | jaypipes: but these are functional tests that calls the nova API. I don't know if there is a way to make the notification sending synced with the API response sending. | 13:37 |
| *** victorhe has quit IRC | 13:37 | |
| *** lamt has joined #openstack-nova | 13:39 | |
| jaypipes | gibi: again, it was just a thought... :) perhaps we can look into it later. I do, however, like your idea of using a callback instead of a polling mechanism. | 13:41 |
| *** baoli has joined #openstack-nova | 13:43 | |
| *** baoli has quit IRC | 13:43 | |
| *** ducttape_ has joined #openstack-nova | 13:43 | |
| *** baoli has joined #openstack-nova | 13:43 | |
| *** jamesdenton has joined #openstack-nova | 13:44 | |
| gibi | jaypipes: I agree that it would be nice to remove the waiting I just don't see how can it be done. At least not yet. I made a note for myself to look into this when time allows | 13:46 |
| *** mdrabe has joined #openstack-nova | 13:46 | |
| *** prateek_ has quit IRC | 13:46 | |
| *** prateek has joined #openstack-nova | 13:47 | |
| jaypipes | gibi: cool with me :) | 13:47 |
| markus_z | jaypipes: If you find time, https://review.openstack.org/#/c/395067/5 would resolve a (s390x critical) issue. Thanks in advance! | 13:47 |
| gibi | jaypipes: for the polling vs callback idea, I will try to put up a patch and ping you | 13:48 |
| *** tlian has joined #openstack-nova | 13:49 | |
| jaypipes | what my days seem to feel like... https://twitter.com/jaypipes/status/801422826402148353 | 13:51 |
| jaypipes | markus_z: on it. | 13:51 |
| *** derekh is now known as derekh_afk | 13:51 | |
| jaypipes | gibi: sounds good, thanks! :) | 13:52 |
| markus_z | jaypipes: thanks a lot. Btw, I talked with the intel folks this morning about their CI, it should be fixed now AFAIK. | 13:52 |
| jaypipes | markus_z: yeah, honestly, it's the vmware, xen, and intel nfv CIs that have all been super-flaky recently. | 13:53 |
| markus_z | jaypipes: Out of curiosity, is there a pattern in the failures or is it case-by-case? | 13:54 |
| *** jamesdenton has quit IRC | 13:55 | |
| *** kevinz has quit IRC | 13:56 | |
| jaypipes | markus_z: no pattern. typically they just start failing everything, then something gets fixed on them, they work, then start failing everything again. | 13:57 |
| *** prateek_ has joined #openstack-nova | 13:57 | |
| jaypipes | markus_z: I don't know if it's running out of disk space, network connectiviity issues, or actual code problems. | 13:57 |
| jaypipes | but I do know it's super annoying :) | 13:57 |
| *** bkopilov has joined #openstack-nova | 13:59 | |
| *** prateek has quit IRC | 13:59 | |
| *** ayogi has quit IRC | 13:59 | |
| *** kevinz has joined #openstack-nova | 14:00 | |
| *** lpetrut has quit IRC | 14:01 | |
| *** lpetrut has joined #openstack-nova | 14:02 | |
| *** anush has joined #openstack-nova | 14:02 | |
| mdbooth | melwitt: I'm asking for a pass on https://review.openstack.org/#/c/337159/ because it's at the head of my queue, and the thing you want coverage on is deleted a couple of patches down the line anyway :) | 14:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: doc: Add guidline about notification payload https://review.openstack.org/308393 | 14:04 |
| jaypipes | markus_z: done. | 14:04 |
| mdbooth | melwitt: It's a no-op | 14:05 |
| mdbooth | It's just a complex no-op which doesn't immediately look like one. | 14:05 |
| openstackgerrit | jichenjc proposed openstack/nova: Add more log when delete orphan node https://review.openstack.org/401274 | 14:05 |
| mdbooth | The worst kind of no-op. | 14:05 |
| *** bvanhav_ has joined #openstack-nova | 14:05 | |
| *** bvanhav has quit IRC | 14:06 | |
| *** thorst has joined #openstack-nova | 14:08 | |
| *** ijw has joined #openstack-nova | 14:10 | |
| *** emil1 has joined #openstack-nova | 14:12 | |
| *** pcaruana has quit IRC | 14:14 | |
| *** ijw has quit IRC | 14:15 | |
| *** jamesdenton has joined #openstack-nova | 14:15 | |
| *** parora has joined #openstack-nova | 14:19 | |
| *** thorst has quit IRC | 14:19 | |
| *** annegentle has joined #openstack-nova | 14:19 | |
| *** code-R has joined #openstack-nova | 14:20 | |
| markus_z | jaypipes: yeah, I love the rechecks in the morning too :-/ Thanks for the review. | 14:21 |
| *** prateek has joined #openstack-nova | 14:21 | |
| *** prateek_ has quit IRC | 14:22 | |
| mdbooth | Smells like victory. | 14:22 |
| *** amoralej|lunch is now known as amoralej | 14:22 | |
| *** thorst has joined #openstack-nova | 14:22 | |
| *** mriedem has joined #openstack-nova | 14:23 | |
| *** parora has quit IRC | 14:24 | |
| *** prateek_ has joined #openstack-nova | 14:24 | |
| *** jamesdenton has quit IRC | 14:24 | |
| markus_z | alex_xu: Could you maybe have a look at https://review.openstack.org/#/c/395067/5 too? It resolves a (s390x critical) issue which is open since 2 weeks. | 14:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Use is_valid_cidr, is_valid_ipv6_cidr from oslo_utils https://review.openstack.org/320375 | 14:26 |
| *** victorhe has joined #openstack-nova | 14:26 | |
| *** victorhe has quit IRC | 14:26 | |
| *** psachin has quit IRC | 14:26 | |
| *** victorhe has joined #openstack-nova | 14:26 | |
| *** prateek has quit IRC | 14:27 | |
| *** Prophidys has quit IRC | 14:30 | |
| *** thorst has quit IRC | 14:31 | |
| *** tesseract has joined #openstack-nova | 14:32 | |
| *** tesseract is now known as Guest48653 | 14:33 | |
| *** nic has joined #openstack-nova | 14:33 | |
| *** ihrachys has joined #openstack-nova | 14:33 | |
| *** Prophidys has joined #openstack-nova | 14:34 | |
| jaypipes | mdbooth: speaking of victory... how are we with your patches? I've been trying to push through reviews on them. I know you're working with melwitt on that one but how are the remaining ones looking? | 14:38 |
| mdbooth | jaypipes: I'm just rebasing the biggest in the series, the StoragePolicy one | 14:38 |
| mdbooth | jaypipes: However, there are 10 in the queue before that one, so I'm not stressing. | 14:39 |
| *** Guest48653 has quit IRC | 14:39 | |
| mdbooth | I think you're all caught up, btw. | 14:39 |
| *** tesseract- has joined #openstack-nova | 14:39 | |
| mdbooth | Big thank you! | 14:39 |
| jaypipes | np. | 14:40 |
| *** ducttape_ has quit IRC | 14:41 | |
| *** ducttape_ has joined #openstack-nova | 14:42 | |
| *** victorhe has quit IRC | 14:42 | |
| mdbooth | I'm hoping melwitt will give me a pass on the one she -1d earlier so I don't have to rebase the series. She's such a nice person, you know. Also great at parties, I hear, and so fantastically smart. | 14:43 |
| *** victorhe has joined #openstack-nova | 14:43 | |
| *** _ducttape_ has joined #openstack-nova | 14:43 | |
| *** ducttape_ has quit IRC | 14:46 | |
| *** derekh_afk is now known as derekh | 14:47 | |
| *** tesseract- has quit IRC | 14:49 | |
| *** andymaier has joined #openstack-nova | 14:52 | |
| *** jaosorior has quit IRC | 14:53 | |
| jaypipes | bauzas: super simple review series for some additional func tests for placement API here: https://review.openstack.org/#/c/396363 | 14:54 |
| jaypipes | mdbooth: :) | 14:54 |
| *** jaosorior has joined #openstack-nova | 14:54 | |
| *** nherciu_ has quit IRC | 14:55 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Return build_requests instead of instances https://review.openstack.org/374267 | 14:56 |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix context on loaded BDMs and Instances from build_request https://review.openstack.org/401007 | 14:56 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make RPCFixture support multiple connections https://review.openstack.org/396417 | 14:56 |
| *** paw has quit IRC | 14:57 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add context param to confirm_migration virt call https://review.openstack.org/383699 | 15:02 |
| *** oanson has joined #openstack-nova | 15:02 | |
| *** claudiub|2 has joined #openstack-nova | 15:04 | |
| *** thorst has joined #openstack-nova | 15:05 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add more log when delete orphan node https://review.openstack.org/401274 | 15:05 |
| *** yufei has quit IRC | 15:06 | |
| *** kevinz has quit IRC | 15:06 | |
| *** claudiub has quit IRC | 15:07 | |
| *** mpaolino has joined #openstack-nova | 15:08 | |
| *** ihrachys has quit IRC | 15:08 | |
| mriedem | dansmith: in https://review.openstack.org/#/c/401007/1/nova/objects/build_request.py | 15:09 |
| *** josecastroleon has quit IRC | 15:09 | |
| EmilienM | mriedem: hey, I'm seeing vif creation error in Nova compute logs, when testing master in Puppet CI (I tested yesterday and it worked fine) http://logs.openstack.org/38/401038/3/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/9948dfc/logs/nova/nova-compute.txt.gz#_2016-11-23_14_04_46_408 | 15:09 |
| EmilienM | have you seen that recently? | 15:09 |
| *** jaosorior has quit IRC | 15:09 | |
| mriedem | the build_request.create() would kill the context in the build_request.block_device_mapping, yes, | 15:09 |
| EmilienM | it's consistent | 15:09 |
| *** jaosorior has joined #openstack-nova | 15:10 | |
| mriedem | dansmith: but with https://review.openstack.org/#/c/374267/22/nova/compute/api.py@980 we're overwriting that build_request.block_device_mapping anyway | 15:10 |
| mriedem | so it's not the one that's used in the create() call | 15:10 |
| *** mwhahaha has joined #openstack-nova | 15:10 | |
| mriedem | it's the one that came back from _bdm_validate_set_size_and_instance which still has the context | 15:10 |
| mdbooth | mriedem: Any chance of poking this over the line before the onset of turkey coma: https://review.openstack.org/#/c/383859/ | 15:10 |
| mdbooth | And the one before it | 15:10 |
| dansmith | mriedem: yeah, sorry I must have been looking at the older PS when I commented | 15:11 |
| dansmith | mriedem: it's still the right thing to do though, IMHO | 15:11 |
| sean-k-mooney | mriedem: hi just looking at https://review.openstack.org/#/c/334048/7/nova/virt/libvirt/vif.py you are right the check is needed. | 15:11 |
| *** josecastroleon has joined #openstack-nova | 15:11 | |
| mriedem | sean-k-mooney: great | 15:12 |
| mriedem | sean-k-mooney: other than that it looks good to me | 15:12 |
| *** ihrachys has joined #openstack-nova | 15:12 | |
| *** thorst has quit IRC | 15:12 | |
| sean-k-mooney | mriedem: i have two options to impement the check. 1 extend the function signeture to pass the host object to the set_config* function or 2 add the check in _get_config_os_vif | 15:12 |
| sean-k-mooney | mriedem: any preference | 15:12 |
| *** victorhe has quit IRC | 15:14 | |
| mriedem | sean-k-mooney: it's too specific to vhostuser to add it in _get_config_os_vif i think | 15:16 |
| *** nic has quit IRC | 15:16 | |
| *** nandal has quit IRC | 15:16 | |
| *** thorst has joined #openstack-nova | 15:16 | |
| mriedem | sean-k-mooney: both get_config_vhostuser and _set_config_VIFVHostUser call designer.set_vif_host_backend_vhostuser_config - couldn't we just move that into a common private method that also does the version check? | 15:17 |
| mriedem | and then call that from both of ^ ? | 15:17 |
| mriedem | mdbooth: omg | 15:18 |
| mriedem | that's not a small patch | 15:18 |
| mriedem | and i'm brushing up on my BFS and DFS skillz | 15:18 |
| *** sandanar_ has quit IRC | 15:19 | |
| sean-k-mooney | mriedem: the issue is we need the host object to call host.has_min_version(MIN_LIBVIRT_VHOSTUSER_MQ) and while _get_config_os_vif has it id does not pass it to the set_config* functions | 15:19 |
| *** zz_dimtruck is now known as dimtruck | 15:19 | |
| sean-k-mooney | if i had the host object i could pass it to the designer.set_vif_host_backend_vhostuser_config and move the check to there | 15:20 |
| *** rama_y has joined #openstack-nova | 15:20 | |
| *** yufei has joined #openstack-nova | 15:20 | |
| *** thorst has quit IRC | 15:20 | |
| mriedem | ugh this sucks, i mean you can pass the host to _set_config_VIFVHostUser but then you need to add a host=None kwarg to all of those _set_config_* os-vif methods | 15:21 |
| *** victorhe has joined #openstack-nova | 15:21 | |
| sean-k-mooney | mriedem: yep and ill have to likely rework some unit tests | 15:22 |
| mriedem | because they are called dynamically | 15:22 |
| *** victorhe_ has joined #openstack-nova | 15:22 | |
| mriedem | well i don't think we want a vhostuser specific check in _get_config_os_vif | 15:22 |
| *** diogogmt has quit IRC | 15:22 | |
| mriedem | we can agree on that probably | 15:22 |
| *** hongbin has joined #openstack-nova | 15:22 | |
| sean-k-mooney | _get_config_os_vif is the simple sollution but i agree its not really the correct one | 15:23 |
| sean-k-mooney | it will work but its a bit messy | 15:23 |
| sean-k-mooney | what i can do is put one commit below this one to add the host=none to all the functions and then pass it in the current review? or would you be ok with having htis in one commit | 15:24 |
| mriedem | i think the best thing is probably passing a host=None kwarg to the _set_config_* methods | 15:24 |
| mriedem | sean-k-mooney: i think one below this is ideal | 15:24 |
| mriedem | so it's not all cluttered up | 15:24 |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: remove deprecated exception option https://review.openstack.org/397823 | 15:25 |
| sean-k-mooney | mriedem: ok no worries ill make that change and reupload. thanks | 15:25 |
| mriedem | cool | 15:25 |
| *** unicell has quit IRC | 15:25 | |
| *** victorhe has quit IRC | 15:26 | |
| *** gszasz has quit IRC | 15:29 | |
| *** jamesdenton has joined #openstack-nova | 15:29 | |
| mriedem | dansmith: do you know if we do https://review.openstack.org/#/c/401007/2/nova/objects/build_request.py as a pattern in any other objects? | 15:30 |
| mriedem | i'm not really for or against it | 15:30 |
| dansmith | mriedem: meaning other places we should be doing this but aren't? | 15:30 |
| dansmith | I'm sure there are a few | 15:30 |
| *** gszasz has joined #openstack-nova | 15:30 | |
| mriedem | well, i guess just looking at objects.Instance._load_security_groups, | 15:31 |
| mriedem | that results in a SecurityGroupList with a context on it, | 15:31 |
| mriedem | from the same Instance._context | 15:31 |
| mriedem | that loaded it | 15:31 |
| mriedem | so in that respect i'm ok with this then | 15:31 |
| mriedem | +2 | 15:32 |
| *** unicell has joined #openstack-nova | 15:32 | |
| *** ekuris has quit IRC | 15:34 | |
| *** ihrachys has quit IRC | 15:35 | |
| *** kukacz has joined #openstack-nova | 15:35 | |
| *** victorhe_ has quit IRC | 15:35 | |
| *** nkrinner has quit IRC | 15:36 | |
| kukacz | hi, pls when nova API docs mention a feature is available from 2.10 - which release does it map to? | 15:36 |
| *** paw has joined #openstack-nova | 15:36 | |
| mriedem | kukacz: that's a microversion | 15:37 |
| mriedem | kukacz: see http://docs.openstack.org/developer/nova/api_microversion_history.html | 15:37 |
| *** anush has quit IRC | 15:37 | |
| mriedem | 2.10 was added in the liberty release | 15:37 |
| mriedem | client code should do version discovery to determine the min/max supported versions of the compute API though | 15:38 |
| kukacz | mriedem: great, thanks, now I know my Kilo does not qualify :-) | 15:38 |
| kukacz | do you have any trick how to quickly interact with the API endpoint to get something like max microversion supported? | 15:39 |
| mriedem | kukacz: http://developer.openstack.org/api-ref/compute/#api-versions | 15:39 |
| *** nkrinner has joined #openstack-nova | 15:39 | |
| mriedem | find the compute endpoint and do a GET request on the root | 15:39 |
| kukacz | I was trying, but seems I need to send a token along, correct? | 15:40 |
| mriedem | i think so | 15:40 |
| mriedem | the various openstack api endpoints are different in that respect, some require a token and some don't to list versions | 15:40 |
| *** josecastroleon has quit IRC | 15:41 | |
| kukacz | ok. would be nice to have this version check in the standard CLI tools | 15:41 |
| *** mlavalle has joined #openstack-nova | 15:42 | |
| kukacz | mriedem: thanks a lot! | 15:42 |
| *** josecastroleon has joined #openstack-nova | 15:42 | |
| *** yufei has quit IRC | 15:42 | |
| paul-carlton | mriedem, can you take a look at https://review.openstack.org/#/q/status:open+topic:bp/detach-boot-volume when you get time please | 15:44 |
| mriedem | kukacz: the version check is in nova cli | 15:45 |
| mriedem | by default nova cli will request against the latest supported microversion the server supports and that the client can understand | 15:45 |
| mriedem | dansmith: 2 nits in https://review.openstack.org/#/c/374267/ and then i'll +W | 15:45 |
| *** prateek_ has quit IRC | 15:46 | |
| *** sapcc-bot1 has joined #openstack-nova | 15:46 | |
| *** prateek_ has joined #openstack-nova | 15:46 | |
| kukacz | mriedem: you're right, I've totally missed that. there's even "nova version-list". great! | 15:46 |
| *** sapcc-bot has quit IRC | 15:46 | |
| *** nkrinner has quit IRC | 15:46 | |
| kukacz | when I've found that the server only supports v2.0 - it means it doesn't support microversions yet, right? | 15:47 |
| mriedem | well, it means the v2.1 API isn't enabled in api-paste.ini i think | 15:47 |
| mriedem | there would probably be 2 compute services in the service catalog, one for v2.0 and one for v2.1 | 15:48 |
| *** gszasz has quit IRC | 15:48 | |
| mriedem | v2.1 is the api that has microversion support | 15:48 |
| mriedem | v2.0 was deprecated in liberty | 15:48 |
| *** gszasz has joined #openstack-nova | 15:48 | |
| *** _ducttape_ has quit IRC | 15:48 | |
| mriedem | you'd need this https://github.com/openstack/nova/blob/kilo-eol/etc/nova/api-paste.ini#L75 | 15:49 |
| *** ducttape_ has joined #openstack-nova | 15:49 | |
| mriedem | and https://github.com/openstack/nova/blob/kilo-eol/etc/nova/api-paste.ini#L65 | 15:49 |
| kukacz | hmm, I got that enabled in my Kilo config | 15:51 |
| *** yufei has joined #openstack-nova | 15:52 | |
| kukacz | but nova version-list (and the curl responses) only return 2.0 | 15:52 |
| *** ducttape_ has quit IRC | 15:53 | |
| markus_z | gmann: Are you around? | 15:53 |
| *** andymaier has quit IRC | 15:56 | |
| paul-carlton | I-ve changed the version of an object and I get "Some notification objects have changed; please make sure the versions have been bumped, and then update their hashes here." in my test failure, can someone please remind me how to generate a new hash please? | 15:56 |
| lyarwood | paul-carlton: isn't it in the output? | 15:57 |
| paul-carlton | ok, duh! | 15:58 |
| paul-carlton | lyarwood, ta | 15:58 |
| lyarwood | np | 15:58 |
| *** nkrinner has joined #openstack-nova | 16:00 | |
| *** Sukhdev_ has joined #openstack-nova | 16:01 | |
| *** yufei has left #openstack-nova | 16:03 | |
| EmilienM | can someone help with a vif creation failure in nova master? http://logs.openstack.org/38/401038/3/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/9948dfc/logs/nova/nova-compute.txt.gz#_2016-11-23_14_04_46_408 | 16:04 |
| EmilienM | we think it's something new, as our CI test recent packages | 16:05 |
| openstackgerrit | int32bit proposed openstack/nova: Change image info to fit instance xml https://review.openstack.org/399860 | 16:05 |
| *** cdent has quit IRC | 16:05 | |
| EmilienM | it looks like neutron is sending the event a bit too late (3 seconds) http://logs.openstack.org/38/401038/3/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/9948dfc/logs/neutron/server.txt.gz#_2016-11-23_14_04_49_575 | 16:05 |
| *** mjura has quit IRC | 16:07 | |
| *** code-R has quit IRC | 16:07 | |
| mriedem | EmilienM: nova is timing out waiting for the vif plugged event from neutron http://logs.openstack.org/38/401038/3/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/9948dfc/logs/nova/nova-compute.txt.gz#_2016-11-23_14_04_45_972 | 16:07 |
| mriedem | if you look before that the vif is unplugged http://logs.openstack.org/38/401038/3/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/9948dfc/logs/nova/nova-compute.txt.gz#_2016-11-23_14_04_46_255 | 16:08 |
| mnaser | is there an effort right now surrounding the issues with deadlocks in nova with galera? | 16:08 |
| mriedem | probably b/c the instance is being deleted while it's still building | 16:08 |
| mnaser | having a single master only doing all ops with two others is becoming an annoying inefficency | 16:08 |
| *** armax has joined #openstack-nova | 16:09 | |
| *** mpaolino has quit IRC | 16:09 | |
| mriedem | EmilienM: nova starts waiting for the vif plugged event here http://logs.openstack.org/38/401038/3/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/9948dfc/logs/nova/nova-compute.txt.gz#_2016-11-23_13_59_44_515 | 16:10 |
| *** ducttape_ has joined #openstack-nova | 16:10 | |
| mriedem | 5 minutes before the timeout | 16:10 |
| *** cnf has left #openstack-nova | 16:10 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Return build_requests instead of instances https://review.openstack.org/374267 | 16:10 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make RPCFixture support multiple connections https://review.openstack.org/396417 | 16:10 |
| EmilienM | mriedem: so we need to increase timeout or? | 16:11 |
| mriedem | no | 16:11 |
| EmilienM | or find our why it takes longer to spawn a vm | 16:11 |
| mriedem | there is likely a failure in neutron | 16:11 |
| mriedem | neutron failed to sent the vif plugged event | 16:11 |
| mriedem | i saw a network changed event in between the start and timeout | 16:11 |
| EmilienM | I haven't seen anything in Nova since yesterday, I'll look Neutron | 16:11 |
| *** whenry has quit IRC | 16:11 | |
| *** _ducttape_ has joined #openstack-nova | 16:12 | |
| mriedem | i think cd553fe0-442c-4e8a-bb51-9924e5088b17 is the port | 16:12 |
| *** gouthamr has quit IRC | 16:12 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: fix admin context scenarios for cinderclient https://review.openstack.org/384799 | 16:13 |
| *** gouthamr has joined #openstack-nova | 16:13 | |
| mriedem | EmilienM: looks like the port is 'up' at this point http://logs.openstack.org/38/401038/3/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/9948dfc/logs/neutron/server.txt.gz#_2016-11-23_13_59_46_939 | 16:15 |
| *** ducttape_ has quit IRC | 16:15 | |
| *** josecastroleon has quit IRC | 16:15 | |
| *** whenry has joined #openstack-nova | 16:15 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova-specs: Add ScaleIO ephemeral storage backend https://review.openstack.org/304715 | 16:15 |
| *** prashkre has joined #openstack-nova | 16:16 | |
| *** _ducttape_ has quit IRC | 16:16 | |
| *** sridharg has quit IRC | 16:16 | |
| emil1 | i can create two instances .. and then "message": "No valid host was found. There are not enough hosts available." :-( | 16:17 |
| emil1 | What can i do? | 16:17 |
| *** josecastroleon has joined #openstack-nova | 16:17 | |
| *** gjayavelu has joined #openstack-nova | 16:17 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Integrate OSProfiler and Nova https://review.openstack.org/254703 | 16:18 |
| mriedem | emil1: this isn't a support channel. try asking that question in the general #openstack channel or mailing list, or on ask.openstack.org | 16:19 |
| *** Apoorva has joined #openstack-nova | 16:19 | |
| mriedem | mnaser: i don't think anyone has reported issues | 16:20 |
| *** siva_krish has joined #openstack-nova | 16:20 | |
| mriedem | mnaser: or if they have, i'm not sure how, i.e. bugs, mailing list, other? | 16:20 |
| *** TravT has joined #openstack-nova | 16:20 | |
| mnaser | mriedem because I remember it's been suggested for a while to not hit multiple masters with galera | 16:20 |
| mriedem | suggested where? | 16:21 |
| mnaser | let me find | 16:21 |
| EmilienM | mriedem: thx for help, I'll keep investigating and probably file a bug. I'm looking in logstash if some projects also hit this | 16:21 |
| mriedem | EmilienM: looking at neutron git log from yesterday there were several db transactional changes | 16:22 |
| mriedem | something might have been busted there | 16:22 |
| *** mdnadeem has joined #openstack-nova | 16:22 | |
| *** siva_krish has quit IRC | 16:23 | |
| EmilienM | mriedem: I see the error quite a lot in logstash | 16:23 |
| *** zenoway has quit IRC | 16:23 | |
| EmilienM | mriedem: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%20%5C%22VirtualInterfaceCreateException%3A%20Virtual%20Interface%20creation%20failed%5C%22 | 16:24 |
| EmilienM | and it started yesterday | 16:24 |
| *** zenoway has joined #openstack-nova | 16:24 | |
| mnaser | mriedem: i think it has to do with this - https://blueprints.launchpad.net/nova/+spec/lock-free-quota-management | 16:24 |
| mnaser | related bp https://specs.openstack.org/openstack/nova-specs/specs/kilo/approved/lock-free-quota-management.html | 16:24 |
| *** giri has joined #openstack-nova | 16:24 | |
| *** Sukhdev_ has quit IRC | 16:24 | |
| *** jgriffith_away is now known as jgriffith | 16:25 | |
| *** unicell has quit IRC | 16:25 | |
| mriedem | EmilienM: yeah, all in non-voting jobs | 16:25 |
| mriedem | EmilienM: i'm using this | 16:25 |
| mriedem | message:"Timeout waiting for vif plugging callback for instance" AND tags:"screen-n-cpu.txt" | 16:25 |
| EmilienM | ah thx | 16:26 |
| mriedem | i don't know what's different about those jobs | 16:26 |
| *** gjayavelu has quit IRC | 16:26 | |
| mriedem | unless they are just slower, and the changes in neutron made things slow to the point that we hit timeouts now | 16:26 |
| EmilienM | mriedem: we hit that in puppet CI | 16:26 |
| mriedem | mnaser: you might want to talk to jaypipes about https://blueprints.launchpad.net/nova/+spec/lock-free-quota-management then | 16:27 |
| *** andymaier has joined #openstack-nova | 16:27 | |
| mnaser | alright, i'll try to do a bit more research because this stuff is from 2015 as well | 16:27 |
| mriedem | this was the spec https://review.openstack.org/#/c/135296 | 16:27 |
| mriedem | also note that we're looking at redoing quotas | 16:27 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/cells-count-resources-to-check-quota-in-api.html | 16:28 |
| mnaser | i'll have a read | 16:28 |
| mnaser | thanks mriedem | 16:28 |
| *** zenoway has quit IRC | 16:28 | |
| mriedem | np | 16:28 |
| *** bvanhav_ has quit IRC | 16:31 | |
| *** TravT has quit IRC | 16:32 | |
| *** daemontool has joined #openstack-nova | 16:32 | |
| *** Apoorva has quit IRC | 16:32 | |
| *** ftersin has quit IRC | 16:32 | |
| *** lbeliveau has quit IRC | 16:36 | |
| *** weshay is now known as weshay_lunch | 16:37 | |
| *** emil1 has left #openstack-nova | 16:38 | |
| *** thorst has joined #openstack-nova | 16:38 | |
| *** liuyulong has quit IRC | 16:38 | |
| openstackgerrit | Julian proposed openstack/nova: Transform instance.snapshot notifications https://review.openstack.org/330128 | 16:38 |
| *** thorst has quit IRC | 16:39 | |
| mriedem | kashyap: ever seen this? http://logs.openstack.org/81/326681/14/check/gate-tempest-dsvm-neutron-full-ssh/9779454/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-11-23_13_18_44_576 | 16:40 |
| *** rama_y has quit IRC | 16:40 | |
| mriedem | looks like we don't have libguestfs setup properly in our xenial nodes | 16:41 |
| *** annegentle has quit IRC | 16:41 | |
| *** yamahata has joined #openstack-nova | 16:42 | |
| *** mdnadeem has quit IRC | 16:42 | |
| mriedem | or it's just a packaging bug | 16:44 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.rebuild notification https://review.openstack.org/384922 | 16:45 |
| *** nic has joined #openstack-nova | 16:46 | |
| *** markus_z has quit IRC | 16:46 | |
| *** bvanhav_ has joined #openstack-nova | 16:49 | |
| *** rmart04 has quit IRC | 16:50 | |
| *** josecastroleon has quit IRC | 16:51 | |
| *** mtanino has joined #openstack-nova | 16:52 | |
| *** siva_krish has joined #openstack-nova | 16:53 | |
| *** josecastroleon has joined #openstack-nova | 16:53 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: remove deprecated cert_topic option https://review.openstack.org/397351 | 16:53 |
| *** ayogi has joined #openstack-nova | 16:54 | |
| *** andymaier has quit IRC | 16:56 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: move console opt to console group https://review.openstack.org/389647 | 16:57 |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: move console_host to console group https://review.openstack.org/394934 | 16:57 |
| *** Guest83036 has joined #openstack-nova | 16:57 | |
| *** sudipto_ has joined #openstack-nova | 16:57 | |
| kashyap | mriedem: Look. (/me just stepped out from home to a coffee shop) | 16:57 |
| kashyap | Err: s/Look/Looking/ | 16:57 |
| kashyap | Ah, it's missing the 'supermin' package, strange | 16:58 |
| mriedem | kashyap: we're in -dev, looks like a packaging issue | 16:58 |
| kashyap | I'm there, too | 16:58 |
| kashyap | mriedem: Yeah, indeed, one a first look, sounds like a packaging issue | 16:59 |
| *** dtp has joined #openstack-nova | 17:03 | |
| *** artom has quit IRC | 17:03 | |
| *** artom_ has joined #openstack-nova | 17:03 | |
| dtp | is there a cells v2 meeting today? | 17:04 |
| kashyap | dtp: I _think_ it's cancelled | 17:04 |
| dtp | ok, thanks! | 17:05 |
| *** sahid has quit IRC | 17:05 | |
| kashyap | dtp: It is, dansmith sent an email to the upstream list with subject: "[nova] 23-Nov Cells meeting cancelled" | 17:05 |
| dtp | i don't know why i'm having such hard time getting on that email distro | 17:05 |
| dtp | i'll go submit another request... | 17:05 |
| dansmith | dtp: we had one last week and you didn't show up :) | 17:05 |
| dtp | kaaaaaahn :0 | 17:06 |
| *** thorst has joined #openstack-nova | 17:06 | |
| *** thorst_ has joined #openstack-nova | 17:07 | |
| *** beagles is now known as beagles-brb | 17:08 | |
| *** lpetrut has quit IRC | 17:10 | |
| *** thorst has quit IRC | 17:11 | |
| *** thorst_ has quit IRC | 17:11 | |
| *** annegentle has joined #openstack-nova | 17:12 | |
| dtp | is this the list that notice was sent to - http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack | 17:14 |
| *** jaosorior has quit IRC | 17:15 | |
| mriedem | mdbooth: questions in https://review.openstack.org/#/c/393869/ | 17:15 |
| mriedem | dtp: no | 17:15 |
| mriedem | http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev | 17:15 |
| mriedem | subject is tagged with [nova] | 17:16 |
| mdbooth | mriedem: Looking, thanks | 17:16 |
| *** yamahata has quit IRC | 17:17 | |
| *** tjones has joined #openstack-nova | 17:19 | |
| dtp | thanks mriedem | 17:20 |
| jaypipes | mriedem, dansmith, bauzas: your reviews on https://review.openstack.org/#/q/topic:bp/inventory-allocations-ocata would be most appreciated. | 17:21 |
| *** rama_y has joined #openstack-nova | 17:21 | |
| *** josecastroleon has quit IRC | 17:23 | |
| *** josecastroleon has joined #openstack-nova | 17:24 | |
| *** msno has quit IRC | 17:25 | |
| *** Apoorva has joined #openstack-nova | 17:25 | |
| *** jamesden_ has joined #openstack-nova | 17:27 | |
| *** jamesde__ has joined #openstack-nova | 17:28 | |
| *** hogepodge has joined #openstack-nova | 17:28 | |
| mriedem | i reckon i need to review https://review.openstack.org/#/c/386710/ first | 17:29 |
| *** diogogmt has joined #openstack-nova | 17:29 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds Hyper-V OVS ViF driver https://review.openstack.org/140045 | 17:30 |
| *** matrohon has quit IRC | 17:30 | |
| *** sambetts is now known as sambetts|afk | 17:30 | |
| *** jamesdenton has quit IRC | 17:30 | |
| *** jamesden_ has quit IRC | 17:31 | |
| *** tbachman has quit IRC | 17:31 | |
| *** lucasagomes is now known as lucas-hungry | 17:33 | |
| *** lucas-hungry is now known as lucas-afk | 17:33 | |
| *** hogepodge has quit IRC | 17:33 | |
| *** nkrinner is now known as nkrinner_afk | 17:37 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds Hyper-V OVS ViF driver https://review.openstack.org/140045 | 17:37 |
| mriedem | jaypipes: feel free to taunt me when the vikings blow the game in the last 30 seconds tomorrow | 17:39 |
| jaypipes | mriedem: :) | 17:40 |
| jaypipes | mriedem: a review on the nested-resource-providers spec, of course, would be most appreciated. | 17:41 |
| jaypipes | mriedem: thought he inventory-allocations-ocata patch series doesn't have to do with nested providers | 17:41 |
| *** klkumar has quit IRC | 17:42 | |
| *** weshay_lunch is now known as weshay | 17:42 | |
| *** claudiub|2 has quit IRC | 17:42 | |
| *** mkoderer has quit IRC | 17:43 | |
| *** hogepodge has joined #openstack-nova | 17:43 | |
| mriedem | oh i see i guess that's just leftover from ocata | 17:44 |
| *** slaweq_ has quit IRC | 17:44 | |
| *** victorhe has joined #openstack-nova | 17:45 | |
| *** slaweq_ has joined #openstack-nova | 17:46 | |
| *** daemontool has quit IRC | 17:46 | |
| *** matrohon has joined #openstack-nova | 17:46 | |
| *** ayogi has quit IRC | 17:46 | |
| *** gszasz has quit IRC | 17:47 | |
| *** giri has quit IRC | 17:50 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova: guestfs: Don't report exception if there's read access to kernel https://review.openstack.org/401366 | 17:51 |
| *** derekh has quit IRC | 17:52 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Call host connection callbacks asynchronously https://review.openstack.org/393869 | 17:54 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859 | 17:54 |
| *** jamesde__ has quit IRC | 17:54 | |
| *** gjayavelu has joined #openstack-nova | 17:54 | |
| *** josecastroleon has quit IRC | 17:54 | |
| mdbooth | mriedem: Updated, at the cost of my +2 from danpb :( | 17:54 |
| *** josecastroleon has joined #openstack-nova | 17:55 | |
| *** dave-mccowan has joined #openstack-nova | 17:56 | |
| openstackgerrit | Julian proposed openstack/nova: Transform instance.snapshot notifications https://review.openstack.org/330128 | 17:57 |
| *** dave-mccowan has quit IRC | 17:57 | |
| *** gjayavelu has quit IRC | 17:58 | |
| *** tbachman has joined #openstack-nova | 17:58 | |
| mriedem | mdbooth: ok if i'm good when tests are done i'll just fast +W | 17:59 |
| mriedem | since the changes were minor | 18:00 |
| *** ralonsoh has quit IRC | 18:00 | |
| mdbooth | mriedem: That would be awesome, thanks. Be good to get that off my todo list :) | 18:00 |
| *** tbachman_ has joined #openstack-nova | 18:01 | |
| *** beagles-brb is now known as beagles | 18:02 | |
| *** amoralej is now known as amoralej|off | 18:02 | |
| *** tbachman has quit IRC | 18:03 | |
| *** tbachman_ is now known as tbachman | 18:03 | |
| *** mdrabe has quit IRC | 18:06 | |
| openstackgerrit | Julian proposed openstack/nova: Transform instance.snapshot notifications https://review.openstack.org/330128 | 18:07 |
| *** bvanhav__ has joined #openstack-nova | 18:08 | |
| *** bvanhav__ is now known as bvanhav | 18:08 | |
| *** siva_krish has quit IRC | 18:09 | |
| *** bvanhav_ has quit IRC | 18:09 | |
| *** parora has joined #openstack-nova | 18:13 | |
| *** slaweq_ has quit IRC | 18:13 | |
| *** prateek_ has quit IRC | 18:16 | |
| openstackgerrit | int32bit proposed openstack/nova: Fix wait for detach code to handle 'the not found' from libvirt https://review.openstack.org/401375 | 18:16 |
| *** sreeram_v has joined #openstack-nova | 18:20 | |
| *** lpetrut has joined #openstack-nova | 18:20 | |
| *** unicell has joined #openstack-nova | 18:24 | |
| *** prateek_ has joined #openstack-nova | 18:24 | |
| *** sreeram_v has quit IRC | 18:24 | |
| *** josecastroleon has quit IRC | 18:25 | |
| *** unicell has quit IRC | 18:25 | |
| *** josecastroleon has joined #openstack-nova | 18:26 | |
| *** unicell has joined #openstack-nova | 18:27 | |
| *** parora has quit IRC | 18:27 | |
| *** annegentle has quit IRC | 18:31 | |
| *** lpetrut has quit IRC | 18:32 | |
| *** ihrachys has joined #openstack-nova | 18:35 | |
| *** markvoelker has quit IRC | 18:35 | |
| *** siva_krish has joined #openstack-nova | 18:37 | |
| *** Guest83036 has quit IRC | 18:38 | |
| *** sudipto_ has quit IRC | 18:38 | |
| *** prateek_ has quit IRC | 18:38 | |
| *** lpetrut has joined #openstack-nova | 18:41 | |
| *** jgriffith is now known as jgriffith_away | 18:42 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: move console opt to console group https://review.openstack.org/389647 | 18:44 |
| *** cnf has joined #openstack-nova | 18:49 | |
| *** dave-mccowan has joined #openstack-nova | 18:49 | |
| *** victorhe has quit IRC | 18:53 | |
| *** yamahata has joined #openstack-nova | 18:55 | |
| *** victorhe has joined #openstack-nova | 18:55 | |
| *** josecastroleon has quit IRC | 18:56 | |
| *** josecastroleon has joined #openstack-nova | 18:57 | |
| *** tbachman has quit IRC | 18:57 | |
| *** dave-mcc_ has joined #openstack-nova | 18:59 | |
| *** dave-mccowan has quit IRC | 19:02 | |
| *** prashkre has quit IRC | 19:07 | |
| *** nherciu has joined #openstack-nova | 19:12 | |
| *** moshele has joined #openstack-nova | 19:12 | |
| openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 19:12 |
| *** jgriffith_away is now known as jgriffith | 19:14 | |
| *** tbachman has joined #openstack-nova | 19:16 | |
| *** beagles is now known as beagles_brb | 19:21 | |
| *** fragatina has quit IRC | 19:22 | |
| *** jamesdenton has joined #openstack-nova | 19:25 | |
| openstackgerrit | Julian proposed openstack/nova: Transform instance.snapshot notifications https://review.openstack.org/330128 | 19:27 |
| *** catintheroof has joined #openstack-nova | 19:27 | |
| *** josecastroleon has quit IRC | 19:28 | |
| *** paul-carlton has quit IRC | 19:28 | |
| *** josecastroleon has joined #openstack-nova | 19:30 | |
| *** jamesdenton has quit IRC | 19:31 | |
| *** catintheroof has quit IRC | 19:34 | |
| *** victorhe has quit IRC | 19:39 | |
| *** victorhe_ has joined #openstack-nova | 19:39 | |
| *** victorhe has joined #openstack-nova | 19:40 | |
| *** victorhe_ has quit IRC | 19:43 | |
| *** jamesdenton has joined #openstack-nova | 19:46 | |
| *** dave-mcc_ has quit IRC | 19:49 | |
| *** betherly_ has joined #openstack-nova | 19:50 | |
| *** lbeliveau has joined #openstack-nova | 19:51 | |
| *** beagles_brb is now known as beagles | 19:55 | |
| *** fragatina has joined #openstack-nova | 19:56 | |
| *** paw has quit IRC | 19:56 | |
| *** tbachman has quit IRC | 20:00 | |
| *** josecastroleon has quit IRC | 20:02 | |
| *** josecastroleon has joined #openstack-nova | 20:03 | |
| *** ihrachys has quit IRC | 20:07 | |
| *** bvanhav has quit IRC | 20:07 | |
| *** jamesdenton has quit IRC | 20:09 | |
| *** oanson has quit IRC | 20:09 | |
| *** ijw has joined #openstack-nova | 20:11 | |
| *** jamesdenton has joined #openstack-nova | 20:13 | |
| *** ihrachys has joined #openstack-nova | 20:13 | |
| *** ihrachys has quit IRC | 20:14 | |
| *** ihrachys has joined #openstack-nova | 20:14 | |
| *** mvk has quit IRC | 20:15 | |
| melwitt | my intel-nfv-ci recheck from yesterday seems to have done nothing | 20:24 |
| mriedem | jaypipes: reviewed https://review.openstack.org/#/c/386710/ - there is a part in there about the whiteboarding session at the summit which might be worth capturing in there | 20:26 |
| mriedem | melwitt: the intel jobs have been wonky | 20:26 |
| mriedem | melwitt: wznoinsk is the person to ping | 20:26 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: PCI: Race condition between audit and migrate/resize revert https://review.openstack.org/400995 | 20:27 |
| *** slaweq_ has joined #openstack-nova | 20:27 | |
| melwitt | mriedem: thanks | 20:27 |
| *** fragatina has quit IRC | 20:30 | |
| *** josecastroleon has quit IRC | 20:33 | |
| *** nicolasbock has quit IRC | 20:33 | |
| *** josecastroleon has joined #openstack-nova | 20:34 | |
| *** Sukhdev has joined #openstack-nova | 20:36 | |
| *** cfriesen has joined #openstack-nova | 20:37 | |
| *** swamireddy has quit IRC | 20:37 | |
| *** betherly_ is now known as betherly | 20:41 | |
| *** cfriesen has quit IRC | 20:42 | |
| *** anush has joined #openstack-nova | 20:42 | |
| *** armax has quit IRC | 20:43 | |
| *** armax has joined #openstack-nova | 20:45 | |
| *** armax has quit IRC | 20:45 | |
| *** kaisers__ has quit IRC | 20:48 | |
| *** tbachman has joined #openstack-nova | 20:49 | |
| *** gouthamr has quit IRC | 20:50 | |
| *** baoli has quit IRC | 20:51 | |
| *** baoli has joined #openstack-nova | 20:51 | |
| *** flaper87 has quit IRC | 20:52 | |
| *** mvk has joined #openstack-nova | 20:53 | |
| *** moshele has quit IRC | 20:54 | |
| *** baoli has quit IRC | 20:56 | |
| *** cfriesen has joined #openstack-nova | 20:57 | |
| *** fragatina has joined #openstack-nova | 21:01 | |
| *** Guest2 has joined #openstack-nova | 21:02 | |
| *** Guest2 has quit IRC | 21:02 | |
| mriedem | mdbooth: if you're still around i'm going through your nfs mount patch | 21:03 |
| mriedem | found some issues i think | 21:03 |
| *** josecastroleon has quit IRC | 21:04 | |
| *** anush has quit IRC | 21:05 | |
| *** josecastroleon has joined #openstack-nova | 21:05 | |
| *** Guest2 has joined #openstack-nova | 21:06 | |
| *** ihrachys has quit IRC | 21:06 | |
| *** catintheroof has joined #openstack-nova | 21:08 | |
| -openstackstatus- NOTICE: Due to a problem with the cinder volume backing the log server, jobs are failing with POST_FAILURE. Please avoid issuing 'recheck' commands until the issue is resolved. | 21:08 | |
| *** ChanServ changes topic to "Due to a problem with the cinder volume backing the log server, jobs are failing with POST_FAILURE. Please avoid issuing 'recheck' commands until the issue is resolved." | 21:08 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: PCI: Release PCI devices on drop_move_claim() https://review.openstack.org/370374 | 21:08 |
| mriedem | mdbooth: comments posted https://review.openstack.org/#/c/383859/ | 21:10 |
| *** anush has joined #openstack-nova | 21:11 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add zero down time upgrade steps https://review.openstack.org/393931 | 21:11 |
| *** unicell has quit IRC | 21:14 | |
| *** Apoorva has quit IRC | 21:16 | |
| *** kaisers_ has joined #openstack-nova | 21:18 | |
| *** jamesdenton has quit IRC | 21:18 | |
| *** unicell1 has joined #openstack-nova | 21:21 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix race in test_volume_swap_server_with_error https://review.openstack.org/394585 | 21:21 |
| *** jamielennox|away is now known as jamielennox | 21:22 | |
| *** tbachman_ has joined #openstack-nova | 21:23 | |
| *** tbachman has quit IRC | 21:24 | |
| *** tbachman_ is now known as tbachman | 21:24 | |
| *** armax has joined #openstack-nova | 21:27 | |
| *** victorhe has quit IRC | 21:29 | |
| mriedem | artom_: looks like some easy comments to address in here https://review.openstack.org/#/c/305120/ | 21:29 |
| mriedem | artom_: had a +2 already, maybe if you make jordanP happy we'll get that elusive +W | 21:30 |
| mriedem | i can address those comments if yo'ure busy | 21:30 |
| *** dtp has quit IRC | 21:32 | |
| *** ijw has quit IRC | 21:34 | |
| *** josecastroleon has quit IRC | 21:35 | |
| *** Jeffrey4l has quit IRC | 21:35 | |
| *** Jeffrey4l has joined #openstack-nova | 21:35 | |
| *** ijw has joined #openstack-nova | 21:37 | |
| *** catintheroof has quit IRC | 21:40 | |
| *** SayyedMohammad has quit IRC | 21:41 | |
| *** SayyedMohammad has joined #openstack-nova | 21:41 | |
| *** victorhe has joined #openstack-nova | 21:43 | |
| *** pradk has quit IRC | 21:43 | |
| *** lpetrut has quit IRC | 21:45 | |
| mriedem | artom_: updated it myself | 21:46 |
| *** thorst_ has joined #openstack-nova | 21:47 | |
| *** kwathore has quit IRC | 21:47 | |
| *** kwathore has joined #openstack-nova | 21:47 | |
| *** msayyed has joined #openstack-nova | 21:48 | |
| *** SayyedMohammad has quit IRC | 21:51 | |
| *** gjayavelu has joined #openstack-nova | 21:54 | |
| mriedem | dansmith: want to bump this in? https://review.openstack.org/#/c/389246/ | 21:57 |
| mriedem | it's for stable | 21:57 |
| *** anush has quit IRC | 21:59 | |
| dansmith | mriedem: I *want* that to be my final act before slipping into stretchy pants | 22:00 |
| mriedem | dansmith: i'm shocked, you even have pants on? | 22:00 |
| dansmith | you know I always have pants on | 22:00 |
| mriedem | never nude | 22:00 |
| dansmith | not while talking to you | 22:01 |
| *** tjones has left #openstack-nova | 22:02 | |
| *** Guest2 has quit IRC | 22:03 | |
| artom_ | mriedem, thanks | 22:03 |
| artom_ | I think JordanP wanted it quicker - which oomichi mentioned as well, specifically by splitting it into volume tests and NIC tests | 22:04 |
| artom_ | That's why I hadn't addressed the jordan's comments - I wanted to split it, and kept putting it off | 22:05 |
| artom_ | Also, my WIP test for tagged attach depend on it, so it makes everything more annoying | 22:05 |
| mriedem | i guess i missed all of that, | 22:05 |
| mriedem | but, | 22:05 |
| mriedem | all of this infra is built into this single test, you'd have to duplicate a lot of garbage to split it | 22:05 |
| artom_ | Honestly though, if it merges with the changes you've made, good. | 22:06 |
| mriedem | s/duplicate/generalize/ | 22:06 |
| artom_ | Better to have something merged and not perfect, than perfect and not merged | 22:06 |
| mriedem | wow i didn't know genericize was a real word | 22:06 |
| mriedem | i thought it was like jazzercise | 22:06 |
| artom_ | mriedem, there's a way to do it more or less elegantly | 22:06 |
| *** cfriesen has quit IRC | 22:07 | |
| mriedem | artom_: sorry, i'm way past that now and looking at http://www.jazzercise.com/ | 22:07 |
| artom_ | I don't think it saves much time, but at least split up no one can argue to make it shorter, because it will literally be atomic | 22:07 |
| artom_ | OK, no literally | 22:07 |
| artom_ | mriedem, fair point. | 22:07 |
| artom_ | I raise you https://www.youtube.com/watch?v=TIfAkOBMf5A | 22:08 |
| mriedem | that reminds me, | 22:08 |
| mriedem | i had proposed to my wife that for next halloween i'm going to be richard simmons and she'll be liza minelli | 22:09 |
| mriedem | she wasn't really on board with that idea | 22:09 |
| * artom_ needs to pick up daughter from daycare. | 22:10 | |
| *** takashin has joined #openstack-nova | 22:16 | |
| mriedem | rpodolyaka: small issue in https://review.openstack.org/#/c/392185/ | 22:18 |
| *** slagle has quit IRC | 22:18 | |
| mriedem | if you want to clean that up quick i'll +2 | 22:18 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix a typo in nova/api/openstack/compute/volumes.py https://review.openstack.org/400533 | 22:18 |
| *** thorst_ has quit IRC | 22:20 | |
| *** sreeram_v has joined #openstack-nova | 22:21 | |
| *** sreeram_v has quit IRC | 22:26 | |
| *** matrohon has quit IRC | 22:33 | |
| *** nic has quit IRC | 22:38 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Remove sata bus for virtuozzo hypervisor https://review.openstack.org/282398 | 22:38 |
| *** nherciu has quit IRC | 22:40 | |
| *** beagles is now known as beagles_brb | 22:43 | |
| *** swamireddy has joined #openstack-nova | 22:45 | |
| *** catintheroof has joined #openstack-nova | 22:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Placement api: Add informative message to 404 response. https://review.openstack.org/392321 | 22:51 |
| *** catintheroof has quit IRC | 22:52 | |
| *** whenry has quit IRC | 22:52 | |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Ocata_Release_Schedule" | 22:53 | |
| -openstackstatus- NOTICE: The affected filesystems on the log server are repaired. Please leave 'recheck' comments on any changes which failed with POST_FAILURE. | 22:53 | |
| *** victorhe has quit IRC | 22:57 | |
| *** dimtruck is now known as zz_dimtruck | 22:59 | |
| *** xyang1 has quit IRC | 22:59 | |
| *** catintheroof has joined #openstack-nova | 23:00 | |
| mriedem | anthonyper: rfolco: can you comment on the libvirt-xen and ppc64 sections in here? https://review.openstack.org/#/c/397871/3/doc/source/support-matrix.ini | 23:01 |
| rfolco | mriedem, I did already previously. You mean confirm min versions ? | 23:03 |
| *** annegentle has joined #openstack-nova | 23:03 | |
| mriedem | rfolco: ah thanks i missed that | 23:03 |
| artom_ | roflco is the name of my comedy club | 23:11 |
| * mriedem cues rimshot | 23:11 | |
| *** mtanino has quit IRC | 23:12 | |
| *** beagles_brb is now known as beagles | 23:14 | |
| mriedem | alaski: so you don't like this, but you're +2 :) https://review.openstack.org/#/c/401007/ | 23:17 |
| *** sreeram_v has joined #openstack-nova | 23:19 | |
| *** anush has joined #openstack-nova | 23:20 | |
| *** catinthe_ has joined #openstack-nova | 23:23 | |
| *** sreeram_v has quit IRC | 23:24 | |
| *** catintheroof has quit IRC | 23:26 | |
| gmann | mriedem: oomichi alex_xu : +1 we should fix all status code together | 23:32 |
| oomichi | gmann: yeah, I don't prefer much much much microversions :) | 23:33 |
| oomichi | but maybe we will miss something I believe as human | 23:33 |
| gmann | oomichi: i remember we have a spec for pushing all api-wg guidlines like status code etc all together in single microversion | 23:34 |
| oomichi | like deprecated proxying APIs | 23:34 |
| oomichi | gmann: oh, I don't know that | 23:34 |
| mriedem | this? https://review.openstack.org/#/c/307675/ | 23:34 |
| gmann | oomichi: yea thats true, we are human :) | 23:34 |
| gmann | mriedem: humm, may be but let me search if anyother | 23:35 |
| gmann | i think thats what i saw. | 23:36 |
| gmann | so can we think about other more suitable guidelines like status code and other if any and push them all together in single version ? | 23:36 |
| gmann | or will it be huge one and mix changes and hard tyo adopt by user ? | 23:37 |
| gmann | tyo->to | 23:37 |
| *** anush has quit IRC | 23:37 | |
| mriedem | that would also be hard to review | 23:37 |
| mriedem | with that i need to get out of here before i get in trouble with the family | 23:40 |
| mriedem | see you all next week | 23:40 |
| gmann | yea | 23:41 |
| *** mriedem has quit IRC | 23:41 | |
| gmann | :), see you. | 23:41 |
| *** mlavalle has quit IRC | 23:42 | |
| *** lamt has quit IRC | 23:43 | |
| *** tlian2 has joined #openstack-nova | 23:46 | |
| *** tlian has quit IRC | 23:48 | |
| *** jamesdenton has joined #openstack-nova | 23:50 | |
| *** tlian2 has quit IRC | 23:51 | |
| *** jamesdenton has quit IRC | 23:52 | |
| *** catinthe_ has quit IRC | 23:55 | |
| *** tlian has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!