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

Clean up obsolete 'version' tag from docker-compose files #5826

Merged
merged 1 commit into from
Aug 11, 2024

Conversation

vvs-personalstash
Copy link
Contributor

Description of the changes

  • As the latest updates in docker compose has deprecated the 'version' tag.This Pr aims to cleanup the version tags in the various docker-compose files so as to cleanup the log currently displaying 'version' is obselete when we run a docker-compose file.

Checklist

Signed-off-by: vvs-personalstash <viralkverma0987@gmail.com>
@vvs-personalstash vvs-personalstash requested a review from a team as a code owner August 11, 2024 21:01
@dosubot dosubot bot added area/storage docker Pull requests that update Docker code labels Aug 11, 2024
@yurishkuro yurishkuro added the changelog:ci Change related to continuous integration / testing label Aug 11, 2024
Copy link
Member

@yurishkuro yurishkuro left a comment

Choose a reason for hiding this comment

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

Thanks!

@yurishkuro yurishkuro enabled auto-merge (squash) August 11, 2024 21:46
@yurishkuro
Copy link
Member

hint: to make checkbox as checked you just need to put x in it: * [x] (you can also do it directly on the webpage by clicking)

Copy link

codecov bot commented Aug 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.81%. Comparing base (a6ccf56) to head (a91bdea).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5826      +/-   ##
==========================================
+ Coverage   96.79%   96.81%   +0.01%     
==========================================
  Files         342      342              
  Lines       16525    16525              
==========================================
+ Hits        15996    15998       +2     
+ Misses        341      340       -1     
+ Partials      188      187       -1     
Flag Coverage Δ
badger_v1 8.05% <ø> (ø)
badger_v2 1.81% <ø> (ø)
cassandra-3.x-v1 16.61% <ø> (ø)
cassandra-3.x-v2 1.74% <ø> (ø)
cassandra-4.x-v1 16.61% <ø> (ø)
cassandra-4.x-v2 1.74% <ø> (ø)
elasticsearch-6.x-v1 18.77% <ø> (-0.02%) ⬇️
elasticsearch-7.x-v1 18.84% <ø> (ø)
elasticsearch-8.x-v1 19.03% <ø> (+0.01%) ⬆️
elasticsearch-8.x-v2 1.81% <ø> (ø)
grpc_v1 9.52% <ø> (ø)
grpc_v2 7.14% <ø> (ø)
kafka-v1 9.74% <ø> (ø)
kafka-v2 1.81% <ø> (ø)
memory_v2 1.81% <ø> (ø)
opensearch-1.x-v1 18.89% <ø> (+0.01%) ⬆️
opensearch-2.x-v1 18.89% <ø> (+0.01%) ⬆️
opensearch-2.x-v2 1.81% <ø> (ø)
unittests 95.29% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yurishkuro yurishkuro merged commit 666c208 into jaegertracing:main Aug 11, 2024
45 of 48 checks passed
@yurishkuro yurishkuro changed the title Clean up obselete 'version' tag from docker-compose files Clean up obsolete 'version' tag from docker-compose files Aug 11, 2024
JaredTan95 pushed a commit to JaredTan95/jaeger that referenced this pull request Aug 13, 2024
…ing#5826)

## Description of the changes
- As the latest updates in docker compose has deprecated the 'version'
tag.This Pr aims to cleanup the version tags in the various
docker-compose files so as to cleanup the log currently displaying
```'version' is obselete ``` when we run a docker-compose file.

## 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: vvs-personalstash <viralkverma0987@gmail.com>
Signed-off-by: Jared Tan <jian.tan@daocloud.io>
JaredTan95 pushed a commit to JaredTan95/jaeger that referenced this pull request Aug 14, 2024
…ing#5826)

## Description of the changes
- As the latest updates in docker compose has deprecated the 'version'
tag.This Pr aims to cleanup the version tags in the various
docker-compose files so as to cleanup the log currently displaying
```'version' is obselete ``` when we run a docker-compose file.

## 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: vvs-personalstash <viralkverma0987@gmail.com>
Signed-off-by: Jared Tan <jian.tan@daocloud.io>
JaredTan95 pushed a commit to JaredTan95/jaeger that referenced this pull request Aug 28, 2024
…ing#5826)

## Description of the changes
- As the latest updates in docker compose has deprecated the 'version'
tag.This Pr aims to cleanup the version tags in the various
docker-compose files so as to cleanup the log currently displaying
```'version' is obselete ``` when we run a docker-compose file.

## 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: vvs-personalstash <viralkverma0987@gmail.com>
Signed-off-by: Jared Tan <jian.tan@daocloud.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/storage changelog:ci Change related to continuous integration / testing docker Pull requests that update Docker code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants