Make config change triggers a default

pkg/api/serialization_test.go

@@ -112,6 +112,10 @@ func fuzzInternalObject(t *testing.T, forVersion string, item runtime.Object, se
 				j.To.Name = strings.Replace(j.To.Name, ":", "-", -1)
 			}
 		},
+		func(j *deploy.DeploymentConfig, c fuzz.Continue) {
+			c.FuzzNoCustom(j)
+			j.Triggers = []deploy.DeploymentTriggerPolicy{{Type: deploy.DeploymentTriggerOnConfigChange}}
+		},
 		func(j *deploy.DeploymentStrategy, c fuzz.Continue) {
 			c.FuzzNoCustom(j)
 			mkintp := func(i int) *int64 {

pkg/deploy/api/v1/defaults.go

@@ -12,6 +12,13 @@ func init() {
 	}
 
 	err := api.Scheme.AddDefaultingFuncs(
+		func(obj *DeploymentConfigSpec) {
+			if obj.Triggers == nil {
+				obj.Triggers = []DeploymentTriggerPolicy{
+					{Type: DeploymentTriggerOnConfigChange},
+				}
+			}
+		},
 		func(obj *DeploymentStrategy) {
 			if len(obj.Type) == 0 {
 				obj.Type = DeploymentStrategyTypeRolling

pkg/deploy/api/v1/defaults_test.go

@@ -31,10 +31,15 @@ func roundTrip(t *testing.T, obj runtime.Object) runtime.Object {
 	return obj3
 }
 
-func TestDefaults_rollingParams(t *testing.T) {
+func TestDefaults(t *testing.T) {
 	c := &current.DeploymentConfig{}
 	o := roundTrip(t, runtime.Object(c))
 	config := o.(*current.DeploymentConfig)
+
+	if len(config.Spec.Triggers) != 1 && config.Spec.Triggers[0].Type != current.DeploymentTriggerOnConfigChange {
+		t.Errorf("expected default trigger for config: %#v", config.Spec.Triggers)
+	}
+
 	strat := config.Spec.Strategy
 	if e, a := current.DeploymentStrategyTypeRolling, strat.Type; e != a {
 		t.Errorf("expected strategy type %s, got %s", e, a)