Improve error handling when loading yaml stream #11962
Conversation
marcusburghardt
force-pushed
the
improve_yaml_processing
branch
from
d7a57cd
to
1281cbd
Compare
|
Start a new ephemeral environment with changes proposed in this pull request: Fedora Environment Oracle Linux 8 Environment |
|
🤖 A k8s content image for this PR is available at: Click here to see how to deploy itIf you alread have Compliance Operator deployed: Otherwise deploy the content and operator together by checking out ComplianceAsCode/compliance-operator and: |
During tests with Ansible macros it was noticed a case where the Ansible
task names include "{{{ rule_title }}}" and an specific rule has special
symbols in its title. This was causing the load of the stream to fail
but the exception was not clear or easy to debug. So, it was created an
exception for this case giving some hints for the developer.
marcusburghardt
force-pushed
the
improve_yaml_processing
branch
from
1281cbd
to
47b5c3b
Compare
|
Code Climate has analyzed commit 47b5c3b and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 33.3% (50% is the threshold). This pull request will bring the total coverage in the repository to 59.4% (0.0% change). View more on Code Climate. |
Description:
During tests with Ansible macros it was noticed a case where the Ansible task names include "{{{ rule_title }}}" and an specific rule has special symbols in its title.
This was causing the load of the stream to fail but the exception was not clear or easy to debug.
So, it was created an exception for this case giving some hints for the developer.
Rationale: