~andersson123/autopkgtest-cloud:apache_logging_monitoring

Last commit made on 2024-04-17
Get this branch:
git clone -b apache_logging_monitoring https://git.launchpad.net/~andersson123/autopkgtest-cloud
Only Tim Andersson can upload to this branch. If you are Tim Andersson please log in for upload directions.

Branch merges

Branch information

Name:
apache_logging_monitoring
Repository:
lp:~andersson123/autopkgtest-cloud

Recent commits

1aea767... by Tim Andersson

feat: web: add apache monitoring and reporting to autopkgtest-web

This commit introduces a new script, `apache-request-monitoring`, which
monitors the exit codes of http requests to our apache server.

It captures the exit code and the count of said exit code in the last 5
minutes.

This commit also adds the necessary service file, and adds the needed juju
config options for the charm (the influx creds).

Succeeded
[SUCCEEDED] pre_commit:0 (build)
[SUCCEEDED] unit_tests:0 (build)
[SUCCEEDED] build_charms:0 (build)
13 of 3 results
ac0e188... by Skia

Merge branch 'skia/private_ppa_noble'

https://code.launchpad.net/~hyask/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/464327

aa5d84f... by Skia

fix: mojo: add keyserver.u.c to no_proxy

With the new mechanism replacing `apt-key`, we need to teach `wget` to
ignore the proxy when fetching the key.

39e36d6... by Skia

fix: worker: unify PPA handling on all releases

This removes the `noble`-specific quirk, since it works for all releases.

1b4863d... by Skia

fix: worker: make private PPA work for noble

This is done with two changes:
  * Get the current release with `lsb_release` instead of parsing
    non-deb822 sources.list file. This is because `noble` is
    deb822-only, thus the old parsing fails.
  * Move away from `apt-key`. This tool has been deprecated after Jammy
    and shouldn't be used anymore.

c7a3c92... by Tim Andersson

fix: web: fix no content-length header for static file endpoints

britney recently started having an issue with downloading the
autopkgtest.db - suddenly the static endpoint had stopped returning the
"Content-Length" header for these static get requests.

This turned out to be a recent change from a security update for the
apache2 package, see below:
https://launchpad.net/ubuntu/+source/apache2/2.4.41-4ubuntu3.17
https://launchpadlibrarian.net/724225454/apache2_2.4.41-4ubuntu3.16_2.4.41-4ubuntu3.17.diff.gz

It was verified in staging that apache2/2.4.41-4ubuntu3.17 was the
problematic version - I tested apache2/2.4.41-4ubuntu3.16 and the
Content-Length header was present when downloading the db.

The header was no longer present because apache2 now serves static files
by default with the "chunked" transfer-encoding, rather than the
"identity" transfer-encoding (which includes the Content-Length header,
Content-Length header is not compatible with a chunked transfer
encoding, see [1]).

This bug:
https://bugs.launchpad.net/ubuntu/+source/apache2/+bug/2061816

Was opened, where a discussion with the package maintainer was had, and
the conclusion was that this new behaviour by default was intended and
is a necessary security patch.

I was pointed to this thread:
https://bz.apache.org/bugzilla/show_bug.cgi?id=68872

Which helpfully had a solution that I modified for our web app.

All static files with the config present in this commit now get served
with the "identity" transfer-encoding and have the appropriate
Content-Length header.

[1] https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Transfer-Encoding#directives

Succeeded
[SUCCEEDED] pre_commit:0 (build)
[SUCCEEDED] unit_tests:0 (build)
[SUCCEEDED] build_charms:0 (build)
13 of 3 results
c8753f2... by Skia

Merge branch 'skia/fix_push-amqp'

https://code.launchpad.net/~hyask/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/464402

f05467c... by Skia

fix: worker: unbreak the 'bytes' detection heuristic

Messages coming from `readline()` always end with a `\n`, so the
heuristic needs to take that into account.

f5e6d8f... by Skia

fix: worker: make push-amqp stop looping at the end of the list

At the end of a test list, `push-amqp` would loop on empty messages,
because the `break` needs to happen before the `continue`.

2350046... by Tim Andersson

Merge remote-tracking branch 'andersson123/docs-add-queue-cleanup-section'