-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Geoserver importer NO_CRS task status doesn't trigger the CRS selection step #9016
Comments
#9021) * [Fixes #9016] Geoserver importer NO_CRS task status doesn't trigger the CRS selection step * [CircleCI] Fix tests * [CircleCI] Fix tests * [Minor * [Minor] Improve syntax and tests * [Minor] Improve syntax and tests * [CircleCI] Fix tests * [CircleCI] Fix tests * [CircleCI] Fix tests * [CircleCI] Fix tests * [CircleCI] Fix tests * [Pep8] Flake8 issues fixes Co-authored-by: mattiagiupponi <51856725+mattiagiupponi@users.noreply.github.com>
#9021) (#9046) * [Fixes #9016] Geoserver importer NO_CRS task status doesn't trigger the CRS selection step * [CircleCI] Fix tests * [CircleCI] Fix tests * [Minor * [Minor] Improve syntax and tests * [Minor] Improve syntax and tests * [CircleCI] Fix tests * [CircleCI] Fix tests * [CircleCI] Fix tests * [CircleCI] Fix tests * [CircleCI] Fix tests * [Pep8] Flake8 issues fixes Co-authored-by: mattiagiupponi <51856725+mattiagiupponi@users.noreply.github.com> Co-authored-by: Alessio Fabiani <alessio.fabiani@geo-solutions.it> Co-authored-by: mattiagiupponi <51856725+mattiagiupponi@users.noreply.github.com>
…oesn't trigger the CRS selection step
The problem is still there @afabiani. It can be reproduced on master with this sample tiff |
@giohappy I don't think this one is related to this issue, or at least it does not depend from the Importer... I'll check
while uploading a dataset without a valid CRS resulting in |
@afabiani I've tested the sample I attached both locally and master. Both cases I get the progress bar stuck and the importer session task[0] with NO_CRS. |
we should explictly send "mosaic=False" from the upload form |
No, we should assume mosaic False by default, not the reverse. Also considering that support for mosaic is a not well defined status right now. |
Found the difference. You were trying to upload the zip file, which triggers the mosaic workflow, because of the field. EDIT: issue created #9097 |
About the raster files, currently the workflow does not envisage passing through the |
@afabiani either we support it, or we return an error in case the importer is stuck with an invalid CRS. |
yes, I agree @giohappy |
…he CRS selection step (#9099) * [Fixes #9016] Geoserver importer NO_CRS task status doesn't trigger the CRS selection step * [Fixes #9016] Geoserver importer NO_CRS task status doesn't trigger the CRS selection step * [Fixes #9016] Geoserver importer NO_CRS task status doesn't trigger the CRS selection step * [CircleCI] Fix Tests * [CircleCI] Fix Docker Build * [CircleCI] Fix Tests * [CircleCI] Fix Tests
…he CRS selection step (#9099) (#9103) * [Fixes #9016] Geoserver importer NO_CRS task status doesn't trigger the CRS selection step * [Fixes #9016] Geoserver importer NO_CRS task status doesn't trigger the CRS selection step * [Fixes #9016] Geoserver importer NO_CRS task status doesn't trigger the CRS selection step * [CircleCI] Fix Tests * [CircleCI] Fix Docker Build * [CircleCI] Fix Tests * [CircleCI] Fix Tests Co-authored-by: Alessio Fabiani <alessio.fabiani@geo-solutions.it>
When an invalid .prj file is sent to Geoserver the importer blocks in a
PENDING
status with the first task inNO_CRS
status.GeoNode's upload keeps returning the
PENDING
status but with an emptyresume_url
, which is expected to be populated to let the client redirect to the CRS selection (as with thesrs
step).The result is a stale import that can only be deleted with the delete button (
delete_url
).This was tested on master with this shepafile.
I suspect it also affects 3.3.x.
The text was updated successfully, but these errors were encountered: