Skip to content

Conversation

battermann
Copy link
Contributor

@battermann battermann commented Dec 9, 2022

[2022-12-09] (Chart Release )

Bug fixes and other updates

Internal changes

@battermann battermann temporarily deployed to cachix December 9, 2022 11:50 — with GitHub Actions Inactive
@battermann battermann temporarily deployed to cachix December 9, 2022 11:50 — with GitHub Actions Inactive
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Dec 9, 2022
@battermann battermann force-pushed the release_2022-12-09_09_57 branch from 894666d to d980b5e Compare December 9, 2022 11:52
@battermann battermann temporarily deployed to cachix December 9, 2022 11:52 — with GitHub Actions Inactive
@battermann battermann temporarily deployed to cachix December 9, 2022 11:52 — with GitHub Actions Inactive
@battermann battermann changed the base branch from develop to master December 9, 2022 11:52
@battermann battermann temporarily deployed to cachix December 9, 2022 12:00 — with GitHub Actions Inactive
@smatting smatting self-requested a review December 9, 2022 14:49
@smatting smatting self-requested a review December 9, 2022 14:50
@battermann battermann temporarily deployed to cachix December 9, 2022 15:22 — with GitHub Actions Inactive
@smatting smatting temporarily deployed to cachix December 9, 2022 17:11 — with GitHub Actions Inactive
jschaul and others added 3 commits December 12, 2022 11:50
* WIP

* make inputs explicit

* fixup for paths

* use alpine base image

* Support from flokli to fix issues

* formatting

* add nixified nginz-disco image

* attempt to add to default.nix

* nginx: [warn] could not build optimal variables_hash, you should increase either variables_hash_max_size: 1024 or variables_hash_bucket_size: 64; ignoring variables_hash_bucket_size

* PR feedback: openssl 1.1

* changelog

* also upload nginz and nginz-disco images

* PR feedback

* nix/nginz.nix: add nginz to contents

cannon has a preStop lifecycle hook that currently invokes nginx -c …
quit on stop.

* nginz container: drop exposed ports

All config is defined in the helm chart, just by building the container
we don't know which ports are exposed.

* services/nginz: drop Dockerfile, README and Makefile

We use Nix to build this now.

* Hi CI

* remove no-longer-necessary openssl1.1 override

Co-authored-by: Florian Klink <[email protected]>
As it turns out, 'skopeo --retry 5' doesn't actually help us. Wrap it in
bash, then.

To work around:

```
*** Uploading /tmp/tmp.kIKRERgZ1H/image to quay.io/wire/spar-integration:4.25.22Getting image source signatures

Copying blob f4f33343fcb5 skipped: already exists

Copying blob a3ab88edf03d skipped: already exists

Copying blob 9360a695c022 skipped: already exists

Copying blob 62d7b43f88a6 skipped: already exists

Copying blob 134eff2df9f9 skipped: already exists

Copying blob 8834895fc941 skipped: already exists

Copying blob 52a0756d3ab1 done  ======================>----------] 30.0MiB / 40.3MiB

Copying blob fa04d4e808c5 done  ---------------------------------] 8.0b / 190.0KiB

Copying blob 6c806be006f4 skipped: already exists

FATA[0004] trying to reuse blob sha256:95218c34e1598cf423f77062d98259bedcc19c9b8f4d937d0905895ee7b0242e at destination: too many requests to registry

make: *** [Makefile:242: upload-images] Error 1

make: Leaving directory '/tmp/build/80754af9/wire-server'
```
@battermann battermann force-pushed the release_2022-12-09_09_57 branch from 30cbb7b to e946300 Compare December 12, 2022 12:23
@battermann battermann temporarily deployed to cachix December 12, 2022 12:23 — with GitHub Actions Inactive
@battermann battermann merged commit 6ed493a into master Dec 12, 2022
@battermann battermann deleted the release_2022-12-09_09_57 branch December 12, 2022 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants