Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: downgrade Azure IMDS required version #8578

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

smira
Copy link
Member

@smira smira commented Apr 10, 2024

Fixes #8555

It seems that older version supports same set of fields we actually use in our platform code, so we can safely downgrade to the version supported by Azure Stack Hub.

I used
this repo to check schemas across versions.

@smira
Copy link
Member Author

smira commented Apr 10, 2024

/promote e2e-azure

@smira
Copy link
Member Author

smira commented Apr 10, 2024

This seems to break Azure 😨

@smira
Copy link
Member Author

smira commented Apr 17, 2024

@smira smira added the platform/azure Azure platform-related label May 31, 2024
@smira
Copy link
Member Author

smira commented May 31, 2024

[    8.644394] [talos] fetching network config from "http://169.254.169.254/metadata/instance/network/interface?api-version=2021-12-13&format=json"
[    8.663187] [talos] fetching load balancer metadata from: "http://169.254.169.254/metadata/loadbalancer?api-version=2021-12-13&format=json"

seems to work fine on Azure (proper)

@smira smira force-pushed the fix/azure-stack-hub branch 2 times, most recently from f3781e9 to bfbbed5 Compare May 31, 2024 17:41
@smira
Copy link
Member Author

smira commented May 31, 2024

/m

Fixes siderolabs#8555

It seems that older version supports same set of fields we actually use
in our platform code, so we can safely downgrade to the version
supported by Azure Stack Hub.

I used
[this repo](https://github.com/Azure/azure-rest-api-specs/tree/main/specification/imds/data-plane/Microsoft.InstanceMetadataService/stable)
to check schemas across versions.

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
@smira
Copy link
Member Author

smira commented Jun 3, 2024

/m

@talos-bot talos-bot merged commit 9a23d84 into siderolabs:main Jun 3, 2024
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Proposed
Development

Successfully merging this pull request may close these issues.

talosctl bootstrap fails on Azure Stack Hub
3 participants