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

Enable Collection for Checkmk 2.3.0 #544

Merged
merged 46 commits into from
May 24, 2024
Merged

Conversation

robin-checkmk
Copy link
Member

@robin-checkmk robin-checkmk commented Jan 30, 2024

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: e.g. #1

What is the new behavior?

Other information

ToDos

  • Idempotency for the rule module is broken with Checkmk 2.3.0. Former versions work.
  • The discovery module is a bad shape when it comes to discovery modes. We need to align it with the current state of the REST API, while making sure it stays backwards compatible.

@robin-checkmk robin-checkmk self-assigned this Jan 30, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation role:server This affects the server role role:agent This affects the agent role labels Jan 30, 2024
@github-actions github-actions bot added the module:user This affects the user module label Jan 30, 2024
Base automatically changed from build/simplify-tests to devel January 30, 2024 22:42
@robin-checkmk robin-checkmk changed the title Enable early Checkmk 2.3.0 tests Enable Checkmk 2.3.0 Tests Mar 14, 2024
@robin-checkmk robin-checkmk changed the title Enable Checkmk 2.3.0 Tests Enable Collection for Checkmk 2.3.0 Apr 1, 2024
@robin-checkmk robin-checkmk added the release:5.0.0 Affects the mentioned release. label Apr 18, 2024
@robin-checkmk robin-checkmk mentioned this pull request May 24, 2024
7 tasks
@robin-checkmk robin-checkmk marked this pull request as ready for review May 24, 2024 18:48
@robin-checkmk
Copy link
Member Author

The failing tests are mostly not relevant, except for the rules, but those errors are already fixed in devel.
I will merge this PR into devel and we will fix follow-up issues there.

@robin-checkmk robin-checkmk merged commit 89e2007 into devel May 24, 2024
407 of 437 checks passed
@robin-checkmk robin-checkmk deleted the build/checkmk-2-3-support branch May 24, 2024 18:50
@github-actions github-actions bot locked and limited conversation to collaborators May 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation module:activation This affects the activation module module:bakery This affects the bakery module module:contact_group This affects the contact_group module module:discovery This affects the discovery module module:downtime This affects the downtime module module:folder This affects the folder module module:host_group This affects the host_group module module:host This affects the host module module:password module:rule This affects the rule module module:service_group This affects the service_group module module:tag_group This affects the tag_group module module:timeperiod This affects the timeperiod module module:user This affects the user module release:5.0.0 Affects the mentioned release. role:agent This affects the agent role role:server This affects the server role
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants