On Tue, Nov 03, 2020 at 07:17:16PM +0800, Wei Jihui wrote:
[Why] if we add new field in job yaml like this: on_fail: sleep: 300 if will do not add sleep to pp
it will not add sleep to pp.
[How] when add sleep to pp, we check sleep's parent hash, not all job. it will find sleep in on_fail'keys.
Signed-off-by: Wei Jihui weijihuiall@163.com
lib/job.rb | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/lib/job.rb b/lib/job.rb index 5c2c9517..50323d31 100755 --- a/lib/job.rb +++ b/lib/job.rb @@ -576,10 +576,10 @@ class Job @job[comment_to_symbol('auto generated by add_pp()')] = nil @job["pp"] = Hash.new() init_program_options()
- for_each_in(@job, @referenced_programs.keys) do |_pk, h, p_n, p_args|
k = hash_key_re_string(@job, p_n, @job['pp'].keys)if k && !@job[k].is_a?(Hash)@job['pp'][k] = @job[k]
- for_each_in(@job, @referenced_programs.keys) do |_pk, h, p_n, _p_args|
k = hash_key_re_string(h, p_n, @job['pp'].keys)if k && !h[k].is_a?(Hash) && !h[k].nil?
!h[k].nil? is awkward, how about just use h[k], and h[k] should be judged nil before 'is_a': if k && h[k] && !h[k].is_a?(Hash)
Thanks, Xijian