Add support for automated migration API, plus other improvements#5
Open
lolodi wants to merge 19 commits intoAzureManagedRedis:mainfrom
Open
Add support for automated migration API, plus other improvements#5lolodi wants to merge 19 commits intoAzureManagedRedis:mainfrom
lolodi wants to merge 19 commits intoAzureManagedRedis:mainfrom
Conversation
Contributor
lolodi
commented
Mar 19, 2026
lolodi
commented
- Added Welcome to contributors
- Add automated migration capabilities support
- Simplify skill using skill-creator
- Add evals for testing
- Fix metrics aggregation for clustered caches
- Add version file
- Add better Claude setup instructions
… the same region and subscription
The Azure Monitor API returns only shard 0 data by default. Both get_acr_metrics scripts now detect shard count via az redis show and request per-shard metrics with \\=ShardId eq '*'\ for clustered caches. Memory and bandwidth are summed across shards; server load and connected clients use max-per-shard. Updated SKILL.md to document the aggregation behavior. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
(cherry picked from commit 6de9d66)
Move the 170-line automated migration section from SKILL.md into references/automated-migration.md, replacing it with a concise summary and pointers. This follows the progressive disclosure pattern — the agent gets key facts inline but loads full details only when needed. SKILL.md: 405 -> 257 lines (-37%) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Convert line endings to LF (was CRLF causing 'bash\r' errors on Linux) - Add .gitattributes to enforce LF for .sh files - Pass JSON body directly to az rest instead of temp file (@file syntax) - Auto-disable color output when stdout is not a terminal - Use bash builtin for lowercase instead of tr subprocess - Move local declarations out of while loop in poll_status Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
1 participant
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.