search

SUSE / technical-reference-documentation

SUSE Technical Reference Documentation
https://documentation.suse.com/trd-supported.html
7 stars 22 forks source link

update file content needed for metadata context #121

Open bwgartner opened 1 month ago

bwgartner commented 1 month ago

Updated only the kubernetes reference files ... adoc/docinfo.xml and adoc/SA_vars.adoc and adoc/SA.adoc to just cover most all of the metadata variables. If this is accepted/usable, then I will likely do another PR to update the product versions across the kubernetes reference files and even add more content in this space as well.

chabowski commented 1 month ago

@bwgartner Hi Bryan! Unfortunately I cannot merge this PR. Several validation checks failed, and it looks to me as if there are issues with some titles. Tom is out this week, and only back next week - I assume we need his help with the investigations where we have issues, and if the rest (of metadata) is okay. Also, would you mind rebasing or updating your branch to the the latest main - this will also help to avoid merge conflicts. I think Tom is back on Tuesday - I will ask him to have a look at the PR as soon as he finds time.

Thanks very much! Meike

fsundermeyer commented 1 month ago

Locally, for DC-kubernetes_rc_k3s-slemicro_ampere-altra I also get

asciidoctor: WARNING: tables must have at least one body row

tomschr commented 3 weeks ago

@bwgartner Looking at the GitHub Action I see a lot of those error messages:

asciidoctor: WARNING: skipping reference to missing attribute: title

You are probably already aware that this is some issue with the AD source code. Sounds to me like there are some if clauses that caused the :title: attribute to be NOT defined.