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

PR title is not populated from commit #5865

Open
roblourens opened this issue Mar 22, 2024 · 6 comments · Fixed by #5893
Open

PR title is not populated from commit #5865

roblourens opened this issue Mar 22, 2024 · 6 comments · Fixed by #5893
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug
Milestone

Comments

@roblourens
Copy link
Member

Type: Bug

I used to be able to make a commit, then "Create Pull Request", and the PR title would be whatever my last commit message was. Now this happens sometimes but usually the PR title is empty.

2024-03-22 13:28:40.781 [info] Review+2> Resolved PR #199720, state is 1
2024-03-22 13:28:41.930 [info] Review+2> This PR is merged
2024-03-22 13:55:58.950 [info] Review+2> Validate state in progress
2024-03-22 13:55:58.950 [info] Review+2> Validating state...
2024-03-22 13:55:58.982 [info] PullRequestGitHelper> associate roblou/immense-snake with base branch microsoft/vscode#main
2024-03-22 13:55:59.033 [info] Found GitHub remote for folder /Users/roblou/code/vscode
2024-03-22 13:55:59.042 [info] Using in-memory cached assignable users.
2024-03-22 13:55:59.580 [info] Review+2> No matching pull request metadata found locally for current branch roblou/immense-snake
2024-03-22 13:55:59.632 [info] Review+2> No matching pull request metadata found on GitHub for current branch roblou/immense-snake
2024-03-22 13:56:08.600 [info] Review+2> Validate state in progress
2024-03-22 13:56:08.600 [info] Review+2> Validating state...
2024-03-22 13:56:08.735 [info] Found GitHub remote for folder /Users/roblou/code/vscode
2024-03-22 13:56:08.742 [info] Using in-memory cached assignable users.
2024-03-22 13:56:09.039 [info] Review+2> No matching pull request metadata found locally for current branch roblou/immense-snake
2024-03-22 13:56:09.107 [info] Review+2> No matching pull request metadata found on GitHub for current branch roblou/immense-snake
2024-03-22 13:56:25.891 [info] CreatePullRequestViewProvider> Initializing "create" view: {"defaultBaseRemote":{"owner":"microsoft","repositoryName":"vscode"},"defaultBaseBranch":"main","defaultCompareRemote":{"owner":"microsoft","repositoryName":"vscode"},"defaultCompareBranch":"roblou/immense-snake","defaultTitle":"","defaultDescription":"<!-- Thank you for submitting a Pull Request. Please:\n* Read our Pull Request guidelines:\n  https://github.com/microsoft/vscode/wiki/How-to-Contribute#pull-requests\n* Associate an issue with the Pull Request.\n* Ensure that the code is up-to-date with the `main` branch.\n* Include a description of the proposed changes and how to test them.\n-->\n","defaultMergeMethod":"squash","baseHasMergeQueue":false,"remoteCount":1,"allowAutoMerge":true,"mergeMethodsAvailability":{"merge":true,"squash":true,"rebase":true},"autoMergeDefault":true,"createError":"","labels":[],"isDraftDefault":false,"isDarkTheme":true,"generateTitleAndDescriptionTitle":"Generate with Copilot","creating":false,"initializeWithGeneratedTitleAndDescription":false}

Extension version: 0.85.2024032111
VS Code version: Code - Insiders 1.88.0-insider (409da0445d397e37a790ce5d2499688955c3fcfa, 2024-03-22T05:51:39.055Z)
OS version: Darwin x64 23.4.0
Modes:

System Info
Item Value
CPUs Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: enabled
webgl: enabled
webgl2: enabled
webgpu: enabled
Load (avg) 6, 5, 4
Memory (System) 32.00GB (0.04GB free)
Process Argv --enable-proposed-api roblourens.chat-agent-dalle --enable-proposed-api ms-vscode.dscopilot-agent --crash-reporter-id a2c1d522-e9f0-4e3f-b78a-9253f1c2d2d3
Screen Reader no
VM 0%
A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256197
vscod805cf:30301675
vsaa593:30376534
py29gd2263:30784851
vscaac:30438845
c4g48928:30535728
2i9eh265:30646982
962ge761:30841072
pythongtdpath:30726887
welcomedialog:30812478
pythonidxpt:30768918
pythonnoceb:30776497
asynctok:30898717
dsvsc013:30777762
dsvsc014:30777825
dsvsc015:30821418
pythontestfixt:30866404
pythonregdiag2:30926734
pyreplss1:30879911
pythonmypyd1:30859725
pythoncet0:30859736
h48ei257:30992801
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
d6fi2104:30968146
cppperfnew:30980852
d34g3935:30961436
fegfb526:30952798
bg6jg535:30979844
ccp1r6:30958151
pythonait:30973460
7ghi1836:30988844

@alexr00 alexr00 self-assigned this Mar 26, 2024
@alexr00 alexr00 added the bug Issue identified by VS Code Team member as probable bug label Mar 26, 2024
@alexr00 alexr00 added this to the March 2024 milestone Mar 26, 2024
alexr00 added a commit that referenced this issue Mar 26, 2024
@alexr00
Copy link
Member

alexr00 commented Mar 26, 2024

Has your system been slower recently? Since October there's been a 5 second timeout to get the latest commit message. You might be hitting that. I've added logs so we can see if that's actually what's happening.

@alexr00 alexr00 modified the milestones: March 2024, April 2024 Mar 26, 2024
alexr00 added a commit that referenced this issue Mar 26, 2024
@alexr00
Copy link
Member

alexr00 commented Apr 4, 2024

Closing since I can't repro. If you see it again please share logs (info log level is fine) so we can see if the problem is that the timeout is being hit. I've also added telemetry so I can see how often this happens.

@roblourens
Copy link
Member Author

I don't know what's special about my setup

2024-04-12 12:23:53.743 [info] Found GitHub remote for folder /Users/roblou/code/vscode
2024-04-12 12:23:53.747 [info] Using in-memory cached assignable users.
2024-04-12 12:23:53.997 [info] Review+0> No matching pull request metadata found locally for current branch roblou/mature-caterpillar
2024-04-12 12:23:54.147 [info] Review+0> No matching pull request metadata found on GitHub for current branch roblou/mature-caterpillar
2024-04-12 12:24:06.268 [info] Review+0> Validate state in progress
2024-04-12 12:24:06.268 [info] Review+0> Validating state...
2024-04-12 12:24:06.405 [info] Found GitHub remote for folder /Users/roblou/code/vscode
2024-04-12 12:24:06.409 [info] Using in-memory cached assignable users.
2024-04-12 12:24:06.709 [info] Review+0> No matching pull request metadata found locally for current branch roblou/mature-caterpillar
2024-04-12 12:24:06.803 [info] Review+0> No matching pull request metadata found on GitHub for current branch roblou/mature-caterpillar
2024-04-12 12:24:53.316 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:24:59.873 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:25:06.395 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:25:13.118 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:25:37.572 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:25:52.516 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:26:04.213 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:26:10.165 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:26:16.442 [info] Review+0> Validate state in progress
2024-04-12 12:26:16.443 [info] Review+0> Validating state...
2024-04-12 12:26:16.587 [info] Found GitHub remote for folder /Users/roblou/code/vscode
2024-04-12 12:26:16.591 [info] Using in-memory cached assignable users.
2024-04-12 12:26:16.842 [info] Review+0> No matching pull request metadata found locally for current branch roblou/mature-caterpillar
2024-04-12 12:26:16.921 [info] Review+0> No matching pull request metadata found on GitHub for current branch roblou/mature-caterpillar
2024-04-12 12:27:22.521 [info] CreatePullRequestViewProvider> Timeout getting last commit message
2024-04-12 12:27:22.522 [info] CreatePullRequestViewProvider> Initializing "create" view: {"defaultBaseRemote":{"owner":"microsoft","repositoryName":"vscode"},"defaultBaseBranch":"main","defaultCompareRemote":{"owner":"microsoft","repositoryName":"vscode"},"defaultCompareBranch":"roblou/mature-caterpillar","defaultTitle":"","defaultDescription":"<!-- Thank you for submitting a Pull Request. Please:\n* Read our Pull Request guidelines:\n  https://github.com/microsoft/vscode/wiki/How-to-Contribute#pull-requests\n* Associate an issue with the Pull Request.\n* Ensure that the code is up-to-date with the `main` branch.\n* Include a description of the proposed changes and how to test them.\n-->\n","defaultMergeMethod":"merge","baseHasMergeQueue":false,"remoteCount":1,"allowAutoMerge":true,"mergeMethodsAvailability":{"merge":true,"squash":true,"rebase":true},"autoMergeDefault":true,"createError":"","labels":[],"isDraftDefault":false,"isDarkTheme":true,"generateTitleAndDescriptionTitle":"Generate with Copilot","creating":false,"initializeWithGeneratedTitleAndDescription":false}

@roblourens roblourens reopened this Apr 12, 2024
@roblourens
Copy link
Member Author

#5923 would explain why it's slow for me though

@alexr00
Copy link
Member

alexr00 commented Apr 15, 2024

2024-04-12 12:27:22.521 [info] CreatePullRequestViewProvider> Timeout getting last commit message

You're definitely hitting the timeout. #5923 is from trying to find a PR template, which I can fix by using GitHub API instead of searching the filesystem.

@alexr00
Copy link
Member

alexr00 commented Apr 19, 2024

I checked the data, and there isn't any spike in the number of users hitting the timeout. It seems to stay between 0 and 20 users per day. Let me know if you see this again with #5923 fixed!

@alexr00 alexr00 modified the milestones: April 2024, On Deck Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants