Skip to content

Add compatibility for metal-console API V2. - #722

Draft
Gerrit91 wants to merge 8 commits into
masterfrom
metal-console-dual-api
Draft

Add compatibility for metal-console API V2.#722
Gerrit91 wants to merge 8 commits into
masterfrom
metal-console-dual-api

Conversation

@Gerrit91

@Gerrit91 Gerrit91 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Description

This is the approach for single metal-console deployment with dual API compatibility.

Depends on:

Used AI-Tools ✨

  • None used for generation

@metal-robot metal-robot Bot added the area: deployment Affects the deployment area. label Aug 14, 2026
@Gerrit91
Gerrit91 marked this pull request as ready for review August 17, 2026 13:17
@Gerrit91
Gerrit91 requested a review from a team as a code owner August 17, 2026 13:17
@Gerrit91
Gerrit91 requested a review from vknabel August 17, 2026 13:17

@vknabel vknabel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

fine from my perspective. Will approve once metal-stack/metal-console#61 has been merged and released.

identifier: metal-console
name: metal-console
description: tenant used by the metal-console
register: _tenant

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Please check back with #453 where I made some important additions (run_once here and use-latest-identifier for the token module).

- name: Create api token for metal-console
delegate_to: localhost
environment:
METAL_APIV2_URL: "{{ metal_apiserver_url }}"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Cross-usage of variable. In #453, I added a shared partition variable for this. Please use them and create a default for this role with a fallback to the partition default.

@Gerrit91
Gerrit91 marked this pull request as draft August 19, 2026 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: deployment Affects the deployment area.

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants