Skip to content

Conversation

@tkatila
Copy link
Contributor

@tkatila tkatila commented Aug 8, 2023

No description provided.

mythi and others added 4 commits August 8, 2023 10:01
setInitContainer() adds "init-sriov-numvfs" to initContainers
but uses initcontainerName constant to search where to add
the QAT configMap volumeMount. Fix by moving all code to use
the const.

It was also noticed in the controller logs that setting Pod
Volumes is not idempotent but broken DaemonSet gets created:

""intel-device-plugins-manager: Reconciler error "err="DaemonSet.apps
\"intel-qat-plugin\" is invalid: spec.template.spec.volumes[6].name:
Duplicate value: \"qat-config\"" controller="qatdeviceplugin"
controllerGroup="deviceplugin.intel.com"

Finally, change 'qat-config' to 'intel-qat-config-volume' to
better describe that it's a volume.

Signed-off-by: Mikko Ylinen <[email protected]>
Signed-off-by: Tuomas Katila <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #1498 (5109d80) into release-0.26 (c17149f) will decrease coverage by 0.20%.
The diff coverage is 18.75%.

❗ Current head 5109d80 differs from pull request most recent head 56d16bf. Consider uploading reports for the commit 56d16bf to get more accurate results

@@               Coverage Diff                @@
##           release-0.26    #1498      +/-   ##
================================================
- Coverage         50.89%   50.70%   -0.20%     
================================================
  Files                44       44              
  Lines              4985     4990       +5     
================================================
- Hits               2537     2530       -7     
- Misses             2302     2314      +12     
  Partials            146      146              
Files Changed Coverage Δ
pkg/controllers/qat/controller.go 9.72% <0.00%> (-5.11%) ⬇️
pkg/controllers/reconciler.go 4.25% <ø> (ø)
cmd/qat_plugin/dpdkdrv/dpdkdrv.go 83.27% <100.00%> (+0.11%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

mythi and others added 2 commits August 8, 2023 16:34
@mythi mythi merged commit abd7ba5 into intel:release-0.26 Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants