Thursday, 2018-03-15

*** sean-k-m1 has joined #openstack-nova00:02
jaypipesefried: +2 from me.00:02
*** gjayavelu has joined #openstack-nova00:03
*** sean-k-mooney has quit IRC00:04
*** amodi has quit IRC00:06
openstackgerritMatt Riedemann proposed openstack/nova master: Make nova-cells-v1 run with neutron  https://review.openstack.org/54978900:06
*** yamamoto has joined #openstack-nova00:07
*** tetsuro has joined #openstack-nova00:17
*** liverpooler has quit IRC00:17
*** xinliang has quit IRC00:22
*** Tom-Tom has joined #openstack-nova00:25
*** odyssey4me has quit IRC00:29
*** odyssey4me has joined #openstack-nova00:29
*** Tom-Tom has quit IRC00:30
*** xinliang has joined #openstack-nova00:34
mriedemjaypipes: that vmware resource pool thing is basically equivalent to AZ:HOST:NODE:CLUSTER:POOL right?00:35
*** abalutoiu has quit IRC00:39
*** tbachman has joined #openstack-nova00:40
*** chyka has joined #openstack-nova00:43
*** moshele has joined #openstack-nova00:43
*** stakeda has joined #openstack-nova00:44
*** wolverin_ has quit IRC00:46
*** wolverineav has joined #openstack-nova00:46
*** chyka has quit IRC00:47
*** suresh12 has quit IRC00:47
*** suresh12 has joined #openstack-nova00:48
*** r-daneel has quit IRC00:48
*** Dinesh_Bhor has joined #openstack-nova00:50
*** wolverineav has quit IRC00:51
*** jichen has joined #openstack-nova00:52
*** suresh12 has quit IRC00:52
*** suresh12 has joined #openstack-nova00:55
*** Dinesh_Bhor has quit IRC00:56
*** zhaochao has joined #openstack-nova00:57
*** suresh12 has quit IRC01:00
*** phuongnh has joined #openstack-nova01:01
*** dave-mccowan has joined #openstack-nova01:05
*** jmlowe has quit IRC01:08
*** jmlowe has joined #openstack-nova01:10
*** tiendc has joined #openstack-nova01:12
*** moshele has quit IRC01:14
*** mriedem has quit IRC01:24
*** gjayavelu has quit IRC01:24
*** Dinesh_Bhor has joined #openstack-nova01:24
*** nicolasbock has quit IRC01:29
*** suresh12 has joined #openstack-nova01:29
*** fragatina has quit IRC01:32
*** Tom-Tom has joined #openstack-nova01:32
*** andreas_s has joined #openstack-nova01:33
*** suresh12 has quit IRC01:34
*** Tom-Tom_ has joined #openstack-nova01:35
*** r-daneel has joined #openstack-nova01:37
*** Tom-Tom has quit IRC01:37
*** andreas_s has quit IRC01:37
*** tianhui has joined #openstack-nova01:39
*** Tom-Tom_ has quit IRC01:41
*** nicolasbock has joined #openstack-nova01:41
*** Tom-Tom has joined #openstack-nova01:42
*** yamahata has quit IRC01:43
*** yufei has joined #openstack-nova01:45
*** fragatina has joined #openstack-nova01:46
*** hongbin has joined #openstack-nova01:46
*** OctopusZhang__ has joined #openstack-nova01:52
*** yufei has quit IRC01:52
*** OctopusZhang has joined #openstack-nova01:53
*** hshiina has joined #openstack-nova01:53
*** fragatina has quit IRC01:54
*** claudiub|2 has quit IRC01:55
*** OctopusZhang__ has quit IRC01:57
*** harlowja has quit IRC01:59
*** lei-zh has joined #openstack-nova02:00
*** salv-orl_ has joined #openstack-nova02:04
*** namnh has joined #openstack-nova02:05
*** salv-orlando has quit IRC02:07
*** Zames has joined #openstack-nova02:11
openstackgerritTetsuro Nakamura proposed openstack/nova master: Fix allocation_candidates not to ignore shared RPs  https://review.openstack.org/53339602:12
openstackgerritTetsuro Nakamura proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs  https://review.openstack.org/51960102:12
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support relay RP for allocation candidates  https://review.openstack.org/53343702:12
openstackgerritTetsuro Nakamura proposed openstack/nova master: remove not necessary short cut  https://review.openstack.org/55312202:12
*** Tom-Tom has quit IRC02:13
*** Tom-Tom has joined #openstack-nova02:13
*** Tom-Tom_ has joined #openstack-nova02:14
*** Zames has quit IRC02:15
*** r-daneel has quit IRC02:15
*** r-daneel has joined #openstack-nova02:16
*** Tom-Tom has quit IRC02:17
*** p_d has joined #openstack-nova02:18
*** p_d has joined #openstack-nova02:20
*** david-lyle has quit IRC02:21
openstackgerritTetsuro Nakamura proposed openstack/nova master: Fix comments in get_all_with_shared()  https://review.openstack.org/53319502:21
*** mingyu_ has joined #openstack-nova02:23
*** mingyu has quit IRC02:25
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events  https://review.openstack.org/54327702:26
*** zhurong has joined #openstack-nova02:29
*** chyka has joined #openstack-nova02:32
*** itlinux has quit IRC02:32
*** Tom-Tom_ has quit IRC02:36
*** phuongnh has quit IRC02:37
*** chyka has quit IRC02:37
*** phuongnh has joined #openstack-nova02:40
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies  https://review.openstack.org/54692502:42
*** naichuans has joined #openstack-nova02:43
*** Tom-Tom has joined #openstack-nova02:44
*** Dinesh_Bhor has quit IRC02:44
*** Dinesh_Bhor has joined #openstack-nova02:45
*** psachin has joined #openstack-nova02:46
*** wolverineav has joined #openstack-nova02:49
*** Spaz-Work has joined #openstack-nova02:59
openstackgerritmelanie witt proposed openstack/nova master: only increment disk address unit for scsi devices  https://review.openstack.org/53831003:04
*** bkopilov has quit IRC03:06
*** Tom-Tom has quit IRC03:08
*** Tom-Tom has joined #openstack-nova03:09
*** wolverineav has quit IRC03:10
*** sree has joined #openstack-nova03:10
*** wolverineav has joined #openstack-nova03:10
*** p_d has quit IRC03:13
*** dave-mccowan has quit IRC03:14
*** germs has quit IRC03:14
*** wolverineav has quit IRC03:15
*** germs has joined #openstack-nova03:15
*** germs has quit IRC03:15
*** germs has joined #openstack-nova03:15
*** germs has quit IRC03:16
*** germs has joined #openstack-nova03:17
*** germs has quit IRC03:17
*** germs has joined #openstack-nova03:17
*** fragatin_ has joined #openstack-nova03:20
*** annp has joined #openstack-nova03:26
*** nicolasbock has quit IRC03:27
*** felipemonteiro_ has joined #openstack-nova03:32
*** tojuvone_ is now known as tojuvone03:32
*** felipemonteiro__ has joined #openstack-nova03:33
*** janki has joined #openstack-nova03:34
*** felipemonteiro_ has quit IRC03:36
*** hshiina has quit IRC03:40
*** hshiina has joined #openstack-nova03:40
*** links has joined #openstack-nova03:41
*** Tom-Tom has quit IRC03:45
*** Tom-Tom has joined #openstack-nova03:45
*** felipemonteiro__ has quit IRC03:45
*** felipemonteiro__ has joined #openstack-nova03:46
*** hongbin has quit IRC03:52
*** zhurong has quit IRC03:53
*** OctopusZhang has quit IRC04:08
*** germs has quit IRC04:09
*** harlowja has joined #openstack-nova04:11
*** Swami has quit IRC04:14
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380504:19
*** bkopilov has joined #openstack-nova04:21
*** sridharg has joined #openstack-nova04:29
*** andreas_s has joined #openstack-nova04:31
openstackgerritBrooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS  https://review.openstack.org/53316804:33
Spaz-Work^Ignore, just a rebase, will fix tests in a bit04:33
*** gyee has quit IRC04:35
*** andreas_s has quit IRC04:36
*** gjayavelu has joined #openstack-nova04:42
*** abhishekk has joined #openstack-nova04:44
*** lpetrut has joined #openstack-nova04:46
*** harlowja has quit IRC04:50
*** yamahata has joined #openstack-nova04:50
openstackgerritGiridhar Jayavelu proposed openstack/nova-specs master: VMware: place instances on resource pool  https://review.openstack.org/54906704:59
*** vivsoni has joined #openstack-nova05:00
*** jmlowe has quit IRC05:01
*** felipemonteiro__ has quit IRC05:01
*** Dinesh_Bhor has quit IRC05:02
*** Dinesh_Bhor has joined #openstack-nova05:03
*** vivsoni has quit IRC05:08
*** lpetrut has quit IRC05:11
*** alexchadin has joined #openstack-nova05:11
*** fragatin_ has quit IRC05:11
*** udesale has joined #openstack-nova05:13
*** ratailor has joined #openstack-nova05:14
*** alexchadin has quit IRC05:17
*** alexchadin has joined #openstack-nova05:17
*** gjayavelu has quit IRC05:29
*** mdnadeem has joined #openstack-nova05:39
*** moshele has joined #openstack-nova05:47
*** suresh12 has joined #openstack-nova05:48
*** claudiub|2 has joined #openstack-nova05:48
*** OctopusZhang has joined #openstack-nova05:51
*** moshele has quit IRC05:52
*** suresh12 has quit IRC05:53
*** anadahz1BP3JC has joined #openstack-nova05:56
anadahz1BP3JCTHIS 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
anadahz1BP3JCTHIS 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
anadahz1BP3JCTHIS 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
anadahz1BP3JCTHIS 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
anadahz1BP3JCOctopusZhang 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 jber05:56
*** anadahz1BP3JC has quit IRC05:56
Spaz-WorkAigoo..05:56
*** OctopusZhang has quit IRC05:58
openstackgerritjichenjc proposed openstack/nova master: Move test_allocation_candidates.py  https://review.openstack.org/55314906:01
openstackgerritjichenjc proposed openstack/nova master: Move test_allocation_candidates.py  https://review.openstack.org/55315006:05
Spaz-WorkSpent 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.. Monnndayyyyysssssss06:08
*** chyka has joined #openstack-nova06:09
*** Dinesh_Bhor has quit IRC06:10
*** OctopusZhang has joined #openstack-nova06:10
*** Dinesh_Bhor has joined #openstack-nova06:10
*** sidx64 has joined #openstack-nova06:12
openstackgerritjichenjc proposed openstack/nova master: Move test_resource_provider.py  https://review.openstack.org/55315006:13
openstackgerritjichenjc proposed openstack/nova master: Move test_resource_class_cache.py  https://review.openstack.org/55315306:13
*** chyka has quit IRC06:13
*** sidx64 has quit IRC06:13
*** Dinesh_Bhor has quit IRC06:16
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/54877206:18
*** Eran_Kuris has joined #openstack-nova06:19
*** OctopusZhang has quit IRC06:21
*** sidx64 has joined #openstack-nova06:24
*** cfriesen_ has joined #openstack-nova06:26
*** cfriesen has quit IRC06:27
*** yamahata has quit IRC06:28
*** OctopusZhang has joined #openstack-nova06:30
*** kholkina has joined #openstack-nova06:34
*** ccamacho has quit IRC06:39
*** sidx64_ has joined #openstack-nova06:39
*** Dinesh_Bhor has joined #openstack-nova06:40
*** sidx64 has quit IRC06:41
*** sidx64 has joined #openstack-nova06:42
*** sidx64_ has quit IRC06:44
*** Eran_Kuris has quit IRC06:53
*** namnh has quit IRC06:53
*** moshele has joined #openstack-nova06:58
openstackgerritjichenjc proposed openstack/nova master: Move test_resource_provider.py  https://review.openstack.org/55315006:59
openstackgerritjichenjc proposed openstack/nova master: Move test_resource_class_cache.py  https://review.openstack.org/55315306:59
*** Eran_Kuris has joined #openstack-nova07:08
*** sidx64_ has joined #openstack-nova07:12
*** sidx64 has quit IRC07:13
*** alexchadin has quit IRC07:14
*** OctopusZhang has quit IRC07:14
*** OctopusZhang has joined #openstack-nova07:14
*** OctopusZhang is now known as yufei07:14
*** OctopusZhang__ has joined #openstack-nova07:15
*** yufei has quit IRC07:15
*** OctopusZhang__ is now known as yufei07:15
*** sidx64 has joined #openstack-nova07:15
*** sidx64_ has quit IRC07:17
*** gaoyan has joined #openstack-nova07:17
*** OctopusZhang_ has joined #openstack-nova07:18
*** gaoyan has quit IRC07:19
*** alexchadin has joined #openstack-nova07:19
*** sar has joined #openstack-nova07:21
*** yufei has quit IRC07:22
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/55321107:26
*** Tom-Tom_ has joined #openstack-nova07:27
*** andreas_s has joined #openstack-nova07:27
*** ccamacho has joined #openstack-nova07:28
*** ccamacho has quit IRC07:29
*** Tom-Tom_ has quit IRC07:29
*** ccamacho has joined #openstack-nova07:29
*** Tom-Tom_ has joined #openstack-nova07:29
*** Tom-Tom has quit IRC07:30
*** phuongnh has quit IRC07:33
*** Tom-Tom_ has quit IRC07:33
*** alexchadin has quit IRC07:41
*** gaoyan has joined #openstack-nova07:42
*** alexchadin has joined #openstack-nova07:42
*** sahid has joined #openstack-nova07:43
*** pcaruana has joined #openstack-nova07:48
*** lpetrut has joined #openstack-nova07:49
*** pcaruana has quit IRC07:50
*** pcaruana has joined #openstack-nova07:50
*** pcaruana is now known as pcaruana|wfh|07:53
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/55325508:01
*** sidx64 has quit IRC08:02
*** OctopusZhang__ has joined #openstack-nova08:03
*** OctopusZhang_ has quit IRC08:03
*** Amit82 has joined #openstack-nova08:04
*** sidx64 has joined #openstack-nova08:04
Amit82Hi All08:05
Amit82Can LXC container orchestration be done using HEAT?08:05
*** OctopusZhang_ has joined #openstack-nova08:06
*** OctopusZhang__ has quit IRC08:09
*** OctopusZhang_ is now known as yufei08:09
*** AlexeyAbashkin has joined #openstack-nova08:13
*** sshwarts has joined #openstack-nova08:19
*** sidx64 has quit IRC08:21
openstackgerritJianghua Wang proposed openstack/os-traits master: GPU: define traits for display heads  https://review.openstack.org/55327708:23
*** sahid_ has joined #openstack-nova08:23
*** sahid_ has quit IRC08:23
*** sahid has quit IRC08:23
*** sahid has joined #openstack-nova08:23
*** tesseract has joined #openstack-nova08:24
*** tssurya has joined #openstack-nova08:40
*** gjayavelu has joined #openstack-nova08:42
*** jpena|off is now known as jpena08:43
*** sidx64 has joined #openstack-nova08:48
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380508:50
*** ralonsoh has joined #openstack-nova08:52
*** hieulq has joined #openstack-nova08:58
*** rcernin has quit IRC09:00
*** tssurya has quit IRC09:02
*** ispp is now known as burned09:04
*** r-daneel has quit IRC09:07
*** slunkad has quit IRC09:10
*** cdent has joined #openstack-nova09:12
*** lucas-afk is now known as lucasagomes09:12
*** alexchadin has quit IRC09:12
*** alexchadin has joined #openstack-nova09:13
*** alexchadin has quit IRC09:13
*** slunkad has joined #openstack-nova09:14
*** alexchadin has joined #openstack-nova09:14
*** alexchadin has quit IRC09:14
*** alexchadin has joined #openstack-nova09:15
*** alexchadin has quit IRC09:15
*** alexchadin has joined #openstack-nova09:15
*** alexchadin has quit IRC09:16
*** gjayavelu has quit IRC09:16
*** alexchadin has joined #openstack-nova09:16
*** alexchadin has quit IRC09:16
*** tssurya has joined #openstack-nova09:18
openstackgerritBrooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS  https://review.openstack.org/53316809:20
lennybHi, is it possible to clean/updated nova database? I have Orphaned Exception ( master branch ) http://paste.openstack.org/show/701494/09:22
openstackgerritZhenyu Zheng proposed openstack/nova master: WIP  https://review.openstack.org/55328809:24
*** gaoyan has quit IRC09:25
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path  https://review.openstack.org/49720109:27
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647509:27
*** nicolasbock has joined #openstack-nova09:27
Spaz-WorkOoOo09:27
*** janki has quit IRC09:28
claudiub|2jichen: hi. around?09:32
*** claudiub|2 is now known as claudiub09:32
jichenclaudiub: yes09:32
jichenclaudiub: what's up?09:32
claudiubcool. regarding the live-resize. do you really want to support live-downsize as well?09:32
*** alexchadin has joined #openstack-nova09:32
claudiubthere's quite a bit of opposition to that. :)09:32
claudiubor maybe we can add support for upsizing first, and then look into downsizing afterwards09:34
*** amoralej|off is now known as amoralej09:34
jichenno, actually, I am ok if we only support upsize at first cycle09:34
claudiubthe nova cores seems to be fine with upsizing.'09:34
jichenexactly, I agree with you09:35
jichenupsizing should be first priority09:35
claudiubgreat, then I will amend the spec to mention this fact.09:35
jichenok, perfect, thank you~09:35
*** Dinesh_Bhor has quit IRC09:35
*** lei-zh has quit IRC09:40
*** derekh has joined #openstack-nova09:40
*** Eran_Kuris has quit IRC09:42
*** Eran_Kuris has joined #openstack-nova09:42
*** mgoddard_ has joined #openstack-nova09:43
*** shaohe_feng has joined #openstack-nova09:50
*** sidx64 has quit IRC09:52
*** jichen has quit IRC09:55
*** sidx64 has joined #openstack-nova09:58
*** Tom-Tom has joined #openstack-nova10:09
*** stakeda has quit IRC10:10
*** yufei has quit IRC10:10
*** ragiman has joined #openstack-nova10:12
*** sree has quit IRC10:13
*** Tom-Tom has quit IRC10:13
*** Eran_Kuris has quit IRC10:14
*** sree has joined #openstack-nova10:14
*** Eran_Kuris has joined #openstack-nova10:18
*** maciejjozefczyk has quit IRC10:18
*** sree has quit IRC10:19
openstackgerritClaudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize  https://review.openstack.org/14121910:20
*** maciejjozefczyk has joined #openstack-nova10:26
*** sridharg has quit IRC10:29
*** hoangcx has quit IRC10:32
*** abhishekk has quit IRC10:35
*** hoangcx has joined #openstack-nova10:36
*** moshele has quit IRC10:36
*** moshele has joined #openstack-nova10:37
*** liuzz has quit IRC10:38
openstackgerritClaudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize  https://review.openstack.org/14121910:39
*** alexchadin has quit IRC10:46
*** gongysh has joined #openstack-nova10:54
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name  https://review.openstack.org/52779110:57
*** sidx64 has quit IRC10:59
*** sidx64 has joined #openstack-nova11:00
stephenfinDoes 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-WorkI wish I knew more about specs11:02
kashyapstephenfin: What might be a better way?11:02
stephenfinkashyap: Something that lets you comment on the rendered spec would be a huge help11:03
stephenfina la Google Docs11:03
kashyapstephenfin: 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-nova11:03
kashyapstephenfin: What is it that is specifically bothering you?11:03
*** yamamoto has quit IRC11:04
stephenfinTake this spec, for example https://review.openstack.org/#/c/552722/11:04
* kashyap clicks11:04
stephenfinartom 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 about11:04
kashyapThe pig is still loading11:04
kashyapstephenfin: And11:05
stephenfinHowever, I can't click those links because it's just text. All the rst->html magic hasn't been applied yet11:05
kashyap(I saw that spec, meaning to review)11:05
kashyapAh-ha11:05
stephenfinSimilarly, for my NUMA aware vSwitch spec, I included a load of images but none of those render in the rST, naturally enough11:05
kashyapHmm, nod11:06
stephenfinSo 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 comments11:06
stephenfinIt's just...silly :)11:06
stephenfinAlso, Realistically, no one should care about the formatting of the source for a doc. It's the output that matters11:06
* kashyap goes to add a comment in the spec, before he forgets11:06
stephenfinBut yeah, </rant> :)11:07
stephenfinstupid Gerrit :mutter mutter:11:07
kashyapstephenfin: Yeah, I hear the pain, I just try to focus on the core content, and the formatting not to be egregious11:07
stephenfinkashyap: Do. It would be a good thing to finally resolve11:08
kashyapstephenfin: BTW, slightly related: Some people seem to think 'patchwork' is "dead"11:08
*** sidx64 has quit IRC11:08
stephenfinOh, really?11:08
kashyapstephenfin: And QEMU folks have written their own: http://patchew.org/11:08
kashyapThere's a bit of overlap with 'patchwork'11:08
kashyapstephenfin: See this snippet:11:08
kashyap17:59 < kashyap> bonzini: Isn't there some overlap between 'patchew' and 'patchwork'?11:08
kashyap17:59 < bonzini> kashyap: yeah, but patchwork seemed dead when patchew was started11:08
kashyapFrom a week or so ago11:09
kashyap(On OFTC, #qemu)11:09
stephenfinYup, that's been around for a while. At the time Patchwork _was_ dead11:09
kashyapAh-ha, I see11:09
stephenfinWas only when I and another Intel dev started working on it (for freedesktop) that it started moving forward again11:09
kashyapMost people used 'patchwork' as just a "tracker" of patches, though11:10
stephenfinGuess the two could be folded in but meh, I've barely any time to work on Patchwork of late11:10
Spaz-WorkNova meeting is 2100 today right?11:11
*** sidx64 has joined #openstack-nova11: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 cave11:12
*** damien_r has quit IRC11:14
*** sidx64 has quit IRC11:14
*** udesale has quit IRC11:15
*** tiendc has quit IRC11:16
*** liuyulong has quit IRC11:16
kashyapSpaz-Work: Not sure the hour; there are two timings11:17
*** gongysh has quit IRC11:17
Spaz-WorkYeah think this week is the 2100.. sitting here late in the day trying to convert that to CDT.  Guess i'll read the logs :D11:18
Spaz-WorkPoor night shift hours.11:18
*** dtruong_ has joined #openstack-nova11:18
*** lpetrut_ has joined #openstack-nova11:18
*** lpetrut has quit IRC11:18
*** andreas_s has quit IRC11:18
*** tbachman_ has joined #openstack-nova11:19
*** Spaz-Home has joined #openstack-nova11:19
*** andreas_s has joined #openstack-nova11:19
*** cdent_ has joined #openstack-nova11:19
*** artom_ has joined #openstack-nova11:20
*** bhagyashri_s has joined #openstack-nova11:20
*** mnaser_ has joined #openstack-nova11:20
*** Kevin_Zheng_ has joined #openstack-nova11:20
*** karlamrhein_ has joined #openstack-nova11:20
*** pooja-jadhav has joined #openstack-nova11:20
*** Alexey_Abashkin has joined #openstack-nova11:20
*** lpetrut has joined #openstack-nova11:22
*** vipuls has joined #openstack-nova11:22
*** dosaboy_ has joined #openstack-nova11:22
*** frickler_ has joined #openstack-nova11:23
*** Anticime1 has joined #openstack-nova11:23
*** kuzko_ has joined #openstack-nova11:23
*** jog0 has joined #openstack-nova11:23
*** sahid_ has joined #openstack-nova11:24
*** andreas_s has quit IRC11:24
*** sorrison_ has joined #openstack-nova11:24
*** jgriffit1 has joined #openstack-nova11:24
*** weezhard has joined #openstack-nova11:25
*** tetsuro has quit IRC11:26
*** slaweq_ has joined #openstack-nova11:26
*** cdent has quit IRC11:27
*** pcaruana|wfh| has quit IRC11:27
*** ttsiouts has quit IRC11:27
*** kaisers1 has quit IRC11:27
*** dtruong has quit IRC11:27
*** sorrison has quit IRC11:27
*** vipul has quit IRC11:27
*** bkero- has quit IRC11:27
*** jogo has quit IRC11:27
*** MikeG451 has quit IRC11:27
*** dosaboy has quit IRC11:27
*** Kevin_Zheng has quit IRC11:27
*** karlamrhein has quit IRC11:27
*** mnaser has quit IRC11:27
*** ingy has quit IRC11:27
*** frickler has quit IRC11:27
*** Anticimex has quit IRC11:27
*** cdent_ is now known as cdent11:27
*** mnaser_ is now known as mnaser11:27
*** karlamrhein_ is now known as karlamrhein11:27
*** pcaruana|wfh| has joined #openstack-nova11:28
*** ccamacho1 has joined #openstack-nova11:28
*** ccamacho has quit IRC11:28
*** sshwarts has quit IRC11:28
*** AlexeyAbashkin has quit IRC11:28
*** moshele has quit IRC11:28
*** sahid has quit IRC11:28
*** tbachman has quit IRC11:28
*** Spazmotic has quit IRC11:28
*** FL1SK has quit IRC11:28
*** liusheng has quit IRC11:28
*** krtaylor has quit IRC11:28
*** jgriffith has quit IRC11:28
*** weezhard_ has quit IRC11:28
*** lpetrut_ has quit IRC11:28
*** artom has quit IRC11:28
*** pooja_jadhav has quit IRC11:28
*** bhagyashris_ has quit IRC11:28
*** slaweq has quit IRC11:28
*** kuzko has quit IRC11:28
*** ingy1 has joined #openstack-nova11:28
*** liusheng has joined #openstack-nova11:28
*** krtaylor has joined #openstack-nova11:28
*** Kevin_Zheng_ has quit IRC11:28
*** Kevin_Zheng_ has joined #openstack-nova11:28
*** mnaser has quit IRC11:28
*** mnaser has joined #openstack-nova11:28
*** karlamrhein has quit IRC11:28
*** karlamrhein has joined #openstack-nova11:28
*** Alexey_Abashkin is now known as AlexeyAbashkin11:28
*** kaisers1 has joined #openstack-nova11:28
*** ttsiouts has joined #openstack-nova11:29
*** tbachman_ is now known as tbachman11:29
*** sshwarts has joined #openstack-nova11:31
*** bkero- has joined #openstack-nova11:31
openstackgerritZhenyu Zheng proposed openstack/nova master: WIP  https://review.openstack.org/55328811:32
*** mdbooth has joined #openstack-nova11:32
mdboothjohnthetubaguy, any chance you might be able to squint at this bugfix for me: https://review.openstack.org/#/c/551302/11:33
mdboothIncidentally, who has anything to do with Hyper-V?11:33
*** MikeG451 has joined #openstack-nova11:34
stephenfinmdbooth: claudiub is your man11:35
*** frickler_ is now known as frickler11:35
mdboothclaudiub, Any chance you could look at ^^^ for me?11:35
mdboothlyarwood, it deletes a bunch of connection_info-related libvirt driver code in live migration if you'd care to look11:36
mdboothclaudiub, 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 IRC11:38
openstackgerritSurya Seetharaman proposed openstack/nova master: Add disabled column to cell_mappings table.  https://review.openstack.org/55250511:38
lyarwoodmdbooth: ack, will look11:40
*** pchavva has quit IRC11:40
*** sidx64 has joined #openstack-nova11:45
*** annp has quit IRC11:48
*** sidx64 has quit IRC11:52
*** liverpooler has joined #openstack-nova11:52
*** Amit82 has quit IRC11:52
*** sidx64 has joined #openstack-nova11:54
openstackgerritHironori Shiina proposed openstack/nova master: ironic: Get correct inventory for deployed node  https://review.openstack.org/55336711:58
*** ameeda has joined #openstack-nova11:58
ameedaHello, 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|lunch12:00
lyarwoodmdbooth: 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/152489812:02
openstackLaunchpad 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 IRC12:03
openstackgerritLee Yarwood proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest  https://review.openstack.org/52810412:03
lyarwoodmdbooth: ^ you can use this change from melwitt to test LM with volume backed instances12:04
*** yamamoto has joined #openstack-nova12:04
*** dtantsur|afk is now known as dtantsur12:04
*** odyssey4me has quit IRC12:05
*** odyssey4me has joined #openstack-nova12:05
*** yamamoto has quit IRC12:09
*** pcaruana|wfh| is now known as pcaruana12:14
*** edmondsw has joined #openstack-nova12:15
*** yamamoto has joined #openstack-nova12:16
*** elmaciej has joined #openstack-nova12:17
*** claudiub|2 has joined #openstack-nova12:20
*** jpena is now known as jpena|lunch12:21
*** takedakn has joined #openstack-nova12:21
*** takedakn has quit IRC12:22
*** claudiub has quit IRC12:23
*** sidx64 has quit IRC12:24
*** moshele has joined #openstack-nova12:27
mdboothlyarwood, Will do, thanks!12:28
*** lucasagomes is now known as lucas-hungry12:28
mdboothlyarwood: Short of adding a depends on to my patch, what's the best way to run against that?12:30
*** sidx64 has joined #openstack-nova12:30
mdboothlyarwood: Not that I'd mind adding a depends on if the test patch is likely to get some priority.12:30
lyarwoodmdbooth: another change on top of https://review.openstack.org/528104 that depends on your change?12:31
mdboothlyarwood: Good idea, thanks.12:31
openstackgerritMatthew Booth proposed openstack/nova master: DNM: Run volume-backed live migration tests  https://review.openstack.org/55337712:33
*** moshele has quit IRC12:33
*** mikal has joined #openstack-nova12:42
*** psachin has quit IRC12:44
*** jmlowe has joined #openstack-nova12:44
*** bhujay has joined #openstack-nova12:44
*** mikal_ has quit IRC12:45
*** r-daneel has joined #openstack-nova12:46
*** suresh12 has joined #openstack-nova12:49
*** FL1SK has joined #openstack-nova12:50
*** Tom-Tom has joined #openstack-nova12:50
*** david-lyle has joined #openstack-nova12:52
*** suresh12 has quit IRC12:53
*** Zames has joined #openstack-nova12:54
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name  https://review.openstack.org/52779112:55
*** moshele has joined #openstack-nova12:56
*** Zames has quit IRC12:57
*** moshele has quit IRC12:58
*** sean-k-m1 is now known as sean-k-mooney12:59
*** dave-mccowan has joined #openstack-nova13:01
*** sridharg has joined #openstack-nova13:01
lpetrutmdbooth: 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-nova13:02
*** lyan is now known as Guest7904613:02
mdboothlpetrut: What changes is that I'm now passing the *old* bdms to post_live_migration instead of the new ones13:02
mdboothSo connection_info will relate to the source host, which is where it's actually running, rather than the destination13:03
lpetrutmdbooth: 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
mdboothlpetrut: Potentially a lot, I think13:03
mdboothThat particular issue has been around for ages13:03
lpetrutfor FC it would look different but that's fine as disconnect_volume is a noop for FC in our case13:03
*** andreas_s has joined #openstack-nova13:03
*** alexchadin has joined #openstack-nova13:03
mdboothThe 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* host13:04
lpetrutgot it13:04
*** sean-k-mooney has quit IRC13:05
mdboothlpetrut: This was the original bug: https://bugs.launchpad.net/nova/+bug/147541113:05
openstackLaunchpad 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
mdboothThat's obviously from 201513:06
*** hshiina has quit IRC13:06
mdboothAffects 3par, apparently: https://bugs.launchpad.net/nova/+bug/128803913:06
openstackLaunchpad 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
mdboothAnyway, if Hyper-V uses that cinder driver, my expectation is that my new change will fix it13:08
lpetrutthanks for the link, indeed, that should fix it13:08
*** eharney has joined #openstack-nova13:09
*** mriedem has joined #openstack-nova13:09
lpetrutI was thinking about submitting a bug report against our driver for posterity reasons, marking your patch as a fix13:09
lpetrutI was not aware of the 3par issue, high chance is that we were leaking volume connections and nobody noticed13:10
*** shaohe_feng has quit IRC13:11
mdboothIs anybody aware of a way to make devstack do git checkouts, install pip dependencies, and then stop?13:11
*** shaohe_feng has joined #openstack-nova13:12
*** david-lyle has quit IRC13:12
*** sidx64 has quit IRC13:13
*** belmoreira has joined #openstack-nova13:13
*** sree has joined #openstack-nova13:14
*** sidx64 has joined #openstack-nova13:15
*** david-lyle has joined #openstack-nova13:15
*** sidx64 has quit IRC13:15
*** abhishekk has joined #openstack-nova13:16
*** yamamoto has quit IRC13:18
*** damien_r has joined #openstack-nova13:18
*** sree has quit IRC13:18
*** yamamoto has joined #openstack-nova13:19
*** Zames has joined #openstack-nova13:21
*** amodi has joined #openstack-nova13:22
kashyapmtreinish might know ^13:23
*** yamamoto has quit IRC13:23
*** amoralej|lunch is now known as amoralej13:24
cdentmdbooth: is this in a session you're running by hand or something you want to automate13:24
*** sidx64 has joined #openstack-nova13:25
*** david-lyle has quit IRC13:25
*** idlemind has joined #openstack-nova13:25
*** gouthamr_ has joined #openstack-nova13:26
*** gouthamr_ is now known as gouthamr13:26
cdentmdbooth: if the former, probably just hack an exit in stack.sh13:28
ameedaany 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 jpena13:29
ameedaI set glance conn timeout to 0 , with no lucks13:29
*** Zames has quit IRC13:29
cdentmdbooth: also grep on TRACK_DEPENDS; let's you bail out after install13:30
openstackgerritMatt Riedemann proposed openstack/nova master: Make nova-cells-v1 run with neutron  https://review.openstack.org/54978913:30
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup tempest-dsvm-cells-rc blacklist  https://review.openstack.org/55340113:30
tssuryamriedem : owe you one for the NewtonCellCheck patch, thanks13:31
*** awaugama has joined #openstack-nova13:31
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup tempest-dsvm-cells-rc blacklist  https://review.openstack.org/55340113:31
mriedemtssurya: np13:31
*** ralonsoh_ has joined #openstack-nova13:32
*** Tom-Tom has quit IRC13:32
ameedamriedem: Hello :) , could you please help me with my issue ?13:32
mriedemameeda: um, check the logs13:32
*** jmlowe has quit IRC13:33
ameedamriedem: let me check , thanks13:33
*** dave-mccowan has quit IRC13:34
*** sidx64 has quit IRC13:35
*** ratailor has joined #openstack-nova13:35
*** ralonsoh has quit IRC13:35
mdboothcdent: 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-nova13:36
*** burt has joined #openstack-nova13:36
*** awaugama has quit IRC13:36
kholkinahi all! could you please review the spec? https://review.openstack.org/#/c/54796413:36
*** felipemonteiro_ has joined #openstack-nova13:38
*** p_d has joined #openstack-nova13:39
*** jmlowe has joined #openstack-nova13:40
claudiub|2mdbooth: sorry, I was out for a bit. Will look at it.13:40
*** awaugama has joined #openstack-nova13:40
mdboothclaudiub|2: Thanks. lpetrut already looked, btw.13:41
*** pchavva has joined #openstack-nova13:41
*** bkopilov has joined #openstack-nova13:41
*** felipemonteiro__ has quit IRC13:42
*** mvk has quit IRC13:42
*** lucas-hungry is now known as lucasagomes13:42
*** links has quit IRC13:42
*** udesale has joined #openstack-nova13:43
ameedamriedem: I check this log /var/log/glance/api.log, I didn't find anything helpful , also this command journalctl -u openstack-glance-api.service -f13:44
*** felipemonteiro_ has quit IRC13:45
*** felipemonteiro_ has joined #openstack-nova13:45
SamYapleameeda: how did you deploy glance? uwsgi? apache? eventlet?13:45
mriedemkholkina: -2 on that idea, you can already update user-data via rebuild now13:46
mriedema lot of thought and discussion already went into adding that to rebuild13:47
mriedemsee http://lists.openstack.org/pipermail/openstack-operators/2017-October/014309.html13:47
ameedaSamYaple: I create python project to use contact with openstack via rest api, I use http requests as rest client13:47
mriedemameeda: this isn't really the channel for this type of question - try #openstack or #openstack-glance13:47
*** sidx64 has joined #openstack-nova13:47
*** r-daneel has quit IRC13:48
ameedaeverything 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 openstack13:49
*** licanwei has quit IRC13:50
ameedamriedem: I will try the other channels, thank you13:50
*** mlavalle has joined #openstack-nova13:51
efriedjaypipes, cdent: Opinion on https://review.openstack.org/#/c/548975/5/nova/scheduler/client/report.py@592 ?13:52
*** janki has joined #openstack-nova13:52
cdentefried: explicit is better than implicit13:52
cdentsome version ought to be listed there13:52
*** sidx64 has quit IRC13:52
kholkinamriedem, the spec means to update user-data on booted instance, without rebuild. It may be useful to provide a cloud-config13:52
efriedcdent: Even if it's '1.0'?13:54
jaypipesefried: yeah, agree with alex_xu_ it would be good to be consistent there.13:54
cdentefried: yes13:54
jaypipesefried: no biggie though, really13:54
*** sidx64 has joined #openstack-nova13:54
efriedcdent: 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
cdentno13:55
cdentIt'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 base13:55
alex_xu_that method is only used for create root rp?13:55
edleafeIt also looks consistent in that block to have a version in both13:56
*** crushil has joined #openstack-nova13:56
cdentthat 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 versin13:56
*** sean-k-mooney has joined #openstack-nova13:56
efriedfixing.13:57
efriedalex_xu_: yes13:57
efriedoh13:57
efriedmaybe not.  Stand by.13:57
efriedHah, 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-nova13:58
openstackgerritEric Fried proposed openstack/nova master: Stop assuming initial provider generation is 0  https://review.openstack.org/54897513:58
efriedalex_xu_, jaypipes, cdent: Done ^13:59
*** lei-zh has quit IRC13:59
*** lei-zh has joined #openstack-nova13:59
ratailoralex_xu_, jaypipes could you pl have a look at https://review.openstack.org/#/c/401009/13:59
*** esberglu has joined #openstack-nova14:00
*** bhujay has quit IRC14:01
*** gongysh has joined #openstack-nova14:01
belmoreiradansmith, mriedem in my cellsV1/V2 migration plan I have some issues that I would like to discuss you. Let me know when you are available14:03
*** tidwellr has joined #openstack-nova14:03
*** jaosorior has quit IRC14:03
*** ralonsoh__ has joined #openstack-nova14:04
*** ralonsoh__ is now known as ralonsoh14:04
*** ralonsoh_ has quit IRC14:07
openstackgerritRajesh Tailor proposed openstack/nova master: Reset instance status on invalid host  https://review.openstack.org/40100914:08
dansmithbelmoreira: go for it14:08
belmoreirathanks dansmith14:08
belmoreiraWe use the same cloud to deploy VMs and baremetal. Baremetal is dedicated to few cells.14:09
belmoreiraWhen 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
dansmithbelmoreira: the baremetal filters are deprecated14: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 identified14: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
dansmithbelmoreira: 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 uniform14:10
belmoreiraI missed that14:11
mriedemthere was a caveat with dropping the ironic host manager for the scheduler, would have to dig it back up,14:11
mriedembut i think it's that not all filter scheduler filters will be able to handle baremetal instances, numatopologyfilter specifically i think14:11
belmoreiraI'm finding the release notes about this. thanks14:12
mriedemi'll be damned if i can find the way to those docs in ironic via their homepage14:12
belmoreira:)14:13
mriedembelmoreira: https://docs.openstack.org/ironic/latest/install/configure-nova-flavors.html#scheduling-based-on-resource-classes14:13
mriedemsearch box ftw14: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 IRC14:13
alex_xu_it pretty looks like an API layer check.14:13
*** mvk has joined #openstack-nova14:14
belmoreirathanks14:14
mriedemalex_xu_: i wouldn't be opposed to that, replied14:14
alex_xu_mriedem: ok14:14
mriedembelmoreira: https://github.com/openstack/nova/commit/c99fc64271f41a028c75e96348813f673d04e24514:15
mriedemNUMATopologyFilter isn't enabled by default, but i know a lot of people use it14:16
mriedemif 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 breaks14:17
mgoddard_mriedem can I offer you a throat sweet?14:18
mriedemwtf14:18
mgoddard_for your cough14:18
mriedemis that british for "i'm going to slit your throat?"14:18
mriedemoh14:18
*** crushil has quit IRC14:19
mgoddard_oh wow, crossed wires there14:19
mriedemi consider phlegm a feature so no thanks14:19
dansmithPaaS - Phlegm as a Service?14:19
mgoddard_I could take a look at this14:19
mriedemi'm having trouble remembering off the top of my head what it was about the NUMATopologyFilter that makes it not work with ironic14:20
*** yamamoto has joined #openstack-nova14:20
dansmithmriedem: that needs fixing if so14:20
dansmithmaybe we could get stephenfin or someone to look at that once we know it's really a problem14:20
stephenfinI didn't do it14:21
stephenfinI've never used ironic so I'm probably not your guy14:22
mriedemapparently 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 default14:23
mriedemgood news, mogan does too http://codesearch.openstack.org/?q=IRONIC_USE_RESOURCE_CLASSES&i=nope&files=&repos=14:23
dansmithstephenfin: we need someone that understands the numa filter not necessarily ironic14:23
stephenfinOh, that I can do14:23
dansmithmriedem: but no numafilter presumably14:23
mriedemnope, 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 up14:24
*** sar has quit IRC14:24
dansmithack14:24
mriedemand then report a bug to stephenfin14:24
openstackgerritRadoslav Gerganov proposed openstack/nova master: WIP: VMware: implement get_serial_console  https://review.openstack.org/55341114:24
stephenfin🎉14:25
mriedemis that a lego brick? a touch tone phone?14:25
dansmithparty horn?14:25
*** hieulq has quit IRC14:25
stephenfinhttps://emojipedia.org/party-popper/14:25
mriedemoh, doesn't render for me14:25
mriedempidgin14:25
*** yamamoto has quit IRC14:25
stephenfinWindows?14:25
mriedemyeah14:26
* stephenfin thought Windows 10 supported all these. Seems not (F27 + Hexchat, fwiw)14:26
stephenfinBut yeah, assign me to that. Soon as I get my server back, I can take a look14:26
stephenfinI'm on spec review duty all day today14:26
*** ratailor has quit IRC14:27
*** lei-zh has quit IRC14:30
*** r-daneel has joined #openstack-nova14:30
*** gagehugo has left #openstack-nova14:30
*** gongysh has quit IRC14:31
*** germs has joined #openstack-nova14:33
*** germs has quit IRC14:33
*** germs has joined #openstack-nova14:33
*** hieulq has joined #openstack-nova14:33
*** germs has quit IRC14:33
mriedemtest patch https://review.openstack.org/55341214:33
mriedemmgoddard_: ^14:33
*** elmaciej has quit IRC14:34
mgoddard_mriedem: nice, I'll keep an eye on it14:35
*** germs has joined #openstack-nova14:36
*** yamahata has joined #openstack-nova14:39
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name  https://review.openstack.org/52779114:40
openstackgerritStephen Finucane proposed openstack/nova-specs master: Few correction in the server filter/sort spec  https://review.openstack.org/52701914:41
*** zhaochao has quit IRC14:42
Spaz-Homedansmith, 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-HomeTime for me to head to bed.. have a good day folks.14:44
dansmithSpaz-Home: niachuan asked us to do the meeting for him because he can't join14:45
dansmithwhich I will14:45
dansmithbut yeah14:45
Spaz-HomeAhh that's unfortunate14:45
Spaz-HomeIt's hard on our hours14:45
*** jaosorior has joined #openstack-nova14:47
mriedembhagyashri_s: i've abandoned your spec since it looks, well, abandoned https://review.openstack.org/#/c/511825/14:47
mriedemneeds to be moved to the rocky directory structure and incorporate the direction from the PTG14:47
*** moshele has joined #openstack-nova14:47
*** mdbooth has quit IRC14:48
mriedemsimondodsley: are you going to move this to rocky? https://review.openstack.org/#/c/340168/ it's proposed against pike14:51
mriedemsean-k-mooney: does anyone care about this anymore? https://review.openstack.org/#/c/471815/14:54
*** ameeda has quit IRC14:54
mriedemi don't know if baoli is around anymore re: cisco14:54
mriedemmgagne: ^ seems like something that would be up your alley14:54
mriedemor vdrok ^ i.e. vlan trunk port metadata bliss14:56
*** eharney has quit IRC14:56
openstackgerritNguyen Hai proposed openstack/python-novaclient master: Fix local test fails with pypy  https://review.openstack.org/55342614:56
*** eharney has joined #openstack-nova14:56
openstackgerritNguyen Hai proposed openstack/python-novaclient master: Fix local test fails with pypy  https://review.openstack.org/55342614:57
*** mdnadeem has quit IRC14:58
*** abhishekk has quit IRC14:59
*** Spaz-Work has quit IRC15:00
openstackgerritSurya Seetharaman proposed openstack/nova master: Add disabled field to CellMapping object  https://review.openstack.org/55009015:00
mgagnemriedem: thanks for the heads-up15:00
dansmithmriedem: comments in that rebuild-bfv spec15:00
mriedemmgagne: 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 feedback15:00
dansmithmriedem: 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 me15:00
*** felipemonteiro_ has quit IRC15:01
*** felipemonteiro_ has joined #openstack-nova15:01
efriedI 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_food15:02
efriedReason 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#L83315:03
efriedBut that information sure would be useful for people implementing that method.15:03
mriedemefried: just say "see the update_provider_tree spec for more details"15:03
mriedemor something15:04
efriedCan I say "rocky" too?15:04
efriedThat'd give them enough info to find it in one shot.15:04
mriedemsure15:04
mriedemor, move the dev details of that spec into devref15:04
*** BlackDex has joined #openstack-nova15:04
mriedemhttps://docs.openstack.org/nova/latest/reference/index.html15:05
*** Zames has joined #openstack-nova15:06
dansmithmriedem: tested by a real live person: https://review.openstack.org/#/c/55269115:06
dansmithper your ask15:06
*** sree has joined #openstack-nova15:07
*** mdbooth has joined #openstack-nova15:07
openstackgerritSurya Seetharaman proposed openstack/nova master: Add disabled field to CellMapping object  https://review.openstack.org/55009015:08
*** sree_ has joined #openstack-nova15:08
efriedmriedem: 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
openstackgerritClaudiu Belu proposed openstack/nova-specs master: Adds spec for instance live resize  https://review.openstack.org/14121915:08
*** beagles_food is now known as beagles15:09
*** sree_ is now known as Guest641615:09
*** Zames has quit IRC15:09
*** udesale has quit IRC15:10
*** sree has quit IRC15:12
mriedemefried: idk, choose your own adventure15:12
*** kholkina has quit IRC15:12
mriedemlinking to a devref is probably ok, the spec URL is long as hell15:12
*** jobewan has joined #openstack-nova15:13
*** r-daneel_ has joined #openstack-nova15:14
*** r-daneel has quit IRC15:14
*** r-daneel_ is now known as r-daneel15:14
mriedemdansmith: agree the create/delete volume thing sucks15:16
mriedemdon't know how easy it would be for cinder to add an api to re-image a volume15:16
mriedemsmcginnis: ^?15:16
*** krtaylor has quit IRC15:16
smcginnisSo detach, re-lay down an image on the vol, reattach?15:17
openstackgerritMerged openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/55325515:17
mriedemnot sure if we'd need the detach/re-attach part15:17
*** felipemonteiro__ has joined #openstack-nova15:17
dansmithsmcginnis: yeah15:17
mriedembut that's what happens during rebuild anyway yeah15:17
dansmithmriedem: I assume we would for some backends15:17
mdboothmriedem: Would you mind taking a look at https://review.openstack.org/#/c/551302/ ? I think it's your bailiwick.15:17
mriedemis a bailiwick related to a throat sweet?15:18
*** tidwellr has quit IRC15:18
smcginnismriedem: I think we would need the detach reattach.15:18
smcginnisWe need to attach it to a cinder node to reimage it.15:19
mriedemsmcginnis: that's fine probably b/c we do it anyway for rebuild15:19
*** hrw has joined #openstack-nova15:19
smcginnisUnless nova wants to take care of that part and leave cinder out of it.15:19
hrwmorning15:19
*** tidwellr has joined #openstack-nova15:19
smcginnisBasically just duplicate what we do in cinder.15:19
*** Spaz-Work has joined #openstack-nova15:19
smcginnisPull the image to the compute node, write it over the volume, then start up the instance again.15:19
mriedemit seems like nova doing things with the internals of the volume is a bad idea15:19
mdboothmriedem: Not especially :) I think there's still a literal bailiwick in the channel islands.15:19
hrwcan someone help me with understanding how nova compares cpus on live migration?15:20
dansmithmriedem: I dunno, it's no different than a regular disk image,15:20
openstackgerritMathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information  https://review.openstack.org/31262615:20
smcginnismriedem: I agree, but it would be an "optimization" to not have to tear things down and start over.15:20
dansmithmriedem: the thing is I would expect cinder to maybe be able to do it more efficiently in some cases (i.e. ceph)15:20
smcginnisEspecially 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 IRC15:20
dansmithsmcginnis: we're talking about bfv here15:20
dansmithsmcginnis: we're doing a rebuild, so we'd detach15:20
smcginnisdansmith: Yeah, so that would be an extra complication.15:20
smcginnisdansmith: Oh, right!15:20
openstackgerritMathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information  https://review.openstack.org/31262615:21
mriedemthe volume would remain in-use during all of this from a state pov15:21
smcginnisSo delete instance, detach volume, cinder rewrites it, reattach, create new "rebuilt" instance using it.15:21
*** felipemonteiro_ has quit IRC15:21
*** yamamoto has joined #openstack-nova15:21
mdboothdansmith: Could also be a multi-attach volume, btw.15:21
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Add CellMappingList.get_all_enabled() query method  https://review.openstack.org/55018815:22
mriedemthis is the code that detaches the volume today for rebuild https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L307215:22
dansmithmdbooth: if people are multi-attaching bfv then I give up15:22
mriedemwe create a 2nd attachment to keep the volume 'in-use' and then delete the first attachment15:22
dansmithwell, unless it's read only I guess15:22
mdboothdansmith: It's a thing with real hardware.15:22
dansmithbut yeah, if we can't undo all the attachments, we'd have to fail I think15:23
mriedembefore we spawn the guest again, we 'attach' again https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L310315:23
* mdbooth would overwrite it in place regardless.15:23
hrwhttps://bugs.launchpad.net/nova/+bug/1756118 reported15:23
openstackLaunchpad bug 1756118 in OpenStack Compute (nova) "No live migration on aarch64" [Undecided,New]15:23
mdbooth... unless the volume was attached RO15:23
openstackgerritJianghua Wang proposed openstack/os-traits master: GPU: define traits for maximum display heads  https://review.openstack.org/55327715:24
mriedemhrw: "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
hrwmriedem: it works in newton.15:25
mdboothThat's a good point. Can you rebuild a server whose root disk is read-only?15:25
* mdbooth would say no.15:25
hrwmriedem: on newton we have libvirt 2.2.10 and live migration from nova level on aarch64 works.15:25
hrwmriedem: on queens we have libvirt 3.8/3.10 and it does not on nova level but does on libvirt level15:25
mriedemhrw: ok that's info that should be pointed out in the bug report then15:26
hrwwill do15:26
hrwadded15:27
hrwmriedem: bug raport is more to signal issue as I will dig into it to find out what is going on15:27
*** yamamoto has quit IRC15:27
openstackgerritMerged openstack/os-traits master: Add HW_NIC_SRIOV_TRUSTED trait  https://review.openstack.org/55087315:28
hrwmriedem: have to find out where get_connection().CompareCPU() is defined15:28
hrwmriedem: as it looks like suspicious place ;d15:29
mriedemmy guess would be nova.virt.libvirt.host15:29
*** itlinux has joined #openstack-nova15:29
hrwmriedem: nova.virt.libvirt.host.compare_cpu() calls self.get_connection.compareCPU()15:31
mriedemmdbooth: 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 attachment15:32
dansmithright,15:32
dansmithat least for the moment15:32
mriedemand if it's a leap year because that's a lot of conditionals15:32
*** chyka has joined #openstack-nova15:33
*** sar has joined #openstack-nova15:33
*** p_d has quit IRC15:34
*** moshele has joined #openstack-nova15:37
*** chyka_ has joined #openstack-nova15:37
*** chyka has quit IRC15:37
*** dave-mccowan has joined #openstack-nova15:38
*** moshele has quit IRC15:39
mriedemwould 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 forward15:42
dansmithoooh15:42
dansmithI found a typo I could use to crush your stats15:43
*** lpetrut_ has joined #openstack-nova15:43
*** lpetrut has quit IRC15:43
*** yamahata has quit IRC15:43
* penick watches dansmith crush mriedem's dreams 15:44
dansmithpenick: all while making this face: https://cheappopculture.files.wordpress.com/2015/02/revolting-blob.png?w=82515:46
artom_As a non-core observer, I feel like singing Sweet Dreams, but with Sweet changed to Crushed, would be appropriate15:46
openstackgerritMathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information  https://review.openstack.org/31262615:46
penickhahaha, that got a real laugh outta me15:47
mriedemwe changed to "extra_spces" in a microversion in queens15:47
mriedemyou must have missed that15:47
dansmithoh mah bad15:48
penickCrushed dreams are made of theeeese, who am I to disagree?, travel the world, to the p-t-g, everybody's like minus two15:48
artom_penick, noice15:48
penick*fistpump*15:49
*** lpetrut_ has quit IRC15:49
artom_Are... are we bros now? Is this what being a bro is like?15:49
*** ragiman has quit IRC15:49
*** gongysh has joined #openstack-nova15:51
*** elod is now known as elod_off15:51
openstackgerritMerged openstack/nova-specs master: Add extra-specs to the flavor show and detail API calls  https://review.openstack.org/54415815:51
*** kmalloc has joined #openstack-nova15: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 IRC15:54
cdentcfriesen_: 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
openstackLaunchpad bug 1708958 in OpenStack Compute (nova) "disabling a compute service does not disable the resource provider" [Wishlist,Confirmed]15:56
*** hamzy has quit IRC15:56
mriedemdisable != delete15:56
cfriesen_yeah15:57
*** ratailor has joined #openstack-nova15:57
mriedemyou can delete a service record via the api,15:57
mriedembut not the compute node record15:57
mriedemdeleting the compute node record is the issue15:57
cfriesen_yes15:57
mriedemand no we don't have anything in place to clean that stuff up15:57
mriedemwe don't have any interfaces for deleting compute node records either15:57
mriedemwe've talked about adding that to nova-manage in the past15:57
*** dtruong_ has quit IRC15:58
penickartom_ almost. Before becoming bros you have to go through the ceremony of course.15:58
*** dtruong has joined #openstack-nova15:58
mriedemwe 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@22815: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 issue15:59
mriedemif 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 placement15:59
cfriesen_ew16:00
mriedemcfriesen_: well, how did they delete the compute node record? directly in the DB?16:00
mriedemif you're monkeying with the db, then you are also going to need to deal with cleaning up placement as a result16:00
mriedemat least for now, until we have some interface for this16:00
cfriesen_mriedem: I don't think they did delete the comptue node record, just the service16:00
mriedemthen i don't see how the compute node uuid would change unless the hostname changed or something16:01
*** ralonsoh_ has joined #openstack-nova16:01
mriedemif the hostname changes, you have other problems16:01
*** amodi has quit IRC16:01
mriedemi 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 nodes16:02
mriedemand placement entries for those compute nodes16:02
*** alexchadin has quit IRC16:03
mriedemanyway, 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 spec16:03
*** harlowja has joined #openstack-nova16:03
mriedemif it were easy it would have been done by now16:03
tssuryaI am cleaning up compute node and rp records in my patch16:03
tssuryawhen we delete a cell or a host16:04
dansmithtssurya: s/host/host mapping/ right?16:04
*** ralonsoh has quit IRC16:04
tssuryayes16:04
mriedemcfriesen_: in the meantime, they can use osc-placement 1.0 to delete the old compute node RP in placement16:05
cfriesen_okay, thanks for the info.  tssurya, can you invite me to review that patch?16:06
mriedemhttps://review.openstack.org/#/c/542964/ ?16:07
cfriesen_mriedem: I think that's basically what they were planning on doing for now16:07
tssuryahttps://review.openstack.org/#/c/546660/16:07
tssuryamriedem: its the one I posted, same one needing the functional test16:08
*** gongysh has quit IRC16:08
tssuryafor the placement side16:08
*** burt has quit IRC16:08
tssuryafor deleting the RP record16:08
tssuryacfriesen_ : done, I hope I got the name right16:09
tssuryafor the review invite16: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
openstackLaunchpad 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, thanks16:10
mriedemcfriesen_: probably ask sahid_ about that bug16:10
mriedemmarkus_z would have been good too but he's no longer around16:10
*** cfriesen_ is now known as cfriesen16:11
*** alexchadin has joined #openstack-nova16:11
*** amodi has joined #openstack-nova16:13
cfriesensahid_:  (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 remember16:17
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Resource provider examples  https://review.openstack.org/55346116: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 IRC16:18
sahid_it seems that during hard reboot we redefine the domain xml16:19
*** sidx64 has quit IRC16:19
*** Guest6416 has quit IRC16:20
*** sree has joined #openstack-nova16:20
*** jaosorior has quit IRC16:21
*** yamahata has joined #openstack-nova16:22
*** BlackDex has joined #openstack-nova16:23
*** yamamoto has joined #openstack-nova16:23
*** sree has quit IRC16:25
*** hongbin has joined #openstack-nova16:25
*** burt has joined #openstack-nova16:27
*** masber has joined #openstack-nova16:28
*** yamamoto has quit IRC16:29
*** AlexeyAbashkin has quit IRC16: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 IRC16:35
openstackgerritChris Dent proposed openstack/nova-specs master: Spec for isolating configuration of placement database  https://review.openstack.org/55292716:37
*** moshele has joined #openstack-nova16:37
cdentstephenfin: addressed your suggestions on ^16:37
*** andreas_s has quit IRC16:38
*** gongysh has joined #openstack-nova16:38
*** gongysh has quit IRC16:39
openstackgerritMatt Riedemann proposed openstack/nova master: Unmap compute nodes when deleting host mappings in delete cell operation  https://review.openstack.org/54296416:41
*** ccamacho1 has quit IRC16:41
mriedemtssurya: would you like to propose a backport to stable/queens for ^ ?16:42
*** andreas_s has joined #openstack-nova16:43
tssuryamriedem: yes I will do it16:43
*** lpetrut has joined #openstack-nova16:43
openstackgerritAlvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances  https://review.openstack.org/32325016:45
*** janki has quit IRC16:46
cfriesensahid_: 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
jaypipesartom_: sure, what's up?16:46
openstackgerritsahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/49745716:47
cfriesensahid: 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
openstackgerritEric Fried proposed openstack/nova master: ProviderTree.{add|remove}_{traits|aggregates}  https://review.openstack.org/55347516:47
efrieddansmith, jaypipes, cdent, edleafe: One half of the result of the "need to be able to merge traits/aggs" discussion ^16:48
*** alexchadin has quit IRC16:48
cfriesensahid_: I'm wondering whether we should do the serial port removal from ALLOCATED_PORTS in _undefine_domain() instead.16:48
openstackgerritEric Fried proposed openstack/nova master: update_provider_tree devref and docstring updates  https://review.openstack.org/55347616:49
efrieddansmith, 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 artom16:50
tssuryamriedem: 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 both16:51
artomjaypipes, 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 spec16:51
artomjaypipes, and what I *think* you're saying is that, we can continue using the current compute-node-claims-the-resources way for now16:51
*** gbarros has quit IRC16:52
*** andreas_s has quit IRC16:52
openstackgerritSurya Seetharaman proposed openstack/nova stable/queens: Extending delete_cell --force to delete instance_mappings  https://review.openstack.org/55347816:53
sahid_cfriesen: i think you also have to look at the method where we define domain xml16:53
edleafeefried: looking...16:53
*** wolverineav has joined #openstack-nova16:53
*** masber has quit IRC16:53
*** wolverineav has quit IRC16:53
*** wolverineav has joined #openstack-nova16:53
sahid_cfriesen: get_config_xml or somethinf16:54
openstackgerritsahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/49745716:54
jaypipesartom: 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/pinning16:54
artomjaypipes, 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 overhaul16:56
jaypipesartom: ++16:57
cfriesensahid_: 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 IRC16:57
jaypipesartom: 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
cfriesensahid_: removing the TCP ports from ALLOCATED_PORTS while they're still defined in the domain is just asking for trouble.16:58
artomjaypipes, totally fair.16:58
openstackgerritMerged openstack/nova stable/pike: doc: fix the link for the evacuate cli  https://review.openstack.org/54285616:58
artomjaypipes, 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 patches17:01
jaypipesartom: 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 writing17:01
jaypipesthe 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 internally17:02
*** gjayavelu has joined #openstack-nova17:03
*** lpetrut has joined #openstack-nova17:03
artomjaypipes, thanks for setting me straight :)17:03
dansmithsahid_: I already said the implementation looks right, barring the gaps in testing17:03
cfriesensahid_: cool, if I get some time I'll hold you to that. :)17:04
cfriesenjaypipes: 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 version17:05
sahid_cfriesen: :)17:05
*** ralonsoh__ has joined #openstack-nova17:06
artomcfriesen, 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
dansmithsahid_: you didn't answer my question about the neutron events in the tests17:07
claudiub|2dansmith: 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 IRC17:07
*** ratailor has quit IRC17:08
*** tidwellr has quit IRC17:08
dansmithclaudiub|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
cfriesenartom: yes, I think it's really NUMA-aware live migration.17:08
claudiub|2nope, I'm trying to add an item to the Migration.migration_type enum.17:09
*** ralonsoh_ has quit IRC17:09
dansmithclaudiub|2: oh just adding something to an existing enum?17:09
*** harlowja has quit IRC17:09
claudiub|2yep17:09
dansmithjaypipes: ^ hopefully that is not a big deal to do online?17:10
claudiub|2this is the commit: https://review.openstack.org/#/c/185961/4 but postgresql seems unhappy about it.17:10
jaypipesdansmith: no, it's virtually instantaneous17:10
dansmithjaypipes: ack17:10
dansmithclaudiub|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 think17:10
artomcfriesen, not a bad idea :)17:11
jaypipeshonestly, we really shouldn't be using the ENUM type anyway... but meh17:11
artomSo, 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
dansmithjaypipes: yeah I'm not a fan myself17:12
*** hamzy has joined #openstack-nova17:13
claudiub|2dansmith: 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 it17:13
claudiub|2DETAIL:  table migrations column migration_type depends on type migration_type17:13
cfriesenartom: I'm just updating the review with some comments right now.  gimme a couple minutes17:13
dansmithclaudiub|2: I don't think I can help you with that one :)17:14
dansmithclaudiub|2: but I don't think it's any of our doing17:14
claudiub|2I sea.17:15
*** dtantsur is now known as dtantsur|afk17:15
openstackgerritSurya Seetharaman proposed openstack/nova stable/queens: Unmap compute nodes when deleting host mappings in delete cell operation  https://review.openstack.org/55349617:16
claudiub|2jaypipes: do you have any ideas? ^17:16
*** ralonsoh__ has quit IRC17:16
*** hamzy_ has joined #openstack-nova17:16
*** gouthamr has quit IRC17:18
openstackgerritMerged openstack/nova master: Update deprecated log-config option in docs  https://review.openstack.org/55182517:18
*** gouthamr has joined #openstack-nova17:18
*** hamzy has quit IRC17:19
mriedemefried: how do i get an endpoint url from a ksa adapter object?17:20
*** fragatina has joined #openstack-nova17:21
efriedmriedem: stand by.17:21
cfriesenartom: okay, updated the review with some extra info17:21
*** fragatina has quit IRC17:21
efriedmriedem: .get_endpoint()17:21
*** tidwellr has joined #openstack-nova17:22
*** fragatina has joined #openstack-nova17:22
* efried applauds apt method name choice17:24
sean-k-mooneymriedem: exposing the vlans for trunk ports in metadata, i think the kuryr team did but honest answer is i dont know17:25
*** yamamoto has joined #openstack-nova17:25
tssuryadansmith, 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 IRC17:25
*** felipemonteiro__ has joined #openstack-nova17:26
dansmithtssurya: generally just fix up the backport and note the reason for the conflicts in the commit message17:26
*** hamzy__ has joined #openstack-nova17:26
dansmithI wouldn't be too opposed to backporting the purge stuff personally, but it's technically not a candidate17:26
tssuryadansmith: okay thanks17:26
tssuryayea I can fix up the backport17:27
tssuryaI didn't know if it was an allowed practice17:27
dansmithtssurya: if you survey some other backports you'll see some "Conflicts:" examples17:27
tssuryadansmith: thanks will look them up17:27
dansmithtssurya: example: https://review.openstack.org/#/c/540145/17:27
jaypipesclaudiub|2: added review note.17:28
tssuryadansmith: perfect thank you!17:28
claudiub|2\o/ thanks. :D17:28
*** liverpooler has quit IRC17:29
*** hamzy_ has quit IRC17:29
*** yamamoto has quit IRC17:30
*** krtaylor has joined #openstack-nova17:36
*** moshele has quit IRC17:37
*** liverpooler has joined #openstack-nova17:42
*** imacdonn has quit IRC17:44
*** imacdonn has joined #openstack-nova17:44
*** r-daneel_ has joined #openstack-nova17:50
*** r-daneel has quit IRC17:51
*** r-daneel_ is now known as r-daneel17:51
*** Matias has quit IRC17:52
*** suresh12 has joined #openstack-nova17:54
*** slaweq_ has quit IRC17:55
*** slaweq has joined #openstack-nova17:55
*** lucasagomes is now known as lucas-afk17:56
cfriesenare there any known issues with nic tagging in Newton?17:56
stephenfinartom: ^ ?17:57
cfriesenI'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-nova17:57
cdentjaypipes: you might enjoy this buglet: https://bugs.launchpad.net/nova/+bug/175615117:58
openstackLaunchpad bug 1756151 in OpenStack Compute (nova) "placement os-traits sync checked every request" [Low,Triaged]17:58
*** mgoddard_ has quit IRC17:58
*** suresh12 has quit IRC17:59
*** felipemonteiro__ has quit IRC17:59
*** moshele has joined #openstack-nova18:01
cfriesenstephenfin: artom: ah yes, microversion 2.37 broke tagging, and 2.42 added it back in.18:03
*** mvenesio has joined #openstack-nova18:03
*** derekh has quit IRC18:04
*** mvk has quit IRC18:05
*** david-lyle has joined #openstack-nova18:05
*** suresh12 has joined #openstack-nova18:07
mriedemmelwitt: have you heard anything new about a project update session at the summit?18:13
*** jpena is now known as jpena|off18:14
melwittmriedem: no, I'm gonna ask anne about it18:15
mriedemnova project update doesn't fall under the category of CI/CD, HPC or EDGE so probably not18:15
mriedemi replied to the ML thread18:15
mriedemquite dickishly18:15
melwittheh. I tried to find them in the summit schedule and found none18:16
mriedemsame18:16
mriedemnova project update: feature k8s!18:17
mriedem*featuring18:17
mriedemdamn18:17
*** moshele has quit IRC18:17
*** harlowja has joined #openstack-nova18:20
jaypipesefried, edleafe: where are we on your battling microversion changes?18:20
efriedjaypipes: It's tied up on the home stretch while zuul unwinds its panties.18:21
efriedSee topic18:21
jaypipesk18:21
*** alexchadin has joined #openstack-nova18:21
*** sridharg has quit IRC18:22
openstackgerritMatt Riedemann proposed openstack/nova master: Use ksa adapter for cinder client  https://review.openstack.org/50834518:24
mriedemefried: i've tried to rebase this but there are some known broken things in it ^18:24
efriedmriedem: Yeah, I had to put it aside for other "more urgent" things.18:25
efriedI swear I had it working at some point in the cycle.18:26
*** moshele has joined #openstack-nova18:26
efriedbut 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-nova18:27
openstackgerritClaudiu Belu proposed openstack/nova master: compute: Adds instance live-resize  https://review.openstack.org/24858118:28
openstackgerritClaudiu Belu proposed openstack/nova master: db: Adds live-resize to Migration model migration_type  https://review.openstack.org/18596118:28
*** burt has joined #openstack-nova18:28
*** david-lyle has quit IRC18:29
*** moshele has quit IRC18:30
*** wolverineav has quit IRC18:31
*** wolverineav has joined #openstack-nova18:31
*** jmlowe has quit IRC18:31
*** yamamoto has quit IRC18:33
*** tidwellr has quit IRC18:33
*** david-lyle has joined #openstack-nova18:33
openstackgerritSurya Seetharaman proposed openstack/nova master: Add disabled field to CellMapping object  https://review.openstack.org/55009018:34
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Add CellMappingList.get_all_enabled() query method  https://review.openstack.org/55018818:34
*** wolverineav has quit IRC18:35
*** jmlowe has joined #openstack-nova18:36
*** wolverineav has joined #openstack-nova18:37
*** david-lyle has quit IRC18:37
*** tesseract has quit IRC18:39
*** mgoddard_ has joined #openstack-nova18:39
cdentnice message efried18:40
openstackgerritSurya Seetharaman proposed openstack/nova master: Add CellMappingList.get_all_enabled() query method  https://review.openstack.org/55018818:40
efriedThanks cdent.  I guess if anyone was gonna bother to read the whole thing, it'd be you :)18:40
cdentyeah, spose so18:41
cdentI think you'll find there's a vast army of silent readers out there18:41
cdenttheir silence can sometimes be rather disturbing18:41
*** felipemonteiro_ has quit IRC18:42
*** felipemonteiro has joined #openstack-nova18:42
*** felipemonteiro_ has joined #openstack-nova18:43
*** alexchadin has quit IRC18:44
efriedcdent: 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
cdentefried: 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
efriedThat explains a lot.18:47
*** felipemonteiro has quit IRC18:47
efriedI 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
edleafeheh, just started reading efried's email18:49
*** tssurya has quit IRC18:49
* efried clicks stopwatch18:49
edleafeAn hourglass would be good enough :)18:50
*** mvk has joined #openstack-nova18:51
cdentefried: I'm certain that I miss stuff, but I'm usually grazing for meaning, not details18:52
efriedI should develop that skill.  FOMO.18:52
cdentmaybe not, probably useful to have both styles around18:53
mriedemmgoddard_: 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_09317218:53
mriedemthat devstack change didn't blow up18:53
dansmithsweet18:53
cdentfor me it's in part a learned skill to intentionally miss out on some stuff18:53
*** AlexeyAbashkin has joined #openstack-nova18:54
*** moshele has joined #openstack-nova18:54
mriedemcould have sworn something about the IronicNodeState object was different such that the filter would fail with it18:55
mriedemalso, it's not like we actually have pci requests in these CI jobs18:55
edleafeI 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 details18:55
edleafeefried: excellent recap18:55
dansmithmriedem: 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
dansmithbut I dunno what that would be really18:56
* cdent goes away for a while18:56
efriededleafe: Thanks.18:56
mriedemdansmith: the other thing might have been something to do with allocation ratios, but grasping at straws18:56
*** beagles is now known as beagles|biab18:57
dansmithwe don't have ratios for those types though18:57
mriedemgoes into the numa topology limits18:58
mriedemthe cpu and ram allocation ratois18:58
*** AlexeyAbashkin has quit IRC18:58
cfriesenmriedem: 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
mriedemcfriesen: is there anything you guys don't have a private patch for?18:59
cfriesenmriedem: we try to upstream stuff, but it takes forever18:59
mgoddard_mriedem: that's good news!18:59
*** moshele has quit IRC18:59
cfriesenmriedem: plus, we only need to worry about one hypervisor19:00
mriedemmgoddard_: well, it's kind of a false positive i think19:00
mriedemcfriesen: it takes even longer when you don't even propose them19:00
dansmithmriedem: 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
mriedemi think host_topology_and_format_from_host might be the thing19:01
mriedemthat's just always None for ironic i think19:02
cfriesenmriedem: 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
dansmithmriedem: 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
mriedemnvm, that would just filter out that ironic node19:03
mriedem / host19:03
openstackgerritEric Berglund proposed openstack/nova master: WIP: Resize  https://review.openstack.org/55358319:03
mriedemcfriesen: if you could figure out *why* you "Looks like we modified NUMATopologyFilter to check the hypervisor type" that would be nice19:03
mriedembecause at some point i thought these wouldn't work but i can't figure out why now19:04
dansmithyeah at least upstreaming the bug would be worthwhile19:04
mriedem++19:04
dansmithalthough they do that, so if you did in this case, then .. cool :)19:04
cfriesenmriedem: let me check with the author19:04
*** esberglu has quit IRC19:07
*** jamesdenton has quit IRC19:08
*** esberglu has joined #openstack-nova19:09
*** artom_ has joined #openstack-nova19:10
sean-k-mooneymriedem: cfriesen  perhaps because you wanted to avoid qemu hosts when there are numa requests?19:10
*** artom has quit IRC19: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
cfriesensean-k-mooney: don't think so, this was specifically part of allowing one nova-scheduler to handle both libvirt/kvm and ironic compute nodes19:12
sean-k-mooneycfriesen: ah ok19:12
*** esberglu_ has joined #openstack-nova19:12
sean-k-mooneymgoddard_: 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 use19:13
mgoddard_exactly19:13
mgoddard_same with cpu pinning19:14
sean-k-mooneymgoddard_: no cpu pinning is different19:14
mgoddard_and hyperthreading19:14
cfriesenmriedem: 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-mooneycpu pinning does not make sense in a phyical server context19:14
dansmithit wouldn't matter these days anyway, ironic nodes report a CUSTOM_IRONIC_FOO resource and no cpu/mem19:14
*** esberglu has quit IRC19:14
mgoddard_sean-k-mooney: ok, you're right about pinning. hyperthreading could (but doesn't) work though19:15
dansmithcfriesen: mriedem because we store service_id in the compute node, so we won't find the existing one when we re-create the service19:15
dansmithcfriesen: 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 that19:16
sean-k-mooneymgoddard_: hyperthreading is a bios config option and should work but nova does not allow you to enable hyperthreading as a flavor extra spec19:16
dansmith(and thus placement(19:16
sean-k-mooneymgoddard_: the closest you have to that is setting the tread count in the cpu topology extra specs19:16
*** david-lyle has joined #openstack-nova19:17
mriedemi didn't realize that service_destroy also deleted the related compute node record19:17
cfriesenme either...had to go read the code.19:17
*** hemna_ has joined #openstack-nova19:18
dansmithcfriesen: I thought you said it didn't?19:18
mriedemok so if service delete also deletes the compute node record, then yeah we should also remove the compute node RP in placement19:18
mriedemi said deleting the service didn't delete the compute node record19:19
dansmithoh okay19:19
mriedemthere was some bug about a year ago we were both talking about this19:19
mriedembut https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L40419:19
cfriesenI should be able to open a bug and post a WIP fix later today19:21
mriedemhttps://github.com/openstack/nova/commit/f0d44c5b09f3f3c84038d40b621bb629a1f8110e#diff-3104166b3e802b86db6c5fa92ad08f4319:21
mriedem^ is why i thought this19:21
mriedemsee the exchange between myself and alex19:23
mriedemso in this case, they deleted the service (and compute node record) but didn't stop the nova-compute service,19:24
mriedemso it re-created the compute node record19:24
*** hongbin has quit IRC19:25
mriedemso in that bug, when they listed compute nodes, the api tries to find the related service which was deleted and the api blows up19:25
mriedembecause you don't recreate the service until you restart the service19: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 now19:25
melwittgibi: 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-summary19:26
mriedemmelwitt: he's on vacation19:26
*** hongbin has joined #openstack-nova19:26
melwitturgh right19:27
melwittgot focused only on the april vacation dates19:27
mriedemcfriesen: 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 running19:27
dansmith...and we should delete the RP in placement19:28
mriedemwhat about the host mapping?19:28
mriedemthat's also orphaned right?19:28
dansmithwe probably should delete it in placement before we delete in our db even19:28
dansmithyeah19:28
mriedemso,19:28
*** yamamoto has joined #openstack-nova19:29
mriedem1. delete RP in placement (because if we fail to delete the compute node or the service, it will get re-created),19:29
mriedem2. delete the service/node,19:29
mriedem3. delete the host mapping19:29
mriedem?19:29
dansmithwell, 1. also because if we fail, they can't do it again if we delete the local entry19:29
*** frickler has quit IRC19:29
dansmithtechnically, we can find the service record again if we delete the host mapping,19:30
dansmithso probably good to delete that first, then node, then service last of all19:30
cfriesenwhat's the host mapping used for?19:30
dansmithso placement, mapping, node, service19:30
dansmithcfriesen: finding it by name in a cell19:30
mriedemcfriesen: to map the host19:30
cfriesenah, right19:30
mriedemnode and service are in the same db api call,19:30
*** frickler has joined #openstack-nova19:30
mriedemunless we delete the compute node record(s?) directly in the api19:30
mriedemwhich would be my preference b/c the auto/implicit delete in the db api is kind of gross19:31
dansmithsame db call, but node could go first19:32
dansmithbut yeah, don't so much care about those two,19:32
dansmithas I'm not sure there are any FKs that would prevent us from deleting one and not the other19:32
*** yamamoto has quit IRC19:33
mriedemcompute_nodes.service_id doesn't have a FK on it19:33
dansmithyeah19:33
cfriesendansmith: what are the implications of a stale host mapping?19:34
*** damien_r has quit IRC19:34
dansmithcfriesen: cruft19:34
dansmithwith no easy way to find and clean it up later19:34
cfriesengot it, thanks19:35
mriedemalso,19:35
mriedemi think you could re-discover the compute host in another cell,19:35
dansmithtrue19:35
mriedemand then the old host mapping would be pointed at the wrong cell19:35
dansmithyeah19:35
mriedemi feel like cern mentioned that19:35
claudiub|2did they express a con...cern?19:36
cfriesenew19:36
mriedemcfriesen: i think this block is where you add the new deletes https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/services.py#L21319:36
dansmithclaudiub|2: nice :)19:36
mriedemclaudiub|2: excellent19:36
claudiub|2:D19:36
*** felipemonteiro_ has quit IRC19:40
*** felipemonteiro_ has joined #openstack-nova19:40
*** ircuser-1 has joined #openstack-nova19:41
*** beagles|biab is now known as beagles19:42
cfriesenmriedem: you think that's better than https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4820 ?19:48
mriedemcfriesen: could happen in compute api code too, doesn't matter a ton to me19:49
mriedemcfriesen: note that this will also need an upgrade impact release note because the api will now require placement to be configured to do this19:49
*** mgoddard_ has quit IRC19:50
mriedemi think we should backport this though...so not sure how sticky that gets19:50
cfriesenmriedem:  if placement isn't configured, wouldn't it be okay to just continue on since there wouldn't be any records?19:50
mriedemi mean [placement] configured specifically in the nova.conf you're running against nova-api,19:51
mriedemwhich hasn't technically required placement before this19:51
mriedemas far as i know19:51
mriedemconductor, compute and scheduler all require placement already, but api doesnt19:51
cfriesenah, got it.  I guess that'd be one advantage of doing it in nova-compute19:51
mriedemthere is no rpc cast from api to compute to delete itself19:51
*** tssurya has joined #openstack-nova19:52
cfriesenyeah, got confused by the nova/compute/api.py path for a sec....too many things going on today19:52
*** mvenesio has quit IRC19:53
*** dave-mccowan has quit IRC19:53
*** fragatina has quit IRC19:55
*** suresh12 has quit IRC19:57
*** suresh12 has joined #openstack-nova19:57
*** Zames has joined #openstack-nova19:58
cdentjaypipes, 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
efriedcdent: It does need an owner afaik.  And it is not immediately urgent afaik.  Thank you for volunteering.19:59
*** suresh12 has quit IRC19:59
efriedmriedem: ^19:59
*** suresh12 has joined #openstack-nova19:59
jaypipescdent: needs an owner, yes.20:00
edleafeI can pair with someone on that20:01
cfriesenmriedem: on the mixed baremetal/hypervisor scheduler thing, I suspect the NUMATopologyFilter is actually okay as long as we don't have spec_obj.numa_topology20:01
*** eharney has quit IRC20:02
*** Zames has quit IRC20:03
mriedemcfriesen: in that case the filter should just filter out an ironic host/node20:03
mriedembut that's what i was trying to figure out20:03
mriedemi.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
mriedembut i think that should be ok?20:04
cfriesenmriedem: as long as requests for ironic instances don't specify a desired numa topology then I think it should be good20:04
mriedemif we have a libvirt and ironic host as candidates, the ironic would is filtered out and the libvirt one should be ok20:04
cdentedleafe: have you got something queued up after member_of?20:04
mriedemcfriesen: well yeah, baremetal flavors shouldn't have numa things in them20:05
edleafein OpenStack, just the api-sig stuff: http and version discovery20:05
*** jmlowe has quit IRC20:06
cdentdo 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 weeks20:06
*** lpetrut has quit IRC20:06
edleafeok, I guess I'll grab it, and anyone who wants in on the fun is welcome to join20:07
openstackgerritJay Pipes proposed openstack/nova master: mirror nova host aggregate members to placement  https://review.openstack.org/55359720:08
jaypipesthx edleafe20:08
* edleafe goes to figure out just how much crap he's volunteered for...20:09
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: add a note in DELETE /os-services about deleting computes  https://review.openstack.org/55359820:10
mriedemcfriesen: ^ while i'm remembering this compute delete thing20:10
openstackgerritsahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/49745720:11
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: add a note in DELETE /os-services about deleting computes  https://review.openstack.org/55359820:11
*** dave-mccowan has joined #openstack-nova20:11
*** sahid_ has quit IRC20:15
openstackgerritsean mooney proposed openstack/nova master: add mtu to libvirt xml for ethernet and bridge types  https://review.openstack.org/55307220:24
*** yamamoto has joined #openstack-nova20:29
*** liverpooler has quit IRC20:30
*** cdent has quit IRC20:30
*** mvenesio has joined #openstack-nova20:31
*** yamamoto has quit IRC20:35
tssuryacfriesen: just read the service-delete stuff ; its also been on our to-do list; so thanks for doing it.20:37
cfriesentssurya: just filed https://bugs.launchpad.net/nova/+bug/175617920:38
openstackLaunchpad bug 1756179 in OpenStack Compute (nova) "deleting a nova-compute service leaves orphaned records in placement" [Undecided,New]20:38
tssuryacfriesen: ack20:39
mriedemcfriesen: are you also going to work on the patch?20:42
*** tbachman has quit IRC20:43
openstackgerritMichael Still proposed openstack/nova master: Move configurable mkfs to privsep.  https://review.openstack.org/55192120:43
openstackgerritMichael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep.  https://review.openstack.org/55224120:43
openstackgerritMichael Still proposed openstack/nova master: Move xenapi disk resizing to privsep.  https://review.openstack.org/55224220:43
openstackgerritMichael Still proposed openstack/nova master: Move xenapi partition copies to privsep.  https://review.openstack.org/55360520:43
openstackgerritLance Bragstad proposed openstack/nova master: Add scope_types to server policies  https://review.openstack.org/52577220:44
lbragstadcc mriedem melwitt ^ new version availble re the RBAC cross-project sessions from dublin20:45
lbragstadlet 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 questions20:45
melwittthanks for the heads up20:48
cfriesentssurya: 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
cfriesentssurya: 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 earliest20:49
tssuryacfriesen : yea, sure I can work on it20:50
tssuryaregarding removal of "service" records, when I guess it makes sense to do when force deleting the cell and host20:51
tssuryamriedem, dansmith : ^^20:51
openstackgerritLance Bragstad proposed openstack/nova master: Add scope_types to server policies  https://review.openstack.org/52577220:51
mriedemthis gets quite messy20:52
mriedembecause deleting a compute service in the api today also removes the compute host from any aggregates it's in20:52
tssuryayes it will get messy,20:52
*** dave-mccowan has quit IRC20:53
cfriesenif you're deleting a cell or a host, doesn't it make sense to delete everything logically associated with that cell/host?20:53
mriedemsure, but,20:53
mriedemwhere does it end20:53
*** mvk has quit IRC20:53
tssuryaI think it does make sense to do it, but to put it all in nova-manage... it gets dirtier and more complex20:54
mriedemif only we had, oh idk, a rest api for managing cells....20:54
tssuryamriedem: :)20:55
dansmithoh yeah, that'd make it so much better20:55
mriedemit would solve all problems20:56
mriedemcells api could be the new secretary of state20:56
mriedemRIP REX20:56
melwittnova meeting in 3 minutes20:57
*** sree has joined #openstack-nova20:58
*** s1061123 has quit IRC20:58
dansmithtssurya: 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 things21:00
dansmithcmd.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-nova21:00
tssuryaI totally agree that it is getting messier21:00
*** fragatina has joined #openstack-nova21:00
tssurya(mostly because of me in the recent months)21:01
mriedemlet's blame belmiro21:01
tssuryahehe yea!21:01
*** sree has quit IRC21:02
*** edmondsw has quit IRC21:02
openstackgerritLance Bragstad proposed openstack/nova master: Set scope_types for admin_action policies  https://review.openstack.org/55361221:06
*** mvk has joined #openstack-nova21:06
*** moshele has joined #openstack-nova21:08
openstackgerritLance Bragstad proposed openstack/nova master: Set scope_types for admin password policies  https://review.openstack.org/55361321:16
*** sar has quit IRC21:22
*** r-daneel_ has joined #openstack-nova21:23
*** liverpooler has joined #openstack-nova21:23
*** r-daneel has quit IRC21:24
*** r-daneel_ is now known as r-daneel21:24
*** mikal_ has joined #openstack-nova21:30
*** awaugama has quit IRC21:31
*** yamamoto has joined #openstack-nova21:31
*** mikal has quit IRC21:32
openstackgerritClaudiu Belu proposed openstack/nova master: db: Adds live-resize to Migration model migration_type  https://review.openstack.org/18596121:33
*** yamamoto has quit IRC21:36
*** itlinux has quit IRC21:39
*** dave-mccowan has joined #openstack-nova21:39
*** itlinux has joined #openstack-nova21:41
*** itlinux has quit IRC21:41
*** r-daneel_ has joined #openstack-nova21:41
*** r-daneel has quit IRC21:41
*** r-daneel_ is now known as r-daneel21:41
*** esberglu_ has quit IRC21:41
*** gjayavelu has quit IRC21:43
*** esberglu has joined #openstack-nova21:44
*** felipemonteiro__ has joined #openstack-nova21:45
*** esberglu_ has joined #openstack-nova21:45
*** gjayavelu has joined #openstack-nova21:45
*** gjayavelu has joined #openstack-nova21:46
*** amodi has quit IRC21:47
*** pchavva has quit IRC21:47
*** esberglu has quit IRC21:48
*** felipemonteiro_ has quit IRC21:49
*** esberglu_ has quit IRC21:50
*** gjayavelu has quit IRC21:54
*** mikal_ is now known as mikal21:55
mriedemKevin_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
mriedemhttps://bugs.launchpad.net/nova/+bug/163915221:59
openstackLaunchpad 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
dansmithmriedem: it requires conf22:00
dansmithso I would kinda expect not22:00
dansmithwe will backport it because it's important enough, but I don't know that it fits the usual backport candidate mold22:00
mriedemwe can backport bug fixes sometimes as long as the config doesn't change any default behavior22:00
*** tbachman has joined #openstack-nova22:00
dansmithit should be safe from that regard22:00
dansmithit's really a feature used to fix a bug, IMHO,22:01
mriedemby "we will backport" you mean the royal we?22:01
dansmithso a bp isn't wrong I don't think22:01
dansmithmriedem: we as in redhat22:01
mriedemthat's what i meant :)22:01
*** moshele has quit IRC22:01
mriedemalright i guess people that want it in stable can always patch out of tree if we're not going to backport upstream22:02
*** danpawlik has joined #openstack-nova22:02
dansmithI'm not saying we can't or shouldn't22:02
mriedemi heard windriver has already had this for a year22:02
*** rcernin has joined #openstack-nova22:02
dansmithI would have expected you to say no22:02
dansmithif you think it's legit, then I think it's reasonable22:02
mriedemhaving a blueprint tag on it makes it more awkward to backport22:02
*** danpawlik has quit IRC22:02
dansmithwell,22:02
mriedemi don't have the context on the thing,22:02
dansmithit's not a bug in nova22:02
mriedemjust saw 'meltdown'22:02
dansmithit's a bug in .. I dunno.. the whole arrangement of things22:03
*** tssurya has quit IRC22:03
mriedemin computers?22:03
mriedemjust say computers22:03
dansmithnova doesn't let you specify a thing that only the operator can know, and is needed to mitigate performance loss after applying the meltdown fixes22:03
dansmithit's completely a nova feature,22:03
dansmithbut operators will need it to fix a bug or regressed behavior in lots of other things (kernel, qemu, etc)22:03
mriedemhmm, this reminds me,22:04
mriedemstable/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 UCA22:04
dansmithso I guess I'd leave it as a bp and just call it out in the backport as why we're making an exception22:04
mriedemmnaser: ^22:04
mriedemyou might be interested in that22:05
mriedemhttps://review.openstack.org/#/c/534384/22:05
*** liverpooler has quit IRC22:05
* mnaser looks22:05
mriedemmnaser 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 minute22:06
mriedemdansmith: ^ sound like the same issue?22:06
dansmithlibvirt startup .. meaning a guest?22:06
*** burt has quit IRC22:06
mriedemno, libvirtd22:06
mriedemor the connection to libvirt22:06
*** liverpooler has joined #openstack-nova22:06
dansmithmm, I dunno, that's a question for kashyap I think22:07
dansmithmaybe if the fixes have been applied on the host,22:07
dansmithbut pcid isn't passed it makes our guests just super slow for some things22:07
mriedemhttps://review.openstack.org/#/c/547676/22:07
dansmithso it could be22:07
mriedemhttps://launchpad.net/~ubuntu-cloud-archive/+archive/ubuntu/ocata-staging/+sourcepub/8774271/+listing-archive-extra22:07
mriedemthe libvirt cve is i nthere22:07
dansmithI mean, that's what the assertion is about the problem we're trying to solve22:07
dansmiththat _some_ things are like 30+% slower after the fixes22:07
*** gjayavelu has joined #openstack-nova22:08
mnaserso the reason why it affected ocata and pike was because first of all devstack didnt use pike uca in pike, it used ocata uca22:08
mnaserand the older release of uca didn't have this feature where it could ask qemu what cpu features it supported22:08
mnaserso 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 etc22:09
*** tssurya has joined #openstack-nova22:09
dansmithdear god22:09
mnaserwhich would make the start up super sloooow22:09
*** masber has joined #openstack-nova22:09
dansmithso that's unrelated I believe, unless the need for the new cpu feature discovery was because of the meltdown patch,22:09
dansmithbut I expect kashyap's fix won't solve that as it's a qemu-missing-a-thing problem22:10
mriedemok22:10
*** gouthamr has quit IRC22:16
*** danpawlik has joined #openstack-nova22:17
*** tssurya has quit IRC22:19
*** claudiub|2 has quit IRC22:20
*** tbachman has quit IRC22:21
mriedemdansmith: if you guys have vgpu people internally besides bauzas that can take a look at this https://bugs.launchpad.net/nova/+bug/175246322:21
openstackLaunchpad bug 1752463 in OpenStack Compute (nova) "Attaching virtual GPU devices to guests in nova" [Medium,Incomplete]22:22
*** danpawlik has quit IRC22:22
*** mvenesio has quit IRC22:22
*** mvenesio has joined #openstack-nova22:23
melwittmriedem, 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
dansmithmelwitt: I'm cool with that yeah22:26
*** felipemonteiro__ has quit IRC22:27
dansmithmriedem: maybe, but that seems pretty low priority and can just wait for him to be back right?22:27
*** mvenesio has quit IRC22:27
*** Guest79046 has quit IRC22:28
mriedemdansmith: sure22:28
*** gjayavelu has quit IRC22:30
*** hemna_ has quit IRC22:30
*** yamamoto has joined #openstack-nova22:32
*** gjayavelu has joined #openstack-nova22:34
melwittmriedem: are you cool with the cpu model flags bp and the idea of backporting it?22:37
mriedemi'm not crazy about it being a bp if we know we're going to backport it22:37
mriedemi'd rather pretend it's not and say it's a bug fix22:38
*** yamamoto has quit IRC22:38
mriedemhttps://review.openstack.org/#/c/534384/5//COMMIT_MSG@28 definitely makes it sound feature-y too though22:39
mriedem"besides the fix, you can also configure nova for other goodies!"22:39
melwittheh. yeah. this is definitely an odd case22:39
mriedemnow, if pcid as the extra flag is the only thing you'd care about if you only care about the fix,22:40
mriedemthen this could be a [workarounds] option with a boolean to set pcid or not,22:41
mriedemand backport that22:41
mriedemthen the free-form wild west extra flags thing that is more of a feature could only be master22:41
mriedemand we'd deprecate the workaround in master22:41
mriedemdansmith: ^ what do you think about that?22:42
dansmithI dunno, it seems like too much concern over a relatively minor thing22:44
dansmithand I kinda expect that we'll backport the main thing ourselves anywaym22:44
*** mvenesio has joined #openstack-nova22:44
dansmithjust so that people with queens and people with rocky don't need a different fix for this22:44
dansmithbut if that's how it has to go, then okay22:44
mriedemi'm not the ptl, not my say, but i'd be more comfortable with that22:46
* mriedem watches that bus go by22:46
dansmithI guess my point is,22:46
dansmithwe're going to backport the meat of the feature anyway, because we need it for the workaround22:46
dansmithand constraining the workaround to one flag doesn't really do anything to make it safer,22:47
dansmithunless 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
mriedemand are all of a sudden reporting bugs about it22:47
dansmithbut if they're going to do that,22:47
dansmiththey'll do it with rocky when it's out22:47
*** dave-mccowan has quit IRC22:48
dansmithif 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
dansmiththere 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 field22:48
dansmithjust seems like it's too much monkeywork so we don't have a blueprint on a backport,22:48
mriedemwell, 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 tooling22:48
dansmithwhen we can clearly point to not-our-fault reasons why this is needed and not high risk22:48
dansmithyeah, it just doesn't seem worth the pain to me22:49
*** chyka has joined #openstack-nova22:49
mriedemprecedent is another concern for me here,22:49
mriedemtrue it's a no-op by default,22:49
cfriesendansmith: I agree with you, but I also suspect someone is going to try enabling pdpe1gb or vmx on queens.22:49
mriedembut then people could use that justification against us later22:50
dansmithcfriesen: so?22:50
melwittthe workaround idea seems good until we get to the point where people would have to go from workaround -> full feature to not regress performance22:50
cfriesendansmith: so mriedem's point about it likely causing more bug reports against stable22:50
*** suresh12 has quit IRC22:50
*** chyka_ has quit IRC22:50
mriedemmelwitt: it's just updating your configs in rocky22:50
dansmithcfriesen: but those bug reports will be against whatever release we allow the full feature in22:50
mriedemto not use the deprecated option22:50
*** hongbin has quit IRC22:50
dansmithmriedem: every time we do that it's upgrade pain and it just seems like we have too much of that already22:51
cfriesendansmith: not necessarily, nested vmx for example is buggy in older systems but may be stable by the time rocky is released22:51
melwittmriedem: but wouldn't you also have to add "pcid" to your rocky config elsewhere then?22:51
dansmithI get the precedent thing22:51
mriedemmelwitt: yes, you'd do it on the new option22:51
dansmithcfriesen: that's a super weak reason :)22:51
dansmithcfriesen: because (a) I don't think it will be and (b) people can already do crazy stuff with lots of options22:52
dansmithcfriesen: 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 way22:52
cfriesendansmith: true enough.  for what it's worth, adding pdpe1gb seems to be generally fine.22:52
dansmithheh22:53
cfriesenfor the subset of intel hosts I've dealt with anyway22:53
*** salv-orl_ has quit IRC22:53
*** salv-orlando has joined #openstack-nova22:54
melwittwe can just slap "experimental" on it on the backport, use at your own risk22:54
dansmithit's not experimental but sure :)22:54
dansmithmelwitt: I need to be done for the day anyway,22:55
melwittwell, 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 enable22:55
dansmithso you make the call, my input is all above22:55
dansmithmelwitt: it won't be any less experimental later,22:55
dansmithit just lets you choose cpu flags22:56
dansmithit will be just as dangerous in five years if you use it wrong22:56
melwittokay22:56
* dansmith &22:56
*** salv-orlando has quit IRC22:59
*** mvenesio has quit IRC22:59
*** masber has quit IRC23:02
*** suresh12 has joined #openstack-nova23:05
*** jobewan has quit IRC23:08
*** sorrison_ has quit IRC23:14
*** sticker has joined #openstack-nova23:17
*** amodi has joined #openstack-nova23:18
openstackgerritEric Fried proposed openstack/nova master: Stop assuming initial provider generation is 0  https://review.openstack.org/54897523:20
mriedemyikun: 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@4923:21
efriedjaypipes, alex_xu_: Trivial pep/tox fixes, ready for re+A  --^23:21
*** chyka has quit IRC23:25
*** yamamoto has joined #openstack-nova23:27
*** hamzy__ has quit IRC23:29
*** crushil has joined #openstack-nova23:31
*** danpawlik has joined #openstack-nova23:35
*** rcernin has quit IRC23:39
*** danpawlik has quit IRC23:39
*** mlavalle has quit IRC23:40
*** chyka has joined #openstack-nova23:47
*** tbachman has joined #openstack-nova23:48
alex_xu_efried: done23:48
*** chyka has quit IRC23:51
*** salv-orlando has joined #openstack-nova23:54
*** crushil has quit IRC23:55
*** david-lyle has quit IRC23:59
*** salv-orlando has quit IRC23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!