Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-26
15:18:57 mriedem so you need to get paged
15:18:59 dansmith right, only an exception will stop it from what I see
15:19:08 bauzas okay, that's my question
15:19:15 bauzas because it's threaded ?
15:19:22 dansmith bauzas: so just raise an exception, which will be ugly, and we could maybe clean that up in service.py after the fact to make it a little less ugly on the eyes
15:19:23 bauzas I mean gt
15:19:30 dansmith nothing related to threading
15:19:33 bauzas anyway
15:19:37 bauzas let's do the exception
15:19:46 bauzas no time for introspecting the caller :)
15:20:31 mriedem i guess https://review.openstack.org/#/c/425524/ didn't work huh
15:21:00 dansmith I didn't look, this conversation was going on when I showed up
15:21:01 dansmith however,
15:21:05 openstackgerrit Gábor Antal proposed openstack/nova: Fixed indentation in virt/libvirt/driver.py https://review.openstack.org/425748
15:21:08 pkoniszewski_ hey, can i have -W/+W there https://review.openstack.org/#/c/407518/ ? jenkins does not want to start gate job for some reason
15:21:10 dansmith the moreI explain it the more I think it's probably the better option anyway
15:21:50 mriedem bauzas: you can remove the dependency on https://review.openstack.org/#/c/425524/ when you respin your nova scheduler change,
15:21:53 mriedem it didn't work anyway
15:22:00 cdent mriedem: I checked into the grenade job and as far as I could tell, the subnode wasn't getting config'd, still
15:22:08 mriedem cdent: yeah it wasn't
15:22:13 mriedem that features yaml is all magic to me
15:22:51 bauzas mriedem: ack
15:23:16 pkoniszewski_ sfinucan: hi, are you around?
15:23:47 sfinucan pkoniszewski_: aye
15:23:57 pkoniszewski_ sfinucan: can i have -W/+W there https://review.openstack.org/#/c/407518/ ? jenkins does not want to start gate job for some reason
15:24:20 openstackgerrit Stephen Finucane proposed openstack/nova: Change exponential function to linear https://review.openstack.org/407518
15:24:35 sfinucan pkoniszewski_: done
15:24:40 pkoniszewski_ sfinucan: thanks!
15:25:32 openstackgerrit Artom Lifshitz proposed openstack/nova: Fix tag attribute disappearing in 2.33 and 2.37 https://review.openstack.org/424759
15:27:00 dansmith I think we probably need to touch the oslo service stuff to make a really graceful exit from service startup
15:27:49 dansmith actually, no
15:28:05 mriedem i don't think so,
15:28:13 mriedem we had a bug where the pci whitelist parser was blowing up,
15:28:14 openstackgerrit Claudiu Belu proposed openstack/nova: Hyper-V PCI Passthrough https://review.openstack.org/420614
15:28:17 mriedem and that just killed the process
15:28:28 mriedem ^ came from the RT on init_host
15:28:35 dansmith mriedem: no I mean to avoid a traceback
15:28:43 mriedem oh
15:29:25 openstackgerrit Dan Smith proposed openstack/nova: WIP: Allow graceful service exits from compute https://review.openstack.org/425751
15:29:43 dansmith mriedem: something like this ^ to clean up what bauzas is adding now .. but later
15:30:03 bauzas dansmith: I'm just writing UTs and then boooom
15:30:13 sfinucan claudiub|2: OK, I'd still like Moshe to jump in but I'm personally happy with that as is
15:30:43 claudiub|2 sfinucan: cool, thanks! :D
15:30:46 bauzas dansmith: you okay if I'm just shamely copying your exception ?
15:30:55 claudiub|2 sfinucan: will try to reach out to him as well, as soon as possible.
15:31:02 bauzas dansmith: mine is too much explicit (PlacementNotConfigured)
15:31:07 sfinucan Side note: we *really* need to find a way to standardized flavor extra specs like we do ImageMetaData
15:31:19 bauzas dansmith: but I was basically doing the same, including the try/except clause in the cmd
15:33:40 mriedem i don't think create() is the correct method to handle,
15:33:46 mriedem that just creates the service object,
15:33:48 dansmith bauzas: just use something else and we can merge my graceful bit later and I will clean it up
15:33:49 mriedem start() is the thing that blows up
15:34:03 bauzas dansmith: okay
15:34:08 dansmith mriedem: yep, you're right
15:34:16 dansmith mriedem: but I think I want to catch around both
15:34:25 dansmith mriedem: in case we had to abort in low-level init or something
15:34:27 mriedem i just don't know if serve() or wait() calls start()
15:34:31 andreykurilin johnthetubaguy: I answered all your comments and all both patches are vaiting for your review:)
15:34:51 dansmith serve calls launch
15:34:54 dansmith which calls start I think
15:35:04 dansmith this is why I made it fail so we can see :)
15:35:17 openstackgerrit Dan Smith proposed openstack/nova: WIP: Allow graceful service exits from compute https://review.openstack.org/425751
15:41:45 cdent mriedem: if we stick to one full circle, I think we should consider that a win
15:42:04 mriedem heh
15:42:12 openstackgerrit melanie witt proposed openstack/nova: Count resources to check quota for cells https://review.openstack.org/416521
15:42:25 mriedem sfinucan: if you're looking for things, https://review.openstack.org/#/c/404990/ and the one after it have +2s
15:42:37 mriedem pretty simple
15:44:41 sfinucan mriedem: I'll give it a shot
15:45:55 mriedem none of that is sqla stuff
15:46:00 mriedem even though it's a jaypipes patch
15:46:00 raj_singh mriedem: If you have time, I got +2's on last two patches of service token bp.
15:47:03 mfeoktistov sfinucan: mriedem: mdbooth: sorry for disturbing you, I updated https://review.openstack.org/#/c/312488/ (rootwrap instead of privsep and small cleanup). Virtuozzo Storage CI fails because we have problem with cinder now, it's not releated to this review.
15:47:39 melwitt johnthetubaguy, mriedem: I took the WIP off the counting quota patch last night after adding more test coverage
15:48:08 bauzas interesting, how can I test that I don't have a conf section yet ?
15:48:28 andreykurilin mriedem bauzas johnthetubaguy: hi all! Do we want to put latest 7.1.0 (which is not released) to u-c ? If yes, there is only ~ 4 hours away now to make a release and propose c change to g-r repo
15:48:30 bauzas I could mock the CONF object tho but I'd prefer to mimic
15:48:37 dansmith bauzas: don't test for the section, test for the keys
15:48:53 bauzas dansmith: same question, how could I unset that ?
15:48:56 mriedem mfeoktistov: +2
15:49:50 mriedem melwitt: how comfortable are you with the counting quotas stuff? is it too risky at this point? and if counting quotas doesn't get in, should we still block the patches below it to move those other quotas tables to the api db (except reservations and usages)?
15:50:16 mriedem andreykurilin: are the patches all in for novaclient 7.1.0 yet?
15:50:38 mdbooth mfeoktistov: Did you change anything else?
15:50:38 mriedem bauzas: by default the tests won't have anything set in the CONF.placement values
15:50:42 mriedem unless we have defaults
15:50:45 openstackgerrit Chris Dent proposed openstack/nova: Add more debug logging on RP inventory update failures https://review.openstack.org/414230
15:50:53 mriedem mfeoktistov: he moved some small things around it looks like
15:51:02 mriedem mdbooth: ^
15:51:09 mdbooth IIRC I was +1 on this before, so if they only thing you've change is privsep I'm still +1.
15:51:11 cdent mriedem: that's ^ the long promised test additions to that logging thing you did a while back. with some adjustments
15:51:17 andreykurilin mriedem: only one(+one not critical in the chain) left - https://review.openstack.org/#/c/424256/ . It has one +2 and answered all comments
15:51:58 bauzas mriedem: dansmith: the only problem I see is that we expect keystone config options
15:52:05 mriedem andreykurilin: ok, looks like johnthetubaguy has been reviewing https://review.openstack.org/#/c/424256/ so i'll leave the +W to him
15:52:10 bauzas so I need to verify them
15:52:18 melwitt mriedem: I'm pretty confident in the counting stuff. I think it's no worse than what's already there. we would trade known issues and out-of-sync problems for something with a lot smaller footprint, but obviously has some element of the unknown compared to something we've had for years
15:53:19 mfeoktistov mdbooth: no, just rootwrap instead of privsep, move two lines of code from libvirt_driver to libvirt_utils and fix tests for it
15:53:23 mriedem bauzas: i think you just need to check for CONF.placement.password
15:53:26 mriedem that *has* to be set
15:53:38 bauzas mriedem: okay, I can just do that
15:53:42 melwitt mriedem: as for the bottom patches, if we assume a single cell, I think it still works, it's just reservations and usages would be in the main db whereas the limits and classes would be in the api db.
15:53:54 mriedem http://logs.openstack.org/72/404472/20/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/8aafdf2/logs/etc/nova/nova.conf.txt.gz

Earlier   Later