Closed
Bug 1036425
Opened 10 years ago
Closed 10 years ago
Minor release-runner.sh improvements
Categories
(Release Engineering :: Release Automation: Other, defect, P3)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Sylvestre, Assigned: Sylvestre)
Details
(Whiteboard: [shipit])
Attachments
(1 file, 2 obsolete files)
1.62 KB,
patch
|
rail
:
review+
Sylvestre
:
checked-in+
|
Details | Diff | Splinter Review |
I know it is a bit of a bikeshed but I like the script I use when they check their arguments. + the patch uses the "notify_to" value from the ini file instead of the hardcoded value.
Attachment #8453079 -
Flags: review?(bhearsum)
Comment 1•10 years ago
|
||
Comment on attachment 8453079 [details] [diff] [review] 0001-Check-the-input-arguments.patch Review of attachment 8453079 [details] [diff] [review]: ----------------------------------------------------------------- ::: buildfarm/release/release-runner.sh @@ +16,5 @@ > if [ -z "$LOGFILE" ]; then > LOGFILE=/var/log/supervisor/release-runner.log > fi > + > +touch $LOGFILE &> /dev/null I don't think this will work. The script runs as ctlbld, but the log file is owned by root. supervisord logs whatever the script prints to stdout/stderr, not the script itself.
Assignee | ||
Comment 2•10 years ago
|
||
Right. I am not checking that the directory of LOGFILE exists.
Attachment #8453079 -
Attachment is obsolete: true
Attachment #8453079 -
Flags: review?(bhearsum)
Attachment #8454408 -
Flags: review?(rail)
Comment 3•10 years ago
|
||
Comment on attachment 8454408 [details] [diff] [review] 0001-Check-the-input-arguments.patch Review of attachment 8454408 [details] [diff] [review]: ----------------------------------------------------------------- ::: buildfarm/release/release-runner.sh @@ +39,4 @@ > > # Sleep time after a failure, in seconds. > SLEEP_TIME=60 > +NOTIFY_TO=$(grep "notify_to:" $CONFIG|sed -e "s|.*<\(.*\)>|\1|g") Can you test if the value is empty and fallback to release@ if it's empty?
Attachment #8454408 -
Flags: review?(rail) → review+
Comment 5•10 years ago
|
||
Comment on attachment 8454425 [details] [diff] [review] 0001-Check-the-input-arguments.patch Feel free to land it anytime. Just ping me to deploy it.
Attachment #8454425 -
Flags: review+
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8454425 [details] [diff] [review] 0001-Check-the-input-arguments.patch Checked in: https://hg.mozilla.org/build/tools/rev/7f3a0e70a2ab Deploy when you want. It should not change anything on your side.
Attachment #8454425 -
Flags: checked-in+
Updated•10 years ago
|
Assignee: nobody → sledru
Comment 7•10 years ago
|
||
Looks like this was deployed already.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•