Task #10969 (closed)
Bug: CLI Chrgp: use DoAll request
| Reported by: | sbesson | Owned by: | mtbcarroll |
|---|---|---|---|
| Priority: | critical | Milestone: | 5.1.0-m4 |
| Component: | Client | Version: | 5.0.0-beta1 |
| Keywords: | n.a. | Cc: | fs@…, sbesson |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | 0.0d |
| Sprint: | n.a. |
Description
As reported in https://github.com/openmicroscopy/openmicroscopy/pull/1195, the following chgrp command:
jrs-macbookpro-25031:ome cblackburn$ dist/bin/omero chgrp 29 /Image:707 /Image:708
currently fails for a Fileset composed of Images 707 and 708 only. This bug should be fixed by allowing chgrp transaction to use DoAll? requests.
Change History (10)
comment:1 Changed 6 years ago by jamoore
comment:2 Changed 6 years ago by jamoore
- Milestone changed from 5.0.0 to 5.0.0-beta3
- Version set to 5.0.0-beta1
comment:3 Changed 6 years ago by jamoore
- Priority changed from major to critical
- Summary changed from CLI Chrgp: use DoAll request to Bug: CLI Chrgp: use DoAll request
Not bumping to beta2 since the issue already existed in beta1. This will be a priority for beta3 though.
comment:4 Changed 5 years ago by jamoore
- Owner set to mtbcarroll
comment:5 Changed 5 years ago by mtbcarroll
Josh: Is it clear that you do want me to attempt to rewrite the Python here to conflate them? Sometimes, as with things entailing original files and the directories they are in, it is important that the deletions occur in the correct sequence. Do we need to expand the CLI argument syntax to indicate groups of objects that must be in separate requests?
comment:6 Changed 5 years ago by jamoore
Hmmmm..... Perhaps we first need a suggestion of what is most likely to work in the most situations, before any rewriting. It might be that the command needs a DoAll regardless, which is pretty straight-forward.
comment:7 Changed 5 years ago by mtbcarroll
- Milestone changed from 5.1.0 to 5.1.0-m4
- Status changed from new to accepted
comment:8 Changed 5 years ago by mtbcarroll
- Resolution set to fixed
- Status changed from accepted to closed
Fixed by https://github.com/openmicroscopy/openmicroscopy/pull/3410. If people do need truly separate requests, they can make multiple CLI calls!
comment:9 Changed 5 years ago by Mark Carroll <M.T.B.Carroll@…>
- Remaining Time set to 0
(In [1e6e7ee55c7916b8e1fd43b98637867395f8ba96/ome.git] on branch develop) fix #10969: rearrange DoAll?'s in cli.py
comment:10 Changed 5 years ago by Josh Moore <josh@…>
(In [8ad7626357765bb8c20ae277fddbd0008a1edab9/ome.git] on branch develop) Merge pull request #3410 from mtbc/trac-10969-cli-chgrp-doall
See reported instance under https://github.com/openmicroscopy/openmicroscopy/pull/1845