Skip to content

remove env var and bind-mounting of storage related to profile - #7

Merged
Jun Aishima (JunAishima) merged 1 commit into
NSLS2:mainfrom
JunAishima:remove-tiled-profile-items
Oct 2, 2026
Merged

Jun Aishima (JunAishima) merged 1 commit into
NSLS2:mainfrom
JunAishima:remove-tiled-profile-items

Conversation

@JunAishima

Copy link
Copy Markdown
Contributor
  • these are no longer necessary since we are using the Tiled URI

 * these are no longer necessary since we are using the Tiled URI
Copilot AI balanced review requested due to automatic review settings October 2, 2026 14:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

Workflows use the configured Tiled URI and retain access to API-key credentials through /srv.

Review effort: Balanced
Findings: None

What changed in this PR

Removes obsolete Tiled profile configuration now that workflows connect through the Tiled URI.

Changes:

  • Removes TILED_SITE_PROFILES.
  • Removes the Tiled profiles bind mount.
File Description
prefect.yaml Simplifies deployment configuration by removing obsolete Tiled profile settings.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@JunAishima
Jun Aishima (JunAishima) merged commit d7f7db5 into NSLS2:main Oct 2, 2026
1 check passed
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