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

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

Draft
letian wants to merge 7 commits from nmstate into master
pull from: nmstate
merge into: NationTech:master
Conversation 0 Commits 7 Files Changed 18 +406 -208
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)

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 considering installing an operator from the OperatorHub.io catalog didn't work (timeout error with DeadlineExceeded probably caused by an insufficient CPU/Memory allocation to query such a big catalog), it was the simplest way to achieve what we wanted for now: install the NMState operator to configure bonds on a host.

## 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) ## 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 considering installing an operator from the OperatorHub.io catalog didn't work (timeout error with DeadlineExceeded probably caused by an insufficient CPU/Memory allocation to query such a big catalog), it was the simplest way to achieve what we wanted for now: install the NMState operator to configure bonds on a host.
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
Some checks failed
Run Check Script / check (pull_request) Failing after 1m23s
Details
This pull request has changes conflicting with the target branch.
  • harmony/src/domain/topology/ha_cluster.rs
  • harmony/src/domain/topology/k8s.rs
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin nmstate:nmstate
git checkout nmstate
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?