Skip to content
This repository was archived by the owner on Dec 16, 2025. It is now read-only.

Prepare deployment#1

Merged
joshuai96 merged 9 commits intomainfrom
feat/local-deployment
Nov 29, 2023
Merged

Prepare deployment#1
joshuai96 merged 9 commits intomainfrom
feat/local-deployment

Conversation

@joshuai96
Copy link
Member

Setup and configure local deployment for development.

Ensure high reuseability to deploy to other infrastructure too.

Signed-off-by: Joshua Irmer <irmer@gonicus.de>
Signed-off-by: Joshua Irmer <irmer@gonicus.de>
@joshuai96 joshuai96 force-pushed the feat/local-deployment branch from 7291b46 to 5391e3f Compare September 21, 2023 13:35
Signed-off-by: Joshua Irmer <irmer@gonicus.de>
Signed-off-by: Joshua Irmer <irmer@gonicus.de>
@joshuai96 joshuai96 force-pushed the feat/local-deployment branch from c33cbbd to 2f6ebaf Compare September 21, 2023 14:04
@joshuai96 joshuai96 marked this pull request as ready for review September 21, 2023 14:07
@joshuai96 joshuai96 requested a review from joshmue September 21, 2023 14:08
jschoone and others added 4 commits September 28, 2023 14:30
Signed-off-by: Joshua Irmer <irmer@gonicus.de>
Signed-off-by: Joshua Irmer <irmer@gonicus.de>
Signed-off-by: Joshua Irmer <irmer@gonicus.de>
}

http://api.test {
reverse_proxy scs-status-page-oathkeeper:4455
Copy link
Contributor

Choose a reason for hiding this comment

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

This will possibly lead caddy to establish a permanent/kept-alive connection to a single backend, preventing load balancing. Should be fine for the kind environment, though.

Copy link
Contributor

Choose a reason for hiding this comment

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

As discussed, using Gateway API would probably help here; Does not strictly block merging, though.

@joshuai96

Signed-off-by: Joshua Irmer <irmer@gonicus.de>
@fkr
Copy link
Member

fkr commented Nov 20, 2023

Why is this still open? It was approved Oct. 20th, it was invoiced but is still open (a month later)

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.

4 participants