Add browser pool profile refresh policy - #111
Open
IlyaasK wants to merge 2 commits into
Open
Conversation
IlyaasK
force-pushed
the
hypeship/browser-pool-data-source-acceptance
branch
from
August 3, 2026 14:26
3259a54 to
7d4f4a5
Compare
IlyaasK
force-pushed
the
hypeship/browser-pool-refresh-on-profile-update-resource
branch
from
August 3, 2026 14:31
a342a28 to
20143f3
Compare
IlyaasK
force-pushed
the
hypeship/browser-pool-data-source-acceptance
branch
from
August 3, 2026 14:53
7d4f4a5 to
eef1ff4
Compare
IlyaasK
force-pushed
the
hypeship/browser-pool-refresh-on-profile-update-resource
branch
from
August 3, 2026 14:53
20143f3 to
e43bf84
Compare
Expose Kernel's durable refresh_on_profile_update setting through the browser pool resource. Preserve API defaults when omitted, retain explicit false values across profile changes, validate the profile dependency, and cover create/update/read behavior.
IlyaasK
force-pushed
the
hypeship/browser-pool-refresh-on-profile-update-resource
branch
from
August 3, 2026 18:25
09b0ad5 to
f4021b9
Compare
IlyaasK
changed the base branch from
hypeship/browser-pool-data-source-acceptance
to
fix/browser-pool-qa-boundaries
August 3, 2026 18:25
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
refresh_on_profile_updatebrowser-pool policy as an optional/computed Terraform attributetruerequiresprofile_idWhy
Kernel's API and Go SDK already support this durable browser-pool setting. Terraform should manage the same desired state as the CLI without introducing browser-pool runtime operations.
The resource keeps an explicitly configured
falsevalue when a profile changes. When the attribute is omitted, profile attachment, replacement, or removal leaves the value to Kernel's profile-dependent default. Unrelated updates preserve the value already read from the API and avoid a noisy unknown plan.Intentionally unsupported
This does not add a browser-pool data source, browser acquire/release/flush operations, leased-browser state, force deletion, profile mutation, or any other runtime/session behavior.
Verification
gofmt -l cmd internalgo test -short -timeout=2m ./...go vet ./...terraform fmt -check -recursive examplesbash scripts/check-docs.shbash scripts/check-markdown-links.shLive acceptance was not rerun for this history-only restack. The browser-pool acceptance workflow remains the final live gate after the resource stack is assembled.