I've reproduce this issue, and it seems to be related to the YAML::Tiny perl module complaining about malformed antora.yml (this one more especially https://pagure.io/fedora-docs/pages/blob/master/f/homepage/antora.yml).
And it's not completely wrong, as the ROOT::index.adoc part should be quoted here to avoid colons messing with the format.
I've not tried yet, but since @jibecfed is not affected, I suspect a more up-to-date library might be a little bit more flexible in this regards.

EDIT: Confirmed. The YAML::Tiny module shipped in rhel7 (1.51) can't read the antora.yml file, the new one directly from cpan (1.73) can.

So the solution is to quote the file where it should be quoted ? (or upgrade the server to EL 8)

Exactly. Another solution would be to move this app to openshift.

Le lun. 20 juil. 2020 =C3=A0 11:59, Michael S pagure@pagure.io a =C3=A9cr=
it :

misc added a new comment to an issue you are following:
So the solution is to quote the file where it should be quoted ? (or upgrade the server to EL 8)

To reply, visit the link below or just reply to this email
https://pagure.io/fedora-infrastructure/issue/9126

PR was merged, we'll see tonight if it have an impact: https://pagure.io/fedora-docs/pages/pull-request/10

sorry, but adding the quote had no impact

When i try to build locally on RHEL 7, I get it updated (index.adoc) no matter the type of quote I use (as I wondered about double quote vs single quote). Maybe the change was right but not reflected in prod for another reason ?

@darknao, since you can reproduce, can you tell if the quote change by @jibecfed work fine for you ?

The quote fixed the initial issue being the index.adoc not updated.
Now we have a second one, that the push to git is not working for some reason.

the pre commit hook maybe?

found it:
in build.py :

shutil.copytree(os.path.join(dir_path, '.githooks'), os.path.join(repo_dir, '.githooks'), dirs_exist_ok=True)

the dirs_exist_ok is added in python3.8, we are in python3.6.

Here is a patch proposal, if that can help.

This remove the use of dirs_exist_ok with shutil.copytree and remove any local changes to your local cloned repositories so you can copy your githook back without errors.
The git reset can also be seen as a security measure as I assume there shouldn't be any local uncommitted changes at this point.

i "applied" the patch and it failed because of an existing folder=2E

note: did an ugly commit yesterday because git complained abouy you patch =
when doing git patch --check

i won't have access to computer for the next few days=2E

Misc, you have commit access on the repository=2E i would appreciate if yo=
u may have the opportunity to give it a look=2E

in addition, it looks like the script is executed on sundries01 and sundri=
es02 because i get two emails=2E
is it because of some migration or a real issue?

ok my bad, I was mistakenly thinking there was a .gitignore rules for the githook directory, so my git reset doesn't achieve the intended goal here.

After a more thoroughly reading of your script, there is a few things to fix.
If you need to overwrite .githook, then you may want to use distutil.dir_util.copy_tree instead.

Your pre-commit hook is not working for at least two reasons.
There is a syntax error line 26 : (("1" + "2")) doesn't work in bash4, ((1 + 2)) does.
This hook is also never called anyway. The usual hook directory is .git/hooks, and the core.hookspath config is only working since git 2.9 (el7 use git 1.8).
That is probably a good thing actually, since your hook do a git rm of your file, which completely remove it from the repo. I think you may want to only unstage it instead with git reset or simply abort the commit.

Most of these errors are related to rhel7 binaries. I can only recommend you to set up a dev/test environment using el7, if you can.
Moving this app to openshift may also be a good idea, and give you more control over the running environment.
I can probably help you with that, if you want.

And as of sundries02, I think that is a real issue too.

Openshift was the plan (and ideally, split that per repo so we do not wait several, but can schedule them and/or update them at different speed), but sundries was easier to do. I was waiting for the DC move to be officially over before working on that, but feel free to do it.

For the 2 sundries, I guess there was a double deployment on nasible due to the main sundries being down or something. It should be fixable in ansible without too much fuzz.

Ok, so I here is my proposal:
I can make a PR on your translations-scripts repo with a new Dockerfile that will be used by openshift, among a few fixes on your pre-commit hook & build.py.
Then another PR on ansible repo to create the associated openshift-app.
I will probably generate a new ssh key pair, so I'll send you the new pub key at this time.
Once this is done and working, we'll remove the scripts from sundries 1&2.
How does that sound to you ?

The ssh key should be stored in a secret on openshift, IMHO. The way I did in the past was to generate it and display when running the script, I guess it coud be more elegant. Alternatively, we can also just ask to a sysadmin to move the existing one in a openshift secret and that's it.

I think the key should be somewhere like in /srv/private so it can be easily used by ansible to generate the openshift secret.

So where are we here?

This is waiting for https://pagure.io/fedora-infra/ansible/pull-request/184 to be reviewed/merged.
In the mean time, translations-scripts has been temporary fixed to work with el7 binaries, so docs seems to be up to date for now.

So, the request is merged, but we have a few issues to work through on it.

Since the issue the ticket was raised about is fixed / worked around now, I am gonna close this. Feel free to re-open if you see this being out of date again...