docs: show benchmark results for storage capacity tracking #350
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
k8s-ci-robot
added
release-note-none
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pohly The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
approved
jsafrane
reviewed
Mar 7, 2022
docs/storage-capacity-tracking.md
Outdated
Comment on lines
281
to
283
| There were other intermittent problems accessing the apiserver. Tweaking the | ||
| way how external-provisioner creates CSIStorageCapacity objects solved this | ||
| problem and the same test passed all three times that it was run: |
There was a problem hiding this comment.
That was meant to be a link to kubernetes-csi/external-provisioner#711. I just used the defaults from that updated external-provisioner - text updated.
pohly
force-pushed
the
storage-capacity-experiment
branch
from
d6d9639
to
d117785
Compare
|
/hold Let me add results for 100 nodes with pods that have no volumes... |
k8s-ci-robot
added
the
do-not-merge/hold
This was done in preparation for promoting storage capacity tracking to GA. The csi-driver-hostpath repo seems like a suitable place to hold this text because it showcases how to use the distributed deployment example.
|
/hold cancel Baseline for 100 nodes was added. The overhead for pods with volumes was actually slightly lower than it was for 10 nodes. |
k8s-ci-robot
removed
the
do-not-merge/hold
pohly
force-pushed
the
storage-capacity-experiment
branch
from
d117785
to
f053a7b
Compare
|
/lgtm |
k8s-ci-robot
added
the
lgtm
|
/retest |
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.
What type of PR is this?
/kind documentation
What this PR does / why we need it:
This was done in preparation for promoting storage capacity tracking to GA. The
csi-driver-hostpath repo seems like a suitable place to hold this text because
it showcases how to use the distributed deployment example.
Does this PR introduce a user-facing change?: