mirror of https://github.com/kubernetes/kops.git
Merge pull request #10106 from johngmyers/tf-json
Remove dependency of TerraformJSON feature flag
This commit is contained in:
commit
d739bae871
|
|
@ -194,9 +194,9 @@ func TestBastionAdditionalUserData(t *testing.T) {
|
||||||
|
|
||||||
// TestMinimalJSON runs the test on a minimal data set and outputs JSON
|
// TestMinimalJSON runs the test on a minimal data set and outputs JSON
|
||||||
func TestMinimalJSON(t *testing.T) {
|
func TestMinimalJSON(t *testing.T) {
|
||||||
featureflag.ParseFlags("+TerraformJSON,-Terraform-0.12")
|
featureflag.ParseFlags("+TerraformJSON")
|
||||||
unsetFeaureFlag := func() {
|
unsetFeaureFlag := func() {
|
||||||
featureflag.ParseFlags("-TerraformJSON,+Terraform-0.12")
|
featureflag.ParseFlags("-TerraformJSON")
|
||||||
}
|
}
|
||||||
defer unsetFeaureFlag()
|
defer unsetFeaureFlag()
|
||||||
newIntegrationTest("minimal-json.example.com", "minimal-json").withJSONOutput().runTestTerraformAWS(t)
|
newIntegrationTest("minimal-json.example.com", "minimal-json").withJSONOutput().runTestTerraformAWS(t)
|
||||||
|
|
|
||||||
|
|
@ -725,7 +725,7 @@ func (c *ApplyClusterCmd) Run(ctx context.Context) error {
|
||||||
checkExisting = false
|
checkExisting = false
|
||||||
outDir := c.OutDir
|
outDir := c.OutDir
|
||||||
tfVersion := terraform.Version011
|
tfVersion := terraform.Version011
|
||||||
if featureflag.Terraform012.Enabled() {
|
if featureflag.Terraform012.Enabled() && !featureflag.TerraformJSON.Enabled() {
|
||||||
tfVersion = terraform.Version012
|
tfVersion = terraform.Version012
|
||||||
}
|
}
|
||||||
tf := terraform.NewTerraformTarget(cloud, region, project, outDir, tfVersion, cluster.Spec.Target)
|
tf := terraform.NewTerraformTarget(cloud, region, project, outDir, tfVersion, cluster.Spec.Target)
|
||||||
|
|
|
||||||
|
|
@ -216,7 +216,7 @@ func (t *LaunchTemplate) RenderTerraform(target *terraform.TerraformTarget, a, e
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
if d != nil {
|
if d != nil {
|
||||||
if featureflag.Terraform012.Enabled() {
|
if featureflag.Terraform012.Enabled() && !featureflag.TerraformJSON.Enabled() {
|
||||||
userDataResource := fi.WrapResource(fi.NewBytesResource(d))
|
userDataResource := fi.WrapResource(fi.NewBytesResource(d))
|
||||||
|
|
||||||
tf.UserData, err = target.AddFile("aws_launch_template", fi.StringValue(e.Name), "user_data", userDataResource, true)
|
tf.UserData, err = target.AddFile("aws_launch_template", fi.StringValue(e.Name), "user_data", userDataResource, true)
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue