Skip to content
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.

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

Terraform major upgrade #2852

Merged
merged 38 commits into from
Jan 26, 2022
Merged

Terraform major upgrade #2852

merged 38 commits into from
Jan 26, 2022

Conversation

seriva
Copy link
Collaborator

@seriva seriva commented Jan 10, 2022

@seriva seriva changed the title Terraform major upgrade research Terraform major upgrade Jan 14, 2022
@seriva seriva marked this pull request as ready for review January 14, 2022 07:53
Copy link
Contributor

@atsikham atsikham left a comment

Choose a reason for hiding this comment

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

In addition to minor changes suggested: did you use terraform fmt? If no, it might be the right PR to do. Continue to check.

docs/home/howto/UPGRADE.md Outdated Show resolved Hide resolved
Copy link
Contributor

@erzetpe erzetpe left a comment

Choose a reason for hiding this comment

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

Additionally perform rebase from develop please.

docs/home/howto/UPGRADE.md Outdated Show resolved Hide resolved
Co-authored-by: Anatoli Tsikhamirau <[email protected]>
@seriva
Copy link
Collaborator Author

seriva commented Jan 18, 2022

Additionally perform rebase from develop please.

Done.

@seriva seriva requested a review from cicharka January 18, 2022 22:13
docs/changelogs/CHANGELOG-2.0.md Outdated Show resolved Hide resolved
@seriva seriva requested review from atsikham and erzetpe January 19, 2022 08:38
@seriva
Copy link
Collaborator Author

seriva commented Jan 19, 2022

In addition to minor changes suggested: did you use terraform fmt? If no, it might be the right PR to do. Continue to check.

Ill check it.

Applied to both AWS and Azure templates.

erzetpe
erzetpe previously approved these changes Jan 19, 2022
cicharka
cicharka previously approved these changes Jan 19, 2022
Copy link
Contributor

@cicharka cicharka left a comment

Choose a reason for hiding this comment

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

LGTM

@cicharka
Copy link
Contributor

/azp run

@seriva seriva dismissed stale reviews from cicharka and erzetpe via efd8fca January 19, 2022 12:10
@seriva seriva requested review from cicharka and erzetpe January 19, 2022 12:14
* query_yes_no for assert_incompatible_terraform added
This template when paresd by TerraformTemplateGenerator is failing due
to: `ERROR epicli - 'cli.helpers.ObjDict.ObjDict object' has no
attribute 'storage_container'`. Removal of commented lines helps parser
to fulfil its job.
@cicharka
Copy link
Contributor

/azp run

Copy link
Contributor

@atsikham atsikham left a comment

Choose a reason for hiding this comment

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

LGTM

@cicharka
Copy link
Contributor

Tested with:
provider: Azure:
✔️ apply v2.0 (all components and apps enabled)
✔️ apply v1.3, manual upgrade of Terraform, re-apply v2.0 (all components and apps enabled/minimal cluster)
✔️ delete v2.0
provider: AWS:
✔️ apply v2.0 (all components and apps enabled)
✔️ delete v2.0

@cicharka
Copy link
Contributor

Tested:

  • provider Azure:
    • Ubuntu and RedHat (all components and apps enabled):
      • init v2.0dev -> apply v2.0dev -> delete v2.0dev
      • upgrade from v1.0.1 -> v2.0dev:
        • includes manual Terraform upgrade
        • re-applying and re-upgrade to test idempotency
        • backup and recovery commands
        • delete
  • provider AWS:
    • Ubuntu and RedHat (all components and apps enabled):
      • init v2.0dev -> apply v2.0dev -> delete v2.0de
        All scenarios verified with epicli test .

@seriva seriva merged commit 2a45947 into hitachienergy:develop Jan 26, 2022
rafzei pushed a commit to rafzei/epiphany that referenced this pull request Feb 1, 2022
- Bumped Terraform from 0.12.6 to 1.1.3.
- Bumped azurerm provider from 1.38 to 2.91.
- Bumped AWS provider from 2.26 to 3.70.1
- Updated azurerm provider scripts
- Updated AWS provider scripts
- Updated Azure-cli from 2.29 to 2.32
- Fix for hitachienergy#1569.
- Added Terraform manual update documentation
- Added incompatible terraform check
- Added changelog.
- Bumped version to 2.0.0dev
- Apply terraform fmt to enforce standard on templates
rafzei pushed a commit to rafzei/epiphany that referenced this pull request Feb 1, 2022
- Bumped Terraform from 0.12.6 to 1.1.3.
- Bumped azurerm provider from 1.38 to 2.91.
- Bumped AWS provider from 2.26 to 3.70.1
- Updated azurerm provider scripts
- Updated AWS provider scripts
- Updated Azure-cli from 2.29 to 2.32
- Fix for hitachienergy#1569.
- Added Terraform manual update documentation
- Added incompatible terraform check
- Added changelog.
- Bumped version to 2.0.0dev
- Apply terraform fmt to enforce standard on templates
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