Skip to content

fix(aws): recognize natGateway interface type for EIP association - #1949

Merged
ioito merged 1 commit into
yunionio:masterfrom
ioito:hotfix/qx-aws-eip-nat
Nov 7, 2026
Merged

ioito merged 1 commit into
yunionio:masterfrom
ioito:hotfix/qx-aws-eip-nat

Conversation

@ioito

@ioito ioito commented Nov 7, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • 公网 NAT 的 EIP:DescribeAddresses 返回 serviceManaged=rnat,且无 networkInterfaceId/instanceId,原逻辑无法识别关联
  • 同时兼容 ENI interfaceType=natGateway(及文档中的 nat_gateway)
  • 按 AllocationId/PublicIp 反查 NAT;GetIEips 优先按 AllocationId 查询

Test plan

  • 同步公网 NAT 绑定的 EIP 后,associate_type=natgateway 且 associate_id 指向对应 NAT
  • 未绑定资源的 EIP 仍保持未关联

Public NAT Gateway EIPs often have serviceManaged=rnat without
networkInterfaceId. Also recognize ENI interfaceType natGateway
(not only nat_gateway), resolve NAT by allocation/public IP, and
prefer AllocationId when listing NAT EIPs.
@ioito
ioito force-pushed the hotfix/qx-aws-eip-nat branch from 8f9492f to cc116e1 Compare November 7, 2026 08:40
@ioito
ioito merged commit a0566ee into yunionio:master Nov 7, 2026
1 check passed
ioito added a commit that referenced this pull request Nov 7, 2026
…stream-release-3.11

Automated cherry pick of #1949: fix(aws): recognize natGateway interface type for EIP association
ioito added a commit that referenced this pull request Nov 7, 2026
…stream-release-4.0

Automated cherry pick of #1949: fix(aws): recognize natGateway interface type for EIP association
ioito added a commit that referenced this pull request Nov 7, 2026
…stream-release-4.1

Automated cherry pick of #1949: fix(aws): associate NAT EIP via serviceManaged=rnat and natGateway
ioito added a commit that referenced this pull request Nov 7, 2026
…stream-release-4.1.0

Automated cherry pick of #1949: fix(aws): associate NAT EIP via serviceManaged=rnat and natGateway
ioito added a commit that referenced this pull request Nov 7, 2026
…stream-release-4.0.5

Automated cherry pick of #1949: fix(aws): associate NAT EIP via serviceManaged=rnat and natGateway
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant