Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-12
03:50:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: Make dest send NUMAMigrateData to the source https://review.openstack.org/634828
03:50:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Use live_migration_claim() to check dest resources https://review.openstack.org/634606
03:50:39 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: update NUMA-related XML on the source https://review.openstack.org/635229
06:07:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for vol-backed snapshots https://review.openstack.org/615084
06:43:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix resetting non-persistent fields when saving obj https://review.openstack.org/636271
07:12:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix resetting non-persistent fields when saving obj https://review.openstack.org/636271
08:06:52 openstackgerrit Merged openstack/nova master: doc: update the security groups admin doc https://review.openstack.org/634735
08:07:01 openstackgerrit Merged openstack/nova master: doc: link admin/configuration from admin home page https://review.openstack.org/634728
08:42:28 openstackgerrit Merged openstack/nova stable/queens: Fix InstanceNotFound during _destroy_evacuated_instances https://review.openstack.org/623355
09:46:11 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459
09:46:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421
09:46:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
09:46:13 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311
09:46:13 openstackgerrit Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310
09:46:14 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543
10:05:35 lyarwood bauzas: https://review.openstack.org/#/c/628726/ - would you mind hitting this stable backport today if you have a chance?
10:06:11 bauzas lyarwood: yup, actively working on fixing some problems with the reshape change, but sure will look
10:07:17 lyarwood bauzas: ack thanks
10:14:36 gibi lyarwood: hi! If we are at stable backports, could you check https://review.openstack.org/#/c/635921/1 and https://review.openstack.org/#/c/623358/1 ?
10:14:51 lyarwood gibi: ack can do
10:14:56 gibi thanks!
10:20:06 bauzas gibi: around ?
10:20:14 abhishekk hi all, any idea how can I print queries in logs?
10:20:20 bauzas gibi: I'm puzzled with testing your functional tests again my new revision
10:20:38 abhishekk I have tried to set default_log_levels = sqlalchemy=INFO but its not working
10:20:51 gibi bauzas: push your new revision up and I will check the functional test
10:20:56 bauzas ack
10:21:01 bauzas it will fail your change tho
10:21:06 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Reject migration requests when source is down https://review.openstack.org/623489
10:21:33 bauzas gibi: I'll just add a new change for correctly using the right mdev before tho
10:21:36 gibi bauzas: you mean it will fail the functional test? That is OK I will check that
10:21:46 lyarwood sean-k-mooney / stephenfin / mdbooth_ ^ I'd appreciate your thoughts on my bugfix above if you have time today
10:22:33 lyarwood oh and is anyone else having weird issues building releasenotes on f29?
10:23:54 stephenfin and tries building releasenotes to check
10:24:44 lyarwood stephenfin: /src/openstack/nova/releasenotes/source/rocky.rst:3436:Duplicate explicit target name: "rest api version history". - is the error I'm getting locally
10:27:45 stephenfin lyarwood: That doesn't sound like something F29 specific. That would happen if someone had defined the same target in two release notes, which it looks like someone has
10:28:01 stephenfin though it's building just fine for me...
10:28:42 lyarwood stephenfin: kk there must be some cruft in my local repo somewhere
10:29:10 stephenfin 'git clean -xfd -e .tox' <-- I should have this aliased
10:42:21 openstackgerrit Boris Bobrov proposed openstack/nova master: Fix memtune parameters according to libvirt docs https://review.openstack.org/636301
10:43:43 openstackgerrit Boris Bobrov proposed openstack/nova master: Fix memtune parameters according to libvirt docs https://review.openstack.org/636301
12:00:24 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add method to allow reset fields for root bdm in BDM obj https://review.openstack.org/614672
12:10:32 openstackgerrit Boris Bobrov proposed openstack/nova master: Add configs for AMD SEV https://review.openstack.org/636318
12:45:27 openstackgerrit Jim Rollenhagen proposed openstack/nova master: Ironic: bump minimum API version to 1.38 https://review.openstack.org/636326
12:45:46 jroll TheJulia: ^ sanity check me that this is cleanup we want to do, please
13:14:09 mriedem http://logs.openstack.org/67/620667/3/check/tempest-full/8fa0aa0/controller/logs/screen-n-api.txt.gz?level=TRACE#_Feb_12_00_25_30_615423
13:14:09 mriedem tssurya: i think we might have some traces in the api logs that are saying a cell is down when it's really something else
13:14:37 mriedem although it looks like there are some MQ issues in those logs
13:37:39 lyarwood mriedem: https://review.openstack.org/#/c/551302/ - has been rebased and should be ready to review again if you have anytime this week.
13:38:25 mriedem lyarwood: yup in the queue
13:38:45 mriedem did you see the conversation in there about rollback also being busted?
13:40:06 lyarwood mriedem: so I think mdbooth was addressing that in a follow up here https://review.openstack.org/#/c/551349/2
13:40:56 lyarwood mdbooth: ^ I can also rebase / refresh that if you'd like
13:42:11 mdbooth lyarwood mriedem: Yeah, although not having time to address that too is probably the biggest reason I haven't looked at this in ages.
13:42:33 mdbooth Definitely a thing, though.
13:49:23 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543
13:57:30 openstackgerrit Boris Bobrov proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.openstack.org/636334
14:00:55 stephenfin sean-k-mooney: Thoughts? http://paste.openstack.org/show/B9unIL8e2KpeSMBGaINe/
14:02:03 stephenfin sean-k-mooney: I'm annoyed by the fact we have no central definition (for docs) or validation of flavour extra specs. Thinking it might be a good, easy spec for T (for the ones we're not deprecating with the cpu-resources spec
14:05:43 bauzas stephenfin: we have documentation for flavor extra specs
14:05:52 bauzas unless I misunderstand your concern
14:06:27 sean-k-mooney stephenfin: we do it called the glance metadef registry
14:06:31 bauzas https://docs.openstack.org/nova/latest/user/flavors.html
14:06:51 stephenfin bauzas: Yup, but it's not "in code" or verified against the code, unlike e.g. the nova.conf options
14:06:57 bauzas ahah
14:07:04 bauzas well
14:07:08 gibi bauzas: if you have time, can you push this two-patch backport? https://review.openstack.org/#/c/623358/1
14:07:08 bauzas AFAIK, we said meh
14:07:14 sean-k-mooney stephenfin: https://github.com/openstack/glance/blob/master/etc/metadefs/compute-cpu-pinning.json
14:07:21 bauzas gibi: looking
14:07:22 sean-k-mooney that is in code in a way
14:07:24 gibi bauzas: thanks
14:07:51 bauzas gibi: oh that one
14:07:58 bauzas gibi: silverlining, already reviewed it
14:08:53 sean-k-mooney stephenfin: it is intended to be used for validating glance metadata entries and its used but horizon to generate teh ui with the dropdown vlaues and for validation
14:09:22 sean-k-mooney it also has defintions for the flavor extra specs too.
14:09:45 mriedem tssurya: dansmith: i left comments in the down cell servers view builder change, the series is going to have to be rebased
14:10:00 sean-k-mooney stephenfin: what might make sense it to extract it into a seperate repo that glance,nova and cinder can all import and share
14:10:00 stephenfin sean-k-mooney: Aye, I knew about. If that was ever kept up-to-date though, I'm not sure if it still is. Barely any edits in years https://github.com/openstack/glance/tree/master/etc/metadefs
14:10:33 stephenfin I didn't know that it supported flavour extra specs though. That's good to know
14:10:59 stephenfin sean-k-mooney: Aye, I think so. I'll have a look into it when I get a chance. Just been annoying me for some time now
14:11:11 sean-k-mooney stephenfin: ya it was orginally created to keep the falvor extra specs and image metadata in sync
14:11:38 sean-k-mooney if we did extract it i dont see why we could not autogenerate the docs for it too
14:13:36 openstackgerrit Artom Lifshitz proposed openstack/nova master: Remove _legacy_dict methods https://review.openstack.org/636210
14:13:37 openstackgerrit Artom Lifshitz proposed openstack/nova master: New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
14:13:37 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add migration param to check_can_live_migrate_destination https://review.openstack.org/634605
14:13:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Use live_migration_claim() to check dest resources https://review.openstack.org/634606
14:13:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Introduce live_migration_claim() https://review.openstack.org/635669
14:13:39 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: update NUMA-related XML on the source https://review.openstack.org/635229
14:13:39 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: Make dest send NUMAMigrateData to the source https://review.openstack.org/634828
14:13:42 kashyap Gosh, this serial / char devices code is making me go slowly mental...
14:13:44 sean-k-mooney stephenfin: can we put it as a cross project topic with the glance and maybe cinder folks for the ptg
14:14:16 stephenfin sean-k-mooney: Good idea, that
14:15:32 sean-k-mooney stephenfin: it looks like it also cover nova host aggrage metadata too https://github.com/openstack/glance/blob/master/etc/metadefs/compute-aggr-num-instances.json
14:15:34 bauzas lyarwood: I just agree with mriedem_snow on https://review.openstack.org/#/c/628726/4
14:15:46 bauzas lyarwood: we should make backports easy by providing a series
14:15:48 bauzas WDYT ?
14:16:36 bauzas since you already identified the source of the conflicts, and you also want to backport the missing changes, I don't see any difficulty in making the backports intertwined
14:16:36 lyarwood bauzas: yeah ack sorry I totally forgot about that
14:16:50 bauzas lyarwood: ack, ping me when you're done then

Earlier   Later