Skip to content

Require ownership before adopting service Docker setup folders and apps - #2281

Merged
amangalampalli-ks merged 2 commits into
fix/service-setup-ownership-checkfrom
fix/service-setup-ownership-check-int
Aug 12, 2026
Merged

Require ownership before adopting service Docker setup folders and apps#2281
amangalampalli-ks merged 2 commits into
fix/service-setup-ownership-checkfrom
fix/service-setup-ownership-check-int

Conversation

@amangalampalli-ks

Copy link
Copy Markdown
Contributor

Summary

  • Service Docker and integration setup adopted shared folders, KSM apps, and config records by hardcoded name with no ownership check, so a shared same-name folder could receive the operator’s credentials.
  • Setup now only reuses objects owned by the current account; non-owned name matches are skipped and a new owned object is created.

Changes

  • Gate folder, KSM app, and config-record adoption in the shared Docker setup base on ownership.
  • Apply the same ownership checks to integration folder/record lookup on re-run.
  • Allow forced KSM app create when a non-owned same-title app would otherwise block setup.
  • Add unit tests for squat skip, owned preference, and create fallback.

@amangalampalli-ks amangalampalli-ks self-assigned this Aug 11, 2026
@amangalampalli-ks
amangalampalli-ks marked this pull request as ready for review August 11, 2026 05:51
@amangalampalli-ks
amangalampalli-ks force-pushed the fix/service-setup-ownership-check-int branch from 70f7ad3 to 95cec08 Compare August 12, 2026 05:30
@amangalampalli-ks
amangalampalli-ks merged commit aff9257 into fix/service-setup-ownership-check Aug 12, 2026
4 checks passed
craiglurey pushed a commit that referenced this pull request Aug 12, 2026
…ps (#2281) (#2285)

* Require ownership before adopting service Docker setup folders and apps

* Use folder_add instead of mkdir to create duplicate name share folder
@adeshmukh-ks
adeshmukh-ks deleted the fix/service-setup-ownership-check-int branch August 13, 2026 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants