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

[darc/maestro++] Support removing repo merge policies #2552

Open
mmitche opened this issue Aug 1, 2019 · 0 comments
Open

[darc/maestro++] Support removing repo merge policies #2552

mmitche opened this issue Aug 1, 2019 · 0 comments

Comments

@mmitche
Copy link
Member

mmitche commented Aug 1, 2019

@dougbu commented on Wed Jul 03 2019

Currently the only darc commands related to repository policies are get-repository-policies and set-repository-policies. While that covers CRU because set-repository-policies both creates and updates (well, overwrites) policies, there is no command to delete repo policies.

For example, we have old policies for non-existent branches that won't be serviced. They just add noise to darc get-repository-policies --all output. I know --all isn't the default but it's very useful when creating policies for branches that don't yet exist.

For example

darc get-repository-policies --repo https://github.com/aspnet/Extensions --all
https://github.com/aspnet/Extensions @ master
- Merge Policies:
  Standard
https://github.com/aspnet/Extensions @ release/3.0-preview3
- Merge Policies:
  Standard
https://github.com/aspnet/Extensions @ release/3.0-preview4
- Merge Policies:
  Standard
https://github.com/aspnet/Extensions @ release/3.0-preview5
- Merge Policies:
  Standard
https://github.com/aspnet/Extensions @ release/3.0-preview53
- Merge Policies:
  Standard
https://github.com/aspnet/Extensions @ release/3.0-preview6
- Merge Policies:
  Standard
https://github.com/aspnet/Extensions @ release/3.0-preview7
- Merge Policies:
  Standard

Hmm, that output includes policies for a branch that never existed. The 'release/3.0-preview53' repo policies must have been accidentally created (probably by me ☹️).

@dkurepa dkurepa transferred this issue from dotnet/arcade Jun 2, 2023
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

No branches or pull requests

2 participants