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.

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

Issue #94: Remove Process monitor job from Linux and create a new connector for process monitoring in Linux #109

Merged
merged 15 commits into from
Sep 5, 2024
Merged

Issue #94: Remove Process monitor job from Linux and create a new connector for process monitoring in Linux #109

merged 15 commits into from
Sep 5, 2024

Conversation

SafaeAJ
Copy link
Contributor

@SafaeAJ SafaeAJ commented Aug 16, 2024 •
edited
Loading

  • Create LinuxProcess.yaml
  • Update Linux.yaml

@SafaeAJ SafaeAJ changed the title Issue #94: Remove Process monitor job from Linux and create a new con… Issue #94: Remove Process monitor job from Linux and create a new connector for process monitoring in Linux Aug 16, 2024
SafaeAJ added 3 commits August 16, 2024 09:50
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
d631558 
…nector for process monitoring in Linux

* Update System.yaml
@SafaeAJ
Merge branch 'feature/issue-94-remove-process-monitor-job-from-linux-…
4312298 
…and-create-a-new-connector-for-process-monitoring-in-linux' of https://github.com/sentrysoftware/metricshub-community-connectors into feature/issue-94-remove-process-monitor-job-from-windows-and-linux-connectors-then-create-a-new-connector-for-process-monitoring-in-unix-linux-and-window
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
81853dc 
…nector for process monitoring in Linux

* Update System.yaml
@SafaeAJ SafaeAJ self-assigned this Aug 16, 2024
@SafaeAJ SafaeAJ added enhancement New feature or request developer_tested The changes are tested by the developer labels Aug 16, 2024
@SafaeAJ
Merge remote-tracking branch 'remotes/origin/main' into feature/issue…
e6347a5 
…-94-remove-process-monitor-job-from-windows-and-linux-connectors-then-create-a-new-connector-for-process-monitoring-in-unix-linux-and-window
SafaeAJ added 2 commits August 27, 2024 09:49
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
629e456 
…nector for process monitoring in Linux

* Update LinuxProcess.yaml
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
f1d866a 
…nector for process monitoring in Linux

* Update comment in linuxprocess.yaml
bertysentry
bertysentry requested changes Aug 27, 2024
View reviewed changes
Copy link
Member

@bertysentry bertysentry left a comment

Choose a reason for hiding this comment

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

A lot of changes are still required, but we're getting there. It's going to be a great connector!

src/main/connector/system/LinuxProcess/LinuxProcess.yaml Show resolved Hide resolved
src/main/connector/system/LinuxProcess/LinuxProcess.yaml Outdated Show resolved Hide resolved
src/main/connector/system/LinuxProcess/LinuxProcess.yaml Outdated Show resolved Hide resolved
src/main/connector/system/LinuxProcess/LinuxProcess.yaml Outdated Show resolved Hide resolved
src/main/connector/system/LinuxProcess/LinuxProcess.yaml Show resolved Hide resolved
src/main/connector/system/LinuxProcess/LinuxProcess.yaml Show resolved Hide resolved
src/main/connector/system/System/System.yaml Outdated Show resolved Hide resolved
src/main/connector/system/System/System.yaml Outdated Show resolved Hide resolved
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
cc7b8c4 
…nector for process monitoring in Linux

* Update linuxprocess.yaml
SafaeAJ added 3 commits August 29, 2024 11:41
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
5a76feb 
…nector for process monitoring in Linux

* Update linuxprocess.yaml
@SafaeAJ
Merge remote-tracking branch 'remotes/origin/main' into feature/issue…
71648a6 
…-94-remove-process-monitor-job-from-windows-and-linux-connectors-then-create-a-new-connector-for-process-monitoring-in-unix-linux-and-window
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
d8596e6 
…nector for process monitoring in Linux

* Update comment in linuxprocess.yaml
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
4a9eb8b 
…nector for process monitoring in Linux

* Update linuxprocess.yaml
bertysentry
bertysentry requested changes Aug 29, 2024
View reviewed changes
Copy link
Member

@bertysentry bertysentry left a comment

Choose a reason for hiding this comment

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

I think these are the last requested changes! 😁

src/main/connector/system/LinuxProcess/LinuxProcess.yaml Outdated Show resolved Hide resolved
src/main/connector/system/LinuxProcess/LinuxProcess.yaml Outdated Show resolved Hide resolved
src/main/connector/system/LinuxProcess/LinuxProcess.yaml Outdated Show resolved Hide resolved
src/main/connector/system/LinuxProcess/LinuxProcess.yaml Show resolved Hide resolved
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
fa8daff 
…nector for process monitoring in Linux

* Update linuxprocess.yaml
bertysentry
bertysentry previously approved these changes Aug 29, 2024
View reviewed changes
@SafaeAJ
Issue #94: Remove Process monitor job from Linux and create a new con…
06aaff6 
…nector for process monitoring in Linux

 * Update LinuxProcess.yaml
@NassimBtk NassimBtk merged commit 684a0c0 into main Sep 5, 2024
1 check passed
@NassimBtk NassimBtk deleted the feature/issue-94-remove-process-monitor-job-from-linux-and-create-a-new-connector-for-process-monitoring-in-linux branch September 5, 2024 11:58
@NathalieLe NathalieLe added this to the 1.0.05 milestone Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bertysentry bertysentry bertysentry approved these changes

@MedMaalej MedMaalej Awaiting requested review from MedMaalej

@NassimBtk NassimBtk Awaiting requested review from NassimBtk

@alexdgilbert alexdgilbert Awaiting requested review from alexdgilbert

@FK-sentry FK-sentry Awaiting requested review from FK-sentry

@CherfaElyes CherfaElyes Awaiting requested review from CherfaElyes

@TNohaic TNohaic Awaiting requested review from TNohaic

Assignees

@SafaeAJ SafaeAJ

Labels
developer_tested The changes are tested by the developer enhancement New feature or request
Projects
None yet
Milestone
v1.0.05
Development

Successfully merging this pull request may close these issues.
5 participants
@SafaeAJ @bertysentry @alexdgilbert @NassimBtk @NathalieLe