Release prep for 0.5.2 - #198
Merged
Merged
Conversation
SQLite remains the default; PostgreSQL and MariaDB/MySQL are selected via discrete LOGSTASHUI_DB_* env vars. Django ORM only (no new CRUD layer).
TDD tasks for env-selected SQLite/PostgreSQL/MySQL, extras, Docker matrix, and BETA migrate-engine.
Record engines, extras, migrator, test matrix, and operator env in the same style as the 0.5.1 notes.
Check server version before migrate and on --skip-migrate. Strip passwords, upsert --write-env, register psycopg wait_select, seed JSONField in live dump/load, drop psql from offline docs.
If loaddata fails the target may be partial; drop/recreate and retry. A production migrator could wrap loaddata in transaction.atomic().
After testing the generated docker image it was discovered that it was not adding the pod's own IP to the ALLOWED_HOSTS. This made it so that aliveness checks were getting a 400 error. That has been addressed here. A few more edits to .gitignore for a scratch directory and relocating other lines All other changes are documentation.
Optional freeze script (wheels + docker zip + experimental PyInstaller). Default uv build unchanged. CPython 3.12 linux-x86_64, [databases] included.
Add bin/freeze_logstashui.sh for wheels, docker save, and experimental PyInstaller zips. Default uv build is unchanged. Isolated wheels install is pip --no-index on CPython 3.12 x86_64.
Show a cyan LS X.Y.Z oval for the running Logstash version on
ConnectionManager and the Policy editor Agents tab. Persist VERSION
pins as {download_dir}/logstash-X.Y.Z/bin. Agents newer than preferred
show "unreleased version" instead of a backwards Upgrade.
Migrated all database tests to `tests/integration` or `tests/unit` Now using `testcontainers[postgres]` and `testcontainers[mysql]` for integration tests. - Docker required. Tests will be skipped if docker is not present or not working Starting the migration of tests away from `src/logstashui/*/tests`
There was a hidden env var in `settings.py` that was missed with regards to disabling TLS. If DEBUG=false, it would set SECURE_SSL_REDIRECT to true. Now it sets SECURE_SSL_REDIRECT to the value of LOGSTASHUI_TLS, which defaults to true if unset.
Keep serving. Message unchanged. wsgi.py swallow left as-is.
Tracing stays off until LOGSTASHUI_OTEL=true. Native wheel extra unchanged.
ERROR-and-continue when the extra is missing. HTTP/4318 only.
DjangoInstrumentor mutates MIDDLEWARE; that must happen before get_wsgi_application() snapshots the stack.
type: ignore[import-not-found] on optional opentelemetry imports.
Standalone freeze stays experimental with no automated smoke.
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.
Reading the diff on the CHANGELOG is probably the fastest way forward here. I could paste it all in here again, but why?