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

[receiver/sshcheck] enable Windows #30648

Merged
merged 2 commits into from
Apr 29, 2024

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Jan 17, 2024

Removes the checks forbidding to use sshcheckreceiver on Windows.

Fixes #30650

@atoulme atoulme added the Run Windows Enable running windows test on a PR label Jan 17, 2024
@atoulme atoulme added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jan 22, 2024
Copy link
Contributor

github-actions bot commented Feb 5, 2024

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Feb 5, 2024
@atoulme atoulme removed the Stale label Feb 14, 2024
@atoulme atoulme changed the title [WIP][receiver/sshcheck] try to enable Windows [receiver/sshcheck] try to enable Windows Feb 17, 2024
@atoulme atoulme removed the Skip Changelog PRs that do not require a CHANGELOG.md entry label Feb 17, 2024
@atoulme atoulme marked this pull request as ready for review February 17, 2024 00:22
@atoulme atoulme requested a review from a team as a code owner February 17, 2024 00:22
@atoulme atoulme changed the title [receiver/sshcheck] try to enable Windows [receiver/sshcheck] enable Windows Mar 3, 2024
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@atoulme
Copy link
Contributor Author

atoulme commented Mar 20, 2024

@codeboten @nslaughter please take a look

Copy link
Contributor

@pjanotti pjanotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Apr 10, 2024
@atoulme atoulme removed the Stale label Apr 13, 2024
@atoulme
Copy link
Contributor Author

atoulme commented Apr 13, 2024

Still waiting for review.

Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @atoulme!

Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@atoulme atoulme force-pushed the try_windows_sshcheckreceiver branch from 8779750 to 6a0b14f Compare April 28, 2024 21:54
@codeboten codeboten merged commit 06c970d into open-telemetry:main Apr 29, 2024
185 checks passed
@github-actions github-actions bot added this to the next release milestone Apr 29, 2024
rimitchell pushed a commit to rimitchell/opentelemetry-collector-contrib that referenced this pull request May 8, 2024
Removes the checks forbidding to use sshcheckreceiver on Windows.

Fixes open-telemetry#30650

Co-authored-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[receiver/sshcheck] Windows support
4 participants