[ZBX-23184] Suspision that YAML export is not compliant with standard Created: 2023 Aug 01 Updated: 2023 Aug 16 Resolved: 2023 Aug 16 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | Frontend (F) |
Affects Version/s: | 6.0.20, 6.4.4, 7.0.0alpha3 |
Fix Version/s: | None |
Type: | Problem report | Priority: | Trivial |
Reporter: | Jeroen Baten | Assignee: | Zabbix Development Team |
Resolution: | Won't fix | Votes: | 1 |
Labels: | None | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified | ||
Environment: |
Zabbix server 6.2.9 on Centos 8 stream |
Sprint: | Sprint 103 (Aug 2023) |
Description |
Steps to reproduce:
Result: zabbix_export: {{Also, using yamllint says: }}
Also, there are a lot of errors because the lines are too long. Expected: I looked into this and found the Yaml standard. Turns out, version prior to 1.2 use "- - - " was used to sepearate documents so it makes sense not to start with three dashes when exporting a single template. See https://yaml.org/spec/1.0/ and search for three dashes and you will find it in section 2.2. Starting from Yaml standard version 1.2.0 that seems to have changed.
I just thought I bring this to your attention. I hope you find this helpful. Regards, Jeroen Baten
|
Comments |
Comment by dimir [ 2023 Aug 11 ] |
The official documentation mentions "three dashes": FYI this is one of the good candidates for YAML reference: https://camel.readthedocs.io/en/latest/yamlref.html Probably we should decide on the YAML version we support and document it. |
Comment by Alexander Vladishev [ 2023 Aug 11 ] |
The second document clearly states that these attributes are optional:
|
Comment by dimir [ 2023 Aug 11 ] |
Agree. There is also
Do we ever use directives? |
Comment by Alexander Vladishev [ 2023 Aug 11 ] |
Our YAML files do not contain directives. Therefore, in our case, the triple-dash is optional. I suggest to close this issue as WON'T FIX. |