Skip to content
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.

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

add --ampls-resource-id and --enable-high-log-scale-mode options #7754

Merged
merged 28 commits into from
Jul 31, 2024
Merged
Show file tree
Hide file tree
Changes from 17 commits
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
ae7e6bb
add --ampls-resource-id and --enable-high-log-scale-mode options
ganga1980 Jun 28, 2024
1aed0eb
fix pr feedback
ganga1980 Jun 28, 2024
064d7e0
fix pr feedback
ganga1980 Jun 28, 2024
85bbb7c
resolve merge conflicts
ganga1980 Jun 28, 2024
c560c45
Merge branch 'Azure:main' into main
ganga1980 Jun 28, 2024
103ee15
resolve merge conflicts
ganga1980 Jun 28, 2024
6121ff8
Merge branch 'main' of github.com:ganga1980/azure-cli-extensions into…
ganga1980 Jun 28, 2024
3f7c7dd
add live tests
ganga1980 Jun 28, 2024
13ee00f
fix pr feedback
ganga1980 Jun 28, 2024
65c4e64
fix pr feedback
ganga1980 Jun 28, 2024
2124e2a
fix pr feedback
ganga1980 Jun 28, 2024
5f3d03b
lint errors
ganga1980 Jun 28, 2024
3dcda47
fix merge conflicts
ganga1980 Jul 16, 2024
3c7de7e
comment tests since require latest az cli
ganga1980 Jul 17, 2024
2534b11
fix merge conflicts
ganga1980 Jul 17, 2024
a760193
Merge branch 'Azure:main' into main
ganga1980 Jul 22, 2024
728aa2e
re-enable tests
ganga1980 Jul 22, 2024
8077f6a
fix pr feedback
ganga1980 Jul 22, 2024
ca413b9
fix merge conflicts
ganga1980 Jul 26, 2024
c5b5ef6
fix lint errors
ganga1980 Jul 26, 2024
957b9ef
Merge branch 'Azure:main' into main
ganga1980 Jul 29, 2024
56ba2ce
Merge branch 'Azure:main' into main
ganga1980 Jul 30, 2024
0f37111
Merge branch 'Azure:main' into main
ganga1980 Jul 31, 2024
8f9fb5c
fix bug
ganga1980 Jul 31, 2024
bebdae5
fix lint errors
ganga1980 Jul 31, 2024
ad84ede
fix lint errors
ganga1980 Jul 31, 2024
007f7af
fix lint errors
ganga1980 Jul 31, 2024
9a6d01c
bump aks-preview version
ganga1980 Jul 31, 2024
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
5 changes: 5 additions & 0 deletions src/aks-preview/HISTORY.rst
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,11 @@ Pending
++++++++
* [BREAKING CHANGE]: Remove support for `az aks get-os-options` command.

6.0.0b2
ganga1980 marked this conversation as resolved.
Show resolved Hide resolved
++++++++
* Add option `--enable-high-log-scale-mode` to `az aks create --enable-addons monitoring` and `az aks enable-addons -a monitoring` .
* Add option `--ampls-resource-id` to `az aks create --enable-addons monitoring` and `az aks enable-addons -a monitoring` .

6.0.0b1
++++++++
* [BREAKING CHANGE]: Remove support for `az aks mesh` egress gateway commands.
Expand Down
24 changes: 24 additions & 0 deletions src/aks-preview/azext_aks_preview/_help.py
Original file line number Diff line number Diff line change
Expand Up @@ -287,6 +287,12 @@
- name: --data-collection-settings
type: string
short-summary: Path to JSON file containing data collection settings for Monitoring addon.
- name: --enable-high-log-scale-mode
type: bool
short-summary: Enable High Log Scale Mode for Container Logs.
- name: --ampls-resource-id
type: string
short-summary: Resource ID of Azure Monitor Private Link scope for Monitoring Addon.
- name: --enable-cluster-autoscaler
type: bool
short-summary: Enable cluster autoscaler, default value is false.
Expand Down Expand Up @@ -2298,6 +2304,12 @@
- name: --data-collection-settings
type: string
short-summary: Path to JSON file containing data collection settings for Monitoring addon.
- name: --enable-high-log-scale-mode
type: bool
short-summary: Enable High Log Scale Mode for Container Logs.
- name: --ampls-resource-id
type: string
short-summary: Resource ID of Azure Monitor Private Link scope for Monitoring Addon.
- name: --subnet-name -s
type: string
short-summary: The subnet name for the virtual node to use.
Expand Down Expand Up @@ -2365,6 +2377,12 @@
- name: --data-collection-settings
type: string
short-summary: Path to JSON file containing data collection settings for Monitoring addon.
- name: --enable-high-log-scale-mode
type: bool
short-summary: Enable High Log Scale Mode for Container Logs.
- name: --ampls-resource-id
type: string
short-summary: Resource ID of Azure Monitor Private Link scope for Monitoring Addon.
- name: --subnet-name -s
type: string
short-summary: The subnet name for the virtual node to use.
Expand Down Expand Up @@ -2446,6 +2464,12 @@
- name: --data-collection-settings
type: string
short-summary: Path to JSON file containing data collection settings for Monitoring addon.
- name: --enable-high-log-scale-mode
type: bool
short-summary: Enable High Log Scale Mode for Container Logs.
- name: --ampls-resource-id
type: string
short-summary: Resource ID of Azure Monitor Private Link scope for Monitoring Addon.
- name: --subnet-name -s
type: string
short-summary: The subnet name for the virtual node to use.
Expand Down
8 changes: 8 additions & 0 deletions src/aks-preview/azext_aks_preview/_params.py
Original file line number Diff line number Diff line change
Expand Up @@ -632,6 +632,8 @@ def load_arguments(self, _):
)
c.argument("enable_syslog", arg_type=get_three_state_flag(), is_preview=True)
c.argument("data_collection_settings", is_preview=True)
c.argument("enable_high_log_scale_mode", arg_type=get_three_state_flag(), is_preview=True)
c.argument("ampls_resource_id", is_preview=True)
c.argument("aci_subnet_name")
c.argument("appgw_name", arg_group="Application Gateway")
c.argument("appgw_subnet_cidr", arg_group="Application Gateway")
Expand Down Expand Up @@ -1853,6 +1855,8 @@ def load_arguments(self, _):
)
c.argument("enable_syslog", arg_type=get_three_state_flag(), is_preview=True)
c.argument("data_collection_settings", is_preview=True)
c.argument("enable_high_log_scale_mode", arg_type=get_three_state_flag(), is_preview=True)
c.argument("ampls_resource_id", is_preview=True)
c.argument(
"dns_zone_resource_id",
deprecate_info=c.deprecate(
Expand Down Expand Up @@ -1908,6 +1912,8 @@ def load_arguments(self, _):
)
c.argument("enable_syslog", arg_type=get_three_state_flag(), is_preview=True)
c.argument("data_collection_settings", is_preview=True)
c.argument("enable_high_log_scale_mode", arg_type=get_three_state_flag(), is_preview=True)
c.argument("ampls_resource_id", is_preview=True)
c.argument(
"dns_zone_resource_id",
deprecate_info=c.deprecate(
Expand Down Expand Up @@ -1946,6 +1952,8 @@ def load_arguments(self, _):
)
c.argument("enable_syslog", arg_type=get_three_state_flag(), is_preview=True)
c.argument("data_collection_settings", is_preview=True)
c.argument("enable_high_log_scale_mode", arg_type=get_three_state_flag(), is_preview=True)
c.argument("ampls_resource_id", is_preview=True)
c.argument(
"dns_zone_resource_id",
deprecate_info=c.deprecate(
Expand Down
14 changes: 7 additions & 7 deletions src/aks-preview/azext_aks_preview/addonconfiguration.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,9 @@ def enable_addons(
dns_zone_resource_ids=None,
enable_msi_auth_for_monitoring=True,
enable_syslog=False,
data_collection_settings=None
data_collection_settings=None,
ampls_resource_id=None,
enable_high_log_scale_mode=False,
):
instance = client.get(resource_group_name, name)
# this is overwritten by _update_addons(), so the value needs to be recorded here
Expand Down Expand Up @@ -99,9 +101,7 @@ def enable_addons(
rotation_poll_interval=rotation_poll_interval,
no_wait=no_wait,
dns_zone_resource_id=dns_zone_resource_id,
dns_zone_resource_ids=dns_zone_resource_ids,
enable_syslog=enable_syslog,
data_collection_settings=data_collection_settings,
dns_zone_resource_ids=dns_zone_resource_ids
)

if CONST_MONITORING_ADDON_NAME in instance.addon_profiles and instance.addon_profiles[
Expand All @@ -124,7 +124,9 @@ def enable_addons(
create_dcr=True,
create_dcra=True,
enable_syslog=enable_syslog,
data_collection_settings=data_collection_settings
data_collection_settings=data_collection_settings,
ampls_resource_id=ampls_resource_id,
enable_high_log_scale_mode=enable_high_log_scale_mode
)
else:
# monitoring addon will use legacy path
Expand Down Expand Up @@ -204,8 +206,6 @@ def update_addons(
dns_zone_resource_id=None,
dns_zone_resource_ids=None,
no_wait=False, # pylint: disable=unused-argument
enable_syslog=False, # pylint: disable=unused-argument
data_collection_settings=None, # pylint: disable=unused-argument
):
# parse the comma-separated addons argument
addon_args = addons.split(',')
Expand Down
30 changes: 24 additions & 6 deletions src/aks-preview/azext_aks_preview/custom.py
Original file line number Diff line number Diff line change
Expand Up @@ -534,6 +534,8 @@ def aks_create(
enable_msi_auth_for_monitoring=True,
enable_syslog=False,
data_collection_settings=None,
ampls_resource_id=None,
enable_high_log_scale_mode=False,
aci_subnet_name=None,
appgw_name=None,
appgw_subnet_cidr=None,
Expand Down Expand Up @@ -2009,6 +2011,8 @@ def aks_addon_enable(
dns_zone_resource_ids=None,
enable_syslog=False,
data_collection_settings=None,
ampls_resource_id=None,
enable_high_log_scale_mode=False
):
return enable_addons(
cmd,
Expand All @@ -2033,6 +2037,8 @@ def aks_addon_enable(
dns_zone_resource_ids=dns_zone_resource_ids,
enable_syslog=enable_syslog,
data_collection_settings=data_collection_settings,
ampls_resource_id=ampls_resource_id,
enable_high_log_scale_mode=enable_high_log_scale_mode
)


Expand Down Expand Up @@ -2063,6 +2069,8 @@ def aks_addon_update(
dns_zone_resource_ids=None,
enable_syslog=False,
data_collection_settings=None,
ampls_resource_id=None,
enable_high_log_scale_mode=False
):
instance = client.get(resource_group_name, name)
addon_profiles = instance.addon_profiles
Expand Down Expand Up @@ -2112,6 +2120,8 @@ def aks_addon_update(
dns_zone_resource_ids=dns_zone_resource_ids,
enable_syslog=enable_syslog,
data_collection_settings=data_collection_settings,
ampls_resource_id=ampls_resource_id,
enable_high_log_scale_mode=enable_high_log_scale_mode
)


Expand Down Expand Up @@ -2146,6 +2156,8 @@ def aks_disable_addons(cmd, client, resource_group_name, name, addons, no_wait=F
create_dcra=True,
enable_syslog=False,
data_collection_settings=None,
ampls_resource_id=None,
enable_high_log_scale_mode=False
)
except TypeError:
pass
Expand Down Expand Up @@ -2188,6 +2200,8 @@ def aks_enable_addons(
dns_zone_resource_ids=None,
enable_syslog=False,
data_collection_settings=None,
ampls_resource_id=None,
enable_high_log_scale_mode=False,
aks_custom_headers=None,
):
headers = get_aks_custom_headers(aks_custom_headers)
Expand Down Expand Up @@ -2223,8 +2237,6 @@ def aks_enable_addons(
no_wait=no_wait,
dns_zone_resource_id=dns_zone_resource_id,
dns_zone_resource_ids=dns_zone_resource_ids,
enable_syslog=enable_syslog,
data_collection_settings=data_collection_settings,
)
if (
CONST_MONITORING_ADDON_NAME in instance.addon_profiles and
Expand Down Expand Up @@ -2255,14 +2267,21 @@ def aks_enable_addons(
create_dcra=True,
enable_syslog=enable_syslog,
data_collection_settings=data_collection_settings,
ampls_resource_id=ampls_resource_id,
enable_high_log_scale_mode=enable_high_log_scale_mode
)
else:
# monitoring addon will use legacy path
if enable_syslog:
raise ArgumentUsageError(
"--enable-syslog can not be used without MSI auth.")
if enable_high_log_scale_mode:
raise ArgumentUsageError(
"--enable-high-log-scale-mode can not be used without MSI auth.")
if data_collection_settings is not None:
raise ArgumentUsageError("--data-collection-settings can not be used without MSI auth.")
if ampls_resource_id is not None:
raise ArgumentUsageError("--ampls-resource-id can not be used without MSI auth.")
ensure_container_insights_for_monitoring(
cmd,
instance.addon_profiles[CONST_MONITORING_ADDON_NAME],
Expand Down Expand Up @@ -2311,7 +2330,7 @@ def aks_rotate_certs(cmd, client, resource_group_name, name, no_wait=True):
return sdk_no_wait(no_wait, client.begin_rotate_cluster_certificates, resource_group_name, name)


def _update_addons(cmd, # pylint: disable=too-many-branches,too-many-statements
def _update_addons(cmd, # pylint: disable=too-many-branches,too-many-statements,
instance,
subscription_id,
resource_group_name,
Expand All @@ -2333,9 +2352,8 @@ def _update_addons(cmd, # pylint: disable=too-many-branches,too-many-statements
rotation_poll_interval=None,
dns_zone_resource_id=None,
dns_zone_resource_ids=None,
no_wait=False, # pylint: disable=unused-argument
enable_syslog=False,
data_collection_settings=None):
no_wait=False,): # pylint: disable=unused-argument

ManagedClusterAddonProfile = cmd.get_models(
"ManagedClusterAddonProfile",
resource_type=CUSTOM_MGMT_AKS_PREVIEW,
Expand Down
Loading
Loading