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

#4953 package jaxws opentracing to embedded Payara / FISH-994 #5020

Merged

Conversation

sgflt
Copy link
Contributor

@sgflt sgflt commented Dec 1, 2020

  • JaxwsContainerRequestTracing is now discovered by ServiceLocator

Description

Fixes packaging of embedded Payara. Any deployed webservice could not handle requests, because at least one implementation of opentracing is required.

Important Info

Blockers

Testing

New tests

Testing Performed

Manual run of integration tests with embedded Payara.

Testing Environment

Documentation

Notes for Reviewers

- JaxwsContainerRequestTracing is now discovered by ServiceLocator
@lprimak
Copy link
Contributor

lprimak commented Dec 7, 2020

jenkins test

@lprimak lprimak added PR: CLA CLA submitted on PR by the contributor Status: Accepted Confirmed defect or accepted improvement to implement, issue has been escalated to Platform Dev Type: Bug Label issue as a bug defect Type: Community Contribution labels Dec 7, 2020
@lprimak lprimak merged commit 6f99973 into payara:master Dec 19, 2020
@sgflt sgflt deleted the 4953-fix-embedded-packaging-to-fix-opentracing branch December 28, 2020 20:31
@AlanRoth AlanRoth changed the title #4953 package jaxws opentracing to embedded Payara #4953 package jaxws opentracing to embedded Payara / FISH-994 Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: CLA CLA submitted on PR by the contributor Status: Accepted Confirmed defect or accepted improvement to implement, issue has been escalated to Platform Dev Type: Bug Label issue as a bug defect
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants