Skip to content

Conversation

Copy link

Copilot AI commented Jan 29, 2026

Description

The keystore is referenced in management.md but lacked dedicated user documentation. Users couldn't discover supported key types (RSA, X25519, WiFi PSK, WireGuard PSK) or usage patterns without exploring YANG models.

Changes:

  • doc/keystore.md: New documentation covering asymmetric keys (SSH RSA, WireGuard X25519), symmetric keys (WiFi/WireGuard PSKs), key management operations, NACM access controls, and troubleshooting
  • doc/management.md: Added reference to keystore.md in SSH section
  • mkdocs.yml: Added keystore.md to System navigation

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):
Original prompt

This section details on the original issue you should resolve

<issue_title>Missing documentation for Infix keystore</issue_title>
<issue_description>### Current Behavior

Infix management documentation refers to the keystore as part of the SSH section, but there is no user documenation on the Infix keystore and how to use it.

As minimum, the existing functionality should be documented. Perhaps there should also be additional functionality, but that would render a new issue (or the title of this issue should be changed).

The following functionality is available for configuration

admin@example:/config/> edit keystore
admin@example:/config/keystore/> help
  asymmetric-key                    An asymmetric key.
  symmetric-key                     A symmetric key.
admin@example:/config/keystore/>

The current online help is sparse, and I cannot say if it is correct

admin@example:/config/keystore/> help asymmetric-key 
NAME
        name <string>

DESCRIPTION
        An arbitrary name for the asymmetric key.

admin@example:/config/keystore/> help symmetric-key 
NAME
        name <string>

DESCRIPTION
        An arbitrary name for the symmetric key.

admin@example:/config/keystore/>

This is what the config output looks like on infix default (qemu).

admin@example:/config/keystore/> show
asymmetric-keys {
  asymmetric-key genkey {
    public-key-format ssh-public-key-format;
    public-key MIIBCgKCAQEAm6uCENSafz7mIfIJ8O.... AQAB;
    private-key-format rsa-private-key-format;
    cleartext-private-key MIIEvAIBADANBgkqhkiG9w0BAQEFAASCBKYw...b7dyPr4mpHg==;
  }
}


admin@example:/config/keystore/>

Expected Behavior

User documentation for the keystore could perhaps be put in separate file like "doc/keystore.md".
It could contain rudimentary information on how it works, and example of how to use it.

Steps To Reproduce

Not applicable

Additional information

None</issue_description>

<agent_instructions>This is a documentation-only change, so please add [skip ci] to the commit message(s).</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Add documentation for Infix keystore usage Add keystore documentation Jan 29, 2026
Copilot AI requested a review from troglobit January 29, 2026 20:35
@troglobit troglobit requested review from jovatn and removed request for troglobit January 29, 2026 20:39
@troglobit troglobit marked this pull request as ready for review January 29, 2026 20:39
[skip ci]

Fixes #914

Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@troglobit troglobit force-pushed the copilot/add-infix-keystore-documentation branch from d74a6bb to f44dd91 Compare January 30, 2026 10:06
@troglobit troglobit requested a review from mattiaswal January 30, 2026 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing documentation for Infix keystore

3 participants