Fix missing module name
Details
Details
- Reviewers
- None
test suite passes
Diff Detail
Diff Detail
- Repository
- rLTRN libtaskotron
- Branch
- feature/kojifix
- Lint
No Linters Available - Unit
No Unit Test Coverage
Fix missing module name
test suite passes
| No Linters Available |
| No Unit Test Coverage |
| Path | Packages | |||
|---|---|---|---|---|
| M | libtaskotron/koji_utils.py (4 lines) |
| Commit | Tree | Parents | Author | Summary | Date |
|---|---|---|---|---|---|
| 3c01805f7b3d | 9cb82c306442 | b420fe644aea | Kamil Páral | koji_utils: fix undefined name error (Show More…) | Jun 23 2014, 1:08 PM |
| Show All 18 Lines | |||||
| 19 | from libtaskotron import config | 19 | from libtaskotron import config | ||
| 20 | 20 | | |||
| 21 | 21 | | |||
| 22 | class KojiClient(object): | 22 | class KojiClient(object): | ||
| 23 | '''Helper Koji methods. | 23 | '''Helper Koji methods. | ||
| 24 | 24 | | |||
| 25 | :ivar koji.ClientSession session: Koji client session | 25 | :ivar koji.ClientSession session: Koji client session | ||
| 26 | ''' | 26 | ''' | ||
| 27 | | ||||
| 27 | def __init__(self, koji_session=None): | 28 | def __init__(self, koji_session=None): | ||
| 28 | '''Create a new KojiClient | 29 | '''Create a new KojiClient | ||
| 29 | 30 | | |||
| 30 | :param koji_session: an existing Koji session instance or ``None`` if | 31 | :param koji_session: an existing Koji session instance or ``None`` if | ||
| 31 | you want a new default session to be created | 32 | you want a new default session to be created | ||
| 32 | :type koji_session: :class:`koji.ClientSession` | 33 | :type koji_session: :class:`koji.ClientSession` | ||
| 33 | ''' | 34 | ''' | ||
| 34 | self.session = (koji_session or | 35 | self.session = (koji_session or | ||
| 35 | koji.ClientSession(config.get_config().koji_url)) | 36 | koji.ClientSession(config.get_config().koji_url)) | ||
| 36 | self._rpm_to_build_cache = {} | 37 | self._rpm_to_build_cache = {} | ||
| 37 | 38 | | |||
| 39 | | ||||
| 38 | def rpm_to_build(self, rpm, prefetch=True): | 40 | def rpm_to_build(self, rpm, prefetch=True): | ||
| 39 | '''Get koji build object for the rpm. | 41 | '''Get koji build object for the rpm. | ||
| 40 | 42 | | |||
| 41 | :param str rpm: filename as either `/my/path/nvr.a.rpm` or `nvr.a.rpm` | 43 | :param str rpm: filename as either `/my/path/nvr.a.rpm` or `nvr.a.rpm` | ||
| 42 | :param bool prefetch: if set to True, get list of all rpms for the build | 44 | :param bool prefetch: if set to True, get list of all rpms for the build | ||
| 43 | and store it for future use - this will speed up subsequent queries | 45 | and store it for future use - this will speed up subsequent queries | ||
| 44 | for rpms belonging to already queried build. | 46 | for rpms belonging to already queried build. | ||
| 45 | :rtype: bunch | 47 | :rtype: bunch | ||
| ▲ Show 20 Lines • Show All 89 Lines • ▼ Show 20 Line(s) | 136 | if not os.path.isdir(rpm_dir): | |||
| 135 | raise exc.TaskotronRemoteError( | 137 | raise exc.TaskotronRemoteError( | ||
| 136 | "Can't create directory: %r It is an already " | 138 | "Can't create directory: %r It is an already " | ||
| 137 | "existing file.", rpm_dir) | 139 | "existing file.", rpm_dir) | ||
| 138 | else: | 140 | else: | ||
| 139 | try: | 141 | try: | ||
| 140 | file_utils.makedirs(rpm_dir) | 142 | file_utils.makedirs(rpm_dir) | ||
| 141 | except OSError, e: | 143 | except OSError, e: | ||
| 142 | log.exception("Can't create directory: %s", rpm_dir) | 144 | log.exception("Can't create directory: %s", rpm_dir) | ||
| 143 | raise TaskotronRemoteError(e) | 145 | raise exc.TaskotronRemoteError(e) | ||
| 144 | 146 | | |||
| 145 | 147 | | |||
| 146 | rpm_urls = self.nvr_to_urls(nvr, | 148 | rpm_urls = self.nvr_to_urls(nvr, | ||
| 147 | arches=arches, | 149 | arches=arches, | ||
| 148 | debuginfo=debuginfo, | 150 | debuginfo=debuginfo, | ||
| 149 | src=src) | 151 | src=src) | ||
| 150 | rpm_files = [] | 152 | rpm_files = [] | ||
| 151 | log.info('Fetching %s RPMs for: %s', len(rpm_urls), nvr) | 153 | log.info('Fetching %s RPMs for: %s', len(rpm_urls), nvr) | ||
| ▲ Show 20 Lines • Show All 55 Lines • Show Last 20 Lines | |||||