Skip to content

Commit

Permalink
try the first actual redis test
Browse files Browse the repository at this point in the history
  • Loading branch information
Florian Goth committed Jan 3, 2024
1 parent c7e73d8 commit 878c943
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 40 deletions.
37 changes: 1 addition & 36 deletions molecule/default/converge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,39 +8,4 @@
## Setup test enviroment
# test whether redis server accepts connections on the unix socket
- name: test connection to socket
copy:
content: |
<html>Success! The test page is displayed correctly</html>
dest: "{{www_dir}}/index.html"
group: "{{ nginx_user }}"
owner: "{{ nginx_group }}"
mode: 0754
register: deployed
- name: Deploy nginx configuration
copy:
content: |
server {
listen 80 default_server;
listen [::]:80 default_server;
root {{ www_dir }};
index index.html index.htm;
location / {
try_files $uri $uri/ =404;
}
}
dest: "{{vhosts_dir}}/default.conf"
owner: root
group: root
mode: 0644
when: deployed.changed and deploy_nginx_config == true

- name: Perform nginx check
shell: nginx -t
when: deployed.changed

# Restart of nginx is required for the new configuration
- name: Restart nginx
systemd:
name: nginx
state: restarted
when: deployed.changed
shell: redis-benchmark -q -n 100 -s /var/run/redis/{{ redis_instancename }}.sock

Check failure on line 11 in molecule/default/converge.yml

View workflow job for this annotation

GitHub Actions / Lint

11:81 [line-length] line too long (86 > 80 characters)
4 changes: 0 additions & 4 deletions tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,6 @@
- -m
- '0750'
- "/var/lib/redis/{{ redis_instancename }}/"
# args:
# cmd: >
# "/usr/bin/install -d -o redis -g redis -m 0750
# /var/lib/redis/{{ redis_instancename }}/"

- name: Ensure Redis is running and enabled on boot.
service: "name={{ redis_daemon }} state=started enabled=yes"

0 comments on commit 878c943

Please sign in to comment.