Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploys Kruize UI #161

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bhanvimenghani
Copy link

This pr aims to make changes to the clowdapp yaml file to deploy kruize ui along with kruize and ros ocp.

Copy link
Collaborator

@patilsuraj767 patilsuraj767 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @bhanvimenghani. I have added the comment below I think you will first need to check clowder document.
Also I think your IDE is formatting the whole document that is the reason I can see many changes under Files Changed tab. If you are using vscode can you please disable/remove any custom yaml plugin which is installed and then try again to format the file.
If that didn't work can you also check this - https://stackoverflow.com/questions/61978428/yaml-file-formatting-in-vscode

Comment on lines +119 to +180
pods:
- name: kruize-ui-nginx-pod
containers:
- name: kruize-ui-nginx-container
image: quay.io/kruize/kruize-ui:0.0.3
imagePullPolicy: Always
env:
- name: KRUIZE_UI_ENV
value: "production"
volumeMounts:
- name: nginx-config-volume
mountPath: /etc/nginx/nginx.conf
subPath: nginx.conf
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You cannot directly declare pods here, You will need to create another deployment. Please refer CloudApp API reference doc - https://consoledot.pages.redhat.com/clowder/dev/api_reference.html#k8s-api-github-com-redhatinsights-clowder-apis-cloud-redhat-com-v1alpha1-podspec

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.

2 participants