Case insensitive parsing or Plugins and PluginSettings #2054

larohra · 2020-10-28T19:23:40Z

Description

TFS task with more details - Bug 8088875: Make ExtensionConfig parsing consistent for Windows and Linux

Issue #

The title of the PR is clear and informative.
There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
If applicable, the PR references the bug/issue that it fixes in the description.
New Unit tests were added for the changes made and Travis.CI is passing.

azurelinuxagent/common/protocol/goal_state.py

larohra added 2 commits October 28, 2020 10:51

Parsing settings is case invariant now

a3c6d88

Added test case for the scenario

90340eb

larohra requested review from kevinclark19a, narrieta, pgombar and ZhidongPeng as code owners October 28, 2020 19:23

kevinclark19a previously approved these changes Oct 28, 2020

View reviewed changes

pgombar previously approved these changes Oct 28, 2020

View reviewed changes

pgombar reviewed Oct 28, 2020

View reviewed changes

azurelinuxagent/common/protocol/goal_state.py Outdated Show resolved Hide resolved

Added a check for None

83b29ae

larohra dismissed stale reviews from pgombar and kevinclark19a via 83b29ae October 28, 2020 20:52

larohra requested review from pgombar and kevinclark19a October 28, 2020 20:52

narrieta approved these changes Oct 28, 2020

View reviewed changes

pgombar approved these changes Oct 28, 2020

View reviewed changes

larohra merged commit 9c9893e into Azure:develop Oct 28, 2020

larohra deleted the case-insensitive-parsing branch October 28, 2020 22:25