You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
While looking at #1824 I used config snippets designed for Epiphany 0.5 with epicli built from develop branch. After this PR #1687 snippets in question are not compatible with current code, but we don't assert that in any way! 😱
In result epcli apply on Azure/RHEL without explicitly defined infrastructure/virtual-machine for repository VM fails because default config for the VM is to use UbuntuServer OS image.
To Reproduce
Steps to reproduce the behavior:
use config snippet for older Epiphany (for example from 0.5)
Expected behavior
Our epicli command should assert that cluster config is not compatible, inform the user in an error message and point to documentation describing solution to this problem.
Note that the user may not explicitly specify all components and then default values are used, which are not always set to 0, so validating only the input yaml file may not be sufficient unless we force the user to explicitly specify all components or we validate the manifest file.
✅ An assertion has been added to check whether all components specified explicitly in the yaml configuration file and those enabled implicitly in the defaults, have the same operating system defined.
Describe the bug
While looking at #1824 I used config snippets designed for Epiphany 0.5 with epicli built from develop branch. After this PR #1687 snippets in question are not compatible with current code, but we don't assert that in any way! 😱
In result
epcli apply
on Azure/RHEL without explicitly definedinfrastructure/virtual-machine
for repository VM fails because default config for the VM is to use UbuntuServer OS image.To Reproduce
Steps to reproduce the behavior:
epicli apply ...
Please notice the
*.deb
extenstion in the log. 😱Expected behavior
Our
epicli
command should assert that cluster config is not compatible, inform the user in an error message and point to documentation describing solution to this problem.Config files
OS (please complete the following information):
Cloud Environment (please complete the following information):
Additional context
N/A
The text was updated successfully, but these errors were encountered: