Skip to content

Conversation

SarahFrench
Copy link
Member

A quick and dirty recreation of #37323 by cherry picking commits onto the base branch. This work's been blocked for a while so it was impacted by lots of changes on main I've been doing in the meantime.

@SarahFrench SarahFrench added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Sep 19, 2025
@radeksimko radeksimko force-pushed the radek/pss-read-write branch 6 times, most recently from ac83419 to bc638ec Compare September 25, 2025 16:33
Base automatically changed from radek/pss-read-write to main September 26, 2025 16:03
SarahFrench and others added 21 commits September 29, 2025 16:11
…arsed config.

This can only be done once modules have been parsed and the required providers data is available. There are multiple places where config is parsed, into either Config or Module structs, so this needs to be implemented in multiple places.
…ath for adding a state store for the first time
SarahFrench and others added 29 commits September 29, 2025 16:13
This change was following discussions with Product
…is currently only possible for testing init commands interacting with the PSS command
Otherwise the provider never receives any state data to write!
* Update Read/WriteStateBytes RPCs to match hashicorp/terraform-plugin-go#531

* Run `make protobuf`

* Run `make generate`

* Update use of `proto.ReadStateBytes_ResponseChunk` in tests

* Fix how diagnostics are handled alongside EOF error, update ReadStateBytes test

* More fixes - test setup was incorrect

I think? I assume that a response would be returned full of zero-values when EOF is encountered.
@SarahFrench SarahFrench force-pushed the pss/init-of-new-pss-workingdir-2 branch from d08b300 to 24a44d7 Compare September 29, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog-needed Add this to your PR if the change does not require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants