Skip to content

Commit 421be1c

Browse files
authored
Merge pull request kubernetes#87400 from neolit123/1.18-remove-coredns-fg
kubeadm: remove the deprecated GA CoreDNS feature-gate
2 parents 88a0574 + 261f260 commit 421be1c

File tree

4 files changed

+0
-21
lines changed

4 files changed

+0
-21
lines changed

cmd/kubeadm/app/cmd/BUILD

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ go_test(
8585
"//cmd/kubeadm/app/apis/output/v1alpha1:go_default_library",
8686
"//cmd/kubeadm/app/cmd/options:go_default_library",
8787
"//cmd/kubeadm/app/constants:go_default_library",
88-
"//cmd/kubeadm/app/features:go_default_library",
8988
"//cmd/kubeadm/app/util:go_default_library",
9089
"//cmd/kubeadm/app/util/config:go_default_library",
9190
"//cmd/kubeadm/app/util/output:go_default_library",

cmd/kubeadm/app/cmd/init_test.go

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,13 @@ limitations under the License.
1717
package cmd
1818

1919
import (
20-
"fmt"
2120
"io/ioutil"
2221
"os"
2322
"path/filepath"
2423
"testing"
2524

2625
"k8s.io/apimachinery/pkg/util/sets"
2726
"k8s.io/kubernetes/cmd/kubeadm/app/cmd/options"
28-
"k8s.io/kubernetes/cmd/kubeadm/app/features"
2927
)
3028

3129
const (
@@ -86,13 +84,6 @@ func TestNewInitData(t *testing.T) {
8684
},
8785
expectError: true,
8886
},
89-
{
90-
name: "fail if deprecated feature gates are set",
91-
flags: map[string]string{
92-
options.FeatureGatesString: fmt.Sprintf("%s=true", features.CoreDNS),
93-
},
94-
expectError: true,
95-
},
9687
{
9788
name: "fails if invalid preflight checks are provided",
9889
flags: map[string]string{

cmd/kubeadm/app/features/features.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -28,19 +28,12 @@ import (
2828
)
2929

3030
const (
31-
32-
// CoreDNS is GA in v1.11
33-
CoreDNS = "CoreDNS"
3431
// IPv6DualStack is expected to be alpha in v1.16
3532
IPv6DualStack = "IPv6DualStack"
3633
)
3734

38-
var coreDNSMessage = "featureGates:CoreDNS has been removed in v1.13\n" +
39-
"\tUse kubeadm-config to select which DNS addon to install."
40-
4135
// InitFeatureGates are the default feature gates for the init command
4236
var InitFeatureGates = FeatureList{
43-
CoreDNS: {FeatureSpec: featuregate.FeatureSpec{Default: true, PreRelease: featuregate.Deprecated}, HiddenInHelpText: true, DeprecationMessage: coreDNSMessage},
4437
IPv6DualStack: {FeatureSpec: featuregate.FeatureSpec{Default: false, PreRelease: featuregate.Alpha}},
4538
}
4639

cmd/kubeadm/test/cmd/init_test.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -352,10 +352,6 @@ func TestCmdInitFeatureGates(t *testing.T) {
352352
name: "no feature gates passed",
353353
args: "",
354354
},
355-
{
356-
name: "feature gate CoreDNS=true",
357-
args: "--feature-gates=CoreDNS=true",
358-
},
359355
{
360356
name: "feature gate IPv6DualStack=true",
361357
args: "--feature-gates=IPv6DualStack=true",

0 commit comments

Comments
 (0)