context-data.md containg invalid identifiers #10149

Closed
opened 2025-12-29 21:27:32 +01:00 by adam · 3 comments
Owner

Originally created by @Domoninic on GitHub (Aug 28, 2024).

Change Type

Correction

Area

Features

Proposed Changes

The file context-data.md contains invalid identifiers "ntp-servers" and "syslog-servers".
If these are used in Config contexts and rendered in Config Data they cause an errors, e.g. :
"jinja2.exceptions.UndefinedError: 'syslog' is undefined"
This means the samples cannot be used as is to learn about Context data and Config templates.
Identifiers should be valid Python Names (i.e not "-" )

Originally created by @Domoninic on GitHub (Aug 28, 2024). ### Change Type Correction ### Area Features ### Proposed Changes The file context-data.md contains invalid identifiers "ntp-servers" and "syslog-servers". If these are used in Config contexts and rendered in Config Data they cause an errors, e.g. : "jinja2.exceptions.UndefinedError: 'syslog' is undefined" This means the samples cannot be used as is to learn about Context data and Config templates. Identifiers should be valid Python Names (i.e not "-" )
adam added the type: documentationnetbox labels 2025-12-29 21:27:32 +01:00
adam closed this issue 2025-12-29 21:27:32 +01:00
Author
Owner

@Domoninic commented on GitHub (Aug 28, 2024):

If this is assigned to me I can do a have a PR ready to submit to correct the identifiers.

@Domoninic commented on GitHub (Aug 28, 2024): If this is assigned to me I can do a have a PR ready to submit to correct the identifiers.
Author
Owner

@jeremystretch commented on GitHub (Feb 7, 2025):

This does not appear to be a NetBox issue.

@jeremystretch commented on GitHub (Feb 7, 2025): This does not appear to be a NetBox issue.
Author
Owner

@Domoninic commented on GitHub (Feb 7, 2025):

This does not appear to be a NetBox issue.

The sample code in the Netbox documentation is using invalid identifiers and generates errors when used. How is this not a Netbox issue?

@Domoninic commented on GitHub (Feb 7, 2025): > This does not appear to be a NetBox issue. The sample code in the Netbox documentation is using invalid identifiers and generates errors when used. How is this not a Netbox issue?
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#10149