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.

Sign up for GitHub

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

Jump to bottom

Migrate AggregatorConfig to OpenEoBackendConfig config system #112

Closed
soxofaan opened this issue May 16, 2023 · 2 comments
Closed

Migrate AggregatorConfig to OpenEoBackendConfig config system #112

soxofaan opened this issue May 16, 2023 · 2 comments
Assignees
@soxofaan
Labels
architecture config devops technical debt

Comments

@soxofaan
Copy link
Member

current AggregatorConfig is dict based, with some extra's, but should migrate to the newer OpenEoBackendConfig from openeo-python-driver
soxofaan added a commit that referenced this issue May 16, 2023
@soxofaan
Issue #112 TODO note about migrating AggregatorConfig to OpenEoBacken…
0defb67 
…dConfig
soxofaan added a commit that referenced this issue Jun 20, 2023
@soxofaan
Issue #112 WIP
7ad2884
soxofaan added a commit that referenced this issue Jun 22, 2023
@soxofaan
Issue #112 introduce OpenEoBackendConfig usage
0a6687a 
still need to be merged/unified with existing AggregatorConfig

related: Open-EO/openeo-python-driver#204
@soxofaan soxofaan self-assigned this Feb 1, 2024
@soxofaan soxofaan changed the title Move to OpenEoBackendConfig config system Migrate AggregatorConfig to OpenEoBackendConfig config system Feb 2, 2024
@soxofaan
Copy link
Member Author

soxofaan commented Feb 2, 2024 •
edited
Loading

related:

soxofaan added a commit that referenced this issue Feb 6, 2024
@soxofaan
Issue #112 Support aggregator_config as config var name to simplify…
d8c73af 
… config migration
soxofaan added a commit that referenced this issue Feb 6, 2024
@soxofaan
Issue #112 minor logging tweak
c7a095b
soxofaan added a commit to Open-EO/openeo-python-driver that referenced this issue Feb 7, 2024
@soxofaan
Eliminate need to subclass ConfigGetter
5397ae6 
to reduce OO overkill

related to Open-EO/openeo-aggregator#112
soxofaan added a commit that referenced this issue Feb 7, 2024
@soxofaan
Issue #112 introduce get_backend_config for AggregatorBackendConfig
0615689
soxofaan added a commit that referenced this issue Feb 7, 2024
@soxofaan
Issue #112 replace (test related) flask_error_handling config with cr…
aae2338 
…eate_app option
soxofaan added a commit that referenced this issue Feb 7, 2024
@soxofaan
Issue #112 port streaming_chunk_size config, introduce config_overrid…
d1f9b75 
…es for testing
soxofaan added a commit that referenced this issue Feb 7, 2024
@soxofaan
Bump version to 0.18.0 (#112 related)
fb0ccee
soxofaan added a commit that referenced this issue Feb 8, 2024
@soxofaan
Issue #112 Add support for standard `OpenEoBackendConfig.oidc_provide…
0864205 
…rs` and deprecate `AggregatorConfig.configured_oidc_providers`
soxofaan added a commit that referenced this issue Feb 8, 2024
@soxofaan
Issue #112 Improve return type hint of get_backend_config
d5da115
soxofaan added a commit that referenced this issue Feb 8, 2024
@soxofaan
Issue #112 Add support for standard `OpenEoBackendConfig.oidc_provide…
c805972 
…rs` and deprecate `AggregatorConfig.configured_oidc_providers`
soxofaan added a commit that referenced this issue Feb 8, 2024
@soxofaan
Issue #112 Improve return type hint of get_backend_config
a893678
soxofaan added a commit that referenced this issue Feb 8, 2024
@soxofaan
Remove auth_entitlement_check=False from configs
04b9983 
It's False by default
and removing it simplifies config migration #112
soxofaan added a commit that referenced this issue Feb 8, 2024
@soxofaan
Issue #112 move auth_entitlement_check config to AggregatorBackendConfig
793c8d9 
auth_entitlement_check is currently unused, so no migration path necessary
soxofaan added a commit that referenced this issue Feb 8, 2024
@soxofaan
Issue #112 move collection_whitelist to AggregatorBackendConfig
556d87e 
currently unused, so no migration path necessary
soxofaan added a commit that referenced this issue Feb 8, 2024
@soxofaan
Issue #112 move collection_whitelist to AggregatorBackendConfig
4b2f80f 
currently unused, so no migration path necessary

related to eu-cdse/openeo-cdse-infra#54
soxofaan added a commit that referenced this issue Feb 20, 2024
@soxofaan
Issue #112 move connections_cache_ttl to AggregatorBackendConfig
09171b8
soxofaan added a commit that referenced this issue Feb 21, 2024
@soxofaan
Issue #112 replace kazoo_client_factory config with zk_memoizer_tracking
47a02e6 
Old system in prime_caches depended on mutable configs, so some refactoring was required to move to alternative with unmutable configs
soxofaan added a commit that referenced this issue Feb 21, 2024
@soxofaan
Issue #112 move connections_cache_ttl to AggregatorBackendConfig
9d3ca4d
soxofaan added a commit that referenced this issue Feb 21, 2024
@soxofaan
Issue #112 replace kazoo_client_factory config with zk_memoizer_tracking
32cda56 
Old system in prime_caches depended on mutable configs, so some refactoring was required to move to alternative with unmutable configs
soxofaan added a commit that referenced this issue Feb 21, 2024
@soxofaan
Issue #112 Add AggregatorBackendConfig.zookeeper_prefix
185aa3e 
 and deprecate `AggregatorConfig.zookeeper_prefix`
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 remove remove AggregatorConfig.memoizer
95f8f65 
Replaced by `AggregatorBackendConfig.memoizer`
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 Eliminate unused AggregatorConfig arguments
71bdcab
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 Eliminate unused AggregatorConfig usage from prime_caches
36d6878
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 eliminate config fixture from conftest.py
30331d3
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 eliminate AggregatorConfig loading related logic
bce68fe
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 AggregatorConfig import cleanup
3ced197
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 some final cleanups
dd8527a
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 remove AggregatorConfig.memoizer
6877dfc 
Replaced by `AggregatorBackendConfig.memoizer`
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 Eliminate unused AggregatorConfig arguments
3f4fd30
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 Eliminate unused AggregatorConfig usage from prime_caches
c117f8b
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 eliminate config fixture from conftest.py
dd412d0
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 eliminate AggregatorConfig loading related logic
59c6c82
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 AggregatorConfig import cleanup
e2739c9
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 some final cleanups
c8b60fe
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 some final cleanups
d15bb53
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 Eliminate unused AggregatorConfig usage from prime_caches
915bb2f
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 eliminate config fixture from conftest.py
ef0c933
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 eliminate AggregatorConfig loading related logic
0c3aa8d
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 AggregatorConfig import cleanup
a63a803
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 some final cleanups
2d1674e
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 finalizing
c1e4549
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
Issue #112 remove AggregatorConfig definition from configs
85150ae
soxofaan added a commit that referenced this issue Mar 1, 2024
@soxofaan
fixup! Issue #112 remove AggregatorConfig definition from configs
6f8e8a8
soxofaan added a commit that referenced this issue Mar 4, 2024
@soxofaan
Issue #112 Remove now unused AggregatorConfig definition
15ee0a8
@soxofaan
Copy link
Member Author

soxofaan commented Mar 4, 2024

finalized with 15ee0a8

@soxofaan soxofaan closed this as completed Mar 4, 2024
soxofaan added a commit that referenced this issue Mar 5, 2024
@soxofaan
Issue #112 update README on AggregatorConfig removal
d055988
@soxofaan soxofaan mentioned this issue Apr 10, 2024
Closed
@soxofaan soxofaan mentioned this issue May 7, 2024
Closed
@soxofaan soxofaan mentioned this issue Jun 28, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@soxofaan soxofaan

Labels
architecture config devops technical debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@soxofaan