Merge "Don't use RBE for bp2build/queryview/api_bp2build/json_module_graph, even with USE_RBE=true." am: b54a3d08ac
am: 2ec2dbc3ec
am: 9eba82453d
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2643172 Change-Id: If1efb45745501cef1a329c6ec8c0d97cf2bde513 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -1375,10 +1375,15 @@ func (c *configImpl) StartGoma() bool {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func (c *configImpl) UseRBE() bool {
|
func (c *configImpl) UseRBE() bool {
|
||||||
|
// These alternate modes of running Soong do not use RBE / reclient.
|
||||||
|
if c.Bp2Build() || c.Queryview() || c.ApiBp2build() || c.JsonModuleGraph() {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
authType, _ := c.rbeAuth()
|
authType, _ := c.rbeAuth()
|
||||||
// Do not use RBE with prod credentials in scenarios when stubby doesn't exist, since
|
// Do not use RBE with prod credentials in scenarios when stubby doesn't exist, since
|
||||||
// its unlikely that we will be able to obtain necessary creds without stubby.
|
// its unlikely that we will be able to obtain necessary creds without stubby.
|
||||||
if !c.StubbyExists() && strings.Contains(authType, "use_google_prod_creds"){
|
if !c.StubbyExists() && strings.Contains(authType, "use_google_prod_creds") {
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
if v, ok := c.Environment().Get("USE_RBE"); ok {
|
if v, ok := c.Environment().Get("USE_RBE"); ok {
|
||||||
|
Reference in New Issue
Block a user