-
Notifications
You must be signed in to change notification settings - Fork 273
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'main' of github.com:open-telemetry/opentelemetry-dotnet…
…-contrib into mike/baggage-predicate # Conflicts: # src/OpenTelemetry.Extensions/CHANGELOG.md
- Loading branch information
Showing
81 changed files
with
1,020 additions
and
218 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,25 +1,55 @@ | ||
name: 'Add labels for component found in bug issue descriptions' | ||
name: 'Add labels to issues and pull requests' | ||
on: | ||
issues: | ||
types: [opened] | ||
types: [ opened ] | ||
|
||
pull_request_target: | ||
branches: [ 'main*', 'instrumentation*', 'exporter*', 'extensions*' ] | ||
|
||
permissions: | ||
issues: write | ||
pull-requests: write | ||
|
||
jobs: | ||
add-labels: | ||
if: ${{ !github.event.issue.pull_request }} | ||
add-labels-on-issues: | ||
if: github.event_name == 'issues' && !github.event.issue.pull_request | ||
|
||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: check out code | ||
uses: actions/checkout@v4 | ||
|
||
- name: Add labels for component found in bug issue descriptions | ||
shell: pwsh | ||
run: | | ||
.\build\scripts\add-labels.ps1 -issueNumber $env:ISSUE_NUMBER -issueBody $env:ISSUE_BODY | ||
Import-Module .\build\scripts\add-labels.psm1 | ||
AddLabelsOnIssuesForComponentFoundInBody ` | ||
-issueNumber ${{ github.event.issue.number }} ` | ||
-issueBody $env:ISSUE_BODY | ||
env: | ||
GH_TOKEN: ${{ github.token }} | ||
ISSUE_NUMBER: ${{ github.event.issue.number }} | ||
ISSUE_BODY: ${{ github.event.issue.body }} | ||
|
||
add-labels-on-pull-requests: | ||
if: github.event_name == 'pull_request_target' | ||
|
||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: check out code | ||
uses: actions/checkout@v4 | ||
with: | ||
ref: ${{ github.event.repository.default_branch }} # Note: Do not run on the PR branch we want to execute add-labels.psm1 from main on the base repo only because pull_request_target can see secrets | ||
|
||
- name: Add labels for files changed on pull requests | ||
shell: pwsh | ||
run: | | ||
Import-Module .\build\scripts\add-labels.psm1 | ||
AddLabelsOnPullRequestsBasedOnFilesChanged ` | ||
-pullRequestNumber ${{ github.event.pull_request.number }} ` | ||
-labelPackagePrefix 'comp:' | ||
env: | ||
GH_TOKEN: ${{ github.token }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,148 @@ | ||
function AddLabelsOnIssuesForComponentFoundInBody { | ||
param( | ||
[Parameter(Mandatory=$true)][int]$issueNumber, | ||
[Parameter(Mandatory=$true)][string]$issueBody | ||
) | ||
|
||
$match = [regex]::Match($issueBody, '^[#]+ Component\s*OpenTelemetry\.((?:.|\w+)+)') | ||
if ($match.Success -eq $false) | ||
{ | ||
Return | ||
} | ||
|
||
gh issue edit $issueNumber --add-label $("comp:" + $match.Groups[1].Value.ToLower()) | ||
} | ||
|
||
Export-ModuleMember -Function AddLabelsOnIssuesForComponentFoundInBody | ||
|
||
function AddLabelsOnPullRequestsBasedOnFilesChanged { | ||
param( | ||
[Parameter(Mandatory=$true)][int]$pullRequestNumber, | ||
[Parameter(Mandatory=$true)][string]$labelPackagePrefix # 'pkg:' on main repo, 'comp:' on contrib repo | ||
) | ||
|
||
# Note: This function is intended to work on main repo and on contrib. Please | ||
# keep them in sync. | ||
|
||
$repoLabels = gh label list --json name,id -L 200 | ConvertFrom-Json | ||
|
||
$filesChangedOnPullRequest = gh pr diff $pullRequestNumber --name-only | ||
|
||
$labelsOnPullRequest = (gh pr view $pullRequestNumber --json labels | ConvertFrom-Json).labels | ||
|
||
$visitedProjects = New-Object System.Collections.Generic.HashSet[string] | ||
$labelsToAdd = New-Object System.Collections.Generic.HashSet[string] | ||
$labelsToRemove = New-Object System.Collections.Generic.HashSet[string] | ||
|
||
# Note: perf label may be added but it is kind of a guess so we don't remove | ||
# it automatically in order to also allow manual inclusion after reviewing files | ||
$managedLabels = 'infra', 'documentation', 'dependencies' | ||
$rootInfraFiles = 'global.json', 'NuGet.config', 'codeowners' | ||
$documentationFiles = 'readme.md', 'contributing.md', 'releasing.md', 'versioning.md' | ||
|
||
foreach ($fileChanged in $filesChangedOnPullRequest) | ||
{ | ||
$fileChanged = $fileChanged.ToLower() | ||
$fullFileName = [System.IO.Path]::GetFileName($fileChanged) | ||
$fileName = [System.IO.Path]::GetFileNameWithoutExtension($fileChanged) | ||
$fileExtension = [System.IO.Path]::GetExtension($fileChanged) | ||
|
||
if ($fileChanged.StartsWith('src/') -or $fileChanged.StartsWith('test/')) | ||
{ | ||
$match = [regex]::Match($fileChanged, '^(?:(?:src)|(?:test))\/(.*?)\/.+$') | ||
if ($match.Success -eq $false) | ||
{ | ||
continue | ||
} | ||
$rawProjectName = $match.Groups[1].Value | ||
if ($rawProjectName.Contains(".benchmarks") -or $rawProjectName.Contains(".stress")) | ||
{ | ||
$added = $labelsToAdd.Add("perf") | ||
} | ||
|
||
$projectName = $rawProjectName.Replace(".tests", "").Replace(".benchmarks", "").Replace(".stress", "") | ||
if ($visitedProjects.Contains($projectName)) | ||
{ | ||
continue | ||
} | ||
|
||
$added = $visitedProjects.Add($projectName); | ||
|
||
foreach ($repoLabel in $repoLabels) | ||
{ | ||
if ($repoLabel.name.StartsWith($labelPackagePrefix)) | ||
{ | ||
$package = $repoLabel.name.Substring($labelPackagePrefix.Length).ToLower() | ||
if ($package.StartsWith('opentelemetry') -eq $false) | ||
{ | ||
# Note: On contrib labels don't have "OpenTelemetry." prefix | ||
$package = 'opentelemetry.' + $package | ||
} | ||
if ($package -eq $projectName) | ||
{ | ||
$added = $labelsToAdd.Add($repoLabel.name) | ||
break | ||
} | ||
} | ||
} | ||
} | ||
|
||
if ($documentationFiles.Contains($fullFileName) -or | ||
$fileChanged.StartsWith('docs/') -or | ||
$fileChanged.StartsWith('examples/')) | ||
{ | ||
$added = $labelsToAdd.Add("documentation") | ||
} | ||
|
||
if ($fileChanged.StartsWith('build/') -or | ||
$fileChanged.StartsWith('.github/') -or | ||
$rootInfraFiles.Contains($fullFileName) -or | ||
$fileExtension -eq ".props" -or | ||
$fileExtension -eq ".targets" -or | ||
$fileChanged.StartsWith('test\openTelemetry.aotcompatibility')) | ||
{ | ||
$added = $labelsToAdd.Add("infra") | ||
} | ||
|
||
if ($fileChanged.StartsWith('test\benchmarks')) | ||
{ | ||
$added = $labelsToAdd.Add("perf") | ||
} | ||
|
||
if ($fullFileName -eq 'directory.packages.props') | ||
{ | ||
$added = $labelsToAdd.Add("dependencies") | ||
} | ||
} | ||
|
||
foreach ($labelOnPullRequest in $labelsOnPullRequest) | ||
{ | ||
if ($labelsToAdd.Contains($labelOnPullRequest.name)) | ||
{ | ||
$removed = $labelsToAdd.Remove($labelOnPullRequest.name) | ||
} | ||
elseif ($labelOnPullRequest.name.StartsWith($labelPackagePrefix) -or | ||
$managedLabels.Contains($labelOnPullRequest.name)) | ||
{ | ||
$added = $labelsToRemove.Add($labelOnPullRequest.name) | ||
} | ||
} | ||
|
||
if ($labelsToAdd.Count -gt 0) | ||
{ | ||
foreach ($label in $labelsToAdd) | ||
{ | ||
gh pr edit $pullRequestNumber --add-label $label | ||
} | ||
} | ||
|
||
if ($labelsToRemove.Count -gt 0) | ||
{ | ||
foreach ($label in $labelsToRemove) | ||
{ | ||
gh pr edit $pullRequestNumber --remove-label $label | ||
} | ||
} | ||
} | ||
|
||
Export-ModuleMember -Function AddLabelsOnPullRequestsBasedOnFilesChanged |
Oops, something went wrong.