Skip to content
This repository has been archived by the owner on Oct 5, 2022. It is now read-only.

Commit

Permalink
Bugfixes: ensure proper region for register-image cmd and fix mismatc…
Browse files Browse the repository at this point in the history
…h on temp_dir between script and vars file
  • Loading branch information
Craig Barrett committed Jan 17, 2017
1 parent bf3d628 commit 62f5cf3
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions playbooks/build-ami.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@
# TODO: handle case where AMI name already exists
- name: Register AMI from
command: aws ec2 register-image
--region {{ ec2_region }}
--name '{{ ami_name }}'
--description '{{ ami_description }}'
--architecture {{ ami_architecture }}
Expand All @@ -52,8 +53,6 @@
build_user: "{{ ansible_user_id }}"
build_controller_host: "{{ ansible_hostname }}"

- fail:

- name: Find local git revision
command: git rev-parse HEAD
register: local_git_revision
2 changes: 1 addition & 1 deletion playbooks/group_vars/all
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
key_pair_name: vyos-build-ami
ec2_region: us-east-1
temp_folder: .tmp
temp_folder: "{{ playbook_dir }}/files/ssh-keys"
key_pair_file: "{{ temp_folder }}/{{ key_pair_name }}.pem"
instance_type: t2.medium
placeholder: vyos-build-ami
Expand Down

0 comments on commit 62f5cf3

Please sign in to comment.