Reload tests broken in Py3
Bug #1855708 reported by
Erno Kuvaja
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Glance |
Fix Released
|
Critical
|
Khuong Luu | ||
| Train |
Fix Committed
|
Undecided
|
Erno Kuvaja | ||
Bug Description
Looks like our reload is also broken in PY3 that has gone unnoticed as the test breakage was blamed on ssl.
Test fails on timeout at worker change.
| Changed in glance: | |
| status: | New → Triaged |
| importance: | Undecided → Critical |
| Changed in glance: | |
| status: | Triaged → Invalid |
| importance: | Critical → Undecided |
| Changed in glance: | |
| status: | Invalid → Triaged |
| importance: | Undecided → Critical |
| Changed in glance: | |
| assignee: | nobody → Khuong Luu (organic-doge) |
To post a comment you must log in.

We have removed ssl support [1] this cycle which results in removing some ssl related config options, but we didn't removed all of its usage and two config options 'key_file' and 'cert_file' are still used in reload functionality [2] which is causing error when SIGHUP is sent to parent process and results in termination due to error and childs remaining stale.
If I comment/remove those two options from [2] then SIGHUP works just fine.
[1] https:/ /review. opendev. org/#/c/ 697969/ 5/glance/ common/ wsgi.py /github. com/openstack/ glance/ blob/master/ glance/ common/ utils.py (L560, L561, L562)
[2] https:/
I have verified the same by adding new store to enabled_backends config option and it is crating the new store (file) as well as adding image to this newly created store.