Skip to content
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

(backup): Support for advancedBackupSettings #14803

Closed
2 tasks
jhojgaard opened this issue May 20, 2021 · 7 comments · Fixed by #15934
Closed
2 tasks

(backup): Support for advancedBackupSettings #14803

jhojgaard opened this issue May 20, 2021 · 7 comments · Fixed by #15934
Labels
@aws-cdk/aws-backup Related AWS Backup effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. good first issue Related to contributions. See CONTRIBUTING.md p2

Comments

@jhojgaard
Copy link

AWS Backup BackupPlan construct is missing support for AdvancedBackupSetting
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-backup-backupplan-advancedbackupsettingresourcetype.html

Use Case

This is needed to support Windows VSS in AWS Backup configured with CDK

Proposed Solution

Add something like: backupOptions: {"WindowsVSS": "enabled"} to support the advanced backup options

Other

  • 👋 I may be able to implement this feature request
  • ⚠️ This feature might incur a breaking change

This is a 🚀 Feature Request

@jhojgaard jhojgaard added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels May 20, 2021
@github-actions github-actions bot added the @aws-cdk/aws-backup Related AWS Backup label May 20, 2021
@MrArnoldPalmer MrArnoldPalmer added effort/small Small work item – less than a day of effort good first issue Related to contributions. See CONTRIBUTING.md p2 and removed needs-triage This issue or PR still needs to be triaged. labels May 20, 2021
@Patil2099
Copy link

@MrArnoldPalmer I would love to work on this issue. Can you guide me a little on how can I solve it?

@MrArnoldPalmer
Copy link
Contributor

hey @Patil2099. This should be pretty straightforward if you checkout the BackupPlan resource you will need to add a new optional property to BackupPlanProps that then is pass in the constructor to the CfnBackupPlan arguments.

@BenChaimberg BenChaimberg changed the title (aws backup): Support for advancedBackupSettings(module name): short issue description (aws backup): Support for advancedBackupSettings May 28, 2021
@MrArnoldPalmer MrArnoldPalmer removed their assignment Jun 21, 2021
@BenChaimberg BenChaimberg changed the title (aws backup): Support for advancedBackupSettings (backup): Support for advancedBackupSettings Jun 22, 2021
@BenChaimberg BenChaimberg removed their assignment Jun 22, 2021
@BenChaimberg
Copy link
Contributor

I am unassigning myself and marking this issue as p2, which means that we are unable to work on this immediately.

We use +1s to help prioritize our work, and are happy to reevaluate this issue based on community feedback. You can reach out to the cdk.dev community on Slack to solicit support for reprioritization.

Please feel free to work on this issue yourself and request my review if you submit a PR or would like design help! See CONTRIBUTING.md for guidelines.

@cyuste
Copy link
Contributor

cyuste commented Aug 9, 2021

As a workaround, you can use this code

 const plan = backup.BackupPlan.daily35DayRetention(this, 'Plan');

const cfnPlan = plan.node.defaultChild as backup.CfnBackupPlan;
const currPlan = cfnPlan.backupPlan as backup.CfnBackupPlan.BackupPlanResourceTypeProperty;
const backupPlan : backup.CfnBackupPlan.BackupPlanResourceTypeProperty = {
  ...currPlan,
  advancedBackupSettings: [{
    backupOptions: {"WindowsVSS": "enabled"},
    resourceType : 'EC2',
  }]
};
cfnPlan.backupPlan = backupPlan;

@fitzoh
Copy link

fitzoh commented Nov 25, 2021

I wasn't sure how to do @cyuste 's workaround in python, so here's an alternate method in python:

        plan = aws_backup.BackupPlan.daily35_day_retention(self, "plan")
        cfn_plan: aws_backup.CfnBackupPlan = plan.node.default_child
        cfn_plan.add_property_override(
            "BackupPlan.AdvancedBackupSettings",
            [
                {
                    "BackupOptions": {"WindowsVSS": "enabled"},
                    "ResourceType": "EC2",
                }
            ],
        )

Also looks like #15934 might be getting close to the finish line

@mergify mergify bot closed this as completed in #15934 Dec 1, 2021
mergify bot pushed a commit that referenced this issue Dec 1, 2021
Closes #14803.
Reference Pull Request : #14891 

Lets you set the 'WindowsVss' option when you create a new backup plan like this:

```ts
const plan = new BackupPlan(stack, 'Plan', {
  windowsVss: true,
});
```
----
*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@github-actions
Copy link

github-actions bot commented Dec 1, 2021

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

@fitzoh
Copy link

fitzoh commented Dec 4, 2021

Thanks @cokia ! 🎉

TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this issue Feb 21, 2022
Closes aws#14803.
Reference Pull Request : aws#14891 

Lets you set the 'WindowsVss' option when you create a new backup plan like this:

```ts
const plan = new BackupPlan(stack, 'Plan', {
  windowsVss: true,
});
```
----
*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-backup Related AWS Backup effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. good first issue Related to contributions. See CONTRIBUTING.md p2
Projects
None yet
6 participants