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

Support for AWS XRay #7165

Merged
merged 1 commit into from
Feb 16, 2020
Merged

Support for AWS XRay #7165

merged 1 commit into from
Feb 16, 2020

Conversation

patriot1burke
Copy link
Contributor

Support for AWS XRay Issue #6785

@patriot1burke
Copy link
Contributor Author

@sherl0cks @christianThor Please reverify that this works for you still.

@christianThor
Copy link

@patriot1burke tested and it works like a charm 👍

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

I added some suggestions inline. Most are purely cosmetic, except for the remark on Jackson.

docs/src/main/asciidoc/amazon-lambda-http.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/amazon-lambda-http.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/amazon-lambda-http.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/amazon-lambda.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/amazon-lambda.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/amazon-lambda.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/amazon-lambda-http.adoc Outdated Show resolved Hide resolved
@gastaldi
Copy link
Contributor

Please rebase this PR

@gastaldi gastaldi added the triage/needs-rebase This PR needs to be rebased first because it has merge conflicts label Feb 12, 2020
@patriot1burke
Copy link
Contributor Author

@gastaldi I'm such a git idiot...I did a git fetch upstream git rebase -i upstream/master. What did I do wrong?

improve

finish tracing

Update docs/src/main/asciidoc/amazon-lambda.adoc

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update docs/src/main/asciidoc/amazon-lambda.adoc

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update docs/src/main/asciidoc/amazon-lambda.adoc

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update docs/src/main/asciidoc/amazon-lambda-http.adoc

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update docs/src/main/asciidoc/amazon-lambda-http.adoc

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update docs/src/main/asciidoc/amazon-lambda-http.adoc

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

rename

Update extensions/amazon-lambda-xray/runtime/src/main/resources/META-INF/quarkus-extension.yaml

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update docs/src/main/asciidoc/amazon-lambda-http.adoc

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

moar fixes

Update docs/src/main/asciidoc/amazon-lambda.adoc

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update extensions/amazon-lambda-xray/runtime/pom.xml

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update extensions/amazon-lambda-xray/pom.xml

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>

Update extensions/amazon-lambda-xray/deployment/pom.xml

Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>
@gastaldi
Copy link
Contributor

git rebase master && git push -f would have been enough :)

@patriot1burke
Copy link
Contributor Author

patriot1burke commented Feb 13, 2020 via email

@gastaldi
Copy link
Contributor

Ah then git checkout master && git pull && git checkout - would help

@gastaldi gastaldi removed the triage/needs-rebase This PR needs to be rebased first because it has merge conflicts label Feb 13, 2020
@gastaldi gastaldi added this to the 1.3.0 milestone Feb 13, 2020
Copy link
Contributor

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

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

LGTM and according to #7165 (comment) it seems to work :)

@patriot1burke
Copy link
Contributor Author

@gsmet Think I resolved your concerns. Thanks for editing the docs.

@gsmet gsmet merged commit e71f96a into quarkusio:master Feb 16, 2020
@gsmet
Copy link
Member

gsmet commented Feb 16, 2020

Merged, thanks!

@gsmet gsmet changed the title Support for AWS XRay Issue #6785 Support for AWS XRay Feb 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants