Friday, 2019-01-25

*** mmethot has quit IRC00:00
*** mmethot has joined #openstack-nova00:00
*** mmethot has quit IRC00:04
*** macza has quit IRC00:05
*** slaweq has joined #openstack-nova00:11
*** slaweq has quit IRC00:15
*** ade_lee has quit IRC00:16
*** breton has joined #openstack-nova00:16
*** ade_lee has joined #openstack-nova00:16
*** rcernin has quit IRC00:18
*** rcernin has joined #openstack-nova00:19
*** markvoelker has quit IRC00:20
*** erlon has quit IRC00:29
*** dklyle has quit IRC00:32
*** macza has joined #openstack-nova00:33
*** macza has quit IRC00:34
*** efried_2dmtg has quit IRC00:35
*** macza has joined #openstack-nova00:37
*** dklyle has joined #openstack-nova00:48
*** wolverineav has quit IRC00:50
*** tstrul has joined #openstack-nova00:52
*** TxGirlGeek has joined #openstack-nova00:54
*** tstrul has quit IRC00:55
*** tstrul has joined #openstack-nova00:55
*** erlon has joined #openstack-nova00:56
*** khomesh|gone is now known as khomesh00:57
*** tstrul has quit IRC00:58
*** tstrul has joined #openstack-nova00:59
*** tstrul has quit IRC00:59
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9)  https://review.openstack.org/57558101:03
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10)  https://review.openstack.org/57601701:03
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11)  https://review.openstack.org/57601801:03
*** wolverineav has joined #openstack-nova01:04
*** whoami-rajat has joined #openstack-nova01:06
*** _fragatina has quit IRC01:08
*** gyee has quit IRC01:10
*** dklyle has quit IRC01:10
*** tbachman has joined #openstack-nova01:13
*** awalende has joined #openstack-nova01:15
*** awalende has quit IRC01:19
*** _fragatina has joined #openstack-nova01:28
openstackgerritmelanie witt proposed openstack/nova master: Raise 403 instead of 500 error from attach volume API  https://review.openstack.org/63290401:29
openstackgerritmelanie witt proposed openstack/nova master: Add configuration of maximum disk devices to attach  https://review.openstack.org/61677701:29
*** TxGirlGeek has quit IRC01:30
*** moshele has joined #openstack-nova01:33
*** _fragatina has quit IRC01:37
*** moshele has quit IRC01:40
*** moshele has joined #openstack-nova01:41
*** ileixe has joined #openstack-nova01:51
*** lbragstad has quit IRC01:52
*** cfriesen has quit IRC01:55
*** wolverineav has quit IRC01:56
*** igordc has quit IRC01:57
*** wolverineav has joined #openstack-nova02:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12)  https://review.openstack.org/57601902:08
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13)  https://review.openstack.org/57602002:08
*** sapd1 has joined #openstack-nova02:08
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14)  https://review.openstack.org/57602702:08
*** khomesh is now known as khomesh|afk02:08
*** khomesh|afk is now known as khomesh02:10
*** wolverineav has quit IRC02:11
*** slaweq has joined #openstack-nova02:11
*** moshele has quit IRC02:11
*** slaweq has quit IRC02:15
*** mmethot has joined #openstack-nova02:16
*** bhagyashris has joined #openstack-nova02:19
*** markvoelker has joined #openstack-nova02:25
*** hongbin has joined #openstack-nova02:25
*** lbragstad has joined #openstack-nova02:35
*** Dinesh_Bhor has joined #openstack-nova02:40
openstackgerritTakashi NATSUME proposed openstack/nova master: Make quota options deprecated  https://review.openstack.org/63310202:44
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15)  https://review.openstack.org/57603102:45
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16)  https://review.openstack.org/57629902:45
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17)  https://review.openstack.org/57634402:46
*** wolverineav has joined #openstack-nova02:47
*** artom has quit IRC02:50
*** markvoelker has quit IRC02:55
*** ebbex has quit IRC02:55
*** bhagyashris has quit IRC02:57
*** psachin has joined #openstack-nova03:04
*** sapd1 has quit IRC03:04
*** udesale has joined #openstack-nova03:06
*** khomesh is now known as khomesh|bfst03:09
openstackgerritZhang Hua proposed openstack/nova stable/pike: Refix disk size during live migration with disk over-commit  https://review.openstack.org/63137203:09
openstackgerritZhang Hua proposed openstack/nova stable/ocata: Fix disk size during live migration with disk over-commit  https://review.openstack.org/53074403:13
*** whoami-rajat has quit IRC03:15
*** hongbin_ has joined #openstack-nova03:19
*** sapd1 has joined #openstack-nova03:21
*** hongbin has quit IRC03:21
openstackgerritZhang Hua proposed openstack/nova stable/ocata: Refix disk size during live migration with disk over-commit  https://review.openstack.org/63137603:24
openstackgerritZhang Hua proposed openstack/nova stable/pike: Fix disk size during live migration with disk over-commit  https://review.openstack.org/53074303:25
*** khomesh|bfst is now known as khomesh03:31
*** dklyle has joined #openstack-nova03:40
*** whoami-rajat has joined #openstack-nova03:45
*** itlinux has joined #openstack-nova03:47
*** markvoelker has joined #openstack-nova03:52
*** dklyle has quit IRC04:02
*** bhagyashris has joined #openstack-nova04:06
*** slaweq has joined #openstack-nova04:11
openstackgerritTakashi NATSUME proposed openstack/nova master: Add minimum value in maximum_instance_delete_attempts  https://review.openstack.org/63308504:15
*** khomesh is now known as khomesh|afk04:16
*** slaweq has quit IRC04:16
*** khomesh|afk is now known as khomesh04:18
*** markvoelker has quit IRC04:25
*** ileixe has quit IRC04:26
*** sapd1 has quit IRC04:32
*** wolverineav has quit IRC04:43
*** hongbin_ has quit IRC04:43
*** sapd1 has joined #openstack-nova04:45
*** wolverineav has joined #openstack-nova04:46
*** wolverineav has quit IRC04:50
*** khomesh is now known as khomesh|afk04:53
*** wolverineav has joined #openstack-nova04:54
*** macza has quit IRC04:55
*** imacdonn has joined #openstack-nova04:57
*** udesale has quit IRC05:01
*** udesale has joined #openstack-nova05:04
*** ileixe has joined #openstack-nova05:08
*** sapd1 has quit IRC05:08
*** slaweq has joined #openstack-nova05:11
*** khomesh|afk is now known as khomesh05:12
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18)  https://review.openstack.org/57667305:14
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19)  https://review.openstack.org/57667605:15
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20)  https://review.openstack.org/57668905:15
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21)  https://review.openstack.org/57670905:15
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22)  https://review.openstack.org/57671205:16
*** slaweq has quit IRC05:16
openstackgerritYongli He proposed openstack/nova master: Add server subresouce toplogy API  https://review.openstack.org/62147605:16
*** markvoelker has joined #openstack-nova05:22
*** _fragatina has joined #openstack-nova05:27
*** macza has joined #openstack-nova05:39
openstackgerritYongli He proposed openstack/nova master: Adds the server group info into show server detail API.  https://review.openstack.org/62147405:40
*** spsurya has joined #openstack-nova05:40
openstackgerritTakashi NATSUME proposed openstack/nova master: Add a warning for max_concurrent_live_migrations  https://review.openstack.org/63224005:43
*** udesale has quit IRC05:45
*** sapd1 has joined #openstack-nova05:45
*** udesale has joined #openstack-nova05:46
*** wolverineav has quit IRC05:54
*** markvoelker has quit IRC05:54
*** wolverineav has joined #openstack-nova05:55
*** wolverineav has quit IRC05:59
*** wolverineav has joined #openstack-nova06:01
*** wolverineav has quit IRC06:06
*** slaweq has joined #openstack-nova06:11
*** slaweq has quit IRC06:15
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove unused quota options  https://review.openstack.org/63310206:17
*** macza_ has joined #openstack-nova06:24
*** macza has quit IRC06:28
*** macza_ has quit IRC06:28
*** wolverineav has joined #openstack-nova06:35
*** takashin has left #openstack-nova06:45
*** sridharg has joined #openstack-nova07:11
*** Luzi has joined #openstack-nova07:12
*** ociuhandu has joined #openstack-nova07:20
*** ociuhandu_ has quit IRC07:20
*** Luzi has quit IRC07:31
*** ebbex has joined #openstack-nova07:35
*** Luzi has joined #openstack-nova07:45
*** lbragstad has quit IRC07:45
*** igordc has joined #openstack-nova07:48
*** wolverineav has quit IRC07:51
*** markvoelker has joined #openstack-nova07:52
*** slaweq has joined #openstack-nova08:00
*** panda|off is now known as panda08:04
*** moshele has joined #openstack-nova08:06
*** awalende has joined #openstack-nova08:07
*** igordc has quit IRC08:08
*** ralonsoh has joined #openstack-nova08:09
*** rpittau has joined #openstack-nova08:10
*** helenaAM has joined #openstack-nova08:11
*** ccamacho has joined #openstack-nova08:13
*** tkajinam has quit IRC08:15
*** sapd1 has quit IRC08:22
*** moshele has quit IRC08:23
*** yan0s has joined #openstack-nova08:23
*** markvoelker has quit IRC08:25
openstackgerritGhanshyam Mann proposed openstack/nova stable/queens: Migrate nova v2.0 legacy job to zuulv3  https://review.openstack.org/62057808:36
openstackgerritGhanshyam Mann proposed openstack/nova stable/pike: Migrate nova v2.0 legacy job to zuulv3  https://review.openstack.org/62057908:37
openstackgerritDongcan Ye proposed openstack/nova master: Raise BuildAbortException while updating instance task_state conflict  https://review.openstack.org/63316008:37
*** sapd1 has joined #openstack-nova08:41
*** tosky has joined #openstack-nova08:49
*** dklyle has joined #openstack-nova08:50
*** s10 has joined #openstack-nova08:50
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script  https://review.openstack.org/61416708:54
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal  https://review.openstack.org/59160708:54
*** abhishekk has joined #openstack-nova09:07
*** moshele has joined #openstack-nova09:10
*** _fragatina has quit IRC09:18
*** _fragatina has joined #openstack-nova09:18
*** _fragatina has quit IRC09:20
*** _fragatina has joined #openstack-nova09:21
*** markvoelker has joined #openstack-nova09:22
*** rtjure has quit IRC09:24
*** rtjure has joined #openstack-nova09:26
*** udesale has quit IRC09:29
*** udesale has joined #openstack-nova09:30
*** derekh has joined #openstack-nova09:31
*** erlon has quit IRC09:32
*** moshele has quit IRC09:37
*** ondrejme has joined #openstack-nova09:41
*** _fragatina has quit IRC09:47
*** rpittau has quit IRC09:51
*** rpittau has joined #openstack-nova09:53
*** markvoelker has quit IRC09:55
*** bhagyashris has quit IRC09:57
stephenfinadrianc: Sweet. I have another old ConnectX-3 NIC ordered, so I should be able to test it myself as soon as that arrives10:08
sean-k-mooneystephenfin: the exra docs for sriov that adrianc was working on?10:09
sean-k-mooneyor live migration10:09
stephenfinLive migration10:09
stephenfinAlso the docs though, I guess?10:09
sean-k-mooneyah10:09
sean-k-mooneystephenfin: https://review.openstack.org/#/c/627312/10:10
sean-k-mooneystephenfin: hopefully they will help you set up the new card when it arrives10:11
*** s10 has quit IRC10:11
stephenfinsean-k-mooney++ Very useful indeed10:13
*** Dinesh_Bhor has quit IRC10:19
*** erlon has joined #openstack-nova10:22
*** erlon has quit IRC10:22
*** erlon has joined #openstack-nova10:23
*** ioni has joined #openstack-nova10:23
*** erlon has quit IRC10:24
*** khomesh is now known as khomesh|afk10:24
*** erlon has joined #openstack-nova10:24
ionihello guys10:33
*** erlon_ has joined #openstack-nova10:33
ionii have a lot of warnings regarding resources not beeing updated after migraton(i think the migration had an error regarding neutron port allocation) but the instance ended up on a new compute node10:34
ionihttps://paste.xinu.at/ecVs8/10:34
ioniis there a way to sync the used resources ?10:34
ionii'm running queens at this time10:35
*** erlon has quit IRC10:36
openstackgerritKevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter  https://review.openstack.org/63317810:38
*** sapd1 has quit IRC10:43
*** khomesh|afk is now known as khomesh10:45
*** macza has joined #openstack-nova10:47
*** khomesh is now known as khomesh|afk10:47
*** macza_ has joined #openstack-nova10:48
*** macza has quit IRC10:51
*** markvoelker has joined #openstack-nova10:52
*** macza_ has quit IRC10:52
*** gibi has quit IRC10:54
*** gibi has joined #openstack-nova10:54
*** khomesh|afk is now known as khomesh10:55
openstackgerritBalazs Gibizer proposed openstack/nova master: Add functional test for libvirt vgpu reshape  https://review.openstack.org/63155911:04
*** abhishekk has quit IRC11:07
*** jaosorior has quit IRC11:07
*** ociuhandu has quit IRC11:07
*** ileixe has quit IRC11:19
openstackgerritStephen Finucane proposed openstack/nova master: Handle libvirt reporting incorrect 4k page quantities  https://review.openstack.org/63103811:24
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Split '_get_host_numa_topology' function  https://review.openstack.org/63123711:24
openstackgerritStephen Finucane proposed openstack/nova master: Make overcommit check for pinned instance pagesize aware  https://review.openstack.org/63105311:24
*** markvoelker has quit IRC11:25
*** tbachman has quit IRC11:44
*** khomesh is now known as khomesh|gone11:49
kashyapsean-k-mooney: Hi, something seems to be off in your line-break formatting in the LP bug comments (https://bugs.launchpad.net/nova/+bug/1811870)11:49
openstackLaunchpad bug 1811870 in OpenStack Compute (nova) "libvirt reporting incorrect value of 4k (small) pages" [High,In progress] - Assigned to Stephen Finucane (stephenfinucane)11:49
kashyapIn comment#1, the line breaks in a paragraph seem to come in randomly, which slightly impairs readability.  :-)11:50
sean-k-mooneythat because for longer comment i type them up somewhere else then copy past and the web form messes up the original formating i had11:53
ionii found that allocation refresh is not requred on compute nodes greater than pike: https://git.openstack.org/cgit/openstack/nova/commit/?id=c3eb6ea3f6f31ae9473d0d2a4fa3f7c3a590d5f211:54
sean-k-mooneythings like text formating is very unimportant to me so i rarely if ever check if it messed it up before i leave teh comment11:54
ioniis it safe to run refresh allocation on queens if already have resouces that aren't deallocated after a migration?11:55
sean-k-mooneyioni: you mean the placement refresh11:55
ionisean-k-mooney, i'm not sure: https://paste.xinu.at/ecVs8/11:56
ionii have this warnings11:56
sean-k-mooneynon of the placement folks seam to be about at the momment. i would ping cdent or efried later when they come online11:58
ionii guess is about allocations11:58
ionisean-k-mooney, ok, i'll wait12:00
sean-k-mooneyjaypipes: you might know the answer as to how to correct https://paste.xinu.at/ecVs8/12:01
sean-k-mooneyioni: jaypipes is logged in to irc but its still early for him12:01
ioniit's fine, i can wait12:03
ionigood thing that you guys changed from debug to warning12:03
ionii had problems in the past scheduling instances on some hypervisors12:03
ionibut now i know why12:03
*** artom has joined #openstack-nova12:04
sean-k-mooneyioni: if you are sure the instance is not running on the host you might be able to manually clean up old allocation safely via the osc placemnet cli12:10
sean-k-mooneyi dont know if there is a way to get nova do do it however12:10
ionisean-k-mooney, pretty sure is not on that hypervisor. openstack server show it's showing the new one and i can see the libvirt file on the node12:11
ionithe only thing is that the resources weren't deallocated from the old one12:12
ionithat warning12:12
ionii cannot find where in databases i need to update12:12
ionii tried to find it in the tables without success12:12
sean-k-mooneyya in which case its probaly safe to manually remove the allocation via  the placemnt api12:12
sean-k-mooneyi would avoid direct db updates unless it cant be fixed via the api12:13
sean-k-mooneyioni: if you ask in #openstack-placement there maybe someone about who can suggest hot to heal the allocations correctly12:13
ioniok, thanks12:13
sean-k-mooneyioni: looks like edleafe: is logged on but its seams pretty quite12:14
ionisean-k-mooney, alright, i have asked in the channel12:18
*** cdent has joined #openstack-nova12:19
*** wolverineav has joined #openstack-nova12:28
*** tssurya has joined #openstack-nova12:30
openstackgerritMartin Midolesov proposed openstack/nova master: Implementing graceful shutdown.  https://review.openstack.org/60870412:31
*** wolverineav has quit IRC12:32
openstackgerritMerged openstack/nova master: Move interface enabling to privsep.  https://review.openstack.org/62152812:32
kashyapsean-k-mooney: Yeah, FWIW, I do that too, just write the comment in `vim`, and wrap the lines to 72 (or your choice) & then copy / paste.12:33
sean-k-mooneykashyap: my choise would be ~15012:33
kashyapHehe12:33
sean-k-mooneyi hate the 80 colum limit and i hate passionetly that we test for 79 is pep8 for some reason even when we say it should be 8012:34
stephenfinsean-k-mooney: That's easily fixed, if you were so inclined12:37
sean-k-mooneyya it should be12:39
sean-k-mooneyi think its testing <80 instead of <=8012:39
sean-k-mooneywe dont explcitly set it12:39
janguttersean-k-mooney: 79 is clearly better because it's a prime number. 80 just has too many factors and it's not a pure power of two either.12:41
sean-k-mooneytrue but in that case i vote for rounding up not down :)12:43
janguttersean-k-mooney: 80's possibly a holdover from 4:3 aspect ratio, but it still makes sense for people holding their phone in portrait mode.12:44
janguttersean-k-mooney: that's actually an argument to go for a 60 column limit.12:45
sean-k-mooneythere is also an argument to got to 160 or 190 the fortran uses12:45
sean-k-mooneypep8 package doc example seam to encurate 160 indretly https://pep8.readthedocs.io/en/release-1.7.x/intro.html#configuration12:47
janguttersean-k-mooney: I just had a fringe idea - what if 79 characters is because there's a secret conspiracy to allow CRLF line endings?12:48
sean-k-mooneythey are both non pritable chariters so it does not matter12:49
sean-k-mooneyanyway i need to go rebase some things and fix some typos :)12:49
janguttersean-k-mooney: without nits like these to pick on Fridays, the world would be a lot more boring. Good luck!12:50
cdentthe world is unable to avoid this conversation isn't it12:51
cdentI get unhappy when code lines are longer than 50 characters or so.12:51
* cdent really likes vertical code12:51
cdentMy standards have completely collapsed since joining the openstack scene12:52
janguttercdent: I notice you didn't specifically say _coding_ standards.12:55
*** whoami-rajat has quit IRC12:55
cdentjangutter: that's correct12:56
sean-k-mooneycdent: i hate wasting vertical space it almost -2 able in my book12:56
cdentsean-k-mooney I WILL FIGHT YOU OVER THIS :)12:56
sean-k-mooneyits not but it severly hurts at leas my readablity of the code12:56
sean-k-mooneycdent: :)12:57
cdentI tend to associated wide code with not extracting methods enough12:57
janguttercdent: one more level of indirection can't hurt, right?12:57
openstackgerritStephen Finucane proposed openstack/nova stable/rocky: Fix typo  https://review.openstack.org/63319612:58
openstackgerritStephen Finucane proposed openstack/nova stable/rocky: Fix overcommit for NUMA-based instances  https://review.openstack.org/63319712:58
cdentbut interestingly it also has a lot to do with readability for me too: if I have to go to the right I get confused. sean-k-mooney if we combined our brains we'd either be able to read nothing or everything, not sure12:58
sean-k-mooneycdent: that can be true but when we have function like post_live_migration_at_destination(...)12:58
sean-k-mooneyi can sometimes be hard to call them with out it requreing multiple lines12:58
stephenfinlyarwood: Any chance you could you take a look at those two backports today?12:59
janguttercdent: I tend to notice something's wrong when the member I'm looking up gets to about 50 characters.12:59
sean-k-mooneycdent: it only becomes annoying to me or hard to read when a function has multiple argumetns and i can nolonger clearly see everything returns and accpes at once13:00
sean-k-mooneyif the lines are too short or too long it become an issue but its also an issue if we have lost of indetation13:00
sean-k-mooneyfor example in conditional within a fucntion within a fuction within a class.13:01
sean-k-mooneyin which case the satment may be only 30 letters wide but its wrapped because of the nested ness of the code13:02
*** avolkov has joined #openstack-nova13:02
* cdent nods13:02
sean-k-mooneyi think the best evolutionin in modern programming was clang-format13:03
jangutterI've seen code (and yonks ago, written) with indents being 2 spaces. Doing it once makes you forever want 4 or more.13:03
sean-k-mooneyit inspired go to creat go formate and decide its a tools proplem to make the code consitent and readable13:03
*** _fragatina has joined #openstack-nova13:03
*** _fragatina has quit IRC13:04
sean-k-mooneyjangutter: i think the kernel still has an intend of 8 to discurabe deep nesting13:04
*** _fragatina has joined #openstack-nova13:04
janguttersean-k-mooney: yep, indents shall be 1 tab and tabs are 8 characters wide like $deity intended.13:04
sean-k-mooneyyaml uses indentaion of 2 spaces i think by convention13:05
*** tbachman has joined #openstack-nova13:11
*** mriedem has joined #openstack-nova13:12
*** ccamacho has quit IRC13:14
*** cdent has quit IRC13:14
*** tbachman has quit IRC13:15
mdboothsean-k-mooney: I think 80 is arbitrary these days, but having a width limit around there does make sense even with very wide monitors, as it allows you to put multiple files next to each other. Most lines are not that long, so if you allow the occasional one to be that long you end up with truncation when viewing multiple files. Basically: imposing a (arbitrary) limit makes the code denser on screen, which is good no matter how big13:17
mdboothyour monitor is.13:17
mdboothMy 2c.13:17
sean-k-mooneymdbooth: well im not saying we sure raise it even if i personally would it jsut annoys me that wehn reviewing in gerit or using an ide that set the 80 colum marker that the ci is actully enforice 7913:18
sean-k-mooneyyou look at it either locally or in gerrit and it looks like it does not cross the virtical line that is rendered but actully its still rejected13:20
mdboothsean-k-mooney: I have vim set to show modifications in the margin, so my terminals all have to be 82 chars anyway :) I don't care what the actual value is, tbh, as long as it exists and is consistent.13:20
sean-k-mooneyya its why i have never bother to "fix" it13:20
mdboothOk, yeah, I can see that would be annoying if all the tools default to 80.13:21
lyarwoodstephenfin: ack will do13:21
*** cdent has joined #openstack-nova13:21
*** tbachman has joined #openstack-nova13:21
*** awalende has quit IRC13:26
*** awalende has joined #openstack-nova13:27
*** ccamacho has joined #openstack-nova13:29
*** rcernin has quit IRC13:30
*** sridharg has quit IRC13:31
*** awalende has quit IRC13:31
stephenfinTIL the 79 character default is in flake8, not tox.ini/setup.cfg13:39
stephenfinsean-k-mooney: Fancy looking at https://review.openstack.org/#/c/631053/ and predecessors again today/early next week?13:40
jangutterAnd 79 is actually codified in pep8 itself https://www.python.org/dev/peps/pep-0008/#maximum-line-length13:40
sean-k-mooneysure i was just looking at your chages for requiring the plugin atribute for os-vif and the noop plugin13:40
sean-k-mooneyjangutter: ya they choose 79 so that if you have emacs in whitespace-newline-mode where it add a $ to mark the end of the line it wont wrap13:42
jangutterthink if we type 79 enough, 80 is going to look weird to sean-k-mooney?13:42
sean-k-mooneyjangutter: but all modern tools use 80 as teh defulat so its annoying13:43
*** eharney has joined #openstack-nova13:44
*** jaypipes is now known as leakypipes13:50
lyarwoodstephenfin: https://review.openstack.org/#/c/633197/1/nova/objects/numa.py - stupid o.vo question in there if you have a second, I assume it's fine just wanted to check.13:52
* stephenfin clicks13:52
sean-k-mooneylyarwood: it should not no13:53
sean-k-mooneylyarwood: the function signitru is part of the class deffinion not the serialised object13:54
stephenfinlyarwood: Yeah, what Sean says. The scheduler, for example, will use its own version of the object13:55
lyarwoodsean-k-mooney: right, wasn't sure so it was worth asking. Thanks.13:55
stephenfin*definition13:55
sean-k-mooneylyarwood: its worth asking but no we are fine13:55
sean-k-mooneythis would matter for signiture of rpc methods where we are invokeing a remote method but not in this case13:58
*** _fragatina has quit IRC13:58
stephenfinlyarwood: I want tooling to automatically add (and remove, if abandoned) patches submitted to internal Gerrit to their BZ13:59
stephenfin...and reject the patch if there is no BZ linked13:59
sean-k-mooneystephenfin: sound like a nice proejct. to discuss downstream ...13:59
*** mlavalle has joined #openstack-nova13:59
stephenfinGoddddd damn it13:59
stephenfinGood thing I don't work in finance14:00
sean-k-mooneyi mean if someone upstream whats to write that for use tye are wellcome. im sure jangutter has plenty of time to do it:)14:00
*** mmethot has quit IRC14:02
janguttersean-k-mooney: if someone writes that, EVERYONE's going to pile on stuff to the weirdest BZ's.14:03
*** mmethot has joined #openstack-nova14:03
jangutterstephenfin, sean-k-mooney: the OpenContrail project has something like that. The unintended consequence is that BZ quality goes down :-(14:04
* cdent hires stephenfin as his financial adviser14:04
openstackgerritMatt Riedemann proposed openstack/nova master: Change InstanceFaultRollback handling in _error_out_instance_on_exception  https://review.openstack.org/63321214:05
mriedemmdbooth: i think you're familiar with _error_out_instance_on_exception - ^ is something that's always bugged me14:05
openstackgerritMatthew Booth proposed openstack/nova master: WIP: Don't age swap backing files in the image cache by default  https://review.openstack.org/63321314:05
openstackgerritMerged openstack/nova master: Kill @safe_connect in _get_provider_traits  https://review.openstack.org/61361314:06
mdboothmriedem: Looking, but ^^^ while you're here (discussion in bug: thanks for that)14:06
mriedemi didn't really understand what swap disks have to do with the image cache manager14:07
mdboothmriedem: Right.14:07
mriedemit's blank right? so why does the *image* cache manager worry about it?14:07
*** moshele has joined #openstack-nova14:07
mdboothmriedem: It's not blank, it's formatted.14:08
mdboothSo it's basically a cache of the output of mkswap.14:08
mriedemi mean there is no glance image in swap disks yeah?14:08
mdboothNo, there's no glance image.14:08
mdboothIt saves a completely inconsequential amount of time during creation, and is likely a performance hit at runtime.14:09
mdboothAnd will save essentially no disk space.14:09
mdboothSo we really shouldn't do it all, but cleaning that up would be a significant task. We'd need an upgrade task to rebase disks in use which already use this.14:09
mdboothHowever, aging these cached formatted swap files is really unlikely to be of much use to anybody.14:10
mdboothThis user has found it's really expensive, so my suggestion is just don't age them.14:10
mdboothmriedem: Anyway, gimme a shout if you think ^^^ is an approach you could get behind and I'll fix that patch up properly.14:11
mriedemi just don't really understand the use case for why these are in the image cache14:13
mriedemgranted, i'm not super familiar with the image cache manager to begin with14:13
mriedembeyond it's used to cache and (on a timer) age/invalidate images in the cache so new versions can be fetched14:13
mriedembut i thought that was only for root disks14:14
mdboothmriedem: I don't know the origin, so I can only guess. However ephemeral filesystem are also in there, and there are certain filesystems which *are* expensive to initialise.14:14
mdboothMy guess is swap was simply handled the same way.14:14
mdboothAlthough to be clear, I would remove ephemeral filesystems too. We're already not aging them.14:14
mriedemi don't see where ephemerals are in here14:17
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix ComputeNode ovo compatibility code  https://review.openstack.org/63119914:18
mdboothext3 vs ext4 fs creation comparison: http://paste.openstack.org/show/743462/14:18
mdboothmriedem: I had to check closely, but I'm confident we don't age ephemeral backing files.14:19
mdboothreasonably confident :) That code is very hard to read.14:19
mdboothI believe ext3 was the default, might still be, and ext3 fs creation can be slow.14:20
mriedemyeah i remember a ML thread from robert collins several years ago about changing the default to ext414:20
mdboothI suspect that's how this feature came about. But note that we still have to do this for other imagebackends anyway, and ext4 fs creation is really fast anyway.14:20
mriedemlooks like ext4 is the default now14:22
openstackgerritMerged openstack/nova master: Add missing ws seperator between words  https://review.openstack.org/63260614:22
mriedemhttps://github.com/openstack/nova/blob/master/nova/privsep/fs.py#L25814:22
kashyapmdbooth: (Our of curiosity, I ran your same test on an NVMe: http://paste.openstack.org/show/743468/)14:26
mdboothkashyap: That's consistent, although I doubt either of our tests were actually hitting disk.14:27
*** BjoernT has joined #openstack-nova14:27
mdboothkashyap: Likely due to RAM/CPU speed differences.14:27
kashyapOh, indeed.  I always forget that point14:28
kashyap(I recall that `fio` is the better way to do benchmarks)14:28
*** mmethot has quit IRC14:28
kashyap(Reference: http://blog.vmsplice.net/2017/11/common-disk-benchmarking-mistakes.html)14:29
*** mmethot has joined #openstack-nova14:29
*** awaugama has joined #openstack-nova14:31
kashyapmriedem: Completely unrelated, posted the initial investigation from last night here: https://bugs.launchpad.net/nova/+bug/1813192/comments/114:33
openstackLaunchpad bug 1813192 in OpenStack Compute (nova) "libvirt: instance delete fails with "Cannot destroy instance, operation time out: libvirt.libvirtError: Timed out during operation: cannot acquire state change lock (held by remoteDispatchDomainBlockJobAbort)" in bionic nodes (libvirt 4.0.0, qemu 2.11)" [Medium,Confirmed]14:33
kashyapI did some more digging this morning for an hour, will write it up today.14:33
kashyapThe unsolved problem for me here is: how to get tracebacks ("thread all apply bt") from the hung QEMU (or libvirtd) from the CI hosts.14:34
kashyapThe only current awful way for that is to abuse 'rootwrap' to elevate privileges and then upload a debug build of libvirt/QEMU.14:35
kashyapThen you can reproduce the problem, because "Heisenbug".14:35
*** liuyulong has joined #openstack-nova14:37
mriedem:/14:38
mriedemit's only 3 hits in 7 days in the gate so it's not burning us down like some of the other known bugs in the gate14:39
kashyapYeah, it's not.  But I'd like to track down these gnarly bugs.  Because if you recall, we had the same problem with the fated "live snapshot" bug14:44
kashyap... that made us reap the whirlind.14:44
*** moshele has quit IRC14:44
kashyap(Typo earlier: "Then you can" --> "Then you can't")14:45
kashyapmriedem: Mind pointing me to the "known bugs" in the gate?14:45
* kashyap goes to check: http://status.openstack.org/elastic-recheck/14:46
mriedemkashyap: http://status.openstack.org/elastic-recheck/14:46
mriedemhttp://status.openstack.org/elastic-recheck/#181319814:46
kashyapYep, remembered the URL14:46
mriedemhttp://status.openstack.org/elastic-recheck/#181314714:46
mriedemthose are 2 big new ones14:46
*** whoami-rajat has joined #openstack-nova14:46
*** dave-mccowan has joined #openstack-nova14:47
kashyapThanks14:47
*** lbragstad has joined #openstack-nova14:49
*** dave-mccowan has quit IRC14:51
*** psachin has quit IRC14:53
openstackgerritRalf Haferkamp proposed openstack/nova master: compute: reset instance events on reload  https://review.openstack.org/63322014:53
*** pelement has left #openstack-nova14:55
mriedemmelwitt: do you expect to start on https://blueprints.launchpad.net/nova/+spec/count-quota-usage-from-placement soon? given you're out next week and we've got 6 weeks left i'm a bit worried about that one14:56
*** cfriesen has joined #openstack-nova15:00
*** cdent has quit IRC15:00
*** hongbin has joined #openstack-nova15:01
melwittmriedem: yeah, I hope to have something WIP uploaded by late today15:02
*** Shilpa has quit IRC15:04
*** spsurya has quit IRC15:07
*** tbachman has quit IRC15:12
*** tbachman has joined #openstack-nova15:22
*** dklyle has quit IRC15:22
*** spsurya has joined #openstack-nova15:30
openstackgerritsean mooney proposed openstack/nova master: Libvirt: do not set mac when unplugging macvtap vf  https://review.openstack.org/62484215:30
openstackgerritsean mooney proposed openstack/nova master: Add free for claimed, allocated devices  https://review.openstack.org/61612015:30
openstackgerritsean mooney proposed openstack/nova master: Allow per-port modification of vnic_type and profile  https://review.openstack.org/60736515:30
openstackgerritsean mooney proposed openstack/nova master: Add get_instance_pci_request_from_vif  https://review.openstack.org/61992915:30
openstackgerritsean mooney proposed openstack/nova master: SR-IOV Live migration indirect port support  https://review.openstack.org/62011515:30
openstackgerritsean mooney proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration  https://review.openstack.org/62958915:30
*** moshele has joined #openstack-nova15:31
*** jmlowe has joined #openstack-nova15:34
*** Luzi has quit IRC15:35
openstackgerritJan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif  https://review.openstack.org/57132515:35
openstackgerritJan Gutter proposed openstack/nova master: Add support for vrouter HW datapath offloads  https://review.openstack.org/57208215:35
*** betherly has joined #openstack-nova15:35
*** READ10 has joined #openstack-nova15:36
*** betherly has quit IRC15:36
*** betherly has joined #openstack-nova15:36
*** cdent has joined #openstack-nova15:37
*** moshele has quit IRC15:38
*** bnemec has quit IRC15:43
*** lbragstad is now known as elbragstad15:43
*** udesale has quit IRC15:56
openstackgerritMatt Riedemann proposed openstack/nova master: Raise InstanceFaultRollback for UnableToMigrateToSelf from _prep_resize  https://review.openstack.org/63322715:58
openstackgerritMatt Riedemann proposed openstack/nova master: Change InstanceFaultRollback handling in _error_out_instance_on_exception  https://review.openstack.org/63321216:00
openstackgerritMatt Riedemann proposed openstack/nova master: Raise InstanceFaultRollback for UnableToMigrateToSelf from _prep_resize  https://review.openstack.org/63322716:00
*** efried has joined #openstack-nova16:02
*** TxGirlGeek has joined #openstack-nova16:02
*** dklyle has joined #openstack-nova16:03
openstackgerritMatt Riedemann proposed openstack/nova master: Change InstanceFaultRollback handling in _error_out_instance_on_exception  https://review.openstack.org/63321216:04
openstackgerritMatt Riedemann proposed openstack/nova master: Raise InstanceFaultRollback for UnableToMigrateToSelf from _prep_resize  https://review.openstack.org/63322716:04
*** wolverineav has joined #openstack-nova16:04
*** efried is now known as efried_mtg16:05
*** elbragstad has quit IRC16:06
*** lbragstad has joined #openstack-nova16:07
*** lbragstad is now known as elbragstad16:07
*** macza has joined #openstack-nova16:08
*** wolverineav has quit IRC16:08
melwittmriedem: I got confused by your comments on https://bugs.launchpad.net/nova/+bug/1811235 is it that cold migrate to same host is only valid for the vmware driver? and compute manager should be kicking it out unilaterally?16:11
openstackLaunchpad bug 1811235 in OpenStack Compute (nova) "instance's vm-state becomes error when cold-migrate instance to same host failed" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)16:11
mriedemstephenfin: i'll likely need you to run it up the flagpole with danpb about why libvirt.sysinfo_serial was never guest-specific16:11
mriedemi don't have the history on that and it wasn't found in the original changes16:11
stephenfinmriedem: Can do16:11
melwitts/should/shouldn't/16:12
mriedemthe compute manager doesn't kick cold migrate to the same host out unilateraly, the api is16:12
mriedemwith that 2.56 change to specify host on cold migrate16:12
mriedemthat's a different issue from the bug though,  which was reported against ocata16:12
mriedembefore 2.56 existed16:12
*** macza has quit IRC16:13
mriedemi believe migrate_to_same_host exists for the vmware driver b/c you can have a single nova-compute service in your openstack deployment which is actually managing a vcenter cluster with 1000 ESXi hosts in it16:13
mriedemso your only choice to cold migrate is to the same host16:13
melwitthe says he thinks it exists in ocata too from reading the code. right? I'm having trouble parsing this entire bug :(16:13
mriedemmy patches above are working on fixing the bug he's hitting which exists in ocata16:14
melwittcomment #216:14
mriedem2.56 does not16:14
mriedem2.56 was queens16:14
mriedemcomment 2 is saying 2 things16:14
mriedemfirst is the bug i'm fixing above16:14
mriedemsecond is the 2.56 issue i think16:15
mriedemwhich is a different bug imo16:15
melwittyeah, I think he's not saying he's reporting it for ocata. "First, why the latest version of nova not supports migrating instance to same host event through some drivers support that function..Second, if there is any problem when user migrates instance without migration destination host but there is just one compute node."16:15
*** ondrejme has quit IRC16:15
melwitteh, I dunno, I think I'm just confused by the wording of these comments16:15
*** yan0s has quit IRC16:17
mriedemi left a comment to try and answer the questions from comment 216:18
melwittthanks. I think I see now, he's saying why go to error status when it "fails" because migrating to self doesn't really do anything16:20
melwittok that comment makes it clear. good16:21
mriedemcorrect, but the rejection in the API is not the same as the bug in the compute that sets the instance to error status16:21
mriedemi've got fixes posted above for the latter, not the former16:21
melwittgotcha. I've got it starred for review16:21
*** panda is now known as panda|off16:23
mriedemmelwitt: if https://blueprints.launchpad.net/openstack/?searchtext=conf-max-attach-volumes is ready for review again you want to queue it up in runways?16:29
mriedemjangutter's can probably go in the open slot as well16:29
melwittmriedem: sure. I don't know why I didn't think of doing that16:30
*** kaliya has joined #openstack-nova16:30
*** wolverineav has joined #openstack-nova16:34
janguttermelwitt, mriedem: thanks!16:34
*** kaliya has quit IRC16:35
mriedempretty easy fix here for someone https://review.openstack.org/#/c/624959/16:35
mriedemneeds to be backported to queens as well16:35
melwittjangutter: the bp dependencies show that vrouter-hw-offloads also depends on the generic-os-vif-offloads too. should I add that patch link in there too for the runway review?16:39
sean-k-mooneymelwitt: i think the os-vif stuff was all in the 1.14 release we did this week16:40
* melwitt reads note on blueprint tracking etherpad16:40
janguttermelwitt: the dependency's unblocked on that one (generic-os-vif-offloads has follow-on work)16:40
sean-k-mooneyjangutter: more stuff in os-vif "e.g. that i should review" or in nova16:41
melwittoh, I see, the dependency piece is unblocked but bp still needs more work to be complete16:41
sean-k-mooneywhich i should proably also review16:41
melwittok thank you16:41
janguttermelwitt: the dominoes are all ready to fall... for the vrouter bits though.16:41
melwittcoolness16:42
*** tssurya has quit IRC16:43
*** tstrul has joined #openstack-nova16:45
janguttersean-k-mooney: for the generic-os-vif-offloads I think there's just one more bit in nova (for Stein) to populate the data. The next os-vif change should really only be appropriate for Train.16:45
*** macza has joined #openstack-nova16:48
sean-k-mooneyok16:48
sean-k-mooneyif there is stuff i need to review for stien let me know. the non client lib frezze is in 3-4 weeks so if another os-vif release is need we need to merge the code before then16:49
sean-k-mooneyform the non-client-lib freeze untill rc1 os-vif will basically be frozen16:50
sean-k-mooneyform rc1/rc2 on it will be reopen for train16:50
melwittfeb 28 is non-client lib freeze16:51
janguttersean-k-mooney: thanks, the only open one I have for Stein is the serialization cleanup patch, and that kinda-sorta depends on a style choice.16:51
sean-k-mooneyok thats what i had understood16:52
janguttersean-k-mooney: I _think_ the only affected party to that would be kuryr and not Nova.16:52
*** betherly has quit IRC16:53
*** rpittau has quit IRC16:54
*** dklyle has quit IRC17:00
*** helenaAM has quit IRC17:04
*** derekh has quit IRC17:05
*** jangutter has quit IRC17:06
melwittleakypipes, mriedem: am I correct in my understanding that because this spec about "how to use initial allocation ratios" got stuck https://review.openstack.org/544683, we don't and will not have support for being able to set allocation ratios per aggregate?17:06
*** dklyle has joined #openstack-nova17:07
mriedemyou can do it via placement externally if you want, and could have since forever - and is what we said people could do in dublin17:08
mriedembut no one worked on that,17:08
mriedemor added anything to osc-placement for it17:08
mriedemthe nova spec is about mirroring aggregate allocation ratios to placement17:09
mriedemwhich is a nice to have but less necessary since you can do that directly with placement and we have rbac support in placement now too (which was a previous reason this should go through nova - for rbac)17:09
mriedemso the only sticking point on the nova spec is deciding if nova should do that mirroring / proxy orchestration work17:10
mriedemclearly that isn't going to happen in stein though17:10
mriedembbiab17:10
*** mriedem is now known as mriedem_afk17:10
*** gyee has joined #openstack-nova17:11
*** mmethot_ has joined #openstack-nova17:11
efried_mtgsean-k-mooney: not sure if you knew about this, but when resubmitting a series, git-review normally re-topics the whole series to match the top patch. You can make it not do that - i.e. leave the topic unchanged (or empty if it's the first time you're uploading it) by using the -T flag.17:12
melwittmriedem_afk: thanks, I'll make some notes about it on the blueprint tracking etherpad17:12
*** mrhillsman is now known as mrhillsman_lunch17:13
sean-k-mooneyefried_mtg: ya it has bitten me in the past17:13
*** brault has quit IRC17:13
melwittI've had a hard time following what was going on there. didn't realize the only thing left was mirroring, and I agree I'm not sure whether we should17:13
sean-k-mooneyefried_mtg: generally its does what i want as if i am working on a patch serise its all for the same bug/blueprint but it can be annoying17:14
*** brault has joined #openstack-nova17:14
*** mmethot has quit IRC17:14
sean-k-mooneyefried_mtg: i just like the fact i can "git checkout -b whatever" and then "git revew -f" and have the topic set correctly to "whatever"17:15
*** _fragatina has joined #openstack-nova17:16
*** _fragatina has quit IRC17:16
leakypipesmelwitt: I'll abandon that spec.17:16
*** _fragatina has joined #openstack-nova17:17
sean-k-mooneyleakypipes: you spec for mirroring nova aggrages as placement aggrates was complete in rocky rigt? seperate form the allcoation ratio stuff17:18
melwittleakypipes: k. I was only asking because I'm trying to round up things we identified as priority items and that one I had trouble parsing what the second spec was about17:18
melwitt*priority items at the ptg17:18
*** dklyle has quit IRC17:19
melwittsean-k-mooney: mirroring aggregates was a different thing yeah, and is already in nova17:19
leakypipessean-k-mooney: yes.17:19
leakypipesmelwitt: np17:20
sean-k-mooneycool so in thery you coudl write a scipt/tool to find all the RPs with a specific resouce in an aggrate and set the allocation ratio in the inventory relitivly simply if you wanted too17:20
melwittyeah, we talked about adding a command to osc-placement CLI to help with that. we should do that. maybe I'll try to do it if I can get my other stuff done17:26
*** pelement has joined #openstack-nova17:27
*** _fragatina has quit IRC17:32
*** _fragatina has joined #openstack-nova17:32
*** mmethot_ has quit IRC17:34
*** mmethot_ has joined #openstack-nova17:35
*** mmethot_ has quit IRC17:36
openstackgerritKevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter  https://review.openstack.org/63317817:39
openstackgerritKevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter  https://review.openstack.org/63317817:40
*** _fragatina has quit IRC17:41
*** _fragatina has joined #openstack-nova17:41
*** dklyle has joined #openstack-nova18:00
*** ccamacho has quit IRC18:05
*** _fragatina has quit IRC18:12
*** _fragatina has joined #openstack-nova18:13
*** wolverineav has quit IRC18:13
*** mrhillsman_lunch is now known as mrhillsman18:15
artomWe can just disable the Keystone middleware and run without authentication, right?18:17
sean-k-mooneyartom: you can disable auth yes18:17
artom(Asking for a performance thing downstream, suspecting Keystone is the culprit, want to propose testing without it)18:17
sean-k-mooneyi guessed. let me see if i can find the config18:21
artomsean-k-mooney, too late, email went out. I'll blame you if people pick me up on it :D18:21
cdentartom, sean-k-mooney: I'm not positive but I think you have to choose a different auth strategy (noauth2) and still set x-auth-token headers so that policy checks still have something to chew on. If that can be avoided too, I'd love to hear about it.18:22
sean-k-mooneyartom: i think you set this https://docs.openstack.org/nova/latest/configuration/config.html#keystone_authtoken.auth_type18:23
artomcdent, right, but that avoids the authentication calls to the Keystone API, right?18:23
cdentyes18:23
*** _fragatina has quit IRC18:23
artomOK, because that's what I care about18:23
cdentI do that noauth2 stuff all the time with placement and it seems to work fine with a 'x-auth-token: amdin' header18:23
sean-k-mooneycdent: ya our functional test use noauth2 i think when they spin up apis18:24
cdentexcept I spell admin correctly...18:24
cdentsigh18:24
sean-k-mooneyjust alias it it will be fine18:24
cdent:)18:24
artomAm Din. You be?18:24
cdentMe be me18:25
sean-k-mooneyi have always beeen temeted to make "suod" and alais of "sudo"18:25
artomAnd "temeted" an alias of "tempted", and "alais" of "alias"18:25
artom;)18:25
artomcdent, but will the FCC let you be?18:25
sean-k-mooneyhehe the problem is i mispell things inconsitently18:26
cdentartom++18:26
artomcdent, .... and now I'm listening to it18:27
cdentyeah, I was headed that way too18:27
*** wolverineav has joined #openstack-nova18:33
*** whoami-rajat has quit IRC18:34
*** wolverineav has quit IRC18:35
*** wolverineav has joined #openstack-nova18:35
*** mriedem_afk is now known as mriedem18:41
*** sapd1_ has quit IRC18:47
*** sapd1_ has joined #openstack-nova18:47
*** wolverineav has quit IRC19:00
*** wolverineav has joined #openstack-nova19:02
*** erlon_ has quit IRC19:02
*** bnemec has joined #openstack-nova19:02
*** tstrul has quit IRC19:08
*** bnemec is now known as beekneemech19:11
*** igordc has joined #openstack-nova19:11
openstackgerritMatt Riedemann proposed openstack/nova master: Isolate cell-targeting code in MigrationTask  https://review.openstack.org/62131019:11
openstackgerritMatt Riedemann proposed openstack/nova master: Extract compute API _create_image to compute.utils  https://review.openstack.org/62131119:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid  https://review.openstack.org/61401219:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field  https://review.openstack.org/61403519:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method  https://review.openstack.org/61403619:11
openstackgerritMatt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells  https://review.openstack.org/61403719:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellWeigher  https://review.openstack.org/61435319:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method  https://review.openstack.org/62131319:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add Instance.hidden field  https://review.openstack.org/63112319:11
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add TargetDBSetupTask  https://review.openstack.org/62789219:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask  https://review.openstack.org/63158119:11
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask  https://review.openstack.org/62789019:11
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask  https://review.openstack.org/62789119:11
*** cdent has quit IRC19:24
*** wolverineav has quit IRC19:33
*** ralonsoh has quit IRC19:33
*** wolverineav has joined #openstack-nova19:34
*** wolverineav has quit IRC19:38
*** wolverineav has joined #openstack-nova19:39
*** beekneemech has quit IRC19:42
melwittis there no way to link to an archived ML post from openstack-dev@?19:43
*** _fragatina has joined #openstack-nova19:43
*** mgariepy has joined #openstack-nova19:43
melwittfrom before the switch to the new list19:43
melwittok, I found a way. just couldn't get to the root page for openstack-dev anymore19:44
*** betherly has joined #openstack-nova19:45
*** wolverineav has quit IRC19:45
*** mmethot has joined #openstack-nova19:49
*** dklyle has quit IRC19:51
sean-k-mooneyare block migrtation broken?19:57
sean-k-mooneyhttp://logs.openstack.org/20/616120/9/check/legacy-grenade-dsvm-neutron-multinode-live-migration/b90819f/logs/testr_results.html.gz19:57
sean-k-mooneytest_live_block_migration is presuably ment to you know use a block migrtation so the fact taht it is not on shared storage is well expected19:58
artomsean-k-mooney, multiqueue is an imageprop, right?20:06
artomI can't find it under flavor extra specs20:06
artom(In parallel, is there a guide as to when something's a prop vs an extra spec?)20:07
*** avolkov has quit IRC20:07
sean-k-mooneyif its virt dirver specific it an image prop if its generic its in the falvor is one guide the other is should the admin or tenant be able to set it20:08
sean-k-mooneyand yes its hw_vif_multiqueue_enabled=true|false20:08
sean-k-mooneyin the image meta20:08
artom"is one guide the other is should the admin or tenant be able to set it" Wat.20:09
*** tbachman has quit IRC20:09
sean-k-mooneyadmin only = flavor, tenant setable = imgage20:09
artomAh :) Thanks!20:10
*** agopi has joined #openstack-nova20:10
sean-k-mooneyo/20:11
*** dklyle has joined #openstack-nova20:13
*** wolverineav has joined #openstack-nova20:18
openstackgerritEric Fried proposed openstack/nova master: Add a warning for max_concurrent_live_migrations  https://review.openstack.org/63224020:19
*** TxGirlGeek has quit IRC20:20
*** tbachman has joined #openstack-nova20:21
*** wolverineav has quit IRC20:23
*** mgariepy has quit IRC20:25
*** zul has quit IRC20:26
*** tbachman has quit IRC20:28
*** mmethot has quit IRC20:29
*** med_ has quit IRC20:30
*** mmethot has joined #openstack-nova20:31
*** efried_mtg has quit IRC20:31
*** READ10 has quit IRC20:35
*** spsurya has quit IRC20:37
openstackgerritMaciej Kucia proposed openstack/nova master: SR-IOV passthrough: Check PF only if VF is enabled  https://review.openstack.org/47664220:38
*** slaweq has quit IRC20:49
*** wolverineav has joined #openstack-nova20:50
*** wolverineav has quit IRC20:55
*** dklyle has quit IRC20:55
*** dklyle has joined #openstack-nova20:59
leakypipesmriedem, stephenfin: if either of you have a moment, https://review.openstack.org/#/c/631684/ should be an easy one.21:08
leakypipessean-k-mooney: ^^ you too if you care to have a look.21:09
*** tbachman has joined #openstack-nova21:12
*** dklyle has quit IRC21:13
*** wolverineav has joined #openstack-nova21:14
mriedemsean-k-mooney: i reported a bug for that job failure yesterday21:15
mriedemhttps://bugs.launchpad.net/nova/+bug/181321621:15
openstackLaunchpad bug 1813216 in OpenStack Compute (nova) "legacy-grenade-dsvm-neutron-multinode-live-migration failing with "is not on shared storage: Shared storage live-migration requires either shared storage or boot-from-volume with no local disks." since Jan 21" [Undecided,Confirmed]21:15
mriedemit's a recent regression but i don't know what is causing it21:15
mriedemcould be a change i made in devstack recently which would impact grenade like jobs21:15
mriedemartom: image properties are doc'ed here https://docs.openstack.org/glance/latest/admin/useful-image-properties.html21:16
mriedemalthough it's missing quite a few21:16
*** slaweq has joined #openstack-nova21:16
*** TxGirlGeek has joined #openstack-nova21:17
openstackgerritAlex Kavanagh (tinwood) proposed openstack/nova master: Add LXD to the nova.objects.fields.HypervisorDriver Enum  https://review.openstack.org/63327921:18
*** slaweq has quit IRC21:21
*** TxGirlGeek has quit IRC21:28
mriedemstephenfin: questoins in that dns patch21:28
mriedemoh nvm that was for leakypipes21:29
*** wolverineav has quit IRC21:29
*** wolverineav has joined #openstack-nova21:30
*** wolverineav has quit IRC21:30
mriedemleakypipes: how far back is that change going to go? presumably at least ocata. if so, it would probably be good to do the network caching i mention here https://review.openstack.org/#/c/631684/2/nova/network/neutronv2/api.py@63521:32
mriedemrather than in a follow up21:32
leakypipesmriedem: ack, though I'm not sure we should ask Hang to do that. Probably better if I do it in a pre-cursor or something?21:33
*** jangutter has joined #openstack-nova21:33
mriedempre-cursor?21:33
mriedemif you want to make that change to this patch set i'm ok with it - and will honor your +221:34
leakypipesmriedem: ftr, that change is being applied to our queens/rocky release, so I'm not entirely concerned about the ocata thing :) but yes, I can make the change you requested on line 635 to that patch.21:40
leakypipesmriedem: and I don't expect you to honor my +2 or anything :)21:41
leakypipesI see no honor in reviews :)21:41
mriedemi mean i think you can still +2 after that, or i'll just proxy your +221:41
leakypipesack21:41
mriedemsince you're a southerner honor is a big thing21:42
leakypipesmriedem: indeed. I am SOOOO Southern now.21:50
*** wolverineav has joined #openstack-nova21:52
*** pelement has left #openstack-nova21:52
*** awaugama has quit IRC21:58
*** awalende has joined #openstack-nova21:58
*** hongbin has quit IRC22:02
*** hongbin has joined #openstack-nova22:03
*** TxGirlGeek has joined #openstack-nova22:07
*** jangutter has quit IRC22:08
mnaserhi, just got bit by this -- https://review.openstack.org/#/c/626218/1 -- anyone on stable mind pushing it through?22:15
mnaserpatch on top of it has +w too22:16
*** awalende has quit IRC22:31
*** awalende has joined #openstack-nova22:31
*** awalende has quit IRC22:35
*** slaweq has joined #openstack-nova22:38
*** jmlowe has quit IRC22:41
*** slaweq has quit IRC22:43
*** KeithMnemonic has quit IRC22:51
openstackgerritMatt Riedemann proposed openstack/nova master: Add Instance.hidden field  https://review.openstack.org/63112322:53
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add TargetDBSetupTask  https://review.openstack.org/62789222:53
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask  https://review.openstack.org/63158122:53
openstackgerritMatt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method  https://review.openstack.org/62131322:53
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask  https://review.openstack.org/62789022:53
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask  https://review.openstack.org/62789122:53
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add prep_snapshot_based_resize_at_dest compute method  https://review.openstack.org/63329322:53
mriedemsmcginnis: can you hit this for mnaser? https://review.openstack.org/#/c/626218/22:54
mriedemand the one on top22:54
mriedemoh nvm the one on top is +W already22:54
mnaseryup :>22:54
*** slaweq has joined #openstack-nova22:57
*** kaliya has joined #openstack-nova22:58
*** kaliya has quit IRC23:00
*** slaweq has quit IRC23:05
*** bnemec has joined #openstack-nova23:12
*** dklyle has joined #openstack-nova23:13
*** BjoernT has quit IRC23:16
*** betherly has quit IRC23:16
*** betherly has joined #openstack-nova23:16
*** dklyle has quit IRC23:20
*** wolverineav has quit IRC23:22
*** wolverineav has joined #openstack-nova23:23
*** efried has joined #openstack-nova23:23
*** wolverineav has quit IRC23:27
*** wolverineav has joined #openstack-nova23:27
*** xek has quit IRC23:30
*** mmethot has quit IRC23:32
*** mmethot has joined #openstack-nova23:32
*** betherly has quit IRC23:32
*** mmethot has quit IRC23:33
*** mmethot has joined #openstack-nova23:33
*** mmethot has quit IRC23:34
*** mmethot has joined #openstack-nova23:35
*** mmethot has quit IRC23:41
*** mmethot has joined #openstack-nova23:42
*** mmethot has quit IRC23:43
*** bnemec has quit IRC23:43
*** mmethot has joined #openstack-nova23:44
*** bnemec has joined #openstack-nova23:44
*** mmethot has quit IRC23:44
*** mmethot has joined #openstack-nova23:45
*** elbragstad has quit IRC23:54
*** bnemec has quit IRC23:59
*** bnemec has joined #openstack-nova23:59

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