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

[release/7.0] [Android] Fix path to ping on Android API 21-27 #80120

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jan 3, 2023

Backport of #79076 to release/7.0

/cc @simonrozsival

Customer Impact

This issue affects only a subset of customers on older Android versions. The System.Net.NetworkInformation.Ping class didn't work in apps running on these systems.

Testing

Tested locally on Android emulators with APIs 21-27.

Risk

Low.

@ghost
Copy link

ghost commented Jan 3, 2023

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #79076 to release/7.0

/cc @simonrozsival

Customer Impact

Testing

Risk

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

Author: github-actions[bot]
Assignees: -
Labels:

area-System.Net

Milestone: -

@ghost
Copy link

ghost commented Jan 3, 2023

Tagging subscribers to 'arch-android': @steveisok, @akoeplinger
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #79076 to release/7.0

/cc @simonrozsival

Customer Impact

Testing

Risk

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

Author: github-actions[bot]
Assignees: -
Labels:

area-System.Net, os-android

Milestone: -

@carlossanlop
Copy link
Member

carlossanlop commented Jan 4, 2023

@simonrozsival when this is ready, can you please add the servicing-consider label and send an email to Tactics requesting approval? And please get a code review sign-off.

@simonrozsival
Copy link
Member

@carlossanlop thanks for the reminder. I'll do that as soon as possible.

@steveisok steveisok added the Servicing-consider Issue for next servicing release review label Jan 11, 2023
@carlossanlop
Copy link
Member

@simonrozsival Friday 13th is the last day for merging backports for the February release. Please make sure to send the email to Tactics requesting approval. The sooner we can merge, the more time we have to verify there are no breaks in the base branch.

@simonrozsival simonrozsival added this to the 7.0.x milestone Jan 12, 2023
@simonrozsival
Copy link
Member

@carlossanlop I've sent the email today.

The CI failure is unrelated to this PR (#78778).

@leecow leecow added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Jan 12, 2023
@leecow leecow modified the milestones: 7.0.x, 7.0.3 Jan 12, 2023
@carlossanlop
Copy link
Member

Approved by Tactics (7.0.3).
Signed off by area owner.
No OOB changes needed (Android code).
CI failure unrelated: #78778
Ready to merge. :shipit:

@carlossanlop carlossanlop merged commit 1947ae4 into release/7.0 Jan 12, 2023
@carlossanlop carlossanlop deleted the backport/pr-79076-to-release/7.0 branch January 12, 2023 21:31
@ghost ghost locked as resolved and limited conversation to collaborators Feb 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants