TPT-4284:ansible implement support for reserved ip for ipv4 by mgwoj · Pull Request #788 · linode/ansible_linode

mgwoj · 2026-04-29T13:30:14Z

📝 Description

Implements support for Linode Reserved IPv4 addresses in ansible_linode.

New modules:

reserved_ip — create, update tags, and delete reserved IP addresses
reserved_ip_info — retrieve details for a single reserved IP by address
reserved_ip_list — list and filter all reserved IPs on an account

Updated modules:

instance — new ipv4 parameter to assign a reserved IP at creation time (create-only; excluded from update path to prevent handle_updates errors)
nodebalancer — new ipv4 parameter to assign a reserved IP at creation time (create-only; excluded from create params when None and popped before handle_updates)

Other changes:

Updated ip_info and ip doc fragments with new fields: reserved, tags, interface_id, assigned_entity, vpc_nat_1_1
Added integration tests: reserved_ip_basic, reserved_ip_info, reserved_ip_list, instance_reserved_ip, nodebalancer_reserved_ip
Pinned requirements.txt to linode_api4 @ git+https://github.com/linode/linode_api4-python.git@proj/reserved-ips

Generated/updated docs via make gendocs

✔️ How to Test

make test-unit

# Modified modules (regression check)
make TEST_SUITE="ip_info instance_basic nodebalancer_basic" test-int

# New reserved IP functionality
make TEST_SUITE="reserved_ip_basic reserved_ip_info reserved_ip_list instance_reserved_ip nodebalancer_reserved_ip" test-int

Release v0.41.1

Release v0.42.0

Release v0.43.0

Release v0.44.0

Release v0.45.0

Copilot

Pull request overview

Adds first-class support for Linode Reserved IPv4 addresses to the linode.cloud Ansible collection, including new modules for managing/querying reserved IPs and create-time assignment support for Instances and NodeBalancers.

Changes:

Introduces reserved_ip, reserved_ip_info, and reserved_ip_list modules plus associated doc fragments and integration tests.
Adds create-only ipv4 parameters to instance and nodebalancer to allow assigning a reserved IPv4 at creation time.
Updates IP-related doc fragments/docs to include new Reserved IP fields and pins linode_api4 to a Git ref.

Reviewed changes

Copilot reviewed 22 out of 23 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/integration/targets/reserved_ip_basic/tasks/main.yaml	New integration test covering reserve/update-tags/delete flow
tests/integration/targets/reserved_ip_info/tasks/main.yaml	New integration test for querying a reserved IP by address
tests/integration/targets/reserved_ip_list/tasks/main.yaml	New integration test for listing reserved IPs
tests/integration/targets/instance_reserved_ip/tasks/main.yaml	New integration test for assigning a reserved IP on instance create
tests/integration/targets/nodebalancer_reserved_ip/tasks/main.yaml	New integration test for assigning a reserved IP on nodebalancer create
requirements.txt	Switches `linode_api4` dependency to a Git URL
plugins/modules/reserved_ip.py	New module to create/update tags/delete reserved IPs
plugins/modules/reserved_ip_info.py	New info module to fetch a reserved IP by address
plugins/modules/reserved_ip_list.py	New list module to enumerate reserved IPs
plugins/modules/instance.py	Adds create-only `ipv4` param and excludes it from update reconciliation
plugins/modules/nodebalancer.py	Adds create-only `ipv4` param and excludes it from update reconciliation
plugins/module_utils/doc_fragments/reserved_ip.py	Doc fragment samples/examples for reserved_ip
plugins/module_utils/doc_fragments/reserved_ip_info.py	Doc fragment samples/examples for reserved_ip_info
plugins/module_utils/doc_fragments/reserved_ip_list.py	Doc fragment samples/examples for reserved_ip_list
plugins/module_utils/doc_fragments/instance.py	Adds instance example using reserved IPv4
plugins/module_utils/doc_fragments/nodebalancer.py	Adds nodebalancer example using reserved IPv4
plugins/module_utils/doc_fragments/ip.py	Updates IP sample fields
plugins/module_utils/doc_fragments/ip_info.py	Updates IP info sample fields
docs/modules/instance.md	Generated docs updated for new instance param
docs/modules/nodebalancer.md	Generated docs updated for new nodebalancer param
docs/modules/ip_info.md	Generated docs updated for new IP fields
docs/modules/ip_rdns.md	Generated docs updated for new IP fields
README.md	Adds module links for reserved_ip*, but docs appear missing

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

Copilot

Pull request overview

Copilot reviewed 25 out of 26 changed files in this pull request and generated 4 comments.

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

zliang-akamai · 2026-05-05T16:18:53Z

Some findings:

We may need to update ip or reserved_ip module for the updated /networking/ips behavior to support promoting an IP to be a reserved IP. I haven't verified if it can work this way, but the gap does exist. For cleanest solution, we might need another brand new module for this. Optionally, we can also state that this feature is intentionally unsupported because users can still use generic API call module for it, your call.
Should we declare the behavior of linode.cloud.reserved_ip that it is not idempotent when creating by region? Re-running the documented example will allocate another billable reserved IP each run.

These are mostly for discussion, the overall changes looks great!

mawilk90 · 2026-05-12T12:01:10Z

 | `type` | <center>`str`</center> | <center>Optional</center> | The type of address you are requesting. Only IPv4 addresses may be allocated through this operation.  **(Choices: `ipv4`)** |
-| `address` | <center>`str`</center> | <center>Optional</center> | The IP address to delete.  **(Conflicts With: `linode_id`,`public`,`type`)** |
+| `address` | <center>`str`</center> | <center>Optional</center> | The IP address to update or delete. Required when updating an existing IP (e.g., promoting to reserved) or when deleting (state=absent).  **(Conflicts With: `linode_id`,`public`,`type`)** |
+| `reserved` | <center>`bool`</center> | <center>Optional</center> | Whether this IP address should be reserved. Setting to true promotes an existing allocated IP to a reserved IP via PUT /networking/ips/{address}. Requires the address parameter.   |


API Spec says that "when 'reserved' is true, one of 'region' or 'linode_id' must be set" so shouldn't we also add a region as one of optional fields?

mawilk90 · 2026-05-13T08:15:45Z

What about /tags endpoints specified in the API Spec? Should we also cover them here?

mawilk90 · 2026-05-18T07:17:56Z

There are some conflicts to be resolved but generally it looks good to me

mawilk90 · 2026-05-20T15:19:28Z

@mgwoj, I've just noticed that PR goes directly to dev, could you please create a dedicated project branch?

…ent-support-for-reserved-ip-for-ipv4

lgarber-akamai · 2026-05-21T16:50:39Z

@@ -92,10 +141,135 @@ def __init__(self) -> None:
            ],


The required_together value might need to be updated to properly account for the new cases. Not sure if this will make a difference in practice though 🤔

Copilot

Pull request overview

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

    def _handle_present(self) -> None:
        params = filter_null_values(self.module.params)
+        address = params.get("address")
+
+        if address is not None:
+            self._handle_update_existing_ip(address, params)
+            return
+
        linode_id = params.get("linode_id")
        public = params.get("public")
+        region = params.get("region")
+        reserved = params.get("reserved")
+
+        if region is not None and reserved:
+            # Allocate a new reserved IP via POST /networking/ips
+            ip_type = params.get("type", "ipv4")
+            try:


+        if linode_id is None:
+            self.fail(
+                msg="linode_id, public, and type are required when creating a new IP"


yec-akamai and others added 7 commits September 3, 2025 10:11

Merge pull request linode#707 from linode/dev

58d89e6

Release v0.41.1

Merge pull request linode#715 from linode/dev

f1c8618

Release v0.42.0

Merge pull request linode#727 from linode/dev

04f653e

Release v0.43.0

Merge pull request linode#743 from linode/dev

4b21f9a

Release v0.44.0

Merge pull request linode#771 from linode/dev

88ad63c

Release v0.45.0

TPT-4284: ansible: Implement support for Reserved IP for IPv4

29d682e

TPT-4284: ansible: Implement support for Reserved IP for IPv4

5035ca4

mgwoj requested a review from Copilot April 29, 2026 13:30

mgwoj requested review from a team as code owners April 29, 2026 13:30

mgwoj added the enhancement issues that request a enhancement label Apr 29, 2026

mgwoj requested review from ezilber-akamai and psnoch-akamai and removed request for a team April 29, 2026 13:30

Copilot started reviewing on behalf of mgwoj April 29, 2026 13:30 View session

github-advanced-security AI found potential problems Apr 29, 2026

View reviewed changes

Comment thread plugins/modules/instance.py Fixed

Comment thread plugins/modules/reserved_ip.py Fixed

Comment thread plugins/modules/reserved_ip.py Fixed

Comment thread plugins/modules/reserved_ip_info.py Fixed

Copilot AI reviewed Apr 29, 2026

View reviewed changes

Comment thread plugins/modules/reserved_ip.py

Comment thread plugins/modules/reserved_ip.py Outdated

Comment thread plugins/modules/instance.py

Comment thread README.md

mgwoj changed the title ~~Feature/tpt 4284 ansible implement support for reserved ip for ipv4~~ TPT-4284:ansible implement support for reserved ip for ipv4 Apr 30, 2026

mgwoj added 2 commits April 30, 2026 13:59

TPT-4284: ansible: Implement support for Reserved IP for IPv4

ef52f17

TPT-4284: ansible: Implement support for Reserved IP for IPv4

13f8a4d

mgwoj requested a review from Copilot April 30, 2026 12:12

Copilot started reviewing on behalf of mgwoj April 30, 2026 12:13 View session

Copilot AI reviewed Apr 30, 2026

View reviewed changes

Comment thread plugins/modules/instance.py

Comment thread plugins/module_utils/doc_fragments/reserved_ip_list.py Outdated

Comment thread docs/modules/reserved_ip_list.md Outdated

Comment thread plugins/modules/reserved_ip_info.py

TPT-4284: ansible: Implement support for Reserved IP for IPv4

df9151d

zliang-akamai approved these changes May 5, 2026

View reviewed changes

mawilk90 requested review from mawilk90 and removed request for psnoch-akamai May 11, 2026 09:00

TPT-4284:ansible implement support for reserved ip for ipv4

9f57ef6

TPT-4284:ansible implement support for reserved ip for ipv4

9fcdc13

mawilk90 reviewed May 12, 2026

View reviewed changes

Comment thread tests/integration/targets/instance_reserved_ip/tasks/main.yaml Outdated

mawilk90 reviewed May 12, 2026

View reviewed changes

mawilk90 reviewed May 13, 2026

View reviewed changes

Comment thread tests/integration/targets/ip_basic/tasks/main.yaml

TPT-4284:ansible implement support for reserved ip for ipv4

6a34db3

mgwoj requested a review from zliang-akamai May 13, 2026 11:34

mawilk90 approved these changes May 18, 2026

View reviewed changes

mgwoj added 2 commits May 20, 2026 12:45

TPT-4284:ansible implement support for reserved ip for ipv4

21ef844

TPT-4284: ansible implement support for reserved ip for ipv4

de93357

mawilk90 mentioned this pull request May 20, 2026

Refactor instance_basic testing playbook #805

Open

mgwoj changed the base branch from dev to proj/reserved-ips May 21, 2026 07:06

Merge branch 'proj/reserved-ips' into feature/TPT-4284-ansible-implem…

dc14dcc

…ent-support-for-reserved-ip-for-ipv4

github-advanced-security AI found potential problems May 21, 2026

View reviewed changes

Comment thread plugins/modules/ip.py Fixed

Comment thread plugins/modules/ip.py Fixed

Comment thread plugins/modules/ip.py Fixed

Comment thread plugins/modules/reserved_ip.py Fixed

Comment thread plugins/modules/ip.py Fixed

lgarber-akamai reviewed May 21, 2026

View reviewed changes

mgwoj added 3 commits May 25, 2026 11:36

TPT-4284: ansible implement support for reserved ip for ipv4

221fa5b

TPT-4284: ansible implement support for reserved ip for ipv4

87cbdee

TPT-4284: ansible implement support for reserved ip for ipv4

87c247b

mgwoj requested review from Copilot, lgarber-akamai and mawilk90 May 25, 2026 09:44

Copilot started reviewing on behalf of mgwoj May 25, 2026 09:45 View session

Copilot AI reviewed May 25, 2026

View reviewed changes

mgwoj added 2 commits May 25, 2026 12:08

TPT-4284: ansible implement support for reserved ip for ipv4

7a4cc2c

TPT-4284: ansible implement support for reserved ip for ipv4

fb5d092

Conversation

mgwoj commented Apr 29, 2026

📝 Description

✔️ How to Test

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

zliang-akamai commented May 5, 2026

Uh oh!

Uh oh!

mawilk90 May 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

mawilk90 commented May 13, 2026

Uh oh!

mawilk90 commented May 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mawilk90 commented May 20, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lgarber-akamai May 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

mawilk90 May 12, 2026 •

edited

Loading

mawilk90 commented May 18, 2026 •

edited

Loading