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

#111/#294/#297 Add health check and indexes to persistence #310

Merged
merged 19 commits into from
Apr 3, 2023
Merged

#111/#294/#297 Add health check and indexes to persistence #310

merged 19 commits into from
Apr 3, 2023

Conversation

TWilkin
Copy link
Owner

@TWilkin TWilkin commented Mar 27, 2023 •
edited
Loading

  • Resolves Add indexes to database columns #294 by adding indexes to type, entity, action and timestamp.
  • Perform updates for persistence, common and common-test.
  • Add FileService to common library to include the touch method.
  • Add HealthService to persistence to monitor whether MQTT and the database are accessible.
  • Add liveness/readiness checks to persistence deployment.
  • Rename database backup job.
  • Fix version.sh script as the service name was hard-coded to energy-monitor.
  • Add missing tests for persistence service.

@TWilkin
Bump persistence to v0.2.0
bac1955
@TWilkin
Fix hard-coded service name in version script
400499c
@TWilkin
#294 Updating packages for persistence
f385b59
@TWilkin
Update common and common-test dependencies
ff2727a
@TWilkin
#294 Add the index to each column
f900b0f
@TWilkin
#111 Add HealthService to persistence
e8b9282
@TWilkin
#111 Add FileService with touch to common
ffe8049
@TWilkin
#111 Touch the file each time the health is okay
f5b1a8b
@TWilkin
#111 Also get the health status of the database
0bfb295
@TWilkin
#111 Missed underscore types dependency
fb181de
@TWilkin
#111 Include health check shell script in common dist
e75a437
@TWilkin
#111 Add probes to persistence
26d7737
@TWilkin
Rename database-backup
877ba78
@TWilkin
#111 Fixes to health service
c77679a
@TWilkin
#294 Use default names for index
9a9611a
@TWilkin
#111 Add tests for FileService to common library
78ec564
@TWilkin
#111 Add tests for HealthService
c0140ba
@TWilkin
#111 Also add a test for HealthService.start
bb85402
@TWilkin TWilkin added bug Something isn't working enhancement New feature or request persistence Relating to the persistence service tests Relating to unit tests labels Mar 27, 2023
@TWilkin TWilkin added this to the Kubernetes milestone Mar 27, 2023
@TWilkin TWilkin added documentation Improvements or additions to documentation kubernetes Relating to the Kubernetes deployment labels Mar 27, 2023
@TWilkin TWilkin changed the title #111/#294 Add health check and indexes to persistence #111/#294/#297 Add health check and indexes to persistence Mar 27, 2023
@TWilkin TWilkin merged commit 0f637b5 into master Apr 3, 2023
@TWilkin TWilkin deleted the 294.persistence-indexes branch April 3, 2023 12:30
@TWilkin TWilkin linked an issue Apr 3, 2023 that may be closed by this pull request
Add health checks to services #111
Closed
@TWilkin TWilkin linked an issue Apr 3, 2023 that may be closed by this pull request
Add tests to all services #297
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request kubernetes Relating to the Kubernetes deployment persistence Relating to the persistence service tests Relating to unit tests
Projects
None yet
Milestone
Kubernetes
Development

Successfully merging this pull request may close these issues.

Add tests to all services Add indexes to database columns Add health checks to services
1 participant
@TWilkin