Skip to content

[DNM/WIP] PSS: multistage provider download experiment #37350

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
Draft
wants to merge 13 commits into
base: pss/multistage-provider-download-prep
Choose a base branch
from
Draft

[DNM/WIP] PSS: multistage provider download experiment #37350

wants to merge 13 commits into from
+1,499 −26

Conversation

SarahFrench
Copy link
Member

WIP

Target Release

1.14.x

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

SarahFrench added 13 commits July 18, 2025 19:10
@SarahFrench
Add forked version of run logic that's only used if experiments are…
488465a 
… enabled
@SarahFrench
Add getProvidersFromConfig method, initially as an exact copy of getP…
c11f436 
…roviders
@SarahFrench
Make getProvidersFromConfig not use state to get providers
a311f66
@SarahFrench
Reorder actions in experimental init - load in full config before con…
f855bbc 
…figuring the backend.
@SarahFrench
Add appendLockedDependencies method to Meta to allow multi-phase …
92623a4 
…saving to the dep locks file
@SarahFrench
Update test
c4f02bf
@SarahFrench
Update experimental init to use new getProvidersFromConfig method
63f6af9
@SarahFrench
Remove old usage of getProviders
89efc84 
At this point we're unable to download the providers only described by state
@SarahFrench
Remove misleading comment
e3ac65e 
This isn't accurate after previous reordering!
@SarahFrench
Add new getProvidersFromState method that only accepts state informat…
4e0739b 
…ion as input for getting providers. Use in experimental init and append values to existing deps lock file
@SarahFrench
Update messages sent to view about provider download phases
58eecc4
@SarahFrench
Change init to save updates to the deps lock file only once
a2e7be3
@SarahFrench
Make Terraform output report that a lock file _will_ be made after pr…
35e5fd6 
…oviders are determined from config
@SarahFrench SarahFrench added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
no-changelog-needed Add this to your PR if the change does not require a changelog entry
Projects
None yet
Milestone
No milestone
Development

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