-
Notifications
You must be signed in to change notification settings - Fork 2
CSI compatibility mode #175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
ftl
wants to merge
55
commits into
main
Choose a base branch
from
csi-compatibility-mode
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,436
−27
Open
Changes from all commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
e95a47d
define the CompatibilityMode field
ftl 0395e83
add the default value for CSI.CompatibilityMode
ftl 9e3f0cf
provide access to the new CompatibilityMode field
ftl b6327d4
render the second DaemonSet in compatibility mode, if parameterized a…
ftl 2e7ef9f
install control-plane stackit-blockstorage chart (compat) as ManagedR…
nschad ded54b7
fix chart path
nschad 315a7b0
fix chart path and releaseName
nschad 7ea066f
fix path?
nschad 700c9cf
Move some things around
nschad 599e6c5
fix values?
nschad 69803f5
fix type assertion
nschad 256fe30
fix namespace
nschad d8d047c
use different name for MR release
nschad 053f355
apply fixes and inject images manually
nschad d8f5d45
remove the duplicated DaemonSet as it is not needed
ftl 3992bb6
add a symmetric method for the Shoot controlplane in Compatibility Mode
ftl 8470869
use the correct image vector for the Shoot controlplane
ftl 38631b6
delete compatibility mode managed resources when not needed
ftl da9924a
control the legacy-volume-creation flag
ftl 553c207
make sure to use the correct namespace and origin
ftl eb9840b
make generate
ftl 680824c
extract the CSI compatibility related code and use dependency injecti…
ftl fc3d2b7
fix typo
ftl 480a667
add a unit test scaffolding
ftl f6159d0
add checks for the relevant command line flags in the DaemonSets
ftl faa4edd
extract function
ftl 3b4737f
separate chart rendering and deployment
ftl 6dd7153
create the renderer only once
ftl d15c800
fix the compatibility mode accessor
ftl 88c40ea
simplify test assertions
ftl 7ece65f
extract helper methods and constants
ftl ed07511
handle unset compatibility mode
ftl 6116e66
go mod tidy
ftl 89bee29
modernize
ftl e324f1f
move the legacy-volume-creation flag to the seed control plane
ftl f6cca4e
use a different healthz port in the second csi drive instance, also i…
ftl 48bbe2f
fix typo
ftl 3d9af01
use a better prefix
ftl 38c6ad3
fix: deploy seed part in correct namespace; findImage by specific ver…
nschad 2cbfc21
make stackit-blockstorage-csi-driver (shoot) chart customizable for d…
nschad e85d5cf
make health-port of CSI livenessprobe configurable
nschad 24f65b7
only deploy snapshot-controller once
nschad 5610a2d
use the correct namespace
ftl 05ef572
fix: deploy snapshotcontroller by default; but NOT if in compatibilit…
nschad 0804373
check the liveness-probe healthz port
ftl f78f764
check the csi drive name in COMPAT(BLOCK) mode
ftl 9899a45
check that there is no snapshot controller deployed with the
ftl af3c625
create an integration test that checks if the compatibility related
ftl 60a7f79
add test to cover the transition from compat to default
ftl 6ce05fd
go mod tidy
ftl 4d8bffc
add a description for the two CSI config fields
ftl 7eef97a
use the flag instead of a string comparison
ftl ef0520e
update api documentation
ftl eacb4a1
fix wrong check
ftl ecafaa8
replace switch/case with if and early bail out
ftl File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.