-
Notifications
You must be signed in to change notification settings - Fork 10
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
Image storage: serve iso images, lithops support #778
Image storage: serve iso images, lithops support #778
Conversation
* remove deprecated bootstrap_and_run, refactor dependent scripts * configure policy for image storage bucket on startup * fix sci-test * update graphql Annotation resolver
* remove dataset indexing from ServerAnnotationJob * linting
* thumnail and coloc gen with lithops * simplify ImageStorage init
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for cleaning up the lint issues. Generally looks good.
My comment on ImageStorage._configure_bucket
is non-critical - skip it if you don't have time.
Action is only needed on the migration comment if you don't already have a plan for how to deal with the transition period.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* feat: serve iso image urls from graphql * remove deprecated bootstrap_and_run, refactor dependent scripts * configure policy for image storage bucket on startup * fix sci-test * update graphql Annotation resolver * fix: test_es_export.py * feat: make image storage work with lithops * remove dataset indexing from ServerAnnotationJob * linting * test: fix * test: fix sci test * fix: code polishing * fix: catch import error * feat: delete ion thumbnail on ds delete * fix: pass ImageStorage objects to lithops functions * thumnail and coloc gen with lithops * simplify ImageStorage init * test: fix colocalization * fix: move image storage bucket configuration to module level init
ServerAnnotationJob
.bootstrap_and_run
, refactor dependent scripts.Annotation
resolver.pylint==2.7.0
complaints.