The default bug view has changed. See this FAQ.

release l10n repacks failed due to failed "rm"

RESOLVED FIXED

Status

Release Engineering
Release Automation
P2
critical
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: rail, Assigned: coop)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Smells like a winrm issue to me.

It runs "rm -rf current" and then tries to "mkdir current", but the directory is still there.

command: START
command: rm -rf /c/builds/moz2_slave/rel-m-beta-w32_rpk_5-000000000/mozilla-beta/obj-l10n/dist/current
command: cwd: c:\builds\moz2_slave\rel-m-beta-w32_rpk_5-000000000
command: output:
command: END (0.00s elapsed)

command: START
command: mkdir /c/builds/moz2_slave/rel-m-beta-w32_rpk_5-000000000/mozilla-beta/obj-l10n/dist/current
command: cwd: c:\builds\moz2_slave\rel-m-beta-w32_rpk_5-000000000
command: output:
command: ERROR
Traceback (most recent call last):
  File "c:\builds\moz2_slave\rel-m-beta-w32_rpk_5-000000000\scripts\lib\python\util\commands.py", line 47, in run_cmd
    return subprocess.check_call(cmd, **kwargs)
  File "c:\mozilla-build\python27\lib\subprocess.py", line 511, in check_call
    raise CalledProcessError(retcode, cmd)
CalledProcessError: Command '['mkdir', '/c/builds/moz2_slave/rel-m-beta-w32_rpk_5-000000000/mozilla-beta/obj-l10n/dist/current']' returned non-zero exit status 1
mkdir: cannot create directory `/c/builds/moz2_slave/rel-m-beta-w32_rpk_5-000000000/mozilla-beta/obj-l10n/dist/current': File exists
command: END (0.01s elapsed)

Updated

3 years ago
Blocks: 1013700
(Assignee)

Comment 1

3 years ago
Are we using a jacuzzi for this? We could roll back just the slaves in that jacuzzi temporarily to unblock while I debug.

 cp /bin/rm-msys.exe /bin/rm.exe
Not currently, no.
(Assignee)

Comment 3

3 years ago
The release isn't blocked on this any more, so downgrading to critical.
Severity: blocker → critical
(Assignee)

Updated

3 years ago
Assignee: nobody → coop
Status: NEW → ASSIGNED
Priority: -- → P2
(Assignee)

Comment 4

3 years ago
Created attachment 8428770 [details] [diff] [review]
Convert paths to Windows-style prior to removal

We still need a platform-specific switch in the code, but at least we're using a single tool again.

I was able to successfully repack Windows beta builds in staging with this patch. Log is here:

http://dev-master1.srv.releng.scl3.mozilla.com:8044/builders/release-mozilla-beta-win32_standalone_repack/builds/14/steps/run_script/logs/stdio
Attachment #8428770 - Flags: review?(rail)
(Reporter)

Comment 5

3 years ago
Comment on attachment 8428770 [details] [diff] [review]
Convert paths to Windows-style prior to removal

Review of attachment 8428770 [details] [diff] [review]:
-----------------------------------------------------------------

You can also call msys2windows() for all platforms, it has that check internally.
Attachment #8428770 - Flags: review?(rail) → review+
(Assignee)

Comment 6

3 years ago
Comment on attachment 8428770 [details] [diff] [review]
Convert paths to Windows-style prior to removal

Review of attachment 8428770 [details] [diff] [review]:
-----------------------------------------------------------------

(In reply to Rail Aliiev [:rail] from comment #5) 
> You can also call msys2windows() for all platforms, it has that check
> internally.

Hawt. Landed with simplified logic.

https://hg.mozilla.org/build/tools/rev/2ee071e99643
Attachment #8428770 - Flags: checked-in+
(Assignee)

Updated

3 years ago
Blocks: 692715
This is pretty crappy -- applying msys2windows() just for this call and not elsewhere seems wrong.

How about I make winrm check for something in the environment.. maybe OSTYPE=msys (that seems to be set), and then have it handle msys style paths?
(Assignee)

Comment 8

3 years ago
(In reply to Vladimir Vukicevic [:vlad] [:vladv] from comment #7)
> This is pretty crappy -- applying msys2windows() just for this call and not
> elsewhere seems wrong.
> 
> How about I make winrm check for something in the environment.. maybe
> OSTYPE=msys (that seems to be set), and then have it handle msys style paths?

Sure, if you want. I don't want to sign you up for more work.
(Assignee)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.