[Barefoot] Enable 'TELEMETRY_WRITABLE' for telemetry container

[YuliiaBashko]

Why I did it

For testing gNMI/gNOI we need to enable r/w mode for telemetry container

How I did it

Added TELEMETRY_WRITABLE=y option to make command

How to verify it

Build SONiC with command make TELEMETRY_WRITABLE=y target/sonic-aboot-barefoot.swi

Which release branch to backport (provide reason below if selected)

• 201811
• 201911
• 202006
• 202012

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

[YuliiaBashko]

@xumia Could you please take a look?

[xumia]

Do you mean as below? the value ${{ parameters.buildOptions }} is replaced in the template compiling process.
BUILD_OPTIONS: ${{ parameters.buildOptions }} TELEMETRY_WRITABLE=y

See https://github.com/Azure/sonic-buildimage/blob/03c6b1bf7c9462a8a2b8a17c8ecfe5e6cc5a77c8/.azure-pipelines/azure-pipelines-build.yml#L30

The variables in jobVariables applying to all job groups can be overridden by the variables in each of the jobGroups.

[YuliiaBashko]

I see, thanks, I`ll fix this. So it means that my change wouldn`t work at all?

[YuliiaBashko]

@xumia Could you please check my new change?

[lguohan]

telemtry_writable should be consistent across different platform. we cannot enable write only on a particular platform.

[YuliiaBashko]

telemtry_writable should be consistent across different platform. we cannot enable write only on a particular platform.

Could we enable telemtry_writable for all platforms? If yes, I will update pull request with needed changes.

[lguohan]

that is not the current plan. the viable solution is to make the telemetry writable as a runtime option, then user can enable it on the fly and no longer a build option.