Vereto/atmos
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove shared creds file

Browse source
This commit is contained in:
Conor McManus 2023-01-24 17:14:05 +01:00
parent 9491a890ab
commit c97850e916
3 changed files with 10 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
README.md
View file

@ -49,7 +49,6 @@ To get the most out of Terraform workspaces it is recommended that the AWS provi
provider "aws" { provider "aws" {
region = "${var.region}" region = "${var.region}"
profile = "${var.workspace}" profile = "${var.workspace}"
shared_credentials_file = ${var.shared_credentials_file}
} }
``` ```
@ -61,7 +60,7 @@ variable "workspace" {
} }
``` ```
This will make Terraform lookup AWS credentials from the `~/.aws/credentials` file using the workspace name as the stanza name. For example the credentials file would look like the shared-creds file in this repo. This will make Terraform lookup AWS credentials from the `~/.aws/credentials` file using the workspace name as the stanza name.
## atmos -e ## atmos -e

1
atmos.py
View file

@ -35,7 +35,6 @@ def determine_actions(args, params):
if (args.command in env_actions) and not (args.n): # Append with env context if (args.command in env_actions) and not (args.n): # Append with env context
cmd = cmd + ' -var-file=vars/{env}.tfvars'.format(env=workspace_vars) cmd = cmd + ' -var-file=vars/{env}.tfvars'.format(env=workspace_vars)
cmd = cmd + ' -var "workspace={env}"'.format(env=workspace) cmd = cmd + ' -var "workspace={env}"'.format(env=workspace)
cmd = cmd + ' -var "shared_credentials_file={aws_creds_file}"'.format(aws_creds_file=aws_creds_file)
for param in params: # Pass terraform params directly through for param in params: # Pass terraform params directly through
cmd = cmd + ' ' + param cmd = cmd + ' ' + param

18
tests.py
View file

@ -22,35 +22,35 @@ class DetermineActionsTests(unittest.TestCase):
@patch("workspaces.get_env") @patch("workspaces.get_env")
def test_whenCalledWithInitCommand_shouldAppendVarsAndCreds(self, mocked_get_env): def test_whenCalledWithInitCommand_shouldAppendVarsAndCreds(self, mocked_get_env):
"""Case where var-file, -var workpace=xyz and -var shared_credentials is appended""" """Case where var-file, -var workpace=xyz is appended"""
self.input_args.command = "init" self.input_args.command = "init"
mocked_get_env.return_value = "mytestenv" mocked_get_env.return_value = "mytestenv"
atmos.determine_actions(self.input_args, []) atmos.determine_actions(self.input_args, [])
atmos.run_cmd.assert_called_with('terraform init -var-file=vars/mytestenv.tfvars -var "workspace=mytestenv" -var "shared_credentials_file=$HOME/.aws/credentials"') atmos.run_cmd.assert_called_with('terraform init -var-file=vars/mytestenv.tfvars -var "workspace=mytestenv"')
@patch("workspaces.get_env") @patch("workspaces.get_env")
def test_whenCalledWithPlanCommand_shouldAppendVarsAndCreds(self, mocked_get_env): def test_whenCalledWithPlanCommand_shouldAppendVarsAndCreds(self, mocked_get_env):
"""Case where var-file, -var workpace=xyz and -var shared_credentials is appended""" """Case where var-file, -var workpace=xyz is appended"""
self.input_args.command = "plan" self.input_args.command = "plan"
mocked_get_env.return_value = "mytestenv" mocked_get_env.return_value = "mytestenv"
atmos.determine_actions(self.input_args, []) atmos.determine_actions(self.input_args, [])
atmos.run_cmd.assert_called_with('terraform plan -var-file=vars/mytestenv.tfvars -var "workspace=mytestenv" -var "shared_credentials_file=$HOME/.aws/credentials"') atmos.run_cmd.assert_called_with('terraform plan -var-file=vars/mytestenv.tfvars -var "workspace=mytestenv"')
@patch("workspaces.get_env") @patch("workspaces.get_env")
def test_whenCalledWithApplyCommand_shouldAppendVarsAndCreds(self, mocked_get_env): def test_whenCalledWithApplyCommand_shouldAppendVarsAndCreds(self, mocked_get_env):
"""Case where var-file, -var workpace=xyz and -var shared_credentials is appended""" """Case where var-file, -var workpace=xyz is appended"""
self.input_args.command = "apply" self.input_args.command = "apply"
mocked_get_env.return_value = "mytestenv" mocked_get_env.return_value = "mytestenv"
atmos.determine_actions(self.input_args, []) atmos.determine_actions(self.input_args, [])
atmos.run_cmd.assert_called_with('terraform apply -var-file=vars/mytestenv.tfvars -var "workspace=mytestenv" -var "shared_credentials_file=$HOME/.aws/credentials"') atmos.run_cmd.assert_called_with('terraform apply -var-file=vars/mytestenv.tfvars -var "workspace=mytestenv"')
@patch("workspaces.get_env") @patch("workspaces.get_env")
def test_whenCalledWithDestroyCommand_shouldAppendVarsAndCreds(self, mocked_get_env): def test_whenCalledWithDestroyCommand_shouldAppendVarsAndCreds(self, mocked_get_env):
"""Case where var-file, -var workpace=xyz and -var shared_credentials is appended""" """Case where var-file, -var workpace=xyz is appended"""
self.input_args.command = "destroy" self.input_args.command = "destroy"
mocked_get_env.return_value = "mytestenv" mocked_get_env.return_value = "mytestenv"
atmos.determine_actions(self.input_args, []) atmos.determine_actions(self.input_args, [])
atmos.run_cmd.assert_called_with('terraform destroy -var-file=vars/mytestenv.tfvars -var "workspace=mytestenv" -var "shared_credentials_file=$HOME/.aws/credentials"') atmos.run_cmd.assert_called_with('terraform destroy -var-file=vars/mytestenv.tfvars -var "workspace=mytestenv"')
@patch("workspaces.workspace_manager") @patch("workspaces.workspace_manager")
def test_whenInAGitRepo_andManualArgIsNotGiven_andEnvironmentArgIsNotGiven_shouldCallTheWorkspaceManager(self, mocked_workspace_manager): def test_whenInAGitRepo_andManualArgIsNotGiven_andEnvironmentArgIsNotGiven_shouldCallTheWorkspaceManager(self, mocked_workspace_manager):
@ -66,4 +66,4 @@ class DetermineActionsTests(unittest.TestCase):
if __name__ == '__main__': if __name__ == '__main__':
unittest.main() unittest.main()