Bug 2222191 - [cnv-4.12] manually increasing the number of virt-api pods does not work
Summary: [cnv-4.12] manually increasing the number of virt-api pods does not work
Keywords:
Status: ON_QA
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Virtualization
Version: 4.12.5
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 4.12.6
Assignee: sgott
QA Contact: Kedar Bidarkar
URL:
Whiteboard:
Depends On: 2227746
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-07-12 08:17 UTC by Barak
Modified: 2023-08-16 12:22 UTC (History)
1 user (show)

Fixed In Version: hco-bundle-registry-container-v4.12.6-8 virt-launcher-container-v4.12.6-1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github kubevirt kubevirt pull 10087 0 None Merged [Release-0.58] Fix patch api conflict 2023-08-16 12:00:42 UTC
Red Hat Issue Tracker CNV-30881 0 None None None 2023-07-12 08:17:59 UTC

Description Barak 2023-07-12 08:17:09 UTC
--------------------------------------------------
Description of problem:
--------------------------------------------------
as a part of the new KubeVirt rate limiting testing, I tried to increase the number of virt-api pods using this which had no issues before,
I suspect the new auto virt-api auto-scaler overwrites the manual patch.

================================================================================================
apiVersion: hco.kubevirt.io/v1beta1
kind: HyperConverged
metadata:
  annotations:
    deployOVS: "false"
    hco.kubevirt.io/tuningPolicy: '{"qps":200,"burst":400}'
    kubevirt.kubevirt.io/jsonpatch: '[{"op": "add", "path": "/spec/customizeComponents/patches",
      "value": [{"resourceType": "Deployment", "resourceName": "virt-api", "type":
      "json", "patch": "[{\"op\": \"replace\", \"path\": \"/spec/replicas\", \"value\":
      6}]"}]}]'
=================================================================================================
oc get deployment virt-api -o yaml

readyReplicas: 2
  replicas: 2
  updatedReplicas: 2
=================================================================================================
oc -n openshift-cnv get kubevirt kubevirt -o yaml

customizeComponents:
      patches:
      - patch: '[{"op": "replace", "path": "/spec/replicas", "value": 6}]'
        resourceName: virt-api
        resourceType: Deployment
        type: json
=================================================================================================
Boaz Ben Shabat, 10:29 AM
[root@e29-h19-740xd storms_tool]# oc get pod -n openshift-cnv|grep virt-api
virt-api-8795754c5-npd5m                               1/1     Running   1               30h
virt-api-8795754c5-v8tn5                               1/1     Running   1               30h
=================================================================================================


--------------------------------------------------
Version-Release number of selected component (if applicable):
--------------------------------------------------


--------------------------------------------------
How reproducible:
--------------------------------------------------
every time.

--------------------------------------------------
Steps to Reproduce:
--------------------------------------------------

apply the following patch:

oc patch hco -n openshift-cnv kubevirt-hyperconverged --type=merge -p '{"metadata":{"annotations":{"kubevirt.kubevirt.io/jsonpatch":"[{\"op\": \"add\", \"path\": \"/spec/customizeComponents/patches\", \"value\": [{\"resourceType\": \"Deployment\", \"resourceName\": \"virt-api\", \"type\": \"json\", \"patch\": \"[{\\\"op\\\": \\\"replace\\\", \\\"path\\\": \\\"/spec/replicas\\\", \\\"value\\\": 6}]\"}]}]"}}}'


--------------------------------------------------
Expected results:
--------------------------------------------------
the number of virt-api pods should increase to 6


Note You need to log in before you can comment on or make changes to this bug.