Merge lp:~psivaa/auto-package-testing/rc-file-and-debug-log into lp:~canonical-ci-engineering/auto-package-testing/add-boottest-requests

Proposed by Para Siva
Status: Merged
Merged at revision: 398
Proposed branch: lp:~psivaa/auto-package-testing/rc-file-and-debug-log
Merge into: lp:~canonical-ci-engineering/auto-package-testing/add-boottest-requests
Diff against target: 50 lines (+11/-2)
3 files modified
etc/boottest-britney.rc (+7/-0)
jenkins/aptcache.py (+3/-2)
jenkins/run-boottest-jenkins.py (+1/-0)
To merge this branch: bzr merge lp:~psivaa/auto-package-testing/rc-file-and-debug-log
Reviewer Review Type Date Requested Status
Vincent Ladeuil (community) Approve
Review via email: mp+249165@code.launchpad.net

Commit message

Adding the rc file specific to CI deployment in tachash. and logging improvement

Description of the change

This is still work in progress, but proposing just to make sure all of us and tachash are on the same page. The rc file is needed with the right information specific to tachash as /var/lib/jenkins/CI/boottest-britney.rc.

To post a comment you must log in.
399. By Para Siva

Fixing the cache updating issue, looking at the right timestamp file and making it sure we open it to update the timestamp

Revision history for this message
Vincent Ladeuil (vila) wrote :

Well done !

review: Approve
400. By Para Siva

Removing defuct entry in rc file

Revision history for this message
Vincent Ladeuil (vila) wrote :

Thanks, sorry for mentioning that issue with the dead entry on irc instead of here.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'etc/boottest-britney.rc'
2--- etc/boottest-britney.rc 1970-01-01 00:00:00 +0000
3+++ etc/boottest-britney.rc 2015-02-10 11:07:00 +0000
4@@ -0,0 +1,7 @@
5+# Default configuration file for adt-britney
6+# This file should be placed under
7+# /var/lib/jenkins/CI/
8+aptroot: /var/lib/jenkins/CI/boottest/aptroot
9+apturi: http://ftpmaster.internal/ubuntu/
10+components: main restricted universe multiverse
11+datadir: /var/local/boottest/
12
13=== modified file 'jenkins/aptcache.py'
14--- jenkins/aptcache.py 2014-04-17 09:56:58 +0000
15+++ jenkins/aptcache.py 2015-02-10 11:07:00 +0000
16@@ -54,12 +54,12 @@
17 self.proposed = False
18
19 self.timestampfile = os.path.normpath(
20- os.path.join(rootdir, 'var/cache/apt/pkgcache.bin'))
21+ os.path.join(rootdir, 'var/cache/apt/cache.timestamp'))
22
23 super(AptCache, self).__init__(rootdir=rootdir, memonly=False)
24 self.rootdir = rootdir
25
26- def update(self, cache_age=60, force_update=False):
27+ def update(self, cache_age=10, force_update=False):
28 ''' Equivalent of apt-get update
29
30 :param cache_age: Update the cache when it is older than
31@@ -78,6 +78,7 @@
32 not os.path.exists(self.timestampfile)) or (
33 (currtime - timestamp) > cache_age * 60):
34 logging.debug('Updating cache')
35+ open(self.timestampfile, 'w').close()
36 super(AptCache, self).update()
37 super(AptCache, self).open()
38 else:
39
40=== modified file 'jenkins/run-boottest-jenkins.py'
41--- jenkins/run-boottest-jenkins.py 2015-01-29 22:15:22 +0000
42+++ jenkins/run-boottest-jenkins.py 2015-02-10 11:07:00 +0000
43@@ -174,6 +174,7 @@
44 out,
45 err))
46 log.error(msg)
47+ log.info(err)
48
49 copy_results(log, release)
50 log.info("===== Done =====")

Subscribers

People subscribed via source and target branches