*** tetsuro has joined #openstack-nova | 00:01 | |
*** gyee has quit IRC | 00:55 | |
*** vishwanathj has quit IRC | 00:57 | |
*** k_mouza has quit IRC | 01:16 | |
*** Swami has quit IRC | 01:24 | |
*** brinzhang has joined #openstack-nova | 01:29 | |
*** jdillaman has quit IRC | 01:37 | |
*** Dinesh_Bhor has joined #openstack-nova | 01:49 | |
*** mriedem has quit IRC | 01:50 | |
*** jdillaman has joined #openstack-nova | 02:04 | |
*** erlon_ has quit IRC | 02:10 | |
*** hongbin has joined #openstack-nova | 02:13 | |
*** tbachman has quit IRC | 02:14 | |
*** READ10 has quit IRC | 02:21 | |
*** mrsoul has joined #openstack-nova | 02:44 | |
*** tiendc has joined #openstack-nova | 02:44 | |
*** markvoelker has joined #openstack-nova | 02:45 | |
*** markvoelker has quit IRC | 02:45 | |
*** markvoelker has joined #openstack-nova | 02:45 | |
*** markvoelker has quit IRC | 02:50 | |
*** markvoelker has joined #openstack-nova | 02:55 | |
*** hshiina_ has joined #openstack-nova | 02:59 | |
*** hshiina has quit IRC | 03:02 | |
openstackgerrit | Fan Zhang proposed openstack/nova master: Retry after hitting libvirt error VIR_ERR_OPERATION_INVALID in live migration. https://review.openstack.org/612272 | 03:05 |
---|---|---|
*** fungi has quit IRC | 03:06 | |
*** fungi has joined #openstack-nova | 03:09 | |
*** fungi has quit IRC | 03:10 | |
*** Dinesh_Bhor has quit IRC | 03:38 | |
*** fungi has joined #openstack-nova | 03:40 | |
*** fungi has quit IRC | 03:41 | |
*** fungi has joined #openstack-nova | 03:45 | |
*** zhanglong has joined #openstack-nova | 03:49 | |
*** sapd1 has quit IRC | 03:58 | |
*** sapd1 has joined #openstack-nova | 03:58 | |
*** sambetts|afk has quit IRC | 04:11 | |
*** hshiina__ has joined #openstack-nova | 04:13 | |
*** hshiina_ has quit IRC | 04:16 | |
*** sambetts_ has joined #openstack-nova | 04:18 | |
*** hshiina__ has quit IRC | 04:19 | |
openstackgerrit | Merged openstack/nova master: Improve formats of the Compute API guide https://review.openstack.org/615809 | 04:20 |
*** hshiina has joined #openstack-nova | 04:24 | |
*** hshiina_ has joined #openstack-nova | 04:26 | |
*** hshiina has quit IRC | 04:29 | |
*** janki has joined #openstack-nova | 04:36 | |
*** Dinesh_Bhor has joined #openstack-nova | 04:42 | |
*** cfriesen has quit IRC | 04:53 | |
*** Dinesh_Bhor has quit IRC | 05:15 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:20 | |
*** munimeha1 has quit IRC | 05:28 | |
*** tetsuro has quit IRC | 05:33 | |
*** hshiina_ has quit IRC | 05:38 | |
*** hshiina has joined #openstack-nova | 05:38 | |
*** hshiina has quit IRC | 05:43 | |
*** hshiina has joined #openstack-nova | 05:51 | |
*** hshiina_ has joined #openstack-nova | 05:52 | |
*** hshiina has quit IRC | 05:56 | |
*** hshiina__ has joined #openstack-nova | 05:56 | |
*** hshiina_ has quit IRC | 06:00 | |
*** hongbin has quit IRC | 06:02 | |
*** TuanDA has joined #openstack-nova | 06:09 | |
*** cfriesen has joined #openstack-nova | 06:17 | |
*** gokhan has joined #openstack-nova | 06:51 | |
*** brinzhang has quit IRC | 06:54 | |
*** brinzhang has joined #openstack-nova | 06:54 | |
*** sahid has joined #openstack-nova | 06:58 | |
*** cfriesen has quit IRC | 07:00 | |
*** dpawlik has joined #openstack-nova | 07:04 | |
*** brinzhang has quit IRC | 07:20 | |
*** brinzhang has joined #openstack-nova | 07:20 | |
*** tssurya has joined #openstack-nova | 07:23 | |
*** adrianc has joined #openstack-nova | 07:30 | |
*** pcaruana has joined #openstack-nova | 07:34 | |
*** slaweq has joined #openstack-nova | 07:39 | |
*** alexchadin has joined #openstack-nova | 07:39 | |
*** jangutter has quit IRC | 07:57 | |
*** jangutter has joined #openstack-nova | 07:57 | |
*** bnemec has quit IRC | 08:00 | |
openstackgerrit | liuyamin proposed openstack/nova master: Fix version details API does not return 200 OK https://review.openstack.org/616444 | 08:03 |
*** helenafm has joined #openstack-nova | 08:10 | |
*** Dinesh_Bhor has quit IRC | 08:11 | |
*** Dinesh_Bhor has joined #openstack-nova | 08:12 | |
*** ralonsoh has joined #openstack-nova | 08:17 | |
*** adrianc has quit IRC | 08:19 | |
*** sapd1 has quit IRC | 08:20 | |
*** adrianc has joined #openstack-nova | 08:20 | |
*** sapd1 has joined #openstack-nova | 08:20 | |
*** bnemec has joined #openstack-nova | 08:26 | |
*** Luzi has joined #openstack-nova | 08:30 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | 08:34 |
*** jpena|off is now known as jpena | 08:36 | |
*** moshele has joined #openstack-nova | 08:37 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | 08:40 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | 08:40 |
*** Cathyz has joined #openstack-nova | 08:42 | |
*** hshiina__ has quit IRC | 08:46 | |
*** sapd1_ has joined #openstack-nova | 08:47 | |
*** sapd1 has quit IRC | 08:49 | |
openstackgerrit | Jeffrey Zhang proposed openstack/nova master: Add feature to flatten the volume from glance image snapshort https://review.openstack.org/616461 | 08:50 |
*** zhanglong has quit IRC | 09:00 | |
openstackgerrit | Jeffrey Zhang proposed openstack/nova master: Add feature to flatten the volume from glance image snapshort https://review.openstack.org/616461 | 09:06 |
*** Cathyz has quit IRC | 09:11 | |
*** sahid has quit IRC | 09:13 | |
*** sahid has joined #openstack-nova | 09:13 | |
*** BlackDex has quit IRC | 09:20 | |
*** jcosmao has joined #openstack-nova | 09:20 | |
*** BlackDex has joined #openstack-nova | 09:25 | |
*** derekh has joined #openstack-nova | 09:32 | |
*** moshele has quit IRC | 09:39 | |
*** ttsiouts has joined #openstack-nova | 09:54 | |
*** dtantsur|afk is now known as dtantsur | 09:55 | |
*** sapd1_ has quit IRC | 09:55 | |
*** ttsiouts has quit IRC | 09:56 | |
*** ttsiouts has joined #openstack-nova | 09:57 | |
*** mvkr has quit IRC | 09:58 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add extra unit test coverage for @run_once https://review.openstack.org/616485 | 09:58 |
*** ttsiouts_ has joined #openstack-nova | 09:59 | |
*** ttsiouts has quit IRC | 09:59 | |
gibi | sean-k-mooney: I left some comments in https://review.openstack.org/#/c/610034 and proposed a followup ^^ | 10:01 |
*** sapd1_ has joined #openstack-nova | 10:05 | |
*** ttsiouts_ has quit IRC | 10:19 | |
*** ttsiouts has joined #openstack-nova | 10:20 | |
*** ttsiouts has quit IRC | 10:24 | |
*** mvkr has joined #openstack-nova | 10:26 | |
*** tetsuro has joined #openstack-nova | 10:27 | |
*** brinzhang has quit IRC | 10:28 | |
*** panda|off is now known as panda | 10:31 | |
*** ttsiouts has joined #openstack-nova | 10:34 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send rp_uuid in the port binding https://review.openstack.org/569459 | 10:35 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | 10:36 |
*** efried has quit IRC | 10:38 | |
*** efried has joined #openstack-nova | 10:38 | |
*** adrianc has quit IRC | 11:01 | |
*** erlon_ has joined #openstack-nova | 11:08 | |
*** davidsha has joined #openstack-nova | 11:12 | |
*** adrianc has joined #openstack-nova | 11:22 | |
*** ttsiouts has quit IRC | 11:35 | |
*** ttsiouts has joined #openstack-nova | 11:35 | |
*** ttsiouts has quit IRC | 11:38 | |
*** ttsiouts_ has joined #openstack-nova | 11:38 | |
sean-k-mooney | gibi: hi | 11:56 |
gibi | sean-k-mooney: hi | 11:56 |
sean-k-mooney | i was under the impression that try finally was not legal in python but reading the reference docs on the try statement i apears to be | 11:56 |
sean-k-mooney | i can remove teh except blocks | 11:57 |
gibi | sean-k-mooney: yeah, try:finally: without except is totally valid | 11:57 |
gibi | sean-k-mooney: also if you want, you can squash my followup into your patch | 11:57 |
sean-k-mooney | i 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 backport | 11:58 |
gibi | sean-k-mooney: sure, that totally works for me | 11:59 |
sean-k-mooney | thanks 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|lunch | 12:03 | |
*** Luzi has quit IRC | 12:05 | |
*** TuanDA has quit IRC | 12:05 | |
*** tetsuro has quit IRC | 12:06 | |
gibi | sean-k-mooney: :) | 12:11 |
*** ttsiouts_ has quit IRC | 12:14 | |
*** ttsiouts has joined #openstack-nova | 12:14 | |
*** ttsiouts has quit IRC | 12:19 | |
*** Luzi has joined #openstack-nova | 12:20 | |
*** janki has quit IRC | 12:20 | |
*** dtantsur is now known as dtantsur|brb | 12:32 | |
openstackgerrit | sean mooney proposed openstack/nova master: Harden placement init under wsgi https://review.openstack.org/610034 | 12:33 |
*** Dinesh_Bhor has quit IRC | 12:43 | |
*** ttsiouts has joined #openstack-nova | 12:52 | |
*** ccamacho has quit IRC | 12:53 | |
*** ttsiouts has quit IRC | 12:56 | |
*** ttsiouts has joined #openstack-nova | 12:57 | |
*** panda is now known as panda|off | 12:59 | |
*** dpawlik has quit IRC | 13:11 | |
sean-k-mooney | gibi: i have cross ported the squashed patch to placement. can you abandon https://review.openstack.org/#/c/616485 when you get a chance | 13:13 |
*** ccamacho has joined #openstack-nova | 13:18 | |
sean-k-mooney | bauzas: 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 summit | 13:19 |
*** jpena|lunch is now known as jpena | 13:24 | |
bauzas | sean-k-mooney: I can try :) | 13:25 |
bauzas | but still on a customer issue | 13:25 |
* bauzas raises fist at anti-affinity checks | 13:25 | |
*** dpawlik has joined #openstack-nova | 13:26 | |
*** eharney has joined #openstack-nova | 13:27 | |
sean-k-mooney | bauzas: for the live migration case | 13:27 |
bauzas | indeed | 13:27 |
bauzas | pbd'ing | 13:27 |
*** janki has joined #openstack-nova | 13:29 | |
sean-k-mooney | bauzas: 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 correct | 13:29 |
*** dpawlik has quit IRC | 13:31 | |
bauzas | sean-k-mooney: no, it's not a concurrent issue | 13:32 |
bauzas | sean-k-mooney: I guess the problem is about the persisted RequestGroup | 13:32 |
bauzas | oops s/RequestGroup/RequestSpec | 13:32 |
bauzas | AFAICS, the instancegroup.members is wrong | 13:33 |
bauzas | anyway | 13:33 |
bauzas | I'll continue to look | 13:33 |
sean-k-mooney | bauzas: do you have a functional test that can reporduce? | 13:33 |
*** tiendc has quit IRC | 13:33 | |
sean-k-mooney | i have a couple of small patches i need to work on but i can try and help later once i get them written/pushed | 13:34 |
sean-k-mooney | i 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 |
bauzas | I also have to go in a garage | 13:35 |
bauzas | see you in 1h | 13:35 |
sean-k-mooney | o/ | 13:35 |
*** k_mouza has joined #openstack-nova | 13:38 | |
*** adrianc has quit IRC | 13:40 | |
*** adrianc has joined #openstack-nova | 13:41 | |
jroll | kashyap: when you have a few minutes, curious your thoughts on this: https://review.openstack.org/#/c/616405/ | 13:47 |
kashyap | jroll: Hi | 13:48 |
jroll | hi :) | 13:48 |
kashyap | Let me check w/ the libvirt upstream folks | 13:48 |
kashyap | jroll: I'd expect the reporter to also note what precise failure they're seeing | 13:50 |
kashyap | Let me ask on the change | 13:50 |
jroll | kashyap: good point | 13:51 |
*** ccamacho has quit IRC | 13:52 | |
kashyap | Asked on the change. | 13:52 |
*** ccamacho has joined #openstack-nova | 13:53 | |
*** moshele has joined #openstack-nova | 13:54 | |
*** dpawlik has joined #openstack-nova | 13:58 | |
*** dpawlik has quit IRC | 13:59 | |
*** ccamacho has quit IRC | 14:02 | |
*** derekh has quit IRC | 14:02 | |
*** davidsha has quit IRC | 14:02 | |
*** jcosmao has quit IRC | 14:02 | |
*** bnemec has quit IRC | 14:02 | |
*** sambetts_ has quit IRC | 14:02 | |
*** jdillaman has quit IRC | 14:02 | |
*** irclogbot_2 has quit IRC | 14:02 | |
*** kukacz has quit IRC | 14:02 | |
*** gokhan has quit IRC | 14:02 | |
*** chudler has quit IRC | 14:02 | |
*** artom has quit IRC | 14:02 | |
*** fanzhang has quit IRC | 14:02 | |
*** ircuser-1 has quit IRC | 14:02 | |
*** jaosorior has quit IRC | 14:02 | |
*** kevinbenton has quit IRC | 14:02 | |
*** tonyb has quit IRC | 14:02 | |
*** burt has quit IRC | 14:02 | |
*** eandersson has quit IRC | 14:02 | |
*** gibi has quit IRC | 14:02 | |
*** gryf has quit IRC | 14:02 | |
*** kevinbenton has joined #openstack-nova | 14:02 | |
*** dpawlik has joined #openstack-nova | 14:04 | |
*** kukacz has joined #openstack-nova | 14:04 | |
*** jdillaman has joined #openstack-nova | 14:04 | |
*** dpawlik has quit IRC | 14:04 | |
*** sambetts_ has joined #openstack-nova | 14:04 | |
*** dpawlik has joined #openstack-nova | 14:04 | |
*** tonyb has joined #openstack-nova | 14:07 | |
*** gibi has joined #openstack-nova | 14:07 | |
*** bnemec has joined #openstack-nova | 14:07 | |
*** jaosorior has joined #openstack-nova | 14:08 | |
*** jmlowe has quit IRC | 14:12 | |
*** mriedem has joined #openstack-nova | 14:13 | |
*** ttsiouts has quit IRC | 14:16 | |
*** ttsiouts has joined #openstack-nova | 14:17 | |
openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: De-dupe subnet IDs when calling neutron /subnets API https://review.openstack.org/616551 | 14:18 |
dansmith | mriedem: good for me to just double-ack this series from two stable cores? https://review.openstack.org/#/c/612496 | 14:19 |
*** ttsiouts has quit IRC | 14:19 | |
*** ttsiouts has joined #openstack-nova | 14:20 | |
mriedem | yeah | 14:20 |
maciejjozefczyk | mriedem: stephenfin johnthetubaguy PTAL https://review.openstack.org/#/c/591607 | 14:22 |
mriedem | s10 (vlad) should probably look at that again to make sure he's happy with it | 14:23 |
*** davidsha has joined #openstack-nova | 14:25 | |
mriedem | dansmith: if you're in that stable kind of mood https://review.openstack.org/#/q/topic:bug/1797580+(status:open+OR+status:merged) | 14:26 |
mriedem | this has probably hung around too long as well https://review.openstack.org/#/q/topic:bug/1746863+status:open | 14:27 |
dansmith | no good deed, eh? | 14:28 |
*** janki has quit IRC | 14:30 | |
maciejjozefczyk | mriedem: right, I don't know his nick or he's simply off (s10) | 14:30 |
mriedem | must be off | 14:33 |
mriedem | dansmith: you are rewarded with more reviews | 14:33 |
kashyap | jroll: BTW, I agree with you there; added a comment in the change | 14:34 |
mriedem | bauzas: 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 forward | 14:34 |
kashyap | mriedem: About the 'virsh capabilities' ping you made last night, I see that's resolved as to what tssurya was looking for. | 14:34 |
tssurya | kashyap: hi, yes thanks we found out the code and as suspected it was adding the offline cpus as well | 14:35 |
kashyap | Hi; cool. (My "waking hours" are in the CET timezone) | 14:36 |
*** alexchadin has quit IRC | 14:39 | |
*** moshele has quit IRC | 14:44 | |
frickler | aarents: 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-author | 14:48 |
openstack | Launchpad bug 1801702 in OpenStack Compute (nova) "Spawn may fail when cache=none on block device with logical block size > 512" [Undecided,New] | 14:48 |
frickler | nova bug triagers: ^^ that bug is critical for us, please assign some importance | 14:49 |
*** tbachman has joined #openstack-nova | 14:53 | |
*** awaugama has joined #openstack-nova | 14:54 | |
mriedem | frickler: done | 15:01 |
sean-k-mooney | frickler: 4k sectore are becomming more common even when not using nvme so yes this should be fixed | 15:01 |
mriedem | i suggested trying in order from largest to smallest | 15:01 |
mriedem | could be a config option i guess....but that seems unnecessary | 15:02 |
*** dtantsur|brb is now known as dtantsur | 15:02 | |
*** faridda has joined #openstack-nova | 15:03 | |
sean-k-mooney | mriedem: 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 it | 15:03 |
frickler | 4096 should always work since it is a multiple of 512, but I'm not 100% sure about other possible regressions | 15:03 |
sean-k-mooney | frickler: alot of 512 driver these days are actully 4096 with 512 sector emulation | 15:04 |
*** mvkr has quit IRC | 15:04 | |
*** jmlowe has joined #openstack-nova | 15:05 | |
sean-k-mooney | frickler: i do know that older windows versions ran slower/ had issue with 4k native boot media | 15:05 |
*** faridda has quit IRC | 15:08 | |
*** ccamacho has joined #openstack-nova | 15:09 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack/nova master: Make supports_direct_io work on 4096b sector size https://review.openstack.org/616580 | 15:12 |
jroll | kashyap: thanks! | 15:12 |
mriedem | alex_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 ups | 15:14 |
*** ttsiouts has quit IRC | 15:15 | |
*** ttsiouts has joined #openstack-nova | 15:15 | |
*** ttsiouts has quit IRC | 15:19 | |
*** mvkr has joined #openstack-nova | 15:20 | |
aspiers | mriedem, gibi: I'm reasonably optimistic that https://review.openstack.org/#/c/609779/14 can be the final patchset :-) | 15:22 |
*** ttsiouts has joined #openstack-nova | 15:22 | |
*** jmlowe has quit IRC | 15:22 | |
gibi | aspiers: I've opened your patch for rereview | 15:28 |
mriedem | aspiers: i just need to go over it again, been busy this week before the summit | 15:32 |
*** mchlumsky has joined #openstack-nova | 15:35 | |
aspiers | thanks both! hopefully nothing controversial since your last reviews :) | 15:35 |
*** hamzy has quit IRC | 15:36 | |
mriedem | i can't remember my last review on it | 15:38 |
mriedem | except concerns about move operations | 15:38 |
*** moshele has joined #openstack-nova | 15:38 | |
aspiers | according to Gerrit it was on patch set 7 | 15:39 |
mriedem | tssurya: https://review.openstack.org/#/c/569055/ needs work - i can update it quick though | 15:41 |
mriedem | since the changes needed are pretty minor | 15:41 |
*** rajinir has joined #openstack-nova | 15:42 | |
openstackgerrit | Martin Midolesov proposed openstack/nova master: VMware: Expose esx hosts to Openstack https://review.openstack.org/613626 | 15:42 |
openstackgerrit | Martin Midolesov proposed openstack/nova master: [vmware] Cross vCenter live migration https://review.openstack.org/616590 | 15:42 |
openstackgerrit | Eric Fried proposed openstack/nova master: Cap oslo.service to 1.32.0 (temporarily) https://review.openstack.org/616591 | 15:43 |
efried | bnemec, melwitt: ^ | 15:43 |
efried | mriedem: Can we get a quick merge on --^ to help us unwind the mess described therein? | 15:43 |
*** ttsiouts has quit IRC | 15:46 | |
*** ttsiouts has joined #openstack-nova | 15:47 | |
*** moshele has quit IRC | 15:48 | |
*** ttsiouts has quit IRC | 15:49 | |
*** ttsiouts has joined #openstack-nova | 15:49 | |
*** derekh has joined #openstack-nova | 15:52 | |
*** artom has joined #openstack-nova | 15:54 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Make _instances_cores_ram_count() be smart about cells https://review.openstack.org/569055 | 15:54 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add DownCellFixture https://review.openstack.org/614810 | 15:54 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: [WIP] Add os_compute_api:servers:create:cell_down policy https://review.openstack.org/614783 | 15:54 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: API microversion bump for handling-down-cell https://review.openstack.org/591657 | 15:54 |
mriedem | melwitt: 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 instances | 15:56 |
melwitt | efried: ack | 15:57 |
melwitt | mriedem: ack | 15:57 |
openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware: Attach volumes using adapter type from instance https://review.openstack.org/616599 | 15:58 |
*** ttsiouts has quit IRC | 16:07 | |
dansmith | mriedem: so on that.. is there any reason we need to or should limit that based on config? | 16:07 |
dansmith | mriedem: because it doesn't really make sense to look at cells we think aren't going to have anything | 16:08 |
mriedem | maybe not | 16:08 |
*** ttsiouts has joined #openstack-nova | 16:08 | |
*** pvc has joined #openstack-nova | 16:09 | |
pvc | hi anyone uses cyborg here? | 16:09 |
melwitt | dansmith: tssurya did that to avoid a useless lookup in the single cell case, similar to instance list | 16:11 |
dansmith | melwitt: we can look at the cell cache and know whether or not we have any more than one non-zero cell right? | 16:11 |
tssurya | mriedem thanks | 16:13 |
melwitt | maybe. 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 deployment | 16:14 |
*** pcaruana has quit IRC | 16:14 | |
melwitt | we had been thinking this would be the same situation | 16:14 |
tssurya | dansmith: 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 |
dansmith | melwitt: tssurya yeah I was thinking the instance list thing had more distinction than just the "could be running one cell" | 16:15 |
dansmith | like, | 16:15 |
*** mvkr has quit IRC | 16:16 | |
dansmith | if you have multiple cells but you don't keep people to a single cell, | 16:16 |
dansmith | then getting the mappings every time they do a list is pointless | 16:16 |
dansmith | I guess it's similar for quota | 16:16 |
dansmith | but quota would be a check on boot, which is less often than list, which could be getting hit really hard in a polling loop | 16:16 |
melwitt | oh, right, it was to save on pulling mappings | 16:16 |
dansmith | anyway, just struck me as less sane to have that be configurable for the quota one | 16:17 |
*** dklyle has quit IRC | 16:17 | |
*** imacdonn has quit IRC | 16:17 | |
*** imacdonn has joined #openstack-nova | 16:18 | |
*** gyee has joined #openstack-nova | 16:18 | |
mriedem | efried: zuul failed | 16:19 |
efried | y | 16:19 |
efried | by which I meant | 16:19 |
efried | f | 16:19 |
*** tssurya has quit IRC | 16:19 | |
*** panda|off is now known as panda|rover | 16:19 | |
efried | ahjeez. More chicken/egg. | 16:20 |
*** hamzy has joined #openstack-nova | 16:22 | |
*** Luzi has quit IRC | 16:22 | |
*** dklyle has joined #openstack-nova | 16:23 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Add a default Virtio RNG device to guests https://review.openstack.org/616603 | 16:27 |
*** mvkr has joined #openstack-nova | 16:28 | |
efried | melwitt, 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 |
openstackgerrit | Eric Fried proposed openstack/nova master: Cap oslo.service to 1.32.0 (temporarily) https://review.openstack.org/616591 | 16:37 |
*** slaweq has quit IRC | 16:40 | |
*** helenafm has quit IRC | 16:48 | |
*** slaweq has joined #openstack-nova | 16:58 | |
*** sahid has quit IRC | 16:59 | |
efried | melwitt, mriedem: btw, we're discussing this in openstack-requirements if you're interested in following along. | 17:02 |
*** ivve has joined #openstack-nova | 17:02 | |
*** panda|rover is now known as panda|rover|off | 17:02 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724 | 17:05 |
openstackgerrit | Merged openstack/nova stable/rocky: Add regression test for bug 1797580 https://review.openstack.org/611938 | 17:07 |
openstack | bug 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 |
openstackgerrit | Eric Fried proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724 | 17:08 |
*** nicolasbock_ has joined #openstack-nova | 17:09 | |
efried | ah, rats, forgot to remove the g-r exclusions... | 17:10 |
*** derekh has quit IRC | 17:14 | |
*** derekh has joined #openstack-nova | 17:15 | |
*** ttsiouts has quit IRC | 17:16 | |
*** k_mouza has quit IRC | 17:24 | |
*** davidsha has quit IRC | 17:25 | |
*** ccamacho has quit IRC | 17:32 | |
*** mvkr has quit IRC | 17:35 | |
*** moshele has joined #openstack-nova | 17:37 | |
*** moshele has quit IRC | 17:48 | |
*** liuyulong has quit IRC | 17:50 | |
*** jpena is now known as jpena|off | 17:51 | |
*** nicolasbock has joined #openstack-nova | 17:56 | |
nicolasbock | Hi, 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 |
nicolasbock | Did I misunderstand the meaning of disk in flavor? | 17:56 |
nicolasbock | I 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-mooney | nicolasbock: glance shows the actual file size | 17:57 |
nicolasbock | But isn't it Nova that's complaining? | 17:57 |
sean-k-mooney | glance has extra metadata to store the minium required disk space to unpack the image | 17:57 |
nicolasbock | So the qcow2 is unpacked? | 17:58 |
sean-k-mooney | nicolasbock: the nova flavor disk attribute is the size the qcow will be unpacked too | 17:58 |
*** k_mouza has joined #openstack-nova | 17:58 | |
sean-k-mooney | nicolasbock: if the qcow file system is bigger nova will complain | 17:58 |
sean-k-mooney | nova is allowed to expand the disk but not srink it | 17:58 |
nicolasbock | Ok, 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 FS | 17:59 |
*** irclogbot_2 has joined #openstack-nova | 17:59 | |
nicolasbock | sean-k-mooney (IRC): you are saying that the disksize inside the image should be <= the disk size of the flavor? | 17:59 |
sean-k-mooney | nicolasbock: can you paste a copy of the error to paste.openstack.org | 17:59 |
sean-k-mooney | nicolasbock: yes | 18:00 |
sean-k-mooney | nicolasbock: nova enforces that the virtual size the disk could grow too is <= the disk in the flavor | 18:00 |
nicolasbock | sean-k-mooney (IRC): http://paste.openstack.org/show/734437/ | 18:01 |
*** ircuser-1 has joined #openstack-nova | 18:01 | |
*** derekh has quit IRC | 18:01 | |
sean-k-mooney | ya so in that case the image seams to be larger then the flavor allows | 18:01 |
sean-k-mooney | nicolasbock: it looks like you tried to boot a 65GiB image on a 50GiB flavor | 18:02 |
nicolasbock | The partition inside the image is 80GiB large though | 18:03 |
nicolasbock | I 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-mooney | strang well the error is clearly showing the glance value it might check both | 18:04 |
nicolasbock | Ok, as long is this is expected behavior I am good. I was just expecting something else :) | 18:05 |
sean-k-mooney | the behavior looks correct to me. out of interest what were you expecting | 18:05 |
sean-k-mooney | was it 80GiB is lager then 50GiB | 18:05 |
nicolasbock | I was expecting the size of the unpacked disk | 18:06 |
efried | mriedem: Does deleting the compute service today delete the provider in placement? | 18:06 |
sean-k-mooney | i 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 it | 18:06 |
sean-k-mooney | nicolasbock: the glance vs flavor check can happen at the api level the actual size can only happen on the compute nodes | 18:07 |
nicolasbock | Ok that's a good point | 18:07 |
nicolasbock | I am going to resize the FS and the disk of that qcow2 | 18:08 |
nicolasbock | sean-k-mooney (IRC): Thanks for the help! | 18:08 |
sean-k-mooney | nicolasbock: if you flavor allowsed 70Gib then it would likely pass the api check and hit the compute node check | 18:08 |
sean-k-mooney | nicolasbock: no worries | 18:09 |
nicolasbock | But it would blow up then, no? | 18:09 |
sean-k-mooney | yes just much later | 18:09 |
nicolasbock | There is another check on the compute node? | 18:09 |
nicolasbock | Ok | 18:09 |
nicolasbock | Thanks! | 18:09 |
*** pcaruana has joined #openstack-nova | 18:10 | |
*** moshele has joined #openstack-nova | 18:15 | |
openstackgerrit | Merged openstack/nova stable/rocky: Don't persist RequestSpec.requested_destination https://review.openstack.org/611939 | 18:22 |
*** adrianc has quit IRC | 18:24 | |
*** k_mouza has quit IRC | 18:25 | |
*** jmlowe has joined #openstack-nova | 18:31 | |
*** moshele has quit IRC | 18:44 | |
*** jmlowe has quit IRC | 18:59 | |
*** dpawlik has quit IRC | 19:00 | |
*** dpawlik has joined #openstack-nova | 19:00 | |
melwitt | mriedem: should we remove use-nested-allocation-candidates from the runway now? I noticed that it seemed to morph into bandwidth-resource-provider | 19:05 |
*** ralonsoh has quit IRC | 19:24 | |
*** jmlowe has joined #openstack-nova | 19:26 | |
*** erlon_ has quit IRC | 19:31 | |
*** mriedem has quit IRC | 19:32 | |
*** k_mouza has joined #openstack-nova | 19:44 | |
*** k_mouza has quit IRC | 19:45 | |
jmlowe | Has anybody had problems with instances getting multiple placement allocations? | 19:45 |
*** k_mouza has joined #openstack-nova | 19:48 | |
*** awaugama has quit IRC | 19:49 | |
*** k_mouza has quit IRC | 19:52 | |
*** dave-mccowan has joined #openstack-nova | 19:56 | |
*** mriedem has joined #openstack-nova | 19:59 | |
mriedem | efried: yes the api attempts to delete the resource provider when a compute service is deleted | 19:59 |
mriedem | melwitt: idk about that use-nested-alloc-candidates bp, or what's left, probably need to ask gibi/tetsuro | 20:00 |
melwitt | mriedem: yeah, I'll ask gibi. thanks | 20:01 |
*** dpawlik has quit IRC | 20:10 | |
mriedem | efried: is https://review.openstack.org/#/c/616371/ expected to be failing? | 20:15 |
mriedem | yay 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 rocky | 20:18 |
mriedem | efried: melwitt: ok i'm +2 on those two nova changes | 20:21 |
melwitt | thank ya | 20:22 |
mriedem | i'm not sure why https://review.openstack.org/#/c/616371/ is failing though | 20:22 |
mriedem | oh i see, | 20:22 |
mriedem | it needs https://review.openstack.org/#/c/615724/ | 20:22 |
mriedem | but that depeneds on 616731 | 20:22 |
mriedem | *371 | 20:23 |
mriedem | so we just need to land the nova changes | 20:23 |
mriedem | well... | 20:23 |
mriedem | but the top nova change can't land w/o that requirements dependency, which is failing b/c the code is still mocking the removed thing | 20:23 |
*** awaugama has joined #openstack-nova | 20:23 | |
*** eharney has quit IRC | 20:24 | |
melwitt | yeah :/ | 20:24 |
melwitt | efried halp | 20:24 |
efried | melwitt: 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 |
melwitt | damn, sorry to hear that | 20:25 |
melwitt | hope you are ok | 20:25 |
efried | I'll know more in an hour. Hopefully no orbital/sinus fractures. | 20:26 |
efried | so, that bizarre requirements chain... | 20:26 |
efried | checking... | 20:26 |
*** dpawlik has joined #openstack-nova | 20:26 | |
efried | Three 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-nova | 20:29 | |
efried | melwitt, mriedem: Heading back to the requirements channel to pick this up... | 20:29 |
*** dpawlik has quit IRC | 20:30 | |
*** dave-mccowan has quit IRC | 20:33 | |
*** jmlowe has quit IRC | 20:38 | |
*** mriedem has quit IRC | 20:41 | |
*** dklyle has quit IRC | 20:45 | |
*** dklyle has joined #openstack-nova | 20:45 | |
*** mriedem has joined #openstack-nova | 20:45 | |
mriedem | dansmith: 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 instances | 20:46 |
*** dklyle has quit IRC | 20:46 | |
openstackgerrit | Merged openstack/nova master: Fix support matrix for VMware UEFI support https://review.openstack.org/615870 | 20:46 |
*** takashin has joined #openstack-nova | 20:47 | |
dansmith | mriedem: yeah, that's what I was saying | 20:47 |
dansmith | mriedem: 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 reason | 20:48 |
mriedem | in 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 anyway | 20:50 |
melwitt | nova meeting in 10 min | 20:50 |
dansmith | mriedem: yep | 20:50 |
mriedem | so we might as well just always do that | 20:50 |
dansmith | mriedem: this is my point | 20:50 |
mriedem | in the case of a cloud that's not multi-cell, do we care about optimizing this? | 20:51 |
melwitt | yeah, I think I got confused by the other instance list check where we could save on pruning cells if there's only one cell | 20:51 |
melwitt | I had thought there was a config option for avoiding that for single cell. and confused it with the quota situation | 20:51 |
dansmith | mriedem: yeah you could filter out cell0 from the cache and if the result is only one avoid the lookup | 20:51 |
mriedem | instances in cell0 still count against your quota though | 20:52 |
mriedem | i'm pretty sure | 20:52 |
melwitt | yeah they do | 20:52 |
melwitt | error instances, for example | 20:52 |
dansmith | yeah, I know, | 20:52 |
melwitt | oh, but you're saying don't count cell0 | 20:52 |
melwitt | yeah | 20:52 |
dansmith | so you could also just say len(cache)==2, but.. checking that there is only one non-cell0 cell seems like a better assertion | 20:52 |
mriedem | wfm | 20:53 |
dansmith | other things would be wrong if those weren't the same, for sure, it just seems more of a positive thing | 20:53 |
dansmith | but that's just a detail, either way that optimization is easy | 20:53 |
melwitt | yeah. 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 |
dansmith | if the cache isn't populated then we have to populate it before we look at the cache | 20:54 |
dansmith | that's how all that code works | 20:54 |
melwitt | ok | 20:54 |
dansmith | so is_single_cell() makes a _load_cells() call first | 20:54 |
*** jmlowe has joined #openstack-nova | 21:05 | |
artom | This is weird, why is conductor failing with 'CantStartEngineError: No sql_connection parameter is established' | 21:05 |
*** brault has quit IRC | 21:06 | |
mriedem | artom: it's trying to hit the API DB | 21:06 |
mriedem | and not configured for it | 21:06 |
mriedem | if you're using devstack | 21:06 |
mriedem | the cell conductor that is | 21:06 |
artom | mriedem, that's exactly it, it's almost creey ;) | 21:06 |
mriedem | e.g. resize fails on compute1, casts back to cell conductor which looks up the instance mapping and kablamo | 21:07 |
mriedem | known issue | 21:07 |
artom | It's a reschedule, the (presumably cell) conductor is trying to get the AZ | 21:07 |
mriedem | yup, i know the bug | 21:07 |
mriedem | sec | 21:07 |
mriedem | artom: linked from https://blueprints.launchpad.net/nova/+spec/fix-reschedule-up-calls | 21:08 |
mriedem | i have a fix for one of those | 21:08 |
artom | Thanks :) | 21:08 |
mriedem | discussed at the ptg | 21:08 |
openstackgerrit | Jack Ding proposed openstack/nova master: Add cache=none option for qemu-img convert https://review.openstack.org/616692 | 21:08 |
mriedem | the other, the one you're hitting, is a bigger change b/c we have to pass the azs down to the cell | 21:08 |
artom | Actually also affects plain boot | 21:08 |
mriedem | on the Selection object | 21:08 |
artom | Hah, so I kinda need this fixed so that I can reproduce *another* bug correctly | 21:09 |
mriedem | artom: b/c of this right? https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L678 | 21:09 |
artom | mriedem, yep | 21:09 |
mriedem | artom: well as a workaround just configure the [api_database]/connection in your nova-cell1.conf | 21:09 |
mriedem | and restart the cell conductor | 21:09 |
artom | Ah, true | 21:09 |
mriedem | i think devstack is the only thing out there that really does this split MQ thing | 21:10 |
mriedem | everyone else just runs with api db configured everywhere | 21:10 |
artom | Doesn't that kind of defeat the scalability point of the API db? | 21:10 |
mriedem | it's not really a scalability thing, | 21:11 |
mriedem | it's an isolation thing | 21:11 |
artom | OK - 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 |
mriedem | https://docs.openstack.org/nova/latest/user/cellsv2-layout.html#operations-requiring-upcalls | 21:11 |
mriedem | b/c dude | 21:11 |
artom | (Not being flippant on purpose or anything, genuinely trying to grok this) | 21:12 |
mriedem | b/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 cells | 21:13 |
artom | Ah, I see. | 21:14 |
mriedem | like cells v1 did | 21:14 |
mriedem | it had to sync everything | 21:14 |
artom | The 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 cells | 21:14 |
artom | It's the *cells DBs* that are the scalability stuff | 21:14 |
mriedem | yeah | 21:15 |
artom | Got it. Thanks :) | 21:15 |
*** mriedem has left #openstack-nova | 21:18 | |
*** mriedem has joined #openstack-nova | 21:18 | |
*** pvc has quit IRC | 21:32 | |
*** nicolasbock_ has quit IRC | 21:32 | |
*** eharney has quit IRC | 21:37 | |
*** ivve has quit IRC | 21:38 | |
openstackgerrit | Doug Hellmann proposed openstack/nova master: Cap oslo.service to 1.32.0 (temporarily) https://review.openstack.org/616591 | 21:39 |
openstackgerrit | Doug Hellmann proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724 | 21:39 |
openstackgerrit | Doug Hellmann proposed openstack/nova master: remove mocks of oslo.service private members https://review.openstack.org/616697 | 21:39 |
*** hamzy has quit IRC | 21:43 | |
openstackgerrit | Doug Hellmann proposed openstack/nova master: remove mocks of oslo.service private members https://review.openstack.org/616697 | 21:51 |
openstackgerrit | Doug Hellmann proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724 | 21:51 |
efried | melwitt, mriedem: Y'all still around to approve the new version of the oslo.service mess? | 21:51 |
melwitt | efried: yeah | 21:52 |
melwitt | the top change says merge conflict tho | 21:53 |
efried | melwitt: First https://review.openstack.org/616697 | 21:53 |
efried | yeah, we're still working on the requirements bit in the middle. | 21:53 |
*** artom has quit IRC | 21:53 | |
melwitt | ok. I didn't think that depends-on could cause the red "merge conflict" message | 21:54 |
efried | yes | 21:54 |
efried | which is a good thing, else we be like, "why is my damn patch stuck?" | 21:55 |
melwitt | TIL | 21:56 |
efried | except 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 IRC | 22:00 | |
mriedem | efried: on https://review.openstack.org/#/c/616697/ the only downside is the slower tests right? | 22:02 |
efried | melwitt: correct, immediately resolved by your patch. | 22:02 |
mriedem | i'm mriedem | 22:02 |
efried | My 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 | +W | 22:03 |
efried | thanks | 22:03 |
efried | oh, and I do have a fractured orbital, though my vision is supposedly fine. | 22:09 |
mriedem | gdi eric, you need a helmet | 22:22 |
mriedem | while looking up helmets, i found this https://pbs.twimg.com/media/DiAdSIWUwAEXkAg.jpg | 22:22 |
mriedem | looks like jerry rice in there | 22:22 |
openstackgerrit | Jack Ding proposed openstack/nova master: Change align_size to 4096 to support 4Kn drives https://review.openstack.org/616701 | 22:26 |
*** mriedem has quit IRC | 22:36 | |
*** mriedem has joined #openstack-nova | 22:36 | |
*** k_mouza has joined #openstack-nova | 22:42 | |
openstackgerrit | Merged openstack/nova master: remove mocks of oslo.service private members https://review.openstack.org/616697 | 22:44 |
*** mriedem has quit IRC | 22:46 | |
*** eharney has joined #openstack-nova | 22:47 | |
*** k_mouza has quit IRC | 22:47 | |
*** manjeets has joined #openstack-nova | 22:54 | |
*** k_mouza has joined #openstack-nova | 22:57 | |
*** awaugama has quit IRC | 23:02 | |
*** k_mouza has quit IRC | 23:02 | |
*** k_mouza has joined #openstack-nova | 23:07 | |
*** k_mouza has quit IRC | 23:08 | |
*** k_mouza has joined #openstack-nova | 23:08 | |
*** artom has joined #openstack-nova | 23:14 | |
*** slaweq has quit IRC | 23:15 | |
*** med_ has joined #openstack-nova | 23:26 | |
openstackgerrit | Eric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache https://review.openstack.org/615646 | 23:32 |
openstackgerrit | Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677 | 23:32 |
openstackgerrit | Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695 | 23:32 |
openstackgerrit | Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705 | 23:32 |
openstackgerrit | Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033 | 23:32 |
openstackgerrit | Eric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache https://review.openstack.org/615646 | 23:33 |
openstackgerrit | Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677 | 23:33 |
openstackgerrit | Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695 | 23:33 |
openstackgerrit | Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705 | 23:33 |
openstackgerrit | Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033 | 23:33 |
*** dklyle has joined #openstack-nova | 23:44 | |
*** erlon_ has joined #openstack-nova | 23:46 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!