-
Notifications
You must be signed in to change notification settings - Fork 216
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.
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
Adding ability to fill ci/override.json with parameter overrides. #58
Conversation
taskcat/stacker.py
Outdated
param_path = self.parameter_path | ||
_url_prefix=re.search(r'^http.*/.*/(?!\/.*?$)', param_path).group[:-1] | ||
override_url = "{}/{}".format(_url_prefix, 'override.json') | ||
r = requests.get(override_url) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To match the changes in development can you make this an s3 obj get call instead. This is necessary to support private buckets
@@ -943,6 +972,9 @@ def stackcreate(self, taskcat_cfg, test_list, sprefix): | |||
cfn = self._boto_client.get('cloudformation', region=region) | |||
s_parmsdata = requests.get(self.get_parameter_path()).text |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as line 267
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. changes incoming.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small changes.
See comments:
Feature Add:
Items like KeyPair can be global. (meaning they are usually the same across repos) As a feature update it would be good if we can a global override file. For example ~/.taskcat_global_overides.json if this file exists all tests would use it. else, check the ci folder for project level overides. This way we don't need to specify it in every ci folder unless the params are project specfic
Changes pushed in 4966cc2. Please review. |
Fixes #57