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

'Give deployer complete control over Trino additionalCatalogs' #65

Closed
mikej888 opened this issue Oct 23, 2023 · 1 comment
Closed

'Give deployer complete control over Trino additionalCatalogs' #65

mikej888 opened this issue Oct 23, 2023 · 1 comment

Comments

@mikej888
Copy link
Collaborator

In relation to monolithic_mk8s_example/1-vm-setup-and-deploy.yaml (originally from DARE-SeRP-Dev-Deployment Version: 43f688f (Thu Sep 14 17:15:42 2023 +0100) main branch).

I updated the appTrino additionalCatalogs minio catalog name in teleport-values.yaml to be edinburgh (a name that might be more usable for a researcher than minio. When I requested ArgoCD rereploy the Trino coordinator, the Trino pod pod/app-trino-coordinator-.... failed with:

2023-10-19T15:45:04.632Z	ERROR	main	io.trino.server.Server
No factory for connector 'hive_hadoop2'.  Available factories: [hive,
ignite, delta_lake, memory, kudu, blackhole, kinesis, redis, accumulo,
gsheets, elasticsearch, jmx, postgresql, redshift, sqlserver,
localfile, example_http, tpch, iceberg, mysql, bigquery, prometheus,
mongodb, pinot, tpcds, hudi, oracle, mariadb, druid, trino, cassandra,
clickhouse, kafka, singlestore, atop, raptor_legacy, trino_thrift,
phoenix5]

This seems to be because DARE-Trino's charts/trino/values.yaml defines:

additionalCatalogs:
    minio: |
      connector.name = hive-hadoop2
      ...

If there is no minio catalog in any overriding values, then this default catalog remains. So, in my case, I had edinburgh and minio catalogs.

Could values be updated to provide an empty additionalCatalogs block and/or could the charts/trino/templates/configmap-catalog.yaml be updated to replace this block with that from a deployer? I think any deployer should have complete control over the catalogs deployed and their names.

@mikej888
Copy link
Collaborator Author

mikej888 commented Jan 8, 2024

Closing as fixed by fix(chart): set default extra catalogs to empty (#70).

@mikej888 mikej888 closed this as completed Jan 8, 2024
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

No branches or pull requests

1 participant