Skip to content

Conversation

@joaopalet
Copy link
Contributor

@joaopalet joaopalet commented Nov 13, 2024

Draft PR to allow state move from old Argus to new Observability resources

  • Follows instructions the official Terraform documentation but does not yet work
  • Still gives error that provider "does not support moved operations across different resource types"

Tip

Workaround for users who want to migrate their Argus instance to Observability instance

  1. Create the new stackit_observability_instance in the config (should have same fields and values as the old Argus one)
  2. Import the existing instance to the new stackit_observability_instance resource
    • $ terraform import stackit_observability_instance.example <PROJECT_ID>,<INSTANCE_ID>
  3. Delete the old Argus instance resource from the state
    • $ terraform state rm stackit_argus_instance.<INSTANCE_RESOURCE_NAME>

Related to #532

@marceljk marceljk marked this pull request as ready for review January 16, 2025 15:07
@marceljk
Copy link
Contributor

Moving a argus resource to observability works. Descriptions of the argus resources extended where it's explained how to do it

@marceljk marceljk changed the title feat: Allow move state on new Observability resources (not working) feat: Allow move state on new Observability resources Jan 16, 2025
bahkauv70
bahkauv70 previously approved these changes Jan 17, 2025
Copy link
Contributor

@bahkauv70 bahkauv70 left a comment

Choose a reason for hiding this comment

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

Looks good to me, test worked flawlessly. One minor thing only: Would it make sense to add examples of the move blocks from the terraform files to the documentation of the deprecated argus resources?

@marceljk
Copy link
Contributor

Looks good to me, test worked flawlessly. One minor thing only: Would it make sense to add examples of the move blocks from the terraform files to the documentation of the deprecated argus resources?

@bahkauv70 Yes, I think it's a good idea. I'll add some examples to the documentation

@marceljk marceljk merged commit 1a66887 into main Jan 20, 2025
3 checks passed
@marceljk marceljk deleted the feat/allow-state-move-agus-observability branch January 20, 2025 08:54
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.

4 participants