Skip to content

Document cluster-scoped template variables for resource tags#4680

Open
gjoranv wants to merge 1 commit intomasterfrom
gjoranv/cluster-template-variables
Open

Document cluster-scoped template variables for resource tags#4680
gjoranv wants to merge 1 commit intomasterfrom
gjoranv/cluster-template-variables

Conversation

@gjoranv
Copy link
Copy Markdown
Member

@gjoranv gjoranv commented Apr 29, 2026

Adds ${clustername} and ${clustertype} to the template variables section of the <resource-tags> reference.

Reformats the variable list as a table with consistent descriptions for all seven variables.

Companion PRs: vespaai/cloud#2588, vespa-engine/vespa#36597

@gjoranv gjoranv force-pushed the gjoranv/cluster-template-variables branch 3 times, most recently from 1798fe1 to 77f8335 Compare April 29, 2026 14:28
@gjoranv gjoranv requested a review from Copilot April 29, 2026 14:36
Copy link
Copy Markdown
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

Updates the <resource-tags> reference documentation to include cluster-scoped template variables and present all supported variables in a consistent tabular format.

Changes:

  • Add ${clustername} and ${clustertype} to the <resource-tags> template variable list.
  • Reformat the template variable list from bullets to a table with descriptions and examples.
  • Clarify combination behavior and failure mode for unknown variables.

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

Comment thread en/reference/applications/deployment.html Outdated
- Add ${clustername} and ${clustertype} to the template variables list
- Note they resolve per host at apply time, not at deployment time
- Add example showing per-cluster cost tracking pattern
@gjoranv gjoranv force-pushed the gjoranv/cluster-template-variables branch from 77f8335 to f43c252 Compare April 29, 2026 14:48
@gjoranv gjoranv requested review from frodelu and kkraune April 29, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants