Skip to content

New functionalities and modules #106

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

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Conversation

balucio
Copy link

@balucio balucio commented Nov 13, 2024

Added module public_cloud_instance_manage.py to start, stop, shelve and unshelve OVH cloud instance
Force deletion also for not active OVH Cloud Instances
New Call for Dedicated Server specification
Allows disk group selection during installation
Cloud Instance Deletion

Copy link
Collaborator

@tortuegenialez tortuegenialez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this PR!
Some improvement are suggested.
Also: could you please squash you commits?
Could you use a tool to format following black formating ( to resolve issues with different formating of doc string for exemple)
And finaly, could you remove the module public_cloud_instance_shelving.py to avoid duplicated code.

module: dedicated_server_compatible_templates
short_description: Retrieve all compatible templates for a OVH dedicated server
description:
- This module retrieves all compatibile templates for a OVH dedicated server
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: compatible

options:
service_name:
required: true
description: The service_name
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for snake case here: service name


EXAMPLES = r'''
- name: Retrieve all compatible templates for an OVH dedicated server
synthesio.ovh.dedicated_server_compatibile_templates:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: compatible


module = AnsibleModule(
argument_spec=module_args,
supports_check_mode=True
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's no support for check mode in this module

register: dedicated_templates
'''

RETURN = ''' # '''
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are barely not using this, but your module is returning a dictionary

force_delete:
required: false
description:
- Force the instance deletion even if it is not in ACTIVE status
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's even if it is in ACTIVE status?

DOCUMENTATION = '''
---
module: public_cloud_instance_manage
short_description: Start or stop a OVH public cloud instance
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start/Stop/Shelve/Unshelve

options:
service_name:
required: true
description: The service_name
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

service name

synthesio.ovh.public_cloud_instance_manage:
instance_id: "{{ instance_id }}"
service_name: "{{ service_name }}"
instance_action: stopped
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the option for this is stop

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