open-io / oio-sds

High Performance Software-Defined Object Storage for Big Data and AI, that supports Amazon S3 and Openstack Swift
https://www.openio.io
Other
664 stars 93 forks source link

CI: convert remaining tests using python 2 to python 3.6 #2095

Closed murlock closed 3 years ago

murlock commented 3 years ago
SUMMARY

Drop Python 2 Since gateway will use Python 3.7, launch tests with Python 3.7

ISSUE TYPE
COMPONENT NAME

oio

SDS VERSION
master
fvennetier commented 3 years ago

Building with both Python 3.6 and 3.7 seems overkill. I suggest we reduce the number of test environments.

ilitzroth commented 3 years ago

Please keep me informed about the python version. I'm currently using 3.6 if we're moving to 3.7 some things need to be adapted (docker files, CMakeLists.txt), but I don't expect much fallout.

murlock commented 3 years ago

We still target Python 3.6 at this time, I'll fix the error and reduce number of test environmment.

fvennetier commented 3 years ago

Now that #2096 is merged, we need to make a cascade merge, and the tests should be ok again.

codecov-io commented 3 years ago

Codecov Report

Merging #2095 (2933bf3) into master (529896d) will decrease coverage by 0.44%. The diff coverage is 93.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2095      +/-   ##
==========================================
- Coverage   79.77%   79.34%   -0.43%     
==========================================
  Files         368      367       -1     
  Lines       72242    72192      -50     
  Branches     6450     6468      +18     
==========================================
- Hits        57623    57270     -353     
- Misses      14417    14718     +301     
- Partials      202      204       +2     
Impacted Files Coverage Δ
tests/functional/blob/test_mover.py 92.93% <0.00%> (-7.07%) :arrow_down:
tests/functional/account/test_backend.py 100.00% <100.00%> (ø)
tests/functional/cli/container/test_container.py 60.96% <100.00%> (ø)
tests/functional/conscience/test_conscience.py 95.06% <100.00%> (ø)
tests/functional/m2_filters/test_filters.py 97.06% <100.00%> (ø)
sqliterepo/election.c 76.91% <0.00%> (-12.17%) :arrow_down:
tests/unit/api/test_container_client.py 92.86% <0.00%> (-7.14%) :arrow_down:
oio/content/ec.py 56.46% <0.00%> (-5.84%) :arrow_down:
sqliterepo/gridd_client_pool.c 77.64% <0.00%> (-3.50%) :arrow_down:
oio/conscience/client.py 77.60% <0.00%> (-3.20%) :arrow_down:
... and 65 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 529896d...2933bf3. Read the comment docs.