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

Ignore Authenticate Reunion NuGet sources #11145

Merged
merged 3 commits into from
Nov 7, 2022
Merged

Ignore Authenticate Reunion NuGet sources #11145

merged 3 commits into from
Nov 7, 2022

Conversation

rmarinho
Copy link
Member

@rmarinho rmarinho commented Nov 7, 2022

Description of Change

  • Seems the reunion feed is not working, i don't think we need it now
  • Also fix a issue where agent cleaner was cleaning the installed dotnet version

@rmarinho rmarinho requested a review from mattleibow November 7, 2022 11:45
@jfversluis jfversluis added area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions t/housekeeping ♻︎ labels Nov 7, 2022
@mattleibow
Copy link
Member

Seems the reunion feed is not working, i don't think we need it now

Maybe the token expired? I had to create a PAT for the service connection

Also, we use it for the reunion maestro: https://github.com/dotnet/maui/pulls/app%2Freunion-maestro

@rmarinho rmarinho merged commit 340dc2a into main Nov 7, 2022
@rmarinho rmarinho deleted the fix-reunion-feeds branch November 7, 2022 14:23
@rmarinho
Copy link
Member Author

rmarinho commented Nov 7, 2022

/backport to net7.0

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2022

Started backporting to net7.0: https://github.com/dotnet/maui/actions/runs/3411158906

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2022

@rmarinho backporting to net7.0 failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patch

Applying: Ignore Authenticate Reunion NuGet sources
Applying: Try install .net after provisioning run
error: sha1 information is lacking or useless (eng/pipelines/common/provision.yml).
error: could not build fake ancestor
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 Try install .net after provisioning run
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2022

@rmarinho an error occurred while backporting to net7.0, please check the run log for details!

Error: git am failed, most likely due to a merge conflict.

rmarinho added a commit that referenced this pull request Nov 7, 2022
* Ignore Authenticate Reunion NuGet sources

* Try install .net after provisioning run

* Fix identation
# Conflicts:
#	eng/pipelines/common/provision.yml
jfversluis pushed a commit that referenced this pull request Nov 8, 2022
* Ignore Authenticate Reunion NuGet sources (#11145)

* Ignore Authenticate Reunion NuGet sources

* Try install .net after provisioning run

* Fix identation
# Conflicts:
#	eng/pipelines/common/provision.yml

* Fix bad merge on provisionator version
@github-actions github-actions bot locked and limited conversation to collaborators Dec 16, 2023
@samhouts samhouts added the fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! t/housekeeping ♻︎
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants