Thursday, 2018-11-08

*** tetsuro has joined #openstack-nova00:01
*** gyee has quit IRC00:55
*** vishwanathj has quit IRC00:57
*** k_mouza has quit IRC01:16
*** Swami has quit IRC01:24
*** brinzhang has joined #openstack-nova01:29
*** jdillaman has quit IRC01:37
*** Dinesh_Bhor has joined #openstack-nova01:49
*** mriedem has quit IRC01:50
*** jdillaman has joined #openstack-nova02:04
*** erlon_ has quit IRC02:10
*** hongbin has joined #openstack-nova02:13
*** tbachman has quit IRC02:14
*** READ10 has quit IRC02:21
*** mrsoul has joined #openstack-nova02:44
*** tiendc has joined #openstack-nova02:44
*** markvoelker has joined #openstack-nova02:45
*** markvoelker has quit IRC02:45
*** markvoelker has joined #openstack-nova02:45
*** markvoelker has quit IRC02:50
*** markvoelker has joined #openstack-nova02:55
*** hshiina_ has joined #openstack-nova02:59
*** hshiina has quit IRC03:02
openstackgerritFan Zhang proposed openstack/nova master: Retry after hitting libvirt error VIR_ERR_OPERATION_INVALID in live migration.  https://review.openstack.org/61227203:05
*** fungi has quit IRC03:06
*** fungi has joined #openstack-nova03:09
*** fungi has quit IRC03:10
*** Dinesh_Bhor has quit IRC03:38
*** fungi has joined #openstack-nova03:40
*** fungi has quit IRC03:41
*** fungi has joined #openstack-nova03:45
*** zhanglong has joined #openstack-nova03:49
*** sapd1 has quit IRC03:58
*** sapd1 has joined #openstack-nova03:58
*** sambetts|afk has quit IRC04:11
*** hshiina__ has joined #openstack-nova04:13
*** hshiina_ has quit IRC04:16
*** sambetts_ has joined #openstack-nova04:18
*** hshiina__ has quit IRC04:19
openstackgerritMerged openstack/nova master: Improve formats of the Compute API guide  https://review.openstack.org/61580904:20
*** hshiina has joined #openstack-nova04:24
*** hshiina_ has joined #openstack-nova04:26
*** hshiina has quit IRC04:29
*** janki has joined #openstack-nova04:36
*** Dinesh_Bhor has joined #openstack-nova04:42
*** cfriesen has quit IRC04:53
*** Dinesh_Bhor has quit IRC05:15
*** Dinesh_Bhor has joined #openstack-nova05:20
*** munimeha1 has quit IRC05:28
*** tetsuro has quit IRC05:33
*** hshiina_ has quit IRC05:38
*** hshiina has joined #openstack-nova05:38
*** hshiina has quit IRC05:43
*** hshiina has joined #openstack-nova05:51
*** hshiina_ has joined #openstack-nova05:52
*** hshiina has quit IRC05:56
*** hshiina__ has joined #openstack-nova05:56
*** hshiina_ has quit IRC06:00
*** hongbin has quit IRC06:02
*** TuanDA has joined #openstack-nova06:09
*** cfriesen has joined #openstack-nova06:17
*** gokhan has joined #openstack-nova06:51
*** brinzhang has quit IRC06:54
*** brinzhang has joined #openstack-nova06:54
*** sahid has joined #openstack-nova06:58
*** cfriesen has quit IRC07:00
*** dpawlik has joined #openstack-nova07:04
*** brinzhang has quit IRC07:20
*** brinzhang has joined #openstack-nova07:20
*** tssurya has joined #openstack-nova07:23
*** adrianc has joined #openstack-nova07:30
*** pcaruana has joined #openstack-nova07:34
*** slaweq has joined #openstack-nova07:39
*** alexchadin has joined #openstack-nova07:39
*** jangutter has quit IRC07:57
*** jangutter has joined #openstack-nova07:57
*** bnemec has quit IRC08:00
openstackgerritliuyamin proposed openstack/nova master: Fix version details API does not return 200 OK  https://review.openstack.org/61644408:03
*** helenafm has joined #openstack-nova08:10
*** Dinesh_Bhor has quit IRC08:11
*** Dinesh_Bhor has joined #openstack-nova08:12
*** ralonsoh has joined #openstack-nova08:17
*** adrianc has quit IRC08:19
*** sapd1 has quit IRC08:20
*** adrianc has joined #openstack-nova08:20
*** sapd1 has joined #openstack-nova08:20
*** bnemec has joined #openstack-nova08:26
*** Luzi has joined #openstack-nova08:30
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331708:34
*** jpena|off is now known as jpena08:36
*** moshele has joined #openstack-nova08:37
openstackgerritTakashi NATSUME proposed openstack/nova master: Add API ref guideline for body text  https://review.openstack.org/60562808:40
openstackgerritTakashi NATSUME proposed openstack/nova master: Add API ref guideline for body text  https://review.openstack.org/60562808:40
*** Cathyz has joined #openstack-nova08:42
*** hshiina__ has quit IRC08:46
*** sapd1_ has joined #openstack-nova08:47
*** sapd1 has quit IRC08:49
openstackgerritJeffrey Zhang proposed openstack/nova master: Add feature to flatten the volume from glance image snapshort  https://review.openstack.org/61646108:50
*** zhanglong has quit IRC09:00
openstackgerritJeffrey Zhang proposed openstack/nova master: Add feature to flatten the volume from glance image snapshort  https://review.openstack.org/61646109:06
*** Cathyz has quit IRC09:11
*** sahid has quit IRC09:13
*** sahid has joined #openstack-nova09:13
*** BlackDex has quit IRC09:20
*** jcosmao has joined #openstack-nova09:20
*** BlackDex has joined #openstack-nova09:25
*** derekh has joined #openstack-nova09:32
*** moshele has quit IRC09:39
*** ttsiouts has joined #openstack-nova09:54
*** dtantsur|afk is now known as dtantsur09:55
*** sapd1_ has quit IRC09:55
*** ttsiouts has quit IRC09:56
*** ttsiouts has joined #openstack-nova09:57
*** mvkr has quit IRC09:58
openstackgerritBalazs Gibizer proposed openstack/nova master: Add extra unit test coverage for @run_once  https://review.openstack.org/61648509:58
*** ttsiouts_ has joined #openstack-nova09:59
*** ttsiouts has quit IRC09:59
gibisean-k-mooney: I left some comments in https://review.openstack.org/#/c/610034 and proposed a followup ^^10:01
*** sapd1_ has joined #openstack-nova10:05
*** ttsiouts_ has quit IRC10:19
*** ttsiouts has joined #openstack-nova10:20
*** ttsiouts has quit IRC10:24
*** mvkr has joined #openstack-nova10:26
*** tetsuro has joined #openstack-nova10:27
*** brinzhang has quit IRC10:28
*** panda|off is now known as panda10:31
*** ttsiouts has joined #openstack-nova10:34
openstackgerritBalazs Gibizer proposed openstack/nova master: Send rp_uuid in the port binding  https://review.openstack.org/56945910:35
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331710:36
*** efried has quit IRC10:38
*** efried has joined #openstack-nova10:38
*** adrianc has quit IRC11:01
*** erlon_ has joined #openstack-nova11:08
*** davidsha has joined #openstack-nova11:12
*** adrianc has joined #openstack-nova11:22
*** ttsiouts has quit IRC11:35
*** ttsiouts has joined #openstack-nova11:35
*** ttsiouts has quit IRC11:38
*** ttsiouts_ has joined #openstack-nova11:38
sean-k-mooneygibi: hi11:56
gibisean-k-mooney: hi11:56
sean-k-mooneyi was under the impression that try finally was not legal in python but reading the reference docs on the try statement i apears to be11:56
sean-k-mooneyi can remove teh except blocks11:57
gibisean-k-mooney: yeah, try:finally: without except is totally valid11:57
gibisean-k-mooney: also if you want, you can squash my followup into your patch11:57
sean-k-mooneyi was just going to ask if i could add you as a co-author and do that to reduce the number of patches i need to backport11:58
gibisean-k-mooney: sure, that totally works for me11:59
sean-k-mooneythanks ill make those change and resubmit once i finish makeing a cup of coffee. im alway happy to learn new things and til try finally is ok in python :)12:01
*** jpena is now known as jpena|lunch12:03
*** Luzi has quit IRC12:05
*** TuanDA has quit IRC12:05
*** tetsuro has quit IRC12:06
gibisean-k-mooney: :)12:11
*** ttsiouts_ has quit IRC12:14
*** ttsiouts has joined #openstack-nova12:14
*** ttsiouts has quit IRC12:19
*** Luzi has joined #openstack-nova12:20
*** janki has quit IRC12:20
*** dtantsur is now known as dtantsur|brb12:32
openstackgerritsean mooney proposed openstack/nova master: Harden placement init under wsgi  https://review.openstack.org/61003412:33
*** Dinesh_Bhor has quit IRC12:43
*** ttsiouts has joined #openstack-nova12:52
*** ccamacho has quit IRC12:53
*** ttsiouts has quit IRC12:56
*** ttsiouts has joined #openstack-nova12:57
*** panda is now known as panda|off12:59
*** dpawlik has quit IRC13:11
sean-k-mooneygibi: i have cross ported the squashed patch to placement. can you abandon https://review.openstack.org/#/c/616485 when you get a chance13:13
*** ccamacho has joined #openstack-nova13:18
sean-k-mooneybauzas: if you have time today can you rereview https://review.openstack.org/#/c/610034/ i adressed your feedback for extra comments + some other nits. i would like to try and get this merged today if possible so i can backport it before everyone goes to the summit13:19
*** jpena|lunch is now known as jpena13:24
bauzassean-k-mooney: I can try :)13:25
bauzasbut still on a customer issue13:25
* bauzas raises fist at anti-affinity checks13:25
*** dpawlik has joined #openstack-nova13:26
*** eharney has joined #openstack-nova13:27
sean-k-mooneybauzas: for the live migration case13:27
bauzasindeed13:27
bauzaspbd'ing13:27
*** janki has joined #openstack-nova13:29
sean-k-mooneybauzas: my understanding was we relied on the filters to check the host did not already have a member of the server group on the host in that case so the late check is not need  however i belive there is a race when doing multiple live migration or booting more instnaces into the server group that was not handeled correct13:29
*** dpawlik has quit IRC13:31
bauzassean-k-mooney: no, it's not a concurrent issue13:32
bauzassean-k-mooney: I guess the problem is about the persisted RequestGroup13:32
bauzasoops s/RequestGroup/RequestSpec13:32
bauzasAFAICS, the instancegroup.members is wrong13:33
bauzasanyway13:33
bauzasI'll continue to look13:33
sean-k-mooneybauzas: do you have a functional test that can reporduce?13:33
*** tiendc has quit IRC13:33
sean-k-mooneyi have a couple of small patches i need to work on but i can try and help later once i get them written/pushed13:34
sean-k-mooneyi have other stuff i need to work on but i could also do with a break/context switch for a few hours so if i can help let me know.13:34
bauzasI also have to go in a garage13:35
bauzassee you in 1h13:35
sean-k-mooneyo/13:35
*** k_mouza has joined #openstack-nova13:38
*** adrianc has quit IRC13:40
*** adrianc has joined #openstack-nova13:41
jrollkashyap: when you have a few minutes, curious your thoughts on this: https://review.openstack.org/#/c/616405/13:47
kashyapjroll: Hi13:48
jrollhi :)13:48
kashyapLet me check w/ the libvirt upstream folks13:48
kashyapjroll: I'd expect the reporter to also note what precise failure they're seeing13:50
kashyapLet me ask on the change13:50
jrollkashyap: good point13:51
*** ccamacho has quit IRC13:52
kashyapAsked on the change.13:52
*** ccamacho has joined #openstack-nova13:53
*** moshele has joined #openstack-nova13:54
*** dpawlik has joined #openstack-nova13:58
*** dpawlik has quit IRC13:59
*** ccamacho has quit IRC14:02
*** derekh has quit IRC14:02
*** davidsha has quit IRC14:02
*** jcosmao has quit IRC14:02
*** bnemec has quit IRC14:02
*** sambetts_ has quit IRC14:02
*** jdillaman has quit IRC14:02
*** irclogbot_2 has quit IRC14:02
*** kukacz has quit IRC14:02
*** gokhan has quit IRC14:02
*** chudler has quit IRC14:02
*** artom has quit IRC14:02
*** fanzhang has quit IRC14:02
*** ircuser-1 has quit IRC14:02
*** jaosorior has quit IRC14:02
*** kevinbenton has quit IRC14:02
*** tonyb has quit IRC14:02
*** burt has quit IRC14:02
*** eandersson has quit IRC14:02
*** gibi has quit IRC14:02
*** gryf has quit IRC14:02
*** kevinbenton has joined #openstack-nova14:02
*** dpawlik has joined #openstack-nova14:04
*** kukacz has joined #openstack-nova14:04
*** jdillaman has joined #openstack-nova14:04
*** dpawlik has quit IRC14:04
*** sambetts_ has joined #openstack-nova14:04
*** dpawlik has joined #openstack-nova14:04
*** tonyb has joined #openstack-nova14:07
*** gibi has joined #openstack-nova14:07
*** bnemec has joined #openstack-nova14:07
*** jaosorior has joined #openstack-nova14:08
*** jmlowe has quit IRC14:12
*** mriedem has joined #openstack-nova14:13
*** ttsiouts has quit IRC14:16
*** ttsiouts has joined #openstack-nova14:17
openstackgerritVlad Gusev proposed openstack/nova stable/queens: De-dupe subnet IDs when calling neutron /subnets API  https://review.openstack.org/61655114:18
dansmithmriedem: good for me to just double-ack this series from two stable cores? https://review.openstack.org/#/c/61249614:19
*** ttsiouts has quit IRC14:19
*** ttsiouts has joined #openstack-nova14:20
mriedemyeah14:20
maciejjozefczykmriedem: stephenfin johnthetubaguy PTAL https://review.openstack.org/#/c/59160714:22
mriedems10 (vlad) should probably look at that again to make sure he's happy with it14:23
*** davidsha has joined #openstack-nova14:25
mriedemdansmith: if you're in that stable kind of mood https://review.openstack.org/#/q/topic:bug/1797580+(status:open+OR+status:merged)14:26
mriedemthis has probably hung around too long as well https://review.openstack.org/#/q/topic:bug/1746863+status:open14:27
dansmithno good deed, eh?14:28
*** janki has quit IRC14:30
maciejjozefczykmriedem: right, I don't know his nick or he's simply off (s10)14:30
mriedemmust be off14:33
mriedemdansmith: you are rewarded with more reviews14:33
kashyapjroll: BTW, I agree with you there; added a comment in the change14:34
mriedembauzas: you were +2 on this https://review.openstack.org/#/c/610034/ - if you still are I guess we should get that in so we can move the mirrored placement change forward14:34
kashyapmriedem: About the 'virsh capabilities' ping you made last night, I see that's resolved as to what tssurya was looking for.14:34
tssuryakashyap: hi, yes thanks we found out the code and as suspected it was adding the offline cpus as well14:35
kashyapHi; cool.  (My "waking hours" are in the CET timezone)14:36
*** alexchadin has quit IRC14:39
*** moshele has quit IRC14:44
frickleraarents: we are affected by https://bugs.launchpad.net/nova/+bug/1801702 and your patch works fine for us, too. just needed to find the correct location for that function in pike. do you want to submit it yourself to upstream? otherwise I'd push it and add you as co-author14:48
openstackLaunchpad bug 1801702 in OpenStack Compute (nova) "Spawn may fail when cache=none on block device with logical block size > 512" [Undecided,New]14:48
fricklernova bug triagers: ^^ that bug is critical for us, please assign some importance14:49
*** tbachman has joined #openstack-nova14:53
*** awaugama has joined #openstack-nova14:54
mriedemfrickler: done15:01
sean-k-mooneyfrickler: 4k sectore are becomming more common even when not using nvme so yes this should be fixed15:01
mriedemi suggested trying in order from largest to smallest15:01
mriedemcould be a config option i guess....but that seems unnecessary15:02
*** dtantsur|brb is now known as dtantsur15:02
*** faridda has joined #openstack-nova15:03
sean-k-mooneymriedem: perhaps that said i was under the impression there was a way to ask the device for its native alingment so we might be able to discover it15:03
frickler4096 should always work since it is a multiple of 512, but I'm not 100% sure about other possible regressions15:03
sean-k-mooneyfrickler: alot of 512 driver these days are actully 4096 with 512 sector emulation15:04
*** mvkr has quit IRC15:04
*** jmlowe has joined #openstack-nova15:05
sean-k-mooneyfrickler: i do know that older windows versions ran slower/ had issue with 4k native boot media15:05
*** faridda has quit IRC15:08
*** ccamacho has joined #openstack-nova15:09
openstackgerritJens Harbott (frickler) proposed openstack/nova master: Make supports_direct_io work on 4096b sector size  https://review.openstack.org/61658015:12
jrollkashyap: thanks!15:12
mriedemalex_xu: replied to you in https://review.openstack.org/#/c/615341/ - i have a goal in what i'm cleaning up so i'm not sure how much we should pack into each change, versus me just doing follow ups15:14
*** ttsiouts has quit IRC15:15
*** ttsiouts has joined #openstack-nova15:15
*** ttsiouts has quit IRC15:19
*** mvkr has joined #openstack-nova15:20
aspiersmriedem, gibi: I'm reasonably optimistic that https://review.openstack.org/#/c/609779/14 can be the final patchset :-)15:22
*** ttsiouts has joined #openstack-nova15:22
*** jmlowe has quit IRC15:22
gibiaspiers: I've opened your patch for rereview15:28
mriedemaspiers: i just need to go over it again, been busy this week before the summit15:32
*** mchlumsky has joined #openstack-nova15:35
aspiersthanks both! hopefully nothing controversial since your last reviews :)15:35
*** hamzy has quit IRC15:36
mriedemi can't remember my last review on it15:38
mriedemexcept concerns about move operations15:38
*** moshele has joined #openstack-nova15:38
aspiersaccording to Gerrit it was on patch set 715:39
mriedemtssurya: https://review.openstack.org/#/c/569055/ needs work - i can update it quick though15:41
mriedemsince the changes needed are pretty minor15:41
*** rajinir has joined #openstack-nova15:42
openstackgerritMartin Midolesov proposed openstack/nova master: VMware: Expose esx hosts to Openstack  https://review.openstack.org/61362615:42
openstackgerritMartin Midolesov proposed openstack/nova master: [vmware] Cross vCenter live migration  https://review.openstack.org/61659015:42
openstackgerritEric Fried proposed openstack/nova master: Cap oslo.service to 1.32.0 (temporarily)  https://review.openstack.org/61659115:43
efriedbnemec, melwitt: ^15:43
efriedmriedem: Can we get a quick merge on --^ to help us unwind the mess described therein?15:43
*** ttsiouts has quit IRC15:46
*** ttsiouts has joined #openstack-nova15:47
*** moshele has quit IRC15:48
*** ttsiouts has quit IRC15:49
*** ttsiouts has joined #openstack-nova15:49
*** derekh has joined #openstack-nova15:52
*** artom has joined #openstack-nova15:54
openstackgerritMatt Riedemann proposed openstack/nova master: Make _instances_cores_ram_count() be smart about cells  https://review.openstack.org/56905515:54
openstackgerritMatt Riedemann proposed openstack/nova master: Add DownCellFixture  https://review.openstack.org/61481015:54
openstackgerritMatt Riedemann proposed openstack/nova master: [WIP] Add os_compute_api:servers:create:cell_down policy  https://review.openstack.org/61478315:54
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: API microversion bump for handling-down-cell  https://review.openstack.org/59165715:54
mriedemmelwitt: dansmith: i think https://review.openstack.org/#/c/569055/ is good to go and something we can backport to rocky - that's the one that limits counting instances/cores/ram to only cells in which the tenant has instances15:56
melwittefried: ack15:57
melwittmriedem: ack15:57
openstackgerritIvaylo Mitev proposed openstack/nova master: VMware: Attach volumes using adapter type from instance  https://review.openstack.org/61659915:58
*** ttsiouts has quit IRC16:07
dansmithmriedem: so on that.. is there any reason we need to or should limit that based on config?16:07
dansmithmriedem: because it doesn't really make sense to look at cells we think aren't going to have anything16:08
mriedemmaybe not16:08
*** ttsiouts has joined #openstack-nova16:08
*** pvc has joined #openstack-nova16:09
pvchi anyone uses cyborg here?16:09
melwittdansmith: tssurya did that to avoid a useless lookup in the single cell case, similar to instance list16:11
dansmithmelwitt: we can look at the cell cache and know whether or not we have any more than one non-zero cell right?16:11
tssuryamriedem thanks16:13
melwittmaybe. I don't remember the details around the config option to avoid a "by project" check to save extra unneeded lookups if an operator is running only a single cell deployment16:14
*** pcaruana has quit IRC16:14
melwittwe had been thinking this would be the same situation16:14
tssuryadansmith: yea, we did the config for listing to avoid the lookup so thought of doing the same here,16:15
tssurya(sorry leaving office now..)16:15
dansmithmelwitt: tssurya yeah I was thinking the instance list thing had more distinction than just the "could be running one cell"16:15
dansmithlike,16:15
*** mvkr has quit IRC16:16
dansmithif you have multiple cells but you don't keep people to a single cell,16:16
dansmiththen getting the mappings every time they do a list is pointless16:16
dansmithI guess it's similar for quota16:16
dansmithbut quota would be a check on boot, which is less often than list, which could be getting hit really hard in a polling loop16:16
melwittoh, right, it was to save on pulling mappings16:16
dansmithanyway, just struck me as less sane to have that be configurable for the quota one16:17
*** dklyle has quit IRC16:17
*** imacdonn has quit IRC16:17
*** imacdonn has joined #openstack-nova16:18
*** gyee has joined #openstack-nova16:18
mriedemefried: zuul failed16:19
efriedy16:19
efriedby which I meant16:19
efriedf16:19
*** tssurya has quit IRC16:19
*** panda|off is now known as panda|rover16:19
efriedahjeez. More chicken/egg.16:20
*** hamzy has joined #openstack-nova16:22
*** Luzi has quit IRC16:22
*** dklyle has joined #openstack-nova16:23
openstackgerritKashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Add a default Virtio RNG device to guests  https://review.openstack.org/61660316:27
*** mvkr has joined #openstack-nova16:28
efriedmelwitt, mriedem: See https://review.openstack.org/616604 <== this will come first; then the nova cap (https://review.openstack.org/616591) then the u-c bumps (https://review.openstack.org/#/c/615676/ and/or https://review.openstack.org/#/c/616371/, but at least the latter) then melwitt's actual fix (https://review.openstack.org/#/c/615724/)16:30
openstackgerritEric Fried proposed openstack/nova master: Cap oslo.service to 1.32.0 (temporarily)  https://review.openstack.org/61659116:37
*** slaweq has quit IRC16:40
*** helenafm has quit IRC16:48
*** slaweq has joined #openstack-nova16:58
*** sahid has quit IRC16:59
efriedmelwitt, mriedem: btw, we're discussing this in openstack-requirements if you're interested in following along.17:02
*** ivve has joined #openstack-nova17:02
*** panda|rover is now known as panda|rover|off17:02
openstackgerritEric Fried proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait  https://review.openstack.org/61572417:05
openstackgerritMerged openstack/nova stable/rocky: Add regression test for bug 1797580  https://review.openstack.org/61193817:07
openstackbug 1797580 in OpenStack Compute (nova) rocky "NoValidHost during live migration after cold migrating to a specified host" [High,In progress] https://launchpad.net/bugs/1797580 - Assigned to Matt Riedemann (mriedem)17:07
openstackgerritEric Fried proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait  https://review.openstack.org/61572417:08
*** nicolasbock_ has joined #openstack-nova17:09
efriedah, rats, forgot to remove the g-r exclusions...17:10
*** derekh has quit IRC17:14
*** derekh has joined #openstack-nova17:15
*** ttsiouts has quit IRC17:16
*** k_mouza has quit IRC17:24
*** davidsha has quit IRC17:25
*** ccamacho has quit IRC17:32
*** mvkr has quit IRC17:35
*** moshele has joined #openstack-nova17:37
*** moshele has quit IRC17:48
*** liuyulong has quit IRC17:50
*** jpena is now known as jpena|off17:51
*** nicolasbock has joined #openstack-nova17:56
nicolasbockHi, when creating a server from an image with a flavor that has a disk of 50 GiB I am told that the the image size is too large. I am quoted the actual file size of the qcow2 file and not the size of the partition on that image.17:56
nicolasbockDid I misunderstand the meaning of disk in flavor?17:56
nicolasbockI assumed that it's referring to the disk size, i.e. that it should relate to the disk that's encoded in the qcow2.17:57
sean-k-mooneynicolasbock: glance shows the actual file size17:57
nicolasbockBut isn't it Nova that's complaining?17:57
sean-k-mooneyglance has extra metadata to store the minium required disk space to unpack the image17:57
nicolasbockSo the qcow2 is unpacked?17:58
sean-k-mooneynicolasbock: the nova flavor disk attribute is the size the qcow will be unpacked too17:58
*** k_mouza has joined #openstack-nova17:58
sean-k-mooneynicolasbock: if the qcow file system is bigger nova will complain17:58
sean-k-mooneynova is allowed to expand the disk but not srink it17:58
nicolasbockOk, I get that sean-k-mooney (IRC) , but the error message is quoting the file size of the qcow2 image and not the size of the unpacked FS17:59
*** irclogbot_2 has joined #openstack-nova17:59
nicolasbocksean-k-mooney (IRC): you are saying that the disksize inside the image should be <= the disk size of the flavor?17:59
sean-k-mooneynicolasbock: can you paste a copy of the error to paste.openstack.org17:59
sean-k-mooneynicolasbock: yes18:00
sean-k-mooneynicolasbock: nova enforces that the virtual size the disk could grow too is <= the disk in the flavor18:00
nicolasbocksean-k-mooney (IRC): http://paste.openstack.org/show/734437/18:01
*** ircuser-1 has joined #openstack-nova18:01
*** derekh has quit IRC18:01
sean-k-mooneyya so in that case the image seams to be larger then the flavor allows18:01
sean-k-mooneynicolasbock: it looks like you tried to boot a 65GiB image on a 50GiB flavor18:02
nicolasbockThe partition inside the image is 80GiB large though18:03
nicolasbockI mean not that I am arguing with Nova here I am just saying that the error message was confusing to me :)18:03
sean-k-mooneystrang well the error is clearly showing the glance value it might check both18:04
nicolasbockOk, as long is this is expected behavior I am good. I was just expecting something else :)18:05
sean-k-mooneythe behavior looks correct to me. out of interest what were you expecting18:05
sean-k-mooneywas it 80GiB is lager then 50GiB18:05
nicolasbockI was expecting the size of the unpacked disk18:06
efriedmriedem: Does deleting the compute service today delete the provider in placement?18:06
sean-k-mooneyi would guess we have an optimization that checks the glance size first sice we already have that before actully checking the size of the file which would require downloading it18:06
sean-k-mooneynicolasbock: the glance vs flavor check can happen at the api level the actual size can only happen on the compute nodes18:07
nicolasbockOk that's a good point18:07
nicolasbockI am going to resize the FS and the disk of that qcow218:08
nicolasbocksean-k-mooney (IRC): Thanks for the help!18:08
sean-k-mooneynicolasbock: if you flavor allowsed 70Gib then it would likely pass the api check and hit the compute node check18:08
sean-k-mooneynicolasbock: no worries18:09
nicolasbockBut it would blow up then, no?18:09
sean-k-mooneyyes just much later18:09
nicolasbockThere is another check on the compute node?18:09
nicolasbockOk18:09
nicolasbockThanks!18:09
*** pcaruana has joined #openstack-nova18:10
*** moshele has joined #openstack-nova18:15
openstackgerritMerged openstack/nova stable/rocky: Don't persist RequestSpec.requested_destination  https://review.openstack.org/61193918:22
*** adrianc has quit IRC18:24
*** k_mouza has quit IRC18:25
*** jmlowe has joined #openstack-nova18:31
*** moshele has quit IRC18:44
*** jmlowe has quit IRC18:59
*** dpawlik has quit IRC19:00
*** dpawlik has joined #openstack-nova19:00
melwittmriedem: should we remove use-nested-allocation-candidates from the runway now? I noticed that it seemed to morph into bandwidth-resource-provider19:05
*** ralonsoh has quit IRC19:24
*** jmlowe has joined #openstack-nova19:26
*** erlon_ has quit IRC19:31
*** mriedem has quit IRC19:32
*** k_mouza has joined #openstack-nova19:44
*** k_mouza has quit IRC19:45
jmloweHas anybody had problems with instances getting multiple placement allocations?19:45
*** k_mouza has joined #openstack-nova19:48
*** awaugama has quit IRC19:49
*** k_mouza has quit IRC19:52
*** dave-mccowan has joined #openstack-nova19:56
*** mriedem has joined #openstack-nova19:59
mriedemefried: yes the api attempts to delete the resource provider when a compute service is deleted19:59
mriedemmelwitt: idk about that use-nested-alloc-candidates bp, or what's left, probably need to ask gibi/tetsuro20:00
melwittmriedem: yeah, I'll ask gibi. thanks20:01
*** dpawlik has quit IRC20:10
mriedemefried: is https://review.openstack.org/#/c/616371/ expected to be failing?20:15
mriedemyay https://review.openstack.org/#/c/614489/ - hopefully we don't have to do the same dance in stable/rocky, but we probably will if the requirements team ever wants to increase the upper-constraint on oslo.service in rocky20:18
mriedemefried: melwitt: ok i'm +2 on those two nova changes20:21
melwittthank ya20:22
mriedemi'm not sure why https://review.openstack.org/#/c/616371/ is failing though20:22
mriedemoh i see,20:22
mriedemit needs https://review.openstack.org/#/c/615724/20:22
mriedembut that depeneds on 61673120:22
mriedem*37120:23
mriedemso we just need to land the nova changes20:23
mriedemwell...20:23
mriedembut the top nova change can't land w/o that requirements dependency, which is failing b/c the code is still mocking the removed thing20:23
*** awaugama has joined #openstack-nova20:23
*** eharney has quit IRC20:24
melwittyeah :/20:24
melwittefried halp20:24
efriedmelwitt: o/ Sorry, was getting a CT of my face. Took an elbow yesterday. (Don't worry, I'm sure it'll still be rainbow-colored next week for ya)20:24
melwittdamn, sorry to hear that20:25
melwitthope you are ok20:25
efriedI'll know more in an hour. Hopefully no orbital/sinus fractures.20:26
efriedso, that bizarre requirements chain...20:26
efriedchecking...20:26
*** dpawlik has joined #openstack-nova20:26
efriedThree out of four are passing zuul, but https://review.openstack.org/#/c/616371/ isn't, and it's the cross-nova check that's being pooty...20:29
*** eharney has joined #openstack-nova20:29
efriedmelwitt, mriedem: Heading back to the requirements channel to pick this up...20:29
*** dpawlik has quit IRC20:30
*** dave-mccowan has quit IRC20:33
*** jmlowe has quit IRC20:38
*** mriedem has quit IRC20:41
*** dklyle has quit IRC20:45
*** dklyle has joined #openstack-nova20:45
*** mriedem has joined #openstack-nova20:45
mriedemdansmith: replied in https://review.openstack.org/#/c/569055/ for a case in which i think it makes sense to only ever count quota for cells in which the project has instances20:46
*** dklyle has quit IRC20:46
openstackgerritMerged openstack/nova master: Fix support matrix for VMware UEFI support  https://review.openstack.org/61587020:46
*** takashin has joined #openstack-nova20:47
dansmithmriedem: yeah, that's what I was saying20:47
dansmithmriedem: we could still ignore the failed cell 5 in your example and let the create continue, but we'd have sat around waiting for it for no reason20:48
mriedemin order to ignore it though, we'd have to determine if the project has instances in that cell, right? which means doing the instance mapping by project_id lookup anyway20:50
melwittnova meeting in 10 min20:50
dansmithmriedem: yep20:50
mriedemso we might as well just always do that20:50
dansmithmriedem: this is my point20:50
mriedemin the case of a cloud that's not multi-cell, do we care about optimizing this?20:51
melwittyeah, I think I got confused by the other instance list check where we could save on pruning cells if there's only one cell20:51
melwittI had thought there was a config option for avoiding that for single cell. and confused it with the quota situation20:51
dansmithmriedem: yeah you could filter out cell0 from the cache and if the result is only one avoid the lookup20:51
mriedeminstances in cell0 still count against your quota though20:52
mriedemi'm pretty sure20:52
melwittyeah they do20:52
melwitterror instances, for example20:52
dansmithyeah, I know,20:52
melwittoh, but you're saying don't count cell020:52
melwittyeah20:52
dansmithso you could also just say len(cache)==2, but.. checking that there is only one non-cell0 cell seems like a better assertion20:52
mriedemwfm20:53
dansmithother things would be wrong if those weren't the same, for sure, it just seems more of a positive thing20:53
dansmithbut that's just a detail, either way that optimization is easy20:53
melwittyeah. we could have a "is_single_cell" method. though I wonder how that works before the cache has been populated. I guess we know it has been before quota checks happen?20:53
dansmithif the cache isn't populated then we have to populate it before we look at the cache20:54
dansmiththat's how all that code works20:54
melwittok20:54
dansmithso is_single_cell() makes a _load_cells() call first20:54
*** jmlowe has joined #openstack-nova21:05
artomThis is weird, why is conductor failing with 'CantStartEngineError: No sql_connection parameter is established'21:05
*** brault has quit IRC21:06
mriedemartom: it's trying to hit the API DB21:06
mriedemand not configured for it21:06
mriedemif you're using devstack21:06
mriedemthe cell conductor that is21:06
artommriedem, that's exactly it, it's almost creey ;)21:06
mriedeme.g. resize fails on compute1, casts back to cell conductor which looks up the instance mapping and kablamo21:07
mriedemknown issue21:07
artomIt's a reschedule, the (presumably cell) conductor is trying to get the AZ21:07
mriedemyup, i know the bug21:07
mriedemsec21:07
mriedemartom: linked from https://blueprints.launchpad.net/nova/+spec/fix-reschedule-up-calls21:08
mriedemi have a fix for one of those21:08
artomThanks :)21:08
mriedemdiscussed at the ptg21:08
openstackgerritJack Ding proposed openstack/nova master: Add cache=none option for qemu-img convert  https://review.openstack.org/61669221:08
mriedemthe other, the one you're hitting, is a bigger change b/c we have to pass the azs down to the cell21:08
artomActually also affects plain boot21:08
mriedemon the Selection object21:08
artomHah, so I kinda need this fixed so that I can reproduce *another* bug correctly21:09
mriedemartom: b/c of this right? https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L67821:09
artommriedem, yep21:09
mriedemartom: well as a workaround just configure the [api_database]/connection in your nova-cell1.conf21:09
mriedemand restart the cell conductor21:09
artomAh, true21:09
mriedemi think devstack is the only thing out there that really does this split MQ thing21:10
mriedemeveryone else just runs with api db configured everywhere21:10
artomDoesn't that kind of defeat the scalability point of the API db?21:10
mriedemit's not really a scalability thing,21:11
mriedemit's an isolation thing21:11
artomOK - I just thought, if you're going to have all conductors talk to the api db, then why have a separate API db in the first place?21:11
mriedemhttps://docs.openstack.org/nova/latest/user/cellsv2-layout.html#operations-requiring-upcalls21:11
mriedemb/c dude21:11
artom(Not being flippant on purpose or anything, genuinely trying to grok this)21:12
mriedemb/c some things are in the api db schema like aggregates and cells, and some things are cell-specific, and we don't need to replicate that api db information across all cells21:13
artomAh, I see.21:14
mriedemlike cells v1 did21:14
mriedemit had to sync everything21:14
artomThe API DB wasn't born out of a need to reduce load, it was born out of a need to keep a single source of truth for things that are common across all cells21:14
artomIt's the *cells DBs* that are the scalability stuff21:14
mriedemyeah21:15
artomGot it. Thanks :)21:15
*** mriedem has left #openstack-nova21:18
*** mriedem has joined #openstack-nova21:18
*** pvc has quit IRC21:32
*** nicolasbock_ has quit IRC21:32
*** eharney has quit IRC21:37
*** ivve has quit IRC21:38
openstackgerritDoug Hellmann proposed openstack/nova master: Cap oslo.service to 1.32.0 (temporarily)  https://review.openstack.org/61659121:39
openstackgerritDoug Hellmann proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait  https://review.openstack.org/61572421:39
openstackgerritDoug Hellmann proposed openstack/nova master: remove mocks of oslo.service private members  https://review.openstack.org/61669721:39
*** hamzy has quit IRC21:43
openstackgerritDoug Hellmann proposed openstack/nova master: remove mocks of oslo.service private members  https://review.openstack.org/61669721:51
openstackgerritDoug Hellmann proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait  https://review.openstack.org/61572421:51
efriedmelwitt, mriedem: Y'all still around to approve the new version of the oslo.service mess?21:51
melwittefried: yeah21:52
melwittthe top change says merge conflict tho21:53
efriedmelwitt: First https://review.openstack.org/61669721:53
efriedyeah, we're still working on the requirements bit in the middle.21:53
*** artom has quit IRC21:53
melwittok. I didn't think that depends-on could cause the red "merge conflict" message21:54
efriedyes21:54
efriedwhich is a good thing, else we be like, "why is my damn patch stuck?"21:55
melwittTIL21:56
efriedexcept that might not be the actual problem here, sigh. Doug and I are still unwinding. But this is gonna work, so feel free to sprinkle +2s around like Halloween candy.21:56
*** manjeets has quit IRC22:00
mriedemefried: on https://review.openstack.org/#/c/616697/ the only downside is the slower tests right?22:02
efriedmelwitt: correct, immediately resolved by your patch.22:02
mriedemi'm mriedem22:02
efriedMy bad, you're both 7c starting with m and a very similar shade of green and melwitt was the last one to respond.22:03
mriedem+W22:03
efriedthanks22:03
efriedoh, and I do have a fractured orbital, though my vision is supposedly fine.22:09
mriedemgdi eric, you need a helmet22:22
mriedemwhile looking up helmets, i found this https://pbs.twimg.com/media/DiAdSIWUwAEXkAg.jpg22:22
mriedemlooks like jerry rice in there22:22
openstackgerritJack Ding proposed openstack/nova master: Change align_size to 4096 to support 4Kn drives  https://review.openstack.org/61670122:26
*** mriedem has quit IRC22:36
*** mriedem has joined #openstack-nova22:36
*** k_mouza has joined #openstack-nova22:42
openstackgerritMerged openstack/nova master: remove mocks of oslo.service private members  https://review.openstack.org/61669722:44
*** mriedem has quit IRC22:46
*** eharney has joined #openstack-nova22:47
*** k_mouza has quit IRC22:47
*** manjeets has joined #openstack-nova22:54
*** k_mouza has joined #openstack-nova22:57
*** awaugama has quit IRC23:02
*** k_mouza has quit IRC23:02
*** k_mouza has joined #openstack-nova23:07
*** k_mouza has quit IRC23:08
*** k_mouza has joined #openstack-nova23:08
*** artom has joined #openstack-nova23:14
*** slaweq has quit IRC23:15
*** med_ has joined #openstack-nova23:26
openstackgerritEric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache  https://review.openstack.org/61564623:32
openstackgerritEric Fried proposed openstack/nova master: Reduce calls to placement from _ensure  https://review.openstack.org/61567723:32
openstackgerritEric Fried proposed openstack/nova master: Consolidate inventory refresh  https://review.openstack.org/61569523:32
openstackgerritEric Fried proposed openstack/nova master: Commonize _update code path  https://review.openstack.org/61570523:32
openstackgerritEric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next  https://review.openstack.org/61603323:32
openstackgerritEric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache  https://review.openstack.org/61564623:33
openstackgerritEric Fried proposed openstack/nova master: Reduce calls to placement from _ensure  https://review.openstack.org/61567723:33
openstackgerritEric Fried proposed openstack/nova master: Consolidate inventory refresh  https://review.openstack.org/61569523:33
openstackgerritEric Fried proposed openstack/nova master: Commonize _update code path  https://review.openstack.org/61570523:33
openstackgerritEric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next  https://review.openstack.org/61603323:33
*** dklyle has joined #openstack-nova23:44
*** erlon_ has joined #openstack-nova23:46

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