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

Improvement to schedular config file #450

Closed
sushanthakumar opened this issue Jan 5, 2021 · 1 comment
Closed

Improvement to schedular config file #450

sushanthakumar opened this issue Jan 5, 2021 · 1 comment

Comments

@sushanthakumar
Copy link
Collaborator

Issue/Feature Description:
Currently schedular config file used for performance collection is a json file having an entry for each storage registered for storage. Each entry had storage id and metric parameters like below

{"storages": [{"id": "e744abbc-9bb2-41f9-bbc3-742bf294347f", "array_polling": {"perf_collection": true, "interval": 12, "is_historic": true}}]}

During storage delete, these entries need to be deleted. All the entries in array need to be traversed

Need to relook if this can be maintained as map of storage entries

Why this issue to fixed / feature is needed(give scenarios or use cases):
During storage delete, these entries need to be deleted. All the entries in array need to be traversed

How to reproduce, in case of a bug:

Other Notes / Environment Information: (Please give the env information, log link or any useful information for this issue)
Proposed solution: Backends which do not support query alert mechanism can throw unsupported feature exception so that user can handle accordingly

@sushanthakumar
Copy link
Collaborator Author

Addressed through #451, closing the issue

amanr032 pushed a commit to amanr032/delfin that referenced this issue Dec 5, 2022
… etcd containers to improve the reliability of SODA services when deployed as containers (sodafoundation#450)

* Added restart always policy to api, controller, dock and auth containers

* Added restart policy always for the etcd container

* Added stop and remove etcd container and volume based on configuration. Updated the container config for etcd with volume information.

* Added missing logical operator from condiional check of etcd db removal
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