Skip to content

Error log linking update production

Will Meek requested to merge error_log_links_update_prod_address into master

What does this MR do and why?

Describe in detail what your merge request does and why.

As per https://gitlab.com/gitlab-org/quality/team-tasks/-/issues/1244

It was noticed that Production logs didn't log URLs eg.

https://ops.gitlab.net/gitlab-org/quality/production/-/jobs/6625044

The targeted URL is https://gitlab.com rather than https://www.gitlab.com

This MR updates this

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Due to targeting production and being used on HTTP 500 level failure conditions, not straightforward to test.

If we look at logs from production failures (eg. https://ops.gitlab.net/gitlab-org/quality/production/-/jobs/6625044 ) that target eg. https://gitlab.com, our unit tests prove that that URL will now match as production for logging.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading