rubrik-modules-for-ansible icon indicating copy to clipboard operation
rubrik-modules-for-ansible copied to clipboard

rubrik_create_sla

Open KowMangler opened this issue 4 years ago • 4 comments

Description

add allowed backup window start hour, min and duration

Related Issue

This project only accepts pull requests related to open issues.

  • If suggesting a new feature or change, please discuss it in an issue first.

Enhancement #103

  • If fixing a bug, there should be an issue describing it with steps to reproduce

Please link to the issue here

Motivation and Context

Why is this change required? What problem does it solve?

** allows backups to be relegated to maint windows **

How Has This Been Tested?

  • Please describe in detail how you tested your changes.
  • Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code, etc.

RHEL7 Python3.6 Ansible-2.9

Ran against 5.0.3 virtual Edge appliance. Requires rubrik python sdk with code from https://github.com/rubrikinc/rubrik-sdk-for-python/pull/247

Screenshots (if appropriate):

SLA Settings created via ansible module image

idempotent check image

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [X] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply. If you're unsure about any of these, don't hesitate to ask. We're here to help!

  • [X] My code follows the code style of this project.
  • [X] My change requires a change to the documentation.
  • [X] I have updated the documentation accordingly.
  • [X] I have read the CONTRIBUTION document.
  • [ ] I have updated the CHANGELOG file accordingly for the version that this merge modifies.
  • [X] I have added tests to cover my changes.
  • [X] All new and existing tests passed.

KowMangler avatar Aug 06 '20 19:08 KowMangler