search

containers / qm

QM is a containerized environment for running Functional Safety qm (Quality Management) software
https://github.com/containers/qm
GNU General Public License v2.0
23 stars 26 forks source link

e2e/tier-0 tests is failing in setup #310

Closed weiwang-linda closed 10 months ago

weiwang-linda commented 10 months ago

I encountered an issue with c9s. [ INFO ] Creating quadlet container file: container-radio in node1 and moving it to container qm to start the quadlet service... Job for container-radio.service failed because the control process exited with error code. See "systemctl status container-radio.service" and "journalctl -xeu container-radio.service" for details. [ FAILED ] unable to start the quadlet service container-radio with exit code: 1

The container-radio service start failed from qm

Jan 17 01:26:17 node1 systemd[1]: Starting The sleep container... Jan 17 01:26:17 node1 container-radio[59]: time="2024-01-17T01:26:17Z" level=warning msg="Failed to read /etc/containers/storage.conf toml: line 53 (last key \"storage.options.additionalimagestores\"): expected value but found \"Allows\" instead\n" Jan 17 01:26:17 node1 container-radio[59]: time="2024-01-17T01:26:17Z" level=error msg="toml: line 53 (last key \"storage.options.additionalimagestores\"): expected value but found \"Allows\" instead" Jan 17 01:26:17 node1 systemd[1]: container-radio.service: Main process exited, code=exited, status=1/FAILURE

It is broken grep here If you update the containers.conf to this additionalimagestores = [ "/var/lib/shared", ] all the other next till next session should stay commented

https://github.com/containers/qm/blob/c883d171c2d993206d6c48e73c1fc8d314699527/setup#L100-L104

So it is easy to reproduce since the origin file exist ls -ltr /usr/lib/qm/rootfs/usr/share/containers/storage.conf need to update correct regexp to give the fixed output

Yarboa commented 10 months ago

@weiwang-linda I think this sed is more safe to use 

-e '/additionalimages.*/a\"/var\/lib\/shared/\",'  # from that one 
-e '/additionalimages.*/{n;s/.*/\"\/var\/lib\/shared\",/;}'
Yarboa commented 10 months ago

@dougsland It seems that bluechi has changed, tier-0 tests are failing

Yarboa commented 10 months ago

There has a change from ManagerHost -> ControllerAddress

Need to change e2e scripts e "s/^#ManagerHost=/ManagerHost='"

dougsland commented 10 months ago

There has a change from ManagerHost -> ControllerAddress

Need to change e2e scripts e "s/^#ManagerHost=/ManagerHost='"

Correct. They mentioned in this week meeting, this change is going to happen.