| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-25 | |||
| 16:36:58 | sean-k-mooney | libvirt/qemu should be validteing there input and should reject it if invalide | |
| 16:37:35 | sean-k-mooney | you could optimes by checking in the driver but not sure that needs to be in the spec | |
| 16:37:49 | cfriesen | makes sense | |
| 16:39:20 | sean-k-mooney | stephenfin: if you have a config or provide a way to bypass the check im ok with it | |
| 16:40:17 | sean-k-mooney | stephenfin: if i cant turn it off you will break people and im not really ok with saying vpp ovs-dpdk, vrouter and snabb swtich deployments can nolonger live migrate | |
| 16:58:49 | kmalloc | cdent: out of curiosity, does Nova really support passing config data via paste-ini or is it mostly assumed you're passing through config via .conf files [even to middleware]? | |
| 16:59:11 | kmalloc | cdent: looking at how deep i need to support config passing into the new loader when allowing folks to drop paste. | |
| 16:59:28 | kmalloc | if you know. | |
| 16:59:43 | cdent | it's not so much the passing of config data (although that may be in there) but the idea that paste.ini can be edited to add other middlewares | |
| 17:00:06 | kmalloc | right. that is fine. | |
| 17:00:09 | kmalloc | i'm planning on supporting that | |
| 17:00:23 | cdent | and so it falls under the domain of stuff that we need to carry on dealing with incase people have their own changes | |
| 17:00:38 | kmalloc | the other question i have is: if i only allow middleware to be injected into a specific place in the pipeline, where would that be? | |
| 17:00:46 | kmalloc | or do we need it to really be very dynamic | |
| 17:00:47 | kmalloc | ? | |
| 17:00:56 | cdent | sean dague had more to say on the topic way back in the day, and probably new more about what was happening in the wild | |
| 17:01:23 | kmalloc | right. i can flip the coin either way, obviously i wont allow injecting things *after* nova's application | |
| 17:01:38 | cdent | I'm unable to answer that question with about "where" with regard to nova | |
| 17:01:56 | cdent | some of the nova-api oriented folks might be able to (gmann, alex_xu ) | |
| 17:02:04 | kmalloc | thnx :) | |
| 17:02:31 | kmalloc | i figure nova is the ideal target to aim for here, since much follows in it's lead | |
| 17:03:20 | cdent | yup | |
| 17:04:05 | kmalloc | keystone was the outlier but we drew the line and said "nope, no unwhitelisted middleware in our app space" but mostly for security/maintenance/behavior reasons | |
| 17:04:39 | kmalloc | it's super easy to add outer middleware to anything, but i don't want to block people who legitimately need things somewhere in the middle | |
| 17:04:47 | kmalloc | say post keystonemiddleware. | |
| 17:07:32 | melwitt | they're APAC, so you might consider the dev ML with [nova][api] | |
| 17:15:00 | kmalloc | melwitt: ah thnx. yeah Pacific has poor overlap with <insert most currently relevant timezones for OpenStack here> | |
| 17:15:18 | kmalloc | melwitt: yeah i'll get an email spun up as well. | |
| 17:16:01 | melwitt | sounds cool | |
| 17:32:04 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Integration tests using gabbi-tempest https://review.openstack.org/613386 | |
| 17:46:28 | cfriesen | is there a current recommendation on whether to enable the "send_service_user_token" config option? It still defaults to "False", just wondering what the downsides would be...is there an additional keystone round-trip? | |
| 17:51:13 | artom | stephenfin, sorry, missed your ping, I think you settled on config option? I'm cool with that | |
| 17:51:35 | melwitt | cfriesen: found this in the spec, maybe that is the main downside https://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/use-service-tokens.html#performance-impact | |
| 18:26:02 | openstackgerrit | Merged openstack/nova master: api-ref: Add descriptions of error cases https://review.openstack.org/610429 | |
| 18:32:58 | Sundar | melwitt: Do specs need to merged this week (for Stein milestone 1) to be considered for implementation in Stein? | |
| 18:33:23 | melwitt | Sundar: no. our spec freeze is milestone 2, Jan 10 | |
| 18:35:01 | Sundar | melwitt: Thanks. Is there any way to accelerate the coverage for the spec https://review.openstack.org/#/c/603955/ ? | |
| 18:35:26 | Sundar | I understand there are lots of other stuff in the Nova plate | |
| 18:37:13 | melwitt | Sundar: looks like there was some review last week, and you've made some updates | |
| 18:39:13 | melwitt | I can mention it in the nova meeting today that it's been updated and ready for review again | |
| 18:39:36 | Sundar | Great. Thanks! | |
| 18:47:58 | melwitt | sean-k-mooney: could you please reply on this os-vif release review about whether we need a major version bump? https://review.openstack.org/612844 | |
| 19:07:11 | openstack | bug 1764556 in OpenStack Compute (nova) ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Medium,In progress] https://launchpad.net/bugs/1764556 - Assigned to melanie witt (melwitt) | |
| 19:07:11 | openstackgerrit | melanie witt proposed openstack/nova master: Add functional recreate test for bug 1764556 https://review.openstack.org/562041 | |
| 19:07:12 | openstackgerrit | melanie witt proposed openstack/nova master: Don't generate service UUID for deleted services https://review.openstack.org/582408 | |
| 19:07:12 | openstackgerrit | melanie witt proposed openstack/nova master: Add functional regression test for bug 1778305 https://review.openstack.org/582407 | |
| 19:07:13 | openstack | bug 1778305 in OpenStack Compute (nova) "Nova may erronously look up service version of a deleted service, when hostname have been reused" [Undecided,In progress] https://launchpad.net/bugs/1778305 - Assigned to melanie witt (melwitt) | |
| 19:10:56 | melwitt | dangit, added a test change to the wrong commit | |
| 19:21:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Default AZ for instance if cross_az_attach=False and checking from API https://review.openstack.org/469675 | |
| 19:23:47 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement stable/queens: Do not depend on jenkins user in devstack gate https://review.openstack.org/556666 | |
| 19:23:48 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement stable/queens: Migrate legacy-osc-placement-dsvm-functional job in-tree https://review.openstack.org/556635 | |
| 19:30:10 | openstack | bug 1764556 in OpenStack Compute (nova) ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Medium,In progress] https://launchpad.net/bugs/1764556 - Assigned to melanie witt (melwitt) | |
| 19:30:10 | openstackgerrit | melanie witt proposed openstack/nova master: Add functional recreate test for bug 1764556 https://review.openstack.org/562041 | |
| 19:30:11 | openstackgerrit | melanie witt proposed openstack/nova master: Don't generate service UUID for deleted services https://review.openstack.org/582408 | |
| 19:30:11 | openstackgerrit | melanie witt proposed openstack/nova master: Add functional regression test for bug 1778305 https://review.openstack.org/582407 | |
| 19:30:12 | openstack | bug 1778305 in OpenStack Compute (nova) "Nova may erronously look up service version of a deleted service, when hostname have been reused" [Undecided,In progress] https://launchpad.net/bugs/1778305 - Assigned to melanie witt (melwitt) | |
| 19:42:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use RequestSpec.user_id in scheduler.utils.claim_resources https://review.openstack.org/568917 | |
| 19:42:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanups for the scheduler code https://review.openstack.org/529343 | |
| 19:45:04 | openstackgerrit | Matt Riedemann proposed openstack/nova master: RT: replace _instance_in_resize_state with _is_trackable_migration https://review.openstack.org/560467 | |
| 20:08:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Keep attach_mode as top-level field in _translate_attachment_ref https://review.openstack.org/574413 | |
| 20:08:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790 | |
| 20:26:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add post-test hook for testing evacuate https://review.openstack.org/602174 | |
| 20:26:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add volume-backed evacuate test https://review.openstack.org/604397 | |
| 20:26:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Refactor TestEvacuateDeleteServerRestartOriginalCompute https://review.openstack.org/608705 | |
| 20:26:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix InstanceNotFound during _destroy_evacuated_instances https://review.openstack.org/606122 | |
| 20:26:59 | openstack | bug 1794996 in OpenStack Compute (nova) "_destroy_evacuated_instances fails and kills n-cpu startup if lazy-loading flavor on a deleted instance" [High,In progress] https://launchpad.net/bugs/1794996 - Assigned to Matt Riedemann (mriedem) | |
| 20:26:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional regression test for bug 1794996 https://review.openstack.org/606106 | |
| 20:27:00 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Run evacuate tests with local/lvm and shared/rbd storage https://review.openstack.org/604400 | |
| 20:50:10 | melwitt | nova meeting in 10 min | |
| 21:20:22 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM: update_provider_tree() (compatible) https://review.openstack.org/468560 | |
| 21:21:15 | efried | jaypipes: trivial update on rebase -^ | |
| 21:22:53 | aspiers | jaypipes, sean-k-mooney, mriedem, artom: I'm (perhaps naively) hoping that this can be the final patchset of https://review.openstack.org/#/c/609779/ ;-) | |
| 21:23:13 | artom | aspiers, heh, I don't have +2 powers | |
| 21:23:25 | aspiers | artom: that's OK, a +1 is welcome too ;-) | |
| 21:23:26 | artom | I've been seeing all the activity and been meaning to look it over | |
| 21:23:45 | artom | I'll try *real hard this time* to actually get to it ;) | |
| 21:24:49 | aspiers | Thanks :) No rush. Just wanted to point out my hope that I've addressed enough of the feedback that it should be stabilising now and the flurry of activity should slow right down | |
| 21:46:25 | efried | mriedem: https://blueprints.launchpad.net/nova/+spec/run-meta-api-per-cell lgtm, except I would like to see the actual config option section/name in that text. | |
| 21:46:39 | mriedem | i don't have it yet... | |
| 21:46:41 | efried | since it's taking the place of a spec, which would surely have that. | |
| 21:47:02 | mriedem | CONF.api.local_meta_per_cell? | |
| 21:51:38 | openstackgerrit | Merged openstack/nova master: Make CellDatabases fixture reentrant https://review.openstack.org/611665 | |
| 21:51:54 | efried | metadata_host and metadata_port are in the [DEFAULT] section. (Not arguing anything yet, just collating data) | |
| 21:52:38 | efried | other metadata_* options too ^ | |
| 21:53:11 | efried | but metadata_cache_expiration is in [api] | |
| 21:55:53 | efried | mriedem: [api] section works; [cells] is another possibility. | |
| 21:56:36 | efried | mriedem: I would spell out 'metadata' (local_metadata_per_cell). | |
| 21:56:59 | efried | and that's about all the opinion I've got on the matter. I defer to someone who actually understands the feature. | |
| 21:57:26 | melwitt | fwiw, I agree to use 'metadata' instead of just meta | |
| 21:58:11 | melwitt | I notice we have other cells related options in [api] so I think that's fine | |
| 21:58:26 | mriedem | efried: we don't use [cells] anymore since that's cells v1 | |
| 21:58:32 | efried | okey | |
| 21:58:40 | mriedem | https://docs.openstack.org/nova/latest/configuration/config.html#cells | |
| 21:58:42 | efried | The [api] section says it's for the nova API | |
| 21:58:44 | mriedem | DEPRECATED: Cells options allow you to use cells v1 functionality in an OpenStack deployment. | |
| 21:58:53 | mriedem | metadata api can be run from the nova api service | |
| 21:58:54 | efried | but that may implicitly incorporate the metadata API. | |
| 21:58:59 | efried | Yeah, that. | |
| 21:59:40 | melwitt | yeah, the all-in-one metadata API does from nova api service | |
| 22:00:15 | efried | so an aside, should we be moving [DEFAULT]metadata_* into [api]? | |