Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-10
14:21:19 sean-k-mooney leave a comment in the next version based on your testing
14:21:33 yonglihe Sure.
14:21:44 sean-k-mooney e.g. why you are doing whatever test you come up with
14:23:11 sean-k-mooney im not sure but i need to go work on somehting before a meeting soon
14:23:29 yonglihe I gonna to verify if that code working on real stack. thanks. I got to run also.
14:23:30 sean-k-mooney i think the task state + checking for soft delete would be enough
14:23:56 yonglihe Ok , i gonna checking that,
14:24:14 yonglihe Have a good day.
14:32:19 sean-k-mooney johnthetubaguy: by the way i -1 the unifed limits spec but if just declare the discoverablity as out of scope to be resolved in a futrue version of the sdk or in a follow up spec then im fine to swap to +1 on that spec
14:32:48 sean-k-mooney once the nits inline are adressed that is
14:33:09 johnthetubaguy sean-k-mooney: cool, I will respin
14:34:34 sean-k-mooney is it safe to assume that the keysone limits api wont have limits for compute resouces if we have not enabled it for nova
14:35:23 sean-k-mooney if so the sdk could check that once an cache it or something like that to determin if it should use keystone limits or nova
14:36:54 sean-k-mooney so the actully check would be. 1.) is keystone version new enough to support limtis 2.) are instance limits reported in keysonte limits api, if yes to both use keystone if the either are false use nova
14:37:56 sean-k-mooney johnthetubaguy: would somthing like ^ work
14:40:00 johnthetubaguy sean-k-mooney: Its not a smooth transition that way... nova might still be ignoring keystone, but it might be good enough (assuming default keystone policy means you can list the limits of your project)
14:40:20 johnthetubaguy it gets worse with hierarchy mind, as everything seems to
14:41:18 sean-k-mooney well we dont need to encode that in the spec i was just wondering if there was a huristic the sdk could resonably use
14:42:27 sean-k-mooney it could be as simple as stick it in a config option for now
14:43:45 sean-k-mooney the clouds.yaml has profiles for clouds that encode there knonw behavior for non discoverable things
14:43:56 sean-k-mooney so that is always an option for limits
14:44:41 sean-k-mooney at least i think nodepool gets that stuff from clouds.yaml for the sdk
14:52:06 openstackgerrit Dakshina Ilangovan proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.opendev.org/612497
14:59:56 spatel sean-k-mooney: did you ever see this error http://paste.openstack.org/show/754267/
15:07:05 stephenfin johnthetubaguy: Seeing as you've been reviewing docs, could you have a look at https://review.opendev.org/#/c/667164/ ?
15:07:29 johnthetubaguy stephenfin: sure
15:15:37 openstackgerrit John Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec https://review.opendev.org/602201
15:15:48 openstackgerrit Stephen Finucane proposed openstack/nova master: ec2: Remove unused functions from 'ec2utils' https://review.opendev.org/662501
15:15:49 openstackgerrit Stephen Finucane proposed openstack/nova master: ec2: Pre-move cleanup of utils https://review.opendev.org/662504
15:15:49 openstackgerrit Stephen Finucane proposed openstack/nova master: ec2: Remove ec2.CloudController https://review.opendev.org/662503
15:15:49 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove unused ec2 objects https://review.opendev.org/662502
15:15:50 openstackgerrit Stephen Finucane proposed openstack/nova master: ec2: Move ec2utils functions to their callers https://review.opendev.org/662505
15:15:51 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Remove 'Debug' middleware https://review.opendev.org/662506
15:19:07 openstackgerrit Stephen Finucane proposed openstack/nova master: vif: Resolve a TODO and update another https://review.opendev.org/666631
15:19:07 openstackgerrit Stephen Finucane proposed openstack/nova master: vif: Stop using getattr for VIF lookups https://review.opendev.org/666630
15:19:08 openstackgerrit Stephen Finucane proposed openstack/nova master: vif: Remove dead minimum libvirt checks https://review.opendev.org/666814
15:20:35 Nick_A We are trying to offload the _base nova image cache using cephfs. our idea was to do a read-only mirror of glance's image pool and mount that to _base on compute. Would nova have problems with that setup?
15:21:04 openstackgerrit Merged openstack/nova master: Perf: Use dicts for ProviderTree roots https://review.opendev.org/637225
15:21:35 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Split '_get_host_numa_topology' function https://review.opendev.org/631237
15:21:35 openstackgerrit Stephen Finucane proposed openstack/nova master: Handle libvirt reporting incorrect 4k page quantities https://review.opendev.org/631038
15:21:36 openstackgerrit Stephen Finucane proposed openstack/nova master: Make overcommit check for pinned instance pagesize aware https://review.opendev.org/631053
15:22:30 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Support filtering of hosts by forbidden aggregates https://review.opendev.org/667952
15:25:54 openstackgerrit Stephen Finucane proposed openstack/nova stable/stein: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670096
15:26:03 openstackgerrit Stephen Finucane proposed openstack/nova stable/rocky: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670097
15:28:05 openstackgerrit Stephen Finucane proposed openstack/nova stable/stein: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670096
15:28:37 openstackgerrit Stephen Finucane proposed openstack/nova stable/rocky: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670097
15:29:21 openstackgerrit Stephen Finucane proposed openstack/nova stable/queens: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670100
15:35:33 mriedem stephenfin: i see https://review.opendev.org/#/c/667164/2 is already approved and you've started backports, but some issues inline
15:35:45 stephenfin looking
15:36:43 openstackgerrit sean mooney proposed openstack/nova master: WIP: coverage job changes https://review.opendev.org/669912
15:36:59 stephenfin mriedem: Those all look fair. Want me to pull it out of the gate and address those?
15:37:15 mriedem that would be nice yes
15:37:20 artom There's a thing to re-run just the tests that failed in the last run... right?
15:37:28 mriedem artom: not in the gate
15:37:32 mriedem locally,
15:37:33 artom mriedem, locally
15:37:36 mriedem tox -e <whatever> -- --failing
15:37:41 artom mriedem++
15:38:59 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
15:40:07 stephenfin mriedem: 'openstack quota list --compute' won't show the _default_ quotas though, will it?
15:40:13 stephenfin (RE: https://review.opendev.org/#/c/667164/2/doc/source/admin/quotas.rst@68)
15:41:13 mriedem stephenfin: honestly i don't know how that command works under the covers,
15:41:25 mriedem "default" doesn't really mean anything for the compute API if you're not looking at the default class
15:41:31 mriedem neutron has an api to see 'defaults'
15:41:39 mriedem but i don't think cinder does and nova doesn't besides the class
15:43:49 stephenfin looks like 'quota list' is no good to us
15:43:51 stephenfin quota informations for requested project"
15:43:51 stephenfin "List quotas for all projects with non-default quota values or list detailed
15:44:10 stephenfin I'll add a note that there are non-compute quotas mixed in there
15:48:11 mriedem yeah this feels like a wild goose chase
15:48:19 mriedem sorting out the clis and docs is a mess
15:50:05 openstackgerrit sean mooney proposed openstack/nova master: WIP: coverage job changes https://review.opendev.org/669912
15:59:46 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
15:59:55 stephenfin mriedem: ^ addressed what I could of them
16:00:01 mriedem ack
16:00:42 spatel sean-k-mooney: nevermind after reload rabbitmq cluster it fixed issue.
16:02:55 sean-k-mooney spatel: sorry i was on a call earlier but yes we have seen that before
16:03:43 sean-k-mooney restarting rabbit or the compute agged typicaly fixes it
16:03:51 spatel same here
16:04:32 sean-k-mooney its related to how oslo.messaging handels retrasmits and connection issue but i think they are trying to fix it
16:04:53 sean-k-mooney its not really a nova issue as we dont actuly handel creating the queue/exchanges
16:05:08 sean-k-mooney that is all done use by oslo.messaging
16:14:15 openstackgerrit Sylvain Bauza proposed openstack/nova master: WIP: Add a placement audit command https://review.opendev.org/670112
16:31:23 sean-k-mooney alex_xu: efried thanks for your input on https://review.opendev.org/#/c/662264/. I have abandoned it for train so we can focus on similar uses of placements new feature in train and come back to cache in U
16:32:22 sean-k-mooney if im working on that again in U ill propose a placmeent native version that may pull some of the provider yaml stuff it that makes progress
16:40:34 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
16:40:35 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Scrub available quotas https://review.opendev.org/670125
16:40:35 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
16:42:18 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
16:42:19 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
16:42:19 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Scrub available quotas https://review.opendev.org/670125
16:48:08 openstackgerrit Merged openstack/nova master: db: Add vpmems to instance_extra https://review.opendev.org/662697
16:59:01 Sundar Hello gibi, Could you look at https://review.opendev.org/657464 ? I understand it merited a -1 earlier, but I think we have resolved the naming issues in the Cyborg-Nova spec. Thanks.
16:59:45 openstackgerrit Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/667177
16:59:46 openstackgerrit Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442
16:59:46 openstackgerrit Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
17:06:40 stephenfin mriedem, melwitt: If we removed the injected_files, injected_file_content_bytes and injected_file_path_bytes quotas in API v2.57, does that mean the conf options in '[quotas]' should have a note saying the quotas aren't used anymore or something?
17:06:41 openstackgerrit Merged openstack/nova master: Remove 'nova.virt.libvirt.compat' https://review.opendev.org/669120
17:07:15 stephenfin I'm not grasping whether those quotas actually do anything any more and, if so, what's enforcing it
17:16:29 sean-k-mooney stephenfin: i think we still need them since we still support it for older microversions

Earlier   Later