Skip to content
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

Broken sync request handling #134

Closed
soxofaan opened this issue Jan 17, 2024 · 4 comments
Closed

Broken sync request handling #134

soxofaan opened this issue Jan 17, 2024 · 4 comments

Comments

@soxofaan
Copy link
Member

New feature in openeo_driver was just added in Open-EO/openeo-python-driver#246 : it calls backend_implementation.processing.validate on sync request handling.

This causes failing tests in aggregator: tests.test_views.TestProcessing.test_result_backend_by_process detect additional error logging because now each sync processing request in aggregator forcibly triggers a /validation requests, which is not mocked in these tests

@soxofaan
Copy link
Member Author

will start with quick fix to pin down the openeo_driver dependency to get tests working again

soxofaan added a commit that referenced this issue Jan 17, 2024
to avoid problems due to new validation-on-sync-request feature
@soxofaan
Copy link
Member Author

tests are passing again with pin openeo_driver>=0.75.0.dev<0.80.0.dev

@soxofaan
Copy link
Member Author

under Open-EO/openeo-python-driver#248 we're going to revise the validation-on-sync-processing, which was reverted in openeo_driver 0.81.0, so we can remove that upper constraint again

@soxofaan
Copy link
Member Author

Additional work (if any) depends on Open-EO/openeo-python-driver#248 but going to close this one for now

@soxofaan soxofaan self-assigned this Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant