Fixes for python3

Depends on Phys!762 (merged), Analysis!682 (merged) and !429 (merged).

added 1 deleted label

added 1 commit

• 3b100b96 - Fixed formatting

Compare with previous version

• [2020-08-25 15:25] Validation started with lhcb-run2-patches-py3#10
• [2020-08-26 00:15] Validation started with lhcb-run2-patches-py3#11
• [2020-08-27 00:15] Validation started with lhcb-run2-patches-py3#12
• [2020-08-27 14:53] Validation started with lhcb-run2-patches-py3#13
• [2020-08-28 00:12] Validation started with lhcb-run2-patches-py3#14
• [2020-08-28 10:36] Validation started with lhcb-run2-patches-py3#15
• [2020-08-29 00:22] Validation started with lhcb-run2-patches-py3#16
• [2020-08-29 12:53] Validation started with lhcb-run2-patches-py3#17
• [2020-08-30 00:08] Validation started with lhcb-run2-patches-py3#18
• [2020-08-30 12:52] Validation started with lhcb-run2-patches-py3#19
• [2020-08-31 00:14] Validation started with lhcb-run2-patches-py3#20
• [2020-08-31 11:17] Validation started with lhcb-run2-patches-py3#21
• [2020-09-01 00:20] Validation started with lhcb-run2-patches-py3#22
• [2020-09-02 00:11] Validation started with lhcb-run2-patches-py3#23
• [2020-09-02 10:45] Validation started with lhcb-run2-patches-py3#24
• [2020-09-02 13:46] Validation started with lhcb-run2-patches-py3#25
• [2020-09-02 15:37] Validation started with lhcb-run2-patches-py3#26
• [2020-09-03 00:13] Validation started with lhcb-run2-patches-py3#27
• [2020-09-03 11:39] Validation started with lhcb-run2-patches-py3#28
• [2020-09-04 00:19] Validation started with lhcb-run2-patches-py3#29
• [2020-09-04 11:43] Validation started with lhcb-run2-patches-py3#30
• [2020-09-05 00:28] Validation started with lhcb-run2-patches-py3#31
• [2020-09-06 00:12] Validation started with lhcb-run2-patches-py3#32
• [2020-09-07 00:15] Validation started with lhcb-run2-patches-py3#33
• [2020-09-08 00:14] Validation started with lhcb-run2-patches-py3#34
• [2020-09-09 00:22] Validation started with lhcb-run2-patches-py3#35
• [2020-09-10 00:14] Validation started with lhcb-run2-patches-py3#36
• [2020-09-11 00:16] Validation started with lhcb-run2-patches-py3#37
• [2020-09-12 00:20] Validation started with lhcb-run2-patches-py3#38
• [2020-09-13 00:09] Validation started with lhcb-run2-patches-py3#39
• [2020-09-15 01:06] Validation started with lhcb-run2-patches-py3#40
• [2020-09-16 00:17] Validation started with lhcb-run2-patches-py3#41
• [2020-09-17 00:22] Validation started with lhcb-run2-patches-py3#42
• [2020-09-18 00:20] Validation started with lhcb-run2-patches-py3#43
• [2020-09-18 08:47] Validation started with lhcb-run2-patches-py3#44
• [2020-09-18 15:14] Validation started with lhcb-run2-patches-py3#45
• [2020-09-20 00:07] Validation started with lhcb-run2-patches-py3#46
• [2020-09-21 00:14] Validation started with lhcb-run2-patches-py3#47

mentioned in merge request Phys!762 (merged)

added 1 commit

• 8e251415 - Update CheckFSRs.py to use subprocess.check_output instead of subprocess.getoutput

Compare with previous version

added 1 commit

• 7087364a - Apply Eduardos suggestion to fix merge-small-files test

Compare with previous version

resolved all threads

added 1 commit

• f14b99b3 - Fixed formatting

Compare with previous version

changed the description

So I think we cannot go any further until !429 (merged) is merged. I don't think python3 can open the db files from the StrippingArchive produced with python2. A quick google suggests python3 shelve can't open ones created with python2.

changed the description

How far are you from being able to merge !429 (merged)? If there is some real issue such as the one you mention then we have to live with it, right? In good CI tools such as pytest you can easily skip certain tests based on conditions (not Python 2, etc.). Thanks for the investigation.

added 8 commits

• f14b99b3...a20c04dd - 4 commits from branch run2-patches
• 4e671d16 - update tests
• 38a5ab19 - python 3 compatibility
• a917470d - Update CheckFSRs.py to use subprocess.check_output instead of subprocess.getoutput
• c324af8d - Apply Eduardos suggestion to fix merge-small-files test

Compare with previous version

added 3 commits

• 718c103f - python 3 compatibility
• 7e16d2a8 - Update CheckFSRs.py to use subprocess.check_output instead of subprocess.getoutput
• 8523e3cd - Apply Eduardos suggestion to fix merge-small-files test

Compare with previous version

added all-slots modernisation labels

changed title from run2-patches python3 to Fixes for python3

resolved all threads

• [2020-09-21 13:13] Validation started with lhcb-run2-patches-py3#48
• [2020-09-22 00:11] Validation started with lhcb-run2-patches-py3#49
• [2020-09-22 00:11] Validation started with lhcb-run2-patches-dev4#458
• [2020-09-22 00:14] Validation started with lhcb-run2-patches#822