missing consol logging makes bmh provisioning fail

[knfoo]

After upgrade to v0.4.1 the bmh no longer logs syslog to the console during the different phases.
It seems to do with this: #160
This for some odd reason has the effect on my baremetal nodes (Hauwei ibmc managed, UEFI and Legacy bootmode) that it fails in random phases like clean wait, inspection wait.
I am not able to get anything to provision the servers as they all get stuck in clean wait.
After help from the community slack https://kubernetes.slack.com/archives/CHD49TLE7/p1606294233210600
I was able to get syslog to consol re-enabled in the ironic-image.

What did to get it back in working state:
I added ssh-debugging in order to get in the the image and try to run some debugging steps to figure out what was wrong
#226

Then I rebuild the ironic-image to get the required parameters included in the inspector.ipxe.j2- I have this file do to the work in the #226 - and the ironic.conf.j2 and enabled the systemd.journald.forward_to_console=yes

This getting the console logging back I am now able to get the bmh to behave as expected, from inspection to full deployment.

At least for me having the possibility to switch on console logging via a env variable would work better than having to hack the image on every new release.

[knfoo]

The changes I did is available in this fork
https://github.com/knfoo/ironic-image

[derekhiggins]

I've updated #226 to remove console=ttyS0 form the kernel, it can now instead be configured.

[knfoo]

That sounds great

[fmuyassarov]

/kind feature

[metal3-io-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues will close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle stale

[metal3-io-bot]

Stale issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle stale.

/close

[metal3-io-bot]

@metal3-io-bot: Closing this issue.

In response to this:

Stale issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle stale.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.