Skip to content

All ComponentResource classes should call the registerOutputs method #2544

@cptanalatriste

Description

@cptanalatriste

✅ Checklist

  • I have searched open and closed issues for duplicates.
  • This is a request for a new feature in the Data Safe Haven or an upgrade to an existing feature.
  • The feature is still missing in the latest version.
  • I have read through the documentation.
  • This isn't an open-ended question (open a discussion if it is).

🍓 Suggested change

Per the Pulumi documentation on Component resources:

Failing to call registerOutputs could cause serious issues with your component resource:

The component will appear as “creating…” indefinitely in the Pulumi Console
Outputs will not be saved to the state file, potentially causing data loss
The component lifecycle will not complete properly, which may affect dependency tracking and updates

🚂 How could this be done?

Add the missing method call in all DSH components. For example, this is missing in EntraApplicationComponent and LocalDnsRecordComponent.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew functionality that should be added to the Safe Haven

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions