Allow dependabot to upgrade database versions used in integration tests #5485
Labels
enhancement
good first issue
Good for beginners
help wanted
Features that maintainers are willing to accept but do not have cycles to implement
Comments
yurishkuro
added
enhancement
help wanted
4 tasks
yurishkuro
pushed a commit
that referenced
this issue
May 28, 2024
) ## Which problem is this PR solving? Part of #5485 ## Description of the changes * updated `script/es-integration-test.sh` to use docker compose files * updated `.github/workflows/ci-elasticsearch.yml` now it only passes the major version for elasticsearch * updated `.github/workflows/ci-opensearch.yml` now it only passes major version for opensearch * added docker compose files at `docker-compose/elasticsearch` and `docker-compose/opensearch` * removed` setup_es` and `setup_opensearch` function and a generalized `setup_db()` function ## How was this change tested? - CI ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Your Name <[email protected]> Signed-off-by: mehul gautam <[email protected]> Co-authored-by: Your Name <[email protected]>
|
@yurishkuro can you assign me this issue I would also like to work on Kafka and Cassandra ci |
4 tasks
yurishkuro
added a commit
that referenced
this issue
May 31, 2024
## Which problem is this PR solving? Part of #5485 ## Description of the changes * In `script/kafka-integration-test.sh` initialized kafka by docker compose * updated `.github/workflow/ci-kafka.yml` to take logs from docker compose * added comments in `script/kafka-integration-test.sh` and fixed a typo * add new locations to dependabot config to allow it to upgrade versions in the new docker compose files ## How was this change tested? - CI ## Checklist - [ ] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [ ] I have signed all commits - [ ] I have added unit tests for the new functionality - [ ] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Your Name <[email protected]> Signed-off-by: mehul gautam <[email protected]> Signed-off-by: Yuri Shkuro <[email protected]> Co-authored-by: Your Name <[email protected]> Co-authored-by: Yuri Shkuro <[email protected]>
varshith257
pushed a commit
to varshith257/jaeger
that referenced
this issue
Jun 2, 2024
…egertracing#5490) ## Which problem is this PR solving? Part of jaegertracing#5485 ## Description of the changes * updated `script/es-integration-test.sh` to use docker compose files * updated `.github/workflows/ci-elasticsearch.yml` now it only passes the major version for elasticsearch * updated `.github/workflows/ci-opensearch.yml` now it only passes major version for opensearch * added docker compose files at `docker-compose/elasticsearch` and `docker-compose/opensearch` * removed` setup_es` and `setup_opensearch` function and a generalized `setup_db()` function ## How was this change tested? - CI ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Your Name <[email protected]> Signed-off-by: mehul gautam <[email protected]> Co-authored-by: Your Name <[email protected]> Signed-off-by: Vamshi Maskuri <[email protected]>
varshith257
pushed a commit
to varshith257/jaeger
that referenced
this issue
Jun 2, 2024
## Which problem is this PR solving? Part of jaegertracing#5485 ## Description of the changes * In `script/kafka-integration-test.sh` initialized kafka by docker compose * updated `.github/workflow/ci-kafka.yml` to take logs from docker compose * added comments in `script/kafka-integration-test.sh` and fixed a typo * add new locations to dependabot config to allow it to upgrade versions in the new docker compose files ## How was this change tested? - CI ## Checklist - [ ] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [ ] I have signed all commits - [ ] I have added unit tests for the new functionality - [ ] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Your Name <[email protected]> Signed-off-by: mehul gautam <[email protected]> Signed-off-by: Yuri Shkuro <[email protected]> Co-authored-by: Your Name <[email protected]> Co-authored-by: Yuri Shkuro <[email protected]> Signed-off-by: Vamshi Maskuri <[email protected]>
4 tasks
yurishkuro
pushed a commit
that referenced
this issue
Jun 3, 2024
## Which problem is this PR solving? - #5485 ## Description of the changes - updated `cassandra-integration-test.sh` to use docker compose - update dependabot config ## How was this change tested? - `bash scripts/cassandra-integration-test.sh 3 v003 v1` ` bash scripts/cassandra-integration-test.sh 3 v003 v2 ` bash `scripts/cassandra-integration-test.sh 4 v004 v1` `scripts/cassandra-integration-test.sh 4 v004 v2` ## Checklist - [ ] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [ ] I have signed all commits - [ ] I have added unit tests for the new functionality - [ ] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Your Name <[email protected]> Co-authored-by: Your Name <[email protected]>
4 tasks
yurishkuro
pushed a commit
that referenced
this issue
Jun 10, 2024
## Which problem is this PR solving? - #5485 ## Description of the changes - changing how major version is passed eg form 7.x -> 7 ## How was this change tested? - ## Checklist - [ ] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [ ] I have signed all commits - [ ] I have added unit tests for the new functionality - [ ] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: mehul gautam <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In the storage integration tests we run backends using Docker images. The versions for those images are hardcoded in various places:
jaeger/.github/workflows/ci-elasticsearch.yml
Line 28 in 584b6ff
jaeger/scripts/kafka-integration-test.sh
Line 21 in 584b6ff
None of these places are supported by the
dependabot, so we're not getting regular upgrades and not testing via the latest versions of the storage backends.One solution could be to use docker-compose files to bring the databases up instead of running a docker command directly. E.g. we could have
docker-compose-elasticsearch-v7.yamlanddocker-compose-elasticsearch-v8.yamland simplify the matrix in the workflow to only specify the major version and then pick the right file at runtime. Then wen can configure dependabot to look at those docker-compose files where it should be able to upgrade the versions as needed.We could try it with opensearch and see if dependabot will upgrade the versions. We also need to be careful to prevent dependabot from upgrading the major version, e.g. integration test against v1.x of opensearch should continue running against v1.x (whichever the latest release is for that major), not bump it to v2.x which is the absolute latest across majors.
Tracker
The text was updated successfully, but these errors were encountered: