Update wg template output

[marko-bekhta]

Localized sites are failing the sync, e.g. https://github.com/quarkusio/ja.quarkus.io/actions/runs/11649704974/job/32437525219

This was originally fixed here #2098

But looks like the wg sync action was executed (cedef7a) that added back the file without the .template

[github-actions]

🙈 The PR is closed and the preview is expired.

[cescoffier]

This is breaking the sync.
Why this was merged without review?

[cescoffier]

I honestly don't know how this is fixing the issue and for sure it's not working anymore, as the new data is in a new file.

[marko-bekhta]

hey @cescoffier the sync was already broken and this was an attempt to fix it. There's a follow up opened here: #2155
sorry for the inconvenience 🙈

[cescoffier]

Which sync was broken? The working group one? It worked this weekend (at least Friday night).

[marko-bekhta]

the sync of localized sites...

[cescoffier]

What does it need? The WG sync is very simple, it just runs a script updating a file under _data (and push the change to git)

[cescoffier]

I don't know what that line is doing:

subprocess.CalledProcessError: Command 'PERLLIB=vendor/po4a/lib vendor/po4a/po4a-updatepo --msgmerge-opt --no-fuzzy-matching --master-charset UTF-8 -f yaml --master upstream/working-groups/templates/wg.yaml --po l10n/po/ja_JP/working-groups/templates/wg.yaml.po' returned non-zero exit status 255.

But I guess it's where the issue needs to be fixed - not by breaking the working group sync process.

[marko-bekhta]

ooook ... let me try to summarize it.

see the sync of a localized site e.g. this one: https://github.com/quarkusio/ja.quarkus.io/actions/runs/11649704974/job/32437525219

the problem is in the templates/wg.yaml. Which was already renamed in the past (in this PR #2098) to a wg.yml.template.

with the changes merged in this PR https://github.com/quarkusio/quarkusio.github.io/pull/2041/files
the templates/wg.yaml was added back for some reason (while it should've not).

[marko-bekhta]

now see this PR: #2155
which I opened right away after realizing that the update to the GH action was wrong ...

[yrodiere]

Why this was merged without review?

It was merged with my review, I just did not formally add the approval. And, apparently, was wrong. Sorry.

Do I understand correctly that #2155 fixed things?

[cescoffier]

I approved and merged #2155 - thanks!

@yrodiere it reverts the WG sync change that broke it. But it does not fix the localized web site issue.

[cescoffier]

@yrodiere @marko-bekhta

Seems like the issue in in https://github.com/doc-l10n-kit/quarkus-l10n-utils/blob/main/lib/quarkus_l10n_utils/po_manager.py#L146. But it's calling a perl script.

[cescoffier]

I fear it's this: https://github.com/mquinson/po4a/blob/326600634c81a49b41fe5e4e1bfaae2847e0963d/po4a-updatepo#L16 (My Perl is a bit rusty)