Reload tests broken in Py3

Bug #1855708 reported by Erno Kuvaja
18
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.

Erno Kuvaja (jokke)
Changed in glance:
status: New → Triaged
importance: Undecided → Critical
Erno Kuvaja (jokke)
Changed in glance:
status: Triaged → Invalid
importance: Critical → Undecided
Erno Kuvaja (jokke)
Changed in glance:
status: Invalid → Triaged
importance: Undecided → Critical
Changed in glance:
assignee: nobody → Khuong Luu (organic-doge)
Revision history for this message
Abhishek Kekane (abhishek-kekane) wrote :

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
[2] https://github.com/openstack/glance/blob/master/glance/common/utils.py (L560, L561, L562)

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.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to glance (master)

Fix proposed to branch: master
Review: https://review.opendev.org/713076

Changed in glance:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (master)

Reviewed: https://review.opendev.org/713076
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=30ece7aa28ce8bef1bfe6d4ac68485e6b980f955
Submitter: Zuul
Branch: master

commit 30ece7aa28ce8bef1bfe6d4ac68485e6b980f955
Author: Khuong Luu <email address hidden>
Date: Fri Mar 13 14:54:54 2020 -0700

    Remove usages of SSL related config options in utils

    SSL support was removed by [1] which resulted in removal of some
    SSL related config options, but two config options 'key_file' and
    'cert_file' are still used in utils.stash_conf_values() which is called
    in reload functionality. When SIGHUP is sent to the parent process,
    this usage causes it to crash and its children processes become orphaned,
    which then causes the reload test to be stale (fail).

    This change also includes:
    - Remove 2 attributes and the decorator that help to skip this test
      (self.diable, self.disabled_message, and @skip_if_disabled)

    [1] https://review.opendev.org/#/c/697969/5/glance/common/wsgi.py

    Change-Id: I361c9d0b53a67b1cc60f0f3012e22accdcc46055
    Closes-Bug: #1855708

Changed in glance:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/glance 20.0.0.0b3

This issue was fixed in the openstack/glance 20.0.0.0b3 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (stable/train)

Reviewed: https://review.opendev.org/713720
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=f3cbe90b3eab1f1952798937f177f23c4ef34e17
Submitter: Zuul
Branch: stable/train

commit f3cbe90b3eab1f1952798937f177f23c4ef34e17
Author: Erno Kuvaja <email address hidden>
Date: Wed Mar 18 19:20:10 2020 +0000

    Fix reloading under PY3

    This is not direct backport of Ussuri fix
    I361c9d0b53a67b1cc60f0f3012e22accdcc46055 but rather
    adaptation of it to facilitate SSL/TLS termination under
    py27.

    Co-Authored-By: Khuong Luu <email address hidden>

    Change-Id: Ifd25ed15211ebc864f3a61558834bb1dae16a062
    Closes-Bug: #1855708

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.