tripleo

Swift: swift processes doesn't support SIGHUP to reload configurations

Bug #1936763 reported by Takashi Kajinami
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
In Progress
Medium
Takashi Kajinami

Bug Description

Description
===========
Currently SIGHUP is widely used to reload services inside containers.
However swift processes don't use SIGHUP to reload config files but to terminate processes.
For server processes we should use SIGUSR1 instead while daemon processes doesn't support reloading configuration files.

Steps to reproduce
==================
* Check /etc/systemd/system/multi-user.target.wants/tripleo_swift*.service

Expected result
===============
* SIGUSR1 is used to reload server services
* No ExecReload is defined for daemon services

Actual result
=============
* SIGHUP is used for all services for reload

Environment
===========
The issue was initially found in stable/train deployment

Logs & Configs
==============
N/A

See original description
Takashi Kajinami (kajinamit)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-ansible (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/tripleo-ansible/+/801216

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801217

Changed in tripleo:
status: New → In Progress
Takashi Kajinami (kajinamit)
Changed in tripleo:
assignee: nobody → Takashi Kajinami (kajinamit)
Takashi Kajinami (kajinamit)
Changed in tripleo:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-ansible (master)

Reviewed: https://review.opendev.org/c/openstack/tripleo-ansible/+/801216
Committed: https://opendev.org/openstack/tripleo-ansible/commit/74df889180a343c74485293dbcbd90b1d5a0eac9
Submitter: "Zuul (22348)"
Branch: master

commit 74df889180a343c74485293dbcbd90b1d5a0eac9
Author: Takashi Kajinami <email address hidden>
Date: Mon Jul 19 09:10:46 2021 +0900

    Allow customizing reload commands

    This change introduces ability to use arbitrary signal for reload and
    disable complete reload capability.
    Currently server services of Swift use SIGUSR1 instead of SIGHUP for
    reload and also its backend daemons like replicator doesn't support
    reloading based on signal.

    Related-Bug: #1936763
    Change-Id: I8c43a83d321c926b65857161e5800a6fda958eb6

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-heat-templates (master)

Change abandoned by "Takashi Kajinami <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801217
Reason: This hasn't attract much attention for long so I'll abandon this.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.