| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-18 | |||
| 16:55:03 | efried | mriedem: I reckon we're deprecating that option soon | |
| 16:55:34 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove put_allocations() from report client https://review.openstack.org/644292 | |
| 16:55:34 | efried | I think we maybe wanted to give CERN a little while to play first | |
| 16:56:35 | mdbooth | Eurgh: http://logs.openstack.org/52/626952/5/check/openstack-tox-py37/47f5826/job-output.txt.gz | |
| 16:56:57 | mdbooth | eventlet monkey patching for command line tools which import nova? | |
| 16:57:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add docs for compute capabilities as traits https://review.openstack.org/644293 | |
| 16:57:12 | mriedem | efried: oh.....well i guess i need to update this then ^ but that wasn't mentioned in the reno | |
| 16:57:44 | mdbooth | It really would be better to monkey patch at specific entry points :/ | |
| 16:59:12 | efried | mriedem: hm, the default there is five minutes, was there any science behind "60 seconds" or was that anecdotal? | |
| 17:01:07 | mriedem | anecdotal | |
| 17:01:11 | mriedem | i'm trying again now | |
| 17:01:13 | mriedem | with resource_provider_association_refresh=0 | |
| 17:03:09 | mdbooth | What's api-paste.ini used for, and what are the implications of making changes there? | |
| 17:04:58 | cdent | mdbooth: it configures middleware used by the api | |
| 17:05:21 | cdent | the difficulty with make changes there is that it is in that hybrid realm of config file and also stuff that's required | |
| 17:05:38 | mdbooth | cdent: So changes to the one we have in tree would have deployment implications? | |
| 17:05:44 | cdent | yes | |
| 17:05:48 | mdbooth | Eurgh | |
| 17:05:53 | cdent | However, we have changed it in the past | |
| 17:05:56 | cdent | so it's not impossible | |
| 17:06:23 | mdbooth | I basically wanted to airlift it into a module which isn't called __init__.py | |
| 17:06:24 | cdent | there's been talk of trying to get rid of it for a long time (similar to policy-defaults-in-code) | |
| 17:06:37 | mriedem | rpms should mark that file as config so updates shouldn't overwrite it | |
| 17:06:49 | mdbooth | However, that would change a bunch of object paths, and therefore require changes in api-paste.ini | |
| 17:07:18 | mriedem | api-paste.ini at this point is mostly just a relic | |
| 17:07:20 | cdent | mdbooth: yeah, this is one of the places I fell down when doing my own roam through __init__ files | |
| 17:07:29 | mdbooth | Or... I could just move monkey patching to the top of it and leave it there... | |
| 17:07:31 | mriedem | when nova and openstack in general were more open to extension points | |
| 17:09:15 | mdbooth | My latest version of the monkey patching change moves monkey patching into nova/__init__.py, and also adds a couple of assertions which will explode if you've imported libraries with known problems before monkey patching | |
| 17:09:27 | mdbooth | Which sounded great, until I realised that command line tools also import nova | |
| 17:09:41 | mdbooth | testing, ftw | |
| 17:10:15 | mdbooth | So now I'm looking again at monkey patching only for certain entry points | |
| 17:10:38 | mdbooth | But we can't monkey patch lower than nova/api/openstack/... because __init__.py imports the world | |
| 17:10:59 | cdent | ayup | |
| 17:11:36 | mdbooth | I was hoping to move that out of the way, but that looks depressingly involved | |
| 17:21:42 | mriedem | efried: hmm, what am i missing with trying to hup n-cpu in devstack: sudo systemctl kill -s HUP devstack@n-cpu.service | |
| 17:21:46 | mriedem | doesn't seem to be doing anything | |
| 17:22:39 | efried | mriedem: have you tried just finding the pid? | |
| 17:26:27 | mriedem | hmm, i must have killed it somehow | |
| 17:26:27 | mriedem | Active: inactive (dead) since Mon 2019-03-18 17:09:49 UTC; 16min ago | |
| 17:31:15 | mriedem | got it | |
| 17:31:15 | mriedem | INFO nova.compute.manager [None req-65477139-47bd-4419-b5b3-d729fdadb369 None None] Reloading compute RPC API | |
| 17:32:15 | efried | mriedem: how did you do it? | |
| 17:32:26 | mriedem | kill -HUP <pid> | |
| 17:34:55 | efried | mriedem: based on the systemctl man page, it's possible you would have needed to say -s SIGHUP (not just HUP) and specify a --kill-who=main (or something?) to target the actual n-cpu process rather than its control process. | |
| 17:35:23 | efried | so yeah, kill -HUP <pid> sounds easier to me :P | |
| 17:35:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add docs for compute capabilities as traits https://review.openstack.org/644293 | |
| 17:40:08 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix SSL infinite recursion https://review.openstack.org/626952 | |
| 19:06:46 | melwitt | mriedem: new bug I tagged for stein-rc-potential https://bugs.launchpad.net/nova/+bug/1820588 | |
| 19:06:47 | openstack | Launchpad bug 1820588 in OpenStack Compute (nova) "Bandwidth resource is leaked if a bound port is deleted in neutron" [Low,Confirmed] | |
| 19:22:26 | aspiers | mriedem, kashyap: Sorry, had to take a few days break due to RSI issues. (Cautiously) back now and trying to catch up | |
| 19:36:30 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP/PoC: Use openstacksdk for placement https://review.openstack.org/643664 | |
| 19:36:31 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP/PoC: Use SDK instead of ironicclient for node.get https://review.openstack.org/642899 | |
| 19:53:48 | openstackgerrit | Eric Fried proposed openstack/nova master: qemu: Make disk image conversion dramatically faster https://review.openstack.org/640781 | |
| 19:54:47 | efried | kashyap: I added a colon to make that go ^ | |
| 19:54:54 | efried | you may (or may not) wish to rebase its successors. | |
| 20:12:37 | efried | mdbooth: I've only superficially been following along, but when you rip out eventlet monkey-patching, some deepcopyZ will start to fail on lock artifacts. | |
| 20:13:30 | efried | mdbooth: Not sure the best solution for this; the naïf one is to implement __deepcopy__ on the thing holding the lock to create a new one (which is what the monkey-patched version did). | |
| 20:28:34 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Add a prelude release note for the 19.0.0 Stein GA https://review.openstack.org/644412 | |
| 20:38:51 | openstackgerrit | Merged openstack/nova master: Remove obsolete policy configuration details from docs https://review.openstack.org/643949 | |
| 20:41:45 | mnaser | hi everyone -- is there anyone doing any work on policy/scopes at the moment? | |
| 20:44:39 | melwitt | mnaser: not that I know of, but I was thinking of trying to get some stuff going next cycle for scope types. I proposed a forum session which might get combined with a keystone proposed session of similar topic | |
| 20:44:58 | mnaser | melwitt: cool, ill tag along for that perhaps | |
| 20:58:47 | openstackgerrit | Merged openstack/python-novaclient master: Update master for stable/stein https://review.openstack.org/644183 | |
| 21:12:11 | openstackgerrit | Lance Bragstad proposed openstack/nova master: Clarify policy shortcomings in policy enforcement doc https://review.openstack.org/643960 | |
| 21:12:12 | openstackgerrit | Lance Bragstad proposed openstack/nova master: Remove additional policy configuration details from policy doc https://review.openstack.org/644423 | |
| 23:13:00 | mriedem | lbragstad: typo here? https://review.openstack.org/#/c/644423/ | |
| 23:49:36 | openstackgerrit | Lance Bragstad proposed openstack/nova master: Remove additional policy configuration details from policy doc https://review.openstack.org/644423 | |
| 23:50:02 | openstackgerrit | Lance Bragstad proposed openstack/nova master: Clarify policy shortcomings in policy enforcement doc https://review.openstack.org/643960 | |
| #openstack-nova - 2019-03-19 | |||
| 00:14:48 | openstackgerrit | Merged openstack/nova master: qemu: Make disk image conversion dramatically faster https://review.openstack.org/640781 | |
| 00:30:28 | openstackgerrit | Adam Spiers proposed openstack/nova master: Remove unnecessary default provider_tree when getting traits https://review.openstack.org/644452 | |
| 01:54:00 | openstackgerrit | Rui Zang proposed openstack/nova-specs master: support virtual persistent memory https://review.openstack.org/601596 | |
| 04:41:25 | openstackgerrit | chenker proposed openstack/python-novaclient master: Optimize limit option docs string description for novaclient https://review.openstack.org/643035 | |
| 04:46:34 | openstackgerrit | chenker proposed openstack/python-novaclient master: Optimize limit option docs string description for novaclient https://review.openstack.org/643035 | |
| 05:11:20 | openstackgerrit | Rui Zang proposed openstack/nova-specs master: Virtual persistent memory libvirt driver implementation https://review.openstack.org/622893 | |
| 06:35:47 | Kevin_Zheng | ping gmann | |
| 06:53:15 | openstackgerrit | melanie witt proposed openstack/nova master: Add a prelude release note for the 19.0.0 Stein GA https://review.openstack.org/644412 | |
| 07:56:26 | openstackgerrit | Merged openstack/nova master: Remove stale aggregates notes from scheduler evolution doc https://review.openstack.org/643613 | |
| 08:26:47 | kashyap | efried: Thank you for putting it through. | |
| 08:39:54 | openstackgerrit | Yongli He proposed openstack/nova master: Clean up orphan instances https://review.openstack.org/627765 | |
| 08:56:24 | openstackgerrit | Seyeong Kim proposed openstack/nova stable/rocky: Extract compute API _create_image to compute.utils https://review.openstack.org/643852 | |
| 08:56:24 | openstackgerrit | Seyeong Kim proposed openstack/nova stable/rocky: Share snapshot image membership with instance owner https://review.openstack.org/643853 | |
| 09:32:25 | openstackgerrit | zhangyangyang proposed openstack/nova master: Drop py35 jobs and add py36 jobs https://review.openstack.org/643871 | |
| 10:05:26 | gibi | melwitt: I don't think https://bugs.launchpad.net/nova/+bug/1820588 is rc potential, I left some reasoning in the bug | |
| 10:05:27 | openstack | Launchpad bug 1820588 in OpenStack Compute (nova) "Bandwidth resource is leaked if a bound port is deleted in neutron" [Low,Confirmed] | |
| 10:06:16 | sean-k-mooney | gibi: that will be fixed by handeling the networks-status event in the periodic jobs right? | |
| 10:06:37 | sean-k-mooney | gibi: have you been working on a fix for that already or is it still todo? | |
| 10:07:08 | sean-k-mooney | i vaguely rememebr talking about it a week or two ago that we would have to do something to handel this edgecase | |
| 10:07:55 | kashyap | aspiers: No problem; I myself will be off next week | |
| 10:09:10 | sean-k-mooney | gibi: even if its not rc potential which i kind of agree with i think this should be backportable shortly after GA | |
| 10:11:10 | gibi | sean-k-mooney: when the bound port is deleted in neutron nova gets the notification and detaches the port from the server. I don't know how the periodic is involved here | |
| 10:11:49 | gibi | sean-k-mooney: moreover I till not conviced that why the admin cannot detach the port in nova first and only delete it after in neutron | |
| 10:12:40 | gibi | detach + delete works properly with bandwidth | |
| 10:19:48 | sean-k-mooney | gibi: when a bound port is delete neutron send a notification to nova which the periodic jobs will recive | |
| 10:20:19 | sean-k-mooney | gibi: this also does not require admin acess | |
| 10:20:57 | sean-k-mooney | a tenant can simple do a openstack port delete to cause this issue | |
| 10:21:42 | aspiers | kashyap: you wanted me to split out the addition of getDomainCapabilities() from the <sev> parsing, right? | |
| 10:21:54 | kashyap | aspiers: Yes, please | |
| 10:22:05 | aspiers | kashyap: the problem is, that code doesn't parse anything else in the output | |
| 10:22:07 | sean-k-mooney | gibi: but you are right that this should be done by a nova interface detach in general | |