Skip to content

Pre-flight checks for CSI Reverse Proxy#379

Merged
Sahiba-Gupta merged 3 commits into
mainfrom
pvt/precheck_error
Dec 13, 2024
Merged

Pre-flight checks for CSI Reverse Proxy#379
Sahiba-Gupta merged 3 commits into
mainfrom
pvt/precheck_error

Conversation

@Sahiba-Gupta

@Sahiba-Gupta Sahiba-Gupta commented Dec 5, 2024 •
edited by AkshaySainiDell
Loading

Copy link
Copy Markdown
Contributor

This PR will solve the prefilght check for reverseproxy.

Checklist:

  • Have you run format,vet & lint checks against your submission?
  • Have you made sure that the code compiles?
  • Did you run the unit & integration tests successfully?
  • Have you maintained at least 90% code coverage?
  • Have you commented your code, particularly in hard-to-understand areas
  • Have you done corresponding changes to the documentation
  • Did you run tests in a real Kubernetes cluster?
  • Backward compatibility is not broken

How Has This Been Tested?
This has been tested locally . tested the image by deploying driver using helm and operator. Also ran cert-csi.

[root@master-1-wcJauzYo5qu0m dell-csi-helm-installer]# kubectl get pods -n powermax
NAME                                 READY   STATUS    RESTARTS      AGE
powermax-controller-74cdc978-29p5x   6/6     Running   8 (44s ago)   77s
powermax-node-tpq4s                  2/2     Running   0             77s
powermax-revproxy-86954b76cf-dllp8   1/1     Running   0             4m31
[2:40](https://delloneisg.slack.com/archives/D03KC23RTC5/p1733211646615879)
[root@lglbg082-Rhel8-jenkins Preflight-Scans]# ./preflight check container 10.247.98.98:5000/revproxy:sg > preflight_checks.logs
time="2024-12-03T07:39:51Z" level=info msg="certification library version" version="1.10.2 <commit: 3a93f15cba1f3a4517c02f10914d6cff5cfa5c60>"
time="2024-12-03T07:39:52Z" level=info msg="running checks for 10.247.98.98:5000/revproxy:sg for platform amd64"
time="2024-12-03T07:39:52Z" level=info msg="target image" image="10.247.98.98:5000/revproxy:sg"
time="2024-12-03T07:39:54Z" level=info msg="check completed" check=HasLicense result=PASSED
time="2024-12-03T07:39:54Z" level=info msg="check completed" check=HasUniqueTag result=PASSED
time="2024-12-03T07:39:54Z" level=info msg="check completed" check=LayerCountAcceptable result=PASSED
time="2024-12-03T07:39:54Z" level=info msg="check completed" check=HasNoProhibitedPackages result=PASSED
time="2024-12-03T07:39:54Z" level=info msg="check completed" check=HasRequiredLabel result=PASSED
time="2024-12-03T07:39:54Z" level=info msg="USER revproxy:revproxy specified that is non-root" check=RunAsNonRoot
time="2024-12-03T07:39:54Z" level=info msg="check completed" check=RunAsNonRoot result=PASSED
time="2024-12-03T07:39:56Z" level=info msg="check completed" check=HasModifiedFiles result=PASSED
time="2024-12-03T07:39:57Z" level=info msg="check completed" check=BasedOnUbi result=PASSED
time="2024-12-03T07:39:57Z" level=info msg="This image's tag sg will be paired with digest sha256:22b5877122c590d82ae17e5ab016d9f0e009e72c07dc3754c9e785f1b34d8a18 once this image has been published in accordance with Red Hat Certification policy. You may then add or remove any supplemental tags through your Red Hat Connect portal as you see fit."
time="2024-12-03T07:39:57Z" level=info msg="Preflight result: PASSED"

@kumarkgosa kumarkgosa self-requested a review December 6, 2024 13:11
@adarsh-dell

adarsh-dell commented Dec 9, 2024

Copy link
Copy Markdown

Hi @Sahiba-Gupta ,

I noticed that some of the checks are failing. Additionally, the controller pod has restarted 8 times within approximately 1 minute, which seems unusual. Could you please look into this and share any insights on the reason for these frequent restarts?

Thank you!

@Sahiba-Gupta Sahiba-Gupta merged commit 8b0db82 into main Dec 13, 2024
@Sahiba-Gupta Sahiba-Gupta deleted the pvt/precheck_error branch December 13, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abhi16394 abhi16394 Awaiting requested review from abhi16394

@adarsh-dell adarsh-dell Awaiting requested review from adarsh-dell

@bpjain2004 bpjain2004 Awaiting requested review from bpjain2004

@donatwork donatwork Awaiting requested review from donatwork

@tdawe tdawe Awaiting requested review from tdawe tdawe is a code owner

@kumarkgosa kumarkgosa Awaiting requested review from kumarkgosa

1 more reviewer

@AkshaySainiDell AkshaySainiDell AkshaySainiDell approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Sahiba-Gupta @adarsh-dell @kumarkgosa @AkshaySainiDell