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

Get & Store Multiple in Configuration Store RPC Interface #175

Open
davidraker opened this issue Jun 14, 2023 · 0 comments
Open

Get & Store Multiple in Configuration Store RPC Interface #175

davidraker opened this issue Jun 14, 2023 · 0 comments

Comments

@davidraker
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Currently, to store configurations in the config store, each must be stored individually with a separate RPC request. This can be very time-intensive when storing a large number of files. While testing automated deployment of a building with 1000+ device entries, ansible needed an hour to install them all.

A similar problem occurs with the web API (when managing devices) because it must request all these files using the management methods; first requesting a list and then requesting each individual file. This is not as time consuming as storing the files, but still slows down considerably for large device sets.

Describe the solution you'd like

The Configuration Store should have RPC methods for storing and retrieving multiple configurations at once. When these methods are called, all of the updates would be made in a single round-trip, and callbacks would not be triggered until the process is complete.
These should be available as "manage" methods to support use cases such as the web API.

Describe alternatives you've considered

  • The storing of many files simultaneously could be done by directly modifying the configuration store JSON file, but this probably would not be able to update the agent if it were running, and violates the expectation of the configuration store service itself.

  • The web API use case could probably be supported by new methods on the driver agent itself instead of interacting with the configuration store.

Additional context
Add any other context or screenshots about the feature request here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant