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.

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

Setting Ingress to use the active Vault Pod #257

Closed
wants to merge 1 commit into from
Closed

Setting Ingress to use the active Vault Pod #257

wants to merge 1 commit into from

Conversation

skj-dev
Copy link
Contributor

@skj-dev skj-dev commented Apr 13, 2020

The Ingress service defaults to the vault service which contains all the running Pods. For some operations, this can cause a error when a standby Pod happens to be connected to with the following message:Error [action description]: redirect would cause protocol downgrade.

Since the chart creates a vault-active service that points to the currently active Pod, this PR adds the -acvive suffix to the service used by the Ingress so the active Pod is always being talked to by clients, which alleviates the above error.

Signed-off-by: Sean Johnson <sean@ttys0.net>
@hashicorp-cla
Copy link

hashicorp-cla commented Apr 13, 2020

CLA assistant check
All committers have signed the CLA.

@pcman312 pcman312 added bug Something isn't working chart Area: helm chart labels Apr 13, 2020
@georgekaz
Copy link
Contributor

I came across this too but this solution is not good because it only creates the -active service if you're using ha. I made a change to achieve the same but with some extra logic. I'll put that PR up but I don't think this one should be accepted Sorry @ttyS0 .

@georgekaz
Copy link
Contributor

This PR achieves the same: #270

@skj-dev
Copy link
Contributor Author

skj-dev commented Apr 22, 2020

@georgekaz has a better approach.

@skj-dev skj-dev closed this Apr 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working chart Area: helm chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants