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

add Kubernetes manifest. Fixes #687 #735

Merged
merged 4 commits into from
May 15, 2024
Merged

add Kubernetes manifest. Fixes #687 #735

merged 4 commits into from
May 15, 2024

Conversation

SlavikCA
Copy link
Contributor

No description provided.

@kroese
Copy link
Collaborator

kroese commented May 15, 2024

@SlavikCA Is there a specific reason why this example kubernetes .yml uses different values then the example compose .yml? For example, the RAM_SIZE, the DISK_SIZE, the DISK_FMT (qcow2), al differ from the default config. I would like to keep them the same to avoid confusion.

@SlavikCA
Copy link
Contributor Author

Good point about defaults. Updated.

@kroese
Copy link
Collaborator

kroese commented May 15, 2024

I see a notice about DSM requiring HTTPS with Kubernetes, so should port 5001 also be added to .yml or how did you do it?

@SlavikCA
Copy link
Contributor Author

When accessing DSM UI via IP address - there is no error.
The error is only when using DNS, and that's usually the case when using Ingress. And when using Ingress the port 5001 is not necessary for HTTPS

@kroese kroese merged commit 3126b38 into vdsm:master May 15, 2024
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.

None yet

2 participants