-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Cherry-pick #20689 to 7.x: Rename cloud.provider to azure
instead of az
in add_cloud_metadata processor
#20824
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…a processor (elastic#20689) * mofidy doc * rename azure provider in cloud data * changelo (cherry picked from commit d34d1b6)
botelastic
bot
added
the
needs_team
Indicates that the issue/PR needs a Team:* label
label
Aug 27, 2020
narph
added
[zube]: In Review
backport
Team:Platforms
Label for the Integrations - Platforms team
labels
Aug 27, 2020
Pinging @elastic/integrations-platforms (Team:Platforms) |
botelastic
bot
removed
the
needs_team
Indicates that the issue/PR needs a Team:* label
label
Aug 27, 2020
zube
bot
added
[zube]: Inbox
[zube]: In Review
and removed
[zube]: In Review
[zube]: Inbox
labels
Aug 27, 2020
narph
added
Team:Integrations
Label for the Integrations team
and removed
Team:Platforms
Label for the Integrations - Platforms team
labels
Aug 27, 2020
Collaborator
💔 Tests FailedExpand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
jsoriano
approved these changes
Aug 27, 2020
kaiyan-sheng
approved these changes
Aug 27, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Cherry-pick of PR #20689 to 7.x branch. Original message:
What does this PR do?
Rename cloud.provider to
azure
instead ofaz
in add_cloud_metadataWhy is it important?
The cloud.provider's value is
az
but it is matched toazure
in theazure
module.Not sure why it is shorten
azure
toaz
. Looking around I do not see theaz
terminology being a popular one or even used anywhere (maybe in the az client). Unlikeaws
orgcp
,azure
is does not to abbreviate toaz
, using azure instead seems to me to be the most reasonable solution.Also, the elastic documentation shows that
azure
should be used instead.Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Related issues