mdraid name creates broken configuration

This is not a MAAS issue but a curtin issue (what MAAS uses for the installation).

Hi there,

Could you please provide the curtin configuration sent to the machine?

maas <user> machine get-curtin-config <system-id>

Curtin can validate the name value in the config; if it matches kernel names (mdXX) then the path can be /dev/{name} if it doesn't match then it needs to use /dev/md/{name}.

Ryan,
I can understand that, but if the case some form of variable check has to be put in place.

Here the logs requested by Andres:

ubuntu@maas:~$ maas admin machine get-curtin-config pctfwe
Success.
Machine-readable output follows:
apt:
  preserve_sources_list: false
  primary:
  - arches:
    - default
    uri: http://archive.ubuntu.com/ubuntu
  proxy: xxxxxxxx
  security:
  - arches:
    - default
    uri: http://archive.ubuntu.com/ubuntu
cloudconfig:
  maas-cloud-config:
    content: xxxxxxxxxx
    path: /etc/cloud/cloud.cfg.d/90_maas_cloud_config.cfg
  maas-datasource:
    content: 'datasource_list: [ MAAS ]'
    path: /etc/cloud/cloud.cfg.d/90_maas_datasource.cfg
  maas-reporting:
    content: xxxxxxxxx
    path: /etc/cloud/cloud.cfg.d/90_maas_cloud_init_reporting.cfg
  maas-ubuntu-sso:
    content: '#cloud-config

      snappy: {email: <email address hidden>}

      '
    path: /etc/cloud/cloud.cfg.d/90_maas_ubuntu_sso.cfg
debconf_selections:
  grub2: grub2 grub2/update_nvram boolean false
  maas: 'cloud-init cloud-init/datasources multiselect MAAS

    cloud-init cloud-init/maas-metadata-url string http://xxxxxxxxxx/MAAS/metadata/
xxxxxx

    cloud-init cloud-init/local-cloud-config string apt:\n preserve_sources_list:
    false\n primary:\n - arches: [default]\n uri: http://archive.ubuntu.com/ubuntu\n proxy:
    http://xxxxxxxx:8000/\n security:\n - arches: [default]\n uri: http://archive.ubuntu.com/ubuntu\napt_preserve_sources_list:
    true\napt_proxy: http://146.118.53.253:8000/\nmanage_etc_hosts: false\nmanual_cache_clean:
    true\nreporting:\n maas: {consumer_key: xxxxxxxxx, endpoint: ''http://xxxxxxx5240/MAAS/metadata/status/pctfwe'',\n token_key:
    xxxxxxxx, token_secret: xxxxxxx,\n type:
    webhook}\nsystem_info:\n package_mirrors:\n - arches: [i386, amd64]\n failsafe:
    {primary: ''http://archive.ubuntu.com/ubuntu'', security: ''http://security.ubuntu.com/ubuntu''}\n search:\n primary:
    [''http://archive.ubuntu.com/ubuntu'']\n security: [''http://archive.ubuntu.com/ubuntu'']\n -
    arches: [default]\n failsafe: {primary: ''http://ports.ubuntu.com/ubuntu-ports'',
    security: ''http://ports.ubuntu.com/ubuntu-ports''}\n search:\n primary:
    [''http://ports.ubuntu.com/ubuntu-ports'']\n security: [''http://ports.ubuntu.com/ubuntu-ports'']\n

    '
early_commands:
  driver_00:
  - sh
  - -c
  - echo third party drivers not installed or necessary.
install:
  error_tarfile: /tmp/curtin-logs.tar
  log_file: /tmp/install.log
  post_files:
  - /tmp/install.log
  - /tmp/curtin-logs.tar
kernel:
  mapping: {}
  package: linux-signed-generic
late_commands:
  maas:
  - wget
  - --no-proxy
  - xxxxxx
  - --post-data
  - op=netboot_off
  - -O
  - /dev/null
  repos_conf:
  - curtin
  - in-target
  - --
  - sh
  - -c
  - git clone xxxxxxt
    /tmp/cloud-baremetal-provisioning
  zone_script:
  - curtin
  - in-target
  - --
  - sh
  - -c
  - /bin/bash xxxx
    -r xenial
network:
  config:
  - id: enp0s29u1u1u5
    mac_address: 42:f2:e9:75:6c:87
    mtu: 1500
    name: enp0s29u1u1u5
    subnets:
    - type: manual
    type: physical
  - id: enp10s0
    mac_addr...

On Tue, Nov 20, 2018 at 1:30 AM Luca Cervigni
<email address hidden> wrote:
>
> Ryan,
> I can understand that, but if the case some form of variable check has to be put in place.

I was just confirming the behavior you're seeing and that it is a bug
that we need to fix.
It's fine for MAAS to allow you to name the raid array; the installer
code (curtin) will need to
handle the non mdX names and specify the path correctly.

[Expired for curtin because there has been no activity for 60 days.]

This bug is fixed with commit a0ede163 to curtin on branch master.
To view that commit see the following URL:
https://git.launchpad.net/curtin/commit/?id=a0ede163