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.

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

resource/host: Streamline the resource's state management logic #18

Merged
merged 2 commits into from
Oct 23, 2024
Merged

resource/host: Streamline the resource's state management logic #18

merged 2 commits into from
Oct 23, 2024

Conversation

janartodesk
Copy link
Member

  • Leverage plan modifiers for transferring state between Host resource's lifecycle stages.
  • Encapsulate state population logic into State.ApplyHost() and State.ApplyEnrollmentCode() helpers.

@janartodesk
Implement State.ApplyHost() and State.ApplyEnrollmentCode() helpers
b52fdc2 
Encapsulate state population logic into the `State` struct.
@janartodesk
Use inline plan modifiers for state transitions
4c197c0 
Let Terraform do the heavy lifting transporting the resource's state between lifecycle stages.
@janartodesk janartodesk added the maintenance House keeping label Oct 23, 2024
@janartodesk janartodesk self-assigned this Oct 23, 2024
@janartodesk janartodesk marked this pull request as ready for review October 23, 2024 05:48
@janartodesk janartodesk requested a review from a team as a code owner October 23, 2024 05:48
@janartodesk janartodesk merged commit 55e8a7a into main Oct 23, 2024
4 checks passed
@janartodesk janartodesk deleted the dev/resource-host-state-streamlining branch October 23, 2024 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@janartodesk janartodesk

Labels
maintenance House keeping
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@janartodesk