Memset dns id to 0 #494
Memset dns id to 0 #494
Conversation
jotak
added
the
ok-to-test
|
New image: It will expire after two weeks. To deploy this build, run from the operator repo, assuming the operator is running: USER=netobserv VERSION=5443cf8 make set-agent-image |
|
That fixes the problem: CPU -22% https://docs.google.com/spreadsheets/d/1oc9wGpQym9i4-LzJpTdHL_8yD1DwCyX6Ux9AGa0-h7E/edit?gid=1897911977#gid=1897911977 |
|
pls add in the description this was a regression caused by #469 for better tracking, I checked other hooks they seem fine and initialize their key. |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jotak 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 |
That's a regression following the map split PR, which removed
packedattributes; now that map keys aren't packed anymore, it becomes critical to initialize them: failing to do so results in having many map misses on lookup due to pad fields having random values.