Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

feat: Initial support for graceful shutdown #789

Merged
merged 33 commits into from
May 9, 2022

Conversation

arthurpitman
Copy link
Collaborator

@arthurpitman arthurpitman commented Apr 27, 2022

Addresses most of #729, adding initial support for graceful shutdown. Still to be added is support for Keptn clients, which will follow once keptn/go-utils has been updated appropriately.

Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
internal/credentials/k8s_secret_reader.go Show resolved Hide resolved
internal/dynatrace/events_client.go Show resolved Hide resolved
internal/dynatrace/events_client.go Show resolved Hide resolved
internal/dynatrace/events_client.go Show resolved Hide resolved
internal/monitoring/problem_notification_creation.go Outdated Show resolved Hide resolved
internal/onboard/service_sync.go Show resolved Hide resolved
@arthurpitman arthurpitman force-pushed the feature/729/add-context branch 2 times, most recently from 232ba0c to ebf5038 Compare April 28, 2022 10:36
internal/health/health_endpoint.go Show resolved Hide resolved
internal/health/health_endpoint.go Show resolved Hide resolved
internal/health/health_endpoint.go Show resolved Hide resolved
internal/health/health_endpoint.go Show resolved Hide resolved
internal/health/health_endpoint.go Show resolved Hide resolved
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
Signed-off-by: Arthur Pitman <[email protected]>
cmd/main.go Show resolved Hide resolved
internal/event_handler/error_handler.go Show resolved Hide resolved
internal/event_handler/no_op_handler.go Show resolved Hide resolved
internal/problem/problem_event_handler.go Show resolved Hide resolved
cmd/then_context.go Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented May 6, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 13 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2022

The following Docker Images have been built:
keptncontrib/dynatrace-service:0.23.0-dev-PR-789,keptncontrib/dynatrace-service:0.23.0-dev-PR-789.202205060940

Copy link
Contributor

@j-poecher j-poecher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@arthurpitman arthurpitman merged commit 6daf187 into master May 9, 2022
@arthurpitman arthurpitman deleted the feature/729/add-context branch May 9, 2022 11:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants