Skip to content

Commit

Permalink
Fix test_aws_setup_credentials
Browse files Browse the repository at this point in the history
Signed-off-by: Alina Buzachis <abuzachis@redhat.com>
  • Loading branch information
alinabuzachis committed Nov 8, 2023
1 parent 989d13a commit 5f5fe40
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .config/ansible-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@ skip_list:
- no-handler # Tasks that run when changed should likely be handlers.
- no-changed-when # Commands should not change things if nothing needs doing.
exclude_paths:
- roles/aws_setup_credentials/ # Skip defaults/main.yml file
- tests/integration

Check failure on line 11 in .config/ansible-lint.yml

View workflow job for this annotation

GitHub Actions / ansible-lint

yaml[new-line-at-end-of-file]

No new line character at the end of file
2 changes: 1 addition & 1 deletion roles/aws_setup_credentials/defaults/main.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
aws_connection_env:
aws_setup_credentials__connection_env:
aws_profile:
env:
- AWS_PROFILE
Expand Down
2 changes: 1 addition & 1 deletion roles/aws_setup_credentials/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@

- name: Create auth credentials
ansible.builtin.include_tasks: read_option.yml
with_dict: "{{ aws_connection_env }}"
with_dict: "{{ aws_setup_credentials__connection_env }}"
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
ansible.builtin.include_role:
name: cloud.aws_ops.aws_setup_credentials
vars:
aws_security_token: '{{ security_token | default(omit) }}' # noqa var-naming
aws_security_token: '{{ security_token | default(omit) }}'

- name: Trying calling module using generating credentials
module_defaults:
Expand Down

0 comments on commit 5f5fe40

Please sign in to comment.