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.