Friday, 2019-10-25

*** TxGirlGeek has joined #openstack-nova00:00
*** TxGirlGeek has quit IRC00:00
*** jawad_axd has joined #openstack-nova00:03
*** gyee has quit IRC00:05
*** jawad_axd has quit IRC00:08
*** slaweq has joined #openstack-nova00:17
*** andreaf has joined #openstack-nova00:18
*** slaweq has quit IRC00:22
*** jawad_axd has joined #openstack-nova00:24
*** markvoelker has joined #openstack-nova00:27
*** jawad_axd has quit IRC00:29
*** markvoelker has quit IRC00:30
*** markvoelker has joined #openstack-nova00:30
*** benj_ has quit IRC00:50
*** ricolin has joined #openstack-nova00:50
*** markvoelker has quit IRC00:54
*** ricolin has quit IRC00:55
*** markvoelker has joined #openstack-nova00:57
*** benj_ has joined #openstack-nova01:05
*** jawad_axd has joined #openstack-nova01:05
*** markvoelker has quit IRC01:08
*** dviroel has quit IRC01:08
*** jawad_axd has quit IRC01:10
*** tbachman has quit IRC01:18
*** tbachman_ has joined #openstack-nova01:18
*** tbachman has joined #openstack-nova01:26
*** tbachman_ has quit IRC01:26
*** jawad_axd has joined #openstack-nova01:26
*** jawad_axd has quit IRC01:30
*** nanzha has joined #openstack-nova01:37
*** nanzha has quit IRC01:42
*** jawad_axd has joined #openstack-nova01:47
*** nanzha has joined #openstack-nova01:49
*** jawad_axd has quit IRC01:51
*** abaindur has quit IRC01:53
*** jawad_axd has joined #openstack-nova02:08
*** jawad_axd has quit IRC02:12
*** nanzha has quit IRC02:20
*** nanzha has joined #openstack-nova02:21
*** jawad_axd has joined #openstack-nova02:28
*** jawad_axd has quit IRC02:32
*** jawad_axd has joined #openstack-nova02:49
*** jawad_axd has quit IRC02:53
*** mkrai has joined #openstack-nova03:08
*** markvoelker has joined #openstack-nova03:09
*** jawad_axd has joined #openstack-nova03:10
*** benj_ has quit IRC03:10
*** markvoelker has quit IRC03:13
*** jawad_axd has quit IRC03:14
*** psachin has joined #openstack-nova03:34
*** nanzha has quit IRC03:38
*** nanzha has joined #openstack-nova03:39
*** benj_ has joined #openstack-nova03:40
*** benj_ has quit IRC03:53
*** janki has joined #openstack-nova03:56
*** igordc has quit IRC04:01
*** benj_ has joined #openstack-nova04:05
*** benj_ has quit IRC04:10
*** VedaAnnayappa has joined #openstack-nova04:11
*** gbarros_ has quit IRC04:11
VedaAnnayappahi04:11
VedaAnnayappaI need help with tempest test bed.04:11
*** benj_ has joined #openstack-nova04:14
*** mkrai has quit IRC04:15
VedaAnnayappais there any impact on nova testcases in py37?04:17
*** janki has quit IRC04:19
*** janki has joined #openstack-nova04:22
*** ricolin has joined #openstack-nova04:24
*** VedaAnnayappa has quit IRC04:30
*** nanzha has quit IRC04:39
*** nanzha has joined #openstack-nova04:44
*** udesale has joined #openstack-nova04:44
*** mkrai has joined #openstack-nova04:54
*** tkajinam has quit IRC05:01
*** bcm has joined #openstack-nova05:07
*** hamzy has quit IRC05:19
*** hamzy has joined #openstack-nova05:21
*** janki has quit IRC05:30
*** artom has quit IRC05:31
*** janki has joined #openstack-nova05:33
*** janki has quit IRC05:33
*** janki has joined #openstack-nova05:33
openstackgerritMerged openstack/nova master: Add regression test for bug 1824435  https://review.opendev.org/68820505:36
openstackbug 1824435 in OpenStack Compute (nova) "fill_virtual_interface_list migration fails on second attempt" [Medium,In progress] https://launchpad.net/bugs/1824435 - Assigned to melanie witt (melwitt)05:36
*** janki has quit IRC05:39
*** slaweq has joined #openstack-nova05:39
*** tkajinam has joined #openstack-nova05:44
*** slaweq has quit IRC05:44
*** slaweq has joined #openstack-nova05:45
*** nanzha has quit IRC05:46
*** nanzha has joined #openstack-nova05:50
*** nanzha has quit IRC05:54
*** nanzha has joined #openstack-nova05:54
*** dpawlik has joined #openstack-nova06:17
*** tesseract has joined #openstack-nova06:19
*** maciejjozefczyk has joined #openstack-nova06:23
*** udesale has quit IRC06:24
*** udesale has joined #openstack-nova06:25
*** rcernin has quit IRC06:26
*** udesale has quit IRC06:27
*** jawad_axd has joined #openstack-nova06:52
*** udesale has joined #openstack-nova06:52
*** Dinesh_Bhor has joined #openstack-nova06:58
*** abaindur has joined #openstack-nova06:59
*** ccamacho has joined #openstack-nova07:01
*** trident has quit IRC07:02
*** trident has joined #openstack-nova07:06
*** otsukahy has joined #openstack-nova07:07
*** markvoelker has joined #openstack-nova07:11
*** slaweq has quit IRC07:12
*** ttsiouts has joined #openstack-nova07:13
*** markvoelker has quit IRC07:16
*** ttsiouts has quit IRC07:26
*** tetsuro_ has joined #openstack-nova07:26
*** ttsiouts has joined #openstack-nova07:27
*** ttsiouts has quit IRC07:28
*** ttsiouts_ has joined #openstack-nova07:28
*** tetsuro has quit IRC07:29
*** abaindur has quit IRC07:41
*** tetsuro_ has quit IRC07:42
*** tetsuro has joined #openstack-nova07:42
*** ttsiouts_ has quit IRC07:43
*** ttsiouts has joined #openstack-nova07:43
*** ttsiouts_ has joined #openstack-nova07:47
*** ttsiouts has quit IRC07:48
*** udesale has quit IRC07:49
*** udesale has joined #openstack-nova07:49
*** rpittau|afk is now known as rpittau07:51
*** zbr has joined #openstack-nova07:53
*** ralonsoh has joined #openstack-nova07:55
*** otsukahy has quit IRC07:55
*** efried has quit IRC08:02
*** efried has joined #openstack-nova08:02
*** tssurya has joined #openstack-nova08:04
*** rcernin has joined #openstack-nova08:09
*** tkajinam has quit IRC08:10
*** ttsiouts_ has quit IRC08:12
*** ttsiouts has joined #openstack-nova08:13
*** ttsiouts has quit IRC08:17
*** ivve has joined #openstack-nova08:21
*** mkrai has quit IRC08:23
*** ttsiouts has joined #openstack-nova08:24
*** rcernin has quit IRC08:28
*** trident has quit IRC08:29
*** dpawlik has quit IRC08:34
*** trident has joined #openstack-nova08:35
*** nanzha has quit IRC08:41
*** nanzha has joined #openstack-nova08:42
*** derekh has joined #openstack-nova08:43
*** tssurya_ has joined #openstack-nova08:57
*** tssurya has quit IRC08:58
*** mkrai has joined #openstack-nova09:00
*** dpawlik has joined #openstack-nova09:02
*** maciejjozefczyk has quit IRC09:09
*** markvoelker has joined #openstack-nova09:12
*** cdent has joined #openstack-nova09:16
*** markvoelker has quit IRC09:17
*** nanzha has quit IRC09:19
*** xek_ has joined #openstack-nova09:19
*** otsukahy has joined #openstack-nova09:23
*** FlorianFa has joined #openstack-nova09:28
*** nanzha has joined #openstack-nova09:31
*** abaindur has joined #openstack-nova09:42
*** lpetrut has joined #openstack-nova09:42
*** otsukahy has quit IRC09:42
*** dpawlik has quit IRC09:48
*** rcernin has joined #openstack-nova09:49
*** otsukahy has joined #openstack-nova09:49
*** otsukahy has quit IRC09:50
*** otsukahy has joined #openstack-nova09:51
*** nanzha has quit IRC09:51
*** lpetrut has quit IRC09:55
*** otsukahy has quit IRC09:56
*** nanzha has joined #openstack-nova10:01
*** mkrai has quit IRC10:02
*** rpittau is now known as rpittau|bbl10:08
*** ttsiouts has quit IRC10:11
*** ttsiouts has joined #openstack-nova10:12
*** otsukahy has joined #openstack-nova10:12
*** dtantsur|afk has quit IRC10:16
*** ttsiouts has quit IRC10:16
*** dtantsur has joined #openstack-nova10:16
*** nanzha has quit IRC10:18
*** dpawlik has joined #openstack-nova10:19
*** nanzha has joined #openstack-nova10:21
*** dpawlik has quit IRC10:24
openstackgerritMerged openstack/nova master: libvirt: Ignore volume exceptions during post_live_migration  https://review.opendev.org/68262210:28
*** otsukahy has quit IRC10:31
openstackgerritLee Yarwood proposed openstack/nova stable/train: libvirt: Ignore volume exceptions during post_live_migration  https://review.opendev.org/69128110:33
openstackgerritLee Yarwood proposed openstack/nova stable/stein: libvirt: Ignore volume exceptions during post_live_migration  https://review.opendev.org/69128210:33
openstackgerritLee Yarwood proposed openstack/nova stable/rocky: libvirt: Ignore volume exceptions during post_live_migration  https://review.opendev.org/69128310:34
openstackgerritLee Yarwood proposed openstack/nova stable/queens: libvirt: Ignore volume exceptions during post_live_migration  https://review.opendev.org/69128410:34
*** otsukahy has joined #openstack-nova10:37
*** tbachman has quit IRC10:40
*** otsukahy has quit IRC10:50
*** ccamacho has quit IRC10:55
*** mkrai has joined #openstack-nova10:58
*** lpetrut has joined #openstack-nova11:02
*** lpetrut has quit IRC11:05
*** dpawlik has joined #openstack-nova11:10
*** artom has joined #openstack-nova11:12
*** rcernin has quit IRC11:18
*** otsukahy has joined #openstack-nova11:21
sean-k-mooneyaspiers: are you around for a quick question about amd cpus and openstack11:22
sean-k-mooneyaspiers: mainly have you tried to use cpu pinning and have you turned on SMT/hyperthreading when useing openstack? if so did it all just work fine? im ignoring the slighly dumb numa topology of eypc gen 1.11:24
sean-k-mooneyaspiers: i have no reason to belive that nova will need any modifcation to work with amd eypc cpu for normal vms but im closing a ticket downstream saying i am happy to rely on the testing done by libvirt and qemu in this area.11:26
*** ccamacho has joined #openstack-nova11:27
*** ttsiouts has joined #openstack-nova11:32
*** otsukahy has quit IRC11:32
*** mkrai has quit IRC11:43
*** mdbooth has quit IRC11:44
*** otsukahy has joined #openstack-nova11:50
*** mdbooth has joined #openstack-nova11:52
*** ttsiouts has quit IRC11:54
*** otsukahy has quit IRC11:54
*** ttsiouts has joined #openstack-nova11:54
*** ttsiouts has quit IRC11:59
*** ttsiouts_ has joined #openstack-nova11:59
*** otsukahy has joined #openstack-nova12:00
*** rpittau|bbl is now known as rpittau12:04
*** artom has quit IRC12:05
*** tbachman has joined #openstack-nova12:12
*** slaweq has joined #openstack-nova12:15
*** otsukahy has quit IRC12:15
*** dave-mccowan has joined #openstack-nova12:16
*** otsukahy has joined #openstack-nova12:17
*** otsukahy has quit IRC12:23
*** otsukahy has joined #openstack-nova12:26
*** jawad_axd has quit IRC12:28
*** jawad_axd has joined #openstack-nova12:30
*** jawad_ax_ has joined #openstack-nova12:32
*** jawad_axd has quit IRC12:34
*** kaisers has joined #openstack-nova12:35
*** jawad_ax_ has quit IRC12:37
*** otsukahy has quit IRC12:37
*** ttsiouts_ has quit IRC12:38
*** ttsiouts has joined #openstack-nova12:39
*** ttsiouts has quit IRC12:43
*** jmlowe has quit IRC12:44
*** ygk_12345 has joined #openstack-nova12:47
*** FlorianFa has quit IRC12:48
*** ttsiouts has joined #openstack-nova12:49
*** dpawlik has quit IRC12:57
*** maciejjozefczyk has joined #openstack-nova12:58
*** READ10 has joined #openstack-nova12:59
*** maciejjozefczyk has quit IRC13:04
*** markvoelker has joined #openstack-nova13:15
*** mriedem has joined #openstack-nova13:17
*** henriqueof has joined #openstack-nova13:18
*** markvoelker has quit IRC13:19
*** eharney has quit IRC13:30
mriedemgibi: do you think you'll be able to get to these today? https://review.opendev.org/#/q/topic:bp/support-move-ops-with-qos-ports-ussuri+status:open13:34
ygk_12345hi all13:35
ygk_12345can someone point me to any resource in understanding the code for  nova wsgi.py file ?13:35
gibimriedem: if I don't have to solve the rollback of the parent_ifname ugliness there then yes I can fix your comments to clean up the patches13:35
mriedemyeah i'm not asking for that13:36
gibimriedem: OK, then I will go and fix those patches up13:36
*** tesseract has quit IRC13:36
* gibi puts on headphone13:36
*** tssurya_ has quit IRC13:37
*** dpawlik has joined #openstack-nova13:37
*** dviroel has joined #openstack-nova13:39
mriedemygk_12345: http://lists.openstack.org/pipermail/openstack-discuss/2019-October/010307.html may be helpful13:45
*** nweinber_ has joined #openstack-nova13:48
*** artom has joined #openstack-nova13:49
openstackgerritMerged openstack/nova master: [Trivial] Add missing ws between words  https://review.opendev.org/68992113:49
*** ttsiouts has quit IRC13:50
*** ttsiouts has joined #openstack-nova13:51
*** gbarros has joined #openstack-nova13:53
*** ygk_12345 has quit IRC13:53
*** kaisers is now known as kaisers_away13:54
*** gbarros has quit IRC13:55
*** ttsiouts has quit IRC13:55
*** kaisers_away is now known as kaisers13:56
*** kaisers is now known as kaisers_away13:56
mriedemlyarwood: very weird that your stein backport fails pep8 https://review.opendev.org/#/c/691282/ for an unused variable but it didn't on master or train,13:57
*** otsukahy has joined #openstack-nova13:57
mriedemsomething must have changed since train with flake8 rules13:57
*** ttsiouts has joined #openstack-nova13:57
mriedemmy guess would be something due to https://github.com/openstack/nova/commit/3e65f778bdb51e2cf0693f4e9b1b4539ea5c294c#diff-b91f3d5bd63fcd17221b267e851608e813:59
*** priteau has joined #openstack-nova13:59
*** mlavalle has joined #openstack-nova13:59
*** kaisers_away is now known as kaisers13:59
*** kaisers is now known as kaisers_away14:00
lyarwoodmriedem: what the flying friday14:01
mriedempycodestyle doesn't check for F84114:01
* lyarwood looks14:02
mriedemhttps://pycodestyle.readthedocs.io/en/latest/intro.html#error-codes14:02
mriedemstephenfin: ^14:03
sean-k-mooneymriedem: stephenfin is currently in japan watching the rugby world cup14:04
mriedemyeah i know he's not around14:05
*** kaisers_away is now known as kaisers14:05
*** kaisers is now known as kaisers_away14:05
bbobrovdansmith: hi, i will be fixing amd sev code14:06
*** ileixe___ has joined #openstack-nova14:07
*** benj_ has quit IRC14:08
*** kaisers_away is now known as kaisers14:09
*** gbarros has joined #openstack-nova14:09
*** kaisers is now known as kaisers_away14:09
mriedemlyarwood: if you're able it would be good to move forward on these rocky backports today https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/rocky+label:Code-Review=214:10
mriedembecause those are also in queens https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/queens14:10
*** benj has joined #openstack-nova14:11
*** benj is now known as Guest1461114:12
*** sapd1 has joined #openstack-nova14:13
*** kaisers_away is now known as kaisers14:13
lyarwoodmriedem: yup stuck in calls for a while but I'll take a look once I'm out14:13
*** kaisers is now known as kaisers_away14:14
*** kaisers_away is now known as kaisers14:14
*** kaisers is now known as kaisers_away14:14
*** kaisers_away is now known as kaisers14:14
*** mkrai has joined #openstack-nova14:14
*** markvoelker has joined #openstack-nova14:16
dansmithmriedem: thanks for that notification sample patch14:17
dansmithmriedem: so onto the more progress-related notifications, I think we could do one of two things:14:17
*** dpawlik has quit IRC14:17
dansmith1. Have the computes all emit a notification when they've downloaded an image, which won't include the aggregate14:17
dansmith2. Have the conductor emit one notification per compute after it reports what it did14:18
dansmith3. Have conductor emit a progress notification as it is doing it with numbers that can be turned into percentage (like 30% complete or N/M complete)14:18
dansmithI can see benefits and drawbacks to all of those approaches14:19
dansmith1-2 don't let you predict completion, 3 doesn't given you much info, like which ones failed14:19
openstackgerritEric Fried proposed openstack/os-traits master: Stop testing with py2  https://review.opendev.org/69099714:20
openstackgerritEric Fried proposed openstack/os-resource-classes master: Stop testing with py2  https://review.opendev.org/69099614:22
*** eharney has joined #openstack-nova14:23
openstackgerritEric Fried proposed openstack/os-vif master: Drop python2 support and testing  https://review.opendev.org/69136414:24
mriedemdansmith: couldn't 2 and 3 be combined?14:28
openstackgerritEric Fried proposed openstack/python-novaclient master: Stop supporting and testing python2  https://review.opendev.org/69136514:28
dansmithmriedem: ah, so per compute, but with progress? sure14:28
mriedemalso, i think you can do 3 without 3 using the existing notification,14:28
mriedemso you'd listen for aggregate.cache_images.start which passes the list of hosts being processed already,14:29
mriedemthen you listen for per-compute notifications for that aggregate, and since you know how many are in the aggregate from the first notification, you can do your own progress tracking14:29
mriedemyou know you're done when you get aggregate.cache_images.end14:29
mriedemso i think i'd just go with 214:29
dansmiththat is assuming you don't have multiple operations going at once and/or that you don't have overlapping aggregates, both of which I think are not reasonable14:30
mriedem1 isn't great if the compute is down14:30
dansmithnot reasonable assumptions I mean14:30
mriedemthe initial notification has the aggregate name in it though14:30
mriedemso even if you have hosts that overlap aggregates i don't think that's a problem from a tracking pov14:30
dansmithyou have to have aggregate in each notification to link all that up14:31
dansmitheither way,14:31
dansmith2 with a total and complete count makes the most sense to me I think14:31
mriedemhaving the aggregate in the per-compute notification is fine i think14:31
dansmithI wasn't sure if once per compute was too much, but sounds  like you're thinking that way14:31
*** johnthetubaguy has joined #openstack-nova14:31
mriedemyou'd have the aggregate, the compute, maybe the image(s) and the status14:32
mriedemwe could make it configurable14:32
mriedemlike we do for bdms in notifications14:32
mriedemso by default we don't send all that noise, but if you need it it's there14:32
mriedemor we wait until a consumer comes to us with their actual requirement for what they need to get out of thise14:32
mriedem*this14:32
dansmithyou really kinda need to know what the status of the images are too if you want this to be coherent14:33
dansmithwell, I have one customer requirement so I have some of that14:33
dansmithlemme run with 2+info and we can argue about the contents and/or what should be included by default or by config in gerrit14:33
mriedemfetching the image status....that means doing yet another GET on each image after we've validated them in the API which would make me sad14:34
dansmithno,14:34
dansmithI mean the result of the operation from thecompute14:34
mriedemoh yeah that's fine, i said that'd be in there above14:34
dansmithyeah14:35
*** jaosorior has joined #openstack-nova14:38
*** ccamacho has quit IRC14:39
dansmithmriedem: https://pastebin.com/cCYBiD3h14:40
dansmithemitted from conductor after each host14:40
openstackgerritEric Fried proposed openstack/os-traits master: Stop testing with py2 and switch to ussuri jobs  https://review.opendev.org/69099714:42
*** damien_r has quit IRC14:43
*** kaisers is now known as kaisers_away14:43
*** jangutter has quit IRC14:43
openstackgerritEric Fried proposed openstack/os-traits master: Stop testing with py2 and switch to ussuri jobs  https://review.opendev.org/69099714:45
openstackgerritEric Fried proposed openstack/os-resource-classes master: Stop testing with py2  https://review.opendev.org/69099614:46
gregworkhas anyone seen a situation where nova is not getting the update from cloudbase-init on the admin password on provision?  eg. running nova get-password instance mykey.pem returns nothing occasionally14:47
gregworkin the console log of the instance I see cloudbase set the admin password and notify metadata14:48
mriedemdansmith: seems reasonable14:49
mriedemgregwork: asking in #openstack-hyper-v might be more productive14:55
openstackgerritEric Fried proposed openstack/os-vif master: Drop python2 support and testing  https://review.opendev.org/69136414:55
mriedemor poking through open/merged changes https://review.opendev.org/#/q/project:x/cloudbase-init14:55
gregworkmriedem: i didnt even know that existed, thanks :)14:55
dansmithgibi: you around?14:59
gibidansmith: yep14:59
gibidansmith: I saw your convo with Matt about the instance cache notification14:59
dansmithgibi: I dunno if you noticed any of that between mriedem and I, but I'm moving on with more rich notifications for progress14:59
*** kaisers_away is now known as kaisers15:00
*** kaisers is now known as kaisers_away15:00
gibidansmith: I skimmed the convo (busy with fixing up patches) overall seems good to me15:00
gregworkmriedem: hmmn that channel has 3 people in it including me :/15:00
*** ivve has quit IRC15:00
dansmithgibi: cool, so.. I guess I need a new action for the per-host notification, but it strikes me as a good candidate to instead have a new phase, like "progress"  so... start..progress...progress...progress...end15:00
dansmithgibi: I assume we don't have that for some reason15:00
gibidansmith: there is instance.update for example so I have nothing against adding progress as well15:01
dansmithgibi: but that's an action, not a phase yeah?15:02
gibidansmith: hm. good point. your notification would be like image_cache.start, image_cache.progress ... image_cache.end15:03
dansmithgibi:  right, that's what would make the most sense to me15:03
gibidansmith: I have no problem with this either15:03
*** xek__ has joined #openstack-nova15:03
mriedemgregwork: lots of the cloudbase people are in eastern europe i think15:03
mriedemlpetrut is maybe your best bet15:04
dansmithgibi: okay I'll add it and you can see what you think in the review15:04
gregworkso this is definitely a clousbase agent problem and not a wierd nova issue i ran into ?15:04
gibidansmith: sure. I will check15:04
gregwork*cloudbase15:04
dansmithgibi: thanks15:04
mriedemgregwork: i have no idea15:04
mriedembut i'd start there rather than here15:05
*** ileixe___ has left #openstack-nova15:05
gregworkok thanks for the contact info15:05
*** xek_ has quit IRC15:06
*** sapd1 has quit IRC15:11
*** sapd1_ has quit IRC15:13
*** sapd1_ has joined #openstack-nova15:14
*** sapd1 has joined #openstack-nova15:14
*** gyee has joined #openstack-nova15:14
openstackgerritEric Fried proposed openstack/python-novaclient master: Stop supporting and testing python2  https://review.opendev.org/69136515:19
openstackgerritBalazs Gibizer proposed openstack/nova master: Allow evacuating server with port resource request  https://review.opendev.org/68838715:23
gibimriedem: one up ^^ one more to go15:24
*** sapd1 has quit IRC15:25
*** TxGirlGeek has joined #openstack-nova15:26
openstackgerritBalazs Gibizer proposed openstack/nova master: Enable evacuation with qos ports  https://review.opendev.org/68868815:32
gibimriedem: done as well ^^15:32
*** otsukahy has quit IRC15:32
*** macz has joined #openstack-nova15:33
*** ttsiouts has quit IRC15:33
openstackgerritJohn Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec  https://review.opendev.org/60220115:34
gibithis marks the end of my work week. have a nice weekend folks!15:34
*** ttsiouts has joined #openstack-nova15:34
dansmithgibi: wait15:37
dansmithWAIT15:37
dansmithgibi: in functional, I'm doing wait_for_versioned_notification() by the things it returns have empty payloads.. do I need to enable some config to have that work?15:37
*** ttsiouts has quit IRC15:38
dansmithmriedem: do you know?15:40
mriedemno, you just need to stub the fake notifier stuff15:41
dansmithokay, I got that of course and am getting the notifs but payloads are empty15:41
dansmithmaybe I'm emitting wrong15:41
dansmithmust be because the other ones have payloads15:46
dansmithdanigt15:46
dansmithhelps if I register the payload too :D15:48
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Fail compute service delete if resource provider delete fails  https://review.opendev.org/67810015:48
mriedem^ is wip until i get some reviews on what's proposed there which is based on what was discussed in the ML for not orphaning resource providers when deleting a compute service that still has allocations against it15:49
mriedemwe continue to get bugs in triage for this kind of thign15:50
mriedeme.g. https://bugs.launchpad.net/nova/+bug/184970115:50
openstackLaunchpad bug 1849701 in OpenStack Compute (nova) "Resource_provider entry related to a deleted compute node, unable to migrate vms to the node" [Undecided,New]15:50
*** jaosorior has quit IRC15:51
*** xek_ has joined #openstack-nova15:56
*** kaisers_away is now known as kaisers15:58
*** kaisers is now known as kaisers_away15:58
*** xek__ has quit IRC15:59
*** kaisers_away is now known as kaisers16:00
*** kaisers is now known as kaisers_away16:00
*** rpittau is now known as rpittau|afk16:05
openstackgerritJohn Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec  https://review.opendev.org/60220116:06
*** derekh has quit IRC16:07
*** otsukahy has joined #openstack-nova16:08
mriedemlooks like docs job is busted16:09
cdentdansmith, mriedem : when nova-compute comes up and tries to initiate talking to the conductor it uses an "early_timeout" and then after 10 attempts fails over to the global timeout. If those 10 attempts pass, is it possible the nova-compute will sit doing nothing on that long timeout for a long time before talking to the conductor, or should it loop quickly if there is an actual connection failed (rather than timing out16:10
cdentwaiting for response)? I'm reading some bugs where it seems like the sit doing nothing is happening, but it seems illogical.16:10
mriedemhttps://zuul.opendev.org/t/openstack/build/10344924bca44849b06b8fae29b5494f/log/job-output.txt#65216:10
mriedemcdent: not sure off the top of my head, probably a better question for dansmith16:11
*** otsukahy has quit IRC16:12
cdentin placement we addressed that docs issue by having a different set of requirements for docs16:13
openstackgerritMatt Riedemann proposed openstack/nova master: Enable evacuation with qos ports  https://review.opendev.org/68868816:13
openstackgerritMatt Riedemann proposed openstack/nova master: Allow evacuating server with port resource request  https://review.opendev.org/68838716:14
dansmithcdent: not sure what you mean by early timeout16:14
cdentdansmith: one sec, will get code link16:14
dansmithcdent: it should be calling ping for the conductor pretty regularly and wait forever16:14
openstackgerritMatt Riedemann proposed openstack/nova master: Enable evacuation with qos ports  https://review.opendev.org/68868816:14
cdentdansmith: https://github.com/openstack/nova/blob/master/nova/conductor/api.py#L4116:15
dansmithoh right16:16
cdentafter early attempts expires, if the global rpc timeout is huge (because people do that...) it appears like it can be a very long time before "ready" happens16:16
cdentbut I would have thought that if the conductor is literally not there, you would know sooner16:16
dansmithcdent: people who set the global timeout large are begging for problems, so I don't think we should do something different here16:16
dansmithhowever,16:16
mriedemvmware sets the timeout large because they are pulling 1K+ instances from conductor on startup yeah?16:17
cdentbut could it be that if the message bus is up but the conductor is not, the ping will hang out in rabbit... waiting16:17
dansmiththe reason for that was because if you have a power outage and start a thousand computes, they need to chill if they can't get a response right away.. just because they have a connection doesn't mean they're not overwhelming the conductor(s), which is why we have the backoff16:17
dansmithcdent: sure16:17
*** tbachman has quit IRC16:17
cdentmriedem: probably something like that, yeah16:17
cdentOkay, cool, this confirms what I thought was happening, but hadn't been able to trace it all the way down16:18
cdentmriedem: there also appears to be an ordering problem in service restarts16:18
cdentthanks to you both16:19
cdentI'll go write down more stuff that people will ignore...16:19
cdent(not you, I mean internal people)16:19
mriedem*those* people16:19
openstackgerritJohn Garbutt proposed openstack/nova master: WIP: Integrating with unified limits  https://review.opendev.org/61518016:19
cdentthem16:19
*** TxGirlGeek has quit IRC16:22
mriedemblarg, yeah we have doc/requirements.txt but https://review.opendev.org/#/c/689152/ threw requirements.txt back into the install for the docs job16:24
mriedemfixing16:24
openstackgerritDan Smith proposed openstack/nova master: Log some stats for image pre-cache  https://review.opendev.org/68817316:26
openstackgerritDan Smith proposed openstack/nova master: Add Aggregate image caching progress notifications  https://review.opendev.org/69139016:26
mriedemhttps://bugs.launchpad.net/nova/+bug/184987016:29
openstackLaunchpad bug 1849870 in OpenStack Compute (nova) "docs build failing on psycopg2 dependency" [Undecided,New]16:29
*** tbachman has joined #openstack-nova16:31
openstackgerritMatt Riedemann proposed openstack/nova master: Stop building docs with (test-)requirements.txt  https://review.opendev.org/69139116:39
mriedem^ will hopefully unblock the gate for nova ^16:39
*** nanzha has quit IRC16:41
dansmithmriedem: should we slam that in?16:41
mriedemlet's see if it passes the docs job first, it works for me locally16:42
mriedemguess it could take awhile to get a node...16:42
dansmithokay16:42
mriedemoh it got a node http://zuul.openstack.org/stream/63d6bbd171134e00877ae6ce61856a84?logfile=console.log16:42
mriedembuild baby build!16:42
*** kaisers_away is now known as kaisers16:45
mriedemdansmith: looks like it passed so you can commence to slamming16:47
*** udesale has quit IRC16:48
dansmithmriedem: you going to fix the typo?16:48
mriedemsure, sec16:48
dansmithor just clarifying for review?16:48
dansmithack16:48
openstackgerritMatt Riedemann proposed openstack/nova master: Stop building docs with (test-)requirements.txt  https://review.opendev.org/69139116:49
openstackgerritDan Smith proposed openstack/nova master: Add Aggregate image caching progress notifications  https://review.opendev.org/69139016:50
mriedemmelwitt: were you going to backport https://review.opendev.org/#/c/688206/ and the one below it?17:00
mriedemi'm planning a train release next week so probably good to get that in it17:00
melwittmriedem: yes. I wasn't sure if I should wait for the top patch to merge first before proposing17:01
mriedemi don't think you need to wait17:01
melwittok17:01
*** kaisers is now known as kaisers_away17:02
*** TxGirlGeek has joined #openstack-nova17:03
*** otsukahy has joined #openstack-nova17:04
dansmithmriedem: what's the trick for generating the notification samples?17:09
*** priteau has quit IRC17:12
*** tbachman has quit IRC17:14
*** igordc has joined #openstack-nova17:14
*** otsukahy has quit IRC17:19
*** abaindur has quit IRC17:21
*** abaindur has joined #openstack-nova17:21
*** tbachman has joined #openstack-nova17:30
mriedemyou have to create them manually17:32
mriedemthe json file i mean17:33
dansmithWAT17:33
* dansmith wants a refund17:33
mriedemall sales are final17:33
* dansmith calls the BBB17:34
dansmithif I add this is there something that will check my work?17:35
*** xek_ has quit IRC17:35
mriedemthe functional test should be comparing against the sample17:35
dansmithoh if I write that test okay17:35
*** xek_ has joined #openstack-nova17:35
dansmithI should rebase mine on yours I guess17:36
mriedemyeah so instead of this https://review.opendev.org/#/c/691390/2/nova/tests/functional/compute/test_cache_image.py17:36
mriedemright17:36
dansmithnot instead, in addition right?17:36
mriedemi don't see why we'd need both17:36
dansmithbecause it already has the error case and other-agg case17:37
*** kaisers_away is now known as kaisers17:37
mriedemthis is where the test in mine validates the sample against the actual notification generated by the test https://review.opendev.org/#/c/691129/1/nova/tests/functional/notification_sample_tests/test_aggregate.py@19917:37
*** kaisers is now known as kaisers_away17:37
mriedemoh yeah ok that makes sense17:37
*** kaisers_away is now known as kaisers17:37
*** kaisers is now known as kaisers_away17:38
*** cdent has quit IRC17:38
openstackgerritMerged openstack/nova stable/train: libvirt: Ignore volume exceptions during post_live_migration  https://review.opendev.org/69128117:39
*** ralonsoh has quit IRC17:39
*** kaisers_away is now known as kaisers17:39
*** kaisers is now known as kaisers_away17:40
openstackgerritmelanie witt proposed openstack/nova stable/train: Add regression test for bug 1824435  https://review.opendev.org/69140217:40
openstackbug 1824435 in OpenStack Compute (nova) "fill_virtual_interface_list migration fails on second attempt" [Medium,In progress] https://launchpad.net/bugs/1824435 - Assigned to melanie witt (melwitt)17:40
openstackgerritmelanie witt proposed openstack/nova stable/train: Remove redundant call to get/create default security group  https://review.opendev.org/69140317:40
openstackgerritMerged openstack/nova stable/queens: Fix unit of hw_rng:rate_period  https://review.opendev.org/68915517:43
*** jdillaman has quit IRC17:44
*** zbr has quit IRC17:45
*** kaisers_away is now known as kaisers17:46
*** kaisers is now known as kaisers_away17:46
*** ivve has joined #openstack-nova17:49
dansmithmriedem: I don't really understand why we specify the payload in a file, but also some fields of the payload to verify in the doc schema file or whatever17:50
dansmithmriedem: this stuff: https://review.opendev.org/#/c/691129/1/doc/notification_samples/aggregate-cache_images-end.json@517:51
*** kaisers_away is now known as kaisers17:58
*** kaisers is now known as kaisers_away17:59
*** kaisers_away is now known as kaisers18:03
*** kaisers is now known as kaisers_away18:03
openstackgerritDan Smith proposed openstack/nova master: Add notification sample test for aggregate.cache_images.start|end  https://review.opendev.org/69112918:04
openstackgerritDan Smith proposed openstack/nova master: Log some stats for image pre-cache  https://review.opendev.org/68817318:04
openstackgerritDan Smith proposed openstack/nova master: Add Aggregate image caching progress notifications  https://review.opendev.org/69139018:04
*** luksky has joined #openstack-nova18:09
*** TxGirlGeek has quit IRC18:14
*** psachin has quit IRC18:15
*** bnemec is now known as beekneemech18:16
*** jmlowe has joined #openstack-nova18:17
openstackgerritMerged openstack/nova master: Stop building docs with (test-)requirements.txt  https://review.opendev.org/69139118:21
mriedemdansmith: that's an override for what's in the common file18:21
mriedemotherwise you just get what's in "$ref": "common_payloads/AggregatePayload.json#",18:22
dansmithso I can just put my stuff in the common file? I mean, that seemed to work fine, but didn't know if it wasn't "right"18:22
dansmithif so, the rev I just pushed may be acceptable then18:23
mriedemit's a little weird to have that much stuff in the common file but not necessarily wrong, the only field you override is the host one18:24
*** takamatsu has quit IRC18:24
mriedemi'd probably just leave it until we have more samples that re-use the common payload and it no longer makes sense,18:25
mriedemer yeah, that's it18:25
dansmithyeah, I mean, doesn't seem like much point unless you need to sub things18:25
*** kaisers_away is now known as kaisers18:25
*** kaisers is now known as kaisers_away18:26
*** ricolin has quit IRC18:31
openstackgerritEric Fried proposed openstack/python-novaclient master: Stop supporting and testing python2  https://review.opendev.org/69136518:33
*** markvoelker has quit IRC18:36
*** markvoelker has joined #openstack-nova18:38
*** efried has quit IRC18:44
*** efried has joined #openstack-nova18:46
openstackgerritEric Fried proposed openstack/os-vif master: Drop python2 support and testing  https://review.opendev.org/69136418:56
*** gmann is now known as gmann_afk18:57
openstackgerritEric Fried proposed openstack/os-vif master: Drop python2 support and testing  https://review.opendev.org/69136419:00
openstackgerritEric Fried proposed openstack/os-vif master: Drop python2 support and testing  https://review.opendev.org/69136419:00
*** zbr has joined #openstack-nova19:01
*** eharney has quit IRC19:05
*** xek_ has quit IRC19:05
*** xek_ has joined #openstack-nova19:05
*** KeithMnemonic has quit IRC19:10
melwittI dunno if anyone is looking at the docs job failure yet, looks like psycopg missing libpq-dev package? https://zuul.opendev.org/t/openstack/build/57fdb73f89084fa9b845561ec724e11d/log/job-output.txt#66419:10
*** KeithMnemonic has joined #openstack-nova19:11
*** mkrai has quit IRC19:11
dansmithpretty sure mriedem already had a fix up for that19:11
efriedyeah, it merged19:11
efriedhttps://review.opendev.org/#/c/691391/219:12
efriedyou fast approved it dansmith :P "pretty sure" indeed.19:12
dansmithefried: I didn't look at her link before answering, hence my pretty sure19:12
*** KeithMnemonic has quit IRC19:14
*** KeithMnemonic has joined #openstack-nova19:15
*** beekneemech has quit IRC19:16
*** TxGirlGeek has joined #openstack-nova19:16
*** bnemec has joined #openstack-nova19:18
*** JamesBenson has joined #openstack-nova19:25
*** READ10 has quit IRC19:27
melwittoh ok, I missed the discussion and didn't see an ML thread19:27
melwittglad it's fixed19:28
*** lifeless has quit IRC19:29
*** TxGirlGeek has quit IRC19:36
*** tbachman has quit IRC19:37
mriedemwe SEV1 IPL'ed that APAR19:42
mriedemefried: those py2 removal patches are going to be the end of one of us19:43
efriedmriedem: I figured there would be some churn. I'm a zuul dunce to begin with.19:43
efriedbut I feel like we're ahead of the game anyway, we've got months to get it sorted.19:44
mriedemway to jinx yourself19:44
openstackgerritEric Fried proposed openstack/os-vif master: Drop python2 support and testing  https://review.opendev.org/69136419:47
melwittIPL and APAR, had to google those19:51
*** nweinber_ has quit IRC19:56
*** looking4somethin has joined #openstack-nova19:57
efriedmriedem: join us in -release if you please?19:57
looking4somethinyes19:58
looking4somethinmriedem are you talking to me?19:59
efriedlooking4somethin: sorry, you arrived in the middle of a conversation20:01
*** jawad_axd has joined #openstack-nova20:01
looking4somethinawesome!20:01
efriedlooking4somethin: Have you tried randomizing your placement allocation candidates?20:02
efriedlanding on one compute isn't really a bug20:02
efriedI can't remember if we pack or spread by default -- dansmith?20:02
looking4somethinnot really20:02
looking4somethinhow can I do it.20:02
melwittefried: pack is default20:03
*** artom has quit IRC20:03
efriedlooking4somethin: there you go ^ so by default you'll go to one compute until it's "full".20:03
efriedIf you prefer to spread, you can do things with weighers.20:03
melwittyou need to adjust some config options if you would prefer VMs to spread first onto compute hosts20:04
looking4somethinthat's it then. however I have observed some spreading previously but right now one compute is getting everything.20:04
melwittyou can use https://docs.openstack.org/nova/rocky/configuration/config.html#placement.randomize_allocation_candidates to have placement randomize the computes it returns to nova20:05
melwittyou can use https://docs.openstack.org/nova/rocky/configuration/config.html#filter_scheduler.host_subset_size to make nova choose randomly from a larger subset of hosts20:06
efriedmelwitt: If there's a weigher in play that packs by default, that won't actually help (unless the deployment is bigger than the ?limit)20:06
melwittoh, I see20:06
melwittyeah20:06
efriedto spread, I thought you had to implement a weigher that weighs emptier hosts heavier.20:07
looking4somethinefried how I can do that20:07
melwittyou may have to adjust individual weighers multipliers if any of them are doing packing behavior20:07
efriedblind leading blind here...20:07
looking4somethinto use the emptier hosts above all others20:07
*** eharney has joined #openstack-nova20:08
looking4somethinin nova.conf right?20:08
melwittthere is also https://docs.openstack.org/nova/rocky/configuration/config.html#filter_scheduler.shuffle_best_same_weighed_hosts if you want to randomize hosts that are all weighed the same20:08
melwittyeah nova.conf20:08
efriedhm, CPUWeigher has a docstring that says spread is the default.20:08
efriedditto RAMWeigher20:09
melwitthere's the docs for all the weighers so you have to read those to see which one might be causing behavior you don't like by default https://docs.openstack.org/nova/latest/user/filter-scheduler.html#weights20:09
efriedphew, an rtfm answer, thanks melwitt20:09
looking4somethinthanks melwitt20:10
*** gmann_afk is now known as gmann20:11
looking4somethinhow can I replace (in nova.conf) nova.scheduler.weights.all_weighers by nova.scheduler.weights.CPUWeigher  or just to select one weigher20:18
looking4somethinI want to allocate to a node that have the most vcpus available20:19
*** mdbooth has quit IRC20:19
efriedlooking4somethin: It looks to me like you would say20:24
efried[scheduler]20:24
efriedweight_classes = 'nova.scheduler.weights.cpu.CPUWeigher'20:24
*** maciejjozefczyk has joined #openstack-nova20:25
looking4somethinefried I will try it tomorrow, thanks20:31
efriedgood luck20:31
efriedsweet dreams20:31
efried(are made of this)20:31
mriedemmelwitt: efried: isn't that twice this week someone has been in here asking how to randomize allocations? wasn't eandersson asking about the same thing?20:34
mriedemmaybe we need a more obvious doc about the various knobs used to make that happen20:35
mriedemsince it seems like there are a few things to look at20:35
efriedexcept I think in both cases randomize turned out not to be the answer.20:35
mriedemwell, i don't mean the actual placement randomize option,20:35
mriedembut shuffling a host within the same weighted subset20:35
eanderssonThere are three options, and all seem very similar to me (especially if not reading the source code first)20:36
*** looking4somethin has quit IRC20:36
mriedemmy point is, can we get people to (1) stop reading source code and (2) find what they are looking for in the docs?20:36
eandersson100% this ^20:36
mriedemi'm thinking about it b/c i've been working on a troubleshooting doc for orphaned allocations20:36
mriedemwhich i'm about to birth20:37
* mriedem gets the ice chips20:37
eanderssonbtw we did like 6k cold migrations and live migrations with Rocky20:38
eanderssonDon't ask me why :p20:39
eanderssonAnd 99% were successful20:39
eanderssonWe had some really bad failures20:39
eanderssonThe worst was some sort of neutron port corruption20:39
eanderssonwhere the port was on the wrong compute, and in some cases completely corrupt in the neutron db20:39
eanderssonBut at least no orphaned placement allocations so that is great20:40
openstackgerritMatt Riedemann proposed openstack/nova master: doc: add troubleshooting guide for cleaning up orphaned allocations  https://review.opendev.org/69142720:44
mriedemeandersson: port issues with the host binding details specifically? because someone else had reported issues with live migrations and weird port binding UniqueConstraint issues20:45
eanderssonYea I think so20:45
eanderssonI can probably dig up some more details20:45
eanderssonbut nothing much in the logs unfortunately20:45
*** jawad_axd has quit IRC20:46
eanderssonlive migrations actually worked pretty well overall in Rocky20:46
eanderssonBut only when emptying one host and live migrating all of them into an empty host20:47
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: libvirt: Ignore volume exceptions during post_live_migration  https://review.opendev.org/69128220:47
*** kaisers_away is now known as kaisers20:47
*** kaisers is now known as kaisers_away20:48
mriedemeandersson: https://review.opendev.org/#/c/653506/20:48
mriedem^ was the solution for the other person that reported that issue20:48
mriedemso far obviously there hasn't been enough motivation to pick that back up and address comments20:49
eanderssonI am not too  worried about it at the moment as live migrations only works in the above scenario for us20:50
eanderssonbut maybe I can upgrade to Train in the lab and see if I can reproduce it.20:50
*** markvoelker has quit IRC20:50
eanderssonAnd test that patch out.20:50
*** KeithMnemonic has quit IRC21:02
mriedemi'm out21:08
mriedemo/21:08
*** mriedem has quit IRC21:08
*** markvoelker has joined #openstack-nova21:23
*** maciejjozefczyk has quit IRC21:26
*** markvoelker has quit IRC21:33
*** xek_ has quit IRC21:41
*** xek has joined #openstack-nova21:42
*** JamesBen_ has joined #openstack-nova21:59
*** JamesBenson has quit IRC22:00
*** JamesBen_ has quit IRC22:04
*** ociuhandu has joined #openstack-nova22:43
*** kaisers_away is now known as kaisers22:45
*** kaisers is now known as kaisers_away22:45
*** ivve has quit IRC22:47
*** gyee has quit IRC22:53
*** tbachman has joined #openstack-nova22:55
*** macz has quit IRC22:55
*** ociuhandu has quit IRC23:04
*** luksky has quit IRC23:07
*** jawad_axd has joined #openstack-nova23:11
*** jawad_axd has quit IRC23:16
*** markvoelker has joined #openstack-nova23:16
*** markvoelker has quit IRC23:28
*** tbachman has quit IRC23:36
openstackgerritMerged openstack/nova stable/rocky: cleanup evacuated instances not on hypervisor  https://review.opendev.org/68755023:37
openstackgerritMerged openstack/nova stable/rocky: Add functional test for resize crash compute restart revert  https://review.opendev.org/68756323:37
openstackgerritMerged openstack/nova stable/rocky: Prevent init_host test to interfere with other tests  https://review.opendev.org/68786223:37
*** igordc has quit IRC23:42
*** jawad_axd has joined #openstack-nova23:51
*** jawad_axd has quit IRC23:55

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