Skip to content

distrodefs: add cache for decoded YAML files#2161

Draft
lzap wants to merge 1 commit intoosbuild:mainfrom
lzap:cache-loader1
Draft

distrodefs: add cache for decoded YAML files#2161
lzap wants to merge 1 commit intoosbuild:mainfrom
lzap:cache-loader1

Conversation

@lzap
Copy link
Contributor

@lzap lzap commented Jan 29, 2026

This cache is unique for each filesystem (it can be mocked in tests) and for each value type. Each returned value is a deep copy of the cached prototype. This significantly speeds up loading of oses and image types. It speeds up the "ibcli list" from 4s to 0.6s.


This is nicer implementation that I created in the morning, I am just pushing it here so I will not forget to take a look later. This is not meant to be merged yet.

This cache is unique for each filesystem (it can be mocked in tests) and for each value type.
Each returned value is a deep copy of the cached prototype. This significantly speeds up loading
of oses and image types. It speeds up the "ibcli list" from 4s to 0.6s.
@lzap lzap requested a review from a team as a code owner January 29, 2026 15:17
@achilleas-k
Copy link
Member

I am just pushing it here so I will not forget to take a look later. This is not meant to be merged yet.

In that case, I hope you don't mind, I'll mark it as draft.

@achilleas-k achilleas-k marked this pull request as draft January 29, 2026 18:36
Copy link
Member

@supakeen supakeen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to comment to get this out of my queue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants