corvus-dotnet / corvus-dotnet/Corvus.Deployment
Add a new providers for `Set-TemporaryAzureResourceNetworkAccess`
- Dominant language
- PowerShell
- Stars
- 3
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
We have a requirement to extend the capabilities of this [Cmdlet](https://github.com/corvus-dotnet/Corvus.Deployment/blob/main/module/functions/azure/Set-TemporaryAzureResourceNetworkAccess.ps1) to support the following Azure resources:
* Key Vault
* Synapse Workspace
This will involve writing an additional 'provider' for each resource type. Documentation for how to approach this can be found [here](https://github.com/corvus-dotnet/Corvus.Deployment/tree/main/module/functions/azure/_azureResourceNetworkAccessHandlers#readme), it would also be useful to feedback any improvements to the documentation as part of this work.
There are also two sets of [Pester v4](https://pester.dev/docs/v4/quick-start) tests related to this:
* [Set-TemporaryAzureResourceNetworkAccess.Handlers.Tests.ps1](https://github.com/corvus-dotnet/Corvus.Deployment/blob/main/module/functions/azure/Set-TemporaryAzureResourceNetworkAccess.Handlers.Tests.ps1) - unit tests to ensure that all handlers implement the required functions
* [Set-TemporaryAzureResourceNetworkAccess.Tests.ps1](https://github.com/corvus-dotnet/Corvus.Deployment/blob/main/module/functions/azure/Set-TemporaryAzureResourceNetworkAccess.Tests.ps1) - integration tests to validate each handler
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with the _azureResourceNetworkAccessHandlers README and module/functions/azure/Set-TemporaryAzureResourceNetworkAccess.ps1, then review the existing providers. Use Set-TemporaryAzureResourceNetworkAccess.Handlers.Tests.ps1 and Set-TemporaryAzureResourceNetworkAccess.Tests.ps1 as the checks for completion: Key Vault and Synapse Workspace handlers should implement the required functions and pass integration tests. Note any useful documentation improvements.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- azure, powershell
- Domain
- cloud, devops
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100