Skip to content

Update .NET SDK to 9.0.100-alpha.1.24070.3 #53525

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

Closed

Conversation

github-actions[bot]
Copy link
Contributor

Updates the .NET SDK to version 9.0.100-alpha.1.24070.3, which also updates the .NET runtime from version 9.0.0-alpha.1.24057.2 to version 9.0.0-alpha.1.24066.33.

This pull request was auto-generated by GitHub Actions.

Update .NET SDK to version 9.0.100-alpha.1.24070.3.

---
updated-dependencies:
- dependency-name: Microsoft.NET.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot requested review from wtgodbe and a team as code owners January 22, 2024 12:07
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Jan 22, 2024
@BrennanConroy
Copy link
Member

@wtgodbe Is there a way to configure this action to update a previous PR? Or wait for the previous PR to be merged before opening a new one?

@martincostello
Copy link
Member

It should already do that itself via the default close-superseded option of true.

@martincostello
Copy link
Member

Actually, that's not what you asked for - it would do the opposite. It would generate a new one and close the previous.

@martincostello
Copy link
Member

martincostello commented Jan 22, 2024

If you configured it to use a fixed well-known branch name, then you'd get the behaviour you wanted, where it would only generate a new PR if the branch didn't currently exist. That relies on branches being cleaned-up post-merge.

Doing the opposite of the current behaviour (which I'm not sure without debugging, why this one didn't close #53449) or updating an existing PR in place would need adding as a feature.

@martincostello
Copy link
Member

I'm not sure without debugging, why this one didn't close #53449

I had a quick look into this, and couldn't see an obvious reason why this didn't happen (this is the relevant code). Maybe some sort of timing/indexing issue with the GitHub API results at the time it ran?

@sebastienros
Copy link
Member

I am going to merge the previous one for now since it passed. Then I'll unblock this one.

@wtgodbe
Copy link
Member

wtgodbe commented Jan 24, 2024

I tried triggering the workflow again, let's see if it closes this

@wtgodbe
Copy link
Member

wtgodbe commented Jan 24, 2024

That created #53600 & didn't close this. Maybe it's important whether the worflow was manually triggered or triggered by automation? It's only run on the automated schedule 1 time, for this PR.

@martincostello
Copy link
Member

I'll have to have another look at why to might be failing to find the old PRs when it opens a new one.

I wonder if this size of this repo (number of issues/PRs etc) compared to the ones where I've seen the auto-close functionality work fine is causing some sort of timing issue that means some sort of eventual consistency in the GitHub API is noticeable enough for it to not work correctly for what the code does.

@martincostello
Copy link
Member

@wtgodbe Could you try manually running it again, but with debugging enabled? It might give me some output that gives a clue as to what's happening.

@wtgodbe
Copy link
Member

wtgodbe commented Jan 24, 2024

@martincostello I set the ACTIONS_RUNNER_DEBUG repo variable to true and kicked off another run: https://git.1-hub.cndotnet/aspnetcore/actions/runs/7646679279. Hopefully that produces the extra logs

@martincostello
Copy link
Member

martincostello commented Jan 24, 2024

I don't see any of the purple debug output (like this).

I think you need to re-run it with the checkbox ticked (after closing the PR it opened and deleting the branch):

image

@wtgodbe
Copy link
Member

wtgodbe commented Jan 24, 2024

I think you need to re-run it with the checkbox ticked (after closing the PR it opened and deleting the branch):

Done. Weird, I don't see that checkbox when just queuing a new job, only on retry.

@martincostello
Copy link
Member

Thanks 👍 - I'll have a dig though and see if I can work out what's going on.

@wtgodbe
Copy link
Member

wtgodbe commented Jan 24, 2024

Thanks 👍 - I'll have a dig though and see if I can work out what's going on.

Thanks, I really appreciate it!

@martincostello
Copy link
Member

#53603 should sort it - the logic to use it was incorrectly behind an if condition for another option you aren't using.

@sebastienros
Copy link
Member

Close it since this one should get in?

@wtgodbe wtgodbe closed this Jan 24, 2024
@wtgodbe wtgodbe deleted the update-dotnet-sdk-9.0.100-alpha.1.24070.3 branch January 24, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants