Skip to content
This repository was archived by the owner on May 27, 2026. It is now read-only.

op-mode: T7583: add a new operational command runner - #40

Merged
dmbaturin merged 1 commit into
vyos-legacy:currentfrom
dmbaturin:T7583-op-mode-runner
Aug 12, 2025
Merged

op-mode: T7583: add a new operational command runner#40
dmbaturin merged 1 commit into
vyos-legacy:currentfrom
dmbaturin:T7583-op-mode-runner

Conversation

@dmbaturin

Copy link
Copy Markdown
Contributor

Change summary

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe): internal change.

Related Task(s)

Related PR(s)

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

@dmbaturin
dmbaturin force-pushed the T7583-op-mode-runner branch 3 times, most recently from a0bd7f3 to 592add1 Compare August 7, 2025 12:54
@dmbaturin
dmbaturin marked this pull request as ready for review August 7, 2025 12:55
@dmbaturin
dmbaturin force-pushed the T7583-op-mode-runner branch from 592add1 to dc72756 Compare August 7, 2025 12:55
@dmbaturin
dmbaturin force-pushed the T7583-op-mode-runner branch from dc72756 to ad5b369 Compare August 7, 2025 12:56
@jestabro

jestabro commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

PR for build dependencies:
vyos/vyos-build#1006

@jestabro

jestabro commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Required bug fix to the op-mode-cache:
vyos/vyos-1x#4653

@jestabro jestabro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested, and with the build dependency PR and the bug fix PR above, all looks good to me.

@dmbaturin
dmbaturin merged commit 9285b34 into vyos-legacy:current Aug 12, 2025
2 of 3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants