search

irods / irods_capability_storage_tiering

BSD 3-Clause "New" or "Revised" License
5 stars 10 forks source link

Avoid using "DOUBLE" in default delay parameters and example tiering rule #191

Closed alanking closed 8 months ago

alanking commented 2 years ago

The default delay parameters for migration and example storage tiering rule using the "DOUBLE" keyword in the execution frequency hint. As this is known to be faulty, we should avoid using it and its usage replaced with "REPEAT".

Additionally, the example rule can have its delay parameters reduced in time (i.e. 1 hour is a very long time for an example/demo). Despite the fact that it is more in line with a real world use case, it make demonstrations and testing annoying to work with.

trel commented 2 years ago

related to https://github.com/irods/irods/issues/5964

alanking commented 2 years ago

Still need to update the README and the default delay parameters for data migration: https://github.com/irods/irods_capability_storage_tiering/blob/a3a6baab10f2f456fcfea315bfb490636198f034/storage_tiering_configuration.hpp#L32

I've also noticed a file called sample_configuration.json in this repository which contains the DOUBLE keyword, but nothing seems to be using this. We can probably remove this file.

trel commented 2 years ago

We can probably remove this file

agreed - it appears out of date - and the README.md should have all that type of information in it already.

trel commented 10 months ago

https://github.com/irods/irods/issues/5964 has now been fixed in 4.3.1

alanking commented 8 months ago

Removing the 4-2-stable checkbox - we can cherry-pick this and many other fixes if/when another 4.2 release is required.