Skip to content

Cherry-pick OtherConfig commits#111

Merged
heyvister1 merged 5 commits intoMellanox:network-operator-26.1.xfrom
rollandf:otherconfigbp261
Dec 23, 2025
Merged

Cherry-pick OtherConfig commits#111
heyvister1 merged 5 commits intoMellanox:network-operator-26.1.xfrom
rollandf:otherconfigbp261

Conversation

@rollandf
Copy link
Member

@rollandf rollandf commented Dec 23, 2025

No description provided.

e0ne and others added 5 commits December 23, 2025 12:37
@e0ne @rollandf
Make ovs-vswitchd service 'other_config' option configurable
3e241c9 
Signed-off-by: Ivan Kolodiazhnyi <ikolodiazhny@nvidia.com>
(cherry picked from commit 6493e92)
(cherry picked from commit 58efb4f)
@e0ne @rollandf
fix: Typo in ovs-vswitchd.service unit file
01314ea 
Signed-off-by: Ivan Kolodiazhnyi <ikolodiazhny@nvidia.com>
(cherry picked from commit 2cfba75)
(cherry picked from commit e05610a)
@almaslennikov @rollandf
fix: run ovs-vsctl service command from the bash shell
14a82f1 
otherwise && and | operators aren't gonna work

Signed-off-by: Alexander Maslennikov <amaslennikov@nvidia.com>
(cherry picked from commit 4b9597c)
@almaslennikov @rollandf
fix: do not pass arguments to systemctl daemon-reload
48b370c 
this command does not take arguments thus failing before

Signed-off-by: Alexander Maslennikov <amaslennikov@nvidia.com>
(cherry picked from commit 3072ff0)
(cherry picked from commit d409c77)
@e0ne @rollandf
Make externalIds list sorted for systemd unit file
7d6d292 
Signed-off-by: Ivan Kolodiazhnyi <ikolodiazhny@nvidia.com>
(cherry picked from commit fefa935)
(cherry picked from commit 62f62c1)
@github-actions
Copy link

github-actions bot commented Dec 23, 2025

Thanks for your PR,
To run vendors CIs, Maintainers can use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs, Maintainers can use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@github-actions github-actions bot added the tests label Dec 23, 2025
@e0ne e0ne closed this Dec 23, 2025
@e0ne e0ne reopened this Dec 23, 2025
@github-actions
Copy link

github-actions bot commented Dec 23, 2025

Thanks for your PR,
To run vendors CIs, Maintainers can use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs, Maintainers can use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@e0ne e0ne closed this Dec 23, 2025
@e0ne e0ne reopened this Dec 23, 2025
@github-actions
Copy link

github-actions bot commented Dec 23, 2025

Thanks for your PR,
To run vendors CIs, Maintainers can use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs, Maintainers can use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@heyvister1
Copy link
Collaborator

heyvister1 commented Dec 23, 2025

/test-all

@heyvister1 heyvister1 closed this Dec 23, 2025
@heyvister1 heyvister1 reopened this Dec 23, 2025
@github-actions
Copy link

github-actions bot commented Dec 23, 2025

Thanks for your PR,
To run vendors CIs, Maintainers can use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs, Maintainers can use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@rollandf rollandf closed this Dec 23, 2025
@rollandf rollandf reopened this Dec 23, 2025
@github-actions
Copy link

github-actions bot commented Dec 23, 2025

Thanks for your PR,
To run vendors CIs, Maintainers can use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs, Maintainers can use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

e0ne
e0ne approved these changes Dec 23, 2025
View reviewed changes
Copy link
Collaborator

@e0ne e0ne left a comment

Choose a reason for hiding this comment

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

LGTM

@heyvister1 heyvister1 merged commit c2f33f9 into Mellanox:network-operator-26.1.x Dec 23, 2025
24 of 34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@e0ne e0ne e0ne approved these changes

@heyvister1 heyvister1 heyvister1 approved these changes

Assignees

No one assigned

Labels

tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@rollandf @heyvister1 @e0ne @almaslennikov