Skip to content

Fix redis vars on example playbook for single host deploy #202

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
lae merged 3 commits into from
Jun 30, 2025
Merged

Fix redis vars on example playbook for single host deploy #202

lae merged 3 commits into from
Jun 30, 2025

Conversation

@egypcio
Copy link
Contributor

@egypcio egypcio commented Jun 27, 2025

by merging this request, we:

  • set the proper redis_download_url, directly defining it to use HTTPS;
  • update redis_version to follow the latest stable v6 release (6.2.14);
  • correct the redis_checksum to match the release tarball.

this change respects the upstream role from @DavidWittman, though keeps using redis v6 for minimum impacts.

here's an output of how the previous URL was not working, and this it's fine after the update:

  • before
$ curl -s -o before https://download.redis.io/redis-6.2.14.tar.gz
$ file before
before: HTML document, ASCII text, with CRLF line terminators
$ sha256sum before 
340c8464c2007ce3f80682e15dfafa4180b641d53c14201b929906b7b0284d87  before
$ cat before 
<html>
<head><title>404 Not Found</title></head>
<body>
<center><h1>404 Not Found</h1></center>
<hr><center>nginx/1.18.0 (Ubuntu)</center>
</body>
</html>
  • after
$ curl -s -o after https://download.redis.io/releases/redis-6.2.14.tar.gz
$ file after 
after: gzip compressed data, was "redis-6.2.14.tar", last modified: Wed Oct 18 07:56:35 2023, max compression, from Unix, original size modulo 2^32 11284480
$ sha256sum after 
34e74856cbd66fdb3a684fb349d93961d8c7aa668b06f81fd93ff267d09bc277  after

lae
lae approved these changes Jun 30, 2025
View reviewed changes
Copy link
Owner

@lae lae left a comment

Choose a reason for hiding this comment

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

💯

@lae lae merged commit 3d2acdf into lae:develop Jun 30, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lae lae lae approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@egypcio @lae