ManageIQ / manageiq-appliance-build

Scripts to build ManageIQ appliances
Apache License 2.0
10 stars 55 forks source link

Fix log file names for release build script #478

Closed bdunne closed 3 years ago

bdunne commented 3 years ago

Currently:

[root@build-lasker manageiq-appliance-build]# /build/manageiq-appliance-build/bin/release-build.sh lasker-1-beta1
lasker-1-beta1 release build kicked off, see logs @ /build/logs/lasker-1-beta1*.log ...

But in the log directory:

-rw-r--r--.  1 root root   135657 Apr 20 17:28 _rpm.log
simaishi commented 3 years ago

Oops.. sorry, I introduced this bug in https://github.com/ManageIQ/manageiq-appliance-build/pull/457...

miq-bot commented 3 years ago

Checked commit https://github.com/bdunne/manageiq-appliance-build/commit/1c699a4006d70c46723cc151d04b49b9fd73cb66 with ruby 2.6.3, rubocop 0.82.0, haml-lint 0.35.0, and yamllint 0 files checked, 0 offenses detected Everything looks fine. :cookie:

Fryguy commented 3 years ago

Backported to lasker in commit 7ef3f29a090eb1506372805d4ff566ac9978356f.

commit 7ef3f29a090eb1506372805d4ff566ac9978356f
Author: Keenan Brock <keenan@thebrocks.net>
Date:   Tue Apr 20 14:00:27 2021 -0400

    Merge pull request #478 from bdunne/fix_log_names

    Fix log file names for release build script

    (cherry picked from commit 8f5b3d77e32c8310a6c4211f361ae472d17303b8)