~kissiel/checkbox/job-unit-manpage-flags-update

« back to all changes in this revision

Viewing changes to plainbox/plainbox/impl/depmgr.py

  • Committer: Zygmunt Krynicki
  • Date: 2015-02-03 07:29:11 UTC
  • mto: This revision was merged to the branch mainline in revision 3564.
  • Revision ID: zygmunt.krynicki@canonical.com-20150203072911-tf6tcii5agm3fnjd
plainbox:depmgr: use job.id for logging

JobDefinition uses a rather verbose repr() that currently also defaults
to using the summary field. (aka human readable text). This causes
dependency analysis logs to be overly verbose. Since changing the repr()
of something as common as a job definition class would require many
fixes to unit tests it's cheaper to fix just this one place instead.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>

Show diffs side-by-side

added added

removed removed

Lines of Context:
259
259
        to be raised. Calls _visit recursively on all dependencies.
260
260
        """
261
261
        color = self._job_color_map[job.id]
262
 
        logger.debug(_("Visiting job %s (color %s)"), job, color)
 
262
        logger.debug(_("Visiting job %s (color %s)"), job.id, color)
263
263
        if color == self.COLOR_WHITE:
264
264
            # This node has not been visited yet. Let's mark it as GRAY (being
265
265
            # visited) and iterate through the list of dependencies