-
Notifications
You must be signed in to change notification settings - Fork 43
[8.17](backport #3466) [Asset Inventory][Azure] Fix Azure service names (cloud.service.name) #3541
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
base: 8.17
Are you sure you want to change the base?
Conversation
…#3466) * update test cases * update Azure service names * apply review remarks * rename FlexibleMySQLDBAssetType * rename MySQLDBAssetType * gofmt (cherry picked from commit 7e3234f) # Conflicts: # internal/inventory/azurefetcher/fetcher_account.go # internal/inventory/azurefetcher/fetcher_account_test.go # internal/inventory/azurefetcher/fetcher_activedirectory.go # internal/inventory/azurefetcher/fetcher_activedirectory_test.go # internal/inventory/azurefetcher/fetcher_resource_graph.go # internal/inventory/azurefetcher/fetcher_resource_graph_test.go # internal/inventory/azurefetcher/fetcher_storage.go # internal/inventory/azurefetcher/fetcher_storage_test.go # internal/resources/fetching/fetchers/azure/assets_fetcher.go
|
Cherry-pick of 7e3234f has failed: To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
|
This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏 |
7 similar comments
|
This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏 |
Summary of your changes
Fixes missing
cloud.service.namevalues for Azure assets (were all assignedAzure).Related Issues
Fixes https://github.com/elastic/security-team/issues/13159
This is an automatic backport of pull request #3466 done by [Mergify](https://mergify.com).