Skip to content

Conversation

@ajankovicTT
Copy link
Contributor

What's changed

Preliminary test infra documentation, useful to the rest of LLK team and anyone who knows how to write tests. Includes getting started document with code snippets, preliminary architecture overview and debuging tips&trick for some bugs myself and other people came across.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

Thank you for your contribution! 🚀
If you want to run metal post-commit tests, you can add the metal-post-commit-tests label to this pull request.
📖 For more information, please refer to our CONTRIBUTING guide.

@ajankovicTT ajankovicTT force-pushed the ajankovic/test-infra-docs branch from bfa440d to c8be111 Compare January 8, 2026 14:49
@ajankovicTT ajankovicTT changed the title docs: Testing infra documentation docs: Preliminary testing infra documentation Jan 8, 2026
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jan 8, 2026
@ajankovicTT ajankovicTT marked this pull request as ready for review January 8, 2026 15:13
Copilot AI review requested due to automatic review settings January 8, 2026 15:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds preliminary documentation for the LLK (Low-Level Kernel) test infrastructure. The documentation aims to help team members understand how to write, run, and debug tests for the test infrastructure.

  • Getting started guide with code examples for writing functional and performance tests
  • Architecture overview explaining compilation process and memory layouts
  • Debugging guide with tips and common issue checklists

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 13 comments.

File Description
docs/tests/infra_architecture.md Detailed architecture documentation covering file paths, TestConfig object, kernel compilation, L1 memory layouts, and runtime behavior (contains multiple TODO sections)
docs/tests/getting_started.md Step-by-step guide for writing and running functional/performance tests with Python and C++ code examples
docs/tests/debugging_guide.md Troubleshooting checklist organized by compilation, runtime, and assertion errors (contains some TODO sections)
docs/tests/images/compilation-graph.svg Diagram illustrating the kernel compilation pipeline from source files to ELF binaries

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ajankovicTT ajankovicTT self-assigned this Jan 8, 2026
@ajankovicTT ajankovicTT force-pushed the ajankovic/test-infra-docs branch from 1155d83 to 983baed Compare January 9, 2026 09:19
@ajankovicTT ajankovicTT enabled auto-merge January 14, 2026 11:07
@ajankovicTT ajankovicTT force-pushed the ajankovic/test-infra-docs branch from 983baed to bd27fce Compare January 16, 2026 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants