[Azure App Service] bug fixes by giorgi-imerlishvili-elastic · Pull Request #15591 · elastic/integrations

giorgi-imerlishvili-elastic · 2025-10-07T13:17:03Z

Proposed commit message

Fix: [Azure App Service]: Expects JSON object but is unparsed JSON string #11169
Fix: [Azure App Service] Missing azure.subscription_id #11172

Checklist

I have reviewed tips for building integrations and this pull request is aligned with them.
I have verified that all data streams collect metrics or logs.
I have added an entry to my package's changelog.yml file.
I have verified that Kibana version constraints are current according to guidelines.
I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

[ ]

How to test this PR locally

Related issues

Screenshots

…tions # Conflicts: # packages/statsd_input/changelog.yml # packages/statsd_input/manifest.yml

…tions

…e and queue

…tions

This reverts commit f9c172f.

…11169" This reverts commit 59c2da8.

elastic-vault-github-plugin-prod · 2025-10-07T14:20:31Z

🚀 Benchmarks report

Package `azure_app_service` 👍(0) 💚(0) 💔(1)

Expand to view

Data stream	Previous EPS	New EPS	Diff (%)	Result
`app_service_logs`	4608.29	1669.45	-2938.84 (-63.77%)	💔

To see the full report comment with /test benchmark fullreport

lucian-ioan · 2025-10-16T03:55:49Z

+  - set:
+      field: azure.app_service.properties_raw
+      value: "{{{azure.app_service.properties}}}"


Why is this renaming needed?

Because in appservice-platformlogs-inner-pipeline.yml in azure.app_service.log field we were setting value of azure.app_service.properties which were expected as string (actually it's json string).
But in other pipelines we need to decode that JSON in order to get values of decoded fields.

So this properties_raw is just temporary field in order to set azure.app_service.log field in appservice-platformlogs-inner-pipeline.yml

lucian-ioan

Will test this PR when it's ready for review. It also needs a changelog entry.

muthu-mps · 2025-10-20T04:43:21Z

Will test this PR when it's ready for review. It also needs a changelog entry.

@giorgi-imerlishvili-elastic - Can you address this comment?

giorgi-imerlishvili-elastic · 2025-10-22T00:50:04Z

Will test this PR when it's ready for review. It also needs a changelog entry.

@giorgi-imerlishvili-elastic - Can you address this comment?

I was thinking to make it ready for review once this is merged #15487
because that PR adds another changelog to this package

lucian-ioan

Pipeline improvements LGTM.

muthu-mps · 2025-10-23T07:15:58Z

      ignore_failure: true
+  - set:
+      field: azure.app_service.properties_raw
+      value: "{{{azure.app_service.properties}}}"


@giorgi-imerlishvili-elastic - Can we add comment about this change. Otherwise this seems to be like adding a field and removing the same field without any processing. Also in the platformlogs-inner-pipeline the properties field is removed. Is it valid?

# Conflicts: # packages/azure_app_service/changelog.yml # packages/azure_app_service/manifest.yml

elasticmachine · 2025-10-30T14:27:07Z

💚 Build Succeeded

Buildkite Build
Commit: 402470f

History

💚 Build #33248 succeeded b2e3da4
💚 Build #32981 succeeded cfe1de1
💚 Build #32972 succeeded cbea5ec
💚 Build #32431 succeeded f491e49

cc @giorgi-imerlishvili-elastic

elastic-vault-github-plugin-prod · 2025-10-30T15:08:03Z

Package azure_app_service - 0.10.0 containing this change is available at https://epr.elastic.co/package/azure_app_service/0.10.0/

fix Missing azure.subscription_id fix Expects JSON object but is unparsed JSON string

giorgi-imerlishvili-elastic added 30 commits July 17, 2025 12:15

Update manifest.yml format version to 3.4

b50ad39

Merge branch 'main' of github.com:giorgi-imerlishvili-elastic/integra…

af28b64

…tions # Conflicts: # packages/statsd_input/changelog.yml # packages/statsd_input/manifest.yml

Update manifest.yml format version to 3.4

1551485

Update changelog.yml

144c534

set statsd.*.* object type to double

7f5b6a1

Merge branch 'elastic:main' into main

d958843

remove non-mandatory source setting from manifest.yml

d674291

Merge branch 'elastic:main' into main

e74c058

Add system test for RabbitMQ

d4c8c97

Merge branch 'elastic:main' into main

bae056a

Merge branch 'main' of github.com:giorgi-imerlishvili-elastic/integra…

5714ca1

…tions

modify manifest.yml and changelog.yml

5d183ad

add system tests for rabbitMQ data streams: connection, exchange, nod…

971c768

…e and queue

Merge branch 'elastic:main' into main

b2a38cb

Merge branch 'main' of github.com:giorgi-imerlishvili-elastic/integra…

ece7b92

…tions

update readme

83d8c87

update field type and system test file yaml format

a5f6775

fix lint errors

7880f2e

fix sonar cube errors

1a9a85c

add comments

d101751

add unit tests for simulate_queue_connection.py

ea3cd82

add more comments

fdbb43a

generate new sample events

f37b4b7

update readme

f72a6a7

update readme

4962cdc

Changes based on PR review

9dc1ec5

Make scripts executable and remove relevant commands from dockerfile

263636b

update readme

00e0fbf

Merge branch 'elastic:main' into main

1de2781

changes based on feedback

2e16eda

giorgi-imerlishvili-elastic added 2 commits October 7, 2025 17:12

Reapply "fix Missing azure.subscription_id"

93bbd1c

This reverts commit f9c172f.

Reapply "fix Expects JSON object but is unparsed JSON string elastic#…

f491e49

…11169" This reverts commit 59c2da8.

giorgi-imerlishvili-elastic self-assigned this Oct 7, 2025

giorgi-imerlishvili-elastic added bugfix Pull request that fixes a bug issue Integration:azure_app_service Azure App Service labels Oct 7, 2025

giorgi-imerlishvili-elastic mentioned this pull request Oct 7, 2025

[Azure App Service] New dashboard #15487

Merged

5 tasks

muthu-mps requested a review from lucian-ioan October 15, 2025 07:49

lucian-ioan reviewed Oct 16, 2025

View reviewed changes

add changelog

cbea5ec

giorgi-imerlishvili-elastic marked this pull request as ready for review October 22, 2025 07:15

giorgi-imerlishvili-elastic requested a review from a team as a code owner October 22, 2025 07:15

Merge branch 'main' into azure_app_service_bugfixes

cfe1de1

giorgi-imerlishvili-elastic requested a review from lucian-ioan October 22, 2025 09:37

andrewkroh added the Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] label Oct 22, 2025

lucian-ioan approved these changes Oct 22, 2025

View reviewed changes

muthu-mps reviewed Oct 23, 2025

View reviewed changes

giorgi-imerlishvili-elastic added 5 commits October 28, 2025 11:23

add comment about field properties_raw

b2e3da4

Merge branch 'elastic:main' into main

00dc7b2

Merge branch 'elastic:main' into main

4e94e8d

Merge branch 'elastic:main' into main

9356c79

Merge branch 'main' into azure_app_service_bugfixes

402470f

# Conflicts: # packages/azure_app_service/changelog.yml # packages/azure_app_service/manifest.yml

muthu-mps approved these changes Oct 30, 2025

View reviewed changes

giorgi-imerlishvili-elastic merged commit ab7dba9 into elastic:main Oct 30, 2025
7 checks passed

tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025

[Azure App Service] bug fixes (elastic#15591)

ae6c6bf

fix Missing azure.subscription_id fix Expects JSON object but is unparsed JSON string

Navigation Menu

Uh oh!

[Azure App Service] bug fixes - #15591

[Azure App Service] bug fixes#15591
giorgi-imerlishvili-elastic merged 47 commits into
elastic:mainfrom
giorgi-imerlishvili-elastic:azure_app_service_bugfixes

giorgi-imerlishvili-elastic commented Oct 7, 2025 •

edited

Loading

elastic-vault-github-plugin-prod Bot commented Oct 7, 2025 •

edited

Loading

lucian-ioan Oct 16, 2025

giorgi-imerlishvili-elastic Oct 22, 2025

lucian-ioan left a comment

muthu-mps commented Oct 20, 2025

giorgi-imerlishvili-elastic commented Oct 22, 2025

lucian-ioan left a comment

muthu-mps Oct 23, 2025

elasticmachine commented Oct 30, 2025

Uh oh!

elastic-vault-github-plugin-prod Bot commented Oct 30, 2025

Labels

5 participants

Uh oh!

Conversation

giorgi-imerlishvili-elastic commented Oct 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Proposed commit message

Checklist

Author's Checklist

How to test this PR locally

Related issues

Screenshots

elastic-vault-github-plugin-prod Bot commented Oct 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🚀 Benchmarks report

Package azure_app_service 👍(0) 💚(0) 💔(1)

lucian-ioan Oct 16, 2025

Choose a reason for hiding this comment

giorgi-imerlishvili-elastic Oct 22, 2025

Choose a reason for hiding this comment

lucian-ioan left a comment

Choose a reason for hiding this comment

muthu-mps commented Oct 20, 2025

giorgi-imerlishvili-elastic commented Oct 22, 2025

lucian-ioan left a comment

Choose a reason for hiding this comment

muthu-mps Oct 23, 2025

Choose a reason for hiding this comment

elasticmachine commented Oct 30, 2025

💚 Build Succeeded

History

Uh oh!

elastic-vault-github-plugin-prod Bot commented Oct 30, 2025

Labels

5 participants

giorgi-imerlishvili-elastic commented Oct 7, 2025 •

edited

Loading

elastic-vault-github-plugin-prod Bot commented Oct 7, 2025 •

edited

Loading

Package `azure_app_service` 👍(0) 💚(0) 💔(1)