-
Notifications
You must be signed in to change notification settings - Fork 1
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
refactor concrete configs out of openeo-aggregator #117
Comments
I tried to work with more explicit env vars for clarity (replace Lines 15 to 16 in d057d00
but after a lot of head scratching I suspect these "docker_run_options" don't do anything anymore, the real env settings are now in |
now managed from hiera-resenv (see GDD-2861)
started with new project |
Also to take into account here: https://github.com/Open-EO/openeo-aggregator/tree/master/integration-tests is currently openeo.cloud oriented as well. Make it more reusable or split off to standalone project? |
refs and related: #117 #143 #146 eu-cdse/openeo-cdse-infra#175
Strongly related with this ticket is refactoring out the concrete dockerfile (which has hard references to configs): |
to allow including example configs in src tree
started working some more on this, but got sidetracked by #151 due to change in openeo_driver |
to allow including example configs in src tree
to allow including example configs in src tree
Hi @tcassaert, I already took various steps in various repo's, but now I'm a bit unsure about the next steps. Can you help me? I'll try to explain the situation below: Current setup of openEO Platform (aka openeo.cloud) build/deploy workflow:
openeo-aggregator:
image: "vito-docker-private-dev.artifactory.vgt.vito.be/openeo-aggregator"
ports:
- 8081:8080
env:
- "OPENEO_BACKEND_CONFIG=/home/openeo/aggregator/conf/aggregator.dev.py"
- "ENV=dev"
subscribe: "Docker::Image[openeo-aggregator]"
This has to/is being split up a bit. Target situation:
I'm a bit unsure on how to do these lasts steps correctly in the appropriate repos, with the appropriate tools and workflows. To avoid the risk of having to "hot swap" several things, I think it makes sense to:
(Once the above is done, I can continue removing the then unused resources from the openeo-aggregator repo) |
The flow that's implemented now:
https://openeocloud-dev.vito.be is already using the new deployment and the integrationtests ran successfully against it. |
@tcassaert I see you added a requirements.txt to the root of https://github.com/Open-EO/openeo-aggregator (which is referenced from the Dockerfile) this is quite confusing, as toplevel requirements.txt typically list dependencies for the python package itself, but this is just for the docker build that installs the package. Is it possible to move the docker resources, e.g., to a |
…ntext is root of project refs #117
Good suggestion, fixed! |
Added the pipeline flow documentation to https://git.vito.be/projects/TPT/repos/openeo_aggregator_cdse/browse/README.md and also added a link to it on confluence. |
As we're starting to play with multiple aggregator instances, it's confusing to have the original concrete (openeo platform) configs still hardcoded in this repo. It's probably better to refactor them out.
(would also help with #94)
The text was updated successfully, but these errors were encountered: