Skip to content

TPT-4552: Add modules for object storage global quota#824

Open
zliang-akamai wants to merge 4 commits into
devfrom
zhiwei/obj-quota
Open

TPT-4552: Add modules for object storage global quota#824
zliang-akamai wants to merge 4 commits into
devfrom
zhiwei/obj-quota

Conversation

@zliang-akamai

Copy link
Copy Markdown
Member

📝 Description

Add list and info modules for the new object storage global quota endpoints.

✔️ How to Test

make integration-test TEST_SUITE=object_storage_global_quota_info
make integration-test TEST_SUITE=object_storage_global_quota_list```

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

Adds new Ansible collection modules to list and fetch details for the Linode Object Storage global quota endpoints, along with generated docs and integration coverage.

Changes:

  • Introduce object_storage_global_quota_list and object_storage_global_quota_info modules.
  • Add documentation fragments and generated module docs, and link the modules in README.md.
  • Add integration tests for the new list/info modules and adjust ListModule’s default description when filters are disabled.

Reviewed changes

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

Show a summary per file
File Description
tests/integration/targets/object_storage_global_quota_list/tasks/main.yaml New integration test for the list module.
tests/integration/targets/object_storage_global_quota_info/tasks/main.yaml New integration test for the info module (includes quota usage).
README.md Adds the new modules to the module index tables.
plugins/modules/object_storage_global_quota_list.py New list module wired to /object-storage/global-quotas with filters disabled.
plugins/modules/object_storage_global_quota_info.py New info module with optional secondary result to fetch quota usage.
plugins/module_utils/linode_common_list.py Makes default list-module description accurate when disable_filters=True.
plugins/module_utils/doc_fragments/object_storage_global_quota_list.py Adds specdoc examples and sample results for list module docs.
plugins/module_utils/doc_fragments/object_storage_global_quota_info.py Adds specdoc examples and sample results for info module docs.
docs/modules/object_storage_global_quota_list.md Generated docs for list module.
docs/modules/object_storage_global_quota_info.md Generated docs for info module (needs return behavior clarification).

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

Comment thread tests/integration/targets/object_storage_global_quota_info/tasks/main.yaml Outdated
Comment thread docs/modules/object_storage_global_quota_info.md
Comment thread tests/integration/targets/object_storage_global_quota_list/tasks/main.yaml Outdated

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

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

Comment thread pyproject.toml
]
max-positional-arguments = 12
py-version = "3.9"
py-version = "3.10"

from __future__ import absolute_import, division, print_function

from typing import Any
client: LinodeClient,
object_storage_global_quota: dict[str, Any],
params: list[InfoModuleParam],
) -> dict[str, Any] | None:
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.

2 participants