Remove --verbose from devicelab task executions. (#162644)

These can be useful, but were probably left in past the point where they
are always useful:

- https://github.com/flutter/flutter/pull/58018
- https://github.com/flutter/flutter/pull/56342
- https://github.com/flutter/flutter/pull/74080

... compared to the cost of reading these logs with 1000s of lines of
`stdout: ` output.

Will ask the CI gods if any of this was load-bearing before sending out
for review.
This commit is contained in:
Matan Lurey 2025-02-06 16:25:20 -08:00 committed by GitHub
parent db7e82bdfb
commit fdeb7b93aa
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
14 changed files with 21 additions and 77 deletions

View File

@ -22,7 +22,6 @@ Future<void> main() async {
'--target-platform=android-arm',
'--obfuscate',
'--split-debug-info=foo/',
'--verbose',
],
);
});
@ -67,7 +66,6 @@ Future<void> main() async {
'--split-debug-info=foo/',
'--no-debug',
'--no-profile',
'--verbose',
],
);
});

View File

@ -93,7 +93,7 @@ Future<void> main() async {
section('Build release AAR');
await inDirectory(projectDir, () async {
await flutter('build', options: <String>['aar', '--verbose']);
await flutter('build', options: <String>['aar']);
});
final String repoPath = path.join(projectDir.path, 'build', 'host', 'outputs', 'repo');

View File

@ -85,7 +85,6 @@ Future<void> _testBuildIosFramework(Directory projectDir, {bool isModule = false
'build',
options: <String>[
'ios-framework',
'--verbose',
'--output=$outputDirectoryName',
'--obfuscate',
'--split-debug-info=symbols',
@ -470,7 +469,6 @@ Future<void> _testBuildMacOSFramework(Directory projectDir) async {
'build',
options: <String>[
'macos-framework',
'--verbose',
'--output=$outputDirectoryName',
'--obfuscate',
'--split-debug-info=symbols',

View File

@ -44,7 +44,7 @@ class NewGalleryChromeRunTest {
await flutter('build', options: <String>['web', '-v', '--release', '--no-pub']);
final List<String> options = <String>['-d', 'chrome', '--verbose', '--resident'];
final List<String> options = <String>['-d', 'chrome', '--resident'];
final Process process = await startFlutter('run', options: options);
final Completer<void> stdoutDone = Completer<void>();

View File

@ -81,10 +81,7 @@ Future<void> main() async {
section('Build app bundle using the flutter tool - flavor: flavor_underscore');
int exitCode = await inDirectory(project.rootPath, () {
return flutter(
'build',
options: <String>['appbundle', '--flavor=flavor_underscore', '--verbose'],
);
return flutter('build', options: <String>['appbundle', '--flavor=flavor_underscore']);
});
if (exitCode != 0) {
@ -114,10 +111,7 @@ Future<void> main() async {
section('Build app bundle using the flutter tool - flavor: production');
exitCode = await inDirectory(project.rootPath, () {
return flutter(
'build',
options: <String>['appbundle', '--flavor=production', '--verbose'],
);
return flutter('build', options: <String>['appbundle', '--flavor=production']);
});
if (exitCode != 0) {

View File

@ -16,7 +16,7 @@ Future<void> main() async {
await runPluginProjectTest((FlutterPluginProject pluginProject) async {
section('APK content for task assembleDebug without explicit target platform');
await inDirectory(pluginProject.exampleAndroidPath, () {
return flutter('build', options: <String>['apk', '--debug', '--verbose']);
return flutter('build', options: <String>['apk', '--debug']);
});
Iterable<String> apkFiles = await getFilesInApk(pluginProject.debugApkPath);
@ -42,7 +42,7 @@ Future<void> main() async {
section('APK content for task assembleRelease without explicit target platform');
await inDirectory(pluginProject.exampleAndroidPath, () {
return flutter('build', options: <String>['apk', '--release', '--verbose']);
return flutter('build', options: <String>['apk', '--release']);
});
apkFiles = await getFilesInApk(pluginProject.releaseApkPath);
@ -67,12 +67,7 @@ Future<void> main() async {
await inDirectory(pluginProject.exampleAndroidPath, () {
return flutter(
'build',
options: <String>[
'apk',
'--release',
'--verbose',
'--target-platform=android-arm,android-arm64',
],
options: <String>['apk', '--release', '--target-platform=android-arm,android-arm64'],
);
});
@ -100,7 +95,6 @@ Future<void> main() async {
options: <String>[
'apk',
'--release',
'--verbose',
'--split-per-abi',
'--target-platform=android-arm,android-arm64',
],
@ -136,7 +130,7 @@ Future<void> main() async {
section('gradlew assembleRelease');
await inDirectory(project.rootPath, () {
return flutter('build', options: <String>['apk', '--release', '--verbose']);
return flutter('build', options: <String>['apk', '--release']);
});
// When the platform-target isn't specified, we generate the snapshots

View File

@ -21,7 +21,7 @@ class HelloWorldMemoryTest extends MemoryTest {
print('launching $project$test on device...');
await flutter(
'run',
options: <String>['--verbose', '--release', '--no-resident', '-d', device!.deviceId, test],
options: <String>['--release', '--no-resident', '-d', device!.deviceId, test],
);
await Future<void>.delayed(const Duration(milliseconds: 1500));
await recordStart();

View File

@ -35,7 +35,7 @@ Future<void> main() async {
await inDirectory(projectDir, () async {
final String buildOutput = await evalFlutter(
'build',
options: <String>['ios', '--no-codesign', '--release', '--verbose'],
options: <String>['ios', '--no-codesign', '--release'],
);
if (!buildOutput.contains('-destination generic/platform=watchOS')) {
print(buildOutput);

View File

@ -85,7 +85,7 @@ dependencies:
section('Build ephemeral host app in release mode without CocoaPods');
await inDirectory(projectDir, () async {
await flutter('build', options: <String>['ios', '--no-codesign', '--verbose']);
await flutter('build', options: <String>['ios', '--no-codesign']);
});
// Check the tool is no longer copying to the legacy xcframework location.
@ -606,7 +606,7 @@ end
);
if (!xcodebuildOutput.contains(
'flutter --verbose --local-engine-src-path=bogus assemble',
RegExp('flutter.*--local-engine-src-path=bogus assemble'),
) || // Verbose output
!xcodebuildOutput.contains(
'Unable to detect a Flutter engine build directory in bogus',

View File

@ -280,7 +280,7 @@ public class DummyPluginAClass {
section('Build plugin A example iOS app');
await inDirectory(exampleApp, () async {
await evalFlutter('build', options: <String>['ios', '--no-codesign', '--verbose']);
await evalFlutter('build', options: <String>['ios', '--no-codesign']);
});
final Directory appBundle = Directory(

View File

@ -37,13 +37,7 @@ Future<void> main() async {
'integration_test.podspec',
);
await exec('pod', <String>[
'lib',
'lint',
iosintegrationTestPodspec,
'--use-libraries',
'--verbose',
]);
await exec('pod', <String>['lib', 'lint', iosintegrationTestPodspec, '--use-libraries']);
final String macosintegrationTestPodspec = path.join(
integrationTestPackage,
@ -51,7 +45,7 @@ Future<void> main() async {
'macos',
'integration_test_macos.podspec',
);
await _tryMacOSLint(macosintegrationTestPodspec, <String>['--verbose']);
await _tryMacOSLint(macosintegrationTestPodspec, <String>[]);
});
section('Create Objective-C plugin');
@ -76,13 +70,7 @@ Future<void> main() async {
final String objcPluginPath = path.join(tempDir.path, objcPluginName);
final String objcPodspecPath = path.join(objcPluginPath, 'ios', '$objcPluginName.podspec');
await inDirectory(tempDir, () async {
await exec('pod', <String>[
'lib',
'lint',
objcPodspecPath,
'--allow-warnings',
'--verbose',
]);
await exec('pod', <String>['lib', 'lint', objcPodspecPath, '--allow-warnings']);
});
section('Lint Objective-C iOS podspec plugin as library');
@ -94,7 +82,6 @@ Future<void> main() async {
objcPodspecPath,
'--allow-warnings',
'--use-libraries',
'--verbose',
]);
});
@ -120,13 +107,7 @@ Future<void> main() async {
final String swiftPluginPath = path.join(tempDir.path, swiftPluginName);
final String swiftPodspecPath = path.join(swiftPluginPath, 'ios', '$swiftPluginName.podspec');
await inDirectory(tempDir, () async {
await exec('pod', <String>[
'lib',
'lint',
swiftPodspecPath,
'--allow-warnings',
'--verbose',
]);
await exec('pod', <String>['lib', 'lint', swiftPodspecPath, '--allow-warnings']);
});
section('Lint Swift iOS podspec plugin as library');
@ -138,7 +119,6 @@ Future<void> main() async {
swiftPodspecPath,
'--allow-warnings',
'--use-libraries',
'--verbose',
]);
});
@ -150,17 +130,13 @@ Future<void> main() async {
'$swiftPluginName.podspec',
);
await inDirectory(tempDir, () async {
await _tryMacOSLint(macOSPodspecPath, <String>['--allow-warnings', '--verbose']);
await _tryMacOSLint(macOSPodspecPath, <String>['--allow-warnings']);
});
section('Lint Swift macOS podspec plugin as library');
await inDirectory(tempDir, () async {
await _tryMacOSLint(macOSPodspecPath, <String>[
'--allow-warnings',
'--use-libraries',
'--verbose',
]);
await _tryMacOSLint(macOSPodspecPath, <String>['--allow-warnings', '--use-libraries']);
});
section('Create Objective-C application');

View File

@ -20,14 +20,7 @@ void main() {
await inDirectory(appDir, () async {
return flutter(
'drive',
options: <String>[
'--verbose',
'-d',
device.deviceId,
'--route',
'/smuggle-it',
'lib/route.dart',
],
options: <String>['-d', device.deviceId, '--route', '/smuggle-it', 'lib/route.dart'],
);
});
return TaskResult.success(null);

View File

@ -23,14 +23,7 @@ void main() {
await inDirectory(appDir, () async {
return flutter(
'drive',
options: <String>[
'--verbose',
'-d',
device.deviceId,
'--route',
'/smuggle-it',
'lib/route.dart',
],
options: <String>['-d', device.deviceId, '--route', '/smuggle-it', 'lib/route.dart'],
);
});
section('TEST WHETHER `flutter run --route` WORKS');
@ -42,7 +35,6 @@ void main() {
'run',
// --fast-start does not support routes.
options: <String>[
'--verbose',
'--disable-service-auth-codes',
'--no-fast-start',
'--no-publish-port',

View File

@ -28,7 +28,6 @@ void main() {
final Process run = await startFlutter(
'run',
options: <String>[
'--verbose',
'--no-fast-start',
'--no-publish-port',
'--disable-service-auth-codes',