Skip to content

feat(docker-compose): add a way to change default datasource #8127

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

Open
wants to merge 224 commits into
base: main
Choose a base branch
from

Conversation

riipeckx
Copy link

@riipeckx riipeckx commented Oct 2, 2024

Summary

As I'd like to use PostgreSQL, I added a way to set the default datasource via environment variables.

sstojak1 and others added 30 commits June 14, 2024 17:26
…pache#7559)

* adding pull_request_id index to  pull_request_commits/comments tables

* change the pull_request_commits primary key columns order

* adding Apache license header

* only run for mysql

* adding support for postgres

---------

Co-authored-by: Josip Stojak <Josip.Stojak@infobip.com>
Co-authored-by: Josip Stojak <Josip.Stojak@infobip.com>
* feat: gerrit python plugin

Signed-off-by: Ji Bin <matrixji@live.com>

* feat: gerrit plugin support incremental sync

Signed-off-by: Ji Bin <matrixji@live.com>

---------

Signed-off-by: Ji Bin <matrixji@live.com>
Co-authored-by: Lynwee <linwei.hou@merico.dev>
…he#7599)

* hudson.maven.MavenModuleSet is now treated as a scope

* refactor: scope is now the default

---------

Co-authored-by: Josip Stojak <Josip.Stojak@infobip.com>
* fix: pr reviewers and assignees script version error

* fix: adapt pg test
* fix: git extractor commit rawdataorigin is empty

* fix: remove debugging print
…lt (apache#7660)

* fix(gitextractor): update database store, set it incremental by default

* fix(gitextractor): add `SetIncrementalMode` to store

* style(gitextractor): fix ci errors
* refactor(framework): add some logs and dispart `Init` from `CreateAndRunApiServer`

* refactor(framework): update initial steps
Co-authored-by: Startrekzky <kaiyun.zhang@merico.dev>
Co-authored-by: Startrekzky <kaiyun.zhang@merico.dev>
Co-authored-by: Startrekzky <kaiyun.zhang@merico.dev>
mintsweet and others added 9 commits October 8, 2024 17:35
Co-authored-by: Startrekzky <kaiyun.zhang@merico.dev>
* feat(sonarcloud): add org to the connections table

* feat: add sonarqube cloud connection config

* fix: sonarcloud now working

* fix: linting

* fix: copywriting for sonarqube cloud

* feat: added cq_issue_impacts table

* fix: unit test

* fix: adjust sonarqube cloud config

* feat: add sonarcloud dashboard (apache#8140)

Co-authored-by: Startrekzky <kaiyun.zhang@merico.dev>

---------

Co-authored-by: mintsweet <0x1304570@gmail.com>
Co-authored-by: Louis.z <louis.s4372121@gmail.com>
Co-authored-by: Startrekzky <kaiyun.zhang@merico.dev>
…th $__timeFilter(date) (apache#8147)

Co-authored-by: Volodymyr Shulga <vshulga@siftscience.com>
* feat(api_collector_stateful): handle case were response has records from both before & after createdAfter of last collection

* feat(circleci-plugin): incremental collection for pipelines

* feat(api_collector_stateful): expose Input collector arg for StatefulFinalizableEntity to collect data based on previous collection

* feat(circleci-plugin): incremental data collection for workflows

* feat(circleci-plugin): incremental data collection for jobs

* refactor(circleci-plugin): use common query param function

* refactor(circleci-plugin): use BuildQueryParamsWithPageToken func when collecting unfinished job details
…8136)

* feat: generating new github access token to every gitextractor task

Signed-off-by: Caio Queiroz <caiogqueiroz@gmail.com>

* feat: using DynamicGitUrl interface to implement the git url logic

* refactor: remove unused code

* fix: unit test

* fix: lint

---------

Signed-off-by: Caio Queiroz <caiogqueiroz@gmail.com>
- add env SKIP_SUBTASK_PROGRESS to decide wether skip subtask progress updating to db
d4x1 and others added 2 commits October 30, 2024 17:07
* feat(tapd): add more log when converting unicode

* feat(tapd): ignore errors when converting unicode fails
@klesh
Copy link
Contributor

klesh commented Nov 1, 2024

I am afraid this would affect helm chart and release procedure as well.
We need more input about this
@matrixji @JorgeGar @d4x1 What do you guys think?

@d4x1
Copy link
Contributor

d4x1 commented Nov 1, 2024

@klesh Yes hel charts should be updated too
@riipeckx https://github.com/apache/incubator-devlake-helm-chart/blob/main/charts/devlake/values.yaml

Startrekzky and others added 3 commits November 1, 2024 18:47
* bug: add missing migration script to register.go

* bug: add missing migration script to register.go - increaseProjectKeyLength

* bug: rename script column name

* increasing id instead of project_key

---------

Co-authored-by: Josip Stojak <Josip.Stojak@infobip.com>
@vi-mer
Copy link

vi-mer commented Nov 12, 2024

I'll check this out asap :)

usharerose and others added 3 commits November 14, 2024 10:47
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Nov 14, 2024
@riipeckx
Copy link
Author

riipeckx commented Nov 15, 2024

I'm working on helm charts updates in the given repo.

Copy link
Contributor

@klesh klesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most Grafana dashboards are using MySQL dialect AFAIK.
I don't think it is a good idea to touch them 😂, are you planning to rewrite all those queries.

@d4x1
Copy link
Contributor

d4x1 commented Nov 19, 2024

LGTM. Please consider @klesh 's comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ext This issue or PR relates to external components, such as Grafana pr-type/feature-development This PR is to develop a new feature size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.