NationTech/harmony
6
0
Fork 1
Code Issues 35 Pull Requests 7 Actions Packages Projects Releases 1 Wiki Activity

fix(host_network): adjust bond & port-channel configuration (partial) #175

Merged
letian merged 12 commits from nmstate into master 2025-10-29 17:09:16 +00:00
Conversation 0 Commits 12 Files Changed 22 +551 -232
letian commented 2025-10-23 20:39:48 +00:00
Owner
Copy Link

Description

  • Replace the CatalogSource approach to install the OperatorHub.io catalog by a more simple & straightforward way to install NMState
  • Improve logging
  • Add report summarizing the host network configuration that was applied (which host, bonds, port-channels)
  • Fix command to find next available port channel id

Extra info

Using the apply_url approach to install the NMState operator isn't the best approach: it's harder to maintain and upgrade. But it helps us achieve waht we wanted for now: install the NMState Operator to configure bonds on a host.

The preferred approach, installing an operator from the OperatorHub.io catalog, didn't work for now. We had a timeout error with DeadlineExceeded probably caused by an insufficient CPU/Memory allocation to query such a big catalog, even though we tweaked the RAM allocation (we couldn't find a way to do it for CPU).

Spent too much time on this so we stopped these efforts for now. It would be good to get back to it when we need to install something else from a custom catalog.

## Description * Replace the CatalogSource approach to install the OperatorHub.io catalog by a more simple & straightforward way to install NMState * Improve logging * Add report summarizing the host network configuration that was applied (which host, bonds, port-channels) * Fix command to find next available port channel id ## Extra info Using the `apply_url` approach to install the NMState operator isn't the best approach: it's harder to maintain and upgrade. But it helps us achieve waht we wanted for now: install the NMState Operator to configure bonds on a host. The preferred approach, installing an operator from the OperatorHub.io catalog, didn't work for now. We had a timeout error with DeadlineExceeded probably caused by an insufficient CPU/Memory allocation to query such a big catalog, even though we tweaked the RAM allocation (we couldn't find a way to do it for CPU). Spent too much time on this so we stopped these efforts for now. It would be good to get back to it when we need to install something else from a custom catalog.
letian added 7 commits 2025-10-23 20:39:48 +00:00
add more logging and add reporting message
Some checks failed
Run Check Script / check (pull_request) Failing after 1m23s
Details
89cb23f4f7
letian added 1 commit 2025-10-28 19:42:54 +00:00
fix command to find next available port-channel id
Some checks failed
Run Check Script / check (pull_request) Failing after 1m13s
Details
ec77afa605
letian added 1 commit 2025-10-28 19:52:57 +00:00
Merge remote-tracking branch 'origin/master' into nmstate
Some checks failed
Run Check Script / check (pull_request) Failing after 1m16s
Details
f63f77550f
letian added 1 commit 2025-10-28 22:32:30 +00:00
improve more logs, better error messages when switch setup failed
Some checks failed
Run Check Script / check (pull_request) Failing after 1m12s
Details
981529751a
letian added 1 commit 2025-10-29 15:27:27 +00:00
adjust nm config policy serialization
Some checks failed
Run Check Script / check (pull_request) Failing after 1m28s
Details
d6c6192c6b
letian added 1 commit 2025-10-29 16:21:28 +00:00
revert opnsense config change
All checks were successful
Run Check Script / check (pull_request) Successful in 1m18s
Details
b8ac3580f8
letian changed title from WIP: fix(host_network): adjust bond & port-channel configuration to fix(host_network): adjust bond & port-channel configuration (partial) 2025-10-29 17:08:16 +00:00
letian merged commit c80ede706b into master 2025-10-29 17:09:16 +00:00
letian deleted branch nmstate 2025-10-29 17:09:17 +00:00
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
1 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#175
No description provided.
Delete Branch "nmstate"

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?