Skip to content

feat(tcc): add validation support for Intel TCC profiles#18

Merged
ishamisu merged 3 commits into
mainfrom
heamings/dev/tcc-validate-config
May 22, 2026
Merged

feat(tcc): add validation support for Intel TCC profiles#18
ishamisu merged 3 commits into
mainfrom
heamings/dev/tcc-validate-config

Conversation

@ishamisu
Copy link
Copy Markdown
Contributor

This update introduces validation support for TCC configuration using the command: cmd>tch tcc validate .

Changes

  • Added YANG module schema definitions to support Intel TCC profiles
  • Implemented configuration validation for Intel TCC using the new schema
  • The universal parser now recognizes the Intel vendor-specific schema.

Testing Result: PASSED
((tcc_venv) ) root@p12hl01zlohx1:/home/hmsu/networking/iTCH/iTCH-TCC-Profile/time-confighub# PYTHONPATH=src python3 -m time_config_hub.cli tcc validate examples/scenario-intel-tcc/scenario-rtc-testbench-1ms-rtworkload.xml
Validating TCC configuration file...
TCC configuration file examples/scenario-intel-tcc/scenario-rtc-testbench-1ms-rtworkload.xml is valid.
Configuration file examples/scenario-intel-tcc/scenario-rtc-testbench-1ms-rtworkload.xml is valid
✓ Configuration validated successfully

- Added YANG module schema definitions to support Intel TCC profiles
- Implemented configuration validation for Intel TCC using the new schema
@ishamisu ishamisu self-assigned this May 13, 2026
@ishamisu ishamisu added the enhancement New feature or request label May 13, 2026
Copy link
Copy Markdown
Contributor

@AzuraTarmiziIntel AzuraTarmiziIntel left a comment

Choose a reason for hiding this comment

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

Need more discussion.

Comment thread examples/scenario-intel-tcc/scenario-rtc-testbench-1ms-rtworkload.xml Outdated
Comment thread src/yang_modules/intel-tcc-config.yang
Copy link
Copy Markdown
Contributor

@engkeong-sve engkeong-sve left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@liowsj liowsj left a comment

Choose a reason for hiding this comment

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

some changes to naming required.

Comment thread examples/scenario-intel-tcc/scenario-rtc-testbench-1ms-rtworkload.xml Outdated
Comment thread examples/scenario-intel-tcc/scenario-rtc-testbench-1ms-rtworkload.xml Outdated
@ishamisu ishamisu changed the base branch from heamings/dev/tcc to main May 22, 2026 02:56
Copy link
Copy Markdown
Contributor

@liowsj liowsj left a comment

Choose a reason for hiding this comment

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

LGTM

@ishamisu ishamisu merged commit 684734a into main May 22, 2026
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants