NationTech/harmony
5
0
Fork 2
Code Issues 37 Pull Requests 15 Actions Packages Projects Releases 1 Wiki Activity

feat: implementation for opnsense os-node_exporter #173

Open
wjro wants to merge 9 commits from feat/install_opnsense_node_exporter into master
pull from: feat/install_opnsense_node_exporter
merge into: NationTech:master
Conversation 2 Commits 9 Files Changed 17 +379 -21
wjro commented 2025-10-22 15:29:38 +00:00
Owner
Copy Link

Added node_exporter to opnsense config and its implementation for OPNSenseFirewall

Added node_exporter to opnsense config and its implementation for OPNSenseFirewall
wjro added 1 commit 2025-10-22 15:29:39 +00:00
feat: implementation for opnsense os-node_exporter
Some checks failed
Run Check Script / check (pull_request) Failing after 41s
Details
8126b233d8
letian commented 2025-10-22 15:32:13 +00:00
Owner
Copy Link

Could you make a quick screen recording (or screenshot) of this change in action? 📹

Could you make a quick screen recording (or screenshot) of this change in action? 📹
letian reviewed 2025-10-22 15:34:42 +00:00
opnsense-config/src/modules/node_exporter.rs Outdated
@ -0,0 +20,4 @@
pub fn get_full_config(&self) -> &Option<NodeExporter> {
&self.opnsense.opnsense.node_exporter
}
fn with_node_exporter<F, R>(&mut self, f: F) -> R
letian commented 2025-10-22 15:34:42 +00:00
Owner
Copy Link

white line missing

white line missing
wjro marked this conversation as resolved
letian reviewed 2025-10-22 15:34:54 +00:00
opnsense-config/src/modules/node_exporter.rs Outdated
@ -0,0 +27,4 @@
match &mut self.opnsense.opnsense.node_exporter.as_mut() {
Some(node_exporter) => f(node_exporter),
None => unimplemented!(
"
letian commented 2025-10-22 15:34:54 +00:00
Owner
Copy Link

is it expected to have a line return here?

is it expected to have a line return here?
wjro marked this conversation as resolved
letian commented 2025-10-22 15:37:15 +00:00
Owner
Copy Link

Looking good

Looking good <img src="https://i.giphy.com/FT6DYlscgddh8cTR6d.webp"/>
wjro added 1 commit 2025-10-22 15:51:28 +00:00
fix: removed unimplemnted marco and returned Err instead
Some checks failed
Run Check Script / check (pull_request) Failing after 29s
Details
5af13800b7 
some formatting error
wjro added 1 commit 2025-10-22 18:39:20 +00:00
fix: added impl node exporter for hacluster topology and dummy infra
All checks were successful
Run Check Script / check (pull_request) Successful in 1m26s
Details
5ab58f0253
letian reviewed 2025-10-22 18:55:43 +00:00
harmony/src/domain/topology/node_exporter.rs
@ -0,0 +10,4 @@
}
//TODO complete this impl
impl std::fmt::Debug for dyn NodeExporter {
letian commented 2025-10-22 18:55:43 +00:00
Owner
Copy Link

I think we should leave it to the implementers to define their Debug impl. So on line 6 we should have pub trait NodeExporter: std::fmt::Debug + Send + Sync instead.

I think we should leave it to the implementers to define their `Debug` impl. So on line 6 we should have `pub trait NodeExporter: std::fmt::Debug + Send + Sync` instead.
wjro added 2 commits 2025-10-28 19:45:34 +00:00
wip: cargo fmt
All checks were successful
Run Check Script / check (pull_request) Successful in 1m16s
Details
9ba939bde1
wjro added 1 commit 2025-10-29 17:26:24 +00:00
wjro added 1 commit 2025-10-29 17:42:28 +00:00
Merge remote-tracking branch 'origin/master' into feat/install_opnsense_node_exporter
All checks were successful
Run Check Script / check (pull_request) Successful in 1m21s
Details
09a06f136e
wjro added 1 commit 2025-10-29 17:50:03 +00:00
Merge remote-tracking branch 'origin/master' into feat/install_opnsense_node_exporter
Some checks failed
Run Check Script / check (pull_request) Failing after 25s
Details
ee278ac817
wjro changed title from WIP: feat: implementation for opnsense os-node_exporter to feat: implementation for opnsense os-node_exporter 2025-10-29 17:50:12 +00:00
wjro added 1 commit 2025-10-29 17:54:03 +00:00
fix:cargo fmt
All checks were successful
Run Check Script / check (pull_request) Successful in 1m21s
Details
c2fa4f1869
All checks were successful
Run Check Script / check (pull_request) Successful in 1m21s
Details
This pull request has changes conflicting with the target branch.
  • harmony/src/domain/topology/ha_cluster.rs
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/install_opnsense_node_exporter:feat/install_opnsense_node_exporter
git checkout feat/install_opnsense_node_exporter
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Architecture Decision Record

   
Good first issue

   
RFC
No Label
Architecture Decision Record
Good first issue
RFC
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: NationTech/harmony#173
No description provided.
Delete Branch "feat/install_opnsense_node_exporter"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?