Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

feat: adding windows defender process exclusions #2967

Merged
merged 1 commit into from Apr 2, 2020

Conversation

marosset
Copy link
Contributor

@marosset marosset commented Mar 25, 2020

Adding Windows defender process exclusions for kubelet.exe and csi-proxy-server.exe to reduce unnecessary cpu load on Windows nodes

Reason for Change:

This is an ask from @mikkelhegn from AKS team

Issue Fixed:

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Mar 25, 2020

Codecov Report

Merging #2967 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2967   +/-   ##
=======================================
  Coverage   72.57%   72.57%           
=======================================
  Files         141      141           
  Lines       25934    25934           
=======================================
  Hits        18821    18821           
  Misses       6014     6014           
  Partials     1099     1099           
Impacted Files Coverage Δ
pkg/engine/templates_generated.go 34.82% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43f76e5...d91deea. Read the comment docs.

Copy link
Member

@mboersma mboersma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Caveat: this looks straightforward to me, but I am far from a PowerShell or Windows expert.

@acs-bot
Copy link

acs-bot commented Mar 27, 2020

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: marosset, mboersma

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mboersma
Copy link
Member

Given current constraints in Azure, we may not get the 1.15 e2e to pass for a while, so I think we can consider this green enough to merge. 💚

@marosset marosset merged commit 54c9756 into Azure:master Apr 2, 2020
@marosset marosset deleted the kubelet-defender-exception branch April 2, 2020 16:47
@marosset
Copy link
Contributor Author

marosset commented Apr 2, 2020

Thanks Matt

AbelHu pushed a commit to AbelHu/aks-engine that referenced this pull request Apr 22, 2020
AbelHu pushed a commit to AbelHu/aks-engine that referenced this pull request Apr 22, 2020
andyliuliming pushed a commit that referenced this pull request Apr 22, 2020
…d processes (#2967) (#3118)

Co-authored-by: Mark Rossetti <marosset@microsoft.com>
AbelHu added a commit to AbelHu/aks-engine that referenced this pull request Apr 22, 2020
…d processes (Azure#2967) (Azure#3118)

Co-authored-by: Mark Rossetti <marosset@microsoft.com>
xuto2 pushed a commit that referenced this pull request Apr 23, 2020
* Upgrade Windows VHD image to April release (#3111)

* chore: adding azure-cni v1.0.33 artifacts to VHDs (#2790)

* chore: update Azure CNI to v1.0.33 (#2825)

* feat: collect Windows CSE logs during log collection (#2858)

* fix: Windows no outbound fixes (#2883)

* fix: fixing nssm logging in windows CSE (#2890)

* fix: Get WindowsVersion from registry instead of calling Get-ComputerInfo (#2954)

* feat: Updating AKS to use April 2020 Windows VHDs by default (#3060)

Co-authored-by: Mark Rossetti <marosset@microsoft.com>
Co-authored-by: Jaeryn <13284103+jaer-tsun@users.noreply.github.com>
Co-authored-by: Kalya Subramanian <42158129+ksubrmnn@users.noreply.github.com>

* feat: adding kubelet and csi-proxy-server as windows defender excluded processes (#2967) (#3118)

Co-authored-by: Mark Rossetti <marosset@microsoft.com>

Co-authored-by: Mark Rossetti <marosset@microsoft.com>
Co-authored-by: Jaeryn <13284103+jaer-tsun@users.noreply.github.com>
Co-authored-by: Kalya Subramanian <42158129+ksubrmnn@users.noreply.github.com>
AbelHu pushed a commit to AbelHu/aks-engine that referenced this pull request Apr 23, 2020
andyliuliming pushed a commit that referenced this pull request Apr 23, 2020
* feat: Adding WindowsNodeReset.ps1 script to reset/cleanup state for nodes (#2457)

* Adding WindowsNodeReset.ps1 script to reset/cleanup state for windox^C nodes

* fix linting errors

* fixing  comments per CR feedback

* chore: adding azure-cni v1.0.33 artifacts to VHDs (#2790)

* chore: update Azure CNI to v1.0.33 (#2825)

* feat: collect Windows CSE logs during log collection (#2858)

* fix: Windows no outbound fixes (#2883)

* fix: fixing nssm logging in windows CSE (#2890)

* fix: Get WindowsVersion from registry instead of calling Get-ComputerInfo (#2954)

* feat: adding kubelet and csi-proxy-server as windows defender excluded processes (#2967)

* feat: Updating AKS to use April 2020 Windows VHDs by default (#3060)

* feat: Updating Windows VHDs with 4B patches (#3115)

Co-authored-by: Mark Rossetti <marosset@microsoft.com>
Co-authored-by: Jaeryn <13284103+jaer-tsun@users.noreply.github.com>
Co-authored-by: Kalya Subramanian <42158129+ksubrmnn@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants