Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add two feature gates : Image based deployment and additional runtimeClasses #394

Closed
wants to merge 16 commits into from
Closed

Add two feature gates : Image based deployment and additional runtimeClasses #394

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
27a7e94
Move featuregates to separate package
bpradipt Apr 23, 2024
b6bacc1
Add method to check for feature cm
bpradipt Apr 25, 2024
2559ba0
Add support for layered image based deployment
bpradipt Apr 23, 2024
c4de12f
Add separate event handler for configmap updates
bpradipt Apr 8, 2024
c69ccba
Allow multiple runtimeClass creation
bpradipt Apr 6, 2024
1fc86ae
Add feature for creating additional runtimeClasses
bpradipt Apr 6, 2024
36272dc
Update featuregate sample yaml
bpradipt Apr 23, 2024
d1eea4b
remove unused method
bpradipt Apr 24, 2024
fb547fe
Rename DefaultFeatureGates to DefaultFeatureGatesStatus
bpradipt Apr 25, 2024
0e53603
Handle featuregate cm delete
bpradipt Apr 24, 2024
6c004fd
Create featuregate status configmap at start
bpradipt Apr 25, 2024
9e96e8d
Handle missing featuregate status configmap
bpradipt Apr 25, 2024
c59f99b
Remove duplicate code for reading yamls
bpradipt Apr 25, 2024
27599db
Rename FeatureGatesConfigMapName to FeatureGatesStatusConfigMapName
bpradipt Apr 25, 2024
9a30d74
Revert fg status configmap to default on KataConfig deletion
bpradipt Apr 25, 2024
5f47dc5
Add unit tests for fg method
bpradipt Apr 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
12 changes: 6 additions & 6 deletions internal/featuregates/features.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ import "strings"
*/

const (
FeatureGatesConfigMapName = "osc-feature-gates"
LayeredImageDeployment = "LayeredImageDeployment"
LayeredImageDeploymentConfig = "layeredimagedeployment-config"
AdditionalRuntimeClasses = "AdditionalRuntimeClasses"
AdditionalRuntimeClassesConfig = "additionalruntimeclasses-config"
FeatureGatesStatusConfigMapName = "osc-feature-gates"
Copy link
Contributor

Choose a reason for hiding this comment

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

Again, a good change but should be squashed with the commit where the original name was first introduced.

LayeredImageDeployment = "LayeredImageDeployment"
LayeredImageDeploymentConfig = "layeredimagedeployment-config"
AdditionalRuntimeClasses = "AdditionalRuntimeClasses"
AdditionalRuntimeClassesConfig = "additionalruntimeclasses-config"
)
bpradipt marked this conversation as resolved.
Show resolved Hide resolved

// Sample ConfigMap with Features
bpradipt marked this conversation as resolved.
Show resolved Hide resolved
Expand Down Expand Up @@ -66,7 +66,7 @@ func GetFeatureGateConfigMapName(feature string) string {
// and do the match
func IsFeatureGateConfigMap(configMapName string) bool {
switch configMapName {
case FeatureGatesConfigMapName, LayeredImageDeploymentConfig, AdditionalRuntimeClassesConfig:
case FeatureGatesStatusConfigMapName, LayeredImageDeploymentConfig, AdditionalRuntimeClassesConfig:
return true
default:
return false
Expand Down
2 changes: 1 addition & 1 deletion main.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ func main() {
FeatureGates: &featuregates.FeatureGates{
Client: mgr.GetClient(),
Namespace: OperatorNamespace,
ConfigMapName: featuregates.FeatureGatesConfigMapName,
ConfigMapName: featuregates.FeatureGatesStatusConfigMapName,
},
}).SetupWithManager(mgr); err != nil {
setupLog.Error(err, "unable to create KataConfig controller for OpenShift cluster", "controller", "KataConfig")
Expand Down