Rename C and CO to U and UO #3
Merged
Conversation
This file contains 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
| - *Burstable CPU performance* - The workload has a baseline compute | ||
| performance but is permitted to burst beyond this baseline, if | ||
| excess compute is available | ||
| - *Cache backed RAM* - VM RAM is cached based in order to provide memory |
There was a problem hiding this comment.
- will leverage file system cache and be fast, but as soon as node memory arises, the memory of the VMs will be pushed out of RAM, because the cache will be dropped.
The close distance to CX but the fact that C and CO are "workload agnostic" was confusing. Thus rename those types which are agnostic to use a different prefix. The prefix is now U, a hint at the expansion "Universal". Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Was messing up memory ratios because sorting was broken Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
|
Merging for now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The close distance to CX but the fact that C and CO are "workload agnostic" was confusing.
Thus rename those types which are agnostic to use a different prefix. The prefix is now U, a hint at the expansion "Universal".