Commit 37315572 authored by Derek Beckett's avatar Derek Beckett Committed by Commit Bot

Adding functionality for full proto names

There are some device protos which have the same field names (e.g. enabled).
Currently the policy_server cannot differentiate between them. This will
all setting these by: "groupname.fieldname", e.g:
"device_power_peak_shift.enabled".

TEST=Locally deployed and tested
BUG=TBD

Change-Id: I9565f3082ed192e787b2a00efec1ae29ebe195f2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790534Reviewed-by: default avatarDenis Kuznetsov <antrim@chromium.org>
Reviewed-by: default avatarRoman Sorokin [CET] <rsorokin@chromium.org>
Commit-Queue: Derek Beckett <dbeckett@chromium.org>
Cr-Commit-Position: refs/heads/master@{#694856}
parent a23122ae
......@@ -1004,7 +1004,12 @@ class PolicyRequestHandler(BaseHTTPServer.BaseHTTPRequestHandler):
# policy config file.
for field in group_message.DESCRIPTOR.fields:
field_value = None
if field.name in policies:
full_name = '{}.{}'.format(group.name, field.name)
if full_name in policies:
got_fields = True
field_value = policies[full_name]
self.SetProtobufMessageField(group_message, field, field_value)
elif field.name in policies:
got_fields = True
field_value = policies[field.name]
self.SetProtobufMessageField(group_message, field, field_value)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment