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

[CM] Fix enroll under Windows #12799

Merged
merged 2 commits into from
Jul 9, 2019
Merged

Conversation

adriansr
Copy link
Contributor

@adriansr adriansr commented Jul 5, 2019

Enrolling of a Beat to Central Management was failing under Windows because the Beat tries to create a configuration backup file that has a colon character in its name, which is not allowed under Windows.

Fixes #12797

Enrolling of a Beat to Central Management was failing under Windows
because the Beat tries to create a configuration backup file that has
a colon character in its name, which is not allowed under Windows.

Fixes elastic#12797
@adriansr adriansr added bug review libbeat x-pack Issues and pull requests for X-Pack features. labels Jul 5, 2019
@adriansr adriansr changed the title [CM] Fix enroll under Windows (#12797) [CM] Fix enroll under Windows Jul 5, 2019
@adriansr adriansr requested a review from michalpristas July 5, 2019 13:57
@urso
Copy link

urso commented Jul 8, 2019

An unit test would be nice. Then we would have caught this earlier, right?

@adriansr adriansr merged commit 4eb411b into elastic:master Jul 9, 2019
@adriansr adriansr added the v7.2.1 label Jul 9, 2019
adriansr added a commit to adriansr/beats that referenced this pull request Jul 9, 2019
Enrolling of a Beat to Central Management was failing under Windows
because the Beat tries to create a configuration backup file that has
a colon character in its name, which is not allowed under Windows.

Fixes elastic#12797

(cherry picked from commit 4eb411b)
adriansr added a commit to adriansr/beats that referenced this pull request Jul 9, 2019
Enrolling of a Beat to Central Management was failing under Windows
because the Beat tries to create a configuration backup file that has
a colon character in its name, which is not allowed under Windows.

Fixes elastic#12797

(cherry picked from commit 4eb411b)
@adriansr adriansr added the v7.3.0 label Jul 9, 2019
adriansr added a commit that referenced this pull request Jul 9, 2019
Enrolling of a Beat to Central Management was failing under Windows
because the Beat tries to create a configuration backup file that has
a colon character in its name, which is not allowed under Windows.

Fixes #12797

(cherry picked from commit 4eb411b)
adriansr added a commit that referenced this pull request Jul 9, 2019
Enrolling of a Beat to Central Management was failing under Windows
because the Beat tries to create a configuration backup file that has
a colon character in its name, which is not allowed under Windows.

Fixes #12797

(cherry picked from commit 4eb411b)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Enrolling of a Beat to Central Management was failing under Windows
because the Beat tries to create a configuration backup file that has
a colon character in its name, which is not allowed under Windows.

Fixes elastic#12797

(cherry picked from commit 8534dbc)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Enrolling of a Beat to Central Management was failing under Windows
because the Beat tries to create a configuration backup file that has
a colon character in its name, which is not allowed under Windows.

Fixes elastic#12797

(cherry picked from commit 8534dbc)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug libbeat review v7.2.1 v7.3.0 x-pack Issues and pull requests for X-Pack features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Filebeat] fails to enroll to centralized management in OS Windows
2 participants