Optimize keyword handling and driver manifest caching#989
Open
cvaroqui wants to merge 6 commits intoopensvc:mainfrom
Open
Optimize keyword handling and driver manifest caching#989cvaroqui wants to merge 6 commits intoopensvc:mainfrom
cvaroqui wants to merge 6 commits intoopensvc:mainfrom
Conversation
The keyword.Keyword struct is big and complex (contains list of types, ...) so using the concrete type in func prototypes caused lots of allocations and resulted in partial copies. Use pointers to stop copying, as kw definition are immutable anyway. Save cpu time.
* Cache the manifests to avoid rebuilding the kw lists * Add a DriverID() driver.ID func to the resource.Driver intf * Use DriverID() as the manifest cache key * Use directly DriverID() instead of Manifest().DriverID() in a lot of codepaths (lighter, lockless response)
Triggered by a corner case where topology is flex but object.Status.Flex is nil.
Which blocked the CI on: Go build and test: drivers/resappforking/main_test.go#L242 method T.DriverID already declared at drivers/resappforking/manifest.go:35:13
The "lxc info <name>" command exits when no error and empty output when the container is down and defined. The abort routine did not test the output length so reported the container up on peer when actually not.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.