Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-10-25
16:34:18 stephenfin Also, mriedem noted about doing it higher but it seemed no simpler. This doesn't affect anything but libvirt so that still seems like the place to do this.
16:34:21 sean-k-mooney cfriesen: prably use two traits one for the version and a second for the type
16:34:27 cfriesen sean-k-mooney: basically, with TPM 2.0 there are two different ways of talking to the TPM.
16:34:33 cfriesen and qemu supports both
16:34:44 sean-k-mooney ok
16:34:48 cfriesen so I was thinking it's not really a trait
16:34:56 cfriesen since it's not schedulable, at least for qemu
16:35:02 sean-k-mooney well then it s a flavor extra spec
16:35:09 sean-k-mooney and or image meta
16:36:00 sean-k-mooney so retuired trait for tpm_2.0 and hw:tpm_connection_mode=CRB
16:36:04 cfriesen if they specify CRB with TPM 1.2, should I ignore the CRB request or error out?
16:36:27 sean-k-mooney cfriesen: proabley error but you could leave that up to qemu
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.

Earlier   Later