ec_deployment: Add snapshot settings by pascal-hofmann · Pull Request #620 · elastic/terraform-provider-ec · GitHub
Skip to content

ec_deployment: Add snapshot settings - #620

Merged
tobio merged 2 commits into
elastic:masterfrom
pascal-hofmann:ec_deployment-with-snapshots
Apr 19, 2023
Merged

tobio merged 2 commits into
elastic:masterfrom
pascal-hofmann:ec_deployment-with-snapshots

Conversation

@pascal-hofmann

Copy link
Copy Markdown
Contributor

Description

Add basic snapshot and snapshot repository configuration to ec_deployment resource.

Related Issues

#602

Motivation and Context

In Elastic Cloud Enterprise snapshot settings can be configured via the ECE UI / REST API. This PR adds the same functionality to the ec_deployment resource.

How Has This Been Tested?

  • Unit tests for the new attributes have been added.
  • Additional manual tests against our local dev ECE instance were performed.

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (improves code quality but has no user-facing effect)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation

Readiness Checklist

  • My code follows the code style of this project
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed - Existing tests were already broken. Can't run all tests because of missing credentials for EC SaaS.

@elasticmachine

Copy link
Copy Markdown
Collaborator

@pascal-hofmann

pascal-hofmann commented Apr 14, 2023 •

Copy link
Copy Markdown
Contributor Author

@dimuon @tobio This PR goes hand in hand with #613.

I'll start working on a data source for snapshot repositories now. With this I think the snapshot support in this provider will be in a very good shape. ➡️ I just figured that we don't need this right now, so I will skip this.

@pascal-hofmann
pascal-hofmann force-pushed the ec_deployment-with-snapshots branch from d5814bf to b3a0a92 Compare April 19, 2023 07:41

@tobio tobio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code LGTM. Just need some time to test this one against local ECE. Thanks for these improvements @pascal-hofmann!

@tobio

tobio commented Apr 19, 2023

Copy link
Copy Markdown
Contributor

jenkins test this

@tobio
tobio merged commit 014f412 into elastic:master Apr 19, 2023
@pascal-hofmann
pascal-hofmann deleted the ec_deployment-with-snapshots branch April 19, 2023 18:46
@pascal-hofmann

Copy link
Copy Markdown
Contributor Author

@tobio Thanks for the quick review and merge. Is there already a planned release date for 0.7.0?

@tobio

tobio commented Apr 27, 2023

Copy link
Copy Markdown
Contributor

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.

3 participants