Comments (8)
This sounds very related to #7154 and irods/python-irodsclient#462.
Have you tried removing the storage resource(s) from the resource hierarchy and attempting to write data directly to them? I'm guessing that you will get a different error (namely, USER_FILE_TOO_LARGE
).
from irods.
Very good - let us know how it goes.
Marking this as duplicate for now.
from irods.
also... related to #3841 ?
from irods.
I considered #3841 before submitting this issue, but I thought this was different, because I couldn't reproduce the issue we're seeing on our environments using iput
. This issue is also not intermittent.
Creating a data object directly on a unixfilesystem resource that has free_space
and minimum_free_space_for_create_in_bytes
configured results in a irods.exception.USER_FILE_TOO_LARGE
when using PRC. It looks like this is indeed a symptom of #7154 and irods/python-irodsclient#462 .
Thank you for the information 👍 Knowing the root cause, I think we will be able to work around it.
from irods.
Since #7154 is closed - closing this issue as well.
The workaround for using Python is to set the filesize to 0 when talking to a pre-4.3.1 iRODS server.
from irods.
Information for other people who encounter the same issue:
We ended up using the following solution for the time being: https://utrechtuniversity.github.io/yoda/design/processes/automatic-resource-balancing.html
It basically involves keeping track of which resources are (almost) full in a Redis key-value store. Our pep_resource_resolve_hierarchy_pre
checks the resource information in Redis when a new data object is created, and changes votes so that new data objects get created on resources that have enough free space.
from irods.
Apologies for commenting on a closed Issue, but I just came across this and would it be possible to get some further clarification please?
I think that this affects all versions of iRODS before 4.3.1 but was introduced at least after 4.2.7 (since I've been able to use the PRC on a 4.2.7 system but not tried the most recent PRC version). It would be really good to know;
- what version this got introduced in.
- If the version of the PRC is related (i.e. can you use an earlier one? I think not but...)
- If other API's (e.g. C) are affected.
@trel does mention that
The workaround for using Python is to set the filesize to 0 when talking to a pre-4.3.1 iRODS server.
However, the documentation for the PRC doesn't mention this as of a few minutes ago, and its not clear to me what affect setting the file size will have on other operations that might depend on it (e.g. it might try and create a file on a resource without sufficient space, or fail to recognise a partial upload?). Could we get an update to the PRC docs to mention this workaround please (especially if its only affecting the PRC, but even if it affects the other API's too).
Thanks!
from irods.
- what version this got introduced in.
I haven't taken the time to track this down, but I can almost guarantee that 4.2.9 is the first version in which this is an issue.
- If the version of the PRC is related (i.e. can you use an earlier one? I think not but...)
I think the PRC has been leaning on the -1 dataSize behavior for a long time.
The inappropriate type choice for the problematic member variable has been in iRODS for a long time and has only "recently" become an issue because the usage of the member variable is leaning on the signed-ness which had been assumed but never actually real.
- If other API's (e.g. C) are affected.
All APIs - regardless of language - are affected by this because the problem is in a type used by the voting machinery deep in the server. We added a test using the C client API call and demonstrated the same problem. The reason this doesn't manifest in other clients is because the dataSize is not being set to -1 by the iput
, istream
, etc. machinery.
However, the documentation for the PRC doesn't mention this as of a few minutes ago, and its not clear to me what affect setting the file size will have on other operations that might depend on it (e.g. it might try and create a file on a resource without sufficient space, or fail to recognise a partial upload?).
Setting the file size to 0 is equivalent to setting it to -1 in a streaming situation. The server only cares that the dataSize is a non-positive value, so 0 or -1 should result in the same behavior.
As for the insufficient space question, streaming data to the resource would result in the same issue regardless of whether -1 or 0 is used. Streaming inherently doesn't know how much data will be uploaded. If the size of the data is known, a put should be used instead of just opening the object. ...However, this also would not solve the issue as specifying the dataSize via put
is not supported yet :( Please see irods/python-irodsclient#462
Could we get an update to the PRC docs to mention this workaround please (especially if its only affecting the PRC, but even if it affects the other API's too).
We could add a Troubleshooting section to the PRC documentation for this, sure. I've created an issue for this here: irods/python-irodsclient#481
Hope that helps...
from irods.
Related Issues (20)
- Redundant tests: `test_all_rules.Test_AllRules.test_msiServerMonPerf_default_3736`, `irods.test.test_all_rules.Test_AllRules.test_rulemsiServerMonPerf_r`
- `irodsServerMonPerf` needs some attention.
- Investigate Include What You Use HOT 1
- Sister projects with decoupled or semi-decoupled version numbers should include irods version number in package revision
- Consider deprecating/removing/replacing/disabling-by-default `msiServerMonPerf`/`irodsServerMonPerf`
- itree in json format
- Consider migrating from Boost.Regex to C++ stdlib
- Remove and replace usage of Boost.Format in favor of fmt
- `irods_kvp_string_parser.hpp` `#include`s `boost/foreach.hpp` twice but never uses it
- Catch2 v3 compatibility
- `irods_plugin_base.hpp` contains many unqualified calls to `std::forward`
- `obf.cpp:obfiDecode`: value of `nout` is never used.
- `irods_server_properties.cpp`: ambiguous calls to `log_server::debug` HOT 4
- `apiHandler.hpp` contains many unqualified calls to `std::forward`
- 4.3.2 post-release Include What You Use sweep
- Automating Data Removal HOT 1
- OpenedDataObjInp object and data object path HOT 3
- irsync creates target collection when using -l option HOT 3
- Investigate multiple replicas of the same data object with the identical physical paths
- The unit test cases under the unit_test directory failed during execution.
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from irods.