Skip to content

Add telemetry spans to __app deploy command#2838

Open
sfc-gh-jfloyd wants to merge 20 commits intoapps-devfrom
jfloyd/add-telemetry-spans-app-deploy
Open

Add telemetry spans to __app deploy command#2838
sfc-gh-jfloyd wants to merge 20 commits intoapps-devfrom
jfloyd/add-telemetry-spans-app-deploy

Conversation

@sfc-gh-jfloyd
Copy link
Copy Markdown
Contributor

Wrap the five deployment phases (bundle, upload, build, deploy_service, endpoint_provision) with CLIMetrics.span() context managers so each phase duration and errors are captured in CLI telemetry payloads.

.... Generated with Cortex Code

Pre-review checklist

  • I've confirmed that instructions included in README.md are still correct after my changes in the codebase.
  • I've added or updated automated unit tests to verify correctness of my new code.
  • I've added or updated integration tests to verify correctness of my new code.
  • I've confirmed that my changes are working by executing CLI's commands manually on MacOS.
  • I've confirmed that my changes are working by executing CLI's commands manually on Windows.
  • I've confirmed that my changes are up-to-date with the target branch.
  • I've described my changes in the release notes.
  • I've described my changes in the section below.
  • I've described my changes in the documentation.

Changes description

...

@sfc-gh-jfloyd sfc-gh-jfloyd force-pushed the jfloyd/add-telemetry-spans-app-deploy branch from f321e17 to 3e11add Compare March 27, 2026 16:15
@sfc-gh-jfloyd sfc-gh-jfloyd changed the base branch from main to apps-dev March 27, 2026 16:27
sfc-gh-gbloom and others added 16 commits March 27, 2026 12:35
Wrap the five deployment phases (bundle, upload, build, deploy_service,
endpoint_provision) with CLIMetrics.span() context managers so each
phase duration and errors are captured in CLI telemetry payloads.

.... Generated with [Cortex Code](https://docs.snowflake.com/en/user-guide/cortex-code/cortex-code)

Co-Authored-By: Cortex Code <noreply@snowflake.com>
When a build job or deployed service enters FAILED state, fetch the
last 50 lines of container logs via SYSTEM$GET_SERVICE_LOGS and print
them before re-raising the error. This gives immediate visibility into
why a build or deploy failed without requiring a manual log query.

.... Generated with [Cortex Code](https://docs.snowflake.com/en/user-guide/cortex-code/cortex-code)

Co-Authored-By: Cortex Code <noreply@snowflake.com>
@sfc-gh-jfloyd sfc-gh-jfloyd force-pushed the jfloyd/add-telemetry-spans-app-deploy branch from 3e11add to 58d89b4 Compare March 31, 2026 14:34
@sfc-gh-jfloyd sfc-gh-jfloyd marked this pull request as ready for review March 31, 2026 15:07
@sfc-gh-jfloyd sfc-gh-jfloyd requested a review from a team as a code owner March 31, 2026 15:07
@sfc-gh-gbloom sfc-gh-gbloom force-pushed the apps-dev branch 2 times, most recently from fd3736d to c209ec4 Compare April 1, 2026 15:20
@sfc-gh-gbloom
Copy link
Copy Markdown
Contributor

Opened #2873 with the same logic on latest main

@sfc-gh-gbloom sfc-gh-gbloom force-pushed the apps-dev branch 2 times, most recently from cff080d to 2679878 Compare April 15, 2026 20:50
@sfc-gh-gbloom sfc-gh-gbloom requested a review from a team as a code owner April 15, 2026 20:50
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.

2 participants