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

Added a docker-compose for hotrod e2e test #5740

Merged
merged 25 commits into from
Jul 17, 2024

Conversation

hellspawn679
Copy link
Contributor

Which problem is this PR solving?

Description of the changes

  • added docker-compose file to test hotrod so that is can connect to jaeger

How was this change tested?

Checklist

Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
@hellspawn679 hellspawn679 requested a review from a team as a code owner July 13, 2024 07:18
@dosubot dosubot bot added the docker Pull requests that update Docker code label Jul 13, 2024
Copy link

codecov bot commented Jul 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.66%. Comparing base (c68136e) to head (9d82e79).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5740   +/-   ##
=======================================
  Coverage   96.66%   96.66%           
=======================================
  Files         341      341           
  Lines       16451    16451           
=======================================
  Hits        15902    15902           
  Misses        361      361           
  Partials      188      188           
Flag Coverage Δ
badger_v1 8.05% <ø> (ø)
badger_v2 1.90% <ø> (ø)
cassandra-3.x-v1 16.61% <ø> (ø)
cassandra-3.x-v2 1.82% <ø> (ø)
cassandra-4.x-v1 16.61% <ø> (ø)
cassandra-4.x-v2 1.82% <ø> (ø)
elasticsearch-6.x-v1 18.77% <ø> (-0.02%) ⬇️
elasticsearch-7.x-v1 18.83% <ø> (ø)
elasticsearch-8.x-v1 19.03% <ø> (ø)
elasticsearch-8.x-v2 1.90% <ø> (+0.01%) ⬆️
grpc_v1 9.52% <ø> (ø)
grpc_v2 7.47% <ø> (ø)
kafka 9.74% <ø> (ø)
memory_v2 1.90% <ø> (ø)
opensearch-1.x-v1 18.89% <ø> (ø)
opensearch-2.x-v1 18.88% <ø> (-0.02%) ⬇️
opensearch-2.x-v2 1.90% <ø> (+0.01%) ⬆️
unittests 95.08% <ø> (ø)

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
Copy link
Member

I suggest you explain your plan, doesn't seem it's going in the right direction. We already have all the pieces you included in this PR

@hellspawn679
Copy link
Contributor Author

@yurishkuro my plan is to build a local hotrod image(localhost:5000/jaegertracing/example-hotrod:${GITHUB_SHA:-latest}) and test it using docker-compose(that way we won't have any error for port 4318) then we will hit the frontend end point and get the hotrod logs and get the tracesID from there

hellspawn679 and others added 2 commits July 13, 2024 23:42
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
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.

you have UI submodule affected in this PR, please remove.

examples/hotrod/docker-compose.yml Outdated Show resolved Hide resolved
hellspawn679 and others added 5 commits July 14, 2024 20:40
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
@yurishkuro yurishkuro added the changelog:test Change that's adding missing tests or correcting existing tests label Jul 14, 2024
hellspawn679 and others added 2 commits July 15, 2024 03:01
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
hellspawn679 and others added 6 commits July 16, 2024 01:24
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
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.

lgtm, a couple open questions

mehul gautam and others added 2 commits July 17, 2024 04:39
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
hellspawn679 and others added 3 commits July 17, 2024 11:55
Signed-off-by: mehul gautam <mehulsharma4786@gamil.com>
@yurishkuro yurishkuro merged commit edfee78 into jaegertracing:main Jul 17, 2024
43 checks passed
@yurishkuro
Copy link
Member

Thanks!


bash scripts/build-upload-a-docker-image.sh -c example-hotrod -d examples/hotrod -p "${platforms}"
Copy link
Member

Choose a reason for hiding this comment

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

this line should not have been deleted. It needs to run as the last step of the script.

FlamingSaint pushed a commit to FlamingSaint/jaeger that referenced this pull request Jul 20, 2024
## Which problem is this PR solving?
- jaegertracing#5735

## Description of the changes
- added docker-compose file to test hotrod so that is can connect to
jaeger

## 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 <mehulsharma4786@gamil.com>
Co-authored-by: mehul gautam <mehulsharma4786@gamil.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:test Change that's adding missing tests or correcting existing tests docker Pull requests that update Docker code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants