-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
FabricBot: Onboarding to GitOps.ResourceManagement because of FabricBot decommissioning #4971
base: main
Are you sure you want to change the base?
Conversation
…t as FabricBot replacement Details on the replacement service and the syntax of the new yaml configuration file is available publicly at: https://microsoft.github.io/GitOps/policies/resource-management.html Please review and merge this PR to complete the process of onboarding to the new service.
Thanks dotnet-policy-service[bot] for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a lot of gaps on this config migration. This repository is not managed by the .NET team so it wasn't on our group's radar.
Owners of this repo are welcome to connect with me, @wtgodbe, and @mkArtakMSFT if they'd like to reference how we approached overcoming our feature gaps, but most of the issues surfaced in my review don't have clear solutions.
"subCapability": "IssuesOnlyResponder", | ||
"version": "1.0", | ||
"config": { | ||
"taskName": "Add needs triage and bug label to new issues", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This task didn't get ported, presumably because it references isPartOfProject
which is not supported.
"issues", | ||
"project_card" | ||
], | ||
"taskName": "Add Voting to new Feature Requests", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feature didn't get ported over because addReactionToIssue
is not supported.
- addLabel: | ||
label: 'needs attention :wave:' | ||
- addReply: | ||
reply: "This issue has been marked as \"needs attention \U0001F44B\" due to no activity for **15 days**. Please triage the issue so the fix can be established. " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reply: "This issue has been marked as \"needs attention \U0001F44B\" due to no activity for **15 days**. Please triage the issue so the fix can be established. " | |
reply: "This issue has been marked as \"needs attention :wave:" due to no activity for **15 days**. Please triage the issue so the fix can be established. " | |
action: Opened | ||
then: | ||
- addReply: | ||
reply: "Thanks ${issueAuthor} for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request \U0001F64C" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reply: "Thanks ${issueAuthor} for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request \U0001F64C" | |
reply: "Thanks ${issueAuthor} for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request :raised_hands:" | |
- addLabel: | ||
label: 'needs attention :wave:' | ||
- addReply: | ||
reply: "This PR has been marked as \"needs attention \U0001F44B\" and awaiting a response from the team. " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reply: "This PR has been marked as \"needs attention \U0001F44B\" and awaiting a response from the team. " | |
reply: "This PR has been marked as \"needs attention :wave:\" and awaiting a response from the team. " | |
"issues", | ||
"project_card" | ||
], | ||
"taskName": "Remove milestone if the PR is removed from the project column", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Project board automation is not supported in the new service, so related tasks were not ported.
} | ||
} | ||
], | ||
"taskName": "Move to In-progress column in the project once In-PR label applied" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Project board automation is not supported in the new service, so related tasks were not ported.
"issues", | ||
"project_card" | ||
], | ||
"taskName": "Move to In-progress column in the project once In-PR label applied", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Project board automation is not supported in the new service, so related tasks were not ported.
"issues", | ||
"project_card" | ||
], | ||
"taskName": "Move to In-progress column in the project once In-PR label applied", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Project board automation is not supported in the new service, so related tasks were not ported.
"issues", | ||
"project_card" | ||
], | ||
"taskName": "Move to In-progress column in the project once In-PR label applied", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Project board automation is not supported in the new service, so related tasks were not ported.
CommunityToolkit/.github#5 needs to be merged to add the policy service bot to the CLA bypass list. |
This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 15 days. It will be closed if no further activity occurs within 30 days of this comment. |
This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 15 days. It will be closed if no further activity occurs within 30 days of this comment. |
@dotnet-policy-service[bot] please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement ( “Agreement” ) is agreed to by the party signing below ( “You” ), 1. Definitions. “Code” means the computer software code, whether in human-readable or machine-executable form, “Project” means any of the projects owned or managed by .NET Foundation and offered under a license “Submit” is the act of uploading, submitting, transmitting, or distributing code or other content to any “Submission” means the Code and any other copyrightable material Submitted by You, including any 2. Your Submission. You must agree to the terms of this Agreement before making a Submission to any 3. Originality of Work. You represent that each of Your Submissions is entirely Your 4. Your Employer. References to “employer” in this Agreement include Your employer or anyone else 5. Licenses. a. Copyright License. You grant .NET Foundation, and those who receive the Submission directly b. Patent License. You grant .NET Foundation, and those who receive the Submission directly or c. Other Rights Reserved. Each party reserves all rights not expressly granted in this Agreement. 6. Representations and Warranties. You represent that You are legally entitled to grant the above 7. Notice to .NET Foundation. You agree to notify .NET Foundation in writing of any facts or 8. Information about Submissions. You agree that contributions to Projects and information about 9. Governing Law/Jurisdiction. This Agreement is governed by the laws of the State of Washington, and 10. Entire Agreement/Assignment. This Agreement is the entire agreement between the parties, and .NET Foundation dedicates this Contribution License Agreement to the public domain according to the Creative Commons CC0 1. |
Add prIssueManagement.yml to onboard repo to GitOps.ResourceManagement as FabricBot replacement
Details on the replacement service and the syntax of the new yaml configuration file is available publicly at: https://microsoft.github.io/GitOps/policies/resource-management.html
Please review and merge this PR to complete the process of onboarding to the new service.