(Win8 and Win2k8 machines) Please deploy _dumbwin32proc.py to the right place

RESOLVED FIXED

Status

task
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: armenzg, Assigned: q)

Tracking

Details

Recently, in bug 798170 we have deployed _dumbwin32proc.py to the right places on our win7, winxp and win64 machines. This allows buildbot to kill jobs when stopped through web UI.

We need to deploy this to the win8 machines as well and I would like to use GPO to deploy it rather than having to manually log in into all machines (besides we need it for future re-imaging).

These are the steps taken as Administrator:
cd C:\mozilla-build\buildbotve\Lib\site-packages\twisted\internet
del _dumbwin32proc.pyc
move _dumbwin32proc.py _dumbwin32proc.py.bak
C:\mozilla-build\wget\wget.exe -q --no-check-certificate https://hg.mozilla.org/build/opsi-package-sources/raw-file/e55c081cb8cf/twisted_dumbwin32proc/CLIENT_DATA/_dumbwin32proc.py

We also need the same change to be added to the task sequence for our Win64-ix builders.

Thanks in advance!
Assignee: server-ops-releng → q
Blocks: 853824
Replaced on the WDS server so we should be set for the next re-image. I will set the file to go out to the Win 8 machines on Monday.
(In reply to Q from comment #1)
> Replaced on the WDS server so we should be set for the next re-image. I will
> set the file to go out to the Win 8 machines on Monday.

Q++!! Thanks!
File going out to win 8 boxes!
All set.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You rock! Thanks!
I've found that t-w864-ix-001 does not have this change. Is it because it is on the loaner DC? (was DC the correct term?)

t-w84-ix-003 seems to be fine.

On another note, I noticed that I can't ssh into t-w864-ix-002.


C:\Users\cltbld.T-W864-IX-001>cd C:\mozilla-build\buildbotve\Lib\site-packages\twisted\internet

C:\mozilla-build\buildbotve\Lib\site-packages\twisted\internet>C:\mozilla-build\wget\wget.exe -q --no-check-certificate https://hg.mozilla.org/build/opsi-package-sources/raw-file/e55c081cb8cf/twisted_dumbwin32proc/CLIENT_DATA/_dumbwin32proc.py

C:\mozilla-build\buildbotve\Lib\site-packages\twisted\internet>diff -pU 8 _dumbwin32proc.py _dumbwin32proc.py.1
--- _dumbwin32proc.py   2013-01-17 15:02:07 -0800
+++ _dumbwin32proc.py.1 2013-05-09 15:15:36 -0700
@@ -233,18 +233,17 @@ class Process(_pollingfile._PollingTimer

         self._addPollableResource(_Reaper(self))


     def signalProcess(self, signalID):
         if self.pid is None:
             raise error.ProcessExitedAlready()
         if signalID in ("INT", "TERM", "KILL"):
-            win32process.TerminateProcess(self.hProcess, 1)
-
+            os.popen('taskkill /T /F /PID %s' % self.pid)

     def _getReason(self, status):
         if status == 0:
             return error.ProcessDone(status)
         return error.ProcessTerminated(status)


     def write(self, data):
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Fixed for the Loaner OU
Depends on: 781277
This is now covered in both GPO and in the install TS for all OSes
Status: REOPENED → RESOLVED
Closed: 7 years ago6 years ago
Resolution: --- → FIXED
Component: Server Operations: RelEng → RelOps
Product: mozilla.org → Infrastructure & Operations
You need to log in before you can comment on or make changes to this bug.