Skip to content

fix: breaking change in enabling monitoring for OpenShift managed cluster #11778

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
mmyyrroonn merged 29 commits into from
Jan 14, 2020
Merged

fix: breaking change in enabling monitoring for OpenShift managed cluster #11778

Changes from 25 commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
b633783
add cluster spn with metric publisher role to aks cluster resource
ganga1980 Feb 14, 2019
34b4ff2
refactor the code
ganga1980 Feb 14, 2019
5d5e06f
fix default workspace issue in china cloud
ganga1980 Feb 15, 2019
13b28a1
add metrics publisher role assignment only in public cloud
ganga1980 Feb 15, 2019
20b8de5
fix pr feedback
ganga1980 Feb 15, 2019
459c11e
fix build error
ganga1980 Feb 20, 2019
77582df
fix lint error
ganga1980 Feb 20, 2019
86b7364
Merge remote-tracking branch 'upstream/dev' into dev
ganga1980 May 18, 2019
552d381
update region mapping
ganga1980 May 18, 2019
c345d65
order regions in sortorder
ganga1980 May 18, 2019
72aa35f
fix trailing whitespaces
ganga1980 May 18, 2019
f98956b
Merge branch 'dev' of https://github.com/Azure/azure-cli into dev
ganga1980 Jun 3, 2019
c284e2c
fix default log analytics region mapping
ganga1980 Jun 3, 2019
6849944
Merge branch 'dev' of https://github.com/Azure/azure-cli into dev
ganga1980 Jun 4, 2019
da318f6
merge upstream changes
ganga1980 Jul 15, 2019
c88769c
fix map index error if key not found
ganga1980 Jul 18, 2019
df2bd1b
Merge branch 'dev' of https://github.com/Azure/azure-cli into dev
ganga1980 Jul 19, 2019
20a1e9e
fix lint errors
ganga1980 Jul 19, 2019
36d0726
fix uninitialized error
ganga1980 Jul 20, 2019
5f2f854
get latest changes from upstream
ganga1980 Nov 26, 2019
38eb2f9
Merge branch 'dev' of https://github.com/Azure/azure-cli into dev
ganga1980 Dec 1, 2019
6e5e85f
update to new API version for ARO
ganga1980 Dec 2, 2019
6fbbcc6
use latest api version for ARO
ganga1980 Dec 3, 2019
296e1b3
Merge branch 'dev' of https://github.com/Azure/azure-cli into dev
ganga1980 Jan 6, 2020
0f753b1
revert to workspace_resource_id in os management profile
ganga1980 Jan 6, 2020
a9a196e
fix test case
ganga1980 Jan 10, 2020
b15b774
fixed test failure
ganga1980 Jan 13, 2020
c898fa2
added recording file
ganga1980 Jan 13, 2020
ae6c3e7
Merge branch 'dev' of https://github.com/Azure/azure-cli into dev
ganga1980 Jan 13, 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
2 changes: 1 addition & 1 deletion src/azure-cli/azure/cli/command_modules/acs/custom.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3202,7 +3202,7 @@ def openshift_create(cmd, client, resource_group_name, name, # pylint: disable=
workspace_id = '/' + workspace_id
if workspace_id.endswith('/'):
workspace_id = workspace_id.rstrip('/')
monitor_profile = OpenShiftManagedClusterMonitorProfile(enabled=True, workspace_id=workspace_id) # pylint: disable=line-too-long
monitor_profile = OpenShiftManagedClusterMonitorProfile(enabled=True, workspace_resource_id=workspace_id) # pylint: disable=line-too-long
else:
monitor_profile = None

Expand Down