autotest: Mark internal tko/db functions private.
BUG=chromium:833988
TEST=None
Change-Id: I373626ff1f1f567f0608edab7cac4e74f6675329
Reviewed-on: https://chromium-review.googlesource.com/1018539
Commit-Ready: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>
diff --git a/tko/db.py b/tko/db.py
index 4400abf..17262c2 100644
--- a/tko/db.py
+++ b/tko/db.py
@@ -443,13 +443,13 @@
@return The dict of data inserted into the tko_jobs table.
"""
- job.machine_idx = self.lookup_machine(job.machine)
+ job.machine_idx = self._lookup_machine(job.machine)
if not job.machine_idx:
- job.machine_idx = self.insert_machine(job, commit=commit)
+ job.machine_idx = self._insert_machine(job, commit=commit)
elif job.machine:
# Only try to update tko_machines record if machine is set. This
# prevents unnecessary db writes for suite jobs.
- self.update_machine_information(job, commit=commit)
+ self._update_machine_information(job, commit=commit)
afe_job_id = utils.get_afe_job_id(tag)
@@ -590,7 +590,7 @@
return {'hostname': hostname, 'machine_group': group, 'owner': owner}
- def insert_machine(self, job, commit = None):
+ def _insert_machine(self, job, commit = None):
"""Inserts the job machine.
@param job: The job object.
@@ -601,7 +601,7 @@
return self.get_last_autonumber_value()
- def update_machine_information(self, job, commit = None):
+ def _update_machine_information(self, job, commit = None):
"""Updates the job machine information.
@param job: The job object.
@@ -613,7 +613,7 @@
commit=commit)
- def lookup_machine(self, hostname):
+ def _lookup_machine(self, hostname):
"""Look up the machine information.
@param hostname: The hostname as string.