canonical-ubuntu-qa team mailing list archive

[Tim Andersson <mp+467708@xxxxxxxxxxxxxxxxxx>]

The proposal to merge ~andersson123/autopkgtest-cloud:web-reactive-status-fix into autopkgtest-cloud:master has been updated.

Commit message changed to:

fix: web: Don't let autopkgtest-web unit stay in "maintenance" status

In prod, with the recent changes to the web unit regarding restarting
the services, this change was missed.

Without it, if the systemd units get written, and none of them have
changed, the status will never get set to active. The status getting set
to active was intended to be done in the
`restart_all_autopkgtest_web_services` function, however, if the files
haven't changed, this won't happen.

This commit amends the issue by the status to active at the end of the
`set_up_systemd_units` function. If the units have changed, it'll be set
back to maintenance in the `restart_all_autopkgtest_web_services`
function, so there won't be any false "active" statuses.

For more details, see:
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/467708
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~andersson123/autopkgtest-cloud:web-reactive-status-fix into autopkgtest-cloud:master.