Skip to content

test: unskip wired stream crud defs - #614

Merged
margaretjgu merged 14 commits into
mainfrom
test/unskip-wired-stream-crud
Sep 10, 2026
Merged

test: unskip wired stream crud defs#614
margaretjgu merged 14 commits into
mainfrom
test/unskip-wired-stream-crud

Conversation

@margaretjgu

@margaretjgu margaretjgu commented Sep 4, 2026

Copy link
Copy Markdown
Member

Relates to #593. Unskips wired stream CRUD. Enable teardown leaves streams on, query streams are created via PUT /_query, and attachments use a provisioned dashboard saved object.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ BASH shellcheck 2 0 0 0.33s
✅ COPYPASTE jscpd yes no no 0.52s
✅ REPOSITORY gitleaks yes no no 50.43s
✅ REPOSITORY git_diff yes no no 0.11s
✅ REPOSITORY secretlint yes no no 8.97s
✅ REPOSITORY trivy yes no no 21.22s
✅ TYPESCRIPT eslint 3 0 0 3.08s
✅ YAML yamllint 17 0 0 1.36s

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

See detailed reports in MegaLinter artifacts
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@JoshMock

JoshMock commented Sep 8, 2026

Copy link
Copy Markdown
Member

LGTM, but seeing the following when trying to setup wired streams in CI:

{"statusCode":409,"error":"Conflict","message":"Could not acquire lock for applying changes","attributes":{}}

@margaretjgu
margaretjgu force-pushed the test/unskip-wired-stream-crud branch from 6d03943 to 86a510f Compare September 9, 2026 18:41
@margaretjgu

Copy link
Copy Markdown
Member Author

LGTM, but seeing the following when trying to setup wired streams in CI:

{"statusCode":409,"error":"Conflict","message":"Could not acquire lock for applying changes","attributes":{}}

The 409 lock is handled in run-kb-tests-runner.sh on main now. A lock waits on /api/streams/_status instead of re-POSTing. A name conflict deletes leftover logs data streams and POSTs once.

@margaretjgu
margaretjgu force-pushed the test/unskip-wired-stream-crud branch from 96396b8 to 94a5978 Compare September 9, 2026 20:02
@margaretjgu
margaretjgu force-pushed the test/unskip-wired-stream-crud branch from 94a5978 to eacfb06 Compare September 10, 2026 17:05
@github-actions

Copy link
Copy Markdown
Contributor

PR type mismatch: title uses test: unskip wired stream crud defs, diff looks like fix instead.

The diff includes a production change that sends stream content import as multipart instead of JSON, which is a bug fix, not test-only work.

This affects the version bump and changelog entry release-please generates on merge. Update the PR title (and squash commit message) to match, or push a follow-up commit that justifies the current prefix.

@margaretjgu
margaretjgu merged commit 40c989b into main Sep 10, 2026
35 checks passed
@margaretjgu
margaretjgu deleted the test/unskip-wired-stream-crud branch September 10, 2026 20:10
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