8000 [cloud-provider-zvirt] Enable migrator from terraform state to opentofu by name212 · Pull Request #13504 · deckhouse/deckhouse · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[cloud-provider-zvirt] Enable migrator from terraform state to opentofu #13504

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
May 20, 2025
Merged

[cloud-provider-zvirt] Enable migrator from terraform state to opentofu #13504

merged 2 commits into from
May 20, 2025

Conversation

name212
Copy link
Member
@name212 name212 commented May 18, 2025 •
edited
Loading

Description

In the previous pr https://github.com/deckhouse/deckhouse/pull/13386/files we forget to enable migrator to opentofu states from terraform and did not add hook for fire alert when cluster has terraform state.

Alse we added comment in terraform_versions.yaml for providers that using terraform with instruction to migrate to opentofu.

For cloud-provider-dynamics we did not add migrator because we do not have working installation with this provider.

Why do we need it, and what problem does it solve?

We have Zvirt installation on terraform. We need to enable migrator for smooth migration from terraform to opentofu.

Also we added comment in terraform_versions.yaml to prevent forgetting enable migrator and metrics hook when we migrate from terraform to opentofu

Why do we need it in the patch release (if we do)?

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: cloud-provider-zvirt
type: chore
summary: Enable migrator from terraform state to opentofu.
impact_level: low

@name212
[cloud-provider-zvirt] Enable migrator from terraform state to opentofu
351701d 
Signed-off-by: Nikolay Mitrofanov <nikolay.mitrofanov@flant.com>
@github-actions github-actions bot added area/cluster-and-infrastructure Pull requests that update infra modules go Pull requests that update Go code labels May 18, 2025
@name212 name212 added this to the v1.70.0 milestone May 18, 2025
@name212
[cloud-provider-zvirt] Enable migrator from terraform state to opentofu
0abedff 
Signed-off-by: Nikolay Mitrofanov <nikolay.mitrofanov@flant.com>
@name212 name212 added the skip/e2e Skip running e2e test for pass pull request check label May 18, 2025
@name212 name212 marked this pull request as ready for review May 18, 2025 23:03
@name212 name212 requested review from sprait and aleksey-su as code owners May 18, 2025 23:03
@name212 name212 merged commit ad4b2a0 into main May 20, 2025
46 checks passed
@name212 name212 deleted the migration-to-tofu-zvirt-fix branch May 20, 2025 14:52
YuryLysov pushed a commit that referenced this pull request May 30, 2025
@name212 @YuryLysov
[cloud-provider-zvirt] Enable migrator from terraform state to opento…
a1ce01b 
…fu (#13504)

Signed-off-by: Nikolay Mitrofanov <nikolay.mitrofanov@flant.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aleksey-su aleksey-su aleksey-su approved these changes

@sprait sprait sprait approved these changes

Assignees

@name212 name212

Labels
area/cluster-and-infrastructure Pull requests that update infra modules go Pull requests that update Go code skip/e2e Skip running e2e test for pass pull request check
Projects
None yet
Milestone
v1.70.0
Development

Successfully merging this pull request may close these issues.
3 participants
@name212 @aleksey-su @sprait
0