leapp upgrades with kernel args is not working as expected

Bug #1890080 reported by Saravanan KR
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Saravanan KR

Bug Description

During leapp upgrade the grub file is updated wrongly as below:

[root@overcloud-computeovsdpdk-0 ~]# cat /etc/default/grub
GRUB_TIMEOUT=5
GRUB_DISTRIBUTOR="$(sed 's, release .*$,,g' /etc/system-release)"
GRUB_DEFAULT=saved
GRUB_DISABLE_SUBMENU=true
GRUB_TERMINAL_OUTPUT="console"
GRUB_CMDLINE_LINUX="console=tty0 console=ttyS0,115200n8 crashkernel=auto rhgb quiet <<<<<---- Closing quote missing after leapp upgrade
TRIPLEO_HEAT_TEMPLATE_KERNEL_ARGS= default_hugepagesz=1GB hugepagesz=1G hugepages=20 iommu=pt intel_iommu=on isolcpus=2-11" <<<<<---- opening quote missing after leapp upgrade.
GRUB_CMDLINE_LINUX="${GRUB_CMDLINE_LINUX:+$GRUB_CMDLINE_LINUX }${TRIPLEO_HEAT_TEMPLATE_KERNEL_ARGS}"
GRUB_DISABLE_RECOVERY="true"
GRUB_CMDLINE_LINUX_DEFAULT="${GRUB_CMDLINE_LINUX_DEFAULT:+$GRUB_CMDLINE_LINUX_DEFAULT }\$tuned_params"
GRUB_INITRD_OVERLAY="${GRUB_INITRD_OVERLAY:+$GRUB_INITRD_OVERLAY }\$tuned_initrd"
GRUB_ENABLE_BLSCFG=true

This is beacause, leapp does not consider any entry starting other than GRUB.

Saravanan KR (skramaja)
Changed in tripleo:
importance: Undecided → High
assignee: nobody → Saravanan KR (skramaja)
milestone: none → victoria-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-upgrade (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/744388

Changed in tripleo:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-upgrade (master)

Reviewed: https://review.opendev.org/744388
Committed: https://git.openstack.org/cgit/openstack/tripleo-upgrade/commit/?id=e5197c4469c7d85c86a8c0f73638bc3d44f919e3
Submitter: Zuul
Branch: master

commit e5197c4469c7d85c86a8c0f73638bc3d44f919e3
Author: Saravanan KR <email address hidden>
Date: Sun Aug 2 16:21:06 2020 +0530

    Add BootParams service to all roles

    BootParams service should be added to all the roles during
    the ugprade. It is responsible to set the kernel args and
    reboot if required.
    Related-Bug: #1890080

    Change-Id: Id68c62486e8b12e21931a4255da48effb0ed4690

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.opendev.org/742625
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=f217eccc7720250a64a0877061063acf0951b495
Submitter: Zuul
Branch: master

commit f217eccc7720250a64a0877061063acf0951b495
Author: Saravanan KR <email address hidden>
Date: Thu Jul 23 14:38:25 2020 +0530

    Align kernel args for system upgrade using leapp

    leapp is looking for errors in /etc/default/grub and
    fixes it during upgrade. It does not recognize entries
    starting other than "GRUB", as tripleo sets the entry
    starting with TRIPLEO_ instead of GRUB, append the
    entry with GRUB_ so that leapp upgrades correctly.
    Closes-Bug: #1890080
    Change-Id: I9238cd35a1114b1649a38ab4f24225865bcecf19

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/745058

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/745059

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-upgrade (stable/ussuri)

Related fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/745060

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-upgrade (stable/train)

Related fix proposed to branch: stable/train
Review: https://review.opendev.org/745061

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/ussuri)

Reviewed: https://review.opendev.org/745058
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=28ed5c0b83f594dae6cfdc266648a64e852adbab
Submitter: Zuul
Branch: stable/ussuri

commit 28ed5c0b83f594dae6cfdc266648a64e852adbab
Author: Saravanan KR <email address hidden>
Date: Thu Jul 23 14:38:25 2020 +0530

    Align kernel args for system upgrade using leapp

    leapp is looking for errors in /etc/default/grub and
    fixes it during upgrade. It does not recognize entries
    starting other than "GRUB", as tripleo sets the entry
    starting with TRIPLEO_ instead of GRUB, append the
    entry with GRUB_ so that leapp upgrades correctly.
    Closes-Bug: #1890080
    Change-Id: I9238cd35a1114b1649a38ab4f24225865bcecf19
    (cherry picked from commit f217eccc7720250a64a0877061063acf0951b495)

tags: added: in-stable-ussuri
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/train)

Reviewed: https://review.opendev.org/745059
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=2f382ac90a635e8f5d0f2244be4b18893ec82202
Submitter: Zuul
Branch: stable/train

commit 2f382ac90a635e8f5d0f2244be4b18893ec82202
Author: Saravanan KR <email address hidden>
Date: Thu Jul 23 14:38:25 2020 +0530

    Align kernel args for system upgrade using leapp

    leapp is looking for errors in /etc/default/grub and
    fixes it during upgrade. It does not recognize entries
    starting other than "GRUB", as tripleo sets the entry
    starting with TRIPLEO_ instead of GRUB, append the
    entry with GRUB_ so that leapp upgrades correctly.
    Closes-Bug: #1890080
    Change-Id: I9238cd35a1114b1649a38ab4f24225865bcecf19
    (cherry picked from commit f217eccc7720250a64a0877061063acf0951b495)

tags: added: in-stable-train
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-upgrade (stable/ussuri)

Reviewed: https://review.opendev.org/745060
Committed: https://git.openstack.org/cgit/openstack/tripleo-upgrade/commit/?id=8ed2b58d39d103fd9c7ea03d80a771d89a50c518
Submitter: Zuul
Branch: stable/ussuri

commit 8ed2b58d39d103fd9c7ea03d80a771d89a50c518
Author: Saravanan KR <email address hidden>
Date: Sun Aug 2 16:21:06 2020 +0530

    Add BootParams service to all roles

    BootParams service should be added to all the roles during
    the ugprade. It is responsible to set the kernel args and
    reboot if required.
    Related-Bug: #1890080

    Change-Id: Id68c62486e8b12e21931a4255da48effb0ed4690
    (cherry picked from commit e5197c4469c7d85c86a8c0f73638bc3d44f919e3)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-upgrade (stable/train)

Reviewed: https://review.opendev.org/745061
Committed: https://git.openstack.org/cgit/openstack/tripleo-upgrade/commit/?id=51429783dac83feadb72561aa6caa8da970b0001
Submitter: Zuul
Branch: stable/train

commit 51429783dac83feadb72561aa6caa8da970b0001
Author: Saravanan KR <email address hidden>
Date: Sun Aug 2 16:21:06 2020 +0530

    Add BootParams service to all roles

    BootParams service should be added to all the roles during
    the ugprade. It is responsible to set the kernel args and
    reboot if required.
    Related-Bug: #1890080

    Change-Id: Id68c62486e8b12e21931a4255da48effb0ed4690
    (cherry picked from commit e5197c4469c7d85c86a8c0f73638bc3d44f919e3)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 11.4.0

This issue was fixed in the openstack/tripleo-heat-templates 11.4.0 release.

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.