Releases: lsst-sqre/times-square
0.17.0
New features
-
GET /v1/github/rendered/{display_path}
that returns a rendered GitHub-backed notebook template. This route provides the same functionality asGET /v1/pages/{page}/rendered
, but finds the page based on its GitHub URL path. This is an additional path, all existing functionality, including the existing template rendering path, remains unchanged.We need to deploy Times Square to environments where some users should not have permissions to execute notebooks, but they should have permissions to render notebook templates for certain GitHub-based notebooks. This will let us configure that access via methods that apply permissions based on URL paths.
What's Changed
Full Changelog: 0.16.0...0.17.0
0.16.0
New features
- Instrument with Sentry. Don't trace the SSE endpoint because Sentry holds spans in memory as long as the connection is open.
What's Changed
- DM-48476: Basic Sentry integration by @fajpunk in #90
- Update change log for version 0.16.0 release by @fajpunk in #91
New Contributors
Full Changelog: 0.15.0...0.16.0
0.15.0
Backwards-incompatible changes
- Migrate the database to use
TEXT
column types where previously we usedVARCHAR
columns with a (now unnecessary) length limit. This change requires a database migration on deployment. In Postgres there is no functional or performance difference betweenVARCHAR
andTEXT
columns. This change simplifies the database schema and reduce the risk of future issues with column length limits.
Bug fixes
- In the
cli
tox environment, fix the name of the executable to betimes-square
rather thantimessquare
.
Other changes
- Improved the developer documentation for database migration to concretely provide copy-and-paste-able commands for preparing and running database migrations.
What's Changed
- DM-48413: Adopt TEXT column types where useful by @jonathansick in #89
Full Changelog: 0.14.0...0.15.0
0.14.0
Other changes
-
Times Square now uses Alembic to manage database schema versioning and migrations.
-
Begin SQLAlchemy 2 adoption with the new
DeclarativeBase
,mapped_column
, and theMapped
type. -
Update the
make update
command in the Makefile to use the--universal
flag withuv pip compile
. -
Fix type checking issues for Pydantic fields to use empty lists as the default, rather than using a default factory.
-
Explicitly set
asyncio_default_fixture_loop_scope
tofunction
. An explicit setting is now required by pytest-asyncio.
What's Changed
- DM-48307: Updates for new ruff, mypy, Pydantic and Sqlalchemy changes by @jonathansick in #87
- DM-48385: Add Alembic integration by @jonathansick in #88
Full Changelog: 0.13.0...0.14.0
0.13.0
New features
-
Times Square now specifies a timeout for notebook executions with Noteburst. This provides better feedback for notebook executions that have hung, either when rendering new pages for users, for when doing a GitHub Check Run. Currently this timeout is set app-wide with the
TS_DEFAULT_EXECUTION_TIMEOUT
environment variable. The default timeout is 60 seconds. In the future we intend to add per-notebook timeout configuration. -
Times Square enforces a timeout when polling for Noteburst results during a GitHub Check run. This prevents the Check Run from hanging indefinitely if the Noteburst service is unable to time out a notebook execution or fails for any reason. This timeout is configurable via the
TS_CHECK_RUN_TIMEOUT
environment variable. The default timeout is 600 seconds.
What's Changed
- DM-46137: Enforce timeouts in GitHub Checks runs by @jonathansick in #81
Full Changelog: 0.12.0...0.13.0
0.12.0
New features
-
Improved feedback in GitHub Check Runs:
-
If a YAML file has bad syntax causing a parsing error, we now post the YAML error as a check run annotation — including the location of the error in the file.
-
If the notebook has incorrect Jinja templating in its Markdown cells, we now post a check run annotation with the error message and cell number.
-
Other changes
-
Adopt uv and tox-uv for pinning and installing dependencies
-
Pin the tox dependencies with a
requirements/tox.[in|txt]
file -
Adapt configuration for tox-docker version 5's randomized connection ports
-
Adopt ruff-shared.toml for common SQuaRE ruff configuration.
-
The GitHub Check Run service methods are now part of a new
GitHubCheckRunService
class, separate from theGitHubRepoService
. This internal change helps clarify what functionality is needed for the GitHub Checks functionality, as opposed to syncing data with GitHub repositories.
What's Changed
- [neophile] Update dependencies by @neophile-square in #73
- DM-46065: Update for tox-docker 5 and ruff-shared.toml by @jonathansick in #78
- Bump python from 3.12.2-slim-bullseye to 3.12.5-slim-bullseye by @dependabot in #79
- DM-46095: Improve GitHub check run feedback by @jonathansick in #80
Full Changelog: 0.11.0...0.12.0
0.11.0
New features
- New support for background recomputation of a page instance (cached HTML) with the new
DELETE /v1/pages/:page/html?{params}
endpoint. This endpoint triggers a Noteburst computation of the page instance and deletes the currently-cached HTML once that computation is complete. This API provides a way for users to request a recomputation of a page instance without affecting other users that may be viewing that page instance. - A new server-sent events endpoint for getting updates on the status of a page instance's computation and HTML rendering:
GET /v1/pages/:page/html/events?{params}
. This endpoint should replace the current practice of clients polling theGET /v1/pages/:page/htmlstatus
endpoint to determine when a page instance's HTML is ready to be displayed. The events endpoint also provides additional metadata, such as the time when the current computation job was queued so that clients can provide more detailed status information to users. This endpoint works well with the newDELETE /v1/pages/:page/html?{params}
endpoint, as it can provide updates on the status of the recomputation job while still linking to the existing cached HTML.
What's Changed
- Bump pre-commit/action from 3.0.0 to 3.0.1 by @dependabot in #71
- DM-43327: Add background replacement of cached page renders by @jonathansick in #72
Full Changelog: 0.10.0...0.11.0
0.10.0
New features
- We've adopted Safir's
safir.fastapi.ClientRequestError
so that errors like 404 and 422 (input validation) now use the same error format as FastAPI uses for its built-in model validation errors. For parameter errors from endpoints likeGET /v1/pages:page/html
the parameter name is now part of theloc
field in the error message. - Times Square and its worker now send uncaught exceptions to a Slack webhook for better error reporting. The webhook URL is set in the
TS_SLACK_WEBHOOK_URL
environment variable.
Other changes
- Updated to Python 3.12
- Updated to Pydantic 2
- Adopted Ruff for linting and formatting, replacing black, flake8, and isort.
- Switch to using Annotated for Pydantic models and FastAPI path functions.
What's Changed
- [neophile] Update dependencies by @neophile-square in #62
- [neophile] Update dependencies by @neophile-square in #63
- [neophile] Update dependencies by @neophile-square in #64
- [neophile] Update dependencies by @neophile-square in #65
- DM-43173: Update to ruff linting and pydantic 2 by @jonathansick in #69
- DM-43264: Adopt Safir ClientRequestError and Slack messaging for internal errors by @jonathansick in #70
Full Changelog: 0.9.2...0.10.0
0.9.2
Bug fixes
- Fix how strings are rendered in the parameters cell of notebooks. Previously string parameters were missing Python quotes. Now parameters are passed to Jinja in their
repr
string forms to be proper Python code.
What's Changed
- Bump actions/checkout from 3 to 4 by @dependabot in #58
- Bump python from 3.11.4-slim-bullseye to 3.11.5-slim-bullseye by @dependabot in #57
- [neophile] Update dependencies by @neophile-square in #59
- DM-40865: Fix rendering string values in parameters cell by @jonathansick in #60
Full Changelog: 0.9.1...0.9.2
0.9.1
Bug fixes
- When a page is updated (e.g., from a GitHub pull request merge), the HTML renders for that page are cleared.
- Fixed a bug where updating a page and executing it with defaults would result in two requests to Noteburst.
- Deleting a page now deletes the page's HTML renders.
What's Changed
- [neophile] Update dependencies by @neophile-square in #55
- DM-40250: Delete nbhtml instances from redis when a page updates by @jonathansick in #56
New Contributors
- @neophile-square made their first contribution in #55
Full Changelog: 0.9.0...0.9.1