*** sean-k-m1 has joined #openstack-nova | 00:02 | |
jaypipes | efried: +2 from me. | 00:02 |
---|---|---|
*** gjayavelu has joined #openstack-nova | 00:03 | |
*** sean-k-mooney has quit IRC | 00:04 | |
*** amodi has quit IRC | 00:06 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-cells-v1 run with neutron https://review.openstack.org/549789 | 00:06 |
*** yamamoto has joined #openstack-nova | 00:07 | |
*** tetsuro has joined #openstack-nova | 00:17 | |
*** liverpooler has quit IRC | 00:17 | |
*** xinliang has quit IRC | 00:22 | |
*** Tom-Tom has joined #openstack-nova | 00:25 | |
*** odyssey4me has quit IRC | 00:29 | |
*** odyssey4me has joined #openstack-nova | 00:29 | |
*** Tom-Tom has quit IRC | 00:30 | |
*** xinliang has joined #openstack-nova | 00:34 | |
mriedem | jaypipes: that vmware resource pool thing is basically equivalent to AZ:HOST:NODE:CLUSTER:POOL right? | 00:35 |
*** abalutoiu has quit IRC | 00:39 | |
*** tbachman has joined #openstack-nova | 00:40 | |
*** chyka has joined #openstack-nova | 00:43 | |
*** moshele has joined #openstack-nova | 00:43 | |
*** stakeda has joined #openstack-nova | 00:44 | |
*** wolverin_ has quit IRC | 00:46 | |
*** wolverineav has joined #openstack-nova | 00:46 | |
*** chyka has quit IRC | 00:47 | |
*** suresh12 has quit IRC | 00:47 | |
*** suresh12 has joined #openstack-nova | 00:48 | |
*** r-daneel has quit IRC | 00:48 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:50 | |
*** wolverineav has quit IRC | 00:51 | |
*** jichen has joined #openstack-nova | 00:52 | |
*** suresh12 has quit IRC | 00:52 | |
*** suresh12 has joined #openstack-nova | 00:55 | |
*** Dinesh_Bhor has quit IRC | 00:56 | |
*** zhaochao has joined #openstack-nova | 00:57 | |
*** suresh12 has quit IRC | 01:00 | |
*** phuongnh has joined #openstack-nova | 01:01 | |
*** dave-mccowan has joined #openstack-nova | 01:05 | |
*** jmlowe has quit IRC | 01:08 | |
*** jmlowe has joined #openstack-nova | 01:10 | |
*** tiendc has joined #openstack-nova | 01:12 | |
*** moshele has quit IRC | 01:14 | |
*** mriedem has quit IRC | 01:24 | |
*** gjayavelu has quit IRC | 01:24 | |
*** Dinesh_Bhor has joined #openstack-nova | 01:24 | |
*** nicolasbock has quit IRC | 01:29 | |
*** suresh12 has joined #openstack-nova | 01:29 | |
*** fragatina has quit IRC | 01:32 | |
*** Tom-Tom has joined #openstack-nova | 01:32 | |
*** andreas_s has joined #openstack-nova | 01:33 | |
*** suresh12 has quit IRC | 01:34 | |
*** Tom-Tom_ has joined #openstack-nova | 01:35 | |
*** r-daneel has joined #openstack-nova | 01:37 | |
*** Tom-Tom has quit IRC | 01:37 | |
*** andreas_s has quit IRC | 01:37 | |
*** tianhui has joined #openstack-nova | 01:39 | |
*** Tom-Tom_ has quit IRC | 01:41 | |
*** nicolasbock has joined #openstack-nova | 01:41 | |
*** Tom-Tom has joined #openstack-nova | 01:42 | |
*** yamahata has quit IRC | 01:43 | |
*** yufei has joined #openstack-nova | 01:45 | |
*** fragatina has joined #openstack-nova | 01:46 | |
*** hongbin has joined #openstack-nova | 01:46 | |
*** OctopusZhang__ has joined #openstack-nova | 01:52 | |
*** yufei has quit IRC | 01:52 | |
*** OctopusZhang has joined #openstack-nova | 01:53 | |
*** hshiina has joined #openstack-nova | 01:53 | |
*** fragatina has quit IRC | 01:54 | |
*** claudiub|2 has quit IRC | 01:55 | |
*** OctopusZhang__ has quit IRC | 01:57 | |
*** harlowja has quit IRC | 01:59 | |
*** lei-zh has joined #openstack-nova | 02:00 | |
*** salv-orl_ has joined #openstack-nova | 02:04 | |
*** namnh has joined #openstack-nova | 02:05 | |
*** salv-orlando has quit IRC | 02:07 | |
*** Zames has joined #openstack-nova | 02:11 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix allocation_candidates not to ignore shared RPs https://review.openstack.org/533396 | 02:12 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601 | 02:12 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support relay RP for allocation candidates https://review.openstack.org/533437 | 02:12 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: remove not necessary short cut https://review.openstack.org/553122 | 02:12 |
*** Tom-Tom has quit IRC | 02:13 | |
*** Tom-Tom has joined #openstack-nova | 02:13 | |
*** Tom-Tom_ has joined #openstack-nova | 02:14 | |
*** Zames has quit IRC | 02:15 | |
*** r-daneel has quit IRC | 02:15 | |
*** r-daneel has joined #openstack-nova | 02:16 | |
*** Tom-Tom has quit IRC | 02:17 | |
*** p_d has joined #openstack-nova | 02:18 | |
*** p_d has joined #openstack-nova | 02:20 | |
*** david-lyle has quit IRC | 02:21 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix comments in get_all_with_shared() https://review.openstack.org/533195 | 02:21 |
*** mingyu_ has joined #openstack-nova | 02:23 | |
*** mingyu has quit IRC | 02:25 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events https://review.openstack.org/543277 | 02:26 |
*** zhurong has joined #openstack-nova | 02:29 | |
*** chyka has joined #openstack-nova | 02:32 | |
*** itlinux has quit IRC | 02:32 | |
*** Tom-Tom_ has quit IRC | 02:36 | |
*** phuongnh has quit IRC | 02:37 | |
*** chyka has quit IRC | 02:37 | |
*** phuongnh has joined #openstack-nova | 02:40 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925 | 02:42 |
*** naichuans has joined #openstack-nova | 02:43 | |
*** Tom-Tom has joined #openstack-nova | 02:44 | |
*** Dinesh_Bhor has quit IRC | 02:44 | |
*** Dinesh_Bhor has joined #openstack-nova | 02:45 | |
*** psachin has joined #openstack-nova | 02:46 | |
*** wolverineav has joined #openstack-nova | 02:49 | |
*** Spaz-Work has joined #openstack-nova | 02:59 | |
openstackgerrit | melanie witt proposed openstack/nova master: only increment disk address unit for scsi devices https://review.openstack.org/538310 | 03:04 |
*** bkopilov has quit IRC | 03:06 | |
*** Tom-Tom has quit IRC | 03:08 | |
*** Tom-Tom has joined #openstack-nova | 03:09 | |
*** wolverineav has quit IRC | 03:10 | |
*** sree has joined #openstack-nova | 03:10 | |
*** wolverineav has joined #openstack-nova | 03:10 | |
*** p_d has quit IRC | 03:13 | |
*** dave-mccowan has quit IRC | 03:14 | |
*** germs has quit IRC | 03:14 | |
*** wolverineav has quit IRC | 03:15 | |
*** germs has joined #openstack-nova | 03:15 | |
*** germs has quit IRC | 03:15 | |
*** germs has joined #openstack-nova | 03:15 | |
*** germs has quit IRC | 03:16 | |
*** germs has joined #openstack-nova | 03:17 | |
*** germs has quit IRC | 03:17 | |
*** germs has joined #openstack-nova | 03:17 | |
*** fragatin_ has joined #openstack-nova | 03:20 | |
*** annp has joined #openstack-nova | 03:26 | |
*** nicolasbock has quit IRC | 03:27 | |
*** felipemonteiro_ has joined #openstack-nova | 03:32 | |
*** tojuvone_ is now known as tojuvone | 03:32 | |
*** felipemonteiro__ has joined #openstack-nova | 03:33 | |
*** janki has joined #openstack-nova | 03:34 | |
*** felipemonteiro_ has quit IRC | 03:36 | |
*** hshiina has quit IRC | 03:40 | |
*** hshiina has joined #openstack-nova | 03:40 | |
*** links has joined #openstack-nova | 03:41 | |
*** Tom-Tom has quit IRC | 03:45 | |
*** Tom-Tom has joined #openstack-nova | 03:45 | |
*** felipemonteiro__ has quit IRC | 03:45 | |
*** felipemonteiro__ has joined #openstack-nova | 03:46 | |
*** hongbin has quit IRC | 03:52 | |
*** zhurong has quit IRC | 03:53 | |
*** OctopusZhang has quit IRC | 04:08 | |
*** germs has quit IRC | 04:09 | |
*** harlowja has joined #openstack-nova | 04:11 | |
*** Swami has quit IRC | 04:14 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | 04:19 |
*** bkopilov has joined #openstack-nova | 04:21 | |
*** sridharg has joined #openstack-nova | 04:29 | |
*** andreas_s has joined #openstack-nova | 04:31 | |
openstackgerrit | Brooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168 | 04:33 |
Spaz-Work | ^Ignore, just a rebase, will fix tests in a bit | 04:33 |
*** gyee has quit IRC | 04:35 | |
*** andreas_s has quit IRC | 04:36 | |
*** gjayavelu has joined #openstack-nova | 04:42 | |
*** abhishekk has joined #openstack-nova | 04:44 | |
*** lpetrut has joined #openstack-nova | 04:46 | |
*** harlowja has quit IRC | 04:50 | |
*** yamahata has joined #openstack-nova | 04:50 | |
openstackgerrit | Giridhar Jayavelu proposed openstack/nova-specs master: VMware: place instances on resource pool https://review.openstack.org/549067 | 04:59 |
*** vivsoni has joined #openstack-nova | 05:00 | |
*** jmlowe has quit IRC | 05:01 | |
*** felipemonteiro__ has quit IRC | 05:01 | |
*** Dinesh_Bhor has quit IRC | 05:02 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:03 | |
*** vivsoni has quit IRC | 05:08 | |
*** lpetrut has quit IRC | 05:11 | |
*** alexchadin has joined #openstack-nova | 05:11 | |
*** fragatin_ has quit IRC | 05:11 | |
*** udesale has joined #openstack-nova | 05:13 | |
*** ratailor has joined #openstack-nova | 05:14 | |
*** alexchadin has quit IRC | 05:17 | |
*** alexchadin has joined #openstack-nova | 05:17 | |
*** gjayavelu has quit IRC | 05:29 | |
*** mdnadeem has joined #openstack-nova | 05:39 | |
*** moshele has joined #openstack-nova | 05:47 | |
*** suresh12 has joined #openstack-nova | 05:48 | |
*** claudiub|2 has joined #openstack-nova | 05:48 | |
*** OctopusZhang has joined #openstack-nova | 05:51 | |
*** moshele has quit IRC | 05:52 | |
*** suresh12 has quit IRC | 05:53 | |
*** anadahz1BP3JC has joined #openstack-nova | 05:56 | |
anadahz1BP3JC | THIS IS A FREENODE BREAKING NEWS ALERT!! Hitechcg AND opal ARE GOING AT IT RIGHT NOW WITH A LOT OF FIGHTING AND ARGUING WOW YOU DON'T WANT TO MISS THIS!! TYPE /JOIN ## TO SEE THE ACTION...AGAIN TYPE /JOIN ## TO SEE THE ACTION!! | 05:56 |
anadahz1BP3JC | THIS IS A FREENODE BREAKING NEWS ALERT!! Hitechcg AND opal ARE GOING AT IT RIGHT NOW WITH A LOT OF FIGHTING AND ARGUING WOW YOU DON'T WANT TO MISS THIS!! TYPE /JOIN ## TO SEE THE ACTION...AGAIN TYPE /JOIN ## TO SEE THE ACTION!! | 05:56 |
anadahz1BP3JC | THIS IS A FREENODE BREAKING NEWS ALERT!! Hitechcg AND opal ARE GOING AT IT RIGHT NOW WITH A LOT OF FIGHTING AND ARGUING WOW YOU DON'T WANT TO MISS THIS!! TYPE /JOIN ## TO SEE THE ACTION...AGAIN TYPE /JOIN ## TO SEE THE ACTION!! | 05:56 |
anadahz1BP3JC | THIS IS A FREENODE BREAKING NEWS ALERT!! Hitechcg AND opal ARE GOING AT IT RIGHT NOW WITH A LOT OF FIGHTING AND ARGUING WOW YOU DON'T WANT TO MISS THIS!! TYPE /JOIN ## TO SEE THE ACTION...AGAIN TYPE /JOIN ## TO SEE THE ACTION!! | 05:56 |
anadahz1BP3JC | OctopusZhang claudiub|2 mdnadeem alexchadin ratailor udesale Dinesh_Bhor yamahata abhishekk sridharg bkopilov Tom-Tom links hshiina janki annp sree Spaz-Work psachin naichuans phuongnh mingyu_ r-daneel namnh salv-orl_ lei-zh tianhui tiendc zhaochao jichen stakeda tbachman xinliang odyssey4me tetsuro yamamoto sean-k-m1 hamzy Matias artom owalsh tojuvone rcernin gagehugo gbarros mvk mrjk imacdonn jamesdenton sapd_ arvindn05 jber | 05:56 |
*** anadahz1BP3JC has quit IRC | 05:56 | |
Spaz-Work | Aigoo.. | 05:56 |
*** OctopusZhang has quit IRC | 05:58 | |
openstackgerrit | jichenjc proposed openstack/nova master: Move test_allocation_candidates.py https://review.openstack.org/553149 | 06:01 |
openstackgerrit | jichenjc proposed openstack/nova master: Move test_allocation_candidates.py https://review.openstack.org/553150 | 06:05 |
Spaz-Work | Spent an hour trying to troubleshoot why my Tox was failing due to an 8 day old commit and was just my Tox ENV being dirty.. sigh.. Monnndayyyyysssssss | 06:08 |
*** chyka has joined #openstack-nova | 06:09 | |
*** Dinesh_Bhor has quit IRC | 06:10 | |
*** OctopusZhang has joined #openstack-nova | 06:10 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:10 | |
*** sidx64 has joined #openstack-nova | 06:12 | |
openstackgerrit | jichenjc proposed openstack/nova master: Move test_resource_provider.py https://review.openstack.org/553150 | 06:13 |
openstackgerrit | jichenjc proposed openstack/nova master: Move test_resource_class_cache.py https://review.openstack.org/553153 | 06:13 |
*** chyka has quit IRC | 06:13 | |
*** sidx64 has quit IRC | 06:13 | |
*** Dinesh_Bhor has quit IRC | 06:16 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | 06:18 |
*** Eran_Kuris has joined #openstack-nova | 06:19 | |
*** OctopusZhang has quit IRC | 06:21 | |
*** sidx64 has joined #openstack-nova | 06:24 | |
*** cfriesen_ has joined #openstack-nova | 06:26 | |
*** cfriesen has quit IRC | 06:27 | |
*** yamahata has quit IRC | 06:28 | |
*** OctopusZhang has joined #openstack-nova | 06:30 | |
*** kholkina has joined #openstack-nova | 06:34 | |
*** ccamacho has quit IRC | 06:39 | |
*** sidx64_ has joined #openstack-nova | 06:39 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:40 | |
*** sidx64 has quit IRC | 06:41 | |
*** sidx64 has joined #openstack-nova | 06:42 | |
*** sidx64_ has quit IRC | 06:44 | |
*** Eran_Kuris has quit IRC | 06:53 | |
*** namnh has quit IRC | 06:53 | |
*** moshele has joined #openstack-nova | 06:58 | |
openstackgerrit | jichenjc proposed openstack/nova master: Move test_resource_provider.py https://review.openstack.org/553150 | 06:59 |
openstackgerrit | jichenjc proposed openstack/nova master: Move test_resource_class_cache.py https://review.openstack.org/553153 | 06:59 |
*** Eran_Kuris has joined #openstack-nova | 07:08 | |
*** sidx64_ has joined #openstack-nova | 07:12 | |
*** sidx64 has quit IRC | 07:13 | |
*** alexchadin has quit IRC | 07:14 | |
*** OctopusZhang has quit IRC | 07:14 | |
*** OctopusZhang has joined #openstack-nova | 07:14 | |
*** OctopusZhang is now known as yufei | 07:14 | |
*** OctopusZhang__ has joined #openstack-nova | 07:15 | |
*** yufei has quit IRC | 07:15 | |
*** OctopusZhang__ is now known as yufei | 07:15 | |
*** sidx64 has joined #openstack-nova | 07:15 | |
*** sidx64_ has quit IRC | 07:17 | |
*** gaoyan has joined #openstack-nova | 07:17 | |
*** OctopusZhang_ has joined #openstack-nova | 07:18 | |
*** gaoyan has quit IRC | 07:19 | |
*** alexchadin has joined #openstack-nova | 07:19 | |
*** sar has joined #openstack-nova | 07:21 | |
*** yufei has quit IRC | 07:22 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/553211 | 07:26 |
*** Tom-Tom_ has joined #openstack-nova | 07:27 | |
*** andreas_s has joined #openstack-nova | 07:27 | |
*** ccamacho has joined #openstack-nova | 07:28 | |
*** ccamacho has quit IRC | 07:29 | |
*** Tom-Tom_ has quit IRC | 07:29 | |
*** ccamacho has joined #openstack-nova | 07:29 | |
*** Tom-Tom_ has joined #openstack-nova | 07:29 | |
*** Tom-Tom has quit IRC | 07:30 | |
*** phuongnh has quit IRC | 07:33 | |
*** Tom-Tom_ has quit IRC | 07:33 | |
*** alexchadin has quit IRC | 07:41 | |
*** gaoyan has joined #openstack-nova | 07:42 | |
*** alexchadin has joined #openstack-nova | 07:42 | |
*** sahid has joined #openstack-nova | 07:43 | |
*** pcaruana has joined #openstack-nova | 07:48 | |
*** lpetrut has joined #openstack-nova | 07:49 | |
*** pcaruana has quit IRC | 07:50 | |
*** pcaruana has joined #openstack-nova | 07:50 | |
*** pcaruana is now known as pcaruana|wfh| | 07:53 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/553255 | 08:01 |
*** sidx64 has quit IRC | 08:02 | |
*** OctopusZhang__ has joined #openstack-nova | 08:03 | |
*** OctopusZhang_ has quit IRC | 08:03 | |
*** Amit82 has joined #openstack-nova | 08:04 | |
*** sidx64 has joined #openstack-nova | 08:04 | |
Amit82 | Hi All | 08:05 |
Amit82 | Can LXC container orchestration be done using HEAT? | 08:05 |
*** OctopusZhang_ has joined #openstack-nova | 08:06 | |
*** OctopusZhang__ has quit IRC | 08:09 | |
*** OctopusZhang_ is now known as yufei | 08:09 | |
*** AlexeyAbashkin has joined #openstack-nova | 08:13 | |
*** sshwarts has joined #openstack-nova | 08:19 | |
*** sidx64 has quit IRC | 08:21 | |
openstackgerrit | Jianghua Wang proposed openstack/os-traits master: GPU: define traits for display heads https://review.openstack.org/553277 | 08:23 |
*** sahid_ has joined #openstack-nova | 08:23 | |
*** sahid_ has quit IRC | 08:23 | |
*** sahid has quit IRC | 08:23 | |
*** sahid has joined #openstack-nova | 08:23 | |
*** tesseract has joined #openstack-nova | 08:24 | |
*** tssurya has joined #openstack-nova | 08:40 | |
*** gjayavelu has joined #openstack-nova | 08:42 | |
*** jpena|off is now known as jpena | 08:43 | |
*** sidx64 has joined #openstack-nova | 08:48 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | 08:50 |
*** ralonsoh has joined #openstack-nova | 08:52 | |
*** hieulq has joined #openstack-nova | 08:58 | |
*** rcernin has quit IRC | 09:00 | |
*** tssurya has quit IRC | 09:02 | |
*** ispp is now known as burned | 09:04 | |
*** r-daneel has quit IRC | 09:07 | |
*** slunkad has quit IRC | 09:10 | |
*** cdent has joined #openstack-nova | 09:12 | |
*** lucas-afk is now known as lucasagomes | 09:12 | |
*** alexchadin has quit IRC | 09:12 | |
*** alexchadin has joined #openstack-nova | 09:13 | |
*** alexchadin has quit IRC | 09:13 | |
*** slunkad has joined #openstack-nova | 09:14 | |
*** alexchadin has joined #openstack-nova | 09:14 | |
*** alexchadin has quit IRC | 09:14 | |
*** alexchadin has joined #openstack-nova | 09:15 | |
*** alexchadin has quit IRC | 09:15 | |
*** alexchadin has joined #openstack-nova | 09:15 | |
*** alexchadin has quit IRC | 09:16 | |
*** gjayavelu has quit IRC | 09:16 | |
*** alexchadin has joined #openstack-nova | 09:16 | |
*** alexchadin has quit IRC | 09:16 | |
*** tssurya has joined #openstack-nova | 09:18 | |
openstackgerrit | Brooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168 | 09:20 |
lennyb | Hi, is it possible to clean/updated nova database? I have Orphaned Exception ( master branch ) http://paste.openstack.org/show/701494/ | 09:22 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/553288 | 09:24 |
*** gaoyan has quit IRC | 09:25 | |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | 09:27 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | 09:27 |
*** nicolasbock has joined #openstack-nova | 09:27 | |
Spaz-Work | OoOo | 09:27 |
*** janki has quit IRC | 09:28 | |
claudiub|2 | jichen: hi. around? | 09:32 |
*** claudiub|2 is now known as claudiub | 09:32 | |
jichen | claudiub: yes | 09:32 |
jichen | claudiub: what's up? | 09:32 |
claudiub | cool. regarding the live-resize. do you really want to support live-downsize as well? | 09:32 |
*** alexchadin has joined #openstack-nova | 09:32 | |
claudiub | there's quite a bit of opposition to that. :) | 09:32 |
claudiub | or maybe we can add support for upsizing first, and then look into downsizing afterwards | 09:34 |
*** amoralej|off is now known as amoralej | 09:34 | |
jichen | no, actually, I am ok if we only support upsize at first cycle | 09:34 |
claudiub | the nova cores seems to be fine with upsizing.' | 09:34 |
jichen | exactly, I agree with you | 09:35 |
jichen | upsizing should be first priority | 09:35 |
claudiub | great, then I will amend the spec to mention this fact. | 09:35 |
jichen | ok, perfect, thank you~ | 09:35 |
*** Dinesh_Bhor has quit IRC | 09:35 | |
*** lei-zh has quit IRC | 09:40 | |
*** derekh has joined #openstack-nova | 09:40 | |
*** Eran_Kuris has quit IRC | 09:42 | |
*** Eran_Kuris has joined #openstack-nova | 09:42 | |
*** mgoddard_ has joined #openstack-nova | 09:43 | |
*** shaohe_feng has joined #openstack-nova | 09:50 | |
*** sidx64 has quit IRC | 09:52 | |
*** jichen has quit IRC | 09:55 | |
*** sidx64 has joined #openstack-nova | 09:58 | |
*** Tom-Tom has joined #openstack-nova | 10:09 | |
*** stakeda has quit IRC | 10:10 | |
*** yufei has quit IRC | 10:10 | |
*** ragiman has joined #openstack-nova | 10:12 | |
*** sree has quit IRC | 10:13 | |
*** Tom-Tom has quit IRC | 10:13 | |
*** Eran_Kuris has quit IRC | 10:14 | |
*** sree has joined #openstack-nova | 10:14 | |
*** Eran_Kuris has joined #openstack-nova | 10:18 | |
*** maciejjozefczyk has quit IRC | 10:18 | |
*** sree has quit IRC | 10:19 | |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219 | 10:20 |
*** maciejjozefczyk has joined #openstack-nova | 10:26 | |
*** sridharg has quit IRC | 10:29 | |
*** hoangcx has quit IRC | 10:32 | |
*** abhishekk has quit IRC | 10:35 | |
*** hoangcx has joined #openstack-nova | 10:36 | |
*** moshele has quit IRC | 10:36 | |
*** moshele has joined #openstack-nova | 10:37 | |
*** liuzz has quit IRC | 10:38 | |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219 | 10:39 |
*** alexchadin has quit IRC | 10:46 | |
*** gongysh has joined #openstack-nova | 10:54 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 10:57 |
*** sidx64 has quit IRC | 10:59 | |
*** sidx64 has joined #openstack-nova | 11:00 | |
stephenfin | Does anyone else think reviewing specs in Gerrit is a PITA? How the heck did the docs team do it for so long? | 11:00 |
Spaz-Work | I wish I knew more about specs | 11:02 |
kashyap | stephenfin: What might be a better way? | 11:02 |
stephenfin | kashyap: Something that lets you comment on the rendered spec would be a huge help | 11:03 |
stephenfin | a la Google Docs | 11:03 |
kashyap | stephenfin: Not that I'm advocating "for Gerrit" (with which I have a "can't live with you, but have to live without you" relation) | 11:03 |
*** pchavva has joined #openstack-nova | 11:03 | |
kashyap | stephenfin: What is it that is specifically bothering you? | 11:03 |
*** yamamoto has quit IRC | 11:04 | |
stephenfin | Take this spec, for example https://review.openstack.org/#/c/552722/ | 11:04 |
* kashyap clicks | 11:04 | |
stephenfin | artom has kindly gone through and linked to a load of places in the code in order to explain some of the things he's talking about | 11:04 |
kashyap | The pig is still loading | 11:04 |
kashyap | stephenfin: And | 11:05 |
stephenfin | However, I can't click those links because it's just text. All the rst->html magic hasn't been applied yet | 11:05 |
kashyap | (I saw that spec, meaning to review) | 11:05 |
kashyap | Ah-ha | 11:05 |
stephenfin | Similarly, for my NUMA aware vSwitch spec, I included a load of images but none of those render in the rST, naturally enough | 11:05 |
kashyap | Hmm, nod | 11:06 |
stephenfin | So you end up with this schizophrenic style review where you're having to review the HTML version then jump back to the rST version to leave comments | 11:06 |
stephenfin | It's just...silly :) | 11:06 |
stephenfin | Also, Realistically, no one should care about the formatting of the source for a doc. It's the output that matters | 11:06 |
* kashyap goes to add a comment in the spec, before he forgets | 11:06 | |
stephenfin | But yeah, </rant> :) | 11:07 |
stephenfin | stupid Gerrit :mutter mutter: | 11:07 |
kashyap | stephenfin: Yeah, I hear the pain, I just try to focus on the core content, and the formatting not to be egregious | 11:07 |
stephenfin | kashyap: Do. It would be a good thing to finally resolve | 11:08 |
kashyap | stephenfin: BTW, slightly related: Some people seem to think 'patchwork' is "dead" | 11:08 |
*** sidx64 has quit IRC | 11:08 | |
stephenfin | Oh, really? | 11:08 |
kashyap | stephenfin: And QEMU folks have written their own: http://patchew.org/ | 11:08 |
kashyap | There's a bit of overlap with 'patchwork' | 11:08 |
kashyap | stephenfin: See this snippet: | 11:08 |
kashyap | 17:59 < kashyap> bonzini: Isn't there some overlap between 'patchew' and 'patchwork'? | 11:08 |
kashyap | 17:59 < bonzini> kashyap: yeah, but patchwork seemed dead when patchew was started | 11:08 |
kashyap | From a week or so ago | 11:09 |
kashyap | (On OFTC, #qemu) | 11:09 |
stephenfin | Yup, that's been around for a while. At the time Patchwork _was_ dead | 11:09 |
kashyap | Ah-ha, I see | 11:09 |
stephenfin | Was only when I and another Intel dev started working on it (for freedesktop) that it started moving forward again | 11:09 |
kashyap | Most people used 'patchwork' as just a "tracker" of patches, though | 11:10 |
stephenfin | Guess the two could be folded in but meh, I've barely any time to work on Patchwork of late | 11:10 |
Spaz-Work | Nova meeting is 2100 today right? | 11:11 |
*** sidx64 has joined #openstack-nova | 11:11 | |
* kashyap still loves the traditional patch workflow of QEMU / Kernel. Yes, the learning curve is steep, but by _Jove_ is it a joy! | 11:11 | |
* kashyap stops complaining & crawls back into his cave | 11:12 | |
*** damien_r has quit IRC | 11:14 | |
*** sidx64 has quit IRC | 11:14 | |
*** udesale has quit IRC | 11:15 | |
*** tiendc has quit IRC | 11:16 | |
*** liuyulong has quit IRC | 11:16 | |
kashyap | Spaz-Work: Not sure the hour; there are two timings | 11:17 |
*** gongysh has quit IRC | 11:17 | |
Spaz-Work | Yeah think this week is the 2100.. sitting here late in the day trying to convert that to CDT. Guess i'll read the logs :D | 11:18 |
Spaz-Work | Poor night shift hours. | 11:18 |
*** dtruong_ has joined #openstack-nova | 11:18 | |
*** lpetrut_ has joined #openstack-nova | 11:18 | |
*** lpetrut has quit IRC | 11:18 | |
*** andreas_s has quit IRC | 11:18 | |
*** tbachman_ has joined #openstack-nova | 11:19 | |
*** Spaz-Home has joined #openstack-nova | 11:19 | |
*** andreas_s has joined #openstack-nova | 11:19 | |
*** cdent_ has joined #openstack-nova | 11:19 | |
*** artom_ has joined #openstack-nova | 11:20 | |
*** bhagyashri_s has joined #openstack-nova | 11:20 | |
*** mnaser_ has joined #openstack-nova | 11:20 | |
*** Kevin_Zheng_ has joined #openstack-nova | 11:20 | |
*** karlamrhein_ has joined #openstack-nova | 11:20 | |
*** pooja-jadhav has joined #openstack-nova | 11:20 | |
*** Alexey_Abashkin has joined #openstack-nova | 11:20 | |
*** lpetrut has joined #openstack-nova | 11:22 | |
*** vipuls has joined #openstack-nova | 11:22 | |
*** dosaboy_ has joined #openstack-nova | 11:22 | |
*** frickler_ has joined #openstack-nova | 11:23 | |
*** Anticime1 has joined #openstack-nova | 11:23 | |
*** kuzko_ has joined #openstack-nova | 11:23 | |
*** jog0 has joined #openstack-nova | 11:23 | |
*** sahid_ has joined #openstack-nova | 11:24 | |
*** andreas_s has quit IRC | 11:24 | |
*** sorrison_ has joined #openstack-nova | 11:24 | |
*** jgriffit1 has joined #openstack-nova | 11:24 | |
*** weezhard has joined #openstack-nova | 11:25 | |
*** tetsuro has quit IRC | 11:26 | |
*** slaweq_ has joined #openstack-nova | 11:26 | |
*** cdent has quit IRC | 11:27 | |
*** pcaruana|wfh| has quit IRC | 11:27 | |
*** ttsiouts has quit IRC | 11:27 | |
*** kaisers1 has quit IRC | 11:27 | |
*** dtruong has quit IRC | 11:27 | |
*** sorrison has quit IRC | 11:27 | |
*** vipul has quit IRC | 11:27 | |
*** bkero- has quit IRC | 11:27 | |
*** jogo has quit IRC | 11:27 | |
*** MikeG451 has quit IRC | 11:27 | |
*** dosaboy has quit IRC | 11:27 | |
*** Kevin_Zheng has quit IRC | 11:27 | |
*** karlamrhein has quit IRC | 11:27 | |
*** mnaser has quit IRC | 11:27 | |
*** ingy has quit IRC | 11:27 | |
*** frickler has quit IRC | 11:27 | |
*** Anticimex has quit IRC | 11:27 | |
*** cdent_ is now known as cdent | 11:27 | |
*** mnaser_ is now known as mnaser | 11:27 | |
*** karlamrhein_ is now known as karlamrhein | 11:27 | |
*** pcaruana|wfh| has joined #openstack-nova | 11:28 | |
*** ccamacho1 has joined #openstack-nova | 11:28 | |
*** ccamacho has quit IRC | 11:28 | |
*** sshwarts has quit IRC | 11:28 | |
*** AlexeyAbashkin has quit IRC | 11:28 | |
*** moshele has quit IRC | 11:28 | |
*** sahid has quit IRC | 11:28 | |
*** tbachman has quit IRC | 11:28 | |
*** Spazmotic has quit IRC | 11:28 | |
*** FL1SK has quit IRC | 11:28 | |
*** liusheng has quit IRC | 11:28 | |
*** krtaylor has quit IRC | 11:28 | |
*** jgriffith has quit IRC | 11:28 | |
*** weezhard_ has quit IRC | 11:28 | |
*** lpetrut_ has quit IRC | 11:28 | |
*** artom has quit IRC | 11:28 | |
*** pooja_jadhav has quit IRC | 11:28 | |
*** bhagyashris_ has quit IRC | 11:28 | |
*** slaweq has quit IRC | 11:28 | |
*** kuzko has quit IRC | 11:28 | |
*** ingy1 has joined #openstack-nova | 11:28 | |
*** liusheng has joined #openstack-nova | 11:28 | |
*** krtaylor has joined #openstack-nova | 11:28 | |
*** Kevin_Zheng_ has quit IRC | 11:28 | |
*** Kevin_Zheng_ has joined #openstack-nova | 11:28 | |
*** mnaser has quit IRC | 11:28 | |
*** mnaser has joined #openstack-nova | 11:28 | |
*** karlamrhein has quit IRC | 11:28 | |
*** karlamrhein has joined #openstack-nova | 11:28 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 11:28 | |
*** kaisers1 has joined #openstack-nova | 11:28 | |
*** ttsiouts has joined #openstack-nova | 11:29 | |
*** tbachman_ is now known as tbachman | 11:29 | |
*** sshwarts has joined #openstack-nova | 11:31 | |
*** bkero- has joined #openstack-nova | 11:31 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/553288 | 11:32 |
*** mdbooth has joined #openstack-nova | 11:32 | |
mdbooth | johnthetubaguy, any chance you might be able to squint at this bugfix for me: https://review.openstack.org/#/c/551302/ | 11:33 |
mdbooth | Incidentally, who has anything to do with Hyper-V? | 11:33 |
*** MikeG451 has joined #openstack-nova | 11:34 | |
stephenfin | mdbooth: claudiub is your man | 11:35 |
*** frickler_ is now known as frickler | 11:35 | |
mdbooth | claudiub, Any chance you could look at ^^^ for me? | 11:35 |
mdbooth | lyarwood, it deletes a bunch of connection_info-related libvirt driver code in live migration if you'd care to look | 11:36 |
mdbooth | claudiub, I don't think it's an issue for Hyper-V, except that it might potentially fix bugs. It's a change, though, which is why I highlight it. | 11:37 |
*** ratailor has quit IRC | 11:38 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add disabled column to cell_mappings table. https://review.openstack.org/552505 | 11:38 |
lyarwood | mdbooth: ack, will look | 11:40 |
*** pchavva has quit IRC | 11:40 | |
*** sidx64 has joined #openstack-nova | 11:45 | |
*** annp has quit IRC | 11:48 | |
*** sidx64 has quit IRC | 11:52 | |
*** liverpooler has joined #openstack-nova | 11:52 | |
*** Amit82 has quit IRC | 11:52 | |
*** sidx64 has joined #openstack-nova | 11:54 | |
openstackgerrit | Hironori Shiina proposed openstack/nova master: ironic: Get correct inventory for deployed node https://review.openstack.org/553367 | 11:58 |
*** ameeda has joined #openstack-nova | 11:58 | |
ameeda | Hello, I am trying to upload image from server1 to openstack "server2" using rest api. when I did that I got error message "504 Gateway Time-out The server didn't respond in time." | 11:59 |
*** amoralej is now known as amoralej|lunch | 12:00 | |
lyarwood | mdbooth: LGTM, however there's no volume backed LM tests at present http://logs.openstack.org/02/551302/6/check/legacy-grenade-dsvm-neutron-multinode-live-migration/3e0cf62/job-output.txt.gz#_2018-03-13_19_30_46_058750 - due to https://bugs.launchpad.net/nova/+bug/1524898 | 12:02 |
openstack | Launchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,In progress] - Assigned to Lee Yarwood (lyarwood) | 12:02 |
*** bkopilov has quit IRC | 12:03 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest https://review.openstack.org/528104 | 12:03 |
lyarwood | mdbooth: ^ you can use this change from melwitt to test LM with volume backed instances | 12:04 |
*** yamamoto has joined #openstack-nova | 12:04 | |
*** dtantsur|afk is now known as dtantsur | 12:04 | |
*** odyssey4me has quit IRC | 12:05 | |
*** odyssey4me has joined #openstack-nova | 12:05 | |
*** yamamoto has quit IRC | 12:09 | |
*** pcaruana|wfh| is now known as pcaruana | 12:14 | |
*** edmondsw has joined #openstack-nova | 12:15 | |
*** yamamoto has joined #openstack-nova | 12:16 | |
*** elmaciej has joined #openstack-nova | 12:17 | |
*** claudiub|2 has joined #openstack-nova | 12:20 | |
*** jpena is now known as jpena|lunch | 12:21 | |
*** takedakn has joined #openstack-nova | 12:21 | |
*** takedakn has quit IRC | 12:22 | |
*** claudiub has quit IRC | 12:23 | |
*** sidx64 has quit IRC | 12:24 | |
*** moshele has joined #openstack-nova | 12:27 | |
mdbooth | lyarwood, Will do, thanks! | 12:28 |
*** lucasagomes is now known as lucas-hungry | 12:28 | |
mdbooth | lyarwood: Short of adding a depends on to my patch, what's the best way to run against that? | 12:30 |
*** sidx64 has joined #openstack-nova | 12:30 | |
mdbooth | lyarwood: Not that I'd mind adding a depends on if the test patch is likely to get some priority. | 12:30 |
lyarwood | mdbooth: another change on top of https://review.openstack.org/528104 that depends on your change? | 12:31 |
mdbooth | lyarwood: Good idea, thanks. | 12:31 |
openstackgerrit | Matthew Booth proposed openstack/nova master: DNM: Run volume-backed live migration tests https://review.openstack.org/553377 | 12:33 |
*** moshele has quit IRC | 12:33 | |
*** mikal has joined #openstack-nova | 12:42 | |
*** psachin has quit IRC | 12:44 | |
*** jmlowe has joined #openstack-nova | 12:44 | |
*** bhujay has joined #openstack-nova | 12:44 | |
*** mikal_ has quit IRC | 12:45 | |
*** r-daneel has joined #openstack-nova | 12:46 | |
*** suresh12 has joined #openstack-nova | 12:49 | |
*** FL1SK has joined #openstack-nova | 12:50 | |
*** Tom-Tom has joined #openstack-nova | 12:50 | |
*** david-lyle has joined #openstack-nova | 12:52 | |
*** suresh12 has quit IRC | 12:53 | |
*** Zames has joined #openstack-nova | 12:54 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 12:55 |
*** moshele has joined #openstack-nova | 12:56 | |
*** Zames has quit IRC | 12:57 | |
*** moshele has quit IRC | 12:58 | |
*** sean-k-m1 is now known as sean-k-mooney | 12:59 | |
*** dave-mccowan has joined #openstack-nova | 13:01 | |
*** sridharg has joined #openstack-nova | 13:01 | |
lpetrut | mdbooth: Hi, thanks for letting us know. I took a look over the patch and it looks good, it shouldn't affect the Hyper-V driver. High chance is that we'd only have issues with the Dell iSCSI driver that you're mentioning in the commit (which should be fixed by it). | 13:02 |
*** lyan has joined #openstack-nova | 13:02 | |
*** lyan is now known as Guest79046 | 13:02 | |
mdbooth | lpetrut: What changes is that I'm now passing the *old* bdms to post_live_migration instead of the new ones | 13:02 |
mdbooth | So connection_info will relate to the source host, which is where it's actually running, rather than the destination | 13:03 |
lpetrut | mdbooth: do you know by any chance if there are any other iSCSI drivers that would return different connection info among hosts (i.e. have a different conn info on the destination)? | 13:03 |
mdbooth | lpetrut: Potentially a lot, I think | 13:03 |
mdbooth | That particular issue has been around for ages | 13:03 |
lpetrut | for FC it would look different but that's fine as disconnect_volume is a noop for FC in our case | 13:03 |
*** andreas_s has joined #openstack-nova | 13:03 | |
*** alexchadin has joined #openstack-nova | 13:03 | |
mdbooth | The new issue is that there's one driver where initialize_connection doesn't return the same value if you call it a second time for the *same* host | 13:04 |
lpetrut | got it | 13:04 |
*** sean-k-mooney has quit IRC | 13:05 | |
mdbooth | lpetrut: This was the original bug: https://bugs.launchpad.net/nova/+bug/1475411 | 13:05 |
openstack | Launchpad bug 1475411 in nova (Ubuntu Trusty) "During post_live_migration the nova libvirt driver assumes that the destination connection info is the same as the source, which is not always true" [Undecided,Fix released] | 13:05 |
mdbooth | That's obviously from 2015 | 13:06 |
*** hshiina has quit IRC | 13:06 | |
mdbooth | Affects 3par, apparently: https://bugs.launchpad.net/nova/+bug/1288039 | 13:06 |
openstack | Launchpad bug 1288039 in OpenStack Compute (nova) "live-migration cinder boot volume target_lun id incorrect" [High,Fix released] - Assigned to Anthony Lee (anthony-mic-lee) | 13:06 |
mdbooth | Anyway, if Hyper-V uses that cinder driver, my expectation is that my new change will fix it | 13:08 |
lpetrut | thanks for the link, indeed, that should fix it | 13:08 |
*** eharney has joined #openstack-nova | 13:09 | |
*** mriedem has joined #openstack-nova | 13:09 | |
lpetrut | I was thinking about submitting a bug report against our driver for posterity reasons, marking your patch as a fix | 13:09 |
lpetrut | I was not aware of the 3par issue, high chance is that we were leaking volume connections and nobody noticed | 13:10 |
*** shaohe_feng has quit IRC | 13:11 | |
mdbooth | Is anybody aware of a way to make devstack do git checkouts, install pip dependencies, and then stop? | 13:11 |
*** shaohe_feng has joined #openstack-nova | 13:12 | |
*** david-lyle has quit IRC | 13:12 | |
*** sidx64 has quit IRC | 13:13 | |
*** belmoreira has joined #openstack-nova | 13:13 | |
*** sree has joined #openstack-nova | 13:14 | |
*** sidx64 has joined #openstack-nova | 13:15 | |
*** david-lyle has joined #openstack-nova | 13:15 | |
*** sidx64 has quit IRC | 13:15 | |
*** abhishekk has joined #openstack-nova | 13:16 | |
*** yamamoto has quit IRC | 13:18 | |
*** damien_r has joined #openstack-nova | 13:18 | |
*** sree has quit IRC | 13:18 | |
*** yamamoto has joined #openstack-nova | 13:19 | |
*** Zames has joined #openstack-nova | 13:21 | |
*** amodi has joined #openstack-nova | 13:22 | |
kashyap | mtreinish might know ^ | 13:23 |
*** yamamoto has quit IRC | 13:23 | |
*** amoralej|lunch is now known as amoralej | 13:24 | |
cdent | mdbooth: is this in a session you're running by hand or something you want to automate | 13:24 |
*** sidx64 has joined #openstack-nova | 13:25 | |
*** david-lyle has quit IRC | 13:25 | |
*** idlemind has joined #openstack-nova | 13:25 | |
*** gouthamr_ has joined #openstack-nova | 13:26 | |
*** gouthamr_ is now known as gouthamr | 13:26 | |
cdent | mdbooth: if the former, probably just hack an exit in stack.sh | 13:28 |
ameeda | any help? Hello, I am trying to upload image from server1 to openstack "server2" using rest api. when I did that I got error message "504 Gateway Time-out The server didn't respond in time." | 13:29 |
*** jpena|lunch is now known as jpena | 13:29 | |
ameeda | I set glance conn timeout to 0 , with no lucks | 13:29 |
*** Zames has quit IRC | 13:29 | |
cdent | mdbooth: also grep on TRACK_DEPENDS; let's you bail out after install | 13:30 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-cells-v1 run with neutron https://review.openstack.org/549789 | 13:30 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup tempest-dsvm-cells-rc blacklist https://review.openstack.org/553401 | 13:30 |
tssurya | mriedem : owe you one for the NewtonCellCheck patch, thanks | 13:31 |
*** awaugama has joined #openstack-nova | 13:31 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup tempest-dsvm-cells-rc blacklist https://review.openstack.org/553401 | 13:31 |
mriedem | tssurya: np | 13:31 |
*** ralonsoh_ has joined #openstack-nova | 13:32 | |
*** Tom-Tom has quit IRC | 13:32 | |
ameeda | mriedem: Hello :) , could you please help me with my issue ? | 13:32 |
mriedem | ameeda: um, check the logs | 13:32 |
*** jmlowe has quit IRC | 13:33 | |
ameeda | mriedem: let me check , thanks | 13:33 |
*** dave-mccowan has quit IRC | 13:34 | |
*** sidx64 has quit IRC | 13:35 | |
*** ratailor has joined #openstack-nova | 13:35 | |
*** ralonsoh has quit IRC | 13:35 | |
mdbooth | cdent: Automate. I'm trying to build a base devstack image with all the config-agnostic stuff pre-done. Will look at TRACK_DEPENDS, thanks. | 13:36 |
*** felipemonteiro__ has joined #openstack-nova | 13:36 | |
*** burt has joined #openstack-nova | 13:36 | |
*** awaugama has quit IRC | 13:36 | |
kholkina | hi all! could you please review the spec? https://review.openstack.org/#/c/547964 | 13:36 |
*** felipemonteiro_ has joined #openstack-nova | 13:38 | |
*** p_d has joined #openstack-nova | 13:39 | |
*** jmlowe has joined #openstack-nova | 13:40 | |
claudiub|2 | mdbooth: sorry, I was out for a bit. Will look at it. | 13:40 |
*** awaugama has joined #openstack-nova | 13:40 | |
mdbooth | claudiub|2: Thanks. lpetrut already looked, btw. | 13:41 |
*** pchavva has joined #openstack-nova | 13:41 | |
*** bkopilov has joined #openstack-nova | 13:41 | |
*** felipemonteiro__ has quit IRC | 13:42 | |
*** mvk has quit IRC | 13:42 | |
*** lucas-hungry is now known as lucasagomes | 13:42 | |
*** links has quit IRC | 13:42 | |
*** udesale has joined #openstack-nova | 13:43 | |
ameeda | mriedem: I check this log /var/log/glance/api.log, I didn't find anything helpful , also this command journalctl -u openstack-glance-api.service -f | 13:44 |
*** felipemonteiro_ has quit IRC | 13:45 | |
*** felipemonteiro_ has joined #openstack-nova | 13:45 | |
SamYaple | ameeda: how did you deploy glance? uwsgi? apache? eventlet? | 13:45 |
mriedem | kholkina: -2 on that idea, you can already update user-data via rebuild now | 13:46 |
mriedem | a lot of thought and discussion already went into adding that to rebuild | 13:47 |
mriedem | see http://lists.openstack.org/pipermail/openstack-operators/2017-October/014309.html | 13:47 |
ameeda | SamYaple: I create python project to use contact with openstack via rest api, I use http requests as rest client | 13:47 |
mriedem | ameeda: this isn't really the channel for this type of question - try #openstack or #openstack-glance | 13:47 |
*** sidx64 has joined #openstack-nova | 13:47 | |
*** r-daneel has quit IRC | 13:48 | |
ameeda | everything working on my project, also I test the project on packstack, its working, but this case have to copy image file from server1 to openstack server, so the image file not exists on the same openstack | 13:49 |
*** licanwei has quit IRC | 13:50 | |
ameeda | mriedem: I will try the other channels, thank you | 13:50 |
*** mlavalle has joined #openstack-nova | 13:51 | |
efried | jaypipes, cdent: Opinion on https://review.openstack.org/#/c/548975/5/nova/scheduler/client/report.py@592 ? | 13:52 |
*** janki has joined #openstack-nova | 13:52 | |
cdent | efried: explicit is better than implicit | 13:52 |
cdent | some version ought to be listed there | 13:52 |
*** sidx64 has quit IRC | 13:52 | |
kholkina | mriedem, the spec means to update user-data on booted instance, without rebuild. It may be useful to provide a cloud-config | 13:52 |
efried | cdent: Even if it's '1.0'? | 13:54 |
jaypipes | efried: yeah, agree with alex_xu_ it would be good to be consistent there. | 13:54 |
cdent | efried: yes | 13:54 |
jaypipes | efried: no biggie though, really | 13:54 |
*** sidx64 has joined #openstack-nova | 13:54 | |
efried | cdent: Okay. Note that there's places we don't do that, elsewhere in the report client. A big enough deal to put up a patch to fill those in? | 13:55 |
cdent | no | 13:55 |
cdent | It's, as jay says, no biggie, but if you're adding something new may as well make the version explicit, even if it is the base | 13:55 |
alex_xu_ | that method is only used for create root rp? | 13:55 |
edleafe | It also looks consistent in that block to have a version in both | 13:56 |
*** crushil has joined #openstack-nova | 13:56 | |
cdent | that will help make things more clear if at some point you do need to add a parent or something there and it doesn't work and you're all wtf, why is this not taking my parent, oh the versin | 13:56 |
*** sean-k-mooney has joined #openstack-nova | 13:56 | |
efried | fixing. | 13:57 |
efried | alex_xu_: yes | 13:57 |
efried | oh | 13:57 |
efried | maybe not. Stand by. | 13:57 |
efried | Hah, you're right, it can assign a parent. So my change was blatantly wrong on that point. Good catch, alex_xu_. Fixing... | 13:58 |
*** lei-zh has joined #openstack-nova | 13:58 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Stop assuming initial provider generation is 0 https://review.openstack.org/548975 | 13:58 |
efried | alex_xu_, jaypipes, cdent: Done ^ | 13:59 |
*** lei-zh has quit IRC | 13:59 | |
*** lei-zh has joined #openstack-nova | 13:59 | |
ratailor | alex_xu_, jaypipes could you pl have a look at https://review.openstack.org/#/c/401009/ | 13:59 |
*** esberglu has joined #openstack-nova | 14:00 | |
*** bhujay has quit IRC | 14:01 | |
*** gongysh has joined #openstack-nova | 14:01 | |
belmoreira | dansmith, mriedem in my cellsV1/V2 migration plan I have some issues that I would like to discuss you. Let me know when you are available | 14:03 |
*** tidwellr has joined #openstack-nova | 14:03 | |
*** jaosorior has quit IRC | 14:03 | |
*** ralonsoh__ has joined #openstack-nova | 14:04 | |
*** ralonsoh__ is now known as ralonsoh | 14:04 | |
*** ralonsoh_ has quit IRC | 14:07 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova master: Reset instance status on invalid host https://review.openstack.org/401009 | 14:08 |
dansmith | belmoreira: go for it | 14:08 |
belmoreira | thanks dansmith | 14:08 |
belmoreira | We use the same cloud to deploy VMs and baremetal. Baremetal is dedicated to few cells. | 14:09 |
belmoreira | When consolidating the scheduler at top level (not per cell as in cellsV1) not seeing how can dedicate schedulers to ironic cells because they need the baremetal_filters enabled. | 14:09 |
dansmith | belmoreira: the baremetal filters are deprecated | 14:09 |
-openstackstatus- NOTICE: POST_FAILURE results on Tempest-based jobs since the most recent Zuul security fixes are being investigated; rechecking those won't help for now but we'll keep you posted once a solution is identified | 14:10 | |
*** ChanServ changes topic to "POST_FAILURE results on Tempest-based jobs since the most recent Zuul security fixes are being investigated; rechecking those won't help for now but we'll keep you posted once a solution is identified" | 14:10 | |
dansmith | belmoreira: the way of the light is custom resource classes for the ironic nodes, and flavors configured to give you the right thing when you ask, which makes scheduling uniform | 14:10 |
belmoreira | I missed that | 14:11 |
mriedem | there was a caveat with dropping the ironic host manager for the scheduler, would have to dig it back up, | 14:11 |
mriedem | but i think it's that not all filter scheduler filters will be able to handle baremetal instances, numatopologyfilter specifically i think | 14:11 |
belmoreira | I'm finding the release notes about this. thanks | 14:12 |
mriedem | i'll be damned if i can find the way to those docs in ironic via their homepage | 14:12 |
belmoreira | :) | 14:13 |
mriedem | belmoreira: https://docs.openstack.org/ironic/latest/install/configure-nova-flavors.html#scheduling-based-on-resource-classes | 14:13 |
mriedem | search box ftw | 14:13 |
alex_xu_ | mriedem: does make sense to move the get_compute_nodes before the state change https://review.openstack.org/#/c/401009/9/nova/compute/api.py@4222 ? | 14:13 |
*** BlackDex has quit IRC | 14:13 | |
alex_xu_ | it pretty looks like an API layer check. | 14:13 |
*** mvk has joined #openstack-nova | 14:14 | |
belmoreira | thanks | 14:14 |
mriedem | alex_xu_: i wouldn't be opposed to that, replied | 14:14 |
alex_xu_ | mriedem: ok | 14:14 |
mriedem | belmoreira: https://github.com/openstack/nova/commit/c99fc64271f41a028c75e96348813f673d04e245 | 14:15 |
mriedem | NUMATopologyFilter isn't enabled by default, but i know a lot of people use it | 14:16 |
mriedem | if we had someone that was familiar enough with both nova and ironic (cough jroll johnthetubaguy mgoddard_ cough) we could have them play around with updating an ironic CI job to not use the ironic host manager and baremetal filters and see what breaks | 14:17 |
mgoddard_ | mriedem can I offer you a throat sweet? | 14:18 |
mriedem | wtf | 14:18 |
mgoddard_ | for your cough | 14:18 |
mriedem | is that british for "i'm going to slit your throat?" | 14:18 |
mriedem | oh | 14:18 |
*** crushil has quit IRC | 14:19 | |
mgoddard_ | oh wow, crossed wires there | 14:19 |
mriedem | i consider phlegm a feature so no thanks | 14:19 |
dansmith | PaaS - Phlegm as a Service? | 14:19 |
mgoddard_ | I could take a look at this | 14:19 |
mriedem | i'm having trouble remembering off the top of my head what it was about the NUMATopologyFilter that makes it not work with ironic | 14:20 |
*** yamamoto has joined #openstack-nova | 14:20 | |
dansmith | mriedem: that needs fixing if so | 14:20 |
dansmith | maybe we could get stephenfin or someone to look at that once we know it's really a problem | 14:20 |
stephenfin | I didn't do it | 14:21 |
stephenfin | I've never used ironic so I'm probably not your guy | 14:22 |
mriedem | apparently ironic has a CI job (or defalut?) to use resource classes so they don't use the ironic hos tmanager https://review.openstack.org/#/c/499106/ | 14:22 |
mgoddard_ | yes, ironic devstack plugin uses resource classes by default | 14:23 |
mriedem | good news, mogan does too http://codesearch.openstack.org/?q=IRONIC_USE_RESOURCE_CLASSES&i=nope&files=&repos= | 14:23 |
dansmith | stephenfin: we need someone that understands the numa filter not necessarily ironic | 14:23 |
stephenfin | Oh, that I can do | 14:23 |
dansmith | mriedem: but no numafilter presumably | 14:23 |
mriedem | nope, so i think i can just hack up devstack and say don't use the baremetal filters and add numatopologyfilter to the list of default enabled filters and see what blows up | 14:24 |
*** sar has quit IRC | 14:24 | |
dansmith | ack | 14:24 |
mriedem | and then report a bug to stephenfin | 14:24 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova master: WIP: VMware: implement get_serial_console https://review.openstack.org/553411 | 14:24 |
stephenfin | 🎉 | 14:25 |
mriedem | is that a lego brick? a touch tone phone? | 14:25 |
dansmith | party horn? | 14:25 |
*** hieulq has quit IRC | 14:25 | |
stephenfin | https://emojipedia.org/party-popper/ | 14:25 |
mriedem | oh, doesn't render for me | 14:25 |
mriedem | pidgin | 14:25 |
*** yamamoto has quit IRC | 14:25 | |
stephenfin | Windows? | 14:25 |
mriedem | yeah | 14:26 |
* stephenfin thought Windows 10 supported all these. Seems not (F27 + Hexchat, fwiw) | 14:26 | |
stephenfin | But yeah, assign me to that. Soon as I get my server back, I can take a look | 14:26 |
stephenfin | I'm on spec review duty all day today | 14:26 |
*** ratailor has quit IRC | 14:27 | |
*** lei-zh has quit IRC | 14:30 | |
*** r-daneel has joined #openstack-nova | 14:30 | |
*** gagehugo has left #openstack-nova | 14:30 | |
*** gongysh has quit IRC | 14:31 | |
*** germs has joined #openstack-nova | 14:33 | |
*** germs has quit IRC | 14:33 | |
*** germs has joined #openstack-nova | 14:33 | |
*** hieulq has joined #openstack-nova | 14:33 | |
*** germs has quit IRC | 14:33 | |
mriedem | test patch https://review.openstack.org/553412 | 14:33 |
mriedem | mgoddard_: ^ | 14:33 |
*** elmaciej has quit IRC | 14:34 | |
mgoddard_ | mriedem: nice, I'll keep an eye on it | 14:35 |
*** germs has joined #openstack-nova | 14:36 | |
*** yamahata has joined #openstack-nova | 14:39 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 14:40 |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Few correction in the server filter/sort spec https://review.openstack.org/527019 | 14:41 |
*** zhaochao has quit IRC | 14:42 | |
Spaz-Home | dansmith, I confirmed either Jianghua or Niachuan will be at the meeting later today, sir. You should be good to go on reviewing that spec with them. | 14:44 |
Spaz-Home | Time for me to head to bed.. have a good day folks. | 14:44 |
dansmith | Spaz-Home: niachuan asked us to do the meeting for him because he can't join | 14:45 |
dansmith | which I will | 14:45 |
dansmith | but yeah | 14:45 |
Spaz-Home | Ahh that's unfortunate | 14:45 |
Spaz-Home | It's hard on our hours | 14:45 |
*** jaosorior has joined #openstack-nova | 14:47 | |
mriedem | bhagyashri_s: i've abandoned your spec since it looks, well, abandoned https://review.openstack.org/#/c/511825/ | 14:47 |
mriedem | needs to be moved to the rocky directory structure and incorporate the direction from the PTG | 14:47 |
*** moshele has joined #openstack-nova | 14:47 | |
*** mdbooth has quit IRC | 14:48 | |
mriedem | simondodsley: are you going to move this to rocky? https://review.openstack.org/#/c/340168/ it's proposed against pike | 14:51 |
mriedem | sean-k-mooney: does anyone care about this anymore? https://review.openstack.org/#/c/471815/ | 14:54 |
*** ameeda has quit IRC | 14:54 | |
mriedem | i don't know if baoli is around anymore re: cisco | 14:54 |
mriedem | mgagne: ^ seems like something that would be up your alley | 14:54 |
mriedem | or vdrok ^ i.e. vlan trunk port metadata bliss | 14:56 |
*** eharney has quit IRC | 14:56 | |
openstackgerrit | Nguyen Hai proposed openstack/python-novaclient master: Fix local test fails with pypy https://review.openstack.org/553426 | 14:56 |
*** eharney has joined #openstack-nova | 14:56 | |
openstackgerrit | Nguyen Hai proposed openstack/python-novaclient master: Fix local test fails with pypy https://review.openstack.org/553426 | 14:57 |
*** mdnadeem has quit IRC | 14:58 | |
*** abhishekk has quit IRC | 14:59 | |
*** Spaz-Work has quit IRC | 15:00 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add disabled field to CellMapping object https://review.openstack.org/550090 | 15:00 |
mgagne | mriedem: thanks for the heads-up | 15:00 |
dansmith | mriedem: comments in that rebuild-bfv spec | 15:00 |
mriedem | mgagne: do you still plan on pushing this spec? https://review.openstack.org/#/c/312626/ it's targeted at pike and would need to be moved to rocky and address reviewer feedback | 15:00 |
dansmith | mriedem: I might be the only one, but doing a create, delete and re-pointing at a new volume seems like not a great experience to me | 15:00 |
*** felipemonteiro_ has quit IRC | 15:01 | |
*** felipemonteiro_ has joined #openstack-nova | 15:01 | |
efried | I assume (because I've not seen it done) that it's Wrong to point to specs from comments/docstrings. | 15:01 |
*** beagles is now known as beagles_food | 15:02 | |
efried | Reason I ask: http://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/update-provider-tree.html#computedriver-update-provider-tree contains a lot more information than I want to put into https://github.com/openstack/nova/blob/master/nova/virt/driver.py#L833 | 15:03 |
efried | But that information sure would be useful for people implementing that method. | 15:03 |
mriedem | efried: just say "see the update_provider_tree spec for more details" | 15:03 |
mriedem | or something | 15:04 |
efried | Can I say "rocky" too? | 15:04 |
efried | That'd give them enough info to find it in one shot. | 15:04 |
mriedem | sure | 15:04 |
mriedem | or, move the dev details of that spec into devref | 15:04 |
*** BlackDex has joined #openstack-nova | 15:04 | |
mriedem | https://docs.openstack.org/nova/latest/reference/index.html | 15:05 |
*** Zames has joined #openstack-nova | 15:06 | |
dansmith | mriedem: tested by a real live person: https://review.openstack.org/#/c/552691 | 15:06 |
dansmith | per your ask | 15:06 |
*** sree has joined #openstack-nova | 15:07 | |
*** mdbooth has joined #openstack-nova | 15:07 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add disabled field to CellMapping object https://review.openstack.org/550090 | 15:08 |
*** sree_ has joined #openstack-nova | 15:08 | |
efried | mriedem: And then can I link to the devref, or do I still have to say, "See the [name] section of the devref for details" ? | 15:08 |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219 | 15:08 |
*** beagles_food is now known as beagles | 15:09 | |
*** sree_ is now known as Guest6416 | 15:09 | |
*** Zames has quit IRC | 15:09 | |
*** udesale has quit IRC | 15:10 | |
*** sree has quit IRC | 15:12 | |
mriedem | efried: idk, choose your own adventure | 15:12 |
*** kholkina has quit IRC | 15:12 | |
mriedem | linking to a devref is probably ok, the spec URL is long as hell | 15:12 |
*** jobewan has joined #openstack-nova | 15:13 | |
*** r-daneel_ has joined #openstack-nova | 15:14 | |
*** r-daneel has quit IRC | 15:14 | |
*** r-daneel_ is now known as r-daneel | 15:14 | |
mriedem | dansmith: agree the create/delete volume thing sucks | 15:16 |
mriedem | don't know how easy it would be for cinder to add an api to re-image a volume | 15:16 |
mriedem | smcginnis: ^? | 15:16 |
*** krtaylor has quit IRC | 15:16 | |
smcginnis | So detach, re-lay down an image on the vol, reattach? | 15:17 |
openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/553255 | 15:17 |
mriedem | not sure if we'd need the detach/re-attach part | 15:17 |
*** felipemonteiro__ has joined #openstack-nova | 15:17 | |
dansmith | smcginnis: yeah | 15:17 |
mriedem | but that's what happens during rebuild anyway yeah | 15:17 |
dansmith | mriedem: I assume we would for some backends | 15:17 |
mdbooth | mriedem: Would you mind taking a look at https://review.openstack.org/#/c/551302/ ? I think it's your bailiwick. | 15:17 |
mriedem | is a bailiwick related to a throat sweet? | 15:18 |
*** tidwellr has quit IRC | 15:18 | |
smcginnis | mriedem: I think we would need the detach reattach. | 15:18 |
smcginnis | We need to attach it to a cinder node to reimage it. | 15:19 |
mriedem | smcginnis: that's fine probably b/c we do it anyway for rebuild | 15:19 |
*** hrw has joined #openstack-nova | 15:19 | |
smcginnis | Unless nova wants to take care of that part and leave cinder out of it. | 15:19 |
hrw | morning | 15:19 |
*** tidwellr has joined #openstack-nova | 15:19 | |
smcginnis | Basically just duplicate what we do in cinder. | 15:19 |
*** Spaz-Work has joined #openstack-nova | 15:19 | |
smcginnis | Pull the image to the compute node, write it over the volume, then start up the instance again. | 15:19 |
mriedem | it seems like nova doing things with the internals of the volume is a bad idea | 15:19 |
mdbooth | mriedem: Not especially :) I think there's still a literal bailiwick in the channel islands. | 15:19 |
hrw | can someone help me with understanding how nova compares cpus on live migration? | 15:20 |
dansmith | mriedem: I dunno, it's no different than a regular disk image, | 15:20 |
openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information https://review.openstack.org/312626 | 15:20 |
smcginnis | mriedem: I agree, but it would be an "optimization" to not have to tear things down and start over. | 15:20 |
dansmith | mriedem: the thing is I would expect cinder to maybe be able to do it more efficiently in some cases (i.e. ceph) | 15:20 |
smcginnis | Especially as I'm not sure how that would work with a boot volume. We can't detach those normally, right. | 15:20 |
*** moshele has quit IRC | 15:20 | |
dansmith | smcginnis: we're talking about bfv here | 15:20 |
dansmith | smcginnis: we're doing a rebuild, so we'd detach | 15:20 |
smcginnis | dansmith: Yeah, so that would be an extra complication. | 15:20 |
smcginnis | dansmith: Oh, right! | 15:20 |
openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information https://review.openstack.org/312626 | 15:21 |
mriedem | the volume would remain in-use during all of this from a state pov | 15:21 |
smcginnis | So delete instance, detach volume, cinder rewrites it, reattach, create new "rebuilt" instance using it. | 15:21 |
*** felipemonteiro_ has quit IRC | 15:21 | |
*** yamamoto has joined #openstack-nova | 15:21 | |
mdbooth | dansmith: Could also be a multi-attach volume, btw. | 15:21 |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Add CellMappingList.get_all_enabled() query method https://review.openstack.org/550188 | 15:22 |
mriedem | this is the code that detaches the volume today for rebuild https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L3072 | 15:22 |
dansmith | mdbooth: if people are multi-attaching bfv then I give up | 15:22 |
mriedem | we create a 2nd attachment to keep the volume 'in-use' and then delete the first attachment | 15:22 |
dansmith | well, unless it's read only I guess | 15:22 |
mdbooth | dansmith: It's a thing with real hardware. | 15:22 |
dansmith | but yeah, if we can't undo all the attachments, we'd have to fail I think | 15:23 |
mriedem | before we spawn the guest again, we 'attach' again https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L3103 | 15:23 |
* mdbooth would overwrite it in place regardless. | 15:23 | |
hrw | https://bugs.launchpad.net/nova/+bug/1756118 reported | 15:23 |
openstack | Launchpad bug 1756118 in OpenStack Compute (nova) "No live migration on aarch64" [Undecided,New] | 15:23 |
mdbooth | ... unless the volume was attached RO | 15:23 |
openstackgerrit | Jianghua Wang proposed openstack/os-traits master: GPU: define traits for maximum display heads https://review.openstack.org/553277 | 15:24 |
mriedem | hrw: "nova doesn't support operation x for an architecture that no one started caring about nova supporting until recently" isn't a bug :) | 15:24 |
hrw | mriedem: it works in newton. | 15:25 |
mdbooth | That's a good point. Can you rebuild a server whose root disk is read-only? | 15:25 |
* mdbooth would say no. | 15:25 | |
hrw | mriedem: on newton we have libvirt 2.2.10 and live migration from nova level on aarch64 works. | 15:25 |
hrw | mriedem: on queens we have libvirt 3.8/3.10 and it does not on nova level but does on libvirt level | 15:25 |
mriedem | hrw: ok that's info that should be pointed out in the bug report then | 15:26 |
hrw | will do | 15:26 |
hrw | added | 15:27 |
hrw | mriedem: bug raport is more to signal issue as I will dig into it to find out what is going on | 15:27 |
*** yamamoto has quit IRC | 15:27 | |
openstackgerrit | Merged openstack/os-traits master: Add HW_NIC_SRIOV_TRUSTED trait https://review.openstack.org/550873 | 15:28 |
hrw | mriedem: have to find out where get_connection().CompareCPU() is defined | 15:28 |
hrw | mriedem: as it looks like suspicious place ;d | 15:29 |
mriedem | my guess would be nova.virt.libvirt.host | 15:29 |
*** itlinux has joined #openstack-nova | 15:29 | |
hrw | mriedem: nova.virt.libvirt.host.compare_cpu() calls self.get_connection.compareCPU() | 15:31 |
mriedem | mdbooth: dansmith: i think we'd just say you can't rebuild a volume-backed server with a new image using a multiattach volume if that volume has >1 attachment | 15:32 |
dansmith | right, | 15:32 |
dansmith | at least for the moment | 15:32 |
mriedem | and if it's a leap year because that's a lot of conditionals | 15:32 |
*** chyka has joined #openstack-nova | 15:33 | |
*** sar has joined #openstack-nova | 15:33 | |
*** p_d has quit IRC | 15:34 | |
*** moshele has joined #openstack-nova | 15:37 | |
*** chyka_ has joined #openstack-nova | 15:37 | |
*** chyka has quit IRC | 15:37 | |
*** dave-mccowan has joined #openstack-nova | 15:38 | |
*** moshele has quit IRC | 15:39 | |
mriedem | would like to see another core review the flavor extra specs in GET /flavors/detail spec https://review.openstack.org/#/c/544158/ - discussed at the ptg and it's pretty straight forward | 15:42 |
dansmith | oooh | 15:42 |
dansmith | I found a typo I could use to crush your stats | 15:43 |
*** lpetrut_ has joined #openstack-nova | 15:43 | |
*** lpetrut has quit IRC | 15:43 | |
*** yamahata has quit IRC | 15:43 | |
* penick watches dansmith crush mriedem's dreams | 15:44 | |
dansmith | penick: all while making this face: https://cheappopculture.files.wordpress.com/2015/02/revolting-blob.png?w=825 | 15:46 |
artom_ | As a non-core observer, I feel like singing Sweet Dreams, but with Sweet changed to Crushed, would be appropriate | 15:46 |
openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information https://review.openstack.org/312626 | 15:46 |
penick | hahaha, that got a real laugh outta me | 15:47 |
mriedem | we changed to "extra_spces" in a microversion in queens | 15:47 |
mriedem | you must have missed that | 15:47 |
dansmith | oh mah bad | 15:48 |
penick | Crushed dreams are made of theeeese, who am I to disagree?, travel the world, to the p-t-g, everybody's like minus two | 15:48 |
artom_ | penick, noice | 15:48 |
penick | *fistpump* | 15:49 |
*** lpetrut_ has quit IRC | 15:49 | |
artom_ | Are... are we bros now? Is this what being a bro is like? | 15:49 |
*** ragiman has quit IRC | 15:49 | |
*** gongysh has joined #openstack-nova | 15:51 | |
*** elod is now known as elod_off | 15:51 | |
openstackgerrit | Merged openstack/nova-specs master: Add extra-specs to the flavor show and detail API calls https://review.openstack.org/544158 | 15:51 |
*** kmalloc has joined #openstack-nova | 15:53 | |
cfriesen_ | I've got a question about service delete and placement...currently when we delete a service we don't delete the resource provider(s) for that compute node. If someone adds that compute node back later, this causes accounting issues because it has the same name but a different UUID. | 15:53 |
cfriesen_ | Should we be calling delete_resource_provider() when deleting the service? | 15:53 |
*** BlackDex has quit IRC | 15:54 | |
cdent | cfriesen_: I _think_ there's a bug for that. | 15:54 |
cfriesen_ | https://bugs.launchpad.net/nova/+bug/1708958 is that the one you were thinking of? | 15:56 |
openstack | Launchpad bug 1708958 in OpenStack Compute (nova) "disabling a compute service does not disable the resource provider" [Wishlist,Confirmed] | 15:56 |
*** hamzy has quit IRC | 15:56 | |
mriedem | disable != delete | 15:56 |
cfriesen_ | yeah | 15:57 |
*** ratailor has joined #openstack-nova | 15:57 | |
mriedem | you can delete a service record via the api, | 15:57 |
mriedem | but not the compute node record | 15:57 |
mriedem | deleting the compute node record is the issue | 15:57 |
cfriesen_ | yes | 15:57 |
mriedem | and no we don't have anything in place to clean that stuff up | 15:57 |
mriedem | we don't have any interfaces for deleting compute node records either | 15:57 |
mriedem | we've talked about adding that to nova-manage in the past | 15:57 |
*** dtruong_ has quit IRC | 15:58 | |
penick | artom_ almost. Before becoming bros you have to go through the ceremony of course. | 15:58 |
*** dtruong has joined #openstack-nova | 15:58 | |
mriedem | we have another issue with cleaning up stuff in placement mentioned here https://review.openstack.org/#/c/502306/17/specs/rocky/approved/bandwidth-resource-provider.rst@228 | 15:58 |
cfriesen_ | mriedem: we've got a reported bug that deleting then re-adding a compute node with the same name results in not being able to schedule due to this placement issue | 15:59 |
mriedem | if you disable a compute or it's down, and delete instances from it and then purge them from the db, the allocations for those instances against that compute are never removed from placement | 15:59 |
cfriesen_ | ew | 16:00 |
mriedem | cfriesen_: well, how did they delete the compute node record? directly in the DB? | 16:00 |
mriedem | if you're monkeying with the db, then you are also going to need to deal with cleaning up placement as a result | 16:00 |
mriedem | at least for now, until we have some interface for this | 16:00 |
cfriesen_ | mriedem: I don't think they did delete the comptue node record, just the service | 16:00 |
mriedem | then i don't see how the compute node uuid would change unless the hostname changed or something | 16:01 |
*** ralonsoh_ has joined #openstack-nova | 16:01 | |
mriedem | if the hostname changes, you have other problems | 16:01 |
*** amodi has quit IRC | 16:01 | |
mriedem | i think this also might tie into something tssurya was looking at which is having a way to delete host mappings and also make that recursive to delete compute nodes | 16:02 |
mriedem | and placement entries for those compute nodes | 16:02 |
*** alexchadin has quit IRC | 16:03 | |
mriedem | anyway, there is enough here that we should (1) have a way to cleanup compute nodes and their associated records and (2) that is complicated enough that it probably deserves at least a short spec | 16:03 |
*** harlowja has joined #openstack-nova | 16:03 | |
mriedem | if it were easy it would have been done by now | 16:03 |
tssurya | I am cleaning up compute node and rp records in my patch | 16:03 |
tssurya | when we delete a cell or a host | 16:04 |
dansmith | tssurya: s/host/host mapping/ right? | 16:04 |
*** ralonsoh has quit IRC | 16:04 | |
tssurya | yes | 16:04 |
mriedem | cfriesen_: in the meantime, they can use osc-placement 1.0 to delete the old compute node RP in placement | 16:05 |
cfriesen_ | okay, thanks for the info. tssurya, can you invite me to review that patch? | 16:06 |
mriedem | https://review.openstack.org/#/c/542964/ ? | 16:07 |
cfriesen_ | mriedem: I think that's basically what they were planning on doing for now | 16:07 |
tssurya | https://review.openstack.org/#/c/546660/ | 16:07 |
tssurya | mriedem: its the one I posted, same one needing the functional test | 16:08 |
*** gongysh has quit IRC | 16:08 | |
tssurya | for the placement side | 16:08 |
*** burt has quit IRC | 16:08 | |
tssurya | for deleting the RP record | 16:08 |
tssurya | cfriesen_ : done, I hope I got the name right | 16:09 |
tssurya | for the review invite | 16:09 |
cfriesen_ | on a totally different issue, I opened https://bugs.launchpad.net/nova/+bug/1755981 Basically there seems to be a race due to the fact that we remove serial ports from ALLOCATED_PORTS but leave them in the guest definition. | 16:10 |
openstack | Launchpad bug 1755981 in OpenStack Compute (nova) "powering off and on an instance can result in instance boot failure due to serial port handling race" [Undecided,New] | 16:10 |
cfriesen_ | tssurya: yes, thanks | 16:10 |
mriedem | cfriesen_: probably ask sahid_ about that bug | 16:10 |
mriedem | markus_z would have been good too but he's no longer around | 16:10 |
*** cfriesen_ is now known as cfriesen | 16:11 | |
*** alexchadin has joined #openstack-nova | 16:11 | |
*** amodi has joined #openstack-nova | 16:13 | |
cfriesen | sahid_: (or anyone else) regarding the above bug, why do we remove the port from ALLOCATED_PORTS when powering off? wouldn't it make more sense to remove them when undefining or changing the libvirt domain? The alternative seems to be to detach the serial ports from the libvirt domain when doing a power off, which is kind of icky. | 16:14 |
sahid_ | cfriesen: no comment or todo to explain why? I can't really remember | 16:17 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Resource provider examples https://review.openstack.org/553461 | 16:18 |
sahid_ | cfriesen: i just looked at the code, furing power_on we are doing a hard-reboot (don't ask me why) | 16:18 |
*** belmoreira has quit IRC | 16:18 | |
sahid_ | it seems that during hard reboot we redefine the domain xml | 16:19 |
*** sidx64 has quit IRC | 16:19 | |
*** Guest6416 has quit IRC | 16:20 | |
*** sree has joined #openstack-nova | 16:20 | |
*** jaosorior has quit IRC | 16:21 | |
*** yamahata has joined #openstack-nova | 16:22 | |
*** BlackDex has joined #openstack-nova | 16:23 | |
*** yamamoto has joined #openstack-nova | 16:23 | |
*** sree has quit IRC | 16:25 | |
*** hongbin has joined #openstack-nova | 16:25 | |
*** burt has joined #openstack-nova | 16:27 | |
*** masber has joined #openstack-nova | 16:28 | |
*** yamamoto has quit IRC | 16:29 | |
*** AlexeyAbashkin has quit IRC | 16:32 | |
artom_ | jaypipes, wanna talk about https://review.openstack.org/#/c/552722/1/specs/rocky/approved/live-migration-with-cpu-pinning.rst@16? | 16:34 |
*** burt has quit IRC | 16:35 | |
openstackgerrit | Chris Dent proposed openstack/nova-specs master: Spec for isolating configuration of placement database https://review.openstack.org/552927 | 16:37 |
*** moshele has joined #openstack-nova | 16:37 | |
cdent | stephenfin: addressed your suggestions on ^ | 16:37 |
*** andreas_s has quit IRC | 16:38 | |
*** gongysh has joined #openstack-nova | 16:38 | |
*** gongysh has quit IRC | 16:39 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/542964 | 16:41 |
*** ccamacho1 has quit IRC | 16:41 | |
mriedem | tssurya: would you like to propose a backport to stable/queens for ^ ? | 16:42 |
*** andreas_s has joined #openstack-nova | 16:43 | |
tssurya | mriedem: yes I will do it | 16:43 |
*** lpetrut has joined #openstack-nova | 16:43 | |
openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 16:45 |
*** janki has quit IRC | 16:46 | |
cfriesen | sahid_: I'm back. So on power_off() we _destroy() the instance (but leave it defined) and remove the serial ports from ALLOCATED_PORTS. Then on power_on() we hard reboot the instance, which calls _destroy() again, which removes the ports from ALLOCATED_PORTS again, which might result in removing ports currently in use by another instance. | 16:46 |
jaypipes | artom_: sure, what's up? | 16:46 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 16:47 |
cfriesen | sahid: I assume we call _hard_reboot() to clean up as many things as possible about the instance (given the comment by mdbooth in _hard_reboot()) | 16:47 |
openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.{add|remove}_{traits|aggregates} https://review.openstack.org/553475 | 16:47 |
efried | dansmith, jaypipes, cdent, edleafe: One half of the result of the "need to be able to merge traits/aggs" discussion ^ | 16:48 |
*** alexchadin has quit IRC | 16:48 | |
cfriesen | sahid_: I'm wondering whether we should do the serial port removal from ALLOCATED_PORTS in _undefine_domain() instead. | 16:48 |
openstackgerrit | Eric Fried proposed openstack/nova master: update_provider_tree devref and docstring updates https://review.openstack.org/553476 | 16:49 |
efried | dansmith, jaypipes, cdent, edleafe, mriedem: The other half ^ | 16:49 |
efried | ...and now to send out that dev ML note, so I don't get in trouble with mriedem... | 16:49 |
*** artom_ is now known as artom | 16:50 | |
tssurya | mriedem: there is a small problem with the backport, I guess the above patch sits on this one -> https://review.openstack.org/#/c/540073/ , so will backport both | 16:51 |
artom | jaypipes, so, what I think I failed in communicating is that getting NUMA resources modelled in placement and claimed by the scheduler (Sylvain's spec) is a dependency of my spec | 16:51 |
artom | jaypipes, and what I *think* you're saying is that, we can continue using the current compute-node-claims-the-resources way for now | 16:51 |
*** gbarros has quit IRC | 16:52 | |
*** andreas_s has quit IRC | 16:52 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Extending delete_cell --force to delete instance_mappings https://review.openstack.org/553478 | 16:53 |
sahid_ | cfriesen: i think you also have to look at the method where we define domain xml | 16:53 |
edleafe | efried: looking... | 16:53 |
*** wolverineav has joined #openstack-nova | 16:53 | |
*** masber has quit IRC | 16:53 | |
*** wolverineav has quit IRC | 16:53 | |
*** wolverineav has joined #openstack-nova | 16:53 | |
sahid_ | cfriesen: get_config_xml or somethinf | 16:54 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 16:54 |
jaypipes | artom: I don't think it's necessary to depend on bauzas' spec. I think that would endanger making progress on fixing live migration for NUMA/pinning | 16:54 |
artom | jaypipes, right, so leave the NUMA RPs thing to itself, and just fix the pin mappings for now. I'd tend to agree, for what it's worth - feels like a more incremental step, and not a major overhaul | 16:56 |
jaypipes | artom: ++ | 16:57 |
cfriesen | sahid_: in the _hard_reboot() case we call destroy(), which does _destroy() and then cleanup(), and that will undefine the instance. Arguably that's the point where we should remove the TCP port from ALLOCATED_PORTS. Then we generate new xml with new TCP ports and all should be happy. | 16:57 |
*** lpetrut has quit IRC | 16:57 | |
jaypipes | artom: and all I was asking was that you make a little more explicit in the spec what is and what isn't "claimed". | 16:57 |
cfriesen | sahid_: removing the TCP ports from ALLOCATED_PORTS while they're still defined in the domain is just asking for trouble. | 16:58 |
artom | jaypipes, totally fair. | 16:58 |
openstackgerrit | Merged openstack/nova stable/pike: doc: fix the link for the evacuate cli https://review.openstack.org/542856 | 16:58 |
artom | jaypipes, err, you're using "claimed" because the compute node doesn't actually talk to placement to "claim" the pinned pCPUs, right? | 16:58 |
sahid_ | cfriesen: if you can make all of that better i will be happy to review any of the your patches | 17:01 |
jaypipes | artom: no, I'm saying that "claim resources" means something very specific in the scheduler -- it is the call to placement to PUT /allocations/{instance_uuid}. And that does *not* include any NUMA resources right now. So I want the spec to be clear about that. When you say "claim in the scheduler", that's not actually what happens. The "claim on the compute" is the old way of allocating resources from the compute node to the instance by writing | 17:01 |
jaypipes | the record to the compute_nodes cell DB table. that is still done for NUMA and PCI resources in the resource tracker's instance_claim() method. | 17:01 |
sahid_ | dansmith: i updated the patch related to live-migration, the point is to have if possible something like an agrement on one of the version so i could make it tested internally | 17:02 |
*** gjayavelu has joined #openstack-nova | 17:03 | |
*** lpetrut has joined #openstack-nova | 17:03 | |
artom | jaypipes, thanks for setting me straight :) | 17:03 |
dansmith | sahid_: I already said the implementation looks right, barring the gaps in testing | 17:03 |
cfriesen | sahid_: cool, if I get some time I'll hold you to that. :) | 17:04 |
cfriesen | jaypipes: artom: this spec only talks about CPU pinning, but we also need to recalculate the destination NUMA node for hugepage-backed instances even without CPU pinning. | 17:05 |
sahid_ | dansmith: i would like avoid any difference, so if you have a moment please have a look in the last version | 17:05 |
sahid_ | cfriesen: :) | 17:05 |
*** ralonsoh__ has joined #openstack-nova | 17:06 | |
artom | cfriesen, jaypipes, so should we just extend this to NUMA live migration, all the while keeping the old claim on the compute way of allocating resources? | 17:06 |
dansmith | sahid_: you didn't answer my question about the neutron events in the tests | 17:07 |
claudiub|2 | dansmith: hellou. Ehm, I saw that we don't allow certain DB operations in nova (drops and alters). I'm trying to add an item to an enum, but afaik, that requires an alter. Or is there a better way to do it? | 17:07 |
*** mdbooth has quit IRC | 17:07 | |
*** ratailor has quit IRC | 17:08 | |
*** tidwellr has quit IRC | 17:08 | |
dansmith | claudiub|2: yeah, we banned alters because they're not (usually) additive and doable online.. I think we had one in the past we exempted because we confirmed with jaypipes that it was lightweight.. does yours fit that description? | 17:08 |
cfriesen | artom: yes, I think it's really NUMA-aware live migration. | 17:08 |
claudiub|2 | nope, I'm trying to add an item to the Migration.migration_type enum. | 17:09 |
*** ralonsoh_ has quit IRC | 17:09 | |
dansmith | claudiub|2: oh just adding something to an existing enum? | 17:09 |
*** harlowja has quit IRC | 17:09 | |
claudiub|2 | yep | 17:09 |
dansmith | jaypipes: ^ hopefully that is not a big deal to do online? | 17:10 |
claudiub|2 | this is the commit: https://review.openstack.org/#/c/185961/4 but postgresql seems unhappy about it. | 17:10 |
jaypipes | dansmith: no, it's virtually instantaneous | 17:10 |
dansmith | jaypipes: ack | 17:10 |
dansmith | claudiub|2: so I think there should be at least one more exception in whatever test that is, so you can copy that for yours I think | 17:10 |
artom | cfriesen, not a bad idea :) | 17:11 |
jaypipes | honestly, we really shouldn't be using the ENUM type anyway... but meh | 17:11 |
artom | So, I have to bounce of a lunch thing, and ideally I'd have liked more discussion about this before rewriting the spec, but I think I'll just bite the bullet and rewrite the spec :) | 17:11 |
dansmith | jaypipes: yeah I'm not a fan myself | 17:12 |
*** hamzy has joined #openstack-nova | 17:13 | |
claudiub|2 | dansmith: ack, done that, but it seems like postgresql is dropping the column on alter, for some reason, and this exception is raised: oslo_db.exception.DBError: (psycopg2.InternalError) cannot drop type migration_type because other objects depend on it | 17:13 |
claudiub|2 | DETAIL: table migrations column migration_type depends on type migration_type | 17:13 |
cfriesen | artom: I'm just updating the review with some comments right now. gimme a couple minutes | 17:13 |
dansmith | claudiub|2: I don't think I can help you with that one :) | 17:14 |
dansmith | claudiub|2: but I don't think it's any of our doing | 17:14 |
claudiub|2 | I sea. | 17:15 |
*** dtantsur is now known as dtantsur|afk | 17:15 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553496 | 17:16 |
claudiub|2 | jaypipes: do you have any ideas? ^ | 17:16 |
*** ralonsoh__ has quit IRC | 17:16 | |
*** hamzy_ has joined #openstack-nova | 17:16 | |
*** gouthamr has quit IRC | 17:18 | |
openstackgerrit | Merged openstack/nova master: Update deprecated log-config option in docs https://review.openstack.org/551825 | 17:18 |
*** gouthamr has joined #openstack-nova | 17:18 | |
*** hamzy has quit IRC | 17:19 | |
mriedem | efried: how do i get an endpoint url from a ksa adapter object? | 17:20 |
*** fragatina has joined #openstack-nova | 17:21 | |
efried | mriedem: stand by. | 17:21 |
cfriesen | artom: okay, updated the review with some extra info | 17:21 |
*** fragatina has quit IRC | 17:21 | |
efried | mriedem: .get_endpoint() | 17:21 |
*** tidwellr has joined #openstack-nova | 17:22 | |
*** fragatina has joined #openstack-nova | 17:22 | |
* efried applauds apt method name choice | 17:24 | |
sean-k-mooney | mriedem: exposing the vlans for trunk ports in metadata, i think the kuryr team did but honest answer is i dont know | 17:25 |
*** yamamoto has joined #openstack-nova | 17:25 | |
tssurya | dansmith, mriedem: since I had to rebase this regression (https://review.openstack.org/#/c/550967/) bug on top of the new purge command got merged before this, does this mean we will have to backport that new command into queens too ? | 17:25 |
*** felipemonteiro__ has quit IRC | 17:25 | |
*** felipemonteiro__ has joined #openstack-nova | 17:26 | |
dansmith | tssurya: generally just fix up the backport and note the reason for the conflicts in the commit message | 17:26 |
*** hamzy__ has joined #openstack-nova | 17:26 | |
dansmith | I wouldn't be too opposed to backporting the purge stuff personally, but it's technically not a candidate | 17:26 |
tssurya | dansmith: okay thanks | 17:26 |
tssurya | yea I can fix up the backport | 17:27 |
tssurya | I didn't know if it was an allowed practice | 17:27 |
dansmith | tssurya: if you survey some other backports you'll see some "Conflicts:" examples | 17:27 |
tssurya | dansmith: thanks will look them up | 17:27 |
dansmith | tssurya: example: https://review.openstack.org/#/c/540145/ | 17:27 |
jaypipes | claudiub|2: added review note. | 17:28 |
tssurya | dansmith: perfect thank you! | 17:28 |
claudiub|2 | \o/ thanks. :D | 17:28 |
*** liverpooler has quit IRC | 17:29 | |
*** hamzy_ has quit IRC | 17:29 | |
*** yamamoto has quit IRC | 17:30 | |
*** krtaylor has joined #openstack-nova | 17:36 | |
*** moshele has quit IRC | 17:37 | |
*** liverpooler has joined #openstack-nova | 17:42 | |
*** imacdonn has quit IRC | 17:44 | |
*** imacdonn has joined #openstack-nova | 17:44 | |
*** r-daneel_ has joined #openstack-nova | 17:50 | |
*** r-daneel has quit IRC | 17:51 | |
*** r-daneel_ is now known as r-daneel | 17:51 | |
*** Matias has quit IRC | 17:52 | |
*** suresh12 has joined #openstack-nova | 17:54 | |
*** slaweq_ has quit IRC | 17:55 | |
*** slaweq has joined #openstack-nova | 17:55 | |
*** lucasagomes is now known as lucas-afk | 17:56 | |
cfriesen | are there any known issues with nic tagging in Newton? | 17:56 |
stephenfin | artom: ^ ? | 17:57 |
cfriesen | I'm failing schema validation, wondering if it's something we screwed up but I don't see us changing anything in that area. | 17:57 |
cfriesen | (by "we" I mean my organization) | 17:57 |
*** felipemonteiro_ has joined #openstack-nova | 17:57 | |
cdent | jaypipes: you might enjoy this buglet: https://bugs.launchpad.net/nova/+bug/1756151 | 17:58 |
openstack | Launchpad bug 1756151 in OpenStack Compute (nova) "placement os-traits sync checked every request" [Low,Triaged] | 17:58 |
*** mgoddard_ has quit IRC | 17:58 | |
*** suresh12 has quit IRC | 17:59 | |
*** felipemonteiro__ has quit IRC | 17:59 | |
*** moshele has joined #openstack-nova | 18:01 | |
cfriesen | stephenfin: artom: ah yes, microversion 2.37 broke tagging, and 2.42 added it back in. | 18:03 |
*** mvenesio has joined #openstack-nova | 18:03 | |
*** derekh has quit IRC | 18:04 | |
*** mvk has quit IRC | 18:05 | |
*** david-lyle has joined #openstack-nova | 18:05 | |
*** suresh12 has joined #openstack-nova | 18:07 | |
mriedem | melwitt: have you heard anything new about a project update session at the summit? | 18:13 |
*** jpena is now known as jpena|off | 18:14 | |
melwitt | mriedem: no, I'm gonna ask anne about it | 18:15 |
mriedem | nova project update doesn't fall under the category of CI/CD, HPC or EDGE so probably not | 18:15 |
mriedem | i replied to the ML thread | 18:15 |
mriedem | quite dickishly | 18:15 |
melwitt | heh. I tried to find them in the summit schedule and found none | 18:16 |
mriedem | same | 18:16 |
mriedem | nova project update: feature k8s! | 18:17 |
mriedem | *featuring | 18:17 |
mriedem | damn | 18:17 |
*** moshele has quit IRC | 18:17 | |
*** harlowja has joined #openstack-nova | 18:20 | |
jaypipes | efried, edleafe: where are we on your battling microversion changes? | 18:20 |
efried | jaypipes: It's tied up on the home stretch while zuul unwinds its panties. | 18:21 |
efried | See topic | 18:21 |
jaypipes | k | 18:21 |
*** alexchadin has joined #openstack-nova | 18:21 | |
*** sridharg has quit IRC | 18:22 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345 | 18:24 |
mriedem | efried: i've tried to rebase this but there are some known broken things in it ^ | 18:24 |
efried | mriedem: Yeah, I had to put it aside for other "more urgent" things. | 18:25 |
efried | I swear I had it working at some point in the cycle. | 18:26 |
*** moshele has joined #openstack-nova | 18:26 | |
efried | but by the end, it was definitely busted and I couldn't figure out how to fix it without spending a big chunk of time. | 18:26 |
*** yamamoto has joined #openstack-nova | 18:27 | |
openstackgerrit | Claudiu Belu proposed openstack/nova master: compute: Adds instance live-resize https://review.openstack.org/248581 | 18:28 |
openstackgerrit | Claudiu Belu proposed openstack/nova master: db: Adds live-resize to Migration model migration_type https://review.openstack.org/185961 | 18:28 |
*** burt has joined #openstack-nova | 18:28 | |
*** david-lyle has quit IRC | 18:29 | |
*** moshele has quit IRC | 18:30 | |
*** wolverineav has quit IRC | 18:31 | |
*** wolverineav has joined #openstack-nova | 18:31 | |
*** jmlowe has quit IRC | 18:31 | |
*** yamamoto has quit IRC | 18:33 | |
*** tidwellr has quit IRC | 18:33 | |
*** david-lyle has joined #openstack-nova | 18:33 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add disabled field to CellMapping object https://review.openstack.org/550090 | 18:34 |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Add CellMappingList.get_all_enabled() query method https://review.openstack.org/550188 | 18:34 |
*** wolverineav has quit IRC | 18:35 | |
*** jmlowe has joined #openstack-nova | 18:36 | |
*** wolverineav has joined #openstack-nova | 18:37 | |
*** david-lyle has quit IRC | 18:37 | |
*** tesseract has quit IRC | 18:39 | |
*** mgoddard_ has joined #openstack-nova | 18:39 | |
cdent | nice message efried | 18:40 |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add CellMappingList.get_all_enabled() query method https://review.openstack.org/550188 | 18:40 |
efried | Thanks cdent. I guess if anyone was gonna bother to read the whole thing, it'd be you :) | 18:40 |
cdent | yeah, spose so | 18:41 |
cdent | I think you'll find there's a vast army of silent readers out there | 18:41 |
cdent | their silence can sometimes be rather disturbing | 18:41 |
*** felipemonteiro_ has quit IRC | 18:42 | |
*** felipemonteiro has joined #openstack-nova | 18:42 | |
*** felipemonteiro_ has joined #openstack-nova | 18:43 | |
*** alexchadin has quit IRC | 18:44 | |
efried | cdent: I'm quite a slow reader, and I often feel that pain when something's important enough that I know I gotta read it, but really long (like dhellman's missive on requirements). I guess for people who read at normal speeds, it's not such an onerous task. | 18:46 |
cdent | efried: yeah, I've been reminded many times that my attitude towards reading comes from something of a position of privilege. Apparently I read _very_ fast when it comes to email and similar forms. | 18:47 |
efried | That explains a lot. | 18:47 |
*** felipemonteiro has quit IRC | 18:47 | |
efried | I envy those who can read fast and not miss stuff. That's why I read slow - because I'm being real thorough (terrified of missing some detail or - gods forbid - failing to catch a typo!) | 18:48 |
edleafe | heh, just started reading efried's email | 18:49 |
*** tssurya has quit IRC | 18:49 | |
* efried clicks stopwatch | 18:49 | |
edleafe | An hourglass would be good enough :) | 18:50 |
*** mvk has joined #openstack-nova | 18:51 | |
cdent | efried: I'm certain that I miss stuff, but I'm usually grazing for meaning, not details | 18:52 |
efried | I should develop that skill. FOMO. | 18:52 |
cdent | maybe not, probably useful to have both styles around | 18:53 |
mriedem | mgoddard_: stephenfin: dansmith: so maybe the numa topology filter is ok with ironic http://logs.openstack.org/12/553412/1/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/2b781af/logs/screen-n-sch.txt.gz#_Mar_15_15_24_11_093172 | 18:53 |
mriedem | that devstack change didn't blow up | 18:53 |
dansmith | sweet | 18:53 |
cdent | for me it's in part a learned skill to intentionally miss out on some stuff | 18:53 |
*** AlexeyAbashkin has joined #openstack-nova | 18:54 | |
*** moshele has joined #openstack-nova | 18:54 | |
mriedem | could have sworn something about the IronicNodeState object was different such that the filter would fail with it | 18:55 |
mriedem | also, it's not like we actually have pci requests in these CI jobs | 18:55 |
edleafe | I tried to learn to skim back in college. I always felt like I never got anything out of it, so I went back to reading details | 18:55 |
edleafe | efried: excellent recap | 18:55 |
dansmith | mriedem: that's true, I guess a flavor or request with pci or numa could break if there are ironic hosts in there somehow, | 18:55 |
dansmith | but I dunno what that would be really | 18:56 |
* cdent goes away for a while | 18:56 | |
efried | edleafe: Thanks. | 18:56 |
mriedem | dansmith: the other thing might have been something to do with allocation ratios, but grasping at straws | 18:56 |
*** beagles is now known as beagles|biab | 18:57 | |
dansmith | we don't have ratios for those types though | 18:57 |
mriedem | goes into the numa topology limits | 18:58 |
mriedem | the cpu and ram allocation ratois | 18:58 |
*** AlexeyAbashkin has quit IRC | 18:58 | |
cfriesen | mriedem: we have a private patch to enable ironic and regular nodes...had to make some of the scheduler filters check the hypervisor type. | 18:58 |
mriedem | cfriesen: is there anything you guys don't have a private patch for? | 18:59 |
cfriesen | mriedem: we try to upstream stuff, but it takes forever | 18:59 |
mgoddard_ | mriedem: that's good news! | 18:59 |
*** moshele has quit IRC | 18:59 | |
cfriesen | mriedem: plus, we only need to worry about one hypervisor | 19:00 |
mriedem | mgoddard_: well, it's kind of a false positive i think | 19:00 |
mriedem | cfriesen: it takes even longer when you don't even propose them | 19:00 |
dansmith | mriedem: it's not a false positive, it's just not comprehensive.. it means something, it just doesn't mean it all works fine :) | 19:00 |
mriedem | i think host_topology_and_format_from_host might be the thing | 19:01 |
mriedem | that's just always None for ironic i think | 19:02 |
cfriesen | mriedem: yeah, I know. I don't control how much upstreaming time I get. Looks like we modified NUMATopologyFilter to check the hypervisor type, and modified AggregateInstanceExtraSpecsFilter to ignore the "baremetal" and "storage" keys. | 19:02 |
dansmith | mriedem: if that breaks if it's none, that probably also precludes using any two hypervisors together where one is libvirt with numa and the other doesn't support it, right? | 19:02 |
mriedem | nvm, that would just filter out that ironic node | 19:03 |
mriedem | / host | 19:03 |
openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: Resize https://review.openstack.org/553583 | 19:03 |
mriedem | cfriesen: if you could figure out *why* you "Looks like we modified NUMATopologyFilter to check the hypervisor type" that would be nice | 19:03 |
mriedem | because at some point i thought these wouldn't work but i can't figure out why now | 19:04 |
dansmith | yeah at least upstreaming the bug would be worthwhile | 19:04 |
mriedem | ++ | 19:04 |
dansmith | although they do that, so if you did in this case, then .. cool :) | 19:04 |
cfriesen | mriedem: let me check with the author | 19:04 |
*** esberglu has quit IRC | 19:07 | |
*** jamesdenton has quit IRC | 19:08 | |
*** esberglu has joined #openstack-nova | 19:09 | |
*** artom_ has joined #openstack-nova | 19:10 | |
sean-k-mooney | mriedem: cfriesen perhaps because you wanted to avoid qemu hosts when there are numa requests? | 19:10 |
*** artom has quit IRC | 19:10 | |
mgoddard_ | perhaps the bug is just that the NUMA filter doesn't work for ironic, rather than that it rejects all bare metal hosts? | 19:11 |
cfriesen | sean-k-mooney: don't think so, this was specifically part of allowing one nova-scheduler to handle both libvirt/kvm and ironic compute nodes | 19:12 |
sean-k-mooney | cfriesen: ah ok | 19:12 |
*** esberglu_ has joined #openstack-nova | 19:12 | |
sean-k-mooney | mgoddard_: well if i ask for a 2 numa node instance the ironic should be able to select a node with 2 numa nodes however i dont think ironic adds numa info into the compute nodes table for the filter to use | 19:13 |
mgoddard_ | exactly | 19:13 |
mgoddard_ | same with cpu pinning | 19:14 |
sean-k-mooney | mgoddard_: no cpu pinning is different | 19:14 |
mgoddard_ | and hyperthreading | 19:14 |
cfriesen | mriedem: back to the nova-compute service delete issue, when deleting a service via the API, nova.db.sqlalchemy.api.service_destroy() will soft-delete both the service and the service and the compute_node entry, but placement is still around. Then we create the compute node again and get new service and compute_node entries with a different uuid but the same hostname. | 19:14 |
sean-k-mooney | cpu pinning does not make sense in a phyical server context | 19:14 |
dansmith | it wouldn't matter these days anyway, ironic nodes report a CUSTOM_IRONIC_FOO resource and no cpu/mem | 19:14 |
*** esberglu has quit IRC | 19:14 | |
mgoddard_ | sean-k-mooney: ok, you're right about pinning. hyperthreading could (but doesn't) work though | 19:15 |
dansmith | cfriesen: mriedem because we store service_id in the compute node, so we won't find the existing one when we re-create the service | 19:15 |
dansmith | cfriesen: mriedem I bet that api was never updated when we added the node concept.. it probably needs to delete the node(s) as well when it does that | 19:16 |
sean-k-mooney | mgoddard_: hyperthreading is a bios config option and should work but nova does not allow you to enable hyperthreading as a flavor extra spec | 19:16 |
dansmith | (and thus placement( | 19:16 |
sean-k-mooney | mgoddard_: the closest you have to that is setting the tread count in the cpu topology extra specs | 19:16 |
*** david-lyle has joined #openstack-nova | 19:17 | |
mriedem | i didn't realize that service_destroy also deleted the related compute node record | 19:17 |
cfriesen | me either...had to go read the code. | 19:17 |
*** hemna_ has joined #openstack-nova | 19:18 | |
dansmith | cfriesen: I thought you said it didn't? | 19:18 |
mriedem | ok so if service delete also deletes the compute node record, then yeah we should also remove the compute node RP in placement | 19:18 |
mriedem | i said deleting the service didn't delete the compute node record | 19:19 |
dansmith | oh okay | 19:19 |
mriedem | there was some bug about a year ago we were both talking about this | 19:19 |
mriedem | but https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L404 | 19:19 |
cfriesen | I should be able to open a bug and post a WIP fix later today | 19:21 |
mriedem | https://github.com/openstack/nova/commit/f0d44c5b09f3f3c84038d40b621bb629a1f8110e#diff-3104166b3e802b86db6c5fa92ad08f43 | 19:21 |
mriedem | ^ is why i thought this | 19:21 |
mriedem | see the exchange between myself and alex | 19:23 |
mriedem | so in this case, they deleted the service (and compute node record) but didn't stop the nova-compute service, | 19:24 |
mriedem | so it re-created the compute node record | 19:24 |
*** hongbin has quit IRC | 19:25 | |
mriedem | so in that bug, when they listed compute nodes, the api tries to find the related service which was deleted and the api blows up | 19:25 |
mriedem | because you don't recreate the service until you restart the service | 19:25 |
*** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Rocky_Release_Schedule" | 19:25 | |
-openstackstatus- NOTICE: The regression stemming from one of yesterday's Zuul security fixes has been rectified, and Devstack/Tempest jobs which encountered POST_FAILURE results over the past 24 hours can safely be rechecked now | 19:25 | |
melwitt | gibi: I'm working on the nova/neutron session summary, would be helpful if you could fill in any gaps for the bandwidth-based scheduling agreements/decisions when you get a chance https://etherpad.openstack.org/p/nova-ptg-rocky-neutron-summary | 19:26 |
mriedem | melwitt: he's on vacation | 19:26 |
*** hongbin has joined #openstack-nova | 19:26 | |
melwitt | urgh right | 19:27 |
melwitt | got focused only on the april vacation dates | 19:27 |
mriedem | cfriesen: yeah so to summarize that, deleting the service deletes the compute node record, and the compute node record shouldn't show back up unless the service is running | 19:27 |
dansmith | ...and we should delete the RP in placement | 19:28 |
mriedem | what about the host mapping? | 19:28 |
mriedem | that's also orphaned right? | 19:28 |
dansmith | we probably should delete it in placement before we delete in our db even | 19:28 |
dansmith | yeah | 19:28 |
mriedem | so, | 19:28 |
*** yamamoto has joined #openstack-nova | 19:29 | |
mriedem | 1. delete RP in placement (because if we fail to delete the compute node or the service, it will get re-created), | 19:29 |
mriedem | 2. delete the service/node, | 19:29 |
mriedem | 3. delete the host mapping | 19:29 |
mriedem | ? | 19:29 |
dansmith | well, 1. also because if we fail, they can't do it again if we delete the local entry | 19:29 |
*** frickler has quit IRC | 19:29 | |
dansmith | technically, we can find the service record again if we delete the host mapping, | 19:30 |
dansmith | so probably good to delete that first, then node, then service last of all | 19:30 |
cfriesen | what's the host mapping used for? | 19:30 |
dansmith | so placement, mapping, node, service | 19:30 |
dansmith | cfriesen: finding it by name in a cell | 19:30 |
mriedem | cfriesen: to map the host | 19:30 |
cfriesen | ah, right | 19:30 |
mriedem | node and service are in the same db api call, | 19:30 |
*** frickler has joined #openstack-nova | 19:30 | |
mriedem | unless we delete the compute node record(s?) directly in the api | 19:30 |
mriedem | which would be my preference b/c the auto/implicit delete in the db api is kind of gross | 19:31 |
dansmith | same db call, but node could go first | 19:32 |
dansmith | but yeah, don't so much care about those two, | 19:32 |
dansmith | as I'm not sure there are any FKs that would prevent us from deleting one and not the other | 19:32 |
*** yamamoto has quit IRC | 19:33 | |
mriedem | compute_nodes.service_id doesn't have a FK on it | 19:33 |
dansmith | yeah | 19:33 |
cfriesen | dansmith: what are the implications of a stale host mapping? | 19:34 |
*** damien_r has quit IRC | 19:34 | |
dansmith | cfriesen: cruft | 19:34 |
dansmith | with no easy way to find and clean it up later | 19:34 |
cfriesen | got it, thanks | 19:35 |
mriedem | also, | 19:35 |
mriedem | i think you could re-discover the compute host in another cell, | 19:35 |
dansmith | true | 19:35 |
mriedem | and then the old host mapping would be pointed at the wrong cell | 19:35 |
dansmith | yeah | 19:35 |
mriedem | i feel like cern mentioned that | 19:35 |
claudiub|2 | did they express a con...cern? | 19:36 |
cfriesen | ew | 19:36 |
mriedem | cfriesen: i think this block is where you add the new deletes https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/services.py#L213 | 19:36 |
dansmith | claudiub|2: nice :) | 19:36 |
mriedem | claudiub|2: excellent | 19:36 |
claudiub|2 | :D | 19:36 |
*** felipemonteiro_ has quit IRC | 19:40 | |
*** felipemonteiro_ has joined #openstack-nova | 19:40 | |
*** ircuser-1 has joined #openstack-nova | 19:41 | |
*** beagles|biab is now known as beagles | 19:42 | |
cfriesen | mriedem: you think that's better than https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4820 ? | 19:48 |
mriedem | cfriesen: could happen in compute api code too, doesn't matter a ton to me | 19:49 |
mriedem | cfriesen: note that this will also need an upgrade impact release note because the api will now require placement to be configured to do this | 19:49 |
*** mgoddard_ has quit IRC | 19:50 | |
mriedem | i think we should backport this though...so not sure how sticky that gets | 19:50 |
cfriesen | mriedem: if placement isn't configured, wouldn't it be okay to just continue on since there wouldn't be any records? | 19:50 |
mriedem | i mean [placement] configured specifically in the nova.conf you're running against nova-api, | 19:51 |
mriedem | which hasn't technically required placement before this | 19:51 |
mriedem | as far as i know | 19:51 |
mriedem | conductor, compute and scheduler all require placement already, but api doesnt | 19:51 |
cfriesen | ah, got it. I guess that'd be one advantage of doing it in nova-compute | 19:51 |
mriedem | there is no rpc cast from api to compute to delete itself | 19:51 |
*** tssurya has joined #openstack-nova | 19:52 | |
cfriesen | yeah, got confused by the nova/compute/api.py path for a sec....too many things going on today | 19:52 |
*** mvenesio has quit IRC | 19:53 | |
*** dave-mccowan has quit IRC | 19:53 | |
*** fragatina has quit IRC | 19:55 | |
*** suresh12 has quit IRC | 19:57 | |
*** suresh12 has joined #openstack-nova | 19:57 | |
*** Zames has joined #openstack-nova | 19:58 | |
cdent | jaypipes, efried, edleafe: on consumer generations. does that need an owner? is anything depending on it? I figure I can probably do it if it isn't needed asap (got forbidden first) | 19:59 |
efried | cdent: It does need an owner afaik. And it is not immediately urgent afaik. Thank you for volunteering. | 19:59 |
*** suresh12 has quit IRC | 19:59 | |
efried | mriedem: ^ | 19:59 |
*** suresh12 has joined #openstack-nova | 19:59 | |
jaypipes | cdent: needs an owner, yes. | 20:00 |
edleafe | I can pair with someone on that | 20:01 |
cfriesen | mriedem: on the mixed baremetal/hypervisor scheduler thing, I suspect the NUMATopologyFilter is actually okay as long as we don't have spec_obj.numa_topology | 20:01 |
*** eharney has quit IRC | 20:02 | |
*** Zames has quit IRC | 20:03 | |
mriedem | cfriesen: in that case the filter should just filter out an ironic host/node | 20:03 |
mriedem | but that's what i was trying to figure out | 20:03 |
mriedem | i.e. if NUMATopologyFilter just doesn't work with an all ironic deployment, then don't enable it, but if you're using a single scheduler for mixed VM/BM, that filter would also kick out requests for numa for ironic hosts in the deployment, | 20:04 |
mriedem | but i think that should be ok? | 20:04 |
cfriesen | mriedem: as long as requests for ironic instances don't specify a desired numa topology then I think it should be good | 20:04 |
mriedem | if we have a libvirt and ironic host as candidates, the ironic would is filtered out and the libvirt one should be ok | 20:04 |
cdent | edleafe: have you got something queued up after member_of? | 20:04 |
mriedem | cfriesen: well yeah, baremetal flavors shouldn't have numa things in them | 20:05 |
edleafe | in OpenStack, just the api-sig stuff: http and version discovery | 20:05 |
*** jmlowe has quit IRC | 20:06 | |
cdent | do you want something immediately queued up? If so you can be primary on consumer generations and I can be your buddy. Otherwise I can start it up in a few weeks | 20:06 |
*** lpetrut has quit IRC | 20:06 | |
edleafe | ok, I guess I'll grab it, and anyone who wants in on the fun is welcome to join | 20:07 |
openstackgerrit | Jay Pipes proposed openstack/nova master: mirror nova host aggregate members to placement https://review.openstack.org/553597 | 20:08 |
jaypipes | thx edleafe | 20:08 |
* edleafe goes to figure out just how much crap he's volunteered for... | 20:09 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add a note in DELETE /os-services about deleting computes https://review.openstack.org/553598 | 20:10 |
mriedem | cfriesen: ^ while i'm remembering this compute delete thing | 20:10 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 20:11 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add a note in DELETE /os-services about deleting computes https://review.openstack.org/553598 | 20:11 |
*** dave-mccowan has joined #openstack-nova | 20:11 | |
*** sahid_ has quit IRC | 20:15 | |
openstackgerrit | sean mooney proposed openstack/nova master: add mtu to libvirt xml for ethernet and bridge types https://review.openstack.org/553072 | 20:24 |
*** yamamoto has joined #openstack-nova | 20:29 | |
*** liverpooler has quit IRC | 20:30 | |
*** cdent has quit IRC | 20:30 | |
*** mvenesio has joined #openstack-nova | 20:31 | |
*** yamamoto has quit IRC | 20:35 | |
tssurya | cfriesen: just read the service-delete stuff ; its also been on our to-do list; so thanks for doing it. | 20:37 |
cfriesen | tssurya: just filed https://bugs.launchpad.net/nova/+bug/1756179 | 20:38 |
openstack | Launchpad bug 1756179 in OpenStack Compute (nova) "deleting a nova-compute service leaves orphaned records in placement" [Undecided,New] | 20:38 |
tssurya | cfriesen: ack | 20:39 |
mriedem | cfriesen: are you also going to work on the patch? | 20:42 |
*** tbachman has quit IRC | 20:43 | |
openstackgerrit | Michael Still proposed openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921 | 20:43 |
openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep. https://review.openstack.org/552241 | 20:43 |
openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242 | 20:43 |
openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605 | 20:43 |
openstackgerrit | Lance Bragstad proposed openstack/nova master: Add scope_types to server policies https://review.openstack.org/525772 | 20:44 |
lbragstad | cc mriedem melwitt ^ new version availble re the RBAC cross-project sessions from dublin | 20:45 |
lbragstad | let me know if you'd like to go through it (i know mriedem mentioned something about reviewing it as a group) - i'm happy to help answer questions | 20:45 |
melwitt | thanks for the heads up | 20:48 |
cfriesen | tssurya: for your review at https://review.openstack.org/#/c/546660 should we also delete the "service" records along with the compute node records? | 20:49 |
cfriesen | tssurya: if you want to submit a patch for that bug, go for it. I probably won't get to it till late next week at the earliest | 20:49 |
tssurya | cfriesen : yea, sure I can work on it | 20:50 |
tssurya | regarding removal of "service" records, when I guess it makes sense to do when force deleting the cell and host | 20:51 |
tssurya | mriedem, dansmith : ^^ | 20:51 |
openstackgerrit | Lance Bragstad proposed openstack/nova master: Add scope_types to server policies https://review.openstack.org/525772 | 20:51 |
mriedem | this gets quite messy | 20:52 |
mriedem | because deleting a compute service in the api today also removes the compute host from any aggregates it's in | 20:52 |
tssurya | yes it will get messy, | 20:52 |
*** dave-mccowan has quit IRC | 20:53 | |
cfriesen | if you're deleting a cell or a host, doesn't it make sense to delete everything logically associated with that cell/host? | 20:53 |
mriedem | sure, but, | 20:53 |
mriedem | where does it end | 20:53 |
*** mvk has quit IRC | 20:53 | |
tssurya | I think it does make sense to do it, but to put it all in nova-manage... it gets dirtier and more complex | 20:54 |
mriedem | if only we had, oh idk, a rest api for managing cells.... | 20:54 |
tssurya | mriedem: :) | 20:55 |
dansmith | oh yeah, that'd make it so much better | 20:55 |
mriedem | it would solve all problems | 20:56 |
mriedem | cells api could be the new secretary of state | 20:56 |
mriedem | RIP REX | 20:56 |
melwitt | nova meeting in 3 minutes | 20:57 |
*** sree has joined #openstack-nova | 20:58 | |
*** s1061123 has quit IRC | 20:58 | |
dansmith | tssurya: fwiw, my preference is that we not put as much stuff in cmd.manage as we have now, but break things out into modules like all the rest of our code, and make cmd.manage just the cli interface to those things | 21:00 |
dansmith | cmd.manage grew up over time, getting larger and larger, with people caring less about it because "meh, it's just nova-manage" | 21:00 |
*** s1061123 has joined #openstack-nova | 21:00 | |
tssurya | I totally agree that it is getting messier | 21:00 |
*** fragatina has joined #openstack-nova | 21:00 | |
tssurya | (mostly because of me in the recent months) | 21:01 |
mriedem | let's blame belmiro | 21:01 |
tssurya | hehe yea! | 21:01 |
*** sree has quit IRC | 21:02 | |
*** edmondsw has quit IRC | 21:02 | |
openstackgerrit | Lance Bragstad proposed openstack/nova master: Set scope_types for admin_action policies https://review.openstack.org/553612 | 21:06 |
*** mvk has joined #openstack-nova | 21:06 | |
*** moshele has joined #openstack-nova | 21:08 | |
openstackgerrit | Lance Bragstad proposed openstack/nova master: Set scope_types for admin password policies https://review.openstack.org/553613 | 21:16 |
*** sar has quit IRC | 21:22 | |
*** r-daneel_ has joined #openstack-nova | 21:23 | |
*** liverpooler has joined #openstack-nova | 21:23 | |
*** r-daneel has quit IRC | 21:24 | |
*** r-daneel_ is now known as r-daneel | 21:24 | |
*** mikal_ has joined #openstack-nova | 21:30 | |
*** awaugama has quit IRC | 21:31 | |
*** yamamoto has joined #openstack-nova | 21:31 | |
*** mikal has quit IRC | 21:32 | |
openstackgerrit | Claudiu Belu proposed openstack/nova master: db: Adds live-resize to Migration model migration_type https://review.openstack.org/185961 | 21:33 |
*** yamamoto has quit IRC | 21:36 | |
*** itlinux has quit IRC | 21:39 | |
*** dave-mccowan has joined #openstack-nova | 21:39 | |
*** itlinux has joined #openstack-nova | 21:41 | |
*** itlinux has quit IRC | 21:41 | |
*** r-daneel_ has joined #openstack-nova | 21:41 | |
*** r-daneel has quit IRC | 21:41 | |
*** r-daneel_ is now known as r-daneel | 21:41 | |
*** esberglu_ has quit IRC | 21:41 | |
*** gjayavelu has quit IRC | 21:43 | |
*** esberglu has joined #openstack-nova | 21:44 | |
*** felipemonteiro__ has joined #openstack-nova | 21:45 | |
*** esberglu_ has joined #openstack-nova | 21:45 | |
*** gjayavelu has joined #openstack-nova | 21:45 | |
*** gjayavelu has joined #openstack-nova | 21:46 | |
*** amodi has quit IRC | 21:47 | |
*** pchavva has quit IRC | 21:47 | |
*** esberglu has quit IRC | 21:48 | |
*** felipemonteiro_ has quit IRC | 21:49 | |
*** esberglu_ has quit IRC | 21:50 | |
*** gjayavelu has quit IRC | 21:54 | |
*** mikal_ is now known as mikal | 21:55 | |
mriedem | Kevin_Zheng_: you reported the bug related to this https://blueprints.launchpad.net/nova/+spec/add-server-group-remove-member-notifications but for telemetry and searchlight - given the states of those projects, do they actually care about solving this now? | 21:59 |
mriedem | https://bugs.launchpad.net/nova/+bug/1639152 | 21:59 |
openstack | Launchpad bug 1639152 in OpenStack Compute (nova) "Send out notification about server group changes when delete instances" [Wishlist,Confirmed] - Assigned to Takashi NATSUME (natsume-takashi) | 21:59 |
dansmith | mriedem: it requires conf | 22:00 |
dansmith | so I would kinda expect not | 22:00 |
dansmith | we will backport it because it's important enough, but I don't know that it fits the usual backport candidate mold | 22:00 |
mriedem | we can backport bug fixes sometimes as long as the config doesn't change any default behavior | 22:00 |
*** tbachman has joined #openstack-nova | 22:00 | |
dansmith | it should be safe from that regard | 22:00 |
dansmith | it's really a feature used to fix a bug, IMHO, | 22:01 |
mriedem | by "we will backport" you mean the royal we? | 22:01 |
dansmith | so a bp isn't wrong I don't think | 22:01 |
dansmith | mriedem: we as in redhat | 22:01 |
mriedem | that's what i meant :) | 22:01 |
*** moshele has quit IRC | 22:01 | |
mriedem | alright i guess people that want it in stable can always patch out of tree if we're not going to backport upstream | 22:02 |
*** danpawlik has joined #openstack-nova | 22:02 | |
dansmith | I'm not saying we can't or shouldn't | 22:02 |
mriedem | i heard windriver has already had this for a year | 22:02 |
*** rcernin has joined #openstack-nova | 22:02 | |
dansmith | I would have expected you to say no | 22:02 |
dansmith | if you think it's legit, then I think it's reasonable | 22:02 |
mriedem | having a blueprint tag on it makes it more awkward to backport | 22:02 |
*** danpawlik has quit IRC | 22:02 | |
dansmith | well, | 22:02 |
mriedem | i don't have the context on the thing, | 22:02 |
dansmith | it's not a bug in nova | 22:02 |
mriedem | just saw 'meltdown' | 22:02 |
dansmith | it's a bug in .. I dunno.. the whole arrangement of things | 22:03 |
*** tssurya has quit IRC | 22:03 | |
mriedem | in computers? | 22:03 |
mriedem | just say computers | 22:03 |
dansmith | nova doesn't let you specify a thing that only the operator can know, and is needed to mitigate performance loss after applying the meltdown fixes | 22:03 |
dansmith | it's completely a nova feature, | 22:03 |
dansmith | but operators will need it to fix a bug or regressed behavior in lots of other things (kernel, qemu, etc) | 22:03 |
mriedem | hmm, this reminds me, | 22:04 |
mriedem | stable/ocata and stable/pike started crapping recently (right before the ptg) because of qemu / libvirt package updates for CVEs related to meltdown and spectre in the UCA | 22:04 |
dansmith | so I guess I'd leave it as a bp and just call it out in the backport as why we're making an exception | 22:04 |
mriedem | mnaser: ^ | 22:04 |
mriedem | you might be interested in that | 22:05 |
mriedem | https://review.openstack.org/#/c/534384/ | 22:05 |
*** liverpooler has quit IRC | 22:05 | |
* mnaser looks | 22:05 | |
mriedem | mnaser had to patch devstack on ocata and pike specifically to wait longer for nova-compute to start up b/c libvirt startup was going from like 10 seconds to a minute | 22:06 |
mriedem | dansmith: ^ sound like the same issue? | 22:06 |
dansmith | libvirt startup .. meaning a guest? | 22:06 |
*** burt has quit IRC | 22:06 | |
mriedem | no, libvirtd | 22:06 |
mriedem | or the connection to libvirt | 22:06 |
*** liverpooler has joined #openstack-nova | 22:06 | |
dansmith | mm, I dunno, that's a question for kashyap I think | 22:07 |
dansmith | maybe if the fixes have been applied on the host, | 22:07 |
dansmith | but pcid isn't passed it makes our guests just super slow for some things | 22:07 |
mriedem | https://review.openstack.org/#/c/547676/ | 22:07 |
dansmith | so it could be | 22:07 |
mriedem | https://launchpad.net/~ubuntu-cloud-archive/+archive/ubuntu/ocata-staging/+sourcepub/8774271/+listing-archive-extra | 22:07 |
mriedem | the libvirt cve is i nthere | 22:07 |
dansmith | I mean, that's what the assertion is about the problem we're trying to solve | 22:07 |
dansmith | that _some_ things are like 30+% slower after the fixes | 22:07 |
*** gjayavelu has joined #openstack-nova | 22:08 | |
mnaser | so the reason why it affected ocata and pike was because first of all devstack didnt use pike uca in pike, it used ocata uca | 22:08 |
mnaser | and the older release of uca didn't have this feature where it could ask qemu what cpu features it supported | 22:08 |
mnaser | so on libvirt start up with older libvirt instances, it would boot up a very small tiny qemu instance to probe it for things like cpu features etc | 22:09 |
*** tssurya has joined #openstack-nova | 22:09 | |
dansmith | dear god | 22:09 |
mnaser | which would make the start up super sloooow | 22:09 |
*** masber has joined #openstack-nova | 22:09 | |
dansmith | so that's unrelated I believe, unless the need for the new cpu feature discovery was because of the meltdown patch, | 22:09 |
dansmith | but I expect kashyap's fix won't solve that as it's a qemu-missing-a-thing problem | 22:10 |
mriedem | ok | 22:10 |
*** gouthamr has quit IRC | 22:16 | |
*** danpawlik has joined #openstack-nova | 22:17 | |
*** tssurya has quit IRC | 22:19 | |
*** claudiub|2 has quit IRC | 22:20 | |
*** tbachman has quit IRC | 22:21 | |
mriedem | dansmith: if you guys have vgpu people internally besides bauzas that can take a look at this https://bugs.launchpad.net/nova/+bug/1752463 | 22:21 |
openstack | Launchpad bug 1752463 in OpenStack Compute (nova) "Attaching virtual GPU devices to guests in nova" [Medium,Incomplete] | 22:22 |
*** danpawlik has quit IRC | 22:22 | |
*** mvenesio has quit IRC | 22:22 | |
*** mvenesio has joined #openstack-nova | 22:23 | |
melwitt | mriedem, dansmith: so we're agreed to approve https://blueprints.launchpad.net/nova/+spec/libvirt-cpu-model-extra-flags as a bp and we'll backport it, calling out in the backport why we're making an exception and backporting bp work. is that right? | 22:26 |
dansmith | melwitt: I'm cool with that yeah | 22:26 |
*** felipemonteiro__ has quit IRC | 22:27 | |
dansmith | mriedem: maybe, but that seems pretty low priority and can just wait for him to be back right? | 22:27 |
*** mvenesio has quit IRC | 22:27 | |
*** Guest79046 has quit IRC | 22:28 | |
mriedem | dansmith: sure | 22:28 |
*** gjayavelu has quit IRC | 22:30 | |
*** hemna_ has quit IRC | 22:30 | |
*** yamamoto has joined #openstack-nova | 22:32 | |
*** gjayavelu has joined #openstack-nova | 22:34 | |
melwitt | mriedem: are you cool with the cpu model flags bp and the idea of backporting it? | 22:37 |
mriedem | i'm not crazy about it being a bp if we know we're going to backport it | 22:37 |
mriedem | i'd rather pretend it's not and say it's a bug fix | 22:38 |
*** yamamoto has quit IRC | 22:38 | |
mriedem | https://review.openstack.org/#/c/534384/5//COMMIT_MSG@28 definitely makes it sound feature-y too though | 22:39 |
mriedem | "besides the fix, you can also configure nova for other goodies!" | 22:39 |
melwitt | heh. yeah. this is definitely an odd case | 22:39 |
mriedem | now, if pcid as the extra flag is the only thing you'd care about if you only care about the fix, | 22:40 |
mriedem | then this could be a [workarounds] option with a boolean to set pcid or not, | 22:41 |
mriedem | and backport that | 22:41 |
mriedem | then the free-form wild west extra flags thing that is more of a feature could only be master | 22:41 |
mriedem | and we'd deprecate the workaround in master | 22:41 |
mriedem | dansmith: ^ what do you think about that? | 22:42 |
dansmith | I dunno, it seems like too much concern over a relatively minor thing | 22:44 |
dansmith | and I kinda expect that we'll backport the main thing ourselves anywaym | 22:44 |
*** mvenesio has joined #openstack-nova | 22:44 | |
dansmith | just so that people with queens and people with rocky don't need a different fix for this | 22:44 |
dansmith | but if that's how it has to go, then okay | 22:44 |
mriedem | i'm not the ptl, not my say, but i'd be more comfortable with that | 22:46 |
* mriedem watches that bus go by | 22:46 | |
dansmith | I guess my point is, | 22:46 |
dansmith | we're going to backport the meat of the feature anyway, because we need it for the workaround | 22:46 |
dansmith | and constraining the workaround to one flag doesn't really do anything to make it safer, | 22:47 |
dansmith | unless the operator is going to just do crazy stuff with it (not sure what that would be even), | 22:47 |
mriedem | ^ is what i'm worried about, they try the 'other use cases' in stable, | 22:47 |
mriedem | and are all of a sudden reporting bugs about it | 22:47 |
dansmith | but if they're going to do that, | 22:47 |
dansmith | they'll do it with rocky when it's out | 22:47 |
*** dave-mccowan has quit IRC | 22:48 | |
dansmith | if we have a workaround, then people have to set that to make the thing work, and then when they roll to rocky (or S) and performance regresses for some reason they forgot about, | 22:48 |
dansmith | there will be bugs where we tell them that that workaround was a one-time deal and now they have to set pcid in this other field | 22:48 |
dansmith | just seems like it's too much monkeywork so we don't have a blueprint on a backport, | 22:48 |
mriedem | well, we'd use the workaround if it were set while it's deprecated, but we'd mark it deprecated to drop it later, but you'd have rocky to transition your tooling | 22:48 |
dansmith | when we can clearly point to not-our-fault reasons why this is needed and not high risk | 22:48 |
dansmith | yeah, it just doesn't seem worth the pain to me | 22:49 |
*** chyka has joined #openstack-nova | 22:49 | |
mriedem | precedent is another concern for me here, | 22:49 |
mriedem | true it's a no-op by default, | 22:49 |
cfriesen | dansmith: I agree with you, but I also suspect someone is going to try enabling pdpe1gb or vmx on queens. | 22:49 |
mriedem | but then people could use that justification against us later | 22:50 |
dansmith | cfriesen: so? | 22:50 |
melwitt | the workaround idea seems good until we get to the point where people would have to go from workaround -> full feature to not regress performance | 22:50 |
cfriesen | dansmith: so mriedem's point about it likely causing more bug reports against stable | 22:50 |
*** suresh12 has quit IRC | 22:50 | |
*** chyka_ has quit IRC | 22:50 | |
mriedem | melwitt: it's just updating your configs in rocky | 22:50 |
dansmith | cfriesen: but those bug reports will be against whatever release we allow the full feature in | 22:50 |
mriedem | to not use the deprecated option | 22:50 |
*** hongbin has quit IRC | 22:50 | |
dansmith | mriedem: every time we do that it's upgrade pain and it just seems like we have too much of that already | 22:51 |
cfriesen | dansmith: not necessarily, nested vmx for example is buggy in older systems but may be stable by the time rocky is released | 22:51 |
melwitt | mriedem: but wouldn't you also have to add "pcid" to your rocky config elsewhere then? | 22:51 |
dansmith | I get the precedent thing | 22:51 |
mriedem | melwitt: yes, you'd do it on the new option | 22:51 |
dansmith | cfriesen: that's a super weak reason :) | 22:51 |
dansmith | cfriesen: because (a) I don't think it will be and (b) people can already do crazy stuff with lots of options | 22:52 |
dansmith | cfriesen: you could work around this by just defining your own cpu models in the libvirt system xml, and you can do way crazier stuff to your systems that way | 22:52 |
cfriesen | dansmith: true enough. for what it's worth, adding pdpe1gb seems to be generally fine. | 22:52 |
dansmith | heh | 22:53 |
cfriesen | for the subset of intel hosts I've dealt with anyway | 22:53 |
*** salv-orl_ has quit IRC | 22:53 | |
*** salv-orlando has joined #openstack-nova | 22:54 | |
melwitt | we can just slap "experimental" on it on the backport, use at your own risk | 22:54 |
dansmith | it's not experimental but sure :) | 22:54 |
dansmith | melwitt: I need to be done for the day anyway, | 22:55 |
melwitt | well, what I mean is if you use it and it breaks something in queens/pike/ocata then we're probably not going to fix things to accommodate the various flags they enable | 22:55 |
dansmith | so you make the call, my input is all above | 22:55 |
dansmith | melwitt: it won't be any less experimental later, | 22:55 |
dansmith | it just lets you choose cpu flags | 22:56 |
dansmith | it will be just as dangerous in five years if you use it wrong | 22:56 |
melwitt | okay | 22:56 |
* dansmith & | 22:56 | |
*** salv-orlando has quit IRC | 22:59 | |
*** mvenesio has quit IRC | 22:59 | |
*** masber has quit IRC | 23:02 | |
*** suresh12 has joined #openstack-nova | 23:05 | |
*** jobewan has quit IRC | 23:08 | |
*** sorrison_ has quit IRC | 23:14 | |
*** sticker has joined #openstack-nova | 23:17 | |
*** amodi has joined #openstack-nova | 23:18 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Stop assuming initial provider generation is 0 https://review.openstack.org/548975 | 23:20 |
mriedem | yikun: Kevin_Zheng_: replied with my opinion on the limited affinity spec options for how to model the limit in the API https://review.openstack.org/#/c/546925/2/specs/rocky/approved/allow-specifying-limit-for-affrinity-group.rst@49 | 23:21 |
efried | jaypipes, alex_xu_: Trivial pep/tox fixes, ready for re+A --^ | 23:21 |
*** chyka has quit IRC | 23:25 | |
*** yamamoto has joined #openstack-nova | 23:27 | |
*** hamzy__ has quit IRC | 23:29 | |
*** crushil has joined #openstack-nova | 23:31 | |
*** danpawlik has joined #openstack-nova | 23:35 | |
*** rcernin has quit IRC | 23:39 | |
*** danpawlik has quit IRC | 23:39 | |
*** mlavalle has quit IRC | 23:40 | |
*** chyka has joined #openstack-nova | 23:47 | |
*** tbachman has joined #openstack-nova | 23:48 | |
alex_xu_ | efried: done | 23:48 |
*** chyka has quit IRC | 23:51 | |
*** salv-orlando has joined #openstack-nova | 23:54 | |
*** crushil has quit IRC | 23:55 | |
*** david-lyle has quit IRC | 23:59 | |
*** salv-orlando has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!