Skip to content

SkipResourceType#466

Merged
daltondhcp merged 3 commits intoAzure:mainfrom
Jefajers:SkipResourceType
Nov 9, 2021
Merged

SkipResourceType#466
daltondhcp merged 3 commits intoAzure:mainfrom
Jefajers:SkipResourceType

Conversation

@Jefajers
Copy link
Copy Markdown
Member

@Jefajers Jefajers commented Nov 9, 2021

Overview/Summary

Adds the capability to exclude/skip resource types at rg scope that should not or cant be synced.
By default it excludes/skips: 'Microsoft.VSOnline/plans','Microsoft.PowerPlatform/accounts','Microsoft.PowerPlatform/enterprisePolicies'

This PR fixes/adds/changes/removes

  1. closing Resource discovery fails on Microsoft.VSOnline/plans #427
  2. adds new setting called "SkipResourceType"

Breaking Changes

N/A

Testing Evidence

Tested that declared resource types are excluded from workflows.

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

@Jefajers Jefajers requested a review from a team as a code owner November 9, 2021 10:43
@daltondhcp
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Comment thread docs/wiki/Settings.md Outdated
Co-authored-by: Johan Dahlbom <johan@dahlbom.eu>
@daltondhcp daltondhcp linked an issue Nov 9, 2021 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@daltondhcp daltondhcp left a comment

Choose a reason for hiding this comment

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

LGTM

@daltondhcp daltondhcp merged commit c8d3290 into Azure:main Nov 9, 2021
@Jefajers Jefajers deleted the SkipResourceType branch November 9, 2021 13:26
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.

Resource discovery fails on Microsoft.VSOnline/plans

2 participants