Skip to content

fix: WPB-25755 remove old images not in use anymore#909

Merged
mohitrajain merged 3 commits into
masterfrom
WPB-25755-remove-old-system-containers
Jun 2, 2026
Merged

fix: WPB-25755 remove old images not in use anymore#909
mohitrajain merged 3 commits into
masterfrom
WPB-25755-remove-old-system-containers

Conversation

@mohitrajain

@mohitrajain mohitrajain commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

remove old images not in use anymore from proc_system_containers.sh

bash-5.2# for img in 'docker.io/library/nginx:1.25.4-alpine' 'bats/bats:1.11.1' 'cr.step.sm/smallstep/step-ca:0.25.3-rc7' 'registry.k8s.io/ingress-nginx/kube-webhook-certgen:v20231011-8b53cabe0' 'quay.io/jetstack/cert-manager-webhook:v1.16.3'; do   echo "=== $img ===";   kubectl get pods -A -o json | jq -r --arg IMG "$img" '
    .items[]
    | select(
        ([.spec.containers[]?.image, .spec.initContainers[]?.image] | any(. == $IMG))
      )
    | "\(.metadata.namespace)/\(.metadata.name)"
  '; done
=== docker.io/library/nginx:1.25.4-alpine ===
=== bats/bats:1.11.1 ===
=== cr.step.sm/smallstep/step-ca:0.25.3-rc7 ===
=== registry.k8s.io/ingress-nginx/kube-webhook-certgen:v20231011-8b53cabe0 ===
=== quay.io/jetstack/cert-manager-webhook:v1.16.3 ===
cert-manager-ns/cert-manager-webhook-7494fb9bdd-b6fgn

following images are coming from cert-manager:

  {
    "chart": "cert-manager",
    "images": [
      "quay.io/jetstack/cert-manager-cainjector:v1.16.3",
      "quay.io/jetstack/cert-manager-controller:v1.16.3",
      "quay.io/jetstack/cert-manager-startupapicheck:v1.16.3",
      "quay.io/jetstack/cert-manager-webhook:v1.16.3"
    ]
  },

Change type

  • Fix
  • Feature
  • Documentation
  • Security / Upgrade

Basic information

  • THIS CHANGE REQUIRES A DEPLOYMENT PACKAGE RELEASE
  • THIS CHANGE REQUIRES A WIRE-DOCS RELEASE

Testing

  • I ran/applied the changes myself, in a test environment.
  • The CI job attached to this repo will test it for me.

Offline Build CI (label-based)

Add one or more labels to trigger offline builds:

  • build-default - Full production build (ansible, terraform, all packages)
  • build-dev - WIAB/dev build
  • build-wiab-staging - WIAB-staging build
  • build-min - Minimal build (fastest, essential charts only)
  • build-all - Run all three builds

Note: No builds run by default. Add a label to trigger CI.

Tracking

  • I added a new entry in an appropriate subdirectory of changelog.d
  • I mentioned this PR in Jira, OR I mentioned the Jira ticket in this PR.
  • I mentioned this PR in one of the issues attached to one of our repositories.

Knowledge Transfer

  • An Asciinema session is attached to the Jira ticket.

Motivation

Objective

Reason

Use case

@Veki301 Veki301 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

* Enable MLS protocol in production values

Mohit had me deploy a wire-server (wire in a box) install following the current instructions. The install itself went mostly well, but recently I tried to set up a new user and log into the webapp, and that produced an error related to MLS/Proteus.
The fix was to change the brig config `setEnableMLS` to `true`, which caused the backend and frontend to "match".
This is a commit aimed at creating a PR that fixes this issue moving forward.

* fix: WPB-25755 Enable MLS protocol in production values for brig

* fix: WPB-25755 Enable MLS protocol in dev values for brig

---------

Co-authored-by: mohitrajain <mohit.rajain@wire.com>
@sonarqubecloud

sonarqubecloud Bot commented Jun 2, 2026

Copy link
Copy Markdown

@mohitrajain mohitrajain merged commit 2dd562a into master Jun 2, 2026
14 of 18 checks passed
@mohitrajain mohitrajain deleted the WPB-25755-remove-old-system-containers branch June 2, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants