Draft
Conversation
- Create new @lage-run/azure-blob-cache-storage plugin package - Add CustomCacheStoragePlugin and CustomCacheStorageConfig types to backfill-config - Remove AzureBlobCacheStorageConfig from CacheStorageConfig union - Remove @azure/storage-blob from backfill-cache dependencies - Remove Azure code from backfill-cache (moved to plugin) - Remove Azure credential handling from @lage-run/cache - Update tests for new custom plugin config format - Map legacy BACKFILL_CACHE_PROVIDER=azure-blob env var to custom plugin config Co-authored-by: ecraig12345 <5864305+ecraig12345@users.noreply.github.com>
- Update docsite remote-cache guide with plugin installation and config - Update backfill README with custom plugin and inline provider sections - Update @lage-run/config reference docs example - Update @lage-run/cache README example Co-authored-by: ecraig12345 <5864305+ecraig12345@users.noreply.github.com>
Co-authored-by: ecraig12345 <5864305+ecraig12345@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Move Azure cache implementation to plugin
Move Azure cache implementation to plugin
Mar 19, 2026
ecraig12345
reviewed
Mar 19, 2026
Co-authored-by: ecraig12345 <5864305+ecraig12345@users.noreply.github.com>
ecraig12345
reviewed
Mar 19, 2026
Member
ecraig12345
left a comment
There was a problem hiding this comment.
See comments. You can make breaking API changes if necessary to get the new API specified in the issue.
ecraig12345
reviewed
Mar 19, 2026
… .npmignore, resolve plugin relative to cwd, remove old CustomStorageConfig, fix emails in change file Co-authored-by: ecraig12345 <5864305+ecraig12345@users.noreply.github.com>
…simplify CacheOptions, move plugin impl to separate file, fix mock Co-authored-by: ecraig12345 <5864305+ecraig12345@users.noreply.github.com>
…version bumps Co-authored-by: ecraig12345 <5864305+ecraig12345@users.noreply.github.com> Agent-Logs-Url: https://github.com/microsoft/lage/sessions/cfdef28c-2024-4a09-9d85-8e96286c05ba
ecraig12345
reviewed
Mar 20, 2026
…t.ts Co-authored-by: ecraig12345 <5864305+ecraig12345@users.noreply.github.com> Agent-Logs-Url: https://github.com/microsoft/lage/sessions/cff14f63-a48c-4434-9eb9-21adc4c67087
ecraig12345
reviewed
Mar 20, 2026
| `process.env.BACKFILL_CACHE_PROVIDER_OPTIONS`is evaluated via backfill (see [`getEnvConfig()`](https://github.com/microsoft/lage/blob/master/packages/backfill-config/src/envConfig.ts#L82) in `backfill-config`). | ||
|
|
||
| For "azure-blob" cache provider with a non-sas/key-based `connectionString`(storage account endpoint) requiring azure identity authentication do not use `BACKFILL_CACHE_PROVIDER_OPTIONS`, instead populate the required env variables according to the desired identity/environment. (See [Azure Idenity SDK](https://learn.microsoft.com/en-us/javascript/api/overview/azure/identity-readme)) and set `credentialName` property in the `lage.config.js` under `cacheOptions.cacheStorageConfig.options.credentialName` or via env var `AZURE_IDENTITY_CREDENTIAL_NAME` Supported options are: | ||
| For the Azure Blob cache provider with a non-sas/key-based `connectionString` (storage account endpoint) requiring [Azure Identity](https://learn.microsoft.com/en-us/javascript/api/overview/azure/identity-readme) authentication, you can pass a `credentialName` option in the plugin config or via the `AZURE_IDENTITY_CREDENTIAL_NAME` environment variable. (Do not use `BACKFILL_CACHE_PROVIDER_OPTIONS` in this case.) Supported options are: |
Member
There was a problem hiding this comment.
Do not use
BACKFILL_CACHE_PROVIDER_OPTIONSin this case.
Figure out why this was needed and try to get rid of it maybe?
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.
files)cwdin getCacheStorageProvider.tsOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.