Commit graph

74,924 commits

Message SHA1
RemoteAuth Boilerplate
Stub out code and test directories

BUG: 290094832
Test: atest FrameworksNetTests RemoteAuthUnitTests
Change-Id: I850242640f32bea3d46febcde5f3719619b1ea07

Casey Borders   authored and   Salil Rajadhyaksha   commited 2023-07-31 21:41:26 +00:00
13a35ea213
Merge "Revert onNetworkActive callback behavior on U+" into main
Motomu Utsumi   authored and   Gerrit Code Review   commited 2023-07-31 10:34:41 +00:00
3f35bb2fce
Revert onNetworkActive callback behavior on U+
aosp/2605757 changed onNetworkActive callback behavior on U+ but this
change was not merged to U release branch.
This CL reverts the behavior change and keep the current behavior on U+.

Bug: 279380356
Bug: 291870075
Test: atest FrameworksNetTests
Change-Id: I91ccc7ee93f0827986401f316997b4a2df11318f

Motomu Utsumi   commited 2023-07-31 19:33:24 +09:00
1e51a64988
Merge "Deflake doTestPlatformVpnWithException related tests" into main
Chiachang Wang   authored and   Gerrit Code Review   commited 2023-07-31 08:54:14 +00:00
52f3e8f29e
Deflake doTestPlatformVpnWithException related tests
This commit updates test to
- Execute IkeSessionCallback inside executor. The callback should
  be called from executor in production code but test trigger the
  callback from test main thread which will result in unexpected
  code interaction.
- Verify the timer directly. The origin design verifies
  interactions with Captor with uncertain interaction. Test
  should be able to verify the schedule() calls with the expected
  timeout value. Also, in lower performance device, device may
  still process previous interaction. Add longer timeout to reduce
  flakes.

Bug: 293233390
Test: atest FrameworksNetTests
Test: atest VpnTest#testStartPlatformVpnFailedWithRecoverableError
      --rerun-until-failure 100
Change-Id: Ie42a2597a382800d6484a086b6d808490dfbdd32

chiachangwang   commited 2023-07-31 04:59:58 +00:00
9f9a4dc69c
Merge changes Ia5fc3e53,Ie1ab6013,I1cd5acb3 into main
* changes:
  Unregister BroadcastReceiver and OnSubscriptionsChangedListener.
  Disable KeepaliveStatsTracker before T.
  Disable KeepaliveStatsTracker instead of throwing.

Hansen Kurli   authored and   Gerrit Code Review   commited 2023-07-28 11:20:21 +00:00
ebdda22240
Merge "Skip presubmit for flaky NetworkPolicy related tests due to Out of SLO" into main
Junyu Lai   authored and   Gerrit Code Review   commited 2023-07-28 05:26:51 +00:00
24420ad536
Merge "Fix EtherentTetheringTest flaky" into main
Mark Chien   authored and   Gerrit Code Review   commited 2023-07-28 04:29:42 +00:00
013425e27e
Merge "Address comment from ag/12107302" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-07-28 03:48:05 +00:00
88c497c2e5
Merge "Skip presubmit for flaky tests" into main
Remi NGUYEN VAN   authored and   Gerrit Code Review   commited 2023-07-28 03:38:33 +00:00
dff0f956e3
Merge changes from topic "hostsidevpntestflaky" into main
* changes:
  Skip presubmit for flaky tests in FrameworksNetTests due to Out of SLO
  Skip presubmit for flaky tests in HostsideVpnTests due to Out of SLO

Junyu Lai   authored and   Gerrit Code Review   commited 2023-07-28 02:06:17 +00:00
a6be6cd48a
Merge "Increase timeout to deflake tests" into main
Chiachang Wang   authored and   Gerrit Code Review   commited 2023-07-28 00:37:42 +00:00
32a8cbce6f
Fix EtherentTetheringTest flaky
The first tethering test case may experience tethering restart
with IP conflict handling. Tethering would cache the last
upstreams so that the next enabled tethering avoids
picking up the address that is in conflict with the upstreams.
To protect subsequent tests, turn tethering on and off before
running them.

Bug: 254183718
Test: atest CtsTetheringTest
      atest MtsTetheringTestLatestSdk
Change-Id: I03729d3212a03bd9fcfe2c09cf327840f13b250c

Mark   authored and   Mark Chien   commited 2023-07-27 16:04:28 +00:00
f5e4b90ae2
Increase timeout to deflake tests
In failed log, VPN still process the migration which does not
send the network capabilities update to network agent.
In low performance devices like cuttlefish, it may sometimes
cause flaky. Specify a longer timeout to deflake the test.

Bug: 293233390
Test: atest FrameworksNetTests
Change-Id: I4e6c80c03e19efb448ee2806c5a250425b0e068f

chiachangwang   authored and   Chiachang Wang   commited 2023-07-27 09:17:00 +00:00
595fac0840
Merge "Fix ConnectivityManagerTest#testFactoryReset flaky" into main
Mark Chien   authored and   Gerrit Code Review   commited 2023-07-27 09:11:20 +00:00
734738767c
Merge "Revert "Diable ConnectivityManagerTest#testFactoryReset"" into main
Mark Chien   authored and   Gerrit Code Review   commited 2023-07-27 09:11:11 +00:00
76edec4f97
Address comment from ag/12107302
Test: trivial doc-only change
Bug: 140807677
Change-Id: I57440e8b54709a2f0e8c875d30f2c295fa3e1f3d

Chalard Jean   commited 2023-07-27 17:29:18 +09:00
f8d1d6c4f9
Fix ConnectivityManagerTest#testFactoryReset flaky
Listen SoftAp callback to make sure soft ap is disabled
before start hotspot.

Bug: 277856014
Test: atest ConnectivityManagerTest#testFactoryReset

Change-Id: I95050be703b7c97da7a14541ac74718fdcbd7f13

Mark   commited 2023-07-27 03:40:39 +00:00
5fab2d6c17
Skip presubmit for flaky tests
Skip presubmit for tests that have been reported to be >0.5% flaky in
presubmit, or in mainline-presubmit. Tests that only meet that criteria
in mainline-presubmit are only marked as @SkipMainlinePresubmit.

Ensure that the tests are still run in postsubmit so they can be moved
back to presubmit after getting fixed.

Test: TH
Bug: 284534251
Bug: 292174275
Bug: 292173706
Bug: 292174832
Bug: 289879947
Bug: 286701510
Bug: 293239598
Bug: 279846955
Bug: 277567450
Bug: 293239828
Bug: 293240365
Change-Id: I986a1079709057d3210016c50b632fe0f3bff1e2

Remi NGUYEN VAN   commited 2023-07-27 02:36:38 +00:00
8db52a5987
Revert "Diable ConnectivityManagerTest#testFactoryReset"
This reverts commit f866acb6e7.

Reason for revert: enable the test

Change-Id: I00c2b32f0bf6fdb825d0edba8dcb74bcc7ad2fbf

Mark Chien   authored and   Mark   commited 2023-07-27 02:33:32 +00:00
dfdc21bc6b
Merge "[Cronet] skip presubmit for flaky tests" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-07-26 10:29:40 +00:00
b868a462fd
Skip presubmit for flaky NetworkPolicy related tests due to Out of SLO
Test: TH
Bug: 279052235
Bug: 279829773
Bug: 293225660
Change-Id: I68fe484e920c0c655176fc1909646c721c3eeefe

Junyu Lai   commited 2023-07-26 16:43:42 +08:00
7274332ff3
Skip presubmit for flaky tests in FrameworksNetTests due to Out of SLO
Test: TH
Bug: 293233390
Change-Id: Ic1d0d86b3f8dfc8e8122c6b05973fab8b57e52b4

Junyu Lai   commited 2023-07-26 16:31:46 +08:00
b8fc755259
Skip presubmit for flaky tests in HostsideVpnTests due to Out of SLO
Test: TH
Bug: 291701394
Change-Id: Iea67a9402f2712746c8e7424aacf66f82099700c

Junyu Lai   commited 2023-07-26 16:20:09 +08:00
7eb00f79c3
Merge "Add test for benchmarking NetworkStatsRecorder" into main
Junyu Lai   authored and   Gerrit Code Review   commited 2023-07-26 07:43:14 +00:00
5be6224203
Merge "Exclude some DnsResolverTest tests from some CtsNetTestCasesLatestSdk" into main
Mike Yu   authored and   Gerrit Code Review   commited 2023-07-26 03:17:08 +00:00
2657ee261f
Merge "Create directory for RemoteAuth module" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-07-26 02:17:59 +00:00
25e205c4c7
Merge "Report registration metrics data" into main
Paul Hu   authored and   Gerrit Code Review   commited 2023-07-26 00:45:34 +00:00
6be7db441c
Create directory for RemoteAuth module
Inital commit for RemoteAuth adding the directory and the
OWNERS.

BUG: 290094832
Test: atest FrameworksNetTests
Change-Id: I8f5f109b7b01238388152c5a1caee72a753fe7fd

Casey Borders   commited 2023-07-25 21:29:02 +00:00
7eea36a648
[Cronet] skip presubmit for flaky tests
Bug: b/293141085
Bug: b/292298108
Test: n/a
Change-Id: Ie7190c0222bd75f3f2df0c5b757b1ab9d0114d77

Chidera Olibie   commited 2023-07-25 20:56:29 +00:00
cd038a07fb
Merge "clatd.c - add support for UDPLITE" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-07-25 18:39:31 +00:00
7c47221756
Merge "[Cronet] Cancel stream before shutdown" into main
Chidera Olibie   authored and   Gerrit Code Review   commited 2023-07-25 18:31:11 +00:00
89e4a09d62
clatd.c - add support for UDPLITE
This is trivial - as the UDPLITE pseudoheader is identical
to the UDP pseudoheader (except that the UDPLITE pseudo length
is derived from the IPv4 total length / IPv6 payload length
field, instead of being copied from the UDPLITE header 'coverage
length' field - but this doesn't matter, as it [ie. the udplite
payload length] doesn't change during 464xlat translation).

Additionally UDPLITE never sends a checksum value of 0,
as at least 8 bytes (the UDPLITE header) *must* be included
in the checksum field, and a 0 must be sent as 0xFFFF.

See: https://datatracker.ietf.org/doc/html/rfc3828

Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I00a110b793fcf3cf705a9a706811da7866c3e810

Maciej Ĺ»enczykowski   commited 2023-07-25 16:17:48 +00:00
43d31bbaf6
[Cronet] Cancel stream before shutdown
This fixes the flakiness. Because we call to a real server,
the server may not be reachable and our assumption would fail.
When this happens, the stream hangs and engine shutdown fails with
active request error. Hence cancel the stream before calling
shutdown.

Also @Ignored this test despite the fix as a safety measure to meet
the SLO. Will re-enable afterwards.

Bug: b/292298108
Test: atest NetHttpCoverageTests --test-filter
BidirectionalStreamTest#testBidirectionalStream_GetStream_CompletesSuccessfully
Change-Id: Iaae61fc62a148b58af86fb29c248399041785f9b

Change-Id: Iea9ef0b8468ec6fa05c306ea394485cec706b9ef

Chidera Olibie   commited 2023-07-25 16:05:10 +00:00
54db2af362
Merge "Add polling and logging when launcher is settings" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-07-25 08:12:30 +00:00
9a9852d782
Add test for benchmarking NetworkStatsRecorder
While previous tests benchmarking performance of reading
NetworkStatsCollection, this measures performance impact
caused by implementation of FileRotator.

Test: atest ConnectivityBenchmarkTests
Bug: 269409485
Change-Id: Ifcfc034128d6d78d759f80a43785b9f2b1ebb9c9

Junyu Lai   commited 2023-07-25 13:37:30 +08:00
4881aa7dc6
Merge "Move QueryTaskConfig to a standalone class" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-07-25 04:51:55 +00:00
4525688b2d
Merge "Ensure the ServiceTypeClient Executor threads can be shutdown" into main
Yuyang Huang   authored and   Gerrit Code Review   commited 2023-07-25 03:05:08 +00:00
54125ef545
Merge "Add test for benchmarking NetworkStatsCollection" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-07-24 11:24:52 +00:00
e8397922b4
Move QueryTaskConfig to a standalone class
Bug: 292470176
Test: atest FrameworksNetTests CtsNetTestCases
Change-Id: Iff15e7b59277444bbaa92b98318bb4f910eea070

Yuyang Huang   commited 2023-07-24 18:34:09 +09:00
bbd573b92f
Add test for benchmarking NetworkStatsCollection
Add a test for benchmarking NetworkStatsCollection.
The test use data that generated from real user dataset which
has many uid records and agreed to share the dataset for testing
purpose.

Test: atest ConnectivityBenchmarkTests
Bug: 269409485

Change-Id: I8981bfc9f65636897c6fc735cda5353b448a0db2

Junyu Lai   commited 2023-07-24 15:35:23 +08:00
ba7a6888cf
Ensure the ServiceTypeClient Executor threads can be shutdown
When ServiceTypeClient is not used, the Executor threads should be
shutdown as well.

Bug: 287013457
Test: atest CtsNetTestCases FrameworksNetTests
Change-Id: I9dae01e7b6c2bcd395a98f4362f0c36f1535b2a6

Yuyang Huang   commited 2023-07-24 16:14:19 +09:00
66cd57f2d8
Report registration metrics data
Collect information about registration successes, failures, and
unregistrations, then build metrics data from this information
and report it.

Bug: 287546772
Test: atest FrameworksNetTestCases NsdManagerTest
Change-Id: I6324279b479da2e61b7519d96df5ad24a432e54a

Paul Hu   commited 2023-07-24 14:50:48 +08:00
777ed05c73
Merge "Address leftover comments" into main
Paul Hu   authored and   Gerrit Code Review   commited 2023-07-21 02:59:28 +00:00
bbaacf4d38
ethernet: add null check before reporting server interface state
If the tethering interface disappears while in server mode (i.e.
tethering is enabled), registering a new InterfaceStateListener will
trigger an NPE.

Test: TH
Bug: 290971780
Change-Id: I1d10575e80e68c77ec4771ac37e5fbc07e03c708

Patrick Rohr   commited 2023-07-20 15:57:52 +00:00
df50b6caac
Merge "Add NetworkNsdReportedMetrics" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-07-20 10:26:45 +00:00
fe4fcecbc1
Add NetworkNsdReportedMetrics
Add helper class NetworkNsdReportedMetrics to build and report
nsd metrics data.

Bug: 287546772
Test: m
Change-Id: I8271811c52a88329fd54e8e0f6cc455d3a3d22f7

Paul Hu   commited 2023-07-20 16:35:50 +08:00
cdef353532
Address leftover comments
- Rename QuerySentResult to QuerySentArguments
- Add constants INVALID_TRANSACTION_ID
- Remove useSessionIdToScheduleMdnsTask check
- Update subtype in MdnsServiceTypeClientTests

Bug: 265787401
Test: atest FrameworksNetTests
Change-Id: I5e544d505c645262c213d40cdd94a79088ac4886

Paul Hu   commited 2023-07-20 05:34:57 +00:00
2048dd2ad3
Merge "Create v4 mapped v6 correctly" into main
Chiachang Wang   authored and   Gerrit Code Review   commited 2023-07-20 05:06:18 +00:00
49ef2e917c
Create v4 mapped v6 correctly
Tests verifies the behavior with v4 mapped v6. The
taken parameter should be a v6 address. However,
InetAddresses.parseNumericAddress() will translate a v4
mapped v6 address into v4. This does not actually test
the behavior. The constant should be created via
Inet6Address.getByAddress() so that it will be an
expected Inet6Address.

Bug: 291869581
Test: atest NattKeepalivePacketDataTest
      w/ and w/o aosp/2626269
Change-Id: I2f086f0fe7d1dd038f5d850f2281deb807c837cf

chiachangwang   commited 2023-07-20 03:14:08 +00:00
65240048ce