John Skarbek activity https://gitlab.com/skarbek 2026-03-17T14:26:02Z tag:gitlab.com,2026-03-17:5213204371 John Skarbek commented on merge request !9216 at GitLab.org / omnibus-gitlab 2026-03-17T14:26:02Z skarbek John Skarbek

@dmakovey example pipeline: https://dev.gitlab.org/gitlab/omnibus-gitlab/-/pipelines/425571

The manifest-upload job runs a few stages prior to the check-packages-availability trigger job. This appears to become a blocking job, 😬

Could we add an empty needs that way there's no dependency on the pipeline stage?

Also question. I'm noticing this rule:

    - if: '$PIPELINE_TYPE =~ /_(RC|TAG)_BUILD_PIPELINE$/'
      when: manual

Is on this job. What plays those? I'm not aware of release-tools performing this action 🤔

tag:gitlab.com,2026-03-17:5213089079 John Skarbek deleted project branch sk/update-mirrors-doc-release-platform at GitLab.org / release / GitLab Release Docs 2026-03-17T14:03:40Z skarbek John Skarbek

John Skarbek (97fef25f) at 17 Mar 14:03

tag:gitlab.com,2026-03-17:5213088234 John Skarbek pushed to project branch master at GitLab.org / release / GitLab Release Docs 2026-03-17T14:03:31Z skarbek John Skarbek

John Skarbek (cd06dda9) at 17 Mar 14:03

Merge branch 'sk/update-mirrors-doc-release-platform' into 'master'

... and 1 more commit

tag:gitlab.com,2026-03-17:5213088225 John Skarbek accepted merge request !1015: docs: Update mirrors documentation with Release Platform module at GitLab.org / release / GitLab Release Docs 2026-03-17T14:03:31Z skarbek John Skarbek

Summary

Updates the security mirrors documentation to recommend the new YAML-driven Release Platform module as the preferred method for creating security mirrors.

Changes

  • Added Release Platform module section as the preferred method for creating security mirrors
  • Wrapped legacy Terraform instructions in a collapsible <details> block
  • Added "Importing Existing Projects" section explaining current limitations
  • Updated prerequisites to reference the new import section
  • Added PSIRT approval to security mirror requirements

Why

The Release Platform module (work item #50) provides a standardized, YAML-driven way to provision the complete 3-mirror architecture with built-in integrations for common-ci-tasks, Vault, and more. This simplifies onboarding new projects and ensures consistent configuration.

References

tag:gitlab.com,2026-03-16:5209940409 John Skarbek commented on merge request !998 at GitLab.com / GitLab Infrastructure Team / ArgoCD / ArgoCD Applications 2026-03-16T20:11:39Z skarbek John Skarbek

Ok, after a bit of research, I see no evidence that this is actually true 😒

tag:gitlab.com,2026-03-16:5209921522 John Skarbek commented on merge request !998 at GitLab.com / GitLab Infrastructure Team / ArgoCD / ArgoCD Applications 2026-03-16T20:04:54Z skarbek John Skarbek

@myacksmith I'm simply copying and pasting the work that you've done. Did you attempt apache style annotation at all? I'm willing to give this a shot. Curious your take prior to moving forward and accepting such suggestion.

tag:gitlab.com,2026-03-16:5209917580 John Skarbek deleted project branch sh-gprd-cny-load-shedding at GitLab.com / GitLab Infrastructure Team / Kubernetes Workloads / GitLab.com 2026-03-16T20:03:45Z skarbek John Skarbek

John Skarbek (4a3f776d) at 16 Mar 20:03

tag:gitlab.com,2026-03-16:5209916862 John Skarbek pushed to project branch master at GitLab.com / GitLab Infrastructure Team / Kubernetes Workloads / GitLab.com 2026-03-16T20:03:35Z skarbek John Skarbek

John Skarbek (0a52bcf3) at 16 Mar 20:03

Merge branch 'sh-gprd-cny-load-shedding' into 'master'

... and 1 more commit

tag:gitlab.com,2026-03-16:5209916815 John Skarbek accepted merge request !5257: gprd-cny: enable Workhorse load shedding at GitLab.com / GitLab Infrastructure Team / Kubernetes Workloads / GitLa... 2026-03-16T20:03:34Z skarbek John Skarbek

What does this MR do?

This enables the Workhorse load shedding feature added in gitlab-org/gitlab!218865. In gprd, we use the following Prometheus query to determine that a backlog of 50-100 might be a reasonable threshold to start:

max by (pod) (
 max_over_time(puma_queued_connections{type="api"}[5m:15s])
)

In addition this enables NGINX to retry on 504 errors up to 3 times.

Relates to gitlab-com/gl-infra/production#21451

Author Check-list

Please read the Contributing document and once you do, complete the following:

  • Check if all of the following apply:
    • Assign to the correct reviewer per the contributing document
    • Apply the correct metadata per the contributing document
    • Link to related MRs for applying the changes on other environments
    • Link to related Chef changes
    • If necessary link to a Criticality 4 Change Request issue

Reviewer Check-list

  • Check if all of the following apply:
    • Reviewed the diff jobs to confirm changes are as expected
    • No changes shown in the diffs not associated with this MR - This may require a rebase or further investigation

Applier Check-list

  • Make sure there is no ongoing deployment for the affected envs before merging (see #announcements slack channel)
tag:gitlab.com,2026-03-16:5209893963 John Skarbek commented on merge request !998 at GitLab.com / GitLab Infrastructure Team / ArgoCD / ArgoCD Applications 2026-03-16T19:56:14Z skarbek John Skarbek

This is ready for review!

tag:gitlab.com,2026-03-16:5209859883 John Skarbek opened merge request !998: Add JSON access log format to Pulp API and Content pods in pre at GitLab.com / GitLab Infrastructure Team / ArgoCD / ... 2026-03-16T19:43:02Z skarbek John Skarbek

Summary

Configure gunicorn_access_logformat for both API and Content pods in the pre environment to output structured JSON logs for better observability and log parsing.

Changes

  • Added JSON access log format for API pods with fields: remote_ip, method, path, status, duration_us, correlation_id
  • Added JSON access log format for Content pods with fields: remote_ip, request, status, duration_us, correlation_id

Why not workers?

Worker pods run Pulp's distributed task worker process (not gunicorn), so gunicorn_access_logformat is not applicable to them. Workers process asynchronous tasks from a queue, not HTTP requests.

Relates to https://gitlab.com/gitlab-org/build/team-tasks/-/work_items/63