Skip to content

[Modules] Fix Data Protection Backup Vault module#1534

Merged
eriqua merged 3 commits intomainfrom
users/erikag/fix-dataProtection
Jun 16, 2022
Merged

[Modules] Fix Data Protection Backup Vault module#1534
eriqua merged 3 commits intomainfrom
users/erikag/fix-dataProtection

Conversation

@eriqua
Copy link
Copy Markdown
Contributor

@eriqua eriqua commented Jun 15, 2022

Description

  • Aligning module to latest telemetry approach
  • Aligning module to latest lock approach
  • Adding lock test to parameter file
  • Updating API

Pipeline references

For module/pipeline changes, please create and attach the status badge of your successful run.

Pipeline
DataProtection: BackupVaults

Type of Change

Please delete options that are not relevant.

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Update to documentation

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

@eriqua eriqua added bug Something isn't working [cat] modules category: modules labels Jun 15, 2022
@eriqua eriqua self-assigned this Jun 15, 2022
@eriqua eriqua mentioned this pull request Jun 15, 2022
15 tasks
@eriqua eriqua marked this pull request as ready for review June 15, 2022 15:31
@eriqua eriqua enabled auto-merge (squash) June 15, 2022 15:32
@eriqua eriqua added the [prio] high importance of the issue: high priority label Jun 15, 2022
@eriqua eriqua merged commit 79c1b22 into main Jun 16, 2022
@eriqua eriqua deleted the users/erikag/fix-dataProtection branch June 16, 2022 09:16
@eriqua eriqua linked an issue Jun 17, 2022 that may be closed by this pull request
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working [cat] modules category: modules [prio] high importance of the issue: high priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug Report]: Fix failing module pipelines

2 participants