grub/grub.cfg is not world readable pxe fails

[Narinder Gupta]

after deployment of the Ironic conductor find out that /tftpboot/brub/grub.cfg file is not world readable causing the pxe boot the baremetal server to goto grub pompt.

After change permission to 644 of /tftpboot/brub/grub.cfg able to download grub and start pxe booting but charm reload it to 640 again. This request is to change the permission of grub.cfg to 644

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/855311

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/855311
Committed: https://opendev.org/openstack/charm-ironic-conductor/commit/546547c4fcbf9f9b1075953ea9051134381aef01
Submitter: "Zuul (22348)"
Branch: master

commit 546547c4fcbf9f9b1075953ea9051134381aef01
Author: Liam Young <email address hidden>
Date: Wed Aug 31 06:34:06 2022 +0000

    Make grub.cfg world readable

    Closes-Bug: #1988231
    Change-Id: Ib271f3a961e3a4b6ab7cd9dda6fbe9e2766452fc

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/yoga
Review: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/855366

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/855369

[Narinder Gupta (narindergupta)]

we would require fix i ussuri edge as well as we are deploying it from that channel.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/855366
Committed: https://opendev.org/openstack/charm-ironic-conductor/commit/584783b8c5e58c37961e6d5637bc10fbce2761c6
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 584783b8c5e58c37961e6d5637bc10fbce2761c6
Author: Liam Young <email address hidden>
Date: Wed Aug 31 06:34:06 2022 +0000

    Make grub.cfg world readable

    Closes-Bug: #1988231
    Change-Id: Ib271f3a961e3a4b6ab7cd9dda6fbe9e2766452fc
    (cherry picked from commit 546547c4fcbf9f9b1075953ea9051134381aef01)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/855369
Committed: https://opendev.org/openstack/charm-ironic-conductor/commit/eca09c0d8b65ad05e803e2014a405090d7ca60e5
Submitter: "Zuul (22348)"
Branch: stable/xena

commit eca09c0d8b65ad05e803e2014a405090d7ca60e5
Author: Liam Young <email address hidden>
Date: Wed Aug 31 13:33:48 2022 +0000

    Set perms on grub.cfg

    This is based on commits 546547c4 and a3b68bd8 from master.

    Closes-Bug: #1988231
    Change-Id: I20b9227bac52041a5620b00ad4d10e195a0caf57

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/872451

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/872452

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/872453

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/872452
Committed: https://opendev.org/openstack/charm-ironic-conductor/commit/4b3a5d8bc0ea0e626b5431f2b213c9c41d577286
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 4b3a5d8bc0ea0e626b5431f2b213c9c41d577286
Author: Liam Young <email address hidden>
Date: Wed Aug 31 13:33:48 2022 +0000

    Set perms on grub.cfg

    This is based on commits 546547c4 and a3b68bd8 from master.

    Closes-Bug: #1988231
    Change-Id: I20b9227bac52041a5620b00ad4d10e195a0caf57
    (cherry picked from commit eca09c0d8b65ad05e803e2014a405090d7ca60e5)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/872453
Committed: https://opendev.org/openstack/charm-ironic-conductor/commit/3687c915a42a3d374f6b822560a80312fc02f86a
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 3687c915a42a3d374f6b822560a80312fc02f86a
Author: Liam Young <email address hidden>
Date: Wed Aug 31 13:33:48 2022 +0000

    Set perms on grub.cfg

    This is based on commits 546547c4 and a3b68bd8 from master.

    Closes-Bug: #1988231
    Change-Id: I20b9227bac52041a5620b00ad4d10e195a0caf57
    (cherry picked from commit eca09c0d8b65ad05e803e2014a405090d7ca60e5)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/charm-ironic-conductor/+/872451
Committed: https://opendev.org/openstack/charm-ironic-conductor/commit/abe039924a9f9f7863bad0883cc5cad9e9f7cbf4
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit abe039924a9f9f7863bad0883cc5cad9e9f7cbf4
Author: Liam Young <email address hidden>
Date: Wed Aug 31 13:33:48 2022 +0000

    Set perms on grub.cfg

    This is based on commits 546547c4 and a3b68bd8 from master.

    Closes-Bug: #1988231
    Change-Id: I20b9227bac52041a5620b00ad4d10e195a0caf57
    (cherry picked from commit eca09c0d8b65ad05e803e2014a405090d7ca60e5)

[Felipe Reyes (freyes)]

Backports of this fix have been merged from Xena to Ussuri

Users running ussuri make sure to get charm's revision 64 to get the fix.

$ juju info --series focal --channel ussuri/stable ironic-conductor
name: ironic-conductor
...
channels: |
  ussuri/stable: 3687c91 2023-02-03 (64) 28MB
  ussuri/candidate: ↑
  ussuri/beta: ↑
  ussuri/edge: ↑