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

KOTS incluster registry S3 dialog doesn't request region #9800

Closed
mrzarquon opened this issue May 5, 2022 · 1 comment · Fixed by #10577
Closed

KOTS incluster registry S3 dialog doesn't request region #9800

mrzarquon opened this issue May 5, 2022 · 1 comment · Fixed by #10577
Assignees
Labels
self-hosted: eks Self hosted support for AWS EKS team: delivery Issue belongs to the self-hosted team type: bug Something isn't working

Comments

@mrzarquon
Copy link
Contributor

Bug description

The S3 region defaults to "local" in the installation if one is just using S3 for the the registry backend and not anything else. In order to fix this, the deployment can be edited by hand, but a subsequent redeployment by kots when changing other settings will remove the change.

Introduced by #9701

Steps to reproduce

Install in AWS using the getting starting guide and provide S3 credentials for the incluster registry.

kubectl edit deployment registry -n gitpod

Change region to correct region, restart just the registry deployment and it works, but doing a redeployment if making another change (or updating) will revert the change.

Workspace affected

No response

Expected behavior

No response

Example repository

No response

Anything else?

No response

@mrsimonemms
Copy link
Contributor

mrsimonemms commented Jun 9, 2022

This is caused by the separation of the S3 credentials between the object store and the registry - originally, these were linked but now they are (correctly) made independent.

How to fix this ticket:

  1. add a region and endpoint param to the S3Storage struct
  2. add the region and endpoint param to the KOTS config

NB I'm proposing adding the endpoint (and defaulting to s3.amazonaws.com) to avoid any problems when we eventually get a customer who wants to use an S3 compatible storage in not-AWS - IT WILL HAPPEN

This is causing a customer issue, so I will look at this in next 24 hours. @jimmybrancaccio is liaising with customer and had a test instance all set up

IMPORTANT once this is in main, we should cut a KOTS hotfix release with this in it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
self-hosted: eks Self hosted support for AWS EKS team: delivery Issue belongs to the self-hosted team type: bug Something isn't working
Projects
No open projects
6 participants