-
Notifications
You must be signed in to change notification settings - Fork 148
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
Fix template files. #154
Fix template files. #154
Conversation
Due to the macro processing issue, an empty line was inserted as the very first line of CRB files. This was causing issues with tools that use strict checking of the YAML file syntax. Specifically it was causing the build using the latest origin-operator-registry image to fail. This patch fixes the templates. Signed-off-by: Lev Veyde <lveyde@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neither of these cluster role bindings are needed anymore. You can just remove both files.
OK, replaced by another set of patches. Please see PR#155. |
We no longer need the CRB files of the CDI and CNA. This patch removes the templates for both. Signed-off-by: Lev Veyde <lveyde@redhat.com> Conflicts: templates/olm-catalog/kubevirt-hyperconverged/VERSION/cdi-operator.crb.yaml.in templates/olm-catalog/kubevirt-hyperconverged/VERSION/cna-operator.crb.yaml.in
We no longer need the CRB files of the CDI and CNA. This patch removes the manifests for both. Signed-off-by: Lev Veyde <lveyde@redhat.com>
Per request added re-based set of patches to this PR as well. |
ci test please |
Which PR is winning here, this or #155 ? |
I think #155 can be closed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (assuming #155 is closed)
Due to the macro processing issue, an empty line was inserted as the very first line of CRB files.
This was causing issues with tools that use strict checking of the YAML file syntax.
Specifically it was causing the build using the latest origin-operator-registry image to fail.
This patch fixes the templates.
Signed-off-by: Lev Veyde lveyde@redhat.com