Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package openQA for openSUSE:Factory checked
in at 2025-08-05 14:21:16
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/openQA (Old)
and /work/SRC/openSUSE:Factory/.openQA.new.1085 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "openQA"
Tue Aug 5 14:21:16 2025 rev:723 rq:1297547 version:5.1754297241.bf37533a
Changes:
--------
--- /work/SRC/openSUSE:Factory/openQA/openQA.changes 2025-08-02
00:42:15.731818986 +0200
+++ /work/SRC/openSUSE:Factory/.openQA.new.1085/openQA.changes 2025-08-05
14:21:56.754890999 +0200
@@ -1,0 +2,7 @@
+Mon Aug 04 16:11:31 UTC 2025 - ok...@suse.com
+
+- Update to version 5.1754297241.bf37533a:
+ * Stop flagging obs_rsync errors as failures
+ * Mention use of CRITICAL_LOAD_AVG_THRESHOLD
+
+-------------------------------------------------------------------
Old:
----
openQA-5.1753967288.48036af4.obscpio
New:
----
openQA-5.1754297241.bf37533a.obscpio
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ openQA-client-test.spec ++++++
--- /var/tmp/diff_new_pack.F2dCiS/_old 2025-08-05 14:21:58.518964724 +0200
+++ /var/tmp/diff_new_pack.F2dCiS/_new 2025-08-05 14:21:58.518964724 +0200
@@ -1,7 +1,7 @@
#
# spec file for package openQA-client-test
#
-# Copyright (c) 2025 SUSE LLC
+# Copyright (c) 2025 SUSE LLC and contributors
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -18,7 +18,7 @@
%define short_name openQA-client
Name: %{short_name}-test
-Version: 5.1753967288.48036af4
+Version: 5.1754297241.bf37533a
Release: 0
Summary: Test package for %{short_name}
License: GPL-2.0-or-later
++++++ openQA-devel-test.spec ++++++
--- /var/tmp/diff_new_pack.F2dCiS/_old 2025-08-05 14:21:58.550966061 +0200
+++ /var/tmp/diff_new_pack.F2dCiS/_new 2025-08-05 14:21:58.554966229 +0200
@@ -1,7 +1,7 @@
#
# spec file for package openQA-devel-test
#
-# Copyright (c) 2025 SUSE LLC
+# Copyright (c) 2025 SUSE LLC and contributors
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -18,7 +18,7 @@
%define short_name openQA-devel
Name: %{short_name}-test
-Version: 5.1753967288.48036af4
+Version: 5.1754297241.bf37533a
Release: 0
Summary: Test package for %{short_name}
License: GPL-2.0-or-later
++++++ openQA-test.spec ++++++
--- /var/tmp/diff_new_pack.F2dCiS/_old 2025-08-05 14:21:58.582967399 +0200
+++ /var/tmp/diff_new_pack.F2dCiS/_new 2025-08-05 14:21:58.586967566 +0200
@@ -1,7 +1,7 @@
#
# spec file for package openQA-test
#
-# Copyright (c) 2025 SUSE LLC
+# Copyright (c) 2025 SUSE LLC and contributors
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -18,7 +18,7 @@
%define short_name openQA
Name: %{short_name}-test
-Version: 5.1753967288.48036af4
+Version: 5.1754297241.bf37533a
Release: 0
Summary: Test package for openQA
License: GPL-2.0-or-later
++++++ openQA-worker-test.spec ++++++
--- /var/tmp/diff_new_pack.F2dCiS/_old 2025-08-05 14:21:58.618968903 +0200
+++ /var/tmp/diff_new_pack.F2dCiS/_new 2025-08-05 14:21:58.622969071 +0200
@@ -1,7 +1,7 @@
#
# spec file for package openQA-worker-test
#
-# Copyright (c) 2025 SUSE LLC
+# Copyright (c) 2025 SUSE LLC and contributors
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -18,7 +18,7 @@
%define short_name openQA-worker
Name: %{short_name}-test
-Version: 5.1753967288.48036af4
+Version: 5.1754297241.bf37533a
Release: 0
Summary: Test package for %{short_name}
License: GPL-2.0-or-later
++++++ openQA.spec ++++++
--- /var/tmp/diff_new_pack.F2dCiS/_old 2025-08-05 14:21:58.654970408 +0200
+++ /var/tmp/diff_new_pack.F2dCiS/_new 2025-08-05 14:21:58.654970408 +0200
@@ -1,7 +1,7 @@
#
# spec file for package openQA
#
-# Copyright (c) 2025 SUSE LLC
+# Copyright (c) 2025 SUSE LLC and contributors
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -97,7 +97,7 @@
%define devel_requires %devel_no_selenium_requires chromedriver
Name: openQA
-Version: 5.1753967288.48036af4
+Version: 5.1754297241.bf37533a
Release: 0
Summary: The openQA web-frontend, scheduler and tools
License: GPL-2.0-or-later
++++++ openQA-5.1753967288.48036af4.obscpio ->
openQA-5.1754297241.bf37533a.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openQA-5.1753967288.48036af4/docs/WritingTests.asciidoc
new/openQA-5.1754297241.bf37533a/docs/WritingTests.asciidoc
--- old/openQA-5.1753967288.48036af4/docs/WritingTests.asciidoc 2025-07-31
15:08:08.000000000 +0200
+++ new/openQA-5.1754297241.bf37533a/docs/WritingTests.asciidoc 2025-08-04
10:47:21.000000000 +0200
@@ -667,6 +667,16 @@
WORKER_CLASS = planet-earth,continent-antarctica,location-my_station
--------------------------------------------------------------------------------
+The load of the system can affect test execution and cause failures due to
delays of command execution and
https://en.wikipedia.org/wiki/Thrashing_(computer_science)[thrashing]. If the
average over a period of 15 minutes exceeds the specified value the worker will
not accept new jobs.
+
+[caption="File: "]
+.workers.ini
+[source,ini]
+--------------------------------------------------------------------------------
+[global]
+# Set to 0 to disable.
+#CRITICAL_LOAD_AVG_THRESHOLD = 40
+--------------------------------------------------------------------------------
=== Running a custom worker engine
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openQA-5.1753967288.48036af4/lib/OpenQA/WebAPI/Plugin/ObsRsync/Task.pm
new/openQA-5.1754297241.bf37533a/lib/OpenQA/WebAPI/Plugin/ObsRsync/Task.pm
--- old/openQA-5.1753967288.48036af4/lib/OpenQA/WebAPI/Plugin/ObsRsync/Task.pm
2025-07-31 15:08:08.000000000 +0200
+++ new/openQA-5.1754297241.bf37533a/lib/OpenQA/WebAPI/Plugin/ObsRsync/Task.pm
2025-08-04 10:47:21.000000000 +0200
@@ -76,7 +76,7 @@
$error =~ s/\s+$//;
$app->log->error('ObsRsync#_run failed (' . $exit_code . '): ' . $error);
$helper->log_failure($project, $job->id);
- return $job->fail({code => $exit_code, message => $error});
+ return $job->finish({code => $exit_code, message => $error});
}
sub update_dirty_status {
@@ -123,7 +123,7 @@
my ($exit_code, $error) = $helper->for_every_batch($alias, $sub);
- return $job->fail({code => $exit_code, message => $error}) if $exit_code;
+ return $job->finish({code => $exit_code, message => $error}) if $exit_code;
return $job->finish('Success');
}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openQA-5.1753967288.48036af4/t/api/14-plugin_obs_rsync_async.t
new/openQA-5.1754297241.bf37533a/t/api/14-plugin_obs_rsync_async.t
--- old/openQA-5.1753967288.48036af4/t/api/14-plugin_obs_rsync_async.t
2025-07-31 15:08:08.000000000 +0200
+++ new/openQA-5.1754297241.bf37533a/t/api/14-plugin_obs_rsync_async.t
2025-08-04 10:47:21.000000000 +0200
@@ -148,38 +148,14 @@
sleep_until_all_jobs_finished($t);
-# MockProjectError will fail so number of finished jobs should remain, but one
job must be failed
-is(_jobs_cnt('finished'), 10, 'Number of finished jobs');
+# MockProjectError should not be raised because errors are ignored
+is(_jobs_cnt('finished'), 11, 'Number of finished jobs');
my ($cnt, $jobs) = _jobs('failed');
-is($cnt, 1, 'Number of finished jobs');
-is($jobs->[0]->{result}->{message}, 'Mock Error', 'Correct error message') if
$cnt;
+is($cnt, 0, 'Number of failed jobs');
-subtest 'test max retry count' => sub {
- # use all concurrency slots to reach concurrency limit
- my @guards = map { $t->app->obs_rsync->concurrency_guard() } (1 ..
$config{queue_limit});
- # put request and make sure it succeeded within 5 sec
- $t->put_ok('/api/v1/obs_rsync/Proj1/runs')->status_is(201, 'trigger
rsync');
-
- my $sleep = .2;
- my $empiristic = 3; # this accounts gru timing in worst case for job
run and retry
- my $max_iterations = ($config{retry_max_count} + 1) * ($empiristic +
$config{retry_interval}) / $sleep;
- for (1 .. $max_iterations) {
- ($cnt, $jobs) = _jobs('finished');
- last if $cnt > 10;
- sleep $sleep;
- }
-
- is($cnt, 11, 'Job should retry succeed');
- is($jobs->[0]->{retries}, $config{retry_max_count}, 'Job retris is
correct');
- is(ref $jobs->[0]->{result}, 'HASH', 'Job retry result is hash');
- is(
- $jobs->[0]->{result}->{message},
- "Exceeded retry count $config{retry_max_count}. Consider job will be
re-triggered later",
- 'Job retry message'
- ) if ref $jobs->[0]->{result} eq 'HASH';
- # unlock guards
- @guards = undef;
-};
+($cnt, $jobs) = _jobs('finished');
+is($jobs->[0]->{result}->{message}, 'Mock Error', 'Correct error message');
+is($jobs->[0]->{result}->{code}, 256, 'Correct exit code') if $cnt;
END {
$gru->signal('TERM');
++++++ openQA.obsinfo ++++++
--- /var/tmp/diff_new_pack.F2dCiS/_old 2025-08-05 14:22:16.219704487 +0200
+++ /var/tmp/diff_new_pack.F2dCiS/_new 2025-08-05 14:22:16.219704487 +0200
@@ -1,5 +1,5 @@
name: openQA
-version: 5.1753967288.48036af4
-mtime: 1753967288
-commit: 48036af4c8bdd6c53a27f5b2135c1aa3d41649bf
+version: 5.1754297241.bf37533a
+mtime: 1754297241
+commit: bf37533a27c8fa526142710b397f61266e8089f2
