| *** jwcroppe has joined #openstack-nova | 00:02 | |
| *** jwcroppe has quit IRC | 00:02 | |
| *** jwcroppe has joined #openstack-nova | 00:02 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 00:05 |
|---|---|---|
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 00:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512 | 00:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43 - Enable cold migration with target host https://review.openstack.org/406707 | 00:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43: List/Show all server migration types https://review.openstack.org/430839 | 00:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 00:07 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 00:07 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608 | 00:08 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Specify keymap on server boot https://review.openstack.org/448830 | 00:08 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 00:08 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 00:09 |
| *** rfolco has joined #openstack-nova | 00:11 | |
| *** suresh12 has quit IRC | 00:12 | |
| *** baoli has quit IRC | 00:16 | |
| *** baoli has joined #openstack-nova | 00:17 | |
| *** suresh12 has joined #openstack-nova | 00:17 | |
| *** baoli has quit IRC | 00:21 | |
| *** suresh12 has quit IRC | 00:22 | |
| *** jamielennox is now known as jamielennox|away | 00:25 | |
| *** adrian_otto has quit IRC | 00:26 | |
| *** dikonoo has joined #openstack-nova | 00:26 | |
| *** dikonoor has joined #openstack-nova | 00:26 | |
| *** ducttape_ has quit IRC | 00:27 | |
| *** jamielennox|away is now known as jamielennox | 00:31 | |
| *** zhurong has joined #openstack-nova | 00:32 | |
| *** tuanluong has joined #openstack-nova | 00:35 | |
| *** thorst has joined #openstack-nova | 00:38 | |
| *** thorst has quit IRC | 00:39 | |
| *** vladikr has quit IRC | 00:41 | |
| *** tovin07_ has joined #openstack-nova | 00:44 | |
| *** abalutoiu has quit IRC | 00:46 | |
| *** tbachman_ has joined #openstack-nova | 00:46 | |
| *** gongysh has joined #openstack-nova | 00:47 | |
| *** raunak has quit IRC | 00:48 | |
| *** zz_dimtruck is now known as dimtruck | 00:48 | |
| *** tbachman has quit IRC | 00:48 | |
| *** tbachman_ is now known as tbachman | 00:48 | |
| *** dharinic has joined #openstack-nova | 00:51 | |
| *** Shunli has joined #openstack-nova | 00:52 | |
| *** armax has quit IRC | 00:54 | |
| *** Dave has quit IRC | 00:54 | |
| *** yingjun has joined #openstack-nova | 00:55 | |
| *** zsli_ has joined #openstack-nova | 00:58 | |
| *** zsli_ has quit IRC | 01:01 | |
| *** Shunli has quit IRC | 01:01 | |
| *** vladikr has joined #openstack-nova | 01:01 | |
| *** Shunli has joined #openstack-nova | 01:01 | |
| *** suresh12 has joined #openstack-nova | 01:02 | |
| *** Dave has joined #openstack-nova | 01:02 | |
| *** ljjjustin has joined #openstack-nova | 01:06 | |
| *** suresh12 has quit IRC | 01:06 | |
| *** ljjjusti_ has joined #openstack-nova | 01:07 | |
| *** phuongnh has joined #openstack-nova | 01:07 | |
| *** thorst has joined #openstack-nova | 01:10 | |
| *** ducttape_ has joined #openstack-nova | 01:10 | |
| *** ljjjustin has quit IRC | 01:10 | |
| *** kevinz has joined #openstack-nova | 01:12 | |
| *** gongysh has quit IRC | 01:16 | |
| *** liusheng has joined #openstack-nova | 01:21 | |
| *** ijw has joined #openstack-nova | 01:23 | |
| *** browne has quit IRC | 01:27 | |
| *** thorst has quit IRC | 01:28 | |
| *** dikonoo has quit IRC | 01:29 | |
| *** dikonoor has quit IRC | 01:29 | |
| *** seungjin has quit IRC | 01:33 | |
| *** thorst has joined #openstack-nova | 01:37 | |
| *** READ10 has quit IRC | 01:38 | |
| *** thorst has quit IRC | 01:38 | |
| *** liangy has joined #openstack-nova | 01:44 | |
| *** lucasxu has joined #openstack-nova | 01:45 | |
| *** brault|away has joined #openstack-nova | 01:48 | |
| *** raunak has joined #openstack-nova | 01:49 | |
| *** Apoorva has quit IRC | 01:50 | |
| *** brault|away has quit IRC | 01:53 | |
| *** oanson has quit IRC | 01:55 | |
| *** oanson has joined #openstack-nova | 01:56 | |
| *** kenperkins has quit IRC | 01:56 | |
| *** dixiaoli has joined #openstack-nova | 01:58 | |
| *** rfolco has quit IRC | 02:00 | |
| *** ducttape_ has quit IRC | 02:04 | |
| *** yamahata has quit IRC | 02:08 | |
| *** ijw has quit IRC | 02:08 | |
| *** lucasxu has quit IRC | 02:10 | |
| *** lucasxu has joined #openstack-nova | 02:11 | |
| *** xinliang has quit IRC | 02:17 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Rename the model object ResourceProviderTraits to ResourceProviderTrait https://review.openstack.org/451635 | 02:19 |
| openstackgerrit | Alex Xu proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625 | 02:19 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201 | 02:19 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | 02:19 |
| *** lucasxu has quit IRC | 02:19 | |
| *** hongbin has joined #openstack-nova | 02:20 | |
| *** smcginnis is now known as smcginnis_vacati | 02:27 | |
| *** liangy has quit IRC | 02:30 | |
| *** xinliang has joined #openstack-nova | 02:30 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the stevedore extension point for server create https://review.openstack.org/451195 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the json-schema extension point of server create https://review.openstack.org/451194 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the usage of extension black/white list opts in BDM tests https://review.openstack.org/450830 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the usage of extension black/white list opts in multi-create https://review.openstack.org/450831 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the usage of extension black/white list opt in scheduler hints https://review.openstack.org/450835 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the usage of extension black/white list opt in user data https://review.openstack.org/450834 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the usage of extension black/white list opt in AZ https://review.openstack.org/450833 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the usage of extension black/white list opt in config drive https://review.openstack.org/450832 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the extension check for os-networks in servers API https://review.openstack.org/450837 | 02:34 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove config opts for extension black/white list https://review.openstack.org/450836 | 02:34 |
| *** rfolco has joined #openstack-nova | 02:36 | |
| *** rfolco has quit IRC | 02:37 | |
| *** thorst has joined #openstack-nova | 02:39 | |
| *** kevinz has quit IRC | 02:40 | |
| *** kevinz has joined #openstack-nova | 02:40 | |
| *** rfolco has joined #openstack-nova | 02:45 | |
| *** rfolco has quit IRC | 02:45 | |
| gmann | alex_xu: johnthetubaguy oomichi can you check this (many API change patches keep appending the field in POST server response) -https://review.openstack.org/#/c/451673/ | 02:46 |
| *** tommylikehu has joined #openstack-nova | 02:51 | |
| *** ducttape_ has joined #openstack-nova | 02:53 | |
| *** thorst has quit IRC | 02:56 | |
| *** thorst has joined #openstack-nova | 02:57 | |
| *** hamzy has joined #openstack-nova | 02:58 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Rework descriptions in os-hypervisors https://review.openstack.org/452030 | 03:00 |
| *** thorst has quit IRC | 03:01 | |
| *** namnh has joined #openstack-nova | 03:01 | |
| *** dimtruck is now known as zz_dimtruck | 03:03 | |
| *** baoli has joined #openstack-nova | 03:07 | |
| namnh | dansmith, Hi Mr.Smith, are you free? could I confirm with you a point | 03:11 |
| *** baoli has quit IRC | 03:15 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add description to policies in admin_password.py https://review.openstack.org/451271 | 03:17 |
| *** ducttape_ has quit IRC | 03:20 | |
| *** amotoki has quit IRC | 03:22 | |
| openstackgerrit | Danfly proposed openstack/nova master: Completely remove mox from unit/network/test_linux_net.py https://review.openstack.org/445504 | 03:25 |
| *** hongbin has quit IRC | 03:31 | |
| *** ducttape_ has joined #openstack-nova | 03:31 | |
| *** Shunli has quit IRC | 03:32 | |
| *** armax has joined #openstack-nova | 03:33 | |
| *** vladikr has quit IRC | 03:34 | |
| *** ducttape_ has quit IRC | 03:34 | |
| *** diga has joined #openstack-nova | 03:35 | |
| *** armax has quit IRC | 03:36 | |
| *** _ducttape_ has joined #openstack-nova | 03:39 | |
| *** ducnc has quit IRC | 03:44 | |
| *** ljjjusti_ has quit IRC | 03:48 | |
| *** ljjjustin has joined #openstack-nova | 03:48 | |
| *** _ducttape_ has quit IRC | 03:48 | |
| *** dixiaoli has quit IRC | 03:48 | |
| *** amotoki has joined #openstack-nova | 03:50 | |
| *** ducttape_ has joined #openstack-nova | 03:51 | |
| *** dixiaoli has joined #openstack-nova | 03:52 | |
| *** mdnadeem has joined #openstack-nova | 03:52 | |
| *** amotoki has quit IRC | 03:56 | |
| *** amotoki has joined #openstack-nova | 04:00 | |
| *** dixiaoli has quit IRC | 04:00 | |
| *** yamamoto has joined #openstack-nova | 04:01 | |
| *** thorst has joined #openstack-nova | 04:01 | |
| openstackgerrit | Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230 | 04:02 |
| *** ijw has joined #openstack-nova | 04:02 | |
| *** Qiming has quit IRC | 04:04 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738 | 04:05 |
| *** ducttape_ has quit IRC | 04:05 | |
| *** Qiming has joined #openstack-nova | 04:06 | |
| *** thorst has quit IRC | 04:06 | |
| *** edmondsw has joined #openstack-nova | 04:08 | |
| *** zhurong has quit IRC | 04:09 | |
| *** edmondsw has quit IRC | 04:12 | |
| *** masayukig has quit IRC | 04:14 | |
| *** gouthamr has quit IRC | 04:14 | |
| *** ducttape_ has joined #openstack-nova | 04:19 | |
| *** amotoki has quit IRC | 04:22 | |
| *** dixiaoli has joined #openstack-nova | 04:24 | |
| *** dixiaoli has quit IRC | 04:25 | |
| *** armax has joined #openstack-nova | 04:26 | |
| *** ducttape_ has quit IRC | 04:27 | |
| *** ducttape_ has joined #openstack-nova | 04:28 | |
| *** zhurong has joined #openstack-nova | 04:30 | |
| *** vks1 has joined #openstack-nova | 04:30 | |
| *** ducttape_ has quit IRC | 04:33 | |
| *** ayogi has joined #openstack-nova | 04:34 | |
| *** psachin has joined #openstack-nova | 04:42 | |
| *** yamamoto has quit IRC | 04:42 | |
| *** bmace has quit IRC | 04:44 | |
| *** yamamoto has joined #openstack-nova | 04:44 | |
| *** bmace has joined #openstack-nova | 04:44 | |
| *** dikonoo has joined #openstack-nova | 04:48 | |
| *** dikonoor has joined #openstack-nova | 04:48 | |
| *** avolkov has quit IRC | 04:54 | |
| *** avolkov has joined #openstack-nova | 04:54 | |
| *** adisky_ has joined #openstack-nova | 04:54 | |
| *** thorst has joined #openstack-nova | 05:02 | |
| *** thorst has quit IRC | 05:07 | |
| *** ratailor has joined #openstack-nova | 05:07 | |
| *** yamamoto has quit IRC | 05:09 | |
| *** udesale has joined #openstack-nova | 05:10 | |
| *** frickler has quit IRC | 05:11 | |
| *** frickler has joined #openstack-nova | 05:12 | |
| *** wagnqwsh has joined #openstack-nova | 05:12 | |
| *** suresh12 has joined #openstack-nova | 05:12 | |
| *** fragatin_ has joined #openstack-nova | 05:14 | |
| *** ijw has quit IRC | 05:15 | |
| *** nkrinner_afk is now known as nkrinner | 05:16 | |
| *** suresh12 has quit IRC | 05:17 | |
| *** fragatina has quit IRC | 05:17 | |
| *** fragatin_ has quit IRC | 05:18 | |
| *** trinaths has joined #openstack-nova | 05:19 | |
| *** dharinic has quit IRC | 05:22 | |
| *** prateek has joined #openstack-nova | 05:23 | |
| *** yamamoto has joined #openstack-nova | 05:28 | |
| *** ducttape_ has joined #openstack-nova | 05:29 | |
| *** brault|away has joined #openstack-nova | 05:30 | |
| *** fragatina has joined #openstack-nova | 05:31 | |
| *** fragatina has quit IRC | 05:31 | |
| *** fragatina has joined #openstack-nova | 05:31 | |
| *** fragatina has quit IRC | 05:34 | |
| *** fragatina has joined #openstack-nova | 05:34 | |
| *** nkorabli has joined #openstack-nova | 05:35 | |
| *** dixiaoli has joined #openstack-nova | 05:35 | |
| *** ducttape_ has quit IRC | 05:35 | |
| *** zhurong has quit IRC | 05:41 | |
| *** takashin has left #openstack-nova | 05:41 | |
| *** yamamoto has quit IRC | 05:41 | |
| *** dixiaoli has quit IRC | 05:43 | |
| *** rcernin has joined #openstack-nova | 05:44 | |
| *** raunak has quit IRC | 05:46 | |
| *** adrian_otto has joined #openstack-nova | 05:52 | |
| *** suresh12 has joined #openstack-nova | 05:52 | |
| *** belmoreira has joined #openstack-nova | 05:55 | |
| *** jaosorior has joined #openstack-nova | 05:56 | |
| *** dixiaoli has joined #openstack-nova | 05:59 | |
| *** dixiaoli has quit IRC | 06:00 | |
| *** thorst has joined #openstack-nova | 06:03 | |
| *** andreas_s has joined #openstack-nova | 06:04 | |
| *** amotoki has joined #openstack-nova | 06:05 | |
| *** zhurong has joined #openstack-nova | 06:05 | |
| *** amotoki_ has joined #openstack-nova | 06:08 | |
| *** thorst has quit IRC | 06:08 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Rework descriptions in os-hypervisors https://review.openstack.org/452030 | 06:10 |
| *** amotoki has quit IRC | 06:10 | |
| *** dixiaoli has joined #openstack-nova | 06:14 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add os-traits reference https://review.openstack.org/447401 | 06:17 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Make metadata doc more readable https://review.openstack.org/450551 | 06:19 |
| gibi | xgerman: hi! I saw you are trying to use soft-anti-affinity | 06:19 |
| *** voelzmo has joined #openstack-nova | 06:19 | |
| *** dikonoo has quit IRC | 06:19 | |
| *** dikonoor has quit IRC | 06:19 | |
| *** suresh12 has quit IRC | 06:20 | |
| gibi | xgerman: The error you linked here https://www.irccloud.com/pastebin/zfgU1sJu/ seems to come from something outside of nova. Do you have more details about what you try to do with nova and how that action fails on nova side? | 06:20 |
| *** slaweq has quit IRC | 06:21 | |
| *** claudiub has joined #openstack-nova | 06:21 | |
| *** slaweq has joined #openstack-nova | 06:22 | |
| *** slaweq has quit IRC | 06:22 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova master: Trivial: Remove unused method https://review.openstack.org/452054 | 06:22 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add api sample tests for security group rules https://review.openstack.org/452055 | 06:22 |
| *** voelzmo has quit IRC | 06:23 | |
| *** slaweq has joined #openstack-nova | 06:24 | |
| *** voelzmo has joined #openstack-nova | 06:29 | |
| *** udesale__ has joined #openstack-nova | 06:29 | |
| *** udesale has quit IRC | 06:31 | |
| *** udesale has joined #openstack-nova | 06:33 | |
| *** udesale__ has quit IRC | 06:34 | |
| *** mnestratov has joined #openstack-nova | 06:35 | |
| *** yamamoto has joined #openstack-nova | 06:42 | |
| *** satyar has joined #openstack-nova | 06:45 | |
| *** ltomasbo|away is now known as ltomasbo | 06:46 | |
| *** markus_z has joined #openstack-nova | 06:48 | |
| *** yamamoto has quit IRC | 06:49 | |
| *** armax has quit IRC | 06:56 | |
| *** ducttape_ has joined #openstack-nova | 07:00 | |
| openstackgerrit | Gergely Csatari proposed openstack/nova master: api-ref: adminPass is optional in Create Server https://review.openstack.org/451497 | 07:00 |
| *** abalutoiu has joined #openstack-nova | 07:00 | |
| *** thorst has joined #openstack-nova | 07:04 | |
| *** ducttape_ has quit IRC | 07:05 | |
| *** nkorabli has quit IRC | 07:06 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: remove ugly local import https://review.openstack.org/450787 | 07:06 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: remove get_nw_info_for_instance from compute.utils https://review.openstack.org/451387 | 07:06 |
| *** karthiks has joined #openstack-nova | 07:07 | |
| *** nkorabli has joined #openstack-nova | 07:07 | |
| *** tesseract has joined #openstack-nova | 07:08 | |
| *** thorst has quit IRC | 07:09 | |
| *** nkorabli has quit IRC | 07:11 | |
| openstackgerrit | Gergely Csatari proposed openstack/nova master: api-ref: networks is mandatory in Create Server https://review.openstack.org/452069 | 07:14 |
| *** dharinic has joined #openstack-nova | 07:16 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: remove get_nw_info_for_instance from compute.utils https://review.openstack.org/451387 | 07:16 |
| openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration https://review.openstack.org/451657 | 07:17 |
| *** yingjun has quit IRC | 07:18 | |
| *** dharinic has quit IRC | 07:23 | |
| *** sridharg has joined #openstack-nova | 07:25 | |
| *** Qiming has quit IRC | 07:26 | |
| *** pcaruana has joined #openstack-nova | 07:27 | |
| *** slaweq_ has joined #openstack-nova | 07:28 | |
| *** slaweq_ has quit IRC | 07:28 | |
| *** dixiaoli has quit IRC | 07:29 | |
| *** udesale has quit IRC | 07:30 | |
| *** udesale has joined #openstack-nova | 07:30 | |
| *** dixiaoli has joined #openstack-nova | 07:32 | |
| *** Qiming has joined #openstack-nova | 07:32 | |
| *** ralonsoh has joined #openstack-nova | 07:34 | |
| *** Nisha_Agarwal has joined #openstack-nova | 07:34 | |
| *** ralonsoh_ has joined #openstack-nova | 07:35 | |
| *** jpena|off is now known as jpena | 07:36 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add empty flavor object info in server api-ref https://review.openstack.org/452078 | 07:36 |
| *** fragatina has quit IRC | 07:37 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Reduced the complexity of the _init_instance method (4) https://review.openstack.org/420632 | 07:38 |
| *** ralonsoh has quit IRC | 07:39 | |
| -openstackstatus- NOTICE: Jobs in gate are failing with POST_FAILURE. Infra roots are investigating | 07:43 | |
| *** ChanServ changes topic to "Jobs in gate are failing with POST_FAILURE. Infra roots are investigating" | 07:43 | |
| *** ralonsoh_ is now known as ralonsoh | 07:44 | |
| *** slaweq_ has joined #openstack-nova | 07:53 | |
| *** slaweq_ has quit IRC | 07:53 | |
| *** slaweq_ has joined #openstack-nova | 07:53 | |
| *** rcernin has quit IRC | 07:54 | |
| *** tesseract has quit IRC | 07:54 | |
| *** tesseract has joined #openstack-nova | 07:55 | |
| *** rcernin has joined #openstack-nova | 07:55 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 07:58 |
| *** zzzeek has quit IRC | 08:00 | |
| *** gszasz has joined #openstack-nova | 08:01 | |
| *** zzzeek has joined #openstack-nova | 08:04 | |
| *** dixiaoli has quit IRC | 08:05 | |
| *** thorst has joined #openstack-nova | 08:05 | |
| *** dixiaoli has joined #openstack-nova | 08:05 | |
| *** slaweq_ has quit IRC | 08:05 | |
| *** slaweq_ has joined #openstack-nova | 08:06 | |
| *** slaweq has quit IRC | 08:06 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 08:06 |
| *** slaweq_ has quit IRC | 08:07 | |
| *** mlakat has joined #openstack-nova | 08:07 | |
| *** mlakat has quit IRC | 08:08 | |
| *** slaweq has joined #openstack-nova | 08:12 | |
| *** karimb has joined #openstack-nova | 08:13 | |
| *** jogo has quit IRC | 08:13 | |
| *** jogo has joined #openstack-nova | 08:15 | |
| *** karimb has quit IRC | 08:17 | |
| *** lpetrut has joined #openstack-nova | 08:18 | |
| *** suresh12 has joined #openstack-nova | 08:20 | |
| *** karimb has joined #openstack-nova | 08:23 | |
| -openstackstatus- NOTICE: logs.openstack.org has corrupted disks, it's being repaired. Please avoid rechecking until this is fixed | 08:24 | |
| *** ChanServ changes topic to "logs.openstack.org has corrupted disks, it's being repaired. Please avoid rechecking until this is fixed" | 08:24 | |
| *** suresh12 has quit IRC | 08:24 | |
| *** thorst has quit IRC | 08:24 | |
| *** lucas-afk is now known as lucasagomes | 08:26 | |
| *** derekh has joined #openstack-nova | 08:32 | |
| *** mlakat has joined #openstack-nova | 08:33 | |
| *** abalutoiu_ has joined #openstack-nova | 08:35 | |
| *** abalutoiu has quit IRC | 08:37 | |
| *** avolkov has quit IRC | 08:37 | |
| *** zyun has joined #openstack-nova | 08:38 | |
| zyun | Hi all | 08:38 |
| zyun | May I ask a question here? | 08:38 |
| zyun | when running the "nova image-create" in vmware env, could I specify this created template stored into specificy datastore? | 08:39 |
| zyun | Many thanks for any clue :) | 08:39 |
| *** dixiaoli has quit IRC | 08:45 | |
| *** jahsis has joined #openstack-nova | 08:46 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 08:47 |
| *** gszasz has quit IRC | 08:58 | |
| *** dixiaoli has joined #openstack-nova | 09:05 | |
| *** baoli has joined #openstack-nova | 09:07 | |
| *** baoli has quit IRC | 09:12 | |
| lyarwood | johnthetubaguy / mdbooth ; Matt is +2 on this now but I'd be happy to respin with the nits corrected this morning if you think it's worthwhile | 09:12 |
| lyarwood | sfinucan: https://review.openstack.org/#/c/451870/ - if you have time, just a silly docs lift and shift to make it easier to find the releasenote guide in dev-ref | 09:13 |
| johnthetubaguy | lyarwood: I think I was +2 before, minus the conf nit, will take a peak | 09:16 |
| *** avolkov has joined #openstack-nova | 09:17 | |
| *** annegentle has quit IRC | 09:18 | |
| *** mlakat has quit IRC | 09:19 | |
| *** cdent has joined #openstack-nova | 09:20 | |
| *** gcb has quit IRC | 09:21 | |
| *** rmart04 has joined #openstack-nova | 09:21 | |
| *** Nisha_Agarwal has quit IRC | 09:21 | |
| lyarwood | we are removing _L*?! I missed that memo. | 09:24 |
| *** zyun has quit IRC | 09:24 | |
| lyarwood | ah found the thread, sorry. | 09:24 |
| sfinucan | lyarwood: Yup, that's an easy win | 09:25 |
| * sfinucan is *always* happy to review anything that improves documentation | 09:25 | |
| johnthetubaguy | lyarwood: yeah, its kinda new, I am just upset we added that logging nonsense it in the first place | 09:25 |
| *** wagnqwsh has quit IRC | 09:26 | |
| lyarwood | sfinucan: :D good to know thanks, I'll follow up with the nits when I get time | 09:26 |
| *** ekuris has quit IRC | 09:28 | |
| johnthetubaguy | lyarwood: did you see the detach patch, I think we might need another refactor in between | 09:29 |
| lyarwood | sfinucan: btw this only came up thanks to my inability to find the releasenote docs while looking into writing one for https://review.openstack.org/#/c/391597/ (encryptor class removal from nova), still can't decide if one is really needed here, any thoughts? | 09:29 |
| * sfinucan looking | 09:29 | |
| lyarwood | johnthetubaguy: yeah I did, what did you have in mind? | 09:31 |
| lyarwood | johnthetubaguy: I don't understand the need for the connection_info['attachment_id'] stuff in the driver bdm tbh | 09:31 |
| johnthetubaguy | lyarwood: FWIW, I always consider my self an operator, and try work out if I want to know anything about the change when I upgrade the system (under the assumption I the operator depend on that thing that just got tweaked) | 09:31 |
| johnthetubaguy | lyarwood: you got a link to file you see that attachement_id stuff being done? | 09:32 |
| *** edmondsw has joined #openstack-nova | 09:32 | |
| johnthetubaguy | oh, I see it now | 09:32 |
| lyarwood | johnthetubaguy: https://review.openstack.org/#/c/438750/4/nova/virt/block_device.py | 09:33 |
| lyarwood | urgh my vps is lagging | 09:33 |
| johnthetubaguy | lyarwood: honestly, that might just be a poor rebase thing | 09:34 |
| *** ekuris has joined #openstack-nova | 09:34 | |
| *** udesale__ has joined #openstack-nova | 09:35 | |
| lyarwood | johnthetubaguy: true, what did you have in mind for the additional refactor before the detach changes? | 09:36 |
| *** psachin has quit IRC | 09:36 | |
| *** kevinz has quit IRC | 09:36 | |
| *** edmondsw has quit IRC | 09:37 | |
| johnthetubaguy | lyarwood: I was expecting all this to happen inside the bdm if you call "detach" or something like that: https://review.openstack.org/#/c/438750/4/nova/compute/api.py | 09:37 |
| *** kevinz has joined #openstack-nova | 09:37 | |
| *** udesale has quit IRC | 09:37 | |
| lyarwood | johnthetubaguy: understood, volume_api or just another method in the driver bdm? | 09:39 |
| johnthetubaguy | lyarwood: good point, probably bdm I guess | 09:40 |
| johnthetubaguy | I could go either way | 09:40 |
| jaosorior | owalsh: alright, oh, so from what I can see in that LP. It seems that there is still a need for the URI there | 09:40 |
| lyarwood | kk, I'll take a look at that and the attach stuff today | 09:40 |
| jaosorior | owalsh: so, why is was that URI being deprecated in the first place? | 09:40 |
| owalsh | jaosorior: not sure | 09:41 |
| johnthetubaguy | lyarwood: actually how do BDMs fit in the non-libvirt drivers these days? | 09:41 |
| jaosorior | seems like it was quite useful and would save a lot of configuration hassles, as opposed to needing to split the config to several options | 09:41 |
| johnthetubaguy | owalsh: jaosorior: I am curious, which URI? | 09:42 |
| owalsh | johnthetubaguy: live_migration_uri | 09:42 |
| johnthetubaguy | owalsh: oh, so bits of that URL are best defaulted by the driver, based on what sort of live-migration you are doing, if I remember correctly, pkoniszewski has more context on that | 09:42 |
| owalsh | johnthetubaguy: https://bugs.launchpad.net/nova/+bug/1671288 | 09:43 |
| openstack | Launchpad bug 1671288 in OpenStack Compute (nova) "live_migration_uri to live_migration_scheme SSH settings" [Undecided,Confirmed] | 09:43 |
| lyarwood | johnthetubaguy: they should all use bdms and driver bdms now AFAIK | 09:43 |
| johnthetubaguy | lyarwood: OK, fingers crossed | 09:43 |
| owalsh | johnthetubaguy: i need to set the ssh user and key | 09:43 |
| johnthetubaguy | owalsh: I see "live_migration_scheme = None" in that configuration reference | 09:44 |
| *** sdague has joined #openstack-nova | 09:44 | |
| jaosorior | johnthetubaguy: so, the main issue is not the scheme or the IP. But the extra configurations one can give as query parameters in the URI | 09:45 |
| jaosorior | johnthetubaguy: which vary depending on the scheme used. | 09:45 |
| johnthetubaguy | jaosorior: owalsh: so the aim was to make it easier to use, so the defaults are correct for all the different hypervisors, if thats not going to work out, totally up for reversing the deprecation | 09:47 |
| johnthetubaguy | jaosorior: owalsh: could you just write out like a before and after, to see how bad it looks? | 09:47 |
| pkoniszewski | so live_migration_uri is mostly used to hack QEMU rather than for "extra configuration" | 09:48 |
| *** rmart04 has quit IRC | 09:48 | |
| pkoniszewski | I understand your problem but I'd like to know your use case for this ssh key | 09:49 |
| jaosorior | pkoniszewski: I wouldn't say that specifying the known hosts, key locations or ssh keys are "hacking qemu" | 09:49 |
| pkoniszewski | does QEMU use root's ssh key? not sure i remember that correctly | 09:49 |
| owalsh | pkoniszewski: it's libvirt, running as root ,so yes | 09:50 |
| pkoniszewski | jaosorior: so mostly operators used this URI to force QEMU to use different link for live migrations | 09:50 |
| pkoniszewski | not to specify keys etc. | 09:50 |
| owalsh | pkoniszewski: I'm adding SSH key setup for the nova user for cold-migration, want to resuse this for live migration to DRY | 09:51 |
| pkoniszewski | got it | 09:51 |
| johnthetubaguy | ah, that makes sense | 09:51 |
| johnthetubaguy | sounds like we should do that out of the box | 09:52 |
| owalsh | pkoniszewski: so qemu+ssh://nova@%s/system?keyfile=/var/lib/nova/.ssh/id_rsa | 09:52 |
| *** adrian_otto has quit IRC | 09:53 | |
| johnthetubaguy | pkoniszewski: whats the best way to do that QEMU? | 09:53 |
| pkoniszewski | well, adding conf option to specify username for SSH makes sense to me | 09:53 |
| pkoniszewski | by default it uses root, this might be a security issue for ome | 09:54 |
| johnthetubaguy | feels like we are adding lots of options at that point... but that might be easiest. | 09:54 |
| *** psachin has joined #openstack-nova | 09:54 | |
| johnthetubaguy | owalsh: I guess migration just uses that key because of the user you are running Nova with, its not a config as such? | 09:55 |
| pkoniszewski | we can do any, teach people (or let them google it) how to configure URI by themselves, or break it into few options | 09:55 |
| johnthetubaguy | yeah, I am just curious what folks would prefer | 09:55 |
| owalsh | johnthetubaguy: yea, /var/lib/nova is the nova home dir | 09:56 |
| pkoniszewski | by default LM uses TCP, SSH is just an option | 09:56 |
| owalsh | pkoniszewski, johnthetubaguy: there is also this https://bugs.launchpad.net/nova/+bug/1677913 | 09:56 |
| openstack | Launchpad bug 1677913 in OpenStack Compute (nova) "percent must be escaped in live_migration_uri" [Undecided,New] | 09:56 |
| johnthetubaguy | pkoniszewski: ssh just gives you the secure transport here? | 09:56 |
| pkoniszewski | yes | 09:56 |
| owalsh | johnthetubaguy: ssh + nc is used to tunnel the libvirt unix socket | 09:57 |
| *** abalutoiu_ has quit IRC | 09:57 | |
| johnthetubaguy | owalsh: that bug about the % needing to be escaped really makes we want us to addd extra config options for the ssl config (and we keep the URL for an extra cycle while that is added) | 09:58 |
| johnthetubaguy | owalsh: would that work for you all? | 09:58 |
| *** BlackDex has joined #openstack-nova | 09:58 | |
| openstackgerrit | Roman Podoliaka proposed openstack/osc-placement master: WIP: implement API version 1.0 https://review.openstack.org/451877 | 09:59 |
| openstackgerrit | Roman Podoliaka proposed openstack/osc-placement master: tests: add a hook for functional testing in the gate https://review.openstack.org/452122 | 09:59 |
| owalsh | johnthetubaguy: yea, might end up with a lot of params though | 10:00 |
| johnthetubaguy | owalsh: but most folks don't need to tweak any of them, and at least it much more discoverable what you can do | 10:00 |
| johnthetubaguy | the alternative seems to involve string substitutions, etc, which is just a bit funky. | 10:01 |
| *** udesale__ has quit IRC | 10:03 | |
| *** kevinz has quit IRC | 10:05 | |
| johnthetubaguy | pkoniszewski: I am curious on the URL substitution, is there a reason why its not the destination host that just provides its whole URL? | 10:05 |
| owalsh | johnthetubaguy: you suggesting we could set it to user@host:port? | 10:06 |
| johnthetubaguy | owalsh: you probably could set that as the host... I guess | 10:07 |
| johnthetubaguy | owalsh: not sure I like that though | 10:07 |
| *** abalutoiu has joined #openstack-nova | 10:07 | |
| owalsh | or user@host:port/system?key=foo | 10:07 |
| *** cdent has quit IRC | 10:07 | |
| johnthetubaguy | not sure | 10:07 |
| johnthetubaguy | it might work | 10:07 |
| owalsh | don't think the latter would work as nova will append /system | 10:08 |
| johnthetubaguy | yeah, that sounds right | 10:08 |
| openstackgerrit | Danfly proposed openstack/nova master: remove mox from unit/virt/vmwareapi/test_driver_api.py https://review.openstack.org/452128 | 10:08 |
| johnthetubaguy | owalsh: really I am suggesting we add a new configuration option, and you keep using the URI for now, and we extend the deprecation period of the URI while we get more feedback | 10:09 |
| *** lpetrut has quit IRC | 10:09 | |
| owalsh | johnthetubaguy: ack, jaosorior ^^ | 10:10 |
| jaosorior | sounds good to me | 10:10 |
| johnthetubaguy | owalsh: jaosorior: would you fancy doing that patch? | 10:11 |
| johnthetubaguy | happy to put it near the top of my review pile once its up | 10:11 |
| owalsh | johnthetubaguy: sure | 10:13 |
| johnthetubaguy | cool, thanks for that info, thats a really interesting use case I had not thought about before | 10:14 |
| * johnthetubaguy learns something new about live-migrate most months this year! | 10:14 | |
| *** amotoki_ has quit IRC | 10:15 | |
| *** lpetrut has joined #openstack-nova | 10:16 | |
| *** diga has quit IRC | 10:17 | |
| owalsh | johnthetubaguy: just thinking...probably need an option for user (ssh only), port (all), and extraparams (all) | 10:17 |
| owalsh | johnthetubaguy: but if user@host:port works with for live_migration_inbound_addr we would just need an extraparams optoin | 10:18 |
| *** ratailor has quit IRC | 10:18 | |
| *** zhurong has quit IRC | 10:18 | |
| *** ljjjustin has quit IRC | 10:19 | |
| *** tuanluong has quit IRC | 10:20 | |
| jaosorior | owalsh: I don't see why adding that in live_migration_inbound_addr | 10:20 |
| jaosorior | wouldn't work | 10:20 |
| *** lpetrut has quit IRC | 10:20 | |
| jaosorior | seems like it's just some substitution | 10:20 |
| owalsh | jaosorior: I was just about to peek at the code :-) | 10:21 |
| *** thorst has joined #openstack-nova | 10:21 | |
| *** tovin07_ has quit IRC | 10:22 | |
| *** GnomeKing has joined #openstack-nova | 10:23 | |
| *** namnh has quit IRC | 10:24 | |
| *** nkrinner is now known as nkrinner_afk | 10:24 | |
| *** sambetts|afk is now known as sambetts | 10:25 | |
| *** thorst has quit IRC | 10:26 | |
| *** satyar has quit IRC | 10:28 | |
| *** lpetrut has joined #openstack-nova | 10:29 | |
| *** lpetrut has quit IRC | 10:29 | |
| *** trinaths has left #openstack-nova | 10:30 | |
| *** ralonsoh_ has joined #openstack-nova | 10:30 | |
| *** lpetrut has joined #openstack-nova | 10:31 | |
| *** lpetrut has quit IRC | 10:31 | |
| *** abalutoiu has quit IRC | 10:31 | |
| *** lpetrut has joined #openstack-nova | 10:32 | |
| *** ratailor has joined #openstack-nova | 10:33 | |
| *** ralonsoh has quit IRC | 10:34 | |
| *** tpatzig_ has joined #openstack-nova | 10:36 | |
| *** carthaca_ has joined #openstack-nova | 10:36 | |
| *** david_1 has joined #openstack-nova | 10:36 | |
| johnthetubaguy | jaosorior: owalsh: as long as we clearly document what you tweak to change those things in the comments, we could entertain that | 10:38 |
| johnthetubaguy | johnthetubaguy: owalsh: by we, I mean I | 10:38 |
| *** tpatzig_ has quit IRC | 10:38 | |
| *** carthaca_ has quit IRC | 10:38 | |
| *** david_1 has quit IRC | 10:38 | |
| *** cdent has joined #openstack-nova | 10:38 | |
| owalsh | johnthetubaguy: you're royalty, yea? | 10:39 |
| johnthetubaguy | heh | 10:39 |
| johnthetubaguy | one does ones best to talk like ones queen | 10:40 |
| cdent | one does | 10:40 |
| *** zhurong has joined #openstack-nova | 10:41 | |
| *** gszasz has joined #openstack-nova | 10:42 | |
| *** abalutoiu has joined #openstack-nova | 10:46 | |
| *** phuongnh has quit IRC | 10:46 | |
| *** dixiaoli has quit IRC | 10:49 | |
| *** psachin has quit IRC | 10:50 | |
| *** dixiaoli has joined #openstack-nova | 10:51 | |
| cdent | Is logs.o.o back to working? | 10:51 |
| *** gcb has joined #openstack-nova | 10:51 | |
| *** psachin has joined #openstack-nova | 10:52 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 11:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Destroy the ReqSpec object when deleting the instance https://review.openstack.org/391060 | 11:00 |
| *** ratailor has quit IRC | 11:00 | |
| *** ratailor has joined #openstack-nova | 11:01 | |
| *** lpetrut has quit IRC | 11:02 | |
| *** dikonoo has joined #openstack-nova | 11:03 | |
| *** dikonoor has joined #openstack-nova | 11:03 | |
| *** slaweq has quit IRC | 11:05 | |
| *** slaweq has joined #openstack-nova | 11:06 | |
| lyarwood | johnthetubaguy: mdbooth is +1 on https://review.openstack.org/439520 btw if you're happy to +W | 11:06 |
| johnthetubaguy | lyarwood: sweet | 11:07 |
| *** gcb has quit IRC | 11:20 | |
| *** suresh12 has joined #openstack-nova | 11:20 | |
| *** thorst has joined #openstack-nova | 11:22 | |
| frickler | cdent: recovery of old logs is ongoing, new jobs should be fine | 11:24 |
| cdent | thanks frickler | 11:24 |
| *** lpetrut has joined #openstack-nova | 11:24 | |
| *** suresh12 has quit IRC | 11:24 | |
| *** lpetrut has quit IRC | 11:25 | |
| *** thorst has quit IRC | 11:27 | |
| *** lucasagomes is now known as lucas-hungry | 11:28 | |
| *** zz_dimtruck is now known as dimtruck | 11:30 | |
| *** thorst has joined #openstack-nova | 11:32 | |
| *** amotoki has joined #openstack-nova | 11:32 | |
| *** gcb has joined #openstack-nova | 11:33 | |
| *** lpetrut has joined #openstack-nova | 11:37 | |
| oomichi | gmann: -1 on https://review.openstack.org/#/c/451673 we need to check tempest jsonschema as qa team ;) | 11:44 |
| *** dimtruck is now known as zz_dimtruck | 11:46 | |
| *** zz_dimtruck is now known as dimtruck | 11:46 | |
| *** nkrinner_afk is now known as nkrinner | 11:47 | |
| *** vladikr has joined #openstack-nova | 11:47 | |
| *** dixiaoli has quit IRC | 11:54 | |
| *** dimtruck is now known as zz_dimtruck | 11:56 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read NIC features in libvirt https://review.openstack.org/444415 | 11:58 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Modify PciPassthroughFilter to accept lists https://review.openstack.org/444946 | 11:58 |
| *** zhurong has quit IRC | 11:59 | |
| *** liverpooler has quit IRC | 12:00 | |
| *** liverpooler has joined #openstack-nova | 12:00 | |
| *** prateek has quit IRC | 12:01 | |
| *** prateek has joined #openstack-nova | 12:01 | |
| *** jjung has joined #openstack-nova | 12:03 | |
| *** ekuris has quit IRC | 12:06 | |
| *** cdent has quit IRC | 12:08 | |
| *** ekuris has joined #openstack-nova | 12:09 | |
| *** edleafe is now known as figleaf | 12:10 | |
| *** aysyd has joined #openstack-nova | 12:11 | |
| *** ekuris has quit IRC | 12:14 | |
| *** adreznec has quit IRC | 12:14 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 12:14 |
| *** adreznec has joined #openstack-nova | 12:17 | |
| *** ducttape_ has joined #openstack-nova | 12:17 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP compute: Centralise terminate and detach volume api calls in bdm https://review.openstack.org/452181 | 12:22 |
| *** lucas-hungry is now known as lucasagomes | 12:22 | |
| *** ducttape_ has quit IRC | 12:30 | |
| *** ekuris has joined #openstack-nova | 12:30 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for POST RP inventories https://review.openstack.org/451855 | 12:30 |
| *** edmondsw has joined #openstack-nova | 12:33 | |
| ildikov | johnthetubaguy: hi, around? | 12:33 |
| johnthetubaguy | I am | 12:33 |
| ildikov | just wanted to ask about the meeting slot | 12:33 |
| ildikov | I guess we should look into either trying to move it back with an hour or to another day, right? | 12:33 |
| johnthetubaguy | ideally, but I am OK reading the logs and the ML to keep track | 12:34 |
| ildikov | can you make it to the let's say first 15-20 minutes if we keep it where it is now? | 12:35 |
| johnthetubaguy | good question, it slightly depends on who is cooking dinner, and thats usually me | 12:35 |
| *** catintheroof has joined #openstack-nova | 12:36 | |
| *** ayogi has quit IRC | 12:36 | |
| *** lpetrut has quit IRC | 12:36 | |
| ildikov | hmm ok, then maybe it would worth looking into other options | 12:36 |
| johnthetubaguy | ildikov: moving it one hour back, to what it was a few weeks ago would work | 12:36 |
| johnthetubaguy | I know thats not the norm, but it could work here | 12:37 |
| ildikov | I will ask around | 12:38 |
| ildikov | and/or create a Doodle poll with the current slot and the one hour earlier for every weekdays and see what's the best option | 12:38 |
| *** vks1 has quit IRC | 12:38 | |
| ildikov | if moving it back with an hour seems to work for the most, I'll leave the poll | 12:39 |
| johnthetubaguy | ildikov: just thinking that was Ok for us all a few weeks back, but I know some have UTC fixed meetings | 12:39 |
| ildikov | it's always tricky with these changes | 12:39 |
| ildikov | although I don't recall any US folks complaining about the change and they switched before us | 12:40 |
| ildikov | so it should be ok in theory | 12:40 |
| johnthetubaguy | yeah, its never easy | 12:41 |
| johnthetubaguy | mdbooth: I just remembered, are you free for a quick hangout this afternoon? | 12:41 |
| mdbooth | johnthetubaguy: Yep | 12:41 |
| johnthetubaguy | mdbooth: does now ish work? link in 5 mins? | 12:41 |
| mdbooth | johnthetubaguy: I should be finished eating by then, sure :) | 12:42 |
| johnthetubaguy | perfect | 12:42 |
| *** awaugama has joined #openstack-nova | 12:44 | |
| *** HW-Peter has joined #openstack-nova | 12:45 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Switch Nova Quobyte volume driver to mount via systemd-run https://review.openstack.org/432344 | 12:51 |
| kaisers | mdbooth: ping ^^^^ (back to systemd-run but with working exit codes) | 12:51 |
| *** andreas_s has quit IRC | 12:52 | |
| openstackgerrit | Samantha Blanco proposed openstack/nova master: Make flavor-rxtx policy more granular https://review.openstack.org/444036 | 12:53 |
| *** kylek3h has quit IRC | 12:54 | |
| *** HW-Peter has quit IRC | 12:55 | |
| *** HW-Peter has joined #openstack-nova | 12:55 | |
| *** HW-Peter has quit IRC | 12:56 | |
| *** HW-Peter has joined #openstack-nova | 12:56 | |
| *** zz_dimtruck is now known as dimtruck | 12:58 | |
| *** arne_r has joined #openstack-nova | 13:00 | |
| *** gouthamr has joined #openstack-nova | 13:01 | |
| *** jjung has quit IRC | 13:04 | |
| *** cdent has joined #openstack-nova | 13:05 | |
| jaosorior | Hey guys, is live-migration already supported by the openstack CLI? Or do I have to use the nova CLI? | 13:06 |
| *** lpetrut has joined #openstack-nova | 13:06 | |
| *** jianghuaw has joined #openstack-nova | 13:09 | |
| *** xyang1 has joined #openstack-nova | 13:09 | |
| *** jamesdenton has joined #openstack-nova | 13:09 | |
| *** cleong has joined #openstack-nova | 13:10 | |
| *** lpetrut has quit IRC | 13:11 | |
| *** jamesden_ has joined #openstack-nova | 13:13 | |
| *** esberglu has joined #openstack-nova | 13:14 | |
| *** baoli has joined #openstack-nova | 13:14 | |
| *** jamesdenton has quit IRC | 13:14 | |
| *** baoli has quit IRC | 13:14 | |
| *** baoli has joined #openstack-nova | 13:15 | |
| *** liangy has joined #openstack-nova | 13:16 | |
| *** dave-mccowan has joined #openstack-nova | 13:19 | |
| *** ratailor has quit IRC | 13:19 | |
| *** mdrabe has joined #openstack-nova | 13:20 | |
| *** kylek3h has joined #openstack-nova | 13:22 | |
| *** tuan_luong has joined #openstack-nova | 13:23 | |
| *** tuanla_ has joined #openstack-nova | 13:24 | |
| *** kylek3h has quit IRC | 13:25 | |
| *** Guest16838 has quit IRC | 13:25 | |
| *** baoli has quit IRC | 13:28 | |
| *** ducttape_ has joined #openstack-nova | 13:30 | |
| *** mdnadeem has quit IRC | 13:31 | |
| *** lpetrut has joined #openstack-nova | 13:33 | |
| *** kylek3h has joined #openstack-nova | 13:33 | |
| *** mariusv_ has quit IRC | 13:34 | |
| *** mariusv has quit IRC | 13:34 | |
| xgerman | gibi: we figured it out - was using the wrong micro version in the client | 13:34 |
| *** mariusv has joined #openstack-nova | 13:35 | |
| *** mariusv has quit IRC | 13:35 | |
| *** mariusv has joined #openstack-nova | 13:35 | |
| *** ducttape_ has quit IRC | 13:35 | |
| *** aunnam_ has joined #openstack-nova | 13:35 | |
| *** Guest16838 has joined #openstack-nova | 13:37 | |
| *** ducttape_ has joined #openstack-nova | 13:38 | |
| *** ekuris has quit IRC | 13:38 | |
| gibi | xgerman: I'm glad that it works for you too | 13:39 |
| xgerman | thanks - sorry, for the confusion | 13:39 |
| *** ralonsoh__ has joined #openstack-nova | 13:41 | |
| *** ekuris has joined #openstack-nova | 13:43 | |
| *** ralonsoh_ has quit IRC | 13:43 | |
| gibi | no problem | 13:44 |
| *** artom_ has quit IRC | 13:46 | |
| *** artom_ has joined #openstack-nova | 13:47 | |
| *** baoli has joined #openstack-nova | 13:47 | |
| *** artom_ has quit IRC | 13:48 | |
| *** artom_ has joined #openstack-nova | 13:48 | |
| *** ZZelle has quit IRC | 13:49 | |
| *** ZZelle has joined #openstack-nova | 13:50 | |
| *** ZZelle is now known as Guest11182 | 13:50 | |
| *** marst has quit IRC | 13:52 | |
| *** efried has joined #openstack-nova | 13:52 | |
| *** mriedem has joined #openstack-nova | 13:53 | |
| *** amoralej is now known as amoralej|lunch | 13:54 | |
| *** Dinesh_Bhor has quit IRC | 13:55 | |
| *** dimtruck is now known as zz_dimtruck | 13:55 | |
| *** zz_dimtruck is now known as dimtruck | 13:55 | |
| *** marst has joined #openstack-nova | 13:58 | |
| *** eharney has joined #openstack-nova | 13:58 | |
| *** burt has joined #openstack-nova | 13:58 | |
| *** amotoki has quit IRC | 13:59 | |
| *** ekuris has quit IRC | 14:00 | |
| openstackgerrit | Steve Noyes proposed openstack/nova master: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/389798 | 14:02 |
| *** lucasxu has joined #openstack-nova | 14:03 | |
| *** artom_ is now known as artom | 14:04 | |
| *** gcb has quit IRC | 14:05 | |
| *** ratailor has joined #openstack-nova | 14:05 | |
| *** fragatina has joined #openstack-nova | 14:06 | |
| mordred | sdague: in the nova api docs, the reboot action and the resize action each show examples of payloads that have more keys and values than are documented | 14:07 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 14:08 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | 14:08 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 14:08 |
| sdague | mordred: pointer? | 14:08 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | 14:08 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 14:08 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 14:08 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 14:08 |
| sdague | mordred: I totally believe this could be the case | 14:08 |
| *** gcb has joined #openstack-nova | 14:09 | |
| mordred | sdague: https://developer.openstack.org/api-ref/compute/?expanded=reboot-server-reboot-action-detail,resize-server-resize-action-detail | 14:09 |
| mordred | sdague: reboot has 'type': 'HARD' in the example (is soft maybe another option?) | 14:09 |
| mordred | and resize has flavorRef and OS-DCF:diskConfig - I'm assuming flavorRef takes a flavor id - not sure what values other than AUTO are good for OS-DCF:diskConfig | 14:10 |
| sdague | yeh, so I have an unsatisfying answer for you | 14:10 |
| mordred | yay! | 14:10 |
| sdague | which is we've just not finished that scrube | 14:10 |
| sdague | let me see if there are some outstanding patches | 14:10 |
| mordred | okie. no worries if it's ongoing - I mostly wanted to report if it was an oversight | 14:10 |
| *** vks1 has joined #openstack-nova | 14:11 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: Add server-action-removefloatingip.json file and update servers-actions.inc https://review.openstack.org/446471 | 14:12 |
| *** nic has joined #openstack-nova | 14:12 | |
| *** Guest11182 is now known as ZZelle_ | 14:12 | |
| sdague | yeh, the file still has the .. needs:parameter_verification | 14:13 |
| sdague | .. needs:example_verification | 14:13 |
| sdague | .. needs:body_verification | 14:13 |
| sdague | at the top of it | 14:13 |
| mordred | cool | 14:13 |
| *** dikonoo has quit IRC | 14:15 | |
| sdague | mordred: if you had to pick one action to be right, which one would you want? I'll write on api-ref patch today on it | 14:15 |
| *** dikonoor has quit IRC | 14:15 | |
| ZZelle_ | Hi everyone | 14:15 |
| *** ekuris has joined #openstack-nova | 14:15 | |
| mriedem | mordred: HARD and SOFT are options https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L239 | 14:16 |
| ZZelle_ | It seems there is two different ways to add a new interface to a VM: using the action AddFixedIp and using os-interface resource ... which one should we prefer? | 14:16 |
| *** nkrinner is now known as nkrinner_afk | 14:16 | |
| mordred | sdague: neither is super urgent - but mriedem just answered reboot for me, so I'll vote for resize :) | 14:16 |
| *** jamielennox is now known as jamielennox|away | 14:16 | |
| *** tuanla_ has quit IRC | 14:17 | |
| *** tuan_luong has quit IRC | 14:17 | |
| sdague | ok | 14:17 |
| mriedem | here is the resize request schema https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L192 | 14:17 |
| mriedem | flavorRef is the only thing that's required | 14:17 |
| mriedem | i believe the diskConfig stuff is all xen | 14:18 |
| mriedem | all xen, all the time | 14:18 |
| mordred | do we expect reboot will ever get a type other than hard or soft? | 14:18 |
| mordred | yay xen! | 14:18 |
| mriedem | ZZelle_: we're going to deprecate addFixedIP | 14:18 |
| mriedem | mordred: such as?/ | 14:18 |
| ZZelle_ | mriedem, ok so i will use os-interface instead of AddFixedIp in OSC | 14:18 |
| mordred | mriedem: dunno. I was thinking of making the python paramter "hard=True" rather than type="HARD"|"SOFT" | 14:19 |
| mordred | mriedem: but making it a bool flag would get weird if in the future we made a "SQUISHY" or something | 14:19 |
| *** tuan_luong has joined #openstack-nova | 14:20 | |
| mriedem | limp? | 14:20 |
| *** tuanla_ has joined #openstack-nova | 14:20 | |
| mordred | yah. please do a limp reboot on that server | 14:21 |
| mriedem | limp == reboot the guest, but make sure my network and volumes are no longer connected | 14:21 |
| mriedem | because i like to be dumb | 14:21 |
| mordred | hahahaha | 14:21 |
| *** nic has quit IRC | 14:21 | |
| mordred | that seems like a good patch for tomorrow | 14:21 |
| *** yamamoto has joined #openstack-nova | 14:21 | |
| mriedem | ZZelle_: fyi https://review.openstack.org/#/c/384261/ | 14:22 |
| ZZelle_ | mriedem, thx | 14:22 |
| mriedem | ZZelle_: also, this is old https://review.openstack.org/#/c/350277/ but i'm not totally sure yet what the long-term plan for os-interface is | 14:23 |
| mriedem | ZZelle_: but for now i think using that for adding an interface is what you want | 14:23 |
| *** satyar has joined #openstack-nova | 14:24 | |
| mriedem | that was proposing to deprecate the GET methods, but i think we said we didn't want that, and we should deprecate os-virtual-interfaces, and make os-interfaces work like os-volume_attachments | 14:25 |
| mriedem | but that's probably all queens work at this point | 14:25 |
| ZZelle_ | mriedem, if we deprecate os-interface how do we attach a new interface to a VM :s? | 14:25 |
| ZZelle_ | mriedem, ok | 14:25 |
| mriedem | the spec was for deprecating the GET methods, | 14:25 |
| mriedem | not POST or DELETE | 14:25 |
| ZZelle_ | mriedem, ok | 14:25 |
| mriedem | it's a bit confusing, sorry about that | 14:25 |
| mriedem | anyway, https://developer.openstack.org/api-ref/compute/#port-interfaces-servers-os-interface is what you want | 14:26 |
| *** jjung has joined #openstack-nova | 14:29 | |
| *** suresh12 has joined #openstack-nova | 14:31 | |
| *** nkorabli has joined #openstack-nova | 14:32 | |
| *** jroll has quit IRC | 14:32 | |
| *** jroll has joined #openstack-nova | 14:34 | |
| *** nkorabli has quit IRC | 14:34 | |
| *** nkorabli has joined #openstack-nova | 14:35 | |
| *** amoralej|lunch is now known as amoralej | 14:36 | |
| *** nkorabli_ has joined #openstack-nova | 14:39 | |
| *** nkorabli has quit IRC | 14:39 | |
| *** prateek has quit IRC | 14:39 | |
| *** adisky_ has quit IRC | 14:39 | |
| *** sneti_ has joined #openstack-nova | 14:41 | |
| *** dillaman has joined #openstack-nova | 14:41 | |
| *** jaypipes has joined #openstack-nova | 14:43 | |
| *** jaypipes is now known as leakypipes | 14:44 | |
| *** ducttape_ has quit IRC | 14:44 | |
| *** ducttape_ has joined #openstack-nova | 14:44 | |
| mriedem | lyarwood: if you haven't started on it yet, i was going to push the changes to cleanup my nits in the detach refactor patch | 14:44 |
| *** dimtruck is now known as zz_dimtruck | 14:45 | |
| *** zz_dimtruck is now known as dimtruck | 14:45 | |
| *** ducttape_ has quit IRC | 14:45 | |
| *** ducttape_ has joined #openstack-nova | 14:45 | |
| lyarwood | mriedem: if you already have a commit feel free to push it | 14:46 |
| *** ducttape_ has quit IRC | 14:46 | |
| *** ducttape_ has joined #openstack-nova | 14:46 | |
| *** ducttape_ has quit IRC | 14:47 | |
| *** jahsis has quit IRC | 14:49 | |
| *** ratailor has quit IRC | 14:49 | |
| mdbooth | kaisers: I looked at --scope. I didn't actually test it, but from reading the docs it was my understanding that a --scope group will be terminated automatically with its parent | 14:50 |
| mdbooth | kaisers: I wouldn't bet my house on that, though. | 14:50 |
| *** stvnoyes has left #openstack-nova | 14:50 | |
| *** dimtruck is now known as zz_dimtruck | 14:54 | |
| dims | dulek : posted a day early? https://review.openstack.org/#/c/452219/ | 14:54 |
| dims | :) | 14:55 |
| *** voelzmo has quit IRC | 14:57 | |
| cdent | oh no, not that time of year :( | 14:57 |
| lyarwood | melwitt: btw https://review.openstack.org/#/c/451870/ is another trivial docs change if you have time today | 14:57 |
| mriedem | heh | 14:58 |
| mriedem | tweetmq? | 14:58 |
| mriedem | isn't there a forum session about dropping all other MQ backends except rabbit? | 14:58 |
| *** ducttape_ has joined #openstack-nova | 14:58 | |
| mriedem | http://forumtopics.openstack.org/cfp/details/8 | 14:58 |
| melwitt | lyarwood: looking | 14:58 |
| melwitt | lol "I would love to see an USPS as a possibility here" | 15:00 |
| mdbooth | kaisers: Hmm, kinda find the reason I thought that now. Could be I was imagining it. Have you tested this manually, btw? | 15:00 |
| mdbooth | kinda? can't | 15:01 |
| *** ijw has joined #openstack-nova | 15:01 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228 | 15:01 |
| leakypipes | sfinucan: quick review on this please? https://review.openstack.org/#/c/451635/2 | 15:02 |
| *** joanna__ has joined #openstack-nova | 15:02 | |
| *** arne_r has quit IRC | 15:05 | |
| *** rcernin has quit IRC | 15:08 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298 | 15:10 |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 15:10 |
| openstackgerrit | Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146 | 15:10 |
| *** belmoreira has quit IRC | 15:10 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 15:12 |
| *** psachin has quit IRC | 15:13 | |
| cfriesen | are there any qemu-knowledgable folks around? When using qcow2 instance storage I'm wondering why we convert backing file images to raw rather than just leaving them in whatever format they were in (raw, qcow2). | 15:14 |
| *** kaisers_ has joined #openstack-nova | 15:16 | |
| kaisers_ | mdbooth: Hi! Yep, i tested this manually. --scope works for me. I actually hacked it into the driver in a test instance and killed the n-cpu process hard, mount point remained active | 15:18 |
| mdbooth | kashyap: That's not quite the same thing | 15:18 |
| kashyap | cfriesen: Good question, I once recall digging it, and now lost it from memory | 15:18 |
| mdbooth | Eurgh | 15:18 |
| mdbooth | kaisers: ^^^ | 15:18 |
| kaisers_ | mdbooth: :) | 15:18 |
| mdbooth | kaisers_: Did you try stopping the nova-compute service? | 15:19 |
| *** ducttape_ has quit IRC | 15:19 | |
| *** nic has joined #openstack-nova | 15:19 | |
| kaisers_ | mdbooth: systemd produces a slice unit who becomes parent of the process | 15:19 |
| kaisers_ | hmm | 15:19 |
| kaisers_ | mdbooth: i can revisit that | 15:19 |
| *** READ10 has joined #openstack-nova | 15:19 | |
| kaisers_ | mdbooth: but that should behave similarily | 15:19 |
| mdbooth | kaisers_: Having read it again, I suspect it should work | 15:20 |
| kaisers_ | mdbooth: as the bug comes up as soon as the cgroup is closed | 15:20 |
| mdbooth | And frankly this is in your driver, so I'd personally be inclined to take your word for it as a functional test would be logistically difficult | 15:20 |
| kashyap | cfriesen: Are you hitting this? -- "Converting image from qcow2 to raw is slow for images with large virtual size" -- https://bugs.launchpad.net/nova/+bug/1484992 | 15:20 |
| openstack | Launchpad bug 1484992 in OpenStack Compute (nova) "Converting image from qcow2 to raw is slow for images with large virtual size" [Wishlist,Confirmed] | 15:20 |
| *** zz_dimtruck is now known as dimtruck | 15:20 | |
| kaisers_ | mdbooth: ack | 15:20 |
| mdbooth | kaisers_: However, given the bug you're trying to fix, if it were me I'd want to test this | 15:20 |
| kaisers_ | yep, i'll do that | 15:21 |
| *** yamamoto has quit IRC | 15:21 | |
| *** armax has joined #openstack-nova | 15:22 | |
| *** hongbin has joined #openstack-nova | 15:23 | |
| sfinucan | leakypipes: Done | 15:24 |
| leakypipes | sfinucan: thanks! | 15:24 |
| leakypipes | sfinucan: or however you say thanks in Irish :P | 15:24 |
| * leakypipes gets sfinucan a Guiness. | 15:24 | |
| sfinucan | leakypipes: Go raibh maith agat ;) | 15:25 |
| leakypipes | ah, well, that too! :) | 15:25 |
| *** yamamoto has joined #openstack-nova | 15:25 | |
| sfinucan | leakypipes: This is one to stick in your review queue for today :) https://review.openstack.org/#/c/361140/ | 15:25 |
| *** trinaths has joined #openstack-nova | 15:25 | |
| sfinucan | ...or for next weeks spec review drive | 15:26 |
| *** adrian_otto has joined #openstack-nova | 15:26 | |
| cfriesen | kashyap: yep. and in our case it's made worse by the fact that we do the conversion with ionice and no output caching (necessary to minimize impact of the conversion IO load on the instances using the same disk) | 15:26 |
| kashyap | cfriesen: I'm looking at this function _disk_qcow2_to_raw() | 15:26 |
| kashyap | And it's not being used anywhere from my current `grep`ing | 15:26 |
| kashyap | It was introduced as a helper to be used in nova/virt/libvirt -> finish_migration() | 15:27 |
| cfriesen | kashyap: in our case we're hitting it when downloading the backing image file during a block live migration for a qcow2-based instance | 15:27 |
| *** jjung has quit IRC | 15:28 | |
| kashyap | cfriesen: Do you have a full test case? | 15:28 |
| *** jaosorior has quit IRC | 15:28 | |
| *** trinaths1 has joined #openstack-nova | 15:29 | |
| cfriesen | kashyap: boot instance with qcow2 storage, live-migrate it to a compute node that doesn't have the image cached yet. | 15:29 |
| kashyap | cfriesen: I _think_ I know why we do the conversion: | 15:29 |
| mriedem | dansmith: weird the logs are not found for https://review.openstack.org/#/c/452006/ | 15:30 |
| mriedem | on the failed jobs | 15:30 |
| *** nic has quit IRC | 15:30 | |
| mriedem | anyway i left some comments | 15:30 |
| kashyap | There's an epic comment from mdbooth in the function finish_migration() | 15:30 |
| *** trinaths has quit IRC | 15:30 | |
| kashyap | cfriesen: It's to avoid CVEs from format probing | 15:30 |
| mriedem | dansmith: i'm be wary of using the IntegratedHelperMixin | 15:30 |
| dansmith | mriedem: known issue apparently | 15:30 |
| leakypipes | sfinucan: yeah, will try to get to that once I complete a review of jianghuaw's VGPU stuff. | 15:30 |
| mriedem | and I'd use USES_DB_SELF = True with the NoDbTestCase i think so you can setup the db yourself | 15:30 |
| dansmith | mriedem: I was copying the server groups test | 15:30 |
| mriedem | avoiding default cells or something | 15:30 |
| kashyap | cfriesen: Hmm, but wait -- format probing is not the only rationale, because to avoid that, the solution is to explicitly specify the format of the backing file | 15:31 |
| *** lucasxu has quit IRC | 15:31 | |
| leakypipes | sfinucan: plus, I'm technically snikitin_'s boss so if I don't get to that review today, he can bitch at me all he wants to over the weekend or Monday ;P | 15:31 |
| mriedem | dansmith: fwiw this is how i've been doing them lately https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1671648.py | 15:31 |
| mriedem | completely self-contained | 15:31 |
| mdbooth | kashyap: IIRC that code is only applicable when the source and destination hosts have different storage configurations | 15:31 |
| kashyap | cfriesen: Can you reproduce it with Git Master? | 15:31 |
| sfinucan | leakypipes: :D | 15:32 |
| mdbooth | As you say, it stops a security bug in that case | 15:32 |
| mdbooth | We really shouldn't be doing it, thoug | 15:32 |
| * kashyap nods | 15:32 | |
| dansmith | mriedem: ack | 15:32 |
| mdbooth | h | 15:32 |
| cfriesen | kashyap: I think that's different. in our case the conversion is done under pre_live_migration(), wheich eventually calls down to libvirt.utils.fetch_image(), which calls fetch_to_raw() | 15:32 |
| cfriesen | kashyap: let me check master | 15:32 |
| kashyap | mdbooth: But cfriesen seems to imply that we unconditionally convert any raw to qcow2 -- do we do that at all? | 15:32 |
| mdbooth | cfriesen: What's the issue? | 15:32 |
| kashyap | cfriesen: Oh right, fetch_to_raw(), it's from utils.py | 15:32 |
| mdbooth | cfriesen: Are you getting different backing file formats on source and destination? | 15:32 |
| kashyap | mdbooth: See the bug in the scrollback | 15:33 |
| kashyap | mdbooth: https://bugs.launchpad.net/nova/+bug/1484992 | 15:33 |
| openstack | Launchpad bug 1484992 in OpenStack Compute (nova) "Converting image from qcow2 to raw is slow for images with large virtual size" [Wishlist,Confirmed] | 15:33 |
| kashyap | Chris is hitting the above | 15:33 |
| cfriesen | mdbooth: no, we're hitting a performance issue when converting a downloaded qcow2 image to raw for use as a backing file, as part of pre_live_migration() | 15:33 |
| mdbooth | cfriesen: Ah | 15:33 |
| mdbooth | So, you don't actually have to convert those | 15:33 |
| mdbooth | That's a config option | 15:34 |
| mdbooth | However, it's an ongoing issue that changing storage config after deployment is a minefield | 15:34 |
| kashyap | Ah, right, I thought I saw one | 15:34 |
| *** suresh12 has quit IRC | 15:34 | |
| cfriesen | mdbooth: force_raw_images ? | 15:35 |
| kashyap | 'force_raw_images = True' | 15:35 |
| mdbooth | cfriesen: Sounds about right, yeah | 15:35 |
| kashyap | Yep, that seems to be it - (BoolOpt) Force backing images to raw format | 15:35 |
| *** wagnqwsh has joined #openstack-nova | 15:35 | |
| mdbooth | cfriesen: You're using LVM? | 15:35 |
| cfriesen | mdbooth: no, qcow2 in this case. | 15:35 |
| cfriesen | mdbooth: lvm in other cases though | 15:35 |
| mdbooth | Ok, because I didn't think live migration worked on lvm... does it? | 15:36 |
| mdbooth | Or is that only cold migration? | 15:36 |
| * mdbooth is quite sure cold migration doesn't work | 15:36 | |
| *** lpetrut has quit IRC | 15:36 | |
| cfriesen | mdbooth: cold works in our environment, we patched it in. :) | 15:36 |
| mdbooth | Hehe, ok | 15:36 |
| mdbooth | cfriesen: Yeah, if you want to avoid the conversion step, force_raw_images = False is what you want | 15:38 |
| mdbooth | It should probably be the default, tbh | 15:38 |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Return 400 when compute host is not found https://review.openstack.org/447355 | 15:38 |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Reset instance status on invalid host https://review.openstack.org/401009 | 15:38 |
| mriedem | lyarwood: ah i realize now we don't have any unit test coverage on detach or driver_detach in nova.virt.block_device now | 15:39 |
| mriedem | you had the TODOs in test_compute_mgr | 15:39 |
| kashyap | mdbooth: There should surely be some rationale documented as to why it was chosen to be 'True' by default? | 15:39 |
| mdbooth | surely | 15:39 |
| cfriesen | mdbooth: do you know if that has any impact on guest performance? I guess it'd be another level of indirection. | 15:39 |
| mdbooth | cfriesen: You'd have to measure it. My guess would be that it wouldn't be significant, though, over what you've already got. | 15:40 |
| kashyap | cfriesen: If it's a qcow2, "it depends", isn't it -- whether you've fully `fallocate`d it (at that point, it is equivalent to raw), or you've only preallocated the qcow2 metadata, etc | 15:40 |
| *** chyka has joined #openstack-nova | 15:41 | |
| *** sridharg has quit IRC | 15:42 | |
| *** artom has quit IRC | 15:42 | |
| cfriesen | mdbooth: hashyap: looks like it was defaulted to False back in "essex", because that was the then-current behaviour. Was added by Vishvananda Ishaya. | 15:42 |
| *** artom has joined #openstack-nova | 15:42 | |
| mdbooth | kashyap: All allocation delay would be on the top layer, regardless of the backing file format. My guess is that for read-only a qcow2 backing file would be very low overhead. | 15:43 |
| lyarwood | mriedem: right, I can follow up with that early next week and move things across I just didn't want the original change to grow out of control tbh | 15:43 |
| kashyap | mdbooth: You are correct. Because the backing file, by definition is read-only | 15:43 |
| kashyap | (Although upstream QEMU seems to be mulling over 'writable' backing files at one point.) | 15:44 |
| mdbooth | kashyap: Multi-writer qcow2 would be awesome, btw | 15:44 |
| *** jjung has joined #openstack-nova | 15:45 | |
| mdbooth | qemu has a copy-on-read feature, don't know if you've seen that. You can use it to cache a remote disk image locally as it's read from a remote source. | 15:45 |
| mdbooth | So, e.g. you can boot using the curl backend directly from an iso hosted on a remote webserver | 15:45 |
| *** kaisers_ has quit IRC | 15:46 | |
| kashyap | mdbooth: Yes, I've seen COR | 15:46 |
| mdbooth | You create a local qcow2 with COR and a remote backing file | 15:46 |
| mdbooth | And you can boot it immediately, without waiting for it to finish downloading at all | 15:46 |
| *** pcaruana has quit IRC | 15:46 | |
| *** kaisers_ has joined #openstack-nova | 15:46 | |
| mdbooth | This would be fantastic for this case. Problem is that because the backing file is in the image cache, it has multiple writers | 15:47 |
| mdbooth | And we can't support that :( | 15:47 |
| kashyap | mdbooth: Oh right, probably you should write up an RFE bug on LP before this goes into the void of scrollback | 15:47 |
| *** yamamoto has quit IRC | 15:47 | |
| cfriesen | mdbooth: kashyap: thanks guys, I've got options to try out now. :) | 15:47 |
| *** artom has quit IRC | 15:48 | |
| *** artom has joined #openstack-nova | 15:48 | |
| mdbooth | cfriesen: Sadly, I don't think this change will cause existing downloads to break | 15:48 |
| *** yamamoto has joined #openstack-nova | 15:48 | |
| mdbooth | I say sadly, because we do format inspection on backing files :( | 15:48 |
| dansmith | mriedem: you use cast-as-call in your test, for a reason I assume.. do you know why I'm getting NotFound and need that poll loop? | 15:49 |
| kashyap | mdbooth: BTW, do you really mean two users trying to simultaneously to a qcow2 file? | 15:49 |
| mdbooth | I don't know of any current security issues relating to this, but it's still a danger | 15:49 |
| *** armax has quit IRC | 15:49 | |
| kashyap | s/users/processes/ | 15:49 |
| *** armax has joined #openstack-nova | 15:50 | |
| *** yamamoto has quit IRC | 15:50 | |
| *** yamamoto has joined #openstack-nova | 15:50 | |
| mdbooth | kashyap: So, you create a qcow2 file with a remote backing file and COR, lets call this cache.qcow | 15:50 |
| *** yamamoto has quit IRC | 15:50 | |
| mdbooth | kashyap: You then create instance_a.qcow2 which has cache.qcow2 as a backing file | 15:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Address comments when moving volume detach to block_device.py https://review.openstack.org/452252 | 15:51 |
| mriedem | lyarwood: ^ | 15:51 |
| mriedem | dansmith: i used cast as call because that's what the test did that i copied from :) | 15:51 |
| dansmith | heh, yeah okay | 15:51 |
| mriedem | dansmith: as for the 404 while it's building... | 15:51 |
| mdbooth | When you read a block from instance_a.qcow2, it'll be looked up in cache.qcow2, miss, be downloaded, and stored in cache.qcow before being returned to instance_a.qcow2 | 15:51 |
| *** kaisers_ has quit IRC | 15:51 | |
| mdbooth | So basically, the qemu process writing to instance_a.qcow2 is also writing to cache.qcow2 | 15:51 |
| mriedem | dansmith: could be hitting some window between when the build request and instance exist for the get() method in the compute API? | 15:51 |
| mriedem | dansmith: i thought we closed that in ocata thoguh | 15:52 |
| mriedem | *though | 15:52 |
| mdbooth | Now simultaneously do the same thing with instance_b.qcow2 | 15:52 |
| *** raunak has joined #openstack-nova | 15:52 | |
| lyarwood | mriedem: ack thanks, LGTM | 15:52 |
| dansmith | mriedem: well, that's my point.. we shouldn | 15:52 |
| dansmith | mriedem: shouldn't have that problem | 15:52 |
| mdbooth | You've now got 2 separate qemu processes writing to cache.qcow2 without coordination | 15:52 |
| mdbooth | I don't believe this will end well | 15:52 |
| kashyap | mdbooth: Hmm. It's not security issues (about multiple processes trying to write to a qcow2 file live), but image being corrupted -- as we speak, there are patches using POSIX / Linux file-locking to prevent that at QEMU-level | 15:52 |
| mdbooth | kashyap: The security issue is image format detection | 15:53 |
| mdbooth | We shouldn't rely on it, but we do | 15:53 |
| kashyap | mdbooth: Oh, you were talking to Chris there, duh :-) | 15:53 |
| mdbooth | But in this case it's convenient that we do, because it won't break anything if you change force_raw | 15:53 |
| dansmith | mriedem: ah, I know why | 15:53 |
| kashyap | mdbooth: But I agree with your design of speeding up remote boots with COR | 15:53 |
| *** ralonsoh__ is now known as ralonsoh | 15:54 | |
| mriedem | dansmith: what i'd do, is temporarily hack your test change to change https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L1144 and do LOG.exception so you get the traceback | 15:54 |
| *** mlavalle has joined #openstack-nova | 15:54 | |
| dansmith | mriedem: I done already figger'd it | 15:54 |
| mriedem | do tell | 15:54 |
| mriedem | no instance mapping? | 15:54 |
| dansmith | no, because that's not a bug | 15:55 |
| *** Swami has joined #openstack-nova | 15:55 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Raise general error message when volume attach fails https://review.openstack.org/320163 | 15:55 |
| dansmith | it's service_version | 15:55 |
| dansmith | because of all the switchin' during service setup | 15:55 |
| dansmith | api thinks api is old | 15:55 |
| *** lucasxu has joined #openstack-nova | 15:55 | |
| *** trinaths1 has quit IRC | 15:55 | |
| dansmith | which is why we have fixtures.AllServicesCurrent() | 15:55 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2386 ? | 15:56 |
| *** fragatina has quit IRC | 15:56 | |
| dansmith | mriedem: yar | 15:56 |
| *** Guest16838 has quit IRC | 15:56 | |
| mriedem | is service_version 0 or soething? | 15:56 |
| mriedem | something | 15:56 |
| mriedem | this reminds me of something i've been meaning to talk to you about, | 15:56 |
| dansmith | it is in the db that is getting consulted | 15:56 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L437 | 15:57 |
| dansmith | HAH | 15:57 |
| mriedem | in the api we check the minimum compute service version for things like get-me-a-network and device tags | 15:57 |
| *** armax has quit IRC | 15:57 | |
| *** cjvolzka has joined #openstack-nova | 15:57 | |
| mriedem | that's not going to work anymore with multicell | 15:57 |
| *** cjvolzka has quit IRC | 15:57 | |
| kashyap | mdbooth: One last on that -- in your scenario of speeding up a remote boot while provisioning, it is still a single QEMU process that is handling COR, and live commit, isn't it... | 15:57 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L583 | 15:57 |
| dansmith | mriedem: well (a) we need to remove those now anyway, but (b) there's an all_cells worker to get that | 15:58 |
| mdbooth | kashyap: Not if you've got 2 instances using the same image backing file | 15:58 |
| kashyap | mdbooth: Ah, right! | 15:58 |
| *** armax has joined #openstack-nova | 15:58 | |
| mdbooth | Both before the backing file download has finished | 15:58 |
| kashyap | mdbooth: But yes, as it stands, upstream doesn't support multiple writers | 15:58 |
| mriedem | dansmith: yeah https://github.com/openstack/nova/blob/master/nova/cmd/api_os_compute.py#L43 | 15:59 |
| kashyap | Probably you want to write an RFE to qemu-{devel, block} lists :-) | 15:59 |
| mdbooth | COR == implicitly writeable backing file | 15:59 |
| mriedem | device tagging was newton too right? | 15:59 |
| mriedem | yeah, i know it wasn't ocat | 15:59 |
| mriedem | *ocata | 15:59 |
| *** armax has quit IRC | 15:59 | |
| dansmith | mriedem: well, it was fixed in ocata api, but yeah | 15:59 |
| mriedem | however | 15:59 |
| mriedem | we have something else using that now in nova.compute.api | 16:00 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1404 | 16:00 |
| mriedem | ^ was added in pike | 16:00 |
| dansmith | ack | 16:00 |
| mriedem | we could remove the checks for get me a network and device tags though probably | 16:00 |
| mriedem | i can't find your nova-status patch | 16:01 |
| mriedem | ah https://review.openstack.org/#/c/442787/ | 16:01 |
| *** ltomasbo is now known as ltomasbo|away | 16:02 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.openstack.org/452006 | 16:03 |
| *** ekuris has quit IRC | 16:04 | |
| *** ducttape_ has joined #openstack-nova | 16:05 | |
| *** ekuris has joined #openstack-nova | 16:05 | |
| *** ducttape_ has quit IRC | 16:05 | |
| *** ssurana has joined #openstack-nova | 16:06 | |
| *** gcb has quit IRC | 16:07 | |
| mriedem | dansmith: that looks much better | 16:09 |
| *** gcb has joined #openstack-nova | 16:09 | |
| *** dtp has joined #openstack-nova | 16:09 | |
| dansmith | I'm thrilled | 16:09 |
| *** Guest16838 has joined #openstack-nova | 16:11 | |
| leakypipes | _gryf: lol at Michal's April Fool's spec :) | 16:11 |
| leakypipes | dulek: :) | 16:12 |
| *** ducttape_ has joined #openstack-nova | 16:12 | |
| *** artom has quit IRC | 16:14 | |
| *** artom has joined #openstack-nova | 16:14 | |
| *** dillaman has quit IRC | 16:14 | |
| * mriedem goes to lunch | 16:15 | |
| *** artom has quit IRC | 16:16 | |
| *** artom has joined #openstack-nova | 16:16 | |
| *** jjung has quit IRC | 16:16 | |
| *** derekh has quit IRC | 16:18 | |
| *** voelzmo has joined #openstack-nova | 16:20 | |
| *** markus_z has quit IRC | 16:22 | |
| *** cdent has quit IRC | 16:25 | |
| *** voelzmo has quit IRC | 16:25 | |
| *** voelzmo has joined #openstack-nova | 16:26 | |
| *** esberglu has quit IRC | 16:27 | |
| *** sneti_ has quit IRC | 16:27 | |
| *** esberglu has joined #openstack-nova | 16:27 | |
| *** lucasagomes is now known as lucas-afk | 16:28 | |
| *** jimbaker has joined #openstack-nova | 16:29 | |
| *** nkorabli_ has quit IRC | 16:29 | |
| *** lucasxu has quit IRC | 16:29 | |
| *** karimb has quit IRC | 16:30 | |
| *** voelzmo has quit IRC | 16:31 | |
| *** satyar has quit IRC | 16:32 | |
| *** karimb has joined #openstack-nova | 16:32 | |
| *** edmondsw has quit IRC | 16:32 | |
| *** esberglu has quit IRC | 16:32 | |
| *** ekuris has quit IRC | 16:33 | |
| ildikov | mriedem: lyarwood: would the Cinder-Nova meeting work for you an hour earlier (1600 UTC) on Thursdays? | 16:33 |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in server_password.py https://review.openstack.org/452259 | 16:33 |
| *** nkorabli has joined #openstack-nova | 16:35 | |
| *** edmondsw has joined #openstack-nova | 16:35 | |
| lyarwood | ildikov: yes | 16:35 |
| *** ducttape_ has quit IRC | 16:36 | |
| *** ZZelle has joined #openstack-nova | 16:36 | |
| *** karimb has quit IRC | 16:36 | |
| ildikov | lyarwood: cool, tnx | 16:36 |
| *** cdent has joined #openstack-nova | 16:38 | |
| *** artom has left #openstack-nova | 16:39 | |
| *** edmondsw has quit IRC | 16:39 | |
| *** ducttape_ has joined #openstack-nova | 16:39 | |
| *** artom has joined #openstack-nova | 16:39 | |
| *** tuan_luong has quit IRC | 16:40 | |
| *** tuanla_ has quit IRC | 16:40 | |
| *** marst has quit IRC | 16:41 | |
| *** Apoorva has joined #openstack-nova | 16:42 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: [WIP] Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 16:43 |
| *** ralonsoh has quit IRC | 16:44 | |
| *** marst has joined #openstack-nova | 16:44 | |
| *** fragatina has joined #openstack-nova | 16:45 | |
| *** karimb has joined #openstack-nova | 16:45 | |
| *** ekuris has joined #openstack-nova | 16:48 | |
| *** ducttape_ has quit IRC | 16:50 | |
| *** yamamoto has joined #openstack-nova | 16:51 | |
| *** ekuris has quit IRC | 16:53 | |
| *** abalutoiu has quit IRC | 16:53 | |
| *** ducttape_ has joined #openstack-nova | 16:53 | |
| *** lucasxu has joined #openstack-nova | 16:54 | |
| *** lpetrut has joined #openstack-nova | 16:54 | |
| *** efried has quit IRC | 16:54 | |
| *** jpena is now known as jpena|off | 16:55 | |
| *** sneti_ has joined #openstack-nova | 16:55 | |
| *** yamamoto has quit IRC | 16:57 | |
| *** suresh12 has joined #openstack-nova | 16:59 | |
| *** gyee has joined #openstack-nova | 16:59 | |
| *** ducttape_ has quit IRC | 17:03 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in server_external_events.py https://review.openstack.org/452263 | 17:05 |
| *** yonglihe has quit IRC | 17:09 | |
| *** armax has joined #openstack-nova | 17:10 | |
| *** sambetts is now known as sambetts|afk | 17:12 | |
| *** armax has quit IRC | 17:16 | |
| *** MVenesio has joined #openstack-nova | 17:18 | |
| *** yamahata has joined #openstack-nova | 17:19 | |
| *** browne has joined #openstack-nova | 17:21 | |
| *** mnestratov has quit IRC | 17:24 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in server_diagnostics.py https://review.openstack.org/452267 | 17:26 |
| JayF | mriedem: https://review.openstack.org/#/c/416487/ mind dropping your procedural -2 from last cycle? | 17:26 |
| JayF | mriedem: we're tying all the patches together now to get it all tested up in tempest and ready to merge (in ironic, then nova) | 17:26 |
| *** awaugama has quit IRC | 17:28 | |
| *** lpetrut has quit IRC | 17:31 | |
| *** lpetrut has joined #openstack-nova | 17:32 | |
| *** swamireddy has quit IRC | 17:36 | |
| *** vks1 has quit IRC | 17:36 | |
| *** mdrabe has quit IRC | 17:38 | |
| jroll | oh we should approve that blueprint | 17:49 |
| *** gszasz has quit IRC | 17:51 | |
| *** armax has joined #openstack-nova | 17:51 | |
| openstackgerrit | Mario Villaplana proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 17:51 |
| JayF | mriedem: \o/ thanks | 17:52 |
| mriedem | yw | 17:53 |
| mriedem | ildikov: works for me | 17:53 |
| sdague | mriedem: so... clarkb has an interesting though on the libvirt malloc crashes, which is that we start doing UCA by default | 17:55 |
| sdague | which would mean jumping up to libvirt 2.5 in master | 17:55 |
| sdague | it has a lot of other implications, he's going to write the whole thing up to share on the ML | 17:55 |
| mriedem | ok | 17:55 |
| sdague | but, I wanted to get that seed out there for everyone to ponder | 17:55 |
| sdague | it would mean we'd start testing some of the pathways that aren't otherwise tested | 17:56 |
| mriedem | well, some of those are things like numa/pci right? | 17:56 |
| *** suresh12 has quit IRC | 17:56 | |
| mriedem | which i don't think we'd be testing regardless | 17:56 |
| clarkb | I think there are newer live migration things too | 17:57 |
| clarkb | at least my tests tripped over that with bad wheel builds against old libvirt | 17:57 |
| *** kaisers_ has joined #openstack-nova | 17:57 | |
| sdague | yeh, postcopy | 17:58 |
| *** unicell has joined #openstack-nova | 17:58 | |
| sdague | and virtlogd | 17:58 |
| mriedem | what are we right now? 1.3.1? | 17:58 |
| sdague | yep | 17:58 |
| sdague | and perf_events? | 17:59 |
| mriedem | yeah, so the only thing of those that would get tested by tempest are probably postcopy | 17:59 |
| mriedem | unless virtlogd ties in with any console-related tests | 17:59 |
| sdague | that seems like everything with higher versions bring in | 17:59 |
| *** Guest16838 has quit IRC | 17:59 | |
| *** baoli has quit IRC | 18:00 | |
| mriedem | yeah the perf events stuff won't be tested by tempest directly | 18:00 |
| sdague | yeh, I think there might be console tests there | 18:00 |
| sdague | anyway, a thing to ponder, details comming. | 18:00 |
| *** baoli has joined #openstack-nova | 18:01 | |
| mriedem | https://www.youtube.com/watch?v=09kuOAr_wkc | 18:02 |
| mriedem | i'll ponder it on the tree of woe | 18:02 |
| *** kaisers_ has quit IRC | 18:02 | |
| *** suresh12 has joined #openstack-nova | 18:02 | |
| openstackgerrit | David Moreau Simard proposed openstack/nova master: Be honest with users when no valid host was found https://review.openstack.org/452279 | 18:02 |
| *** Apoorva_ has joined #openstack-nova | 18:03 | |
| sdague | heh | 18:03 |
| *** Apoorva has quit IRC | 18:05 | |
| *** baoli has quit IRC | 18:10 | |
| leakypipes | sdague, sfinucan: another easy one: https://review.openstack.org/#/c/447401/5 | 18:11 |
| *** baoli has joined #openstack-nova | 18:11 | |
| sdague | bam | 18:11 |
| leakypipes | heh | 18:11 |
| *** lucasxu has quit IRC | 18:12 | |
| *** sneti_ has quit IRC | 18:12 | |
| *** baoli has quit IRC | 18:12 | |
| *** baoli has joined #openstack-nova | 18:13 | |
| *** abalutoiu has joined #openstack-nova | 18:13 | |
| *** baoli has quit IRC | 18:19 | |
| mriedem | i need another specs core on https://review.openstack.org/#/c/441692/ | 18:20 |
| mriedem | i think i addressed all of johnthetubaguy's comments | 18:20 |
| mriedem | i'm not sure if there is something i need to update in there, so i haven't | 18:20 |
| openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/357151 | 18:21 |
| *** tesseract has quit IRC | 18:21 | |
| *** sneti_ has joined #openstack-nova | 18:21 | |
| *** nic has joined #openstack-nova | 18:23 | |
| *** fragatina has quit IRC | 18:24 | |
| sdague | mriedem: have you tried to POC any of that yet? | 18:24 |
| mriedem | sdague: no | 18:24 |
| mriedem | i started by just trying to get devstack setup with searchlight in a ci job | 18:24 |
| mriedem | https://review.openstack.org/#/c/441696/ | 18:25 |
| sdague | how did that go? | 18:25 |
| mriedem | not well | 18:25 |
| mriedem | i didn't spend much time on it | 18:25 |
| mriedem | i was hoping the searchlight devstack plugin would configure nova, but it doesn't | 18:25 |
| mriedem | and now the logs are gone so i don't remember what was failing, i'll recheck it | 18:26 |
| mriedem | also, searchlight doesn't have a dsvm job, so they didn't have something i could copy really | 18:26 |
| sdague | so, it feels like one of those things were could define a direction here as we're going to try, but there should be a bit more POC before we commit (and especially punt on doing any of this in nova) | 18:26 |
| sdague | mriedem: ah, bummer | 18:26 |
| mriedem | sdague: the spec is about the general plan | 18:26 |
| mriedem | just like the original resource providers spec was about the general plan, and changed a lot over time | 18:27 |
| sdague | ok | 18:27 |
| mriedem | i also have http://forumtopics.openstack.org/cfp/details/31 proposed | 18:28 |
| mriedem | i agree poc would be ideal | 18:28 |
| mriedem | chances are this won't happen in nova in pike anyway, since there are dependencies on changes in searchlight first | 18:29 |
| mriedem | which don't have owners | 18:29 |
| mriedem | but it's The Plan (tm) | 18:29 |
| *** thomasem has joined #openstack-nova | 18:30 | |
| *** cdent has quit IRC | 18:30 | |
| sdague | yeh, I guess I'd just soften it further from the recommended mode, given we have no idea if it's going to work. Just say, we think this is going to be a better search approach, and we want to implement an optional backend to get it into the field. | 18:32 |
| sdague | I don't think this is nearly as straight forward an aggressive cut over as placement | 18:32 |
| *** cdent has joined #openstack-nova | 18:32 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: WIP: Add the specified microversion into error msg https://review.openstack.org/451986 | 18:32 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove unnecessary duplicated NOTE https://review.openstack.org/452284 | 18:32 |
| mriedem | i wouldn't say integrating placement has been straightforward | 18:33 |
| *** sneti_ has quit IRC | 18:36 | |
| *** swamireddy has joined #openstack-nova | 18:38 | |
| *** crushil has joined #openstack-nova | 18:39 | |
| *** sridharg has joined #openstack-nova | 18:39 | |
| *** sridharg has quit IRC | 18:40 | |
| *** baoli has joined #openstack-nova | 18:40 | |
| mriedem | melwitt: so, i hate to talk about this, but before i fix the bug that's shown in https://review.openstack.org/#/c/449334/ - i'm thinking about what you said where were are committing the quota usage decrement in the api before we delete the instance, and on failure we try to rollback, but that doesn't work | 18:40 |
| *** baoli has quit IRC | 18:40 | |
| *** baoli has joined #openstack-nova | 18:40 | |
| mriedem | i noticed something funny in the quota object | 18:40 |
| melwitt | okay | 18:41 |
| mriedem | on commit and rollback, if the reservation is not set, it's a noop | 18:41 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/objects/quotas.py#L100 | 18:41 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/objects/quotas.py#L111 | 18:41 |
| mriedem | i was wondering why we didn't fail | 18:41 |
| mriedem | but those are really invalid operations, if you're trying to commit or rollback reservations before you made the reservation, it's a bug | 18:41 |
| mriedem | right? | 18:41 |
| melwitt | fwiw, I think it's a no-op at the db/api level too, so I don't think it matters | 18:41 |
| mriedem | i'm not sure that's a good reason though | 18:42 |
| *** aunnam_ has quit IRC | 18:42 | |
| melwitt | sure | 18:42 |
| mriedem | but yeah, in the db api methods, if they don't find a reservation, they don't do anything | 18:42 |
| mriedem | i was wondering why we didn't get a NotFound error on delete | 18:42 |
| melwitt | yeah, it's not valid to do a commit/rollback without a reserve first | 18:42 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L3934 | 18:42 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L3922 | 18:43 |
| melwitt | I'm not sure what the reasoning is to do a no-op, if there's some case where that would be a thing | 18:43 |
| mriedem | reservations are just uuids right? | 18:43 |
| melwitt | reservations are records in the reservations tables | 18:43 |
| melwitt | *table | 18:43 |
| mriedem | with a uuid | 18:43 |
| mriedem | yeah | 18:43 |
| mriedem | filter(models.Reservation.uuid.in_(reservations)).\ | 18:43 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L3866 | 18:44 |
| mriedem | is on reserve | 18:44 |
| mriedem | we get the list of uuids back | 18:44 |
| mriedem | and store that in the Quota object | 18:44 |
| artom | Can an instance have multiple fixed IPs? | 18:44 |
| mriedem | so maybe i'll tinker with making commit/rollback w/o a reservation in the object a failure | 18:44 |
| melwitt | artom: I think so, if it's on multiple networks | 18:44 |
| mriedem | and see what falls out | 18:44 |
| artom | melwitt, thanks :) | 18:44 |
| mriedem | artom: yeah definitely | 18:44 |
| clarkb | mriedem: artom also ipv4 + ipv6 | 18:44 |
| mriedem | multinic | 18:44 |
| clarkb | er melwitt artom ^ | 18:45 |
| mriedem | and ipv8 | 18:45 |
| mriedem | it's secret | 18:45 |
| mriedem | and new | 18:45 |
| mriedem | full of tomatoe | 18:45 |
| artom | Are we running out of quarks to give IPv6s to? | 18:45 |
| artom | IoSAP? | 18:45 |
| artom | Internet of SubAtomic Particles... | 18:45 |
| artom | Eesh, one bad joke and everyone shuts up? | 18:47 |
| * mriedem changes shirts from the vomit | 18:47 | |
| artom | Wear a bib next time ;) | 18:47 |
| mriedem | you'd have to warn me | 18:48 |
| mriedem | unless i start wearing one full time | 18:48 |
| melwitt | mriedem: yeah, might be worth it to do an experiment. I can't imagine why no-op would be valid other than some sort of race where commit could possibly hit before the reserve and that leaving an orphan reservation is okay | 18:48 |
| melwitt | I can't think of such a race, but I have heard of having a lot of orphaned reservations in the db. so that implies it happens somehow | 18:49 |
| mriedem | maybe comstud is lurking | 18:49 |
| *** suresh12 has quit IRC | 18:50 | |
| mriedem | interesting https://github.com/openstack/nova/blob/master/nova/quota.py#L1401 | 18:51 |
| mriedem | same with rollback https://github.com/openstack/nova/blob/master/nova/quota.py#L1425 | 18:52 |
| mriedem | we just ignore exceptions | 18:52 |
| mriedem | any exceptions | 18:52 |
| mriedem | no matter how much Vek says that's safe, it seems unsafe | 18:52 |
| melwitt | heh | 18:52 |
| mriedem | " because the usage resynchronization will resolve the issue" | 18:52 |
| mriedem | we don't re-sync by default do we? | 18:52 |
| melwitt | we do. on every reserve | 18:53 |
| melwitt | it will do a sync for that project/user combo | 18:53 |
| melwitt | it doesn't help if some other user in your project is out of sync though | 18:53 |
| *** amoralej is now known as amoralej|off | 18:54 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/conf/quota.py#L278 | 18:54 |
| mriedem | i was thinking of ^ | 18:54 |
| melwitt | also a reminder that quota counting will remove reservations and usages | 18:54 |
| melwitt | oh, hm | 18:54 |
| mriedem | and https://github.com/openstack/nova/blob/master/nova/conf/quota.py#L267 | 18:54 |
| mriedem | so i think by default, we don't refresh any of that stuff | 18:55 |
| mriedem | or resync | 18:55 |
| mriedem | so maybe the comment from Vek was more about, it will fix it, because we at rackspace turn those on... | 18:55 |
| melwitt | yeah, okay. I didn't realize that defaulted to off | 18:55 |
| melwitt | if it's on, then on every reserve it does a refresh if needed | 18:55 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L3647 | 18:55 |
| mriedem | yeah if needed | 18:55 |
| mriedem | based on age and until_refresh count | 18:56 |
| melwitt | I think most operators turn it on, since going out of sync happens a fair bit | 18:56 |
| mriedem | so if blindly handling all exceptions https://github.com/openstack/nova/blob/master/nova/quota.py#L1425 and assuming we're ok is dependent on those config options, which default to not doing the thing that comment says we do automatically, | 18:56 |
| mriedem | it seems we should reraise the exception if those options aren't set | 18:56 |
| mriedem | at least one of them | 18:56 |
| mriedem | otherwise we're just silently failing | 18:57 |
| melwitt | I guess I mean if usage != resource count if refreshes. not that that saves a lot I guess | 18:57 |
| melwitt | maybe. it's going to be moot soon, we hope | 18:57 |
| *** baoli has quit IRC | 18:59 | |
| *** dtp has quit IRC | 18:59 | |
| *** kaisers_ has joined #openstack-nova | 18:59 | |
| *** lucasxu has joined #openstack-nova | 19:00 | |
| *** baoli has joined #openstack-nova | 19:00 | |
| *** baoli has quit IRC | 19:00 | |
| mriedem | what kind of state is that series in now? | 19:01 |
| *** lucasxu has quit IRC | 19:02 | |
| *** suresh12 has joined #openstack-nova | 19:02 | |
| melwitt | I'm working on it and plan to push the series up today and see what tempest thinks of it | 19:03 |
| melwitt | there is light at the end of the tunnel finally | 19:04 |
| *** baoli has joined #openstack-nova | 19:05 | |
| *** HW-Peter has quit IRC | 19:06 | |
| *** marst has quit IRC | 19:07 | |
| *** MVenesio has quit IRC | 19:10 | |
| mriedem | ok so by default, reservations expire after 24 hours, | 19:12 |
| mriedem | and there is a periodic task in the scheduler manager that triggers the expire routine | 19:12 |
| openstackgerrit | dane-fichter proposed openstack/nova-specs master: Trusted flavors https://review.openstack.org/312225 | 19:12 |
| *** cleong has quit IRC | 19:12 | |
| *** gcb has quit IRC | 19:12 | |
| *** gcb has joined #openstack-nova | 19:13 | |
| JayF | Is there a standard username expected for login when an instance is rescued via `nova rescue`? | 19:15 |
| JayF | There is not one documented in the client docs, afaict | 19:15 |
| *** cdent has quit IRC | 19:17 | |
| mriedem | https://espaces.edu.au/vwrangler/nectar-topics/nectar-how-tos/using-nova-rescue-to-repair-a-nectar-instance | 19:21 |
| JayF | that one says "admin" | 19:21 |
| mriedem | i'm not totally sure, but i think the username depends on the image used | 19:21 |
| JayF | Rackspace uses root | 19:21 |
| JayF | Yeah, that's what I was afraid of | 19:21 |
| JayF | since ironic provides the image we have to just decide | 19:21 |
| *** ducttape_ has joined #openstack-nova | 19:22 | |
| clarkb | fwiw if it were me, I would expect root | 19:22 |
| JayF | We have it written today for "rescue@" | 19:22 |
| JayF | I think root is more reasonable... but it's unfortunate that it's not dictated | 19:23 |
| JayF | someone, just tell me what to do :P | 19:23 |
| *** sdague_ has joined #openstack-nova | 19:23 | |
| mriedem | dictated by the user making the rescue request? | 19:24 |
| JayF | dictated by a document, or something like that | 19:24 |
| mriedem | because the admin on a windows rescue image is going to be Administrator, right? | 19:24 |
| JayF | I just want to be told what right is, because there being no "right answer" means I have to document the hell out of it | 19:24 |
| JayF | we don't support arbitrary rescue images in ironic's implementatino | 19:24 |
| JayF | not really possible since we're implementing rescue as booting a ramdisk and configuring it | 19:24 |
| clarkb | also you should be able to rescue windows with linux but the other direction is much harder | 19:25 |
| clarkb | (there are even distros out there purpose built for this) | 19:25 |
| mriedem | JayF: what are you going to do in the nova driver impl then? ignore the image ref passed in, or put the instance into error state? | 19:25 |
| JayF | mriedem: ignore the one passed in, I think is how it's written | 19:25 |
| JayF | mariojv: ^ you just looked at that patch, do you know? | 19:25 |
| sdague_ | mriedem: ok, going through https://review.openstack.org/#/c/441692 with a deeper comb. While I realize that it probably isn't going to get done in Pike anyway, it does feel better on an expectation front to say that Pike is about getting an optional facility there for people to play around with instead of jumping all the way to recommended operating mode | 19:25 |
| *** baoli has quit IRC | 19:26 | |
| JayF | mriedem: I think we're happy doing whatever you think is right in that case, but imo ignoring the image request seems the sanest. | 19:26 |
| mriedem | sdague_: isn't that what i said in there? i said a config option, disabled by default | 19:26 |
| mriedem | JayF: that's not honoring the API... | 19:26 |
| JayF | so you think we should put instance into error and raise an instance fault/ | 19:26 |
| mriedem | so i'm not sure if it's better to fault and say you can't do that, or just ignore it | 19:26 |
| JayF | ? | 19:26 |
| *** baoli has joined #openstack-nova | 19:26 | |
| JayF | I mean, technically, it's borderline impossible for us to respect the api with bare metal | 19:27 |
| mriedem | putting the instance into error state is maybe something we shouldn't do | 19:27 |
| mriedem | so it's a question of ignore, or fault | 19:27 |
| sdague_ | https://review.openstack.org/#/c/441692/3/specs/pike/approved/list-instances-using-searchlight.rst@15 ... definitely leans in a different direction | 19:27 |
| *** jwcroppe has quit IRC | 19:27 | |
| mriedem | https://review.openstack.org/#/c/441692/3/specs/pike/approved/list-instances-using-searchlight.rst@48 | 19:28 |
| mriedem | "By default this path will be disabled." | 19:28 |
| JayF | mriedem: I'll put a note on the nova patch for us to think about that case more explicitly -- right now we're trying to get the patches lined up to prove rescue working in tempest | 19:28 |
| *** jwcroppe has joined #openstack-nova | 19:28 | |
| mriedem | JayF: fwiw, when you do boot from volume or attach a volume and provide a device name in the api, the libvirt driver totally ignores that device name and has since liberty | 19:29 |
| mriedem | so we have some precedence for drivers ignoring the api | 19:29 |
| clarkb | JayF: couldn't you publish a rescue image in glance that would be used if specified, and its up to use to figure it out if they specify something different? | 19:29 |
| JayF | mriedem: prior are for ignoring the user, I'll take it :P | 19:29 |
| JayF | clarkb: we're treating it similar to a provisioning ramdisk, configurable by operator/deployer only | 19:29 |
| JayF | clarkb: and I would have to think hard about the security implications of letting an api end-user dictate an image to pxe boot | 19:30 |
| JayF | clarkb: plus it's sorta ... setting up a user for failure. I'd rather ignore their image request than provide support for it, but have building an image that works be borderline impossible without special knowledge. | 19:30 |
| clarkb | JayF: as a user of your APIs I really disagree on that last point | 19:31 |
| clarkb | I'm really tired of clouds making bad decisions for me and forcing me to abide by them when there is no reason to | 19:31 |
| *** ducttape_ has quit IRC | 19:31 | |
| *** baoli has quit IRC | 19:31 | |
| clarkb | image upload is/was that same general problem. So what did people do they ran their own pxe servers and dd'd over ubuntu images with image that then booted pxe | 19:31 |
| clarkb | (it may make sense here but as a general rule I'm not a fan of it) | 19:32 |
| JayF | There /is/ a reason for it here: rescue is not some arbitrary image booted on a hypervisor, it's a ramdisk, that has to have proper hardware support, as well as a properly configured ironic-python-agent. As someone who has operated ironic clouds, I'm not sure I'd want any of my api users being able to pxe boot an arbitrary ramdisk on hardware | 19:32 |
| clarkb | JayF: also fwiw I think your users likely can already pxe boot arbitrary ramdisks on hardware | 19:33 |
| clarkb | (its a really common workaround for the I can't upload my own images problem) | 19:33 |
| JayF | clarkb: not in an ironic configured for localboot only | 19:34 |
| JayF | clarkb: they can localboot anything they want, but not pxe boot it directly from ironic's setup | 19:34 |
| clarkb | JayF: you local boot the image that speaks pxe | 19:34 |
| JayF | oh, I forgot the #1 reason why that likely shouldn't be allowed: | 19:34 |
| JayF | we pxe boot the rescue ramdisk from an internal-use-only rescue network | 19:35 |
| JayF | to flip it over to tenant networks before the rescue is complete | 19:35 |
| JayF | so allowing arbitrary ramdisks to boot in rescue would be enabling a tenant to run arbitrary code inside your rescue network | 19:35 |
| JayF | which would be a major security concern | 19:35 |
| *** karimb has quit IRC | 19:36 | |
| *** kylek3h has quit IRC | 19:38 | |
| *** abalutoiu has quit IRC | 19:41 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure reservation_expire actually expires reservations https://review.openstack.org/452300 | 19:45 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure reservation_expire actually expires reservations https://review.openstack.org/452300 | 19:50 |
| *** kaisers_ has quit IRC | 19:50 | |
| *** sdague_ has quit IRC | 19:51 | |
| *** sdague_ has joined #openstack-nova | 19:56 | |
| -openstackstatus- NOTICE: lists.openstack.org will be offline from 20:00 to 23:00 UTC for planned upgrade maintenance | 19:58 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in security_groups.py https://review.openstack.org/452309 | 20:01 |
| *** liangy has quit IRC | 20:02 | |
| *** liangy has joined #openstack-nova | 20:04 | |
| *** ducttape_ has joined #openstack-nova | 20:11 | |
| *** Qiming has quit IRC | 20:12 | |
| *** tbachman_ has joined #openstack-nova | 20:14 | |
| *** tbachman has quit IRC | 20:15 | |
| *** tbachman_ is now known as tbachman | 20:15 | |
| leakypipes | dansmith: is there such a thing as a cells.json file? | 20:15 |
| dansmith | leakypipes: um, what? | 20:16 |
| leakypipes | can you pop into #openstack-helm right quick? | 20:16 |
| leakypipes | dansmith: ^^ | 20:16 |
| leakypipes | as superdan :) | 20:16 |
| *** Qiming has joined #openstack-nova | 20:16 | |
| *** suresh12 has quit IRC | 20:20 | |
| *** suresh12_ has joined #openstack-nova | 20:20 | |
| *** dtp has joined #openstack-nova | 20:21 | |
| *** gouthamr has quit IRC | 20:22 | |
| *** MikeG451 has quit IRC | 20:22 | |
| *** liangy has quit IRC | 20:23 | |
| *** liangy has joined #openstack-nova | 20:28 | |
| *** READ10 has quit IRC | 20:34 | |
| *** lyan_ has joined #openstack-nova | 20:35 | |
| *** marst has joined #openstack-nova | 20:36 | |
| *** sdague has quit IRC | 20:36 | |
| *** liangy has quit IRC | 20:37 | |
| *** sdague_ has quit IRC | 20:41 | |
| *** Apoorva_ has quit IRC | 20:43 | |
| *** Apoorva has joined #openstack-nova | 20:44 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Reraise reservation commit/rollback errors if not self-healing https://review.openstack.org/452319 | 20:44 |
| *** crushil has quit IRC | 20:45 | |
| *** eharney has quit IRC | 20:45 | |
| *** gouthamr has joined #openstack-nova | 20:46 | |
| *** bnemec is now known as beekneemech | 20:49 | |
| mriedem | wow there was a whole lot of nothing going on in there | 20:51 |
| *** MikeG451 has joined #openstack-nova | 20:54 | |
| *** Qiming has quit IRC | 21:01 | |
| *** Qiming has joined #openstack-nova | 21:02 | |
| openstackgerrit | Scott Moser proposed openstack/nova master: Limit exposure of network device types to the guest. https://review.openstack.org/400883 | 21:04 |
| *** fragatina has joined #openstack-nova | 21:05 | |
| *** kaisers has quit IRC | 21:07 | |
| *** fragatin_ has joined #openstack-nova | 21:08 | |
| *** ijw has quit IRC | 21:10 | |
| *** fragatin_ has quit IRC | 21:10 | |
| *** fragatin_ has joined #openstack-nova | 21:11 | |
| *** fragatina has quit IRC | 21:12 | |
| *** dimtruck is now known as zz_dimtruck | 21:13 | |
| *** xinliang has quit IRC | 21:13 | |
| *** pramodrj07 has joined #openstack-nova | 21:15 | |
| pramodrj07 | Hey All, | 21:15 |
| *** sdague has joined #openstack-nova | 21:16 | |
| *** mvk has quit IRC | 21:18 | |
| pramodrj07 | Can anyone help me in understanding the differnece between security groups in nova and sg in neutron? | 21:21 |
| *** aysyd has quit IRC | 21:22 | |
| mriedem | pramodrj07: security groups in nova are tied to nova-network, which has been deprecated since the newton release | 21:22 |
| pramodrj07 | Oh cool! Thanks for clarification | 21:23 |
| mriedem | the security group resource APIs were deprecated in the 2.36 microversion | 21:23 |
| mriedem | you can provide security groups when creating a server, and nova will associate the security groups with ports that are attached to the server when creating it | 21:24 |
| mriedem | assuming the network that the port is on enables port security | 21:24 |
| *** tbachman has quit IRC | 21:24 | |
| pramodrj07 | Oh ok! So the latest cli client will call neutron security group API when we try to create a sg!? | 21:25 |
| mriedem | which cli? nova boot or openstack server create? | 21:26 |
| pramodrj07 | openstack security group create | 21:26 |
| pramodrj07 | yes Openstack one | 21:26 |
| mriedem | that is openstackclient and will use neutron if there is a network service type in the service catalog | 21:27 |
| *** xinliang has joined #openstack-nova | 21:27 | |
| *** xinliang has joined #openstack-nova | 21:27 | |
| pramodrj07 | Oh ok! Gotcha! | 21:28 |
| pramodrj07 | i am extending a plugin for rally to create sgs along with sg rules for neutron. I saw a similar scenario developed for nova. So wanted a clarification! | 21:30 |
| pramodrj07 | Thanks a lot mriedem | 21:30 |
| mriedem | yw | 21:31 |
| *** suresh12_ has quit IRC | 21:33 | |
| *** sdague has quit IRC | 21:33 | |
| *** xinliang has quit IRC | 21:44 | |
| *** nkorabli has quit IRC | 21:44 | |
| *** zz_dimtruck is now known as dimtruck | 21:44 | |
| *** burt has quit IRC | 21:45 | |
| -openstackstatus- NOTICE: The upgrade maintenance for lists.openstack.org has been completed and it is back online. | 21:50 | |
| *** tbachman has joined #openstack-nova | 21:51 | |
| *** ijw has joined #openstack-nova | 21:53 | |
| *** xyang1 has quit IRC | 21:56 | |
| *** xinliang has joined #openstack-nova | 21:57 | |
| *** xinliang has joined #openstack-nova | 21:57 | |
| *** ijw has quit IRC | 21:58 | |
| *** Swami has quit IRC | 21:59 | |
| *** leakypipes has quit IRC | 21:59 | |
| *** snikitin_ has quit IRC | 22:06 | |
| *** snikitin_ has joined #openstack-nova | 22:06 | |
| *** thorst has quit IRC | 22:10 | |
| *** raunak has quit IRC | 22:12 | |
| *** wagnqwsh has quit IRC | 22:13 | |
| *** wagnqwsh has joined #openstack-nova | 22:14 | |
| *** ijw has joined #openstack-nova | 22:17 | |
| *** figleaf is now known as edleafe | 22:18 | |
| *** tbachman has quit IRC | 22:19 | |
| *** artom has quit IRC | 22:26 | |
| *** artom has joined #openstack-nova | 22:26 | |
| *** jamesden_ has quit IRC | 22:27 | |
| *** artom has quit IRC | 22:30 | |
| *** artom has joined #openstack-nova | 22:30 | |
| *** dimtruck is now known as zz_dimtruck | 22:32 | |
| *** gouthamr has quit IRC | 22:34 | |
| *** catintheroof has quit IRC | 22:36 | |
| mriedem | melwitt: here is the bug for the quota commit/rollback thing in the api https://bugs.launchpad.net/nova/+bug/1678326 | 22:40 |
| openstack | Launchpad bug 1678326 in OpenStack Compute (nova) "Quota is decremented during instance delete in cell0 even if the instance destroy fails" [High,Triaged] - Assigned to Matt Riedemann (mriedem) | 22:40 |
| mriedem | long-time coming on writing that | 22:40 |
| mriedem | i don't have a good way of writing a regression test for that one really, unless i stub out instance.destroy() to raise InstanceNotFound to simulate concurrent deletes | 22:41 |
| melwitt | mriedem: thanks for writing that up | 22:41 |
| melwitt | I'm trying to remember how does the instance destroy fail | 22:42 |
| *** crushil has joined #openstack-nova | 22:42 | |
| *** ducttape_ has quit IRC | 22:42 | |
| *** adrian_otto has quit IRC | 22:43 | |
| *** ducttape_ has joined #openstack-nova | 22:43 | |
| mriedem | it could happen with concurrent delete requests couldn't it? | 22:43 |
| *** ducttape_ has quit IRC | 22:43 | |
| melwitt | I just can't remember what happened to where we figured this out | 22:43 |
| mriedem | idk, it was copied from _delete_while_booting | 22:46 |
| mriedem | which was copied from _delete | 22:46 |
| mriedem | which checks for that here https://github.com/openstack/nova/blob/88bc8dc5ce32748452c9d3acda9f35e77fedb6ce/nova/compute/api.py#L1806 | 22:46 |
| *** ducttape_ has joined #openstack-nova | 22:47 | |
| mriedem | sorry wrong link there | 22:47 |
| melwitt | looks like int32bit described it in a comment on https://bugs.launchpad.net/nova/+bug/1670627 but didn't say how delete failed | 22:48 |
| openstack | Launchpad bug 1670627 in OpenStack Compute (nova) ocata "quota is always in-use after delete the ERROR instances " [Critical,In progress] - Assigned to Matt Riedemann (mriedem) | 22:48 |
| mriedem | https://github.com/openstack/nova/blob/88bc8dc5ce32748452c9d3acda9f35e77fedb6ce/nova/compute/api.py#L2025 | 22:48 |
| melwitt | sorry, krims0n32 is the lp name | 22:48 |
| mriedem | yeah i was wondering about that | 22:49 |
| mriedem | anyway, the old old code just says # NOTE(comstud): Race condition. Instance already gone. | 22:49 |
| mriedem | again, this is all comstud's fault | 22:49 |
| melwitt | https://bugs.launchpad.net/nova/+bug/1670627/comments/27 | 22:49 |
| openstack | Launchpad bug 1670627 in OpenStack Compute (nova) ocata "quota is always in-use after delete the ERROR instances " [Critical,In progress] - Assigned to Matt Riedemann (mriedem) | 22:49 |
| *** zz_dimtruck is now known as dimtruck | 22:49 | |
| melwitt | heh | 22:50 |
| mriedem | 2012 so who knows https://github.com/openstack/nova/commit/c729ba8c0aa4d283e84d139bc98e0e89fd933c4a | 22:50 |
| mriedem | seemed like a good idea at the time | 22:50 |
| *** crushil has quit IRC | 22:51 | |
| mriedem | have you ever noticed this? https://github.com/openstack/nova/blob/88bc8dc5ce32748452c9d3acda9f35e77fedb6ce/nova/compute/api.py#L2010 | 22:51 |
| *** ducttape_ has quit IRC | 22:51 | |
| mriedem | i bet ^ is wrong now with the cellsv2 local delete case too | 22:51 |
| melwitt | yeah, I was just curious how it happened, mostly to make sure that's the only problem and there isn't some additional problem | 22:51 |
| melwitt | yeah, I have seen that before but it gets glossed over bc soft-delete | 22:52 |
| *** adrian_otto has joined #openstack-nova | 22:52 | |
| melwitt | it gets removed in my series (all reserve/commit/rollback removed) | 22:53 |
| mriedem | i noticed it because of this https://github.com/openstack/nova/blob/88bc8dc5ce32748452c9d3acda9f35e77fedb6ce/nova/compute/manager.py#L2334 | 22:53 |
| mriedem | the dumb thing is that we create the reservation in the api then just roll it back right away | 22:53 |
| mriedem | we'd be smarter to just not create the reservations in the first place if it's soft delete | 22:54 |
| mriedem | but i'm sure that was fixed as a bolt-on bug fix to the already 30 bolt-ons to the local delete code | 22:54 |
| melwitt | yeah, for real | 22:54 |
| mriedem | ok so maybe rather than me spending time making quotas.commit/rollback fail instead of noop if there are no reservations, because of counting quotas removing all of that in master, i can just work on the functional regression test recreate and the fix since those have to be backported to ocata | 22:56 |
| mriedem | i wanted to fix this before https://bugs.launchpad.net/nova/+bug/1675570 | 22:56 |
| openstack | Launchpad bug 1675570 in OpenStack Compute (nova) "Volumes are not detached when deleting shelved offloaded servers" [High,In progress] - Assigned to Zhenyu Zheng (zhengzhenyu) | 22:56 |
| melwitt | yeah. I wonder if some of that will get consolidated bc of the need to call _local_delete in some cases | 22:58 |
| mriedem | that's my plan | 22:58 |
| mriedem | rather than duplicate/triplicate everything, just re-use _local_delete | 22:58 |
| mriedem | for the latter bug | 22:58 |
| melwitt | _local_delete is needed if the instance had ever been scheduled. which you can't tell from only instance.host bc of shelved offloaded I guess | 22:59 |
| melwitt | the other case (the 5-time-reverted patch I have up) is an instance in error state. it could have volumes attached even though it has instance.host = None | 23:00 |
| melwitt | because there's code in compute that resets instance.host to None when setting to ERROR state | 23:00 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Make quotas.commit/rollback strict https://review.openstack.org/452346 | 23:00 |
| *** ijw has quit IRC | 23:04 | |
| *** Sukhdev has joined #openstack-nova | 23:07 | |
| *** dimtruck is now known as zz_dimtruck | 23:08 | |
| *** edmondsw has joined #openstack-nova | 23:09 | |
| *** thorst has joined #openstack-nova | 23:11 | |
| *** edmondsw has quit IRC | 23:14 | |
| *** unicell has quit IRC | 23:14 | |
| *** adrian_otto has quit IRC | 23:15 | |
| *** thorst has quit IRC | 23:15 | |
| *** crushil has joined #openstack-nova | 23:15 | |
| *** adrian_otto has joined #openstack-nova | 23:18 | |
| *** zz_dimtruck is now known as dimtruck | 23:20 | |
| *** nrezinorn has joined #openstack-nova | 23:22 | |
| nrezinorn | How new is the server.create_image snapshot function in nova client/api? I am seeing a problem where randomly the snapshot fails, and the VM stays powered off. | 23:25 |
| *** adrian_otto has quit IRC | 23:25 | |
| *** browne has quit IRC | 23:25 | |
| *** ducttape_ has joined #openstack-nova | 23:26 | |
| mriedem | nrezinorn: probably extremely old | 23:28 |
| mriedem | git blame will tell you | 23:28 |
| *** ducttape_ has quit IRC | 23:28 | |
| mriedem | the client side code shouldn't be causing random issues, you're likely hitting something on the server side | 23:28 |
| nrezinorn | its definately stuff server side, as you can see the errors here | 23:28 |
| *** ducttape_ has joined #openstack-nova | 23:28 | |
| nrezinorn | https://gist.github.com/Nrezinorn/decc29d0f142eee34c693e7220090ad9 | 23:28 |
| *** ducttape_ has quit IRC | 23:29 | |
| mriedem | qemu is blowing up | 23:29 |
| nrezinorn | ^in that i took a snapshot fine on that VM ~ 20-30 prior and it worked fine | 23:29 |
| nrezinorn | it doesnt always blow up. | 23:29 |
| *** ducttape_ has joined #openstack-nova | 23:30 | |
| nrezinorn | i will look through more RH bugreports for qemu , is there anything i can enable on the hupervisor to get more info than the error message from nova-compute? | 23:31 |
| mriedem | libvirt debug logging | 23:31 |
| mriedem | sec | 23:31 |
| mriedem | this is how we setup libvirt debug logging for our CI runs https://github.com/openstack-dev/devstack/blob/master/lib/nova_plugins/functions-libvirt#L104 | 23:31 |
| mriedem | what version of qemu/libvirt? | 23:32 |
| *** wagnqwsh has quit IRC | 23:32 | |
| nrezinorn | libvirt-2.0.0-10.el7_3.4.x86_64 qemu-kvm-1.5.3-126.el7_3.3.x86_64 | 23:33 |
| *** chyka has quit IRC | 23:34 | |
| mriedem | hmm, idk then, we're using what's in ubuntu 16.04 right now which is libvirt 1.3.1 | 23:34 |
| nrezinorn | sorry this is on centos ;) | 23:34 |
| nrezinorn | and i already looked to see if there were updated packages for both qemu and libvirt | 23:34 |
| mriedem | that qemu looks pretty old actually | 23:35 |
| nrezinorn | i will attempt the debug stuff you link on monday to see what is really going on. | 23:35 |
| mriedem | https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | 23:35 |
| nrezinorn | pretty old == CentOS7 pretty old ;) | 23:35 |
| mriedem | 7.3 2016-11-03 2.0.0 2.6.0 | 23:35 |
| mriedem | is it centos 7.3? | 23:35 |
| nrezinorn | yes | 23:35 |
| mriedem | oh 7.0 | 23:35 |
| mriedem | if it's 7.3, ^ says you should have qemu 2.6.0 | 23:35 |
| mriedem | not 1.5.3 | 23:35 |
| nrezinorn | hmmm | 23:36 |
| nrezinorn | ok thanks | 23:36 |
| mriedem | np | 23:36 |
| nrezinorn | i will look into why the packages are not lining up :) | 23:36 |
| *** dimtruck is now known as zz_dimtruck | 23:38 | |
| nrezinorn | hmm, wonder where that 2.6.0 rpm lives...on my machine im on now its 7.3 and i only see 1.5.3 like above | 23:38 |
| *** fragatina has joined #openstack-nova | 23:41 | |
| *** thorst has joined #openstack-nova | 23:42 | |
| *** ZZelle has quit IRC | 23:42 | |
| *** lyan_ has quit IRC | 23:42 | |
| mriedem | melwitt: ah writing a regression test for this commit/rollback thing shows me yet another problem :) | 23:44 |
| mriedem | it's a bountiful harvest of bugs | 23:44 |
| melwitt | :( | 23:44 |
| *** fragatin_ has quit IRC | 23:44 | |
| mriedem | nrezinorn: i'm not sure which service channels are used to build that list, or if it's just stock (i assume it's supposed to be stock) | 23:45 |
| mriedem | you could ask sgordon but i think it's too late in the day for his tz | 23:45 |
| mriedem | melwitt: right here https://github.com/openstack/nova/blob/88bc8dc5ce32748452c9d3acda9f35e77fedb6ce/nova/compute/api.py#L1891 | 23:46 |
| mriedem | we don't return | 23:46 |
| mriedem | but we have an instance, and continue going down the old path | 23:46 |
| melwitt | wah wah | 23:47 |
| mriedem | maybe that's ok, idk | 23:47 |
| mriedem | https://github.com/openstack/nova/blob/88bc8dc5ce32748452c9d3acda9f35e77fedb6ce/nova/compute/api.py#L1836-L1906 was all really supposed to be for transition i think | 23:48 |
| mriedem | still have the cell0 case though | 23:48 |
| melwitt | it looks like it should return in that case | 23:49 |
| mriedem | where the quotas are in the main db and the instance is in the cell0 db | 23:49 |
| *** gouthamr has joined #openstack-nova | 23:49 | |
| mriedem | yeah i'm adding that | 23:49 |
| dtp | mriedem is the task of adding a uuid to service model still mine? | 23:51 |
| nrezinorn | mriedem: im going to updgrade to those 2.6.0 RPMs Monday in our testing env - hoping this doesnt bite me though: https://www.redhat.com/archives/rdo-list/2016-December/msg00028.html | 23:51 |
| mriedem | dtp: it is | 23:52 |
| *** unicell has joined #openstack-nova | 23:52 | |
| dtp | cool, planning to start it monday | 23:52 |
| nrezinorn | i'll know more next week, and will come back if i need another drink from the well of knowledge ;) | 23:52 |
| mriedem | nrezinorn: you could ask dmisard in #openstack-operators | 23:52 |
| mriedem | it's a shallow well with me | 23:52 |
| mriedem | but dmisard is around other channels | 23:52 |
| nrezinorn | ill do so monday! 5pm on a friday for me :) | 23:53 |
| mriedem | dmsimard i mean | 23:53 |
| mriedem | 7pm for me | 23:53 |
| mriedem | good luck | 23:53 |
| *** kaisers has joined #openstack-nova | 23:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for quota decrement bug 1678326 https://review.openstack.org/452351 | 23:59 |
| openstack | bug 1678326 in OpenStack Compute (nova) "Quota is decremented during instance delete in cell0 even if the instance destroy fails" [High,Triaged] https://launchpad.net/bugs/1678326 - Assigned to Matt Riedemann (mriedem) | 23:59 |
| mriedem | melwitt: here is the regression test ^ i haven't run unit tests on the bug fix in nova.compute.api | 23:59 |
| mriedem | will do that later | 23:59 |
| mriedem | i'm in trouble for being late to dinner | 23:59 |
| melwitt | k, I'll check it out | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!