OCPCLOUD-2710: implement extended AWS metadata service options - #170
OCPCLOUD-2710: implement extended AWS metadata service options#170RadekManak wants to merge 3 commits into
Conversation
|
@RadekManak: This pull request references OCPCLOUD-2710 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@RadekManak: This pull request references OCPCLOUD-2710 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
|
@RadekManak this will soon also need |
|
@RadekManak this was been open for a little while. Is it still relevant? |
Cover explicit enable/disable and precedence over the dual-stack default.
Temporary replace of github.com/openshift/api to the local workspace checkout so MAPA can build and test against the IMDS API PR.
6ae4120 to
b73bc9b
Compare
|
/remove-lifecycle stale Still relevant — rebased, added HTTPProtocolIPv6 plumbing (and dual-stack default behavior), and updated tests. API PR: openshift/api#2654 |
|
@RadekManak: This pull request references OCPCLOUD-2710 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
No actionable comments were generated in the recent review. 🎉 WalkthroughThe module graph is updated, and instance metadata request construction now maps HTTP endpoint, IPv6 protocol, hop limit, and metadata tag settings. Tests add coverage for explicit values, dual-stack fallback behavior, and combined metadata configurations. ChangesIMDS Metadata Options
Estimated code review effort: 3 (Moderate) | ~20 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@RadekManak: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
This PR extends the AWS Instance Metadata Service (IMDS) configuration options in Machine API Provider for AWS (MAPA) to achieve parity with Cluster API Provider AWS (CAPA).
What this PR does
Adds support for three new metadata service options when creating AWS instances:
HTTPEndpoint- Controls whether the HTTP metadata endpoint is enabled or disabled on EC2 instancesEnabled: The metadata endpoint is accessibleDisabled: The metadata endpoint is not accessibleHTTPPutResponseHopLimit- Configures the desired HTTP PUT response hop limit for instance metadata requests (valid range: 1-64). This is useful for containers and multi-hop network setups where the default hop limit of 1 may be insufficient.InstanceMetadataTags- Controls whether instance tags are accessible from the instance metadata serviceEnabled: Allows instance tags to be retrieved from IMDSDisabled: Instance tags are not accessible from IMDSThese options are plumbed into the
InstanceMetadataOptionsRequestwhen calling the EC2RunInstancesAPI.Why this is needed
CAPA currently supports these metadata service configuration options while MAPA does not. This gap causes issues during CAPI to MAPI conversion when these fields are populated. By implementing these options in MAPA:
API PR: openshift/api#2654
Summary by CodeRabbit
New Features
Bug Fixes
Chores