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

feat: added pathType to ingress #2066

Merged
merged 3 commits into from
Sep 19, 2022
Merged

Conversation

giautm
Copy link
Contributor

@giautm giautm commented Sep 16, 2022

Which problem is this PR solving?

Short description of the changes

  • Added pathType to ingress

@giautm giautm force-pushed the giautm/path-type branch 5 times, most recently from f58eb3f to d8c7a93 Compare September 16, 2022 15:22
@frzifus frzifus self-assigned this Sep 16, 2022
@frzifus
Copy link
Member

frzifus commented Sep 16, 2022

thats awesome, thanks @giautm. I will have a more detailed look later today.

@giautm giautm force-pushed the giautm/path-type branch 2 times, most recently from f490593 to 2fd0dc0 Compare September 16, 2022 15:43
giautm and others added 2 commits September 16, 2022 22:46
Signed-off-by: Giau. Tran Minh <[email protected]>
Signed-off-by: Giau. Tran Minh <[email protected]>
@frzifus
Copy link
Member

frzifus commented Sep 16, 2022

Seems some formating is wrong.

 Checking...
Go fmt, license check, or import ordering failures, run 'make format'
pkg/ingress/query_test.go
make: *** [Makefile:119: check] Error 1

Signed-off-by: Giau. Tran Minh <[email protected]>
@giautm
Copy link
Contributor Author

giautm commented Sep 17, 2022

Hello @frzifus , review again please?

@codecov
Copy link

codecov bot commented Sep 17, 2022

Codecov Report

Base: 87.29% // Head: 87.30% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (86a236e) compared to base (8903e35).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2066   +/-   ##
=======================================
  Coverage   87.29%   87.30%           
=======================================
  Files         100      100           
  Lines        6826     6828    +2     
=======================================
+ Hits         5959     5961    +2     
  Misses        665      665           
  Partials      202      202           
Impacted Files Coverage Δ
apis/v1/jaeger_types.go 87.50% <ø> (ø)
pkg/ingress/query.go 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@frzifus frzifus left a comment

Choose a reason for hiding this comment

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

great work :)

@frzifus frzifus merged commit cc61547 into jaegertracing:main Sep 19, 2022
rubenvp8510 pushed a commit to rubenvp8510/jaeger-operator that referenced this pull request Sep 19, 2022
* feat: added pathType to ingress

Signed-off-by: Giau. Tran Minh <[email protected]>

* fix: update bundle

Signed-off-by: Giau. Tran Minh <[email protected]>

* fix: run make format

Signed-off-by: Giau. Tran Minh <[email protected]>

Signed-off-by: Giau. Tran Minh <[email protected]>
@giautm giautm deleted the giautm/path-type branch September 25, 2022 11:30
@giautm
Copy link
Contributor Author

giautm commented Sep 26, 2022

Hello @frzifus, can you create another release for helm-chart, please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: Support for path / pathType in query ingress
2 participants