Skip to content

[WIP] monit: triggering CI tests #10913

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 37 commits into from
Closed

[WIP] monit: triggering CI tests #10913

wants to merge 37 commits into from
+31 −12

Conversation

@russoz
Copy link
Collaborator

@russoz russoz commented Oct 12, 2025

SUMMARY

Investigation in the unstable tests for monit

ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Refactoring Pull Request
  • Test Pull Request
COMPONENT NAME

monit

@ansibullbot
Copy link
Collaborator

ansibullbot commented Oct 12, 2025

cc @brian-brazil @dstoflet @snopoke
click here for bot help

@ansibullbot ansibullbot added WIP Work in progress bug This issue/PR relates to a bug module module plugins plugin (any type) small_patch Hopefully easy to review labels Oct 12, 2025
@ansibullbot ansibullbot added integration tests/integration tests tests and removed small_patch Hopefully easy to review labels Oct 13, 2025
@ansibullbot ansibullbot added the unit tests/unit label Oct 13, 2025
@ansibullbot ansibullbot removed the module module label Oct 13, 2025
@russoz russoz closed this Oct 13, 2025
@russoz russoz reopened this Oct 13, 2025
@russoz russoz closed this Oct 13, 2025
@russoz russoz reopened this Oct 13, 2025
@russoz
Copy link
Collaborator Author

russoz commented Oct 13, 2025

in the last 3 runs, two succeeded and one failed

@ansibullbot ansibullbot added the module module label Oct 14, 2025
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot
Copy link
Collaborator

ansibullbot commented Oct 14, 2025

The test ansible-test sanity --test pylint [explain] failed with 5 errors:

plugins/modules/monit.py:183:16: unreachable: Unreachable code
plugins/modules/monit.py:195:12: unreachable: Unreachable code
plugins/modules/monit.py:212:8: unreachable: Unreachable code
plugins/modules/monit.py:216:8: unreachable: Unreachable code
plugins/modules/monit.py:226:8: unreachable: Unreachable code

The test ansible-test sanity --test pylint [explain] failed with 5 errors:

plugins/modules/monit.py:183:16: unreachable: Unreachable code
plugins/modules/monit.py:195:12: unreachable: Unreachable code
plugins/modules/monit.py:212:8: unreachable: Unreachable code
plugins/modules/monit.py:216:8: unreachable: Unreachable code
plugins/modules/monit.py:226:8: unreachable: Unreachable code

The test ansible-test sanity --test pylint [explain] failed with 5 errors:

plugins/modules/monit.py:183:16: unreachable: Unreachable code
plugins/modules/monit.py:195:12: unreachable: Unreachable code
plugins/modules/monit.py:212:8: unreachable: Unreachable code
plugins/modules/monit.py:216:8: unreachable: Unreachable code
plugins/modules/monit.py:226:8: unreachable: Unreachable code

click here for bot help

@ansibullbot ansibullbot added the stale_ci CI is older than 7 days, rerun before merging label Oct 22, 2025
@ansibullbot ansibullbot added the needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html label Oct 30, 2025
@russoz
Copy link
Collaborator Author

russoz commented Nov 15, 2025

There was some learning here, but main branch has gone through a lot of changes recently and this one is not really going anywhere. Closing.

@russoz russoz closed this Nov 15, 2025
@russoz russoz deleted the monit-tests branch November 15, 2025 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug This issue/PR relates to a bug has_issue integration tests/integration module module needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html plugins plugin (any type) stale_ci CI is older than 7 days, rerun before merging tests tests unit tests/unit WIP Work in progress

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@russoz @ansibullbot