[Build] Fix the reproducible build variable display error in the slave container (#14543)
Why I did it Enable the reproducible build for PR build for master branch Fix the reproducible build variable display error in the slave container. The below config is none, although the config is set and takes effect. "SONIC_VERSION_CONTROL_COMPONENTS": "none" How I did it Passing the variable through the slave container command line. The variable has been passed to the slave container and the other docker container by a config file, it is only used to display the value during the build. How to verify it See https://dev.azure.com/mssonic/build/_build/results?buildId=247960&view=logs&j=88ce9a53-729c-5fa9-7b6e-3d98f2488e3f&t=88f376cf-c35d-5783-0a48-9ad83a873284 "SONIC_VERSION_CONTROL_COMPONENTS": "deb,py2,py3,web,git,docker"
This commit is contained in:
parent
1bf50a5566
commit
09bd333b63
@ -1,5 +1,2 @@
|
|||||||
variables:
|
variables:
|
||||||
${{ if eq(variables['Build.Reason'],'PullRequest') }}:
|
VERSION_CONTROL_OPTIONS: 'SONIC_VERSION_CONTROL_COMPONENTS=deb,py2,py3,web,git,docker'
|
||||||
VERSION_CONTROL_OPTIONS: 'SONIC_VERSION_CONTROL_COMPONENTS=$([[ "$(System.PullRequest.TargetBranch)" =~ ^20[2-9][0-9]{3}$ ]] && echo deb,py2,py3,web,git,docker)'
|
|
||||||
${{ else }}:
|
|
||||||
VERSION_CONTROL_OPTIONS: 'SONIC_VERSION_CONTROL_COMPONENTS=deb,py2,py3,web,git,docker'
|
|
||||||
|
@ -561,6 +561,7 @@ SONIC_BUILD_INSTRUCTION := $(MAKE) \
|
|||||||
MIRROR_SECURITY_URLS=$(MIRROR_SECURITY_URLS) \
|
MIRROR_SECURITY_URLS=$(MIRROR_SECURITY_URLS) \
|
||||||
GZ_COMPRESS_PROGRAM=$(GZ_COMPRESS_PROGRAM) \
|
GZ_COMPRESS_PROGRAM=$(GZ_COMPRESS_PROGRAM) \
|
||||||
MIRROR_SNAPSHOT=$(MIRROR_SNAPSHOT) \
|
MIRROR_SNAPSHOT=$(MIRROR_SNAPSHOT) \
|
||||||
|
SONIC_VERSION_CONTROL_COMPONENTS=$(SONIC_VERSION_CONTROL_COMPONENTS) \
|
||||||
$(SONIC_OVERRIDE_BUILD_VARS)
|
$(SONIC_OVERRIDE_BUILD_VARS)
|
||||||
|
|
||||||
.PHONY: sonic-slave-build sonic-slave-bash init reset
|
.PHONY: sonic-slave-build sonic-slave-bash init reset
|
||||||
|
Loading…
Reference in New Issue
Block a user