Skip to content

Add TELEMETRY_WRITABLE build option to enable telemetry write mode (default read-only) #4209

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
renukamanavalan merged 16 commits into from
Jun 25, 2020
Merged

Add TELEMETRY_WRITABLE build option to enable telemetry write mode (default read-only) #4209

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
fb9036d
Add build option to make telemetry read-only
seiferteric Feb 28, 2020
2a1e736
Fix indent
seiferteric Feb 29, 2020
8603815
Change TELEMETRY_REAONLY to TELEMETRY_READWRITE since default will be…
seiferteric Mar 10, 2020
762a5b8
Updated supported parameters and config/rules to include TELMETRY_REA…
seiferteric Mar 10, 2020
3cafe6d
Merge remote-tracking branch 'azure/master' into azure_ro_mode
seiferteric Mar 22, 2020
5a6d462
Merge remote-tracking branch 'azure/master' into azure_ro_mode
seiferteric Mar 23, 2020
57c591b
Add SONIC preix to telemetry readonly flag
seiferteric Mar 23, 2020
f514d09
Update rules/config with correct flag name
seiferteric Mar 23, 2020
42ff12c
Merge remote-tracking branch 'azure/master' into azure_ro_mode
seiferteric Mar 23, 2020
4d70acd
Merge remote-tracking branch 'origin/master' into azure_ro_mode
seiferteric Jun 2, 2020
2c35081
Change SONIC_TELEMETRY_READWRITE flag to SONIC_TELEMETRY_WRITABLE
seiferteric Jun 4, 2020
82a0255
Merge remote-tracking branch 'origin/master' into azure_ro_mode
seiferteric Jun 9, 2020
c47035e
Merge remote-tracking branch 'origin/master' into azure_ro_mode
seiferteric Jun 12, 2020
07e9265
Merge remote-tracking branch 'origin/master' into azure_ro_mode
seiferteric Jun 17, 2020
b18cbd6
Remove line readded during merge
seiferteric Jun 17, 2020
6aa7ea1
Change SONIC_TELEMETRY_WRITABLE to TELEMETRY_WRITABLE
seiferteric Jun 23, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions Makefile.work
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,9 @@
# * Default: yes
# * Values: yes, no
# * KERNEL_PROCURE_METHOD: Specifying method of obtaining kernel Debian package: download or build
# * TELEMETRY_WRITABLE: Enable write/config operations via the gNMI interface.
# * Default: unset
# * Values: y
# * SONIC_DPKG_CACHE_METHOD: Specifying method of obtaining the Debian packages from cache: none or cache
# * SONIC_DPKG_CACHE_SOURCE: Debian package cache location when cache enabled for debian packages
# * BUILD_LOG_TIMESTAMP: Set timestamp in the build log (simple/none)
Expand Down Expand Up @@ -200,6 +203,7 @@ SONIC_BUILD_INSTRUCTION := make \
HTTPS_PROXY=$(https_proxy) \
SONIC_ENABLE_SYSTEM_TELEMETRY=$(ENABLE_SYSTEM_TELEMETRY) \
SONIC_ENABLE_RESTAPI=$(ENABLE_RESTAPI) \
TELEMETRY_WRITABLE=$(TELEMETRY_WRITABLE) \
EXTRA_DOCKER_TARGETS=$(EXTRA_DOCKER_TARGETS) \
BUILD_LOG_TIMESTAMP=$(BUILD_LOG_TIMESTAMP) \
SONIC_ENABLE_IMAGE_SIGNATURE=$(ENABLE_IMAGE_SIGNATURE) \
Expand Down
3 changes: 3 additions & 0 deletions rules/config
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,9 @@ ENABLE_RESTAPI = n
# ENABLE_NAT - build docker-sonic-nat for nat support
ENABLE_NAT = y

# TELEMETRY_WRITABLE - Enable write/config operations via the gNMI interface.
# Uncomment to enable:
# TELEMETRY_WRITABLE = y
# INSTALL_KUBERNETES - if set to y kubernetes packages are installed to be able to
# run as worker node in kubernetes cluster.
INSTALL_KUBERNETES = n
Expand Down
2 changes: 2 additions & 0 deletions slave.mk
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -230,6 +230,8 @@ $(info "BLDENV" : "$(BLDENV)")
$(info "VS_PREPARE_MEM" : "$(VS_PREPARE_MEM)")
$(info "ENABLE_SFLOW" : "$(ENABLE_SFLOW)")
$(info "ENABLE_NAT" : "$(ENABLE_NAT)")
$(info "TELEMETRY_WRITABLE" : "$(TELEMETRY_WRITABLE)")
$(info )

include Makefile.cache

Expand Down