redis: Drop redis and redis-sentinel images

We switched to Valkey

Change-Id: Ic40fecb87dd3eaf67eb1d3088978d462ff5b4320
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
This commit is contained in:
Michal Nasiadka 2025-12-03 10:31:30 +01:00
parent b013f9ad1d
commit 2da1676d7a
12 changed files with 10 additions and 113 deletions

View file

@ -133,7 +133,7 @@ Kolla provides images to deploy the following infrastructure components:
direct messaging back end for communication between services.
- `RabbitMQ <https://www.rabbitmq.com/>`__ as a broker messaging back end for
communication between services.
- `Redis Sentinel <https://redis.io/topics/sentinel>`__ provides high availability for redis
- `Valkey Sentinel <https://valkey.io/topics/sentinel>`__ provides high availability for valkey
along with collateral tasks such as monitoring, notification and acts as configuration
provider for clients.
- `Telegraf <https://www.docs.influxdata.com/telegraf/>`__ as a plugin-driven server

View file

@ -46,11 +46,11 @@ ovsdpdk,N,U,U
placement,U,U,U
prometheus,U,U,U
rabbitmq,U,U,U
redis,U,U,U
skyline,U,U,U
tacker,U,U,U
telegraf,N,N,N
tgtd,U,U,U
trove,U,U,U
valkey,U,U,U
watcher,U,U,U
zun,U,U,U

1 Image Rocky Linux Ubuntu Debian
46 placement U U U
47 prometheus U U U
48 rabbitmq U U U
redis U U U
49 skyline U U U
50 tacker U U U
51 telegraf N N N
52 tgtd U U U
53 trove U U U
54 valkey U U U
55 watcher U U U
56 zun U U U

View file

@ -47,12 +47,12 @@ ovsdpdk,N,U,U
placement,T,T,T
prometheus,U,U,U
rabbitmq,T,T,T
redis,T,U,U
skyline,U,U,U
swift,T,T,U
tacker,T,U,U
telegraf,U,U,U
tgtd,N,T,U
trove,U,U,U
valkey,T,U,U
watcher,U,U,U
zun,T,T,U

1 Image Rocky Linux Ubuntu Debian
47 placement T T T
48 prometheus U U U
49 rabbitmq T T T
redis T U U
50 skyline U U U
51 swift T T U
52 tacker T U U
53 telegraf U U U
54 tgtd N T U
55 trove U U U
56 valkey T U U
57 watcher U U U
58 zun T T U

View file

@ -1,17 +0,0 @@
FROM {{ namespace }}/{{ image_prefix }}base:{{ tag }}
{% block labels %}
LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build_date }}"
{% endblock %}
{% block redis_base_header %}{% endblock %}
{% import "macros.j2" as macros with context %}
{{ macros.configure_user(name='redis', homedir='/run/redis') }}
COPY extend_start.sh /usr/local/bin/kolla_extend_start
RUN chmod 644 /usr/local/bin/kolla_extend_start
{{ macros.kolla_patch_sources() }}
{% block redis_base_footer %}{% endblock %}

View file

@ -1,9 +0,0 @@
#!/bin/bash
if [[ ! -d "/var/log/kolla/redis" ]]; then
mkdir -p /var/log/kolla/redis
fi
if [[ $(stat -c %a /var/log/kolla/redis) != "755" ]]; then
chmod 755 /var/log/kolla/redis
fi

View file

@ -1,25 +0,0 @@
FROM {{ namespace }}/{{ image_prefix }}redis-base:{{ tag }}
{% block labels %}
LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build_date }}"
{% endblock %}
{% block redis_sentinel_header %}{% endblock %}
{% import "macros.j2" as macros with context %}
{% if base_package_type == 'rpm' %}
{% set redis_sentinel_packages = ['redis'] %}
{% elif base_package_type == 'deb' %}
{% set redis_sentinel_packages = ['redis-sentinel'] %}
{% endif %}
{{ macros.install_packages(redis_sentinel_packages | customizable("packages")) }}
COPY extend_start.sh /usr/local/bin/kolla_extend_start
RUN chmod 644 /usr/local/bin/kolla_extend_start
{{ macros.kolla_patch_sources() }}
{% block redis_sentinel_footer %}{% endblock %}
{% block footer %}{% endblock %}
USER redis

View file

@ -1,32 +0,0 @@
#!/bin/bash
if [[ ! -d "/var/log/kolla/redis" ]]; then
mkdir -p /var/log/kolla/redis
fi
if [[ $(stat -c %a /var/log/kolla/redis) != "755" ]]; then
chmod 755 /var/log/kolla/redis
fi
# The CONFIG REWRITE command rewrites the redis.conf
# file the server was started with, applying the minimal
# changes needed to make it reflect the configuration
# currently used by the server, which may be different
# compared to the original one because of the use of
# the CONFIG SET command.
#
# https://redis.io/commands/config-rewrite/
#
# Because of above behaviour it's needed to
# hack kolla's CMD.
#
# Without this hack
# /usr/local/bin/kolla_set_configs --check
# is always reporting changed.
#
# Therefore redis-sentinel is always restarted
# even if configuration is not changed from
# kolla-ansible side.
if [ ! -z "${REDIS_CONF}" ] && [ ! -z ${REDIS_GEN_CONF} ]; then
cp ${REDIS_CONF} ${REDIS_GEN_CONF}
fi

View file

@ -1,22 +0,0 @@
FROM {{ namespace }}/{{ image_prefix }}redis-base:{{ tag }}
{% block labels %}
LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build_date }}"
{% endblock %}
{% block redis_header %}{% endblock %}
{% import "macros.j2" as macros with context %}
{% if base_package_type == 'rpm' %}
{% set redis_packages = ['redis'] %}
{% elif base_package_type == 'deb' %}
{% set redis_packages = ['redis-server'] %}
{% endif %}
{{ macros.install_packages(redis_packages | customizable("packages")) }}
{{ macros.kolla_patch_sources() }}
{% block redis_footer %}{% endblock %}
{% block footer %}{% endblock %}
USER redis

View file

@ -64,9 +64,9 @@ _PROFILE_OPTS = [
'openvswitch',
'proxysql',
'rabbitmq',
'redis',
'storm',
'tgtd',
'valkey',
],
help='Infra images'),
cfg.ListOpt('main',
@ -99,10 +99,10 @@ _PROFILE_OPTS = [
'masakari',
'mistral',
'octavia',
'redis',
'tacker',
'telegraf',
'trove',
'valkey',
'zookeeper',
'zun',
],

View file

@ -217,7 +217,7 @@ USERS = {
'uid': 42459,
'gid': 42459,
},
'redis-user': {
'redis-user': { # unused user (redis dropped)
'uid': 42460,
'gid': 42460,
},

View file

@ -26,7 +26,6 @@ UNBUILDABLE_IMAGES = {
"hacluster-pcs", # Missing crmsh package
"nova-spicehtml5proxy", # Missing spicehtml5 package
"ovsdpdk", # Not supported on CentOS
"redis-base", # Missing in CS10
"telegraf", # CS10 no opstools repo
"tgtd", # Not supported on CentOS
},
@ -41,7 +40,6 @@ UNBUILDABLE_IMAGES = {
"hacluster-pcs", # Missing crmsh package
"nova-spicehtml5proxy", # Missing spicehtml5 package
"ovsdpdk", # Not supported on CentOS
"redis-base", # Missing in Rocky10
"telegraf", # CS10 no opstools repo
"tgtd", # Not supported on CentOS
},

View file

@ -0,0 +1,4 @@
---
upgrade:
- |
``redis`` container image has been dropped due to migration to ``valkey``.