Skip to content

(bug) Surface error on missing TemplateResourceRefs#1790

Merged
gianlucam76 merged 1 commit into
projectsveltos:mainfrom
gianlucam76:missing-templateresourcerefs-error
May 15, 2026
Merged

(bug) Surface error on missing TemplateResourceRefs#1790
gianlucam76 merged 1 commit into
projectsveltos:mainfrom
gianlucam76:missing-templateresourcerefs-error

Conversation

@gianlucam76
Copy link
Copy Markdown
Member

If a resource (non optional) referenced in TemplateResourceRefs section is missing, no error showed up in the ClusterSummary making it difficult to detect why the profile was not deployed.

This PR fixes that by making sure the error is reported in the ClusterSummary Status

An example

status:
  dependencies: no dependencies
  featureSummaries:
  - failureMessage: 'Referenced resource: Secret default/imported-secret does not
      exist: secrets "imported-secret" not found'
    featureID: Resources
  nextReconcileTime: "2026-05-15T07:17:30Z"

@gianlucam76 gianlucam76 force-pushed the missing-templateresourcerefs-error branch from fd2ebc1 to b1036dd Compare May 15, 2026 08:17
If a resource (non optional) referenced in TemplateResourceRefs section
is missing, no error showed up in the ClusterSummary making it difficult
to detect why the profile was not deployed.

This PR fixes that by making sure the error is reported in the ClusterSummary
Status

An example

```yaml
status:
  dependencies: no dependencies
  featureSummaries:
  - failureMessage: 'Referenced resource: Secret default/imported-secret does not
      exist: secrets "imported-secret" not found'
    featureID: Resources
  nextReconcileTime: "2026-05-15T07:17:30Z"
```
@gianlucam76 gianlucam76 force-pushed the missing-templateresourcerefs-error branch from b1036dd to 83f29ec Compare May 15, 2026 10:11
@gianlucam76 gianlucam76 merged commit db19aed into projectsveltos:main May 15, 2026
8 checks passed
@gianlucam76 gianlucam76 deleted the missing-templateresourcerefs-error branch May 15, 2026 11:01
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