Skip to content

Clobber token files first#460

Merged
und1sk0 merged 2 commits intomasterfrom
SAN-3488-clobber-vault-token-files
May 6, 2016
Merged

Clobber token files first#460
und1sk0 merged 2 commits intomasterfrom
SAN-3488-clobber-vault-token-files

Conversation

@und1sk0
Copy link
Copy Markdown
Contributor

@und1sk0 und1sk0 commented May 6, 2016

  • Clobber vault token files first.

Reviewers

Tests

Test any modifications on one of our environments.

Deployment (post-merge)

(does not need to be deployed)

@bkendall
Copy link
Copy Markdown
Contributor

bkendall commented May 6, 2016

better idea:

instead of using lineinfile to put the contents of the file in place, use copy with the content parameter:

- name: copy vault auth files
  tags: vault_files
  become: true
  copy:
    dest="/opt/runnable/dock-init/consul-resources/vault/{{ node_env }}/{{ item.file_name }}"
    content="{{ item.value }}"
    mode="0400"
  with_items:
  - { file_name: 'auth-token', value: "{{ vault_auth_token }}" }
  - { file_name: 'token-01', value: "{{ vault_token_01 }}" }
  - { file_name: 'token-02', value: "{{ vault_token_02 }}" }
  - { file_name: 'token-03', value: "{{ vault_token_03 }}" }

@und1sk0
Copy link
Copy Markdown
Contributor Author

und1sk0 commented May 6, 2016

ooh

@anandkumarpatel
Copy link
Copy Markdown

LGTM

1 similar comment
@bkendall
Copy link
Copy Markdown
Contributor

bkendall commented May 6, 2016

LGTM

@und1sk0 und1sk0 merged commit 0959043 into master May 6, 2016
@und1sk0 und1sk0 deleted the SAN-3488-clobber-vault-token-files branch May 6, 2016 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants