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

No downtime changes #140

Merged
merged 46 commits into from
Aug 30, 2024
Merged

No downtime changes #140

merged 46 commits into from
Aug 30, 2024

Conversation

arlowatts
Copy link
Contributor

This PR converts DeploymentConfig to Deployment and implements no-downtime deployments in the PNET APIs.

arlowatts added 30 commits August 23, 2024 11:27
@arlowatts
Update Chart.yaml
e8e6364
@arlowatts
Update to use Deployment instead of DeploymentConfig
782fd9c
@arlowatts
Reorganize values.yaml
c06b216
@arlowatts
Merge branch 'main' into no-downtime-changes
6ed2450
@arlowatts
Increase pnet-proxy response delay for testing
0fe043c
@arlowatts
Change pnet-proxy source for testing
e2fb3e0
@arlowatts
Add health check scripts
96e3f4e
@arlowatts
Update Chart.yaml
db3bba6
@arlowatts
Include shell script in build
4d9f443
@arlowatts
Add directive to copy shell scripts to build output
f2f0bec
@arlowatts
Remove jq dependency
b647542
@arlowatts
Make health check scripts executable
627e1bf
@arlowatts
Fix transaction submit path
ca2171d
@arlowatts
Add health check secrets for service account
de19e8e
@arlowatts
Add health check secrets to Helm chart
128d181
@arlowatts
Remove unused and unnecessary lines from csproj files
53f0ea4
@arlowatts
Update readiness probe to use health check script
7fb0a14
@arlowatts
Clean up health check script
29edc8c
@arlowatts
Revert changes to pnet-proxy
1b28a6f
@arlowatts
Reduce delay to 0 to optimize health checks
6198115
@arlowatts
Switch pnet-proxy to build from no-downtime-changes branch
8e786db
@arlowatts
Add minimum time delay to guarantee that a response is sent
126b6bc
@arlowatts
Increase readiness probe delay
06e94d2
@arlowatts
Add preStop hook to avoid transaction loss
ea0640a
@arlowatts
Explicitly quote strings
96bfa1f
@arlowatts
Add startup probe for smoother start
592b406
@arlowatts
Add Kong-Request-ID header to health check
5d541d6
@arlowatts
Increase failure threshold and termination grace period
797ee72
@arlowatts
Implement health check endpoint at /healthz
a10480a
@arlowatts
Set health check endpoint to /healthz
cc6e500
@arlowatts arlowatts requested a review from daveb-hni August 30, 2024 17:03
daveb-hni
daveb-hni approved these changes Aug 30, 2024
View reviewed changes
Copy link
Collaborator

@daveb-hni daveb-hni left a comment

Choose a reason for hiding this comment

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

Approve

@arlowatts arlowatts merged commit fe3db0b into main Aug 30, 2024
12 checks passed
@arlowatts arlowatts deleted the no-downtime-changes branch August 30, 2024 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daveb-hni daveb-hni daveb-hni approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@arlowatts @daveb-hni