IPA element failed to start on ubuntu because of modprobe path

Bug #1658297 reported by Moshe Levi
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
diskimage-builder
Fix Released
Undecided
Ian Wienand

Bug Description

This commit https://github.com/openstack/diskimage-builder/commit/2854f4063bd2a6dcdb6fa5fab93aa56857e47b59 added pre start /usr/sbin/modprobe vfat
The problem is that on ubuntu the modprobe location is /sbin/modprobe

Changed in diskimage-builder:
assignee: nobody → Ian Wienand (iwienand)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to diskimage-builder (master)

Reviewed: https://review.openstack.org/423893
Committed: https://git.openstack.org/cgit/openstack/diskimage-builder/commit/?id=ca60b2cf7d30a8978ae9166a23bfee1bca8f6f6a
Submitter: Jenkins
Branch: master

commit ca60b2cf7d30a8978ae9166a23bfee1bca8f6f6a
Author: Ian Wienand <email address hidden>
Date: Mon Jan 23 14:20:36 2017 +1100

    ironic-agent: use /sbin for modprobe

    On redhat/fedora /sbin is a symlink to /usr/sbin, but not on all
    platforms. This was put in with
    Ibf74dd1b2678ea76e0676711a7aa5ba6b88d5421

    Change-Id: I7847b29503c3c07503430a7d85a5364911894c6c
    Closes-bug: #1658297

Changed in diskimage-builder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to diskimage-builder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/424988

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on diskimage-builder (master)

Change abandoned by Ian Wienand (<email address hidden>) on branch: master
Review: https://review.openstack.org/424988
Reason: wrong branch

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to diskimage-builder (feature/v2)

Fix proposed to branch: feature/v2
Review: https://review.openstack.org/424989

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on diskimage-builder (feature/v2)

Change abandoned by Ian Wienand (<email address hidden>) on branch: feature/v2
Review: https://review.openstack.org/424989
Reason: i'm going to try this in smaller increments

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to diskimage-builder (feature/v2)

Fix proposed to branch: feature/v2
Review: https://review.openstack.org/428519

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to diskimage-builder (feature/v2)
Download full text (18.6 KiB)

Reviewed: https://review.openstack.org/428519
Committed: https://git.openstack.org/cgit/openstack/diskimage-builder/commit/?id=3f8800832a32ed1523cad6ed5a786141f0a6385a
Submitter: Jenkins
Branch: feature/v2

commit 83bdfe179a549e9ae3cd61e4f15383f77d668c11
Author: Ian Wienand <email address hidden>
Date: Tue Jan 31 14:20:09 2017 +1100

    Git ignore coverage stuff

    One less piece of clutter when you're trying to merge ...

    Change-Id: I794027280359f5d1e02d4f3b99222d35783075d5

commit 11d384e2fa1f550d424f686600d643b3969a40b3
Author: Andrey Shestakov <email address hidden>
Date: Thu Jan 5 14:39:25 2017 +0200

    Add DIB_IPA_COMPRESS_CMD option

    This option allows to specifiy command for compress built initramfs image
    for ironic-agent element. This command can be specified with arguments and
    should read raw data from stdin and write compressed data to stdout.
    Default if "gzip".

    Change-Id: I0fdd2ab91d7bf7aaaa6cdd5278e3902d44c5b883

commit ca60b2cf7d30a8978ae9166a23bfee1bca8f6f6a
Author: Ian Wienand <email address hidden>
Date: Mon Jan 23 14:20:36 2017 +1100

    ironic-agent: use /sbin for modprobe

    On redhat/fedora /sbin is a symlink to /usr/sbin, but not on all
    platforms. This was put in with
    Ibf74dd1b2678ea76e0676711a7aa5ba6b88d5421

    Change-Id: I7847b29503c3c07503430a7d85a5364911894c6c
    Closes-bug: #1658297

commit dc8449bf02316f27b556994644f7c4ce18b995a8
Author: Matthew Thode <email address hidden>
Date: Sun Jan 22 15:53:02 2017 -0600

    move post-install.d to finalize.d

    It looks like I installed the cleanup file in the wrong location. Moving it
    to the correct location and cleaning some more stuff up should allow for a
    much smaller image, currently tested at 300M vs 490M.

    Change-Id: I9d0a24b0fe59d4f2d38aa88ba47e4400e0476b46

commit 90a347707c2c0e411052d9d54e00a3380f3e9ce3
Author: Paul Belanger <email address hidden>
Date: Thu Jan 12 11:55:47 2017 -0500

    Bump fedora release to 25

    Now that fedora-25 is released, bump fedora-minimal / fedora elements
    to use it.

    Change-Id: Ifdb05255e7a138b99099252fe17041fe4621b6f4
    Depends-On: I3cc467a2d08486458b3f625f94ba969532f6cd04
    Signed-off-by: Paul Belanger <email address hidden>

commit a45ad8eaf56c16c8f77dc8f96fecaf224af4d5bf
Author: Andrey Shestakov <email address hidden>
Date: Tue Dec 6 23:59:49 2016 +0200

    Fix dhcp-all-interfaces for ubuntu-minimal xenial

    The start script of dhcp-all-interfaces currently requires ifup command.
    ifup command provided by package ifupdown, which is not installed in
    ubuntu-minimal Xenial.

    This change adds ifupdown package for debian family, as it required for
    dhcp-all-interfaces.

    Closes-bug: #1647853

    Change-Id: I6dfc0108ec067f350b22e9fb933b9e8d47b09fde

commit 0ceb138d877101542934b7729f9497bc4c9a039a
Author: Ian Wienand <email address hidden>
Date: Wed Jan 18 08:32:10 2017 +1100

    Don't set base element path in run_functests.sh

    DIB automatically includes it's base elements in the ELEMENT_PATH;
    this double-set means the base elements are in the path twice, ...

tags: added: in-feature-v2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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