Make a generic logger instead of the nil logger on dependent update

[ggaaooppeenngg]

Signed-off-by: Peng Gao peng.gao.dut@gmail.com

What this PR does / why we need it:

Make a temporary logger for generic object instead of being nil.

Which issue(s) this PR fixes

Fixes #1679 #1678

Checklist:

• Docs included if any changes are user facing

[coveralls]

Pull Request Test Coverage Report for Build 3392042555

• 0 of 30 (0.0%) changed or added relevant lines in 5 files are covered.
• 3 unchanged lines in 1 file lost coverage.
• Overall coverage increased (+0.3%) to 39.932%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pkg/controller.v1/mxnet/mxjob_controller.go	0	3	0.0%
pkg/controller.v1/pytorch/pytorchjob_controller.go	0	3	0.0%
pkg/controller.v1/tensorflow/tfjob_controller.go	0	3	0.0%
pkg/controller.v1/xgboost/xgboostjob_controller.go	0	3	0.0%
pkg/common/util/reconciler.go	0	18	0.0%

Files with Coverage Reduction	New Missed Lines	%
pkg/common/util/reconciler.go	3	16.5%

Totals
Change from base Build 3390627896:	0.3%
Covered Lines:	2366
Relevant Lines:	5925

---

💛 - Coveralls

[djwhatle]

This resolved the issue scheduling TFJobs with volcano for me!

[ggaaooppeenngg]

This pr fixes the previous wrong pr #1666 and uses generic functions for the podgroup callbacks. This pr also adds a test for the XXXFunc which only handles pod and service objects instead of other generic objects. @johnugeorge @D0m021ng @Crazybean-lwb

[johnugeorge]

/lgtm
/approve

[google-oss-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ggaaooppeenngg, johnugeorge

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [johnugeorge]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[johnugeorge]

Dynamic watcher ref: #1688