Skip to content

Commit

Permalink
ci - revert logcli exclusion of mariadb
Browse files Browse the repository at this point in the history
Change-Id: Ie42ffa677cd1b8646a9ff8245eb3fc28c0ab4171
  • Loading branch information
morucci committed Jul 24, 2024
1 parent 2057c0f commit 953180c
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions roles/post/get-loki-logs/tasks/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,10 @@
retries: 10
until: "loki_buildinfo.status == 200 and '2.9.2' in loki_buildinfo.content"

# TODO: Remove exclusion of mariadb (this was done because of the mariadb container logs everything at high rate breaking the capability of loki to wrap logs)
# Remove after the next sf-operator release (it affects only the upgrade job)
- name: Get aggregated logs prefixed by service
ansible.builtin.shell: >
~/bin/logcli query --batch 2000 --forward --since=6h --parallel-duration 15m --parallel-max-workers 4 --part-path-prefix=/tmp/all-query --merge-parts --no-labels --quiet '{namespace="sf", container != "mariadb"} | json | {{ line_format_query }}' > {{ _output_dir_realpath.stdout }}/all.log
~/bin/logcli query --batch 2000 --forward --since=6h --parallel-duration 15m --parallel-max-workers 4 --part-path-prefix=/tmp/all-query --merge-parts --no-labels --quiet '{namespace="sf"} | json | {{ line_format_query }}' > {{ _output_dir_realpath.stdout }}/all.log
- name: Change owner and group for the log dir
ansible.builtin.command: chown -R {{ ansible_user }}:{{ ansible_user }} {{ _output_dir_realpath.stdout }}

0 comments on commit 953180c

Please sign in to comment.