Ubuntu
openipmi package

openipmi fails to start with 'unexpected operator' in /etc/init.d/openinit

Bug #1844448 reported by Justin Greever
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openipmi (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

OS Release: Ubuntu 19.04
Package: openipmi
Version: 2.0.25-2.1ubuntu1

Expected results:
update system, install openipmi, start openipmi via systemctl

Actual results:
update system, install openipmi, start openipmi via systemctl, greeted with an error message and openipmi fails to start

Steps to reproduce issue:
1. fresh install of ubuntu 19.04.
2. sudo apt update && sudo apt upgrade
3. sudo apt install openipmi
4. sudo systemctl start openipmi
5. openipmi fails to start with error "Sep 17 12:25:08 anthrax openipmi[1347]: /etc/init.d/openipmi: 55: [: 5.2: unexpected operator" via journalctl -xe

Line 55 in /etc/init.d/openipmi is:
if [ "${kernel}" == "2.4" ]; then

Steps to fix the issue:
edit /etc/init.d/openipmi
go to line 55 and change the == to a single =
save file
perform systemctl daemon-reload
perform systemctl start openipmi
openipmi starts without issues/errors
verify with systemctl status openipmi

Tags: openipmi

Related branches

ubuntu/+source/openipmi:paelzer-lp-issue-workaround-temp-namespace/2.0.27-and-init-for-focal
Lucas Kanashiro (community): Approve
Canonical Server packageset reviewers: Pending requested
Diff: 35577 lines (+7791/-2554)
110 files modified
Makefile.am (+4/-4)
Makefile.in (+29/-21)
OpenIPMI.spec (+2/-2)
README (+0/-4)
aclocal.m4 (+27/-23)
ar-lib (+1/-1)
cmdlang/Makefile.am (+2/-2)
cmdlang/Makefile.in (+24/-17)
cmdlang/cmd_lanparm.c (+4/-2)
cmdlang/cmd_mc.c (+4/-2)
cmdlang/cmd_pef.c (+4/-2)
cmdlang/cmd_solparm.c (+4/-2)
cmdlang/ipmish.c (+3/-1)
config.h.in (+0/-186)
configure (+573/-1403)
configure.ac (+64/-147)
debian/changelog (+11/-0)
debian/control (+1/-1)
debian/openipmi.init (+15/-5)
debian/patches/sample-fix-readline-include-in-ipmi_serial_bmc_emu.c.patch (+36/-0)
debian/patches/series (+1/-0)
debian/rules (+2/-1)
dev/null (+0/-89)
doc/Makefile.in (+21/-13)
glib/Makefile.in (+21/-13)
include/Makefile.in (+21/-13)
include/OpenIPMI/Makefile.in (+21/-13)
include/OpenIPMI/internal/Makefile.in (+21/-13)
include/OpenIPMI/ipmi_conn.h (+7/-0)
include/OpenIPMI/ipmi_sol.h (+17/-7)
include/OpenIPMI/ipmiif.h (+1/-1)
include/OpenIPMI/os_handler.h (+6/-6)
include/OpenIPMI/selector.h (+22/-0)
lanserv/Makefile.in (+21/-13)
lanserv/OpenIPMI/Makefile.in (+21/-13)
lanserv/OpenIPMI/msg.h (+1/-1)
lanserv/bmc_storage.c (+6/-2)
lanserv/config.c (+2/-0)
lanserv/emu_cmd.c (+1/-1)
lanserv/ipmi_sim.c (+12/-0)
lanserv/lan.conf (+1/-3)
lanserv/lanserv_ipmi.c (+1/-0)
lanserv/marvell-bmc/Makefile.in (+21/-13)
lanserv/persist.c (+3/-0)
lanserv/sdrcomp/Makefile.in (+21/-13)
lanserv/serv.c (+4/-0)
lanserv/sol.c (+14/-7)
lib/Makefile.in (+21/-13)
lib/entity.c (+11/-11)
lib/ipmi.c (+2/-0)
lib/ipmi_lan.c (+23/-10)
lib/ipmi_smi.c (+12/-2)
lib/ipmi_sol.c (+54/-23)
lib/mc.c (+5/-3)
lib/normal_fru.c (+2/-2)
ltmain.sh (+2/-2)
m4/ax_compare_version.m4 (+177/-0)
m4/ax_pkg_swig.m4 (+135/-0)
m4/ax_prog_python_version.m4 (+66/-0)
m4/ax_python_devel.m4 (+327/-0)
m4/libtool.m4 (+0/-1)
man/Makefile.in (+21/-13)
sample/Makefile.am (+1/-2)
sample/Makefile.in (+24/-18)
sample/eventd.c (+1/-0)
sample/example_oem.c (+8/-9)
sample/ipmicmd.c (+2/-2)
swig/Makefile.in (+21/-13)
swig/OpenIPMI.i (+1/-1)
swig/perl/Makefile.am (+3/-1)
swig/perl/Makefile.in (+24/-14)
swig/perl/test_chan.p (+600/-0)
swig/perl/test_fru.p (+1151/-0)
swig/perl/test_lanparm.p (+864/-0)
swig/perl/test_pef.p (+1469/-0)
swig/perl/test_pet.p (+1170/-0)
swig/python/Makefile.am (+10/-15)
swig/python/Makefile.in (+34/-32)
swig/python/openipmigui.py (+1/-1)
swig/python/openipmigui/Makefile.am (+7/-27)
swig/python/openipmigui/Makefile.in (+27/-41)
swig/python/openipmigui/_conn.py (+2/-2)
swig/python/openipmigui/_control.py (+3/-3)
swig/python/openipmigui/_domain.py (+9/-9)
swig/python/openipmigui/_entity.py (+6/-6)
swig/python/openipmigui/_fru.py (+4/-4)
swig/python/openipmigui/_mc.py (+6/-6)
swig/python/openipmigui/_mc_chan.py (+6/-6)
swig/python/openipmigui/_mc_lanparm.py (+5/-5)
swig/python/openipmigui/_mc_pefparm.py (+5/-5)
swig/python/openipmigui/_mc_solparm.py (+5/-5)
swig/python/openipmigui/_mc_user.py (+5/-5)
swig/python/openipmigui/_saveprefs.py (+1/-1)
swig/python/openipmigui/_sel.py (+3/-3)
swig/python/openipmigui/_sensor.py (+3/-3)
swig/python/openipmigui/gui.py (+8/-8)
swig/python/openipmigui/gui_SoL.py (+3/-3)
swig/python/openipmigui/gui_cmdwin.py (+2/-2)
swig/python/openipmigui/gui_domainDialog.py (+2/-2)
swig/python/openipmigui/gui_list.py (+2/-2)
swig/python/openipmigui/gui_setdialog.py (+1/-1)
swig/python/openipmigui/gui_term.py (+1/-1)
swig/python/openipmigui/gui_treelist.py (+2/-2)
tcl/Makefile.am (+3/-3)
tcl/Makefile.in (+24/-16)
ui/Makefile.in (+21/-13)
ui/ui.c (+4/-2)
unix/Makefile.in (+21/-13)
unix/selector.c (+239/-94)
utils/Makefile.in (+21/-13)
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Hi Justin,
I just came by openipmi today and was looking at the bugs.
I can confirm yours - easy to be reproduced.

Changed in openipmi (Ubuntu):
status: New → Confirmed
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

That should be https://sourceforge.net/p/openipmi/code/ci/d7ac7a37321357d17fb581e9b7782f787334919b/
Which is interesting as this should be in 2.0.25 by upstream ...

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

The package seems to carry an outdated local copy ... grml

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

The init scripts have diverged a lot and I'm not going to do historical research for ages ont his.
To help for now I'll pass through the last bunch of upstream fixes to the init script and port those to Ubuntu.

Changed in openipmi (Ubuntu):
status: Confirmed → Triaged
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Ok, with the fixes in place it is much better.
If you happen to test on a container without devices you still get a fail, but a much more reasonable one than a syntax error.

Nov 28 17:46:24 f systemd[1]: Starting LSB: OpenIPMI Driver init script...
Nov 28 17:46:24 f openipmi[29057]: * Starting ipmi drivers
Nov 28 17:46:24 f openipmi[29057]: ...fail!
Nov 28 17:46:24 f openipmi[29057]: ...done.
Nov 28 17:46:24 f systemd[1]: openipmi.service: Control process exited, code=exited, status=1/FAILURE
Nov 28 17:46:24 f systemd[1]: openipmi.service: Failed with result 'exit-code'.
Nov 28 17:46:24 f systemd[1]: Failed to start LSB: OpenIPMI Driver init script.

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I combined it with an upgrade to 2.0.27 for further fixes and opened an MP for review
=> https://code.launchpad.net/~usd-import-team/ubuntu/+source/openipmi/+git/openipmi/+merge/376170

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package openipmi - 2.0.27-0ubuntu1

---------------
openipmi (2.0.27-0ubuntu1) focal; urgency=medium

  * New upstream release 2.0.27
    - d/rules: disable the now default-on python support
    - d/control: now needs libreadline-dev as build-depends
    - d/p/sample-fix-readline-include-in-ipmi_serial_bmc_emu.c.patch: fix
      FTBFS due to readline include in ipmi_serial_bmc_emu.c
  * d/openipmi.init: fix various issues identified upstream (LP: #1844448)

 -- Christian Ehrhardt <email address hidden> Thu, 28 Nov 2019 14:12:35 +0100

Changed in openipmi (Ubuntu):
status: Triaged → Fix Released
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.