| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-02-12 | |||
| 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: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 09:46:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421 | |
| 09:46:13 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310 | |
| 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: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 | 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: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: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 | bauzas | AFAIK, we said meh | |
| 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: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 | 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: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: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: Add migration param to check_can_live_migrate_destination https://review.openstack.org/634605 | |
| 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:38 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: [WIP] Introduce live_migration_claim() https://review.openstack.org/635669 | |
| 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:39 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: [WIP] LM: Make dest send NUMAMigrateData to the source https://review.openstack.org/634828 | |
| 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: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 | lyarwood | bauzas: yeah ack sorry I totally forgot about that | |
| 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:50 | bauzas | lyarwood: ack, ping me when you're done then | |
| 14:17:29 | bauzas | that's what happens when you write poor interfaces | |
| 14:20:57 | NewBruce | sean-k-mooney any chance i can get your thoughts on a bit of a bug that looks to be related to the port binding API changes? | |
| 14:25:07 | sean-k-mooney | NewBruce: i saw you comments this moring | |
| 14:25:19 | NewBruce | awesome, sorry for chasing | |
| 14:25:44 | NewBruce | done some more debugging now, and feels like i isolated the point it fails | |
| 14:25:47 | sean-k-mooney | no worries i just havent had a chance to reply was booking flights and stuff | |
| 14:26:02 | sean-k-mooney | so one general question | |
| 14:26:06 | NewBruce | np. hope your goin' somewhere warm | |