Skip to content

Conversation

xenia-lang
Copy link

This is a suggestion for fixing #186.

@cla-checker-service
Copy link

cla-checker-service bot commented Jan 9, 2023

💚 CLA has been signed

@github-actions github-actions bot added aws-λ-extension AWS Lambda Extension community Issues and PRs created by the community triage Issues and PRs that need to be triaged labels Jan 9, 2023
@ghost
Copy link

ghost commented Jan 9, 2023

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-01-17T00:48:57.018+0000

  • Duration: 5 min 21 sec

Test stats 🧪

Test Results
Failed 0
Passed 208
Skipped 2
Total 210

🤖 GitHub comments

Expand to view the GitHub comments

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!)

@xenia-lang
Copy link
Author

I am confused by the contributor agreement. It says that the field "Title" is required. What am I supposed to put in there?

@xenia-lang
Copy link
Author

/test

Copy link
Member

@axw axw left a comment

Choose a reason for hiding this comment

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

Thanks @JonathanLang-TNG! Looks good, I'd just like to align the env var names with the ones used by Elastic APM agents.

Would you mind adding a couple of basic unit tests? For now I think it would be enough to test just the certificate PEM & verify certs config. You could clone TestInfoProxy and use httptest.NewTLSServer, then check that the proxy code can connect with either verification disabled, or by passing the server's self-signed certificate as the CA cert.

@xenia-lang xenia-lang force-pushed the main branch 2 times, most recently from f96c88e to de0c4c0 Compare January 13, 2023 10:29
Copy link
Member

@axw axw left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@axw
Copy link
Member

axw commented Jan 16, 2023

/test

@axw
Copy link
Member

axw commented Jan 16, 2023

run elasticsearch-ci/docs

@axw
Copy link
Member

axw commented Jan 16, 2023

@JonathanLang-TNG please run go mod tidy to get CI to pass.

@xenia-lang
Copy link
Author

Would be really helpful if I could trigger the CI, so I can check myself if everything is fine...

@axw
Copy link
Member

axw commented Jan 17, 2023

/test

@axw axw merged commit 8201650 into elastic:main Jan 17, 2023
@axw
Copy link
Member

axw commented Jan 17, 2023

Thanks again @JonathanLang-TNG!

@xenia-lang
Copy link
Author

@axw Do you have any idea, when these changes will make into the lambda layer elastic provides on AWS? So, when the next release will be?

@axw
Copy link
Member

axw commented Jan 18, 2023

Typically we create new releases in alignment with the Elastic stack. I'll discuss with the team and see if we have capacity to do some manual testing before 8.7.0 is due, and get back to you.

@axw
Copy link
Member

axw commented Jan 19, 2023

@JonathanLang-TNG we will look at testing and creating a new release soon (sooner than 8.7.0; hopefully in the next couple of weeks, but I can't promise): #364

@axw
Copy link
Member

axw commented Feb 23, 2023

@JonathanLang-TNG apologies for the delay, we have just released this in v1.3.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aws-λ-extension AWS Lambda Extension community Issues and PRs created by the community triage Issues and PRs that need to be triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants