-
-
Notifications
You must be signed in to change notification settings - Fork 261
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
Module for resource management / dynamic resources #1351
Comments
As per RFC4, this Issue has been inactive for 90 days. In order to manage maintenance burden, it will be automatically closed in 7 days. |
@matthesrieke may be related: #1636 |
Thanks, will join the conversation. |
This Issue has been inactive for 90 days. As per RFC4, in order to manage maintenance burden, it will be automatically closed in 7 days. |
Is there any statement from the maintainers (@tomkralidis @justb4 ) on this? I saw #1636 was scheduled for MS 0.17 & 0.18, but as far I can see there are open discussion threads. |
Is your feature request related to a problem? Please describe.
Currently, pygeoapi does not allow to add new resources to the running instance. Resource are defined via config and a restart is required to enable new resources. This feature request is a first step into the goal to allow dynamic additional/removal/update (basically CRUD) of resources at runtime.
Describe the solution you'd like
The idea of this feature request is to refactor the
API
class to use a dedicated class for resource management. Currently,API
uses a lot ofself.config['resources']
style access. By introducing a dedicatedResourceRegistry
(abstract class), the architecture will allow to replace the config-based implementation with a CRUD style registry, even controlled by an API to allow a transactions.The implementing class for
ResourceRegistry
can be controlled via the configuration.Describe alternatives you've considered
Alternatives are to implement some workarounds in the
API
class, but they would make the class even more complex.Additional context
I have seen some related requests in the GitHub discussions here:
A WIP branch is already in progress: master...52North:pygeoapi:feature/resource-registry
The text was updated successfully, but these errors were encountered: