Closed yankay closed 1 week ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: yankay
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/ok-to-test
/ok-to-test
/retest
HI @vannten @erikjiang @cyclinder
Would you please help to review it
"msg": "Failed to create object: b'{\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"admission webhook \\\"virtualmachine-validator.kubevirt.io\\\" denied the request: spec.template.spec.volumes[1].cloudInitNoCloud.userDataBase64.cloudInitNoCloud.userDataBase64 is not a valid base64 value.\",\"reason\":\"Invalid\",\"details\":{\"causes\":[{\"reason\":\"FieldValueInvalid\",\"message\":\"spec.template.spec.volumes[1].cloudInitNoCloud.userDataBase64.cloudInitNoCloud.userDataBase64 is not a valid base64 value.\",\"field\":\"spec.template.spec.volumes[1].cloudInitNoCloud.userDataBase64\"}]},\"code\":422}\n'", "reason": "Unprocessable Entity
Failure (of the openeuler job) looks relevant Also, why put it in extended rather than regular runs ? Not stable enough ?
"msg": "Failed to create object: b'{"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"admission webhook \"virtualmachine-validator.kubevirt.io\" denied the request: spec.template.spec.volumes[1].cloudInitNoCloud.userDataBase64.cloudInitNoCloud.userDataBase64 is not a valid base64 value.","reason":"Invalid","details":{"causes":[{"reason":"FieldValueInvalid","message":"spec.template.spec.volumes[1].cloudInitNoCloud.userDataBase64.cloudInitNoCloud.userDataBase64 is not a valid base64 value.","field":"spec.template.spec.volumes[1].cloudInitNoCloud.userDataBase64"}]},"code":422}\n'", "reason": "Unprocessable Entity
Failure (of the openeuler job) looks relevant Also, why put it in extended rather than regular runs ? Not stable enough ?
HI @VannTen
Thank you for your review. I have fixed the bug related to the base64 string in the code. Additionally, I have adjusted the CI configuration back to regular runs; previously, I had set it to extended runs to make the CI process faster.
/lgtm
What type of PR is this?
What this PR does / why we need it:
Add CI Image for openEuler 24.03
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: