Hotfix/tags

[andre-merzky]

forward compatibility for tags - see radical-cybertools/radical.entk/issues/429

[codecov]

Codecov Report

Merging #2300 (4e83814) into devel (3a53618) will increase coverage by 0.02%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##            devel    #2300      +/-   ##
==========================================
+ Coverage   34.81%   34.84%   +0.02%     
==========================================
  Files          82       82              
  Lines        8908     8912       +4     
==========================================
+ Hits         3101     3105       +4     
  Misses       5807     5807              

Impacted Files	Coverage Δ
src/radical/pilot/agent/scheduler/continuous.py	72.37% <75.00%> (+0.62%)	⬆️

---

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 3a53618...6e968c0. Read the comment docs.

[iparask]

I'm approving of the change since I know it works! I'll try to create a test case for tags.

[andre-merzky]

I'll try to create a test case for tags.

Thank you!

[iparask]

@andre-merzky I added the test case. I'll do more while writing more tests.

[andre-merzky]

@andre-merzky I added the test case. I'll do more while writing more tests.

Is this good to merge, or should we wait for those other tests?

[iparask]

I see that one of the tests fails. After that is fixed we can merge and I will add tests during my testing day. :)

[andre-merzky]

I see that one of the tests fails. After that is fixed we can merge and I will add tests during my testing day. :)

This should be fixed now. I don't see how the test can have worked before though: the unit fed into the scheduler is unit.000000, but the test checks if the tag history has been filled for unit.000001?