Skip to content

Conversation

sc68cal
Copy link

@sc68cal sc68cal commented Jul 29, 2025

This is based off some issues with the following RPM installed via Rocky 8 Linux.

https://rockylinux.pkgs.org/8/rockylinux-appstream-x86_64/redis-5.0.3-5.module+el8.5.0+657+2674830e.x86_64.rpm.html

Some variables were also set, similar to the following comment #286 (comment)

This is based off some issues with the following RPM installed via Rocky
8 Linux.

https://rockylinux.pkgs.org/8/rockylinux-appstream-x86_64/redis-5.0.3-5.module+el8.5.0+657+2674830e.x86_64.rpm.html

Some variables were also set, similar to the following comment
DavidWittman#286 (comment)
@sc68cal
Copy link
Author

sc68cal commented Jul 29, 2025

What is strange to me, is that the package is supposed to create the /etc/redis directory when it installs the package, based on the file listing from pkgs.org but I was only able to get it to install after modifying the task to ensure that the directory existed. Without it, I would get errors and the playbook would fail to completed.

Regardless, this was a set of fixes that I had to make, in order to make it work in my Rocky Linux 8 environment. It may just be something that is specific to my Rocky Linux 8 images or the package mirror I am using, but this may be useful for others.

- ../templates
register: sentinel_unit_file
when: redis_as_service and ansible_service_mgr|default() == "systemd" and not (not redis_install_from_source and redis_service_name == 'redis')
when: redis_as_service and ansible_service_mgr|default() == "systemd"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this actually needs to be

Suggested change
when: redis_as_service and ansible_service_mgr|default() == "systemd"
when: redis_as_service and ansible_service_mgr|default() == "systemd" and redis_service_name == 'redis')

But I was just trying to get this working, and figure out why later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant