Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

T7678: Remove host-resources from CLI#57

Closed
natali-rs1985 wants to merge 2 commits intovyos-legacy:currentfrom
natali-rs1985:T7678
Closed

T7678: Remove host-resources from CLI#57
natali-rs1985 wants to merge 2 commits intovyos-legacy:currentfrom
natali-rs1985:T7678

Conversation

@natali-rs1985
Copy link
Contributor

@natali-rs1985 natali-rs1985 commented Aug 5, 2025

It will be configured in section "system option host-resources"

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)
  • Other (please describe): remove CLI section

Related Task(s)

Related PR(s)

vyos/vyos-1x#4647

Proposed changes

How to test

vyos@vyos# /usr/libexec/vyos/tests/smoke/cli/test_vpp.py -k test_19_host_resources
test_19_host_resources (__main__.TestVPP.test_19_host_resources) ... ok

----------------------------------------------------------------------
Ran 1 test in 25.023s

OK

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

Copy link
Contributor

@sever-sever sever-sever left a comment

Choose a reason for hiding this comment

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

The host-resources calculations and CLI are moving to vyos/vyos-1x#4647

It will be configured in  section "system option host-resources"
@natali-rs1985 natali-rs1985 force-pushed the T7678 branch 2 times, most recently from bf0f29e to 069afd6 Compare August 5, 2025 11:17
@sever-sever
Copy link
Contributor

This PR should be moved to the vyos-1x after merging vyos/vyos-1x#4650

@natali-rs1985
Copy link
Contributor Author

Move this changes to vyos-1x repo

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants