Closed michaelmcfadyensky closed 2 years ago
💚 CLA has been signed
the below badges are clickable and redirect to their specific view in the CI or DOCS
#### Build stats * Start Time: 2022-01-11T20:52:26.108+0000 * Duration: 8 min 11 sec * Commit: 3290a55579c4e05fe8c6450cec749ec5a5878b16 #### Test stats :test_tube: | Test | Results | | ------------ | :-----------------------------: | | Failed | 0 | | Passed | 196 | | Skipped | 0 | | Total | 196 |
To re-run your PR in the CI, just comment with:
/test
: Re-trigger the build.
run
elasticsearch-ci/docs
: Re-trigger the docs validation. (use unformatted text in the comment!)
Hi, Sky have already signed the Elastic contributor-agreement. Is this enough to allow the CLA checks to be marked complete?
The CLA also has to be signed by each individual contributor. Let me know if I can assist.
@michaelmcfadyensky can you sign the CLA using your GitHub email?
@eyalkoren we're currently waiting on getting myself added to Sky's company CLA. @dro25 is chasing this up.
Any updates on the CLA?
The Sky CLA has now been updated to include myself as a contributor
The CLA also has to be signed by each individual contributor using the same GitHub email used by the PR author. It doesn't seem to be the case still. Sorry for insisting on this, but we must do so.
Once this is done, since this has been untouched for a long time, please make sure to merge with latest master and see that the tests pass. Thanks!
cla/check
Hi @michaelmcfadyensky , sorry the CLA is giving you trouble. When you say that the Sky CLA has been updated, can you confirm how that was sent to us? Typically, we have updates sent to cla-questions@elastic.co, but I'm not seeing anything there. I did find the original Sky CLA which was submitted on June 24, 2020.
Also, updates to existing corporate CLAs require manual work on our side to add the new users. Once I confirm receipt of the updated CLA, I'll make sure to update our system and let you know.
@michaelmcfadyensky I received the CLA, added you and the rest of the Sky contributors, and now the CLA checker has approve that part of this PR.
I'll let @eyalkoren and @felixbarny take it from here.
@gtback thanks for your help. It seems I may have jumped the gun with regards to messaging you guys about the updated CLA. I was not aware of the full process - TIL.
@eyalkoren I've pulled in the latest changes from master and have fixed the broken tests
Thanks a lot @gtback and @michaelmcfadyensky!
Relates to elastic/ecs-logging-java#103
This PR includes:
ThrowableHandlingConverter
in the ecs logback encoder (example below)Example usage: