Skip to content

Conversation

@javirln
Copy link
Member

@javirln javirln commented Jan 15, 2026

This PR adds the ability to display a clickable link to view attestations in the platform UI after a successful attestation push command.

Helm Chart:

  • Added platformExternalUrl configuration field to values.yaml for operators to set the platform UI URL
  • Updated configmap template to inject the platform URL into the control plane bootstrap configuration

CLI:

  • Added logic to fetch the platform URL from the control plane's /infoz endpoint
  • Display attestation view link after successful push in the format: https://example.com/attestation/{digest}?tab=summary
  • Link only appears in table output format (not JSON/YAML) and is hidden in dry-run mode

Configuration

Operators can enable this feature by setting the platform URL in Helm values:

controlplane:
  platformExternalUrl: "https://example.com"

When configured, users will see the attestation link immediately after pushing, even if policy violations occur.

Signed-off-by: Javier Rodriguez <[email protected]>
Signed-off-by: Javier Rodriguez <[email protected]>
@javirln javirln requested review from jiparis and migmartri January 15, 2026 10:32
@javirln javirln self-assigned this Jan 15, 2026
Signed-off-by: Javier Rodriguez <[email protected]>

var attestationURL string
if !res.Status.DryRun && flagOutputFormat == output.FormatTable {
platformURL := fetchPlatformURL(ctx)
Copy link
Member

Choose a reason for hiding this comment

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

maybe this should come in the init and stored in the crafting state? That would be more in line on how the attestation process gets context today no?



## @param controlplane.platformExternalUrl Optional external URL for the platform UI. If set, CLI will display attestation view links after push
platformExternalUrl: ""
Copy link
Member

Choose a reason for hiding this comment

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

platform might be overloaded, maybe uiDashboardURL?

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.

2 participants