Skip to content

Commit ceb9176

Browse files
committed
Revert "[FLOC-4086] Warn users about needing access to S3 buckets during CloudFormation"
1 parent 32246e1 commit ceb9176

File tree

2 files changed

+9
-20
lines changed

2 files changed

+9
-20
lines changed

admin/installer/cloudformation.py

+9-16
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ def _validate_cluster_size(size):
141141
# Keys corresponding to CloudFormation user Inputs.
142142
access_key_id_param = template.add_parameter(Parameter(
143143
"AmazonAccessKeyID",
144-
Description="Required: Your Amazon AWS access key ID",
144+
Description="Your Amazon AWS access key ID (mandatory)",
145145
Type="String",
146146
NoEcho=True,
147147
AllowedPattern="[\w]+",
@@ -150,32 +150,25 @@ def _validate_cluster_size(size):
150150
))
151151
secret_access_key_param = template.add_parameter(Parameter(
152152
"AmazonSecretAccessKey",
153-
Description="Required: Your Amazon AWS secret access key",
153+
Description="Your Amazon AWS secret access key (mandatory)",
154154
Type="String",
155155
NoEcho=True,
156156
MinLength="1",
157157
))
158158
keyname_param = template.add_parameter(Parameter(
159159
"EC2KeyPair",
160-
Description="Required: Name of an existing EC2 KeyPair to enable SSH "
161-
"access to the instance",
162-
Type="AWS::EC2::KeyPair::KeyName",
163-
))
164-
secret_access_key_param = template.add_parameter(Parameter(
165-
"S3AccessPolicy",
166-
Description="Required: Is current IAM user allowed to access S3? "
167-
"S3 access is required to distribute Flocker and Docker "
168-
"configuration amongst stack nodes. Reference: "
169-
"http://docs.aws.amazon.com/IAM/latest/UserGuide/"
170-
"access_permissions.html Stack creation will fail if user "
171-
"cannot access S3",
160+
Description="Name of an existing EC2 KeyPair to enable SSH "
161+
"access to the instance (mandatory)",
172162
Type="String",
173-
AllowedValues=["Yes"],
163+
MinLength="1",
164+
AllowedPattern="[\x20-\x7E]*",
165+
MaxLength="255",
166+
ConstraintDescription="can contain only ASCII characters.",
174167
))
175168
volumehub_token = template.add_parameter(Parameter(
176169
"VolumeHubToken",
177170
Description=(
178-
"Optional: Your Volume Hub token. "
171+
"Your Volume Hub token (optional). "
179172
"You'll find the token at https://volumehub.clusterhq.com/v1/token."
180173
),
181174
Type="String",

flocker/acceptance/endtoend/test_installer.py

-4
Original file line numberDiff line numberDiff line change
@@ -268,10 +268,6 @@ def _new_stack(self):
268268
{
269269
'ParameterKey': 'VolumeHubToken',
270270
'ParameterValue': os.environ['VOLUMEHUB_TOKEN']
271-
},
272-
{
273-
'ParameterKey': 'S3AccessPolicy',
274-
'ParameterValue': 'Yes'
275271
}
276272
]
277273

0 commit comments

Comments
 (0)