615097 - config.py shouldn't depend on non Python libraries

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Rail Aliiev [:rail]]

Attachment: Hacky patch

Bug 588150 introduced an new import in config.py which prevents readBranchConfig from [1] evaluating of the content of config.py, which we use for reconfigless slave side scripts, which don't have buildbot master libraries.

It would be great if we avoid external dependencies or wrap the imported classes/functions to allow evaluating config.py without ImportErrors.

1. http://hg.mozilla.org/build/tools/file/aac57c95e2ed/lib/python/release/info.py#l62

The attached patch is a hacky way to resolve the current situation which prevents running release l10n repacks and landing some other release automation related patches.

Comment 1

[Rail Aliiev [:rail]]

Assigning to Syed. According to the IRC conversation, he is going to move the logic to the factories.

Comment 2

[Syed Albiz [:salbiz]]

Attachment: set ccache basedir inside buildfactory rather than config.py

This patch deals with what is currently the only offender in config.py by moving it into the constructor of MercurialBuildFactory. dump_masters.sh output with this change seems fine.

Comment 3

[Rail Aliiev [:rail]]

Comment on attachment 493778 [details] [diff] [review]
set ccache basedir inside buildfactory rather than config.py

Looks good to me. Don't forget to remove the corresponding entry from config.py ;)

Comment 4

[Rail Aliiev [:rail]]

If I understand correctly, this bug blocks 3.6.13 build2 l10n repacks. Setting as "blocker".

Comment 5

[[e-mail] (:bhearsum)]

Comment on attachment 493561 [details] [diff] [review]
Hacky patch

This is the safest thing that will let us start 3.6.13/3.5.16build2, r+.

Comment 6

[[e-mail] (:bhearsum)]

Comment on attachment 493561 [details] [diff] [review]
Hacky patch

changeset:   3386:6cf865c6e3a0

Comment 7

[Syed Albiz [:salbiz]]

Attachment: remove temporary fix & WithProperties statements

I've also backed out the other fix in this patch. Since there shouldn't be any need for it in the future once 3.5.16 is finished and the patch to move the ccache_basedir into the build factory has landed. output of dump-masters.sh and config.py looks good.

Comment 8

[Rail Aliiev [:rail]]

Comment on attachment 493981 [details] [diff] [review]
remove temporary fix & WithProperties statements

Looks good

Comment 9

[Syed Albiz [:salbiz]]

Comment on attachment 493981 [details] [diff] [review]
remove temporary fix & WithProperties statements

Tested a linux build, BASEDIR looks good.

Comment 10

[[e-mail] (:bhearsum)]

Comment on attachment 493981 [details] [diff] [review]
remove temporary fix & WithProperties statements

Landed in changeset:   3609:fc3a0690fad5, merged to the production branch in 3610:e5262b6572a3

Comment 11

[[e-mail] (:bhearsum)]

Comment on attachment 493778 [details] [diff] [review]
set ccache basedir inside buildfactory rather than config.py

Landed in changeset:   1249:f48ae5f00010, merged to the production-0.8 branch in 1250:30537b778f04