race condition in test_upload_image_http_nonexistent_location_url: glance_store doesn't handle properly invalid HTTP reply
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Glance |
Invalid
|
High
|
Cyril Roelandt |
Bug Description
Victor asked for me to have a look at an intermittent failure he was seeing in https:/
it shows up like so:
Traceback (most recent call last):
File "/home/
func(*args, **kwargs)
File "/home/
location_data = self._upload(req, image_meta)
File "/home/
{'status': 'saving'})
File "/home/
from_
File "/home/
headers=
File "/home/
'exc_name': exc_name})
File "/home/
six.
File "/home/
raise value
File "/home/
**kwargs)
File "/home/
return func(self, *args, **kwargs)
File "/home/
headers=
File "/home/
return func(self, method, url, body, headers)
File "/home/
raise exception.
glance.
=======
FAIL: glance.
tags: worker-0
-------
Traceback (most recent call last):
File "/home/
self.
File "/home/
self.
File "/home/
raise mismatch_error
testtools.
and through bisection I can reproduce it with 75 tests - I'm working on shrinking the set but it takes a couple hundred runs to be sure its a false branch, so its not super fast.
Changed in glance: | |
importance: | Undecided → High |
Changed in glance: | |
assignee: | nobody → Cyril Roelandt (cyril-roelandt) |
Down to tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ catch_error_ not_found tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ catch_error_ unhandled tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ create_ option tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ delete_ queued_ image_not_ forced_ confirmed tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ delete_ queued_ images_ not_forced_ confirmed tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ list_cached_ images tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ list_cached_ images_ empty tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ lookup_ command_ supported_ command tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ parse_options_ no_parameters tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ parse_options_ retrieve_ command tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ queue_image_ not_forced_ not_confirmed tests.unit. api.test_ cmd_cache_ manage. TestGlanceCmdMa nage.test_ queue_image_ without_ index tests.unit. api.test_ common. TestSizeChecked Iter.test_ small_last_ chunk tests.unit. api.test_ common. TestSizeChecked Iter.test_ too_little_ data tests.unit. api.test_ property_ protections. TestExtraProper tiesProxy. test_delete_ empty_extra_ property tests.unit. api.test_ property_ protections. TestExtraProper tiesProxy. test_delete_ reserved_ extra_property tests.unit. api.test_ property_ protections. TestProtectedIm ageRepoProxy. test_get_ image tests.unit. async.flows. test_import. TestImportTask. test_complete_ task tests.unit. async.flows. test_import. TestImportTask. test_import_ flow_no_ import_ flows tests.unit. async.test_ taskflow_ executor. TestTaskExecuto r.test_ begin_processin g tests.unit. async.test_ taskflow_ executor. TestTaskExecuto r.test_ task_fail tests.unit. common. scripts. image_import. test_main. TestImageImport .test_create_ image tests.unit. common. scripts. test_scripts_ utils.TestScrip tsUtils. test_set_ base_image_ properties tests.unit. common. scripts. test_scripts_ utils.TestScrip tsUtils. test_validate_ location_ file_location_ error tests.unit. common. scripts. test_scripts_ utils.TestScrip tsUtils. test_validate_ location_ http tests.unit. common. test_client. TestClient. test_make_ auth_plugin tests.unit. common. test_config. TestDefaultConf ig.test_ osprofiler_ disabled tests.unit. common. test_exception. GlanceException TestCase. test_non_ unicode_ error_msg tests.unit. common. test_exception. GlanceException TestCase. test_specified_ error_msg tests.unit. common. test_exception. GlanceException TestCase. test_specified_ error_msg_ with_kwargs tests.unit. common. test_property_ utils.TestPrope rtyRulesWithPol icies.test_ check_property_ rules_create_ none_permitted tests.unit. common. test_property_ utils.TestPrope rtyRulesWithPol icies.test_ check_property_ rules_create_ unpermitted_ policy tests.unit. common. test_property_ utils.TestPrope rtyRulesWithPol icies.test_ check_property_ rules_read_ all_permitted tests.unit. common. test_prope. ..
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.
glance.