autoqa-devel April 2010

autoqa-devel@lists.fedorahosted.org

7 participants
65 discussions

[AutoQA] #143: ResultsDB: create resultsdb prototype - create XMLRPC based interface
by fedora-badges 28 Apr '10

28 Apr '10

#143: ResultsDB: create resultsdb prototype - create XMLRPC based interface ----------------------------+----------------------------------------------- Reporter: jskladan | Owner: jskladan Type: task | Status: new Priority: major | Milestone: Resultdb Component: infrastructure | Version: 1.0 Keywords: | ----------------------------+----------------------------------------------- sub-task for #134. Create XMLRPC based interface implementing API from #131 and using the database from #142 as backend -- Ticket URL: <https://fedorahosted.org/autoqa/ticket/143> AutoQA <http://autoqa.fedorahosted.org> Automated QA project

1 3

[AutoQA] #152: pst: --yum and --rpm options are confusing
by fedora-badges 27 Apr '10

27 Apr '10

#152: pst: --yum and --rpm options are confusing -------------------------+-------------------------------------------------- Reporter: kparal | Owner: psss Type: enhancement | Status: new Priority: major | Milestone: Package Sanity Tests Component: tests | Version: 1.0 Keywords: | -------------------------+-------------------------------------------------- From pst --help: {{{ Modes: -r --rpm test local update (provide packages with full path) -y --yum update from the updates-testing repo (NVRs only) }}} I would expect that --rpm means "use rpm for sanity testing" and --yum means "use yum for sanity testing". But no, yum is always used. This just denotes whether the packages come from local filesystem or from repository. That confused me a little. So I propose to change it like this:[[BR]] 1. Lets get rid of --rpm and --yum altogether[[BR]] 2. We can distinguish local files and repository packages by:[[BR]] 2a) local files end in ".rpm", repository packages don't[[BR]] OR[[BR]] 2b) simply suppose the argument is local file and test its presence - if it is not available, it must be a repository package That should simplify things a little from an end-user perspective. What do you think, Petr? PS: Is full path to local packages really necessary? Isn't relative path enough? -- Ticket URL: <https://fedorahosted.org/autoqa/ticket/152> AutoQA <http://autoqa.fedorahosted.org> Automated QA project

1 0

[AutoQA] #147: pst: Delete phase fails
by fedora-badges 27 Apr '10

27 Apr '10

#147: pst: Delete phase fails --------------------+------------------------------------------------------- Reporter: kparal | Owner: psss Type: defect | Status: new Priority: major | Milestone: Package Sanity Tests Component: tests | Version: 1.0 Keywords: | --------------------+------------------------------------------------------- From unknown reason the Delete phase failed for me. Please see attached log. -- Ticket URL: <https://fedorahosted.org/autoqa/ticket/147> AutoQA <http://autoqa.fedorahosted.org> Automated QA project

1 5

[AutoQA] #141: pst: Blacklisting some packages
by fedora-badges 26 Apr '10

26 Apr '10

#141: pst: Blacklisting some packages --------------------+------------------------------------------------------- Reporter: kparal | Owner: psss Type: task | Status: new Priority: major | Milestone: Package Sanity Tests Component: tests | Version: 1.0 Keywords: pst | --------------------+------------------------------------------------------- Because we use some necessary tools for sanity testing, we won't be probably able to test some packages with it. For example, can we test sanity of 'yum' or 'rpm' packages, when we are using those exact commands to test it? If not, we will need to blacklist them. So the first task is to collect a list of packages we can't test. The second task is to think about what to do with it. Will we just skip them, or will we invent an alternative way to test them somehow? -- Ticket URL: <https://fedorahosted.org/autoqa/ticket/141> AutoQA <http://autoqa.fedorahosted.org> Automated QA project

1 3

[AutoQA] #148: rpmguard: Filter out messages varying only in package version
by fedora-badges 23 Apr '10

23 Apr '10

#148: rpmguard: Filter out messages varying only in package version -------------------------+-------------------------------------------------- Reporter: kparal | Owner: Type: enhancement | Status: new Priority: major | Milestone: Package update tests Component: tests | Version: 1.0 Keywords: | -------------------------+-------------------------------------------------- Many tests containing file path may differ only in directory name containing package version, still they are reported as changed. This could be taken into account and filtered out. This also applies to names of requirements and provisions. Read more here: http://lists.fedoraproject.org/pipermail/test/2010-April/090341.html -- Ticket URL: <https://fedorahosted.org/autoqa/ticket/148> AutoQA <http://autoqa.fedorahosted.org> Automated QA project

1 1

[PATCH] 65, 130 - updated test templates and current tests
by Josef Skladanka 23 Apr '10

23 Apr '10

First patch updates the test_class templates so they have the self.autotest_url variable created during initialise phase. Second patch updates current tests, so they also take advantage from this functionality. J.

1 0

[PATCH 1/2] Ticket 65, 130: getting autotest test-ID to the client
by Josef Skladanka 23 Apr '10

23 Apr '10

Added new option to autoqa --autotest-server which can be used to set hostname of the autotest server, which will hold the results of the test. Since our infrastructure has Autotest and AutoQA on the same machine, default value for this option is the hostname of current machine (socket.gethostname()). Also the templates of control were updated, so they propagate the basic URL to the Autotest results directory. Test_class_templates were updated, so they create the whole URL to the results. --- autoqa | 6 ++++++ hooks/post-koji-build/control.template | 3 ++- hooks/post-koji-build/test_class_template.py | 5 ++++- hooks/post-repo-update/control.template | 3 ++- hooks/post-repo-update/test_class_template.py | 5 ++++- hooks/post-tree-compose/control.template | 3 ++- hooks/post-tree-compose/test_class_template.py | 5 ++++- 7 files changed, 24 insertions(+), 6 deletions(-) diff --git a/autoqa b/autoqa index efb94e3..9f41c00 100755 --- a/autoqa +++ b/autoqa @@ -27,6 +27,7 @@ import optparse import tempfile import StringIO import urlgrabber +import socket from ConfigParser import * from subprocess import call @@ -139,6 +140,9 @@ parser.add_option('--local', action='store_true', dest='local', help='Do not schedule jobs - run test(s) directly on the local machine') parser.add_option('-l', '--list-tests', action='store_true', dest='listtests', help='list the tests for the given hookname - do not run any tests') +parser.add_option('--autotest-server', action='store', default=socket.gethostname(), + help='Sets the autotest-server hostname. Used for creating URLs to results.\ +Hostname of the local machine is used by default.') # Read and validate the hookname # Check for no args, or just -h/--help if len(sys.argv) == 1 or sys.argv[1] in ('-h', '--help'): @@ -197,6 +201,8 @@ for arch in opts.arch: # N.B. process_testdata may grow new keyword arguments if we add new autoqa # args that add another loop here.. testdata = hook.process_testdata(opts, args, arch=arch) + if not 'autotest_server' in testdata.keys(): + testdata['autotest_server'] = opts.autotest_server # XXX FIXME: tests need to be able to indicate that they do not require # any specific arch (e.g. rpmlint can run on any arch) for test in testlist: diff --git a/hooks/post-koji-build/control.template b/hooks/post-koji-build/control.template index b90260f..b7c5261 100644 --- a/hooks/post-koji-build/control.template +++ b/hooks/post-koji-build/control.template @@ -15,4 +15,5 @@ TEST_CATEGORY = 'Functional' # kojitag: koji tag applied to this package job.run_test('testclassname', name=name, envr=envr, kojitag=kojitag, - config=autoqa_conf) + config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/hooks/post-koji-build/test_class_template.py b/hooks/post-koji-build/test_class_template.py index cf91865..462dc91 100644 --- a/hooks/post-koji-build/test_class_template.py +++ b/hooks/post-koji-build/test_class_template.py @@ -35,7 +35,10 @@ class post_repo_update_test_class_name(test.test): #utils.system('yum -y install yum-utils') pass - def run_once(self, envr, name, kojitag): + def run_once(self, envr, name, kojitag, autotest_url): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) parentlist = ' '.split(parents) cmd = 'test_binary --url %s' % baseurl # You can get stuff from the [test] section of autoqa.conf like this diff --git a/hooks/post-repo-update/control.template b/hooks/post-repo-update/control.template index 50ba58d..204dc69 100644 --- a/hooks/post-repo-update/control.template +++ b/hooks/post-repo-update/control.template @@ -19,4 +19,5 @@ TEST_CATEGORY = 'Functional' job.run_test('testclassname', baseurl=url, parents=parents, reponame=reponame, - config=autoqa_conf) + config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/hooks/post-repo-update/test_class_template.py b/hooks/post-repo-update/test_class_template.py index e274f98..c0c0137 100644 --- a/hooks/post-repo-update/test_class_template.py +++ b/hooks/post-repo-update/test_class_template.py @@ -33,7 +33,10 @@ class post_repo_update_test_class_name(test.test): #utils.system('yum -y install yum-utils') pass - def run_once(self, baseurl, parents, reponame): + def run_once(self, baseurl, parents, reponame, autotest_url): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) parentlist = ' '.split(parents) cmd = 'test_binary --url %s' % baseurl # You can get stuff from the [test] section of autoqa.conf like this diff --git a/hooks/post-tree-compose/control.template b/hooks/post-tree-compose/control.template index e327799..ef8ea35 100644 --- a/hooks/post-tree-compose/control.template +++ b/hooks/post-tree-compose/control.template @@ -17,4 +17,5 @@ TEST_CATEGORY = 'Functional' job.run_test('testclassname', baseurl=url, treename=treename, - config=autoqa_conf) + config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/hooks/post-tree-compose/test_class_template.py b/hooks/post-tree-compose/test_class_template.py index 0ffab9d..6084fb1 100644 --- a/hooks/post-tree-compose/test_class_template.py +++ b/hooks/post-tree-compose/test_class_template.py @@ -33,7 +33,10 @@ class post_tree_compose_test_class_name(test.test): #utils.system('yum -y install yum-utils') pass - def run_once(self, baseurl, treename): + def run_once(self, baseurl, treename, autotest_url): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) cmd = 'test_binary --url %s' % baseurl # You can get stuff from the [test] section of autoqa.conf like this email = self.config.get('test','result_email') -- 1.6.6.1

3 5

[PATCH 2/2] Ticket 65, 130: autotest_server param in autoqa.conf
by Josef Skladanka 21 Apr '10

21 Apr '10

autotest_server option added to the autoqa.conf file. If not set, socket.gethostname() is used. Could be overriden by the --autotest-server parameter. --- autoqa | 8 ++++++-- autoqa.conf | 5 +++++ 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/autoqa b/autoqa index 9f41c00..e889a9e 100755 --- a/autoqa +++ b/autoqa @@ -38,6 +38,7 @@ conf = { 'testdir': '/usr/share/autotest/client/site_tests', 'hookdir': '/usr/share/autoqa', 'notification_email': '', + 'autotest_server': socket.gethostname(), } cfg_parser = SafeConfigParser() # used by prep_controlfile try: @@ -140,7 +141,7 @@ parser.add_option('--local', action='store_true', dest='local', help='Do not schedule jobs - run test(s) directly on the local machine') parser.add_option('-l', '--list-tests', action='store_true', dest='listtests', help='list the tests for the given hookname - do not run any tests') -parser.add_option('--autotest-server', action='store', default=socket.gethostname(), +parser.add_option('--autotest-server', action='store', default=None, help='Sets the autotest-server hostname. Used for creating URLs to results.\ Hostname of the local machine is used by default.') # Read and validate the hookname @@ -202,7 +203,10 @@ for arch in opts.arch: # args that add another loop here.. testdata = hook.process_testdata(opts, args, arch=arch) if not 'autotest_server' in testdata.keys(): - testdata['autotest_server'] = opts.autotest_server + if opts.autotest_server is not None: + testdata['autotest_server'] = opts.autotest_server + else: + testdata['autotest_server'] = cfg['autotest_server'] # XXX FIXME: tests need to be able to indicate that they do not require # any specific arch (e.g. rpmlint can run on any arch) for test in testlist: diff --git a/autoqa.conf b/autoqa.conf index ed8662a..75efdd8 100644 --- a/autoqa.conf +++ b/autoqa.conf @@ -13,6 +13,11 @@ notification_email = # Default is to schedule jobs with the autotest server using 'atest'. # This is the same as using 'autoqa --local'. #local = true +# Hostname of Autotest server, which stores the test results. +# The same machine as AutoQA is running on is assumed by default - +# socket.gethostname() is used to determine it. +# This value can be overriden with --autotest-server parameter of AutoQA +#autotest_server = please.set.me # The 'test' section is used for systemwide configuration of autoqa tests [test] -- 1.6.6.1

3 5

[PATCH] Ticket 65, 130: updated current tests
by Josef Skladanka 21 Apr '10

21 Apr '10

Current test were updated, so the all send the URL to the stored results on the Autotest server (on first line of the email). --- tests/conflicts/conflicts.py | 9 +++++++-- tests/conflicts/control | 3 ++- tests/initscripts/control | 3 ++- tests/initscripts/initscripts.py | 9 ++++++--- tests/rats_install/control | 3 ++- tests/rats_install/rats_install.py | 9 +++++++-- tests/rats_sanity/control | 3 ++- tests/rats_sanity/rats_sanity.py | 9 +++++++-- tests/repoclosure/control | 3 ++- tests/repoclosure/repoclosure.py | 9 +++++++-- tests/rpmguard/control | 3 ++- tests/rpmguard/rpmguard.py | 9 +++++++-- tests/rpmlint/control | 3 ++- tests/rpmlint/rpmlint.py | 9 +++++++-- 14 files changed, 62 insertions(+), 22 deletions(-) diff --git a/tests/conflicts/conflicts.py b/tests/conflicts/conflicts.py index 16f8644..9b08b1b 100644 --- a/tests/conflicts/conflicts.py +++ b/tests/conflicts/conflicts.py @@ -33,7 +33,11 @@ class conflicts(test.test): f.write(logdata) f.close() - def run_once(self, baseurl, parents, reponame): + def run_once(self, baseurl, parents, reponame, autotest_url): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) + os.chdir(self.bindir) cmd = './potential_conflict.py --tempcache --newest' cmd += ' --repofrompath=target,%s --repoid=target' % baseurl @@ -65,10 +69,11 @@ class conflicts(test.test): result_email = self.config.get('test','result_email') if result_email: + mail_body = "Full logs available at <%s>\n\n%s" % (results_url, out) utils.send_email(mail_to=result_email, mail_from=self.config.get('test','mail_from'), mail_server=self.config.get('test','smtpserver'), subject='conflicts: %u packages with file conflicts in %s' % (len(conflicting_pkgs), reponame), - body = out) + body = mail_body) # TODO fail test if file conflicts exist? diff --git a/tests/conflicts/control b/tests/conflicts/control index 7c9b780..495f19a 100644 --- a/tests/conflicts/control +++ b/tests/conflicts/control @@ -13,4 +13,5 @@ TEST_CATEGORY = 'Functional' job.run_test('conflicts', baseurl=url, parents=parents, reponame=reponame, - config=autoqa_conf) + config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/tests/initscripts/control b/tests/initscripts/control index fec4fc0..3b9d317 100644 --- a/tests/initscripts/control +++ b/tests/initscripts/control @@ -14,4 +14,5 @@ TEST_CATEGORY = 'Functional' # name: package name # kojitag: koji tag applied to this package -job.run_test('initscripts', name=name, envr=envr, kojitag=kojitag, config=autoqa_conf) +job.run_test('initscripts', name=name, envr=envr, kojitag=kojitag, config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/tests/initscripts/initscripts.py b/tests/initscripts/initscripts.py index 634a04d..1da8a36 100644 --- a/tests/initscripts/initscripts.py +++ b/tests/initscripts/initscripts.py @@ -93,11 +93,13 @@ class initscripts(test.test): - def run_once(self, envr, name, kojitag): + def run_once(self, envr, name, kojitag, autotest_url): """ name - name of the package for initscript test """ - + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) #find all tests for package $name testdir = os.path.join(self.bindir, "./tests/%s" % name) tests = [] @@ -151,7 +153,8 @@ class initscripts(test.test): mail_from = self.config.get('test', 'mail_from') mail_server = self.config.get('test', 'smtpserver') if (mail_to and mail_from and mail_server): - msg = "Following tests returned zero exit code: %s\n" % repr(tests_exit_status_zero) + msg = "Full logs available at <%s>\n\n" % results_url + msg += "Following tests returned zero exit code: %s\n" % repr(tests_exit_status_zero) msg += "Following tests returned NON-zero exit code: %s\n" % repr(tests_exit_status_nonzero) msg += "%s\n" % ("#" * 79,) diff --git a/tests/rats_install/control b/tests/rats_install/control index f3fd56e..2cff980 100644 --- a/tests/rats_install/control +++ b/tests/rats_install/control @@ -16,4 +16,5 @@ TEST_CLASS = 'General' TEST_CATEGORY = 'Functional' TEST_TYPE = 'CLIENT' -job.run_test('rats_install', baseurl=url, treename=treename, config=autoqa_conf) +job.run_test('rats_install', baseurl=url, treename=treename, config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/tests/rats_install/rats_install.py b/tests/rats_install/rats_install.py index 3a46606..d657e34 100644 --- a/tests/rats_install/rats_install.py +++ b/tests/rats_install/rats_install.py @@ -47,7 +47,11 @@ class rats_install(test.test): # use the big hammer until we figure out the right thing to do utils.system('lokkit --disabled') - def run_once(self, baseurl, treename, image_url="", boot_args=""): + def run_once(self, baseurl, treename, autotest_url, image_url="", boot_args=""): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) + os.chdir(self.bindir) cmd = "./install.py -s %s -l %s" % (self.tmpdir, self.resultsdir) if image_url != "": @@ -64,11 +68,12 @@ class rats_install(test.test): result_email = self.config.get('test','result_email') if result_email: + mail_body = "Full logs available at <%s>\n\n%s" % (results_url, out) utils.send_email(mail_to=result_email, mail_from=self.config.get('test','mail_from'), mail_server=self.config.get('test','smtpserver'), subject='rats_install %s %s' % (result, treename), - body = out) + body = mail_body) # grab extra key = val data keyvals = util.read_keyvals() diff --git a/tests/rats_sanity/control b/tests/rats_sanity/control index 299d423..784998c 100644 --- a/tests/rats_sanity/control +++ b/tests/rats_sanity/control @@ -20,4 +20,5 @@ TEST_TYPE = 'CLIENT' job.run_test('rats_sanity', baseurl=url, parents=parents, reponame=reponame, - config=autoqa_conf) + config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/tests/rats_sanity/rats_sanity.py b/tests/rats_sanity/rats_sanity.py index 3e1f7fb..b13bcca 100644 --- a/tests/rats_sanity/rats_sanity.py +++ b/tests/rats_sanity/rats_sanity.py @@ -34,7 +34,11 @@ class rats_sanity(test.test): if os.path.exists(self.irb_conf): utils.system('yum -y install python-fedora') - def run_once(self, baseurl, parents, reponame): + def run_once(self, baseurl, parents, reponame, autotest_url): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) + os.chdir(self.bindir) cmd = "./sanity.py -s %s -l %s" % (self.tmpdir, self.resultsdir) cmd += " %s" % baseurl @@ -47,11 +51,12 @@ class rats_sanity(test.test): result_email = self.config.get('test','result_email') if result_email: + mail_body = "Full logs available at <%s>\n\n%s" % (results_url, out) utils.send_email(mail_to=result_email, mail_from=self.config.get('test','mail_from'), mail_server=self.config.get('test','smtpserver'), subject='rats_sanity %s %s' % (result, reponame), - body = out) + body = mail_body) # grab extra key = val data keyvals = util.read_keyvals() diff --git a/tests/repoclosure/control b/tests/repoclosure/control index 8467545..9a4e187 100644 --- a/tests/repoclosure/control +++ b/tests/repoclosure/control @@ -12,4 +12,5 @@ TEST_CATEGORY = 'Functional' job.run_test('repoclosure', baseurl=url, parents=parents, reponame=reponame, - config=autoqa_conf) + config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/tests/repoclosure/repoclosure.py b/tests/repoclosure/repoclosure.py index 29b0106..4a96d58 100644 --- a/tests/repoclosure/repoclosure.py +++ b/tests/repoclosure/repoclosure.py @@ -30,7 +30,11 @@ class repoclosure(test.test): def setup(self): utils.system('yum -y install yum-utils') - def run_once(self, baseurl, parents='', reponame=''): + def run_once(self, baseurl, autotest_url, parents='', reponame=''): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) + # build the repoclosure commandline cmd = 'repoclosure --tempcache --newest' cmd += ' --repofrompath=target,%s --repoid=target' % baseurl @@ -47,11 +51,12 @@ class repoclosure(test.test): result_email = self.config.get('test','result_email') if result_email: + mail_body = "Full logs available at <%s>\n\n%s" % (results_url, out) utils.send_email(mail_to=result_email, mail_from=self.config.get('test','mail_from'), mail_server=self.config.get('test','smtpserver'), subject='repoclosure: %u packages with unresolved deps in %s' % (unresolved_count, reponame), - body = out) + body = mail_body) if unresolved_count: raise error.TestFail, "%u packages with unresolved deps" % unresolved_count diff --git a/tests/rpmguard/control b/tests/rpmguard/control index 8c6bd12..cb0decb 100644 --- a/tests/rpmguard/control +++ b/tests/rpmguard/control @@ -18,4 +18,5 @@ TEST_CATEGORY = 'Regression' # kojitag: koji tag applied to this package job.run_test('rpmguard', name=name, envr=envr, kojitag=kojitag, - config=autoqa_conf) + config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/tests/rpmguard/rpmguard.py b/tests/rpmguard/rpmguard.py index 61eec64..321dea6 100644 --- a/tests/rpmguard/rpmguard.py +++ b/tests/rpmguard/rpmguard.py @@ -52,7 +52,11 @@ rpmlint | cut -d ' ' -f 1", retain_output=True) os.makedirs(self.rpmdir) - def run_once(self, envr, name, kojitag): + def run_once(self, envr, name, kojitag, autotest_url): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) + # where to log output out = open(os.path.join(self.resultsdir,'rpmguard.log'),'wb') @@ -178,6 +182,7 @@ rpmlint | cut -d ' ' -f 1", retain_output=True) # email results to mailing list result_email = self.config.get('test','result_email') if result_email: + mail_body = "Full logs available at <%s>\n\n%s" % (results_url, log) warning_count = len([line for line in log.splitlines() \ if line.startswith('W: ')]) print 'Sending email with results to %s ...' % result_email @@ -186,7 +191,7 @@ rpmlint | cut -d ' ' -f 1", retain_output=True) mail_server=self.config.get('test','smtpserver'), subject='rpmguard: %u warnings for package %s' % \ (warning_count, envr), - body = log) + body = mail_body) # if some command failed, make this test fail if test_result == 'FAIL': diff --git a/tests/rpmlint/control b/tests/rpmlint/control index a03be4e..c1fb55c 100644 --- a/tests/rpmlint/control +++ b/tests/rpmlint/control @@ -15,4 +15,5 @@ TEST_CATEGORY = 'Functional' # name: package name # kojitag: koji tag applied to this package -job.run_test('rpmlint', name=name, envr=envr, kojitag=kojitag, config=autoqa_conf) +job.run_test('rpmlint', name=name, envr=envr, kojitag=kojitag, config=autoqa_conf, + autotest_url="http://%s/results/%s/" % (autotest_server, job.tag)) diff --git a/tests/rpmlint/rpmlint.py b/tests/rpmlint/rpmlint.py index 7ad6f90..352ba0f 100644 --- a/tests/rpmlint/rpmlint.py +++ b/tests/rpmlint/rpmlint.py @@ -32,7 +32,11 @@ class rpmlint(test.test): def setup(self): utils.system('yum -y install rpmlint') - def run_once(self, envr): + def run_once(self, envr, autotest_url): + # results_url points to the directory on autotest-server with + #the sotred results + results_url = "%s%s/" % (autotest_url, self.__class__.__name__) + koji = autoqa.koji_utils.SimpleKojiClientSession() pkgurls = koji.nvr_to_urls(envr) rpmdir = os.path.join(self.tmpdir, 'rpms') @@ -63,6 +67,7 @@ class rpmlint(test.test): # email results to mailing list result_email = self.config.get('test','result_email') if result_email: + mail_body = "Full logs available at <%s>\n\n%s" % (results_url, self.results) # parse last line for summary ('N errors, N warnings') summary = ((self.results.splitlines()[-1]).split(';')[-1])[1:-1] print 'Sending email with results to %s ...' % result_email @@ -71,7 +76,7 @@ class rpmlint(test.test): mail_server=self.config.get('test','smtpserver'), subject='rpmlint: %s for package %s' % \ (summary, envr), - body = self.results) + body = self.mail_body) # if some command failed, make this test fail if result == 'FAIL': -- 1.6.6.1

2 3

conflicts: 69 packages with file conflicts in f12-updates-testing-x86_64 -- openscada-Special-FlibMath
by James Laska 21 Apr '10

21 Apr '10

-------- Forwarded Message -------- From: autoqa(a)fedoraproject.org Reply-to: fedora-test-list(a)redhat.com To: autoqa-results(a)lists.fedorahosted.org Subject: conflicts: 69 packages with file conflicts in f12-updates-testing-x86_64 Date: Sat, 17 Apr 2010 22:14:25 -0400 Getting complete filelist for: http://download.fedoraproject.org/pub/fedora/linux/updates/12/x86_64/ http://download.fedoraproject.org/pub/fedora/linux/releases/12/Everything/x… http://download.fedoraproject.org/pub/fedora/linux/updates/testing/12/x86_6… 2230289 files found. Looking for duplicated filenames: 263862 duplicates found. Doing more advanced checks to see if these are real conflicts: > <snip> 377 file conflicts found. 7 package conflicts found. > <snip> == File conflicts, listed by conflicting packages == openscada-Special-FLibMath-0.6.4.1-9.fc12.x86_64 openscada-Special-FlibMath-0.6.4-4.fc12.x86_64 /usr/lib64/openscada/spec_FLibMath.so I'm confused by this result for two reasons 1. Why are there multiple versions. Maybe this is the f12 repo, and then f12-updates repo 2. openscada-Special-FLibMath-0.6.4.1-9.fc12 has a %obsoletes [1] to handle obsoleting the older "flibMath" package. Is it not being considered in the conflicts transaction? Thanks, James [1] Obsoletes: %{name}-Special-FlibMath from http://cvs.fedoraproject.org/viewvc/rpms/openscada/F-12/openscada.spec?r1=1…

2 2

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

autoqa-devel April 2010