Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Fix mutating extant resources not forcing new #157

Closed
wants to merge 110 commits into from

Conversation

OJFord
Copy link
Contributor

@OJFord OJFord commented Feb 22, 2023

DAlperin and others added 30 commits May 26, 2022 14:08
Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 2.9.1 to 3.0.0.
- [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
- [Commits](goreleaser/goreleaser-action@v2.9.1...v3.0.0)

---
updated-dependencies:
- dependency-name: goreleaser/goreleaser-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Token parameter & env were renamed
@@ -168,8 +168,11 @@ func (mr flyMachineResourceType) GetSchema(context.Context) (tfsdk.Schema, diag.
},
"volume": {
Required: true,
MarkdownDescription: "Name or ID of volume",
MarkdownDescription: "ID of volume",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

If name did work it technically still will, I just suggest not documenting it because the ID will work better - it will change if the fly_volume is replaced, so we'll detect the change here; if it's recreated with the same name then that wouldn't have changed and wouldn't force a replacement here, so better to use ID.

@zxaos
Copy link
Contributor

zxaos commented Jun 6, 2023

Closed due to history re-write, but refiled as #190

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replacing volume fails