Skip to content

CTOR-1656-move-centreon-plugins-sudoers-to-centreon-plugin-repository#5598

Merged
Evan-Adam merged 4 commits intodevelopfrom
CTOR-1656-move-centreon-plugins-sudoers-to-centreon-plugin-repository
Jul 3, 2025
Merged

CTOR-1656-move-centreon-plugins-sudoers-to-centreon-plugin-repository#5598
Evan-Adam merged 4 commits intodevelopfrom
CTOR-1656-move-centreon-plugins-sudoers-to-centreon-plugin-repository

Conversation

@Evan-Adam
Copy link
Contributor

Description

integrate centreon-plugin-sudoers to centreon-plugins

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Functionality enhancement or optimization (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

How this pull request can be tested ?

Package should be installable on every supported OS.

Checklist

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have rebased my development branch on the base branch (develop).
  • In case of a new plugin, I have created the new packaging directory accordingly.
  • I have implemented automated tests related to my commits.
    • Data used for automated tests are anonymized.
  • I have reviewed all the help messages in all the .pm files I have modified.
    • All sentences begin with a capital letter.
    • All sentences end with a period.
    • I am able to understand all the help messages, if not, exchange with the PO or TW to rewrite them.
  • After having created the PR, I will make sure that all the tests provided in this PR have run and passed.

@Evan-Adam Evan-Adam requested review from a team as code owners May 20, 2025 15:23
@Evan-Adam Evan-Adam requested review from ddes, kduret and omercier May 20, 2025 15:23
@Evan-Adam Evan-Adam changed the base branch from develop to CTOR-1649-plugin-apps-nmap-cli-need-to-secure-the-execution-of-commands May 20, 2025 15:23
@github-actions
Copy link

github-actions bot commented May 20, 2025

Logo
Checkmarx One – Scan Summary & Details8cf2aae1-cb69-4955-8bfe-54c9f6bab0de

Great job, no security vulnerabilities found in this Pull Request

Base automatically changed from CTOR-1649-plugin-apps-nmap-cli-need-to-secure-the-execution-of-commands to develop May 26, 2025 07:29
@Evan-Adam Evan-Adam force-pushed the CTOR-1656-move-centreon-plugins-sudoers-to-centreon-plugin-repository branch from 9b62c04 to 35dd6ef Compare May 26, 2025 07:29
kduret
kduret previously approved these changes Jun 9, 2025
@Evan-Adam
Copy link
Contributor Author

will be integrated by #5597

@Evan-Adam Evan-Adam closed this Jun 10, 2025
@Evan-Adam
Copy link
Contributor Author

en fait j'ai besoin que la librairie soit déplacé dans une PR a part pour que les tests que j'ajoute dans la nouvelle pr (sur le plugin nmap cli) puisse installer le plugin sans avoir besoin du repo centreon 24.10.

@Evan-Adam Evan-Adam requested a review from omercier July 1, 2025 12:56
@scresto31 scresto31 self-requested a review July 2, 2025 11:47
@Evan-Adam Evan-Adam merged commit eec431c into develop Jul 3, 2025
20 checks passed
@Evan-Adam Evan-Adam deleted the CTOR-1656-move-centreon-plugins-sudoers-to-centreon-plugin-repository branch July 3, 2025 14:27
scresto31 pushed a commit that referenced this pull request Jul 10, 2025
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.

6 participants