Disable or configure verbose rsync

[jonahbeckford]

Thank you for your project!

I have a feature request. I would like to have rsync verbose mode (-v) turned off by default or by configuration.

The verbose option is added here:

action/src/action/action.ts

Lines 221 to 240 in cad4964

	private async syncFiles(
	ipAddress: string,
	...excludePaths: string[]
	): Promise<void> {
	core.debug(`Syncing files to VM, excluding: ${excludePaths}`)
	// prettier-ignore
	await exec.exec('rsync', [
	'-auvzrtopg',
	'--exclude', '_actions/cross-platform-actions/action',
	...flatMap(excludePaths, p => ['--exclude', p]),
	`${this.workDirectory}/`,
	`runner@${ipAddress}:work`
	])
	}
	private async syncBack(ipAddress: string): Promise<void> {
	core.info('Syncing back files')
	// prettier-ignore
	await exec.exec('rsync', [
	'-uvzrtopg',

With my project I have 16,000 verbose lines added at the beginning of the step. Here is an example (look at the "Test in VirtualBox" step:

https://github.com/diskuv/dkml-c-probe/actions/runs/2449053587

In the example you will see openbsd 6.9 disabled because the VirtualBox OpenBSD 6.9 image is missing some important syspatches. That problem is not part of this issue

[jacob-carlborg]

Sounds reasonable. I can add the -v flag only in debug mode.

[jacob-carlborg]

You need to update to version 0.6.1.

[jonahbeckford]

I tried v0.6.1 and it now only works if I have debug mode enabled:

Syncing back files
/usr/bin/rsync -uzrtopg  [email protected]:work/ /Users/runner/work
rsync: link_stat "/Users/runner/work/dkml-c-probe/dkml-c-probe/[email protected]:work/." failed: No such file or directory (2)
rsync error: some files could not be transferred (code 23) at /AppleInternal/BuildRoot/Library/Caches/com.apple.xbs/Sources/rsync/rsync-54.120.1/rsync/main.c(996) [sender=2.6.9]
Terminating VM
/usr/bin/sudo kill -s TERM 1469

I think it is because with an empty syncVerboseFlag:

action/src/action/action.ts

Lines 225 to 227 in 52ad2be

	private get syncVerboseFlag(): string {
	return core.isDebug() ? '-v' : ''
	}

in

action/src/action/action.ts

Lines 235 to 242 in 52ad2be

	await exec.exec('rsync', [
	'-auzrtopg',
	this.syncVerboseFlag,
	'--exclude', '_actions/cross-platform-actions/action',
	...flatMap(excludePaths, p => ['--exclude', p]),
	`${this.workDirectory}/`,
	`runner@${ipAddress}:work`
	])

you would get the second argument (this.syncVerboseFlag) as an empty string.

rsync will see the empty string and try to sync that.