Skip to content

Conversation

cold-magma
Copy link
Collaborator

Upgraded java version to 17
Upgraded aws-cloudwatch-exts-commons to 2.2.8
Added stage and resource to the metric path on collect by adding additional dimensions

@cold-magma cold-magma changed the title Feature/java upgrade and api pathwise metrics Java upgrade and api pathwise metrics Aug 25, 2025
@cold-magma cold-magma force-pushed the feature/java_upgrade_and_api_pathwise_metrics branch from e9809f0 to 2518c55 Compare August 25, 2025 06:16
Copy link
Contributor

@sanketmehta28 sanketmehta28 left a comment

Choose a reason for hiding this comment

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

Please respond to queries


DimensionFilter apiNameDimensionFilter = new DimensionFilter();
apiNameDimensionFilter.withName(APINAME);
DimensionFilter apiNameDimensionFilter = DimensionFilter.builder()
Copy link
Contributor

Choose a reason for hiding this comment

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

instead of APIName, why 3 diff dimention filters were created?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ApiName is not a filter in cloudwatch. they changed the schema to ApiId,Stage and Resource
image

for(Dimension dimension : metric.getDimensions()) {
if(dimension.getName().equalsIgnoreCase("ApiName")) {
apiName = dimension.getValue();
for(Dimension dimension : metric.dimensions()) {
Copy link
Contributor

Choose a reason for hiding this comment

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

same as above

awsSecretKey: ""
displayAccountName: "AppD"
regions: ["us-west-2"]
displayAccountName: "Test Gateway"
Copy link
Contributor

Choose a reason for hiding this comment

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

is this display account name is correct?

Copy link
Collaborator Author

@cold-magma cold-magma Aug 25, 2025

Choose a reason for hiding this comment

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

This is just a placeholder. it is customised by customer based on their needs.

Copy link
Contributor

Choose a reason for hiding this comment

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

Even as a place holder, please do restore the previous displayName

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

reverted.

# Global time range configuration, applicable to all the metrics configured below.
metricsTimeRange:
startTimeInMinsBeforeNow: 10
startTimeInMinsBeforeNow: 60
Copy link
Contributor

Choose a reason for hiding this comment

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

what is the purpose of increasing the starttime by 60 min?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is just a placeholder. it is customised by customer based on their metrics size.

Copy link
Contributor

Choose a reason for hiding this comment

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

in that case, please dont change the value so that it does not create any confusion unnecessary

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

reverted.

@sanketmehta28 sanketmehta28 merged commit 6897982 into master Aug 25, 2025
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.

2 participants