Skip to content

Redirect some traffic to fsharp #3431

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

Merged
merged 13 commits into from
Feb 3, 2022
Merged

Redirect some traffic to fsharp #3431

merged 13 commits into from
Feb 3, 2022

Conversation

pbiggar
Copy link
Member

@pbiggar pbiggar commented Feb 1, 2022

This adds a path from the main darklang ingress (the load balancer for darklang.com/*) to the apiserver. I've made it go there just for the trydarkfsharp canvas, which will be enough to test it out.

This will be deployed manually so I can quickly revert if it's going badly.

@pbiggar
Copy link
Member Author

pbiggar commented Feb 3, 2022

I tested all this manually and discovered that nginx wasn't configured correctly. This fixes the nginx config for apiserver, and also for bwdserver (which I didn't test, just applied the same changes).

Alas, this doesn't enable access to the new apiserver, as I couldn't get the ingress (the main load balancer for darklang.com) to point to the new service, as it's in a different kubernetes namespace. I tried to use a workaround I found on stackoverflow, but it doesn't work on GKE. I have a request in with google support and I'll see what they say, but we don't currently have a solution for now.

Note that one solution is to put everything in the same namespace. This is strictly worse from a security perspective, especially since the darklang namespace has a default-deny policy that would be error-prone to add to the default namespace.

@pbiggar pbiggar merged commit cf5f36b into main Feb 3, 2022
@pbiggar pbiggar deleted the paul/some-traffic-to-fsharp branch February 3, 2022 07:52
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

Successfully merging this pull request may close these issues.

1 participant