openstack/kolla-ansible
Code Issues Proposed changes

Do not set pid file for iscsid

Browse Source 
Kolla Ansible runs iscsid in the foreground (-f) and
a recent change to iscsid in CentOS 8 (both Linux and Stream)
caused it to reject setting pid file in such a case.
PID file is irrelevant in this scenario so this commit
removes its parameter.

Closes-Bug: #1933033
Change-Id: Ic0c4beae0c812f3ca68a6ee5cc4daa2fee0f277d
This commit is contained in:
Radosław Piliszek 2021-06-20 14:14:53 +00:00
parent 5665d5e619
commit 18a0af6954
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ansible/roles/iscsi/templates/iscsid.json.j2
View File

@ -1,4 +1,4 @@
{ {
"command": "iscsid -d 8 -f --pid=/run/iscsid.pid", "command": "iscsid -d 8 -f",
"config_files": [] "config_files": []
} }

6
releasenotes/notes/bug-1933033-76746d127285cfe8.yaml Normal file
View File

@ -0,0 +1,6 @@
---
fixes:
- |
Fixes ``iscsid`` failing in current CentOS 8 based images due to
pid file being needlessly set.
`LP#1933033 <https://launchpad.net/bugs/1933033>`__