Skip to content

Commit

Permalink
all: use "no-same-owner" when unpacking binaries
Browse files Browse the repository at this point in the history
This is needed in order to ignore the owner of files used to archive the files. Using owner from archive might cause issues on some systems in case ID was set to a high number. See: VictoriaMetrics/VictoriaMetrics#6788

Signed-off-by: Zakhar Bessarab <[email protected]>
  • Loading branch information
zekker6 committed Aug 15, 2024
1 parent 78cce5d commit 56399a2
Show file tree
Hide file tree
Showing 7 changed files with 14 additions and 0 deletions.
2 changes: 2 additions & 0 deletions roles/single/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,8 @@
src: "{{ victoriametrics_download_url }}"
dest: /tmp/vic-single/
remote_src: yes
extra_opts:
- --no-same-owner
notify: Restart VictoriaMetrics service
register: archive_downloaded
become: no
Expand Down
2 changes: 2 additions & 0 deletions roles/vmagent/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@
src: "{{ vmagent_download_url }}"
dest: /tmp
remote_src: yes
extra_opts:
- --no-same-owner
when:
- not ansible_check_mode
- not vmagent_is_installed.stat.exists or
Expand Down
2 changes: 2 additions & 0 deletions roles/vmalert/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@
src: "{{ vic_vm_alert_download_url }}"
dest: /tmp
remote_src: yes
extra_opts:
- --no-same-owner
delegate_to: localhost
become: no
when:
Expand Down
2 changes: 2 additions & 0 deletions roles/vmauth/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@
src: "{{ vmauth_download_url }}"
dest: /tmp
remote_src: yes
extra_opts:
- --no-same-owner
when:
- not ansible_check_mode
- not vmauth_is_installed.stat.exists or
Expand Down
2 changes: 2 additions & 0 deletions roles/vminsert/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@
src: "{{ vminsert_download_url }}"
dest: /tmp
remote_src: yes
extra_opts:
- --no-same-owner
when:
- not ansible_check_mode
- not vminsert_is_installed.stat.exists or
Expand Down
2 changes: 2 additions & 0 deletions roles/vmselect/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@
src: "{{ vmselect_download_url }}"
dest: /tmp
remote_src: yes
extra_opts:
- --no-same-owner
when:
- not ansible_check_mode
- not vmselect_is_installed.stat.exists or
Expand Down
2 changes: 2 additions & 0 deletions roles/vmstorage/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@
src: "{{ vmstorage_download_url }}"
dest: /tmp
remote_src: yes
extra_opts:
- --no-same-owner
when:
- not ansible_check_mode
- not vmstorage_is_installed.stat.exists or
Expand Down

0 comments on commit 56399a2

Please sign in to comment.