You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by "Tonkonozhenko (via GitHub)" <gi...@apache.org> on 2024/01/23 22:23:10 UTC
[PR] Support postgresql [incubator-devlake-helm-chart]
Tonkonozhenko opened a new pull request, #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "klesh (via GitHub)" <gi...@apache.org>.
klesh commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2050940120
@Tonkonozhenko Maybe you could file a feature request for Grafana?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "klesh (via GitHub)" <gi...@apache.org>.
klesh commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-1907248132
@JorgeGar @matrixji @ZhangNing10 Would you like to take a look? Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "Tonkonozhenko (via GitHub)" <gi...@apache.org>.
Tonkonozhenko commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2037208511
@klesh @JorgeGar @matrixji @ZhangNing10 did you think about supporting Postgres?
I decided to switch to PG completely because I want to collect other data too. I want to use some Airflow DAGs to populate it and do transformations with DBT, but I definitely can't rely on https://github.com/dbeatty10/dbt-mysql.
Looks like on the backend everything works for me with a small change: https://github.com/apache/incubator-devlake/pull/7284
The issue is with Grafana. I migrated several dashboards (Dora, Dora Validation, Gitlab) to PostgreSQL, but I don't know how to make a generic solution.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "Tonkonozhenko (via GitHub)" <gi...@apache.org>.
Tonkonozhenko commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2049509588
@klesh the only option I see is extracting SQL queries from dashboards' JSONs and having two queries (or if conditions) per database.
```
grafana/
dashboards/
DORA.json
queries/
postgresql/
01 Overall DORA Metrics.sql
mysql/
01 Overall DORA Metrics.sql
```
It would be great if Grafana had go templating in queries, but it only has it in alerting.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "richardinemi (via GitHub)" <gi...@apache.org>.
richardinemi commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2058337801
> @richardinemi There is a Space at the beginning of your Connection String, right after the double-quote.
I don’t understand why i have this space if it’s was the problem.
This my config for DB connection.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "richardinemi (via GitHub)" <gi...@apache.org>.
richardinemi commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2056986111
> @klesh I'll try. For now, maybe add support to the chart? I have a working fork: [main...Tonkonozhenko:incubator-devlake-helm-chart:main](https://github.com/apache/incubator-devlake-helm-chart/compare/main...Tonkonozhenko:incubator-devlake-helm-chart:main)
Hello,
I have working with the same subject but i have this error on devlake when he try first connection to Postgres.
>>> panic: parse " postgresql://merico:merico@devlake-pg-pgsql:5432/lake": first path segment in URL cannot contain colon
Wraps: (2) parse " postgresql://merico:merico@devlake-pg-pgsql:5432/lake"
Wraps: (3) first path segment in URL cannot contain colon
Error types: (1) *hintdetail.withDetail (2) *url.Error (3) *errors.errorString
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "JorgeGar (via GitHub)" <gi...@apache.org>.
JorgeGar commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-1907476006
The issue with having PSQL supported by the helm chart is that it sends a confusing message. The chart is offering a toolset that includes both devlake and the grafana boards, but PostgreSQL is only supported by the former.
We removed it from the chart because of this very same reason.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "cosmorocker (via GitHub)" <gi...@apache.org>.
cosmorocker commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2056994873
@richardinemi I see a suspicious space at the beginning of your error before " postgresql://". I suggest checking it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "matrixji (via GitHub)" <gi...@apache.org>.
matrixji commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-1907253076
Do we need to support Postgres? As #148 says, the devlake-dashboard(Grafana) currently supports MySQL syntax only.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "klesh (via GitHub)" <gi...@apache.org>.
klesh commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2042284390
@Tonkonozhenko It would be nice if we could support both. Just like you, we have yet to figure out a generic solution so far. 😂
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "ZhangNing10 (via GitHub)" <gi...@apache.org>.
ZhangNing10 closed pull request #240: Support postgresql
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "Tonkonozhenko (via GitHub)" <gi...@apache.org>.
Tonkonozhenko commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2051579569
@klesh I'll try. For now, maybe add support to the chart? I have a working fork: https://github.com/apache/incubator-devlake-helm-chart/compare/main...Tonkonozhenko:incubator-devlake-helm-chart:main
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "klesh (via GitHub)" <gi...@apache.org>.
klesh commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2058164637
@richardinemi There is a Space at the beginning of your Connection String, right after the double-quote.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Support postgresql [incubator-devlake-helm-chart]
Posted by "richardinemi (via GitHub)" <gi...@apache.org>.
richardinemi commented on PR #240:
URL: https://github.com/apache/incubator-devlake-helm-chart/pull/240#issuecomment-2058431627
> @richardinemi There is a Space at the beginning of your Connection String, right after the double-quote.
I found the problem. It’s work now.
Thank you @klesh .
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org