Bug#1014100: We should do an update of all qiime2 modules as soon as possible (Was: Bug#1014100: qiime: autopkgtest needs update for new version of python-decorator)

2022-06-30 Thread Steffen Möller



On 30.06.22 11:51, Andreas Tille wrote:

I'd like to point out to this serious bug.  Steffen, would you mind
koordinating (not doing alone) an upgrade of the qiime infrastructure to
make sure everything will go smoothly?

I added "to be updated" tags to all the qiime2 packages on

https://docs.google.com/spreadsheets/d/1tApLhVqxRZ2VOuMH_aPUgFENQJfbLlB_PFH_Ah_q7hM/edit#gid=1986383821

Yes, these updates are overdue. I'll see what I can do, help is of
course much appreciated.

Many thanks!

Steffen



Bug#1014100: We should do an update of all qiime2 modules as soon as possible (Was: Bug#1014100: qiime: autopkgtest needs update for new version of python-decorator)

2022-06-30 Thread Andreas Tille
Hi,

I'd like to point out to this serious bug.  Steffen, would you mind
koordinating (not doing alone) an upgrade of the qiime infrastructure to
make sure everything will go smoothly?

Kind regards

 Andreas.

Am Thu, Jun 30, 2022 at 11:30:21AM +0200 schrieb Paul Gevers:
> Source: qiime
> Version: 2021.8.0-2
> Severity: serious
> X-Debbugs-CC: python-decora...@packages.debian.org
> Tags: sid bookworm
> User: debian...@lists.debian.org
> Usertags: needs-update
> Control: affects -1 src:python-decorator
> 
> Dear maintainer(s),
> 
> With a recent upload of python-decorator the autopkgtest of qiime fails in
> testing when that autopkgtest is run with the binary packages of
> python-decorator from unstable. It passes when run with only packages from
> testing. In tabular form:
> 
>passfail
> python-decorator   from testing5.1.1-2
> qiime  from testing2021.8.0-2
> all others from testingfrom testing
> 
> I copied some of the output at the bottom of this report.
> 
> Currently this regression is blocking the migration of python-decorator to
> testing [1]. Of course, python-decorator shouldn't just break your
> autopkgtest (or even worse, your package), but it seems to me that the
> change in python-decorator was intended and your package needs to update to
> the new situation. To me it looks like the references need updating for the
> new version but I might be wrong.
> 
> If this is a real problem in your package (and not only in your
> autopkgtest), the right binary package(s) from python-decorator should
> really add a versioned Breaks on the unfixed version of (one of your)
> package(s). Note: the Breaks is nice even if the issue is only in the
> autopkgtest as it helps the migration software to figure out the right
> versions to combine in the tests.
> 
> More information about this bug and the reason for filing it can be found on
> https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation
> 
> Paul
> 
> [1] https://qa.debian.org/excuses.php?package=python-decorator
> 
> https://ci.debian.net/data/autopkgtest/testing/amd64/q/qiime/23173367/log.gz
> 
> FF.FFF..
> ==
> FAIL: test_callable_and_async_signature_with_artifacts_and_parameters
> (qiime2.sdk.tests.test_method.TestMethod)
> --
> Traceback (most recent call last):
>   File 
> "/tmp/autopkgtest-lxc.t_q3d0jx/downtmp/autopkgtest_tmp/qiime2/sdk/tests/test_method.py",
> line 223, in test_callable_and_async_signature_with_artifacts_and_parameters
> self.assertEqual(parameters, exp_parameters)
> AssertionError: Lists differ: [('in[20 chars]ts1: list">), ('ints2',
> )] != [('in[20 chars]ts1: IntSequence1 |
> IntSequence2">), ('ints2',[147 chars]t">)]
> 
> First differing element 0:
> ('ints1', )
> ('ints1', )
> 
> Diff is 735 characters long. Set self.maxDiff to None to see it.
> 
> ==
> FAIL: test_callable_and_async_signature_with_no_parameters
> (qiime2.sdk.tests.test_method.TestMethod)
> --
> Traceback (most recent call last):
>   File 
> "/tmp/autopkgtest-lxc.t_q3d0jx/downtmp/autopkgtest_tmp/qiime2/sdk/tests/test_method.py",
> line 242, in test_callable_and_async_signature_with_no_parameters
> self.assertEqual(parameters, exp_parameters)
> AssertionError: Lists differ: [('ma[26 chars]ng1: dict">), ('mapping2',
> )] != [('ma[26 chars]ng1: Mapping">),
> ('mapping2', )]
> 
> First differing element 0:
> ('mapping1', )
> ('mapping1', )
> 
> - [('mapping1', ),
> ? ^ ^^
> 
> + [('mapping1', ),
> ?  ^^
> 
> -  ('mapping2', )]
> ? ^ ^^
> 
> +  ('mapping2', )]
> ?  ^^
> 
> 
>