Skip to content

Conversation

@robkooper
Copy link
Member

This is an initial helm chart

@robkooper
Copy link
Member Author

robkooper commented Sep 30, 2022

this is still work in progress, however we got it to work on local machine with docker/kubernetes

  • need to import keycloak.json realm @longshuicy is minifying it
  • need to import theme into keycloak container
  • need to setup redirect in keycloak.json file when importing
  • need to fix hardcode redirect in backend deployment
  • elasticsearch fails in backend, hardcoded url? WARNING:elastic_transport.node_pool:Node <Urllib3HttpNode(http://localhost:9200)> has failed for 2 times in a row, putting on 2 second timeout
  • different username/password for minio fails, hardcoded username/password in backend?
  • allow to configure external keycloak
  • setup externalsecrets as a way to not hardcode the secrets in the helm chart.

- add tar file of theme for keycloak
- use single hostname
@robkooper robkooper mentioned this pull request Oct 1, 2022
@robkooper
Copy link
Member Author

this fixes #112

@robkooper robkooper linked an issue Oct 1, 2022 that may be closed by this pull request
@longshuicy
Copy link
Member

longshuicy commented Oct 3, 2022

for minio i see these hardcoded in config.py?

 MINIO_SERVER_URL: str = "localhost:9000"
    MINIO_BUCKET_NAME: str = "clowder"
    MINIO_ACCESS_KEY: str = "minioadmin"
    MINIO_SECRET_KEY: str = "minioadmin"
    MINIO_UPLOAD_CHUNK_SIZE: int = 10 * 1024 * 1024

For elasticsearch I see those:

# Elasticsearch local config
    elasticsearch_url = "http://localhost:9200"
    elasticsearch_no_of_shards = 5
    elasticsearch_no_of_replicas = 5

@robkooper robkooper changed the title start of helmchart helmchart to deploy clowder2 in kubernetes Oct 7, 2022
@longshuicy longshuicy marked this pull request as ready for review November 10, 2022 19:58
@lmarini
Copy link
Member

lmarini commented Nov 15, 2022

Is this ready to go?

@robkooper
Copy link
Member Author

Did the hardcoded values in minio get fixed? One option is to merge for now and create some issues for the left over pieces.

@robkooper robkooper added helm Helm chart for Kubernetes deployment enhancement New feature or request labels Nov 15, 2022
@longshuicy
Copy link
Member

I created issues to capture the things left to do #200
Maybe a few are duplicated even but we'll see

@lmarini lmarini merged commit 83f55b2 into main Nov 16, 2022
@lmarini lmarini deleted the helm branch November 16, 2022 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request helm Helm chart for Kubernetes deployment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create helm chart

4 participants