Skip to content

fixes imds_available#125

Merged
tomjemmett merged 1 commit into
mainfrom
fix_imds_endpoint
May 20, 2026
Merged

fixes imds_available#125
tomjemmett merged 1 commit into
mainfrom
fix_imds_endpoint

Conversation

@tomjemmett

Copy link
Copy Markdown
Member

fixes #124

@tomjemmett tomjemmett requested a review from StatsRhian May 18, 2026 13:44
@tomjemmett tomjemmett self-assigned this May 18, 2026
@tomjemmett tomjemmett requested a review from francisbarton as a code owner May 18, 2026 13:44
Copilot AI review requested due to automatic review settings May 18, 2026 13:44
@tomjemmett tomjemmett added bug Something isn't working priority: must MoSCoW priority labels May 18, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates Azure IMDS availability detection to probe the instance metadata endpoint referenced by issue #124, and bumps the package patch version.

Changes:

  • Changes ping_msi_endpoint() to query /metadata/instance?api-version=2021-02-01.
  • Bumps package version from 0.3.0 to 0.3.1.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
R/get_auth_token.R Updates the managed identity availability probe endpoint.
DESCRIPTION Bumps the package version for the fix release.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread R/get_auth_token.R
Comment thread R/get_auth_token.R
@tomjemmett

Copy link
Copy Markdown
Member Author

bump for review @StatsRhian / @matt-dray you will probably want this in output/outputs selection/inputs, so will need to bump version in description and regen manifest.

work around if we don't merge is to set IMDS_AVAILABLE on all the apps

@tomjemmett tomjemmett merged commit 98e6a2a into main May 20, 2026
5 checks passed
@tomjemmett tomjemmett deleted the fix_imds_endpoint branch May 20, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: must MoSCoW priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

imds_available isn't working

3 participants