Repository navigation
Sync upstream main through v3.2.5 - #5
Merged
Merged
Conversation
current: eval frequency continues to be 1m unless task is restarted task restart only by - 1. alert disabled and then enabled again 2. server restart 3. alert edited consecutive eval failures stop the task entirely no automatic restart fix: successful retry restores configured eval frequency no task restart required
* tool executer api for external mcp server * deepsource fix * deepsource fix * remove promql tool, remove confirmed from tool call request, coderabbit comment fix
* expose read only tools * segregate read only tools for external use
Bring in the alert retry recovery fix, external MCP tool execution and read-only catalog, and the v3.2.5 release update. Resolve the alert scheduler conflict by preserving PromQL retry cadence and unlimited retries while restoring the configured interval after successful evaluation.
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
2 participants
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.
Description
Sync the fork with
parseablehq/parseable:mainthroughb8fb6ce(v3.2.5), preserving the fork's community PromQL, OIDC authorization, and/nextfrontend changes.Includes four upstream commits:
322e03c: restore the configured alert evaluation interval after a successful retry (upstream #1814).f44c3af: add the external MCP tool executor API (upstream #1816).81e82ab: expose the read-only tool catalog (upstream #1817).b8fb6ce: update the server and classic UI assets to v3.2.5 (upstream #1818).The merge required one conflict resolution in
src/sync.rs. Successful evaluations now restore the configured interval. PromQL alerts retain their configured retry cadence and continue retrying; other alert types retain upstream's one-minute retries and retry limit. The retry warning reports the actual delay.Use a merge commit when merging this PR to retain upstream ancestry for future syncs.
Validation
cargo test --locked --lib: 584 passed, 1 ignored, 0 failed, with the/nextfrontend embedded.node --test scripts/test-community-ui.mjs: 28 passed.cargo fmt --all -- --check: passed.cargo clippy --locked --lib -- -D warnings: passed.git diff origin/main...HEAD --check: passed.This PR has:
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.