Give better log infomation in Pages job

Here is an example pipeline
https://git.drupalcode.org/project/scheduler/-/pipelines/328214

Is the 'pages:deploy' job viewable by a user with higher authority than the project maintainer?

The documentation site is https://project.pages.drupalcode.org/scheduler but this is not shown in any job log or help page. Maybe we can at minimum add this to the pages log?

The only place where we can add some useful logs is here: https://git.drupalcode.org/project/gitlab_templates/-/blob/main/includes...

pages:deploy is a GitLab specific job, so we have no control, but we can definitely add things to the pages job. There is an CI variable with the URL to the generated site, so we can echo that.

Tested here
https://git.drupalcode.org/project/scheduler/-/pipelines/330500

Is there anything else we can add?

I suggested a small change in the MR.

@fjgarlin,
How about adding a line of dashes as a divider?

That's almost exactly the comment that I made in the MR. DIVIDER is just a bunch of * characters.

Working on it.

Perfect. Look at other examples of how it is used within the file for reference.

@fjgarlin ,
Added the divider as desired.
Kindly check and merge :)

I gave feedback in the MR.

@fjgarlin, done with the changes
Please have a look.

Feedback in the MR.

@fjgarlin, please check.

I left a further comment in the MR

@fjgarlin @jonathan1055,
Removed the extra '\n' character and '\' before $CI_PAGES_URL.
Kindly review :)

Thanks for fixing the message. Here is the test before
https://git.drupalcode.org/project/scheduler/-/jobs/3272612

and after
https://git.drupalcode.org/project/scheduler/-/jobs/3279263

All looks good now, but I will make an adjustment for that extra $divider variable in the 'check environment' job

With this MR and $DIVIDER removed - 'environment check' before. The ended green because the pipe syntax was removed and the 'exit 1' was therefore not executed
https://git.drupalcode.org/project/scheduler/-/jobs/3279475

With latest commit to this MR, 'environment check' has the correct message box and ends red as required.
https://git.drupalcode.org/project/scheduler/-/jobs/3280383

I have resolved the various MR comment threads where the code has been fixed, and also reverted the temporoary change that allowed the 'pages' job to be tested.

Unrelated pipeline fail, due to 'check versions' showing that 11.0.6 and 10.3.7 have been released. This is RTBC from me, but will let @fjgarlin do the final review.

Oh wow, I didn't catch the exit 1 being ignored, thanks for testing it!

Did a final check, agree it's good to RTBC and merge.

Downstream with keycdn running pages: https://git.drupalcode.org/issue/gitlab_templates-3485285/-/pipelines/33...

Automatically closed - issue fixed for 2 weeks with no activity.