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

Cherry-pick #13812 to 7.4: Allow users to select the cloud met… #13815

Merged
merged 2 commits into from
Sep 27, 2019

Conversation

urso
Copy link

@urso urso commented Sep 26, 2019

Cherry-pick of PR #13812 to 7.4 branch. Original message:

Resolves: #11145

We introduce a new setting 'providers' to the add_cloud_metadata
processor.

By now all the implementation for metadata providers requires developers
to mark a provider as 'local'. The alibaba and tencent providers are not
marked as local by now.

If the 'providers' setting is not used, then no all providers marked as
'local' are applied. This is a breaking change, because alibaba and
tencent providers will not be enabled anymore by default.

Although it's a breaking change, I prefered to disable non-local providers by default. (I epxect the fix to go into 7.4.0).

If the providers setting is used, only the selected providers will be
used.

The change supports alises for different providers. But the setup will only initialized a provider twice if the alias is used.

* Allow users to select the cloud metadata providers

We introduce a new setting 'providers' to the add_cloud_metadata
processor.

By now all the implementation for metadata providers requires developers
to mark a provider as 'local'. The alibaba and tencent providers are not
marked as local by now.

If the 'providers' setting is not used, then no all providers marked as
'local' are applied. This is a breaking change, because alibaba and
tencent providers will not be enabled anymore by default.

If the providers setting is used, only the selected providers will be
used.

(cherry picked from commit dc99773)
@urso
Copy link
Author

urso commented Sep 27, 2019

Fail on travis is a flaky filebeat test. Unrelated to this PR. All related tests passed on jenkins already.

@urso urso changed the title Cherry-pick #13812 to 7.4: Allow users to select the cloud metadata providers Cherry-pick #13812 to 7.4: Allow users to select the cloud met… Sep 27, 2019
@urso urso merged commit f940c36 into elastic:7.4 Sep 27, 2019
@urso urso deleted the backport_13812_7.4 branch September 27, 2019 00:23
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
elastic#13815)

* Allow users to select the cloud metadata providers (elastic#13812)

* Allow users to select the cloud metadata providers

We introduce a new setting 'providers' to the add_cloud_metadata
processor.

By now all the implementation for metadata providers requires developers
to mark a provider as 'local'. The alibaba and tencent providers are not
marked as local by now.

If the 'providers' setting is not used, then no all providers marked as
'local' are applied. This is a breaking change, because alibaba and
tencent providers will not be enabled anymore by default.

If the providers setting is used, only the selected providers will be
used.

(cherry picked from commit fa6d344)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants