Remove `-t -t` from Drydock SSH interface
ClosedPublic
Actions

Authored by hach-que on Dec 5 2013, 3:56 AM.

Tags

None

Referenced Files

	F18099229: D7710.id17413.diff
	Sat, Aug 9, 2:54 AM

	F18097105: D7710.id17414.diff
	Fri, Aug 8, 11:09 AM

	F18086222: D7710.id17413.diff
	Tue, Aug 5, 8:12 PM

	F17943402: D7710.diff
	Thu, Jul 31, 8:21 AM

	F17938762: D7710.id17414.diff
	Thu, Jul 31, 2:36 AM

	F17917985: D7710.id17414.diff
	Wed, Jul 30, 12:27 AM

	F17908789: D7710.diff
	Tue, Jul 29, 1:59 PM

	F17882661: D7710.id.diff
	Mon, Jul 28, 10:55 PM

Subscribers

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T1049: Implement Harbormaster
T4111: Use Drydock to allocate build machines for Harbormaster

Commits

Restricted Diffusion Commit
rP48e80f05412e: Remove `-t -t` from Drydock SSH interface

Summary

These arguments prevent stderr from being routed correctly for Linux hosts and break Windows entirely. Removing them fixes the issue.

Test Plan

Removed those options and both Linux and Windows hosts had their output fed back into Harbormaster correctly.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley accepted this revision.Dec 5 2013, 3:56 AM

Closed by commit rP48e80f05412e (authored by @hach-que).

Revision Contents
Changeset List

Path

Size

src/

applications/

drydock/

interface/

command/

DrydockSSHCommandInterface.php

4 lines

Diff 17414

View Options

src/applications/drydock/interface/command/DrydockSSHCommandInterface.php

Remove `-t -t` from Drydock SSH interfaceClosedPublicActions