Hello, can you confirm that the helm chart does no...
# helpa
Alessandro Mangone
11/29/2022, 9:45 AMHello, can you confirm that the helm chart does not support deploying lakefs using dynamodb as database?
The database type seems to be hardcoded: https://github.com/treeverse/charts/blob/master/charts/lakefs/templates/_env.tpl#L3-L4
b
Barak Amar
11/29/2022, 9:51 AM
Hi @Alessandro Mangone yes, currently it is hard code - I'll open an issue to fix this one until tomorrow
👍🏻 1
👍 1
Barak Amar
11/29/2022, 11:48 AM
Barak Amar
11/30/2022, 3:57 PM
Pull-request merged with fix for the above.
a
Alessandro Mangone
11/30/2022, 4:07 PMThank you @Barak Amar!
But by having a look at the code I can’t find any reference to dynamodb?
I’m looking at these lines, but I don’t know if they get overridden somewhere else.
Alessandro Mangone
11/30/2022, 4:07 PM(I’m not really proficient in helm charts syntax, so I might be missing something)
b
Barak Amar
11/30/2022, 4:12 PM
So, the change enables you to control the database type and additional configuration as part of
lakefsConfigBarak Amar
11/30/2022, 4:12 PM
Example:
Barak Amar
11/30/2022, 4:14 PM
create myvalues.yaml:
Copy code
secrets:
authEncryptSecretKey: random_string
lakefsConfig: |
database:
type: local
blockstore:
type: local Copy code
$ helm install -f myvalues.yaml my-lakefs lakefs/lakefsBarak Amar
11/30/2022, 4:14 PM
it will deploy lakefs with the above configuration - using local database and local block storage
Barak Amar
11/30/2022, 4:15 PM
in your case you can use 'database.type' 'dynamodb' (like in the example: https://docs.lakefs.io/reference/configuration.html#example-aws-deployment-with-dynamodb-database) to configure which table to use.
Barak Amar
11/30/2022, 4:16 PM
The change you referenced was made to keep the same defaults we had so far and enable any customization using lakefsConfig settings.
👍 1
a
Alessandro Mangone
11/30/2022, 4:19 PMThanks for the clarification! I’ll try updating the chart right now
lakefs 1
Alessandro Mangone
11/30/2022, 4:59 PMSeems to be working as expected! Thanks for the quick update! 🙂
b
Barak Amar
11/30/2022, 5:00 PM
@Alessandro Mangone note I've update the above snippet with additional secret field as it is required.
👍 1
a
Alessandro Mangone
11/30/2022, 5:01 PMYes I already had that from when I tried it using postgres, thanks
8 Views