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

Release v1.32.5 #941

Merged
merged 8 commits into from
Oct 31, 2024
Merged

Conversation

bjrara
Copy link
Member

@bjrara bjrara commented Oct 31, 2024

Description of changes:

  1. Switch isEmpty() to .isPresent() for Optionals Type #939
  2. Comment out E2E Operator Test #940
  3. Add Application Signals runtime metrics #892
  4. Update Dockerfile for corretto-slim build #924

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

yiyuan-he and others added 8 commits October 17, 2024 14:32
*Description of changes:*

Final step of MCM:

https://mcm.amazon.dev/cms/MCM-112315558/execution/step/e035bb70-e64d-4aee-a26f-ac4c2c11c029

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
Update Dockerfile to use public ECR image

*Issue #, if available:*

*Description of changes:*

Update Dockerfile to use public ECR image


By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
Merging release/v1.32.4 into main as part of MCM:

https://mcm.amazon.dev/cms/MCM-112315558/execution/step/9de5087f-2c46-42b8-b760-a9ce99b569df

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
*Issue #, if available:*

*Description of changes:*

Update Dockerfile to use public ECR image


By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
## Feature request
Add runtime metrics collection into Application Signals.

## Description of changes:
This PR is an umbrella PR to track the ongoing changes for runtime
metrics.
1. [Add Application Signals runtime metrics with feature disabled
aws-observability#900](aws-observability#900)
[Merged in main]
2. [feat: Add contract tests for runtime metrics
aws-observability#893](aws-observability#893)
3. Enable runtime metrics by default

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

*Issue #, if available:*

*Description of changes:*


By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

---------

Co-authored-by: Reno Seo <[email protected]>
*Issue #, if available:*
E2E Operator test is currently failing due to a known issue. We do not
have an immediate plan to fix it so commenting it out for now.

Example failed run:
https://github.com/aws-observability/aws-otel-java-instrumentation/actions/runs/11525535146/job/32241628521

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
…y#939)

*Issue #, if available:*
Main-build Enablement E2E tests started failing starting from this
[PR](aws-observability#899).
It was due to the usage of `isEmpty()` for Optional datatypes, which is
available only from Java 11

*Description of changes:*
Change .isEmpty() to .isPresent() for optional types


By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

Co-authored-by: Min Xia <[email protected]>
@bjrara bjrara changed the base branch from main to release/v1.32.x October 31, 2024 16:47
@bjrara bjrara requested a review from a team as a code owner October 31, 2024 16:47
Copy link
Contributor

@vastin vastin left a comment

Choose a reason for hiding this comment

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

LGTM

@bjrara bjrara merged commit db13aa6 into aws-observability:release/v1.32.x Oct 31, 2024
4 checks passed
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.

5 participants