Giter Site home page Giter Site logo

Comments (7)

loganmc10 avatar loganmc10 commented on August 17, 2024

Maybe related to #3951, issue is not present when I set the image tags to v2.4.4

from assisted-service.

pdfruth avatar pdfruth commented on August 17, 2024

There is incomplete JSON data in the file;

https://github.com/openshift/assisted-service/blob/73fd2631a5c4fc60cca8b71548ac272e0801d650/internal/ignition/ignition.go

This chunk of JSON at lines 205 - 208

	{
        "name": "multipathd.service",
        "enabled": true,            <---- notice the contents that should follow is missing
    },

It should be amended to include the service's contents;

	{
        "name": "multipathd.service",
        "enabled": true,
        "contents":  "bla bla bla".   <---- notice the contents that should be placed here
    },

@avishayt please fix the error introduced by commit;
73fd263

from assisted-service.

avishayt avatar avishayt commented on August 17, 2024

I'm guessing that this is because the ISO doesn't have multipathd installed? Please verify.

from assisted-service.

pdfruth avatar pdfruth commented on August 17, 2024

@avishayt
No, I'm suggesting the problem is to do with the internal/ignition/ignition.go code.
It is generating an ignition file that has invalid json. since the change in this commit

As I suggested in my earlier comment, shouldn't there be a contents element containing the service definition for the multipathd.service

from assisted-service.

avishayt avatar avishayt commented on August 17, 2024

This is strange because in OpenShift/RHCOS, it is working. I SSH into an agent and see:

[core@localhost ~]$ systemctl status multipathd
● multipathd.service - Device-Mapper Multipath Device Controller
   Loaded: loaded (/usr/lib/systemd/system/multipathd.service; enabled; vendor preset: enabled)
   Active: active (running) since Sun 2022-06-26 09:14:33 UTC; 3min 20s ago
  Process: 1228 ExecStartPre=/sbin/multipath -A (code=exited, status=0/SUCCESS)
  Process: 1226 ExecStartPre=/sbin/modprobe -a scsi_dh_alua scsi_dh_emc scsi_dh_rdac dm-multipath (code=exited, status=0/SUCCESS)
 Main PID: 1230 (multipathd)
   Status: "up"
    Tasks: 7
   Memory: 15.9M
   CGroup: /system.slice/multipathd.service
           └─1230 /sbin/multipathd -d -s

Jun 26 09:14:33 localhost systemd[1]: Starting Device-Mapper Multipath Device Controller...
Jun 26 09:14:33 localhost multipathd[1230]: --------start up--------
Jun 26 09:14:33 localhost multipathd[1230]: read /etc/multipath.conf
Jun 26 09:14:33 localhost multipathd[1230]: path checkers start up
Jun 26 09:14:33 localhost systemd[1]: Started Device-Mapper Multipath Device Controller.
[core@localhost ~]$ cat /etc/systemd/system/sysinit.target.wants/multipathd.service
[Unit]
Description=Device-Mapper Multipath Device Controller
Wants=systemd-udev-trigger.service systemd-udev-settle.service
Before=iscsi.service iscsid.service lvm2-activation-early.service
Before=local-fs-pre.target blk-availability.service shutdown.target
After=multipathd.socket systemd-udev-trigger.service systemd-udev-settle.service
ConditionPathExists=/etc/multipath.conf
DefaultDependencies=no
Conflicts=shutdown.target
ConditionKernelCommandLine=!nompath
ConditionKernelCommandLine=!multipath=off

[Service]
Type=notify
NotifyAccess=main
LimitCORE=infinity
ExecStartPre=-/sbin/modprobe -a scsi_dh_alua scsi_dh_emc scsi_dh_rdac dm-multipath
ExecStartPre=-/sbin/multipath -A
ExecStart=/sbin/multipathd -d -s
ExecReload=/sbin/multipathd reconfigure
TasksMax=infinity

[Install]
WantedBy=sysinit.target
Also=multipathd.socket

from assisted-service.

vrutkovs avatar vrutkovs commented on August 17, 2024

FCOS doesn't have multipath.service pre-installed, but imo its an Ignition bug - it should not crash on enabling the service which doesn't exist, or at least give a better error message.

Seems we need to remove multipathd ignition part on OKD deployments (iirc it is being enabled with a flag)

from assisted-service.

avishayt avatar avishayt commented on August 17, 2024

That's what I suspected, thanks for verifying @vrutkovs . I'll add multipath to the ignition only if not OKD.

from assisted-service.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.