Project

General

Profile

Bug #32372

Wrong destination path for pre section logs in kickstart_default template

Added by Martin Schlossarek 2 months ago. Updated 22 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Martin Schlossarek
Category:
Templates
Target version:
-
Difficulty:
trivial
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/8457, https://github.com/theforeman/foreman/pull/8478, https://github.com/theforeman/foreman/pull/8462
Fixed in Releases:
2.4.1, 2.5.0
Found in Releases:
2.4.0

Description

Copying the pre section logs (/tmp/*pre*log) to root of sysimage fails because of a wrong destination path:

cp -vf /tmp/*pre*log /tmp/sysimage/root/

/tmp/sysimage does not exist. It should be /mnt/sysimage.

This was introduced in #28521.

Related issues

Related to Foreman - Bug #28521: Default kickstart places log to /mnt/sysimage/root/install.post.logClosed

Associated revisions

Revision ceb276fb (diff)
Added by Martin Schlossarek 2 months ago

Fixes #32372 - Correct path for logs in kickstart

bb4633bcd4facc339a8099f4a8e89312323d8b61 refactored the kickstart but
used an incorrect path for logs in the %pre section. This corrects it.

Revision 4a720456 (diff)
Added by Martin Schlossarek 2 months ago

Refs #32372 - correct remaining paths

bb4633bcd4facc339a8099f4a8e89312323d8b61 refactored the atomic kickstart but
used an incorrect path for logs in the %pre section. This corrects it.

History

#1 Updated by The Foreman Bot 2 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8457 added

#2 Updated by Ewoud Kohl van Wijngaarden 2 months ago

  • Triaged changed from No to Yes
  • Found in Releases 2.4.0 added

#3 Updated by Ewoud Kohl van Wijngaarden 2 months ago

  • Related to Bug #28521: Default kickstart places log to /mnt/sysimage/root/install.post.log added

#4 Updated by The Foreman Bot 2 months ago

  • Fixed in Releases 2.5.0 added

#5 Updated by Anonymous 2 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Lukas Zapletal 2 months ago

  • Status changed from Closed to New

There are two more occurrences of /tmp/sysimage so reopening to keep track of this. A customer just reported this today.

#7 Updated by The Foreman Bot 2 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8462 added

#8 Updated by Martin Schlossarek 2 months ago

Lukas Zapletal wrote:

There are two more occurrences of /tmp/sysimage so reopening to keep track of this. A customer just reported this today.

I also fixed the other two occurrences.

Actually I reported something different, but I also stumbled upon these incorrect paths. Don't know why the Red Hat Support refers to this issue now...

Kind regards,
the customer ;)

#9 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/foreman/pull/8478 added

#10 Updated by Tomer Brisker about 2 months ago

  • Status changed from Ready For Testing to Closed

#11 Updated by Amit Upadhye 22 days ago

  • Fixed in Releases 2.4.1 added

Also available in: Atom PDF